Restoring from checkpoint failed

@mahmoudmohsen213
flash@flash:~/airs$ python3 train.py
2019-03-21 13:46:40.578816: loading data files
WARNING:tensorflow:From train.py:15: load_csv_without_header (from tensorflow.contrib.learn.python.learn.datasets.base) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.data instead.
WARNING:tensorflow:From train.py:33: RunConfig.init (from tensorflow.contrib.learn.python.learn.estimators.run_config) is deprecated and will be removed in a future version.
Instructions for updating:
When switching to tf.estimator.Estimator, use tf.estimator.RunConfig instead.
WARNING:tensorflow:From /usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/dnn.py:378: multi_class_head (from tensorflow.contrib.learn.python.learn.estimators.head) is deprecated and will be removed in a future version.
Instructions for updating:
Please switch to tf.contrib.estimator._head.
WARNING:tensorflow:From /usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py:1180: BaseEstimator.init (from tensorflow.contrib.learn.python.learn.estimators.estimator) is deprecated and will be removed in a future version.
Instructions for updating:
Please replace uses of any Estimator from tf.contrib.learn with an Estimator from tf.estimator.
2019-03-21 13:46:49.793051: training...
WARNING:tensorflow:Casting <dtype: 'int64'> labels to bool.
WARNING:tensorflow:Casting <dtype: 'int64'> labels to bool.
WARNING:tensorflow:Trapezoidal rule is known to produce incorrect PR-AUCs; please switch to "careful_interpolation" instead.
WARNING:tensorflow:Trapezoidal rule is known to produce incorrect PR-AUCs; please switch to "careful_interpolation" instead.
WARNING:tensorflow:From /usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/head.py:678: ModelFnOps.new (from tensorflow.contrib.learn.python.learn.estimators.model_fn) is deprecated and will be removed in a future version.
Instructions for updating:
When switching to tf.estimator.Estimator, use tf.estimator.EstimatorSpec. You can use the estimator_spec method to create an equivalent one.
2019-03-21 13:46:51.593519: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2019-03-21 13:46:51.620702: W tensorflow/core/framework/op_kernel.cc:1273] OP_REQUIRES failed at save_restore_v2_ops.cc:184 : Invalid argument: tensor_name = dnn/hiddenlayer_0/weights; shape in shape_and_slice spec [75,100] does not match the shape stored in checkpoint: [1,100]
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1334, in _do_call
return fn(*args)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1319, in _run_fn
options, feed_dict, fetch_list, target_list, run_metadata)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1407, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.InvalidArgumentError: tensor_name = dnn/hiddenlayer_0/weights; shape in shape_and_slice spec [75,100] does not match the shape stored in checkpoint: [1,100]
[[{{node save/RestoreV2}} = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_INT64], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 1546, in restore
{self.saver_def.filename_tensor_name: save_path})
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 929, in run
run_metadata_ptr)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1152, in _run
feed_dict_tensor, options, run_metadata)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1328, in _do_run
run_metadata)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py", line 1348, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: tensor_name = dnn/hiddenlayer_0/weights; shape in shape_and_slice spec [75,100] does not match the shape stored in checkpoint: [1,100]
[[node save/RestoreV2 (defined at /usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py:1092)  = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_INT64], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
Caused by op 'save/RestoreV2', defined at:
File "train.py", line 70, in
classifier.fit(input_fn=getTrainData, steps=totalTrainingSteps)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py", line 525, in fit
loss = self._train_model(input_fn=input_fn, hooks=hooks)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py", line 1092, in _train_model
config=self._session_config) as mon_sess:
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 504, in MonitoredTrainingSession
stop_grace_period_secs=stop_grace_period_secs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 921, in init
stop_grace_period_secs=stop_grace_period_secs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 643, in init
self._sess = _RecoverableSession(self._coordinated_creator)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 1107, in init
_WrappedSession.init(self, self._create_session())
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 1112, in _create_session
return self._sess_creator.create_session()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 800, in create_session
self.tf_sess = self._session_creator.create_session()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 557, in create_session
self._scaffold.finalize()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 215, in finalize
self._saver.build()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 1114, in build
self._build(self._filename, build_save=True, build_restore=True)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 1151, in _build
build_save=build_save, build_restore=build_restore)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 789, in _build_internal
restore_sequentially, reshape)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 459, in _AddShardedRestoreOps
name="restore_shard"))
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 406, in _AddRestoreOps
restore_sequentially)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 862, in bulk_restore
return io_ops.restore_v2(filename_tensor, names, slices, dtypes)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/gen_io_ops.py", line 1466, in restore_v2
shape_and_slices=shape_and_slices, dtypes=dtypes, name=name)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/ops.py", line 3274, in create_op
op_def=op_def)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/ops.py", line 1770, in init
self._traceback = tf_stack.extract_stack()
InvalidArgumentError (see above for traceback): tensor_name = dnn/hiddenlayer_0/weights; shape in shape_and_slice spec [75,100] does not match the shape stored in checkpoint: [1,100]
[[node save/RestoreV2 (defined at /usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py:1092)  = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_INT64], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "train.py", line 70, in
classifier.fit(input_fn=getTrainData, steps=totalTrainingSteps)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py", line 525, in fit
loss = self._train_model(input_fn=input_fn, hooks=hooks)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py", line 1092, in _train_model
config=self._session_config) as mon_sess:
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 504, in MonitoredTrainingSession
stop_grace_period_secs=stop_grace_period_secs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 921, in init
stop_grace_period_secs=stop_grace_period_secs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 643, in init
self._sess = _RecoverableSession(self._coordinated_creator)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 1107, in init
_WrappedSession.init(self, self._create_session())
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 1112, in _create_session
return self._sess_creator.create_session()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 800, in create_session
self.tf_sess = self._session_creator.create_session()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 566, in create_session
init_fn=self._scaffold.init_fn)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/session_manager.py", line 288, in prepare_session
config=config)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/session_manager.py", line 218, in _restore_checkpoint
saver.restore(sess, ckpt.model_checkpoint_path)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 1582, in restore
err, "a mismatch between the current graph and the graph")
tensorflow.python.framework.errors_impl.InvalidArgumentError: Restoring from checkpoint failed. This is most likely due to a mismatch between the current graph and the graph from the checkpoint. Please ensure that you have not altered the graph expected based on the checkpoint. Original error:
tensor_name = dnn/hiddenlayer_0/weights; shape in shape_and_slice spec [75,100] does not match the shape stored in checkpoint: [1,100]
[[node save/RestoreV2 (defined at /usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py:1092)  = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_INT64], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
Caused by op 'save/RestoreV2', defined at:
File "train.py", line 70, in
classifier.fit(input_fn=getTrainData, steps=totalTrainingSteps)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py", line 525, in fit
loss = self._train_model(input_fn=input_fn, hooks=hooks)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py", line 1092, in _train_model
config=self._session_config) as mon_sess:
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 504, in MonitoredTrainingSession
stop_grace_period_secs=stop_grace_period_secs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 921, in init
stop_grace_period_secs=stop_grace_period_secs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 643, in init
self._sess = _RecoverableSession(self._coordinated_creator)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 1107, in init
_WrappedSession.init(self, self._create_session())
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 1112, in _create_session
return self._sess_creator.create_session()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 800, in create_session
self.tf_sess = self._session_creator.create_session()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 557, in create_session
self._scaffold.finalize()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py", line 215, in finalize
self._saver.build()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 1114, in build
self._build(self._filename, build_save=True, build_restore=True)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 1151, in _build
build_save=build_save, build_restore=build_restore)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 789, in _build_internal
restore_sequentially, reshape)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 459, in _AddShardedRestoreOps
name="restore_shard"))
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 406, in _AddRestoreOps
restore_sequentially)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 862, in bulk_restore
return io_ops.restore_v2(filename_tensor, names, slices, dtypes)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/gen_io_ops.py", line 1466, in restore_v2
shape_and_slices=shape_and_slices, dtypes=dtypes, name=name)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/ops.py", line 3274, in create_op
op_def=op_def)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/ops.py", line 1770, in init
self._traceback = tf_stack.extract_stack()
InvalidArgumentError (see above for traceback): Restoring from checkpoint failed. This is most likely due to a mismatch between the current graph and the graph from the checkpoint. Please ensure that you have not altered the graph expected based on the checkpoint. Original error:
tensor_name = dnn/hiddenlayer_0/weights; shape in shape_and_slice spec [75,100] does not match the shape stored in checkpoint: [1,100]
[[node save/RestoreV2 (defined at /usr/local/lib/python3.5/dist-packages/tensorflow/contrib/learn/python/learn/estimators/estimator.py:1092)  = RestoreV2[dtypes=[DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_INT64], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]

was the checkpoint files created with the same neural network structure as the defined in the code trying to restore it?
was the checkpoint files created from the same tensorflow version?

@mahmoudmohsen213
Tensorflow-gpu 1.12.0
Pillow 4.1.0
Python3.5
CSV created successfully(convertToFeautureFiles.py)
CSV file created inside airs-dataset folder
After all this I am getting the above error when I try to run train.py
from datetime import datetime
import numpy as np
import tensorflow as tf
print(str(datetime.now()) + ': loading data files')
Data sets
trainDataFileName = 'airs-dataset/train.csv'
testDataFileName = 'airs-dataset/test.csv'
validationDataFileName = 'airs-dataset/valid.csv'
Load datasets.
trainData = tf.contrib.learn.datasets.base.load_csv_without_header(
#trainData = tf.data.load_csv_without_header(
filename=trainDataFileName,
target_dtype=np.int,
features_dtype=np.int)
testData = tf.contrib.learn.datasets.base.load_csv_without_header(
#testData = tf.data.load_csv_without_header(
filename=testDataFileName,
target_dtype=np.int,
features_dtype=np.int)
validationData = tf.contrib.learn.datasets.base.load_csv_without_header(
filename=validationDataFileName,
target_dtype=np.int,
features_dtype=np.int)
trainingSteps = 1000
totalTrainingSteps = 5000
featureColumns = [tf.contrib.layers.real_valued_column("", dimension=75)]
hiddenUnits = [100, 150, 100, 50]
classes = 2
modelDir = 'model'
classifierConfig = tf.contrib.learn.RunConfig(save_checkpoints_secs = None, save_checkpoints_steps = trainingSteps)
classifier = tf.contrib.learn.DNNClassifier(feature_columns = featureColumns,
hidden_units = hiddenUnits,
n_classes = classes,
model_dir = modelDir,
config = classifierConfig)
Define the training inputs
def getTrainData():
x = tf.constant(trainData.data)
y = tf.constant(trainData.target)
return x, y
Define the test inputs
def getTestData():
x = tf.constant(testData.data)
y = tf.constant(testData.target)
return x, y
Define the validation inputs
def getValidationData():
x = tf.constant(validationData.data)
y = tf.constant(validationData.target)
return x, y
print(str(datetime.now()) + ': training...')
prevAccuracy = -1.0
for i in range(totalTrainingSteps // trainingSteps):
classifier.fit(input_fn=getTrainData, steps=trainingSteps)
currentAccuracy = classifier.evaluate(input_fn=getValidationData, steps=1)['accuracy']
print(str(datetime.now()) + ': training: validation step: ' + str(i) + ' currentAccuracy:', currentAccuracy)
#if(currentAccuracy <= prevAccuracy):
#    break
prevAccuracy = currentAccuracy
print(str(datetime.now()) + ': training...')
print(totalTrainingSteps)
#input_fn = getTrainData
#print(input_fn)
classifier.fit(input_fn=getTrainData, steps=totalTrainingSteps)
print(str(datetime.now()) + ': testing...')
#input_fn1=getTestData
accuracy = classifier.evaluate(input_fn=getTestData, steps=1)['accuracy']
print(str(datetime.now()) + ': done')
print(str(datetime.now()) + ': accuracy:', accuracy)

Same Error coming. Checkpoint error even with the tensorflow,python and pillow specifications as mentioned in the readme. Kindly correct it.

@shubhamsharma1609 I assure you that I have tested this code back when I pushed it, and I have worked with others as well who were able to use it ,,, I will try to rerun the code and see what went wrong,,, however, as far as I understand the code for images and csv files manipulation is working fine with you, and the problems are related to tensorflow usage,,, so in the mean time you can replicate the same idea with some other framework like keras for example, and still use the code for preparing the csv files.
You can easily see the architecture of the neural network used, and build the same network with other framework to make use of time.

Same Error coming. Checkpoint error even with the tensorflow,python and pillow specifications as mentioned in the readme. Kindly correct it.
Can you write the data to a csv file？


Refactor core functions from @cell-decorated containers

Is your feature request related to a problem? Please describe.
Functions which take a cell, somehow decorate them (by adding grating couplers, straights, pins, etc.), and return a new, decorated cell we often refer to as "containers". These functions, in principle, should be generic, but by registering them with @cell we introduce some problems:

they must be registered in downstream PDKs too for proper functionality
they introduce an extra layer of hierarchy which is not always desired

It would be nice to refactor out the core functions so that they could be used more flexibly.
For example
@cell
def add_grating_couplers(component, ...):
    c = gf.Component()
    # add component and other stuff to c
    return c

Would become
def add_grating_couplers(top_component, inner_component, ...):
   # add inner_component and other stuff to top_component
   return top_component

Note that the current @cell containers could still exist and simply be refactored to use the new, generic functions.
@flaport
@sebastian-goeldi
@joamatab

How can we do this and don't create duplicated cell names, when, for example, using a file watcher?

Why would that matter? This would only impact how the general workflow is. If your cell wants pure instances, this would just recreate that cell. This change would allow you to do the following (the add_gc might not be the best example, but e.g. add_extents is a good example)
@cell
def my_coupled_component() -> gf.Component:
    c = gf.Component()
    my_comp_ref1 = c << my_comp
    add_grating_coupler(my_comp_ref1, gc_array)
    my_comp_ref2 = c << my_comp
    add_grating_coupler(my_comp_ref2, gc_array)
    my_comp_ref3 = c << my_comp
    add_grating_coupler(my_comp_re3, gc_array)
    return c

Without unecessarily creating a hierarchy level for each ref or having to have an intermediate level where it might not be desired

yes exactly, it doesn't matter. these are functions that should be used within other cell functions
I think you mean for your second argument to be c, not gc_array, right @sebastian-goeldi ? or I would suggest as the first argument like this
@cell
def my_coupled_component() -> gf.Component:
    c = gf.Component()
    my_comp_ref1 = c << my_comp
    add_grating_coupler(c, my_comp_ref1)
    my_comp_ref2 = c << my_comp
    add_grating_coupler(c, my_comp_ref2)
    my_comp_ref3 = c << my_comp
    add_grating_coupler(c, my_comp_ref3)
    return c

Ah, yes of course. I mixed my mind a bit up when using the function as I barely use wrapper functions ^^


Window service refactor

Codecov Report

Merging #574 into master will increase coverage by <.01%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #574      +/-   ##
==========================================
+ Coverage   92.88%   92.89%   +<.01%
==========================================
  Files          78       79       +1
  Lines        2587     2589       +2
  Branches        2        2
==========================================
+ Hits         2403     2405       +2
  Misses        182      182
  Partials        2        2

Impacted Files
Coverage Δ

frontend/src/app/redirect.component.ts
81.81% <100%> (+4.89%)
:arrow_up:

frontend/src/app/window.service.ts
75% <75%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b61130b...f5e5457. Read the comment docs.


test: example selecting VC template based on basic one

According to @troyronda:
we should add an additional test case to provide an illustration of how a developer could choose which template to use.
decode into a basic verifiable credential
based on type info in the basic verifiable credential, decode into the more specialized type.
i.e., keep this test function that you created in this PR and add another one to illustrate the above.
Story: #218
Signed-off-by: Dima<EMAIL_ADDRESS>

Codecov Report

Merging #227 into master will increase coverage by 0.13%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #227      +/-   ##
==========================================
+ Coverage   87.42%   87.55%   +0.13%
==========================================
  Files          37       37
  Lines        1733     1736       +3
==========================================
+ Hits         1515     1520       +5
+ Misses        120      119       -1
+ Partials       98       97       -1

Impacted Files
Coverage Δ

pkg/doc/verifiable/credential.go
91.47% <100%> (+1.79%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 00ec3d8...c788425. Read the comment docs.

I think the commit description is now about providing an example rather than a utility function, at this point.

I think the commit description is now about providing an example rather than a utility function, at this point.

Agree with Troy! I've updated the task description.

Remember the commit title too :).

Remember the commit title too :).

Done


DefaultBreadcrumbBarItemStyle is not accessible to customize

DefaultBreadcrumbBarItemStyle needs to be defined in theme resources file.

This is true for the majority of styles for controls authored in Winui2.  However you can just use the default style merging to accomplish customization. ie:

This style will be merged with the DefaultBreadcrumbBarItemStyle automatically because it is the metadata defined default style.
What scenario does adding it to the theme resources enable?


Feature/noref/info card display

Major changes:

Seperated CSS styles from index.html to improve organisation of components
Added info-card display for markers, data pulled from zoo-info.csv file
Added weather widget and map zoom controls
Added webpage banner element

Minor changes:

Colour coding for visitor/guide messages

I am having some issues changing the upstream branch, so I will be making a new branch to work on the weather widget and webpage banner.


Error Page editing your credential page (Bug)

Once you click on edit at credentials page, an error page pops up
I am getting
No route matches [GET] "/user_credentials/5/theme-chrome.js
15:37:11 web.1  |   Rendered user_credentials/_form.html.erb (37.3ms)
15:37:11 web.1  |   Rendered user_credentials/edit.html.erb within layouts/application (39.8ms)
15:37:11 web.1  |   Rendered layouts/_navigation.html.erb (4.9ms)
15:37:11 web.1  |   Rendered layouts/_messages.html.erb (0.2ms)
15:37:11 web.1  |   Rendered application/_upgrade_warning.html.erb (0.1ms)
15:37:11 web.1  |    (0.5ms)  SELECT `agents`.`name`, `agents`.`id` FROM `agents` WHERE `agents`.`user_id` = 1  ORDER BY agents.created_at desc
15:37:11 web.1  |    (0.4ms)  SELECT `scenarios`.`name`, `scenarios`.`id` FROM `scenarios` WHERE `scenarios`.`user_id` = 1
15:37:11 web.1  | Completed 200 OK in 252ms (Views: 244.8ms | ActiveRecord: 2.1ms)
15:37:12 web.1  |
15:37:12 web.1  |
15:37:12 web.1  | Started GET "/user_credentials/5/theme-chrome.js" for <IP_ADDRESS> at 2016-03-29 12:37:12 -0700
15:37:12 web.1  |
15:37:12 web.1  | ActionController::RoutingError (No route matches [GET] "/user_credentials/5/theme-chrome.js"):
15:37:12 web.1  |   config/initializers/silence_worker_status_logger.rb:5:in `call_with_silence_worker_status'
15:37:12 web.1  |

The error goes away when I remove
editor.setTheme("ace/theme/chrome") in app/javascripts/pages/user-credential-page.js.coffee
My guess is that theme-chrome.js used to exist but not any more?
@cantino is this correct and should I remove it?

Good catch!


Won't Start on Linux Mint 17.3

I have installed and re-installed but I keep getting the same issue. It won't start and I get no error message.

Can u start like this: sudo electrum-stratis -v, then it should give some more information.

This is the error I get:
Error: could not find paymentrequest_pb2.py. Create it with 'protoc --proto_path=lib/ --python_out=lib/ lib/paymentrequest.proto'

@Bearclay please complete all the steps to install the wallet, you can find them in the readme:
https://github.com/stratisproject/electrum-stratis/blob/master/README.rst


Automod: Sanitize content on text based triggers

Adds toggles to every text based AMv2 rule for sanitizing text
Also adds "sanitize" function to CCs

Oop test failed.
Lemme see what I did wrong rq lol

Oh I think it's just because the package doesn't exist on the actual repo

Note that there are many more obvious "lookalike" characters than those included here, such as Α (GREEK CAPITAL LETTER ALPHA). I do recognize that you've selected a subset of "common" characters, but these are unlikely to satisfy all clients. For this reason, I'm not convinced that this feature belongs in core; the criteria for whether a character is "spoofed" is too broad for the trigger to be useful.

Note that there are many more obvious "lookalike" characters than those included here, such as Α (GREEK CAPITAL LETTER ALPHA). I do recognize that you've selected a subset of "common" characters, but these are unlikely to satisfy all clients. For this reason, I'm not convinced that this feature belongs in core; the criteria for whether a character is "spoofed" is too broad for the trigger to be useful.

Yes, I am aware that there are a lot more than what I have provided. At this current time, I just nabbed all the characters that https://onlineunicodetools.com/spoof-unicode-text uses, but I do plan on adding more into the map.

I believe this would be something such as the nouns, verbs, etc. lists, as they obviously always can add more, but the lists are still present.

Wrong button

lmk if you can think of any more confusable characters not included

go has its own normalization tools, why use this manual file for char comparisons?
https://go.dev/blog/normalization

go has its own normalization tools, why use this manual file for char comparisons? https://go.dev/blog/normalization

I was not aware of this, nor have I tested that lib yet. I'll check if it's better in a bit

Just tested, and the output of unicode/norm does not give adequate normalization. I.e. "Crillic Small Letter A" is not normalized into "Latin Small Letter A"
Tested code:
package main

import (
	"fmt"
	"unicode"

	"golang.org/x/text/transform"
	"golang.org/x/text/unicode/norm"
)

func isMn(r rune) bool {
	return unicode.Is(unicode.Mn, r)
}

func main() {
	x := []rune{1072, 1072, 1072, 1072} // аааа
	z := []rune{97, 97, 97, 97}         // aaaa
	t := transform.Chain(norm.NFD, transform.RemoveFunc(isMn), norm.NFC)
	normStr1, _, _ := transform.String(t, string(x))

	fmt.Println(normStr1 == string(z))
}

for cyrillic matching, this is not normalization, but this package should help... "the confusables"  : ) https://github.com/oragono/confusables

for such Cyrillic matchings etc, this is already not normalization, but this library should help... "the confusables" : ) https://github.com/oragono/confusables
this is similar to your common/normalize.go, just maybe more complete

The library mentioned just looks to be an overcomplicated replacer.

maybe instead of making separate normalize.go file, just place those functions inside common/util.go

maybe instead of making separate normalize.go file, just place those functions inside common/util.go

honestly, yea.

Any updates & things I need to change?


 git autosquash commit messages, because it can't be marked as an acceptable type.
Testing
Note: Couldn't get local testing working, so I just used this PR and the github action to test.
While having "squash" commits on the PR, I got a passing build with the following warnings:

You have commit messages with warnings
⧗   input: squash: remove failOnWarnings
⚠   Reminder to squash all 'squash' commits [squash-match]
⚠   found 0 problems, 1 warnings

Merge checklist:

[ ] All reviewers approved
[x] CI build is green
[x] If adding new checks, followed how_tos/0001-adding-new-checks.rst
[x] Changelog record added (if needed)
[x] Documentation updated (not only docstrings) (if needed)
[ ] Commits are squashed

Since commitlint already ignores squash! and fixup!, I'm going to close this PR rather than adding custom rules.


Issue 347 remove iam js

This PR removes the dependency on iam-js. iam-js was a wrapper for pbac, it was done while waiting for https://github.com/monken/node-pbac/pull/1 to be merged.
The PR does not remove completely the iam module because the actions still needs to be used. There is also a ticket ( https://github.com/nearform/labs-authorization/issues/338 ) that discusses some shortcomings related to it.
Ref. issue #347

Coverage decreased (-0.02%) to 95.997% when pulling 8d2db07a0ed7528cf311091ff91afe9c0bf9d8bf on issue-347-remove-iam-js into 21e84a6a8cb98612c05cdec3d1186c3028819e34 on master.

Coverage decreased (-0.02%) to 95.997% when pulling 8d2db07a0ed7528cf311091ff91afe9c0bf9d8bf on issue-347-remove-iam-js into 21e84a6a8cb98612c05cdec3d1186c3028819e34 on master.

Coverage increased (+0.1%) to 96.144% when pulling bed4089ff34389046025fadded84782bacb0d15d on issue-347-remove-iam-js into 21e84a6a8cb98612c05cdec3d1186c3028819e34 on master.

Coverage increased (+0.1%) to 96.051% when pulling 162d2f3639aa13cdc4a14be9db032486dd9f82ac on issue-347-remove-iam-js into<PHONE_NUMBER>0070f473531e2f51890c9e74fdc4d4 on master.


Nonstandard relocs

Resolves #64

Is the intention that the 192-255 space is a free-for-all, or that non-standard relocation numbers will be managed in some way? Or is this patch purposely avoiding making such a determination?

My plan was the former: chaos, just like the ISA opcode space for nonstandard extensions.  I guess I should add a sentence to that effect?

Yes, chaos was my assumption but just leaving it marked as "reserved" without further explanation might lead someone to think there's something more managed going on. An extra sentence would be a good idea I think.

Yep.


Setting Bing Maps Key on SPO tenant level?

Hello Dev team,
May I please ask for a confirmation whether one Bing Maps Key can be set on SPO tenant level?
(I am not talking about SP On-Prem farm level, as written in this article, or SPO web level)
I just want to know whether I can set one key on SPO tenant level, so that it can be applicable for all SPO web site collections, or I need different keys for different site collections?
Thanks in advance!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4f512d30-b714-7b3c-a567-c5c58197c636
Version Independent ID: 5e7b7d5a-88eb-4472-074c-469441a27ae5
Content: Set the Bing Maps key at the web and farm level in SharePoint
Content Source: docs/general-development/how-to-set-the-bing-maps-key-at-the-web-and-farm-level-in-sharepoint.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Yes, see the feature "tenant properties": https://docs.microsoft.com/en-us/sharepoint/dev/spfx/tenant-properties


Event fixes and addition

correct use of Event vs Eventf
fix case of message
use common/events
emit event when unhealthy node condition is found

ECOPROJECT-1797

/lgtm


Example on JMP x Flax?

Hey, this mixed-precision functionality is awesom. I note the haiku example and wonder if there is a example on using JMP on Flax?

@tomhennigan Do you have any ideas or comments?

I believe Flax have their own mixed precision support, I am not aware of examples of them using or integrating JMP.
Their imagenet example includes an example of mixed precision training https://github.com/google/flax/tree/main/examples/imagenet/.


Loading screen when tsc working

Is your feature request related to a problem? Please describe.
Many people used to see some "Loading..." sign in terminal when tsc is working. With Vite we cannot understand when tsc is working.
Describe the solution you'd like
Implement some method to check tsc loading.
Describe alternatives you've considered
No response
Additional context
No response
Validations

[X] Read the docs.
[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that asks for the same feature to avoid creating a duplicate.

You mean there should a loading indicator when tsc is loading(especially on the initial).


Rest Client Address Bar is too small

Rest-client address bar looks too small just after the update.

is there any estimate when this fix is going to be available?

That is extremely annoying :/

Problem was caused when Firefox removed support for their custom "-moz-calc" CSS function.  CSS3's "calc" does the same thing.  Fixing this is a simple find-and-replace.
If you would like a workaround and you are using Firefox, you can define a userContent.css file to override the setting locally.  Do a web search on "Firefox userContent.css" for instructions on where to put this file.  Inside the file, add the following:
@-moz-document domain(restclient) {
    #request-url {
    	width: calc(100% - 380px) !important;
    }
}

Restart Firefox and your address bar should be back to normal width.  Downside is that if the CSS ever does get fixed (or changed), you will not pick up the new code, and remembering that you made this custom override may be difficult.

yes, there are definitely a couple of workarounds (another which comes to mind is just edit this prop in inspector) but this one is a bug and should be fixed. moreover, there's PR addressing this problem, it just needs to be merged in.

CSS can also be adjusted via Stylish add-on. No need to bother with userContent.css
And the styling is applied immediately, without browser restart. (the add-on installation does require a restart)
P.S. I've also added this rule:
    #request-body {
        height: 300px !important;
    }

Fixed by https://github.com/chao/RESTClient/pull/265

An update is published.

Fixed in RESTClient 3


Ruby 2.5.7 support

Updating our ruby version, but now with actions our CI is unable to use the latest 2.5.x Ruby to run tests etc.
https://www.ruby-lang.org/en/news/2019/10/01/ruby-2-5-7-released/

Same here. Are there timelines from the time of release to the availability of ruby versions?

I need ruby 2.6.5, and unless we can get support for that then unfortunately it's a no-go from the offset. I think this is going to be a pretty ubiquitous requirement for any serious app.  🙏

Hej! I built https://github.com/clupprich/ruby-build-action over the weekend, which makes use of ruby-build to install exactly the version of Ruby you need.
While that sounds (and actually is) slow, you can use the actions/cache action to cache the Ruby installation in your workflows (which brings the time it takes to setup Ruby down to ~30 seconds).
Let me know if that works for you. Feeback/issue reports/contributions welcome!

@clupprich That absolutely solved this for me, and I imagine it will do so for many others! Great job.
💯 💯 💯 💯 💯 💯

Thanks @clupprich...that worked perfectly! Not sure how useful setup-ruby is when you can't specify the exact ruby version you need.

Not that everyone is testing on Windows, but I created MSP-Greg/actions-ruby, which installs current Ruby versions (along with ruby-head/master) and allows updating the gcc build tools/packages on Windows.
Ruby 2.3.3 is also available, but if used for extension gems, it compiles with the newer 'DevKit'...

+1 for Ruby 2.5.7 support.

+1

I think this requires an update to the virtual environment, so I’ve requested it here: https://github.com/actions/virtual-environments/issues/95

What are the currently supported Ruby versions? Our actions started failing today with a similar message, Version 2.5.5 not found.

The support page linked by @konradpabjan in https://github.com/actions/setup-python/issues/15#issuecomment-551222017 still says that Ruby 2.5.5 is supported.

Ruby 2.5.5 should work. Just tested it out:

https://github.com/konradpabjan/Testing3/commit/9561e0ac0a4e7dcd4ab209aa0aacd3e4df02203c/checks?check_suite_id=365213462#step:4:4
Looking at the tools cache where the Ruby gets downloaded from, I still see version 2.5.5. It gets added here and automatically documented: https://github.com/actions/virtual-environments/blob/9786a9946a6db1a8d0c89948550753f9e01007b3/images/linux/scripts/installers/1804/hosted-tool-cache.sh#L39
If there are issues with the documented versions then actions/virtual-environments is the best place to report it.

2.5.5 is working for us again today. Thanks for testing it.

Closing since 2.5.5 is working and there's already a tracking item in virtual-environments.
Note a related issue: https://github.com/actions/setup-ruby/issues/44


Not working iOS

Worked fine, but when installed this plugin:
2018-10-27 19:06:45.780660+0200 EasyFysio[3835:563802] Apache Cordova native platform version 4.5.5 is starting. 2018-10-27 19:06:45.780993+0200 EasyFysio[3835:563802] Multi-tasking -> Device: YES, App: YES 2018-10-27 19:06:45.849558+0200 EasyFysio[3835:563802] CDVWKWebViewEngine: trying to inject XHR polyfill 2018-10-27 19:06:45.870861+0200 EasyFysio[3835:563802] *** Terminating app due to uncaught exception 'NSUnknownKeyException', reason: '[<WKWebViewConfiguration 0x10260c0e0> setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key _requiresUserActionForMediaPlayback.' *** First throw call stack: (0x18f5fbef8 0x18e7c9a40 0x18f513dcc 0x18ffa44ac 0x100cfbfb0 0x100cb32fc 0x10105d4c8 0x10105cc10 0x10105d0a4 0x10105a7d8 0x100bd9c38 0x1bbe35380 0x1bbe357b0 0x1bc4302d0 0x1bc430878 0x1bc44186c 0x10105f1e0 0x100bd9954 0x1bc3efc68 0x1bc3f13d0 0x1bc3f6e1c 0x1bbc97c18 0x1bbca06f0 0x1bbc97894 0x1bbc98234 0x1bbc96334 0x1bbc95fe0 0x1bbc9b1a0 0x1bbc9c100 0x1bbc9b058 0x1bbc9fd9c 0x1bc3f5314 0x1bbfe0c34 0x19202b890 0x192036658 0x192035d50 0x1019d4de4 0x1019d8a2c 0x19206a640 0x19206a2cc 0x19206a8e8 0x18f58a5b8 0x18f58a538 0x18f589e1c 0x18f584ce8 0x18f5845b8 0x1917f8584 0x1bc3f8bc8 0x100bd9844 0x18f044b94) libc++abi.dylib: terminating with uncaught exception of type NSException

@chris087
Here is solution:
Install plugin using

cordova plugin add https://github.com/katzer/cordova-plugin-background-mode --nofetch
npm install --save @ionic-native/background-mode
Then
ionic cordova build ios

Thanks! The same issue,that works for me .Bu the way ,my problem more serious ，my app crash when xcode simulator.

@l-xue-yu Please check it on real device, or try to set entry component in Xcode if it crash on simulator.
Regards

@l-xue-yu Please check it on real device, or try to set entry component in Xcode if it crash on simulator.
Regards

Thank you for reminding me check in real device.I try in iphone6s and app not crash.

@l-xue-yu  My Pleasure :)


Version 5 conflict with Bootstrap

In version 5.x there is a CSS conflict with Bootstrap.
I'm using the Default theme, but it appears to affect other themes as well.
In default-base.css there is this global CSS selector that sets the box-sizing:
*,
*::before,
*::after {
box-sizing: inherit;
}
Bootstrap 4 and 5 have a similar selector that looks like this:
*,
*::before,
*::after {
box-sizing: border-box;
}
Since it's global (not one of the .rz- selectors) I think it's overriding the Bootstrap selector and causing layout problems.
Specifically it affects the size of standard  text boxes. After updating the Radzen components to version 5, all of my text boxes  appeared too large.
e.g.

Hi @leon42,
box-sizing: inherit; means that it will inherit the formerly set by Bootstrap box-sizing: border-box;

Make sure bootstrap.css is included before default-base.css
Clear you browser cache
I suspect this size change might be caused by a root font-size. Could you inspect the textbox and see what are the computed font-size and padding?

I ran into this issue as well with the bootstrap .nav class and the NavLink component after updating. It might be another point here to help figure out the bootstrap change/issue.
Comparing between Radzen 4.34.4 and 5.0.6 the only computed value to change is box-sizing. All padding and font stuff remained the same. In my case it is now inheriting to "content-type" when previously it was just set to "border-box".

🤦🏻 Oh, misunderstood the issue! As @leon42 was refering to "text boxes" I initially thought it was RadzenTextBox component that got affected, while it was actually the standard input element Bootstrap styles being overridden by Radzen.
Thank you @psantoro3 and @leon42 for clarifying.

I just updated my app, and RadzenNumeric which originally had a w-100 class, the span's inner input is no longer sizing properly.


Drone missing nsswitch.conf

When pulling drone:1.1.0 there seems to be a regression from when this was last fixed in #1306
Drone no longer seems to respect /etc/hosts.

I cannot remember the exact reasons, but the the nsswitch was removed back in 2016

Drone is based on alpine, if I remember correctly Müsli libc didn't rely on nsswitch.conf

If I understand this correctly, it is golang's net package which assumes there will always be an nsswitch, which isn't always the case.
Which I suppose leaves 3 options:

Add an nsswitch anyway so that drone will read /etc/hosts (though as @bradrydzewski suggests this may cause other issues)
Bake support for reading /etc/hosts into drone itself
Status quo and require the end user to mount an nsswitch.conf file at runtime

It would be ideal if the drone images supported Docker linking and Kubernetes hostAlias out of the box.

I can confirm that running drone with the flag -v /etc/nsswitch.conf:/etc/nsswitch.conf makes it work.
So I've made a PR: https://github.com/drone/drone/pull/2820

I wanted to add the nsswitch.conf because I needed to setup the gitea service ip on the /etc/hosts.
If this is your case, just using the steps above won't be enough, as even though the Drone server will access the correct IP, the pipeline slaves won't because those dockers won't have the nsswitch.conf either.
So the solution I've found is to use the host dns server for the containers. You can
specify the dns servers in the docker networks, but if you don't want to specify the --network flag on the drone slaves, you have to modify the bridge0 network. To do so you have to add this configuration to the /etc/docker/daemon.json
{
  "dns": ["<IP_ADDRESS>"]
}

Usually systemd machines have a systemd-resolv service listening on
<IP_ADDRESS>:53, the problem is that you can't access that address from within
the container, so you would have to make systemd-resolv listen to the inet
address of the bridge0 network, or use another dns server that forwards the dns
queries from the docker network to the systemd one or the /etc/hosts file.
$ ifconfig docker0
docker0: flags=4099<UP,BROADCAST,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
...

I've found easier to do the second one, so you just have to install dnsmasq,
configure the listenaddress in /etc/dnsmasq.conf.
listen-address=<IP_ADDRESS>,<IP_ADDRESS>

and restart both dnsmasq and docker services.

closed by f6ba3ea7b2aa10dfc79ca43c156ac2924ba1e039


Upcoming blog posts are promoted

Blog posts which are present in the "upcoming" directory get the New status and are shown on the main page. We need to hide them the same way they are hidden on blog.sourced.tech

Fixed by src-d/blog#106. Closing this as is a blog issue.


Copyedit and clarify phrasing

Copyediting: grammar and usage, slight clarification of explanations

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


Very low testing mIoU

I train the model on VOC2012 segmentation, using 250 epoch and 20 batch_size.
The training accu is 0.985~.(It should be the pixel accu, right?)
But the testing mIoU is just 0.56025, which has a big gap with the result in the paper.
What's your result?
And maybe the hyperparameter is not optimal?
[ 0.9077976 0.75288631 0.47242709 0.58043887 0.50712961 0.47588975
  0.72664924 0.68814923 0.66110743 0.20972129 0.53752346 0.31540707
  0.5731418 0.50105971 0.64968748 0.76978092 0.43357291 0.5765887
  0.32090376 0.61638412 0.49687383]
meanIOU: 0.560625
pixel acc: 0.898940

Which model did you use? And can you list your hyperparameters? The hyperparameters I left in the code were used for resnet.
 The dataset used for those papers should include about 10k images for training.

My hyperparameters is same as your except the batch_size is 20.
And the model is resnet50, same as in your code.
I just changed the file path and the batch_size.
The training images number is 1400+, I don't use the SegmentationClassAUG.
I am training on the SegAug label right now.
I will update my result later.
And what's your result in the original Seg label and the SegAug label?

Update:
I run the model in SegmentationAug data using 25 epoch and 32 batch_size, and keeping other hyperparameters unchangeable.
The testing mIoU is 0.661076.
It seems acceptable.
And maybe I can boost it by training more epoches.
Could you please tell me your results?
I want to compare with yours.
Thanks.

Hmm... Actually, I got 65.75 mIOU using 25 epoch and 32 batch size. Your result is slightly better than mine...

I run more 25 epoches based on the 0.661076 model, but the result is almost the same.
BTW, thank you for your code of the FCN in keras, its very helpful for me!

@Fansiee Glad to help :)

@Fansiee Aren't the classes a bit different in the augmented pascal VOC?
I assume you're discussing these:
    # original PASCAL VOC 2012
    # wget http://host.robots.ox.ac.uk/pascal/VOC/voc2012/VOCtrainval_11-May-2012.tar # 2 GB
    # berkeley augmented Pascal VOC
    # wget http://www.eecs.berkeley.edu/Research/Projects/CS/vision/grouping/semantic_contours/benchmark.tgz # 1.3 GB

@ahundt I think your link is correct and the classes should be the same. Just add all augmented data to your training set and change the hyper parameters, and you should be able to reproduce the result.


Remove file type extension in URL that allows for content negociation

if you curl localhost:8888/users/1.xml you can get XML without passing an Accept header.
But reading more about RESTful design, I learned it goes against the URI definition, because /users/1 would be considered different from /users/1.xml.
It makes sense to me, and it was more of a gimmick to play in the browser anyway.
It will simplify the code and remove some ugly hacks and dependencies on hardcoded mime types.
ref: https://restfulapi.net/content-negotiation/#comment-1099

Also: file extension support imposes restrictions on entities identifiers.


Noble Soul Legacy not adding class level to spell penetration checks

from user:

Correction: Electric and Cold blasts do not add class level to spell penetration bonuses. Fire works as intended. Tested as dhampir isekai vampire on overlord subtype for isekai protagonist with both Kinetic Lord and Noble Soul legacies. Only this mod, the required mods and toybox mods present at time of testing.

Fire with eruption infusion also seems to be not adding class level. Extended Range infusion seemed to work with Fire, Blue Flame, Cold.

@kjk001 The ContextCalculateAbilityParamsBasedOnClass component for each blast ability calculates the the caster level, spell level, casting stat etc. Simply adding the same component with the Isekai Protagonist class reference doesn't work and seems to use only the original ContextCalculateAbilityParamsBasedOnClass compoent. Replacing the component instead will work. However, the original kineticist class will break (i.e. blasts will not scale on caster level, spell level etc.).
Possible solutions:

A solution could be defining a new component class that is a clone of the ContextCalculateAbilityParamsBasedOnClass compoent and add our component, but then we need some way of deciding whether to apply the original kineticist's ContextCalculateAbilityParamsBasedOnClass component or to our own component.
Another solution could be creating a clone of the ability instead and then replace the component with a reference to the Isekai Protagonist class. This will be very work intensive, and new blueprints need to be added.

I have replaced the ContextCalculateAbilityParamsBasedOnClass with my own custom component that computes that highest class level between the kineticist the Isekai Protagonist.


Add italian translation

May need some reviewing. I can notice, and I can't figure out why, that "Press new button" isn't actually getting translated, it stays in english.

Hey Simone,
I found a couple of areas where the translation overlaps the UI. Can you please fix them?

The "+New" button is showing up translated for me. Are you referring to a different button?

Thanks!

Hey, I'll have to be clever and come up with semantically/grammatically correct but also short italian sentences to fix those :D And no, I'm referring to the text that shows up when you rebind in the settings. The original string is "Press new button"

Oh I see, I just pushed a change to dev that should fix that.

I can confirm that text now gets translated correctly


fix(slide-toggle): drag not working in edge

Some browsers like Edge or IE11 emit click events differently than other browsers like Chrome. For example does Edge still emit a click event if the pointer drags something around and is being released over the <label> element. This doesn't happen on Chrome.
Right now this means that the new checked state from the drag is being reverted by the click event that then causes a change event on the underlying input element. To ensure that the behavior is the same across all supported browsers, the checked state from the click event is being ignored while dragging.
Fixes #8391

LGTM


Sequence Diagrams for Search Log

Add sequence diagrams for search logs and word cloud visibility.

Sequence diagrams are added to the Software Design Wiki Page.


scipy pin preventing Python 3.12 support?

Hi, I'm attempting to upgrade some of our code to Python 3.12, and I've noticed a conflict involving scispacy that I think prevents installing it on Python 3.12:

scispacy 0.5.4 pins scipy<1.11 (commit)
scipy<1.11 doesn't support Python 3.12 (docs)
scispacy 0.5.3 only supports spacy<3.7
spaCy doesn't have support for Python 3.12 until 3.7 (docs)

Any idea if it's possible to relax the scipy constraint to allow for running this on Python 3.12?  Thanks!

So, on later versions of scipy this code (https://github.com/allenai/scispacy/blob/021fe76d69b20523d3f94a08b447c27e1a46597e/scispacy/candidate_generation.py#L451-L453) errors with ValueError: Output dtype not compatible with inputs.. This is only required for the code used to create the linkers, not actually for running, so the library should actually function fine with a later version of scipy. But I'd need to dig a bit more to see how to actually resolve this issue.

Can be reproduced locally by running pytest tests/ with scipy>=1.11 installed

It seems like they don't support this, so may take a bit of work to decide what to do...https://github.com/scipy/scipy/issues/7408

Thanks for the quick response!  Yikes... glad it's a relatively small incompatibility, but that's a tricky one.
We use a pretty strict build system, so I can't override the constraint even if the library works OK with newer scipy.  I'll attempt to find a solution for the float16 issue if upgrading becomes a blocker for us.

I'm not able to replicate a functional environment for CandidateGenerator, even using the instructions for a 3.9 conda environment. Strangely this was working recently so I'm not sure if something else changed.
conda create -n scispacy2 python=3.9
conda activate scispacy2
pip install scispacy

>>> from scispacy.candidate_generation import CandidateGenerator
>>> generator = CandidateGenerator(name='umls')

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/alejandro/miniconda3/envs/scispacy2/lib/python3.9/site-packages/scispacy/candidate_generation.py", line 221, in __init__
    self.ann_index = ann_index or load_approximate_nearest_neighbours_index(
  File "/home/alejandro/miniconda3/envs/scispacy2/lib/python3.9/site-packages/scispacy/candidate_generation.py", line 132, in load_approximate_nearest_neighbours_index
    concept_alias_tfidfs = scipy.sparse.load_npz(
  File "/home/alejandro/miniconda3/envs/scispacy2/lib/python3.9/site-packages/scipy/sparse/_data.py", line 72, in astype
    self._deduped_data().astype(dtype, casting=casting, copy=copy),
  File "/home/alejandro/miniconda3/envs/scispacy2/lib/python3.9/site-packages/scipy/sparse/_data.py", line 32, in _deduped_data
    self.sum_duplicates()
  File "/home/alejandro/miniconda3/envs/scispacy2/lib/python3.9/site-packages/scipy/sparse/_compressed.py", line 1118, in sum_duplicates
    self.sort_indices()
  File "/home/alejandro/miniconda3/envs/scispacy2/lib/python3.9/site-packages/scipy/sparse/_compressed.py", line 1164, in sort_indices
    _sparsetools.csr_sort_indices(len(self.indptr) - 1, self.indptr,
ValueError: Output dtype not compatible with inputs.

So, on later versions of scipy this code (
https://github.com/allenai/scispacy/blob/021fe76d69b20523d3f94a08b447c27e1a46597e/scispacy/candidate_generation.py#L451-L453
) errors with ValueError: Output dtype not compatible with inputs.. This is only required for the code used to create the linkers, not actually for running, so the library should actually function fine with a later version of scipy. But I'd need to dig a bit more to see how to actually resolve this issue.

I get the error with scipy==1.10.1 on Azure Databricks


Sagemcom F5684 Not Working

Versions

Key
Value

Home Assistant version
2024.1.6

Integration version
0.3.1

Sagemcom F@st model

Sagemcom F@st firmware (optional)

Describe the bug
Login not working at all, tested both MD5/SHA512, admin & guest, with personal and no password. SSL checked (https) but no SSL certificate validation.
Additional context
No error or logs in log viewer in HAOS (debug is enabled as described in wiki.
Please note that I can get SSH access as admin into the F5684 using a crafted Javascript command via browser console.
Thank you!
[bug]

Hardware V.	F5684S_v2
software V.	AGGHX_1.2.2
GUI V.	5.112.2


New database but "All migrations are already applied"

I did the Prisma2 tutorial, created the first model User, let me generate migration files via prisma2 lift save --name 'init' and ran prisma2 lift up. It worked but didn't check if the database then actually changed. (Am using Postgres via a Docker Compose.)
I then completely stopped the Postgres server and removed its associated volume that was created via my Docker Compose file:
version: "3"

services:

  postgres:
    image: postgres:11-alpine
    restart: always
    ports:
      - "5433:5432"
    volumes:
      - postgres:/var/lib/postgresql
    environment:
      POSTGRES_PASSWORD: "prisma"
      POSTGRES_USER: "prisma"
      POSTGRES_DB: "prisma"

  adminer:
    image: adminer
    restart: always
    ports:
      - 8080:8080

volumes:
  postgres:

So the data was completely gone. I then re-upped the Docker services and ran prisma2 lift up to migrate my Postgres database to the last state.
The output was All migrations are already applied but there are no tables, since I've completely renewed the Postgres server.
Is this intentionally?

Can you please check whether there is a _Migrations table?
I am unable to reproduce this. Please double-check via the UI.

@pantharshit00 may have looked into the wrong schema. It's working. The default schema was default@default, or something like that. Forcibly set it to prisma with an environment variable. Filled with data :)

@pantharshit00 this issue still persists with the SQLite DB.
prisma2 version: 2.0.0-preview018.1
OS: Linux

I have the same issue with a postgreSQL database

I can confirm this should be fixed in preview018.2


refactor: replace wait with waitFor

wait is deprecated (use https://react-hooks-testing-library.com/reference/api#waitfor)
Before

After

Thanks 🙏


[Baremetal] DRAFT: Remove accumulator and IV from cipher context in CCM-only configurations

Summary: This PR proposes to remove the accumulator and IV from mbedtls_cipher_context in in configurations disabling all but CCM, which (a) always passes data in multiples of the underlying cipher's block length, and (b) passes the IV as a parameter in mbedtls_cipher_auth_xxx().
This saves a few bytes of code and RAM.

I am closing this pull request because the baremetal branch is no longer maintained in this repository. If it is still relevant, please open a new pull request in https://github.com/PelionIoT/pelion-crypto .


SPARKC-613 fix direct join for spark sql

Expression strict matching used in CDJExec and CDJStrategy failed when
one of the expressions contained a qualifier. This failed Direct Join
recognition.
Since we care only about expression names and types, this commit
introduces a loosened expression comparison.

+1


mysql dialect DELETE syntax error

Describe the bug
mysql dialect DELETE statement generates error
To Reproduce
package main

import (
	"fmt"

	"github.com/doug-martin/goqu/v9"
	_ "github.com/doug-martin/goqu/v9/dialect/mysql"
)

func main() {
	dialect := goqu.Dialect("mysql")

	sql, _, _ := dialect.Delete("table").ToSQL()
	fmt.Println(sql)
}

go run bug/main.go
DELETE `table` FROM `table`

Expected behavior
expect
DELETE FROM `table`

Dialect:

[ ] postgres (not test)
[x] mysql
[ ] sqlite3  (ok)

Additional context
Add any other context about the problem here.

@Fanduzi Thats not really a bug. Yes, DELETE FROM abcWHEREid = 1; is the shortest form, but the result of the query builder is not wrong. It is the multiple table syntax. See link.

@Fanduzi Thats not really a bug. Yes, DELETE FROM abc WHERE id = 1; is the shortest form, but the result of the query builder is not wrong. It is the multiple table syntax. See link.

you are right, thank you!


Create Typography Module

For Reference:
http://getbootstrap.com/css/#type
http://foundation.zurb.com/sites/docs/typography-base.html

https://codepen.io/iron16/pen/ybPBoG?editors=0100

Closed by style-modules/module-typography@2535f8f

Superseded by #5


Backend route for getting admins list #154

Issue that this pull request solves
Closes: #154
Proposed changes
Added code for getting admins list
Brief description of what is fixed or changed
Added code for getting admins list with different conditions of superAdmin as true, false and null
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] My changes does not break the current system and it passes all the current test cases.

@atharva-upadhye-01 I cannot find API testing file for this. Please add that to your PR!

@atharva-upadhye-01 I cannot find API testing file for this. Please add that to your PR!

@atharva-upadhye-01 I cannot find API testing file for this. Please add that to your PR!

Please re-request for a review after adding API testing

@atharva-upadhye-01 I cannot find API testing file for this. Please add that to your PR!

Please re-request for a review after adding API testing

What will be the preferred package for API testing ?

What will be the preferred package for API testing ?

Which package should be used for API testing ?

We are using Postman fot API testing, check the reference in the issue description for resources

Which package should be used for API testing ?

We are using Postman fot API testing, check the reference in the issue description for resources

Which package should be used for API testing ?

We are using Postman fot API testing, check the reference in the issue description for resources

Yes, I will learn about Postman testing today, till now I used it only for manual testing.
Initially I thought we would be using something like Mocha & Chai, didn't know postman can do this too !

Which package should be used for API testing ?

We are using Postman fot API testing, check the reference in the issue description for resources

Yes, I will learn about Postman testing today, till now I used it only for manual testing.
Initially I thought we would be using something like Mocha & Chai, didn't know postman can do this too !

Added some tests, please review.

Added some tests, please review.

@atharva-upadhye-01 Please make sure your commits do not contain any linter errors.
Please run npm run lint:fix and then npm run lint and fix those errors before committing.
Can you quickly update this? I am in the middle of reviewing the code

@atharva-upadhye-01 Please make sure your commits do not contain any linter errors.
Please run npm run lint:fix and then npm run lint and fix those errors before committing.
Can you quickly update this? I am in the middle of reviewing the code

@atharva-upadhye-01 Please make sure your commits do not contain any linter errors.
Please run npm run lint:fix and then npm run lint and fix those errors(if any) before committing.
Can you quickly update this? I am in the middle of reviewing the code

Yes, doing it right now

It isn't showing me any problems :(

@atharva-upadhye-01 Please make sure your commits do not contain any linter errors.
Please run npm run lint:fix and then npm run lint and fix those errors(if any) before committing.
Can you quickly update this? I am in the middle of reviewing the code

Yes, doing it right now

It isn't showing me any problems :(

@atharva-upadhye-01 Please make sure your commits do not contain any linter errors.
Please run npm run lint:fix and then npm run lint and fix those errors(if any) before committing.
Can you quickly update this? I am in the middle of reviewing the code

Yes, doing it right now

It isn't showing me any problems :(

@atharva-upadhye-01 Can you come for a very quick meet? https://meet.google.com/tac-tkst-fki

@atharva-upadhye-01 Please make sure your commits do not contain any linter errors.
Please run npm run lint:fix and then npm run lint and fix those errors(if any) before committing.
Can you quickly update this? I am in the middle of reviewing the code

Yes, doing it right now

It isn't showing me any problems :(

@atharva-upadhye-01 Can you come for a very quick meet? https://meet.google.com/tac-tkst-fki

@atharva-upadhye-01 Can you come for a very quick meet? https://meet.google.com/tac-tkst-fki

Yes, just a minute I'll grab my BT earphones

@atharva-upadhye-01 Can you come for a very quick meet? https://meet.google.com/tac-tkst-fki

Yes, just a minute I'll grab my BT earphones

@atharva-upadhye-01 Your postman collection is not running. Perhaps you forgot to include the JWT.  I have added it and made the necessary changes. I am merging your PR with a few changes from my side

@atharva-upadhye-01 Your postman collection is not running. Perhaps you forgot to include the JWT.  I have added it and made the necessary changes. I am merging your PR with a few changes from my side

@all-contributors please add @atharva-upadhye-01 for code

@all-contributors please add @atharva-upadhye-01 for code

@atharva-upadhye-01 Please confirm your participation is for which event? SWOC or DWOC or both?

@atharva-upadhye-01 Please confirm your participation is for which event? SWOC or DWOC or both?

@atharva-upadhye-01 Please confirm your participation is for which event? SWOC or DWOC or both?

DWOC

@atharva-upadhye-01 Please confirm your participation is for which event? SWOC or DWOC or both?

DWOC

@atharva-upadhye-01 Your postman collection is not running. Perhaps you forgot to include the JWT. I have added it and made the necessary changes. I am merging your PR with a few changes from my side

I cannot find backend/tests directory in merged commit. Where were those supposed to be placed in ?
Postman didn't tell me anything about JWT.
Can you please explain me what was I supposed to do ?

@atharva-upadhye-01 Your postman collection is not running. Perhaps you forgot to include the JWT. I have added it and made the necessary changes. I am merging your PR with a few changes from my side

I cannot find backend/tests directory in merged commit. Where were those supposed to be placed in ?
Postman didn't tell me anything about JWT.
Can you please explain me what was I supposed to do ?

@atharva-upadhye-01 Your postman collection is not running. Perhaps you forgot to include the JWT. I have added it and made the necessary changes. I am merging your PR with a few changes from my side

I cannot find backend/tests directory in merged commit. Where were those supposed to be placed in ?
Postman didn't tell me anything about JWT.
Can you please explain me what was I supposed to do ?

Yes I have removed them, I already have a Postman team setup, I will incorporate that directly. I will also share the link to it very soon so that the participants can join.
If you run your request in postman it will fail due to auth error.

@atharva-upadhye-01 Your postman collection is not running. Perhaps you forgot to include the JWT. I have added it and made the necessary changes. I am merging your PR with a few changes from my side

I cannot find backend/tests directory in merged commit. Where were those supposed to be placed in ?
Postman didn't tell me anything about JWT.
Can you please explain me what was I supposed to do ?

Yes I have removed them, I already have a Postman team setup, I will incorporate that directly. I will also share the link to it very soon so that the participants can join.
If you run your request in postman it will fail due to auth error.


Roll Engine from e444009bf46b to 57f7719e6eb3 (3 revisions)

https://github.com/flutter/engine/compare/e444009bf46b...57f7719e6eb3
2021-12-15<EMAIL_ADDRESS>Revert "Removed the email warning about notifications entitlement" (flutter/engine#30330)
2021-12-15<EMAIL_ADDRESS>[Win32, Keyboard] Abstract KeyboardManager from FlutterWindowWin32 (flutter/engine#30296)
2021-12-15<EMAIL_ADDRESS>Revert "Run Dart VM tasks on the engine's ConcurrentMessageLoop instead the VM's separate thread pool. (#29819)" (flutter/engine#30332)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Gold has detected about 19 new digest(s) on patchset 3.
View them at https://flutter-gold.skia.org/cl/github/95305


Concurrent Survey Process

Adds funcionality for CSP
https://trello.com/c/zgCyUiHl/204-3-concurrent-survey-process-step-3-ios-sdk
This commit:

Includes end_user_id, user_id and unique_link on requests to responses and
decline
Creates unique_link algorithm to be a SHA256 hash of NPS token,
end_user_email, date and a random string
Adds priority number to responses and declines
Adds WTRApiClientTests to test CSP changes

Code needed a refactor to make it more testable. I've focused on testing
the parts that involved CSP changes. There is still more testing to be
done but I prefered to leave that for another PR, atomic commits.
One thing I did include in this commit was changing identation of
ViewController.m and WTRApiClient.m from 4 spaces to 2. This is
and inconsistency throughout the project. I decided to settle for
2 spaces but haven't changed other files that use 4. Again, I'm
planning on doing this in a separate PR.

Write some tests.

Glad to see tests coming in here, see if you like my suggestions and please add the test for priority. It's almost done, great work!

Looks much better now, hope we had CI configured for this. 👍

:shipit:

@diegoserranoa :+


update camera optimizer to match nerfstudio

camera optimizer has been move to model in https://github.com/nerfstudio-project/nerfstudio/pull/2092
Currently, it will throw an error if we want to init it in datamanager as before:
TypeError: RayGenerator.__init__() takes 2 positional arguments but 3 were given

Sorry! For some reason I did not see a notification for this PR. Thank you for fixing this!

No worries! I'm happy to help in any way I can.

@lsongx If you have any time, can you help me figure out why the results with newer diffusers/nerfstudio look bad? I narrowed it down to an issue with how the edited images look, so I assume it is a diffusers issue, but if you can just print out the edited images and take a look, I would greatly appreciate it

Is it https://github.com/ayaanzhaque/instruct-nerf2nerf/issues/60?
I have trouble replicating this issue. Everything looks fine on my side. Maybe we can discuss it under that issue


Add animation to preview-example.html

Animate

hover over filter.
selected filter.
disabled filter.
items width, height and opacity when shown/hidden.

Also update the preview README gif.

Closed by https://github.com/robertpainsi/filtering.js/commit/b86184a17211c863424ad33492ce719149c316e7


[Security] Workflow audit.yml is using vulnerable action actions/checkout

The workflow audit.yml is referencing action actions/checkout using references v1. However this reference is missing the commit a6747255bd19d7a757dbdda8c654a9f84db19839 which may contain fix to the some vulnerability.
The vulnerability fix that is missing by actions version could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider to update the reference to the action.

Thank you for the report. The workflow has been updated to using checkout v2 in commit 19b0077c69be215fbaa3939acec4dfd5bbfb1ffc which includes the mentioned commit
https://github.com/aumetra/tranquility/blob/19b0077c69be215fbaa3939acec4dfd5bbfb1ffc/.github/workflows/audit.yml#L12


Not working on iOS 16

I am getting the below error on iOS 16. Recording works well on iOS 14 as I have tested.
this._nativeModule.addListener is not a function. (In 'this._nativeModule.addListener(t)', 'this._nativeModule.addListener' is undefined)

Hi @fizawisedv , can you share the code snippet where you have this issue?

It occurs at the startRecording() method. Below is my code.
    try {
        const recordMic = await PermissionService.checkPermission("mic") // ask for mic permission
        const options = {
            mic: recordMic, // defaults to true
            androidBannerStopRecordingHandler: this.androidBannerStopRecordingHandler
        }

        const recordingStatus = await ScreenRecorder.startRecording(options).catch((error) => {
            LOG.error("RECORD SCREEN ERROR: ", error)
        })

        if (recordingStatus === 'started') {
            this.isRecording = true
        }
        else if (recordingStatus === 'userDeniedPermission') SimpleToast.show('Please grant permission to record screen')
        else SimpleToast.show('Recording not supported')
    } catch(e) {
        LOG.error("Recording error: ", e) // this log shows the above error
    }

thanks, will have a look soon

Hi @fizawisedv ,
the issue seems to be caused by the androidBannerStopRecordingHandler option I added recently to improve handling on Android, however, it seems that the implementation is not consistent in iOS.
As a workaround only set androidBannerStopRecordingHandler on Android
         const options = {             mic: recordMic, // defaults to true             androidBannerStopRecordingHandler: Platform.OS === 'ios' ? null : this.androidBannerStopRecordingHandler         }
I'll handle this properly on the next iteration

Fixed in 0.1.4


crush() doesn't handle dots in property names well

import { crush } from 'radash';
const test = {
  prop1: 'value1',
  'prop2.sub1': 'value2',
  prop3: {
    prop4: 'value4',
    'prop5.sub2': 'value5'
  }
};
console.log(crush(test));

outputs
{
  prop1: 'value1',
  'prop2.sub1': undefined,
  'prop3.prop4': 'value4',
  'prop3.prop5.sub2': undefined
}

Whenever there is a dot in a proptery name, the value is undefined.
Even is I'm pushing the limits by having the dots in the property names, these's still no reason for the values to be undefined

+1 for this issue.
@stefaanv I see you had a PR for this, but it got closed. Was it not working?

@Balvajs no, the repo seems to have been abandonned, unfortunately.
A shame really because it's a really good initiative.

@stefaanMLB oh no 😕
Do you know if there is some statement somewhere or more info about abandoning this repo?
2 weeks ago the author made a release, so maybe he could be back 🤞

Hi @Balvajs, this is fixed in the Radashi fork (#95).
You can use it today by installing radashi@beta (see changelog here). An official release is pending. If you have any more ideas or feedback, consider sharing it here.


Add version: JetBrains.Kotlin.Compiler version 1.3.40-eap-67

Pull request has been created with Komac v1.9.1 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116253&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


 After the animation is warmed up, it appears to become smoother.
This is a tracking issue to investigate and solve.

Now that --release builds exist and use AOT mode, there should no longer be initial compile jitter?  If you have specific traces happy to look at them!

Reopening as this is a systemic issue. Report from our mailing list: "2) First time loading shows lag, but after some time experience becomes smoother." and "Initially home screen lags for the first time only, then it gets smoother" and we recently saw it with a customer's app and their initial onboarding screens (on a Pixel, a modern phone).

This is a metabug for tracking these similar issues.
Fixing this is our top performance priority.
Assigning to @cbracken who is currently tasked with investigating this.

Any updates on this ?

I'm on 0.2.8 of the flutter.io Gallery app on the Play Store with a Nexus 4.
The performance degradation between first run and subsequent runs seems almost completely gone compared to previous versions.
There still seems to be a very, very slight difference between first and subsequent runs but it's very, very, slight to the point where I'm not even sure if it's there after looking at it numerous times.
It's still very choppy on my Nexus 7. But I think that is related to https://github.com/flutter/flutter/issues/11861 Or perhaps tablet specific as I'm not sure Google are testing Flutter on tablets.

Any update on this?
We want to launch our app soon... This performance issue is critically for us :/

I'm now actively working on this. Will give an update by the end of this week.

As comment https://github.com/flutter/flutter/issues/813#issuecomment-380273787 suggests, this issue was almost gone until we recently introduced a new regression. The newly regressed jank is not caused by the slow performance (so our dashboard doesn't catch it), but by the missing image cache. Please see this footage for more details: https://drive.google.com/open?id=1Y9tETl-Elr8_VAlapTTPqoqGHMJYmo75 . A patch https://github.com/flutter/flutter/pull/19220 is on the way.
Meanwhile, we do have some performance issues caused by Dart garbage collection. But that's not related with the "first run of animation". It's more like a random jank in the middle of some animations.

I'm sorry to hear that @christian-muertz . A video or a reproduction sample app would be very helpful 😃

@christian-muertz Thanks for the information! However, I do need a little bit more information for debugging:

Run your app in profile mode flutter run --profile (note that the default flutter run and the green play triangle in the IDE are running your app in debug mode, and the first animation is definitely going to be very slow in that mode; that slowness should go away once you run in profile or release mode).

Manipulate your app until right before you start the first slow animation

Go to the observatory web page following the link in the command line (e.g., An Observatory debugger and profiler on Nexus 5X is available at http://<IP_ADDRESS>:38585/)

Click the timeline link in the observatory web page.

Click the all button for Recorded Streams Profile on the left so all the check boxes below are checked.

Click the View Options drop down on the right and check Flow events and Highlight VSync check boxes.

Manipulate your app to start the first slow animation, and probably start a second fast animation

Go back to the web page and click Refresh button on the top right.

Click the save button on the top right, and save the current web page as html and send them to us so we can see which things are causing the jank

Finally, a sample app for us to reproduce the problem is also highly useful. If you have some more time after finishing the steps above, please send us a sample app :smile:

Thanks for your effort. Here are the files!
I will work an sample app and i will send it to you asap :D

Hmm my sample app has no jank

@christian-muertz : according to the timeline trace you sent, we believe that it's caused by font loading. Can you try to add a few empty paragraphs to your first screen with all the fonts that your app uses, and see if there's still a jank when you start the first animation? Hopefully this will also help you create a sample app to demonstrate the issue.

Ok now i broke everything 😅 I cant even start our app in profile mode now 😕
[VERBOSE-3:dart_vm.cc(423)] Error while initializing the Dart VM: Snapshot not compatible with the current VM configuration: the snapshot requires 'product no-type_checks no-asserts no-error_on_bad_type no-error_on_bad_override no-sync_async reify_generic_functions arm64' but the VM has 'release no-type_checks no-asserts no-error_on_bad_type no-error_on_bad_override no-sync_async reify_generic_functions arm64'

Ok problem is gone... Simply changed the app id and the install was successfully...

Ok removed all fonts from our app. Still having the same jank. Here are my new files!
Archiv.zip

Sorry for the late reply. We've had some merging crisis recently. By looking at your new traces, we don't find anything new. Can you please provide us the "open a cupertino route with an empty scaffold" app source code, and the jank video so we can better understand the problem?

I'll post a video soon @liyuqian. We have this same issue. Performance is also our company's number one issue with flutter.

Here I created an app from the Flutter starter app with a page transition to a blank page with an app bar. You can see that the first animation is worse, then as I go back and forward between pages it improves. Notice that the pop animation is always janky. On a Samsung Note 4 (Exynos).
I've included the observatory output and also a video of the same events.
dart-timeline-2018-7-2-617.txt
https://drive.google.com/file/d/1bxAvFuLAzX-V-c3agshNWUxNCjuEnFkA/view?usp=sharing
Here is the additional code to reproduce the app:
onPressed: () => Navigator.push(context, MaterialPageRoute(builder: (context) => PageTwo())),
        tooltip: 'Increment',
        child: new Icon(Icons.navigate_next),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

class PageTwo extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Page two'),
      ),
    );
  }
}```

@Chozzle thank you for the report and the really simple reproduction app! This is caused by the first time to compile a GL shader and we had https://github.com/flutter/flutter/issues/14028 filed to solve it.
You can see a more detailed trace by flutter run --profile --trace-skia:

@liyuqian. I tried MaterialPageRoute.debugEnableFadingRoutes = true; and it makes a big improvement to the feel of the app. I might suggest turning that on by default. Here's the video.
Video

@Chozzle : Yes, with animated opacity, the transition looks much better. We currently don't turn it on because of https://github.com/flutter/flutter/issues/13736 . We've made many progress in the last few months regarding that issue and we're now very close (90%) to our 60fps goal. There's one last step that I plan to finish before October. Let me ask @Hixie to see if we can turn it on a little earlier.

This issue should now be fixed thanks to the closing of https://github.com/flutter/flutter/issues/14028

Reopen as https://github.com/flutter/flutter/issues/14028 has been disabled for stability issues. We'll have a more stable solution soon.

Vote up. This happens to me everytime I start the Reflecty android app by tapping on its notification

@uenx I'm surprised to learn that the animation is smooth after a restart. Can you provide us the version of Reflectly app and the Flutter version being used in that Reflectly version? The only possible way that I can imagine is that Reflectly uses a Flutter version that has our persistent cache before we reverted it. And somehow each morning our persistent cache is removed somehow?

@uenx : it's probably difficult for us to get that information. Also, your animation issue might be caused by some other reasons that are unrelated with Flutter. For example, the app may initiate a costly network communication every morning that blocks the animation. Therefore, I think it's better to directly contact the reflectly app team about this issue:
<EMAIL_ADDRESS>(according to https://reflectly.app/)

This is blocking for customer solaris.

I was going to test out the fix (coming from #27660) in my app, but it seems that the changes were reverted (in #28376).
Is it possible to reopen this issue until final fix lands in master?

Yes, I'll fix the flaky tests soon and reland the PR. Meanwhile, I'll reopen this issue until the reland PR is merged.

@liyuqian What is current status with this? as owner of iPhone XS, I can feel the performance not being the best when generally scrolling or navigating around - I can feel some FPS drop.
Let me know if I can assist at testing solutions. Would be very happy to help.

If the FPS only drops when the animation first appears and it's smooth afterwards, then it's probably related to this bug. Otherwise, it's probably a different issue (so it's better to post it on another thread).
To help us better fix your performance issue, please

Give us your Flutter version: flutter doctor -v
Run your app on a physical device using profile mode: flutter run --profile
Take a video of the performance issue so we can have an intuitive understanding of what happened.
Show us a screenshot of performance overlay (https://flutter.dev/docs/testing/ui-performance#the-performance-overlay) so we know which thread is causing the issue
Open observatory and save a timeline trace of your performance issue and send it to us so we know which functions might be causing the issue (https://medium.com/flutter-io/profiling-flutter-applications-using-the-timeline-a1a434964af3).

Depending on the specific issue, we might need a trace with Skia info. You can get that trace with flutter run --profile --trace-skia.

Construct a single-file minimal Flutter app that can reproduce the issue. That way, we can test our fixes and maybe add it to our continuous performance benchmarks.

@liyuqian has a working solution (via the shader warm-up) for this which he is testing with an internal customer.  I expect we're finally close to closing this. :)  Thanks all for your patience.

Can everyone please check your issue with the tip-of-tree master Flutter? Now the shader persistent cache should be turned on by default in both Android (binary cache) and iOS (SkSL cache), and there's a default pre-warm-up that eliminates some common compilations. Hence I expect that most of these issues should have already been solved.

Unfortunately, this problem is exactly the same on the new master:
https://github.com/flutter/flutter/issues/28113

See also https://github.com/flutter/flutter/pull/27660#issuecomment-474230411

This specific bug was for us to track the performance of the gallery. It is probably best at this point if each occurrence of this gets its own bug with its own small reproducable test case, so we can track each individual cause separately.

Such an anti-climactic ending to a 3.5 year old bug. :)

I guess it's continued here?
https://github.com/flutter/flutter/issues/28113
And partly also the discussion at the end of this PR
https://github.com/flutter/flutter/pull/27660

Reopen this issue as there's still ongoing work related with this.
Meanwhile, I've modified the description of this bug to only include first-animation-jank that caused by shader compilations.
Other first-animation-jank (such as font loading) will be tracked in separate issues such as https://github.com/flutter/flutter/issues/28113

Thumbs on this. Reported 3 month ago and now the app is already on production and a new one is coming soon with the same issue on iOS.
As soon as it launches there’s jittery in the animations/scroll or overall experience for about 3-4 seconds with an iPhone X. After warming up it’s fine.
If you run a static screen before anything else for about this same time (eg. an onboarding screen without animations or scrolls), then, afterwards, it will run smoothly.
Definitely looks like some sort of Skia engine warming up but that only affects in iOS.

That’s exactly the experience we have. The issue is, that the ‚warming up‘ is also persistent when apps are switched, so a 3-4s (!) onboarding screen has to be shown for every rebuild...
...which makes the entire Flutter framework pretty useless for professional apps right now...we desperately hope for a fix.

@msw333, @miguelpruivo Have you been able to test your application behavior using the latest in the master channel to see if @liyuqian's work has resolved it?

I haven't uploaded a solution yet. But I've added some tracing as mentioned in https://github.com/flutter/flutter/issues/28113#issuecomment-480618431 . It would be nice if you can trace your performance issues with master channel and see if the font loading is the cause.

@kf6gpe I’m sitting at 1.5.3 @ dev. When was it synced with master? Anyhow, I’m looking to release an app with this issue (if not fixed until then) early next month.
Fingers crossed for a solution on this. It always look like the app is doing some heavy task in those little seconds.

@miguelcmedeiros : v1.5.3 should have the added trace events. Can you please follow https://github.com/flutter/flutter/issues/813#issuecomment-468751700 (especially item 5) to capture a trace of your issue so we can figure out how to help you?

@liyuqian I'm not sure if this is the correct place to post this, but here is a ~200 LoC single file example which exhibits the issue (as well as #28113 and #31086) on iOS but works fine on Android (with android.permission.INTERNET): https://github.com/yunyu/flutter_scroll_stutter_test/blob/master/lib/main.dart

Noticed that this also happens when the iPhone is in lower power mode. It completely jitters scroll and animations.

@yunyu : Thank you very much for your example app! I did reproduced the stutter on an iPhone X, and it didn't stutter on an iPhone 6S. I checked the trace and it's not related to the shader compilations of this issue so I'll route the discussion to https://github.com/flutter/flutter/issues/31086#issuecomment-487768379

I'm closing this bug since all new complaints here seem to be unrelated with our original issue. I created a new issue https://github.com/flutter/flutter/issues/31881 for our original task of improving flutter_gallery's shader compilation jank. Other iOS related first animation jank issues should be routed to either https://github.com/flutter/flutter/issues/28113 or https://github.com/flutter/flutter/issues/31086. If both of them don't seem to be the issue that you had, please feel free to create a new issue.

I am experiencing this problem on current production app (flutter v1.5.4). The first time my animations run janky, then it runs smooth.
Both Android and iOS.

Experiencing this specifically with hero animations. Other animations seem to run smooth. However, initial hero animation is a bit janky

Same for me, initial run of a hero animation is missing frames.

It seems to happen with all animations at start of an iOS app and/or reloading from background. It’s 2/3 seconds janky and then smooth until closed and reopen again. You can replicate it everytime in an iPhone X (at least).

Anyone find a fix? Or a workaround for this?

Found it lags for the first time on both Android and iOS initial start up

We're coming up with a long-term solution for this and that progress is tracked publicly on https://github.com/flutter/flutter/issues/32170 . Currently, we have a 16-page internal draft design doc with 9 different proposed solutions in it.
In shorter terms, we believe that there might be some specific issues that we can quickly solve just for iPhone X since the issue seems to be less bad on older iPhones. We're currently purchasing iPhone X to add to our device lab so we can debug the performance of iPhone X: https://github.com/flutter/flutter/issues/34590

#32170

This is not only an issue on the iPhone X. We have done testing on the Pixel 3 and other android devices and have found the same effect.

@michinchin : we're aware of that hence https://github.com/flutter/flutter/issues/32170 is not just for iOS, but also for Android, Fuchsia, Desktop, and probably all platforms that Flutter directly compiles into binaries.
iOS currently just has additional issues. For example, we can store binary persistent shader cache on Android, but we can only store GLSL persistent shader cache on iOS. That requires iOS to do one more compilation (compared to Android). For that, we're also enabling Metal backend for iOS which should help this specific issue.

@liyuqian thank you for the updates. Currently (and because I’m an iPhone X user) I immediately noticed it and I must say that despite of being an issue already tracked, it bothers me somehow to have a production app that launches on a scroll screen where it always feel janky as the user starts it on iPhone for the first 2 seconds.
Oddly and in my experience, it seems to affect only the iOS. Android devices were I’ve tested it, seem to be good and smooth right from the launch point.

@miguelcmedeiros : that's exactly the difference between binary persistent shader cache and GLSL persistent shader cache. On Android, there's 0 work if you close the app and re-open the app. On iOS, re-opening the app triggers another GLSL to binary compilation.

@liyuqian got it. Do you have an idea of when a fix may be available? Have you already found a solution for it or still drafting for?
PS: You've tagged the wrong person. 😄

Oops. We haven't picked which solution to pursue. Will update https://github.com/flutter/flutter/issues/32170 with some proper time estimation once we made the decision.

@liyuqian Any news on this?
I'm experiencing jaggy animations every first time i do:
Navigator.push(MaterialPageRoute(fullscreenDialog: true, builder: (context) => SomePage()));

Second time i open a page, it' much smoother.
I'm on an iPhone 7.
flutter --version
Flutter 1.15.18 • channel dev • https://github.com/flutter/flutter.git
Framework • revision<PHONE_NUMBER> (31 hours ago) • 2020-03-06 10:24:23 -0800
Engine • revision 5aff311948
Tools • Dart 2.8.0 (build 2.8.0-dev.12.0 9983424a3c)

@erf : we're actively working on SkSL shader warm-up. Some preliminary results on iOS should be available in 1-2 months.

For progress, please check https://github.com/flutter/flutter/issues/32170 . We'll be referencing it when we're making smaller fixes.

@liyuqian i feel the performance has been improved in the latest dev version 1.16.3 ..?

@erf : it's nice to hear that! We're always working hard to improve the performance. While https://github.com/flutter/flutter/issues/32170 isn't fixed yet, we've made at least 13 other quantified improvements between Sep 2019 and Dec 2019.


LocalSources inaccessible because they are internal

small issue, your example code seems not to work:
        let config = Filestack.Config.builder
            .with(appUrlScheme: "your-scheme")
            .with(videoQuality: .typeHigh)
            .with(imageUrlExportPreset: .current)
            .with(maximumSelectionLimit: 10)
            .withEditorEnabled()
            .with(availableCloudSources: [.dropbox, .googleDrive, .googlePhotos, .customSource])
            .with(availableLocalSources: [.camera])
            .build()

'camera' is inaccessible due to 'internal' protection level

this is also the case in the DemoApp when trying to not use LocalSources.All()

I ran into the same issue. Please advise as this probably makes the product unusable.

+1

Fixed in develop branch.