Auto-translated map_item updates from 4CAT: twitter-import - #102
Draft
4cat-to-zeeschuimer-automation-pr[bot] wants to merge 1 commit into
Draft
4cat-to-zeeschuimer-automation-pr[bot] wants to merge 1 commit into
4cat-to-zeeschuimer-automation-pr[bot] wants to merge 1 commit into
Conversation
…9bbe8814c3c85009f6d12
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Generation parameters
gpt-oss-120b(provider:litellm)7cba15fto 4CAT master (module:twitter-import).Summary
datasources/twitter-import/search_twitter.pymodules/twitter.jsThe following datasources translated successfully but the static lint flagged issues that need human fixes. The auto-generated code was spliced into the JS module as-is; please patch the file directly in this PR.
datasources/twitter-import/search_twitter.py->modules/twitter.js[]or{}is false in Python and true in JavaScript, so this takes the opposite branch on an empty list, and the code inside then indexes element[0]of it. UseArray.isArray(x) && x.length(orObject.keys(x).length). NOTE: the check goes by the name, so a plural holding a string or a boolean is a false positive — ignore the warning if the value is not a list or an object..toFixed(found. It pads with zeros where Python'sstr(round(x, n))does not, turning a coordinate like4.85into4.850000. NOTE: the corrected formString(Number(x.toFixed(n)))contains.toFixed(as well — ignore the warning where the result is already wrapped inNumber(.datasources/twitter-import/search_twitter.py->modules/twitter.js(48.63s)LLM commentary:
Python diff