Skip to content

Commit

Permalink
chore: Remove 'fcomm' package and associated files (#871)
Browse files Browse the repository at this point in the history
- Removed the entire `fcomm` module, 👋
- Deleted numerous `fcomm` related source and snapshot files from the project.
- Removed `test_make_fcomm_examples` from excludedLinux Tests phase in GitHub Actions workflow
- Updated README link to fcomm -> demo under 'Proofs' section.
- Removed `fcomm` as workspace member in `Cargo.toml` and from GitHub Actions CI/CD setup.
  • Loading branch information
huitseeker authored Nov 8, 2023
1 parent 5de43d7 commit d9c81c1
Show file tree
Hide file tree
Showing 29 changed files with 3 additions and 2,635 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/merge-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
- uses: Swatinem/rust-cache@v2
- name: Linux Tests
run: |
cargo nextest run --profile ci --workspace --cargo-profile dev-ci --run-ignored ignored-only -E 'all() - test(test_make_fcomm_examples) - test(test_demo)'
cargo nextest run --profile ci --workspace --cargo-profile dev-ci --run-ignored ignored-only -E 'all() - test(test_demo)'
linux-arm:
if: github.event_name != 'pull_request' || github.event.action == 'enqueued'
Expand Down
2 changes: 1 addition & 1 deletion Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ vergen = { version = "8", features = ["build", "git", "gitcl"] }

[workspace]
resolver = "2"
members = ["fcomm", "lurk-macros", "lurk-metrics"]
members = ["lurk-macros", "lurk-metrics"]

# Dependencies that should be kept in sync through the whole workspace
[workspace.dependencies]
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ Lurk's distinguishing feature relative to most zk-SNARK authoring languages is t

# Proofs

Integration with backend proving systems and tooling for proof generation are both still very early. Performance and user experience still have room for significant optimization and improvement, but simple examples can be found in the [fcomm example directory](fcomm/README.md).
Integration with backend proving systems and tooling for proof generation are both still very early. Performance and user experience still have room for significant optimization and improvement, but simple examples can be found in the [demo example directory](demo/).

# Backends
- Nova is Lurk's officially-supported IVC backend. It uses Lurk Lab's Arecibo fork of the [Nova proving system](https://github.com/lurk-lab/arecibo) and the Pasta Curves.
Expand Down
1 change: 0 additions & 1 deletion fcomm/.clippy.toml

This file was deleted.

2 changes: 0 additions & 2 deletions fcomm/.gitignore

This file was deleted.

46 changes: 0 additions & 46 deletions fcomm/Cargo.toml

This file was deleted.

69 changes: 0 additions & 69 deletions fcomm/README.md

This file was deleted.

4 changes: 0 additions & 4 deletions fcomm/examples/.gitignore

This file was deleted.

49 changes: 0 additions & 49 deletions fcomm/examples/Makefile

This file was deleted.

1 change: 0 additions & 1 deletion fcomm/examples/chained-function.json

This file was deleted.

1 change: 0 additions & 1 deletion fcomm/examples/chained-input.lurk

This file was deleted.

1 change: 0 additions & 1 deletion fcomm/examples/chained-request.json

This file was deleted.

1 change: 0 additions & 1 deletion fcomm/examples/eval-input.lurk

This file was deleted.

1 change: 0 additions & 1 deletion fcomm/examples/fcomm

This file was deleted.

2 changes: 0 additions & 2 deletions fcomm/examples/fibonacci.json

This file was deleted.

11 changes: 0 additions & 11 deletions fcomm/examples/fibonacci.lurk

This file was deleted.

1 change: 0 additions & 1 deletion fcomm/examples/num-list-function.json

This file was deleted.

7 changes: 0 additions & 7 deletions fcomm/examples/num-list-input.lurk

This file was deleted.

Loading

1 comment on commit d9c81c1

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

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

Benchmarks

Table of Contents

Overview

This benchmark report shows the Fibonacci GPU benchmark.
NVIDIA GeForce RTX 4070
AMD Ryzen 9 3950X 16-Core Processor
125.711 GB RAM

Benchmark Results

LEM Fibonacci Prove - rc = 100

fib-ref=5de43d79b43237c0b465ea53a72b1ff6e7ab6126 fib-ref=d9c81c1c2d0c3220884d2c983259721bfd17ecd3
num-100 4.01 s (✅ 1.00x) 4.00 s (✅ 1.00x faster)
num-200 9.00 s (✅ 1.00x) 8.76 s (✅ 1.03x faster)

LEM Fibonacci Prove - rc = 600

fib-ref=5de43d79b43237c0b465ea53a72b1ff6e7ab6126 fib-ref=d9c81c1c2d0c3220884d2c983259721bfd17ecd3
num-100 3.05 s (✅ 1.00x) 3.05 s (✅ 1.00x faster)
num-200 7.09 s (✅ 1.00x) 7.04 s (✅ 1.01x faster)

Made with criterion-table

Please sign in to comment.