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

Compatibility with dm's tests #39

Open
krlmlr opened this issue Nov 24, 2024 · 1 comment · May be fixed by cynkra/dm#2256
Open

Compatibility with dm's tests #39

krlmlr opened this issue Nov 24, 2024 · 1 comment · May be fixed by cynkra/dm#2256

Comments

@krlmlr
Copy link

krlmlr commented Nov 24, 2024

I see, in the dev version, that the genre table has changed, and that the pixar_films table no longer has a missing value. The dm package makes certain assumptions about the structure of the data, and so might other users. Curious to hear your thoughts on how to deal with that.

Releasing dm 1.0.11 now, but I can push an update on short notice if needed.

@erictleung
Copy link
Owner

@krlmlr thanks for the issue. I did notice this too for the dm package as I was doing some testing.

The breaking change for the genres table is to expand the table a bit. I don't expect to change it that often. But I'm finally doing an update of the data, so I found some extra subgenres that I wanted, hence the breaking change.

Happy to make the updates to dm package function. The schema for the genres data is now three columns: film, category, value.

Regarding pixar_films no longer having missing data, that was a feature of at the time, Pixar didn't realize the title of the film. But now, the table I'm pulling from has all the films named. And I don't include any unreleased films anymore. So, for this issue, I'm not sure you want to work around this.

@erictleung erictleung linked a pull request Nov 27, 2024 that will close this issue
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 a pull request may close this issue.

2 participants