[DNM] Align node previews with number format selected in preferences#13076
[DNM] Align node previews with number format selected in preferences#13076aparajit-pratap wants to merge 1 commit intoDynamoDS:masterfrom
Conversation
| return null; | ||
| } | ||
|
|
||
| internal static string PrecisionFormat { get; set; } = "f3"; |
There was a problem hiding this comment.
Same as default value in preferencesettings.numberformat.
src/DynamoCore/Models/DynamoModel.cs
Outdated
|
|
||
| private static void InitializePreferences(IPreferences preferences) | ||
| { | ||
| DynamoUnits.Display.PrecisionFormat = preferences.NumberFormat; |
There was a problem hiding this comment.
What happened to DynamoUnits.Display.PrecisionFormat?
There was a problem hiding this comment.
It's still there in the next line.
|
It looks good @aparajit-pratap - maybe we can add a simple test though using one of the customer graphs that prompted this change? |
|
Let's wait for a decision from @Amoursol, @Jingyi-Wen, and the rest of the team on the next steps for this. For this PR to be given the green light some of the questions that need answering/assumptions made are:
|
@aparajit-pratap thank you for putting this PR in! My answers as below:
|
My thoughts:
|
|
Regarding "We may need another Preference setting here that enables rounding under a certain threshold for users", I believe this is a bigger task that would need addressing at a core level that would affect all numeric results, certainly not in this PR. Until then I suppose we can either wait until those changes happen or first roll out this part where we are only using preferences for display purposes. @Amoursol @Jingyi-Wen any thoughts about the ☝️ suggestion? |
Looks good to me @aparajit-pratap - We can look to further rounding improvement later. |
|
Closing since this is now duplicating #13744 |
Purpose
See slack thread.
Declarations
Check these if you believe they are true
*.resxfilesRelease Notes
(FILL ME IN) Brief description of the fix / enhancement. Mandatory section
FYIs
@Amoursol