# Looking for feedback on package type design - tree log volumes

**URL:** <https://discourse.julialang.org/t/looking-for-feedback-on-package-type-design-tree-log-volumes/8078>\
**Category:** General Usage\
**Created:** [December 30, 2017, 11:23pm UTC](https://discourse.julialang.org/t/looking-for-feedback-on-package-type-design-tree-log-volumes/8078 "2017-12-30T23:23:11Z")\
**Posts on this page:** 1\
**Showing post:** 2

<div class="post-metadata">

**Author:** ![Tamas\_Papp](https://sea2.discourse-cdn.com/julialang/user_avatar/discourse.julialang.org/tamas_papp/32/25949_2.png) [@Tamas\_Papp](https://discourse.julialang.org/u/Tamas_Papp)\
**Post date:** [December 31, 2017, 7:36am UTC](https://discourse.julialang.org/t/looking-for-feedback-on-package-type-design-tree-log-volumes/8078/2 "2017-12-31T07:36:16Z")

</div>

First, [please quote your code with backticks](https://discourse.julialang.org/t/psa-how-to-quote-code-with-backticks/7530).

It is unclear what your question is. Do you want an set of functions that calculates volumes for various objects? Multiple dispatch should be straightforward.

I don’t know what `huber` and `newton` are, but if they result in different volumes, then it is not particularly good design to have them in the same method, and if they don’t, I am not sure why you need them.

It is unclear why you need `LogSegment` to be a macro. A function should be able to provide the same interface.

---

_[View the full topic](https://discourse.julialang.org/t/looking-for-feedback-on-package-type-design-tree-log-volumes/8078)._