Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

alpha #2

Merged
merged 22 commits into from
Mar 21, 2018
Merged

alpha #2

merged 22 commits into from
Mar 21, 2018

Conversation

Superjomn
Copy link
Collaborator

@Superjomn Superjomn commented Mar 15, 2018

A live demo: http://172.19.61.250:8080/.

An initial implementation with a web frontend.

Current the evaluation tasks are working on a busy server, so some tests are fragile and easy to fail, requiring an exclusive machine.

Most of the script is written with xonsh, a python-powered BASH interpreter which enables mixing python with Linux command. In that way, the trivial commands can be better maintained for a growing project.

The baseline is currently maintained at https://github.com/Superjomn/paddle-modelci-baseline, will keep being updated in master branch with the latest successful evaluation result; the latest failed result will be persisted in develop branch.

The ModelCE framework will keep updating with the baseline's master branch, so one can change the baseline directly by editing on the baseline repo.

fixes: #3

@Superjomn Superjomn changed the title init alpha Mar 16, 2018
@panyx0718 panyx0718 self-requested a review March 19, 2018 10:20
Copy link
Contributor

@panyx0718 panyx0718 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can also track memory leak in the future. Currently we seems to have memory leak

@@ -23,16 +23,16 @@ class Factor(object):
self.out_file = out_file
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to add comments for this class

@Superjomn Superjomn merged commit fb088b7 into master Mar 21, 2018
@Superjomn Superjomn deleted the feature/alpha branch March 21, 2018 00:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants