Skip to content

add links to course and book in userguide-selection - #704

Merged
solegalli merged 21 commits into
feature-engine:mainfrom
Bestine:bestine-branch
Oct 19, 2023
Merged

add links to course and book in userguide-selection#704
solegalli merged 21 commits into
feature-engine:mainfrom
Bestine:bestine-branch

Conversation

@Bestine

@Bestine Bestine commented Oct 18, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Oct 18, 2023

Copy link
Copy Markdown

Codecov Report

Merging #704 (9659028) into main (a819eb1) will not change coverage.
The diff coverage is n/a.

❗ Current head 9659028 differs from pull request most recent head ff0905a. Consider uploading reports for the commit ff0905a to get more accurate results

@@           Coverage Diff           @@
##             main     #704   +/-   ##
=======================================
  Coverage   98.03%   98.03%           
=======================================
  Files         100      100           
  Lines        3877     3877           
  Branches      761      761           
=======================================
  Hits         3801     3801           
  Misses         28       28           
  Partials       48       48           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@solegalli solegalli left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @Bestine

Thank you for the initial PR. It's almost there. Please see my comments below and make sure to include the links to the resources.

Thank you!

Comment thread docs/user_guide/selection/index.rst Outdated
- `Feature Selection book <https://leanpub.com/feature-selection-in-machine-learning/>`_
- `Train in data's blog <https://www.blog.trainindata.com/>`_

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove links from this file

All notebooks can be found in a `dedicated repository <https://github.com/feature-engine/feature-engine-examples>`_.

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please don't live so much space between lines

3 -0.011595 0.012191 0.024991 0.022688
4 -0.036385 0.003935 0.015596 -0.031988

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove indentation

In this Kaggle kernel we feature 3 different end-to-end machine learning pipelines using
:class:`DropFeatures()`:

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pleaase move these changes to after the kaggle notebook

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, fine. Am following

4 -0.036385 0.021872 0.003935 -0.031988

For more details about this and other feature selection methods check out these resources:
- `Feature selection for machine learning <https://www.trainindata.com/p/feature-selection-for-machine-learning>` _, online course.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @Bestine

Thank you for the modifications.

Small detail, there needs to be one line between the text and the bulletpoints, otherwise the documentation does not build properly. Please check how I put together text and bullets with links in the files you are modifying as inspiration.

Comment thread docs/user_guide/selection/DropConstantFeatures.rst
Comment thread docs/user_guide/selection/DropCorrelatedFeatures.rst
Comment thread docs/user_guide/selection/DropDuplicateFeatures.rst
Comment thread docs/user_guide/selection/DropFeatures.rst

All notebooks can be found in a `dedicated repository <https://github.com/feature-engine/feature-engine-examples>`_.

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:

- `Kaggle Tips for Feature Engineering and Selection <https://www.youtube.com/watch?v=RtqtM1UJfZc&t=3150s>`_, by Gilberto Titericz.
- `Feature Selection: Beyond feature importance? <https://www.kdnuggets.com/2019/10/feature-selection-beyond-feature-importance.html>`_, KDDNuggets.

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:

3 -0.011595 -0.036656 0.012191 0.022688
4 -0.036385 0.021872 0.003935 -0.031988

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:


- `Recursive feature elimination with Python <https://www.blog.trainindata.com/recursive-feature-elimination-with-python/>`_

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:

or grouping rare categories with the RareLabelEncoder if they are categorical. No newline at end of file
or grouping rare categories with the RareLabelEncoder if they are categorical.

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:

4 -0.036385 0.003935 0.015596 -0.031988


For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:


All notebooks can be found in a `dedicated repository <https://github.com/feature-engine/feature-engine-examples>`_.

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:


All notebooks can be found in a `dedicated repository <https://github.com/feature-engine/feature-engine-examples>`_.

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:


All notebooks can be found in a `dedicated repository <https://github.com/feature-engine/feature-engine-examples>`_.

For more details about this and other feature selection methods check out these resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For more details about this and other feature selection methods check out these resources:
For more details about this and other feature selection methods check out these resources:

@solegalli solegalli left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Bestine

If you look at how the file it's rendered, you'll see that the links that you added show the word and the URL, whereas the links that are in the file, like for examplee Kaggle kernel, do not show the url, instead they appear linked. I think the problem is that there is a space between the final ` and the _. Could you please check?

Additional Resources
--------------------

More details about feature selection can be found in the following resources:

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These lines were not the ones you added, they were there already. Could you please add them back?

@solegalli solegalli changed the title More details about further learning to .rst files add links to course and book in userguide-selection Oct 19, 2023
@solegalli
solegalli merged commit 073a96a into feature-engine:main Oct 19, 2023
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