Closed 3ddiehead closed 3 years ago
It appears that putting in two arguments for strongStart_Hsu is going to be a problem with the JSON string to function-call. For example, when calling the function for the creation of a tableau, 'strongStart_Hsu-w-i' does not work. The system indicates that the "i" was unexpected. I could change the function to fit within the system but it might be a little less clean. Maybe making the categories into options would work? What do you all think?
Ah yes, I forgot about this issue. I was thinking we could make a couple of wrapper functions. In fact, I decided it would be easier to just write them than to try to explain them; see the commit I just pushed to this branch. This is the simplest solution, although there are definitely other ways we could handle this (with an array of categories, for instance). Try those in a tableau and see if they work ok.
On Wed, Jan 20, 2021 at 2:37 PM 3ddiehead notifications@github.com wrote:
It appears that putting in two arguments for strongStart_Hsu is going to be a problem with the JSON string to function-call. For example, when calling the function for the creation of a tableau, 'strongStart_Hsu-w-i' does not work. The system indicates that the "i" was unexpected. I could change the function to fit within the system but it might be a little less clean. Maybe making the categories into options would work? What do you all think?
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/syntax-prosody-ot/main/pull/546#issuecomment-763998379, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC7ROKW4JOXA4O3WISJHSWDS25LK3ANCNFSM4WH4NNKQ .
--
-- Jennifer Bellik, PhD Post-doctoral researcher & lecturer UC Santa Cruz https://people.ucsc.edu/~jbellik/
Those changes work perfectly, here is what I propose for the categories available to k and p: k = syllable, foot, word, phi p = phi, iota (maybe u, but what is it?) Let me know if this sounds good.
We don't actually need 'u'. It was a mistake to include it in the prosodic hierarchy since Junko & Armin have proposed a PH that doesn't have "utterance" in it at all. So for the interface, that selection of values for k looks good, and then we can have distinct entries for p = phi, p = i. These should all go under the "show more" section in Sisterhood.
On Wed, Jan 20, 2021 at 5:10 PM 3ddiehead notifications@github.com wrote:
Those changes work perfectly, here is what I propose for the categories available to k and p: k = syllable, foot, word, phi p = phi, iota (maybe u, but what is it?) Let me know if this sounds good.
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/syntax-prosody-ot/main/pull/546#issuecomment-764144751, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC7ROKUE5TCIM4XJ5DRQ4ADS255G5ANCNFSM4WH4NNKQ .
--
-- Jennifer Bellik, PhD Post-doctoral researcher & lecturer UC Santa Cruz https://people.ucsc.edu/~jbellik/
Alright, added to the interface. Also, there was a minor error in the code for StrongStart (any sister) where the value of a checkbox was "syll" while text read as ι. I changed the readout to σ instead because I assume that's what it was supposed to be.
Oh, thanks for correcting that. Whoops.
On Thu, Jan 21, 2021 at 1:30 PM 3ddiehead notifications@github.com wrote:
Alright, added to the interface. Also, there was a minor error in the code for StrongStart (any sister) where the value of a checkbox was "syll" while text read as ι. I changed the readout to σ instead because I assume that's what it was supposed to be.
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/syntax-prosody-ot/main/pull/546#issuecomment-764953331, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC7ROKSQUWC64GP4HDN6JZTS3CMFXANCNFSM4WH4NNKQ .
--
-- Jennifer Bellik, PhD Post-doctoral researcher & lecturer UC Santa Cruz https://people.ucsc.edu/~jbellik/
Fully functional with automated test document