-
-
Notifications
You must be signed in to change notification settings - Fork 328
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
sagemaker quality of life improvements #245
base: master
Are you sure you want to change the base?
Conversation
@absynthe started a new PR with the quality of life improvements. BTW, do you maybe know how to detect whether we're in a sagemaker environment? |
so something similar to
|
@absynthe I've been toying around with sagemaker studio today, but so far haven't been able to get the dash to run, do you have a guide or HOWTO? |
Hey @oegedijk, I created this working example for you: https://github.com/absynthe/explainer-dashboard-studio-working-example/blob/main/workingExplainerDashboardStudioExample.ipynb . Try to walk through it and let me know if you have any questions. |
There is nothing in the kernel giving SageMaker studio away. However, you can look for the /opt/ml/metadata/resource-metadata.json file in the file system. |
Hi again @oegedijk , I added a test for kwargs: absynthe/explainer-dashboard-studio-working-example@7066886 . This does not seem to work, I assume something is missing for the cli glue to work? We cannot run the dashboard directly in the notebookn using the run command. This is due to the separation between compute and storage of SageMaker Studio as mentioned here: https://aws.amazon.com/blogs/machine-learning/dive-deep-into-amazon-sagemaker-studio-notebook-architecture/ . This is why there is the workaround of exporting the dashboard config to disk and loading again in the terminal window of the JupyterServer App. |
We'd very much like to start using explainer dashboard on SageMaker. Is there anything I can do to help push this forward? |
No description provided.