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

dev to main #26

Merged
merged 25 commits into from
Aug 8, 2024
Merged

dev to main #26

merged 25 commits into from
Aug 8, 2024

Conversation

xmariachi
Copy link
Collaborator

Closes: #XXX

What is the purpose of the change

Merge dev to main

Documentation and Release Note

  • Does this pull request introduce a new feature or user-facing behavior changes?

Where is the change documented?

  • Specification (x/{module}/README.md)
  • Allora documentation site docs.allora.network source code at: https://github.com/allora-network/docs
  • Code comments?
  • [X ] N/A - merge

kpeluso and others added 25 commits August 2, 2024 02:23
* add changelog + security (#9)

* added dockerfile

* fix bug on build

* change the wkdir to /data

* change the wkdir to /

* fix typo

* back to /node

* push to dockerhub instead

* deployment done

---------

Co-authored-by: Kenny P <17100641+kpeluso@users.noreply.github.com>
* change the wkdir to /data

* change the wkdir to /

* fix typo

* back to /node

* deploy 2 reputer and workers for topic 1

* random
* API Adapter with README

* move API adapter readme inside dir

* Config file/env var + factory adapters + minor fixes
Hello, I found some typos in README.md

"intentend" to "intended"

"be use" to "be used"

And there were also missing space in "InferenceEndpointis required if InferenceEntrypoint is defined.". This canged for "InferenceEndpoint is required if InferenceEntrypoint is defined."

Hope this helps.

Signed-off-by: osrm <90407222+osrm@users.noreply.github.com>
* cleanups

* efficient deployment updates

* fix bug

---------

Co-authored-by: Kenny <pelusoken@gmail.com>
* uniform backoff

* README update
* Update and rename start.docker to init.docker

on devnet, when running the command:

```
chmod +x start.docker
./start.docker
```

 the worker node won't register because it is not funded. The wallet for the worker node is being created and the node is attempting to register to the network in one go. However, for the user this will never work. What they will have to do is
1. run the command
2. kill the terminal
3. fund their wallet
4. rerun the chmod command

The fix proposed is:

1. run init.docker to generate address and then you can get the address funded manually
2. run docker compose --build separately to start your services

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* Update README.md

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* Update README.md

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

---------

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>
* add changelog + security (#9)

* release v0.1.0 (#23)

* add changelog + security

* Deployment (#10)

* add changelog + security (#9)

* added dockerfile

* fix bug on build

* change the wkdir to /data

* change the wkdir to /

* fix typo

* back to /node

* push to dockerhub instead

* deployment done

---------

Co-authored-by: Kenny P <17100641+kpeluso@users.noreply.github.com>

* use internal network dns

* Deployment (#13)

* change the wkdir to /data

* change the wkdir to /

* fix typo

* back to /node

* deploy 2 reputer and workers for topic 1

* random

* API Adapter with README ORA-2066 (#12)

* API Adapter with README

* move API adapter readme inside dir

* Config file/env var + factory adapters + minor fixes

* updating readme to tell you to copy the env file and fill it out before running docker-compose (#14)

* readme changes (#15)

* chore: fix typos README.md (#17)

Hello, I found some typos in README.md

"intentend" to "intended"

"be use" to "be used"

And there were also missing space in "InferenceEndpointis required if InferenceEntrypoint is defined.". This canged for "InferenceEndpoint is required if InferenceEntrypoint is defined."

Hope this helps.

Signed-off-by: osrm <90407222+osrm@users.noreply.github.com>

* update dependencies, do not print conf (#18)

* Cleanups (#16)

* cleanups

* efficient deployment updates

* fix bug

---------

Co-authored-by: Kenny <pelusoken@gmail.com>

* fix issues with docker-compose initialize

* readme fix

* forecast should return nodeValue

* Update dependencies + reputer nonce fix (#19)

* fix deprecated docker compose command

* string interpolation

* uniform backoff (#21)

* uniform backoff

* README update

* Update and rename start.docker to init.docker (#22)

* Update and rename start.docker to init.docker

on devnet, when running the command:

```
chmod +x start.docker
./start.docker
```

 the worker node won't register because it is not funded. The wallet for the worker node is being created and the node is attempting to register to the network in one go. However, for the user this will never work. What they will have to do is
1. run the command
2. kill the terminal
3. fund their wallet
4. rerun the chmod command

The fix proposed is:

1. run init.docker to generate address and then you can get the address funded manually
2. run docker compose --build separately to start your services

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* Update README.md

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* Update README.md

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

---------

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* fix the conflicting account issues

* add pull request template

* correct the wkdir for source github actions

---------

Signed-off-by: osrm <90407222+osrm@users.noreply.github.com>
Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>
Co-authored-by: Kenny <pelusoken@gmail.com>
Co-authored-by: Kenny P <17100641+kpeluso@users.noreply.github.com>
Co-authored-by: Diego C <xmariachi@gmail.com>
Co-authored-by: Madison Williams <madisonjw@gmail.com>
Co-authored-by: osrm <90407222+osrm@users.noreply.github.com>
Co-authored-by: kush-alloralabs <kush@alloralabs.xyz>

* remove usage of init.docker

* fix readme conflicts

* fix readme conflicts

* use offchain-node as name

---------

Signed-off-by: osrm <90407222+osrm@users.noreply.github.com>
Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>
Co-authored-by: Kenny P <17100641+kpeluso@users.noreply.github.com>
Co-authored-by: Tobi Okedeji <tobiokedeji@gmail.com>
Co-authored-by: Kenny <pelusoken@gmail.com>
Co-authored-by: Madison Williams <madisonjw@gmail.com>
Co-authored-by: osrm <90407222+osrm@users.noreply.github.com>
Co-authored-by: kush-alloralabs <kush@alloralabs.xyz>
@xmariachi xmariachi marked this pull request as ready for review August 8, 2024 19:48
@xmariachi xmariachi requested review from okedeji and kpeluso and removed request for okedeji August 8, 2024 19:49
@okedeji okedeji merged commit 551f6a0 into main Aug 8, 2024
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.

6 participants