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

Update master to Pastas 1.1.0 #595

Merged
merged 62 commits into from
May 12, 2023
Merged

Update master to Pastas 1.1.0 #595

merged 62 commits into from
May 12, 2023

Conversation

raoulcollenteur
Copy link
Member

@raoulcollenteur raoulcollenteur commented May 11, 2023

Short Description

Add a short description describing the pull request (PR) here.

martinvonk and others added 30 commits February 15, 2023 11:20
- temporary(?) fix for tmin/tmax
- ax.set_xlim("2010", "2015") not working with matplotlib 3.6.1 and pandas 1.5.3
make table optional
commit a37e95c
Merge: 9a3d810 10522d3
Author: Davíd Brakenhoff <d.brakenhoff@artesia-water.nl>
Date:   Mon Feb 20 11:13:50 2023 +0100

    Fix datetime axes with pandas.Timestamp and some other improvements (#557)

commit 10522d3
Author: dbrakenhoff <d.brakenhoff@artesia-water.nl>
Date:   Fri Feb 17 14:26:33 2023 +0100

    add tmin/tmax fix to ps.plots.series()
    revert nb change

commit 51f8282
Author: Martin Vonk <vonk.mart@gmail.com>
Date:   Fri Feb 17 13:24:53 2023 +0100

    fix notebooks pd Timestamp errors

commit 4cc49df
Author: dbrakenhoff <d.brakenhoff@artesia-water.nl>
Date:   Fri Feb 17 12:14:42 2023 +0100

    isort

commit 25eeb8b
Author: dbrakenhoff <d.brakenhoff@artesia-water.nl>
Date:   Fri Feb 17 12:14:28 2023 +0100

    codacy fixes for timer.py

commit 205faa9
Author: dbrakenhoff <d.brakenhoff@artesia-water.nl>
Date:   Fri Feb 17 12:13:57 2023 +0100

    support linear trends in modelcompare

commit 3eac98a
Author: dbrakenhoff <d.brakenhoff@artesia-water.nl>
Date:   Fri Feb 17 12:13:28 2023 +0100

    fix tests
    - temporary(?) fix for tmin/tmax
    - ax.set_xlim("2010", "2015") not working with matplotlib 3.6.1 and pandas 1.5.3
…tion-for-not-showing-statistics-table-in-pastasplotsseries
fix if statement
fix overwrite import
Co-authored-by: Martin Vonk <vonk.mart@gmail.com>
black allows these errors
@raoulcollenteur raoulcollenteur added this to the 1.1 milestone May 11, 2023
@codacy-production
Copy link

codacy-production bot commented May 11, 2023

Coverage summary from Codacy

Merging #595 (a7b11cc) into master (7302511) - See PR on Codacy

Coverage variation Diff coverage
+0.30% (target: +0.00%) 60.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (7302511) 4970 3595 72.33%
Head commit (a7b11cc) 4955 (-15) 3599 (+4) 72.63% (+0.30%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#595) 235 141 60.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@martinvonk
Copy link
Collaborator

I fixed the CI NB fail on #578. So if we merge that, this PR should be fine as well.

dbrakenhoff and others added 2 commits May 11, 2023 14:46
Co-authored-by: Martin Vonk <vonk.mart@gmail.com>
Co-authored-by: Martin Vonk <66305055+martinvonk@users.noreply.github.com>
pastas/modelcompare.py Outdated Show resolved Hide resolved
Copy link
Member

@dbrakenhoff dbrakenhoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made some minor changes in a dbrakenhoff-patch-1.

After merging those, good to go as far as I'm concerned.

@raoulcollenteur raoulcollenteur merged commit 5424a28 into master May 12, 2023
18 checks passed
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

4 participants