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

feat: show visualmap of contract's h24 tx count #297

Merged
merged 6 commits into from
Jun 21, 2024

Conversation

zmcNotafraid
Copy link
Member

@zmcNotafraid zmcNotafraid requested a review from Keith-CY May 7, 2024 09:19
Copy link

vercel bot commented May 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ckb-explorer-frontend-in-magickbase-repo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 21, 2024 8:33am

@Keith-CY Keith-CY requested a review from FrederLu May 7, 2024 10:52
@Keith-CY
Copy link
Member

Keith-CY commented May 8, 2024

@Sven-TBD please have a review

@FrederLu
Copy link

FrederLu commented May 8, 2024

image image

https://ckb-explorer-frontend-in-magickbase-repo-q965n6zra-magickbase.vercel.app/en/script/0xf329effd1c475a2978453c8600e1eaf0bc2087ee093c3ee64cc96ec6847752cb/type?page=2&size=10

1、Counting this '0x177b2ae15abf7e6bae568e2f32a04b928c56cc3499010970b59d4104934385f5' transaction data, there are 17 pieces of data, less than 23 pieces of data.
2、The address shown above is 465. How was this data obtained?


2024-05-08.16.45.39.mov

3、If there is multiple data on the vertical axis, it is difficult to switch to other data. Only one data information can be displayed when selecting up and down.


https://pudge.explorer.nervos.org/scripts#Unique%20Cell

https://ckb-explorer-frontend-in-magickbase-repo-q965n6zra-magickbase.vercel.app/en/script/0x8e341bcfec6393dcd41e635733ff2dca00a6af546949f70c57a706c0f344df8b/type

image

4、Unique Cell is a registered script, and script name information needs to be displayed on the chart.

@zmcNotafraid
Copy link
Member Author

image image
https://ckb-explorer-frontend-in-magickbase-repo-q965n6zra-magickbase.vercel.app/en/script/0xf329effd1c475a2978453c8600e1eaf0bc2087ee093c3ee64cc96ec6847752cb/type?page=2&size=10

1、Counting this '0x177b2ae15abf7e6bae568e2f32a04b928c56cc3499010970b59d4104934385f5' transaction data, there are 17 pieces of data, less than 23 pieces of data. 2、The address shown above is 465. How was this data obtained?

2024-05-08.16.45.39.mov
3、If there is multiple data on the vertical axis, it is difficult to switch to other data. Only one data information can be displayed when selecting up and down.

https://pudge.explorer.nervos.org/scripts#Unique%20Cell

https://ckb-explorer-frontend-in-magickbase-repo-q965n6zra-magickbase.vercel.app/en/script/0x8e341bcfec6393dcd41e635733ff2dca00a6af546949f70c57a706c0f344df8b/type

image 4、`Unique Cell` is a registered script, and `script name` information needs to be displayed on the chart.
  1. h24 tx count was refreshed by job every 1h, and the data source from cell_dependencies table
  2. contract's referering cell outputs
  3. fixed.
  4. will update contract's name attribute

@FrederLu
Copy link

FrederLu commented May 13, 2024

1、The newly added color information will be lost after refreshing the page.

2024-05-13.15.16.38.mov

2、After updating this question, it still has the same issue as before.

image

@zmcNotafraid
Copy link
Member Author

1、The newly added color information will be lost after refreshing the page.

2024-05-13.15.16.38.mov
2、After updating this question, it still has the same issue as before.

image

The second problem has fixed.But the first one I can't resolve it.When I refresh page, I found the echarts has load twice.First time no dataset.Please help me @Keith-CY

@PainterPuppets
Copy link

reference: https://stackoverflow.com/questions/61074770/why-isnt-visual-map-drawn

import the visualmap library in /src/pages/StatisticsChart/common/index.tsx and it can be displayed

import 'echarts/lib/component/visualMap'

Signed-off-by: Miles Zhang <[email protected]>
@Keith-CY
Copy link
Member

LGTM

@Keith-CY Keith-CY merged commit 7412ce9 into Magickbase:develop Jun 21, 2024
3 checks passed
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.

4 participants