Skip to content

Use Beaker queues instead of Google Cloud Pub/Sub for the worker system. #138

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

Merged
merged 4 commits into from
Jun 17, 2025

Conversation

favyen2
Copy link
Collaborator

@favyen2 favyen2 commented May 27, 2025

Main changes:

  • Use Beaker queues instead of Google Cloud Pub/Sub for the worker system
  • Upgrade beaker-py to 2.x (queues are not supported in older version).

Minor updates/fixes:

  • Update helios.Dockerfile to use pytorch 2.7.0 to reduce build time.
  • Update one_off_projects/convert_satlas_webmercator_to_rslearn/lib/__init__.py for new rslearn VectorFormat.encode_vector API.
  • Remove manage_scratch_dir_on_data_disk option since it's not needed anymore (since the Docker volumes are now on the big /data disk across all Beaker nodes).

favyen2 added 2 commits May 27, 2025 13:51
There are several other changes:
- Upgrade beaker-py to 2.x (queues are not supported in older version).
- Update helios.Dockerfile to use pytorch 2.7.0 to reduce build time.
- Update one_off_projects/convert_satlas_webmercator_to_rslearn/lib/__init__.py for new rslearn VectorFormat.encode_vector API.
- Remove manage_scratch_dir_on_data_disk option since it's not needed anymore (since the Docker volumes are now on the big /data disk across all Beaker nodes).
Copy link
Collaborator

@yawenzzzz yawenzzzz left a comment

Choose a reason for hiding this comment

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

LGTM!

@favyen2 favyen2 merged commit 35354c7 into master Jun 17, 2025
4 checks passed
@favyen2 favyen2 deleted the favyen/20250527-beaker-upgrad branch June 17, 2025 17:06
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