score class in the XML serialization #225

Merged
claudio.atzori merged 1 commits from measure_serialization into beta 2 years ago
Owner

This PR includes the class information in the measure XML serialization, which changes from

  <measure id="influence" key="score" value="5.06690394631e-09"/>
  <measure id="popularity_alt" key="score" value="0.0"/>
  <measure id="popularity"  key="score" value="3.11855618382e-09"/>
  <measure id="influence_alt"  key="score" value="0.0"/>
  <measure id="impulse"  key="score" value="0.0"/>

to

  <measure id="influence" score="5.06690394631e-09" class="C"/>
  <measure id="popularity_alt" score="0.0" class="C"/>
  <measure id="popularity" score="3.11855618382e-09" class="C"/>
  <measure id="influence_alt" score="0.0" class="C"/>
  <measure id="impulse" score="0.0" class="C"/>

Note also that the attributes are renamed, I think the updated serialisation is simpler.

This PR includes the class information in the measure XML serialization, which changes from ``` <measure id="influence" key="score" value="5.06690394631e-09"/> <measure id="popularity_alt" key="score" value="0.0"/> <measure id="popularity" key="score" value="3.11855618382e-09"/> <measure id="influence_alt" key="score" value="0.0"/> <measure id="impulse" key="score" value="0.0"/> ``` to ``` <measure id="influence" score="5.06690394631e-09" class="C"/> <measure id="popularity_alt" score="0.0" class="C"/> <measure id="popularity" score="3.11855618382e-09" class="C"/> <measure id="influence_alt" score="0.0" class="C"/> <measure id="impulse" score="0.0" class="C"/> ``` Note also that the attributes are renamed, I think the updated serialisation is simpler.
claudio.atzori added the
enhancement
label 2 years ago
alessia.bardi was assigned by claudio.atzori 2 years ago
miriam.baglioni was assigned by claudio.atzori 2 years ago
claudio.atzori added 1 commit 2 years ago
Owner

I am not familiar with the exact input provided by BIP!Finder, but I think this PR is ok.

I am not familiar with the exact input provided by BIP!Finder, but I think this PR is ok.
Collaborator

The PR is ok to me as well

The PR is ok to me as well
claudio.atzori merged commit 138d1dfbf8 into beta 2 years ago
claudio.atzori deleted branch measure_serialization 2 years ago
The pull request has been merged as 138d1dfbf8.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b measure_serialization beta
git pull origin measure_serialization

Step 2:

Merge the changes and update on Gitea.
git checkout beta
git merge --no-ff measure_serialization
git push origin beta
Sign in to join this conversation.
No reviewers
No Milestone
No project
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: D-Net/dnet-hadoop#225
Loading…
There is no content yet.