Skip to content

predict_time_share needs to be set before sending the completed webhook #1683

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 1 commit into from
May 22, 2024

Conversation

technillogue
Copy link
Contributor

#1643 adding a predict_time_share metric but it was only set on the prediction after the completed webhook was sent

@technillogue technillogue force-pushed the syl/fix-time-share branch 4 times, most recently from 1100a5a to af90118 Compare May 22, 2024 05:03
Signed-off-by: technillogue <technillogue@gmail.com>
@technillogue
Copy link
Contributor Author

on further reflection, this should be moved to director, but that doesn't have to happen right this second necessarily

@technillogue technillogue merged commit 1728da7 into async May 22, 2024
10 checks passed
@technillogue technillogue deleted the syl/fix-time-share branch May 22, 2024 19:23
technillogue added a commit that referenced this pull request Jun 19, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jun 19, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jun 19, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jun 19, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jun 20, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jul 3, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
technillogue added a commit that referenced this pull request Jul 18, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
mattt pushed a commit that referenced this pull request Jul 18, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
mattt pushed a commit that referenced this pull request Jul 19, 2024
predict_time_share tracks the portion of the worker's processing time that was dedicated to each individual prediction. the "cost" of each second is split across the predictions running during that second.

Co-authored-by: Zeke Sikelianos <zeke@sikelianos.com>

predict_time_share needs to be set before sending the completed webhook (#1683)

allow disabling time share metric with COG_DISABLE_TIME_SHARE_METRIC

Signed-off-by: technillogue <technillogue@gmail.com>
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.

1 participant