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

[logger] add log indentation support via logger.block context manager #230

Merged
merged 55 commits into from Mar 7, 2016

Conversation

sdrave
Copy link
Member

@sdrave sdrave commented Mar 3, 2016

@ftalbrecht, you may have a strong opinion on this ...

sdrave and others added 30 commits February 19, 2016 14:48
…eded

as GenericRBReconstructor makes the copy.
If a test is run from the pymortests source directory an import
of 'pickle' (as in the subprocess module) will import the test
module instead of the std lib module.
to prevent finalization of master process resources (e.g MPI state)
[algorithms] add estimate_image, estimate_image_hierarchical and use them
@sdrave sdrave added this to the 0.4 milestone Mar 3, 2016
@sdrave
Copy link
Member Author

sdrave commented Mar 4, 2016

@ftalbrecht, as discussed, I have moved the timestamps to the left. I have also resolved the 'change of timestamp width' issue.

I am wondering: shouldn't also the bold printing be disabled when colors are disabled?

@ftalbrecht
Copy link
Contributor

ftalbrecht commented Mar 5, 2016 via email

@sdrave
Copy link
Member Author

sdrave commented Mar 7, 2016

Bold font is now also disabled. I also moved the pipe characters to the left which looks a bit cleaner to my eye. If no one objects, I will merge this.

@ftalbrecht
Copy link
Contributor

ftalbrecht commented Mar 7, 2016 via email

sdrave added a commit that referenced this pull request Mar 7, 2016
[logger] add log indentation support via logger.block context manager
@sdrave sdrave merged commit dbe607e into master Mar 7, 2016
@sdrave sdrave deleted the logger_blocks branch May 17, 2016 10:36
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.

None yet

3 participants