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

Feature/rpc logger #660

Merged
merged 37 commits into from
Jan 3, 2023
Merged

Feature/rpc logger #660

merged 37 commits into from
Jan 3, 2023

Conversation

cbellot000
Copy link
Contributor

@cbellot000 cbellot000 commented Nov 30, 2022

2 new methods in Session:
handle_events_with_file_logger(self, file_path, verbosity_level=1):
"""Adds an event handler of type file_logger server side.
Events will then be caught and forwarded to the file stream.
Parameters
----------
file_path : str
verbosity_level : int
0, 1 or 2
"""
start_emitting_rpc_log(self):
"""Adds a signal emitter to the session. This emitter will catch all incoming rpc calls.
Adding a handler will enable the logging (
use :func:Session.handle_events_with_file_logger()).
"""

@codecov
Copy link

codecov bot commented Nov 30, 2022

Codecov Report

Merging #660 (3ae0e5c) into master (1bc0fea) will decrease coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #660      +/-   ##
==========================================
- Coverage   88.60%   88.49%   -0.11%     
==========================================
  Files          71       71              
  Lines        8081     8091      +10     
==========================================
  Hits         7160     7160              
- Misses        921      931      +10     

@PProfizi PProfizi added the enhancement New feature or request label Dec 13, 2022
…ture/rpc_logger

# Conflicts:
#	.github/workflows/ci.yml
#	requirements/requirements_dev.txt
#	src/ansys/dpf/core/session.py
@cbellot000 cbellot000 merged commit d55cc67 into master Jan 3, 2023
@cbellot000 cbellot000 deleted the feature/rpc_logger branch January 3, 2023 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants