Skip to content

Eliminate bid adjustment factors - #44

Open
lukasschwab wants to merge 2 commits into
mainfrom
ls/rm-bid-adjustment-factors
Open

Eliminate bid adjustment factors#44
lukasschwab wants to merge 2 commits into
mainfrom
ls/rm-bid-adjustment-factors

Conversation

@lukasschwab

Copy link
Copy Markdown
Member

This system was effectively unused: all auction requests from the CloudX auction-server bear vacuous (i.e. empty, 1x) bid adjustment factors. There are no immediate plans to use non-vaucous adjustmnets factors, so we removed the logic involved from the auction-server entirely.

This removes the corresponding dead logic from openauction.

Old config field `experimental_monorepo_root` is deprecated in current
versions of mise, which yields warning noise:

```
mise WARN  deprecated [config.experimental_monorepo_root]: `experimental_monorepo_root` in .../openauction/.mise/config.toml is deprecated. Use `monorepo_root` instead. This will be removed in mise 2027.12.0.
```
This system was effectively unused: all auction requests from the CloudX
auction-server bear vacuous (i.e. empty, 1x) bid adjustment factors.
There are no immediate plans to use non-vaucous adjustmnets factors, so
we removed the logic involved from the auction-server entirely.

This removes the corresponding dead logic from openauction.
@lukasschwab
lukasschwab requested a review from nickpell as a code owner July 24, 2026 22:12

@nickpell nickpell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks solid. Since this changes the attestation we'll want to run it by demand partners before shipping.

Base automatically changed from ls/monorepo-root to main July 29, 2026 19:03
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