Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat(few-shot): add a few-shot example for #1 #2

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

Closed
ganler wants to merge 1 commit into enhance from enhance-more-fewshot

Conversation

@ganler
Copy link
Collaborator

@ganler ganler commented May 3, 2024

No description provided.

@ganler ganler requested a review from UniverseFly May 3, 2024 02:41
Comment on lines +708 to +709
[Instruction]
Write a Python function that converts altitude to temperature using the standard atmosphere model. The function should also adjust for variations in barometric pressure given by the parameter mmHg.
Copy link
Collaborator

@UniverseFly UniverseFly May 3, 2024

Choose a reason for hiding this comment

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

The instruction loses context. "the standard atmosphere model" is not explained and terms like "barometric pressure", "mmHg" are strange. Try to make it more declarative, logical, and understandable by humans, incorporating more unique coding features.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I thought The Standard Atmosphere was common knowledge: https://engineering.purdue.edu/~andrisan/Courses/AAE490A_S2002/Atmosphere.pdf

Or do you think more explanation is needed regardless of common knowledge? 🤔

[Property]
category: code generation (function implementation)
language: Python
concepts: mathematical modeling, environmental physics, parameter default values
Copy link
Collaborator

@UniverseFly UniverseFly May 3, 2024

Choose a reason for hiding this comment

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

mathematical modeling and environmental physics are too vague concepts. Try to relate to the logic like formula calculation / logarithm using..


[Response]
[Reasoning]
To compute the temperature at a given altitude, considering the barometric pressure, we can use the standard atmospheric model modified for non-standard pressures. The function altitude_to_temperature calculates the temperature at a specified altitude h, adjusting the altitude if the barometric pressure mmHg differs from the standard 29.92 inches of Hg. The temperature calculation depends on whether the altitude is above or below 36,089 feet.
Copy link
Collaborator

@UniverseFly UniverseFly May 3, 2024

Choose a reason for hiding this comment

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

29.92 cannot be derived from the instruction, so the response is not aligned

Copy link
Collaborator

We have a formula calculation example already, so let's pick a different instrucion

@ganler ganler closed this May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@UniverseFly UniverseFly UniverseFly left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /