thoth-station / core

Using Artificial Intelligence to analyse and recommend Software Stacks for Artificial Intelligence applications.
https://thoth-station.github.io/
GNU General Public License v3.0
28 stars 25 forks source link

[3pt] Document Thoth's requirement for dependencies to be in an index #411

Open codificat opened 2 years ago

codificat commented 2 years ago

Problem statement

As a Thoth user, I am frustrated because Thoth can not provide advise about my application software stack, because my Pipfile references modules that are not hosted in a package index (local modules, file / git references).

For example:

And if I run thamos advise and my Pipfile contains an editable install:

$ thamos advise
2022-05-19 13:50:37,067 [163181] WARNING  thamos: Development dependencies will not be considered during the resolution process - see https://thoth-station.ninja/j/no_dev
2022-05-19 13:50:37,067 [163181] INFO     thamos.lib: Using Pipenv files to manage dependencies located in '/home/pep/github/os-climate/aicoe-osc-demo'
2022-05-19 13:50:37,067 [163181] INFO     thamos.lib: Submitting Pipfile.lock as a base for user's stack scoring - see https://thoth-station.ninja/j/user_stack
2022-05-19 13:50:37,068 [163181] ERROR    thamos: Failed to load Pipfile (path: /home/pep/github/os-climate/aicoe-osc-demo/Pipfile: Package 'src' is editable local project instead of a package from a package index

and it is unclear to me why this is an ERROR and how to fix it.

High-level Goals

Documentation exists, clearly reachable within the Thoth documentation, that explains:

Proposal description

Alternatives

Additional context

A blog post was published one year ago about this topic: https://developers.redhat.com/articles/2021/05/26/can-we-consider-editable-bad-practice

However, this is not part of Thoth's documentation.

Acceptance Criteria

Documentation in the Thoth website covers:

And based on that documentation:

codificat commented 2 years ago

/sig user-experience /triage accepted /priority important-soon

codificat commented 2 years ago

Given that the content of the documentation is heavily tied to the guidance service, this is /sig stack-guidance more than /remove-sig user-experience

codificat commented 2 years ago

/assign

codificat commented 2 years ago

A (relatively) related request is https://github.com/thoth-station/adviser/issues/2275.

sesheta commented 2 years ago

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

/lifecycle stale

codificat commented 2 years ago

/wg cnbi

codificat commented 2 years ago

/remove-lifecycle stale

codificat commented 2 years ago

This is delayed waiting for input from the alternative approaches and how they apply to a real-life use case, e.g. https://github.com/thoth-station/opendatahub-cnbi/issues/14