Skip to content

Conversation

@PearsonWhite
Copy link
Contributor

Mixnet analysis changes

  • Iterate list of experiments

  • Caching layer for scraping data

  • Log output for each experiment in separate log

  • Add mixnet plots

@PearsonWhite PearsonWhite self-assigned this Oct 13, 2025
@PearsonWhite PearsonWhite added this to DST Oct 13, 2025
@PearsonWhite PearsonWhite added enhancement New feature or request ift IFT commitments labels Oct 13, 2025
@PearsonWhite PearsonWhite moved this to In Progress in DST Oct 13, 2025
@PearsonWhite PearsonWhite force-pushed the pwhite/mix_3 branch 2 times, most recently from 3a1fce1 to 3317e35 Compare October 17, 2025 12:59
@PearsonWhite PearsonWhite marked this pull request as ready for review October 17, 2025 13:46
@PearsonWhite
Copy link
Contributor Author

The Add files used in mix analysis commit is from a88601e (Alberto/mix).

Since some of the changes are hacky, we may want to merge into a separate branch and then go from there. That way there are more manageable steps for cleanly merging into master.

@PearsonWhite
Copy link
Contributor Author

The ugly decorator for caching things will need to be changed before merging into master. It was a way to quickly get analysis to run much more quickly, but it's not pretty. We could either try to make it cleaner or use a generic function instead. I would say we can just have the base class do it, but it was helpful for the assert_num_nodes too. What do you think should be done with that?

@AlbertoSoutullo
Copy link
Collaborator

The ugly decorator for caching things will need to be changed before merging into master. It was a way to quickly get analysis to run much more quickly, but it's not pretty. We could either try to make it cleaner or use a generic function instead. I would say we can just have the base class do it, but it was helpful for the assert_num_nodes too. What do you think should be done with that?

Hmm as you think it is more manageable. Can't assert_num_nodes be done in upper layers?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ift IFT commitments

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

3 participants