Skip to content

DM-13112 LSST workspace #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

Merged
merged 2 commits into from
Jul 25, 2019
Merged

DM-13112 LSST workspace #15

merged 2 commits into from
Jul 25, 2019

Conversation

tgoldina
Copy link
Contributor

https://jira.lsstcorp.org/browse/DM-13112
Use https://lsst-lsp-int.ncsa.illinois.edu/portal/app to test

Users should be able to save images and tables into their workspace and read saved tables/images back to Firefly. Use image/table "Save" with "workspace" selection to save files into workspace. "External Images" and "External Catalogs" allow to load from workspace.

LSST workspace does not allow to retrieve the whole file tree. One level is retrieved at a time. Click on folder to retrieve its content.

To save file in a new subdirectory, use "Add subfolder" link, enter folder name, and click enter.

@tgoldina tgoldina requested a review from loitly July 17, 2019 21:55
@tgoldina tgoldina self-assigned this Jul 17, 2019
@tgoldina tgoldina force-pushed the dm-13112_lsstworkspace branch from 1565e9e to 5b9bc03 Compare July 18, 2019 00:04
Copy link
Contributor

@loitly loitly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Tested on -int. I am able to save and load from workspace.

@@ -32,6 +32,11 @@ sso.logout.url = "/oauth2/sign_in"
sso.user.profile.url =
__$help.base.url = "onlinehelp/"

workspace.host.url = "https://lsst-lsp-int.ncsa.illinois.edu"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should -stable be added somewhere so that it will work when promoted into -stable?

@tgoldina tgoldina merged commit 14b40b6 into master Jul 25, 2019
@gpdf gpdf deleted the dm-13112_lsstworkspace branch June 23, 2021 01:36
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.

2 participants