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

AA: Deleting features takes a long time #143

Closed
roemhildtg opened this Issue Sep 7, 2016 · 9 comments

Comments

Projects
None yet
5 participants
@roemhildtg

roemhildtg commented Sep 7, 2016

When using a file geodatabase feature class, and AA is activated, if I select a feature and delete it, it takes an oddly long time to perform the delete. Disabling AA and the delete happens instantly.

The feature class I'm working with happens to be in a geometric network, but I believe this occurs with any feature class.

@roemhildtg

This comment has been minimized.

Show comment
Hide comment
@roemhildtg

roemhildtg Sep 21, 2016

Update: This doesn't happen when using a sql based data source, only file gdb type.

roemhildtg commented Sep 21, 2016

Update: This doesn't happen when using a sql based data source, only file gdb type.

@MikeMillerGIS

This comment has been minimized.

Show comment
Hide comment
@MikeMillerGIS

MikeMillerGIS Sep 23, 2016

Member

There is a function: public static void StopOperation(), line 823 in the AttributeAssistantEditorExtension that seems to be the culprit. This is required so the AA will process any related records generated or updated as part of the routine. We could expose an object to bypass this functionality to speed things up.

Member

MikeMillerGIS commented Sep 23, 2016

There is a function: public static void StopOperation(), line 823 in the AttributeAssistantEditorExtension that seems to be the culprit. This is required so the AA will process any related records generated or updated as part of the routine. We could expose an object to bypass this functionality to speed things up.

@MikeMillerGIS

This comment has been minimized.

Show comment
Hide comment
@MikeMillerGIS

MikeMillerGIS Sep 23, 2016

Member

@Vertex3 @SteveGrise You added this function, do you know if we bypass it, what the ramification are?

Member

MikeMillerGIS commented Sep 23, 2016

@Vertex3 @SteveGrise You added this function, do you know if we bypass it, what the ramification are?

@SteveGrise

This comment has been minimized.

Show comment
Hide comment
@SteveGrise

SteveGrise Sep 23, 2016

Hmm, that's from a while ago. I suspect it takes time to listen to each
edit event and it probably takes about the same amount of time to process
even if nothing is triggered...

On Fri, Sep 23, 2016 at 11:17 AM, Michael Miller notifications@github.com
wrote:

@Vertex3 https://github.com/vertex3 @SteveGrise
https://github.com/SteveGrise You added this function, do you know if
we bypass it, what the ramification are?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#143 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEh3BLULwiiUhruP1kCea7I348l2aZy8ks5qs-2ggaJpZM4J3IKS
.

SteveGrise commented Sep 23, 2016

Hmm, that's from a while ago. I suspect it takes time to listen to each
edit event and it probably takes about the same amount of time to process
even if nothing is triggered...

On Fri, Sep 23, 2016 at 11:17 AM, Michael Miller notifications@github.com
wrote:

@Vertex3 https://github.com/vertex3 @SteveGrise
https://github.com/SteveGrise You added this function, do you know if
we bypass it, what the ramification are?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#143 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEh3BLULwiiUhruP1kCea7I348l2aZy8ks5qs-2ggaJpZM4J3IKS
.

@MikeMillerGIS

This comment has been minimized.

Show comment
Hide comment
@MikeMillerGIS

MikeMillerGIS Sep 23, 2016

Member

Added a simple bypass variable. Speeds up batch processing, such as using the buttons.
d4eddf4

Member

MikeMillerGIS commented Sep 23, 2016

Added a simple bypass variable. Speeds up batch processing, such as using the buttons.
d4eddf4

@MikeMillerGIS MikeMillerGIS self-assigned this Sep 23, 2016

@MikeMillerGIS MikeMillerGIS added this to the Next Release milestone Sep 23, 2016

@MikeMillerGIS

This comment has been minimized.

Show comment
Hide comment
@MikeMillerGIS

MikeMillerGIS Sep 23, 2016

Member

Need to document new config tag in the AA configuration file

Member

MikeMillerGIS commented Sep 23, 2016

Need to document new config tag in the AA configuration file

@pLeBlanc93

This comment has been minimized.

Show comment
Hide comment
@pLeBlanc93

pLeBlanc93 Feb 13, 2017

Contributor

verified bypass variable gives signification speed up

Contributor

pLeBlanc93 commented Feb 13, 2017

verified bypass variable gives signification speed up

@pLeBlanc93 pLeBlanc93 added the Verified label Feb 13, 2017

pLeBlanc93 added a commit to pLeBlanc93/local-government-desktop-addins that referenced this issue Feb 13, 2017

@pLeBlanc93

This comment has been minimized.

Show comment
Hide comment
@pLeBlanc93

pLeBlanc93 Feb 14, 2017

Contributor

GenerateID fields

The GenerateID table must have three fields: Name, Counter, and Interval. To use fields with different names, set the SequenceName, SequenceCounter, and SequenceInterval key value to your field names.

Bypass edit operation

When the Attribute Assistant is enabled and features are deleted, related records are also processed by the add-in. For quicker delete operations, set the BypassEditOperationCheck key value to True.

Contributor

pLeBlanc93 commented Feb 14, 2017

GenerateID fields

The GenerateID table must have three fields: Name, Counter, and Interval. To use fields with different names, set the SequenceName, SequenceCounter, and SequenceInterval key value to your field names.

Bypass edit operation

When the Attribute Assistant is enabled and features are deleted, related records are also processed by the add-in. For quicker delete operations, set the BypassEditOperationCheck key value to True.

@pLeBlanc93 pLeBlanc93 removed the doc-needed label Feb 14, 2017

@MikeMillerGIS

This comment has been minimized.

Show comment
Hide comment
@MikeMillerGIS

MikeMillerGIS Feb 14, 2017

Member
Member

MikeMillerGIS commented Feb 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment