-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat: add matlab enabled Docker image #222
Conversation
65bf644
to
157f485
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Just a minor thing about apt-get ordering
Test repo here: I'm having trouble getting it to work as I did before... the logs show something like this:
Repo: https://renkulab.io/projects/lee.gavin.k/test-matlab-docker/ (log into MATLAB with your ETH-provided license, however this might be the issue according to https://github.com/mathworks/jupyter-matlab-proxy/blob/main/MATLAB-Licensing-Info.md). I tried a previous matlab build (it is a slightly different matlab version) here: https://renkulab.io/gitlab/learn-renku/working-with-matlab and it works fine. It could also be the security settings that changed with regards to iframes recently? |
I just tried with the VNC (without the jupyter-matlab-proxy) and it seems to play ball - I think the new security settings + the ways the ports are set up in the jupyter-matlab-proxy make it unhappy. I will reformulate this. |
What do you mean by "new security settings"? |
I just tried the test-matlab-docker and it seems to be ok? Or maybe I'm not looking at the right things? |
New security settings: the ones renku-ui put in to disallow some particular js/browser-based features like interactive plots. Yes I believe I've fixed it, so the latest test-matlab-docker should work. |
yeah but those settings just disallow rendering of JS / iframes in notebook previews. It shouldn't have any effect on the session window afaik, if it does it's a bug. |
Yeah it probably was a bug on matlab's side. Anyway I think this one is alright now - you would access it through the vnc directly. |
So I just tested the latest image and it definitely seems to work - but in the previous versions you had matlab opening up directly - is that no longer needed? My thinking was just if you could open it directly via a proxy then you actually don't need all the vnc stuff. But if this is how it's intended/useful it's also fine. |
also - this PR definitely deserves a description! 😆 |
Yeah, so in the previous version it was using a slightly older matlab docker base, and the vnc-proxy worked. However when I updated it it started bugging and shut down after I entered my license details. So I opted to use our renku-vnc and have users open it from the vnc. |
ok all good - I'll approve then. But please do add a description for posterity ;) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gavin-k-lee I imagine this will make lots of people happy!
@rokroskar sorry for the commit spam - I was trying to get the desktop icon to play nicely but it seems like it would be another rabbit hole to go down if I was to find out the root problem (I think it's down to environment variables). Basically now when you click on the desktop icon you get taken to the /opt/VNC folder but even when I passed in the starting directory, matlab would complain and open it there. I don't think it's a huge issue and we can instruct them to navigate to their project git repo in the container. |
Sounds good - we can try to fix that in a future iteration, I think it's already useful as-is |
After much demand from the community and some prototypes (see here) we should add a Matlab-enabled Renku docker image. This simplifies down the Docker images and we can then make templates from this image.