I feel its better to keep them different. It keeps it simple for user to
use.
On 17 Sep 2013 22:34, "Jason Moore" <notifications@github.com> wrote:

> In pydy_viz/static/js/canvas/addObjects.js:
>
> > +                                      _frame.shape.length,
> > +                                      50, 50, 50);
> > +            break;
> > +
> > +        case "Cylinder":
> > +            var _geometry = new THREE.CylinderGeometry(
> > +                                      _frame.shape.radius,
> > +                                            _frame.shape.radius,
> > +                                      _frame.shape.height,
> > +                                      50,50);
> > +            break;
> > +
> > +        case "Cone":
> > +            var _geometry = new THREE.CylinderGeometry(
> > +                                      _frame.shape.radius,
> > +                                      0.1,
>
> Why do we have a separate Cone class? Seems like you should just have the
> Cylinder class in python and allow the user to set both radii.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/PythonDynamics/pydy-viz/pull/46/files#r6411038>
> .
>