Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
zoidbergwill
@zoidbergwill
Wanna change to using it?
I looked at another one I've wanted to use for awhile but it was too heavy, then I stopped looking
Rudi
@rudigiesler
I think ours is easy enough to not need it
And I think it might be more restricting
I'm already having to craft a custom response to have a redirect with a body
zoidbergwill
@zoidbergwill
Yeah, Flask's responses are super basic.
Rudi
@rudigiesler
ready for review: v
rudigiesler/mathml-to-image-service#11
zoidbergwill
@zoidbergwill
One minor comment
Rudi
@rudigiesler
pushed changes
Rudi
@rudigiesler
#7 is ready for review
zoidbergwill
@zoidbergwill
Sorry, one minor thing.
Rudi
@rudigiesler
Sorry lunch took a while, I've pushed a commit that should fix it
zoidbergwill
@zoidbergwill
:+1:'ed
Rudi
@rudigiesler
thanks
@zoidbergwill are you working on documentation, or should I work on creating something?
zoidbergwill
@zoidbergwill
I've got the basic index page
Rudi
@rudigiesler
I'll write some tests for svg_to_image then
zoidbergwill
@zoidbergwill
Sweet
I've got the basic documentation stuff in a branch
I'll make a PR
I still need to add documentation for the actual use of the API
Rudi
@rudigiesler
Okay sure. Do you want two separate PR's for that?
zoidbergwill
@zoidbergwill
It should be chilled
I'll do it in one
Rudi
@rudigiesler
Okay I'll wait for it to be complete before I review
zoidbergwill
@zoidbergwill
Sweet, thanks
Rudi
@rudigiesler
#13 ready for review
zoidbergwill
@zoidbergwill
Still working on the tests for the svg converting stuff?
Rudi
@rudigiesler
nope, the tests are in the PR
I'm busy working on adding a quality setting
zoidbergwill
@zoidbergwill
Cool, was just wondering if I could review it
That 'ready for review' message for #13 didn't come through until I refreshed the page :/
Should I add the things to #13 that I suggested, while you work on the quality setting stuff?
Well I guess we can merge #13 or I can make a PR into it
zoidbergwill
@zoidbergwill
rudigiesler/mathml-to-image-service#15 ready for review
#14 ready for review
Rudi
@rudigiesler
:+1: on both
zoidbergwill
@zoidbergwill
Thanks
Both merged
zoidbergwill
@zoidbergwill
Gonna check if the read the docs builds, then work on improving it
Rudi
@rudigiesler
okay cool, I've implemented most of the image size reduction things, now to see if it works properly
zoidbergwill
@zoidbergwill
Sorry, I'm being mean on the other pull request.
zoidbergwill
@zoidbergwill
Dammit, I lost internet for like 20 minutes
Rudi
@rudigiesler
haha, sorry was working on the quality feature, I'll take a look at the PR now
I've open #12 for the new feature
zoidbergwill
@zoidbergwill
Sweet, will take a look.
Rudi
@rudigiesler
Leaving for the airport now, not sure if I'll have internet there
zoidbergwill
@zoidbergwill
Cool, I'll leave one comment on #12
I :+1:'ed #13