Skip to content

Commit 44da068

Browse files
committedJun 27, 2024
Revert "always return a scaling factor"
This reverts commit a7791e1.
1 parent 50df842 commit 44da068

File tree

1 file changed

+15
-7
lines changed

1 file changed

+15
-7
lines changed
 

‎pyomo/contrib/pynumero/interfaces/pyomo_nlp.py

+15-7
Original file line numberDiff line numberDiff line change
@@ -299,24 +299,32 @@ def get_inequality_constraint_indices(self, constraints):
299299
# overloaded from NLP
300300
def get_obj_scaling(self):
301301
obj = self.get_pyomo_objective()
302-
val = SuffixFinder('scaling_factor', 1.0).find(obj)
302+
val = SuffixFinder('scaling_factor').find(obj)
303303
return val
304304

305305
# overloaded from NLP
306306
def get_primals_scaling(self):
307-
scaling_suffix_finder = SuffixFinder('scaling_factor', 1.0)
307+
scaling_suffix_finder = SuffixFinder('scaling_factor')
308308
primals_scaling = np.ones(self.n_primals())
309+
ret = None
309310
for i, v in enumerate(self.get_pyomo_variables()):
310-
primals_scaling[i] = scaling_suffix_finder.find(v)
311-
return primals_scaling
311+
val = scaling_suffix_finder.find(v)
312+
if val is not None:
313+
primals_scaling[i] = val
314+
ret = primals_scaling
315+
return ret
312316

313317
# overloaded from NLP
314318
def get_constraints_scaling(self):
315-
scaling_suffix_finder = SuffixFinder('scaling_factor', 1.0)
319+
scaling_suffix_finder = SuffixFinder('scaling_factor')
316320
constraints_scaling = np.ones(self.n_constraints())
321+
ret = None
317322
for i, c in enumerate(self.get_pyomo_constraints()):
318-
constraints_scaling[i] = scaling_suffix_finder.find(c)
319-
return constraints_scaling
323+
val = scaling_suffix_finder.find(c)
324+
if val is not None:
325+
constraints_scaling[i] = val
326+
ret = constraints_scaling
327+
return ret
320328

321329
def extract_subvector_grad_objective(self, pyomo_variables):
322330
"""Compute the gradient of the objective and return the entries

0 commit comments

Comments
 (0)
Please sign in to comment.