Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

XP-255 update codeowners and authors in setup #195

Merged
merged 2 commits into from
Dec 20, 2019
Merged

XP-255 update codeowners and authors in setup #195

merged 2 commits into from
Dec 20, 2019

Conversation

PanicButtonPressed
Copy link
Member

@PanicButtonPressed PanicButtonPressed commented Dec 20, 2019 •
edited by FeliXayn
Loading

References

XP-255

Summary

This PR solves the following:

  • update the author in setup.py
  • remove CODEOWNER for now, since they are not used

Are there any open tasks/blockers for the ticket?

All tasks are done.

Reviewer checklist

Reviewer agreement:

  • Reviewers assign themselves at the start of the review.
  • Reviewers do not commit or merge the merge request.
  • Reviewers have to check and mark items in the checklist.

Merge request checklist

  • Conforms to the merge request title naming XP-XXX <a description in imperative form>.
  • Each commit conforms to the naming convention XP-XXX <a description in imperative form>.
  • Linked the ticket in the merge request title or the references section.
  • Added an informative merge request summary.

Code checklist

  • Conforms to the branch naming XP-XXX-<a_small_stub>.
  • Passed scope checks.
  • Added or updated tests if needed.
  • Added or updated code documentation if needed.
  • Conforms to Google docstring style.
  • Conforms to XAIN structlog style.

@PanicButtonPressed PanicButtonPressed requested review from atymoshchuk, FeliXayn and wilk10 and removed request for tanertopal December 20, 2019 08:53
@FeliXayn FeliXayn self-requested a review December 20, 2019 09:18
@PanicButtonPressed PanicButtonPressed merged commit 4ebc4b9 into development Dec 20, 2019
@PanicButtonPressed PanicButtonPressed deleted the XP-255-update-codeowners-and-authors-in-setup branch December 20, 2019 09:19
@PanicButtonPressed PanicButtonPressed mentioned this pull request Dec 20, 2019
Merged
10 tasks
PanicButtonPressed added a commit that referenced this pull request Dec 20, 2019
@PanicButtonPressed
XP-265 move benchmarks to separate repo (#193)
3f5d9c5 
* XP-265 remove benchmark directory

* XP-265 remove datasets directory

* XP-265 remove references to either examples or benchmark directory

* XP-265 remove benchmark participant

* XP-265 remove benchmark config relations

* XP-265 remove benchmark coordinator, related types and helpers

* XP-265 review comments + changed type import

* XP-265 commenting out benchmarks docs

* XP-243 remove flags from xain-fl codebase

* XP-255 update codeowners and authors in setup (#195)

* XP-255 change setup author to XAIN AG & remove codeowners for now

* XP-255 black reformat

* XP-265 remove benchmark directory

* XP-265 remove datasets directory

* XP-265 remove references to either examples or benchmark directory

* XP-265 remove benchmark participant

* XP-265 remove benchmark config relations

* XP-265 remove benchmark coordinator, related types and helpers

* XP-265 review comments + changed type import

* XP-265 commenting out benchmarks docs

* XP-243 remove flags from xain-fl codebase

* XP-265 add ticket number for deciding about EvoAgg
rsaffi pushed a commit that referenced this pull request Jan 23, 2020
@atymoshchuk @dkravetz
@PanicButtonPressed @wilk10 @Robert-Steiner @little-dude @janpetschexain @finiteprods
XP-227 - Merge v0.3.0 changes into master (#233)
fed62b7 
* XP-337 Clean up docs before generation (#188)

* XP-229 Update Readme.md (#189)

* XP-255 update codeowners and authors in setup (#195)

* XP-265 move benchmarks to separate repo (#193)

* XP-243 remove flags from xain-fl codebase

* XP-255 update codeowners and authors in setup (#195)

* XP-257 cleanup cproto dir (#198)

* XP-261 move tests to own dir (#197)

* XP-168 update setup.py (#191)

* DO-35 📰 ✨ Add placeholder for e2e. Update CI for gitflow

* XP-241 remove legacy participant and sdk dir (#199)

* DO-17 🐳 ✨ Add Dockerfiles, dockerignore and docs (#202)

* XP-354 Remove proto files (#200)

* XP-385 Fix docs badge (#204)

* Xp 273 scripts cleanup (#206)

* XP-357 make controller parametrisable (#201)

* XP-384 remove unused files (#209)

* Do 43 docker compose minio (#208)

* XP-374 Clean up docs (#211)

* XP-271 fix pylint issues (#210)

* XP-424 Remove unused packages (#212)

* DO-49 🐳 ✨ Create initial buckets (#213)

* Xp 373 add sdk as dependency in fl (#214)

* XP-433 Fix docker headings (#218)

* XP-119 Fix gRPC testing setup so that it can run on macOS (#217)

* XP-422 ai metrics (#216)

* XP-308: store aggregated weights in S3 buckets (#215)

* XP-436 Reinstate FINISHED heartbeat from Coordinator (#219)

* XP-208 model sum aggregator as a mock for testing

* XP-208 add test integrating P and C

* XP-208 add identity controller as a mock for test

* XP-208 small typo in id-controller docstring

* XP-208 add coordinator test fixture with mocked components

* XP-208 test for top-level function start_participant
* XP-308: print debugging information for spurious failure

* XP-208 join monitor_thread to confirm response to terminate_event

* XP-208 change mock patch string to object called rather than defined

* XP-208 model sum aggregator as a mock for testing

* XP-208 add test integrating P and C

* XP-208 add identity controller as a mock for test

* XP-208 small typo in id-controller docstring

* XP-208 add coordinator test fixture with mocked components

* XP-208 test for top-level function start_participant

* XP-436 fix bug: coordinator not advertising FINISHED state

* XP-436 revert logging level

* XP-208 join monitor_thread to confirm response to terminate_event

* XP-208 change mock patch string to object called rather than defined

* XP-436 suggested documentation fixes from review

* XP-208 mock the store (following rebase)

* XP-208 fix import

* XP-208 test_start_participant: mark as slow, comment on aggregation

* XP-480 revise message names (#222)

* XP-499 Remove conftest, exclude tests folder (#223)

* XP-333 Replace numproto with xain-proto (#220)

* XP-505: docstrings cleanup (#224)

* XP-508 Replace circleci badge (#225)

* XP-510 allow for zero epochs on cli (#227)

* XP-498: more generic shebangs (#229)

* XP-505: cleanup docstrings in xain_fl.coordinator (#228)

* Prepare release of v0.3.0 (matching version with `xain-sdk` and `xain-proto`) (#230)

Co-authored-by: Anastasiia Tymoshchuk <atymoshchuk@icloud.com>
Co-authored-by: Daniel Kravetz <daniel.kravetz@gmail.com>
Co-authored-by: Felix Reichel <31513604+PanicButtonPressed@users.noreply.github.com>
Co-authored-by: Anselmo Sampietro <ans.samp@gmail.com>
Co-authored-by: Robert Steiner <robertt.debug@gmail.com>
Co-authored-by: Corentin Henry <little-dude@users.noreply.github.com>
Co-authored-by: janpetschexain <58227040+janpetschexain@users.noreply.github.com>
Co-authored-by: kwok <kwok.doc@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@FeliXayn FeliXayn FeliXayn approved these changes

@atymoshchuk atymoshchuk Awaiting requested review from atymoshchuk

@wilk10 wilk10 Awaiting requested review from wilk10

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@PanicButtonPressed @FeliXayn