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

Add Samson.js close #358 #386

Merged
merged 2 commits into from
Oct 7, 2024
Merged

Add Samson.js close #358 #386

merged 2 commits into from
Oct 7, 2024

Conversation

jiaojiaodubai
Copy link
Collaborator

我在开发过程中仅仅随机抽取几个网站进行了测试,@doubanchan 麻烦你审查一下兼容性

@yfdyh000
Copy link
Contributor

yfdyh000 commented Oct 5, 2024

@jiaojiaodubai
Copy link
Collaborator Author

/oa/DArticle.aspx?type=view&id= 系列是否要一并兼容?

看了一下啊,这些页面对元素进行了良好的标记,而且 PDF 下载链接也是直接给出链接,兼容难度不大,如果能多给几个例子会更好。

@jiaojiaodubai jiaojiaodubai linked an issue Oct 5, 2024 that may be closed by this pull request
1 task
@yfdyh000
Copy link
Contributor

yfdyh000 commented Oct 5, 2024

/oa/DArticle.aspx?type=view&id= 系列是否要一并兼容?

看了一下啊,这些页面对元素进行了良好的标记,而且 PDF 下载链接也是直接给出链接,兼容难度不大,如果能多给几个例子会更好。

我随便搜到的,Google里还有一些,不太清楚更多例子指什么样的。

@doubanchan
Copy link

#358 中列出来的期刊论文页面都能抓取。

@jiaojiaodubai
Copy link
Collaborator Author

/oa/DArticle.aspx?type=view&id= 系列是否要一并兼容?

已兼容,请审查。有些期刊(如例1例2)的数据本身没处理好,未予以考虑。

@yfdyh000
Copy link
Contributor

yfdyh000 commented Oct 6, 2024

/oa/DArticle.aspx?type=view&id= 系列是否要一并兼容?

已兼容,请审查。有些期刊(如例1例2)的数据本身没处理好,未予以考虑。

看上去OK。

@jiaojiaodubai jiaojiaodubai merged commit b23e33f into l0o0:master Oct 7, 2024
1 check failed
@jiaojiaodubai jiaojiaodubai deleted the Samson branch October 7, 2024 10:28
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.

samsoncn.com/西安三才
3 participants