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

Scipy curve fit #5919

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Scipy curve fit #5919

wants to merge 19 commits into from

Conversation

ca999
Copy link

@ca999 ca999 commented Jan 5, 2025 •
edited
Loading

Description

  1. Add a new entry in scipy.optimize
  2. Create a new directory: content/scipy/concepts/scipy-optimize/terms/curve-fit
  3. Added curve-fit documentation under curve-fit.md

Issue Solved

closes #5848

Type of Change

  • Adding a new entry

Checklist

  • All writings are my own. - the code example was taken from https://docs.scipy.org/doc/scipy/reference/generated/scipy.optimize.curve_fit.html with minor alterations. Chatgpt was used for better coherence for certain explanations.
  • My entry follows the Codecademy Docs style guide.
  • My changes generate no new warnings.
  • I have performed a self-review of my own writing and code.
  • I have checked my entry and corrected any misspellings.
  • I have made corresponding changes to the documentation if needed.
  • I have confirmed my changes are not being pushed from my forked main branch.
  • I have confirmed that I'm pushing from a new branch named after the changes I'm making.
  • I have linked any issues that are relevant to this PR in the Issues Solved section.

@CLAassistant
Copy link

CLAassistant commented Jan 5, 2025 •
edited
Loading

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
10 out of 11 committers have signed the CLA.

✅ ca999
✅ mamtawardhani
✅ Nil-Morah
✅ Maverick073
✅ vlrnsnk
✅ SaviDahegaonkar
✅ ebikatsudon
✅ jrood
✅ tommaso-berti
✅ hitomipupil
❌ codecademy-gh-api
You have signed the CLA already but the status is still pending? Let us recheck it.

@ca999 ca999 marked this pull request as ready for review January 6, 2025 01:58
@mamtawardhani mamtawardhani self-assigned this Jan 6, 2025
@mamtawardhani
Copy link
Collaborator

CLA assistant check Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.✅ ca999❌ Nayana

Nayana seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @ca999, looks like you have made a commit from a local ID as well, could you please use that to sign the CLA?

@mamtawardhani mamtawardhani added new entry New entry or entries scipy Python scipy enteries labels Jan 6, 2025
mamtawardhani
mamtawardhani approved these changes Jan 8, 2025
View reviewed changes
Copy link
Collaborator

@mamtawardhani mamtawardhani left a comment

Choose a reason for hiding this comment

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

Thank you for contributing to Codecademy Docs @ca999😄

The entry looks good for a second round of review! 🚀

content/scipy/concepts/scipy-optimize/terms/curve-fit/curve-fit.md Outdated Show resolved Hide resolved
@ca999
Copy link
Author

ca999 commented Jan 8, 2025 •
edited
Loading

Hi @mamtawardhani I am unable to get the CLA for the local account. Can you please guide me on how I can get the CLA agreement to the account

ca999 and others added 18 commits January 8, 2025 22:53
@ca999
Finished initial description
9612ccf
@ca999
syntax
d37462d
@ca999
complete
0a24e82
@codecademy-gh-api @ca999
🤖 update concept of the week
e53f07a
@ca999
spellings + code
d424f6f
@mamtawardhani @ca999
Update curve-fit.md
91d3f6f 
minor fixes
@jrood @ca999
Check to verify that there are not missing md files (Codecademy#5553)
edc6db9 
* Github check to verify that there are not missing md files

* linebreaks
@Nil-Morah @ca999
Volatile variables (Codecademy#5721)
c8466ca 
* volatile-variables file created and filled, examples pending

* Created volatile variable term for cpp variables

* File has been added.

* File has been modified.

* minor changes

* Fixes

---------
@tommaso-berti @ca999
feat: [Concept Entry] Git Checkout (Codecademy#5748) (Codecademy#5769)
878dfe4 
* feat: [Concept Entry] Git Checkout (Codecademy#5748)

* Update content/git/concepts/checkout/checkout.md

Co-authored-by: Pragati Verma <[email protected]>

* Update content/git/concepts/checkout/checkout.md

Co-authored-by: Pragati Verma <[email protected]>

* Update content/git/concepts/checkout/checkout.md

Co-authored-by: Pragati Verma <[email protected]>

* Update content/git/concepts/checkout/checkout.md

Co-authored-by: Pragati Verma <[email protected]>

* Update content/git/concepts/checkout/checkout.md

Co-authored-by: Pragati Verma <[email protected]>

* Update content/git/concepts/checkout/checkout.md

Co-authored-by: Pragati Verma <[email protected]>

* Update content/git/concepts/checkout/checkout.md

Co-authored-by: Pragati Verma <[email protected]>

* Apply suggestions from code review

Co-authored-by: Pragati Verma <[email protected]>

* feat: [Concept Entry] Git Checkout (Codecademy#5748): added optional flags

* Update options syntax

* Update checkout.md

* fix: [Bug/Error] Java Map: .replaceAll() (Codecademy#5748)

* revert wrong commit on feat branch

* Minor fixes

* Formatting fixes

---------
@jrood @ca999
Update post-merge copy (Codecademy#5922)
ce0fcdd
@vlrnsnk @ca999
Update json.md (Codecademy#5891)
95ee575 
Remove trailing commas on pseudocode JSON
@SaviDahegaonkar @ca999
[Term Entry] Python statsmodels: anova_lm() (Codecademy#5907)
cb821c7 
* New file has been added.

* Update user-input.md

* Update user-input.md

* File has been modified.

* Update anova-lm.md

major fixes in meta data and some content

* Update anova-lm.md

---------
@Maverick073 @ca999
Added docs for javascript spread operator - Codecademy#5802 (Codecade…
e624ab1 
…my#5827)

* Added docs for javascript spread operator

* Update spread-operator.md

fixes done

* Update spread-operator.md

* Update content/javascript/concepts/spread-operator/spread-operator.md

* Update content/javascript/concepts/spread-operator/spread-operator.md

* Update content/javascript/concepts/spread-operator/spread-operator.md

---------
@mamtawardhani @ca999
Update curve-fit.md
5ba20f4 
fixed formatting
@ebikatsudon @ca999
[Edit] Combine Duplicate Focus Group Entries (Codecademy#5829)
3a93d85 
* merged conflicting focus group entries into one

* updated to fix formatting errors

* Revert yarn.lock changes

* Revert yarn.lock changes

* Revert yarn.lock changes

* Revert yarn.lock changes

* Update focus-group.md

* Update focus-group.md

* Update focus-group.md

* Update focus-group.md

---------
@SaviDahegaonkar @ca999
[Concept Entry] Rust: Ownership (Codecademy#5880)
b95eba3 
* New file has been added.

* Update user-input.md

* Update user-input.md

* File has been modified.

* Update content/rust/concepts/ownership/ownwership.md

* Update content/rust/concepts/ownership/ownwership.md

* Update content/rust/concepts/ownership/ownwership.md

* Rename ownwership.md to ownership.md

fixed minor issues and corrected file name spelling

* Update content/rust/concepts/ownership/ownership.md

* Update ownership.md

* Update ownership.md

---------
@hitomipupil @ca999
[Concept Entry] JavaScript: Nullish Coalescing
73af5de 
* Initial commit

* removed extra information and modified grammar mistakes

* Update nullish-coalescing.md

minor fixes

* Minor changes

---------
@ca999
trying to commit with github email
53cb3ed
@ca999
Copy link
Author

ca999 commented Jan 8, 2025

Hi @mamtawardhani So I update the commit email using the ones related to this account, I think it did the job, I no longer see the local account, but I think all the other commiter also have been added in to the CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mamtawardhani mamtawardhani mamtawardhani approved these changes

Assignees

@mamtawardhani mamtawardhani

Labels
new entry New entry or entries scipy Python scipy enteries status: ready for next review status: review 1️⃣ completed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Term Entry] Python:SciPy scipy.optimize: curve_fit()