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

Reduce block processing logging #235

Closed
Tracked by #242
yousefmoazzam opened this issue Mar 5, 2024 · 2 comments
Closed
Tracked by #242

Reduce block processing logging #235

yousefmoazzam opened this issue Mar 5, 2024 · 2 comments
Labels
enhancement New feature or request logging

Comments

@yousefmoazzam
Copy link
Collaborator

Each iterations over a block within a section seems to print information about the method being run, which can produce a lot of output. The printed information feels like it should be reduced in some manner (maybe only one print per-method in each section, ie, don't print on every block?)
@yousefmoazzam yousefmoazzam added the enhancement New feature or request label Mar 5, 2024
@yousefmoazzam yousefmoazzam changed the title Reduce logging printed to terminal during pipeline execution Reduce block processing logging Mar 19, 2024
@yousefmoazzam yousefmoazzam mentioned this issue Mar 19, 2024
Open
23 tasks
@yousefmoazzam
Copy link
Collaborator Author

Taken from the release TODO:

Fix the logging when the number of prints is related to the number of blocks. Currently it is confusing as we print the same method N-times. I guess we should print for the last block only?

yousefmoazzam added a commit that referenced this issue Apr 26, 2024
@yousefmoazzam
Don't log each method processing each block
8997f67 
Fixes #235
@yousefmoazzam yousefmoazzam mentioned this issue Apr 26, 2024
Merged
2 tasks
yousefmoazzam added a commit that referenced this issue Apr 26, 2024
@yousefmoazzam
Don't log each method processing each block
0520011 
Fixes #235
yousefmoazzam added a commit that referenced this issue May 2, 2024
@yousefmoazzam
Don't log each method processing each block
95165ed 
Fixes #235
yousefmoazzam added a commit that referenced this issue May 3, 2024
@yousefmoazzam
Don't log each method processing each block
5eaba4a 
Fixes #235
@yousefmoazzam
Copy link
Collaborator Author

Completed in merge of #294

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request logging
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yousefmoazzam