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

MAINT: Upgrade to anaconda=2024.02 and cuda=12.3.1 #388

Merged
merged 12 commits into from
Apr 29, 2024
Merged

Conversation

mmcky
Copy link
Contributor

@mmcky mmcky commented Mar 7, 2024

This PR

  • upgrades to anaconda=2024.02
  • upgrades to CUDA=12.3.1

this PR is part of QuantEcon/meta#130

requires: #387

  • this PR is waiting on interpolations to be updated for numba>=0.59

@mmcky mmcky mentioned this pull request Mar 7, 2024
1 task
@mmcky
Copy link
Contributor Author

mmcky commented Mar 7, 2024

  • the broken link is to BK19 as a reference but that doesn't seem to be in our quant-con.bib file
You can read  about Dynamic Mode Decomposition here {cite}`DMD_book` and here [[BK19](https://python.quantecon.org/zreferences.html#id25)] (section 7.2).

@jstac or @thomassargent30 do you know what BK19 would refer to and I can add it as a proper reference using bibtex

@jstac
Copy link
Contributor

jstac commented Mar 7, 2024

@mmcky , it refers to this one:
https://www.databookuw.com/

@mmcky
Copy link
Contributor Author

mmcky commented Mar 8, 2024

@mmcky
Copy link
Contributor Author

mmcky commented Mar 12, 2024

  • this cannot be merged until interpolations is updated or there is a substitute for mlinterp that is suitable for the odu lecture (cc @kp992)
ImportError�[0m: cannot import name 'generated_jit' from 'numba' (/opt/conda/envs/quantecon/lib/python3.11/site-packages/numba/__init__.py)

@kp992
Copy link
Contributor

kp992 commented Mar 12, 2024

I see, from numba 0.60, generated_jit will fail.

@mmcky
Copy link
Contributor Author

mmcky commented Mar 13, 2024

I see, from numba 0.60, generated_jit will fail.

@kp992 I think from numba>=0.59 it will fail -- so that is why we will need to fix interpolation as we use it for minterp and some splines support. From my discussion with Pablo, the generated_jit will need to be replaced with an alternative (overload?) EconForge/interpolation.py#110

@kp992
Copy link
Contributor

kp992 commented Mar 13, 2024

From my discussion with Pablo, the generated_jit will need to be replaced with an alternative (overload?)

I have tried fixing that locally but that's throwing some error. I followed the examples of moving from generated_jit to overload, but didn't work. Let's pin numba < 0.59 until we find and do robust a fix for this issue.

@mmcky
Copy link
Contributor Author

mmcky commented Mar 21, 2024

Copy link

github-actions bot commented Mar 22, 2024

lectures/odu.md Outdated
@@ -30,7 +30,7 @@ In addition to what’s in Anaconda, this lecture deploys the libraries:
---
tags: [hide-output]
---
!pip install interpolation
!pip install git+https://github.com/EconForge/interpolation.py
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jstac this installs interpolation package in the odu lecture from the main branch of the repo as it isn't released to pypi yet. Are you happy to run with this until pypi is updated?

@mmcky
Copy link
Contributor Author

mmcky commented Mar 22, 2024

  • re-enable build cache
  • open an issue to update pip install from main branch of interpolation repo to pypi once released

lectures/odu.md Outdated Show resolved Hide resolved
reverts to pypi version which is currently `2.2.6`
@mmcky
Copy link
Contributor Author

mmcky commented Apr 26, 2024

  • reverted to install interpolation from pypi which is currently 2.2.6

@mmcky mmcky merged commit ca57701 into main Apr 29, 2024
7 checks passed
@mmcky mmcky deleted the anaconda=2024.02 branch April 29, 2024 05:46
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.

3 participants