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

TypeError: copy_events() takes 6 positional arguments but 7 were given #133

Closed
sceniclife opened this issue Sep 22, 2020 · 3 comments · Fixed by #134 or #135
Closed

TypeError: copy_events() takes 6 positional arguments but 7 were given #133

sceniclife opened this issue Sep 22, 2020 · 3 comments · Fixed by #134 or #135

Comments

@sceniclife
Copy link
Contributor

Attempting to copy events will error with the following error:

TypeError: copy_events() takes 6 positional arguments but 7 were given self._target(*self._args, **self._kwargs) self._target(*self._args, **self._kwargs) self.run() File "/Users/viet/.pyenv/versions/3.8.3/lib/python3.8/threading.py", line 870, in run

Environment:
[tool.poetry.dependencies] python = "^3.8" cognite-replicator = "^0.8.1"

@sceniclife
Copy link
Contributor Author

On Pull Request #123: The argument dst_ts was added to the replication.thread parameters. The replication.thread function is used for multiple CDF types… not just TimeSeries. Because dst_ts was added to replication.thread parameters, the Event call to replication.thread is broken

@sceniclife
Copy link
Contributor Author

Possible solution is to add target_external_ids to all other replicate functions like the time_series one. This will add the ability to all other data types (event, filemetadata).

Then change dst_ts to dst_list, make it usable for all other data types.

Then consume the list in the data types copy to only filter out the dst_list

@gaetan-h
Copy link
Contributor

This should now be fixed with #134

This was linked to pull requests Nov 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants