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 (Encyclopedia of China 3rd.js) 新增中国大百科全书 #141 #273

Merged
merged 5 commits into from
Jan 18, 2024

Conversation

jiaojiaodubai
Copy link
Collaborator

@jiaojiaodubai jiaojiaodubai commented Jan 6, 2024

  • 适配首页和专题页
  • 更新内部函数
  • 更新测试用例

@jiaojiaodubai jiaojiaodubai changed the title add (Encyclopedia of China 3rd.js) 适配首页和专题页;更新内部函数;更新测试用例 add (Encyclopedia of China 3rd.js) 新增中国大百科全书 #141 Jan 6, 2024
@doubanchan
Copy link

@jiaojiaodubai 中国大百科全书的条目作者有些格式不同,需要改一下,比如例1。可以再增加一个多人作者的testcast,例2

@jiaojiaodubai
Copy link
Collaborator Author

中国大百科全书的条目作者有些格式不同,需要改一下,比如例1

例1的问题已修复;

可以再增加一个多人作者的testcast,例2

原来已经有一个多作者的例子,这个和原来的有什么区别吗?

@doubanchan
Copy link

之前的case中的作者含有“撰”、“修订“后缀,这个没有。

@jiaojiaodubai
Copy link
Collaborator Author

好怪,现在这个selector,我在浏览器里匹配是对的,但scaffold里总是匹配多个,还得继续改进

@doubanchan
Copy link

@jiaojiaodubai 发现一个问题,case 4(学科列表)未能识别成“multiple”。一个提议,是否在extra增加“英文名称”字段,虽然有时翻译不是很准确,但是大多数应该和英语常见标题一致或类似。如果增加英文字段,有一个问题,比如蒯祥,其中拼音字母a用了U+0251,而不是最常用的U+0061

@jiaojiaodubai
Copy link
Collaborator Author

我现在访问你上面给的两个链接都是502 504异常,过后我再试试

@doubanchan
Copy link

发现好像是整个网站502/504了,主页也是。

@jiaojiaodubai
Copy link
Collaborator Author

case 4(学科列表)未能识别成“multiple”。

实测可以检测为“multiple”。

一个提议,是否在extra增加“英文名称”字段

之前已经有original-title可用,不过selector写得有问题就没匹配上,我再改改。

有一个问题,比如蒯祥,其中拼音字母a用了U+0251,而不是最常用的U+0061

我想,我们不能对数据库的内容负责。

@doubanchan
Copy link

case 4的问题,我清楚浏览器缓存之后好了,暂不清楚原因。第二个拼音字母的话,其他条目似乎都是用U+0061,倾向是编写者的笔误(大百科全书中看得多之后,是能发现错误的)。为什么用“original-title”,而不是trans-title或者eng(lish)-title,按理说中文才是original-title。

@jiaojiaodubai
Copy link
Collaborator Author

为什么用“original-title”,而不是trans-title或者eng(lish)-title,按理说中文才是original-title。

#257

@jiaojiaodubai jiaojiaodubai merged commit 427effd into l0o0:master Jan 18, 2024
@jiaojiaodubai jiaojiaodubai deleted the Encyclopedia branch January 19, 2024 10:08
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