Skip to content

Conversation

@brianhelba
Copy link
Collaborator

No description provided.

@brianhelba brianhelba requested a review from annehaley January 2, 2026 19:42
@cloudflare-workers-and-pages
Copy link

Deploying geoinsight with  Cloudflare Pages  Cloudflare Pages

Latest commit: edaafbe
Status: ✅  Deploy successful!
Preview URL: https://b6e78e6b.geoinsight.pages.dev
Branch Preview URL: https://heroku-worker.geoinsight.pages.dev

View logs

Copy link
Collaborator

@annehaley annehaley left a comment

Choose a reason for hiding this comment

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

If we don't use workers, where will tasks be run? They need to run in an environment where the tasks extras are available.

@brianhelba
Copy link
Collaborator Author

@annehaley I'm happy to discuss this.

However, the Heroku slug size has a hard limit of 500MB, so we can't install the extra dependencies there.

I think this is generally an unfortunate result of the mismatch between the research community (which doesn't care about packaging or optimizing code redistribution size) and the web engineering community (which needs to care about maintaining sane package sizes, to support more advanced deployment automation).

Copy link
Collaborator

@annehaley annehaley left a comment

Choose a reason for hiding this comment

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

After a dicsussion with @brianhelba, we have decided to move our asynchronous tasks to an EC2 worker instead of a Heroku worker.

@annehaley annehaley merged commit 4615586 into master Jan 14, 2026
4 checks passed
@annehaley annehaley deleted the heroku-worker branch January 14, 2026 18:28
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