search

probcomp / Venturecxx

Primary implementation of the Venture probabilistic programming system
http://probcomp.csail.mit.edu/venture/
GNU General Public License v3.0
28 stars 6 forks source link

Put the Puma C++ definitions into their own namespace #15

Open gregory-marton opened 9 years ago

gregory-marton commented 9 years ago

Perhaps something like mit::probcomp::venture or mit_probcomp_venture or just probcomp_venture? Unsure what the right politics are, but having it in its own namespace will help resolve ambiguities when others try to integrate with it, or when we try to integrate with others.

axch commented 8 years ago

This is code style. The umbrella code style issue is currently #429.

axch commented 8 years ago

There is also a political aspect to it, as @gregory-marton points out. I actually don't know the socio-technical interactions of C++ well enough to think through the politics.

riastradh-probcomp commented 8 years ago

I think the namespace probcomp::venture would be perfectly fine, sociopoliticotechnically. Or even just venture.

We should also ruthlessly expunge using from .h files in the process.