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 Ray Fingerprint binproto and Updater Script #439

Closed
wants to merge 4 commits into from

Conversation

mr-mosi
Copy link
Contributor

@mr-mosi mr-mosi commented Mar 25, 2024

Reference to this issue #432

I added updater script and binproto file. It seems to be ready for your check. one version I think it was 0.8.6, does not support dashboard so it skipped.

@maoning maoning self-requested a review March 28, 2024 22:02
maoning added 3 commits March 28, 2024 15:50
Add a newline to make linter happy
Add a newline to make linter happy
copybara-service bot pushed a commit that referenced this pull request Mar 28, 2024
--
38a9064 by mr-mosi <[email protected]>:

binproto file updater script and docker compose file are added

--
02a8743 by Annie <[email protected]>:

Update docker-compose.yml

Add a newline to make linter happy
--
da8f68d by Annie <[email protected]>:

Update versions.txt

Add a newline to make linter happy
--
12e0529 by Annie <[email protected]>:

Update docker-compose.yml

COPYBARA_INTEGRATE_REVIEW=#439 from mr-mosi:ray 12e0529
PiperOrigin-RevId: 620086469
Change-Id: I8ea1335bd9037f2018394f30fdb34fc138c39d40
@maoning
Copy link
Collaborator

maoning commented Mar 28, 2024

Hi @mr-mosi ,

Your PR has been merged. This usually means a reward will be granted. Google will start the internal QC process and the reward amount will be determined based on the quality of the detector report. Please be patient and allow up to a week for the QC process to finish. You'll be notified once the decision is made.

Thanks!

@maoning maoning closed this Mar 28, 2024
@mr-mosi mr-mosi deleted the ray branch April 10, 2024 07:52
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