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

Release v0.5.0 #21

Merged
merged 29 commits into from
Jun 17, 2017
Merged

Release v0.5.0 #21

merged 29 commits into from
Jun 17, 2017

Conversation

paulgoetze
Copy link
Owner

kcning and others added 29 commits December 25, 2016 03:57
        1) add support for adding attribute values passed via a hash
        2) add testcases for it
        1) refactor #attribute_values_to_hash
        2) rewrite comment
        One of the Java method of Instances has unexpected behavior.
        enumerateAttributes will skip the class attribute once it is
        set. Therfore we have to manually add the class attribute,
        otherwise we we will miss it in every method that calls
        Instances#attributes.

See also:
http://weka.sourceforge.net/doc.stable/weka/core/Instances.html#enumerateAttributes--
        1) The previous fix only works when class attribute is the last
        attribute. The bug fix address that problem.

        2) use attribute in the middle as the class attribute for the
        test cases in the spec.
        1) use keyword argument for #attributes and #attribute_names
        2) fix #ensure_attribute_defined! and #attribute_with_name
        3) refactor private methods #attribute_values_to_hash and
        #attribute_values_from_hash
        4) add new test cases and refactor the spec
Feature: allow passing hash to Instances#add_instance
to_m returns a Matrix object holding all instances's values as rows.
…odule

Add curve classes to classifier evaluator namespace
This splits up the different method groups into separate modules and
makes it a bit more readable and changeable.
@paulgoetze paulgoetze merged commit 1e14a52 into master Jun 17, 2017
@paulgoetze paulgoetze deleted the release/0.5.0 branch June 17, 2017 12:58
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.

2 participants