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

Can the editable install come with package data? i.e. pip install -e . #1611

Open
1 task done
noklam opened this issue Oct 26, 2023 · 9 comments
Open
1 task done

Can the editable install come with package data? i.e. pip install -e . #1611

noklam opened this issue Oct 26, 2023 · 9 comments

Comments

@noklam
Copy link
Contributor

noklam commented Oct 26, 2023

Description

Currently, when kedro-viz is installed locally, the assets such as html and css are missing. The contribution guide say that you need to run make build.

This requires the user has npm and make it harder to contribute particularly if one is interested in the Python bit but not the front-end.

Is is possible to make it work without a separate build step? Cc @astrojuanlu

I found some links in setuptools:

Checklist

  • Include labels so that we can categorise your feature request
@astrojuanlu
Copy link
Member

Yes it can be done: #1267 (comment)

@tynandebold tynandebold added Enhancement Python Pull requests that update Python code and removed Issue: Feature Request labels Oct 27, 2023
@tynandebold tynandebold moved this to Inbox in Kedro-Viz Oct 27, 2023
@ravi-kumar-pilla
Copy link
Contributor

@astrojuanlu Thank you for the comment. Kedro Viz sdist already contains the js build. But, this ticket is to include js build in local development I guess. We do not have our js build on Git and if someone wants to contribute only python code still needs the js build as the html folder is missing from the version control.

I do not see how the Manifest.in or the datafiles link you shared might help this usecase @noklam , as they are helpful in including the files which are already in the source tree. Please correct me if I am wrong.

To resolve this issue, we need to somehow include the html folder (js build) in the source tree (VCS), so that users can get the files for local development. Happy to hear any other way to do this. Thank you

@tynandebold tynandebold moved this from Inbox to Backlog in Kedro-Viz Nov 6, 2023
@rashidakanchwala rashidakanchwala added Type: Technical Design and removed Python Pull requests that update Python code labels Nov 9, 2023
@rashidakanchwala
Copy link
Contributor

Closing this issue as it cannot be done

@rashidakanchwala rashidakanchwala closed this as not planned Won't fix, can't repro, duplicate, stale Feb 26, 2024
@github-project-automation github-project-automation bot moved this from Backlog to Done in Kedro-Viz Feb 26, 2024
@rashidakanchwala
Copy link
Contributor

@astrojuanlu - Requested to reopen this issue. Maybe we can invite you to our next backlog grooming and u can give us more information on this and #1267

@astrojuanlu

This comment was marked as outdated.

@astrojuanlu
Copy link
Member

Now that the codebase has been migrated to Node.js 18 #1652 this should be easier to do.

@astrojuanlu

This comment was marked as resolved.

@astrojuanlu astrojuanlu mentioned this issue Feb 26, 2024
5 tasks
@astrojuanlu
Copy link
Member

Explanation (and fix) of #1267 in #1766

@antonymilne
Copy link
Contributor

This is something I've always wanted to have! So +💯 from me if it can be done 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

6 participants