Skip to content
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

Update config-available for clpBNR v0.12.0 #180

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ridgeworks
Copy link

Add clpBNR_toolkit and clpBNR_search to preload on SWISH (now req'd for sandboxing). This should be done as part of any SWISH build incorporating the clpBNR v0.12.0.

clpBNR v0.12.0 includes:
a) new module clpBNR_search which is a version of ECLIPSe search libraries(e.g., fd_search) for clpBNR on SWI-Prolog.
b) additional predicates in module clpBNR_toolkit supporting numerical integration.

Both modules require the addition of sandboxing rules which, I think, dictates that they must be pre-loaded on SWISH. (clpBNR_toolkit could be used previously without preloading because no sandbox predicates were required.)

I've tested the above on a local SWISH server but can retest on a non-production server is desired.

A "process" question: The version of clpBNR (v0.11.7b) currently running on SWISH is several releases old. Is there something I need to do on producing a new version of clpBNR to trigger its inclusion in the next SWISH build? Or can something be done to the SWISH "make" scripts, e.g., pack_upgrade(clpBNR) so this happens automatically? If so will this also include the appropriate copy of the clpBNR quickstart notebook from the Git repo to examples?

Off topic question: Getting the version of available packs on SWISH is a bit problematical because pack-info (and related) are sandboxed on SWISH. I've added clpBNR version to the version info (using the documented hook) but maybe something from the pack manager could be white-listed so this wouldn't be necessary.

Add clpBNR_toolkit and clpBNR to preload on SWISH (now req'd for sanboxing).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant