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

Session errors are logged #64

Merged
merged 5 commits into from
Jul 4, 2019
Merged

Session errors are logged #64

merged 5 commits into from
Jul 4, 2019

Conversation

henhuy
Copy link
Contributor

@henhuy henhuy commented Jun 28, 2019

I need more information on session errors to debug my code.
Session errors are now logged in format:

Session error for app "appname":
Session-Key: current session key
Current session data:
session key: str-output of app-related UserSession

Example for stemp app:
Session error for app "stemp":
Session-Key: None
Current session data:
v3le6kj6hinj1cimdx72x80z4gyhm287: Scenarios: gas,oil,woodchip,pv_heatpump, Demand-Type: 0, Demand-ID: 303, District: {}
hlms88f1lyim1q47q3m6jj29fkxq9iir: Scenarios: , Demand-Type: 0, Demand-ID: 304, District: {}

See https://github.com/rl-institut/WAM_APP_stemp_mv/blob/87c6f4e20a75e2598d954c154bfda5a0eca1c47e/user_data.py#L142 for example of str-representation of UserSession

@nesnoj
Copy link
Member

nesnoj commented Jul 3, 2019

@henhuy I'd like to create a new WAM release, do you wanna merge this one before?

@nesnoj nesnoj mentioned this pull request Jul 3, 2019
@henhuy henhuy merged commit d6b9248 into dev Jul 4, 2019
@henhuy henhuy deleted the enhancement/session_error_log branch June 9, 2020 08:42
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.

2 participants