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

[UX] Nematic functional has an attribute pitch, but corresponds to 2*Pi/pitch #77

Open
joshichaitanya3 opened this issue Jan 24, 2022 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@joshichaitanya3
Copy link
Collaborator

From Amine Missaoui:

"I wanted to calculate the director field for a simple case of an achiral 5CB. So I set the pitch to a large value (about 200 because the "lnem.pitch = inf" command does not work)."

The pitch attribute for the Nematic functional is confusing for the user, since it actually means 2*Pi / pitch (= q).

Don't know whether we want to keep pitch or introduce the q, or keep both.

@joshichaitanya3 joshichaitanya3 added the enhancement New feature or request label Jan 24, 2022
@softmattertheory
Copy link
Contributor

We can rename this to wavevector. @joshichaitanya3 or @ConduitDan please make the change and amend the examples (if any!)

@ConduitDan ConduitDan self-assigned this Feb 22, 2022
@ConduitDan
Copy link
Collaborator

I'll take this on in my upcoming round of fixes

@ConduitDan
Copy link
Collaborator

revisiting this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants