Skip to content

Revert gnark upgrade - #1150

Merged
JuArce merged 3 commits into
stagingfrom
revert_gnark_upgrade
Oct 2, 2024
Merged

Revert gnark upgrade#1150
JuArce merged 3 commits into
stagingfrom
revert_gnark_upgrade

Conversation

@MauroToscano

@MauroToscanoMauroToscano commented Oct 2, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@MauroToscano
MauroToscano changed the base branch from testnet to stagingOctober 2, 2024 18:46
@github-actions

github-actionsBot commented Oct 2, 2024•
edited
Loading

Copy link
Copy Markdown

Changes to gas cost

Generated at commit: a88d497cc7e1d3c59afb248c9ba6525aa56d442b, compared to commit: 2fec13617bee50d83c43bc87bd7bac0bbf63c66d

🧾 Summary (10% most significant diffs)

ContractMethodAvg (+/-)%
AlignedLayerServiceManagercreateNewTask-174 ✅-0.24%
Full diff report 👇
ContractDeployment Cost (+/-)MethodMin (+/-)%Avg (+/-)%Median (+/-)%Max (+/-)%# Calls (+/-)
AlignedLayerServiceManager4,681,797 (0)createNewTask
receive		53,823 (-144)
21,169 (0)		-0.27%
0.00%		73,740 (-174)
44,690 (-187)		-0.24%
-0.42%		73,993 (+6)
45,064 (0)		+0.01%
0.00%		74,770 (+36)
45,064 (0)		+0.05%
0.00%		256 (0)
256 (0)

IAvecilla
IAvecilla approved these changes Oct 2, 2024
View reviewed changes

@IAvecillaIAvecilla left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works fine in my pc

@JuArce
JuArce merged commit ba808d2 into stagingOct 2, 2024
@JuArce
JuArce deleted the revert_gnark_upgrade branch October 2, 2024 20:01
PatStiles pushed a commit that referenced this pull request Oct 3, 2024
@MauroToscano@PatStiles
Revert gnark upgrade (#1150)
b27ffcc
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JuArceJuArceJuArce approved these changes
+2 more reviewers
@uri-99uri-99uri-99 approved these changes
@IAvecillaIAvecillaIAvecilla approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@MauroToscano@uri-99@JuArce@IAvecilla