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

PoSt workers page #114

Merged
merged 31 commits into from
Apr 12, 2022
Merged

PoSt workers page #114

merged 31 commits into from
Apr 12, 2022

Conversation

rjan90
Copy link
Collaborator

@rjan90 rjan90 commented Mar 28, 2022

Initial draft for PoSt workers.

A couple of things is needed before it can be considered ready:

  • Find minimum specs for winningPoSt & windowPoSt
  • Remote storage access section
  • Clean up Env-list
  • Optional / Advanced settings
  • Finish inital testing
  • Spell check

Initial draft for PoSt workers.

A couple of things is needed before it can be merged:
@johnnymatthews
Copy link
Contributor

Nice. I'm gonna change this to a draft PR for now, since you're still working on things.

@johnnymatthews johnnymatthews marked this pull request as draft March 28, 2022 14:51
rjan90 added 4 commits March 29, 2022 09:21
Adds which optional settings can be set when running PoSt-workers
Added a section to clarify how the PoSt workers will work in the case where a SP has multiple partitions in a single proving deadline.
Add FD-limit section
Add Remote Storage Access section
@LexLuthr LexLuthr self-requested a review March 29, 2022 12:49
Copy link
Collaborator

@LexLuthr LexLuthr left a comment

Choose a reason for hiding this comment

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

Add warning flags if required.

content/en/storage-providers/seal-workers/post-workers.md Outdated Show resolved Hide resolved
content/en/storage-providers/seal-workers/post-workers.md Outdated Show resolved Hide resolved
rjan90 added 11 commits March 29, 2022 15:30
Adds a testing the setup section, removes aliases, and some small fixes
Adds full path in the relref for `explained in the setup guide`
Adds a copy proof-params from the lotus-miner instance for faster init
Updates parallel-reads values, and adds a caution
Adding som considerations before running.
Cleaning more env-variables
Remove a false statement about sector storage access.
Small change in sentence
Add OpenCL /  CUDA & Nvidia-driver mentions
Adding output of `proving workers`
Copy link
Collaborator

@LexLuthr LexLuthr left a comment

Choose a reason for hiding this comment

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

Just a few cosmetic changes. PR looks good.

content/en/storage-providers/seal-workers/post-workers.md Outdated Show resolved Hide resolved
content/en/storage-providers/seal-workers/post-workers.md Outdated Show resolved Hide resolved
content/en/storage-providers/seal-workers/post-workers.md Outdated Show resolved Hide resolved
content/en/storage-providers/seal-workers/post-workers.md Outdated Show resolved Hide resolved
content/en/storage-providers/seal-workers/post-workers.md Outdated Show resolved Hide resolved
content/en/storage-providers/seal-workers/post-workers.md Outdated Show resolved Hide resolved
rjan90 and others added 2 commits April 4, 2022 20:16
Fix spelling/typos

Co-authored-by: LexLuthr <88259624+LexLuthr@users.noreply.github.com>
Some cosmetic changes

Co-authored-by: LexLuthr <88259624+LexLuthr@users.noreply.github.com>
rjan90 and others added 4 commits April 4, 2022 20:18
Fix spelling

Co-authored-by: TippyFlits <james.bluett@protocol.ai>
Fix spelling

Co-authored-by: LexLuthr <88259624+LexLuthr@users.noreply.github.com>
Fix spelling

Co-authored-by: LexLuthr <88259624+LexLuthr@users.noreply.github.com>
Link to local fd-limit guide
rjan90 added 2 commits April 4, 2022 23:08
Remove parent cache, as it is not needed for the PoSt worker to function
Enabling a PoSt task will disable other tasks
Add very conservative winningPoSt spec. These are most likely lower
@netlify
Copy link

netlify bot commented Apr 7, 2022

Deploy Preview for lotus-docs ready!

Name Link
🔨 Latest commit 7e27607
🔍 Latest deploy log https://app.netlify.com/sites/lotus-docs/deploys/6255ed6cc4896a000829acf1
😎 Deploy Preview https://deploy-preview-114--lotus-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

johnnymatthews and others added 4 commits April 7, 2022 13:53
Removes advanced PoSt settings, as advanced configs have gotten a separate section now.
Fix a typo, and remove an excessive line from the `Environment variables` section
@rjan90 rjan90 changed the title Draft for PoSt workers page PoSt workers page Apr 12, 2022
@rjan90 rjan90 marked this pull request as ready for review April 12, 2022 20:17
@jennijuju jennijuju merged commit 39d960b into filecoin-project:main Apr 12, 2022
Copy link
Contributor

@johnnymatthews johnnymatthews left a comment

Choose a reason for hiding this comment

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

Some changes needed, mostly minor.

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