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.

Sign up for GitHub

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

Jump to bottom

FeatureBuffer ffm_fields_count removed as it is already present in BlockFFM as ffm_num_fields #107

Merged

ggaspersic merged 1 commit into main from remove_fb_ffm_num_fields

Aug 13, 2023

Collaborator

ggaspersic commented Aug 10, 2023

FeatureBuffer ffm_fields_count has been removed as it is already present in BlockFFM as the ffm_num_fields field.


          FeatureBuffer ffm_fields_count removed as it is already present in Bl…

64de1f6

…ockFFM as ffm_num_fields

ggaspersic requested review from SkBlaz, bbenshalom and yonatankarni

August 10, 2023 20:09

bbenshalom approved these changes

View reviewed changes

ggaspersic merged commit a686fa5 into main

3 checks passed

ggaspersic deleted the remove_fb_ffm_num_fields branch

August 13, 2023 12:28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet