PPO training error with rewards other than 0

Getting the error below while training a PPO. The point that I have discovered during debugging is that the error comes when I pass a reward other than 0. with 0 reward, everything works fine.
I make the agent using the following code
class PPOTrainer:
    def __init__(self, state_dim, action_dim):
        with open("rl_models/data.json") as fp:
            agent = json.load(fp=fp)
        network=[dict(type='dense', size=64),dict(type='dense', size=64)]
        self.agent = Agent.from_spec(spec=agent, kwargs=dict(states=OpenAIGym.state_from_space(state_dim), actions=OpenAIGym.action_from_space(action_dim), network=network))

    def action(self, state):
        return self.agent.act(state, Independent=True)

    def update(self, reward, terminal):
        self.agent.observe(reward, terminal)

where data.json is the ppo.json file that can be found in the example folder.
update:  I have tried other agents as well. Apparently having problem with all.
InvalidArgumentError                      Traceback (most recent call last)
/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py in _do_call(self, fn, *args)
   1322     try:
-> 1323       return fn(*args)
   1324     except errors.OpError as e:

/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py in _run_fn(session, feed_dict, fetch_list, target_list, options, run_metadata)
   1301                                    feed_dict, fetch_list, target_list,
-> 1302                                    status, run_metadata)
   1303

/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/errors_impl.py in __exit__(self, type_arg, value_arg, traceback_arg)
    472             compat.as_text(c_api.TF_Message(self.status.status)),
--> 473             c_api.TF_GetCode(self.status.status))
    474     # Delete the underlying status object from memory otherwise it stays alive

InvalidArgumentError: Must have updates.shape = indices.shape + params.shape[1:], got updates.shape [2,4], indices.shape [1], params.shape [5000,4]
	 [[Node: ppo/observe-timestep/store/ScatterUpdate = ScatterUpdate[T=DT_FLOAT, Tindices=DT_INT32, _class=["loc:@ppo/initialize/latest/initialize/state-state"], use_locking=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](ppo/initialize/latest/initialize/state-state, ppo/observe-timestep/store/mod/_367, ppo/strided_slice, ^ppo/observe-timestep/store/AssignSub/_371)]]
	 [[Node: ppo/observe-timestep/cond/optimization/multi-step/step/subsampling-step/step/adam/step/add_7/_512 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_2223_ppo/observe-timestep/cond/optimization/multi-step/step/subsampling-step/step/adam/step/add_7", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]()]]

During handling of the above exception, another exception occurred:

InvalidArgumentError                      Traceback (most recent call last)
~/environments/vip_protection/main_code/vip_protection_rl.py in <module>()
    195 if __name__ == '__main__':
    196     arglist = parse_args()
--> 197     train(arglist)

~/environments/vip_protection/main_code/vip_protection_rl.py in train(arglist)
    143         for i, agent in enumerate(trainers):
    144             print(rew_n[i])
--> 145             agent.update(rew_n[i], terminal)
    146
    147         if done or terminal:

~/environments/vip_protection/main_code/rl_models/agents.py in update(self, reward, terminal)
     13
     14     def update(self, reward, terminal):
---> 15         self.agent.observe(reward, terminal)
     16 #
     17 # class PPOTrainer(PPOAgent):

~/environments/vip_protection/tensorforce/tensorforce/agents/agent.py in observe(self, terminal, reward)
    224                 self.episode = self.model.observe(
    225                     terminal=self.observe_terminal,
--> 226                     reward=self.observe_reward
    227                 )
    228                 self.observe_terminal = list()

~/environments/vip_protection/tensorforce/tensorforce/models/model.py in observe(self, terminal, reward)
   1240
   1241         self.is_observe = True
-> 1242         episode = self.monitored_session.run(fetches=fetches, feed_dict=feed_dict)
   1243         self.is_observe = False
   1244

/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py in run(self, fetches, feed_dict, options, run_metadata)
    519                           feed_dict=feed_dict,
    520                           options=options,
--> 521                           run_metadata=run_metadata)
    522
    523   def should_stop(self):

/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py in run(self, *args, **kwargs)
    965         raise six.reraise(*original_exc_info)
    966       else:
--> 967         raise six.reraise(*original_exc_info)
    968
    969

/usr/lib/python3/dist-packages/six.py in reraise(tp, value, tb)
    684         if value.__traceback__ is not tb:
    685             raise value.with_traceback(tb)
--> 686         raise value
    687
    688 else:

/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py in run(self, *args, **kwargs)
    950   def run(self, *args, **kwargs):
    951     try:
--> 952       return self._sess.run(*args, **kwargs)
    953     except _PREEMPTION_ERRORS:
    954       raise

/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py in run(self, fetches, feed_dict, options, run_metadata)
   1022                                   feed_dict=feed_dict,
   1023                                   options=options,
-> 1024                                   run_metadata=run_metadata)
   1025
   1026     for hook in self._hooks:

/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/monitored_session.py in run(self, *args, **kwargs)
    825
    826   def run(self, *args, **kwargs):
--> 827     return self._sess.run(*args, **kwargs)
    828
    829

/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py in run(self, fetches, feed_dict, options, run_metadata)
    887     try:
    888       result = self._run(None, fetches, feed_dict, options_ptr,
--> 889                          run_metadata_ptr)
    890       if run_metadata:
    891         proto_data = tf_session.TF_GetBuffer(run_metadata_ptr)

/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py in _run(self, handle, fetches, feed_dict, options, run_metadata)
   1118     if final_fetches or final_targets or (handle and feed_dict_tensor):
   1119       results = self._do_run(handle, final_targets, final_fetches,
-> 1120                              feed_dict_tensor, options, run_metadata)
   1121     else:
   1122       results = []

/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py in _do_run(self, handle, target_list, fetch_list, feed_dict, options, run_metadata)
   1315     if handle is None:
   1316       return self._do_call(_run_fn, self._session, feeds, fetches, targets,
-> 1317                            options, run_metadata)
   1318     else:
   1319       return self._do_call(_prun_fn, self._session, handle, feeds, fetches)

/usr/local/lib/python3.5/dist-packages/tensorflow/python/client/session.py in _do_call(self, fn, *args)
   1334         except KeyError:
   1335           pass
-> 1336       raise type(e)(node_def, op, message)
   1337
   1338   def _extend_graph(self):

InvalidArgumentError: Must have updates.shape = indices.shape + params.shape[1:], got updates.shape [2,4], indices.shape [1], params.shape [5000,4]
	 [[Node: ppo/observe-timestep/store/ScatterUpdate = ScatterUpdate[T=DT_FLOAT, Tindices=DT_INT32, _class=["loc:@ppo/initialize/latest/initialize/state-state"], use_locking=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](ppo/initialize/latest/initialize/state-state, ppo/observe-timestep/store/mod/_367, ppo/strided_slice, ^ppo/observe-timestep/store/AssignSub/_371)]]
	 [[Node: ppo/observe-timestep/cond/optimization/multi-step/step/subsampling-step/step/adam/step/add_7/_512 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_2223_ppo/observe-timestep/cond/optimization/multi-step/step/subsampling-step/step/adam/step/add_7", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]()]]

Caused by op 'ppo/observe-timestep/store/ScatterUpdate', defined at:
  File "/usr/local/bin/ipython", line 11, in <module>
    sys.exit(start_ipython())
  File "/usr/local/lib/python3.5/dist-packages/IPython/__init__.py", line 125, in start_ipython
    return launch_new_instance(argv=argv, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/traitlets/config/application.py", line 658, in launch_instance
    app.start()
  File "/usr/local/lib/python3.5/dist-packages/IPython/terminal/ipapp.py", line 356, in start
    self.shell.mainloop()
  File "/usr/local/lib/python3.5/dist-packages/IPython/terminal/interactiveshell.py", line 480, in mainloop
    self.interact()
  File "/usr/local/lib/python3.5/dist-packages/IPython/terminal/interactiveshell.py", line 471, in interact
    self.run_cell(code, store_history=True)
  File "/usr/local/lib/python3.5/dist-packages/IPython/core/interactiveshell.py", line 2728, in run_cell
    interactivity=interactivity, compiler=compiler, result=result)
  File "/usr/local/lib/python3.5/dist-packages/IPython/core/interactiveshell.py", line 2856, in run_ast_nodes
    if self.run_code(code, result):
  File "/usr/local/lib/python3.5/dist-packages/IPython/core/interactiveshell.py", line 2910, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-1-2d2d706ed1b7>", line 1, in <module>
    get_ipython().run_line_magic('run', 'vip_protection_rl')
  File "/usr/local/lib/python3.5/dist-packages/IPython/core/interactiveshell.py", line 2095, in run_line_magic
    result = fn(*args,**kwargs)
  File "<decorator-gen-60>", line 2, in run
  File "/usr/local/lib/python3.5/dist-packages/IPython/core/magic.py", line 187, in <lambda>
    call = lambda f, *a, **k: f(*a, **k)
  File "/usr/local/lib/python3.5/dist-packages/IPython/core/magics/execution.py", line 775, in run
    run()
  File "/usr/local/lib/python3.5/dist-packages/IPython/core/magics/execution.py", line 761, in run
    exit_ignore=exit_ignore)
  File "/usr/local/lib/python3.5/dist-packages/IPython/core/interactiveshell.py", line 2491, in safe_execfile
    self.compile if shell_futures else None)
  File "/usr/local/lib/python3.5/dist-packages/IPython/utils/py3compat.py", line 186, in execfile
    exec(compiler(f.read(), fname, 'exec'), glob, loc)
  File "/home/hassam/environments/vip_protection/main_code/vip_protection_rl.py", line 197, in <module>
    train(arglist)
  File "/home/hassam/environments/vip_protection/main_code/vip_protection_rl.py", line 110, in train
    trainers = get_trainers_ppo(obs_shape_n, action_shape_n, env.n)
  File "/home/hassam/environments/vip_protection/main_code/vip_protection_rl.py", line 73, in get_trainers_ppo
    agent = PPOTrainer(observation_space_dimension[i], action_space_dimension[i])
  File "/home/hassam/environments/vip_protection/main_code/rl_models/agents.py", line 9, in __init__
    self.agent = Agent.from_spec(spec=agent, kwargs=dict(states=OpenAIGym.state_from_space(state_dim), actions=OpenAIGym.action_from_space(action_dim), network=network))
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/agents/agent.py", line 291, in from_spec
    kwargs=kwargs
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/util.py", line 159, in get_object
    return obj(*args, **kwargs)
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/agents/ppo_agent.py", line 151, in __init__
    entropy_regularization=entropy_regularization
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/agents/learning_agent.py", line 149, in __init__
    batching_capacity=batching_capacity
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/agents/agent.py", line 79, in __init__
    self.model = self.initialize_model()
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/agents/ppo_agent.py", line 179, in initialize_model
    likelihood_ratio_clipping=self.likelihood_ratio_clipping
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/pg_prob_ratio_model.py", line 88, in __init__
    gae_lambda=gae_lambda
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/pg_model.py", line 95, in __init__
    requires_deterministic=False
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/distribution_model.py", line 86, in __init__
    discount=discount
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/memory_model.py", line 106, in __init__
    reward_preprocessing=reward_preprocessing
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/model.py", line 200, in __init__
    self.setup()
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/model.py", line 344, in setup
    independent=independent
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/memory_model.py", line 583, in create_operations
    independent=independent
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/model.py", line 1006, in create_operations
    self.create_observe_operations(reward=reward, terminal=terminal)
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/model.py", line 983, in create_observe_operations
    reward=reward
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/template.py", line 278, in __call__
    result = self._call_func(args, kwargs, check_for_new_variables=False)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/template.py", line 217, in _call_func
    result = self._func(*args, **kwargs)
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/models/memory_model.py", line 481, in tf_observe_timestep
    reward=reward
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/template.py", line 261, in __call__
    return self._call_func(args, kwargs, check_for_new_variables=True)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/template.py", line 217, in _call_func
    result = self._func(*args, **kwargs)
  File "/home/hassam/environments/vip_protection/tensorforce/tensorforce/core/memories/queue.py", line 169, in tf_store
    updates=state
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/gen_state_ops.py", line 786, in scatter_update
    use_locking=use_locking, 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/framework/ops.py", line 2956, in create_op
    op_def=op_def)
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/ops.py", line 1470, in __init__
    self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access

InvalidArgumentError (see above for traceback): Must have updates.shape = indices.shape + params.shape[1:], got updates.shape [2,4], indices.shape [1], params.shape [5000,4]
	 [[Node: ppo/observe-timestep/store/ScatterUpdate = ScatterUpdate[T=DT_FLOAT, Tindices=DT_INT32, _class=["loc:@ppo/initialize/latest/initialize/state-state"], use_locking=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](ppo/initialize/latest/initialize/state-state, ppo/observe-timestep/store/mod/_367, ppo/strided_slice, ^ppo/observe-timestep/store/AssignSub/_371)]]
	 [[Node: ppo/observe-timestep/cond/optimization/multi-step/step/subsampling-step/step/adam/step/add_7/_512 = _Recv[client_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_2223_ppo/observe-timestep/cond/optimization/multi-step/step/subsampling-step/step/adam/step/add_7", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]()]]

Hello,
please make sure to use the independent act flag on the latest master, otherwise every act has to be followed by exactly one observe.

Hi,  i have set that flag true. I am still getting that error.
On Mar 11, 2018 4:05 AM, "michaelschaarschmidt"<EMAIL_ADDRESS>wrote:

Hello,
please make sure to use the independent act flag on the latest master,
otherwise every act has to be followed by exactly one observe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/reinforceio/tensorforce/issues/365#issuecomment-372096998,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOSH8VTyz0IYmN-ihFTCe43IHipjgsXxks5tdNqtgaJpZM4SlgK3
.

Secondly wouldn't it stop it from updating the agent?  Thank you
On Mar 11, 2018 8:07 AM, "Hassam Sheikh"<EMAIL_ADDRESS>wrote:
Hi,  i have set that flag true. I am still getting that error.
On Mar 11, 2018 4:05 AM, "michaelschaarschmidt"<EMAIL_ADDRESS>wrote:

Hello,
please make sure to use the independent act flag on the latest master,
otherwise every act has to be followed by exactly one observe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/reinforceio/tensorforce/issues/365#issuecomment-372096998,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOSH8VTyz0IYmN-ihFTCe43IHipjgsXxks5tdNqtgaJpZM4SlgK3
.

see https://github.com/reinforceio/tensorforce/issues/335

Hi, thank you. I have already seen this issue. Followed everything that is suggested in the issue. There is one thing I found that is super weird is that everything works perfectly when I manually pass 0 reward. As soon as I pass any different number, it gives the same error. Let me try some other things. I will update this

Saw #335 Followed it, still having the problem

You would have to provide the full code you are using for any useful help..

posted on gitter


Update OSM to v0.5.0

Signed-off-by: Waleed Malik<EMAIL_ADDRESS>
What type of PR is this?

/kind feature
What this PR does / why we need it:
This PR intends to update OSM and machine-controller to the latest versions i.e. v.0.5.0 and v1.52.0, respectively.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Update OSM to v0.5.0
Update machine-controller to v1.52.0

@ahmedwaleedmalik can you please add new scenario to generated tests with a config that has OSM enabled.

/kind feature

/test pull-kubeone-e2e-aws-defaults-install-operating-system-manager-v1.22.11 pull-kubeone-e2e-aws-defaults-install-operating-system-manager-v1.23.8  pull-kubeone-e2e-aws-defaults-install-operating-system-manager-v1.24.1

/test pull-kubeone-e2e-aws-defaults-install-operating-system-manager-v1.22.11 pull-kubeone-e2e-aws-defaults-install-operating-system-manager-v1.23.8 pull-kubeone-e2e-aws-defaults-install-operating-system-manager-v1.24.1

/approve
/lgtm
:heart:


feat: moving generated code away from static vars

Description
Removed static aliases in the generated button CSS.
How and where has this been tested?

How this was tested:
Browser(s) and OS(s) this was tested with:

Screenshots

To-do list

[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

[x] This pull request is ready to merge.

One quick thing: it's missing the purple accent color. Suggest referencing the following aliases to bring it back:
var(--spectrum-semantic-cta-background-color-default)
var(--spectrum-semantic-cta-background-color-down)
var(--spectrum-semantic-cta-background-color-hover)
var(--spectrum-semantic-cta-background-color-key-focus)


Generate sql parameters index  error

sql:
select * from foo where a = :a AND b in :b AND c = :c;

generate code:
    var $arrayStartIndex = 2;
    final expandedcategories = $expandVar($arrayStartIndex, categories.length);
    $arrayStartIndex += categories.length;
    customSelect('select * from foo where a = ?1 AND b in ($expandedcategories) AND c = ?2;') // bug!!!!
....

If the "in parameter" order is not the last, the parameters index will be repeated.

Good catch, thank you!
Initially my idea was to stop generating explictly indexed variables with the new code generation option, but that wouldn't work as the new generator re-orders variables in some cases.
Instead, I changed the generator to instead use explicitly-indexed variables everywhere, putting regular variables first. With 6bcb0070b0f5116c0bca966b89c1dad7fedffe0d, your query should be generated as
    var $arrayStartIndex = 3;
    final expandedcategories = $expandVar($arrayStartIndex, categories.length);
    $arrayStartIndex += categories.length;
    customSelect('select * from foo where a = ?1 AND b in ($expandedcategories) AND c = ?2;')


Remove spinner and use angular-loading-bar

Using a angular-loading-bar to handle automatic display of progress when making http calls, this way there is no need to wrap the call in the spinner function
Also changed the spinner image to use an svg so it looks better on hi-res screens.

Coverage decreased (-0.2%) to 58.727% when pulling 37cf35ced6ea479b3030e87d528f6917bd8b802b on ynurmahomed:loading-bar into d3e4fde61da2340d8289f4b527984f4ff2a7d005 on esaude:master.


[ja] Translate docs/concepts/storage/storage-classes/ into Japanese

close #38147

/assign @inductor

/lgtm


Doesn't work on iOS 12.5.5 (more details)

My iPhone is iOS 12.5.5 | Jailbreak with Chimera 1.6.4
Install this tweak using Sileo version 2.2.6
Install Log:

Video Test:
https://user-images.githubusercontent.com/87893636/145152715-263cba58-aae6-43a9-9971-1d4c6d431bc4.mp4
Conflict ???
I see in this post on Reddit say Conflict between YouPiP and Youtube Reborn
Note: I tried uninstalling Youtube Reborn, then reinstalling this tweak but still DOESN'T work

@NguyenASang Can you try reinstalling ForceInPicture?

@NguyenASang So, tapping on PiP button doesn't trigger PiP?

@PoomSmart It seems like it depends on the YouTube app version. I tried downgrading and gathered some info which might be useful for your debugging process.
My config is:
iPad Mini 2 (iOS 12.5.1, Chimera 1.6.4)
YouPiP 1.6.0, ForceInPicture 1.0
15.36.2 is the latest working version.
15.37.3 - 15.40.4 (tapping the icon does nothing)
15.41.2 - 15.49.6 (YouPiP prevents playing videos at all, No stream, tap to retry error)
16.05.7 and above (tapping the icon does nothing)

15.36.2 is the latest working version.
I will test it.

For the record, I have PiP working just fine on iPhone 5s running iOS 12.5.4, YouTube 16.46.5, jailbroken using unc0ver 6.2.0

@NguyenASang Can you try reinstalling ForceInPicture?

Still not working

For the record, I have PiP working just fine on iPhone 5s running iOS 12.5.4, YouTube 16.46.5, jailbroken using unc0ver 6.2.0 - YouPiP 1.6.0 and ForceInPicture 1.0

@PoomSmart Can you try iOS 12.5.5 ? Maybe this error happened with this iOS version

More: now I jailbreak with unc0ver. Not Chimera anymore

Ok guys try version 1.6.1 from my repo @NguyenASang @h0rn3z0r

Ok guys try version 1.6.1 from my repo @NguyenASang @h0rn3z0r

Works on my side with the latest YouTube

@PoomSmart I will test it now :))

@PoomSmart IT'S PERFECT THANKS YOU SO MUCH


Support for disktype for nodegroup

While creating nodegroup we have the option to specify disksize. We also need the option to specify disktype (gp2, gp3 etc..)
<EMAIL_ADDRESS>

I think you need to go with a custom launch template to configure the disk type, it's the same situation with the AWS CLI: create-nodegroup only allows configuring the disk size.
To configure the disk type, you need to create a launch template and attach it to the nodegroup.

@zetamorph is correct - you need to use custom Launch Template


Assets URIs map should use Uri class

The model class Assets has the field uris: Option[Map[String, String]]. It would be more correct for it to have type Option[Map[String, Uri]].

No activity in a while. Closing.


crashed atom

I went to install this and when it was installing it crashed atom and now I can only open in safe mode

Did you find  a solution?
I'm currently facing the same problem...

No I never did unfortunately. I ended up switching to vs code. Sorry you're
having the same problem!
On Tue, Dec 18, 2018, 10:13 AM R.A.Kanzler<EMAIL_ADDRESS>wrote:

Did you find a solution?
I'm currently facing the same problem...
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/snowkit/atom-haxe/issues/73#issuecomment-448254040,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AimnNP_EVhrNaQojuAXb_oy2DG2nh3mGks5u6QYegaJpZM4W71ei
.


AttributeError: 'NoneType' object has no attribute 'tobytes'

when i run using someguy and somegirl it works but when i try to input another audio , here's the problem :
AttributeError: 'NoneType' object has no attribute 'tobytes'
size of the audio is just 4,4 mb and 4:39min audio. does this code only work for 7second audio ?
anyone can help me ? i'm stuck already

C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0>runtime\python.exe infer-web.py --pycmd runtime\python.exe --port 7897
Found GPU NVIDIA GeForce MX130
Set fp16_run to false in 32k.json
Set fp16_run to false in 40k.json
Set fp16_run to false in 48k.json
Use Language: en_US
Running on local URL:  http://<IP_ADDRESS>:7897
To create a public link, set share=True in launch().
loading weights/Tanjiro.pth
gin_channels: 256 self.spk_embed_dim: 109

Traceback (most recent call last):
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\infer-web.py", line 316, in vc_single
audio_opt = vc.pipeline(
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\vc_infer_pipeline.py", line 542, in pipeline
pitch, pitchf = self.get_f0(
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\vc_infer_pipeline.py", line 303, in get_f0
f0 = self.get_f0_official_crepe_computation(x, f0_min, f0_max)
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\vc_infer_pipeline.py", line 142, in get_f0_official_crepe_computation
f0, pd = torchcrepe.predict(
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\torchcrepe\core.py", line 120, in predict
probabilities = infer(frames, model)
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\torchcrepe\core.py", line 564, in infer
return infer.model(frames, embed=embed)
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\torch\nn\modules\module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\torchcrepe\model.py", line 96, in forward
x = self.embed(x)
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\torchcrepe\model.py", line 120, in embed
x = self.layer(x, self.conv1, self.conv1_BN, (0, 0, 254, 254))
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\torchcrepe\model.py", line 132, in layer
x = F.relu(x)
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\torch\nn\functional.py", line 1457, in relu
result = torch.relu(input)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 512.00 MiB (GPU 0; 2.00 GiB total capacity; 851.32 MiB already allocated; 62.24 MiB free; 1.02 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
Traceback (most recent call last):
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\routes.py", line 437, in run_predict
output = await app.get_blocks().process_api(
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\blocks.py", line 1349, in process_api
data = self.postprocess_data(fn_index, result["prediction"], state)
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\blocks.py", line 1283, in postprocess_data
prediction_value = block.postprocess(prediction_value)
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\components.py", line 2586, in postprocess
file_path = self.audio_to_temp_file(
File "C:\RVC\Mangio-RVC-v23.7.0_INFER_TRAIN\Mangio-RVC-v23.7.0\runtime\lib\site-packages\gradio\components.py", line 360, in audio_to_temp_file
temp_dir = Path(dir) / self.hash_bytes(data.tobytes())
AttributeError: 'NoneType' object has no attribute 'tobytes'

out of memory error because your gpu only has 2 gb vram
if u want process longer video, u need to buy better gpu

I was able to get around this by leaving the "Path to the feature index file" input blank. And instead placing your audio file within the "\Mangio-RVC-v23.7.0\audios" directory, you can then select the audio file from the dropdown menu (see below)

That worked for me, let me know how it goes @Kevin201110219


reflect: StructOf isn't behaving properly with methods on embedded structs

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.8.3
What operating system and processor architecture are you using (go env)?
Visible on Go Playground and on Mac OS
$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/mbh475/go_projects"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/xf/5x4cmml91xn880l6v0c9l1dxnrqmjk/T/go-build686415997=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
What did you do?
I wrote a small example to show the difference between delegating a method call to an embedded struct normally and doing so via reflection using reflect.StructOf.  I expected that embedding via reflection would not work, and would panic at runtime when I tried to cast the Interface(). What happened instead is that I was able to not only cast successfully, I could invoke the method and it would use data in a different embedded struct field.
Links:
Two links, one with two embedded structs, where it's exposing the method from the second struct and using the value from the first struct:
https://play.golang.org/p/TWApy-JibG
The second has one embedded struct, one embedded pointer to a struct, again the method from the second is being used with the data from the first, but this time the data is a string not an int:
https://play.golang.org/p/Dq9Y8DAkvA
What did you expect to see?
In both cases, I expected a panic when I tried to cast the generated struct to the interface that's implemented by the embedded struct.
What did you see instead?
Not only did the cast work, but invoking the method worked as well. In the second case, it's multiplying a string by 2 and producing a nonsense result.

Out of curiosity, have you tried Go 1.9beta2?

Actually, I just tested.
In both examples, with Go 1.9beta2 I get:
panic: reflect.StructOf: field 0 has no name

goroutine 1 [running]:
reflect.StructOf(0xc420041e38, 0x3, 0x3, 0x0, 0x0)
        /home/bradfitz/go/src/reflect/type.go:2401 +0x3404
main.main()
        /home/bradfitz/reflect1.go:50 +0x562
exit status 2

So maybe this is fixed. I haven't read the repro code in detail yet.

@jonbodner, is Go 1.9's behavior what you expect?

From https://github.com/golang/go/issues/20632#issuecomment-307530427:

The current situation is that it sometimes works and sometimes doesn't. Try adding another field to your StructOf call, such that the interface is not the first field. Horrible things will happen.

I was a little curious about where the value for f.A was coming from (in the bar2impl.Double call), after playing around with the order of the fields in the StructOf call, it appears that it's coming from the first field of the first element (i.e. Baz.D in your examples). What's even weirder is that attempting to print either f or f.B in the call results in a panic, but f.A can be printed just fine.

@bradfitz was able to repro this on 1.9beta2 after adding the name fields.

@kshvmdn, thanks.
Closing as a dup of #15924.  I'll drop a reference to this bug there.

I'm going to reopen this, because I think this is a special case that is worse than #15924.  #15924 is about the fact that we don't generate methods that we should.  The problem here is that we are keeping some methods, but those methods work incorrectly.

/cc @crawshaw @mdempsky @josharian


Provide a webpack 2 example

Hi.
I have been using universal-webpack for a while and recently became curious in migrating my project to webpack 2. However, even though in one of the closed issues you mention that universal-webpack should be working with webpack 2 - it doesn't, since passing one raw configuration through webpack 2 works and trying to wrap it in the client_configuration - throws an error "TypeError: undefined is not iterable!".
I couldn't find a source of that error in the configuration though, but I think that it would be really useful if you could try to migrate this project to webpack 2.
Thanks.

Webpack 2 works for me with this project.
npm install<EMAIL_ADDRESS>npm install<EMAIL_ADDRESS>
// Then remove OccurrenceOrderPlugin

It might work with the beta.21, but it will fail at the latest and above. Are you going to do anything about it?

I will make it work with Webpack 2 once it is released and stabilised (along with the corresponding 3rd party plugins).
Until that you can fork this repo and create a pull request called "Webpack 2" in which you're gonna keep the project working and up-to-date with the latest beta version of Webpack 2.

I can confirm this. I don't think that the webpack beta version matters, but if you migrated to the new config syntax of webpack2 the loaders iterations will fail.

Ok then, make up a sample repo, tell me the command and the error message,
and i'll have a look this evening
On Thursday, November 3, 2016, Aris<EMAIL_ADDRESS>wrote:

I can confirm this. I don't think that the webpack beta version matters,
but if you migrated to the new config syntax of webpack2 the loaders
iterations will fail.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/halt-hammerzeit/universal-webpack/issues/38#issuecomment-258123683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdH783lSbjsQV6C9U14Qm2Xb-T1lcJPks5q6cw3gaJpZM4KkqEi
.

Since none of you provided me with a sample project reproducing the issues I made it myself and added the instructions on running webpack-react-redux-isomorphic-render-example with Webpack 2 to that repo.
https://github.com/halt-hammerzeit/webpack-react-redux-isomorphic-render-example/commit/952a9c770db971343339c75a5397fb486c14d59c
universal-webpack has no dependency onWebpack version and worked (and works) with any Webpack version.
This repo is no place for this issue.

Wow, this might be very useful, will definitely check it out. I could post a repo, but I agree with you on the point of waiting for the stable release of the webpack 2, since the config might change multiple times by that time.
However, thank you anyway.

I'm personally not a fan of going with the latest hot stuff since it's almost always means finding bugs and googling the hell out of the internet.

I agree, and to be honest, while trying to state the main features in webpack 2 and googling to make sure I'm correct I have found out that those things can be done with wepback 1 as well, but their support will not be native and might have some troubles to get it working. Nevertheless I would probably stick with your opinion on this one.

Webpack 2 introduces tree shaking.
Which is gonna eliminate unused code and reduce bundle size drastically.
Well, at least that's how everyone thought.
Except it still doesn't work and that's the reason it says BETA.
https://github.com/webpack/webpack/issues/2867

Interesting. Then the effort of fixing the errors during migration to the new version, since I've done everything like in your example and some more things, but still got problems with resolving paths.

Post your errors

Sure, but gonna do it tomorrow

Back here again. My error is not anyhow related to this project, it's a webpack's issue, however, it appears because the new webpack disallows to add an empty string into resolve extensions, and it leads to a fail in the processing of some of the node_modules. But, as I said, since it's not related to this project, I leave it here only because you asked

FYI the newest 0.2.0 release now only supports Webpack 2.
It works for my project.
If you find any bugs in the new version then create an issue.


displayName is not set for the component of "@yamada-ui/card"

script: component-display-name
package: @yamada-ui/card
Description
The component "@yamada-ui/card" uses forwardRef, but displayName is not set, making it difficult to see the display in React Developer Tools.
Additional Information
Set displayName and __ui__ after each component definition.
For example, like this.
const Component = forwardRef((props, ref) => {
  ...
})

Component.displayName = 'Component'
Component.__ui__ = 'Component'

Replace 'component' with the name of the component. For example, for a Select component, it is 'Select'.
__ui__ is an identifier required for internal use by Yamada UI. Be sure to set it together with displayName.

I'll try it.


use online service to handle translation

right now the translations get more complicated now, so it would be nice that we have a service to handle this part and ask volunteers to translate if there is any change.

registered an account on getLocalization now !

sad it seems there is no free account any more. need to find other alternatives

Noticed that getLocalization may have no community account anymore (sad). If anyone has any idea about this, feel free to comment below  !

Maybe you could use Transifex (https://www.transifex.com/) instead?

@seppi91 I didn't find free plan for open source project from the website. Any shortcut ?

ohoho it's on the bottom of plans !! Didn't notice that !!! I just registered an account and will share the news with all contributors here later after I setup all craps :P

Thanks @seppi91 ++++ Hope you can join the translation as well !! See you soon :)

I'll close this issue first and ask for help in another issue ! Thanks all !!
Help to translate Kaku at https://www.transifex.com/kaku/kaku/ !

Something is wrong with your provided hyperlink:
Transifex shows an error:
"Looks like you followed a bad link. If you think it's our fault, please let us know."

@seppi91 sorry, i forgot to change to old one , please try this https://www.transifex.com/kaku/desktop-app/ !

For anyone who is interested in translation, please check the new issue here - https://github.com/EragonJ/Kaku/issues/377


Display globals on the home page

Fixes #69.
You can see end-result here: https://kean-org.github.io/docs/nuke-docs-beta/9.0.0/

Thanks for fixing this, @kean! Really glad to be seeing globals in the HTML output now.


Truncate M values in Trading History

Description
We should truncate values over >1.000.000 to 1.00M in the trading history so that the trading history is readable.
Related issue
closes #2357
Motivation and Context

How Has This Been Tested?

Open the futures page and check the truncated value of trade history table

Screenshots (if appropriate):

@LeifuChen the spec mentions 1M and above but the screenshots shows also thousands being truncated, do we need to make it only truncate 1M and above?

Good catch, I think we should only truncate 1M and upwards in the Trades History table, truncating thousands does not look very useful:


serviceName is not correctly set

Describe what happened
I am setting serviceName in build.gradle but not seeing the service name I set used in the datadog-ci command.
build.gradle:
datadog {
  serviceName: "my-service"
}

$ /Users/***/Desktop/Engineering/packages/native/node_modules/.bin/datadog-ci react-native upload --platform android --service com.***.android.development --bundle /Users/***/Desktop/Engineering/***/packages/native/android/app/build/generated/assets/react/development/release/index.android.bundle --sourcemap /Users/***/Desktop/Engineering/packages/native/android/app/build/generated/sourcemaps/react/development/release/index.android.bundle.map --release-version *** --build-version ***
Starting upload.
Upload of /Users/***/Desktop/Engineering/packages/native/android/app/build/generated/sourcemaps/react/development/release/index.android.bundle.map for bundle /Users/***/Desktop/Engineering/packages/native/android/app/build/generated/assets/react/development/release/index.android.bundle on platform android with project path /Users/allenkinzalow/Desktop/Engineering/packages/native
version: *** build: *** service: com.***.android.development

Steps to reproduce the issue:

Set serviceName under datadog configuration in build.gradle.
Run a release variant build
Observe --service not set to serviceName.

Describe what you expected:
Additional context

Android Build Tools version: 31
Gradle version: 7.0.4
OS version: MacOS 13.2
Plugin version: 1.7.0

Hello @akinzalowevidation! Source maps upload for our React Native SDK is not using dd-sdk-android-gradle-plugin, so anything you have in the datadog extension has no effect. Please open this issue in the React Native SDK repository issue tracker.


Hello What is this wired error for me in response

Summary of problem or feature request
DataTables warning: table id=mytable - Invalid JSON response. For more information about this error, please see http://datatables.net/tn/1

Look at response start with letter W
w{"draw":1,"recordsTotal":500,"recordsFiltered":500,"data":[{"id":"500","name_en":"United States Minor Outlying Islands","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"499","name_en":"Nepal","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"498","name_en":"Greece","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"497","name_en":"Burundi","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"496","name_en":"Lesotho","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"495","name_en":"Angola","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"494","name_en":"Guatemala","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"493","name_en":"Czech Republic","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"492","name_en":"India","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null},{"id":"491","name_en":"Peru","name_ar":"\u0627\u0644\u0628\u0644\u062f \u0628\u0627\u0644\u0639\u0631\u0628\u0649","created_at":"2018-11-25 09:31:12","updated_at":"2018-11-25 09:31:12","deleted_at":null}],"queries":[{"query":"select count(*) as aggregate from (select '1' as row_count from countries) count_row_table","bindings":[],"time":10},{"query":"select * from countries order by id desc limit 10 offset 0","bindings":[],"time":3}],"input":{"draw":"1","columns":[{"data":"id","name":"id","searchable":"true","orderable":"true","search":{"value":null,"regex":"false"}},{"data":"name_en","name":"name_en","searchable":"true","orderable":"true","search":{"value":null,"regex":"false"}},{"data":"name_ar","name":"name_ar","searchable":"true","orderable":"true","search":{"value":null,"regex":"false"}}],"order":[{"column":"0","dir":"desc"}],"start":"0","length":"10","search":{"value":null,"regex":"false"},"_":"1543138665057"}}
Look at letter w **at the beginning of response before {draw: 1,   I looked for it all blade fields nothing there why dose it come with response ? Thanks

System details

Operating System Windows 7
PHP Version 7.2
Laravel Version 5.7
Laravel-Datatables Version 8.x

Your vendor files might be accidentally edited? Try deleting the vendor folder then composer install again.


Is HtmlUnitDriver supported?

Not currently, but you can use Chrome headless instead. For more information, see this SO answer.

Thanka @adrianth,
Do we support any headless browser in opentest which does not need to be installed on the machine. Actually, we wanted to set up our framework on a linux machine which does not have any browser installed in it. For this, we would need headless browsers. Is there any workaround that opentest supports for such scenarios?


Modified Metadata range to have separate pool

The Metadata range should not be shared with the object range.  This
change ensures that their are separate requests to the Pal for meta-data
and object data ranges.  The requests are never combined, and thus
memory cannot flow from being used in malloc to later be used in meta-
data.

@achamayou this PR should fix the memory reporting on OE.


Confirms should be posts

Would need to wrap the buttons in forms

Pending items on the confirm page, both individually and the confirmAll button, should be changed away from links into buttons which submit a POST.  The server side should change to only accept [HttpPost]


fix(dashboard): Resetting the cache when signing out.

fix(dashboard): Resetting the cache when signing out.
Right now, when signing out you get an error regarding a missing field in our cache workspace_members this is because we sign out the user with nhost.auth.signOut(); prior to redirecting and when we query again for the workspace the user has, it comes out as null (as there is no user) and then redirect.

Codecov Report

Merging #1470 (4ad3a9a) into main (c4d5366) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 4ad3a9a differs from pull request most recent head 5de29e6. Consider uploading reports for the commit 5de29e6 to get more accurate results

@@           Coverage Diff           @@
##             main    #1470   +/-   ##
=======================================
  Coverage   85.92%   85.92%
=======================================
  Files          83       83
  Lines        9236     9236
  Branches      498      498
=======================================
  Hits         7936     7936
  Misses       1300     1300

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Shared IPC namespace for containers in a pod

#1615
This PR makes the containers in a pod share ipc namespace in addition to network namespace. The "network container" is renamed to "pod infra container" as per @thockin suggestion.
Here is some output from testing:
[root@kubernetes-minion-1 ~]# docker ps
CONTAINER ID        IMAGE                                  COMMAND                CREATED              STATUS              PORTS                    NAMES
5db6f3cea91c        dockerfile/nginx:latest                "nginx"                About a minute ago   Up About a minute                            k8s_mynginx.6617bbd0_1a59f6e9-a5b9-11e4-a611-0800279696e1.default.api_1a59f6e9-a5b9-11e4-a611-0800279696e1_b7f0a81d
d99d954d65bc        google/cadvisor:0.8.0                  "/usr/bin/cadvisor"    7 minutes ago        Up 7 minutes                                 k8s_cadvisor.4c14154f_cadvisor-agent.file-6bb810db-kubernetes-minion-1.file_3fd0c171b08ee83650fd343a6fdf9653_aa7ac19c
05187563d087        kubernetes/fluentd-elasticsearch:1.0   "/bin/sh -c '/usr/sb   9 minutes ago        Up 9 minutes                                 k8s_fluentd-es.f0e4cc_fluentd-to-elasticsearch.file-8cd71177-kubernetes-minion-1.file_a6e8b87c41aade4c1d35ead1804d8fc4_39c7aec3
c81cc8a37ce4        kubernetes/pause:go                    "/pause"               14 minutes ago       Up 14 minutes       <IP_ADDRESS>:9191->80/tcp     k8s_POD.a29db6ef_1a59f6e9-a5b9-11e4-a611-0800279696e1.default.api_1a59f6e9-a5b9-11e4-a611-0800279696e1_b963898c
39b274cc5f5c        kubernetes/pause:go                    "/pause"               19 minutes ago       Up 19 minutes                                k8s_POD.a812958f_fluentd-to-elasticsearch.file-8cd71177-kubernetes-minion-1.file_a6e8b87c41aade4c1d35ead1804d8fc4_385ae96c
5d9bc25d14ad        kubernetes/pause:go                    "/pause"               19 minutes ago       Up 19 minutes       <IP_ADDRESS>:4194->8080/tcp   k8s_POD.117b915_cadvisor-agent.file-6bb810db-kubernetes-minion-1.file_3fd0c171b08ee83650fd343a6fdf9653_88393ead
[root@kubernetes-minion-1 ~]# docker inspect 5db6f3cea91c | grep Mode
        "IpcMode": "container:c81cc8a37ce411133f366abe4d45e6ea91e6071485e4f29ed12b74419a5fa86b",
        "NetworkMode": "container:c81cc8a37ce411133f366abe4d45e6ea91e6071485e4f29ed12b74419a5fa86b",
[root@kubernetes-minion-1 ~]# docker inspect c81cc8a37ce4 | grep Pid
        "Pid": 13521,
[root@kubernetes-minion-1 ~]# docker inspect 5db6f3cea91c | grep Pid
        "Pid": 18463,
[root@kubernetes-minion-1 ~]# ls -l /proc/13521/ns
total 0
lrwxrwxrwx 1 root root 0 Jan 27 00:25 ipc -> ipc:[4026532395]
lrwxrwxrwx 1 root root 0 Jan 27 00:27 mnt -> mnt:[4026532393]
lrwxrwxrwx 1 root root 0 Jan 27 00:25 net -> net:[4026532398]
lrwxrwxrwx 1 root root 0 Jan 27 00:27 pid -> pid:[4026532396]
lrwxrwxrwx 1 root root 0 Jan 27 00:27 uts -> uts:[4026532394]
[root@kubernetes-minion-1 ~]# ls -l /proc/18463/ns
total 0
lrwxrwxrwx 1 root root 0 Jan 27 00:27 ipc -> ipc:[4026532395]
lrwxrwxrwx 1 root root 0 Jan 27 00:27 mnt -> mnt:[4026532510]
lrwxrwxrwx 1 root root 0 Jan 27 00:27 net -> net:[4026532398]
lrwxrwxrwx 1 root root 0 Jan 27 00:27 pid -> pid:[4026532513]
lrwxrwxrwx 1 root root 0 Jan 27 00:27 uts -> uts:[4026532511]

@thockin @vishh PTAL

Except for the few nits, LGTM!

LGTM!  Thanks

LGTM, except the one small comment about not duplicating a constant.

Does this change our minimum required Docker version, or will this field just be silently ignored on old versions?

@bgrant0607 I believe so, but I can test that and confirm.

@bgrant0607 IpcMode is ignored when used with older docker version (1.3.*)
https://gist.github.com/mrunalp/53d128f4654971033f11
go run ipctest.go
Container started successfully
[mrunal@dhcp-16-139 go]$ sudo docker ps
CONTAINER ID        IMAGE                 COMMAND             CREATED             STATUS              PORTS               NAMES
51b043956885        kubernetes/pause:go   "/pause"            5 seconds ago       Up 4 seconds                            ipctest
a85c1b84f30a        kubernetes/pause:go   "/pause"            4 minutes ago       Up 4 minutes                            hungry_mclean
[mrunal@dhcp-16-139 go]$
[mrunal@dhcp-16-139 go]$
[mrunal@dhcp-16-139 go]$
[mrunal@dhcp-16-139 go]$
[mrunal@dhcp-16-139 go]$
[mrunal@dhcp-16-139 go]$ docker inspect 51b043956885 | grep Mode
        "NetworkMode": "container:a85c1b84f30aa99223",
[mrunal@dhcp-16-139 go]$
[mrunal@dhcp-16-139 go]$ docker inspect 51b043956885 | grep Pid
        "Pid": 15107,
[mrunal@dhcp-16-139 go]$ docker inspect a85c1b84f30a | grep Pid
        "Pid": 14546,
[mrunal@dhcp-16-139 go]$ sudo ls -l /proc/15107/ns
total 0
lrwxrwxrwx. 1 root root 0 Jan 27 09:40 ipc -> ipc:[4026532585]
lrwxrwxrwx. 1 root root 0 Jan 27 09:40 mnt -> mnt:[4026532583]
lrwxrwxrwx. 1 root root 0 Jan 27 09:40 net -> net:[4026532484]
lrwxrwxrwx. 1 root root 0 Jan 27 09:40 pid -> pid:[4026532586]
lrwxrwxrwx. 1 root root 0 Jan 27 09:40 user -> user:[4026531837]
lrwxrwxrwx. 1 root root 0 Jan 27 09:40 uts -> uts:[4026532584]
[mrunal@dhcp-16-139 go]$ sudo ls -l /proc/14546/ns
total 0
lrwxrwxrwx. 1 root root 0 Jan 27 09:38 ipc -> ipc:[4026532481]
lrwxrwxrwx. 1 root root 0 Jan 27 09:38 mnt -> mnt:[4026532479]
lrwxrwxrwx. 1 root root 0 Jan 27 09:38 net -> net:[4026532484]
lrwxrwxrwx. 1 root root 0 Jan 27 09:38 pid -> pid:[4026532482]
lrwxrwxrwx. 1 root root 0 Jan 27 09:38 user -> user:[4026531837]
lrwxrwxrwx. 1 root root 0 Jan 27 09:38 uts -> uts:[4026532480]

Thanks.

@brendanburns @thockin Does this look good now?

LGTM

LGTM from me, also. Kicking travis.

This PR broke e2e because update.sh is depending on the presence of a container named "net"
I'll fix.

mea culpa - too happy to get it in.
On Tue, Jan 27, 2015 at 9:10 PM, Brendan Burns<EMAIL_ADDRESS>wrote:

This PR broke e2e because update.sh is depending on the presence of a
container named "net"
I'll fix.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/3817#issuecomment-71781930
.

np, I'm running it in e2e right now...
On Tue, Jan 27, 2015 at 9:29 PM, Tim Hockin<EMAIL_ADDRESS>wrote:

mea culpa - too happy to get it in.
On Tue, Jan 27, 2015 at 9:10 PM, Brendan Burns<EMAIL_ADDRESS>wrote:

This PR broke e2e because update.sh is depending on the presence of a
container named "net"
I'll fix.
--
Reply to this email directly or view it on GitHub
<
https://github.com/GoogleCloudPlatform/kubernetes/pull/3817#issuecomment-71781930

.

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/3817#issuecomment-71783251
.

Sorry about that! Thanks for fixing.


Add description of missing blacklist_domain config option

The blacklist_domain/blacklist_domains config options are a little-known fact, we need to disclose it in docs.

👍
There are a lot of bundle settings that are undocumented.


Typo in Readme.md

Hello Jeeliz,
Just wanted to inform you that there is a typo in readme.md file
It should be indeed not indead.

Indead, Apple© owns the intellectual property of 3D animated foxes (but not on raccoons yet). Thank you for your understanding.

Nothing major.
Regards,
Abhilash

Hi @abhilash26
Thanx, I have committed the fix.
Best,
Xavier


Change default layout programmatically

What problem does this feature solve?
Role-based applications could load different layouts based on the user's role. Currently, I am utilizing the layout property in every component like:
  layout(context) {
    return context.store.state.auth.user.role
  }

my layout folder is structured like:
layout/

admin.vue
default.vue
manager.vue
norole.vue
user.vue

Which works great until I create a new component and forget to throw this snippet in.
What does the proposed changes look like?
I am wondering if having a global method to set the default layout would be a useful feature. In the scenario of programmatically changing the layout based on a user's role, I would image it would be called like:
this.$layout.setDefault(this.$auth.user.role + '.vue'); // admin role -> admin.vue
// or maybe even bind it?!
this.$layout = this.$auth.user.role + '.vue';

Maybe I am overthinking this. Thanks!

This feature request is available on Nuxt community (#c8764)

I think that's already possible:
$nuxt.setLayout('admin')

Thank you! Do you know of any way I can access this.$nuxt from within vuex?

@russellsean on client-side: window.$nuxt.

PS: Closing here as the requested feature is already included.

I noticed this only changes for the current component. If you route to another page it changes back to the default layout. Is there any way to change the default layout for the entire app?

Jumping in here, is there a way to use setLayout from inside asyncData?
All I found is accessing context.app.methods.setLayout but it doesn't change the actual layout of the printed page.

@russellsean @altryne did you find the solution?

@marko-mlinarevic negative. I took a different approach. I use the same layout for authenticated users, but load components based on their role.

@russellsean I did a hacky solution.
when landing on page I do this in fetch
          beforeNuxtRender(({ nuxtState }) => {
            nuxtState.layout = store.state.layout;
          })

and when changing pages I do this in created

    this.$nuxt.setLayout(this.$store.state.layout)

It works for now

Hi guys the solution on the client works very well, just need a loading to cover the layout changes, but in the fetch the solution for the backend works but throws some errors on the console about the Virtual DOM, is there another solution to resolve in the serve side? Because I think is a feature that the dev wanted to them.

Maybe my approach can help.
In default.vue layout I placed the layout change in the data().
You can place the layout switching logic in each layout.
data() {         if(this.$auth.loggedIn) {             this.$nuxt.setLayout('different-layout');         }     }

When I try changing the layout e.g. $nuxt.setLayout('default') it seems to get overwritten by the current layout almost immediately after the change. $nuxt.layoutName does return the layout I change it to though. Does anyone have any ideas?

    setLayout (layout) {
      if(layout && typeof layout !== 'string') {
        throw new Error('[nuxt] Avoid using non-string value as layout property.')
      }

      if (!layout || !layouts['_' + layout]) {
        layout = 'default'
      }
      this.layoutName = layout
      this.layout = layouts['_' + layout]
      return this.layout
    },
    loadLayout (layout) {
      if (!layout || !layouts['_' + layout]) {
        layout = 'default'
      }
      return Promise.resolve(layouts['_' + layout])
    },

    ```

this.$nuxt.setLayout works beautifully. Like so many things, it'd be nice to be able to find out about it without having to track down a reply to people desperately trying to do the same thing. This should be in the layout documentation. It's not:

I was able to achieve this by editing layouts/default.vue to something like this...
<template>
  <DefaultLayoutV2 v-if="$config.version === 'v2'" />
  <DefaultLayoutV1 v-else />
</template>

<script>
import DefaultLayoutV1 from './default-v1.vue'
import DefaultLayoutV2 from './default-v2.vue'

export default {
  components: { DefaultLayoutV1, DefaultLayoutV2 }
}


Move local embeddings as a back-fill for symf

Internal Slack thread to monitor for decision on removing local embeddings.
PRD draft indicating local embeddings are not to be removed. If that is the case, @beyang wants local embeddings as a backfill for symf, however that is probably very few cases in practice.
After 1:1 sync with @beyang 2024-05-11, the plan is: symf to be the primary source. When local embeddings are available, query them and blend the results—either with a 70/30 60/40 etc. split or with RRF.

completed


listview

Can you nest listview?

You can't nest ScrollView on android. see : https://github.com/facebook/react-native/issues/8024
I added the ScrollViewComponent props in the v0.8.0. You should be able to use it if you want to use ListView (72e2806). See an example here with Flatlist : https://github.com/bamlab/react-native-image-header-scroll-view-example/blob/master/src/Pages/Avignon.js#L34


Refactor Task Manager to track TurbiniaRequest objects

Right now the task manager takes in the requests and pulls out the evidence and processes that.   Right now we copy some of this data to the evidence (request id and config), but really we should just keep and track the Request so we can have access to all of the parameters.  The big thing we want is the Recipe/Config data associated with it.

Now that the task manager tracks jobs and tasks and saves all output evidence related to those, I don't think we need to do this.


move iso script to correct path

thanks, this script will be getting moved around soon anyhow https://github.com/secureblue/secureblue/pull/564

https://github.com/secureblue/secureblue/commit/85ca39551546a8b14b3f2e95a04c0b1a09487a7c


Hot Reload IP

Hey Guys,
Thanks for the great boilerplate!
I have a quick question. I can't seem to get the hot reload working when working with a remote server. I have a local linux server <IP_ADDRESS> that is hosting the site.
When running npm run watch it works no problem when accessing the site via the IP (on nginx).
When running npm run hot it tries to load the js and css from 'localhost:8080'. Is there anyway to change the IP for hot reload?

Try checking with laravel-mix.


[2.0.3-prepare][Fix-8010] Rectify this issue about not editing the form of task node when it was under the mode of full screen.

Purpose of the pull request
Issue: #8010 .
PR: #8039 .

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

:exclamation: No coverage uploaded for pull request base (2.0.3-prepare@600ee5e). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             2.0.3-prepare    #8042   +/-   ##
================================================
  Coverage                 ?   21.05%
  Complexity               ?     1577
================================================
  Files                    ?      534
  Lines                    ?    22399
  Branches                 ?     2641
================================================
  Hits                     ?     4715
  Misses                   ?    17208
  Partials                 ?      476

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 600ee5e...4cb9cba. Read the comment docs.


Small typo

Under Configure the Azure AD Application Registration for group attributes, number 3:
If you want <typo>to</typo> groups in the token to contain the on premises AD group attributes in the optional claims section specify which token type optional claim should be applied to, the name of optional claim requested and any additional properties desired. Multiple token types can be listed:
to => the

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

ID: c5a6779a-58f1-73fa-2b24-5f85deeea52b
Version Independent ID: 481c093b-6db3-c4e5-6644-b1a3a4497a30
Content: Configure group claims for applications with Azure Active Directory
Content Source: articles/active-directory/hybrid/how-to-connect-fed-group-claims.md
Service: active-directory
GitHub Login: @billmath
Microsoft Alias: billmath

@janwillemkeizer
Thank you for your feedback! We will review and provide an update as appropriate.

@janwillemkeizer thanks for the update! We're currently making changes to the docs accordingly. We appreciate the feedback so that we can keep these docs up to grade for the community.
Closing this issue as we are making the change.


Update to design-system 0.31.0 | Convert a-heading inside m-slug-header to m-slug-header_heading

Merge after https://github.com/cfpb/design-system/pull/1732
Contains:
https://github.com/cfpb/design-system/releases/tag/v0.31.0
https://github.com/cfpb/design-system/compare/v0.30.0...v0.31.0
Changes

Convert a-heading inside m-slug-header to m-slug-header_heading
Update meta header to use the one defined in the design system (updated in https://github.com/cfpb/design-system/pull/1731)

How to test this PR

http://localhost:8000/blog post previews meta header (bit above the post title link) is unchanged from production.
http://localhost:8000/rural-or-underserved-tool/ (e.g.) will have slug headers in the right-hand side bar that are unchanged from production.

Hold till we work out what we want to do with
https://www.consumerfinance.gov/compliance/amicus/
https://www.consumerfinance.gov/rules-policy/notice-opportunities-comment/

@anselmbradford
Let's remove the links from those two instances. The overwhelming majority of instances of the meta header component on our website do not include a link. So we can safely assume that the links were added on these two pages due to an inconsistency approach or human error when building the pages in the Wagtail interface. Both pages include a link to go back to the full list of results and filter so, if needed, a user can access the content and filter by the category that they desire.

@natalia-fitzgerald that's great; as Ans alludes to above that change will allow for some additional simplification of the post preview template. @anselmbradford can we go ahead with this PR and then I can open a subsequent one to do that simplification? Or is there something about these changes that requires including that link removal here?

can we go ahead with this PR and then I can open a subsequent one to do that simplification? Or is there something about these changes that requires including that link removal here?

Chatted and we'll go ahead with this PR and then in a subsequent PR clean up the url argument that as of this PR is getting ignored in the post preview.


Mispawned chimney

In Cool Cool Mountain there is a chimney where the snowman's head spawns

oh yea, I'll fix the static level objects on this level soon

Fixed I think... reopen this if not


    add fstats customized for screen-level Q and T with "land only" option

a standalone set of code, script, and rc file  are added to GEOS_Util/post/  for forecast stats customized for 2-meter moisture and temperature verification. It is tested in the cases of  self-verification and  ecmwf-verification, and with "land only" option.

@lltakacs @gmao-jstassi  Larry and Joe,   the modifications/additions  ( for screen level variables and land only option) are not  put into fstats.F90, g5fcst_stats.pl and stats.rc,  instead in the customized  fstatsNx.F90, g5fcst_statsNx.pl and statsNx.rc. Comparison of *stats  vs. *statsNx files will show the modifications.  could you review them and give comments and suggestions.   We have tested and used this addition for fcst verifications  in land-atmos coupled DAS experiments.

Larry and Joe reviewed the draft PR , and sent via email comments and suggestions:
for fstatsNx.F90
" (1) The variable “landonly” is only explicitly initialized if “-land” is present.  You should probably set a default value explicitly to .false.
(2) There are several comments and print-outs that are different from the ones in the Develop version of GIT.  I think you might just have started from an older version.  Never-the-less, it would be best to merge your updates once again to what is in Develop."   -Larry
for g5fcst_statsNx.pl
"I did a xxdiff between the scripts, and I saw that a couple dozen lines have been added or modified. I didn't see anything that looked like a problem. Ideally, you should get someone else to run the code to see it does what they think it should do.
If I get a chance later, after you get your script into the repository, I will take a look and try to merge the two scripts." - Joe

@saraqzhang, re. you last commit:

merge statsNx.F90 to stats.F90

If stats.F90 can now do everything that stats.F90 and statsNx.F90 were doing when you first created the PR, then we should "git remove" statsNx.F90 from the branch before merging the PR.

the modification/addition for new options "land only" and "Nx only" are merged into the stats script and  code.  The "customized" version of the codes  are removed.

@saraqzhang would you mind updating the changelog for this PR so all CI tests will pass? Thanks

this code is compiled/built with MAPL v2.8.0  used in adas,   where MAPL Const only have MAPL_TICE,  not MAPL_CELSIUS_TO_KELVIN.

this code is compiled/built with MAPL v2.8.0

Ouch, that's old.  Oh well.  I guess for the purpose of computing fstats it doesn't really matter whether 273.15 or 273.16 is used.

@gmao-rreichle @saraqzhang do you feel this PR is resolved and ready for merging?

@gmao-rreichle @saraqzhang do you feel this PR is resolved and ready for merging?

I'm happy with the changes.


cannot pickle temp module on custom dataset

using custom dataset
@DATASETS.register_module()
class ANewDataset(CustomDataset):
CLASSES = ("one", "two", "yjtrr")
def load_annotations(self, ann_file):
         # handling files

when I use it for training,
cfg = get_cfg()
datasets = [build_dataset(cfg.data.train)]
model = build_detector(
cfg.model, train_cfg=cfg.get('train_cfg'), test_cfg=cfg.get('test_cfg'))
model.CLASSES = datasets[0].CLASSES
mmcv.mkdir_or_exist(os.path.abspath(cfg.work_dir))
train_detector(model, datasets, cfg, distributed=False, validate=True)
looks like the custom dataset files is converted to a temp file. but during train_detector,
an error was reported:
Python39\lib\multiprocessing\reduction.py", line 60, in dump
ForkingPickler(file, protocol).dump(obj)
_pickle.PicklingError: Can't pickle <class 'tmpbmscwqvw.ANewDataset'>: import of module 'tmpbmscwqvw' failed

Could you provide more information, like the custom dataset code?

custom data code is above,
@DATASETS.register_module()
class ANewDataset(CustomDataset):
CLASSES = ("one", "two", "yjtrr")
def load_annotations(self, ann_file):
# handling files
for datasets = [build_dataset(cfg.data.train)]
it is okay to get the custom data and the calling load_annotations, but the break point in this function is not working
since from custom,
calling self.data_infos = self.load_annotations(local_path)
it went to a temp py file, like
BUILD_MAP, LOAD_FAST(.0), FOR_ITER(to 20),  xxxxx
xxx

========
for loading data, the temp file is still working, but for
dump (reduction.py), ForkingPickler(file, protocol).dump(obj) cannot find the temp file.

@tempbrucefu Sorry, can you fill it out according to the issue template?

This issue will be closed due to inactivity. Feel free to reopen this pr if you have any questions.


How to mix ocamlformat and ocamlformat-mlx

In my own Conform set up config have the ocaml file type mapped to ocamlformat, e.g.:
opts = {
    formatters_by_ft = {
      ocaml = { "ocamlformat" },
    }
}

If I keep that config and open an mlx file with the filetype ocaml.mlx, conform match on the first part of the file type an tries to format with ocamlformat.
If I remove the config above, then this plugin configuration takes over and ocaml.mlx is mapped to ocamlformat-mlx.
Is there a way to have both ocamlformat and ocamlformat-mlx configured with each being applied to the correct filetypes?

I realised we can pass a function to the conform table, so I went with:
function()
  if vim.bo.filetype == "ocaml.mlx" then
    return { "ocamlformat_mlx" }
  else
    return { "ocamlformat" }
  end
end

It seems to do the trick


introduce additional port parameter (-p port_number) for serve command

Typically i have many things running, that would like to use port 8080. So perhaps it's helpfull for others as well, if they could configure the port used by the serve command

Could you please also modify the readme at https://github.com/avisi-cloud/structurizr-site-generatr/blob/main/README.md#start-a-development-web-server-around-the-generated-website and note that 8080 is the default port.
Otherwise, LGTM!

build-docker failed to download ubuntu packages during Dockerfile command
RUN apt update && apt install graphviz --yes && rm -rf /var/lib/apt/lists/*
gh run view<PHONE_NUMBER> --log-failed shows, that some ubuntu repository server can't be reached
build-docker    Build container image   2023-04-26T07:01:25.2641543Z #9 70.75 W: Failed to fetch http://ports.ubuntu.com/ubuntu-ports/dists/jammy-backports/InRelease  Unable to connect to ports.ubuntu.com:80: [IP: <IP_ADDRESS> 80]
Looks like a problem, that is not related to the content of the pull request


Go to Definition does not work

Hi,
could anybody please help me with this problem? How to make it work? Proof (vue3):

~Thanks,
Vadim

Hi @vinogradov, you need to wrap components option by defineComponent to have language support, but please not that the language support for JS is limited.

It was supported by https://github.com/johnsoncodehk/volar/commit/a78acac01eb23562d9ce28906a4a185911c57131.


A description put on a 'Background:' step, is suppressed from the HTML report

Issue:
When you add a description to a 'Background:'  step in a feature file, the Karate HTML report does not display the description.
NOTE: The Cucumber report also suppresses the Background description, but I personally see no reason to be suppressing that.
To Reproduce:

You can reproduce this on ANY project, including the Karate skeleton project.  Just modify the Background on this file to have a description in the background (on the same line as the 'Background:'  statement :
https://github.com/intuit/karate/blob/develop/examples/jobserver/src/test/java/jobtest/simple/simple1.feature

When the tests are done running, refer to the Karate HTML report under   target/surefire-reports/karate-summary.html
You will notice that if you added a description, neither the Karate (nor the Cucumber) HTML report show the description.

won't fix, PR-s welcome against this ticket after discussing implementation approach


set julia min versions to 0.6.0-pre

for newer syntax like abstract type, Base.IndexStyle, etc
on early 0.6.0-dev versions, would be better to stay with the 0.5 version of the package

Codecov Report

Merging #20 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   82.98%   82.98%
=======================================
  Files           6        6
  Lines         288      288
=======================================
  Hits          239      239
  Misses         49       49

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 d04eca5...62bcd87. Read the comment docs.


Improved partial sync API

This is a WIP that I'm publishing for sake of visibility and early feedback.
The Swift API is close to @tgoyne's proposal. The Objective-C API is similar, but uses native KVO for notifying users of changes to the partial sync state. KVO's usability isn't great from Objective-C, so we may want to do something different here. It was just the easiest thing to throw together.

Should unsubscribing live on the Realm, or the Results? It seems like it needs to live on the Realm in order to allow unsubscribing based only on the name. It's a bit unsymmetrical though, since subscribing lives on Results.
How should errors raised by partial sync, if any, be exposed?
With the Swift API they could potentially be passed to the callback as the payload of an enum value. That's not possible with the current approach in Objective-C.

As per a use in the forums, when trying to compile the RelmSwift module after a successful compile of the Realm module the, an error occurs here:
<module-includes>:1:9: note: in file included from <module-includes>:1:
#import "Headers/Realm.h"
        ^
/Users/spector/Library/Developer/Xcode/DerivedData/ToDo-dbvoejisiwkcjwarfsyauvuolqfb/Build/Products/Debug-iphoneos/Realm/Realm.framework/Headers/Realm.h:38:9: error: 'Realm/RLMSyncSubscription.h' file not found
#import <Realm/RLMSyncSubscription.h>
        ^
/Users/spector/Work/Realm/my-first-realm-app/ios/PartialSync/Pods/RealmSwift/RealmSwift/Aliases.swift:20:8: error: could not build Objective-C module 'Realm'
import Realm
       ^

The code is included from the partial sync branch as with the previous report:
  pod 'RealmSwift', :git => 'https://github.com/realm/realm-cocoa', :branch => 'mar/partial-sync', :submodules => true
  pod 'Realm', :git => 'https://github.com/realm/realm-cocoa', :branch => 'mar/partial-sync', :submodules => true

cc @bdash

@dhmspector, I think that'll be fixed by 879c1a3.


Very weird behavior no playback

Started Cumulus -> Sign in -> Go to Feed / Likes / Playlists -> Click Play
-> it shows covers changing a lot and the text gets striked through then it does taht for a while but even if i hit play nothing works

This is a rate limit issue, see https://github.com/gillesdemey/Cumulus/issues/89


Support CustomWriterIdentity

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.
Is your feature request related to a problem? Please describe.
I think we should be able to set a customWriterIdentity in a CreateSinkRequest. Based on rest api: https://cloud.google.com/logging/docs/reference/v2/rest/v2/organizations.sinks/create
Describe the solution you'd like
Allow setting custom writer identity.
Describe alternatives you've considered
Additional context

@cindy-peng any update on this? Is this on the roadmap or is there a different way to implement this?

Sorry for the late reply @johnclara! I agree that adding CustomWriterIdentity  parameter makes sense for the library. Unfortunately the team has been busy working on some other projects. I've added your request to our list and will let you know as soon as we have a plan in place.


Make Timestamp implement java.time.Temporal

If Timestamp implements Temporal, it would be really easy for users to convert to other java.time types. For example, one could call LocalDate.from(myTimestamp). Right now, users have to write their own conversion method, or they have to perform a multi-step conversion going through Date or Calendar.

This may be relevant to your interests: https://github.com/amazon-ion/ion-java/issues/425

Oops. I'll close this in favor of the original issue.


Wrong password

Hello, I am having issues with the latest updates, somehow it keeps telling me wrong password while I am sure it is right.
I did change the SSID and password in the include/user_config file.
Also I don't see the changes I do in user config file really reflecting in build, I change the server IP and make clean then make the project, I flash with new bins and nth happen, just connecting to same IP.

ah found the problem, i was not using the make flash which is now deleting this old config


Add code for finding no. of nodes in a subtree

Closes #42

Description: Add code for finding no. of nodes in a subtree
Checklist
<Put an x between square brackets wherever appropriate without any spaces like [x]>

[ ] I've followed the Contributing guidelines provided in the repository.
[ ] I've made the changes which were demanded in the linked issue.
[ ] My code gave a clean output on debugging. (NO warnings/errors)

@ankiiitraj merge it.


EAS Build looks in wrong place for Sentry CLI

Hi there, thanks for the recent work in improving monorepo support in Expo 43, much appreciated.
I've followed along closely to the changes here and in Expo 43, and attempted to follow a similar migration path for one of my projects (Expo Managed workflow) so it can take advantage of EAS Build and Submit.
It has worked well overall, and I was able to get an Expo 43 monorepo project building on EAS Build for iOS and Android.  However, when I then try and do the final remaining step of removing expo-yarn-workspaces (which seemed necessary as it was causing issues in the Expo Web app), this breaks the mobile app under EAS Build on the Sentry step.  The error reported by EAS Build servers suggests that it's looking in the child package node_modules folder for Sentry, which it won't be able to find anymore as expo-yarn-workspaces is not doing symlinking.
Android error from EAS build log
[stderr] FAILURE: Build failed with an exception.
[stderr] * What went wrong:
[stderr] Execution failed for task ':app:bundleReleaseJsAndAssets_SentryUpload_42'.
[stderr] > A problem occurred starting process 'command '/build/workingdir/build/packages/mobile/node_modules/@sentry/cli/bin/sentry-cli''

iOS error from EAS build log
› Executing RapidStaging » Bundle React Native code and images

❌  error: No such file or directory (os error 2)

I've checked carefully and do believe I'm following all the relevant steps for Sentry setup.
packages/mobile/app.config.js snippet
module.exports = () => {
  // ...
  return {
    expo: {
      // ...
      plugins: ['sentry-expo'],
      hooks: {
        postPublish: [
          {
            file: 'sentry-expo/upload-sourcemaps',
            config: {
              organization: 'redacted',
              project: 'redacted',
              authToken: 'redacted',
            },
          },
        ],
      },
    },
  };
};

packages/mobile/package.json snippet
  "dependencies": {
    "@sentry/react-native": "^2.6.0",
    "expo": "^43.0.2",
    "sentry-expo": "^4.0.0",
  },

Is there some way of hinting to the sentry-expo "plugin" that the project is a monorepo, so it can make the appropriate adjustments for locating Sentry CLI?  Everything I'm reading says that in Expo 43 symlinking with expo-yarn-workspaces is no longer necessary and module resolution should just work, but that doesn't seem to be the case here.
yarn.lock package versions:

@sentry/react-native: 2.6.2
expo: 43.0.2
sentry-expo: 4.0.3

Hi @jamesisaac! We are working with Sentry to get this resolved asap, you can follow along here https://github.com/getsentry/sentry-react-native/issues/1857

Excellent, thanks!  Sorry, presumed sentry-expo was in the expo monorepo, or I would have spotted https://github.com/expo/sentry-expo/issues/217

No worries! We probably will encounter this for some libraries in the future. But, we are actively trying to solve these types of issues either on our (Expo) side, or with the library maintainers. I'm glad you are one of the community members who warns us of these types of issues, so we can resolve them. Thanks for that ❤

I managed to make it work in ejected and not ejected expo projects. The problem only occurred in monorepo; the main problem is that the Sentry CLI path is hardcoded.
Ejected Expo:
This https://github.com/expo/sentry-expo/issues/217#issuecomment-979878785 (iOS section) will do the project to build with EAS correctly.
Not ejected Expo:
By adding nohoist option to yarn workspace:
"nohoist": [
    "@react-native-community/cli-platform-ios",
    "@react-native-community/cli-platform-android",
    "@sentry/cli",
    "@sentry/react-native",
    "expo",
    "react-native",
    "jsc-android",
    "hermes-engine",
    "sentry-expo"
  ]

That's definitely a good workaround for now. At least until https://github.com/getsentry/sentry-react-native/issues/1912 is solved. If Sentry respects the path on iOS (on Android they already do I believe), we can actually use that instead of the workaround.

@byCedric still seeing this issue in a monorepo with expo 48

My error is a little bit different but i assume it is related. Build passes for android. Im using npm so I don't have the nohoist option, is there another work around?


Resolved the Uncaught TypeError and SelectBox not displaying graph names

Hi @MJinH ,
Metadata retrieval was updated here 22dbb4394fce0797c3208407d8bb9eba613beef7 in order to retrieve metadata much faster. It was tested and working. Can you provide more details as to why this is being changed back in this PR?


Cant set center of the map.

I am trying to set the center of the map but to no success. What am i doing wrong?
The map marker is positioned to geolocation but center is not changing.
classSimpleMapPage extends Component {
 mixins: [PureRenderMixin];
 constructor () {
     super();
     this.state = {
         center: {
             lat: 23.938043,
             lng: 73.337157
         },
         zoom: 14
     };
 }

 componentDidMount() {
     navigator.geolocation.getCurrentPosition(
          (position) => {
              this.setState({
                      center: {
                          lat: position.coords.latitude,
                          lng: position.coords.longitude
                      }
                  }
              );
              // var center = this.state.center;
              // center.lat = position.coords.latitude;
              // center.lng = position.coords.longitude;
              // this.setState({center: center});
          },
          (error) => { console.log(error.message),
          {enableHighAccuracy: true, timeout: 20000, maximumAge: 1000}
      );
 }

 render() {
     return (
         <GoogleMap center={this.state.center} zoom={this.state.zoom} >
             <MapMarker lat={this.state.center.lat} lng={this.state.center.lng} text={'Me'}/>
         </GoogleMap>
     );
 }
};`

+1

Works for me https://jsbin.com/ledujubibe/edit?js,console,output

@istarkov it was because I was setting it as an array not an object, my bad, thanks for the quick response 👍

So I found out about this navigator with access to the geolocation methods via searching this repository for the keyword geolocation Could this be something that is pointed on on the main README.md? Or is there somewhere that has a wiki about how to access for API methods like this?


Pr fixs

Fixes #90 #100 #87
Changes you made

There is nothing here on the home page
-delete crispy
Please mention someone for code review
add home and login logout and base templates

Please give a meaningful name to this PR.
Also, please explain in the description what is this PR for and why you did these changes. For example, why did you delete crispy?
There is no need to close this PR, you can edit it.


Firefox and Windows flag the executables as viruses, after running one, a Windows Defender notification pops up and tells me that it installed a Trojan.
To be specific, the "Trojan:Win32/Wacatac.B!ml" is recognized which, if untreated, can cause data leaks.

lol

False positive. Already been discussed. Chrome uses Windows Defender to scan for "malware"
Readme links are links to the files listed above.....
https://github.com/ShadowWhisperer/Remove-MS-Edge/issues/11
https://github.com/ShadowWhisperer/Remove-MS-Edge/issues/40

Ok thanks, maybe add a notice to the README so that new users don't get sudden anxiety :P


Correct git edit link on overview

The following link should go to /settings/git instead of opening a dialog.

hey I want to work on this issue 😀

I set up everything locally but when the dashboard loads I can't see the "OverviewRepository" component on the dashboard. why is this ??

This has been fixed in a past release.
@Aashish-Upadhyay-101: The reason why you didn't see that component is that it's not available locally. We only allow Git-related features in the cloud.


built for mobile

I wonder if rxcpp is built for mobile and is available. Because I want to use Android ndk or iOS.

yes! @igrmk contributed no-exceptions support and added rxcpp to android - https://android-review.googlesource.com/c/platform/external/Reactive-Extensions/RxCpp/+/649610


What are the assumptions about stdgates.inc ?

The OpenQASM3 paper arXiv:2104.14722v1 defines the content of the file stdgates.inc in the section 4.1.1 Standard gate library p13, similarly to qelib1.inc, the OpenQASM2 equivalent, in the paper arXiv:1707.03429, section 3.1 Quantum Experience standard header p10.
The OpenQASM2 experience tells me that the lack of detail about its formal meaning and stability of the standard library content can be problematic. For example, several components assumed that the qelib1.inc will always preserve backward compatibility (new gates can be added, but they cannot be removed). You can see the discussion here:  https://github.com/Qiskit/qiskit-terra/pull/6125
Similarly some components (like qiskit-terra itself) only use the file to extend the gate namespace but ignores the body (a la opaque gate). A change in a definition have no effect at all.
In order to avoid similar issues, I suggest to be more explicit on the assumptions around stdgates.inc and its potential formal semantic of including the file. Consider that extending the default global scope (like in the IBM Quantum Experience) will be a needed use case in the mid-term future.

In OpenQASM3 there is only one built-in gate at the language level: U. We have written stdgates.inc in the paper so that including that file will provide backward compatibility for many OpenQASM2 programs.
You seem to be poking at two issues: (1) formal meaning and (2) stability. I'm not sure I follow what is missing on (1). The formal meaning of include "stdgates.inc;" is equivalent to #include in C, i.e. it is as if one copy-pasted those lines of code from another file into the current source file. On (2), I concur that changing the meaning of symbols in standard libraries is disruptive and should be avoided. However, it is going to be up to OpenQASM3 implementers to provide their own libraries. The version in this repository can serve as a reference to implementers, but I don't see what we can do to enforce constraints on that.

Thanks, this unfolding is useful!
(1) I probably used the term formal wrongly. If we take strictly that include works like the directive in C, that means that the compiler needs a preprocesor to do the inlining and then parse the result. This seems inefficient. Moreover, it makes "standard" gates indistinguishable of the custom gates. For OpenQASM2, Qiskit made the unilateral assumption that the gates in qelib1.inc were "standard". That means that they are not being parsed and namespace is extended with Qiskit standard gate objects that are mapped to the gate names in that file. The underling assumption is that standard gate definitions (their decomposition in other gates) are just a way to referred to their unitary matrices. In other words, the standard gate swap refers to the object SwapGate and how is defined in qelib1.inc turned irrelevant, because the convention was that we were referring to the same meaning. However, a swap gate declaration out of qelib1.inc means that that decomposition have to be used, and no other. These mechanism were never explicit and is what I referred as "assumptions".
(2) Does it means that stdgates.inc is no different that any other library? That would mean that no assumptions can be made about it and it should be parsed and analyse always?
If there is no shortcuts about meaning of a declared gate that means that is not easy to "match" the those gates with other representations. In other words, the only way to know that gate swap a,b { cx a,b; cx b,a; cx a,b; } and SwapGate (the Qiskit object) is the same is by calculating the unitary of both and check for equivalence? Would it be okey to "precompute" the matrix for some gate declarations based on... the name? the hash of its declaration body?

In other words, the standard gate swap refers to the object SwapGate and how is defined in qelib1.inc turned irrelevant, because the convention was that we were referring to the same meaning. However, a swap gate declaration out of qelib1.inc means that that decomposition have to be used, and no other. These mechanism were never explicit and is what I referred as "assumptions".

I can personally confirm that this tacit assumption is quite surprising when one starts to explore the Qasm2 implementation in Qiskit!

In other words, the only way to know that gate swap a,b { cx a,b; cx b,a; cx a,b; } and SwapGate (the Qiskit object) is the same is by calculating the unitary of both and check for equivalence? Would it be okey to "precompute" the matrix for some gate declarations based on... the name? the hash of its declaration body?

In view of the intent of OpenQASM 3 to move ahead with the technology, it is unacceptable to take the position that these gates are effectively part of the language. It's also unacceptable to demand that conforming implementations actually interpret the standard source for these well-known operators as part of every compilation.
So maybe the answer is to relax (or interpret) the role of stdgates.inc so as to allow a #pragma predefined to appear textually in stdgates.inc, i.e., allow an implementation to declare #pragma predefined h before the definition of h in stdgates.inc if, in fact, the Hadamard gate is predefined in the implementation's translator.
In this case, any definition in any user file of  gate h ... would be valid and would indeed be the prevailing definition immediately upon being encountered were the #pragma predefined not to appear in that translation unit (or perhaps #pragma predefined can be described as only in scope for the next immediate appearance of a gate definition and raise an error if the next gate definition encountered is not the name called out by the #pragma predefined immediately above).

So maybe the answer is to relax (or interpret) the role of stdgates.inc so as to allow a #pragma predefined to appear textually in stdgates.inc

There does not appear to be anything to preclude this mechanism in any of:

the grammar
directives.html
Arxiv 2104.14722 sec. 4.1.1  Standard gate library

which prescribes that "AnyOpenQASM 3 circuit that includes the standard library can make use of these gates" but does not specifically state that the definition found therein will be interpreted textually at compile time as opposed to having been built into the translator.

Certainly the latter point could lead to confusion if a user believed that altering the definition in stdgates.inc would alter the compilation unit, but equally certainly the presence of a #pragma predefined would alert the user to the danger.


Error: -daemon is not supported on this operating system

Describe the issue
Error: -daemon is not supported on this operating system
Can you reliably reproduce the issue?
yes
If so, please list the steps to reproduce below:

i follow every single step on https://docs.raptoreum.com/raptoreum-docs/raptoreumcore-cli/installing-and-using-raptoreumcore-cli/ in windows OS (10,11) and
2.it shows up Error: -daemon is not supported on this operating system

Expected behaviour
i should be able to use rtm daemon
Actual behaviour
it doesn't work

As far as i know windows does not support daemonization, more info on bitcoin pr 8813
note: you should be able to run the raptoreumd without daemon=1

perfect!!!! it worked!!! nando you are great indeed sorry if i said that developers were shit. you are not. maybe tri is but you are not indeed.


The repo confluent.packages.io seems broken

I have installed the platform and the repo packages worked fine,now I am trying another installation and I constantly found a Failed Connection for Red Hat distribution
This system is receiving updates from RHN Classic or Red Hat Satellite.
https://packages.confluent.io/rpm/5.1/repodata/repomd.xml: [Errno 14] curl#7 - "Failed connect to packages.confluent.io:443; Operation now in progress"
Trying other mirror.
https://packages.confluent.io/rpm/5.1/repodata/repomd.xml: [Errno 14] curl#7 - "Failed connect to packages.confluent.io:443; Operation now in progress"
Trying other mirror.
https://packages.confluent.io/rpm/5.1/repodata/repomd.xml: [Errno 14] curl#7 - "Failed connect to packages.confluent.io:443; Operation now in progress"

It was a Red Hat distribution problem. Not related with confluent.packages.io repository.


Requested changes from the final review of the formatting guide

Purpose
This has the changes related to the finalizing review phase two of the formatting guideline https://github.com/ballerina-platform/ballerina-lang/issues/14864#issuecomment-487554983
Related #14864

Codecov Report

Merging #15178 into master will decrease coverage by 2.96%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #15178      +/-   ##
============================================
- Coverage     56.93%   53.97%   -2.97%
  Complexity     3103     3103
============================================
  Files           282      320      +38
  Lines         16883    18046    +1163
  Branches       3082     3273     +191
============================================
+ Hits           9613     9741     +128
- Misses         6028     7050    +1022
- Partials       1242     1255      +13

Impacted Files
Coverage Δ
Complexity Δ

tool-plugins/vscode/src/utils/logger.ts
52.63% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/core/index.ts
100% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/debugger/index.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/debugger/manager.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/test-runner/activator.ts
0% <0%> (ø)
0% <0%> (?)

...lugins/vscode/src/core/extended-language-client.ts
12.5% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/debugger/session.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/extension.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/bbe/renderer.ts
0% <0%> (ø)
0% <0%> (?)

tool-plugins/vscode/src/trace-logs/index.ts
100% <0%> (ø)
0% <0%> (?)

... and 28 more

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 abeb9ea...ec5f457. Read the comment docs.


Image digest spec does not include the entire alphabet

Note that algorithm part only allows [A-Fa-f], instead of [A-Za-z]. This looks like a copy-and-paste error from the hex part.
Modify it to match the reference implementation.

This section is being deprecated for https://github.com/opencontainers/distribution-spec/blob/master/spec.md
We will likely remove this soon, but any changes should be filed there