Changeset 20 for opponentmodel/src/main/java/geniusweb

Timestamp:

08/05/20 09:42:15 (4 years ago)

Author:

bart

Message:

Added BOA support, some bug fixes

Location:

opponentmodel/src/main/java/geniusweb/opponentmodel

Files:

• FrequencyOpponentModel.java (modified) (3 diffs)
• OpponentModel.java (modified) (2 diffs)

opponentmodel/src/main/java/geniusweb/opponentmodel/FrequencyOpponentModel.java

@@ -24 +24 @@
  * (as you might expect as {@link NumberValueSetUtilities} is only affected by
  * the endpoints).
- *
+ * <p>
+ * immutable.
  */
 public class FrequencyOpponentModel implements UtilitySpace, OpponentModel {
 
         private static final int DECIMALS = 4; // accuracy of our computations.
+        private static int serial = 1; // counter for auto name generation
+
         private final Domain domain;
         private final Map<String, Map<Value, Integer>> bidFrequencies;
         private final BigDecimal totalBids;
-        private static int serial = 1; // counter for auto name generation
-
-        public FrequencyOpponentModel(Domain domain) {
-                /*
-                 * simply reset the whole model and make a new one... maybe this can be
-                 * done smarter for minor changes? map with empth hashmap for each
-                 * issue.
-                 */
-                this(domain,
+
+        public FrequencyOpponentModel() {
+                this.domain = null;
+                this.bidFrequencies = null;
+                this.totalBids = BigDecimal.ZERO;
+        }
+
+        @Override
+        public FrequencyOpponentModel with(Domain domain, Bid resBid) {
+                // FIXME merge already available frequencies?
+                return new FrequencyOpponentModel(domain,
                                 domain.getIssues().stream().collect(
                                                 Collectors.toMap(iss -> iss, iss -> new HashMap<>())),
@@ -115 +120 @@
         }
 
-        /**
-         * {@inheritDoc}
-         * 
-         * <h1>change info</h1> This replaces update(Bid). Now you pass the entire
-         * action and the progress instead of just the bid.
-         */
         @Override
         public FrequencyOpponentModel with(Action action, Progress progress) {
@@ -127 +126 @@
 
                 Bid bid = ((Offer) action).getBid();
-                String err = domain.isComplete(bid);
-                if (err != null) {
-                        throw new IllegalArgumentException(err);
-                }
                 Map<String, Map<Value, Integer>> newFreqs = cloneMap(bidFrequencies);
                 for (String issue : domain.getIssues()) {
                         Map<Value, Integer> freqs = newFreqs.get(issue);
                         Value value = bid.getValue(issue);
-                        Integer oldfreq = freqs.get(value);
-                        if (oldfreq == null) {
-                                oldfreq = 0;
+                        if (value != null) {
+                                Integer oldfreq = freqs.get(value);
+                                if (oldfreq == null) {
+                                        oldfreq = 0;
+                                }
+                                freqs.put(value, oldfreq + 1);
                         }
-                        freqs.put(value, oldfreq + 1);
                 }
 

opponentmodel/src/main/java/geniusweb/opponentmodel/OpponentModel.java

@@ -2 +2 @@
 
 import geniusweb.actions.Action;
+import geniusweb.issuevalue.Bid;
 import geniusweb.issuevalue.Domain;
 import geniusweb.profile.Profile;
@@ -13 +14 @@
  * Domain as argument. unfortunately this can not be enforced in a java
  * interface
+ * 
+ * <p>
+ * <em>MUST</em> have an empty constructor as these are also used as part of the
+ * BOA framework.
+ * 
  */
 public interface OpponentModel extends Profile {
+
+        /**
+         * Initializes the model. This function must be called first after
+         * constructing an instance. It can also be called later, if there is a
+         * change in the domain or resBid.
+         * 
+         * @param domain the domain to work with
+         * @param resBid the reservation bid, or null if no reservationbid is
+         *               available.
+         * @return OpponentModel that uses given domain and reservationbid.
+         * 
+         */
+        OpponentModel with(Domain domain, Bid resBid);
 
         /**