Implemented Error Handling and Data Validation in Python Files #201

[Anushka-Pote]

Description

Enhance the robustness of the Python files in the project by implementing comprehensive error handling and data validation mechanisms. This will ensure the application can handle unexpected scenarios gracefully and provide meaningful feedback to the user.

Related Issue

Fixes #201

Type of change

• Bug fix
• New feature

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented on my code, particularly in hard-to-understand areas
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[netlify]

✅ Deploy Preview for multiverse-dataverse ready!

Name	Link
🔨 Latest commit	7ef5871
🔍 Latest deploy log	https://app.netlify.com/sites/multiverse-dataverse/deploys/6720fa6ad784320008396999
😎 Deploy Preview	https://deploy-preview-213--multiverse-dataverse.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[github-actions]

🎉 Thank you for your contribution! Your pull request has been submitted successfully. A maintainer from Dataverse team will review it as soon as possible. We appreciate your support in making this project better

[jency1]

@Anushka-Pote Resolve all conflicts.

[Anushka-Pote]

Yeah, I got the conflicts, as there was another linear regression model merge made but, before making the PR I resolved all the the conflicts after sync on my local machine itself. You can see it as one of the 5 commits.
I don't think there should be any conflict left, but I will see it again

[jency1]

@multiverseweb

[Anushka-Pote]

Have I resolved the conflicts or any other change is expected?

[multiverseweb]

@Anushka-Pote I'm testing the PR, will get back to you soon.

[Anushka-Pote]

Heyy
Actually tomorrow is the last day for any PR merges to be allowed, so could you please tell me if there needs any changes here or is it fine??

[jency1]

Heyy Actually tomorrow is the last day for any PR merges to be allowed, so could you please tell me if there needs any changes here or is it fine??

@Anushka-Pote There are conflicts. So you need to resolve it.

[Anushka-Pote]

There were conflicts before i agree, but then I had resolved all
I don't know why this is happening again.

[multiverseweb]

Hey @Anushka-Pote, there are some problems in this PR:

• You have removed the variable z which counts the number of failed login attempts so the code is throwing an error in the terminal in that scenario
  

Add an error message for invalid login attempts. Leave the variable z as it is, I'll later add a feature that disables finance tracker for some time after 2 invalid login attempts

• The colors of pie chart are not same as those used in the line chart.

This can be solved by adding colors=colors[1:len(columns) + 1] attribute here

Kindly solve the stated problems to get the PR merged.

Note

I understand that I'm late in reviewing the code and its the last day of GSSoC, so I'll be merging the PR whether you resolve the stated problems timely or not. Will fix them myself otherwise.
Good luck!

[Anushka-Pote]

Hey I just saw the message right now, And I don't think it will be done by 7 pm to be honest
So how should I work it out?

[multiverseweb]

Hey @Anushka-Pote no problem. Just let me know if you can still work on this PR otherwise I can fix the problems too