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.

Sign up for GitHub

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

Jump to bottom

Log sequence ad usecase #37

Merged
merged 14 commits into from
Feb 24, 2023
Merged

Log sequence ad usecase #37

merged 14 commits into from
Feb 24, 2023

Conversation

tzemicheal
Copy link
Contributor

No description provided.

@tzemicheal
log sequence added
d7bf8ea
@tzemicheal
copyright/style check
8b9df8e
@tzemicheal
copyright/style check
6b8bfde 
	new file:   requirements.txt

	modified:   README.md
	new file:   requirements.txt
	modified:   training-tuning-inference/datatools.py
	modified:   training-tuning-inference/model.py
	modified:   training-tuning-inference/utils.py
@tzemicheal
Merge branch 'log-sequence-ad' of github.com:tzemicheal/morpheus-expe…
6218ad1 
…rimental into log-sequence-ad
@tzemicheal tzemicheal added feature request New feature or request non-breaking Non-breaking change labels Feb 16, 2023
@tzemicheal tzemicheal self-assigned this Feb 17, 2023
@tzemicheal tzemicheal marked this pull request as ready for review February 17, 2023 15:06
@tzemicheal tzemicheal requested a review from a team as a code owner February 17, 2023 15:06
@raykallen raykallen requested a review from gbatmaz February 21, 2023 14:31
gbatmaz
gbatmaz reviewed Feb 21, 2023
View reviewed changes
Copy link
Contributor

@gbatmaz gbatmaz left a comment

Choose a reason for hiding this comment

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

I added some comments for Tad's review. Thanks.

log-sequence-ad/training-tuning-inference/log-sequence-ad.ipynb Outdated Show resolved Hide resolved
log-sequence-ad/training-tuning-inference/log-sequence-ad.ipynb Show resolved Hide resolved
log-sequence-ad/training-tuning-inference/log-sequence-ad.ipynb Show resolved Hide resolved
log-sequence-ad/training-tuning-inference/log-sequence-ad.ipynb Show resolved Hide resolved
log-sequence-ad/training-tuning-inference/log-sequence-ad.ipynb Show resolved Hide resolved
log-sequence-ad/training-tuning-inference/log-sequence-ad.ipynb Outdated Show resolved Hide resolved
log-sequence-ad/training-tuning-inference/utils.py Show resolved Hide resolved
log-sequence-ad/training-tuning-inference/model.py Show resolved Hide resolved
log-sequence-ad/training-tuning-inference/log-sequence-ad.ipynb Outdated Show resolved Hide resolved
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

gbatmaz
gbatmaz approved these changes Feb 24, 2023
View reviewed changes
Copy link
Contributor

@gbatmaz gbatmaz left a comment

Choose a reason for hiding this comment

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

lgtm

raykallen
raykallen approved these changes Feb 24, 2023
View reviewed changes
Copy link
Contributor

@raykallen raykallen left a comment

Choose a reason for hiding this comment

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

lgtm

@raykallen
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit e2608dd into nv-morpheus:branch-23.03 Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gbatmaz gbatmaz gbatmaz approved these changes

@raykallen raykallen raykallen approved these changes

Assignees

@tzemicheal tzemicheal

Labels
feature request New feature or request non-breaking Non-breaking change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tzemicheal @raykallen @gbatmaz