CNN Predict: Key b-1 not found in checkpoint

Hi @brightmart ,
I have trained the CNN using ''train-zhihu4-only-title-all.txt''. When i am using the predict file for prediction on "test-zhihu6-title-desc.txt" using the word2vec as "zhihu-word2vec-title-desc.bin-100", I am getting the following error:
Restoring Variables from Checkpoint
2018-06-27 20:49:22.480037: W tensorflow/core/framework/op_kernel.cc:1318] OP_REQUIRES failed at save_restore_v2_ops.cc:184 : Not found: Key b-1 not found in checkpoint
Traceback (most recent call last):
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1322, in _do_call
return fn(*args)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1307, in _run_fn
options, feed_dict, fetch_list, target_list, run_metadata)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1409, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.NotFoundError: Key b-1 not found in checkpoint
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_INT32, DT_INT32, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "p7_TextCNN_predict.py", line 77, in
saver.restore(sess, tf.train.latest_checkpoint(FLAGS.ckpt_dir))
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 1802, in restore
{self.saver_def.filename_tensor_name: save_path})
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 900, in run
run_metadata_ptr)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1135, in _run
feed_dict_tensor, options, run_metadata)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1316, in _do_run
run_metadata)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1335, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.NotFoundError: Key b-1 not found in checkpoint
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_INT32, DT_INT32, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
Caused by op 'save/RestoreV2', defined at:
File "p7_TextCNN_predict.py", line 74, in
saver = tf.train.Saver()
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 1338, in init
self.build()
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 1347, in build
self._build(self._filename, build_save=True, build_restore=True)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 1384, in _build
build_save=build_save, build_restore=build_restore)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 835, in _build_internal
restore_sequentially, reshape)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 472, in _AddRestoreOps
restore_sequentially)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 886, in bulk_restore
return io_ops.restore_v2(filename_tensor, names, slices, dtypes)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/gen_io_ops.py", line 1463, in restore_v2
shape_and_slices=shape_and_slices, dtypes=dtypes, name=name)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3392, in create_op
op_def=op_def)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1718, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
NotFoundError (see above for traceback): Key b-1 not found in checkpoint
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_INT32, DT_INT32, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]

Python: 2.7 ... Can you help me figure it out as there is no b-1 key in checkpoint?
Thank you..

Hi,
error come from:
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte
try another way to use pretrain word embedding:
import gensim
from gensim.models import KeyedVectors
word2vec_model = KeyedVectors.load_word2vec_format(word2vec_model_path, binary=True, unicode_errors='ignore')
or set use pretrain word embedding flag to false.

发件人: IronMelter<EMAIL_ADDRESS>发送时间: 2018年6月27日 23:14:43
收件人: brightmart/text_classification
抄送: brightmart; Mention
主题: [brightmart/text_classification] CNN Predict: Key b-1 not found in checkpoint (#65)
Hi @brightmarthttps://github.com/brightmart ,
I have trained the CNN using ''train-zhihu4-only-title-all.txt''. When i am using the predict file for prediction on "test-zhihu6-title-desc.txt" using the word2vec as "zhihu-word2vec-title-desc.bin-100", I am getting the following error:
Traceback (most recent call last):
File "/home/user/bikram/temp/data_util_zhihu.py", line 27, in create_vocabulary
vocabulary_word2index, vocabulary_index2word=pickle.load(data_f)
File "/home/user/anaconda3/lib/python3.6/codecs.py", line 321, in decode
(result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte
Traceback (most recent call last):
File "/home/user/bikram/temp/data_util_zhihu.py", line 69, in create_vocabulary_label
vocabulary_word2index_label, vocabulary_index2word_label=pickle.load(data_f)
File "/home/user/anaconda3/lib/python3.6/codecs.py", line 321, in decode
(result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x80 in position 0: invalid start byte
start padding....
end padding...
Restoring Variables from Checkpoint
2018-06-27 20:40:24.135961: W tensorflow/core/framework/op_kernel.cc:1318] OP_REQUIRES failed at save_restore_v2_ops.cc:184 : Not found: Key b-1 not found in checkpoint
Traceback (most recent call last):
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1322, in _do_call
return fn(*args)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1307, in _run_fn
options, feed_dict, fetch_list, target_list, run_metadata)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1409, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.NotFoundError: Key b-1 not found in checkpoint
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_INT32, DT_INT32, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "p7_TextCNN_predict.py", line 77, in
saver.restore(sess, tf.train.latest_checkpoint(FLAGS.ckpt_dir))
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 1802, in restore
{self.saver_def.filename_tensor_name: save_path})
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 900, in run
run_metadata_ptr)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1135, in _run
feed_dict_tensor, options, run_metadata)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1316, in _do_run
run_metadata)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/client/session.py", line 1335, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.NotFoundError: Key b-1 not found in checkpoint
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_INT32, DT_INT32, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]
Caused by op 'save/RestoreV2', defined at:
File "p7_TextCNN_predict.py", line 74, in
saver = tf.train.Saver()
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 1338, in init
self.build()
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 1347, in build
self._build(self._filename, build_save=True, build_restore=True)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 1384, in _build
build_save=build_save, build_restore=build_restore)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 835, in _build_internal
restore_sequentially, reshape)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 472, in _AddRestoreOps
restore_sequentially)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/training/saver.py", line 886, in bulk_restore
return io_ops.restore_v2(filename_tensor, names, slices, dtypes)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/gen_io_ops.py", line 1463, in restore_v2
shape_and_slices=shape_and_slices, dtypes=dtypes, name=name)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 3392, in create_op
op_def=op_def)
File "/home/user/anaconda3/lib/python3.6/site-packages/tensorflow/python/framework/ops.py", line 1718, in init
self._traceback = self._graph._extract_stack() # pylint: disable=protected-access
NotFoundError (see above for traceback): Key b-1 not found in checkpoint
[[Node: save/RestoreV2 = RestoreV2[dtypes=[DT_FLOAT, DT_INT32, DT_INT32, DT_FLOAT, DT_FLOAT, ..., DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT, DT_FLOAT], _device="/job:localhost/replica:0/task:0/device:CPU:0"](_arg_save/Const_0_0, save/RestoreV2/tensor_names, save/RestoreV2/shape_and_slices)]]

Python: 2.7 ... Can you help me figure it out as there is no b-1 key in checkpoint?
Thank you..
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/brightmart/text_classification/issues/65, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ASuYMNHzd34vhVx9uRFOywADEH1a_Iqdks5uA6FjgaJpZM4U55yN.

Hi,
The 'pretrain word embedding' flag is set to False. Also this was because it was reading from the vocab pickle file in 'r' and 'a' instead of 'rb' and 'ab'. I have made those changes. Now the error is just this much  without the utf-8 error. Any suggestions?
Thanks for your time,
Bikram

yes... will i upload the files and you can try to reproduce them in your system??

ok.

发件人: IronMelter<EMAIL_ADDRESS>发送时间: 2018年6月28日 20:31
收件人: brightmart/text_classification
抄送: brightmart; Mention
主题: Re: [brightmart/text_classification] CNN Predict: Key b-1 not found in checkpoint (#65)
yes... will i upload the files and you can try to reproduce them in your system??
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/brightmart/text_classification/issues/65#issuecomment-401018239, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ASuYMFZdxmt4jgIIhJkSLFopyM4TG7Lxks5uBMyZgaJpZM4U55yN.

Hi @brightmart ,
Thank you for your help. The uploaded files are in the URL:
{ https://anonfile.com/oa3ef0f3bb/data_util.py
https://anonfile.com/p931f4fcb5/p7_TextCNN_predict.py
https://anonfile.com/q13af0fcb9/p7_TextCNN_train.py
https://anonfile.com/r033faf2b3/p8_TextRNN_model.py
https://anonfile.com/s533f5f5b1/p7_TextCNN_model.py
https://anonfile.com/tc3afbfbbe/data_util_zhihu.py
https://anonfile.com/u53af4feb9/p8_TextRNN_train.py }
I have used data_util only in training for the CNN. The training files for both CNN and RNN is either 'train-zhihu4-only-title-all.txt' downloaded from Zhihu url or the 'sample_multiple_label.txt' from your repo.
The RNN is also throwing error while getting trained as key not found. Both the CNN and RNN have slightly different error. The CNN is giving error while predicting and the RNN while training.
Note: The use word embedding is set as False while training in CNN. I am using
Regards,

i am using Tensorflow 1.8.0

@bikramkhastgir “Not found: Key b-1 not found in checkpoint”，
i modify the cnn program for train single label，this error also appears when i  predict,has this error been solved? i need you help.

No @kevinsay ... I couldnt figure out exactly which routine needs Key b-1. So i am still hoping for @brightmart to figure it out..

@kevinsay Hi！Did you solve this problem? I met the same error when i predict...

@bikramkhastgir Hi，I think I got the way to solve this problem.. the filter_nums array is different in train and  predict,  the b's name in  model defined by b-%s, s is the filter_num....

@switchhh Were you able to run the train and the predict for CNN. I can see that the flag 'num_classes' is the same for both as 128.

@switchhh Were you able to run it?
I can see "num_filters" is 128 for both train and predict and "filter_sizes" = [6,7,8]


BBO Number with Validation

Adds a BBO number question with validation. Was originally going to keep this unvalidated in AssemblyLine, but the Appeals Court asked for https://github.com/SuffolkLITLab/docassemble-AppealsCivilDocketingStatement/issues/14, so I thought we didn't want to force verification on everyone for the bootcamp.
The validation is just to ensure that the number is 6 digits, possibly with leading 0's. Also points out what character isn't the digit.
Related to https://github.com/SuffolkLITLab/docassemble-AssemblyLine/pull/23.

I thought so too, but it is feedback directly from Paul https://github.com/SuffolkLITLab/docassemble-AppealsCivilDocketingStatement/issues/14.  It is level 4, so we can probably sit on it for a while.


View IP address of incoming request/broadcast

Is this possible with Cote?

I understand that you wouldn't want to necessarily know how to do do this with cote but I need it for security reasons


How to use the latest electron

It would be good to be able to use the latest version of electron rather than be stuck with 1.7.6, however for some reason the dependency on the special version of zerorpc repo you have kindly provided prevents this - see #23.
I thought I had solved the problem by installing the latest versions of electron etc. and inspired by #22, even the latest "official" zerorpc, which resulted in these dependencies in my package.json:
"devDependencies": {
  "electron": "^1.8.4",
  "electron-packager": "^11.1.0",
  "electron-rebuild": "^1.7.3"
},
"dependencies": {
  "zerorpc": "^0.9.7"
}

After running these commands
brew install zeromq
npm install
./node_modules/.bin/electron-rebuild

I was able to run the example successfully.
Unfortunately when I package the app and run it on a virgin Mac system, libzmq.5.dylib is not found - same issue as https://github.com/zeromq/zeromq.js/issues/131.
Despite the advice in that issue to avoid brew's zeromq and to instead install npm's zeromq with npm i zeromq --save - the problem persists because zerorpc uses zmq which "doesn't provide prebuild statically linked binaries. So you can't package your app with this module". Which presumably is one of the reasons that you made your custom version of zerorpc which changes the source code slightly to require zeromq instead of zmq.
Its all a bit confusing - it seems strange to me that npm zeromq would tolerate being 'unpackagable' because it requires zmq.  And I don't understand why using your custom zerorpc repo locks us into an old version of electron (module mismatch issue).
Aside: The need for specific version numbers all over the readme instructions, and the use of custom repos and old versions of electron might put people off this wonderful project/solution.
Is there a way we can use the latest electron e.g. 1.8.4 with this electron-python solution - in a way which is deployable/packagable?

After discovering that installing and running electron-rebuild fixes module mismatch problems (see #23) I can now successfully use the latest version of Electron.
I still have to use fyear's forked zerorpc repository if I want to successfully package my app. The good news is that the forked repo is up to date (as of March 2018) with the official https://github.com/0rpc/zerorpc-node and presumably can be kept up to date using git, so I'm now fine depending on a non official repo.  The various questions re zerorpc and its dependencies remain for the curious, but no longer essential to understand ;-)
Incidentally, I have also since discovered that brew install zeromq is not needed, as the python pip install zerorpc is sufficient and also provides the zerorpc CLI for testing purposes.


Refactor main.js: down to 450 lines

Move all the torrent event handlers to their own file torrent-controller.js
Move all the file extension checking to the same place

@feross @mathiasvr

LGTM 👍


Does this component support client certificates?

Hi,
I could not find in the docs, looks like it comes from ssl context but I'm still not sure...
In my case I have self-signed HTTPS and CA for client certificates.
What kind of options I have to provide in this case?
Thanks

Hi @korotovsky! Yes, this project supports all TLS (formerly known as SSL) context options that are supported by PHP (http://php.net/manual/en/context.ssl.php), as they are simply passed through to the underlying PHP functions.
To request client certificates on the server side this means that you will need to set the capath and verify_peer=true.
To send client certificates on the client side this means that you will need to set the local_cert (and possibly passphrase).
I hope this helps :+1:


test(cli): add test for get_all_envs extension

Description of changes
Increase test coverage of amplify-cli/extensions.
Description of how you validated changes

lerna --scope @aws-amplify/cli run test

Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[x] Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @togami2864, thank you for the contribution 🎉 ! The team will take a look and review.


Git Commit causes tab change (Phase 2)

This is a repeat of issue #354, and appears to have been reverted as of 5.29.1. As near as I can tell, it is identical: every time I do a commit, the leftmost tab becomes the current tab, not the tab that was opened when I do Cmd-Shift-A-P. Again, it seems to be related to the tab-smart-sort package, and as before, it only seems to happen when tab-smart-sort is set to open preferences on the right.
See #354 for details.

That is odd. I can't replicate the issue this time even with the 'Place Special Tabs On Right' setting.
I'm not quite sure what could be causing this now. The only change in the last update was to prevent opening multiple tabs for the commit message so the code that fixed #354 is still in place.

I apologize for not responding sooner, but this seems to be working fine now.


Prototype support for kotlin-inject

This adds code gen support for @evant's https://github.com/evant/kotlin-inject
Adapting from @joshafeinberg's prior prototyping in https://github.com/joshafeinberg/CircuitKotlinInject
This works by introducing an @MergeCircuitComponent annotation that can be used on a scoped "circuit component" interface that code gen then processes and generates a kotlin-inject-compatible component impl for, aggregating bound UI and presenter factories within the module.
Pros: multiplatform!
Cons: aggregating processor, no cross-module hinting like anvil and hilt do
Example
// Given an existing app component
@Component
abstract class AppComponent {
  // ...
}

// Write a merged circuit component that extends from it
// The generic type is the parent component
// The argument is the scope key
@MergeCircuitComponent(AppScope::class)
abstract class AppScopeCircuitComponent(@Component appComponent: AppComponent) : CircuitComponent

// Usage
val appComponent = AppComponent::class.create()
val circuitComponent = AppScopeCircuitComponent::class.create(appComponent)
val circuit = circuitComponent.newBuilder().build()

Everything else works the same before as far as using @CircuitInject.
TODO

Docs
Tests
API improvements

Should we make a separate runtime for the CircuitComponent and new annotations?
Can we further abstract the code gen mode specifics from the Circuit processor? Currently we do custom handling for assisted types

Inexplicable, generated code doesn't seem able to smart-cast data object types to their type when passing it as an assisted param

Thinking it might be better to leave the parent component to the class itself
// Write a merged circuit component that extends from it
// The argument is the scope key
// Primary constructor params are just copied to the generated impl
@MergeCircuitComponent(AppScope::class)
abstract class AppScopeCircuitComponent(@Component val parent: AppComponent) : CircuitComponent

// Generated impl just copies the primary constructor params and passes them up

I'm not sure this has wheels until https://github.com/evant/kotlin-inject/issues/351 is available. I'm also not able to figure out a way to get past this hurdle of deferring generation until kotlin-inject runs 🤔. I suspect there may be a problem with both plugins deferring.

Closing for now, can revisit if there are upstream changes that unlock this


Add StructInheritance cop

Overall the code looks good (apart from my remarks). Next step - checking for classes that can be replaced by a struct. :-)

@bbatsov could you give an example of what you mean? :smile:

I meant that once we're ready with this cop we could create another one checking for code like:
class Person
  attr_accessor :age, :name

  def initialize(age, name)
    ...
  end
end

We could search for more complex patterns, but I think we shouldn't overdo this. Anyways, this was just a random comment. Let's wrap this PR first.

@bbatsov changed the message to match the updated one in ruby-style-guide.

:+1: Thanks!

There's a small gotcha with Struct.new, which I think this cop doesn't handle:
Person = Struct.new(:first, :last) do
  SEPARATOR = ' '.freeze
  def name
    [first, last].join(SEPARATOR)
  end
end

is not equivalent to:
class Person < Struct.new(:first, :last)
  SEPARATOR = ' '.freeze
  def name
    [first, last].join(SEPARATOR)
  end
end

The former creates ::Person and ::SEPARATOR, while the latter creates ::Person and ::Person::SEPARATOR.

Why is it better not to inherit from Struct.new?

@mocoso:

Don't extend an instance initialized by Struct.new. Extending it introduces a superfluous class level and may also introduce weird errors if the file is required multiple times.

https://github.com/bbatsov/ruby-style-guide#no-extend-struct-new

@Koronen Thanks for the clarification (and for your very polite encouragement for me to read the docs) :smile:

Just upgraded and got bit by this cop.  Can the rule be updated to only address empty class blocks (e.g. no additional methods or constants)?  As @tamird alluded to, there are valid use cases for extending Struct.new.  Or at the very least, add more details about why this cop should be included and on by default.

There's a link to the style guide already.

@bbatsov Yes, I saw that, and it didn't clarify anything.

Guess you've never seen Struct.new do ...? Check out the official docs http://ruby-doc.org/core-2.2.0/Struct.html

I've seen that too -- it was in one of the examples earlier in this thread.  But as was already pointed out, it's not equivalent.
I'm just looking for a reason for the hate for this obviously useful pattern.

I understand this well enough. Perhaps we should account for the constant definition, but outside of this corner-case the two setups are pretty much the same and it's generally better to prefer the simpler option (simpler inheritance chain).

I just found this interesting article about defining constants inside a Struct: http://blog.55minutes.com/2012/01/trivia-constants-in-ruby-192-vs-193/
I think I agree with its conclusion: maybe you should just use a simple Class instead.

Sounds good to me. @bquorning Please, log a ticket so we don't forget to implement a check for constants defined in structs eventually.

I was unaware of the perils of constants defined inside blocks, and I agree that constants inside blocks are definitely a risky choice and should be discouraged.  I'm not convinced the author of that article knew about the < Struct.new pattern, so I wouldn't take his failure to mention it as a recommendation against it.


flannel logging lots of L3 Miss

I'm running a kubernetes cluster with flannel/vxlan and I'm seeing a lot of these messages in the logs.  Are these messages benign?  Could not find anywhere mentioning this.  If they're benign it'd be cool if it didn't sound as much like an error...  If it's an error, any idea why?  I setup the cluster with kube-aws pretty recently but I've been seeing this error for a while and then decided to find out why.  Thanks for any help!
I0126 19:12:33.583598 01411 network.go:225] L3 miss: <IP_ADDRESS>

bueller?

It's not an error. The logging in this area has been improved in v0.7.0

Ok great thanks!


Enable the Utility Service by default.

So that it works like this in the different environments:

C++ CODAL programmes: Enabled by default
MicroPython: Enabled by default on pairing mode (BLE only enabled in pairing mode)
MakeCode: To be updated to disable this flags in normal programmes to save some memory, and adding the datalog extension enables them again

Part of:

https://github.com/lancaster-university/codal-microbit-v2/issues/354

This PR is no longer necessary, more info in:

https://github.com/lancaster-university/codal-microbit-v2/issues/354#issuecomment-2023441540


Fixed error messages bug in registration form

Description
This PR fixes a bug we had on the registration form.
When entering an invalid password, for any reason (too common, too short, not strong enough, etc...), the error message that was shown was 'password mismatch'.
Now it will show the correct messages
Close #96
Before

After

Squashed the commits.


Updating dependencies

:tada: This PR is included in version 3.3.1 :tada:
Your semantic-release bot :package::rocket:


Move data-access crate into datafusion directory

Which issue does this PR close?

Closes https://github.com/apache/arrow-datafusion/issues/2424
Rationale for this change

Make the project more consistent. All of the other datafusion sub-crates are under the datafusion directory and there appears to be no reason for data-access to be different.
What changes are included in this PR?

Move data-access crate into datafusion directory
Bump data-access version from 1.0.0 to 7.0.0 for consistency with other crates
Update release scripts

Are there any user-facing changes?

No

nice, LGTM

Hi @andygrove, according to #2076, the reason to move the data-access crate out of the datafusion directory is to make it possible for other crates, like datafusion-objectstore-s3 and datafusion-objectstore-hdfs, to depend on only this crate rather than the whole datafusion crate. Then the datafusion crate may also be able to depend on the object store extensions.
In summary, the data-access crate should not depend on the datafusion. The dependencies should be like follows:
datafusion
     |        \
     |          \
     |            datafusion-objectstore-extensions
     |            /
     |          /
data-access

it looks like this PR just change the data-access code location (for easier crate building). But it doesn't change dependencies.

it looks like this PR just change the data-access code location (for easier crate building). So it doesn't change dependencies

Yes,  that is correct. This PR just moves the code and bumps the version from 1.0.0 to 7.0.0 for consistency with other DataFusion crates.

Thanks for the review @yjshen


feat: add ability to set location

reuse location

Codecov Report

Merging #143 (78b160d) into master (ad2d28a) will decrease coverage by 0.27%.
The diff coverage is 38.46%.

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
- Coverage   99.89%   99.62%   -0.28%
==========================================
  Files          18       18
  Lines        2897     2910      +13
==========================================
+ Hits         2894     2899       +5
- Misses          2        8       +6
- Partials        1        3       +2

Impacted Files
Coverage Δ

errors.go
83.33% <0.00%> (-16.67%)
:arrow_down:

setter.go
96.90% <45.45%> (-3.10%)
:arrow_down:

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 ad2d28a...78b160d. Read the comment docs.

Insufficient unit test coverage of errors.go and setter.go

大哥，可以合并吗，还是要继续补充单测？我已经写了，感觉没啥可补的了呀


Disable test --name filter in the upgrade plugin discover

The fresh implementation of the upgrade execute plugin in #1196 does not support filtering tests using the test --name filter. Here's a short real life example from the tmt repo:
tmt run -av plan -n core test -n smoke execute -h upgrade -u https://github.com/teemtee/upgrade/ -t prepare

The test -n filter is applied to the upgrade tasks as well. We need to find a way how to disable the global cli test filter for upgrade tasks which is implemented here:
https://github.com/teemtee/tmt/blob/7b73789624b27c88e8fcc318940fd45cf4ac1f5d/tmt/base.py#L1329

We initially thought that passing some option to DiscoverFmf.go() and some ifs would be enough but that is not very systematic. I think we need to modify Test._opt such that if some true flag on Test is set, it returns the default value (None or empty list) rather than the actual value. Any other similar approach?


feat(virtualbursting): add limits to the configuration table

This adds the virtual bursting limit settings to the configuration table.
CLOSES DCOS-62250
Testing

Trade-offs

Dependencies

Screenshots

:tada: This PR is included in version 3.1.0-vertical-bursting-snapshot.2 :tada:
The release is available on:

v3.1.0-vertical-bursting-snapshot.2
GitHub release

Your semantic-release bot :package::rocket:


[TFLite] Feature request: Add support for DepthToSpace op

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Ubuntu 16.04
TensorFlow installed from (source or binary):source
TensorFlow version (or github SHA if from source):1.13

Provide the text output from tflite_convert
...  Converting unsupported operation: DepthToSpace

Any other info / logs
DepthToSpace op has been well used for implementing subpixel convolution (aka pixel shuffle) like this super-resolution task.
I know SELECT_TF_OPS option. But its binary size is too huge for mobile application.
Would you like to add DepthToSpace to built-in ops?

@aselle @stakemura  Hi, I am interested in using the depth_to_space op. Is there a workaround for this in the meantime till it is added to the list of supported ops? I am able to convert my model to tflite by specifying
converter.target_ops = [tf.lite.OpsSet.TFLITE_BUILTINS,tf.lite.OpsSet.SELECT_TF_OPS]
converter.allow_custom_ops = True # this is for DEPTH_TO_SPACE

However, the interpreter fails with the error:
ValueError: Didn't find custom op for name 'DEPTH_TO_SPACE' with version 1
Registration failed.

Hey! When will DepthToSpace be added?

If depth_to_space isn’t supported even now, Can you tell me an alternative to depth_to_space which I can convert to tflite.

@stakemura,
Sorry for the delayed response. Can you please let us know if tfl.depth_to_space (TFL::DepthToSpaceOp) is the functionality that you are looking for? Thanks!


refactor to move an ios specific class out of globals.dart

refactor to move an ios specific class out of globals.dart

@yjbanov

LGTM


[BOT PR] to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: istio-testing
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
If they are not already assigned, you can assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
25044f0348204415ab142c7195fcedb3f5352bee
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
25044f0348204415ab142c7195fcedb3f5352bee
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
25044f0348204415ab142c7195fcedb3f5352bee
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[0.69][WIP] Fix S3 addon selector and and deauth modals [#OSF-6156]

Purpose
Fix JS console errors that occurred after authorizing S3 addon on a project settings page.
Also identified and fixed some related errors on user settings page when deauthorizing an addon for a specific node.
Thanks to Nici for the help getting S3 working locally.
Changes

Fix s3 addon for node settings page
Fix "Remove addon" modal dialogs on the user settings page; should now be able to remove an addon by clicking the red "x" next to a project name, without the "remove" button generating JS errors. Special cases addressed in this PR are noted as separate items:

Dataverse
S3
Settings for other addons

Side effects
None expected. Let me know if any addon-related buttons are broken on either the project or user settings "configure addons" pages.
Ticket
https://openscience.atlassian.net/browse/OSF-6156

The parent account "disconnect" buttons also have problems; investigating and will bundle fix into this PR.

Tested out that this works! Looks good to me @abought! 🌲


use with import

This:
import Promise from 'promise-polyfill';
window.Promise = Promise;
import 'whatwg-fetch';

Fails because imports are hoisted, and window.Promise will be undefined when whatwg-fetch gets imported.
Would you be interested in fixing this? I can submit a pull request.

There isn't a good solution for this in a single file. My suggestion is to create 2 files.
One with:
./polyfill.js
import Promise from 'promise-polyfill';
window.Promise = Promise;

Then you can do
import './polyfill';
import 'whatwg-fetch';

Thanks. I ended up using a copy of promise-polyfill, modifying it so it worked like the 'whatwg-fetch' polyfill (in other words, with "import './promise-polyfill'".
You probably have a good reason to export the polyfill when it's used as a module, but that doesn't really fit my use case.
Thank you for your work on this. It's a really nice implementation for Promise. I'm using it to support IE 11.

It was a tough decision. Originally this library polyfilled by default, but I found users who didn't want the global state changed by default. If I don't polyfill, users always have the option to import and do it themselves. If I do it by default, there isn't a good way to turn it off.
I think the best options is to provide both options. If I get some time, I will create a polyfill file as well.
Thanks for your feedback. I appreciate you using the library.


What advantages over competition?

I read your flux evolution blog post (very good stuff) and have been looking at the other flux implementations, so far microcosm is looking the best to me. I still have to do research on some of the others you listed but after looking at this projects README I'm not sure I understand what this provides that the other flux implementations you mention don't provide. Could you add a section to the README explaining?

Maybe no particular advantages, I don't know :-).
It's just something I can play with for my talk at React Europe.
The main goal of this library is to prove that Flux can be implemented in a way compatible with full hot reloading (and explore how this can be done). You can run the example code with npm start, change action creators or stores, and the new logic will kick in before you refresh.

Thanks, just trying to decide which library to use as I think your right in calling this an interesting evolution of flux.

For now, Redux is more of an exploration project than a production-ready Flux lib. It may evolve into one later, but no sooner than a few months. You are free to try it and compare with other approaches though! The API is subject to change.

@gaearon I was thinking about giving some talk about Redux in the next Israel React Meetup, so if you are willing to share some of your contents from the React-Europe conference it can be awesome :-)

@asaf When is your talk?

@gaearon I think the next is not even set yet, so there's enough time, if the content after your talk is going to be public then share it among us :-)


No support for CURLOPT_CAINFO

Need to be able to support CURLOPT_CAINFO to allow loading of cacert file.  In some cases, this is needed, in my case it is needed while running on a windows machine.  php curl on Windows seems to have a problem with SSL requests unless CURLOPT_CAINFO is specified.

You can use the 'cert'request option: http://docs.guzzlephp.org/en/latest/request-options.html#cert

I actually tried that and it didn't work.  I thought it was for
something else.  The terminology doesn't match with curl options.
Here is how I called it:
$res = $client->request('GET', ZDURL . '/organizations.json', ['auth' =>
[ZDUSER . '/token', ZDAPIKEY], 'cert' =>
"C:\Users\Jose\Documents\Development\Php\EnsimZendesk\cacert.pem"]);
Here is the error:
Fatal error: Uncaught exception
'GuzzleHttp\Exception\RequestException' with message 'cURL error 58:
unable to set private key file:
'C:\Users\Jose\Documents\Development\Php\EnsimZendesk\cacert.pem' type
PEM (see http://curl.haxx.se/libcurl/c/libcurl-errors.html)' in
C:\Users\Jose\Documents\Development\Php\EnsimZendesk\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php:187
Stack trace: #0
C:\Users\Jose\Documents\Development\Php\EnsimZendesk\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(150):
GuzzleHttp\Handler\CurlFactory::createRejection(Object(GuzzleHttp\Handler\EasyHandle),
Array) #1
C:\Users\Jose\Documents\Development\Php\EnsimZendesk\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php(103):
GuzzleHttp\Handler\CurlFactory::finishError(Object(GuzzleHttp\Handler\CurlMultiHandler),
Object(GuzzleHttp\Handler\EasyHandle),
Object(GuzzleHttp\Handler\CurlFactory)) #2
C:\Users\Jose\Documents\Development\Php\EnsimZendesk\vendor\guzzlehttp\guzzle\src\Handler\CurlMultiHandler.php(180):
GuzzleHttp\Handler\CurlFactory::fi in
C:\Users\Jose\Documents\Development\Php\EnsimZendesk\vendor\guzzlehttp\guzzle\src\Handler\CurlFactory.php
on line *187

What does work is if I modify CurlFactory.php directly, I add a single
line to getDefaultConf, but obviously not a permanent solution.
 private function getDefaultConf(EasyHandle $easy)
 {
     $conf = [
         '_headers'             => $easy->request->getHeaders(),
         CURLOPT_CUSTOMREQUEST  => $easy->request->getMethod(),
         CURLOPT_URL            => (string) $easy->request->getUri(),
         CURLOPT_RETURNTRANSFER => false,
         CURLOPT_CAINFO         =>

'C:\Users\Jose\Documents\Development\Php\EnsimZendesk\cacert.pem',
CURLOPT_HEADER         => false,
CURLOPT_CONNECTTIMEOUT => 150,
On 11/11/2015 10:25 PM, Jeremy Lindblom wrote:

You can use the |'cert'|request option:
http://docs.guzzlephp.org/en/latest/request-options.html#cert
—
Reply to this email directly or view it on GitHub
https://github.com/guzzle/guzzle/issues/1306#issuecomment-156013533.

I looked at the code further, and the 'cert' option only sets the
CURLOPT_SSLCERT option.  This is completely different than the
CURLOPT_CAINFO option.  Further, a global search in the guzzle source
code shows that nowhere is CURLOPT_CAINFO set.
I think this needs to be supported in future version.
Thanks,
Jose
On 11/11/2015 10:25 PM, Jeremy Lindblom wrote:

You can use the |'cert'|request option:
http://docs.guzzlephp.org/en/latest/request-options.html#cert
—
Reply to this email directly or view it on GitHub
https://github.com/guzzle/guzzle/issues/1306#issuecomment-156013533.

I added this code to applyHandlerOptions in CurlFactory.php and now it works by specifying 'cainfo' option.  Probably should iimplement CURLOPT_CAPATH as well.
    if (isset($options['cainfo'])) {
        $cainfo = $options['cainfo'];
        if (!file_exists($cainfo)) {
            throw new \InvalidArgumentException(
                "CAINFO certificate not found: {$cainfo}"
            );
        }
        $conf[CURLOPT_CAINFO] = $cainfo;
    }

Oh, my bad on 'cert', I was actually thinking of the 'verify' option. That should work, but there is also a way to specify arbitrary curl options as well.
Those are both "request options", but can be applied to every request as explained in the bottom of the client creation section, "All other options passed to the constructor are used as default request options with every request created by the client."

Yep, this is done using verify.

solution1: disable ssl verification
$client->request('GET', '/', [
    'curl' => [
        CURLOPT_SSL_VERIFYPEER => false,
	CURLOPT_SSL_VERIFYHOST => false,
    ],
    //other parameters
    'headers' => [
		'Authorization' => 'Basic '.$this->token,
    ],
    'multipart' => [
		[
			'name'     => 'file',
			'contents' => fopen($uploadFilePath, 'r')
		],
    ]
]);

solution2: specify a cacert.pem file
$client->request('GET', '/', [
    'curl' => [
        CURLOPT_CAINFO => '/path/to/cacert.pem'
    ],
    //other parameters
    'headers' => [
		'Authorization' => 'Basic '.$this->token,
    ],
    'multipart' => [
		[
			'name'     => 'file',
			'contents' => fopen($uploadFilePath, 'r')
		],
    ]
]);

sulation3: specify default cainfo in php.ini
[curl]
; A default value for the CURLOPT_CAINFO option. This is required to be an
; absolute path.
curl.cainfo = /path/to/cacert.pem

I'm not sure if it needs a double quote like this curl.cainfo = "/path/to/cacert.pem"


added homework-6

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


create a dark theme for the website

We use mkdocs’ Terminal theme, and it does not have a theme selection option.
However, mkdocs allows adding custom css and custom javascript.
We can create a css that overrides the default theme; and switch to the dark theme based on user preferences.
When the user clicks a theme switch button, we shall switch to whichever theme the user picks and remember it.
When there is not default theme selection, it should default to the system theme.

Kuzey did this; I’ll close this one it’s tested.

Done.


Add new explorer: thetangle.org

Hi @viossat ,
Thanks for sharing a site.
This site does not work on Safari. Maybe someone who is owner of site will fix it

It works on my side with Safari 10.1.2 and Safari on iOS 10.3.3.

@ertugrulyilmaz , please reconsider this PR. An issue with localStorage has been fixed.

hi @viossat
i added link to tools section.
thanks.


ModuleNotFoundError inside deep.py

While using deeppavlov 0.0.8 package I get the following error when importing find_config from deeppavlov.deep:
File "<removed>", line 16, in <module>
    from deeppavlov.deep import find_config'

File "<removed>/venv/lib/python3.6/site-packages/deeppavlov/deep.py", line 26, in <module>
    from utils.telegram_utils.telegram_ui import interact_model_by_telegram

ModuleNotFoundError: No module named 'utils.telegram_utils'; 'utils' is not a package

The import seems fine in the Github repository at tag 0.0.8 so I'm wondering what problem occurs here. It seems that it is an DeepPavlov issue. But of course please let me know if I'm doing anything wrong here :)

Hi!
Have you installed the package or are you just working with code from github?
You can install it in development mode with pip3 install -e . from the repository root directory.

So I just

created and activated a new virtual environment
pip install deeppavlov
python -c 'from utils.telegram_utils.telegram_ui import interact_model_by_telegram; from deeppavlov.deep import find_config'

and got no errors.
Do you have a way to reproduce your exception?

Found it. Trivial problem. The package utils and deeppavlov are both top-level imports. I had one source file called utils.py causing the utils package to be overridden. Had to rename my utils.py and completely restart the project. Only removing virtual environment was not enough.
Thanks for your help!


X cut doesn't match what's on screen

Describe the bug
Performing an X-Cut in a raster appears to go to the next trace instead of the current one.
To Reproduce
Steps to reproduce the behavior:

Create a raster plot with 3 distinct traces.  By default the Y axis should be numbered 0, .5, 1, 1.5, 2, 2.5, 3.
Verify the first trace is a rectangle starting at 0 and ending at 1 on the Y axis, and the second is from 1 to 2 and third from 2 to 3.

Expected behavior
I would think the X-Cut should match the trace under the cursor when you press the X key, it appears to round up so at 0.5 Y index it prints trace 1 and at 1.5 Y index it prints trace 2.
jsFiddle
If possible, demonstrate the issue by forking this jsFiddle and providing a link to your jsFiddle in the bug report.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

sorry, I should have explained better.  In my case I don't have a lot of raster "lines"; as an example, in a plot there may only  be 6 raster lines so each line is about 1 inch wide on my screen.  If I'm at the top of the 1 inch wide line it shows the x cut properly, if I'm anywhere after 50% into the 1 inch line it shows the x cut for the NEXT raster line.  I do use this a lot.  Most of my plots contain less than 20 lines and since the window is about 10 inches tall on my display, that's a 1/2 inch wide line.  Most of my users will put the curser in vertical and horizontal center of the line which means they will often be viewing the next line when doing an X cut.  I've included a REALLY BAD powerpoint diagram trying to show the issue. I can't upload a real plot for security reasons.  Sorry.  BTW, thanks for the follow up.

@kmun thanks for the picture and description.  I have identified the source
of the issue and will fix it soon.
On Tue, May 18, 2021 at 8:08 AM kmun @.***> wrote:

sorry, I should have explained better. In my case I don't have a lot of
raster "lines"; as an example, in a plot there may only be 6 raster lines
so each line is about 1 inch wide on my screen. If I'm at the top of the 1
inch wide line it shows the x cut properly, if I'm anywhere after 50% into
the 1 inch line it shows the x cut for the NEXT raster line. I do use this
a lot. Most of my plots contain less than 20 lines and since the window is
about 10 inches tall on my display, that's a 1/2 inch wide line. Most of my
users will put the curser in vertical and horizontal center of the line
which means they will often be viewing the next line when doing an X cut.
I've included a REALLY BAD powerpoint diagram trying to show the issue. I
can't upload a real plot for security reasons. Sorry. BTW, thanks for the
follow up.
[image: Example of X-Cut issue]
https://user-images.githubusercontent.com/45660362/118647876-d7a4e580-b7af-11eb-9945-d76e1b422158.jpg
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LGSInnovations/sigplot/issues/103#issuecomment-843113009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABYEIRWANL3YNP3T4NHHSTTOJKCTANCNFSM44C4RLDA
.

@kmun I don't know what version of SigPlot you are running, but I have fixed the issue here:
https://github.com/spectriclabs/sigplot/commit/01b06a046be13d39fb15dfd44a3d4c72b9f0e84c
If you are able to run the master branch from spectriclabs/sigplot then it will include the fix.
The good news is, the fix is very easy, so you should be able to port it into whatever version you are running by altering these lines:
https://github.com/LGSInnovations/sigplot/blob/480cec393280a41359924383d06bb6849eeab879/js/sigplot.js#L3540
https://github.com/LGSInnovations/sigplot/blob/480cec393280a41359924383d06bb6849eeab879/js/sigplot.js#L3684


Add more exercises

Add Difference of Squares, Raindrops, and Clock.

Yeap. You are totally right. :)


RUSTSEC-2021-0064: cpuid-bool has been renamed to cpufeatures

cpuid-bool has been renamed to cpufeatures

Details

Status
unmaintained

Package
cpuid-bool

Version
0.2.0

URL
https://github.com/RustCrypto/utils/pull/381

Date
2021-05-06

Please use the `cpufeatures`` crate going forward:
<https://github.com/RustCrypto/utils/tree/master/cpufeatures>
There will be no further releases of cpuid-bool.
See advisory page for additional details.

Closing as I can't find this dependency in the app.


[HUDI-3664] Handle type conversion for comparison of column range metadata

What is the purpose of the pull request
Fixes an issue with comparison of column range metadata. Instead of using string comparators, we get the type of field from the avro schema and convert to native Java type.
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / 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:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Closing it in favor of #5181


Do not fail on 'Invalid comparator' error

Trying to install definitions with flow-typed install I faced an Invalid comparator error due to GitHub repo dependency in my package.json file.
This solution avoids Invalid comparator errors by ignoring non-semver packages and and installing stubs for them.
Closes #1127

@cema-sp to get this to work for me I had to modify npmLibDefs.js here with the below snippet. Not sure why I needed this in addition to your changes.
if (filter.pkgName.toLowerCase() === def.name.toLowerCase()) {
          try {
            filterMatch = pkgVersionMatch(filter.pkgVersion, def.version);
          } catch(error) {
            if (error.message.indexOf('Invalid comparator') > -1) {
              console.error(
                `\u2022 Unprocessible version of '${filter.pkgName}' ` +
                `in package.json: '${filter.pkgVersion}'.`
              );
            } else {
              throw error;
            }
          }
        }

Hi there, I have been manually removing the github repos in my package.json whenever I need to flow-typed install, which is a bit of a pain.  This is a fairly significant bug in my mind, and it would be awesome to get this merged and released.  Looks like it's been sitting for a while. @cema-sp will you be updating this to incorporate @vinnymac's patch?  If not, I can copy this work and create another PR, because I'd really love to get this fixed.

@IanVS I would like to see these changes get merged in as well. I actually don't use flow at all right now, waiting till bugs such as this get fixed. If their is anything I can do to help, let me know.

@cema-sp There's a new feature issue at #1127 for this exact bug, and a number of other ones. If you can rebase this we can probably get it merged.

Apologize for such a delay.
I found a better and more straightforward solution, please review it.
PR is already rebased.
Resolves #1127

I appreciate your patience, thanks again for the fix 💪


Not loaded

223 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: ree_jp\stackStorage\libs\poggit\libasynql\ConfigException: "Config problem: Database settings are missing or incorrect" (EXCEPTION) in "plugins/StackStrage.phar/src/ree_jp/stackStorage/libs/poggit/libasynql/libasynql" at line 77
224 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #0 plugins/StackStrage.phar/src/ree_jp/stackStorage/sql/StackStorageHelper(29): ree_jp\stackStorage\libs\poggit\libasynql\libasynql::create(object ree_jp\StackStorage\StackStoragePlugin, boolean , array[2])
225 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #1 plugins/StackStrage.phar/src/ree_jp/stackStorage/StackStoragePlugin(30): ree_jp\stackStorage\sql\StackStorageHelper->__construct(object ree_jp\StackStorage\StackStoragePlugin, string[76] /home/gameserver/servers/serveur7/minecraftbedrock/plugin_data/StackStorage/)
226 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #2 pmsrc/src/pocketmine/plugin/PluginBase(116): ree_jp\StackStorage\StackStoragePlugin->onEnable()
227 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #3 pmsrc/src/pocketmine/plugin/PluginManager(552): pocketmine\plugin\PluginBase->setEnabled(boolean 1)
228 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #4 pmsrc/src/pocketmine/Server(1785): pocketmine\plugin\PluginManager->enablePlugin(object ree_jp\StackStorage\StackStoragePlugin)
229 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #5 pmsrc/src/pocketmine/Server(1771): pocketmine\Server->enablePlugin(object ree_jp\StackStorage\StackStoragePlugin)
230 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #6 pmsrc/src/pocketmine/Server(1584): pocketmine\Server->enablePlugins(integer 1)
231 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #7 pmsrc/src/pocketmine/PocketMine(314): pocketmine\Server->__construct(object BaseClassLoader, object pocketmine\utils\MainLogger, string[51] /home/gameserver/servers/serveur7/minecraftbedrock/, string[59] /home/gameserver/servers/serveur7/minecraftbedrock/plugins/)
232 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #8 pmsrc/src/pocketmine/PocketMine(344): pocketmine\server()
233 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #9 pmsrc(11): require(string[106] phar:///home/gameserver/servers/serveur7/minecraftbedrock/PocketMine-MP.phar/src)
  | 1 line
235 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: Error: "Typed static property ree_jp\stackStorage\sql\StackStorageHelper::$instance must not be accessed before initialization" (EXCEPTION) in "plugins/StackStrage.phar/src/ree_jp/stackStorage/StackStoragePlugin" at line 52
236 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #0 pmsrc/src/pocketmine/plugin/PluginBase(118): ree_jp\StackStorage\StackStoragePlugin->onDisable()
237 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #1 pmsrc/src/pocketmine/plugin/PluginManager(639): pocketmine\plugin\PluginBase->setEnabled(boolean )
238 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #2 pmsrc/src/pocketmine/plugin/PluginManager(559): pocketmine\plugin\PluginManager->disablePlugin(object ree_jp\StackStorage\StackStoragePlugin)
239 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #3 pmsrc/src/pocketmine/Server(1785): pocketmine\plugin\PluginManager->enablePlugin(object ree_jp\StackStorage\StackStoragePlugin)
240 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #4 pmsrc/src/pocketmine/Server(1771): pocketmine\Server->enablePlugin(object ree_jp\StackStorage\StackStoragePlugin)
241 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #5 pmsrc/src/pocketmine/Server(1584): pocketmine\Server->enablePlugins(integer 1)
242 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #6 pmsrc/src/pocketmine/PocketMine(314): pocketmine\Server->__construct(object BaseClassLoader, object pocketmine\utils\MainLogger, string[51] /home/gameserver/servers/serveur7/minecraftbedrock/, string[59] /home/gameserver/servers/serveur7/minecraftbedrock/plugins/)
243 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #7 pmsrc/src/pocketmine/PocketMine(344): pocketmine\server()
244 | 2021-09-14 [00:21:24] [Server thread/CRITICAL]: #8 pmsrc(11): require(string[106] phar:///home/gameserver/servers/serveur7/minecraftbedrock/PocketMine-MP.phar/src)

Did you set up sql.yml correctly?
https://poggit.pmmp.io/p/StackStorage/1.0.1#rdesc-section-config

I'm sorry
I made a typo with sql.yaml in sql.yml
Rename the file to sql.yml and try again

Don’t work i don’t understand i must place
database:
mysql or sqlite
type: mysql
# Edit these settings only if you choose "sqlite".

sqlite:
# The file name of the database in the plugin data folder.
You can also put an absolute path here.
file: data.sqlite

Edit these settings only if you choose "mysql".
mysql:
host: <IP_ADDRESS>
# Avoid using the "root" user for security reasons.
username: StackStorage
password: password
# Database name
schema: StackStorage
The maximum number of simultaneous SQL queries
Recommended: 1 for sqlite, 2 for MySQL. You may want to further increase this value if your MySQL connection is very slow.
worker-limit: 2
In sql.yml

Bro i don’t understand your plugin doesn’t work but v0.7.0 work but i have lots of bugs i want to download 1.0.1 but not loaded

Can you send me a valid config please !

2021-09-25 [14:37:08] [Server thread/CRITICAL]: ree_jp\stackStorage\libs\poggit\libasynql\SqlError: "SQL CONNECT error: Connection refused" (EXCEPTION) in "plugins/StackStrage.phar/src/ree_jp/stackStorage/libs/poggit/libasynql/libasynql" at line 141
182 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #0 plugins/StackStrage.phar/src/ree_jp/stackStorage/sql/StackStorageHelper(27): ree_jp\stackStorage\libs\poggit\libasynql\libasynql::create(string[46] object(ree_jp\StackStorage\StackStoragePlugin), array[4], array[2])
183 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #1 plugins/StackStrage.phar/src/ree_jp/stackStorage/StackStoragePlugin(30): ree_jp\stackStorage\sql\StackStorageHelper->__construct(string[46] object(ree_jp\StackStorage\StackStoragePlugin), string[76] /home/gameserver/servers/serveur7/minecraftbedrock/plugin_data/StackStorage/)
184 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #2 pmsrc/src/pocketmine/plugin/PluginBase(116): ree_jp\StackStorage\StackStoragePlugin->onEnable()
185 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #3 pmsrc/src/pocketmine/plugin/PluginManager(552): pocketmine\plugin\PluginBase->setEnabled(boolean 1)
186 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #4 pmsrc/src/pocketmine/Server(1785): pocketmine\plugin\PluginManager->enablePlugin(string[46] object(ree_jp\StackStorage\StackStoragePlugin))
187 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #5 pmsrc/src/pocketmine/Server(1771): pocketmine\Server->enablePlugin(string[46] object(ree_jp\StackStorage\StackStoragePlugin))
188 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #6 pmsrc/src/pocketmine/Server(1584): pocketmine\Server->enablePlugins(integer 1)
189 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #7 pmsrc/src/pocketmine/PocketMine(314): pocketmine\Server->__construct(string[23] object(BaseClassLoader), string[35] object(pocketmine\utils\MainLogger), string[51] /home/gameserver/servers/serveur7/minecraftbedrock/, string[59] /home/gameserver/servers/serveur7/minecraftbedrock/plugins/)
190 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #8 pmsrc/src/pocketmine/PocketMine(344): pocketmine\server()
191 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #9 pmsrc(11): require(string[106] phar:///home/gameserver/servers/serveur7/minecraftbedrock/PocketMine-MP.phar/src)
  | 1 line
193 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: Error: "Typed static property ree_jp\stackStorage\sql\StackStorageHelper::$instance must not be accessed before initialization" (EXCEPTION) in "plugins/StackStrage.phar/src/ree_jp/stackStorage/StackStoragePlugin" at line 52
194 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #0 pmsrc/src/pocketmine/plugin/PluginBase(118): ree_jp\StackStorage\StackStoragePlugin->onDisable()
195 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #1 pmsrc/src/pocketmine/plugin/PluginManager(639): pocketmine\plugin\PluginBase->setEnabled(boolean )
196 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #2 pmsrc/src/pocketmine/plugin/PluginManager(559): pocketmine\plugin\PluginManager->disablePlugin(object ree_jp\StackStorage\StackStoragePlugin)
197 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #3 pmsrc/src/pocketmine/Server(1785): pocketmine\plugin\PluginManager->enablePlugin(object ree_jp\StackStorage\StackStoragePlugin)
198 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #4 pmsrc/src/pocketmine/Server(1771): pocketmine\Server->enablePlugin(object ree_jp\StackStorage\StackStoragePlugin)
199 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #5 pmsrc/src/pocketmine/Server(1584): pocketmine\Server->enablePlugins(integer 1)
200 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #6 pmsrc/src/pocketmine/PocketMine(314): pocketmine\Server->__construct(object BaseClassLoader, object pocketmine\utils\MainLogger, string[51] /home/gameserver/servers/serveur7/minecraftbedrock/, string[59] /home/gameserver/servers/serveur7/minecraftbedrock/plugins/)
201 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #7 pmsrc/src/pocketmine/PocketMine(344): pocketmine\server()
202 | 2021-09-25 [14:37:08] [Server thread/CRITICAL]: #8 pmsrc(11): require(string[106] phar:///home/gameserver/servers/serveur7/minecraftbedrock/PocketMine-MP.phar/src)

sql.zip
Show me your sql.yml

Thank’s i will try,
My sql :
first try:
--- []
...
second try:

database:
mysql or sqlite
type: mysql
# Edit these settings only if you choose "sqlite".

sqlite:
# The file name of the database in the plugin data folder.
You can also put an absolute path here.
file: data.sqlite

Edit these settings only if you choose "mysql".
mysql:
host: <IP_ADDRESS>
# Avoid using the "root" user for security reasons.
username: StackStorage
password: password
# Database name
schema: StackStorage
The maximum number of simultaneous SQL queries
Recommended: 1 for sqlite, 2 for MySQL. You may want to further increase this value if your MySQL connection is very slow.
worker-limit: 2
...

This working with your config !! You can place your config in src i think .. ?

Bro i don’t understand why stackstorage don’t open the storage but create double chest on top of my player it’s according to my position because in certain place, stack storage work ……

2021-09-25 [15:06:04] [Server thread/CRITICAL]: InvalidStateException: "No free window IDs found" (EXCEPTION) in "pmsrc/src/pocketmine/Player" at line 4108
261 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #0 plugins/StackStrage.phar/src/ree_jp/stackStorage/api/GuiAPI(40): pocketmine\Player->addWindow(object ree_jp\stackStorage\virtual\VirtualStackStorage)
262 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #1 plugins/StackStrage.phar/src/ree_jp/stackStorage/gui/StackStorage(50): ree_jp\stackStorage\api\GuiAPI->sendGui(string[7] MadoxMc, object ree_jp\stackStorage\virtual\VirtualStackStorage)
263 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #2 pmsrc/src/pocketmine/scheduler/ClosureTask(61): ree_jp\stackStorage\gui\StackStorage->ree_jp\stackStorage\gui{closure}(integer 14306)
264 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #3 pmsrc/src/pocketmine/scheduler/TaskHandler(137): pocketmine\scheduler\ClosureTask->onRun(integer 14306)
265 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #4 pmsrc/src/pocketmine/scheduler/TaskScheduler(179): pocketmine\scheduler\TaskHandler->run(integer 14306)
266 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #5 pmsrc/src/pocketmine/plugin/PluginManager(654): pocketmine\scheduler\TaskScheduler->mainThreadHeartbeat(integer 14306)
267 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #6 pmsrc/src/pocketmine/Server(2389): pocketmine\plugin\PluginManager->tickSchedulers(integer 14306)
268 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #7 pmsrc/src/pocketmine/Server(2152): pocketmine\Server->tick()
269 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #8 pmsrc/src/pocketmine/Server(1992): pocketmine\Server->tickProcessor()
270 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #9 pmsrc/src/pocketmine/Server(1586): pocketmine\Server->start()
271 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #10 pmsrc/src/pocketmine/PocketMine(314): pocketmine\Server->__construct(object BaseClassLoader, object pocketmine\utils\MainLogger, string[51] /home/gameserver/servers/serveur7/minecraftbedrock/, string[59] /home/gameserver/servers/serveur7/minecraftbedrock/plugins/)
272 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #11 pmsrc/src/pocketmine/PocketMine(344): pocketmine\server()
273 | 2021-09-25 [15:06:04] [Server thread/CRITICAL]: #12 pmsrc(11): require(string[106] phar:///home/gameserver/servers/serveur7/minecraftbedrock/PocketMine-MP.phar/src)
274 | 2021-09-25 [15:06:04] [Server thread/EMERGENCY]: An unrecoverable error has occurred and the server has crashed. Creating a crash dump
275 | 2021-09-25 [15:06:04] [Server thread/EMERGENCY]: Please upload the "/home/gameserver/servers/serveur7/minecraftbedrock/crashdumps/Sat_Sep_25-15.06.04-CEST_2021.log" file to the Crash Archive and submit the link to the Bug Reporting page. Give as much info as you can.
276 | 2021-09-25 [15:06:05] [Server thread/EMERGENCY]: The crash dump has been automatically submitted to the Crash Archive. You can view it on https://crash.pmmp.io/view/5234170 or use the ID #5234170.

Does that mean I can't open the storage with the slapper plugin?
I'm using machine translation so I can't understand it well

I used /slapper edit addcommand rca {player} stackstorage
But i will say you, the stackstorage command don’t working everywhere, i can’t open stackstorage where i want, sometimes i must be 15x the command stackstorage to open the storage, i cant make a video or you can join my server and try the command, just double chest creating on top of my player but storage doesn’t work

To open someone else's storage / stackstorage [xuid]
Please run the
If the person is online you can open it by name
/ stackstorage [name]

No, i tell you, when y do /stackstorage, the storage don’t open, my storage don’t open but double chest is created on top of my head !

https://youtu.be/G6PS0TScPgc

It may be due to the processing speed of the server
I created a trial with the delay of opening the storage 7 times longer than usual
try it
https://poggit.pmmp.io/r/143057/StackStrage_dev-39.phar

Okok thank you i will try now !! And if you want more of this bug i have do another video https://youtu.be/EmvvNAO04AU

Same issue with dev 39 i think it’s with position of player but it’s not sure

The location is not important as the chest is set up to deceive the client
I think the problem is that the chest doesn't become a large chest
Chests are usually connected like this
https://youtu.be/yHcnf3_AKFw

Ooh okok i see !! But why my chest doesn’t become a large chest 🤔 it’s maybe an other plugin ?

is that so
Try disabling other plugins once

Okok i will try without plugin


Comments as paid feature for collectors

Would be nice to add the ability to comment on art from premium collectors
https://docs.textile.io/users/

not relevant


Allow the advisor to filter packages by license classification

Users might not want to disclose the names of (company-)internal packages in the lookup at (public) vulnerability providers. A way to address that would be to have configurable license classification filter for the advisor, so that only packages with licenses of the configured classifications (e.g. only Open Source licenses) are being looked up at the vulnerability provider.

(e.g. only Open Source licenses) are being looked up at the vulnerability provider.

This specific check could be implemented way simpler by just trying to look up the license in the SPDX list.

Also a more generic (Script based?) approach could be implemented that also addresses the needs of https://github.com/oss-review-toolkit/ort/issues/4892.


Write wiki documentation for minimal extension template

As an extension developer I want clear documentation that helps me to understand how to work with the minimal extension template so that I can start developing my own extension
Currently we have two extension templates for extension developers to use. The 'hello world' extension template, and the minimal extension template. The hello world one has proper documentation that explains users how to get started with it.
The minimal one has proper documentation that explains the anatomy of extensions, but it is still missing practical documentation that helps users to get started with it.
The documentation should contain:

A description of what is contained within the minimal template repo, similar to the documentation of the 'hello world' template
A step by step description of how to turn the minimal template into a fully functional one, like the 'hello world' template.

The documentation should be added to this wiki page. It should replace the 'To do' section

As part of this, we should also clarify what the "main" key is in the manifest.json (and probably document all the keys in both manifest.json and package.json).

Here's the current JSDocs for the manifest file https://github.com/paranext/paranext-core/blob/main/src/extension-host/services/extension.service.ts#L40

Looks great! Thanks for the hard work!
I made some smaller tweaks that you can see in the following diffs:

https://github.com/paranext/paranext-extension-template/wiki/Extension-Anatomy/_compare/7a187a300156c50eb4d75dfb17eaaeccc9c99e61...30dd2daa50eddf227f76fd3d0066ff559463ce08
https://github.com/paranext/paranext-extension-template/wiki/Your-First-Extension/_compare/7a187a300156c50eb4d75dfb17eaaeccc9c99e61...67c1f26fd4c3280e438d476b5d4b11416c311d1e
https://github.com/paranext/paranext-extension-template-hello-world/wiki/Your-First-Extension-‐-An-Easy-Start/_compare/44328a5ea7e44c1a1da87adc2a2e126323324b23...33a62deed1e3eba2e67f7172cdd939332f497993
https://github.com/paranext/paranext-extension-template/wiki/PAPI/_compare/18bded63587e1d2da58fbd0f661393771d9d480f...c0aacb53b50d2104152ceb06b990bb0638889b78

Larger recommendation:

Extension Anatomy

Move Data Provider Engine section down after activate and deactivate

@tjcouch-sil thanks for improving the docs. The first link above you have added links with line numbers to files. That seems fragile to me with no way for anyone modifying the file to know this page needs updating. Perhaps still link to the file but use a description to get to the right line.

@tjcouch-sil thanks for improving the docs. The first link above you have added links with line numbers to files. That seems fragile to me with no way for anyone modifying the file to know this page needs updating. Perhaps still link to the file but use a description to get to the right line.

I agree that it's painfully fragile. However, I thought there was enough context to get them close enough for now. If you want to change something, feel free. And there is a link to WebViewProps in papi.d.ts somewhere, too, which is really bad since papi.d.ts is generated. I hope we can replace that once we generate API reference from papi.d.ts and use a markdown header link.


Add links to OpenSSF Vulnerabilities Disclosures WG

We always want projects/SIGs to point to their working group, but this one doesn't. Let's fix that, and tweak some other text along the way so it's clearer for visitors.

@SecurityCRob @dlorenc - seem correct? I want to make sure we have links to the relevant WG.

@SecurityCRob @dlorenc - seem correct? I want to make sure we have links to the relevant WG.

friendly ping on this!

+1

Thanks all!


Can't edit file with " quotation mark in name

Looks like filename in click.edit is not escaped or quoted which causes exception or weird behavior when filename contains ".
Exception when filename contains odd number of "
click.edit(filename='foo"bar')

/bin/sh: 1: Syntax error: Unterminated quoted string
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/fpob/tmp/.direnv/python-3.9.1/lib/python3.9/site-packages/click/termui.py", line 588, in edit
    editor.edit_file(filename)
  File "/home/fpob/tmp/.direnv/python-3.9.1/lib/python3.9/site-packages/click/_termui_impl.py", line 466, in edit_file
    raise ClickException("{}: Editing failed!".format(editor))
click.exceptions.ClickException: vim: Editing failed!

Expected is file named foo"bar opened in the editor.
And when filename contains even number of ", then the editor opens multiple files. For example when I tried to edit filename foo "bar baz" first file was named foo "bar and second baz. I would expect one file named foo "bar baz" to be open in the editor
click.edit(filename='foo "bar baz"')  # opened 2 files
click.edit(filename='foo "bar baz", "foo bar"')  # opened 3 files

I don't know what the exact quoting rules for quotes are, but you'll need to do that, it's not something Click can do. I think it's something like foo'"'bar. Work it out in your terminal, as whatever you use there is what it should be, it's not specific to Click.

I don't know what the exact quoting rules for quotes are, but you'll need to do that, it's not something Click can do. I think it's something like foo'"'bar. Work it out in your terminal, as whatever you use there is what it should be, it's not specific to Click.

https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html -- shell quoting rules.
I believe it is bug in Click. I tried to look into code and changed https://github.com/pallets/click/blob/master/src/click/_termui_impl.py#L480 to c = subprocess.Popen(f'{editor} {shlex.quote(filename)}', env=environ, shell=True). That fixed the problem. Python docs is saying that shlex.quote is quoting compatible with UNIX shell.

https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html -- shell quoting rules.
I believe it is bug in Click. I tried to look into code and changed https://github.com/pallets/click/blob/master/src/click/_termui_impl.py#L480 to c = subprocess.Popen(f'{editor} {shlex.quote(filename)}', env=environ, shell=True). That fixed the problem. Python docs is saying that shlex.quote is quoting compatible with UNIX shell.

We can't call shlex.quote because it does the wrong thing on Windows. It's up to you for now to do that.

We can't call shlex.quote because it does the wrong thing on Windows. It's up to you for now to do that.

Something like
            if WIN:
                c = subprocess.Popen(f'{editor} "{filename}"', env=environ, shell=True)
            else:
                c = subprocess.Popen(f'{editor} {shlex.quote(filename)}', env=environ, shell=True)

would work?

Something like
            if WIN:
                c = subprocess.Popen(f'{editor} "{filename}"', env=environ, shell=True)
            else:
                c = subprocess.Popen(f'{editor} {shlex.quote(filename)}', env=environ, shell=True)

would work?

Then the behavior is different on Windows, so someone else will report that weird filenames don't work consistently. We've had a bunch of issues recently when we tried to switch to shlex.quote. The robust solution is to use subprocess.Popen instead of os.system, see #1476.

Then the behavior is different on Windows, so someone else will report that weird filenames don't work consistently. We've had a bunch of issues recently when we tried to switch to shlex.quote. The robust solution is to use subprocess.Popen instead of os.system, see #1476.


Fix for iOS 16 infinite render loop

Needs testing outside of iOS 16
#29 #28 #27
Submitting as draft

Seems to work well with 16.x and 15.2 so far. Tested with iPad/iOS. Will try to get this wrapped up tomorrow when I have more time

Thanks a lot @MikeMorawski, I'll do some testing and hopefully merge it after


Only Reinstall and Syc are availabe commands.

I have been using this add-on successfully for few times. Now it has only reinstall and sync available, other commands that push to the deck are not available.
What have I done to debug:

Used the reinstall command.
Uninstall and reinstall the add-on.
Restart the machine and vs code.

Any suggestions on how  I can debug this will be very helpful.
System info:
OS: PopOS
VS Code version: 1.62.3

The other commands only work if you're viewing a markdown file
https://github.com/jasonwilliams/anki/blob/master/package.json#L32-L41
Maybe you're viewing a file of a different type?

Yes, that was the exact issue. I had *.txt extension instead of *.md.
Thank you for quick response and a great project.


Migrate all workflows to slbeta4

Description:
$subject we have to do the above since the commands have changed from slbeta4
Similar PR https://github.com/ballerina-platform/module-ballerinax-slack/pull/83

Redundant after https://github.com/ballerina-platform/module-ballerinax-mssql.driver/pull/10


What firewall rule are required to manage a Hyper-V host?

Maybe I missed this but I don't see where it's listed what firewall rules are required to be opened for managing hyper-v hosts? I think this would be useful for organization using group policy and DSC to deploy firewall rules.

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

ID: 5c4bb2d4-9b81-b153-6936-3ca0f95bf631
Version Independent ID: 968f574d-3be8-2af6-7470-d4e1f65b4a48
Content: Remotely manage Hyper-V hosts
Content Source: WindowsServerDocs/virtualization/hyper-v/manage/Remotely-manage-Hyper-V-hosts.md
Service: na
Product: windows-server-threshold
GitHub Login: @KBDAzure
Microsoft Alias: kathydav

In Windows Firewall, the following rule:
Windows Remote Management (HTTP-In)
opens port 5985/TCP.


Fix fragment_shading_rate_dynamic sample

Description
Fix the declaration of the compute_buffer in a loop as an r-value reference to a simple reference.
Fixes #913
General Checklist:
Please ensure the following points are checked:

[X] My code follows the coding style
[X] I have reviewed file licenses
[X] I have commented any added functions (in line with Doxygen)
[X] I have commented any code that could be hard to understand
[X] My changes do not add any new compiler warnings
[X] My changes do not add any new validation layer errors or warnings
[X] I have used existing framework/helper functions where possible
[X] My changes do not add any regressions
[X] I have tested every sample to ensure everything runs correctly
[X] This PR describes the scope and expected impact of the changes I am making

Note: The Samples CI runs a number of checks including:

[X] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[X] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[X] I have tested the sample on at least one compliant Vulkan implementation

I'm actually not sure if that sample should work at all. It allocates the compute command buffers from the base class command pool, which may be limited to the graphics queue family index.
Imo it should be adjusted to create it's own compute command pool, like we do in the other compute related samples.

I'm actually a bit confused, because I recently did a batch test against my allocated-refactor branch which does NOT have this fix and it passed without crashing.  And the error you're seeing is what I was getting before I made this change on the sample, so I think the sample is more subtle than I suspected at first...

Imo it should be adjusted to create it's own compute command pool, like we do in the other compute related samples.

I initially tried that approach because I noticed that FragmentShadingRateDynamic::build_command_buffers resets this command pool, which invalidates any of the compute buffers allocated from it, but that didn't seem to help.  That's also where my investigation with git bisect led me.
I'm going to give that another shot and test it again checking both release and debug modes this time.

OK, so using a dedicated compute queue for the compute operations would require significant refactoring of the example.  While the acquire and release barriers in the compute command buffers are pretty straight-forward, it's not immediately clear to me where they'd go on the graphics side.  It looks like the acquire would have to go in the draw_cmd_buffers but the release would have to go in the small_command_buffers, I think.  At any rate, I'm going to just try to resolve this with a unified queue for now.


Unable to compile example in stand-alone repository using main due to Serde error

Hello,
I tried building the quickstart example in a stand-alone project today, and was unable to do so. I've tested that cloning neuronika off the main branch builds (and the example runs), but when pulling it in as a dependency, I am consistently running into the error
error[E0277]: the trait bound `NeuralNetwork: serde::de::Deserialize<'_>` is not satisfied
    --> src/main.rs:53:9
     |
53   |     } = serde_json::from_str(
     |         ^^^^^^^^^^^^^^^^^^^^ the trait `serde::de::Deserialize<'_>` is not implemented for `NeuralNetwork`
     |
note: required by a bound in `serde_json::from_str`
    --> /home/chrism/.cargo/registry/src/github.com-1ecc6299db9ec823/serde_json-1.0.79/src/de.rs:2611:8
     |
2611 |     T: de::Deserialize<'a>,
     |        ^^^^^^^^^^^^^^^^^^^ required by this bound in `serde_json::from_str`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `neuronika_example` due to previous error

Running this code with 0.2 doesn't seem feasible due to what I'm assuming are updated trait bounds around Overwrite.
My Cargo.toml file has the following configuration, based on neuronika main's:
[dependencies]
neuronika = {git = "https://github.com/neuronika/neuronika.git", branch = "main", features = ["serialize"]}
ndarray = {version = "0.15.4", features = ["rayon"]}
serde_json = "1.0.72"
serde = {version = "1.0.130", features = ["derive"]}

I am compiling with rustc 1.60.0-stable on an x86 platform using Pop!_OS 21.04.
I'm very interested in this library, and would love to experiment with starting to solve basic classification problem like MNIST or CIFAR-10 with it. Any advice on how I might be able to resolve these issues would be greatly appreciated. Thanks for your time!

I'm afraid it doesn't look like this fixes the issue. Thanks for your consideration, but since it sounds like you're doing a lot of re-factoring at the moment (really looking forward to the features you mentioned landing!), I'll probably just wait until the next release to really start digging in.

How are you running the example? You should be using cargo run --example quickstart --features="serialize"

I think we've had a miscommunication. When I clone the neuronika repository and run cargo run --example quickstart --features="serialize", everything compiles fine.
However, when I create a new repository cargo new --bin neurnika_test, copy-paste in the quickstart example code, and try to run it while having the following Cargo.toml file
[dependencies]
neuronika = {git = "https://github.com/neuronika/neuronika.git", branch = "main", features = ["serialize"]}
ndarray = {version = "0.15.4", features = ["rayon"]}
serde_json = "1.0.72"
serde = {version = "1.0.130", features = ["derive"]}

which should import the neuronika main branch with the serialize feature enabled, I get a compilation error regarding serde trait bounds as seen in my first example. In essence, it seems like in-project development builds are fine, but I am having issues using neuronika main as a dependency in other projects.

Got it. Yes, that's a problem. Would you be able to look into it? Unfortunately, I'll be quite busy for the next couple of weeks.

Just ran into the same error too. Was it resolved? Thanks.

@phatware does the following work? cargo run --example quickstart --features="serialize"

yes, this command works, however creating a new project and using code similar to
fn load_model() -> NeuralNetwork {
let NeuralNetwork {
lin1,
lin2,
lin3,
mut status,
} = serde_json::from_str(
r#"
{
"lin1":{
"weight":{
......
fails with the same error as described before.
Thanks.

@phatware try this  for_pathware.zip, it works for me. Also, forgive me for not taking care of the project in the last few months but I have been very very busy with other stuff. If you are interested in the library do note that soon (around or before October) a big rework of the project with GPU support will land on main. You can check it out at the variable-rework branch.

This worked, thank you. Appreciate quick response. Cheers.

@phatware try this for_pathware.zip, it works for me. Also, forgive me for not taking care of the project in the last few months but I have been very very busy with other stuff. If you are interested in the library do note that soon (around or before October) a big rework of the project with GPU support will land on main. You can check it out at the variable-rework branch.
Also forgive me for misspelling your nickname!

I got the same issue, and it is solved by your zip, thanks.


UI Improvement

This page has no design till now. So grab this opportunity and do any changes that make the UI appealing to the user.
Don't use any other framework except Bootstrap.

I'll take this

#15 Improves UI and other features in the website

I can improve the style.

@marcowright please go ahead.

let me try bro .. @rowhitswami ...

@Barathwaja This application should be light-weight as well as responsive. Do make the change if you
think your solution can fulfill this requirement.


IOTA Explorer failure for private Tangle

I am deploying a private chrysalis-based Tangle using the provided scripts. All of the main Docker containers are built and started properly: coo, node1, spammer and node-autopeering. However, when trying to install the IOTA Explorer, although the explorer-webapp container is up and running, the explorer-api fails. I receive the error "Network is not one of the valid network types" corresponding to the following lines of the explorer code:
https://github.com/iotaledger/explorer/blob/94a5bc12f33995b3cca4942d183ce1c63877b20b/api/src/services/networkService.ts#L83-L86
I found out that the network type ("tangle") configuration provided in the private-network.json config file does not correspond to any of the network types defined in the explorer's scripts, since "tangle" is not one of the available options.
https://github.com/iotaledger/one-click-tangle/blob/e9cceb4f5c08a55b63dd82c7d27652bd15a7e36c/explorer/config/private-network.json#L1-L14
https://github.com/iotaledger/explorer/blob/94a5bc12f33995b3cca4942d183ce1c63877b20b/api/src/models/db/networkType.ts#L1-L18
First, I tried to fix this by modifying the explorer's script and adding a new network type called "tangle" so it matches to the one given in the configuration. Also, another workaround is to change the type in the private-network.json config file. I tried with the "mainnet" option and this way I finally managed to deploy the explorer-api container. However, when accessing the explorer web at http://localhost:8082 nothing was displayed, receiving the error shown in the next image where there seems to be a syntax error while executing the env.js script.

Here come my questions:
First, which of the available network types (defined in the explorer's script) is the most suitable for the deployed private Tangle?
And second, any idea on how to solve the error occured while displaying the explorer graphical interface?
Technical notes:
Ubuntu version: 20.04
Docker version: 23.01
Docker-compose version: 1.25.0
Thank you in advance!

can you try with network: "custom" ?

can you try with network: "custom" ?

I already tried this option, but I still get the same syntax error while rendering the explorer web app.

Add a file named env.js in the client/public folder with this content
window.env = {
    API_ENDPOINT: 'http://localhost:4000/'
};

In there specify the URI of the Explorer-api (by default it should be as above).
I'm sorry for this, i think we are missing this step in the setup docs. 😅

Also, as you already noticed, the value for network in a network config cannot be "tangle" or anything that is not one of the defined values. I would suggest "mainnet" if you are using it for a "chrysalis" network, or "custom" for a stardust one.

I have just applied the changes you suggested and it worked perfecly.
Thank you so much!


Add PulseIn function

Closes #17

I have fixed the test by implementing a new Pin for PulseIn tests. I think this hasn't race parts.

@maruel sorry for bothering you, I have fixed the last issues.

Thanks!


update pinned versions of several dependencies

I am working on incorporating pgstac into a project and I had to downgrade to smart-open==4.2.0 so I thought I would try bumping the pinned version to the latest (6.2.*). Since I was in there already I bumped a few other less-outdated pinned versions, too. All of the tests are passing when I run scripts/test as described in the contributing guidelines!
Maybe there is a good reason smart-open is held back to version 4.2.* but in case there is not one, here is a PR!

@vincentsarago I'm sorry I left this hanging - thanks for taking this over the finish line!


--no-version-reporting --no-check-update flags not working

Hi,
cdk will try to recommend version upgrades by invoking "npm view version cdk-core" outputing:
  **************************************************
  *** Newer version of CDK is available [1.91.0] ***
  *** Upgrade recommended                        ***
  **************************************************

cdk also supports flags to skip this, but cdklocal doesn't seem to be working with those.
Is this possibly an issue with cdklocal or is our usage just wrong?
Thanks,
Patrick

Thanks for reporting @PaddyJM . Please note that cdklocal is simply using whichever version of cdk is installed under the covers (and available in your NODE_PATH). So, technically you should be able to upgrade the CDK version independently, and cdklocal should still work.

cdk also supports flags to skip this, but cdklocal doesn't seem to be working with those.

Can you please share some more details - are you referring to the CDK_DISABLE_VERSION_CHECK environment variable? Thanks

Hi Whummer,
Updating the CDK version is is not the concern. We are deploying into localstack so the version check just times out, slowing down our testing suite as it waits for the timeout on each run.
I tried passing in CDK_DISABLE_VERSION_CHECK with both values "true" and "1", and neither worked. I also tried exporting these variables.
It feels like as CDK accepts the --no-version-reporting --no-check-update flags, it should be possible to pass these down from cdk local? Is this the currently expected behaviour?

I tried looking into this again. Here are some of my notes:

--no-version-reporting doesn't seem to exist anymore and previously wasn't related to updating the npm lib and instead controlled the output of "AWS::CDK::Metadata" in the synthesized template.
The current behavior or aws-cdk in regards to update prompts  is to ask once, record the version that it asked about in ~/.cdk/cache/repo-version-ttl and then it won't check/ask again until one day has passed (based on the file modification date).
I did not find a --no-check-update option for cdk
CDK_DISABLE_VERSION_CHECK=1 can be used to skip this unconditionally. Based on my tests this works with both cdk and cdklocal. I.e. both $ CDK_DISABLE_VERSION_CHECK=1 cdklocal context and $ CDK_DISABLE_VERSION_CHECK=1 cdk context will not print the upgrade message after previously making sure the file at ~/.cdk/cache/repo-version-ttl has been deleted.


Last prediction:  0 always zero

tanzeel@tanzeel-HP-Compaq-dx7500-SFF:~/BitMEX-simple-trading-robot$ python3 main_loop.py
Last prediction: {prediction} 0
Last prediction: {prediction} 0
Last prediction: {prediction} -1
/home/tanzeel/.local/lib/python3.5/site-packages/bravado_core/spec.py:271: Warning: guid format is not registered with bravado-core!
  category=Warning,
Last prediction: {prediction} 0
Last prediction: {prediction} 0
Last prediction: {prediction} 0
Last prediction: {prediction} 0
Last prediction: {prediction} 0
Last prediction: {prediction} 0
Last prediction: {prediction} 0

What am i doing wrong ?
my configuration.py is :

TEST_EXCHANGE=True
API_KEY="XXXXXXXXXXXXXXXXXX"
API_SECRET="XXXXXXXXXXXXXXXXXXXXXX"

AMOUNT_MONEY_TO_TRADE=200 #$
LEVERAGE=25

TIMEFRAME = '1m'

time_to_wait_new_trade = {'1m': 60,
                          '5m': 60*5,
                          '1h': 60*60,
                          '1d': 60*60*24}

Did you manage to figure out what the issue was? I'm also having similar trouble getting it to run.


Error on tryLogin()

After redirection tryLogin throws unhandled error:

Version:
"angular-oauth2-oidc": "4.0.2"
this.oAuthService.tryLogin(); is being called on ngOnInit
AuthConfig:
`
export const authConfig: AuthConfig = {
// Url of the Identity Provider
issuer: getAuthUrl(),
loginUrl: getAuthUrl() + '/oauth2/authorize',
requireHttps: environment.production ? true : false,
// URL of the SPA to redirect the user to after login
redirectUri: window.location.origin + '/',
strictDiscoveryDocumentValidation: true,
silentRefreshRedirectUri: window.location.origin + '/',
oidc: false, // ID_Token
// The SPA's id. The SPA is registerd with this id at the auth-server
clientId: environment.oauthClientId,
showDebugInformation: true,
// set the scope for the permissions the client should request
// The first three are defined by OIDC. The 4th is a usecase-specific one
scope: ''

};
`
The same happens on silentUpdate
The login itself does work, if you "try catch" the error.
Please tell me if you have an idea what went wrong.

The problem is that i try 4.0.2 with angular 5. I try to migrate to 6.


docs: Update rules from the 1.x to 2.x format

Hi here,
This is just a trivial PR in order to update alerting rules from the 1.x to 2.x format.
This PR is maybe linked to https://github.com/prometheus/alertmanager/pull/1062 and https://github.com/prometheus/docs/issues/857 (extract alertmanager documentation).

Thanks!


Scanning hathi pairtree data is fragile: breaks on .DS_Store, additional or missing files

Setting up an issue here to track this PR: https://github.com/Princeton-CDH/ppa-django/pull/529
I've run into a few errors when scanning through the HATHI_DATA folder. If there's a .DS_Store in there, as happens on macs if that folder is ever opened via Finder, or if a single work has a missing .zip file, then the entire indexing or other process (often parallel processes, which break even more messily) will break.

This issue also occasioned this closed PR: https://github.com/Princeton-CDH/ppa-django/pull/515

Fixed by PR #515


Add option to group locally imported packages

Packages which are not pulled from a remote, but read from a local path
can be grouped individually with this option.

Hi @gleich, sorry this took so long. I finally got back to this, rebased to current master and implemented some tests.
Please take another look :)

Thank you so much for this @julianscheel! I'll make sure to review it sometime this weekend. Thanks again!

ping @gleich

Hello @julianscheel. Thank you so much for sending in this PR but I am sadly archiving import_sorter (see #63). This means that your work will not be merged into import_sorter. Thank you so much for the effort in trying to improve the project.