new editions and minor corrections of latin inscriptions

Hi Jonathan,
I think that I have made the requested corrections, and made a pull a
request (not completely sure about this last point, though).
Fortunately, we've had a little break in the conferences. When I
submitted the inscriptions there was a possible problem with ISicily
720, but I'm not sure why. Didn't find I.Lipara in the LGPN, it's not
easy to find a particular book in here...
Jonathan Prag @.***> escribió:

@JonPrag requested changes on this pull request.
thanks again Andoni - sorry I dashed away this morning. Again, lots
of good stuff here. For Lipara, in future, if you decide to work on
more of those, use the Meligunis Lipara XII volume (I've seen here's
a copy in CSAD/LGPN). Note the way to encode glyphs throughout needs
amending (I've flagged all the instances and given examples). You've
picked some tricky ones and coped well. Note the tricky question
around how to handle texts that only survive fromt he antiquarian
tradition. If you correct these before doing any more, then we can
get this pull request merged and start a new one.

@@ -190,22 +195,23 @@

           <ab>

               <lb n="1"/>

               <supplied reason="lost">— — — — —</supplied>V V V ․

               <supplied reason="lost">— — — — — — — —</supplied>

               <lb n="2"/>

               <supplied reason="lost">— — — —

—AREI․INQVFREIVS․#⁷

               <supplied reason="lost">— — —</supplied>

               <lb n="3"/>

               <supplied reason="lost">— —

ἀμερίμνι, οὐδὶς

               ἀθά<supplied

reason="omitted">νατος.

               <lb n="4"/>

           </ab>

I fear this isn't the right xml:lang encoding - a pair of languages
like this means that the first is the semantic language and teh
second the script. The way to do this is to put the xml:lang
attribute and value onto the textpart, ratehr than the edition div.
The textpart should also have subtype of section, so, e.g.:

, so
remove the xml:lang from the edition div, but leave your resp on the
edition div.
cf. https://github.com/ISicily/ISicily/wiki/Encoding:-text

               <lb n="1"/>

               <supplied reason="lost">—</supplied>ΙΡες

Λιλυβα

               <supplied reason="omitted">ῖτα</supplied>

               <supplied reason="lost">ι — —</supplied>

               <lb n="2"/>

               <supplied reason="lost">Μ(ᾶρκον)</supplied>

               Οὐαλέριο<surplus>Ο</surplus>ν

²⁶Οὐαλέριον²⁶ Δι

               ογνή</supplied>-

               <lb n="3"/>

               <supplied reason="lost">το</supplied>υ

Μηγαν(—)

               υἱὸν <supplied reason="lost">Χόρ</supplied>-

               <lb n="4"/>

               <supplied reason="lost">τ</supplied>ωνα

εὐεργέταν.

               <lb n="5"/>

              <lb n="1"/><gap reason="lost" extent="unknown"

unit="character"/> ΙΡες
ΛΔιλυβιοα ῖται
Ah, this is a more complicated case - here it is not a case of the
text being erroneously inscribed, or orthographically unusual, and
so choice isn't the appropriate solution. Rather, the text only
exists in a clearly flawed transcription by Gualtherus of 1624 (and
an even more flawed version in Tardia, somewhat later, which can be
ignored). The version in IG is Kaibel's plausible effort to
interpret the letters transmitted by Gualtherus; and according to
Kaibel's pre-Leiden editorial conventions, his suggested corrections
appear simply in square brackets. But if we use 'choice', this
implies we are correcting what is ont he stone, when actually we /
Kaibel are correction what Gualtherus thought he could read.
Therefore, the appropriate solution here is simply to give Kaibel's
text, and not to use choice or supplied, except for those letters at
the beginnings or ends of lines which are actual supplements, and
not simply corrections of misread letters (such as the M at the
beginning of line two, which is a suggested praenomen on the basis
of other surviving Lilybaeum texts; or the ογνη at the end of line
two).
the letters reported by Gualtherus are then, instead, reported line
by line in the apparatus, in capitals, e.g.
<app loc="line 1"><note>Gualtherus: ΙΡΕΣΔΙΛΙΟΑbOI</note></app>

@@ -192,58 +197,26 @@

add subtype="section" and xml:lang="la

             <ab>

               <lb n="1"/>

               <expan>

                   Γ

                   <ex>αΐου</ex>

               </expan>

               ∙ Κλωδ-<lb n="2"/>ίου Πρε-<lb n="3"/>μίονος.

               <lb n="4"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Γαίου
 Κλωινδ
<g ref="#interpunct">·</g> NOT <g type="interpunct/>
Also, it's clear from https://epigraphy.packhum.org/text/334797 that
the erroneous original reading reported by Kaibel after Salinas was
in fact the perfectly correct Κλωδ and so there's no need for the
correction (Note all Lipara texts have been reedited in the 1990s
from autopsy by Brea, Cavalier and Campagna in I.Lipara (Meligunis
XII). So, it would be better on this occasion to say text according
to I.Lipara

@@ -191,17 +196,19 @@

           <ab>

               <lb n="1"/>

               Λούκιε ❦<lb n="2"/>Καίλιε ❦<lb n="3"/>Φῆλιξ,

❦χρηστὲ χαῖρε. ❦

               <lb n="6"/>

           </ab>

       <ab>

           <lb n="1"/><persName type="attested"><name

type="praenomen">Λούκιε
the hedera and similar symbols, like interpuncts, are done using the
examples in the character declaration file at
https://github.com/ISicily/ISicily/blob/master/alists/charDecl.xml
(as per 'glyphs and symbols' in
https://github.com/ISicily/ISicily/wiki/Encoding:-text, using ref to
point to the example, and encoding the actual unicode symbol
whenever it exists, so:
<g ref="#ivy-leaf">❦</g>

               <lb n="1"/>

               Κούιντος Σόσσις<lb n="2"/>καὶ Πετρονία<lb

n="3"/>Σοσσία Χείᾳτῇ θυγατρί,

               ζησάσῃ<lb n="5"/>ἔτη ∙ ιʹ ∙

               <expan>

                   ἡμέρα

                   <ex>ς</ex>

               </expan>

               ∙ κ∙βʹ.

               <lb n="6"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Κούιντος Σόσσις

              <lb n="2"/>καὶ <persName type="attested"><name

type="nomen">Πετρονία

              <lb n="3"/><name

type="cognomen">Σοσσία Χείᾳ

              <lb n="4"/><hi rend="ligature">τῇ</hi> <hi

rend="ligature">θυγατρί ζησάσῃ

              <lb n="5"/>ἔ<hi rend="ligature">τη</hi><g

type="interpunct"/>ιἡμέρας  κ  β
spacing is missing in the first part of the line, and the g type
needs changing to g ref - but otherwise, nicely done!

               <lb n="1"/>

               Κούιντος Σόσσις<lb n="2"/>καὶ Πετρονία<lb

n="3"/>Σοσσία Χείᾳτῇ θυγατρί,

               ζησάσῃ<lb n="5"/>ἔτη ∙ ιʹ ∙

               <expan>

                   ἡμέρα

                   <ex>ς</ex>

               </expan>

               ∙ κ∙βʹ.

               <lb n="6"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Κούιντος Σόσσις

              <lb n="2"/>καὶ <persName type="attested"><name

type="nomen">Πετρονία

              <lb n="3"/><name

type="cognomen">Σοσσία Χείᾳ

              <lb n="4"/><hi rend="ligature">τῇ</hi> <hi

rend="ligature">θυγατρί ζησάσῃ

              <lb n="5"/>ἔ<hi rend="ligature">τη</hi><g

type="interpunct"/>ιἡμέρας  κ  β

add the apparatus?

@@ -197,45 +197,13 @@

               <lb n="1"/><persName type="attested"><name

type="praenomen">Lucius·Avillius

               <lb n="1"/><persName type="attested"><name

type="praenomen">Lucius
 Avillius
<g ref="#interpunct">·</g>

               <gap reason="lost" extent="unknown" unit="line">

               </gap>

           </ab>

           <ab>

               <lb n="1"/>

               <supplied reason="lost">— — — —— — — — — —</supplied>

               <lb n="2"/>

               <supplied reason="lost">— — —</supplied>SA ∙ F

               <unclear>PI</unclear>

               <supplied reason="lost">— — —</supplied>

               <lb n="3"/>

               <lb n="1"/><gap reason="lost" extent="unknown"

unit="character"/> SA
filia
piissima
<g ref="#interpunct">·</g>

@@ -198,59 +198,11 @@

               <lb n="1"/><persName type="attested"><name

type="praenomen">Marcus·Valerius                    ̣

                   <lb n="2"/><orgName type="tribus"

nymRef="#Pomptina">Pomptina

               <lb n="1"/><persName type="attested"><name

type="praenomen">Marcus
 Valerius                    ̣
g ref etc. again

               ∙ Κα<supplied reason="lost">στρί</supplied>κιος ∙

               <expan>

                   Κυρ

                   <ex>είνα</ex>

               </expan>

               ∙ Προύδηνς ∙ ἱππεὺς ∙

               <expan>

                   Ῥωμ

                   <ex>αίων</ex>

               </expan>

               ∙ πρῶτος ∙ Μελιταίων<lb n="2"/>καὶ ∙ πάτρων ∙

ἄρξας ∙ καὶ ∙ ἀμφιπολεύσας ∙ θεῷ ∙ Αὐγούστῳ

               <lb n="3"/>

               <supplied reason="lost">— — — — —

—ΕΣΧ— —Ν— —Ε․Ι․ΝΕ

               <supplied reason="lost">— — — — — — — —</supplied>

               <lb n="4"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Λούκιος
 Καστρίκιος
Κυρείνα
 Προύδηνς
ἱππεὺς  Ῥωμαίων πρῶτοςΜελιταίων
spacing missing near the end; and change the g types to g refs with
the interpuncts

               <expan>

                   Κυρ

                   <ex>είνα</ex>

               </expan>

               ∙ Προύδηνς ∙ ἱππεὺς ∙

               <expan>

                   Ῥωμ

                   <ex>αίων</ex>

               </expan>

               ∙ πρῶτος ∙ Μελιταίων<lb n="2"/>καὶ ∙ πάτρων ∙

ἄρξας ∙ καὶ ∙ ἀμφιπολεύσας ∙ θεῷ ∙ Αὐγούστῳ

               <lb n="3"/>

               <supplied reason="lost">— — — — —

—ΕΣΧ— —Ν— —Ε․Ι․ΝΕ

               <supplied reason="lost">— — — — — — — —</supplied>

               <lb n="4"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Λούκιος
 Καστρίκιος
Κυρείνα
 Προύδηνς
ἱππεὺς  Ῥωμαίων πρῶτοςΜελιταίων

              <lb n="2"/>καὶ<g type="interpunct"/> πάτρων <g

type="interpunct"/> ἄρξας  καὶ  ἀμφιπολεύσας  θεῷ  Αὐγούστῳ
space after the first και and change g type to g ref etc. throughout

                   Κυρ

                   <ex>είνα</ex>

               </expan>

               ∙ Προύδηνς ∙ ἱππεὺς ∙

               <expan>

                   Ῥωμ

                   <ex>αίων</ex>

               </expan>

               ∙ πρῶτος ∙ Μελιταίων<lb n="2"/>καὶ ∙ πάτρων ∙

ἄρξας ∙ καὶ ∙ ἀμφιπολεύσας ∙ θεῷ ∙ Αὐγούστῳ

               <lb n="3"/>

               <supplied reason="lost">— — — — —

—ΕΣΧ— —Ν— —Ε․Ι․ΝΕ

               <supplied reason="lost">— — — — — — — —</supplied>

               <lb n="4"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Λούκιος
 Καστρίκιος
Κυρείνα
 Προύδηνς
ἱππεὺς  Ῥωμαίων πρῶτοςΜελιταίων

              <lb n="2"/>καὶ<g type="interpunct"/> πάτρων <g

type="interpunct"/> ἄρξας  καὶ  ἀμφιπολεύσας  θεῷ  Αὐγούστῳ

              <lb n="3"/><gap reason="lost" extent="unknown"

unit="character"/> ΕΣΧ  Ν  Ε
 Ι
 ΝΕ

one of the gaps needs turning into a closed element; and, in
principle, if you're going to accept the dots as representative of
characters on the rest of the line, why would one not do so for the
first three dots also?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/ISicily/ISicily/pull/173#pullrequestreview-686780961

--
Andoni Llamazares Martín
Ikasketa Klasikoen saila / Dpto. de Estudios Clásicos
Letren Fakultatea / Facultad de Letras
c/Francisco Tomás y Valiente 1. 01006 Vitoria-Gasteiz
EHU / UPV

OK, I discovered that there was a repetition of the changes in my
stashed changes and the merged version in 720
Jonathan Prag @.***> escribió:

@JonPrag requested changes on this pull request.
thanks again Andoni - sorry I dashed away this morning. Again, lots
of good stuff here. For Lipara, in future, if you decide to work on
more of those, use the Meligunis Lipara XII volume (I've seen here's
a copy in CSAD/LGPN). Note the way to encode glyphs throughout needs
amending (I've flagged all the instances and given examples). You've
picked some tricky ones and coped well. Note the tricky question
around how to handle texts that only survive fromt he antiquarian
tradition. If you correct these before doing any more, then we can
get this pull request merged and start a new one.

@@ -190,22 +195,23 @@

           <ab>

               <lb n="1"/>

               <supplied reason="lost">— — — — —</supplied>V V V ․

               <supplied reason="lost">— — — — — — — —</supplied>

               <lb n="2"/>

               <supplied reason="lost">— — — —

—AREI․INQVFREIVS․#⁷

               <supplied reason="lost">— — —</supplied>

               <lb n="3"/>

               <supplied reason="lost">— —

ἀμερίμνι, οὐδὶς

               ἀθά<supplied

reason="omitted">νατος.

               <lb n="4"/>

           </ab>

I fear this isn't the right xml:lang encoding - a pair of languages
like this means that the first is the semantic language and teh
second the script. The way to do this is to put the xml:lang
attribute and value onto the textpart, ratehr than the edition div.
The textpart should also have subtype of section, so, e.g.:

, so
remove the xml:lang from the edition div, but leave your resp on the
edition div.
cf. https://github.com/ISicily/ISicily/wiki/Encoding:-text

               <lb n="1"/>

               <supplied reason="lost">—</supplied>ΙΡες

Λιλυβα

               <supplied reason="omitted">ῖτα</supplied>

               <supplied reason="lost">ι — —</supplied>

               <lb n="2"/>

               <supplied reason="lost">Μ(ᾶρκον)</supplied>

               Οὐαλέριο<surplus>Ο</surplus>ν

²⁶Οὐαλέριον²⁶ Δι

               ογνή</supplied>-

               <lb n="3"/>

               <supplied reason="lost">το</supplied>υ

Μηγαν(—)

               υἱὸν <supplied reason="lost">Χόρ</supplied>-

               <lb n="4"/>

               <supplied reason="lost">τ</supplied>ωνα

εὐεργέταν.

               <lb n="5"/>

              <lb n="1"/><gap reason="lost" extent="unknown"

unit="character"/> ΙΡες
ΛΔιλυβιοα ῖται
Ah, this is a more complicated case - here it is not a case of the
text being erroneously inscribed, or orthographically unusual, and
so choice isn't the appropriate solution. Rather, the text only
exists in a clearly flawed transcription by Gualtherus of 1624 (and
an even more flawed version in Tardia, somewhat later, which can be
ignored). The version in IG is Kaibel's plausible effort to
interpret the letters transmitted by Gualtherus; and according to
Kaibel's pre-Leiden editorial conventions, his suggested corrections
appear simply in square brackets. But if we use 'choice', this
implies we are correcting what is ont he stone, when actually we /
Kaibel are correction what Gualtherus thought he could read.
Therefore, the appropriate solution here is simply to give Kaibel's
text, and not to use choice or supplied, except for those letters at
the beginnings or ends of lines which are actual supplements, and
not simply corrections of misread letters (such as the M at the
beginning of line two, which is a suggested praenomen on the basis
of other surviving Lilybaeum texts; or the ογνη at the end of line
two).
the letters reported by Gualtherus are then, instead, reported line
by line in the apparatus, in capitals, e.g.
<app loc="line 1"><note>Gualtherus: ΙΡΕΣΔΙΛΙΟΑbOI</note></app>

@@ -192,58 +197,26 @@

add subtype="section" and xml:lang="la

             <ab>

               <lb n="1"/>

               <expan>

                   Γ

                   <ex>αΐου</ex>

               </expan>

               ∙ Κλωδ-<lb n="2"/>ίου Πρε-<lb n="3"/>μίονος.

               <lb n="4"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Γαίου
 Κλωινδ
<g ref="#interpunct">·</g> NOT <g type="interpunct/>
Also, it's clear from https://epigraphy.packhum.org/text/334797 that
the erroneous original reading reported by Kaibel after Salinas was
in fact the perfectly correct Κλωδ and so there's no need for the
correction (Note all Lipara texts have been reedited in the 1990s
from autopsy by Brea, Cavalier and Campagna in I.Lipara (Meligunis
XII). So, it would be better on this occasion to say text according
to I.Lipara

@@ -191,17 +196,19 @@

           <ab>

               <lb n="1"/>

               Λούκιε ❦<lb n="2"/>Καίλιε ❦<lb n="3"/>Φῆλιξ,

❦χρηστὲ χαῖρε. ❦

               <lb n="6"/>

           </ab>

       <ab>

           <lb n="1"/><persName type="attested"><name

type="praenomen">Λούκιε
the hedera and similar symbols, like interpuncts, are done using the
examples in the character declaration file at
https://github.com/ISicily/ISicily/blob/master/alists/charDecl.xml
(as per 'glyphs and symbols' in
https://github.com/ISicily/ISicily/wiki/Encoding:-text, using ref to
point to the example, and encoding the actual unicode symbol
whenever it exists, so:
<g ref="#ivy-leaf">❦</g>

               <lb n="1"/>

               Κούιντος Σόσσις<lb n="2"/>καὶ Πετρονία<lb

n="3"/>Σοσσία Χείᾳτῇ θυγατρί,

               ζησάσῃ<lb n="5"/>ἔτη ∙ ιʹ ∙

               <expan>

                   ἡμέρα

                   <ex>ς</ex>

               </expan>

               ∙ κ∙βʹ.

               <lb n="6"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Κούιντος Σόσσις

              <lb n="2"/>καὶ <persName type="attested"><name

type="nomen">Πετρονία

              <lb n="3"/><name

type="cognomen">Σοσσία Χείᾳ

              <lb n="4"/><hi rend="ligature">τῇ</hi> <hi

rend="ligature">θυγατρί ζησάσῃ

              <lb n="5"/>ἔ<hi rend="ligature">τη</hi><g

type="interpunct"/>ιἡμέρας  κ  β
spacing is missing in the first part of the line, and the g type
needs changing to g ref - but otherwise, nicely done!

               <lb n="1"/>

               Κούιντος Σόσσις<lb n="2"/>καὶ Πετρονία<lb

n="3"/>Σοσσία Χείᾳτῇ θυγατρί,

               ζησάσῃ<lb n="5"/>ἔτη ∙ ιʹ ∙

               <expan>

                   ἡμέρα

                   <ex>ς</ex>

               </expan>

               ∙ κ∙βʹ.

               <lb n="6"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Κούιντος Σόσσις

              <lb n="2"/>καὶ <persName type="attested"><name

type="nomen">Πετρονία

              <lb n="3"/><name

type="cognomen">Σοσσία Χείᾳ

              <lb n="4"/><hi rend="ligature">τῇ</hi> <hi

rend="ligature">θυγατρί ζησάσῃ

              <lb n="5"/>ἔ<hi rend="ligature">τη</hi><g

type="interpunct"/>ιἡμέρας  κ  β

add the apparatus?

@@ -197,45 +197,13 @@

               <lb n="1"/><persName type="attested"><name

type="praenomen">Lucius·Avillius

               <lb n="1"/><persName type="attested"><name

type="praenomen">Lucius
 Avillius
<g ref="#interpunct">·</g>

               <gap reason="lost" extent="unknown" unit="line">

               </gap>

           </ab>

           <ab>

               <lb n="1"/>

               <supplied reason="lost">— — — —— — — — — —</supplied>

               <lb n="2"/>

               <supplied reason="lost">— — —</supplied>SA ∙ F

               <unclear>PI</unclear>

               <supplied reason="lost">— — —</supplied>

               <lb n="3"/>

               <lb n="1"/><gap reason="lost" extent="unknown"

unit="character"/> SA
filia
piissima
<g ref="#interpunct">·</g>

@@ -198,59 +198,11 @@

               <lb n="1"/><persName type="attested"><name

type="praenomen">Marcus·Valerius                    ̣

                   <lb n="2"/><orgName type="tribus"

nymRef="#Pomptina">Pomptina

               <lb n="1"/><persName type="attested"><name

type="praenomen">Marcus
 Valerius                    ̣
g ref etc. again

               ∙ Κα<supplied reason="lost">στρί</supplied>κιος ∙

               <expan>

                   Κυρ

                   <ex>είνα</ex>

               </expan>

               ∙ Προύδηνς ∙ ἱππεὺς ∙

               <expan>

                   Ῥωμ

                   <ex>αίων</ex>

               </expan>

               ∙ πρῶτος ∙ Μελιταίων<lb n="2"/>καὶ ∙ πάτρων ∙

ἄρξας ∙ καὶ ∙ ἀμφιπολεύσας ∙ θεῷ ∙ Αὐγούστῳ

               <lb n="3"/>

               <supplied reason="lost">— — — — —

—ΕΣΧ— —Ν— —Ε․Ι․ΝΕ

               <supplied reason="lost">— — — — — — — —</supplied>

               <lb n="4"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Λούκιος
 Καστρίκιος
Κυρείνα
 Προύδηνς
ἱππεὺς  Ῥωμαίων πρῶτοςΜελιταίων
spacing missing near the end; and change the g types to g refs with
the interpuncts

               <expan>

                   Κυρ

                   <ex>είνα</ex>

               </expan>

               ∙ Προύδηνς ∙ ἱππεὺς ∙

               <expan>

                   Ῥωμ

                   <ex>αίων</ex>

               </expan>

               ∙ πρῶτος ∙ Μελιταίων<lb n="2"/>καὶ ∙ πάτρων ∙

ἄρξας ∙ καὶ ∙ ἀμφιπολεύσας ∙ θεῷ ∙ Αὐγούστῳ

               <lb n="3"/>

               <supplied reason="lost">— — — — —

—ΕΣΧ— —Ν— —Ε․Ι․ΝΕ

               <supplied reason="lost">— — — — — — — —</supplied>

               <lb n="4"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Λούκιος
 Καστρίκιος
Κυρείνα
 Προύδηνς
ἱππεὺς  Ῥωμαίων πρῶτοςΜελιταίων

              <lb n="2"/>καὶ<g type="interpunct"/> πάτρων <g

type="interpunct"/> ἄρξας  καὶ  ἀμφιπολεύσας  θεῷ  Αὐγούστῳ
space after the first και and change g type to g ref etc. throughout

                   Κυρ

                   <ex>είνα</ex>

               </expan>

               ∙ Προύδηνς ∙ ἱππεὺς ∙

               <expan>

                   Ῥωμ

                   <ex>αίων</ex>

               </expan>

               ∙ πρῶτος ∙ Μελιταίων<lb n="2"/>καὶ ∙ πάτρων ∙

ἄρξας ∙ καὶ ∙ ἀμφιπολεύσας ∙ θεῷ ∙ Αὐγούστῳ

               <lb n="3"/>

               <supplied reason="lost">— — — — —

—ΕΣΧ— —Ν— —Ε․Ι․ΝΕ

               <supplied reason="lost">— — — — — — — —</supplied>

               <lb n="4"/>

              <lb n="1"/><persName type="attested"><name

type="praenomen">Λούκιος
 Καστρίκιος
Κυρείνα
 Προύδηνς
ἱππεὺς  Ῥωμαίων πρῶτοςΜελιταίων

              <lb n="2"/>καὶ<g type="interpunct"/> πάτρων <g

type="interpunct"/> ἄρξας  καὶ  ἀμφιπολεύσας  θεῷ  Αὐγούστῳ

              <lb n="3"/><gap reason="lost" extent="unknown"

unit="character"/> ΕΣΧ  Ν  Ε
 Ι
 ΝΕ

one of the gaps needs turning into a closed element; and, in
principle, if you're going to accept the dots as representative of
characters on the rest of the line, why would one not do so for the
first three dots also?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/ISicily/ISicily/pull/173#pullrequestreview-686780961

--
Andoni Llamazares Martín
Ikasketa Klasikoen saila / Dpto. de Estudios Clásicos
Letren Fakultatea / Facultad de Letras
c/Francisco Tomás y Valiente 1. 01006 Vitoria-Gasteiz
EHU / UPV


Clean outputs of pymialsrtk

The outputs of pymialsrtk contain by default too much material.
There should be a default setting that returns a minimal set of outputs, such as the SR reconstruction and its json, the brain mask, as well as labelmaps (#133). The rest should be optional outputs.
In addition,

_reduceFOV folders should be removed.
reports should contain run_ids

In addition, with options to do multiple runs of preprocessing or sr, the output in pymialsrtk is overwritten partially.
We should think of a way of handling it more cleanly.

In addition, we should use a BIDS-compliant formatting without using sr as a tag, but rather use desc-.

Not exhaustive, TBC:

[ ] Remove addition of pre/post fix in filename from many interfaces.
[ ] Define clear BIDS compliant output filenames
[ ] Define minimalist default outputs (SR image, SR mask, SR json, report + eventual custom_interfaces outputs : SR labels, CSV metrics files). Output all files (e.g. all preprocessed LR images) only on request
[ ] Manage multi_parameters custom interface and run ids

See #184 and #185


[BUG] More than 30 GB mobile data a day consumed by Amethyst

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Video and Screenshots
If applicable, add a video and/or screenshots to help explain your problem.
Device (please complete the following information):

Phone Brand/Model [e.g. Pixel 7 Pro]:
Android Version [e.g. 33]:
App Version [e.g. v0.20.3]:
Amber Version (if using it to sign):

Bounty (in Bitcoin sats) offered for a solution
Incentivize developers to work on your issue. Describe clear milestones to claim payment.

Have your tried data management settings?

The point is that I barely used the app that day. Typically I saw data consumption at level of 100 MB a day.

Oh I see!

Do your stats say whether it was foreground or background data?

30 GB foreground, 10 MB background.
Also, phone battery drained quickly at the time.

It could be something user specific with your settings.. Relays, follow list, hastags or such.
Not sure how to investigate it.. Besides maybe trying with your npub.
Feel free to share your npub to: npub142srz0tkhynwkw65g37c4tr4r938rfchchcq9akr4c8h0thyxjuqs9ldrf
But I understand if privacy doesn't allow for it.

npub133s4w0s6gnpxs00yxmtsteejsu0jetqem7r54peza9avhx60raqsla52tn

Got the npub, feel free to delete.
Can you please check your relay settings. Is there any relay that has consumed abnormal amount of data?

How to verify the amount of data that a relay transferred that day? I think I can see current consumption only in the Relays screen.

Yes, that's the screen. Odd, it's all 0

Yes, that's the screen. Odd, it's all 0

That's because the screenshot was made at another day (after I blocked Amethyst app from using mobile data).

Shame, would have been good to see if it's a single relay that is misabehaving.
Also, have your tried "WiFi only" in settings?

There were a couple of MASSIVE videos placed on Nostr in the past week. I noticed I was downloading 50MB/sec of a single 14GB video at some point. I wonder if the video just kept playing in the background after a swipe-up to close app gesture.
Maybe we need to warn users when very large files are posted.

I would guess that there were video data downloaded in a loop in background by Amethyst. I might have seen playback position indicator movement in a media playback widget in notification panel or a lock screen at some moment if I recall correctly.

Maybe we need to warn users when very large files are posted.

Yeah, that's a good idea IMHO.

I wasn't able to find anything wrong with your profile @GregTonoski. I suspect the issue @vitorpamplona mentioned, viewing large video.
Using the "wifi only" settings should help a bit!


Issue5 upgrade dependencies d3plus

All upgrades including the initial migration from d3plus v1 to v2.
Related PR: https://github.com/gdcc/dv-metrics/pull/9
Related Issue: #9
Things to look at:

[x] Graphs have no footer, so most likely place configured text in html element below it.
[x] Texts and formatting not migrated; tooltips can be improved
[x] Links to datasets or files when applicable
[x] colors from config not migrated, default seems acceptable
[x] stacked (chart) not working properly
[x] Missing tickmarks on x-axis for timeseries; seems acceptable when tooltips work
[x] Lots of commented-out code that must be removed when done
[x] Local code in directory d3plus.v1.9.8 should be removed, if we use CDN

Problem with the StackedArea:

Just created an issue at d3plus; https://github.com/d3plus/d3plus/issues/726

Got an answer from d3plus (Dave Landry), with the fix:
.time("date")
.timeline(false)

Sorting problem might be fixable, tested another sorting function and on FF it remaisn werking, just have t test Chrome.
The problem is tha we now use a comparison that results in a boolean, while the sort wants a number.
We had:
.xSort(function(a,b) { return a["count"] < b["count"];})

If we replace it with this
.xSort(function(a,b) { return b["count"] - a["count"];})

It might work on Chrome.

The sorting problem has been fixed, so we now only have problems with the StackedArea.


.Net Core Tests

Branch: dotnet-core
One of the features of the new system is the command dotnet test which executes the test cases for a project. Before merging the dotnet-core branch into master I want to make sure we can run our tests on it.
This may help: https://xunit.github.io/docs/getting-started-dotnet-core.html

Needed to fix some additional tests in ecb9e643ed99e7ff49e8e5111980cead2e66284d but the tests now build completely in .Net Core. The dotnet-core branch has been merged into master.


ARM 64 node support

Please describe the Improvement and/or Feature Request
Wanted to know what pieces are missing in order to support ARM nodes.
Scope (please mark with X where applicable)

New Functionality          [ ]
Install                    [ x]
SMI Traffic Access Policy  [ ]
SMI Traffic Specs Policy   [ ]
SMI Traffic Split Policy   [ ]
Permissive Traffic Policy  [ ]
Ingress                    [ ]
Egress                     [ ]
Envoy Control Plane        [ x]
CLI Tool                   [ ]
Metrics                    [ ]
Certificate Management     [ ]
Sidecar Injection          [ ]
Logging                    [ ]
Debugging                  [ ]
Tests                      [ ]
CI System                  [ ]
Demo                       [ ]
Project Release            [ ]

Possible use cases
using osm install i tried to install osm on my Microk8s (Raspberry pi4, 5 node ARM64) cluster. I could see that the node affinity wasn't allowing the pods to be scheduled. So i patched the affinity to support ARM64 nodes. Then i could see that the envoyproxy containers couldn't be scheduled.
When i looked at the os builds for envoyproxy:v1.19.3 (referenced by the pod i can see no arm64 support.
The same is true of the latest version of envoy-alpine
However the generic envoy supports ARM64.
Also it seems that the envoy-alpine image is being deprecated?
Is this an easy swapping of the image or (as expected :) ) there are more changes needed?

Hi @tbertenshaw
You can change the default sidecar image by editing the MeshConfig:
kubectl patch meshconfig osm-mesh-config -n osm-system -p '{"spec":{"sidecar":{"envoyImage":"envoyproxy/envoy:v1.19.1"}}}' --type=merge
If you update the meshconfig like this, are the envoy sidecar injections able to take place?
We are also in the process of adding an ARM64 test to our release process so hopefully we will be able to catch errors like this in advance down the road.

We have made changes to run osm and its E2E test cases on ARM64 , but haven't yet submitted changes to main repo. We started our journey to implement Pipy as osm proxy control plane. https://github.com/openservicemesh/osm/issues/4632

Try this https://github.com/flomesh-io/osm/wiki/How-to-setup-osm-to-run-on-arm64-platform , it works for me and Flomesh dev team. We run osm on ubunt2x arm64 server version on apple m1 UTM based VM.

closing in favor of https://github.com/openservicemesh/osm/issues/4716


[bug] args in 'Command' is escaped.

Describe the bug
I'm trying to open using 'explorer.exe' command, but the path input is escaped each time, resulting in explorer process ignoring args.
Reproduction
export const getCurrentResourceDir = async () => {
  const resourceDir = await path.resourceDir();

  return await path.resolve(resourceDir, 'data');
};

export const openSavedFolderAsync = async () => {
  const curPath = (await getCurrentResourceDir()).replaceAll('\\\\?\\', '');

  const isWindows = navigator.userAgent.includes('Windows');
  const isMac = navigator.userAgent.includes('Mac');

  // C:\my-project\src-tauri\target\debug\data
  console.log(curPath);

  // args : Array(1)
  // 0 : "C:\\my-project\\src-tauri\\target\\debug\\data"
  // length : 1
  console.log(new Command('explorer', curPath));

  if (isWindows) {
    // FIXME curPath is escaped
    new Command('explorer', curPath).spawn();
  } else if (isMac) {
    new Command('open', curPath).spawn();
  }
};

Expected behavior
Command executes then new folder is opened with designated path.
"explorer 'path'".
if 'path' is not fit with windows directory format, it just opens user's document folder, which I'm experiencing now.
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.22631 X64
    ✔ WebView2: 129.0.2792.89
    ✔ MSVC:
        - Visual Studio Build Tools 2019
        - Visual Studio Community 2022
    ✔ rustc: 1.76.0 (07dca489a 2024-02-04)
    ✔ cargo: 1.76.0 (c84b36747 2024-01-18)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.4.0
    - pnpm: 9.9.0
    - yarn: 1.22.19
    - npm: 9.7.2
    - bun: 1.1.22

[-] Packages
    - tauri [RUST]: 1.8.0
    - tauri-build [RUST]: 1.5.5
    - wry [RUST]: 0.24.11
    - tao [RUST]: 0.16.10
    - @tauri-apps/api [NPM]: 1.6.0 (outdated, latest: 2.0.3)
    - @tauri-apps/cli [NPM]: 1.6.2 (outdated, latest: 2.0.3)

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist
    - devPath: http://localhost:1420/
    - framework: React
    - bundler: Vite

Stack trace
None

Additional context
No response

I don't see any issue with the code you provided except your getCurrentResourceDir doesn't check if the path resolved exists or not (and possibly create it if not) which is why explorer <non-existent-path> will open default Documents dir.

From there I already created data folder and when I typed the exact explorer in cmd, It opened well. It would be good if you try this after you create data folder first.

I just tried that again and it works, here is the code I tested with:
let resourceDir = await path.resourceDir();
let dataDir = await path.resolve(resourceDir, "data");
console.log("dataDir: ", dataDir);

let exists = await fs.exists(dataDir);
console.log("does dataDir exists? ", exists);

// list the directories of `resourceDir` just to make sure
console.log((await new shell.Command("cmd", ["/c", "dir"], { cwd: resourceDir }).execute()).stdout)

// create if not exists
if (!exists) {
  await fs.mkdir(dataDir);
}

// open in file expolorer
await new shell.Command("explorer", dataDir).spawn()

and here is the output
dataDir:  D:\.cargo-target\debug\data

does dataDir exists?  false

Volume in drive D has no label.
  I will try again. Thank you for the example.


fix(iphone): dismissOnWindowClick

Fixes #529

@halafi  A bit more explanation on what the fix exactly does would be useful. But even though +1

the onClick event does not fire on mobile safari, onTouchEnd works

@halafi that is obvious ;). the question is at what action or point it does not fire. because what I see it is within the autoDismiss logic.

closing with click out (actually touch out) does not work when cookie consent is displayed on iPhone Safari and dismissOnWindowClick is enabled

@halafi thx for clarification, I don't want to bug you. But I just wanted to understand what it exactly does and it is probably also useful for others.


🐞 [Bug]: Cannot Deploy Kubernetes workers

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
Dashboard
What happened?

After Deploying Kubernetes and trying to deploy workers through the manage workers button, the Deploy button was still inactive, although all configurations for this worker were done.

Steps To Reproduce

After successfully deploying a K8s cluster.
Navigate to the deployment list and click on Manage Workers
Fill the info for the new worker and click on Deploy

which network/s did you face the problem on?
Test
version
2.5.0-rc2
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
-

same for caprover

verifed on devnet
version : 08a275d
k8s

caprover

https://app.testlodge.com/a/26076/projects/40893/runs/746337?tab=2#executed_case_32672081
https://app.testlodge.com/a/26076/projects/40893/runs/746337?tab=2#executed_case_32672089


Crash in AGProxyFilterContext::setResponseHeaders

https://sentry.adguard.com/adg/adguard-android-rs/issues/6485/activity/

/core/pull-requests/508


A way to set 'push' origin IP address

Is there a way to set push origin IP address? Something similar to proxy_bind
For example I have three public IP's on a streaming server, say <IP_ADDRESS>, <IP_ADDRESS> and <IP_ADDRESS> (I'm going to use NAT IPs just as an example).
The default eth0 is <IP_ADDRESS>
What I would like to do is have nginx+rtmp module receive OBS data on <IP_ADDRESS> and push it to an external server from the same origin IP <IP_ADDRESS>.
And do the same thing with <IP_ADDRESS>, receive rtmp ON it, and push to an external server FROM it.
Right now I am receiving the OBS stream on <IP_ADDRESS>:12002, but when pushing to rtmp://some-stream-consumer.com/live/my-key the stream goes out through the source IP <IP_ADDRESS>
rtmp {
        server {
                listen <IP_ADDRESS>:12002;
                chunk_size 4096;

                application bob {
                        live on;
                        record off;
                        deny play all;
                        meta copy;
                        on_publish http://<IP_ADDRESS>:80/rtmp_authentication.php;
                        push rtmp://some-stream-consumer.com/live/my-key
                }
        }
}

I have no trouble doing a curl curl --interface <IP_ADDRESS> http://example.com/, this works, there is no routing issue or something like that. Just that this option is not available in NginX or RTMP module.
Does anybody have an idea about this?
LE: why do I want to do this? Currently I have a small pool of IPv4 addresses and each stream has it's own source (requirements of the streaming consumer service), and now instead of using multiple OBS and or splitter in the streaming PCs, I want to receive a single OBS stream per PC into a streaming box with all these IPs and push it to it's corresponding external service.

I'm not sure if this module can bind to an interface like you want but if you have build Nginx --with-stream you can use ngx_stream_proxy_module with the proxy_bind directive to achieve that.
The config would be something like this
stream {
  server {
    listen <IP_ADDRESS>:9000;
    proxy_bind <IP_ADDRESS>;
    proxy_pass rtmp://some-stream-consumer.com/live/my-key;
  }
}

<snip>

rtmp {
  server {
    listen <IP_ADDRESS>:12002;
    chunk_size 4096;

    application bob {
      live on;
      record off;
      deny play all;
      meta copy;
      on_publish http://<IP_ADDRESS>:80/rtmp_authentication.php;
      push rtmp://<IP_ADDRESS>:9000;
    }
  }
}

@teh-random-name this is the tested example of what you recommended, and it works with very little changes! Thank you very much!
Yes, a recompile --with-stream was needed
stream {
  server {
    listen <IP_ADDRESS>:9000;
    proxy_bind <IP_ADDRESS>;
    proxy_pass external-stream-consumer.com:1935;
  }
}

rtmp {
  server {
    listen <IP_ADDRESS>:12002;
    chunk_size 4096;

    application bob {
      live on;
      record off;
      deny play all;
      meta copy;
      on_publish http://<IP_ADDRESS>:80/rtmp_authentication.php;
      push rtmp://<IP_ADDRESS>:9000/stream-app/my-key;
    }
  }
}

Where <IP_ADDRESS> is the default IP address of the server, which in this case handles some RTMP authentication too.
And <IP_ADDRESS> is the secondary IP used for IN/OUT RTMP streaming.


How to match without the space before 'at'

It only match when I input with at least a single space, how can I match without space only when it match the keywords?

I have got the solution from the code.

How do you solve it? Can you post your solution?

Ok, I solve it, simply add startWithSpace: false in elm.atwho({..}).


Flasky flash message problem

To replicate the problem in Flasky:
Enter wrong login credentials, message "Invalid username or password." should appear.
On browser, press "back" and "forward", same message should be there.
Any idea on how to fix this? Thanks!

This is a side effect of how message flashing works. The idea is that the message is shown right after the situation that cause the error happened, but then the error message is suppressed. Same thing happens if you hit refresh on a page that has a flashed message, the message is expected to go away in that case too.

@miguelgrinberg thanks a lot for the answer.


Frontend for the application

Hi,
I am willing to contribute to open source and found your project really interesting.
Would like to create a front-end say HTML page for users to search the details.
Please let me know.
Thanks.

You're absolutely welcome to do this :) Thank you!
After you've built such a front-end, we could link it here in this repository in a new section called "SDKs and libraries" or something like this.
Look here for someone else who wants to build a similar thing in the form of a PHP library: https://github.com/delight-im/FreeGeoDB/issues/10


Sending message with # in markdown in backticks ends up with the #'s being escaped by backslashes

The buggy message is: https://vector.im/develop/#/room/#matrix:matrix.org/$1471691556118961CfDix:matrix.org

Example: send "test toto #123" from vector ios
the formatted body will be "test toto \#123"
instead of "test toto #123"


[17.1.5] Can't create alias with name IRC/irc

The name cannot be the internally reserved keyword "irc".
But it is not in the list of known services (although I assume it is in /etc/services

Same for nntps

Same for sieve...
It looks like the drop down does not list all wellknown servers but doesn't allow aliases for them to be create... which is really annoying!

Since we know the list from /etc/services...
It would be nice to option under Firewall -> Settings to toggle showing of all known services vs well known services.
Showing all of them is probably overwhelming for some users, but others might want to see them all.

timeout due to inactivity


Fixed Luhn Summarizer for Crystal 0.31.0

The / operator was changed to return floats: https://github.com/crystal-lang/crystal/pull/8120. This change preserves previous behaviour (floor).

It seems there are more problems with newer crystal versions. The CI failures are in unrelated areas, mostly Readability.

So this repo is actually dead for now since everything in it has been moved into separate shards. The summarizer shard is located here. If you wouldn't mind reopening a PR there. Specs will then just be for the summarizer.


Testing WebKit

It looks like progress is being made on WebKit's WebGPU implementation: https://github.com/WebKit/WebKit/tree/main/Source/WebGPU.
I'm not sure what the actual status is, and it seems to only support Metal at the moment, but it might be possible to at least compile the WGSL compiler on other platforms to do some validation.

Interestingly, WebKit might end up using Dawn on at least some platforms: https://bugs.webkit.org/show_bug.cgi?id=216712.


Hide overlay while quest reward screen

Overlay messes to screenshot, need to hide overlay while quest reward screen (best if hidden when you press button to hide ingame HUD).
Also same function for other screenshoting modes.

There's an overlay toggle hotkey though (Default: Ctrl+Alt+Z). Too much work to find that in memory, implement everything, map it every patch, when you can just press a hotkey.


Rename label in create/alter contract view

In the basic information on a contract, there is a form field labeled “budget.” After introduction of tax deduction, this is ambiguous and should be renamed “net budget,” better yet, “net budget amount.”

The same problem applies to the Create/Alter Budget view, “Budget Total” must be renamed “Net Budget Total.”


[AutoPR batch/data-plane] typo: batch/data-plane/Microsoft.Batch

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4835

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/4031


Update async.md - Minor grammatical changes

Title
Minor grammatical changes
Summary
Minor grammatical changes
Details
Minor grammatical changes
Suggested Reviewers
none

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thanks @NonSecwitter!


[BUG] The same log explorer used for Security Lake data source can’t query MV/CI indices

What is the bug?
After a MV/CI index is created using the securitylake log explorer, if I try to query the created index within the same explorer, it will return index not found error for both SQL and PPL.
How can one reproduce the bug?
Steps to reproduce the behavior:

Go to Data sources -> securitylake -> query data -> open log explorer
Create a materialized view

CREATE MATERIALIZED VIEW last_1day_ct_2024_07_31_mv AS
    SELECT time_dt,
        actor.user.uid AS requestor_arn,
        accountid AS account_id,
        region AS region,
        src_endpoint.ip AS source_ip,
        api.service.name AS service,
        api.operation AS api_operation,
        api.request.data AS request_parameters,
        api.response.data AS response_elements,
        api.response.error AS error,
        api.response.message AS response_message,
        http_request.user_agent AS user_agent
     FROM securitylake.amazon_security_lake_glue_db_us_east_1.amazon_security_lake_table_us_east_1_cloud_trail_mgmt_2_0
     WHERE time_dt BETWEEN CURRENT_TIMESTAMP - INTERVAL '1' DAY AND CURRENT_TIMESTAMP
WITH ( auto_refresh = false)

REFRESH MATERIALIZED VIEW last_1day_ct_2024_07_31_mv

Run any query against the flint index you just created. E.g.
SELECT * FROM flint_securitylake_default_last_1day_ct_2024_07_31_mv

It will error out and say index not found.

What is the expected behavior?
Quey results returned.
What is your host/environment?

OS: [e.g. iOS]
Version 2.13
Plugins

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
Add any other context about the problem.

Catch All Triage - 1, 2, 3

CI (Covering Index) will be utilized automatically when querying source table, as discussed in https://github.com/opensearch-project/opensearch-spark/issues/298.


Integration with bootstrap-dateTimePicker

Hi,
I'm using bootstrap datetime picker on my project (https://eonasdan.github.io/bootstrap-datetimepicker/);
When the date field is with error all the inputs on the screen becomes red and with the same custom error message "invalid date". Even the valid fields become red.
Anyone managed a workaround or is using another date picker that works with the validaor?
Thanks!

oooops, sorry guys.
The error was my fault. I did not put the field inside a form-group.
Now works fine!
:)


Add zenodo DOI to readme

closes #99
I first added Zenodo oauth access for JuliaEcosystem, then turned it on for this repo, then used this script to submit our latest existing release (rather than making a new tag).

hm, not sure why it put my name first, going to try to edit the authors


Write unit test case for sample python code and get the code coverage

File operation
unit test(positive and negative)
code coverage report

Installed git,python
Unit testing info

file operations completed.
am not getting how to write that unit test cases.
i go through that code coverage but it will ask for an code coverage python package.

1.Using python function I have written one code.
2.Prepared one text document which contains theory part of unit testing.

1.Installed code coverage tool in python.
2.Execute the simple python code and get the code coverage report.

preparing ppt for unit test cases and code coverage for python program.


Improve release candidate container build cachability and reproducability

The release candidate container setup script makes reusing cached image
layers difficult because any change to the setup script (e.g. sbt/npm
version) invalidates the previous cached layers. This means anytime a
change is made to this script the entire container image must be rebuilt
which takes a lot of time and wasted space.
To avoid this, the container setup script is removed and all
functionality is moved to the Dockerfile. This results in more layers,
but makes it less likely to need to rebuild layers if there are changes.
Commands that are more likely to change due to versions or contents
(e.g. sbt/yarn install, release-candidate script) are put closer to the
end so if they are changed, less needs to be rebuilt.
More importantly, the installation of yarn and sbt do not provide a
specific version, so the version in the container depends on the latest
version available in the npm/sbt repositories when the container is
built. To improve reproducability, they are changed to install a
pinned version (the latest versions as of this change).
Additional minor updates to the container:

Switch from fedora-minimal to fedora. The size difference is minimal
compared to the final size of the container, and it replaces microdnf
with dnf, which has more capabilities to make the Dockerfile cleaner,
(e.g. quieter build, repo/repofrompath options).
Add the "--group-add keep-groups" option to the podman run command
needed for newer versions of podman.
Some files only need to be verified when building Daffodil and not VS
Code, only mention the correct files to avoid confusion.
Update sbt-pgp plugin to latest version
Update wix_wine.sh to remove debug message that SBT makes look like
errors.

DAFFODIL-2713

@stevedlawrence The changes being done here look pretty good to me. Is then any other process you would like me to do before approving? Like would you like me to pull this down locally and give it a test run to make sure stuff was working up to a certain point?

It's up to each individual committer how they choose to review, so I'll leave it up to you and others.
But if anyone wants to do a test, a good test would be to check out this PR and follow the steps in the release-candidate README.md to build and run the container (making sure to add the --dry-run option to the end of the run command). There shouldn't be anything that looks like an error and the release artifacts should be created. Because it's a dry run, nothing will get published and everything will be erased when you exit the container.
Any additional test to show the layer caching is working correctly is to modify the Dockerfile to use a slightly older yarn/sbt version and rebuild/run the container. The second build should be significantly faster than the first if layers are caching correctly.


Raspberry Pi Zero W compatibility

I'm submitting a ...

[ ] bug report
[x] feature request
[ ] support request

Hello,
I'm very much interested in seeing this working on a Raspberry Pi Zero W.
Will you add support for that?

@T-vK A few months ago I had it working on a Pi Zero, however, I don't remember if I had to make any adjustments. I don't see a reason why it would not. Did you run into a problem?

I haven't tried it yet. I just assumed it would only work on the 3B and 3B+. I'll report back when I have tried it.

@T-vK see issue #2. I managed to run on Rpi Zero W, you just need to change the base image to an armv6 one.

@cecchisandrone after your previous bug report I created the image arm32v6-1.0.4. Thanks
@T-vK use the image cjimti/iotwifi:arm32v6-1.0.4

Awesome. :) It would be nice to have this information in the Readme.


Update framework

This PR updates skelBML to include the latest version of the BioCro C++ framework. It also adds a new unexported function to each derived module library R package: framework_version. Now there are separate skeleton_version and framework_version R functions for each module library, and the main BioCro R package will also have its own framework_version function (pending an open PR on biocro-dev).

But actually, I wonder why include this in the C++ code at all? Wouldn't it be much simpler just to write an R function that returned the skeleton version directly?

It's important to keep in mind that BioCro is conceptualized as C++ software with an R interface. To the extent that it's possible and reasonable, any important BioCro functions should be written in C++; if they must also be available to R users, there should be associated R-to-C code and R code, as shown in Figure 3 of the BioCro II paper (https://academic.oup.com/view-large/figure/354193777/diac003f0003.jpg).
We haven't always done this in the past, but I'm trying to be better about it when adding new functions, even if they are simple ones like this one.

Again, I wonder about the use cases for this function. Unlike the framework version constant, which is associated with a well-defined set of C++ code files that the user of the submodule presumably won't modify, the skeleton consists of some files which the user is expected to modify and others which the user is expected to leave alone. Presumably, the "skeleton version" is mainly relevant to the latter. This all relies on a sort of honor system: if the user of the skeleton modifies files willy-nilly, the skeleton version number is largely irrelevant.

The skeleton version number does refer to a well-defined set of files: the files in the biocro/skelBML repository. As the skeleton library evolves, we may need to add new C++ or R functions to it. In that case, the version of the skeleton used to create a particular R module library package will determine which functions are available. This PR contains an example of adding a new C++ function (R_skeleton_version) and a new R function (skeleton_version). While I don't think these changes will happen very often, I can't guarantee that they will never happen. With this in mind, I think it's better to make the skeleton framework version accessible as soon as possible, so it is available if another package (such as the main BioCro R package) ever needs to require a minimum version.
It's true that this version number relies on an "honor system," but that's not unique to the skeleton library; this consideration also applies to any piece of open source code with a version number, right?

It doesn't seem appropriate, however, for the skeleton_version constant, since this really has nothing to do with the customizations (including the name) of the R module and corresponding module library the user is creating.

I guess I can see where you're coming from. The skeleton version number is namespaced but the framework version number isn't. If the framework version number doesn't need to be namespaced, why does the skeleton version number?
I guess if someone tries to compile a C++ project involving multiple module libraries, and some of those libraries use different versions of the C++ framework or were based on different versions of the skeleton library, there will probably be some compile-time issues. In this case it's probably not important to have either version number as part of a namespace. (Am I thinking about this the right way?)

This prompts me to suggest that it would be a good idea to add, at the top each file listed in the README as a file to avoid modifying, a statement saying as much. (I think the standard SGML/XML/HTML comment markers (<!-- ... -->) work to hide comments in *.md files when viewed using a Markdown viewer.) Moreover, there should perhaps be similar statements at the top of the automatically-generated files—the templated files—produced by the setup script, stating that they are automatically generated, along with a clear indication of whether the resulting file may be customized or should be left alone.

This is a good idea, I will work on adding these statements where appropriate.

Regarding putting the skeleton version information in the C++ code:
My point was that since nearly all of the C++-related files in the skeleton have to do only with the R-interface, it didn't seem to me at all necessary to keep the skeleton version information, which is really only relevant to R users (or so I thought), in the C++ code.
Taking an inventory of the files here to see to what extent changes to the skeleton might be relevant to C++-only users, I came up with the following:
inc — the Boost library has it's own versioning
init.cpp — this is only needed by the R interface
R_* files — these, of course, are only needed by the R interface
module_library.cpp — the user is expected to modify this, and so it didn't seem particularly tied to a particular skeleton version
example_module.h — it seems likely the user won't even keep this file, so again, it doesn't seem worth tracking it with a skeleton version
skeleton_version.h — only a circular argument would say we need to track this at the C++ level
The only file which seemed to me "C++-only" and meant to not be user-modified was the file module_library.h.  However unlikely it may be that this file will change, perhaps for the sake of it alone we should keep the skeleton version information at the C++ level.  There is also the possibility, which I think you alluded to, that we might in future add more such files in this category.
Also, there is the borderline case of the Makevars files (Makevars and Makevars.win).  While presumably these are only used to compile the C++ code in the context of an R framework, it is possible that a user interested only in producing a C++ library file from the C++ code might use an R installation to do so.

It doesn't seem appropriate, however, for the skeleton_version constant, since this really has nothing to do with the customizations (including the name) of the R module and corresponding module library the user is creating.

I guess I can see where you're coming from. The skeleton version number is namespaced but the framework version number isn't. If the framework version number doesn't need to be namespaced, why does the skeleton version number?

I didn't necessarily mean that skeleton_version shouldn't be namespaced.  I was just questioning whether it really belongs in the same namespace as the library modules.
Namespaces are both a way of conceptually organizing code and a way of preventing name clashes.  I think I mainly had the former in mind when I made this comment.

I guess if someone tries to compile a C++ project involving multiple module libraries, and some of those libraries use different versions of the C++ framework or were based on different versions of the skeleton library, there will probably be some compile-time issues. In this case it's probably not important to have either version number as part of a namespace. (Am I thinking about this the right way?)

I'm still rather fuzzy on how using multiple module libraries will work at the C++ level.  It's probably time to start experimenting and time to start thinking more seriously about this.


Python not found when creating virtualenv

Hi,
Thank you for the tutorial.
When I run `reticulate::virtualenv_create(envname = 'python35_env', python= '/usr/bin/python3') I get this error:
"Error in system2(python, args, stdout = TRUE, stderr = FALSE) :
'"/usr/bin/python3"' not found"
I know this is probably a basic issue but do you have any idea why it might not be able to find Python at that path? And are there alternatives I should try?
My Python code is working fine locally. I just want to create a new virtual environment that will let me host the shiny app online.
Let me know if I can provide any more info or run any diagnostic code to check if Python is available/where it is.
Thank you for any help.

Try:
reticulate::virtualenv_create("python35_env", python = "python3")
with a .Rprofile in the same folder that contains this:
Sys.setenv(RETICULATE_PYTHON = "~/.virtualenvs/python35_env/bin/python")

Hi @Alex-A14, I've updated this repo so the app is now fully self-contained (i.e. you no longer need to create the virtualenv separately; the app now handles virtualenv creation automatically). These latest changes should make the tutorial easier to follow, and straightforward to deploy to shinyapps.io. Let me know if you still run into issues!
As for your question about seeing the location where Python is installed on your machine, you can use the command which python. The command python3 --version will print the exact version of Python 3 you have.

Hi. Thanks for the tutorial. I am still confused about "the app now handles "virtualenv creation automatically). Sys.getenv("PYTHON_PATH") returns empty. So, how is ti possible?
Do I need to modify my R profile?

Hi @FrancyJGLisboa - there is an .Rprofile file included in this repo here that sets the PYTHON_PATH env var. Do you have that file in your working directory? You may need to restart the R session to make sure it gets loaded and then Sys.getenv("PYTHON_PATH") will return "python3".

Hi Rani, thanks for the quick answer.
I am almost able to replicate your APP with my settings.
I have to call a data frame using the API TOKEN.
This token needs to be pasted by the user in a Textarea input.
The user also needs to set more five inputs to query the data frame. These
inputs should be used in a python function I am sourcing from a .py file.
I am getting an API error 401. Can you provide any hint on how to use shiny
inputs in a python function that uses pandas?
Thx
On Thu, 21 May 2020 at 13:30, Rani Powers<EMAIL_ADDRESS>wrote:

Hi @FrancyJGLisboa https://github.com/FrancyJGLisboa - there is an
.Rprofile file included in this repo here
https://github.com/ranikay/shiny-reticulate-app/blob/master/.Rprofile
that sets the PYTHON_PATH env var. Do you have that file in your working
directory? You may need to restart the R session to make sure it gets
loaded and then Sys.getenv("PYTHON_PATH") will return "python3".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ranikay/shiny-reticulate-app/issues/1#issuecomment-632192685,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXCGY4MS3OUULOT2SVQXTLRSVJJZANCNFSM4LROLEUA
.

--
Francy J. G. Lisboa
PhD in  Agronomy

To use additional python packages, add them to the PYTHON_DEPENDENCIES list on line 7 of server.R so that they get installed into the virtualenv. When you source your .py file, similar to the way that I did with python_functions.py in this app, make sure to include import pandas at the top of your .py file.


added table of contents

Now there are two TOCs.
Left to the README.md there is an icon. When you click on it, it shows you the TOC.

https://github.blog/changelog/2021-04-13-table-of-contents-support-in-markdown-files/


M3U subtitles support.

M3U based subtitles (see #133 for some context) use multiple files for a single subtitles track. addExternalCaptions() could be extended to handle these types of subtitles. Work required for this feature may overlap with work required to implement TTML support (#111).

Unfortunately, we don't have the bandwidth right now to add this.  It basically amounts to subtitle manifest outside of the main content's manifest, and it doesn't fit into our current architecture.
It is trivial to add vtt subtitles to a DASH manifest, and if you can't modify the manifest, you can always use addExternalCaptions() in the mean time.

Closing as a duplicate of #279, which is more general and covers all HLS support.


Fix for readme fix

In context, the word is "shrunken," not "shrunk" 😅

i changed a word in a readme, so i'm relatively confident these build errors aren't about me

@StoneCypher From CONTRIBUTING.md:

README.md is generated from the source with ljs

So you need to edit the source (arbitraryBless.js in this case), and then run make literate.

i went to go do that, and it looks like someone else already has


Installation Problem

Hello,
How can we create a new directory and install this project on cisco sf300 switch and fs 1130 switch?
Thanks in advance.

Never tried it, don’t have any experience on those switches.  Highly unlikely that it will work.
OOM depends on having a shim that allows it read/write access to the modules over the i2c bus.  Do those switches have an interface to the module EEPROMs?  On ONL and SONiC there are device files in /sys/bus/i2c/devices that go through a kernel driver and provide read/write access.  Is there anything like that available?
Feel free to contact me directly at<EMAIL_ADDRESS><EMAIL_ADDRESS>Don
From: soheer<EMAIL_ADDRESS>Sent: Thursday, October 01, 2020 1:45 AM
To: opencomputeproject/oom<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [opencomputeproject/oom] Installation Problem (#19)
Hello,
How can we create a new directory and install this project on cisco sf300 switch and fs 1130 switch?
Thanks in advance.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/opencomputeproject/oom/issues/19 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AD6VCJ2F6SZC6XSSXI6XOUTSIQ6PNANCNFSM4SAEA6XQ .


Unofficial Fork of app

As discussed briefly in https://github.com/Dar9586/NClientV2/issues/764 , the project is abandoned,
No one came forth wanting to maintain a fork of the app.
As such I created a fork where work and issues should be done from now on.
There is already a release fixing the crash on startup (webp problem) and the image not loading (new subdomains) problems.
There is a migration help in the README on the repository page.
Fork: https://github.com/maxwai/NClientV3
Issues and Pull requests are welcomed. I'm also looking for developers and maintainers that can help with the app.

Thank you. And I try to introduce one:
https://github.com/Kronos2308, you can see his commit;you are all so great

Nice work, starred.

thank you.😘😘😘

@maxwai I've been working on a fork as well but so far I've mostly been
rewriting screen by screen in Kotlin/Compose. Though, at some point I'd
like to address some UI/UX problems as well.
Unfortunately there's just waaaay too much technical debt and legacy
Android code to continue maintaining this codebase as is.

@maxwai I've been working on a fork as well but so far I've mostly been我也一直在研究一个分叉，但到目前为止，我大部分时间都在 rewriting screen by screen in Kotlin/Compose. Though, at some point I'd在 Kotlin/Compose 中逐个屏幕重写。不过，在某些时候我会 like to address some UI/UX problems as well.也想解决一些 UI/UX 问题。
Unfortunately there's just waaaay too much technical debt and legacy不幸的是，有太多的技术债务和遗留问题 Android code to continue maintaining this codebase as is.Android 代码继续保持此代码库的原样。

Thank you for your efforts


oneOf change the branch after creation

Once oneOf branch is selected, it can't be changed from UI. The select box is disappearing.
We need ability to change the oneOf branch after creation.
So we can keep select box and allow user to change it.
All data under this oneOf will be destroyed.

@danziamo any ideas how to fix?


[FEATURE] Handle flow balance separately by using available balance

Basically, we need use available flow balance across our wallet for flow token balance. Which including move and send action.
In this way, even user use max amount. The balance won’t lower than 0.001 or the minimum storage requirement.
Currently, we query flow token balance along with other tokens together and its total balance.
We need handle flow balance separately by using available balance.
Additional condition: If user turn on free gas fee, max transferable amount should be available amount. If user turn off free gas fee, max transferable amount = available amount -0.001

@jaymengxy
We do not need to let max transferable amount = available amount -0.001 when user not turn off gas free fee when send FLOW token between flow accounts.


Added  LAPACK option to test_h5.cpp to compare residuals and number o…

…f operations.

Build failed because it was depending on libs (Intel MKL) that are not available in the automatic build environment.


Proxy connections

:bulb: Idea

Proxy connections through Tor, I2P or basic proxy server to maintain anonymity.
:computer: Target components

Android and iOS apps.

Please provide more information. I have no idea what you mean.

Add ability to proxy the requests made by ntfy allowing us to pass the connection through Tor and I2P and hide our IP from the instance.

Add ability to proxy the requests made by ntfy allowing us to pass the connection through Tor and I2P and hide our IP from the instance.

A good example for that is Tusky which added Proxy Support in 2017.
This way people can use Orbot to torify traffic.

It's also beneficial in cases where one needs to use a Proxy [i.e. Corporate Networks] to bypass blocks of HTTPS and other non-DPI-able traffic. (Yes, those networks do exist and no, I cannot elaborate any further.


No hotkey or smart tips

Hi brother,
First of all thanks for your open source project and it's great and gives me lots of help in my project.
Then there is a advise that is it possible to add a smart tip (showing a list of resource keywords) after I enter a keyword(like ctr+j) or press the space key? and I bet this project would be even more popular if it were possible.
Thanks a again.

Hi,
first that is a cool idea. But this have to be an extra visual studio extension an can not be combined with this library.
I add here some links for later clarification.
https://docs.microsoft.com/en-us/visualstudio/extensibility/walkthrough-displaying-quickinfo-tooltips?view=vs-2019
https://github.com/tomasr/viasfora
I will definitely let this here as Idea, but more for 2021/22
bye
Konrad

Cooool
Noted, thanks

I think support for IntelliSense is possible, check the following first:

localization-of-a-wpf-app-the-simple-approach
https://github.com/jakubfijalkowski/codinginfinity.me/issues/3#issuecomment-1656743656

Then look at my specific solution:
IPConfig/Languages


Disable selection for decorative elements

Use user-select: none to avoid

text-selection for links containing only icons
selection of line numbers from code fences

Done in 03ba3f7ba5990cec2c304726c323f16b17f30f4d and 54385e09bc9a5adc723b63c91c3ba60d6bb027a8


added python 3.11 compatibility

Added python 3.11 compatibility in /src/onnxflow/build.py as per Gleb's request

Hey Arash, I ran the scripts at /mlagility/examples/cli/scripts to test. I should probably should do more thorough testing though...
Not sure about python 3.8, I suspect the cpu/gpu tests will run fine, not sure about the groq tests though. Will look into this and update if necessary. Thanks!

Hey Arash, I ran the scripts at /mlagility/examples/cli/scripts to test. I should probably should do more thorough testing though...
Not sure about python 3.8, I suspect the cpu/gpu tests will run fine, not sure about the groq tests though. Will look into this and update if necessary. Thanks!
That sounds good, as long as it exercises that piece of code. And yeah doesn't matter whether it's CPU/GPU or Groq for this MR.


Add math rendering to posts using KaTeX

What kind of changes does this PR include?

Changes with larger consequences (logic, library updates, etc.)

Description
Adds support of rendering TeX maths equations & symbols inside posts using KaTeX. Wrap a TeX statement with $$ to allow rendering.
Tex rendering before this PR:

Tex rendering after this

After using this for a while, I have realised it does have some issues with the RSS Feed regarding rendering, for example:
Website:

RSS feed (I use Feeder):

This is caused by KaTeX using CSS to render the text, as KaTeX shows both a nice formatted version as you see on the webpage, just also has a version for screen readers to parse, which is hidden, hence the repeating of text in RSS.
I only just now had a look at Astro's RSS documention, so a custom stylesheet might need to be implemented to fix these issues.

ok, having a look on how RSS works in general and how its set up in this template, it seems that when a RSS reader fetches the full article, its up to the RSS reader to format it. the only way to fix it would be to include the content directly into the rss.xml, which might be undesirable.

Ahh I see what you mean. That's a shame but I guess it makes sense it's all css. I know some readers will drop any custom stylesheet, so yeah you're probably right in that it's going to be including the content directly, which still might need 'cleaning-up'.
Umm. I'm not completely against adding the content directly,  e.g. Astro's docs, but I'm guessing it's not going to be much better, unfortunately, but I'll see what output we get.

Here's the KaTeX docs if your curious and the stylesheet:
https://katex.org/docs/browser
<link<EMAIL_ADDRESS>rel="stylesheet">
Now KaTeX isn't the only option, as specified in the remark-math monorepo, MathJax being another option, but I wouldn't recommend. It has an option to turn the LaTeX into an SVG, but does break formatting on the website. The other options would require css aswell.

Hi, I'm interested in this PR. Is there any way for me to help? Not KaTeX specific, but math notation in general, I'd be glad to contribute.

The reason this has gone stale is more so over a design decision, than a technical implementation.
The issue currently is that RSS readers don't render the custom CSS used by KaTeX to render the math notation. The other program that can be used, MathJax, can render it out into images, but this causes issues with in-line rendering.
Basically RSS can't really support math notation in a way I have seen. If you can somehow find a way to overcome that, and also make it look decent on the website itself, then we can implement it.


BVTCK-57 Followup: Avoid the use of @BeforeMethod

It's a follow up on

https://hibernate.atlassian.net/browse/BVTCK-57

As @gunnarmorling mentioned in the comments of https://github.com/beanvalidation/beanvalidation-tck/pull/86#issuecomment-287355830 I've added checkstyle plugin with configs for imports only. Turned out that there were a few violations so I've cleaned those up.
Also with that last commit I've removed the last Before/After's they were used to change Locale.
As enabling other checlstyle rules will require some code modifications (for example NewlineAtEndOfFile is failing in each file :) ), I think it'll be better if done separately from this one.
Let me know if that would be useful and if so - I can work on it :)

@gunnarmorling the issue I see with using checkstyle to avoid the usage of these 2 annotations is that they indeed might be used correctly. Typically, in the locale case, I don't think it's an issue to use them. I don't foresee a lot more of these correct usages though so we might consider this limitation acceptable.

they indeed might be used correctly. Typically, in the locale case, I don't think it's an issue to use them

One can turn off CS for specific code sections via comments (//CHECKSTYLE.OFF ... //CHECKSTYLE.ON). It's a bit verbose, but I think it's a good trade-off as there should be just a few legitimate uses.
That said, is setting the locale on the client-side actually a legitimate use? Shouldn't it be done within the VM running the tests?

That said, is setting the locale on the client-side actually a legitimate use? Shouldn't it be done within the VM running the tests?

It sure can. It's probably more correct but also more verbose. I think Marko found a good trade-off (once my remarks are applied) so I would lean towards accepting it. Just wanted to point this out.

This one still needs more time to be updated to add back @BeforeMethod and add //CHECKSTYLE-OFF for the few locale uses, right?

@gunnarmorling that's right. I'll be doing this a bit later today.

I've reworked last commit and added SuppressionCommentFilter and commented out those two imports with //CHECKSTYLE-OFF //CHECKSTYLE-ON. So this PR seems to be complete now.
As for the other checkstyle rules - would it be of use enabling them and making the code comply those rules?

So this PR seems to be complete now.

Awesome, rebased and applied. Thanks a lot, Marko!

As for the other checkstyle rules - would it be of use enabling them

Not needed from my side. I'm no CheckStyle zealot really. @gsmet?


Create a release

Before we can merge this https://github.com/sigma0-xyz/zkbitcoin/pull/33#issue-2117112857 we would need to create at least release.

The tag of the release commit should start with v e.g. v0.1.0

cc @imikushin

We have our first release: https://github.com/sigma0-xyz/zkbitcoin/releases/tag/v0.1.0


Expose govuk_closed_status in api

Following on from #3781 this will give some valuable contextual
information in interpreting superseded_organisations and
superseding_organisations.

@heathd what's the difference between govuk_closed_status and the existing govuk_status, which can sometimes have the value closed, eg https://www.gov.uk/api/organisations/deputy-prime-ministers-office?

@frankieroberto govuk_closed_status specifically describes the reason for the closure see eg. https://github.com/alphagov/whitehall/blob/master/app/models/organisation.rb#L157

@heathd aha! I did wonder where all the different messages on the closed org pages came from. I'd assumed it was free text.
I'd suggest renaming the field in the API to closed_reason or closed_because or similar, just to make it a bit clearer.

@heathd any chance you (or someone) could take a look at this, whenever you have a spare moment?  I'd like to do a thing with the data...

@thomasleese 🎊  thank you!


docs: update to v0.12.2

Update docs to v0.12.2
Description
This PR updates the docs to the new version v0.12.2. Also it updates the operator and aligned version to v0.12.2

stalled


Add minimum genes parameter to (pairwise) macro-synteny microservices

The macro_synteny_blocks microservices takes a chromosome as an ordered list of gene families and computes pairwise synteny-blocks between it and all other chromosomes in the database. To compute synteny-blocks for multiple chromosomes in parallel, the pairwise synteny-block computations are actually done by the pairwise_macro_synteny_blocks microservice. Specifically, for each chromosome in the database, the macro_synteny_blocks microservice sends a request to the pairwise_macro_synteny_blocks microservice via a gRPC request (all microservices support both HTTP (JSON) requests and gRPC requests).
In this issue we want to add an optional minimum genes parameter to both the macro_synteny_blocks and pairwise_macro_synteny_blocks microservices. The parameter will be optional to maintain backwards compatibility with existing clients. We'll call this parameter optionalChromosomeGenes in the protocol buffer files and min_chromosome_genes in the Python code.
To implement this feature, create a new "macro-synteny-filtering" branch in your fork of the microservices repository. Then:

Add the optional parameter to the macro_synteny_blocks service's protocol buffer and the pairwise_macro_synteny_blocks service's protocol buffer. You will also need to update the macro_synteny_blocks service's copy of the pairwise_macro_synteny_blocks service's protocol buffer.
Update the macro_synteny_blocks service's gRPC server to unpack the new optional parameter. Ditto for the pairwise_macro_synteny_blocks service's gRPC server.
Update the macro_synteny_blocks service's HTTP server to unpack the new optional parameter. Ditto for the pairwise_macro_synteny_blocks service's HTTP server.
Update the macro_synteny_blocks service's argument parser to verify the type and value of the new parameter are valid. Ditto for the pairwise_macro_synteny_blocks service's argument parser.
Update the macro_synteny_blocks service's process function to take the new optional parameter and pass it to the _getTargets and _computePairwiseBlocks methods.

Update the _getTargets method to take the new optional parameter and use it to refine the RediSearch query to limit the chromosomes to ones that have at least as many genes specified by the parameter.
Update the _computePairwiseBlocks method (and subsequently the computerPairwiseMacroSyntenyBlocks function) to take the new optional parameter and use it.

Update the pairwise_macro_synteny_blocks service's process method to use the new optional parameter (hint: it should be used just like the matched parameter).

It may seem redundant to have both microservices filtering on the parameter but we want to prevent the macro_synteny_blocks service from sending unnecessary requests to the pairwise_macro_synteny_blocks service while allowing other apps (such as ZZBrowse) to use the same parameter directly on the pairwise_macro_synteny_blocks service.
 However, don't open a PR in the microservices repo. The issue is part of an epic so you'll want to complete all issues in the epic before making a PR.

Done in commit 1ade8e144d26994c462e2961d11ff3350194535e. Closing.


CoreLocation returns .notDetermined before Pop Up Button is pressed

I've tried using SwiftLocation in a new app I'm creating using async await. I did it according to the readme:
try await location.requestPermission(.whenInUse) // obtain the permissions
let userLocation = try await location.requestLocation() // get the location

I was however surprised that even though the app is supposed to await the requestPermission, in practice the app nearly immediately tried to request a location even though no button on the permission popup was tapped. Obviously no location was returned as no permission was granted.
Investigating the issue I found that CoreLocation immediately returns .notDetermined when the popup appears, at least it does while I was testing it on iOS 17.2. Then when the user taps a button, the correct permission is returned, but by then the await has already been continued with the wrong value: .notDetermined.
So I have added code to ignore .notDetermined - this now works for me, only once a user taps a button on the pop up, does the await return.

I've tried to replicate the issue with no luck:
@IBAction public func ciao() {
        Task {
            do {
                try await location.requestPermission(.whenInUse) // obtain the permissions
                print("requested permission")
                let userLocation = try await location.requestLocation() // get the location
                print("called immediately")
            } catch {
                print(error.localizedDescription)
            }
        }
    }

In your case "called immediately" is executed right after the system popup is showed, right?

I tried again - I think it depends on where you init Location(). If you do it within the Task the issue I describe above occurs. If I init location as part of a struct variable (on the main thread) this issue does not occur.
Maybe initing Location() within a Task isn't valid, but then there should be warnings about this I guess...
        .onAppear {
            Task {
                do {
                    let location = Location()
                    print("requesting permission")
                    try await location.requestPermission(.whenInUse) // obtain the permissions
                    print("requested permission")
                    let userLocation = try await location.requestLocation() // get the location
                    print("called immediately")
                    print("userLocation: \(userLocation)")
                } catch {
                    print(error.localizedDescription)
                }
            }
        }

If you init Location in the main thread you get a warning. Is it safe to init within a Task?


Add Azure Event Hubs as a PubSub component

Describe the proposal
Add Azure Event Hubs as a Dapr PubSub component

To use Azure Event Hubs as a PubSub component:

Create Azure Event Hubs namespace and an Event Hub
Create Azure Blob Storage account and container
Use the following component config

apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: <enter a name e.g. eventhub-pubsub>
spec:
  type: pubsub.azure.eventhubs
  metadata:
    - name: connectionString
      value: "Endpoint=sb://<enter namespace>.servicebus.windows.net/;SharedAccessKeyName=RootManageSharedAccessKey;SharedAccessKey=<enter key>;EntityPath=<enter event hub name>"
    - name: storageAccountName
      value: <enter storageAccountName>
    - name: storageAccountKey
      value: <enter storageAccountKey>
    - name: storageContainerName
      value: <enter storageContainerName>

@pkedy can this issue be closed as completed?


fixed the alignment issue of team members

Description

The alignment of the names of winners was not right. It was because the divs with .podium-items class were flex items because they are inside the .container div which is a flex container and had align-items:flex-end;. That's why the names were aligned to the end of the container. I fixed the issue by converting each .podium-items div into css grid of 2 rows and 1 column. Then changing the align-items: to stretch and setting relevant height did the job.
Related Issue

Fixes # 1
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Motivation and Context

The change is required because it makes the leaderboard look cleaner and easier to read.
How Has This Been Tested?

I tried to increase the length of the name of a winner, and it was still aligning perfectly. Even in the mobile viewport the alignment is ok.
Steps to reproduce:

Open inspect tools and find the html part where the names of winners are written using select tool.
Try to increase the length of any of the name.
Try changing the viewport size.

Screenshots (if appropriate):

Checklist:

[x] I have registered myself at Contrihub website.
[x] My code follows the code style of this project.
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] Any dependent changes have been merged and published in downstream modules
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

@hexwhiz i don't want the podium like this the leader (middle person) should be at the top

Ok @meisabhishekpatel  I will fix that. I have got 2 hrs remaining so if the timer runs out, can I still make the PR?

@meisabhishekpatel Is this ok?

I have added a new commit regarding the height issue. I also added some margin to team names so that name of middle person doesn't overlap with the team's name when zoomed.

@meisabhishekpatel would you please review my PR?

closes #1


Startup event order is different

VisibleChanged happens before Load when loaded using Show() or ShowDialog().
This is detected in Windows 10 targeting framework 4.5.2 and 4.6.1

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 76d3a80d-9468-0283-435b-f45f4827b920
Version Independent ID: c879f822-8677-5e20-bcc7-52e1595d8883
Content: Order of Events in Windows Forms
Content Source: docs/framework/winforms/order-of-events-in-windows-forms.md
Product: dotnet-framework
Technology: dotnet-winforms
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

And also activated runs before load event.

@merriemcgaw @RussKie can you help validate what would be the right order of events in WinForms?

The currently documented order is correct for a form launched via Application.Run(form) or form.Show().
For a form launched via form.ShowDialog() the order of events for the first time should be the same as the documented, e.g.:
form3: HandleCreated
form3: BindingContextChanged
form3: Load
form3: VisibleChanged
form1: Deactivate
form3: Activated
form3: Shown

For the 2nd and subsequent calls it will be:
form3: HandleCreated
form3: Load						<-- this call went off earlier
form3: BindingContextChanged
form3: VisibleChanged
form3: Activated
form3: Shown

This is due to different code flows resolving visibility. A form shown via form.ShowDialog() remains alive after being closed (form1 and form2 will be disposed once closed), and when it is shown again the order of events is different.
https://github.com/dotnet/winforms/blob/9bdd2e4e73ed5e9dd37e6e09a7d38143e64ec025/src/System.Windows.Forms/src/System/Windows/Forms/Form.cs#L2329-L2375
Test harness:
public partial class Form1 : Form
{
    public Form1()
    {
        var form2 = new Form { Name = "form2" };
        BindEvents(form2);

        var form3 = new Form { Name = "form3" };
        BindEvents(form3);

        var button = new Button
        {
            Text = "Show form",
            Location = new System.Drawing.Point(8, 8),
            AutoSize = true
        };
        button.Click += (s, e) => form2.Show();

        var button2 = new Button
        {
            Text = "Show dialog",
            Location = new System.Drawing.Point(8, 40),
            AutoSize = true
        };
        button2.Click += (s, e) => form3.ShowDialog();

        this.Name = "form1";
        this.Controls.Add(button);
        this.Controls.Add(button2);

        BindEvents(this);
    }

    private void BindEvents(Form form)
    {
        form.HandleCreated += (s, e) => Debug.WriteLine($"{form.Name}: HandleCreated");
        form.BindingContextChanged += (s, e) => Debug.WriteLine($"{form.Name}: BindingContextChanged");
        form.Load += (s, e) => Debug.WriteLine($"{form.Name}: Load");
        form.VisibleChanged += (s, e) => Debug.WriteLine($"{form.Name}: VisibleChanged");
        form.Activated += (s, e) => Debug.WriteLine($"{form.Name}: Activated");
        form.Shown += (s, e) => Debug.WriteLine($"{form.Name}: Shown");
        form.Closing += (s, e) => Debug.WriteLine($"{form.Name}: Closing");
        form.FormClosing += (s, e) => Debug.WriteLine($"{form.Name}: FormClosing");
        form.FormClosed += (s, e) => Debug.WriteLine($"{form.Name}: FormClosed");
        form.Deactivate += (s, e) => Debug.WriteLine($"{form.Name}: Deactivate");
    }
}

Tested on net461;net472;netcoreapp3.0

That is very odd, because that is not what I experienced.  As I said, I saw VisibleChanged happen before Load.

Without a repro and stack traces I am unable to provide further comments, sorry.

I will endeavour to provide this as soon as I can.


Add docs for the hostdb cache metrics

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1205/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1098/ for details.


eth_newFilter / eth_getFilterChanges not working

Description
There was a breaking change introduced in this PR
The problem is that whether shared cache is configured or not we try to write filter id to the shared cache:
https://github.com/hashgraph/hedera-json-rpc-relay/blob/main/packages/relay/src/lib/services/ethService/ethFilterService/index.ts#L94
But then we read the key from whatever cache is enabled:
https://github.com/hashgraph/hedera-json-rpc-relay/blob/main/packages/relay/src/lib/services/ethService/ethFilterService/index.ts#L212
As result if there's no shared cache set, this functionality doesn't work.
Steps to reproduce

Set up Hedera JSON RPC relay so that Filters are enabled but there's no Redis cache set up
Create filter:

curl -H "Content-Type: application/json" -X POST -d'{"method":"eth_newFilter","params":[{"topics":["0xd78ad95fa46c994b6551d0da85fc275fe613ce37657fb8d5e3d130840159d822"],"address":["0x1e375f8947dffee57c7713f3176b01858bfba104"]}],"id":2,"jsonrpc":"2.0"}'  http://your-hedera-relay

Try to request filter changes (while specifying the filter ID obtained on the previous step):
curl -H "Content-Type: application/json" -X POST -d'{"method":"eth_getFilterChanges","params":[""],"id":4,"jsonrpc":"2.0"}' http://your-hedera-relay
Receive an error:

{"error":{"code":-32001,"name":"Filter not found","message":"[Request ID: 063e3a42-c3ee-4579-a9d5-cac84cd07a42] Filter not found"},"jsonrpc":"2.0","id":4}

Additional context
No response
Hedera network
mainnet, testnet
Version
v0.50.1
Operating system
Linux

Is this fixed? I am getting same error with redis cache disabled and using version 0.55.0


CI/CD

Hello,
Would it be possible to add a CI pipeline that releases a binary with each tagged commit to eliminate the need for compilation on the server side?
Regards,

Absolutely possible - I need to make a new release soon anyways so I'll try and do this work with that release.

Awesome thanks you !


Faster PDF preview generation

Right now, we use https://github.com/barteksc/PdfiumAndroid for rendering PDF previews.
The library is loaded per each document instead of loading it once per batch,
because it doesn't have API to re-use instance several times.
We should consider alternatives:
https://pspdfkit.com/guides/android/viewer/kotlin/
https://developer.android.com/reference/android/graphics/pdf/PdfRenderer.html
https://github.com/pdf-rs/pdf

Testing data:
https://www.taran.space/dev/TEST_PDF.zip

https://github.com/ARK-Builders/ARK-Navigator/pull/329


Language Selection

It is possible that i just cannot find it (sorry for that case):
TMDB has informations for each movies in different languages but how do i select a specific language in the search request ("EN", "DE", etc) ?
EDIT: Ok found out the DefaultLanguage property sets the language, but it does not affect the results for the "SearchMovieAsync" method ?

If DefaultLanguage is not setting for a single method, it might be a bug - check if the methods have an overload that accepts a language parameter.

OK got the problem: The language string is case sensitive...when i input "DE" (for german) then the language is not recognized but with lower case "de" it works.

Interesting, I don't think it's us that make this into an issue but the api might. Perhaps we should lowercase the input of this field? Or change it over to a an enum but that might be more maintenance for little gain.

Perhaps we should. I'll raise an issue with tmdb to hear if it's always the case, and/or if we can get a static list of languages and countries.
Sent from my iPhone

On 17. aug. 2016, at 08.55, Thomas Vriens<EMAIL_ADDRESS>wrote:
Interesting, I don't think it's us that make this into an issue but the api might. Perhaps we should lowercase the input of this field? Or change it over to a an enum but that might be more maintenance for little gain.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

https://www.themoviedb.org/talk/57b4907e925141712f000bd3


fix: resolved 'Workspace not found' issue in devcontainer setup

Description
On setting up CircuitVerse through dev container, the error Workspace not found was encountered.

Analysis
Upon analyzing the devcontainer setup, I found that this issue was due to the CircuitVerse folder not being mounted into the workspace.
Solution
To resolve this, I added a line in the docker-compose file of the devcontainer to mount the CircuitVerse Project folder into the workspace. This change ensures that developers and contributors can now use the devcontainer for their development environment setup without encountering this error.

Summary by CodeRabbit

New Features

Introduced a volume mapping in the development container configuration, allowing seamless file access between the host and the container for improved development workflow.

@Darshan808 Could you investigate why the checks aren't green

@tachyons
Thank you for your feedback! I have checked and it seems that all the necessary checks are green. Could you please let me know if there is a specific check you'd like me to investigate further? I'd be happy to look into it.

@tachyons  I noticed that one of the checks isn't passing. After consulting with Tanmoy, I found that the issue was related to the CI configuration. I've addressed this in #5080
Once #5080 is merged, the CI tests should pass smoothly, and we can proceed with merging this one as well.

@Darshan808 Great job


Support WMF files that start with META_PLACEABLE record

Hi there, according to MS-WMF <IP_ADDRESS>, it seems that WMF files may start with a META_PLACEABLE record before the META_HEADER record. These files currently give a "Header: Type 52695 must be 1 or 2" error with js-wmf.
I think these files could be supported by updating image_size_prepped_bytes() to check for files that start with the META_PLACEABLE signature of 0x9AC6CDD7, and then skipping ahead 22 bytes to get to the actual META_HEADER record.
As for sample files, it looks like most of the "iPres Systems Showcase" WMF files are of this "placeable" variant.
I'm happy to work on a PR if you would like.

Good catch!  The initial problem was to support Word charts and image annotations, which are usually standard WMF.  A PR would be appreciated :)

Good catch!  The initial problem was to support Word charts and image annotations, which are usually standard WMF.  A PR would be appreciated :)

Hi, may I know is the pull request applied? I run into the same error when trying to display WMF files generated by MathType, a Word extension for editing math formulas. I will be very appreciative of any help on this issue 💯

hello, I am interested in such a version as well

I tried creating my own code from this https://joseluisbz.wordpress.com/2013/08/06/obtaining-size-or-dimension-of-images/
but its not working
var uint8 = Uint8Array.from(atob(base64), c => c.charCodeAt(0));
var dV = new DataView(uint8.buffer);
var w, h;
var magic = dV.getUint32(0, true).toString(16).toUpperCase();
if(magic == '9AC6CDD7') {
w = dV.getUint16(10, true)-dV.getUint16(6, true);
h = dV.getUint16(12, true)-dV.getUint16(8, true);
}
else {
var PWMF = 18;
var SizeR;
while(PWMF < dV.byteLength) {
	SizeR = dV.getUint32(0, true);
	var m = dV.getUint16(4, true).toString(16).toUpperCase();
	console.log(m);
	if (m == "0C02") { //020C
		w = dV.getUint16(PWMF+8, true);
		h = dV.getUint16(PWMF+6, true);
	}
	PWMF += (2*SizeR);
}
}

Add support for META_PLACEABLE record in my fork.
What about support for all the missing record types?


Use uv pip in makefile (PR #7961)

Second stage for #7961

Processing this pull request

Pull request rejected by merge tool. The tests for this branch did not succeed.

Processing this pull request

Merged into branches iso7 in 595fca1bd30e39532974eef3009c2c3ba45dbb0e


Custom identity providers

@jonatanklosko should we move env.sh to extensions/env.sh?

should we move env.sh to extensions/env.sh?

Perhaps we should have $RELEASE_ROOT/user for user-written files, and there we have env.sh and extensions/, wdyt?


"Gura Humorului" is listed in the drop-down menu

Description:
"Gura Humorului" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Gura Humorului" in the search bar.

Expected results
The location is listed in the drop-down menu as "Oraș Gura Humorului, SUCEAVA ".


Blogs - Multi-Cluster monitoring with Prometheus and Submariner

Signed-off-by: I<EMAIL_ADDRESS>

@skitt @sridhargaddam can you please take a look at this? I want to push some changes to the same file and prefer this one to be merged first. Thanks!