12:29 a.m.
Hi,
I'm wondering about the following behavior:
<hansl>
matrix mtemp = ones(12) ~ seq(1,12)' # two columns
cnameset(mtemp, "") # works
cnameset(mtemp, " ") # works
cnameset(mtemp, defarray(" "," ")) # works
cnameset(mtemp, defarray("","")) # gives error: Missing string in
cnameset
</hansl>
I believe that the 4th/last cnameset line should also apply empty names
to the columns, like the 1st and 2nd lines hopefully do. In contrast and
just for comparison, the 3rd line should apply a single blank character
as the name for each column.
This is with the March 5th snapshot.
thanks
sven
6:18 p.m.
On Wed, Mar 19, 2025 at 8:29 PM Sven Schreiber
<sven.schreiber(a)fu-berlin.de> wrote:
I'm wondering about the following behavior:
<hansl>
matrix mtemp = ones(12) ~ seq(1,12)' # two columns
cnameset(mtemp, "") # works
cnameset(mtemp, " ") # works
cnameset(mtemp, defarray(" "," ")) # works
cnameset(mtemp, defarray("","")) # gives error: Missing string in
cnameset
</hansl>
I believe that the 4th/last cnameset line should also apply empty names
to the columns, like the 1st and 2nd lines hopefully do. In contrast and
just for comparison, the 3rd line should apply a single blank character
as the name for each column.
cnameset 1: OK, deletes any existing colnames, as per the doc
cnameset 2: "works" in the sense of not actually flagging an error,
but that's a bug: the single string argument is just a space, but is
not "empty", and so it should fail. It now does in git.
cnameset 3: sets single-space colnames. Weird, but if that's what you want...
cnameset 4: should not fail, but delete existing colnames. It now does in git.
<hansl>
matrix m = mnormal(3,4)
print m
cnameset(m, defarray("one", "two", "three",
"four"))
print m
cnameset(m, defarray("one", "", "two", ""))
print m
# passing an empty string deletes existing colnames
err = cnameset(m, "")
print m
# passing a single string, not empty: not supported
catch cnameset(m, " ")
if $error
printf "error, as expected\n"
endif
cnameset(m, defarray(" "," "," "," "))
printf "what should this variant do?\n"
print m
# passing an array of empty strings deletes existing colnames
cnameset(m, defarray("","","",""))
print m
strings S = empty
# also deletes existing colnames
cnameset(m, S)
print m
</hansl>
Allin
6:38 p.m.
Am 22.03.2025 um 19:18 schrieb Cottrell, Allin:
On Wed, Mar 19, 2025 at 8:29 PM Sven Schreiber
<sven.schreiber(a)fu-berlin.de> wrote:
> <hansl>
> matrix mtemp = ones(12) ~ seq(1,12)' # two columns
> cnameset(mtemp, "") # works
> cnameset(mtemp, " ") # works
> cnameset(mtemp, defarray(" "," ")) # works
> cnameset(mtemp, defarray("","")) # gives error: Missing string in
cnameset
> </hansl>
>
cnameset 1: OK, deletes any existing colnames, as per the doc
cnameset 2: "works" in the sense of not actually flagging an error,
but that's a bug: the single string argument is just a space, but is
not "empty", and so it should fail. It now does in git.
I'm
indifferent about this type of behavior. I thought that the fact
that it worked previously could be interpreted as "k space-separated
substrings" (quoting from the ref) in the sense that the single space is
the separator and thus to the left and to the right of this separator
are two (=k) empty strings. But it's a matter of convention I'd say.
cnameset 3: sets single-space colnames. Weird, but if that's what you want...
cnameset 4: should not fail, but delete existing colnames. It now does in git.
OK great, thanks!
BTW, I transformed your testing code to use rnameset() on the transpose
of the test matrix. It choked on the two lines:
rnameset(m, defarray("one", "", "two", ""))
rnameset(m, defarray("","","",""))
But this is an older snapshot, so maybe you also fixed rnameset already,
in parallel to cnameset.
cheers
sven
13
days inactive
15
days old
2 comments
2 participants
participants (2)
-
Cottrell, Allin -
Sven Schreiber