ProFET icon indicating copy to clipboard operation
ProFET copied to clipboard

removed click, added parameter check

Open MichaelDoron opened this issue 9 years ago • 3 comments

MichaelDoron avatar May 27 '15 05:05 MichaelDoron

Yay. Thank you very much :-)

בעתיד, תהיה אתה מעוניין\יכול עדיין להמשיך להשתתף עוד קצת בקשר לזה? (על דברים פשוטים יחסית. תיעוד )/readme), בדיקה ל exceptions וכו).

אני לא ליד מחשב כרגע, אבל רק לשאול - בשינויים שהעלית, יש אז עכשיו אפשרות לשמירה וטעינה של מודל? ו שמירה ל raw features, בלי לנרמל?) On May 27, 2015 8:28 AM, "Michael Doron" [email protected] wrote:


You can view, comment on, or merge this pull request online at:

https://github.com/ddofer/ProFET/pull/1 Commit Summary

  • removed click, added parameter check

File Changes

  • M CODE/feat_extract/pipeline.py https://github.com/ddofer/ProFET/pull/1/files#diff-0 (63)

Patch Links:

  • https://github.com/ddofer/ProFET/pull/1.patch
  • https://github.com/ddofer/ProFET/pull/1.diff

— Reply to this email directly or view it on GitHub https://github.com/ddofer/ProFET/pull/1.

ddofer avatar May 27 '15 05:05 ddofer

אין את השמירה של המודל או הנרמול, ואני חושב שזה יהיה עמוס לי מדי, עם המחקר אצל עידן והכל.

2015-05-27 8:56 GMT+03:00 Dan Ofer [email protected]:

Yay. Thank you very much :-)

בעתיד, תהיה אתה מעוניין\יכול עדיין להמשיך להשתתף עוד קצת בקשר לזה? (על דברים פשוטים יחסית. תיעוד )/readme), בדיקה ל exceptions וכו).

אני לא ליד מחשב כרגע, אבל רק לשאול - בשינויים שהעלית, יש אז עכשיו אפשרות לשמירה וטעינה של מודל? ו שמירה ל raw features, בלי לנרמל?) On May 27, 2015 8:28 AM, "Michael Doron" [email protected] wrote:


You can view, comment on, or merge this pull request online at:

https://github.com/ddofer/ProFET/pull/1 Commit Summary

  • removed click, added parameter check

File Changes

  • M CODE/feat_extract/pipeline.py https://github.com/ddofer/ProFET/pull/1/files#diff-0 (63)

Patch Links:

  • https://github.com/ddofer/ProFET/pull/1.patch
  • https://github.com/ddofer/ProFET/pull/1.diff

— Reply to this email directly or view it on GitHub https://github.com/ddofer/ProFET/pull/1.

— Reply to this email directly or view it on GitHub https://github.com/ddofer/ProFET/pull/1#issuecomment-105766735.

MichaelDoron avatar May 27 '15 14:05 MichaelDoron

Ok. On May 27, 2015 5:52 PM, "Michael Doron" [email protected] wrote:

אין את השמירה של המודל או הנרמול, ואני חושב שזה יהיה עמוס לי מדי, עם המחקר אצל עידן והכל.

2015-05-27 8:56 GMT+03:00 Dan Ofer [email protected]:

Yay. Thank you very much :-)

בעתיד, תהיה אתה מעוניין\יכול עדיין להמשיך להשתתף עוד קצת בקשר לזה? (על דברים פשוטים יחסית. תיעוד )/readme), בדיקה ל exceptions וכו).

אני לא ליד מחשב כרגע, אבל רק לשאול - בשינויים שהעלית, יש אז עכשיו אפשרות לשמירה וטעינה של מודל? ו שמירה ל raw features, בלי לנרמל?) On May 27, 2015 8:28 AM, "Michael Doron" [email protected] wrote:


You can view, comment on, or merge this pull request online at:

https://github.com/ddofer/ProFET/pull/1 Commit Summary

  • removed click, added parameter check

File Changes

  • M CODE/feat_extract/pipeline.py https://github.com/ddofer/ProFET/pull/1/files#diff-0 (63)

Patch Links:

  • https://github.com/ddofer/ProFET/pull/1.patch
  • https://github.com/ddofer/ProFET/pull/1.diff

— Reply to this email directly or view it on GitHub https://github.com/ddofer/ProFET/pull/1.

— Reply to this email directly or view it on GitHub https://github.com/ddofer/ProFET/pull/1#issuecomment-105766735.

— Reply to this email directly or view it on GitHub https://github.com/ddofer/ProFET/pull/1#issuecomment-105943225.

ddofer avatar May 27 '15 14:05 ddofer