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

Reduce memory requirements of tests #90

Open
Aariq opened this issue Feb 4, 2025 · 1 comment
Open

Reduce memory requirements of tests #90

Aariq opened this issue Feb 4, 2025 · 1 comment

Comments

@Aariq
Copy link
Collaborator

Aariq commented Feb 4, 2025

The long-running tests currently skipped by default are causing my R session to run out of memory. A fix might be as simple as breaking these into separate tests, but it might also be good to see if all unit tests can be made more minimal.

@Aariq
Copy link
Collaborator Author

Aariq commented Feb 4, 2025

This test in particular seems to be huge:

  some_data <- npn_download_status_data(
    request_source = "Unit Test",
    years = 2013,
    class_ids = c(15), #TODO pick a smaller class or year with less data!
    additional_fields = c("Class_ID")
  )

Could write to disk (as one would do in practice), pick a smaller class, and/or pick a year with less data.

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

No branches or pull requests

1 participant