-
Notifications
You must be signed in to change notification settings - Fork 3
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
[ENH] Updating resen-base and resen-core #61
Conversation
… latest jupyter-stacks.
@@ -1,4 +1,5 @@ | |||
FROM earthcubeingeo/resen-base:2020.1.0 | |||
#FROM earthcubeingeo/resen-base:2020.1.0 | |||
FROM resenbasetest:latest |
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.
This needs to be changed to resen-base:2020.2.0 once we are happy with resen-base!
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.
Checked all files. Only one comment.
There's a small issue with the terminal in jupyterlab not starting up with bash. Should be able to be solved with a jupyterlab_config.py file. |
I'm having issues building the resen-core image - looks like jupyterlab is failing.
This issue persists if I try running the commands from within the docker container.
Both these commands produce the error
End of log file:
edit by @asreimer: here's the full log file: |
I cannot reproduce on my fedora VM (has 9GB of RAM available). I think the issue might be related to memory usage during minification: facebook/create-react-app#8320 (comment) (edit: Yes, 100% this must be our problem, see here: webpack-contrib/terser-webpack-plugin#202 (comment)) If you increase the amount of memory that the container is allowed to use while building, does this problem go away? |
I cannot reproduce the error either. My system is similar to @ljlamarche : MacBook Pro 16 GB RAM, and my docker is allowed to use up to 6 CPUs, 4 GB memory, 1GB swap, and disk image size is 60 GB (34 GB used) |
Increasing the CPUs, memory, and swap to @pmreyes2 configuration fixed the problem. Thanks! |
It's scary to think about the amount of system resources that we can
through at things nowadays...
…On 9/15/20 11:30 PM, Pablo M. Reyes wrote:
I cannot reproduce the error either. My system is similar to
@ljlamarche <https://github.com/ljlamarche> : MacBook Pro 16 GB RAM,
and my docker is allowed to use up to 6 CPUs, 4 GB memory, 1GB swap,
and disk image size is 60 GB (34 GB used)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#61 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAMC5ZVKD3UTCNODNDPTPUTSGBLRFANCNFSM4RGVDDVQ>.
|
Tried this out, everything looks good to me! We'll have to update some of the tutorials, but I can do that on a separate branch. |
We're running into this issue: timduly4/pyglow#128 |
I bumped the version of basemap and have fixed the bug with the jupyterlab terminal not using bash. I think we're good to push the new resen-base to dockerhub and merge this into develop. Agree? |
…that fixes an iri2016 issue for dates after year 2010.
I agree that we're ready to push the new resen-base to dockerhub. Do we want to include sciunit2 in this version of resen-core? (Can discuss this Thursday if anyone has strong opinions.) |
updating pip to 20.2.4 (2020-10-16)
After two very small changes: adding pyglow latest commit and recently released sciunit2 for python3 I agree to merge this branch into develop. And yes, resen-base should be pushed to dockerhub. |
In order for bokeh to work, we need to install the following jupyterlab extensions:
https://docs.bokeh.org/en/latest/docs/user_guide/jupyter.html |
@ljlamarche I just added widgets interaction for bokeh. The jupyterlab-manager was already installed for interactions with matplotlib, so just the bokeh interactions was added. I think we can merge this to develop branch to keep fine tuning there. |
Just tested the bokeh widget. I have no further concerns about merging this. |
Accomplishes all checklist items in #59
Testing
Build the resen-base image:
Bulid the resen-core image:
Create a container with resencoretest and test things (e.g.):