Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [nebula-dev] Contributions to XYGraph

Hi,

I've added a bug for this change Christian, https://bugs.eclipse.org/bugs/show_bug.cgi?id=460536.

Wim, speaking for myself, it is quite likely that I'll be around this project in the foreseeable future so I'd be happy to help out as a committer.

Best regards,
Torkild


> 22. feb. 2015 kl. 04.12 skrev Xihui Chen <ztyaner11@xxxxxxxxx>:
> 
> Hi Guys,
> 
> I'm happy to see that more contributions are coming in for XYGraph! Unfortunately, I just changed my job and quite busy recently, so it is great if you guys can take the responsibility! I would recommend to do a format only change with the latest Eclipse default formatter to all the code before adding any feature. And then add all the features with the same formatter so it won't introduce any changes caused by formatting.
> 
> Cheers,
> Xihui
> 
> 
> 
> On Sat, Feb 21, 2015 at 11:53 AM, Christian Pontesegger <christian.pontesegger@xxxxxx> wrote:
> Perfect, so I will simply provide a patch reformatting the XY widget code.
> Is there any formatter you would suggest?
> 
> I see eclipse just recently changed the default formatter and increased line length to 120 or 140 (would have to look this up).
> 
> Agreed to use the default formatter with increased line length?
> 
> br
> Christian
> 
> 
> On 21.02.2015 20:21, Wim Jongman wrote:
>> Hi,
>> 
>> Sounds great guys!
>> 
>> Xihui recently indicated he would not mind some help maintaining and enhancing the visualization widgets.
>> 
>> I'm happy to start committer election for you two if you plan to put some longer term effort into this.
>> 
>> Cheers,
>> 
>> Wim
>> On Feb 21, 2015, at 19:46, "Torkild Ulvøy Resheim" <torkildr@xxxxxxxxx> wrote:
>> Hi Christian,
>> 
>> I'm not a Nebula committer, but we're consuming this widget and it's quite important to us so I'm taking this liberty. Please correct me if I'm wrong.
>> 
>> 
>>  21. feb. 2015 kl. 19.09 skrev Christian Pontesegger <christian.pontesegger@xxxxxx>:
>> 
>> 
>>  Hi,
>> 
>>  as part of the
>> eclipse.org/ease
>>  project we are working on a scripting module for charting. Due to the scripting nature it will allow to interactively create XY charts.
>> 
>>  Therefore we would like to contribute some extensions to the XYGraph widget.
>>  A first approach has been taken with [1], adding styling to chart series, allowing to customize the way
>> single points of a series are rendered.
>>  I see that this topic is already in review. Gerrit reports >2000 LOC here, which would mean we need a CQ for this. Still most changes are caused by a wrong code formatter.
>>  I tried to fix this with the Eclipse built in formatter, but it is also not compliant. What would be the suggested way to commit this change?
>> 
>> 
>> The XYGraph does not appear to have any project wide formatting settings, so using the formatter on this code would most likely result in undesirable changes. I suggest reverting those caused by the formatter.
>> 
>> 
>> 
>>  At a later stage we are planning to commit further improvements like an additional zoom mode, merging rubberband, vertical and horizontal zoom modes. Further we think of adding cursor support (not to be mixed up with markers, which are already
>> supported).
>> 
>> I'm looking forward to this :-)
>> 
>> 
>>  If there is no formatter available we might first create a changeset, simply reformatting affected code (using the Eclipse built-in style?) and provide our additions afterwards. This would help to avoid the CQ.
>> 
>> Yes, good idea.
>> [snip]
>> 
>> Best regards,
>> Torkild
>> 
>> 
>> 
>> _______________________________________________
>> nebula-dev mailing list
>> 
>> nebula-dev@xxxxxxxxxxx
>> 
>> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
>> 
>> https://dev.eclipse.org/mailman/listinfo/nebula-dev
> 
> 
> _______________________________________________
> nebula-dev mailing list
> nebula-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/nebula-dev
> 
> _______________________________________________
> nebula-dev mailing list
> nebula-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/nebula-dev

--
Torkild Ulvøy Resheim
Consultant / Eclipse Committer / Senior Software Developer
Itema AS - http://itema.no


Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail


Back to the top