Code Monkey home page Code Monkey logo

Comments (4)

thomasp85 avatar thomasp85 commented on May 29, 2024 1

I currently maintain ~15 packages and seldom have problems with changing API's (just to share my personal experience). I have no problem removing tibble and your point is valid so consider that done. I would be thrilled if you removed pipe use in the text-explainer as I prefer its use during analysis and not within packages, but being a collaboration I am also OK with others using their preferred approach. The same goes for purrr.

from lime.

thomasp85 avatar thomasp85 commented on May 29, 2024

I never really use magrittr and purrr (and very seldom dplyr - I was apparently lazy here) when developing packages.

I accepted that you wanted to use pipe and purrr, but if you want to remove them from your code I'll be very supportive.

I think the notion of "too many dependencies" is a bit strange though - if a package is used for a valid reason I have no problem including it. If you are more comfortable programming with purrr, I think that's a valid reason...

from lime.

pommedeterresautee avatar pommedeterresautee commented on May 29, 2024

I answer here for both #17 and #20.
There are few reasons why I think it may be good to reduce the list of dependencies:

  • one is from my own exp when writing the XGBoost R package. I made lots of dependencies, in particular on some Viz packages. Viz package author broke the API every 2 months, making it more complete and more complexe. So every 2 months I had to learn the new API and update the package. Very annoying with lots of issues opened because people had a working package and because of update, everything was broken.
  • some dev really hates the hadleyverse for bad/good reasons (I really don't care). Even if we may think tibble offers a better API or a better print, if we don't really need its features, we should not force the user to follow our opinions (I know it s easy to convert to other format). If vanilla data.frame make the work, why we would not use it? (it would be another story if you think to use its features in the future, for instance on XGBoost I only use data.table for its speed in join/select)
  • regarding purrr in particular, it offers a very nice API for someone with a functional background but may discourage other dev to enter in the project just for my own comfort. So I think it s better to remove them.

Please don't hesitate to share your mind, I would be very interested in it.

from lime.

pommedeterresautee avatar pommedeterresautee commented on May 29, 2024

done/closing

from lime.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.