Comments (3)
Hi bạn,
Mình phản hồi 1 số ý như sau:
- Trong giai đoạn phát triển vnstock, mình cũng ưu tiên có tính năng và truy xuất được dữ liệu chính xác so với dữ liệu gốc từ nguồn. Việc optimize hoặc bổ sung data model có thể thực hiện khi có nguồn lực và có thể có chút ý niệm về việc tính năng cần làm có được nhiều người quan tâm hay không (việc này có thể xem thông qua phân tích lượt xem trang và thời gian xem của người dùng qua Google Analytics)
- Không cần thiết sử dụng label tiếng Việt cho các cột dữ liệu bởi trong thực tế code sử dụng tiêu đề tiếng Việt dài dòng và không cần thiết. Việc cung cấp nghĩa của cột có thể bổ sung trong tài liệu hướng dẫn. Nếu xét về việc dữ liệu có giá trị và cần lưu vào database thì lựa chọn tốt vẫn là lưu bằng tiêu đề tiếng Anh chứ ko phải tiếng Việt.
- Mình sẽ bổ sung thêm các bước validate dữ liệu đầu vào khi phát triển thêm code trong giai đoạn tới. Sử dụng pydantic hay không thì mình sẽ nghiên cứu thêm vì phần này mình cũng chưa dùng nhiều.
from vnstock.
Cảm ơn b đã tham gia thảo luận.
ý 2:
- chỉ nói riêng trong funds.py, thì b cũng đồng ý là nên bỏ phần logic đổi tên cột cho đơn giản. M hiểu vậy có đúng ko nhỉ?
ý 1:
- Nói data model thì nghe to bự quá, chứ thực ra thì m muốn chỉ là chuẩn hóa việc đặt tên cột cho df output thành snake_case thay vì để ở dạng Title Case kèm 2 lựa chọn Anh Việt như hiện tại. Và sẽ chỉ mapping lại cột bằng pandas như chúng ta đang làm thôi. M sẽ ko ứng dụng thử nghiệm pydantic. Nếu b ngại cho người dùng cuối phải thêm cài nhiều package.
- vd: thay vì để column label mặc địch từ api là
product.navGrowth6month
, hoặc dịch sang dạng title case kiểuNAV change 6 month
thì sẽ chuẩn mực hóa thànhfund_nav_6_month
- sau này b có hứng làm data model thì cũng chỉ cần lấy cái tên cột đã chuẩn mực hóa. Và sẽ ko cần phải cập nhật lại docs về phần tên cột.
Nếu b đồng thuận là m hiểu đúng ý b như đã diễn giải ở trên thì m có thể pr thêm cho funds.py?
Mục tiêu tuần tự là:
- bỏ logic đổi tên cột
- chuẩn hóa tên cột về dạng snake_case
- Sau khi đạt được đồng thuận về tên đã chuẩn hóa thì cập nhật docs về thay đổi lại tên cột
from vnstock.
We achieve a consensus on:
- standardize column label to snake_case
- remove unnecessary and ambiguous input param
lang
,mode
,decor
, from functions in modulefunds.py
These changes were implemented in PR #108
Close discussion.
from vnstock.
Related Issues (20)
- VNINDEX chỉ số ra số sai HOT 1
- Chưa đoc kỹ tài liệu
- FR: crawl data related to mutual funds from https://fmarket.vn/ HOT 4
- Báo lỗi cho tính năng HOT 2
- Dữ liệu vnindex chỉ đến năm 2020 HOT 2
- FR: implement QA / test for vnstock HOT 2
- Không thể import thư viện HOT 7
- "financial_report" not working. HOT 1
- How can get outstanding shares data for every year? HOT 1
- Please add unit for financial report HOT 1
- VNindex không hiển thị trước 2020 HOT 2
- Hàm `fund_details` vẫn hiển thị kết quả khi tên mã quỹ bị gõ thiếu HOT 3
- Dữ liệu data khung thời gian nhỏ có vấn đề HOT 3
- `financial_ratio_compare` trả về "Excel file format cannot be determined, you must specify an engine manually." HOT 1
- Error in API response HOT 2
- Cải thiện error msg cho hàm `dividend_history` khi cổ phiếu không trả cổ tức HOT 2
- Hàm finance.income_statement và finance.ratio không nhận tham số symbol HOT 3
- Lỗi khi lấy dữ liệu giá cổ phiếu HOT 1
- Lỗi khi lấy giá lịch sử theo tháng
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from vnstock.