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

Add make_view_incremental, tests, documentation, etc. #28

Merged
merged 1 commit into from
Sep 26, 2024

Conversation

jasonmp85
Copy link
Collaborator

Committing this functional subset to get the PR open and start working code into a release.

@jasonmp85
Copy link
Collaborator Author

Had some interrupts since opening this, forgot I did open it, and was pleasantly surprised today that I'd already written a function I know I'll need.

Getting this up to speed!

@jasonmp85 jasonmp85 force-pushed the make_view_incremental branch 2 times, most recently from 861630c to bfc8998 Compare September 25, 2024 21:35
Committing this functional subset to get the PR open and start working code into a release.
@jasonmp85 jasonmp85 merged commit 636153c into main Sep 26, 2024
2 checks passed
@jasonmp85 jasonmp85 deleted the make_view_incremental branch September 26, 2024 16:42
jasonmp85 added a commit that referenced this pull request Sep 26, 2024
Committing this functional subset to get the PR open and start working code into a release.
@Felk
Copy link

Felk commented Sep 27, 2024

Hey there, thanks for this! I'm very excited to try this out! But seeing that you didn't add this to the readme or the recent release notes, do you consider this "unstable"? As in, e.g. I shouldn't open issues if I encounter any problems, and shouldn't expect it to "just work"?

@jasonmp85
Copy link
Collaborator Author

do you consider this "unstable"

No, it's not considered unstable. We do have one issue being fixed in a newer release of IVM that I'm releasing shortly. I'll update the install instructions so you know what to do there, too. I'll keep you posted.

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