Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BHoM_Adapter: refactoring-PullTo-implement batches #92

Open
Tracked by #84
alelom opened this issue May 30, 2019 · 1 comment
Open
Tracked by #84

BHoM_Adapter: refactoring-PullTo-implement batches #92

alelom opened this issue May 30, 2019 · 1 comment
Labels
type:feature New capability or enhancement

Comments

@alelom
Copy link
Member

alelom commented May 30, 2019

The PullTo method (or "Move", as proposed by #91) could implement batches of objects in order to improve stability/performance when dealing with large models.

As discussed with @adecler and @IsakNaslundBh one way of doing it could be to implement PullTo as async. However, this would be in contrast with the rest of the methods, it could be scary to less advanced users, and generally more difficult to debug.

Another way is simply to introduce a mechanism to Pull and Push objects in batches of a certain number.

@alelom alelom added the type:feature New capability or enhancement label May 30, 2019
@al-fisher
Copy link
Member

A global approach to coping with asynchronous events is going to be required anyway - so good to have that formally on the road map. It has been discussed a few times recently in context of both Adapter Pushing/Pulling as well as loading/creation of formulas in Excel_Toolkit etc. etc. So will be good to start considering users exceptions and required UI for asynchronous events in a flow based programming environment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:feature New capability or enhancement
Projects
None yet
Development

No branches or pull requests

2 participants