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

Compute Example Readme flow #1615

Merged
merged 53 commits into from
Dec 14, 2022
Merged

Compute Example Readme flow #1615

merged 53 commits into from
Dec 14, 2022

Conversation

bogdanfazakas
Copy link
Member

@bogdanfazakas bogdanfazakas commented Sep 23, 2022

Fixes #1195.

Changes proposed in this PR:

  • Add new integration test file ComputeExamples.test.ts
  • Add new integration test file ComputeExamples.md

@bogdanfazakas bogdanfazakas changed the title WIP C2DExamples Readme flow Compute Example Readme flow Sep 27, 2022
@bogdanfazakas bogdanfazakas marked this pull request as ready for review September 27, 2022 13:40
ComputeExamples.md Outdated Show resolved Hide resolved
@jamiehewitt15
Copy link
Member

I'm currently getting an error when running this, is it working ok for you?

node dist/compute.js
Aquarius URL: http://172.15.0.5:5000
Provider URL: http://172.15.0.4:8030
Deployed contracts address: [object Object]
Publisher account address: 0xe2DD09d719Da89e5a3D0F2549c7E24566e947260
Consumer account address: 0xBE5449a6A97aD46c8558A3356267Ee5D2731ab5e
dataset id: did:op:7688bf67e4db9a3a4d28e3d97982f8f501fa69d3aed0204a7101e8e626f5c021
algorithm id: did:op:4f175f2689b6e8edd3cf82989df19b3ef4ce391ec5fafb261c5cea99f84d5888
/home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:330
                    computeEnv = computeEnvs.find(function (ce) { return ce.priceMin === 0; });
                                             ^

TypeError: computeEnvs.find is not a function
    at /home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:330:46
    at step (/home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:33:23)
    at Object.next (/home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:14:53)
    at fulfilled (/home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

@jamiehewitt15
Copy link
Member

I've uploaded the code I'm running here, maybe I made some mistake following along

@codeclimate
Copy link

codeclimate bot commented Dec 12, 2022

Code Climate has analyzed commit fce7b32 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 79.9% (0.0% change).

View more on Code Climate.

@bogdanfazakas
Copy link
Member Author

I'm currently getting an error when running this, is it working ok for you?

node dist/compute.js
Aquarius URL: http://172.15.0.5:5000
Provider URL: http://172.15.0.4:8030
Deployed contracts address: [object Object]
Publisher account address: 0xe2DD09d719Da89e5a3D0F2549c7E24566e947260
Consumer account address: 0xBE5449a6A97aD46c8558A3356267Ee5D2731ab5e
dataset id: did:op:7688bf67e4db9a3a4d28e3d97982f8f501fa69d3aed0204a7101e8e626f5c021
algorithm id: did:op:4f175f2689b6e8edd3cf82989df19b3ef4ce391ec5fafb261c5cea99f84d5888
/home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:330
                    computeEnv = computeEnvs.find(function (ce) { return ce.priceMin === 0; });
                                             ^

TypeError: computeEnvs.find is not a function
    at /home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:330:46
    at step (/home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:33:23)
    at Object.next (/home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:14:53)
    at fulfilled (/home/jamie/Desktop/ocean/compute-quickstart/dist/compute.js:5:58)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Can you add a log for computeEnvs, and also check the barge logs for some errors, it looks like the compute env list is empty, it may be something local from barge to the operator when trying to get the defined compute envs.

@jamiehewitt15
Copy link
Member

After pruning all my docker containers and running this again it worked without any modification. Seems all good to me, let's get it merged

@bogdanfazakas bogdanfazakas merged commit a026c12 into main Dec 14, 2022
@bogdanfazakas bogdanfazakas deleted the feature/c2d-flow-readme branch December 14, 2022 15:22
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.

"C2D Flow" README works in v4
4 participants