-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
symbols to export for essentials2023 context #468
Comments
Hmm, I really don't like the namespace inconsistency of removing the |
to be clear - nobody is suggesting removing the old aliases. I'm also happy to have a Bootstrap-specific context, but this opens up the "discover curated contexts UI" question |
Right: adding new aliases while not removing old ones, and ad-hoc names without namespace-prefixes, just makes the global namespace issue worse. Having short names for bootstrap is a worthwhile goal. Polluting the global namespace is an anti-goal... |
@blerner OK, so you're ok with a curated dropdown of contexts in the "Choose Context" menu? Note that this is starting to replicate DrRacket's "Add teachpack" UI... |
I'm more ok with that, yes. I'm also mulling the idea of allowing supplying the context as a URL parameter, so that teachers can create a short-URL that drops students right into the correct context. |
Seems like I no longer need to weigh in? (-: |
I'm fine with adding the listed names to the next I also really like the idea of the dropdown/URL infrastructure for contexts. But the names suggested here (as motivated by Bootstrap) are a reasonable proxy for “stuff first-time programmers could probably use”. Putting it in There is one other interface suggestion worth making here. @schanzer points out that one reason to want them in |
I would love a "what kind of project are you starting?" UI. That puts this all front-and-center. |
how about num-expt and expt? is there a place for |
@retabak good call on num-expt. added
|
@schanzer are pi/e part of this? Can easily add to |
@jpolitz yep! the corresponding issue in pyret-lang can probably be closed once you add these |
@jpolitz just added the basic trig functions to the list. Hopefully these are easy to add. |
@schanzer a request that may seem a little silly but would help me a lot – could you write a short check/examples block that uses all the identifiers you want included on trivial inputs? That will be the spec for what ought to work. Something like:
and so on. This would help me rule out typos and also be a useful agreement on what this is accomplishing. Share link to CPO or just paste it here. |
I was surprised to find that |
@jpolitz not sure if this is related to the constants context, but it would be pretty great to have these symbols available by default as well |
(Think I have this, just confirming) should PI and E be in this file, too? That is, I'm going to include this as part of the spec, too:
|
Yes - and IIRC, Shriram wanted both the upper- and lower-case versions of both. |
Is there any reason you needed to |
Right, but The other spec is that all the imports + definitions should go away and the check block should still succeed if the context is |
And for the default namespace we don't want Both of your comments help clarify that my goal is the |
This was completed some time ago |
The text was updated successfully, but these errors were encountered: