Skip to content

Comments

Add media URL to local path mapping in JSON output#688

Merged
dataabc merged 2 commits intodataabc:masterfrom
sci-m-wang:media-mapping
Feb 2, 2026
Merged

Add media URL to local path mapping in JSON output#688
dataabc merged 2 commits intodataabc:masterfrom
sci-m-wang:media-mapping

Conversation

@sci-m-wang
Copy link
Contributor

@sci-m-wang sci-m-wang commented Feb 1, 2026

原来的版本无法直接保存图片/视频与文本的映射关系,微调了一下代码,可以直接建立图片/视频与帖子的对应关系,便于分析研究

Comment on lines 13 to 28
# Structured media info for better URL <-> local mapping
self.original_pictures_list = []
self.retweet_pictures_list = []
self.media = {
'original_pictures': [],
'retweet_pictures': [],
'video': []
}
# Structured media info for stronger URL<->local mapping
self.original_pictures_list = []
self.retweet_pictures_list = []
self.media = {
'original_pictures': [],
'retweet_pictures': [],
'video': []
}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

感谢贡献代码,很有帮助的特性,不过这两段是不是重复了?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

是的,没注意😂,我来修改一下

Removed unused media-related attributes from the class.
@dataabc dataabc merged commit 45db049 into dataabc:master Feb 2, 2026
@dataabc
Copy link
Owner

dataabc commented Feb 2, 2026

感谢贡献代码,已合并。

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants