Skip to content

[ISSUE 762] Publishing machine learning models #764

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

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rimjhimsudhesh
Copy link
Contributor

Closes #

Description

Checklist (Mandatory for new features)

  • Added Documentation
  • Added Unit Tests

Testing (Mandatory for all changes)

  • GPU Test: test-medium-connected.xml Passed
  • GPU Test: test-large-long.xml Passed

@rimjhimsudhesh rimjhimsudhesh requested a review from stiber January 16, 2025 07:36
Copy link
Contributor

@stiber stiber left a comment

Choose a reason for hiding this comment

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

There are no comments in the Jupyter notebook, nor any other information (like a README) to indicate what this file is about. Moreover, changes should be submitted as PRs to merge into SharedDevelopment, not master.

However, I don't think that this is something that should be added to the repo. Instead, submit this as part of your report and we will add it to Zotero under lab internal documents. Include a text description that can be added to the Zotero metadata. This PR can then be closed without being merged.

@stiber stiber marked this pull request as draft June 18, 2025 18:19
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