Marcos Nicolau comments

Results 9 comments of


                                            Marcos Nicolau

add resource in academics section

Hi, Sure, I'll do it asap! And thank you for the review!!

add resource in academics section

Hello, Sorry that it took me so long to make the changes. I've been on vacation and I didn't have a pc with me! Thanks!

fix(audit): Remove Task data from aggregator after a response has been responded or expires.

> What happens if an answer comes after the batch is closed? It should probably be tested with more operators The aggregator when receiving the response checks that the batch...

feat(aggregation-mode): proving system id for proof commitments

> Add IDs to interface constants > Make proving system u16 I've added the proving system as a `u16`. I don't think having an enum in the function is the...

Error exec 'make ethereum_package_start'

Solved in #2172

Fix: Current ruleset "Review needed" requires a pull request before merging

I was about to address this problem. Your solution looks good, but I don't think it will work since it requires creating a personal access token with the necessary permissions,...

Fix: Current ruleset "Review needed" requires a pull request before merging

> Also can we squash the commits of PRs before merging ? Just added that.

feat: integrate lita verifier

@Oppen, sure here is the reference: - [lita docs](https://lita.gitbook.io/lita-documentation) - [lita release I am using](https://github.com/lita-xyz/llvm-valida-releases/releases/tag/v0.1.0-alpha) - [lita valida vm](https://github.com/valida-xyz/valida)

Inconsistent Verification Results with Specific Zero-Knowledge Proofs

Hi @nihat99. Could you be more specific with your issue: What is your proving system and version? Can you share your program code? How are you sending the proofs (via...