minibmg: change std::list to std::vector

facebookresearch / beanmachine

A library that allows for inference on probabilistic models

https://beanmachine.org/

MIT License

264 stars 49 forks source link

minibmg: change std::list to std::vector #1792

Closed gafter closed 1 year ago

gafter commented 1 year ago

Summary: We change all uses of std::list to std::vector throughout minibmg. The books I've read recommond not using list unless you really need some of its APIs, which we do not. vector is faster and uses less memory.

I'm doing this now because I need it in another diff, and I want to keep each diff as simple as possible.

Differential Revision: D40812419

facebook-github-bot commented 1 year ago

This pull request was exported from Phabricator. Differential Revision: D40812419