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

Add Larch 2nd milestone #1028

Merged
merged 2 commits into from
Nov 3, 2023
Merged

Add Larch 2nd milestone #1028

merged 2 commits into from
Nov 3, 2023

Conversation

radhe-zeeve
Copy link
Contributor

Milestone Delivery Checklist

  • The milestone-delivery-template.md has been copied and updated.
  • This pull request is being made by the same account as the accepted application.
  • I have disclosed any and all sources of reused code in the submitted repositories and have done my due diligence to meet its license requirements.
  • In case of acceptance, an invoice must be submitted and the payment will be transferred to the BTC/ETH/fiat account provided in the application.
  • The delivery is according to the Guidelines for Milestone Deliverables.

Link to the application pull request: w3f/Grants-Program#1744, w3f/Grants-Program#1399

@takahser Article Links will be updated soon, you can start with technical testing.

Signed-off-by: Jasti Sri Radhe Shyam <[email protected]>
@takahser takahser self-assigned this Oct 12, 2023
@takahser takahser self-requested a review October 12, 2023 18:04
@takahser
Copy link
Contributor

Great, thanks @radhe-zeeve. Will be back with feedback soon.

Copy link
Contributor

@takahser takahser left a comment

Choose a reason for hiding this comment

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

@radhe-zeeve thanks for the submission.
Although I'm able to build it, execution fails:

bins % ll
total 130488
-rwxrwxr-x@ 1 xxx  staff  65938514 Oct 16 11:14 larch-linux

bins % ./larch-linux --help
zsh: exec format error: ./larch-linux

bins % ./larch-linux       
zsh: exec format error: ./larch-linux

Note: I'm using the same linux machine template on AWS as for M1, so it's likely not an environment-related problem.

@radhe-zeeve
Copy link
Contributor Author

@takahser Can you tell on what OS did you built the binary and what command was used to build the binary and where are you executing the binary.

@takahser
Copy link
Contributor

@radhe-zeeve I was using ubuntu on aws (a t2 instance). I followed the guide to create the binary, so it was npx lerna run package:linux. It's the same environment as I've already used for M1.

@radhe-zeeve
Copy link
Contributor Author

radhe-zeeve commented Oct 19, 2023

@takahser I have checked on EC2 Ubunut 22 Instance, I am able to build and execute the binary

image

I believe you are trying to execute it on Mac M2: bins % ./larch-linux (% is not the current default shell separator symbol & zsh in error is zsh shell which is also not default shell in Ubuntu ). If this it the case, try to build and execute the binary on ubuntu instance.

@takahser takahser self-requested a review October 30, 2023 16:26
Copy link
Contributor

@takahser takahser left a comment

Choose a reason for hiding this comment

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

@radhe-zeeve I've added my evlauation here. I've set up a new VM on AWS and updated all local dependencies, with that the build now passes. The main problem is currently that there is an error when creating the network from the template through the web UI. I remember we had similar problems in the M1 delivery. If you don't know the cause of the issue we can also investigate this together on a video call which you can book here - whatever works best/easiest for you! :)

Signed-off-by: Jasti Sri Radhe Shyam <[email protected]>
Copy link
Contributor

@takahser takahser left a comment

Choose a reason for hiding this comment

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

@radhe-zeeve thanks for your patience and support. I've just accepted the delivery, you can find the final evaluation here.

@takahser takahser merged commit 6e5354f into w3f:master Nov 3, 2023
3 checks passed
@radhe-zeeve
Copy link
Contributor Author

@takahser I have uploaded new invoice with crypto as the currency transfer medium as mentioned in the Grant, earlier attached with fiat method, if the payment process is initiated with the fiat that's also fine.

@takahser
Copy link
Contributor

@radhe-zeeve thanks for the update. We've recently changed the invoicing process to avoid outdated invoices. Could you upload the invoice also here?

@semuelle
Copy link
Member

@radhe-zeeve, the postal address in your invoice does not match the one in the grant application. Could you change either one to match the other? Thanks!

@semuelle
Copy link
Member

@radhe-zeeve, I don't see the payment address in the invoice. Could you add that?

@radhe-zeeve
Copy link
Contributor Author

It's present in the middle of the invoice.
image

@semuelle
Copy link
Member

Sorry, @radhe-zeeve, my bad. Payment should go out this week if it hasn't yet.

@RouvenP
Copy link

RouvenP commented Nov 28, 2023

hi @radhe-zeeve we just transferred the payment

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.

4 participants