Skip to content

Conversation

@chansen776
Copy link
Contributor

@chansen776 chansen776 commented Jan 9, 2025

Updates to notebooks that use NN's to call old versions of tensorflow to maintain compatibility. Update other outdated notebooks in PM4 and PM5.

Update to check-R-notebooks.yml to install libraries on ubuntu which are needed for various R packages.

add system command in R to force install of old version of tensorflow.

Add line to check-R-notebooks.yml to add libpng-dev which is required for png library in R
Update DoubleML...BERT.ipynb and r_dml...ownership.irnb to fixed tensorflow version.
Remove calls to DoubleML in r-dml-401k.irnb which depend on mlr3 and not available in current R version. (Version with DoubleML moved to Old)

Update check-R-notebooks.yml to install libfontconfig1-dev which is required for systemfonts package used by daggity
@chansen776 chansen776 marked this pull request as ready for review January 10, 2025 16:19
@chansen776 chansen776 requested a review from vsyrgkanis January 10, 2025 16:28
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why did the whole dml401k notebook need to change?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Similarly for this one?

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