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

Perseus tool in geometry section not working #5099

Closed
rifatbd opened this Issue Apr 21, 2016 · 14 comments

Comments

Projects
None yet
4 participants
@rifatbd

rifatbd commented Apr 21, 2016

Summary

In the geometry section (more specifically in Transformations-Congruence-Similarity section) when doing exercise no operation can be done on shapes. Like rotating the shape, or transform or operations like those.

Branch or installer method

  • Installer: Windows installer from pantry (win7 64 bit)
  • Version: 0.16.2

Traceback or relevant snippet from server.log

server.log is empty

How to reproduce

  1. Start Ka lite & naviagte to /learn/khan/math/basic-geo/transformations-congruence-similarity-geo/transformations-basics/performing-reflections-on-the-coordinate-plane/
  2. Try to do the "Reflect" operation by clicking the button underneath the image
  3. Nothing happens

Screenshots

capture1
capture2
capture3
capture4
capture5

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Apr 25, 2016

Member

@rtibbles what's your take on this? You mentioned as a possible dirty resolution to #5036 that exercises could be excluded.

Member

benjaoming commented Apr 25, 2016

@rtibbles what's your take on this? You mentioned as a possible dirty resolution to #5036 that exercises could be excluded.

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Apr 26, 2016

Member

I find that lots of geometry exercises are broken, almost anyone that I click. I guess these are using the "more advanced" features of Perseus.

Using: Latest English content pack + 0.16.3 release.

Here's another example of an error:

screenshot from 2016-04-26 16 07 27

Member

benjaoming commented Apr 26, 2016

I find that lots of geometry exercises are broken, almost anyone that I click. I guess these are using the "more advanced" features of Perseus.

Using: Latest English content pack + 0.16.3 release.

Here's another example of an error:

screenshot from 2016-04-26 16 07 27

@rifatbd

This comment has been minimized.

Show comment
Hide comment
@rifatbd

rifatbd Apr 27, 2016

@benjaoming how can i install the 0.16.3 release? I cant see any installer for windows. Or there is some other method.

rifatbd commented Apr 27, 2016

@benjaoming how can i install the 0.16.3 release? I cant see any installer for windows. Or there is some other method.

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Apr 27, 2016

Member

@rifatbd 0.16.3 doesn't solve this issue, it was released in our PPA and on PyPi but will quickly be made redundant because fixing the above is of utmost priority :)

Member

benjaoming commented Apr 27, 2016

@rifatbd 0.16.3 doesn't solve this issue, it was released in our PPA and on PyPi but will quickly be made redundant because fixing the above is of utmost priority :)

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Apr 27, 2016

Member

@rifatbd thanks for a very thorough issue report btw, made it very easy to realize the problems with exercises.

Member

benjaoming commented Apr 27, 2016

@rifatbd thanks for a very thorough issue report btw, made it very easy to realize the problems with exercises.

@rifatbd

This comment has been minimized.

Show comment
Hide comment
@rifatbd

rifatbd Apr 28, 2016

Good to hear that i was of some help...... :)

rifatbd commented Apr 28, 2016

Good to hear that i was of some help...... :)

@mjptak

This comment has been minimized.

Show comment
Hide comment
@mjptak

mjptak Apr 28, 2016

also glad to see that the very dot that lowers the exercise creation bar is
still intact. Awesome work...awesome applications...awesome volunteers.

On Thu, Apr 28, 2016 at 1:33 AM, rifatbd notifications@github.com wrote:

Good to hear that i was of some help...... :)


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#5099 (comment)

mjptak commented Apr 28, 2016

also glad to see that the very dot that lowers the exercise creation bar is
still intact. Awesome work...awesome applications...awesome volunteers.

On Thu, Apr 28, 2016 at 1:33 AM, rifatbd notifications@github.com wrote:

Good to hear that i was of some help...... :)


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#5099 (comment)

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Apr 28, 2016

Member

It seems like everything will be fine in the next release, we just need to re-generate some of the exercise data before releasing 0.16.4. Within another 24 hours it'll be rolled out :)

Member

benjaoming commented Apr 28, 2016

It seems like everything will be fine in the next release, we just need to re-generate some of the exercise data before releasing 0.16.4. Within another 24 hours it'll be rolled out :)

@rifatbd

This comment has been minimized.

Show comment
Hide comment
@rifatbd

rifatbd Apr 30, 2016

@benjaoming I just upgraded to 0.16.4 from PyPi .. and the issue seems to be solved :)....
One more thing the woff2 fonts are still missing in fresh installation for some reason..... i just had to manually add them.....may b u should check that out too.....
apart from that the very issue of geometric tools is solved 👍 ........
i added a snap of woff2 missing just in case..
capture22

rifatbd commented Apr 30, 2016

@benjaoming I just upgraded to 0.16.4 from PyPi .. and the issue seems to be solved :)....
One more thing the woff2 fonts are still missing in fresh installation for some reason..... i just had to manually add them.....may b u should check that out too.....
apart from that the very issue of geometric tools is solved 👍 ........
i added a snap of woff2 missing just in case..
capture22

@rifatbd

This comment has been minimized.

Show comment
Hide comment
@rifatbd

rifatbd Apr 30, 2016

Anyway looks like there is still some minor issues left .....
When Bengali is selected as Default language this happens....

capture21
capture23

rifatbd commented Apr 30, 2016

Anyway looks like there is still some minor issues left .....
When Bengali is selected as Default language this happens....

capture21
capture23

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Apr 30, 2016

Member

Thanks for testing @rifatbd !!

It seems like said fonts are located in static/html/_static/fonts/.

I would recommend creating two new issues, one for issues regarding Bengali, the other for fonts. I can tag the latter as "beginner friendly". It can be addressed possibly in 0.16.5.

Member

benjaoming commented Apr 30, 2016

Thanks for testing @rifatbd !!

It seems like said fonts are located in static/html/_static/fonts/.

I would recommend creating two new issues, one for issues regarding Bengali, the other for fonts. I can tag the latter as "beginner friendly". It can be addressed possibly in 0.16.5.

@benjaoming benjaoming closed this Apr 30, 2016

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming Apr 30, 2016

Member

Sorry, the fonts are present in the right location, they are however not included in MANIFEST.in

➜  ka-lite git:(0.16.x) find |grep KaTeX_Main-Regular.woff2   
./kalite/distributed/static/fonts/KaTeX_Main-Regular.woff2
./kalite/distributed/static/js/distributed/perseus/ke/local-only/katex/fonts/KaTeX_Main-Regular.woff2
./kalite/static/fonts/KaTeX_Main-Regular.woff2
./kalite/static/js/distributed/perseus/ke/local-only/katex/fonts/KaTeX_Main-Regular.woff2
Member

benjaoming commented Apr 30, 2016

Sorry, the fonts are present in the right location, they are however not included in MANIFEST.in

➜  ka-lite git:(0.16.x) find |grep KaTeX_Main-Regular.woff2   
./kalite/distributed/static/fonts/KaTeX_Main-Regular.woff2
./kalite/distributed/static/js/distributed/perseus/ke/local-only/katex/fonts/KaTeX_Main-Regular.woff2
./kalite/static/fonts/KaTeX_Main-Regular.woff2
./kalite/static/js/distributed/perseus/ke/local-only/katex/fonts/KaTeX_Main-Regular.woff2
@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming May 2, 2016

Member

Gonna test now manually if this works, will give an update and release as 0.16.5 soon.

Member

benjaoming commented May 2, 2016

Gonna test now manually if this works, will give an update and release as 0.16.5 soon.

@benjaoming benjaoming assigned benjaoming and unassigned rtibbles May 2, 2016

@benjaoming

This comment has been minimized.

Show comment
Hide comment
@benjaoming

benjaoming May 2, 2016

Member

Font issue solved in 0.16.5

Member

benjaoming commented May 2, 2016

Font issue solved in 0.16.5

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