Skip to content

Pypi Package#7

Merged
AnanyaRaval merged 47 commits intomainfrom
feature/pypi
Apr 13, 2026
Merged

Pypi Package#7
AnanyaRaval merged 47 commits intomainfrom
feature/pypi

Conversation

@AnanyaRaval
Copy link
Copy Markdown
Collaborator

@AnanyaRaval AnanyaRaval commented Feb 6, 2026

This PR contains changes required for a PyPI package:

  • Fix ruff and lint errors.
  • Add docstrings in numpy format.
  • Add docs.yml workflow file.
  • Add mkdocs documentation for the package API.

@AnanyaRaval AnanyaRaval changed the title Remove extra utils folder Pypi Package Feb 6, 2026
@AnanyaRaval AnanyaRaval marked this pull request as ready for review March 2, 2026 23:32
@AnanyaRaval
Copy link
Copy Markdown
Collaborator Author

Comments by @dhanesh-vector :

  1. Does the Repository structure in README.md reflect the updated contents ?
  2. It seems the /blog directory is part of the commit. I suppose it is required to render the blog page.
  3. If someone want to use this pypi package for their own work, what needs to be changed or adapted ? It should be the datasets file. But I don't see any instructions on the README that reflects this.  perhaps adding a subsection:
           "Using the model on your own dataset" would help clarify. Essentially ensuring that the specific feature columns are properly preprocessed (like normalization, taking care of missing data etc) and returning this
    return (
    x[valid],
    t[valid],
    event_type[valid],
    feature_names,
    n_continuous,
    feature_ranges,
    )

@AnanyaRaval AnanyaRaval requested a review from amrit110 April 6, 2026 22:07
@AnanyaRaval AnanyaRaval merged commit 190d65a into main Apr 13, 2026
5 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.

2 participants