-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
gsoc'20 call summary #38
Comments
|
|
hi @lazycipher would you like another call next week? |
hi @tdhock, Can we have a call? |
to solve that issue just delete that example and then recompile documentation via devtools::document()
|
next week is really not that great for me for a call so let's try to use github ok? |
@tdhock sounds good to me! |
Hii @tdhock, Can you check what's this test renders and probably explain this test a bit to me, please? The test is failing because of line 36. |
this test is supposed to make sure that Inf/-Inf are rendered properly on both the x and y axes (at the edge of the panel).
|
hi what is the above issue? looks like the viz with Inf/-Inf is rendering fine so you just need to make sure the testing code is extracting the positions correctly right? |
If that's the case, then we are looking in the wrong panels for lines. |
@tdhock, I don't know why it's happening! lazycipher@78c8535 |
Fixed this! It was |
great, that makes sense. |
Hi @tdhock, @faizan-khan-iit, I'm having issues in getting the flow of data. I've also made and posted 2 videos of testing on YouTube which shows how it's done on Linux and Windows. Soon I'll be posting a few videos in which I'll be making animated viz. |
Call on 29th July with @faizan-khan-iit,
@faizan-khan-iit, Please let me know if I missed anything. Thank you so much for the detailed discussion. |
problem understand what is R package? no, problem understanding how R package works. that is normal. |
TODO post summary of calls.
The text was updated successfully, but these errors were encountered: