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

agripilot-team: Fixed iJIT_NotifyEvent Issue. #222

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from

Conversation

agripilot-team
Copy link

  • Fixed iJIT_NotifyEvent undefined symbol issue.
  • Resolved model export issue during training.
  • Replaced deprecated append() with concat().
  • Fixed datetime addition issue by converting integer using pd.to_datetime()

rafaspadilha and others added 7 commits November 27, 2024 11:38
This PR pulls the changes made in `dev` branch since last sync (May
2024) to the `main` branch.

---------

Co-authored-by: FarmVibes Release Pipeline <[email protected]>
Co-authored-by: Alex Crown <[email protected]>
Co-authored-by: Bruno Silva <[email protected]>
Co-authored-by: Eduardo Rodrigues <[email protected]>
Co-authored-by: Jessica Wolk <[email protected]>
Co-authored-by: Leonardo Nunes <[email protected]>
Co-authored-by: Naga Bilwanth Gangarapu <[email protected]>
Co-authored-by: Renato Luiz de Freitas Cunha <[email protected]>
Co-authored-by: Rick Gutierrez <[email protected]>
Co-authored-by: Roberto de Moura Estevão Filho <[email protected]>
Co-authored-by: Roberto Santos <[email protected]>
Co-authored-by: Sara Malvar <[email protected]>
Co-authored-by: Naga Bilwanth Gangarapu <[email protected]>
The current build process fails to include model weights in worker
images because we are failing to restore LFS files.

Fixes microsoft#208
Reverting previous two commits on `main` branch. Will trigger release
pipeline correctly this time.
This re-releases microsoft#206 and microsoft#211 now using the release pipeline, which
also updates the prod images in the Container Registry.

----

This PR marks the full release of FarmVibes.AI's source code with an
open source license.

This PR also introduces several significant updates to the development
environment, build workflows, and continuous integration processes. The
most important changes include the addition of a development container
configuration, updates to GitHub workflows for building and testing, and
the introduction of new settings for code linting and formatting.

### Development Environment Updates:
*
[`.devcontainer/devcontainer.json`](diffhunk://#diff-24ad71c8613ddcf6fd23818cb3bb477a1fb6d83af4550b0bad43099813088686R1-R62):
Added a new devcontainer configuration with custom VSCode settings and
extensions, Docker mounts, and environment variables.
*
[`.devcontainer/post-create.sh`](diffhunk://#diff-bcf66a848607b49d2a45edfad434f6808017ee9d18f5a20dde6670745010d89aR1-R71):
Added a post-create script to set up the development environment,
including SSH configuration, Python virtual environment setup, and
installation of necessary tools and dependencies.

### Build and Deployment Workflows:
*
[`.github/workflows/base-build.yml`](diffhunk://#diff-034a91a8f3709fafd1219e587ce389e3a39338ac6adbe543d524bcb1e84b2cf3R1-R64):
Added a new workflow for building and pushing base Docker images,
including steps for Azure CLI login and updating image tags.
*
[`.github/workflows/docker-build.yml`](diffhunk://#diff-3414847e2ad632333f775cabb810f0dc0df61a570365df34750a08b00912fe82R1-R41):
Introduced a workflow for building and pushing service Docker images on
push events to the `dev` and `main` branches.
*
[`.github/workflows/release-to-main.yml`](diffhunk://#diff-b55efbebc7173bd35e8aa625b689d11a521f0ea54f064ae7a2be9cbf5c5fa1a4R1-R54):
Added a workflow to handle the release process, including retagging
Docker images and updating the default image tag in the codebase.

### Continuous Integration:
*
[`.github/workflows/lint-test.yml`](diffhunk://#diff-1dccad827a8e5157df1dcdb70b73e454524c2586ebc64576af82cb348d478892R1-R197):
Created a comprehensive workflow for linting, type checking, and testing
the codebase across multiple packages and environments.

### Code Linting and Formatting:
*
[`.ruff.toml`](diffhunk://#diff-b3236d4cb260ca6acce083493fcdc42816f732fc3d5aefb263a3c3fe9db4f81fR1-R17):
Added configuration for the `ruff` linter, specifying linting rules,
file exclusions, and formatting preferences.

These changes aim to ensure FarmVibes.AI's longevity, to streamline the
development process, ensure consistent code quality, and automate the
build and deployment pipelines.

---------

Co-authored-by: FarmVibes Release Pipeline <[email protected]>
Co-authored-by: Alex Crown <[email protected]>
Co-authored-by: Bruno Silva <[email protected]>
Co-authored-by: Eduardo Rodrigues <[email protected]>
Co-authored-by: Jessica Wolk <[email protected]>
Co-authored-by: Leonardo Nunes <[email protected]>
Co-authored-by: Naga Bilwanth Gangarapu <[email protected]>
Co-authored-by: Rafael Padilha <[email protected]>
Co-authored-by: Rick Gutierrez <[email protected]>
Co-authored-by: Roberto de Moura Estevão Filho <[email protected]>
Co-authored-by: Roberto Santos <[email protected]>
Co-authored-by: Sara Malvar <[email protected]>
Co-authored-by: Naga Bilwanth Gangarapu <[email protected]>
… replaced deprecated append() with concat(), and fixed datetime addition by converting integer using pd.to_datetime().
@agripilot-team
Copy link
Author

@microsoft-github-policy-service agree company="agripilot.ai"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants