MLBazaar / MLBazaar/MLBlocks

Support reading an attribute from a class instead of calling a method?

Open
#72 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

approved new feature
Dominant language
Python
Stars
124
Forks
33
PR merge metrics
No merged PRs in 30d

Description

This is a discussion issue as a followup #71

Should we natively support returning a class attribute value (or multiple) instead of calling a method of the class during the produce phase?

This could be simple achieved at the JSON level by simply not specifying the method key.

Then, we would have:

    "produce": {
        "output": [
            {
                "name": "feature_weights",
                "attribute": "feature_importances_"   # This would be optional, to be used only if the used name does not match the attribute name.
                "type": "list"
            }
        ]
    }

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading follow-up issue #71 and tracing the produce-phase configuration described here. Determine how method-based outputs are currently represented and whether omitting method can identify a class attribute. Done means the proposed attribute form and its optional name mapping have a clear, agreed implementation path.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
machine-learning
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.