-
-
Notifications
You must be signed in to change notification settings - Fork 245
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
update 三期型、四期型 #172
Open
NullYing
wants to merge
3
commits into
mikebryant:master
Choose a base branch
from
NullYing:patch-1
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
update 三期型、四期型 #172
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -22,8 +22,8 @@ | |
"decreasing": "递减型", | ||
"fluctuating": "波动型", | ||
"unknown": "不知道", | ||
"large-spike": "大幅上涨(三期型)", | ||
"small-spike": "小幅上涨(四期型)" | ||
"large-spike": "大幅上涨", | ||
"small-spike": "小幅上涨" | ||
}, | ||
"prices": { | ||
"description": "本周你的岛上大头菜的购买价格是多少?<i>(如果你是第一次购买大头菜,这个字段不可用)</i>", | ||
|
@@ -42,15 +42,15 @@ | |
"wednesday": "周三", | ||
"thursday": "周四", | ||
"friday": "周五", | ||
"saturday" : "周六", | ||
"saturday": "周六", | ||
"sunday": "周日", | ||
"abr": { | ||
"monday": "周一", | ||
"tuesday": "周二", | ||
"wednesday": "周三", | ||
"thursday": "周四", | ||
"friday": "周五", | ||
"saturday" : "周六" | ||
"saturday": "周六" | ||
} | ||
}, | ||
"times": { | ||
|
@@ -77,4 +77,4 @@ | |
"contributors-text": "哦!别忘记感谢那些至今为止做出过贡献的人。", | ||
"contributors": "贡献者" | ||
} | ||
} | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. nit: should have new line at end of file. |
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In fact, I think it would be better to use
三期型
and四期型
(Remove大幅上涨
and小幅上涨
).There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
无法理解啥是三期型四期型,可以麻烦解释一下?对于普通人来说,跟我一样有疑问的会有更多
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
简单来说,三期型上涨三次,四期型上涨四次。
#166
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
那么我觉得可以翻译成这样,而不是简单的三期型和四期型
三期型(上涨三次)
四期型(上涨四次)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
不经过解释的话,很多人都不会知道这两词的意思
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
三期型(大幅上涨三次)
四期型(小幅上涨四次)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's too many word
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps it would be better to explain the patterns by adding title attribute.
(Or add mouse hover tips in other ways)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think 大幅上涨三次 and 小幅上涨四次 is enough to explain, and it easier for regular people to understand.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is an issue open #41 for adding tooltip/hover that explains how the pattern works.
The idea behind the title is to give a simple name that explains that overall result of the pattern, not each phase of that pattern.
For example, "large spike" means the overall result is a big increase in sell price, but doesn't indicate how many periods the increase will last.