This repository has been archived by the owner on Jan 9, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add issue and pull request templates and update documentation
- Loading branch information
1 parent
36ee020
commit 5cc9025
Showing
3 changed files
with
77 additions
and
39 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,57 @@ | ||
#### Description | ||
<!-- | ||
Example: DropFeatures fails on categorical features when assessing a string column | ||
--> | ||
|
||
#### Steps/Code to Reproduce | ||
<!-- | ||
Please add the minimum code required to reproduce the issue if possible. | ||
Example: | ||
```python | ||
import uuid | ||
import numpy as np | ||
import pandas as pd | ||
from foreshadow.preprocessor import Preprocessor | ||
cat1 = [str(uuid.uuid4()) for _ in range(40)] | ||
cat2 = [str(uuid.uuid4()) for _ in range(40)] | ||
input = pd.DataFrame({ | ||
'col1': np.random.choice(cat1, 1000), | ||
'col2': np.random.choice(cat2, 1000) | ||
}) | ||
processor = Preprocessor() | ||
output = processor.fit_transform(input) | ||
``` | ||
If the code is too long, feel free to put it in a public gist and link it in the issue: https://gist.github.com | ||
--> | ||
|
||
#### Expected Results | ||
<!-- | ||
Please add the results that you would expect here. | ||
Example: Error should not be thrown | ||
--> | ||
|
||
#### Actual Results | ||
<!-- | ||
Please place the full traceback here, again use a gist if you feel that it is too long. | ||
--> | ||
|
||
#### Versions | ||
<!-- | ||
Please run the following snippet in your environment and paste the results here. | ||
```python | ||
import platform; print(platform.platform()) | ||
import sys; print("Python", sys.version) | ||
import numpy; print("NumPy", numpy.__version__) | ||
import sklearn; print("Scikit-Learn", sklearn.__version__) | ||
import pandas; print("Pandas", pandas.__version__) | ||
import foreshadow; print("Foreshadow", foreshadow.__version__) | ||
from foreshadow.utils import check_transformer_imports; check_transformer_imports() | ||
``` | ||
--> | ||
|
||
|
||
<!--Thank you for contributing!--> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
<!-- | ||
Thanks you for taking the time to submit a pull request! Please take a look at some guideliens before submitting a pull request: https://github.com/georgianpartners/foreshadow/blob/development/doc/contrib.rst | ||
--> | ||
|
||
### Related Issue | ||
<!-- | ||
Example: Fixes #7. See also #35. | ||
Please use keywords (e.g., Fixes) to create link to the issues or pull requests | ||
you resolved, so that they will automatically be closed when your pull request | ||
is merged. See https://github.com/blog/1506-closing-issues-via-pull-requests | ||
--> | ||
|
||
### Description | ||
<!-- | ||
Please add a narrative description of your the changes made and the rationale behind them. If making a enhancement include the motivation and use cases addressed. | ||
--> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters