PreTeXtBook / pretext

PreTeXt: an authoring and publishing system for scholarly documents
https://pretextbook.org
Other
254 stars 203 forks source link

Sage cells inside knowls do not fully render #190

Closed rbeezer closed 4 years ago

rbeezer commented 8 years ago

As the title says, an instance of a Sage Cell revealed by opening a knowl does not render properly.

  1. When the knowl content is embedded on the page, the cell shows a box, but it is not loaded with text and not functional.
  2. When the knowl is a separate file, the knowl does not display.
  3. In the prototype version of FCLA, knowls from files with Sage cells work fine.

The first two behaviors can be seen in the current "beta" online version of FCLA: http://linear.ups.edu/beta/fcla.html

Third behavior is in original: http://linear.ups.edu/html/fcla.html

kcrisman commented 7 years ago

I think this is a pretty bad bug, because it means that any Sage cells inside of examples (which are rendered as knowls!!!) don't function. Just ran into this now. But, at least for me, examples (and proofs and ...) are "main text" and shouldn't be treated differently. This will make large portions of my text unusable 😦

rbeezer commented 7 years ago

I have the same problem with FCLA, perhaps #286 will fix this.

Until then, just turn off knowling examples and you can see the whole example and Sage should behave.

On 06/02/2016 09:56 AM, kcrisman wrote:

I think this is a pretty bad bug, because it means that any Sage cells inside of /examples/ (which are rendered as knowls!!!) don't function. Just ran into this now. But, at least for me, examples (and proofs and ...) are "main text" and shouldn't be treated differently. This will make large portions of my text unusable 😦

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/rbeezer/mathbook/issues/190#issuecomment-223353904, or mute the thread https://github.com/notifications/unsubscribe/ABy2chu7207JO2J5eyexFQM-XkzD-jNUks5qHwtAgaJpZM4G8p3e.

kcrisman commented 7 years ago

Until then, just turn off knowling examples and you can see the whole example and Sage should behave.

And I assume that I can read exactly how to do that in the sample article ... ?

rbeezer commented 7 years ago

There are now two simple examples of this in the sample article, viewable online. One right before Example 4.2, and one within the knowl for Example 4.2.

http://mathbook.pugetsound.edu/examples/sample-article/html/section-4.html

rbeezer commented 7 years ago

On 06/03/2016 07:22 AM, kcrisman wrote:

Until then, just turn off knowling examples and you can see the whole
example and Sage should behave.

And I assume that I can read exactly how to do that in the sample article ... ?

You shouldn't assume anything about the documentation. In this case, the answer you want is in the real documentation:

http://mathbook.pugetsound.edu/doc/author-guide/html/processing-stringparam.html

kcrisman commented 7 years ago

You shouldn't assume anything about the documentation.

Haha!

In this case, the answer you want is in the real documentation:

Oh, the stringparams again. I hope I remember to do this when I need to start testing the Sage cells.

rbeezer commented 7 years ago

beta link above will not serve as an example right now. I'll post a dedicated version later today.

davidfarmer commented 7 years ago

3 comments earlier there is an example.

rbeezer commented 7 years ago

Right - forgot I made a simple one. That'll work.

kcrisman commented 7 years ago

Until then, just turn off knowling examples and you can see the whole example and Sage should behave.

Umm, no. Now it doesn't show up at all in html. This is with xsltproc --stringparam html.knowl.example 'no'

rbeezer commented 7 years ago

Are you viewing locally or are you putting files on a server and then viewing them?

I believe there are situations where former fails and latter behaves. And I think this is new behavior (relative to the comment you quote).

On 12/01/2016 09:06 AM, kcrisman wrote:

Until then, just turn off knowling examples and you can see the whole example
and Sage should behave.

Umm, no. Now it doesn't show up at all in html. This is with |xsltproc --stringparam html.knowl.example 'no'|

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/rbeezer/mathbook/issues/190#issuecomment-264231605, or mute the thread https://github.com/notifications/unsubscribe-auth/ABy2crrtnipNSBI61jw4Ld5vRmQ6GBrEks5rDv6VgaJpZM4G8p3e.

kcrisman commented 7 years ago

Locally.

kcrisman commented 7 years ago

I would like to suggest that this is the number 1 bug in MBX for my project's usability, as a I have a fair number of Sage cells in examples, which are knowlized. (And we see that the non-knowl version, which I prefer anyway, doesn't currently work.) I don't see any way I can do the JMM workshop without exposing this, and I'd have to work pretty hard to remember not to use those as examples.

rbeezer commented 7 years ago

Whatever problems there were with Sage cells in external file knowls (due to cross-references) should be fixed with a47215d7

Sage cells in embedded knowls are visible, but not functional. Nothing learned on the current fix seems to help. So there is more work to be done here.

rbeezer commented 4 years ago

I noticed the test example in the sample article now seems to be behaving. No idea what might have changed.

Example 4.2 at

https://pretextbook.org/examples/sample-article/html/interesting-corollary.html

has a simple Sage cell within. Now works for me with recent Firefox and Chrome. Any willing Safari testers? @kcrisman - are you able to re-knowl your example and test?

davidfarmer commented 4 years ago

The Sage cell in the knowled Example 4.2 works for me in Safari.

Can I complain about the seemingly random numbering in that section?

On Wed, 16 Oct 2019, Rob Beezer wrote:

I noticed the test example in the sample article now seems to be behaving. No idea what might have changed.

Example 4.2 at

https://pretextbook.org/examples/sample-article/html/interesting-corollary.html

has a simple Sage cell within. Now works for me with recent Firefox and Chrome. Any willing Safari testers? @kcrisman - are you able to re-knowl your example and test?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, orunsubscribe.[AABTULC3EMSMMWBOA3TYNA3QO5TKBA5CNFSM4BXSTXPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOOR PWSZGOEBNVA4Q.gif]

kcrisman commented 4 years ago

has a simple Sage cell within. Now works for me with recent Firefox and Chrome. Any willing Safari testers? @kcrisman https://github.com/kcrisman - are you able to re-knowl your example and test?

Really sorry, no time for this currently :( But I can confirm it works in Safari, which is something!

rbeezer commented 4 years ago

On 10/16/19 12:54 PM, David W. Farmer wrote:

Can I complain about the seemingly random numbering in that section?

Only if

(1) You are not confusing the numbers on solutions showing up somewhere seemingly odd,

and

(2) You put the complaint somewhere besides right here.

;-)

Rob

JeanSebastienTurcotte commented 4 years ago

Are cells problematic again in knowls? I swear knowled cells (multiple) behaved before, but now they have no memory of one another.

rbeezer commented 4 years ago

If I go here:

https://pretextbook.org/examples/sample-article/html/section-sage-cells.html

and put

a=6

in the first cell and run it, and then put

a

as the last line of the second cell, it reports 6.

Do I misunderstand the problem?

Rob

On 12/4/19 4:52 PM, JeanSebastienTurcotte wrote:

Are cells problematic again in knowls? I swear knowled cells (multiple) behaved before, but now they have no memory of one another.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/rbeezer/mathbook/issues/190?email_source=notifications&email_token=AAOLM4SDPD722VGNCCAYCQLQXBGDZA5CNFSM4BXSTXPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF7CPEQ#issuecomment-561915794, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOLM4VIX346ZRX3HWXNGULQXBGDZANCNFSM4BXSTXPA.

JeanSebastienTurcotte commented 4 years ago

It works fine outside knowls, used to work in knowl, but now doesn't anymore

JeanSebastienTurcotte commented 4 years ago

If I go here: https://pretextbook.org/examples/sample-article/html/section-sage-cells.html and put a=6 in the first cell and run it, and then put a as the last line of the second cell, it reports 6. Do I misunderstand the problem? Rob … On 12/4/19 4:52 PM, JeanSebastienTurcotte wrote: Are cells problematic again in knowls? I swear knowled cells (multiple) behaved before, but now they have no memory of one another. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#190?email_source=notifications&email_token=AAOLM4SDPD722VGNCCAYCQLQXBGDZA5CNFSM4BXSTXPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF7CPEQ#issuecomment-561915794>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAOLM4VIX346ZRX3HWXNGULQXBGDZANCNFSM4BXSTXPA.

Take a look at

http://alir.jsturcotte.profweb.ca/sec-prodscal.html

example "Calcul 1.2.12" .

If you execute both cells, everything is fine, but if I delete the first line of the second cell, Sage does not remember that vector u and v were defined in the first cell of that example.

It used to work fine and does so outside of knowls.

rbeezer commented 4 years ago

Sorry for the long delay. I needed to get a block of time to dig into this one. Yes, I think I understand why linking broke (within knowls, only). Discussion and work is on #981, so you'll want to watch that one.

rbeezer commented 4 years ago

I'm going to close this. Cells render and execute, which was at issue here. Linking is a new problem, and is addressed on #981.