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

Dataset_generator + tests #314

Merged
merged 37 commits into from
Jul 26, 2023
Merged

Dataset_generator + tests #314

merged 37 commits into from
Jul 26, 2023

Conversation

lizlouise1335
Copy link
Contributor

Dataset_generator + tests, plus a datetime generator bug fix

@lizlouise1335 lizlouise1335 marked this pull request as ready for review July 18, 2023 15:24
synthetic_data/dataset_generator.py Outdated Show resolved Hide resolved
synthetic_data/dataset_generator.py Outdated Show resolved Hide resolved
Copy link
Contributor

@taylorfturner taylorfturner left a comment

Choose a reason for hiding this comment

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

add assert

tests/test_dataset_generator.py Show resolved Hide resolved
@JGSweets JGSweets enabled auto-merge (squash) July 18, 2023 18:35
auto-merge was automatically disabled July 18, 2023 18:43

Head branch was pushed to by a user without write access

taylorfturner
taylorfturner previously approved these changes Jul 18, 2023
auto-merge was automatically disabled July 18, 2023 20:36

Head branch was pushed to by a user without write access

auto-merge was automatically disabled July 25, 2023 19:40

Head branch was pushed to by a user without write access

auto-merge was automatically disabled July 26, 2023 14:33

Head branch was pushed to by a user without write access

tests/test_dataset_generator.py Outdated Show resolved Hide resolved
tests/test_dataset_generator.py Outdated Show resolved Hide resolved
tests/test_dataset_generator.py Outdated Show resolved Hide resolved
tests/test_dataset_generator.py Outdated Show resolved Hide resolved
auto-merge was automatically disabled July 26, 2023 17:12

Head branch was pushed to by a user without write access

synthetic_data/dataset_generator.py Outdated Show resolved Hide resolved
tests/test_dataset_generator.py Show resolved Hide resolved
tests/test_dataset_generator.py Show resolved Hide resolved
auto-merge was automatically disabled July 26, 2023 18:24

Head branch was pushed to by a user without write access

),
np.array(
[
"010100011010001000001100000001111101111111110110001011101100001010000100101100001010101000100010101010101011000110010110110000011101001011101110011010000011111010001100000000110001100100000000000111001110110110101010110010010000101101110001000101010110101101001011101011110110100110100000111101010101",
Copy link
Contributor

Choose a reason for hiding this comment

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

we will be combining text + string in a subsequent PR

@taylorfturner
Copy link
Contributor

Let's remove to_csv and path along with combining text and string in a subsequent PR

@taylorfturner taylorfturner merged commit 71ee566 into capitalone:feature/simple-tabular-generator Jul 26, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants