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

No way for users to define LICENSE_AMPL #19

Open
yoctozepto opened this issue Jul 22, 2024 · 3 comments
Open

No way for users to define LICENSE_AMPL #19

yoctozepto opened this issue Jul 22, 2024 · 3 comments
Assignees

Comments

@yoctozepto
Copy link
Member

In-test variants had hardcoded LICENSE_AMPL value but the end users are not supposed to hack on the scripts. This should be made easily settable.

@yoctozepto
Copy link
Member Author

I assigned @rudi as the owner of this issue but please discuss and reassign appropriately to get this issue fixed.

@rudi
Copy link
Member

rudi commented Jul 22, 2024

The optimiser-controller creates the cluster with all environment variables specified by the user in the GUI, and the solver start script picks up the value of AMPL_LICENSE (see https://github.com/eu-nebulous/optimiser-solver/blob/3b39ea002fc2b580dbd0eb217c695b47b9fc7677/start-solver.sh#L11).

So it should all work already, but maybe the end user documentation does not tell the user to set AMPL_LICENSE in the GUI?

@yoctozepto
Copy link
Member Author

Possibly, Rudi. @robert-sanfeliu should be most aware.

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

No branches or pull requests

2 participants