Darren's Occasional Nugget 09/26/2022

[Frozen]

---

@Ettepet wrote:

---

@Darren wrote:

---

@DerrickB wrote:

Anyone opposed to me turning a version into a VIPM package? I presume DNatt doesn't mind if I start from his, want to check with others. I would link to this forum in the package description.

---

Sounds great, thanks for doing this Derrick. My preference would be that you use my original VI (I did not see anybody report any bugs with the original version in this thread), and credit me as the author.

---

If your original post does do XML-escaping, regional-number formatting etc. you would be correct.  If it doesn't you would need to correct that.

I will continue to use mine, which also has some efficiency improvements and fits on my display.

---

As I said before, I am not a fan of your compressed coding style. There are better ways to make it fit on your display. 😁

The "efficiencies" are minor at best.

The regional-number formatting is probably important.

[DerrickB]

Darren's does no formatting, it's up to the implementer to decide how to parse the data. That's part of it's charm.

And Darren knows I won't release it as-is.  Sure, there'll be a simple single VI use-case kept intact but it's definitely getting refactored into subvis to start, along with an architecture that'll let me expand capabilities going forward, and working towards other optimizations around memory allocations and repeated accesses to data.

[Frozen]

 @Ettepet

I ran my minimal VIA config against your vi.

33 wires under objects?! 😱

[Ettepet]

---

@Frozen wrote:

---

@Ettepet wrote:

---

@Darren wrote:

---

@DerrickB wrote:

Anyone opposed to me turning a version into a VIPM package? I presume DNatt doesn't mind if I start from his, want to check with others. I would link to this forum in the package description.

---

Sounds great, thanks for doing this Derrick. My preference would be that you use my original VI (I did not see anybody report any bugs with the original version in this thread), and credit me as the author.

---

If your original post does do XML-escaping, regional-number formatting etc. you would be correct.  If it doesn't you would need to correct that.

I will continue to use mine, which also has some efficiency improvements and fits on my display.

---

As I said before, I am not a fan of your compressed coding style. There are better ways to make it fit on your display. 😁

The "efficiencies" are minor at best.

The regional-number formatting is probably important.

---

The XML-Escape codes are mandetory, so you need to make sure you do those.  The efficient code I mentioned mainly concerned that part.  If you use a sub-vi the conversion process will crawl forward for instance, if you meant that approach for fitting the display.

@Frozen wrote:

 @Ettepet

I ran my minimal VIA config against your vi.

33 wires under objects?! 😱

---

I coded for efficiency in speed, not looks.  Some of the code I based it on didn't even properly check on end-conditions, so that saves on wires, sure.

[Darren]

I've made a few modifications to the Read XLSX File VI, and decided to publish it to github. Here's the repo:

https://github.com/dnattinger/read-xlsx-file

And if you just want the VI, you can download it directly with this link.

[Ettepet]

---

@Darren wrote:

I've made a few modifications to the Read XLSX File VI, and decided to publish it to github. Here's the repo:

https://github.com/dnattinger/read-xlsx-file

And if you just want the VI, you can download it directly with this link.

---

Good to see you added the  XML Escape codes.

You should also add a  decimal separator mechanism so everyone can use it, regardless of country. Excel internally uses the United States ".", so it will have to be converted for countries that use a ",".