Not shure if the the github repo is monitored so here is a note on a bug
fix that happens when plotting more than 99 sequential plots:
https://github.com/tcltk/tklib/compare/master...Meshparts:tklib:patch-1
meshparts <alexandru.dadalau@meshparts.de> posted:
Not shure if the the github repo is monitored so here is a note on a bug fix that happens when plotting more than 99 sequential plots:
https://github.com/tcltk/tklib/compare/master...Meshparts:tklib:patch-1
Thanks for pointing this out. I thought that was fixed. But I will have a look!
Arjen <user153@newsgrouper.org.invalid> posted:
meshparts <alexandru.dadalau@meshparts.de> posted:
Not shure if the the github repo is monitored so here is a note on a bug >>> fix that happens when plotting more than 99 sequential plots:
https://github.com/tcltk/tklib/compare/master...Meshparts:tklib:patch-1
Thanks for pointing this out. I thought that was fixed. But I will have a look!
Indeed, it is not yet solved. I found the culprit though that caused the error
for XY-plots. I will have to test that my solution works for all plots, but my change is in plotpriv.tcl:
if { [string match {[0-9]*} $w] } {
set c [string range $w [string first . $w] end]
} else {
set c $w
}
Originally the second line was:
set c [string range $w 2 end]
There are quite a few of these constructs.
Regards,
Arjen
I thought, the link I posted shows my changes, but maybe not, because I forgot to do a pull request?
My fix was:
set c .[join [lrange [split $w .] 1 end] .]
But your fix is shorter and look better.
Regards
Alex
My apologies - I did not look at the ticket, as I remembered seeing thisNo worries, not a big deal. Glad it's fixed now.
bug before (and thinking it fixed ;)). The corrected code was there in a few places, but not everywhere.
| Sysop: | DaiTengu |
|---|---|
| Location: | Appleton, WI |
| Users: | 1,125 |
| Nodes: | 10 (0 / 10) |
| Uptime: | 26:20:12 |
| Calls: | 14,396 |
| Calls today: | 5 |
| Files: | 186,389 |
| D/L today: |
8,335 files (2,169M bytes) |
| Messages: | 2,545,071 |
| Posted today: | 1 |