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

updated the entire app.py #185

Merged
merged 3 commits into from
Oct 26, 2024

Conversation

sanyadureja
Copy link
Contributor

@sanyadureja sanyadureja commented Oct 26, 2024

Description

  • In the "Why Choose Cal?" section, the content has been updated and presented in a structured card format, incorporating the new feature for Enhanced Document Analysis & Language Tools. The user interface has also been improved with the addition of icons and colors for better visual appeal.
  • For the "Available Calculators" section, the calculator information is now organized in a structured format, featuring an expander for detailed insights on each calculator. The UI has been further enhanced with vibrant colors, and calculators are displayed in a two-column table layout for improved readability.
  • Additionally, a "Get Started" section has been added at the bottom, along with a footer for extra information.

Issue Resolved

This PR resolves #128

Changes Made

  1. Why Choose Cal? Section
  • Content Update: Revised text to better highlight Cal's benefits.
  • Structured Card Format: Features displayed in a card layout for clarity.
  • New Feature Addition: Added Enhanced Document Analysis & Language Tools.
  • Enhanced UI: Improved visuals with icons and a new color scheme.
  1. Available Calculators Section
  • Structured Information: Organized calculator details for easier navigation.
  • Expander Feature: Implemented expanders for more info on each calculator.
  • UI Enhancements: Added vibrant colors for a more attractive look.
  • Table Layout: Displayed calculators in a two-column table for better comparison.
  1. Get Started Section
  • New Section: Added a "Get Started" section to guide users on initial steps.
  1. Footer and Additional Information
    Footer Addition: Included a footer with additional links and resources for user convenience.

Additional Details

If you would want me to do some more changes, please give your meaningful suggestions.

Checklist

  • [✅] My code follows the current project structure
  • [✅] I have thoroughly reviewed and updated the requirements.txt file to include any new packages
  • [✅] The predict.py file includes a properly implemented model_details() function, or the notebook contains this function to print a detailed model report. The model will not be accepted without this function, as it is essential for generating the necessary model details.
  • [✅ ] I have added relevant tests (if necessary).
  • [✅] I have added comments in the code where needed.
  • [✅] This PR is submitted under Hacktoberfest.
  • [✅] This PR is submitted under GirlScript Summer of Code (GSSoC-Extd).

@sanyadureja
Copy link
Contributor Author

@yashasvini121 I've completed the issue #128 and raised the PR, please review and give your suggestions (if any).

Looking forward to getting my PR merged. Most importantly add all the labels (GSSOC, Hacktoberfest and level).
Thank you

Copy link
Owner

@yashasvini121 yashasvini121 left a comment

Choose a reason for hiding this comment

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

Great work! See if you can fix the image; if not, let me know, and I'll merge the PR.

Just make the updates in the current branch and commit them—the PR will automatically reflect the changes.

App.py Outdated Show resolved Hide resolved
@sanyadureja
Copy link
Contributor Author

Great work! See if you can fix the image; if not, let me know, and I'll merge the PR.

Just make the updates in the current branch and commit them—the PR will automatically reflect the changes.

Thanks @yashasvini121.
I've few doubts, by current branch you mean update-app.py right? (branch I'm working in)?

@yashasvini121
Copy link
Owner

Yup

@sanyadureja
Copy link
Contributor Author

Yup

@yashasvini121 I have changed the image please check

@sanyadureja
Copy link
Contributor Author

@yashasvini121 I hope PR will be merged now, please do the needful. Thank you

@yashasvini121 yashasvini121 merged commit 33f86d8 into yashasvini121:master Oct 26, 2024
1 check passed
@sanyadureja
Copy link
Contributor Author

sanyadureja commented Oct 26, 2024

@yashasvini121 maybe you wanted some other image.

Can you create an issue related to that with the exact requirement, I'll try working on it.
What do you say?

Also, since I've made a lot of changes and changed the app py completely can't it be a level 2 issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Updating content on App (Home page) of the Application - Enhancing the Predictive Calc
2 participants