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

Bump versions of ftm, ftm-store and servicelayer #234

Merged
merged 4 commits into from
Aug 24, 2023

Conversation

catileptic
Copy link
Contributor

@catileptic catileptic commented Aug 10, 2023

  • bumped dataset to the latest version
  • tested a memorious run with the versions as they currently are in this PR, against a PostgreSQL database
  • ftm, ftm-store and servicelayer have been bumped to their latest version that makes them backwards and forwards compatible with SQLAlchemy 1.4.49 and 2+. This is needed in order for memorious to be able to use the latest version of dataset which hasn't been refactored to use SQLAlchemy 2+ yet
  • bumped the versions of banal and normality to the latest versions, to match the versions used in ftm, ftm-store and servicelayer

Copy link
Contributor

@tillprochaska tillprochaska left a comment

Choose a reason for hiding this comment

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

Not sure about dataset, but followthemoney and followthemoney-store are also installed directly e.g. in our pipelines image, so pinning them to an exact version will likely make it cumbersome to upgrade these versions independently from Memorious. Sometimes it’s important to be able to quickly bump e.g. the FtM version to make use of a new schema from the default model.

setup.py Outdated
"tabulate",
"python-dateutil >= 2.8.2, < 3.0.0",
"dataset >= 1.0.8",
"servicelayer[google,amazon] == 1.20.7",
"dataset == 1.6.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"dataset == 1.6.2",
"dataset >= 1.6.2",

setup.py Outdated
"pantomime == 0.5.1",
"alephclient >= 2.3.5",
"followthemoney >= 2.3.1",
"followthemoney-store >= 3.0.1",
"followthemoney == 3.5.2",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"followthemoney == 3.5.2",
"followthemoney >= 3.5.2",

setup.py Outdated
"followthemoney >= 2.3.1",
"followthemoney-store >= 3.0.1",
"followthemoney == 3.5.2",
"followthemoney-store == 3.0.6",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"followthemoney-store == 3.0.6",
"followthemoney-store >= 3.0.6",

@catileptic catileptic merged commit ba37638 into main Aug 24, 2023
6 checks passed
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.

2 participants