Skip to content

If Workspace is that important, why is it not in this repo? #63

Closed

Description

@pllim

From https://pyautogalaxy.readthedocs.io/en/latest/general/workspace.html and throughout the documentation, it seems like autogalaxy workspace is an integral part of this package (not merely optional supplementary material?). If so, why not include it as part of this package? (Can always exclude from the manifest if you do not want it in the sdist tarball.)

astropy/astropy.github.com#491 (comment)

Activity

  1. pllim commented on Aug 15, 2022

    @pllim
    ContributorAuthor

    Related: If Workspace notebooks are indeed essential, consider adding Jupyter notebook installation instructions to the "conda" section of installation pages.

    Or if you want to lock things down more, provide a conda environment YAML file.

  2. Jammy2211 commented on Nov 27, 2022

    @Jammy2211
    Collaborator

    The primary reason is that the workspace contains notebooks and .fits datasets, which in the past have caused a lot of problems with bulking-out the repo history. By keeping them on a separate repo we avoid the source code history being noisy or (as was the case a long time ago) broken.

    I also didn't find it much help having the workspace buried in the site-packages of an install.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions