Skip to content

Clarify derivatives.#3848

Merged
HansOlsson merged 4 commits intomodelica:masterfrom
HansOlsson:ClarifyDerivative
Mar 16, 2026
Merged

Clarify derivatives.#3848
HansOlsson merged 4 commits intomodelica:masterfrom
HansOlsson:ClarifyDerivative

Conversation

@HansOlsson
Copy link
Copy Markdown
Collaborator

Closes #3846

@HansOlsson HansOlsson requested review from AHaumer and henrikt-ma March 3, 2026 12:47
@HansOlsson HansOlsson added the clarification Specification of feature is unclear, but not incorrect label Mar 4, 2026
@HansOlsson
Copy link
Copy Markdown
Collaborator Author

Pure clarification, no actual change.

Comment thread chapters/functions.tex Outdated
Comment thread chapters/functions.tex Outdated
Comment thread chapters/functions.tex Outdated
Copy link
Copy Markdown
Collaborator

@henrikt-ma henrikt-ma left a comment

Choose a reason for hiding this comment

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

As a way to remedy the lack of a way to express infinite differentiability, perhaps the specification should recommend a specific big value to approximate infinity?!

HansOlsson and others added 2 commits March 10, 2026 09:56
Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>
Co-authored-by: Henrik Tidefelt <henrikt@wolfram.com>
Copy link
Copy Markdown
Collaborator

@henrikt-ma henrikt-ma left a comment

Choose a reason for hiding this comment

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

Looks good to me, but it would also be good to get an approval from @AHaumer.

Comment thread chapters/functions.tex
@HansOlsson
Copy link
Copy Markdown
Collaborator Author

As a way to remedy the lack of a way to express infinite differentiability, perhaps the specification should recommend a specific big value to approximate infinity?!

Good idea, I changed so that 100 is recommended. Basically it is easy to recognize (as any power of 10 would be), and I found 10 to be too small (I could imagine that someone thought it meant exactly 10 derivatives in some weird case), and 1000 excessive.

and pinging @AHaumer

@HansOlsson HansOlsson removed the request for review from AHaumer March 16, 2026 14:20
@HansOlsson HansOlsson merged commit ae31210 into modelica:master Mar 16, 2026
1 check passed
@HansOlsson HansOlsson deleted the ClarifyDerivative branch March 16, 2026 14:21
@HansOlsson HansOlsson added the M37 For pull requests merged into Modelica 3.7 label Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clarification Specification of feature is unclear, but not incorrect M37 For pull requests merged into Modelica 3.7

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarification of derivatives of functions

2 participants