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

Tracking PR for v0.7 release #978

Draft
wants to merge 34 commits into
base: main
Choose a base branch
from
Draft

Tracking PR for v0.7 release #978

wants to merge 34 commits into from

Conversation

bobbinth
Copy link
Contributor

No description provided.

PhilippGackstatter and others added 30 commits November 11, 2024 13:03
* feat: send 503 error when queue is full

* docs: update changelog

* fix: use new method to create an error
…::build_existing` (#969)

* feat: Make `initialize_from_components` private

* feat: Refactor `build_testing` into `build_existing`

* chore: Remove `nonce` field in builder

* chore: Add changelog

* chore: Fix docs

* chore: `make format`

* chore: Fix incorrect changelog

* chore: Refactor build `Auth` as method

* chore: Update `with_assets` doc comment
…or::Error` (#974)

* feat: Remove `Clone`, `PartialEq`, `Eq` from errors

* feat: Refactor `AssetError` and `AssetVaultError`

* feat: Use `thiserror` for `BlockError`

* chore: Make `NoteExecutionHint` tags associated constants

* feat: Use `thiserror` for `NoteError`

* feat: Include printed Report in error

* feat: Add error messages for `TransactionInputError`

* feat: Add error messages for `TransactionOutputError`

* feat: Use `thiserror` for `ProvenTransactionError`

* feat: Update `winter-utils` to `0.10.2`

* feat: Add error messages for `AccountDeltaError`

* fix: Build no-std

* feat: Add error messages for `AccountError`

* chore: Add Changelog entry

* chore: Fix formatting

* feat: Replace `std` with `core::error::Error`

* chore: Alphabetize dependency order

* chore: Include only minor version in version string
* wip: unify queues in proxy

* fix: remove prints, update documentation

* fix: remove sleep

* docs: update changelog

* feat: use struct for queue, add methods for context, move functions to utils

* docs: add separator for structs, add doc comments

* feat: use VecDequeue instead of Vec for queues

* fix: add concurrent to makefile installation

* rename set_available_worker, use assert instead of if

* fix: use constant for interval time

* fix: fix docs, remove unused scope, remove _ from var

* fix: changelog entry

* feat: use u128 for request_id

* docs: update {add, get}_available_worker documentation

* fix: use mayus for structs sections, rename get_available_worker, add info logs, fix documentation
Improves the new `digest!` macro by allowing any length string literals. The previous
implementation only allowed for Felt-multiple lengths.
* feat: AccountHeader serde

* chore: CHANGELOG
* feat: impl extend advice func for TXKernel

* refactor: add as_elements for AccountHeader, use new func in FPI tests

* chore: update doc comment

* refactor: make merkle path mandatory, use new function to provide the merkle path of foreign account

* refactor: update variable name

* refactor: impl StorageSlotHeader, move as_elements there

* refactor: move back the constant, add doc comments
* feat: add conversion from `Account` to `AccountDelta`

* Update objects/src/accounts/delta/storage.rs

Co-authored-by: Philipp Gackstatter <PhilippGackstatter@users.noreply.github.com>

* Update objects/src/accounts/delta/vault.rs

Co-authored-by: Philipp Gackstatter <PhilippGackstatter@users.noreply.github.com>

* refactor: reduce cloning in conversions

* refactor: get rid of unnecessary vector allocations

* docs: update CHANGELOG.md

* fmt: apply rustfmt after merging

* docs: improve function description

---------

Co-authored-by: Philipp Gackstatter <PhilippGackstatter@users.noreply.github.com>
* feat: implement `count` method for (non)fungible asset deltas

* refactor: rename `count` to `num_assets`
* feat: add `get_note_script_hash`

* feat: add test

* fix: kernel offsets

* fix: unit tests

* refactor: rename procedure

* chore: add procedure documentation

* chore: add procedures to docs

* chore: update CHANGELOG

* refactor: remove `Word` import from test
* refactor: require `Sync` for source of `InvalidFaucetAccountId`

* refactor: add `'static` requirement
* feat: dynamic creation and removal of workers

* feat: add command to update workers

* refactor: use two lists for workers in load balancer

* fix: separate update workers command in two
* feat: Use `thiserror` in `miden-lib`

* feat: Use `thiserror` in `miden-tx`

* chore: Add changelog entry

* fix: remote transaction prover internal error usage

* chore: Fix docs build

* chore: Rename `Custom` variant to `Other`
* feat: add Health Check to worker, use health check to remove workers from proxy

feat: add health check server in worker

feat: add gRPC healthcheck methods

feat: add health check in proxy

feat: add ProxyConfig::update_workers method

docs: add entry to changelog

docs: update README

docs: remove old doc from field

chore: remove unused fields in Worker Config, improve Worker::execute documentation

docs: add documentation to LBWrapper, add documentation to BackgroundService, remove unwraps

chore: split BackgroundService implementation in different methods

* review: address review comments
* feat: remove workers from the proxy configuration file

* review: add warning when initializing the proxy w/o workers

* review: update retries documentation

* review: improve documentation about starting proxy w/o workers

Co-authored-by: Tomas Rodriguez Dala <43424983+tomyrd@users.noreply.github.com>

---------

Co-authored-by: Tomas Rodriguez Dala <43424983+tomyrd@users.noreply.github.com>
review: rename to MIDEN_TX_PROVER, add target to parent trace for each request

review: use default implementations directly from trait

review: unpin patch versions in cargo.toml

review: remove debug log with transaction witness

chore: remove unused import

review: improve comment on default methods implementation

review: remove info! log and change log level of the prove_transaction method return

review: use log level from env

review: improve tracing setup documentation

review: refactor logging and tracing sections in readme

review: add transaction ID to worker trace

review: merge tracing setup functions

fix: add default implementation proxy

review: tracing section in readme shorter

review: address readme comments

fix: add missing import

review: move changelog entry to bottom

review: add missing empty line in readme

review: exclude parameters from trace in connected_to_upstream
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.

9 participants