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

[out3]: Fix bugin AIMEC plot #1043

Merged
merged 2 commits into from
Nov 13, 2024
Merged

[out3]: Fix bugin AIMEC plot #1043

merged 2 commits into from
Nov 13, 2024

Conversation

PaulaSp3
Copy link
Contributor

@PaulaSp3 PaulaSp3 commented Nov 13, 2024

change labels in AIMEC "thalweg altitude" plot: delta s and delta z

@PaulaSp3 PaulaSp3 requested review from fso42 and awirb November 13, 2024 08:50
@pep8speaks
Copy link

pep8speaks commented Nov 13, 2024

Hello @PaulaSp3! Thanks for updating this PR.

Line 1705:49: W292 no newline at end of file
Line 1698:9: E231 missing whitespace after ','
Line 1685:44: E241 multiple spaces after ','
Line 1684:39: E241 multiple spaces after ','
Line 1683:39: E241 multiple spaces after ','
Line 1638:78: E127 continuation line over-indented for visual indent
Line 1627:34: E231 missing whitespace after ':'
Line 1606:52: E231 missing whitespace after ','
Line 1579:121: E501 line too long (145 > 120 characters)
Line 1566:52: E231 missing whitespace after ','
Line 1550:111: E203 whitespace before ','
Line 1513:121: E501 line too long (149 > 120 characters)
Line 1513:22: E127 continuation line over-indented for visual indent
Line 1512:121: E501 line too long (126 > 120 characters)
Line 1421:1: E303 too many blank lines (3)
Line 1375:121: E501 line too long (133 > 120 characters)
Line 1375:23: E127 continuation line over-indented for visual indent
Line 1374:38: E225 missing whitespace around operator
Line 1364:20: E251 unexpected spaces around keyword / parameter equals
Line 1312:45: E222 multiple spaces after operator
Line 1258:50: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
Line 1069:121: E501 line too long (121 > 120 characters)
Line 1069:41: E225 missing whitespace around operator
Line 1046:17: E265 block comment should start with '# '
Line 1046:17: E116 unexpected indentation (comment)
Line 1040:9: E265 block comment should start with '# '
Line 982:13: E117 over-indented
Line 971:17: E117 over-indented
Line 970:12: E111 indentation is not a multiple of four
Line 878:67: E127 continuation line over-indented for visual indent
Line 739:5: E265 block comment should start with '# '
Line 724:74: E225 missing whitespace around operator
Line 718:121: E501 line too long (121 > 120 characters)
Line 697:121: E501 line too long (121 > 120 characters)
Line 695:9: E265 block comment should start with '# '
Line 688:121: E501 line too long (124 > 120 characters)
Line 674:121: E501 line too long (123 > 120 characters)
Line 666:121: E501 line too long (138 > 120 characters)
Line 413:49: E231 missing whitespace after ','
Line 413:47: E231 missing whitespace after ','
Line 385:53: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
Line 369:23: E127 continuation line over-indented for visual indent
Line 360:121: E501 line too long (124 > 120 characters)
Line 352:121: E501 line too long (122 > 120 characters)
Line 337:17: E265 block comment should start with '# '
Line 337:17: E116 unexpected indentation (comment)

Comment last updated at 2024-11-13 14:38:21 UTC

@fso42 fso42 added the bug Something isn't working label Nov 13, 2024
@fso42 fso42 added this to the Version 1.10 milestone Nov 13, 2024
@PaulaSp3
Copy link
Contributor Author

PaulaSp3 commented Nov 13, 2024

Before:
avaArzlerUni_releaseFictive_1504379142_C_M_null_dfa_thalwegAltitude

Now: (the labels delta s and delta z are right now)

avaArzlerUni_releaseFictive_1504379142_C_M_null_dfa_thalwegAltitude

@fso42 fso42 self-assigned this Nov 13, 2024
Copy link

codeclimate bot commented Nov 13, 2024

Code Climate has analyzed commit 76a9db2 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 69.6% (0.0% change).

View more on Code Climate.

@fso42 fso42 merged commit 78b0822 into master Nov 13, 2024
4 checks passed
@fso42 fso42 deleted the PS_aimecPlotAxes branch November 13, 2024 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants