Janos Gabler

Results 38 comments of Janos Gabler

Seems like we don't need any dependency and can just use `difflib` from the standard library. ![image](https://user-images.githubusercontent.com/9536380/176413865-2716f91c-da66-47bd-904b-6f92e1a9ee26.png) - [tweet](https://twitter.com/driscollis/status/1541841989120958466?t=TCFqIZkSdc327hY1i2QATg&s=03) - [documentation](https://docs.python.org/3/library/difflib.html#difflib.get_close_matches)

## To-Do - Rewrite the [propose_alternatives](https://github.com/OpenSourceEconomics/estimagic/blob/f62ebc8f8318cb8208d1b2bee9565d92244732e8/src/estimagic/utilities.py#L112) function using difflib - Add some simple unit tests for it [here](https://github.com/OpenSourceEconomics/estimagic/blob/main/tests/test_utilities.py) - Remove `fuzzywuzzy` from environment.yml and setup.cfg

Solved a long time ago (#437)

It looks like the test we need already exists: https://arxiv.org/pdf/1907.13093.pdf

Two more resources: - [JAX enhancement proposal](https://jax.readthedocs.io/en/latest/jep/12049-type-annotations.html) - [Jaxtyping](https://github.com/google/jaxtyping)

The code here can probably be improved by using the history collection introduced in #324

We close this due to a more ambitious visualization projects started in #403

I will close this for now because it is incompatible with some of the planned changes in benchmarking (see #495). I kept the files so we can easily re-add this...

Thanks for opening the issue. In `check_constraints_are_satisfied` we can definitely do what you suggest. An EPP student is currently trying to make that code faster and I think improving the...