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

Merged
merged 70 commits into from
Jan 23, 2025
Merged

Tracking PR for v0.7 release #978

merged 70 commits into from
Jan 23, 2025

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
PhilippGackstatter and others added 28 commits January 14, 2025 10:00
* feat: Add make `clippy-no-std` and `check-no-std`

* fix: no-std lints

* feat: Add clippy no std CI check

* feat: Add developer-friendly test run recipe

* chore: Refactor derives

* chore: Move `StorageValues` to `toml.rs`
* feat: Add pprof flamegraph generation for benchmark

* feat: Remove concurrent feature on miden-objects

* chore: Remove concurrent feature from miden-lib

* chore: Add changelog entry

* chore: Add docs and use default-features false for `pprof`
* feat: make AccountIdError public
* feat: make MAX_DECIMALS and MAX_MAX_SUPPLY public

* docs: update changelog

* review: remove MAX_MAX_SUPPLY
…te-provers (#1047)

* feat: split miden-tx-prover in miden-remote-provers and miden-proving-service

* fix: docs

* review: remove std feature from binary, make concurrent default

* review: add thiserror to remote-provers

* review: remove std dir in binary

* fix: exclude proving service from no-std clippy, fix lint in remote provers

* review: fix typo

Co-authored-by: igamigo <ignacio.amigo@lambdaclass.com>

* review: remove build-remote-provers target from makefile

* review: move proto files to workspace dir

* review: improve clients readme

* chore: update cargo lock

---------

Co-authored-by: igamigo <ignacio.amigo@lambdaclass.com>
* feat: Move duplicate procedures to shared util modules

* chore: Add changelog entry

* chore: Move non-shared procedures back to kernel::account

* chore: Rename `type` -> `id_type`

* chore: Reexport `account_id::{validate->validate_id, is_equal->is_id_equal}`
* refactor: initial kernel procs renaming and reordering

* refactor: update the offsets and procedures names

* chore: fix changelog

* refactor: update doc comments of the offset procedures
* feat: Remove additional decorator operations

* feat: Create `account::get_map_item`

* chore: Update expected stack state

* fix: Outdated docs on `set_map_item`

* chore: Remove unnecessary pad comments
* review: remove unused deps

* review: remove testing feature from docs and README

* review: remove once_cell dep

* review: add license section to readme

* review: use crate_root for proto files dir

* review: rename midne-remote-provers, mark it as no_std in lib.rs

* chore: update comment in build.rs

* review: do not use .take

* review: swap --features and --exclude calls in makefile

* review: fix license section in readme
* feat: missing BlockNumber s

docs: update changelog

* review: replace missing u32 with BlockNumber
* fix: Add #[macro_use]

* fix: Change to suppressing warning
* feat: Add unused deps CI job

* feat: Remove unused deps across base crates

* chore: Add changelog entry
* feat: Type storage placeholders and support for templated maps

* fix: Expose placeholder type

* refactor: Expose MapRepresentation

* reviews: Doc updates, validation for map

* chore: Merge

* chore: Merge fixes

* refactor: change inner type of StorageValue::Map

* docs: Change description

* reviews: Docs and avoid allocation when formatting

* reviews: Change module name and cratea validate function
* refactor: update toml files, update memory.rs

* refactor: update memory.masm, fix kernel data end address

* refactor: building state -- update miden-tx tests

* recator: update unit test, updating integration tests

* refactor: migrate to the VM next branch

* chore: Add shared modules before adding kernel lib

* refactor: update dependencies, fix formatting

* feat: add handling of the FalconSigToStack event to the Host

* feat: Make `EVENT_ID_PREFIX` an associated constant

* chore: Return error if event is an unexpected system event

* chore: update changelog

* refactor: fix the last test

* chore: update benchmark results

* chore: slightly improve the doc comments format

* refactor: update inline comments, small improvements

* refactor: update inline comments in SWAP script

* refactor: update docs, impl parse_word helper func

* chore: update dependencies

---------

Co-authored-by: Philipp Gackstatter <PhilippGackstatter@users.noreply.github.com>
* feat: Turn wrapped VM and crypto errors into source errors

* chore: Remove leftover proc duplication TODO

* chore: Remove outdated decorator bug TODO

* chore: Add changelog entry
* chore: update env var for building proving services proto

* review: rename var to BUILD_GENERATED_FILES, use it in miden-lib

* review: rename to BUILD_GENERATED_FILES_IN_SRC

* review: update constants documentation

* review: add comment to Makefile flag

* review: add doc comments about building to src

* review: refactor CI job to general check of built files

* review: remove env from clippy and format, improve usage of env var

* review: remove duplicated comment

* review: use make check in CI job
@bobbinth bobbinth marked this pull request as ready for review January 23, 2025 08:32
@bobbinth bobbinth merged commit fb26a64 into main Jan 23, 2025
23 checks passed
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.

10 participants