Skip to content

HW2 submitted#3

Open
policyglot wants to merge 3 commits into
css-research:masterfrom
policyglot:master
Open

HW2 submitted#3
policyglot wants to merge 3 commits into
css-research:masterfrom
policyglot:master

Conversation

@policyglot
Copy link
Copy Markdown

No description provided.

@jfan3
Copy link
Copy Markdown

jfan3 commented May 24, 2019

Some models are not run.

@policyglot
Copy link
Copy Markdown
Author

policyglot commented May 24, 2019

Hi Fiona, do you mean that they are not running? I ran all of them before finding the best one. Could you please clarify the issue? Thanks !

@jfan3
Copy link
Copy Markdown

jfan3 commented May 25, 2019

they are not running? I ran all of them before finding the best one. Could you

Hi Abe, I was referring to the following cells:
temp

@policyglot
Copy link
Copy Markdown
Author

policyglot commented May 25, 2019 via email

@jfan3
Copy link
Copy Markdown

jfan3 commented May 26, 2019

Hi Fiona, So are you saying the code is not working for those cells? What's the error exactly? Or are you saying the code was not run for them? To be clear,I ran all the code on the weekend before class and saw all of the results on a plot. But I then tried to abstract away and write cleaner code. So I decided to store all the results in a list and loop over (all_models) and loop over them in the end. I know that the code works, just did not run ALL the models again, since I'd seen the output already. Does that make sense now? Carpe that Diem! Abhishek Pandit https://www.linkedin.com/in/policyglot https://www.linkedin.com/in/abhishek-pandit-91945711 PS- So as to maximize my productivity and presence in the moment, I only check emails at 11 AM and 4 PM on weekdays. For anything more urgent, feel free to call at +1 630-345-2837 or chat either in person or on Skype (ID: abhishekpandit6789)

On Fri, 24 May 2019 at 21:32, Fiona Fan @.***> wrote: they are not running? I ran all of them before finding the best one. Could you Hi Abe, I was referring to the following cells: [image: temp] https://user-images.githubusercontent.com/16408337/58363541-791c2980-7e6b-11e9-941f-e599c84d83b4.png — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#3?email_source=notifications&email_token=AB2XSIO4445ZX7XHVGQHRELPXCQNHA5CNFSM4HMS4EO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWG4D2Q#issuecomment-495829482>, or mute the thread https://github.com/notifications/unsubscribe-auth/AB2XSIP6SOICZN54UZZOI73PXCQNHANCNFSM4HMS4EOQ .

Hi Abe,

I don't run the code because it would take me hours and hours to do so. Thus, I only grade on the outputs. I didn't see the outputs for those cells, that's what I was referring to when I said there are models unrun. It's great if you optimized your code, but you would still need to run the code to receive full credit.

@policyglot
Copy link
Copy Markdown
Author

policyglot commented May 28, 2019 via email

@jfan3
Copy link
Copy Markdown

jfan3 commented May 29, 2019

Yes, naturally it's a long process. I know because I had to go through it already! :) My mistake was to not save the results in a list for the first time when I left the code running overnight. And then I optimized the code without running it all over again for all the cells. I understand that you might have instructions to grade based on the output. But I didn't know all the output had to already be visible (didn't do that for all the previous Perspectives assignments till now). I've definitely put in the work, otherwise I wouldn't get the final result from the plots. If you'd like, we can check with Dr. Soltoff about what he thinks should be done here. Thanks! Carpe that Diem! Abhishek Pandit https://www.linkedin.com/in/policyglot https://www.linkedin.com/in/abhishek-pandit-91945711 PS- So as to maximize my productivity and presence in the moment, I only check emails at 11 AM and 4 PM on weekdays. For anything more urgent, feel free to call at +1 630-345-2837 or chat either in person or on Skype (ID: abhishekpandit6789)

On Sun, 26 May 2019 at 13:29, Fiona Fan @.> wrote: Hi Fiona, So are you saying the code is not working for those cells? What's the error exactly? Or are you saying the code was not run for them? To be clear,I ran all the code on the weekend before class and saw all of the results on a plot. But I then tried to abstract away and write cleaner code. So I decided to store all the results in a list and loop over (all_models) and loop over them in the end. I know that the code works, just did not run ALL the models again, since I'd seen the output already. Does that make sense now? Carpe that Diem! Abhishek Pandit https://www.linkedin.com/in/policyglot https://www.linkedin.com/in/policyglot https://www.linkedin.com/in/abhishek-pandit-91945711 https://www.linkedin.com/in/abhishek-pandit-91945711 PS- So as to maximize my productivity and presence in the moment, I only check emails at 11 AM and 4 PM on weekdays. For anything more urgent, feel free to call at +1 630-345-2837 or chat either in person or on Skype (ID: abhishekpandit6789) … <#m_-7844480312418504209_> On Fri, 24 May 2019 at 21:32, Fiona Fan @.> wrote: they are not running? I ran all of them before finding the best one. Could you Hi Abe, I was referring to the following cells: [image: temp] https://user-images.githubusercontent.com/16408337/58363541-791c2980-7e6b-11e9-941f-e599c84d83b4.png — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#3 <#3>?email_source=notifications&email_token=AB2XSIO4445ZX7XHVGQHRELPXCQNHA5CNFSM4HMS4EO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWG4D2Q#issuecomment-495829482>, or mute the thread https://github.com/notifications/unsubscribe-auth/AB2XSIP6SOICZN54UZZOI73PXCQNHANCNFSM4HMS4EOQ . Hi Abe, I don't run the code because it would take me hours and hours to do so. Thus, I only grade on the outputs. I didn't see the outputs for those cells, that's what I was referring to when I said there are models unrun. It's great if you optimized your code, but you would still need to run the code to receive full credit. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#3?email_source=notifications&email_token=AB2XSIKEDKWNY35W4FFDK4LPXLJIXA5CNFSM4HMS4EO2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWIK64A#issuecomment-496021360>, or mute the thread https://github.com/notifications/unsubscribe-auth/AB2XSIJK34MJOGEIE7WNYADPXLJIXANCNFSM4HMS4EOQ .

Sure! :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants