-
Notifications
You must be signed in to change notification settings - Fork 15
v4.3.1 release #540
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
base: master
Are you sure you want to change the base?
v4.3.1 release #540
Conversation
…A-DISP tile with TiTiler-MultiDim.ipynb
…ualizing_OPERA-DISP_tile_with_TiTiler-MultiDim.ipynb
…A-DISP_tile_with_TiTiler-MultiDim.ipynb
Visualizing OPERA-DISP tile
Direct access
Create EarthCare_access and_visualize.ipynb
add alpha earth rst
fix asset key
Delete Old NISAR Tutorial
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
So far I think this is looking pretty good, I don't think I see any new Hub documentation. I've noticed though that the leaftlet maps don't seem to be rendering in working_with_r/vector_data_visualization and working_with_r/visualizing_with_titiler-pgstac. I'm also not getting an image for the last cell in science/GEE/gee, but not sure if it's just me or not |
|
Looks like those maps were never working, so that should be a blocker on this. |
|
@hrodmn @HarshiniGirish we can an open a new ticket to fix the maps, my suspicion is a difference in how the notebook records the executed R code, unclear if it stores rendered html+js the same way as python. |
|
I filed a ticket for the failed maps here https://github.com/NASA-IMPACT/active-maap-sprint/issues/1326 |
|
The earthcare notebook fails for me with |
|
Direct_access should probably actually reference Pangeo, that's the default python image we want people to use. We'll look into the EarthCARE issue. |
|
Hahah that'd explain why I was struggling so much getting tmap to work when testing the vector data visualization notebook with Henry's fix. I'll run it in the Hub instead |
|
Yes, I meant adding to the Hub images, I just tested and added tmap to our maap-workspaces repo so it will be in our next release |
* fix: load Proj and Leaflet js and css for R notebooks * run titiler-pgstac notebook --------- Co-authored-by: smk0033 <smk0033@uah.edu>
4.3.1 Release Feedback Updates
|
Ran the docs locally and updates seem to have been made and render correctly now. Should be ready for another review! As an FYI, I'm out until January 12th. Feel free to make any other updates or merge if everything looks good |
|
I made a branch Gee-> You need to create a new project to access https://code.earthengine.google.com/?scriptPath=users%2Fmmacander%2Fveg_structure%3Asentinel_1%2Fseasonal_s1_tile_export_v2 Is there another way to make this information more accessible to users? It is unclear what code that should go in sys.path.append('/projects/code/icesat2_boreal/lib') Adding credentials to s3://maap-ops-workspace/{USER}/credentials is not secure since other users can see the credentials. I thought we had a plan to not allow users to see other users buckets, but I tested and that has not been implemented yet. I stopped checking if the notebook runs after this point but checked the code direct_access-> Do users need to be logging into their own AWS account to get the temporary credentials? Because the role that we default assign users for a 2i2c session doesn’t have access to assume other roles. We also don’t have a maap-data-reader role on our AWS account either. I think we need more documentation on how to do this, I am not sure if there are more docs we can just link to vector_data_visualization → works for me |
Some updates:
Note: this release should not have any MAAP Hub documentation