Skip to content

Commit

Permalink
Pin cyipopt to version 1.2 or smaller and fix ruff complaints
Browse files Browse the repository at this point in the history
  • Loading branch information
timmens committed Oct 25, 2023
1 parent 4823580 commit 31bc910
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
2 changes: 1 addition & 1 deletion environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ channels:
- nodefaults
dependencies:
- python=3.10 # dev
- cyipopt # dev
- cyipopt<1.3.0 # dev
- jupyterlab # dev, docs
- nb_black # dev, docs
- nlopt # dev, tests
Expand Down
10 changes: 6 additions & 4 deletions src/estimagic/visualization/estimation_table.py
Original file line number Diff line number Diff line change
Expand Up @@ -1240,8 +1240,9 @@ def _generate_notes_latex(
raise ValueError(
f"""Each custom note can only be of string type.
The following notes:
{[n for n in custom_notes if type(n) != str]} are of types
{[type(n) for n in custom_notes if type(n) != str]}
{[n for n in custom_notes if not isinstance(n, str)]} are of
types
{[type(n) for n in custom_notes if not isinstance(n, str)]}
respectively."""
)
for n in custom_notes:
Expand Down Expand Up @@ -1300,8 +1301,9 @@ def _generate_notes_html(
raise ValueError(
f"""Each custom note can only be of string type.
The following notes:
{[n for n in custom_notes if type(n) != str]} are of types
{[type(n) for n in custom_notes if type(n) != str]}
{[n for n in custom_notes if not isinstance(n, str)]} are of
types
{[type(n) for n in custom_notes if not isinstance(n, str)]}
respectively."""
)
notes_text += """
Expand Down

0 comments on commit 31bc910

Please sign in to comment.