Skip to content

streamline CI and remove ape#703

Closed
wakamex wants to merge 47 commits intomainfrom
add_safety_check_for_calc_spot_price_in_hyperdrive_pricing_model
Closed

streamline CI and remove ape#703
wakamex wants to merge 47 commits intomainfrom
add_safety_check_for_calc_spot_price_in_hyperdrive_pricing_model

Conversation

@wakamex
Copy link
Copy Markdown
Contributor

@wakamex wakamex commented Jul 20, 2023

  • main is now the tag for every docker image built on main
  • testing is now the tag for every docker image built on any other branch (these builds are not automatically triggerred)
  • be careful with the edge tag as it will just pick the latest of either of these two

@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
elf-simulations ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 21, 2023 8:48pm

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4988 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4983 🚀

@wakamex wakamex marked this pull request as draft July 20, 2023 20:50
Copy link
Copy Markdown
Contributor

@sentilesdal sentilesdal left a comment

Choose a reason for hiding this comment

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

let's remove that NaN coercion. would rather see negative values and then debug. calculate_spot_price is just a pure math function, the responsibility to ensure proper values lies upstream

Comment thread elfpy/markets/hyperdrive/hyperdrive_pricing_model_sol.py Outdated
@wakamex wakamex force-pushed the add_safety_check_for_calc_spot_price_in_hyperdrive_pricing_model branch from 2b56d29 to bcbeee6 Compare July 21, 2023 14:39
@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4981 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4945 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4932 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4994 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4995 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4908 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4893 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4893 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4999 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4994 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4979 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4972 🚀

@github-actions
Copy link
Copy Markdown

Remaining rate limit: 4960 🚀

@wakamex wakamex changed the title safety check for calc spot price in hyperdrive pricing model streamline CI and remove ape Jul 21, 2023
@wakamex wakamex marked this pull request as ready for review July 21, 2023 20:53
@wakamex
Copy link
Copy Markdown
Contributor Author

wakamex commented Jul 21, 2023

replaced by #716 #717 and #718

@wakamex wakamex closed this Jul 21, 2023
@wakamex wakamex deleted the add_safety_check_for_calc_spot_price_in_hyperdrive_pricing_model branch July 21, 2023 21:22
@wakamex wakamex mentioned this pull request Jul 21, 2023
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