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

Start Porting Shumai Tests #1128

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

cryptodeal
Copy link

@cryptodeal cryptodeal commented Jun 8, 2023

Original Issue: N/A

Summary

Starting work to port unit tests from shumai -> flashlight. So far have ported any, all, cumsum, argmin, and argmax.

Test Plan (required)

N/A as this is just adding more test coverage, not implementing new features.

cryptodeal and others added 4 commits June 8, 2023 11:46
Summary:
See title. Improves build time for most pkgs, especially since `runtime` depends on FL text as it stands

Pull Request resolved: flashlight#1125

Test Plan: CI + local test

Reviewed By: bwasti

Differential Revision: D46469489

Pulled By: jacobkahn

fbshipit-source-id: 3b86bd31472d414d52d0d7019c4c9d4521e85d84
Summary:
See title. [[context](https://fb.workplace.com/groups/osssupport/posts/24783945017894113/)]

Pull Request resolved: flashlight#1126

Test Plan: CI

Reviewed By: bigfootjon

Differential Revision: D46531543

Pulled By: jacobkahn

fbshipit-source-id: 5e8e8040ee2beeca36cc831766527d283d9320fd
…t#1127)

Summary:
Switching to `macos.m1.medium.gen1` seems to result in OOM/infrastructure fails by default. Reduce build parallelism from CMake's high default to mitigate this

Pull Request resolved: flashlight#1127

Test Plan: CI

Reviewed By: bigfootjon

Differential Revision: D46539373

Pulled By: jacobkahn

fbshipit-source-id: ef14f7762b9fb817e0e44c5c27f42eae2c9d0c30
@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jun 8, 2023
@cryptodeal cryptodeal marked this pull request as draft June 8, 2023 16:54
@cryptodeal cryptodeal changed the title More tests Start Porting Shumai Tests Jun 8, 2023
@cryptodeal cryptodeal marked this pull request as ready for review June 9, 2023 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants