Skip to content

Agents #468

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

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

Agents #468

wants to merge 12 commits into from

Conversation

blaiszik
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Jan 22, 2025

Codecov Report

Attention: Patch coverage is 70.27027% with 88 lines in your changes missing coverage. Please review.

Project coverage is 73.17%. Comparing base (0ccb39b) to head (2a47d5c).
Report is 857 commits behind head on main.

Files with missing lines Patch % Lines
foundry/loaders/spectral.py 31.42% 24 Missing ⚠️
foundry/loaders/molecular.py 41.02% 23 Missing ⚠️
foundry/loaders/image.py 44.44% 15 Missing ⚠️
foundry/loaders/numpy.py 50.00% 8 Missing ⚠️
foundry/loaders/parquet.py 0.00% 8 Missing ⚠️
foundry/foundry.py 55.55% 4 Missing ⚠️
foundry/loaders/tabular.py 93.10% 2 Missing ⚠️
foundry/loaders/tf_wrapper.py 87.50% 1 Missing ⚠️
foundry/loaders/torch_wrapper.py 87.50% 1 Missing ⚠️
foundry/utils/io.py 90.90% 1 Missing ⚠️
... and 1 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #468      +/-   ##
==========================================
+ Coverage   72.08%   73.17%   +1.08%     
==========================================
  Files           9       28      +19     
  Lines         541     1260     +719     
==========================================
+ Hits          390      922     +532     
- Misses        151      338     +187     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

what-the-diff bot commented Jan 22, 2025

PR Summary

  • Coverage Configuration Update
    The team has added configuration files to ensure that only relevant parts of our code are tested for performance and reliability.

  • Improved GitHub Actions Workflow
    We've brought both more breadth and depth to our test routines by expanding the workflow triggers and adding a comprehensive setup strategy. Workflow names have been changed for clarity.

  • README.md Revised
    A progress tracker (the Coverage Badge) was added, and instructions on how to contribute to the project along with example codes have been updated, making it easier for newcomers to understand.

  • New Sample Data File Introduced
    A new JSON data file has been added for testing purposes.

  • New JSON Data File Addition and Error Handling
    A new JSON file and error handling mechanisms have been added for more efficient data validation.

  • Loader Management System Introduction
    A new loader management system has been introduced, making data loading from various formats easier and more structured.

  • Data Loading Logic & Configuration Model
    The process of data loading has been simplified thanks to using the new loader system and implementing a standardized structure of handling configuration across the project.

  • Implementation of the Base Class for Data Loaders
    The team has defined a blueprint for a consistent interface for all data loaders. DataLoader classes for HDF5, images, molecular data, NumPy arrays, and Parquet formats have been developed.

  • Enhanced DataLoader Registration and Handling
    A new file registry.py has been added that provides a straightforward way to manage and register different types of data loaders.

  • Data Loading Specific Enhancements
    Data import from spectral and tabular formats has experienced significant improvements.

  • Wrapper Updates
    Dependencies on outdated packages in TensorFlow and Torch have been removed.

  • Utility Function Additions
    New functions have been added to facilitate optional module imports and notify users when dependencies are missing.

  • I/O and Validation Utilities
    Helper functions for reading various file types (CSV, JSON, Excel) have been introduced, along with functions to verify accuracy and format.

  • PyTest Configuration
    PyTest has been configured to focus on only relevant paths, increasing the efficiency of the testing process.

  • Requirement File Updates
    Necessary and optional development requirements have been clearly defined and separated.

  • Test Script and Unit Test Introduction
    A new script for testing with coverage options has been implemented, and unit tests have been created to mock and test URLs.

  • Uncommented Testing Scenarios
    Previously ignored test functions have been uncommented and adjusted for better functionality.

  • Comprehensive Testing of Loaders
    Significant efforts have been put into testing of various loaders and their functionalities, ensuring that they interact with data formats as expected and deal with errors appropriately.

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.

1 participant