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.

Sign up for GitHub

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

Jump to bottom

dataset: move fraud_detection to solution & remove atomic_edge field #480

Merged
merged 2 commits into from
Feb 22, 2023
Merged

dataset: move fraud_detection to solution & remove atomic_edge field #480

merged 2 commits into from
Feb 22, 2023

Conversation

wey-gu
Copy link
Contributor

@wey-gu wey-gu commented Feb 22, 2023 •
edited
Loading

What type of PR is this?

  • bug
  • feature
  • enhancement

also fixed #479

Aligned with @slimshadylol0303 , fraud_detection dataset should be under solution category.

@wey-gu wey-gu changed the title dataset: move fraud_detection to solution dataset: move fraud_detection to solution & remove atomic_edge field Feb 22, 2023
hetao92
hetao92 approved these changes Feb 22, 2023
View reviewed changes
Copy link
Contributor

@hetao92 hetao92 left a comment

Choose a reason for hiding this comment

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

LGTM

@hetao92 hetao92 merged commit ad2050d into master Feb 22, 2023
@wey-gu wey-gu deleted the wey_dataset_fix branch February 22, 2023 09:54
nebula-bots pushed a commit that referenced this pull request Feb 22, 2023
@wey-gu
dataset: move fraud_detection to solution & remove atomic_edge field (#…
726f044 
…480)

* dataset: move fraud_detection to solution

* remove incompatibility field atomic_edge

fix: #479
@github-actions github-actions bot mentioned this pull request Feb 22, 2023
Merged
3 tasks
hetao92 pushed a commit that referenced this pull request Feb 22, 2023
@github-actions @wey-gu
dataset: move fraud_detection to solution & remove atomic_edge field (#…
debdd87 
…480) (#482)

* dataset: move fraud_detection to solution

* remove incompatibility field atomic_edge

fix: #479

Co-authored-by: Wey Gu <weyl.gu@gmail.com>
@wey-gu wey-gu mentioned this pull request Feb 25, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hetao92 hetao92 hetao92 approved these changes

@slimshadylol0303 slimshadylol0303 slimshadylol0303 approved these changes

Assignees
No one assigned
Labels
already-auto-picked-3.6 v3.6-cherry-pick
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[欢迎页]demo语句报错
3 participants
@wey-gu @hetao92 @slimshadylol0303