Roger Barreto

Results 74 comments of Roger Barreto

@dluc, I agree, will follow your suggestions. Let's leave this PR parked for now, since there are some core changes that can be added as part of it. The current...

Closing for now as this approach will be revisited into a future PR

@DavidLuong98 this PR #952 and #1117 addresses the first and second requests on this Issue. Now you can get more detailed information about your requests using the `ModelResults` property of...

As this changes removes Conditional work all together, I suggest removing the `ConditionalFlowHelper`, `ConditionalFlowConstants` and the UnitTests related to it.

Same here, conflicting with Visual Studio inline shortcut. We got really a new stackoverflow with the disrupting issue. https://superuser.com/questions/1785700/ctrlspace-not-working-anywhere

@shawncal @stephentoub Since we are introducing this such great change my suggestion would to be way more radical and just obsolete the `SKFunctionContextParameter`, `SKFunctionInput`, `SKFunctionContextParameter` attributes altogether. Suggestion: Go straight...

@stephentoub, good point on the Xml part. I think we can move forward using the `SKFunctionContextParameter` for description with a smaller name like Shawn was suggesting then but that got...

@stephentoub I would first like to revisit my suggestions also with @shawncal and @dluc, I'm just thinking what would be my take, but would definitely await for @dluc and @shawncal...

> What's our plan for tracking / removing such temporary measures? Do we actually need them? @stephentoub The plan here is to keep those as extensions while we still use...

@feiyun0112 Thanks for the contribution, this issue is currently under analysis.