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

[Wait for #4494] [tensor_trainer] remove ready_to_complete property #4496

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

[Wait for #4494] [tensor_trainer] remove ready_to_complete property #4496

wants to merge 2 commits into from

Conversation

songgot
Copy link
Contributor

@songgot songgot commented Jun 19, 2024 •
edited
Loading

  • ready_to_complete property is removed

Self evaluation:

  1. Build test: [*] Passed [ ]Failed [ ]Skipped
  2. Run test: [*] Passed [ ]Failed [ ]Skipped

@songgot
[tensor_trainer] Add a function that generates dummy data
2e3e5c9 
When the pipeline stops and no data is input, the necessary data
for the current epoch is generated for the normal termination of nntrainer.
The data generation callback of nntrainer is no longer called
until all the necessary data for the current epoch are received.

Signed-off-by: hyunil park <hyunil46.park@samsung.com>
@songgot
[tensor_trainer] remove ready_to_complete property
1adc06d 
- ready_to_complete property is removed

Signed-off-by: hyunil park <hyunil46.park@samsung.com>
@songgot songgot changed the title [tensor_trainer] remove ready_to_complete property [Wait for #4494] [tensor_trainer] remove ready_to_complete property Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@myungjoo myungjoo Awaiting requested review from myungjoo myungjoo is a code owner

@jijoongmoon jijoongmoon Awaiting requested review from jijoongmoon jijoongmoon is a code owner

@again4you again4you Awaiting requested review from again4you again4you is a code owner

@jaeyun-jung jaeyun-jung Awaiting requested review from jaeyun-jung jaeyun-jung is a code owner

@wooksong wooksong Awaiting requested review from wooksong wooksong is a code owner

@gichan-jang gichan-jang Awaiting requested review from gichan-jang gichan-jang is a code owner

@anyj0527 anyj0527 Awaiting requested review from anyj0527 anyj0527 is a code owner

@jihochu jihochu Awaiting requested review from jihochu jihochu is a code owner

@niley7464 niley7464 Awaiting requested review from niley7464 niley7464 is a code owner

@yeonykim2 yeonykim2 Awaiting requested review from yeonykim2 yeonykim2 is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@songgot