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

update doc #449

Conversation

younesStrittmatter
Copy link
Collaborator

Description

Provide a high-level summary of the change and which issue (either on Github or Notion) it relates to.
Include relevant motivation and context.

If the issue is on Github, simply link to it using the '#' symbol; otherwise, provide a notion page link):

  • resolves #(issue)

Type of change

Delete as appropriate:

  • fix: A bug fix
  • feat: A new feature
  • refactor: A code change that neither fixes a bug nor adds a feature
  • docs: Documentation only changes
  • test: Adding missing tests or correcting existing tests
  • perf: A code change that improves performance
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • ci: Changes to our CI configuration files and scripts (i.e., those in the .github directory)
  • build: Changes that affect the build system or external dependencies (i.e., poetry, conda)

If this is a breaking change (which might apply to any of the other kinds of change), note that below, otherwise delete
this section:

  • BREAKING_CHANGE: {description}

Update the title of the PR to follow the convention:
{type}{! if BREAKING_CHANGE}: {short description}, e.g.

  • "fix: add missing foo in module bar"
  • "feat: new theorist abc",
  • "feat!: replace module x with module y"
  • "docs: add notebooks on theorist z"

... then delete this block.

Features (Optional)

  • bulleted list of features, describing the code contributions in this pull request

Questions (Optional)

  • potential questions for the code reviewer (e.g., if you want feedback on a certain method or class; any feedback required from other code contributors with their mentioning)

Remarks (Optional)

  • any additional remarks for the code reviewer
  • if your remarks or questions pertain to individual lines in the code, you may also include them as comments in the code review window

Screenshots (Optional)

If your PR is related to UI change or anything visual, feel free to include screenshots to give the reviewer straightforward understandings on your changes.

musslick and others added 30 commits June 15, 2023 16:36
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
musslick and others added 29 commits June 16, 2023 11:03
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
Co-authored-by: benwandrew <benwallaceandrew@gmail.com>
…module-contribution-guide

Update theorist-module.md
Doc: extension of contribution documentation
@younesStrittmatter younesStrittmatter merged commit 84f9dae into 446-add-documentation-for-experiment_runner-module-contribution Jun 16, 2023
7 checks passed
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.

None yet

3 participants