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.

Sign up for GitHub

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

Jump to bottom

Fix some pytets warnings #147

Merged
merged 2 commits into from
Nov 9, 2023
Merged

Fix some pytets warnings #147

merged 2 commits into from
Nov 9, 2023

Conversation

GianmarcoCallegher
Copy link
Contributor

No description provided.

@GianmarcoCallegher GianmarcoCallegher self-assigned this Nov 7, 2023
@GianmarcoCallegher
Copy link
Contributor Author

GianmarcoCallegher commented Nov 9, 2023 •
edited
Loading

The UserWarning can be fixed in several ways (look here for more details) by they all involve installing a pip package or tkinter. I would not do that since they appear only in the tests. Maybe we can just ignore the warnings in the tests. What do you think, @jobrachem ?

jobrachem
jobrachem requested changes Nov 9, 2023
View reviewed changes
Copy link
Contributor

@jobrachem jobrachem left a comment

Choose a reason for hiding this comment

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

Makes sense! Just one issue, see the comment on test_goose.py

@jobrachem jobrachem self-requested a review November 9, 2023 23:01
@jobrachem jobrachem merged commit 1c369b1 into main Nov 9, 2023
@jobrachem jobrachem deleted the pytests_warnings branch November 9, 2023 23:03
GianmarcoCallegher added a commit that referenced this pull request Feb 9, 2024
@jobrachem @hriebl
@GianmarcoCallegher @wiep @Seb-Lorek
Try to update calc on init (#92)
df53f7d 
* correct sampling for sigma_sq

* try to update calc on init

Update nodes.py

* fix test

Due to the exception being caught and re-raised in the update() method, we do not get the actual error text here.

* Add Goose-based initialization strategies / jittering (#72)

Co-authored-by: Hannes Riebl <hriebl@uni-goettingen.de>

* Cap BlackJAX version (#95)

* Fix PyMC tutorial (#96)

Co-authored-by: wiep <wiep@users.noreply.github.com>

* Fix mypy errors (#97)

* Improve import name

* Fix title in plot_scatter(), closes #98

* Adapt to BlackJAX 1.0.0 (#100)

Co-authored-by: Hannes Riebl <hriebl@uni-goettingen.de>

* increase timeout

* Update metadata v0.2.5

* update metadata 0.2.6-dev

* Fix Mypy errors (#102)

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>

* More Efficient MVN Degenerate (#101)

* Efficient MVN Degenerate

* Fixed blackjax refactor error

* Refactor

* indent more code

* Update CHANGELOG.md

---------

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>
Co-authored-by: Johannes Brachem <37882800+jobrachem@users.noreply.github.com>

* Fix #103 (#109)

* Update summary_m.py

* Update CHANGELOG.md

* Update deprecation message

* update deprecation messages

* Rename Param and Obs (#130)

* change names

* update usage in tests

* update usage in README

* Update CHANGELOG.md

* Make error_df public (closes #94) (#137)

* Update summary_m.py

* Update CHANGELOG.md

* Fix comparing samplers tutorial (closes #107) (#134)

* adapt to lower/case obs and param

* include rank and log?pdet in graph

* split single NUTS kernel up

* Fix group tutorial (closes #131) (#135)

* adapt to lower/case obs and param

* include rank and log?pdet in graph

* fix wrong variable names

* Update 07-groups.qmd

---------

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettingen.de>

* Expose sampling results (#138)

* export SamplingResults

* Update CHANGELOG.md

* Refactor goose-models to goose-interfaces (closes #110) (#136)

* Rename goose "models" to "interfaces"

* add warnings

* rename and move GooseModel -> LieselInterface

* update names in other parts of the code

* Update CHANGELOG.md

* restore goose.models module

* call super

* MVN Degenerate from smoothing param (#133)

* MVN Degenerate from smoothing param

* Added log prob test

* Fix pre commit error

* Refactor. Back to old naming

---------

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>

* update revision of hooks in pre-commit-config (#139)

* fix typo

* update lsl.Node docs

* update Data docs

* update calc docs

* update dist docs

* update var docs

* Update config to avoid seaborn warnings

* Revert "Update config to avoid seaborn warnings"

This reverts commit 0bed211.

* update obs and param docs

* fix deprecation message of add_groups

* activate docs builds for PRs

* Update config to avoid seaborn warnings (#141)

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>

* update graphbuilder docs

* update model docs

* update enginebuilder docs

* update interface docs

* Update README.md

* Add model and goose overview (#145)

* add overview pages to docs

* Update goose.rst

* Update index.rst

* Update nodes.py

* fix typo

* Jittering Functions documentation (#144)

* Jittering functions documentation

* Simpler example

* Simpler example

* Simpler example

* Removed set_duration and build

* Back to param

* Pre-commit happy

* Updated doc if jitter functions are not provided

---------

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>

* Make GEV tutorials to work (#140)

* Make GEV tutorial to work

* Updated tutorial and jitter fn for tau2

* truncated_normal fixed for pytest

---------

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>

* update linear regression tutorial

* Update 01a-transform.qmd

* Update 02-ls-reg.qmd

* remove mistakenly added lines

* add pointer to tutorials

update link

* update deprecation warning message

* Update docs.yml

* Fix some pytets warnings (#147)

* Fix some pytets warnings

* Update test_goose.py

---------

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>
Co-authored-by: Johannes Brachem <37882800+jobrachem@users.noreply.github.com>

* Update CHANGELOG.md

* Update __version__.py

* Update __version__.py

* Different seed GEV tutorials

* Allow passing key as seed

* add

* change type to be consistent with Goose

* pin pymc version

* Implements NamedTupleInterface (#151)

* Adds NamedTupleInterface

* Format

* Add documentation

* Export NamedTupleInterface in goose.__init__.py

* Format

* Update documentation

* Update CHANGELOG.md

* Updated version

* Update CHANGELOG.md

* fix name `basis_matrix` in 07-groups.qmd (#166)

* Better data generation for location-scale regression tutorial

* Fixed deprecated warnings

* Update docs (#169)

* Update model.rst

* Update __version__.py

* Raise error if duplicate nodes are detected in GraphBuilder.transform (#167)

* Transform method handles nodes duplicates

* Refactor

* Added tests. Removed useless check

* Update CHANGELOG.md

* rename and move test

* rephrase message

* remove unnecessary code

---------

Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>
Co-authored-by: Johannes Brachem <37882800+jobrachem@users.noreply.github.com>

* try to update calc on init

Update nodes.py

* fix test

Due to the exception being caught and re-raised in the update() method, we do not get the actual error text here.

* implement init argument

* fix Calc docs

* Update CHANGELOG.md

* Removed duplicated logger in nodes module

---------

Co-authored-by: Hannes Riebl <hriebl@uni-goettingen.de>
Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettingen.de>
Co-authored-by: wiep <wiep@users.noreply.github.com>
Co-authored-by: Gianmarco Callegher <gianmarco.callegher@uni-goettinge.de>
Co-authored-by: Sebastian Lorek <sebastian.lorek@gmx.de>
Co-authored-by: Sebastian Lorek <76441080+Seb-Lorek@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jobrachem jobrachem Awaiting requested review from jobrachem

Assignees

@GianmarcoCallegher GianmarcoCallegher

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@GianmarcoCallegher @jobrachem