Skip to content

Multiple targets for ping#2

Open
marvinlinthorst wants to merge 3 commits intoDoogeJ:masterfrom
marvinlinthorst:master
Open

Multiple targets for ping#2
marvinlinthorst wants to merge 3 commits intoDoogeJ:masterfrom
marvinlinthorst:master

Conversation

@marvinlinthorst
Copy link

No description provided.

@DoogeJ DoogeJ self-requested a review January 5, 2017 14:12
@DoogeJ DoogeJ self-assigned this Jan 5, 2017
@DoogeJ
Copy link
Owner

DoogeJ commented Jan 5, 2017

testimg
I see some issues with the timers for multiple pings at once, and the graph sometimes behaves weird. Also I would like to keep the customizable intervals per ping. This is possible, but would require stacked graphs I think, or multiple horizontal scales per ping.

Also the added listbox doesnt fit really well in the style yet, and doesn't have a right side anchor so it bugs when resizing. This should be an easy fix :)

The worst part so far is that force-stopping the process makes Windows crash in combination with some network drivers:
wp_20170105_15_14_43_pro

This only happens when using some versions of .NET 4.0 (it triggers a bug thats fixed in the preview versions already) but at least its not release-worthy yet until Microsoft fixes this.

Oh and thanks for renaming the form! Much better named as pingform ;-)

I'll work on this a bit more soon and see if I can make it releaseable!

@DoogeJ DoogeJ added this to the 2.0.0.0 milestone Jan 5, 2017
Copy link
Owner

@DoogeJ DoogeJ left a comment

Choose a reason for hiding this comment

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

See comments on post - will work on this soon(tm) tho!

Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants