[I] [Bug] canOmitUnusedDimensions will mess up the dimension indexes [echarts]

via GitHub Thu, 09 Jan 2025 02:00:38 -0800


Justin-ZS opened a new issue, #20672:
URL: https://github.com/apache/echarts/issues/20672


   ### Version
   
   5.6.0
   
   ### Link to Minimal Reproduction
   
   
[Demo](https://echarts.apache.org/examples/en/editor.html?code=PYBwLglsB2AEC8sDeAoWsDGwA2wBOAXLANprqwDkAxACwDMA7AJwDCAohQDRnrVNMAhABwAxAIxcelKiICCAEQGNJ5aSJZCAbAAYATCvLURchjSEHeVBroEBWNgIvSGQlvLbyn1WSyay2-tyqRj50miJeMvIMIjEUZAC6QbAAnrIAHhAAzkSoqmApIACmRBQAbgCG2ACuRU4gwFkQkDCl2EUAZmBOFZlZADIVAEZF2LlS6AAWEAAmRQDyZUV42BUgRGB4tVIAvmQ7yekZ2USkqnmqsAXFpRgVYEUA5vgpTugNTS3QpUPAYGDAAC2byuwBwkHWyAm5Cyk2AAHcNlsislLpgYB0INASldkajLpsII9HstSs0isD8aoPs0oN9KACQPE0XtVAdEsksssIEUciQpBd8oUcRQhhU8CCsmAKhgANalKUy2WS0ZFDAPGYAWWAc1KgOq2Ah7RBFJAkwqTT5gsu5MBAGUCu1xmjVKAZc0UkRtAA6JjQ9Csy7stFc9rq50u20OlJOqEu8hujAeohiKmXX54OZ4AAqwoVOFmINUGazAHVZmBJkRdP7YMGXasRmM4_HYInk7AxLXA2y09AKoCRbo9NpAC56gBpvMSAY21AGV6AFpAACpgFyMgD6DylGGwJugWF1LbRnpIuiStfSpzECWh9dUjdGEZDcMRuNqafQHXwgPuD0IlGZQbTt7YIMKTANUYD3pc0xzIsyyrJCmzbCyuz4ta6DXCKYoSmmipygq0pyiqYYatqe4UPqhoQCAxppqa5qWhBqhRo6OKoZc7YFF6vrdle2GquG-42g89rMQxbEgO6HGdq-5AlssuY3JQWQFjMRYyfgZYVlWsA1vG16XIBomuuJSaSV2uk8dC_aDqUw7DhOuizouq7rmAm7bruLG1oexA0Ce8ZniQF4WWiBkCaosIIkiL
 
61u-eCfv8pK_sF-nDKMwGgeBYXkFBCxLCsaxRUUV7IQK0LoT84qSvh8qKdVhFqsROoiuRRp1DRgJmhaJxZegTExp5rZtsZHY-n65ksrxRGGbAfWxqxRkSYeqa1ugsk5nminKapq3qcs5YzJW1bdtJoXzQmw2mdxE2WQOQ4jhOdCOcua68q5W40TuTXTd5mh-S6AXEEF10halzZnRFT4ISiMUfl-iUUH-vbQoB6VgdNOUwfl8HIsVSPnGVG2ipV2HVXhSr1eqRRal9lAtZR1HQrRXVWrWs0Da27GHqNV3_tCoYNdNbPTegnMptJO2ZnJhNKdghbi7Aa37Yd2nHbWp0rUNi0pjzeOXFZd12eONBPc5r1uR9HnfacQh_WiANA7zINNtNEOFdJsXxd-pSI-QenoCjFQgWjPWwBjeVwYVuO-yhBMKUTWF86TtXk5NDVUyRzUGq1JodXR3VnULIcixdXNceNjuXPz_EF0J0ZzRrotSRra3ySKMty83u14ErWk6S6fvkOrg2N2Z_fJao-s2XoYgTtO87PS55uM59e5nd5YjaLblz25ewMpc7Ieu8-0Pxh7cM_gj4-D6DqOZWdYewQVx9RwGMdouVlCYVVSpkwRqeU9TUidMqJtUZrnZmgta4iSLprEypcxpjz3uFPid9WZQP6sLWBHZlqDRbtLLa8tFaaSOrpE6oNMEjx1tHG61lKC2RnuOBy89TYbnesvS2Id17HmkjvK-_tyGH0fG7GGcVz7ez4bAAOQdUHxgfljSOSFdboFYh_eO39cLJz_nzFB6caZkSzvTUBaImb0RDoXM6xctawG5uXJRMIUGQIpHXdm8YR6EK7q3fMssVLuMlt3YhKtSFqwERYrBl1bHULRJPOh08Homxeqw9yNM17nl8jw88u8K43hCbWI-UN3awwShfH2r9kY30DhldGsxcqP2xohIMJV8bv0Jl_EmP9NHKn_o1IBBiQE506qYmuTjoGhMbjYxBWT7FTTMe
 
g-uw8S5i07n4zxm1vHbQVl3HuJD-5kIPqMhZnYqGlKibdKeugGHG2YQkt6STV5eXPL9dJgVMl2P4Xs3JQjj4FNEUU8RSDr5NlvlU6C4cn5QxfnWN-BIWnE0Tu0igOFOnaKIronpFE-ntQGfnNBwyMEwLGWXCZrzYBVxkZGWZLiXRuKWVmFZCKCE0r2gEvu_zsnvPmVY0eijJnoGiRQWyuhZ7xMXmw4xK9KWqHXjbJ5gMXmRP3neQRkUvkiM9vDEpkKymAoqcHe-1TMYR2fty455AVEwoTiGJOCK6pdNRZndFDNjHgMGTi4SeL9lWPGca-VyDplDLdXMjmBycGtjwXHduPjGX-IOr3VW8Yh5Bs5UczVJzaH8r0IKxhwqzaisuEUcVVsjybxlQ7YlCaHzKvyaqsRSVWWSPKdI4FNT5FGoaXYs1cdWlwo0dalOyK06APtdnTFecWbxnMQ3A5Xq208pJQ4mZuLA2uODb42l-C1mrqZTG7Zda3mKo9XA7WESTUT1OTE3QmbHpXJFbciV5BvIXq3qoXhdby2V0-VW0-hSva1tnVIypIc5GGvBd6vSHaMKwstfCxFFNulDsMf00djiA13oWoe6xhLvUnqmQLBdKGKErqjXSiN6yiHbsCTs4J7LE3oa5TO4lfKBUTkuU5a5S8xUcJSUeNJp4MkSLfeFD9eJq2_N_WWhtAG9UgtqQo-jKbTWxwgxayuVqYO2sHXqXpjq83OuxeOilBHPWYbk9h9ApLkPOMM7Rzd61w0MtwZs5lcaGw5I5bR5NA9GN6DoEK69Obb2FuILoR5vHnn8dcy6PJwmv0_J_ZfV9EndW1iA2CnGoGoVCk7ZBlT0GbX9oARnTTDqjE6axWO8li7UPnSMwgrD8nfW4f9ZZ_FhGHPLPXR3NrGlyMstnXusGk6k3Hvq-QLzugfNZr84ki2yT7lHmlaF2V4XqMVshtFl0Z9RPxb_YlslaIUt1KKul0qzSsvKfCqpvLIYdE
 
adplpkrqgTF6Yq_hlrNWPNdIsyMwb1miMdcjV1rdytevEv61Z7BH2aEGwm1e1jN6Zt3PjN5OgxbFulp9QC_dHzK3rbRJtuLGqB7_qS7I_VoLDsQrA4piq52YSXb7ddlFt39HFcQxAvDzWD0jWM2yCR5mOffbc9gmzxH7Ohscz15zTssdC_CUSjHvKz3pvG8x7N032GzaR6cOg3C0dytM_Wlb76cfRRi2q4py20o6r25BMnMnW28_bdTygstsQgj5QFkOO9pLI98nzlIgJfjYDtBAAAXjibQI72fgzuLGDoVQuTSRmNkYY7QZiyeJRO-MWBcAXxkCIbQhftCE4AhF1bwizc1oRkn7I0odw4lsAU6AYBSxFCJJMTKoocAqUt0Ba3TaDWpfqWWrERROe1nhAEyPtYc_4FKPnovxfKccjII8PAswIKMiIEIfE7Qujb_xOvx4HeD9SF-P8IEp_VBYGbxULEgwD4gfkwCcElFN9H5JBfXo2QnA38xNiCvcgb_LIAABWACxB_XA1KGgBgDqEaXIHx0SgAAoQAABKBAAAPlgBAGIE3m9H1nxCi3qXrBmHuAtCKEyjOAUxDFAjwAwCKAAAkigKgswiB49sBE9oRk9BxoAmgYA-QqC0RQk-VABT00AHxzWAJA0cVA9ZVRSAQcdZUPWAulPABEDVA3ZRDWPlQAck1AAuZVkMJnUixCqEUOUMJlUPhHUJGyaVbDGzHEnDVxuXlk21yAHkyxFA6FwHuFMOxDpUHEJAwCsLcJgTsN8zh382cMKVcPllUU8OAG8PliUN8MJlnzwAEFeCl0uGEKV1snsKYXCPV0GhcOQGCLQkJjiISI1iSKKD8IoPX0CMyJsPjFCMmwKKcI1mKKQFKKuHKK8O6ESLMLjlSPSKCOkmyLTVyLiSm3aKKKiJKJiN6PiP6KqMGOajqIgAaKCXjHGINnsNhwXgiI6LmK6IWLjgqOWMGmqLpWGIyK2JdB2LOXsJYwOMKNbE
 
6O6NiL6J8JqMJn8PqNGNrAePPSeMcPYzeOOI-MWMqMuNWNuBwHwBGMaOoNsJyNiQcOmLBMrzvkhLOK-IGOSLjj-I2IBO2K0NRPOTCJeJmPBJ-WiI1k-KWO-OuPhLSNuMo1JMGjGwYXyKpMxI2whNOI8LxJWIJLWICJJPuLJImLRJ5JYWpKxLpMGgZOhNbCuJSJZMRLuKEKlINgYX2LlL5LxwFPpKhIuNVNhNpnWM2PZMlM5PJL1NBNzQVPmJNNxMZPxJ-KGI1LZN3RCPtNVwxKdP5NpJdKVNNKZN-KtIlO1LtOlIpKNkdPWXeMFNKHOIjK9Nz01JtJjJRLjMzTnjaMNMuGTNdKFPdJFM9LFP-KRM0NjINnzMTMiJDJONLNTOFJhNFLhMzJ9L6z9LzPskbKOObJxLLJVPjDVMJKjJrL7PrIHMDKTONLDLdLHJdAnJFBuOjKyJ1LOUvUHNmOHJTMoDTI9NqPFOnKBOV13PnKbPikVNbGVLNPHItIoA3PPO3PPUzWeINKDKNIPNbKPPbPNM7MtLPK1K3LrJ3IDMLJ_OLMXPvPDJPPVO7M3KaJdDGwmwLN5JgtUBLKXNHMfNXOfKJOtN9IvNsgwr3JpNvNDPguXIIrRDXK7IRJ7NBxnLOQm1lLY2woQLgvjAfPTKrOJLfIgvPQ4soudJbLwrbPLI7MrKYtZJQuROaPJJh3EuDOosktovwoEr1CnLAtQtTWhymOgoXL_KkoApkqArksoFfP0qUrQpUqgqwtMo0pHOkpXIYqIr0uzPAtzKMoTOvKHNcsPIoGPIrOZOQuEr8tKEAG4DQAQuidKQL_j5YMBqgpQgQ6UARpQtxk0n0SVaD6DThaxBCXRldtAAAdaoQvMQGYYveWCgQAN0VABg7UquqtqvWU0AFU0Hlk6ozW6o1lsA3l0FsAYG9EbwGqGpGrGvlhoGC10CYF0BmrmoWvlhGqYBR00GmoGuYA2q2s5INGwHlmgAOqOpOq0LO
 
v2uwEOvOqutOtupuuususetsIupevuqeruueuaNeu-vereq-rQp-sBr-t-oBqiSBvBpBuBrBr1ghthqhrRHOTCG0CEBB3IDyvQBKrRDKtau0BqokHqsADllQAWjlcb8b1ldB6BtBNAQ14xKaNraaXQxB5qUcmA9rWxmb1rtA2bxrBoxAabbAaBbB2b4x-axBBbhbebWxKbzl-b5YZaxA5aHrPqVbla1aPr1b_rVaNadatbNbQbta9bdaDb9bobDaTbjazaNYGAGAdAddawMb-RBocaqq8aZhAgNYKBCbAB1ZTJvdvWSFtsCYG0AYBmtsCDpDvlvmpDp9ClrpujoYFjvljoB1yYCEDoBFpdBTvmvTszrRDoBoH5poFGrjqzsLs0GLrzvhphonjhtroRurvNqtstshprtGzrvbobvrrbt5Q7t7q7s7p7tgGOoHv7qHpHvHr7uHqnonqbtbrnsbo1jEDoCDp0CEDGodu4GhC3kDBPB2AAG4gA&_source=echarts-doc-preview&version=5.6.0&renderer=svg)
   
   ### Steps to Reproduce
   
   Hover to see the unexpected `NaN` tooltip.
   ```js
     tooltip: {
       trigger: 'axis',
       formatter: (p) => p[0].name,
       show: true
     },
   ```
   
   ### Current Behavior
   
   the `name` of the [tooltip formatter 
params](https://echarts.apache.org/zh/option.html#tooltip.formatter)  is `NaN`
   
   ### Expected Behavior
   
   Show correct category name
   
   ### Environment
   
   ```markdown
   - OS:
   - Browser:
   - Framework:
   ```
   
   
   ### Any additional comments?
   
   It was caused by `canOmitUnusedDimensions`.
   when `canOmitUnusedDimensions` is true
   1. `prepareSeriesDataSchema` only create necessary dimensions
   ![截屏2025-01-09 17 23 
43](https://github.com/user-attachments/assets/929027ca-57e8-4db1-af34-243441a6b230)
   3. `DataStore` is still generated with all dimensions.(see the length of 
`_chunks`)
   ![截屏2025-01-09 17 25 
23](https://github.com/user-attachments/assets/2c1760e1-d3fd-4e5c-bfc3-267d4275aa23)
   5. `SeriesData` uses `_nameDimIdx` to reference the data in `DataStore`. 
       Since unnecessrary dimensions are not created, `_nameDimIdx` is assigned 
the wrong idx
   ![截屏2025-01-09 17 33 
01](https://github.com/user-attachments/assets/130dce2c-32fb-42a6-af88-077a981cb891)
   ![截屏2025-01-09 17 33 
17](https://github.com/user-attachments/assets/c487b373-eca5-4e8c-881a-e84cbc38a209)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

[I] [Bug] canOmitUnusedDimensions will mess up the dimension indexes [echarts]

Reply via email to