8000 Numeric String Type by norberttech · Pull Request #1719 · flow-php/flow · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Numeric String Type #1719

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

Merged
merged 2 commits into from
Jun 17, 2025
Merged

Numeric String Type #1719

merged 2 commits into from
Jun 17, 2025

Conversation

norberttech
Copy link
Member

Resolves: #1717

Change Log


Added

  • NumericString Type to Flow Types library

Fixed

Changed

Removed

Deprecated

Security

@norberttech norberttech linked an issue Jun 17, 2025 that may be closed by this pull request
8000
Copy link
Contributor
github-actions bot commented Jun 17, 2025

Flow PHP - Benchmarks

Results of the benchmarks from this PR are compared with the results from 1.x branch.

Extractors
+-----------------------+------------------------+------+-----+-----------------+------------------+-----------------+
| benchmark             | subject                | revs | its | mem_peak        | mode             | rstdev          |
+-----------------------+------------------------+------+-----+-----------------+------------------+-----------------+
| CSVExtractorBench     | bench_extract_10k      | 1    | 3   | 4.793mb +0.04%  | 417.882ms +0.47% | ±0.96% +55.80%  |
| ExcelExtractorBench   | bench_extract_10k_ods  | 1    | 3   | 65.503mb +0.00% | 1.064s +0.54%    | ±0.73% -18.89%  |
| ExcelExtractorBench   | bench_extract_10k_xlsx | 1    | 3   | 67.549mb +0.00% | 1.670s -0.51%    | ±0.44% -59.28%  |
| JsonExtractorBench    | bench_extract_10k      | 1    | 3   | 5.077mb +0.04%  | 1.166s -0.36%    | ±0.78% -18.96%  |
| ParquetExtractorBench | bench_extract_10k      | 1    | 3   | 86.347mb +0.00% | 909.113ms -1.55% | ±0.23% -26.54%  |
| TextExtractorBench    | bench_extract_10k      | 1    | 3   | 4.524mb +0.04%  | 44.622ms +15.66% | ±1.14% +175.76% |
| XmlExtractorBench     | bench_extract_10k      | 1    | 3   | 4.519mb +0.04%  | 610.160ms +0.35% | ±0.46% -67.03%  |
+-----------------------+------------------------+------+-----+-----------------+------------------+-----------------+
Transformers
+---------------------------------+--------------------------+------+-----+------------------+-----------------+----------------+
| benchmark                       | subject                  | revs | its | mem_peak         | mode            | rstdev         |
+---------------------------------+--------------------------+------+-----+------------------+-----------------+----------------+
| RenameEachEntryTransformerBench | bench_transform_10k_rows | 1    | 3   | 18.523mb +0.01%  | 72.083ms -0.90% | ±0.61% -2.44%  |
| RenameEntryTransformerBench     | bench_transform_10k_rows | 1    | 3   | 123.261mb +0.00% | 64.651ms -0.88% | ±1.27% -38.33% |
+---------------------------------+--------------------------+------+-----+------------------+-----------------+----------------+
Loaders
+--------------------+----------------+------+-----+------------------+------------------+-----------------+
| benchmark          | subject        | revs | its | mem_peak         | mode             | rstdev          |
+--------------------+----------------+------+-----+------------------+------------------+-----------------+
| CSVLoaderBench     | bench_load_10k | 1    | 3   | 62.453mb +0.00%  | 83.428ms -1.51%  | ±0.92% -12.69%  |
| JsonLoaderBench    | bench_load_10k | 1    | 3   | 79.754mb +0.00%  | 100.485ms -1.44% | ±1.36% +829.13% |
| ParquetLoaderBench | bench_load_10k | 1    | 3   | 166.230mb +0.00% | 20.503s -0.85%   | ±0.21% +47.53%  |
| TextLoaderBench    | bench_load_10k | 1    | 3   | 17.830mb +0.01%  | 30.425ms -0.68%  | ±1.13% +434.75% |
+--------------------+----------------+------+-----+------------------+------------------+-----------------+
Building Blocks
+-------------------+----------------------------+------+-----+------------------+------------------+-----------------+
| benchmark         | subject                    | revs | its | mem_peak         | mode             | rstdev          |
+-------------------+----------------------------+------+-----+------------------+------------------+-----------------+
| TypeDetectorBench | bench_type_detector        | 1    | 3   | 42.096mb +0.00%  | 429.093ms -0.49% | ±0.32% -40.07%  |
| TypeDetectorBench | bench_type_detector        | 1    | 3   | 11.473mb +0.02%  | 85.355ms -0.63%  | ±0.96% +64.19%  |
| EntryFactoryBench | bench_entry_factory        | 1    | 3   | 101.802mb +0.00% | 650.048ms +0.78% | ±0.69% +144.22% |
| EntryFactoryBench | bench_entry_factory        | 1    | 3   | 53.152mb +0.00%  | 328.072ms -0.08% | ±0.48% -69.46%  |
| EntryFactoryBench | bench_entry_factory        | 1    | 3   | 14.402mb +0.01%  | 70.545ms +0.02%  | ±1.85% +63.40%  |
| RowsBench         | bench_chunk_10_on_10k      | 2    | 3   | 93.414mb +0.00%  | 3.263ms -9.16%   | ±2.73% +316.56% |
| RowsBench         | bench_diff_left_1k_on_10k  | 2    | 3   | 110.785mb +0.00% | 238.942ms +1.26% | ±0.81% +582.97% |
| RowsBench         | bench_diff_right_1k_on_10k | 2    | 3   | 93.505mb +0.00%  | 24.361ms +2.65%  | ±0.96% +49.22%  |
| RowsBench         | bench_drop_1k_on_10k       | 2    | 3   | 94.289mb +0.00%  | 1.350ms -27.24%  | ±2.28% +24.45%  |
| RowsBench         | bench_drop_right_1k_on_10k | 2    | 3   | 94.289mb +0.00%  | 1.329ms -32.62%  | ±1.35% -48.37%  |
| RowsBench         | bench_entries_on_10k       | 2    | 3   | 92.450mb +0.00%  | 3.362ms -6.09%   | ±0.95% +29.12%  |
| RowsBench         | bench_filter_on_10k        | 2    | 3   | 92.978mb +0.00%  | 15.155ms -7.75%  | ±1.15% -49.44%  |
| RowsBench         | bench_find_on_10k          | 2    | 3   | 92.978mb +0.00%  | 15.358ms -5.92%  | ±1.10% -23.82%  |
| RowsBench         | bench_find_one_on_10k      | 10   | 3   | 91.667mb +0.00%  | 2.000μs +4.93%   | ±0.00% -100.00% |
| RowsBench         | bench_first_on_10k         | 10   | 3   | 91.667mb +0.00%  | 0.400μs 0.00%    | ±0.00% 0.00%    |
| RowsBench         | bench_flat_map_on_1k       | 2    | 3   | 100.728mb +0.00% | 13.979ms -11.56% | ±0.26% -91.60%  |
| RowsBench         | bench_map_on_10k           | 2    | 3   | 130.155mb +0.00% | 66.507ms -2.53%  | ±0.74% -49.81%  |
| RowsBench         | bench_merge_1k_on_10k      | 2    | 3   | 93.498mb +0.00%  | 1.218ms -25.39%  | ±3.32% +11.82%  |
| RowsBench         | bench_partition_by_on_10k  | 2    | 3   | 96.867mb +0.00%  | 62.289ms -1.25%  | ±1.67% +585.56% |
| RowsBench         | bench_remove_on_10k        | 2    | 3   | 94.551mb +0.00%  | 3.424ms -15.56%  | ±2.09% -28.16%  |
| RowsBench         | bench_sort_asc_on_1k       | 2    | 3   | 92.030mb +0.00%  | 39.589ms -3.60%  | ±1.01% -57.90%  |
| RowsBench         | bench_sort_by_on_1k        | 2    | 3   | 92.030mb +0.00%  | 40.090ms +1.06%  | ±0.56% -69.92%  |
| RowsBench         | bench_sort_desc_on_1k      | 2    | 3   | 92.030mb +0.00%  | 39.949ms -1.18%  | ±1.27% +23.72%  |
| RowsBench         | bench_sort_entries_on_1k   | 2    | 3   | 94.110mb +0.00%  | 8.040ms -3.18%   | ±1.12% -46.52%  |
| RowsBench         | bench_sort_on_1k           | 2    | 3   | 91.860mb +0.00%  | 29.815ms -4.61%  | ±0.76% -54.62%  |
| RowsBench         | bench_take_1k_on_10k       | 10   | 3   | 91.667mb +0.00%  | 13.600μs -7.69%  | ±1.20% -14.29%  |
| RowsBench         | bench_take_right_1k_on_10k | 10   | 3   | 91.667mb +0.00%  | 15.500μs -7.99%  | ±0.53% -79.39%  |
| RowsBench         | bench_unique_on_1k         | 2    | 3   | 110.786mb +0.00% | 246.701ms +2.80% | ±1.07% -2.49%   |
+-------------------+----------------------------+------+-----+------------------+------------------+-----------------+

Copy link
codecov bot commented Jun 17, 2025

Codecov Report

Attention: Patch coverage is 0% with 23 lines in your changes missing coverage. Please review.

Project coverage is 81.91%. Comparing base (4d303c2) to head (66185fd).
Report is 2 commits behind head on 1.x.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##              1.x    #1719      +/-   ##
==========================================
- Coverage   82.01%   81.91%   -0.10%     
==========================================
  Files         709      710       +1     
  Lines       19302    19325      +23     
==========================================
  Hits        15830    15830              
- Misses       3472     3495      +23     
Components Coverage Δ
etl 88.36% <ø> (ø)
cli 84.42% <ø> (ø)
lib-array-dot 94.53% <ø> (ø)
lib-azure-sdk 62.56% <ø> (ø)
lib-doctrine-dbal-bulk 93.49% <ø> (ø)
lib-filesystem 78.02% <ø> (ø)
lib-types 54.82% <0.00%> (-0.94%) ⬇️
lib-parquet 84.37% <ø> (ø)
lib-parquet-viewer 82.02% <ø> (ø)
lib-snappy 90.69% <ø> (ø)
bridge-filesystem-async-aws 90.38% <ø> (ø)
bridge-filesystem-azure 89.92% <ø> (ø)
bridge-monolog-http 98.11% <ø> (ø)
symfony-http-foundation 74.41% <ø> (ø)
adapter-chartjs 86.45% <ø> (ø)
adapter-csv 90.37% <ø> (ø)
adapter-doctrine 89.95% <ø> (ø)
adapter-elasticsearch 97.19% <ø> (ø)
adapter-google-sheet 83.87% <ø> (ø)
adapter-http 59.15% <ø> (ø)
adapter-json 90.62% <ø> (ø)
adapter-logger 53.84% <ø> (ø)
adapter-meilisearch 97.75% <ø> (ø)
adapter-parquet 78.64% <ø> (ø)
adapter-text 84.44% <ø> (ø)
adapter-xml 83.15% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@norberttech norberttech merged commit 9c8a4de into 1.x Jun 17, 2025
20 of 22 checks passed
@norberttech norberttech deleted the 1717-proposal-new-type-numeric-string branch June 17, 2025 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Proposal]: new type: numeric-string
1 participant
0