oneapi-src / credit-card-fraud-detection

AI Starter Kit for Credit Card Fraud Detection model using Intel® Extension for Scikit-learn*
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

Update ReadMe #6

Closed vpmodak closed 1 year ago

vpmodak commented 1 year ago

Add details on downsampling post clustering and inference dataset size

vpmodak commented 1 year ago

Hello,

Thanks so much for your response. Will keep this in mind. It was a quick change so, I decided to make it to the main.

Do you want me to delete my fork or we can let it stay there for future edits/PRs? I will make sure those happen on a branch and not the main 😊

Thanks, Viraj

From: Abolfazl Shahbazi @.> Sent: Tuesday, November 22, 2022 9:51 AM To: oneapi-src/credit-card-fraud-detection @.> Cc: Modak, Viraj @.>; Mention @.> Subject: [External] Re: [oneapi-src/credit-card-fraud-detection] Update ReadMe (PR #6)

This message is from an EXTERNAL SENDER - be CAUTIOUS, particularly with links and attachments.


@ashahba approved this pull request.

Hi @vpmodakhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vpmodak&d=DwMFaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=EzPG2shOI35tmQBFzgeeNCLGFyBHqcj-Ldjt9jRC9SE&m=z_YwqqAJ9Cr1aWwLjHGUL2qo4_aeJIlYIQDGX8gbdbGBsLzYehGPV0ciNTMgjOG0&s=DWCsxJE0pStve-mpSfgukkAUJ5RR8Vnc33L8XNAgcXE&e= Thanks for the PR.

My only comment is that usually PRs require to be on a branch other than your main branch, this is because after merge, we usually require the PR owner to delete their branch and that means deleting your main branch.

Feel free to let me know if you need any help with your fork once we merge this to bring it back to normal 😄

LGTM!

— Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_oneapi-2Dsrc_credit-2Dcard-2Dfraud-2Ddetection_pull_6-23pullrequestreview-2D1190537615&d=DwMFaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=EzPG2shOI35tmQBFzgeeNCLGFyBHqcj-Ldjt9jRC9SE&m=z_YwqqAJ9Cr1aWwLjHGUL2qo4_aeJIlYIQDGX8gbdbGBsLzYehGPV0ciNTMgjOG0&s=qJJ918664CK44Bx97_WM88RJd7t8JDrMWCA9huNVzFo&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AXRPUTMXXB3RNNDPQLQ2C5LWJUBXZANCNFSM6AAAAAASFADIFQ&d=DwMFaQ&c=eIGjsITfXP_y-DLLX0uEHXJvU8nOHrUK8IrwNKOtkVU&r=EzPG2shOI35tmQBFzgeeNCLGFyBHqcj-Ldjt9jRC9SE&m=z_YwqqAJ9Cr1aWwLjHGUL2qo4_aeJIlYIQDGX8gbdbGBsLzYehGPV0ciNTMgjOG0&s=tcx5yy_KL4yt0D6x30cMaycniIvYJfwAag9d6GmjigQ&e=. You are receiving this because you were mentioned.Message ID: @.**@.>>


This message is for the designated recipient only and may contain privileged, proprietary, or otherwise confidential information. If you have received it in error, please notify the sender immediately and delete the original. Any other use of the e-mail by you is prohibited. Where allowed by local law, electronic communications with Accenture and its affiliates, including e-mail and instant messaging (including content), may be scanned by our systems for the purposes of information security and assessment of internal compliance with Accenture policy. Your privacy is important to us. Accenture uses your personal data only in compliance with data protection laws. For further information on how Accenture processes your personal data, please see our privacy statement at https://www.accenture.com/us-en/privacy-policy.


www.accenture.com