ContriHUB / OneML_ContriHub

0 stars 11 forks source link

Render the "Visualizing the error(Low Penalty)" in SVM using plotly #27

Closed sidhantagar closed 3 years ago

sidhantagar commented 3 years ago

Rewrite code to render the "Visualizing the error(Low Penalty)" in SVM using plotly.

Refer to expectedoutput2.html in the expected outputs folder for the desired output.

In order to contribute to this issue, refer to the additional contributing guide here

MOULIK-RAZDAN commented 3 years ago

Please Assign.

MOULIK-RAZDAN commented 3 years ago

I am having a doubt

that in matplotlib graph the error values are from 200 to 29

but in expected outputs2.html it is from 146 to 26

so in plotly graph do i need to make it from 146 to 26

sidhantagar commented 3 years ago

Hi Moulik, actually the starting value is random since a seed has not been set hence before the definition of the SVM model add the code line np.random.seed(42) now the graph will always start from a value of 147

MOULIK-RAZDAN commented 3 years ago

Hello Sir, Sorry for the late reply Sir after adding the code line np.random.seed(42) it is giving the same output.

On Tue, Oct 12, 2021 at 9:03 PM Sidhant Agarwal @.***> wrote:

Hi Moulik, actually the starting value is random since a seed has not been set hence before the definition of the SVM model add the code line np.random.seed(42) now the graph will always start from a value of 147

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/ContriHUB/OneML_ContriHub/issues/27#issuecomment-941125939, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOIIKMFXN544MTV3ZMFTW3DUGRILZANCNFSM5FSGIUIA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

sidhantagar commented 3 years ago

Please host your notebook on so that I can review the code changes before merging

MOULIK-RAZDAN commented 3 years ago

Sir,actually don't add the current pull request because I had earlier made changes for Kmeans instead of SVM.

On Tue, Oct 12, 2021 at 10:52 PM Sidhant Agarwal @.***> wrote:

Please host your notebook on so that I can review the code changes before merging

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/ContriHUB/OneML_ContriHub/issues/27#issuecomment-941214776, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOIIKMDUQ3OW6VKH4HT3BC3UGRVEPANCNFSM5FSGIUIA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

sidhantagar commented 3 years ago

Okay open a new PR then

MOULIK-RAZDAN commented 3 years ago

Sir , I made a new PR

MOULIK-RAZDAN commented 3 years ago

Sir I have made a new PR sir this is the link https://github.com/ContriHUB/OneML_ContriHub/pull/53

On Wed, Oct 13, 2021 at 12:20 PM Sidhant Agarwal @.***> wrote:

Okay open a new PR then

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/ContriHUB/OneML_ContriHub/issues/27#issuecomment-941976578, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOIIKMDW7RH57UJGGL7TFUDUGUT2VANCNFSM5FSGIUIA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

sidhantagar commented 3 years ago

Hi moulik could you ping me on teams? I cannot find the code changes made by you to the notebook