[WIP] Update ProposeSweeps to have unique entries by jwood803 · Pull Request #358 · dotnet/machinelearning

jwood803 · 2018-06-14T10:52:49Z

Proposed fix for issue #215.

Updated the ProposeSweeps virtual and override methods to not add to the collection if it already exists to prevent duplicates.

Put this as a work in progress PR since I'm sure I'll need some guidance as to if this solution is even a good one and how it can be further improved.

If anything else is needed, I'll be glad to update this PR.

sfilipi · 2018-06-15T15:14:39Z

Thanks for helping @jwood803. Taking a look now.

sfilipi · 2018-06-15T15:18:11Z

 }
- result.Add(_cache[iPerm]);
+
+ if(!_cache.Contains(_cache[iPerm]))


_cache [](start = 20, length = 6)
did you mean to check if result Contains it? #Closed

Oh, whoops! I think you are correct. I'll update that.

Also, this would avoid returning duplicates, which is good, but we'd still have duplicates stored in the _cache. Might be just good practice to avoid storing the result directly in the _cache
In reply to: 195774135 [](ancestors = 195774135)

This is probably way off, but would something like this work when initializing the _cache?
_cache = (new ParameterSet[_nGridPoints]).Distinct().ToArray(); ``` #Closed

probably not, because the values are generated later, by calling CreateParamSet.
In reply to: 195781506 [](ancestors = 195781506)

Good point! I'll just update to add only if it's contained in the cache.

sfilipi · 2018-06-15T16:11:07Z

There are a few other sweepers, inside the Algorithms folder, that override ProposeSweeps.
Taking a look to see whether they need the same filtering, and I'll update the comment. Probably they do.

In reply to: 397653603 [](ancestors = 397653603)

jwood803 · 2018-06-18T21:09:17Z

Took a shot at updating the KdoSweeper. The others I may need some guidance with as to what the best approach would be, though.

jwood803 · 2018-06-18T21:23:53Z

Whoops! Looks like rebasing closed this. 😅

Probably for the best since #368 looks like a better fix. #Closed

sfilipi · 2018-06-18T21:38:56Z

Thank you for your contribution!

In reply to: 398200508 [](ancestors = 398200508)

jwood803 · 2018-06-14T10:52:49Z

Proposed fix for issue #215.

Updated the ProposeSweeps virtual and override methods to not add to the collection if it already exists to prevent duplicates.

Put this as a work in progress PR since I'm sure I'll need some guidance as to if this solution is even a good one and how it can be further improved.

If anything else is needed, I'll be glad to update this PR.

sfilipi · 2018-06-15T15:14:39Z

Thanks for helping @jwood803. Taking a look now.

sfilipi · 2018-06-15T15:18:11Z

 }
- result.Add(_cache[iPerm]);
+
+ if(!_cache.Contains(_cache[iPerm]))


_cache [](start = 20, length = 6)
did you mean to check if result Contains it? #Closed

Oh, whoops! I think you are correct. I'll update that.

Also, this would avoid returning duplicates, which is good, but we'd still have duplicates stored in the _cache. Might be just good practice to avoid storing the result directly in the _cache
In reply to: 195774135 [](ancestors = 195774135)

This is probably way off, but would something like this work when initializing the _cache?
_cache = (new ParameterSet[_nGridPoints]).Distinct().ToArray(); ``` #Closed

probably not, because the values are generated later, by calling CreateParamSet.
In reply to: 195781506 [](ancestors = 195781506)

Good point! I'll just update to add only if it's contained in the cache.

sfilipi · 2018-06-15T16:11:07Z

There are a few other sweepers, inside the Algorithms folder, that override ProposeSweeps.
Taking a look to see whether they need the same filtering, and I'll update the comment. Probably they do.

In reply to: 397653603 [](ancestors = 397653603)

jwood803 · 2018-06-18T21:09:17Z

Took a shot at updating the KdoSweeper. The others I may need some guidance with as to what the best approach would be, though.

jwood803 · 2018-06-18T21:23:53Z

Whoops! Looks like rebasing closed this. 😅

Probably for the best since #368 looks like a better fix. #Closed

sfilipi · 2018-06-18T21:38:56Z

Thank you for your contribution!

In reply to: 398200508 [](ancestors = 398200508)

jwood803 · 2018-06-14T10:52:49Z

Proposed fix for issue #215.

Updated the ProposeSweeps virtual and override methods to not add to the collection if it already exists to prevent duplicates.

Put this as a work in progress PR since I'm sure I'll need some guidance as to if this solution is even a good one and how it can be further improved.

If anything else is needed, I'll be glad to update this PR.

sfilipi · 2018-06-15T15:14:39Z

Thanks for helping @jwood803. Taking a look now.

sfilipi · 2018-06-15T15:18:11Z

 }
- result.Add(_cache[iPerm]);
+
+ if(!_cache.Contains(_cache[iPerm]))


_cache [](start = 20, length = 6)
did you mean to check if result Contains it? #Closed

Oh, whoops! I think you are correct. I'll update that.

Also, this would avoid returning duplicates, which is good, but we'd still have duplicates stored in the _cache. Might be just good practice to avoid storing the result directly in the _cache
In reply to: 195774135 [](ancestors = 195774135)

This is probably way off, but would something like this work when initializing the _cache?
_cache = (new ParameterSet[_nGridPoints]).Distinct().ToArray(); ``` #Closed

probably not, because the values are generated later, by calling CreateParamSet.
In reply to: 195781506 [](ancestors = 195781506)

Good point! I'll just update to add only if it's contained in the cache.

sfilipi · 2018-06-15T16:11:07Z

There are a few other sweepers, inside the Algorithms folder, that override ProposeSweeps.
Taking a look to see whether they need the same filtering, and I'll update the comment. Probably they do.

In reply to: 397653603 [](ancestors = 397653603)

jwood803 · 2018-06-18T21:09:17Z

Took a shot at updating the KdoSweeper. The others I may need some guidance with as to what the best approach would be, though.

jwood803 · 2018-06-18T21:23:53Z

Whoops! Looks like rebasing closed this. 😅

Probably for the best since #368 looks like a better fix. #Closed

sfilipi · 2018-06-18T21:38:56Z

Thank you for your contribution!

In reply to: 398200508 [](ancestors = 398200508)

jwood803 · 2018-06-14T10:52:49Z

Proposed fix for issue #215.

Updated the ProposeSweeps virtual and override methods to not add to the collection if it already exists to prevent duplicates.

Put this as a work in progress PR since I'm sure I'll need some guidance as to if this solution is even a good one and how it can be further improved.

If anything else is needed, I'll be glad to update this PR.

sfilipi · 2018-06-15T15:14:39Z

Thanks for helping @jwood803. Taking a look now.

sfilipi · 2018-06-15T15:18:11Z

 }
- result.Add(_cache[iPerm]);
+
+ if(!_cache.Contains(_cache[iPerm]))


_cache [](start = 20, length = 6)
did you mean to check if result Contains it? #Closed

Oh, whoops! I think you are correct. I'll update that.

Also, this would avoid returning duplicates, which is good, but we'd still have duplicates stored in the _cache. Might be just good practice to avoid storing the result directly in the _cache
In reply to: 195774135 [](ancestors = 195774135)

This is probably way off, but would something like this work when initializing the _cache?
_cache = (new ParameterSet[_nGridPoints]).Distinct().ToArray(); ``` #Closed

probably not, because the values are generated later, by calling CreateParamSet.
In reply to: 195781506 [](ancestors = 195781506)

Good point! I'll just update to add only if it's contained in the cache.

sfilipi · 2018-06-15T16:11:07Z

There are a few other sweepers, inside the Algorithms folder, that override ProposeSweeps.
Taking a look to see whether they need the same filtering, and I'll update the comment. Probably they do.

In reply to: 397653603 [](ancestors = 397653603)

jwood803 · 2018-06-18T21:09:17Z

Took a shot at updating the KdoSweeper. The others I may need some guidance with as to what the best approach would be, though.

jwood803 · 2018-06-18T21:23:53Z

Whoops! Looks like rebasing closed this. 😅

Probably for the best since #368 looks like a better fix. #Closed

sfilipi · 2018-06-18T21:38:56Z

Thank you for your contribution!

In reply to: 398200508 [](ancestors = 398200508)

jwood803 · 2018-06-14T10:52:49Z

Proposed fix for issue #215.

Updated the ProposeSweeps virtual and override methods to not add to the collection if it already exists to prevent duplicates.

Put this as a work in progress PR since I'm sure I'll need some guidance as to if this solution is even a good one and how it can be further improved.

If anything else is needed, I'll be glad to update this PR.

sfilipi · 2018-06-15T15:14:39Z

Thanks for helping @jwood803. Taking a look now.

sfilipi · 2018-06-15T15:18:11Z

 }
- result.Add(_cache[iPerm]);
+
+ if(!_cache.Contains(_cache[iPerm]))


_cache [](start = 20, length = 6)
did you mean to check if result Contains it? #Closed

Oh, whoops! I think you are correct. I'll update that.

Also, this would avoid returning duplicates, which is good, but we'd still have duplicates stored in the _cache. Might be just good practice to avoid storing the result directly in the _cache
In reply to: 195774135 [](ancestors = 195774135)

This is probably way off, but would something like this work when initializing the _cache?
_cache = (new ParameterSet[_nGridPoints]).Distinct().ToArray(); ``` #Closed

probably not, because the values are generated later, by calling CreateParamSet.
In reply to: 195781506 [](ancestors = 195781506)

Good point! I'll just update to add only if it's contained in the cache.

sfilipi · 2018-06-15T16:11:07Z

There are a few other sweepers, inside the Algorithms folder, that override ProposeSweeps.
Taking a look to see whether they need the same filtering, and I'll update the comment. Probably they do.

In reply to: 397653603 [](ancestors = 397653603)

jwood803 · 2018-06-18T21:09:17Z

Took a shot at updating the KdoSweeper. The others I may need some guidance with as to what the best approach would be, though.

jwood803 · 2018-06-18T21:23:53Z

Whoops! Looks like rebasing closed this. 😅

Probably for the best since #368 looks like a better fix. #Closed

sfilipi · 2018-06-18T21:38:56Z

Thank you for your contribution!

In reply to: 398200508 [](ancestors = 398200508)

jwood803 · 2018-06-14T10:52:49Z

Proposed fix for issue #215.

Updated the ProposeSweeps virtual and override methods to not add to the collection if it already exists to prevent duplicates.

Put this as a work in progress PR since I'm sure I'll need some guidance as to if this solution is even a good one and how it can be further improved.

If anything else is needed, I'll be glad to update this PR.

sfilipi · 2018-06-15T15:14:39Z

Thanks for helping @jwood803. Taking a look now.

sfilipi · 2018-06-15T15:18:11Z

 }
- result.Add(_cache[iPerm]);
+
+ if(!_cache.Contains(_cache[iPerm]))


_cache [](start = 20, length = 6)
did you mean to check if result Contains it? #Closed

Oh, whoops! I think you are correct. I'll update that.

Also, this would avoid returning duplicates, which is good, but we'd still have duplicates stored in the _cache. Might be just good practice to avoid storing the result directly in the _cache
In reply to: 195774135 [](ancestors = 195774135)

This is probably way off, but would something like this work when initializing the _cache?
_cache = (new ParameterSet[_nGridPoints]).Distinct().ToArray(); ``` #Closed

probably not, because the values are generated later, by calling CreateParamSet.
In reply to: 195781506 [](ancestors = 195781506)

Good point! I'll just update to add only if it's contained in the cache.

sfilipi · 2018-06-15T16:11:07Z

There are a few other sweepers, inside the Algorithms folder, that override ProposeSweeps.
Taking a look to see whether they need the same filtering, and I'll update the comment. Probably they do.

In reply to: 397653603 [](ancestors = 397653603)

jwood803 · 2018-06-18T21:09:17Z

Took a shot at updating the KdoSweeper. The others I may need some guidance with as to what the best approach would be, though.

jwood803 · 2018-06-18T21:23:53Z

Whoops! Looks like rebasing closed this. 😅

Probably for the best since #368 looks like a better fix. #Closed

sfilipi · 2018-06-18T21:38:56Z

Thank you for your contribution!

In reply to: 398200508 [](ancestors = 398200508)

jwood803 · 2018-06-14T10:52:49Z

Proposed fix for issue #215.

Updated the ProposeSweeps virtual and override methods to not add to the collection if it already exists to prevent duplicates.

Put this as a work in progress PR since I'm sure I'll need some guidance as to if this solution is even a good one and how it can be further improved.

If anything else is needed, I'll be glad to update this PR.

sfilipi · 2018-06-15T15:14:39Z

Thanks for helping @jwood803. Taking a look now.

sfilipi · 2018-06-15T15:18:11Z

 }
- result.Add(_cache[iPerm]);
+
+ if(!_cache.Contains(_cache[iPerm]))


_cache [](start = 20, length = 6)
did you mean to check if result Contains it? #Closed

Oh, whoops! I think you are correct. I'll update that.

Also, this would avoid returning duplicates, which is good, but we'd still have duplicates stored in the _cache. Might be just good practice to avoid storing the result directly in the _cache
In reply to: 195774135 [](ancestors = 195774135)

This is probably way off, but would something like this work when initializing the _cache?
_cache = (new ParameterSet[_nGridPoints]).Distinct().ToArray(); ``` #Closed

probably not, because the values are generated later, by calling CreateParamSet.
In reply to: 195781506 [](ancestors = 195781506)

Good point! I'll just update to add only if it's contained in the cache.

sfilipi · 2018-06-15T16:11:07Z

There are a few other sweepers, inside the Algorithms folder, that override ProposeSweeps.
Taking a look to see whether they need the same filtering, and I'll update the comment. Probably they do.

In reply to: 397653603 [](ancestors = 397653603)

jwood803 · 2018-06-18T21:09:17Z

Took a shot at updating the KdoSweeper. The others I may need some guidance with as to what the best approach would be, though.

jwood803 · 2018-06-18T21:23:53Z

Whoops! Looks like rebasing closed this. 😅

Probably for the best since #368 looks like a better fix. #Closed

sfilipi · 2018-06-18T21:38:56Z

Thank you for your contribution!

In reply to: 398200508 [](ancestors = 398200508)

jwood803 · 2018-06-14T10:52:49Z

Proposed fix for issue #215.

Updated the ProposeSweeps virtual and override methods to not add to the collection if it already exists to prevent duplicates.

Put this as a work in progress PR since I'm sure I'll need some guidance as to if this solution is even a good one and how it can be further improved.

If anything else is needed, I'll be glad to update this PR.

sfilipi · 2018-06-15T15:14:39Z

Thanks for helping @jwood803. Taking a look now.

sfilipi · 2018-06-15T15:18:11Z

 }
- result.Add(_cache[iPerm]);
+
+ if(!_cache.Contains(_cache[iPerm]))


_cache [](start = 20, length = 6)
did you mean to check if result Contains it? #Closed

Oh, whoops! I think you are correct. I'll update that.

Also, this would avoid returning duplicates, which is good, but we'd still have duplicates stored in the _cache. Might be just good practice to avoid storing the result directly in the _cache
In reply to: 195774135 [](ancestors = 195774135)

This is probably way off, but would something like this work when initializing the _cache?
_cache = (new ParameterSet[_nGridPoints]).Distinct().ToArray(); ``` #Closed

probably not, because the values are generated later, by calling CreateParamSet.
In reply to: 195781506 [](ancestors = 195781506)

Good point! I'll just update to add only if it's contained in the cache.

sfilipi · 2018-06-15T16:11:07Z

There are a few other sweepers, inside the Algorithms folder, that override ProposeSweeps.
Taking a look to see whether they need the same filtering, and I'll update the comment. Probably they do.

In reply to: 397653603 [](ancestors = 397653603)

jwood803 · 2018-06-18T21:09:17Z

Took a shot at updating the KdoSweeper. The others I may need some guidance with as to what the best approach would be, though.

jwood803 · 2018-06-18T21:23:53Z

Whoops! Looks like rebasing closed this. 😅

Probably for the best since #368 looks like a better fix. #Closed

sfilipi · 2018-06-18T21:38:56Z

Thank you for your contribution!

In reply to: 398200508 [](ancestors = 398200508)

jwood803 mentioned this pull request Jun 14, 2018
Update error messages to point to GitHub issues instead of support group #359
Merged

TomFinley requested review from george-microsoft and sfilipi June 14, 2018 16:07

sfilipi reviewed Jun 15, 2018
View reviewed changes

TomFinley mentioned this pull request Jun 18, 2018
Return distinct array of ParameterSet when ProposeSweep is called #368
Merged

ross-p-smith mentioned this pull request Jun 18, 2018
The ProposeSweeps method, in sweepers should return uniqie entries #215
Closed

jwood803 closed this Jun 18, 2018

jwood803 force-pushed the master branch from 519d3ad to fc7286cCompare June 18, 2018 21:19

ghost locked as resolved and limited conversation to collaborators Mar 30, 2022

jwood803 mentioned this pull request Jun 14, 2018
Update error messages to point to GitHub issues instead of support group #359
Merged

TomFinley requested review from george-microsoft and sfilipi June 14, 2018 16:07

sfilipi reviewed Jun 15, 2018
View reviewed changes

TomFinley mentioned this pull request Jun 18, 2018
Return distinct array of ParameterSet when ProposeSweep is called #368
Merged

ross-p-smith mentioned this pull request Jun 18, 2018
The ProposeSweeps method, in sweepers should return uniqie entries #215
Closed

jwood803 closed this Jun 18, 2018

jwood803 force-pushed the master branch from 519d3ad to fc7286cCompare June 18, 2018 21:19

ghost locked as resolved and limited conversation to collaborators Mar 30, 2022

jwood803 mentioned this pull request Jun 14, 2018
Update error messages to point to GitHub issues instead of support group #359
Merged

TomFinley requested review from george-microsoft and sfilipi June 14, 2018 16:07

sfilipi reviewed Jun 15, 2018
View reviewed changes

TomFinley mentioned this pull request Jun 18, 2018
Return distinct array of ParameterSet when ProposeSweep is called #368
Merged

ross-p-smith mentioned this pull request Jun 18, 2018
The ProposeSweeps method, in sweepers should return uniqie entries #215
Closed

jwood803 closed this Jun 18, 2018

jwood803 force-pushed the master branch from 519d3ad to fc7286cCompare June 18, 2018 21:19

ghost locked as resolved and limited conversation to collaborators Mar 30, 2022

jwood803 mentioned this pull request Jun 14, 2018
Update error messages to point to GitHub issues instead of support group #359
Merged

TomFinley requested review from george-microsoft and sfilipi June 14, 2018 16:07

sfilipi reviewed Jun 15, 2018
View reviewed changes

TomFinley mentioned this pull request Jun 18, 2018
Return distinct array of ParameterSet when ProposeSweep is called #368
Merged

ross-p-smith mentioned this pull request Jun 18, 2018
The ProposeSweeps method, in sweepers should return uniqie entries #215
Closed

jwood803 closed this Jun 18, 2018

jwood803 force-pushed the master branch from 519d3ad to fc7286cCompare June 18, 2018 21:19

ghost locked as resolved and limited conversation to collaborators Mar 30, 2022

jwood803 mentioned this pull request Jun 14, 2018
Update error messages to point to GitHub issues instead of support group #359
Merged

TomFinley requested review from george-microsoft and sfilipi June 14, 2018 16:07

sfilipi reviewed Jun 15, 2018
View reviewed changes

TomFinley mentioned this pull request Jun 18, 2018
Return distinct array of ParameterSet when ProposeSweep is called #368
Merged

ross-p-smith mentioned this pull request Jun 18, 2018
The ProposeSweeps method, in sweepers should return uniqie entries #215
Closed

jwood803 closed this Jun 18, 2018

jwood803 force-pushed the master branch from 519d3ad to fc7286cCompare June 18, 2018 21:19

ghost locked as resolved and limited conversation to collaborators Mar 30, 2022

jwood803 mentioned this pull request Jun 14, 2018
Update error messages to point to GitHub issues instead of support group #359
Merged

TomFinley requested review from george-microsoft and sfilipi June 14, 2018 16:07

sfilipi reviewed Jun 15, 2018
View reviewed changes

TomFinley mentioned this pull request Jun 18, 2018
Return distinct array of ParameterSet when ProposeSweep is called #368
Merged

ross-p-smith mentioned this pull request Jun 18, 2018
The ProposeSweeps method, in sweepers should return uniqie entries #215
Closed

jwood803 closed this Jun 18, 2018

jwood803 force-pushed the master branch from 519d3ad to fc7286cCompare June 18, 2018 21:19

ghost locked as resolved and limited conversation to collaborators Mar 30, 2022

jwood803 mentioned this pull request Jun 14, 2018
Update error messages to point to GitHub issues instead of support group #359
Merged

TomFinley requested review from george-microsoft and sfilipi June 14, 2018 16:07

sfilipi reviewed Jun 15, 2018
View reviewed changes

TomFinley mentioned this pull request Jun 18, 2018
Return distinct array of ParameterSet when ProposeSweep is called #368
Merged

ross-p-smith mentioned this pull request Jun 18, 2018
The ProposeSweeps method, in sweepers should return uniqie entries #215
Closed

jwood803 closed this Jun 18, 2018

jwood803 force-pushed the master branch from 519d3ad to fc7286cCompare June 18, 2018 21:19

ghost locked as resolved and limited conversation to collaborators Mar 30, 2022

jwood803 mentioned this pull request Jun 14, 2018
Update error messages to point to GitHub issues instead of support group #359
Merged

TomFinley requested review from george-microsoft and sfilipi June 14, 2018 16:07

sfilipi reviewed Jun 15, 2018
View reviewed changes

TomFinley mentioned this pull request Jun 18, 2018
Return distinct array of ParameterSet when ProposeSweep is called #368
Merged

ross-p-smith mentioned this pull request Jun 18, 2018
The ProposeSweeps method, in sweepers should return uniqie entries #215
Closed

jwood803 closed this Jun 18, 2018

jwood803 force-pushed the master branch from 519d3ad to fc7286cCompare June 18, 2018 21:19

ghost locked as resolved and limited conversation to collaborators Mar 30, 2022

Uh oh!

Conversation

jwood803 commented Jun 14, 2018

Uh oh!

sfilipi commented Jun 15, 2018

Uh oh!

sfilipiJun 15, 2018• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jwood803Jun 15, 2018

Choose a reason for hiding this comment

Uh oh!

sfilipiJun 15, 2018• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jwood803Jun 15, 2018• edited by sfilipi Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sfilipiJun 15, 2018• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jwood803Jun 15, 2018

Choose a reason for hiding this comment

Uh oh!

sfilipi commented Jun 15, 2018

Uh oh!

jwood803 commented Jun 18, 2018

Uh oh!

jwood803 commented Jun 18, 2018• edited by sfilipi Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sfilipi commented Jun 18, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

jwood803 commented Jun 14, 2018

Uh oh!

sfilipi commented Jun 15, 2018

Uh oh!

sfilipiJun 15, 2018• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jwood803Jun 15, 2018

Choose a reason for hiding this comment

Uh oh!

sfilipiJun 15, 2018• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jwood803Jun 15, 2018• edited by sfilipi Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sfilipiJun 15, 2018• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jwood803Jun 15, 2018

Choose a reason for hiding this comment

Uh oh!

sfilipi commented Jun 15, 2018

Uh oh!

jwood803 commented Jun 18, 2018

Uh oh!

jwood803 commented Jun 18, 2018• edited by sfilipi Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

sfilipiJun 15, 2018•
edited
Loading

sfilipiJun 15, 2018•
edited
Loading

jwood803Jun 15, 2018•
edited by sfilipi
Loading

sfilipiJun 15, 2018•
edited
Loading

jwood803 commented Jun 18, 2018•
edited by sfilipi
Loading

sfilipiJun 15, 2018•
edited
Loading

sfilipiJun 15, 2018•
edited
Loading

jwood803Jun 15, 2018•
edited by sfilipi
Loading

sfilipiJun 15, 2018•
edited
Loading

jwood803 commented Jun 18, 2018•
edited by sfilipi
Loading

sfilipiJun 15, 2018•
edited
Loading

sfilipiJun 15, 2018•
edited
Loading

jwood803Jun 15, 2018•
edited by sfilipi
Loading

sfilipiJun 15, 2018•
edited
Loading

jwood803 commented Jun 18, 2018•
edited by sfilipi
Loading