Skip to content
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

0.5.0 rc rebase #321

Merged
merged 12 commits into from
Jan 26, 2024
Merged

0.5.0 rc rebase #321

merged 12 commits into from
Jan 26, 2024

Conversation

NickGeneva
Copy link
Collaborator

Modulus Pull Request

Description

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.
  • The CHANGELOG.md is up to date with these changes.
  • An issue is linked to this pull request.

Dependencies

tge25 and others added 12 commits January 25, 2024 17:43
* Update dataset.py

* Update formatting

---------

Co-authored-by: Mohammad Amin Nabian <m.a.nabiyan@gmail.com>
Co-authored-by: Mohammad Amin Nabian <mnabian@nvidia.com>
* fix small bugs

* formatting

* add documentation for task
* Refactor saving results to improve perf and overlap with inferencing

* Update blossom-ci.yml (NVIDIA#295)

* Change pip install commands with the correct PyPI package name (NVIDIA#298)

* Make number of writer workers parameterized

* add comment for writer workers

* revert readme and blossom-ci

* 2nd revert of readme

* fix black formatting

---------

Co-authored-by: David Pruitt <dpruitt@login-eos01.eos.clusters.nvidia.com>
Co-authored-by: Kaustubh Tangsali <71059996+ktangsali@users.noreply.github.com>
Co-authored-by: Abdullah <37012364+Saydemr@users.noreply.github.com>
…IA#308)

* Update blossom-ci.yml (NVIDIA#295)

* Change pip install commands with the correct PyPI package name (NVIDIA#298)

* update configs

* remove mixture loss

* remove mixture loss from init

---------

Co-authored-by: Kaustubh Tangsali <71059996+ktangsali@users.noreply.github.com>
Co-authored-by: Abdullah <37012364+Saydemr@users.noreply.github.com>
* Update blossom-ci.yml (NVIDIA#295)

* Change pip install commands with the correct PyPI package name (NVIDIA#298)

* reorganize

* remove from init

---------

Co-authored-by: Kaustubh Tangsali <71059996+ktangsali@users.noreply.github.com>
Co-authored-by: Abdullah <37012364+Saydemr@users.noreply.github.com>
… inference (NVIDIA#306)

* Refactor saving results to improve perf and overlap with inferencing

* Update blossom-ci.yml (NVIDIA#295)

* Change pip install commands with the correct PyPI package name (NVIDIA#298)

* Make number of writer workers parameterized

* add comment for writer workers

* revert readme and blossom-ci

* 2nd revert of readme

* Adding NVTX markers and removing redundant regression model inference

* Code formatting

* Adding nvtx to pyproject.toml

* Updating CHANGELOG.md

---------

Co-authored-by: David Pruitt <dpruitt@login-eos01.eos.clusters.nvidia.com>
Co-authored-by: Kaustubh Tangsali <71059996+ktangsali@users.noreply.github.com>
Co-authored-by: Abdullah <37012364+Saydemr@users.noreply.github.com>
Co-authored-by: David Pruitt <dpruitt@nvidia.com>
Co-authored-by: David Pruitt <daviddpruitt@gmail.com>
Co-authored-by: Akshay Subramaniam <asubramaniam@login-eos01.eos.clusters.nvidia.com>
* Update blossom-ci.yml (NVIDIA#295)

* Change pip install commands with the correct PyPI package name (NVIDIA#298)

* add wb logging

* formatting

* make mode configurable

---------

Co-authored-by: Kaustubh Tangsali <71059996+ktangsali@users.noreply.github.com>
Co-authored-by: Abdullah <37012364+Saydemr@users.noreply.github.com>
@NickGeneva NickGeneva added the ! - Release PRs or Issues releating to a release label Jan 26, 2024
@NickGeneva NickGeneva self-assigned this Jan 26, 2024
@NickGeneva
Copy link
Collaborator Author

/blossom-ci

@ktangsali ktangsali self-requested a review January 26, 2024 01:52
@NickGeneva NickGeneva merged commit 318e96b into NVIDIA:main Jan 26, 2024
1 check passed
@ktangsali ktangsali mentioned this pull request Apr 2, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
! - Release PRs or Issues releating to a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants