Fix text style errors

Fix errors on testing with .travis-build.php.
$ php ./.travis-build.php
The following lines end with a space character:
   57: Not every principle herein has to be strictly followed, and even fewer will be universally
   58: agreed upon. These are guidelines and nothing more, but they are ones codified over many
  104: We will read more code than we will ever write. It's important that the code we do write is
  105: readable and searchable. By *not* naming variables that end up being meaningful for
  434: Limiting the amount of function parameters is incredibly important because it makes
  435: testing your function easier. Having more than three leads to a combinatorial explosion
  438: Zero arguments is the ideal case. One or two arguments is ok, and three should be avoided.
  439: Anything more than that should be consolidated. Usually, if you have more than two
  440: arguments then your function is trying to do too much. In cases where it's not, most
  479: This is by far the most important rule in software engineering. When functions do more
  480: than one thing, they are harder to compose, test, and reason about. When you can isolate
  481: a function to just one action, they can be refactored easily and your code will read much
  482: cleaner. If you take nothing else away from this guide other than this, you'll be ahead
  545: class Email
  702: Flags tell your user that this function does more than one thing. Functions should
  703: do one thing. Split out your functions if they are following different code paths
  737: A function produces a side effect if it does anything other than take a value in and
  738: return another value or values. A side effect could be writing to a file, modifying
  741: Now, you do need to have side effects in a program on occasion. Like the previous
  742: example, you might need to write to a file. What you want to do is to centralize where
  743: you are doing this. Don't have several functions and classes that write to a particular
  747: any structure, using mutable data types that can be written to by anything, and not
  748: centralizing where your side effects occur. If you can do this, you will be happier
  789: Polluting globals is a bad practice in many languages because you could clash with another
  790: library and the user of your API would be none-the-wiser until they get an exception in
  792: You could write global function like `config()`, but it could clash with another library
  825: Load configuration and create instance of `Configuration` class
 1133: In PHP you can set `public`, `protected` and `private` keywords for methods.
 1134: Using it, you can control properties modification on an object.
 1279: class Employee
 1293: // Bad because Employees "have" tax data.
 1296: class EmployeeTaxData extends Employee
 1300:
 1316: class EmployeeTaxData
 1330: class Employee
 1477:
 1482:
 1486:     public function getColor()
 1507:
 1512:
 1516:     public function getColor()
 1575: class UserAuth
 1583:
 1590: class UserSettings
 1595:     public function __construct(User $user)
 1782:
 1857: they do not use."
 2041: Do your absolute best to avoid duplicate code. Duplicate code is bad because
 2042: it means that there's more than one place to alter something if you need to
 2045: Imagine if you run a restaurant and you keep track of your inventory: all your
 2052: to have two or more separate functions that do much of the same things. Removing
 2053: duplicate code means creating an abstraction that can handle this set of different
 2059: a good abstraction, do it! Don't repeat yourself, otherwise you'll find yourself
 2160:

The table of contents is not aligned:
--- /www/clean-code-php/current-chapters  2019-07-11 19:11:21.578661500 +0300
+++ /www/clean-code-php/chapters-found    2019-07-11 19:11:21.580663800 +0300
@@ -11,7 +11,7 @@
      * [Don't add unneeded context](#dont-add-unneeded-context)
      * [Use default arguments instead of short circuiting or conditionals](#use-default-arguments-instead-of-short-circuiting-or-conditionals)
   3. [Comparison](#comparison)
-     * [Use identical comparison](#use-identical-comparison)
+     * [Use [identical comparison](http://php.net/manual/en/language.operators.comparison.php)](#use-identical-comparisonhttpphpnetmanualenlanguageoperatorscomparisonphp)
   4. [Functions](#functions)
      * [Function arguments (2 or fewer ideally)](#function-arguments-2-or-fewer-ideally)
      * [Functions should do one thing](#functions-should-do-one-thing)
@@ -33,7 +33,7 @@
   6. [Classes](#classes)
      * [Prefer composition over inheritance](#prefer-composition-over-inheritance)
      * [Avoid fluent interfaces](#avoid-fluent-interfaces)
-     * [Prefer `final` classes](#prefer-final-classes)
+     * [Prefer final classes](#prefer-final-classes)
   7. [SOLID](#solid)
      * [Single Responsibility Principle (SRP)](#single-responsibility-principle-srp)
      * [Open/Closed Principle (OCP)](#openclosed-principle-ocp)

Thank you Peter :+1:


Eudico's State Manager

[x] Mempool integration (0.5 weeks)
[x] Block assembler (0.5 week)
[x] (Re)Configuration

[x] Format of configuration data and config transaction flow (1 week)
[x] Reconfiguration

[x] Instantiating components with new configuration (1 week)
[x] Adding / removing nodes (2 weeks)
[x] Garbage-collection of old components (1 week)

[x] State transfer (1 week)
[ ] Testing / debugging / buffer (1 week)

Estimated duration: 8 weeks

Closed by merging https://github.com/filecoin-project/eudico/pull/234


feat: Categorical hnsw

Build hnsw graph based on other columns, to improve search performance when with filters
Reference: https://qdrant.tech/articles/filtrable-hnsw/

Working on it


Fix issue where inline special attributes were fine with spaces.

The behavior before the changes to the special attributes was that
spaces were not allowed. This allows for not accidentally making content
inside {brackets} disappear. This readds that requirement that spaces
are not allowed between a codespan or a link and their special
attributes.

Codecov Report

:exclamation: No coverage uploaded for pull request head (whitespace-attr-fix@f9e0d59).

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 d971e29...f9e0d59. Read the comment docs.

The coverage is less because code that wasn't working as intended is removed. Whitespace doesn't matter inside special attributes, and are skipped anyways. Let me know if the failing test is a blocker.

Thanks.


Any possible way to remove has-error class when form submit?

Thank you for the awesome library!
I am trying to add shake css animation to input field. In css
.has-error input {
/* css animation codes */
}

In jsx, when submit event occurs, I want to remove .has-error class(es) in all form fields and re-evaluate them to show animation again if error occurs.
All I can think of is use getElementsByClassName and loop to remove .has-error class. Is there a function inside tcomb-form to reset error?

Hi, thanks.

Is there a function inside tcomb-form to reset error?

Officially, no.
Unofficially, as a workaround, you could get the inner components through refs and set the error state by hand, e.g.
var MyType = t.struct({
  name: t.Str,
  surname: t.Str
});

...

<Form ref="form" type={MyType} />

...

this.refs.form.refs.input.refs.name.setState({hasError: true});
this.refs.form.refs.input.refs.surname.setState({hasError: true});


Web spell acting weird

For some reason these zombies (probably all creatures) gets to keep rolling strength without spending their action (set to free action perhaps?) when restrained until they succeed.
Web Being Wacky.zip

This has already been fixed in dev. Will test on your save as well.

Cool! You were on top of it I see

confirmed with your save as well.


data loss on windows platform

Originally, we are using Ambry on linux and everything is fine. But recently, we try to use Ambry on the client side where the OS is Windows 10. After we restart Ambry, all data lost. Log says:
[2017-04-13 09:06:12,176] INFO Bootstrapping AmbryServer (com.github.ambry.server.AmbryMain)
[2017-04-13 09:06:12,179] INFO starting (com.github.ambry.server.AmbryServer)
[2017-04-13 09:06:12,179] INFO Setting up JMX. (com.github.ambry.server.AmbryServer)
[2017-04-13 09:06:12,240] INFO creating configs (com.github.ambry.server.AmbryServer)
[2017-04-13 09:06:12,243] INFO Verifying properties (com.github.ambry.config.VerifiableProperties)
[2017-04-13 09:06:12,244] INFO Property host.name is overridden to localhost (com.github.ambry.config.VerifiableProperties)
[2017-04-13 09:06:12,244] INFO Property store.data.flush.interval.seconds is overridden to 3 (com.github.ambry.config.VerifiableProperties)
[2017-04-13 09:06:12,245] INFO check if node exist in clustermap host localhost port 6667 (com.github.ambry.server.AmbryServer)
[2017-04-13 09:06:12,258] INFO Starting storage manager (com.github.ambry.store.StorageManager)
[2017-04-13 09:06:12,263] INFO Loading segment with name [] (com.github.ambry.store.Log)
[2017-04-13 09:06:12,264] INFO Segment [] has capacity of<PHONE_NUMBER>0 bytes (com.github.ambry.store.Log)
[2017-04-13 09:06:12,487] INFO Index :  C:\Users\DM\Desktop\oss\data\0 log end offset of index  before recovery Name = [] Offset = [0] (com.github.ambry.store.PersistentIndex)
[2017-04-13 09:06:12,487] INFO Index :  C:\Users\DM\Desktop\oss\data\0 performing recovery on index with start offset Name = [] Offset = [0] and end offset 506 (com.github.ambry.store.PersistentIndex)
[2017-04-13 09:06:12,487] ERROR Trying to read more than the available bytes (com.github.ambry.messageformat.BlobStoreRecovery)
[2017-04-13 09:06:12,489] INFO Setting active segment to [] (com.github.ambry.store.Log)
[2017-04-13 09:06:12,489] INFO Index :  C:\Users\DM\Desktop\oss\data\0 Starting hard delete recovery (com.github.ambry.store.PersistentIndex)
[2017-04-13 09:06:12,489] INFO Index : C:\Users\DM\Desktop\oss\data\0 Finished performing hard delete recovery (com.github.ambry.store.PersistentIndex)
[2017-04-13 09:06:12,492] INFO Starting storage manager complete (com.github.ambry.store.StorageManager)
[2017-04-13 09:06:12,493] INFO BlockingChannelConnectionPool started (com.github.ambry.network.BlockingChannelConnectionPool)
[2017-04-13 09:06:12,500] INFO Reading replica tokens for mount path C:\Users\DM\Desktop\oss\data (com.github.ambry.replication.ReplicationManager)
[2017-04-13 09:06:12,500] WARN Number of Datacenters to replicate from is 0, not starting any replica threads (com.github.ambry.replication.ReplicationManager)
[2017-04-13 09:06:12,510] INFO Starting 8 processor threads (com.github.ambry.network.SocketServer)
[2017-04-13 09:06:12,561] INFO Starting acceptor threads (com.github.ambry.network.SocketServer)
[2017-04-13 09:06:12,562] INFO Awaiting socket connections on localhost:6667 (com.github.ambry.network.Acceptor)
[2017-04-13 09:06:12,563] INFO Started server (com.github.ambry.network.SocketServer)
[2017-04-13 09:06:12,563] INFO started (com.github.ambry.server.AmbryServer)
[2017-04-13 09:06:12,563] INFO Server startup time in Ms 383 (com.github.ambry.server.AmbryServer)
[2017-04-13 09:06:59,547] INFO Creating first segment (com.github.ambry.store.PersistentIndex)

I am aware of that Ambry is not tested on Windows. But according to the error log "Trying to read more than the available bytes", this might be fixable. Will you guys fix it on Windows?

Here is the test process on Windows (On linux, no problem):
After clean all the data in the data store, I post some data to Ambry:
curl -i -H "x-ambry-blob-size : `wc -c test.txt | xargs | cut -d" " -f1`" -H "x-ambry-service-id : CUrlUpload"  -H "x-ambry-owner-id : `whoami`" -H "x-ambry-content-type : image/gif" -H "x-ambry-um-description : Demonstration Image" http://<IP_ADDRESS>:1174/ --data-binary @test.txt                     HTTP/1.1 201 Created
Location: /AAEAAQAAAAAAAAAAAAAAJDJjZTMxNjRjLTEzYzMtNGZmYy1iYjM1LTU0MjcxOTdiMmU0OQ
Date: Thu, 13 Apr 2017 01:22:54 GMT
Content-Length: 0
x-ambry-creation-time: Thu, 13 Apr 2017 01:22:54 GMT

a few seconds later, I kill Ambry jvm via a .bat:
FOR /F "tokens=5 delims= " %%P IN ('netstat -a -n -o ^| findstr :1174.*LISTENING') DO TaskKill.exe /F /PID %%P
FOR /F "tokens=5 delims= " %%P IN ('netstat -a -n -o ^| findstr :6667.*LISTENING') DO TaskKill.exe /F /PID %%P

then restart Ambry, the error log says what I posted before. In the datastore directory, "0_index.tmp" and "log_current" files both have data. But ambry failed to recover data from the restart.

@michael2008 "Trying to read more than the available bytes" is expected during recovery. It is the way the recovery code detects that nothing more can be recovered.
In your second example, when you restarted ambry did you see 0_index (not .tmp)? Is the error log you pasted above from this very restart? How big is the file you uploaded? ambry's flushing to disk is async (file system is also flushing regularly but we don't rely on that) - so it is possible that you lose some data if you kill the process. Recovery can only recover complete blobs. It stops when it encounters incomplete blobs. So you might have the killed the process here before all the data was written into the log.

@vgkholla   actually, there is no .tmp file when I restart the server. No matter how long I wait, the recovery would fail. async should work, but it does not, don't know why.

@michael2008 @nsivabalan found the bug that is causing this. Will be fixed shortly. However, keep in mind that the log message can print during recovery even after the bug is fixed if a blob has been half written into the log at the event of a crash.

Being fixed in #602


Coding Style

I don't see many of these coding style from corefx repo anywhere in the docs. Shouldn't they be added somewhere ?
Whether in the Coding Conventions article or another separate article ?
/cc @rpetrusha , @BillWagner , @mairaw

I think this should be discussed before making these changes.  It may be wise to link to the CoreFx repo's coding style, but I don't think we should replicate them in the docs.
Thoughts?

Linking to the corefx coding style seems good I think.


Add 'org.jcp.xml.dsig.internal' as system package.

This package is included in both jdk7 and jdk8.
Use package will be used as a typical jsr105 usage, such as

        String providerName = System.getProperty("jsr105Provider", "org.jcp.xml.dsig.internal.dom.XMLDSigRI");
        XMLSignatureFactory fac =
            XMLSignatureFactory.getInstance("DOM", (Provider) Class.forName(providerName).newInstance());

This is not the correct fix.  None of the other JAXP implementation classes are listed as system packages; this should be fixed in the JAXP redirection code (though I do not know why it has failed in the first place).

Thanks, @dmlloyd. :+1: Problem solved.

Just use something like
XMLSignatureFactory fac = XMLSignatureFactory.getInstance("DOM", "XMLDSig");

is enough.


Adjusting servo end courses and speed (delay)

Hi,
First of all, be aware that I'm in no way a professional developer :)
With the driver you built, I wasn't able to get the full course of my 270deg. servo. A quick investigation revealed that the default values provided by Servo.h when attaching the servo (min pulse width = 544 and max pulse width = 2400) were not ideal for my setup. (Using the cheap servos from Amazon that you listed).
Instead of just editing the Arduino code with alternative (and fixed) values, I decided to add 'pulse min' and 'pulse max' as part of the Ascom driver profile.

While I was there, I also added the servo write "delay" as a parameter too. (the default value 30 isn't the best for my setup).
The three parameters are sent to the Arduino upon connection of the device in Nina. Please check how it's parsed on the Arduino side; it's a little dirty... (but it works :) )
I have no idea how I can properly share my code with you or if you're even interested. I'm simply attaching a ZIP below that contains both the driver and the Arduino file. Let me know if there is a better method.
Motorized_cover_guillaume.zip
Comments/critics very welcome!
-Guillaume
PS: Awesome job you did there! I already built two motorized covers, including one for my C11 and, despite the large size, it works perfectly.

Hello! Great work! Note that this project is now on life support so I will not make any changes to it. However, I have been working on a new version of this project, which brings a lot of improvements, and fixes the servo pulse min/max thing you mentioned. I do not have an ETA to release that project. Thanks!

Hello! Great work! Note that this project is now on life support so I will not make any changes to it. However, I have been working on a new version of this project, which brings a lot of improvements, and fixes the servo pulse min/max thing you mentioned. I do not have an ETA to release that project. Thanks!

I’ll keep an eye on what’s coming next!


TFLite: Cannot run model inference converted with SELECT_TF_OPS: ERROR: Select TensorFlow op(s), included in the given model, is(are) not supported by this interpreter. Make sure you apply/link the Flex delegate before inference.

1. System information

OS Platform and Distribution: Ubuntu 20.04.3 LTS
TensorFlow installation: from source
TensorFlow library: 2.9

2. Code
Provide code to help us reproduce your issues using one of the following options:
Code: https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/examples/minimal
Model: https://tfhub.dev/google/boundless/quarter/1 (basically any model which must be converted with tf.lite.OpsSet.SELECT_TF_OPS)
Model conversion code (with SELECT_TF_OPS):
# Load the original model
model = hub.load(model_handle)
concrete_function = model.signatures['default']

# Convert the model to TensorFlow Lite
converter = tf.lite.TFLiteConverter.from_concrete_functions([concrete_function])
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.target_spec.supported_ops = [tf.lite.OpsSet.SELECT_TF_OPS]

tflite_model = converter.convert()

# Serialize the model
filename = f'boundless_quarter_{quantization}.tflite'
open(filename, 'wb').write(tflite_model)

3. Any other info / logs
Error message when minimal.exe gets model.tflite saved with SELECT_TF_OPS:
user1@ubuntu:~/minimal$ ./minimal ../boundless_quarter_integer.tflite
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
ERROR: Select TensorFlow op(s), included in the given model, is(are) not supported by this interpreter. Make sure you apply/link the Flex delegate before inference. For the Android, it can be resolved by adding "org.tensorflow:tensorflow-lite-select-tf-ops" dependency. See instructions: https://www.tensorflow.org/lite/guide/ops_select
ERROR: Node number 0 (FlexIdentity) failed to prepare.
ERROR: Select TensorFlow op(s), included in the given model, is(are) not supported by this interpreter. Make sure you apply/link the Flex delegate before inference. For the Android, it can be resolved by adding "org.tensorflow:tensorflow-lite-select-tf-ops" dependency. See instructions: https://www.tensorflow.org/lite/guide/ops_select
ERROR: Node number 0 (FlexIdentity) failed to prepare.
Error at /home/user1/tensorflow_src/tensorflow/lite/examples/minimal/minimal.cc:60

Correct output, using any model saved without SELECT_TF_OPS (Expected behaviour):
user1@ubuntu:~/minimal$ ./minimal ../model.tflite
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
=== Pre-invoke Interpreter State ===
Interpreter has 1 subgraphs.

-----------Subgraph-0 has 27 tensors and 4 nodes------------
1 Inputs: [0] -> 4B (0.00MB)
1 Outputs: [8] -> 4B (0.00MB)

Tensor  ID Name                      Type            AllocType          Size (Bytes/MB)    Shape      MemAddr-Offset
Tensor   0 serving_default_dense_... kTfLiteFloat32  kTfLiteArenaRw     4        / 0.00 [1,1] [0, 4)
Tensor   1 dense_2/bias              kTfLiteFloat32  kTfLiteMmapRo      4        / 0.00 [1] [572, 576)
Tensor   2 sequential/dense/MatMul   kTfLiteFloat32  kTfLiteMmapRo      4        / 0.00 [1,1] [480, 484)
Tensor   3 sequential/dense_1/MatMul kTfLiteFloat32  kTfLiteMmapRo      64       / 0.00 [16,1] [336, 400)
Tensor   4 sequential/dense_2/MatMul kTfLiteFloat32  kTfLiteMmapRo      64       / 0.00 [1,16] [192, 256)
Tensor   5 sequential/dense_1/Mat... kTfLiteFloat32  kTfLiteMmapRo      64       / 0.00 [16] [0, 64)
Tensor   6 sequential/dense/MatMul1  kTfLiteFloat32  kTfLiteArenaRw     4        / 0.00 [1,1] [-1, -1)
Tensor   7 sequential/dense/BiasA... kTfLiteFloat32  kTfLiteArenaRw     64       / 0.00 [1,16] [-1, -1)
Tensor   8 StatefulPartitionedCall:0 kTfLiteFloat32  kTfLiteArenaRw     4        / 0.00 [1,1] [64, 68)
Tensor   9 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  10 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  11 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  12 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  13 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  14 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  15 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  16 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  17 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  18 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  19 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  20 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  21 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  22 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  23 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  24 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  25 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  26 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)

kTfLiteArenaRw Info:
Tensor 0 has the max size 4 bytes (0.000 MB).
This memory arena is estimated as[0x55e3cc5bb104, 0x55e3cc5bb0c0), taking 68 bytes (0.000 MB).
One possible set of tensors that have non-overlapping memory spaces with each other, and they take up the whole arena:
Tensor 0 -> 8.

kTfLiteArenaRwPersistent Info: not holding any allocation.

kTfLiteMmapRo Info:
Tensor 3 has the max size 64 bytes (0.000 MB).
This memory arena is estimated as[0x7fb2de3f4648, 0x7fb2de3f4408), taking 576 bytes (0.001 MB).
One possible set of tensors that have non-overlapping memory spaces with each other, and they take up the whole arena:
Tensor 5 -> 4 -> 3 -> 2 -> 1.

kTfLiteDynamic Info: not holding any allocation.

Node   0 Operator Builtin Code   9 FULLY_CONNECTED (delegated by node 3)
  3 Input Tensors:[0,2,-1] -> 0B (0.00MB)
  1 Output Tensors:[6] -> 0B (0.00MB)
Node   1 Operator Builtin Code   9 FULLY_CONNECTED (delegated by node 3)
  3 Input Tensors:[6,3,5] -> 0B (0.00MB)
  1 Output Tensors:[7] -> 0B (0.00MB)
Node   2 Operator Builtin Code   9 FULLY_CONNECTED (delegated by node 3)
  3 Input Tensors:[7,4,1] -> 0B (0.00MB)
  1 Output Tensors:[8] -> 0B (0.00MB)
Node   3 Operator Custom Name TfLiteXNNPackDelegate
  6 Input Tensors:[0-5] -> 204B (0.00MB)
  1 Output Tensors:[8] -> 4B (0.00MB)

Execution plan as the list of 1 nodes invoked in-order: [3]
Among these nodes in the execution plan:
  Node 3 is a TfLiteXNNPackDelegate node (0x55e3cc5ba4f0), which has delegated 3 nodes: [0-2]
--------------Subgraph-0 dump has completed--------------

=== Post-invoke Interpreter State ===
Interpreter has 1 subgraphs.

-----------Subgraph-0 has 27 tensors and 4 nodes------------
1 Inputs: [0] -> 4B (0.00MB)
1 Outputs: [8] -> 4B (0.00MB)

Tensor  ID Name                      Type            AllocType          Size (Bytes/MB)    Shape      MemAddr-Offset
Tensor   0 serving_default_dense_... kTfLiteFloat32  kTfLiteArenaRw     4        / 0.00 [1,1] [0, 4)
Tensor   1 dense_2/bias              kTfLiteFloat32  kTfLiteMmapRo      4        / 0.00 [1] [572, 576)
Tensor   2 sequential/dense/MatMul   kTfLiteFloat32  kTfLiteMmapRo      4        / 0.00 [1,1] [480, 484)
Tensor   3 sequential/dense_1/MatMul kTfLiteFloat32  kTfLiteMmapRo      64       / 0.00 [16,1] [336, 400)
Tensor   4 sequential/dense_2/MatMul kTfLiteFloat32  kTfLiteMmapRo      64       / 0.00 [1,16] [192, 256)
Tensor   5 sequential/dense_1/Mat... kTfLiteFloat32  kTfLiteMmapRo      64       / 0.00 [16] [0, 64)
Tensor   6 sequential/dense/MatMul1  kTfLiteFloat32  kTfLiteArenaRw     4        / 0.00 [1,1] [-1, -1)
Tensor   7 sequential/dense/BiasA... kTfLiteFloat32  kTfLiteArenaRw     64       / 0.00 [1,16] [-1, -1)
Tensor   8 StatefulPartitionedCall:0 kTfLiteFloat32  kTfLiteArenaRw     4        / 0.00 [1,1] [64, 68)
Tensor   9 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  10 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  11 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  12 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  13 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  14 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  15 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  16 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  17 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  18 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  19 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  20 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  21 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  22 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  23 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  24 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  25 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)
Tensor  26 (nil)                     kTfLiteNoType   kTfLiteMemNone     0        / 0.00 (null) [-1, -1)

kTfLiteArenaRw Info:
Tensor 0 has the max size 4 bytes (0.000 MB).
This memory arena is estimated as[0x55e3cc5bb104, 0x55e3cc5bb0c0), taking 68 bytes (0.000 MB).
One possible set of tensors that have non-overlapping memory spaces with each other, and they take up the whole arena:
Tensor 0 -> 8.

kTfLiteArenaRwPersistent Info: not holding any allocation.

kTfLiteMmapRo Info:
Tensor 3 has the max size 64 bytes (0.000 MB).
This memory arena is estimated as[0x7fb2de3f4648, 0x7fb2de3f4408), taking 576 bytes (0.001 MB).
One possible set of tensors that have non-overlapping memory spaces with each other, and they take up the whole arena:
Tensor 5 -> 4 -> 3 -> 2 -> 1.

kTfLiteDynamic Info: not holding any allocation.

Node   0 Operator Builtin Code   9 FULLY_CONNECTED (delegated by node 3)
  3 Input Tensors:[0,2,-1] -> 0B (0.00MB)
  1 Output Tensors:[6] -> 0B (0.00MB)
Node   1 Operator Builtin Code   9 FULLY_CONNECTED (delegated by node 3)
  3 Input Tensors:[6,3,5] -> 0B (0.00MB)
  1 Output Tensors:[7] -> 0B (0.00MB)
Node   2 Operator Builtin Code   9 FULLY_CONNECTED (delegated by node 3)
  3 Input Tensors:[7,4,1] -> 0B (0.00MB)
  1 Output Tensors:[8] -> 0B (0.00MB)
Node   3 Operator Custom Name TfLiteXNNPackDelegate
  6 Input Tensors:[0-5] -> 204B (0.00MB)
  1 Output Tensors:[8] -> 4B (0.00MB)

Execution plan as the list of 1 nodes invoked in-order: [3]
Among these nodes in the execution plan:
  Node 3 is a TfLiteXNNPackDelegate node (0x55e3cc5ba4f0), which has delegated 3 nodes: [0-2]
--------------Subgraph-0 dump has completed--------------

4. Question
How should I build the minimal.cc to use tflite models saved with SELECT_TF_OPS. The platform where I'll use my code is Linux. I created libtensorflow_flex.so using this tutorial , but now I don't know what should I do with created shared library. I am a complete newbie in the usage of tflite models with C++, so you have to assume that I don't know most of the well-known things :)
What steps should I do to get minimal.exe which runs smoothly with the aforementioned models?

Hi @gadagashwini ! Could you please look at this issue?

Regarding the error message that you got:
stderr (/home/user1/.cache/bazel/_bazel_user1/83d974b1842d484c79357d882b1fd6b9/execroot/org_tensorflow/bazel-out/_tmp/actions/stderr-2) exceeds maximum size of --experimental_ui_max_stdouterr_bytes=1048576 bytes; skipping :
To get a more detailed error message, try examining the referenced stderr file, in this case /home/user1/.cache/bazel/_bazel_user1/83d974b1842d484c79357d882b1fd6b9/execroot/org_tensorflow/bazel-out/_tmp/actions/stderr-2
Or alternatively, try building with the flag --experimental_ui_max_stdouterr_bytes=-1 passed to bazel.

@fergushenderson  Here is part of the stderr-2 from aforementioned file:
bazel-out/k8-opt/bin/tensorflow/core/protobuf/_objs/worker_proto_cc_impl/worker.pb.o:worker.pb.cc:function tensorflow::RecvTensorRequest::ByteSizeLong() const: error: undefined reference to 'tensorflow::DeviceLocality::ByteSizeLong() const'
bazel-out/k8-opt/bin/tensorflow/core/protobuf/_objs/worker_proto_cc_impl/worker.pb.o:worker.pb.cc:function tensorflow::RecvTensorRequest::ByteSizeLong() const: error: undefined reference to 'google::protobuf::Any::ByteSizeLong() const'
bazel-out/k8-opt/bin/tensorflow/core/protobuf/_objs/worker_proto_cc_impl/worker.pb.o:worker.pb.cc:function tensorflow::RecvBufRequest::ByteSizeLong() const: error: undefined reference to 'tensorflow::DeviceLocality::ByteSizeLong() const'
bazel-out/k8-opt/bin/tensorflow/core/protobuf/_objs/worker_proto_cc_impl/worker.pb.o:worker.pb.cc:function tensorflow::RecvBufRequest::ByteSizeLong() const: error: undefined reference to 'tensorflow::DeviceLocality::ByteSizeLong() const'
bazel-out/k8-opt/bin/tensorflow/core/protobuf/_objs/worker_proto_cc_impl/worker.pb.o:worker.pb.cc:function tensorflow::RecvBufRequest::ByteSizeLong() const: error: undefined reference to 'google::protobuf::Any::ByteSizeLong() const'

All of the errors are like this ...error: undefined reference to...

@BioWar! Did you try building  "--experimental_ui_max_stdouterr_bytes=-1 " flag too? I did not face any issue in the 2.8 version though with above config as add on to existing cmakelist.txt . Attaching gist and CMakeList.txt for reference. Thanks!

Here is output with default Bazel BUILD in the minimal directory:

How should I build the minimal.cc to use tflite models saved with SELECT_TF_OPS.

You just need to make sure that libtensorflow_flex.so is linked in.
This can be done in several different ways, depending on exactly how you are building minimal.cc.
If you're doing this in Bazel, adding "//tensorflow/lite/delegates/flex:delegate" to "deps" of the "minimal" cc_binary rule (as in your comment above) is the right approach; that will instruct Bazel to build the flex delegate and then link it into the binary.
The cause of the "undefined reference" errors that you got is not immediately obvious to me,
but I suspect the most likely cause is a missing dependency (a missing entry in the "deps") in one of the BUILD files.  Find the source code that defines the missing symbols, find the build target that contains those sources, and make sure that there is a dependency on that build target from the build target for the source file that references those symbols.


App crashing randomly

I admit your hard-work.  So far the best open source code to work on binary media.
But app is crashing at many places.
I am using pixel 3.

yes, I have not undergone large-scale testing

Hi @wlanjie
Am too Facing same issue on my pixel 2 xl.. any update or resolution to it.. if fix provided it will be really great.
Thanks in Advance

if possible request you to provide quick fix..
Thanks

Closing this issue New Lib working Great.

@phoenixvk hey possible to provide working sample code?


ENH: Export current view as CSV

Fixes #77

This should wait on our discussion around TSV vs CSV before merging. As well, I want to point out that the strategy employed in this PR is not to replace the JSONP with TSV/CSV (this departs from our discussion a few weeks ago). Because q2cli still uses the file:// protocol to load visualizations, we are stuck operating in a pretty limited environment. For loading JSON and TSV in this visualization, we have two options: embed the data directly into the HTML, or, the data+padding approach. Ideally we could have replaced the JSONP with TSV, but we are stuck with padding (which would require us to modify the TSVP client-side when someone attempts to download the vanilla TSV, to strip the padding). That seemed like a lot of work, and if we wind up refactoring q2cli to utilize some kind of server (or q2view), this won't matter. Similarly, embedding the TSV directly into the HTML would work, but again, would require a bunch of client-side manipulation to present the user with with cleaned up data. For now, this PR ships both the TSV and the JSONP as-is, which seems acceptable to me.

Thanks @ebolyen, I meant to clarify that in my comment above: the TSV is wholly disconnected from the actual plot (for now), with the exception of the selected level. Otherwise, this is ready for merge.


Debug: Scheme is missing

Steps to Reproduce:

debug a hello world extension
step into code that is not from the extension

=> uri.ts:19 [UriError]: Scheme is missing: {scheme: "", authority: "", path: "Unknown Source", query: "", fragment: ""}
Possibly from somewhere around here: https://github.com/Microsoft/vscode/blob/e055c6ef94e024405f34c0cade76886d42d3fa77/src/vs/workbench/parts/debug/common/debugSource.ts#L50

We would have to add more checks to verify if it has a scheme.
Also adding @weinand

Uri.parse is causing a ton of these in my debug console since v1.27. The problem is that I am using it to validate a URI by checking if scheme is populated. I assume it is expecting that validation to be done prior to parsing, but if so what is the best way to do this? There is no Uri.validate or similar function in the API from what I can tell.

Fixed via 5717c6a539


pretrained model

Hi, can you provide a trained model，thanks

Trained model is available (Note: Please use the newest code!):
Baiduyunpan:
Link:  https://pan.baidu.com/s/1TfeYpkyRORPFLktS9REc2g
Password: 5fe1

@WSTao
Trained model is available (Note: Please use the newest code!):
Baiduyunpan:
Link: https://pan.baidu.com/s/1TfeYpkyRORPFLktS9REc2g
Password: 5fe1

@happinesslz Can you post link to Google Drive / Dropbox? Even with the password, it seems we need to register pan baidu account which may require China mainland phone number.

@jhultman  Ok! Thanks for you suggestion!  I will register Google Drive and add the link to README.

@jhultman
Google Driver Link: https://drive.google.com/drive/folders/1jSuQy5svfkDBw_4JszSzmIp9tpH1Ox7f

@happinesslz Thanks so much!


Search result titles should be on same line as document counter.

Staging:

Production:

Doesn't seem to be a problem currently.


Small error

Dear author,
Good evening. Your project seems interesting, but during the "python3 wallofshame.py --start traffic", I check the following scenario:

Do have any idea about this issue? I've created the user, granted permission and create the API Token.
Thank you for your attention.

Hey @alexandreborges,
Sorry for the delay in response. Thanks for letting us know this. Well, as far as the issue is concerned, it looks like there has been some issue with the API token or framework communicating with the pineapple. I suggest you to try contacting the pineapple(retrieving client details etc.) using the API Token using the modules from pinemodules folder. Feel free to reach out if the issue still persists.
Thank You.


user exit: event handler

abapGit is missing an easy way to extend it with new features that are not suitable for abapGit itself. For example, it would be nice to link to a new page from a wall message (#4653). Another use case would be redirecting menu items to a custom page rather than standard abapGit, for example using a company-specific solution to replace "Advanced > Run Code Inspector".
Suggestion: Add a user-exit to the event handler
CLASS zcl_abapgit_gui_router
 METHOD zif_abapgit_gui_event_handler~on_event.

    " Custom event handling
    rs_handled = zcl_abapgit_exit=>get_instance( )->on_event( ii_event ).

In the exit, you have the option to react to any abapGit UI event. This could be a new event like a "sapevent:my_action" link in a wall message to processing an existing event like "sapevent:repo_code_inspector" for the example above.

Could be a nice feature. By the way I'd add some kind of indication which exits are there and activated. Somewhere at debug page, or maybe a small indication at main. Not sure. I mean exits (unexpected behaviour) are usually very confusing for people, so we should indicate it somehow


Issue with cloud.writeFile

Hi there, big thanks for replacing greenworks. This looks like it will be way easier to use. I'm having a basic issue with steam cloud. I have my app configured in steamworks with 5mb and 10 files max. I'm trying to save a single file that is a short string.
cloud.writeFile('sp-save', 'test')
always returns false. I don't know what that means other than the write failed. I can confirm it failed by noting that
cloud.readFile('sp-save')
returns nothing. How do I troubleshoot this knowing that my app is configured correctly in steamworks?
On steamworks.js boot I do see this:
Setting breakpad minidump AppID = (my app id)
SteamInternal_SetMinidumpSteamID:  Caching Steam ID:<PHONE_NUMBER>2801299 [API loaded no]
Does this mean anything?

I solved this issue by removing the '-' character from my file name.


WSL closes immediately

Environment
Windows build number: Microsoft Windows [Version 10.0.18363.1139]
Your Distribution version: Ubuntu 20.04
Whether the issue is on WSL 2 and/or WSL 1: WSL 2 (Linux version 4.19.128-microsoft-standard (oe-user@oe-host) (gcc version 8.2.0 (GCC)) #1 SMP Tue Jun 23 12:58:10 UTC 2020)

Steps to reproduce
Running: "Open Linux shell here" from context menu, run "wsl" from cmd or PowerShell.
Expected behavior
Shell opens. Shell opens at the location I selected "Open Linux shell here".
Actual behavior
This results in a window which closes immediately. No possibility of interacting or even taking a look a the error message.
If running in the command line this is followed by no response.
wsl only works if distro is mentioned explicitely: wsl -d ubuntu

Which is your default distro?

Which is your default distro?

Or better, output of wsl.exe -l -v.

@therealkenc You are thinking what I am thinking.

I have the same issue.
` wsl.exe -l -v
NAME      STATE           VERSION

Legacy    Running         1`

What is Legacy?

Well that's interesting, insofar as wsl.exe -d ubuntu launches successfully, but Ubuntu is not in your list. Which is at least a little like #6024.
You might try:
C:\> wsl.exe --set-default Ubuntu

Assuming that takes, Ubuntu will be your default. You won't need the -d parameter to wsl.exe, and with fingers crossed, it might launch from the context menu as well.
Before doing so, out of an abundance of caution, you might want to backup your Ubuntu distribution with wsl.exe --export Ubuntu <filename> (if you have any valuable data in there). You can also try installing another distribution with a different name (say Ubuntu-20.04) to see if it shows up in the wsl -l -v list. Then set Ubuntu-20.04 as the default.

No possibility of interacting or even taking a look a the error message.

Trying running wsl.exe -d Legacy from a command prompt. Presumably the Legacy distro will fail to launch.
If it fails to launch, and you like, collect WSL logs and post the backlink to the feedback item here. Although in the circumstance, you are probably shooting for WSL2 Ubuntu anyway.

What is Legacy?

It is the distribution that used to ship with the now-deprecated bash.exe. Ref some commentary in #4717, including --export and --import.

Same thing happens when your default distro is docker-desktop or docker-desktop-data.

Hi, I have the same issue
WSL is closing immediately
wsl.exe -l -v
NAME            STATE           VERSION

Ubuntu-18.04    Stopped         1

Please advise how to fix this

@ktre2017 Does running wsl -d Ubuntu-18.04 work?

@ktre2017 Does running wsl -d Ubuntu-18.04 work?

I tried but it didn't work.

@therealkenc  thanks! basically the default distro was docker

The same thing happened to me.
I was able to find the problem after running as root:
 wsl -d Ubuntu-22.04 --user root

Then I did:
su my_username

And I saw that there was a problem with my shell prompt.
I fixed it, and then everything went back to normal.

Not sure if any can help me with my issue. So I just got a refurb replacement note 8, and the WSL is not opening. I can't even install Ubuntu, I get this error for that,

In my case, I didn't have Ubuntu distribution.
So, I installed it, using this command:
wsl --install -d Ubuntu

Then I used the following command to set the Ubuntu distribution as default:
wsl --set-default Ubuntu

Other useful commands:
# To list all installed distributions
wsl --list --verbose
wsl -l -v

# To see the available distributions to install
wls --list --online

I can't install ubuntu distro using my wsl and open it . I tried to install the Ubuntu using this command
wsl --install -d Ubuntu

And then the Ubuntu shell opens and then it asks to press a key to continue but when I press the key the window immediately closes. And when I try to open again it shows you have no distros installed.
Windows Subsystem for Linux has no installed distributions.
Distributions can be installed by visiting the Microsoft Store:
https://aka.ms/wslstore

I can't install ubuntu distro using my wsl and open it . I tried to install the Ubuntu using this command
wsl --install -d Ubuntu

And then the Ubuntu shell opens and then it asks to press a key to continue but when I press the key the window immediately closes. And when I try to open again it shows you have no distros installed.
Windows Subsystem for Linux has no installed distributions.
Distributions can be installed by visiting the Microsoft Store:
https://aka.ms/wslstore

I tried listing the distros using this command
wsl --list --verbose

but it shows no distros installed.

Finally solved this doing a kernel file from the wsl docs
https://learn.microsoft.com/en-us/windows/wsl/install-manual#step-4---download-the-linux-kernel-update-package

@therealkenc Thank you!

Same same here.
Win 11 just updated.
Freshly installed WSL Debian from the store.
wsl-exe -l -v
issues

Debian Stopped 1

wsl -d debian
issues
Fehler beim bereit stellen eines Ihrer Datei Systeme. Führen Sie "dmesg" aus, um weitere Details anzuzeigen.
Which translate to
Error mounting one of your file systems. Run dmesg to see more details.
All partitions are clean, the last one is Ext4

Had this problem to. Probably because some change to WSL as default.
Solution: If you already have Ubuntu installed
RUN THIS IN POWERSHELL
wsl --set-default Ubuntu


vendor-values: only hide value for the merchant and bag frames

Usecase:

open merchant frame
open crafting frame
try to observe vendor sell value on any item in crafting frame

Thank you :+1:


Incorporate/Store User Feedback

This is a feature request that will help customers provide feedback on the chatbot's accuracy.
For example:

[ ] Thumbs up and down on top right corner of the response.
[ ] If user clicks on thumbs up/down, it stores that feedback in DynamoDB under the user session table. Data could also be stored directly to S3/Athena/OpenSearch for analytics.

Any option can work as long as there is a db/storage for the feedback that can be queried later on.

Duplicate of #151


"extension_" must be pre-pended to any name you want to use

This may seem like a duplicate of https://github.com/MicrosoftDocs/azure-docs/issues/34587 but it is not. You cannot say this doc is correct because it works as listed on the doc. People come here looking for direction on how to do this. I look at this and say to myself "hey let's add gender" to this. In fact on this page: https://docs.microsoft.com/en-us/azure/active-directory-b2c/claimsschema they actually give a close example we can modify which I list below:
<ClaimType Id="city">
<DisplayName>city where you work</DisplayName>
<DataType>string</DataType>
<UserInputType>DropdownSingleSelect</UserInputType>
<Restriction>
<Enumeration Text="Bellevue" Value="bellevue" SelectByDefault="false" />
<Enumeration Text="Redmond" Value="redmond" SelectByDefault="false" />
<Enumeration Text="New York" Value="new-york" SelectByDefault="true" />
</Restriction>
</ClaimType>
Now lets just say I change to gender and make changes to the rest. No matter what is done - THIS WILL NOT WORK. It is not until extension_ is prepended to the word gender will this actually work. If you need to test use the above (hell don't change it to gender just use city).
I have blown nearly two days on this. This has to be called out I the docs somewhere. If it is a requirement then please call it out as such.

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

ID: 29fb9816-df6a-6f60-35d0-c2a04732f295
Version Independent ID: ffc8b13a-b2c9-a502-1580-abea596194ab
Content: Add your own attributes to custom policies in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/active-directory-b2c-create-custom-attributes-profile-edit-custom.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

@dwhiteddsoft Thanks for the feedback. I am testing this in my environment and will get back to you with my findings.


Some types can not be serialized in IL2CPP

Bug description:
Some types, like arrays and enums, can not be serialized when building to IL2CPP.
Expected behavior:
It should serialize.
Additional context:
Can be solved by implementing a resolver that points the types to an ArrayFormatter, GenericEnumFormatter, etc.

Wrong issue.

Has been fixed.


gramaddict 3.26 error

What happened:
ERROR | stat /data/app/com.instagram.android-8d-T-gaUvYpezJ-qxgqfsg==/base.apk
package:/data/app/com.instagram.android-8d-T-gaUvYpezJ-qxgqfsg==/split_arservicesforpersonsegmentation.apk
package:/data/app/com.instagram.android-8d-T-gaUvYpezJ-qxgqfsg==/split_pytorch.apk
package:/data/app/com.instagram.android-8d-T-gaUvYpezJ-qxgqfsg==/split_supmediastreamcontroller.apk: no such file or directory
What you expected to happen:
gramaddict executes normally
How to reproduce it (as minimally and precisely as possible):
run gramaddict
Anything else we need to know?:
Environment:

GramAddict version:
Device Model/Emulator Type:
Android Version:
Instagram Version:
Discord Ticket ID:           *(if submitting crash log)

Relevant Logs:

*Note: if you have a crash log, please do not attach the archive here as this is not a secure place to upload the sensitive data inside. Please open a ticket in Discord in #lobby and provide the ticket ID here.

Hello, I'm aware of this problem, it's due to an atx-agent bug, https://github.com/openatx/atx-agent/pull/111
in the meantime, I'll move to monkey mode (there is already a PR for that)

We're can I find the fix please ?
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Dennis @.>
Sent: Saturday, September 30, 2023 6:40:26 PM
To: GramAddict/bot @.>
Cc: roncam82 @.>; Author @.>
Subject: Re: [GramAddict/bot] gramaddict 3.26 error (Issue #343)
Hello, I'm aware of this problem, it's due to an atx-agent bug, openatx/atx-agent#111https://github.com/openatx/atx-agent/pull/111
in the meantime, I'll move to monkey mode (there is already a PR for that)
—
Reply to this email directly, view it on GitHubhttps://github.com/GramAddict/bot/issues/343#issuecomment-1741810247, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALBM2Q3JLLIHDJWTVK2U6LTX5BDPVANCNFSM6AAAAAA5NUIA6E.
You are receiving this because you authored the thread.Message ID: @.***>

Whats monkey mode?

Whats monkey mode?

https://developer.android.com/studio/test/other-testing-tools/monkey


Import: URIs containing un-escaped \ behave differently on Windows, Unix

Some misbehaving glTF producers (eg. assimp) may produce invalid URIs containing \. When imported, backslashes pass through urllib.parse.unquote unchanged and get passed on through open to the OS. On Windows the \ will work as a folder separator and the URI will silently work. On Unix it will be treated as a normal filename character and (probably) fail.
The behavior should be the same on both OSes.
Note that the glTF validator does not flag URIs with a backslash as invalid. It treats it as though it had been properly percent-encoded, %5C.
Solution
I think the URI should be repaired by replacing backslashes with forward slashes, ie uri = uri.replace('\\', '/') before unquoting. The reasons:

This is what browsers do when they find a backslash. It's what the WHATWG's URL spec says to do.
Backslashes are almost certain to be improperly encoded Windows folder separators (they are not common in Unix filenames).

The alternative is to raise an error and refuse to use the path.
Also the _uri_to_path function used for loading images should be moved out so it can be used for .bins as well. The change would go there.

Hello,
Do you have an example of glTF that have different behaviour on linux and Windows?
Using linux here:
I just tried to modify one with 1 \, but of course the json library complains that json is not valid (on linux again)

Testing with \\, and importer works without any error (on linux again)

Try a .bin. Images go through Blender's own file loading code, which will treat \ as /.
    "buffers" : [
        {
            "uri" : "data\\somefile.bin"
        }
    ]

Proposition of fix in #1834


Added offline handling for all pages in router

Hey @Anandsg I've added the offline handling logic for Router. Though for styling the offline page I'll need some ideas. Do you have any svgs to render for the offline page? As only svg can be rendered in the offline page. Since there's no network, no image can be rendered with img tag. Also thinking of adding a reload button. Any thoughts or suggestion would help

Gotcha. Please check the updated PR @Anandsg


Have the ability to make IBM tests mandatory/optional for PR merging

/kind feature
Which functionality do you think we should add?

[ ] Have the ability to make tests mandatory/optional for PR merging

Why is this needed?
notes
It would be nice to first have #5487 before making test required
https://docs.ci.openshift.org/docs/

Closing this issue as we have made IBM cloud mandatory for PRs.
/close


Service disruption

Changing the namespace scenario to delete objects in the namespace instead of the whole namespace to not need to edit the namespace to remove the finalizer.
These changes will be able to list object types and the ability to remove the larger object types (deployment, replica set and statefulset) as well as services

@paigerube14 let's update docs with scenario name as well as functionality changes: https://github.com/redhat-chaos/krkn/blob/main/docs/namespace_scenarios.md#delete-namespace-scenarios. Also here: https://github.com/redhat-chaos/krkn#kubernetesopenshift-chaos-scenarios-supported. Thoughts?


Conditional requirements

Conditional requirements must be implemented in requirements() method not in configure:
    def configure(self):
        del self.settings.compiler.libcxx

    def requirements(self):
        if not self.options.no_electric_fence and self.settings.os == "Linux":
            self.requires.add("electric-fence/2.2.0@lasote/stable", private=True)
        else:
            if "electric-fence" in self.requires:
                del self.requires["electric-fence"]

        if not self.options.no_zlib:
            self.requires.add("zlib/1.2.11@lasote/stable", private=False)
        else:
            if "zlib" in self.requires:
                del self.requires["zlib"]

This can cause issues for users that are using version_ranges. Reported for 1.0.2k but might affect other versions

Fixed in 1.0.2j and 1.0.2k recipes

@lasote: Is this issue the reason why I cannot install libcurl without ssl support: https://github.com/lasote/conan-libcurl/blob/release/7.50.3/conanfile.py#L50

@jonico what is the issue? are you specifying with_openssl=False ?

@lasote: I tried to build with libcurl:with_openssl=False

@jonico, got it. Try now, I've updated the recipe in conan.io. Use install -u to update your local recipe or delete it first.

@lasote: thanks, works now 🎉


License

Hi,
Thanks for releasing F5 TTS! I noticed the license is CC-BY, which is not normally used for code. Are there any plans to switch to a code license (i.e. MIT/Apache/ISC)?
Thanks!

Sure,
I will switch to MIT if it would bring more convenient use.

Thanks!

Hi,
Just to clarify, are the models also released under the MIT license?
Thanks!

CC-BY-4.0 on HF ckpt repo, though I thought to be almost the same as MIT license (as there's no NC or SA)

CC-BY-4.0 on HF ckpt repo, though I thought to be almost the same as MIT license (as there's no NC or SA)

@fakerybakery Hi, we're sorry that we made a mistake earlier. We just realized that Emilia is using cc-by-nc-4.0, mainly for it's an in-the-wild dataset, and the copyright should goes to the original owners of the videos or audio.
We think we should abide by the protocols, so we change our ckpt repo to cc-by-nc-4.0. Sorry again for any inconvenience this may cause. Our codebase will keeps with MIT license.

@SWivid regarding the licence and the Emilia dataset, the team behind it (who released the dataset itself) https://huggingface.co/datasets/amphion/Emilia obviously don't feel training on the in the wild data makes the models themselves cc-by-nc-4.0 as their own MIT licensed amphion/MaskGCT is MIT licensed.
Also the big player in the SST world, Whisper https://github.com/openai/whisper was trained on almost entirely in the wild data (including YouTube from how often it hallucinates to "please like and subscribe") and is licensed as MIT.

Hi @kylehowells ~
I think we should follow the basic rules, even though some people don't.
We don't actually restrict where the model can be used, the protocol is just to follow the rules. Please be assured that we will not cause you any trouble~

Does this mean this model could be used for commercial purposes or not, I got a bit confused while reading it I see a lot of models use the Emilia and change license.


fix: create service account credentials from json contents

Added code for validating private key before making cloud operations.

This change is

Why don't you start by reproducing the problem?  Start with this test:
https://github.com/googleapis/google-cloud-cpp/blob/82ce9d1b41b5d8ba9fa535922ceca8aa95507a2d/google/cloud/storage/oauth2/service_account_credentials_test.cc#L775-L799
Create a copy that uses an invalid key and see how it behaves?  Does the MakeJWTAssertion() raises an exception with an invalid key? If it does, then we already have enough validation, we just need to perform this validation as part of ParseServiceAccountCredentials().  If it does not, then maybe we should add code like the one in this PR.

Yes, you are correct!
MakeJWTAssertion() already handles an invalid key. I have refactored  code to handle invalid key in CreateServiceAccountCredentailsFromJsonContents as reported in #5315. kindly review.

Please review code changes.

clang-tidy is asking you to reformat the code. Some changes look unnecessary to me, but maybe I am missing something.

Reviewed 1 of 7 files at r6, 4 of 4 files at r7.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @amitchauhan1007)

google/cloud/storage/oauth2/service_account_credentials_test.cc, line 383 at r7 (raw file):

      auto header_bytes =
          storage::internal::UrlsafeBase64Decode(encoded_header);

are these changes still needed?

These changes not required anymore. Updated code by removing "storage::" namespace prefixes.


fix(witness): witness should not error on an empty git repo with no commits

Fixes https://github.com/testifysec/witness/issues/275
When witness runs on an empty git repo without commits, it shouldn't error with reference not found
reproduction

start a new git repo, do not make any commits
run witness on it
observe witness fail

acceptance criteria

witness doesn't fail on step 3

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 48.494%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
1433

Relevant Lines:
2955

💛  - Coveralls


Editing the input field do not update the calender.

Is there a way an edit in the input field results in the calendar to update itself as well?

yes this is work in progress

Great ! Any idea on the estimate release?

i am not promising but i am working, also have hell lot of work from Office :(

I understand. This is great work you have already put up. Thanks !

Up!

@mominsamir Ar you planning to release this as a . version?

version 2.0.5

@mominsamir  This needs to be reopened. The problem is the calender gets the updated value but the ng-model do not get updated. I can submit a pr with my work around on this.

Please submit ur pr and i will meagre it
Sent from my iPhone

On Dec 14, 2016, at 7:23 AM, PraveenKumar Elankovan<EMAIL_ADDRESS>wrote:
@mominsamir This needs to be reopened. The problem is the calender gets the updated value but the ng-model do not get updated. I can submit a pr with my work around on this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I don't know how can I submit a PR. But here is the solution:
On picker.js,
while construction the template, add a ng-change to input tag, say data-ng-change="vm.onDateEdited(vm.value)"
Here is how the template construction looks like:
if(attributes.hasOwnProperty('onFocus')){
inputType =  '<input name="{{vm.fname}}" ng-model="vm.value" '
+ '  type="text" placeholder="{{vm.label}}" data-ng-change="vm.onDateEdited(vm.value)"'
+ '  aria-label="{{vm.fname}}" ng-focus="vm.show()" data-ng-required="vm.isRequired"  ng-disabled="vm.disable"'
+ '  server-error class="sm-input-container" />'
+ '     '
+ '     today' ;
      }else{
         inputType = '      <input class="" name="{{vm.fname}}" ng-model="vm.value" '
                  + '             type="text" placeholder="{{vm.label}}" data-ng-change="vm.onDateEdited(vm.value)"'
                  + '             aria-label="{{vm.fname}}" aria-hidden="true" data-ng-required="vm.isRequired"  ng-disabled="vm.disable"/>'
                  + '     <md-button tabindex="-1" class="sm-picker-icon md-icon-button" aria-label="showCalender" ng-disabled="vm.disable" aria-hidden="true" type="button" ng-click="vm.show()">'
                  + '         <svg  fill="#000000" height="24" viewBox="0 0 24 24" width="24" xmlns="http://www.w3.org/2000/svg"><path d="M7 10l5 5 5-5z"/><path d="M0 0h24v24H0z" fill="none"/></svg>'
                  + '     <i class="material-icons md-light">today</i></md-button>' ;
      }

And write a change handler at its controller.
SMDateTimePickerCtrl.prototype.onDateEdited = function (date) {
var self = this;
if (moment(date, self.format).format(self.format) !== "Invalid date") {
self.currentDate = date;
self.ngModelCtrl.$setViewValue(date);
self.ngModelCtrl.$render();
self.value = date;
}
}


[BUG] AttributeError: 'WebDriver' object has no attribute 'find_element_by_name'

[LOGIN] Logging-in...
[LOGIN] Writing email...
Traceback (most recent call last):
File "C:\Users\david\Desktop\Microsoft-Rewards-Farmer-master\ms_rewards_farmer.py", line 773, in
login(browser, account['username'], account['password'])
File "C:\Users\david\Desktop\Microsoft-Rewards-Farmer-master\ms_rewards_farmer.py", line 46, in login
browser.find_element_by_name("loginfmt").send_keys(email)
AttributeError: 'WebDriver' object has no attribute 'find_element_by_name'
Some fix?

I think this has happened because Selenium has deprecated the function "find_element_by_name". Sadly, I cannot fix this right now, but hopefully someone else will soon.

As a temporary fix, put this in terminal: pip install -Iv "selenium===4.2.0"
This downgrades selenium to a prior version where the error won't appear anymore. However, you will still get a ton of DeprecationWarning warnings. Don't worry, it will work nonetheless.
I will make a pull request to fix this.

This got fixed in the last commit, pull the latest changes and check if the problem still arises.

After doing that it worked for a few days then stopped working again showing the same error.

Same here. I still get this error atm.
[LOGIN] Logging-in...
[LOGIN] Writing email...
Traceback (most recent call last):
File "C:\Users\jonat\Downloads\Microsoft-Rewards-Farmer-master\Microsoft-Rewards-Farmer-master\ms_rewards_farmer.py", line 773, in
login(browser, account['username'], account['password'])
File "C:\Users\jonat\Downloads\Microsoft-Rewards-Farmer-master\Microsoft-Rewards-Farmer-master\ms_rewards_farmer.py", line 46, in login
browser.find_element_by_name("loginfmt").send_keys(email)
AttributeError: 'WebDriver' object has no attribute 'find_element_by_name'

After doing that it worked for a few days then stopped working again showing the same error.

Someone fixed the bug you are having. It's already merged into source; the creator just need to re-release the update. Selenium depreciated the attribute "find_element_by_*"

just go to requirements.txt and replace selenium with selenium==3.14.1

someone re created the bot https://github.com/MehdiRtal/MicrosoftRewardsBot.git

Fixed with v3.0 #188


Added tests for Array2 impls of ArgminConj

Codecov Report
Base: 93.38% // Head: 93.44% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (793d2cd) compared to base (cbc1d15).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #302      +/-   ##
==========================================
+ Coverage   93.38%   93.44%   +0.05%
==========================================
  Files         119      119
  Lines       18914    18939      +25
==========================================
+ Hits        17663    17697      +34
+ Misses       1251     1242       -9

Impacted Files
Coverage Δ

argmin-math/src/ndarray_m/conj.rs
100.00% <100.00%> (+15.51%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Running synthesized files through Tax-Calculator

Issue:

We are creating a synthesized file from the raw PUF for 2011.
We'd like to compare certain complex variables computed from the synthesized data with their counterparts in the raw data, to help us evaluate file quality.
Most notable are adjusted gross income and various tax measures.
They are too complex to calculate from scratch, so we plan to run the synthesized and raw files through Tax-Calculator and compare the resultant computed variables.
The first tax year we can compute in Tax-Calculator is 2013, not 2011.

Proposed solution:

I propose to (and have started to) run the raw and synthesized files through Tax-Calculator with the tax year set to 2013, and will examine the results.
There were some tax-law changes between 2011 and 2013, the biggest of which were top-rate increases. However, I think this approach is reasonable nonetheless because the 2013 law would be applied to both the raw 2011 PUF and the synthesized 2013 PUF. Our concern will be how far apart the two files are, not how different the two tax laws are.
-The approach has the advantage of being tractable and non-cumbersome.

Two questions, especially for @andersonfrailey:

Do we think this makes sense?
I'm thinking that we should do the same for the 3 tax reforms @andersonfrailey put together (run reforms vs. "current law" defined as 2013 law). Does that makes sense? @andersonfrailey, to do that, do I just need to change 2019 to 2013 in the json files?

@feenberg, are you using Tax-Calculator, and if so what tax year are you using?

@donboyd5 I think this makes sense. And you're correct that all you would need to do is change 2019 to 2013 in the reform files.

On Wed, 12 Dec 2018, Don Boyd wrote:

@feenberg, are you using Tax-Calculator, and if so what tax year are you
using?

I am using taxcalc.sas, and comparing to 2011. Is the synthetic data for a
later year? If so what can I use for comparison? I think we should
simulate the same year we have data for, and then age that, not age first
and then simulate. Otherwise testing is rather confusing.
I will have results tomorrow.
dan

That's good - 2011 is the synthesis year.

I have added are a large number of non-calculated variables to the tax reforms previously tested.
synscore4.txt
That is for each of the income and deduction items mentioned, the table shows the difference in the revenue score between PUF and synth4 when $100 is added to that item. I am still working on getting standard errors on the PUF scores.

Apparently Github doesn't like the Linux newline indicator. Here it is with Windows newlines:
synscore4.txt


Parse Issue: "mon-wed every 7 weeks"

The description "mon-wed every 7 weeks" was supposed to be parsed properly, but wasn't.

Example server simply needed to be updated from new API.


Need ability to pass commandline arguments to extension

I'd like a way to launch VSCode with commandline arguments and to have a way to my extension to access these value pairs.

Not the same but you can register a uri handler: https://code.visualstudio.com/api/references/vscode-api#window.registerUriHandler

This is currently beyond our API scope.

🤷 I don't own the feature...

This would be a nice feature indeed but we solved it by having the launcher process update the user profile settings.json file before launch and passing in parameters that way. Works but maybe not super elegant.


Look into implementation/use of revocation-url

louketo seems to be calling the end_session_endpoint on keycloak, which is not the revocation endpoint, I believe? Keycloak seems to have an undocumented revocation endpoint at /revoke, according to https://github.com/keycloak/keycloak/pull/6704.
If this is more of a logout endpoint than revocation endpoint, we should rename the config variable and improve docs.
Token revocation is defined by https://tools.ietf.org/html/rfc7009

when i check my keycloak and also on stackoverflow, keycloak endpoints were:
    "issuer": "http://localhost:8080/auth/realms/demo",
    "authorization_endpoint": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/auth",
    "token_endpoint": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/token",
    "userinfo_endpoint": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/userinfo",
    "end_session_endpoint": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/logout",
    "jwks_uri": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/certs",

so there wasn't any explicit revocation endpoint
also according this issue https://github.com/louketo/louketo-proxy/issues/662, keycloak 10.0.0 introduces new revocation endpoint, however you can set any revocation endpoint through option, we can set new keycloak revocation endpoint as default

when i check my keycloak and also on stackoverflow, keycloak endpoints were:
    "issuer": "http://localhost:8080/auth/realms/demo",
    "authorization_endpoint": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/auth",
    "token_endpoint": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/token",
    "userinfo_endpoint": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/userinfo",
    "end_session_endpoint": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/logout",
    "jwks_uri": "http://localhost:8080/auth/realms/demo/protocol/openid-connect/certs",

so there wasn't any explicit revocation endpoint
also according this issue https://github.com/louketo/louketo-proxy/issues/662, keycloak 10.0.0 introduces new revocation endpoint, however you can set any revocation endpoint through option, we can set new keycloak revocation endpoint as default

as per keycloak implementation https://github.com/keycloak/keycloak/pull/6704, they support in token parameter only refresh tokens, as found loginHandler doesn't provide refresh token in cookies so if logging in via loginHandler we are unable to logout because we don't have refresh token (unless enabled store, didn't check if that's not buggy...)

as per keycloak implementation https://github.com/keycloak/keycloak/pull/6704, they support in token parameter only refresh tokens, as found loginHandler doesn't provide refresh token in cookies so if logging in via loginHandler we are unable to logout because we don't have refresh token (unless enabled store, didn't check if that's not buggy...)


token auth not work well for non-persistent topic get

Describe the bug
This happens in 2.3.2
user report token auth not worked well for non-persistent topic get
when token auth configured, get tentant and get namespace both worked well, but get non-persistent topic failed:

dup with #4560 will close this one


Yarn Warning on Plugin Install Prevents Plugin Installation

Describe the bug
Yarn Warning on Plugin Install Prevents Plugin Installation.
To Reproduce
Steps to reproduce the behavior:

Go to the "response regex" page https://insomnia.rest/plugins/insomnia-plugin-response-regex on the Insomnia Plugin Hub
Click on Install Plugin
Click on install when prompted by Insomnia to install the plugin
See error: "Yarn error warning insomnia-plugin-response-regex > insomnia-xpath ><EMAIL_ADDRESS>Deprecated due to CVE-2021-21366 resolved in 0.5.0"

Expected behavior
Plugin installs
Screenshots

Desktop (please complete the following information):
Version: Insomnia 2021.2.1
Release date: 3/24/2021
OS: Windows_NT x64 10.0.19042
Electron: 9.3.5
Node: 12.14.1
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.73.0-DEV OpenSSL/1.1.1g zlib/1.2.11 brotli/1.0.9 WinIDN libssh2/1.9.0_DEV nghttp2/1.41.0
Additional context
Add any other context about the problem here.

I was researching another issue I am having with Insomnia and realized they might have the same root cause so I thought I would post the details here and if I'm wrong we can split the issue.
The other issue I am having is in using xpath to parse a response.  In the Insomnia debug console I get the following error:
[xmldom error] entity not found:
When I searched on this error I found the following resent issue raised in the xmldom repository:
https://github.com/xmldom/xmldom/issues/57.
Based on the issue and the output of the YARN warning it looks like updating the xmldom package Insomnia is using (I couldn't figure out how to tell when build was in use) to the current 0.5.0 build might solve both issues.
If there is a way for me to do this in my build of Insomnia I'm willing to try but I'll need some detailed instructions on how to do that.

Looks like plugin has a dependency on insomnia-xpath, which includes the older version of xmldom that you mention. I reproduced the issue.
The fix from that issue on xmldom seems to imply we would need to update to at least 0.4.0 in order to absorb the fix.

Dependabot has opened a PR for this already - https://github.com/Kong/insomnia/pull/3188 - so we'll try get it in.
On a related note though, plugin installation in Insomnia happens through a bundled Yarn standalone runner. Its response is parsed as having failed installation even if there are only warnings (such as this case). Ideally we should improve the install process as well to not fail completely on yarn warnings, but that's unrelated to this particular issue which would be resolved by the dependabot PR.


Add analysis report

At the bottom add a report of the test, and/or a button to generate this
e.g
"Test type: between subjects test
Power = 0.8
Effect size = 0.3
Total required sample = 160 (80 in each group)
For details on the calculation see [[link]]

A nice idea but not sure we've got time to implement for this release. Is that cool?

Yes, let's leave this


add support for context aware handler

We would like access to the aws-request-id header that is returned by the runtime API.
Our specific use case is that we use the request id to correlate log entries generated by this lambda invocation. Initially we generated our own correlation id but we noticed we can't link them to the cloudwatch entries but we can link them if we use the lambda request id as the correlation id.

That's really interesting!
I think we can do without the ContextAwareHandler interface. Indeed it is possible to invoke a function with "extra" arguments. If the function doesn't use them or even declare them they will simply be ignored.


UI improvement

Changes

Created separator in CompoundIndex
Improve UI for display filter
Implement Add Item popup
Implement Modify Item popup

Related Issues/ PRs

Fixes #99 #100 #107

Codecov Report

Merging #129 (4e75d30) into develop (7d2529b) will decrease coverage by 1.41%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop     #129      +/-   ##
=============================================
- Coverage      54.17%   52.76%   -1.42%
  Complexity       459      459
=============================================
  Files             98      100       +2
  Lines           1905     1956      +51
  Branches         183      178       -5
=============================================
  Hits            1032     1032
- Misses           807      858      +51
  Partials          66       66

Impacted Files
Coverage Δ

.../seedu/ibook/commons/core/index/CompoundIndex.java
0.00% <ø> (ø)

.../seedu/ibook/logic/commands/DeleteItemCommand.java
0.00% <ø> (ø)

.../seedu/ibook/logic/commands/UpdateItemCommand.java
0.00% <ø> (ø)

...main/java/seedu/ibook/logic/parser/ParserUtil.java
76.19% <0.00%> (ø)

src/main/java/seedu/ibook/ui/CommandBox.java
0.00% <ø> (ø)

src/main/java/seedu/ibook/ui/MainWindow.java
0.00% <0.00%> (ø)

...c/main/java/seedu/ibook/ui/control/ControlBox.java
0.00% <0.00%> (ø)

src/main/java/seedu/ibook/ui/control/Filter.java
0.00% <0.00%> (ø)

src/main/java/seedu/ibook/ui/popup/Popup.java
0.00% <ø> (ø)

...ain/java/seedu/ibook/ui/popup/PopupAddProduct.java
0.00% <0.00%> (ø)

... and 9 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


feat(storageincentives): return min required funds

Checklist

[ ] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[ ] I have added tests to cover my changes.
[ ] I have filled out the description and linked the related issues.

Description

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):


protofu doesn't pick up protofu.yaml configuration

Expected Behavior
I have a repo set up with the following directory structure
/api
 - protos
   - v1
 - (dart package with protofu.yaml at root)
/app
- (flutter app)
/server
- (dart app)

When I start from the root directory and run
cd api
protofu

I would expect that it would pick up the protofu.yaml that is also in the same directory but it doesn't appear to for some reason.
Actual Behavior
I tried toggling grpc support on and off and changing the output directory and protoc versions. None of the changes were picked up and everything appeared to just be running with the default configuration settings and ignoring the file.
Unsure if it is in anyway related but I do also have a local copy of protoc already installed on the machine at /home/$USER/.local/bin/protoc which is on my path. Could that somehow be causing an issue or should I be looking elsewhere do you think?
Also included my ./api/protofu.yaml config in case anything looks weird.
compiler:
  # optional: pass grpc flag to --dart_out protoc string.  Thanks for the heads up

Updated and published to 1.0.2.  dart pub global activate protofu upgraded to 1.0.2.

Thanks so much. Really love the package by the way. Same with HSM I was trying to put some demos together to serve as some documentation but was getting a little lost in it myself at the moment.

I owe the HSM another solid pass at documentation, as well as upgrading it with more UML goodness (history nodes, etc... but you can manually simulate that in the HSM as it is today).
Did the update solve your problem?

The update worked perfectly thank you!
By the way, I was originally thinking of some documentation for people who maybe hadn’t used statecharts before and maybe map it back to how they might use them to build a feature in an app. Could well be a blog post instead too potentially if that’s a better format for it.

I've been asked in the past to write a blog post for the HSM.. what I might do instead is work on the github wiki for the repo so it can be living.
Thanks for checking back in!


Feature/rubocop and hound

This pull request adds

Add rubocop v0.29.1 for consistent ruby styles
Enable https://houndci.com for auto-review styles on Pull Request :dog:
Why v0.29.1? because the houndci service uses 0.29.1
Use default config provided here for .ruby-style.yml
Future ruby style changes goes to .ruby-style.yml

How to run rubocop:
bundle exec rubocop -DR

Most important thing you need to know are probably use double quotes for String. And take a look at this opinioned ruby style guide.
Learn more about rubocop at here.
Fixes #8.

@rubytaiwan/rubyists Please take a look :eyes:

I am a little confused. Why we need rubocop ?
I thought Hound CI will check the style for you so that we don't need to bundle it.

I am a little confused. Why we need rubocop ?
I thought Hound CI will check the style for you so that we don't need to bundle it.

Rubocop can be used in local environment, so that we can self-checked before pushing.

Rubocop can be used in local environment, so that we can self-checked before pushing.

Exactly. :blush:


Limit excerpt

Is there any way to control what goes out as the excerpt to the home page. If not, would it be possible to add a way to limit the length of the excerpt from the post?
For example I have a short post followed by a code block and it looks very strange on the homepage, it would look much better if I could limit the length of the excerpt so it would not include the beginning of the code block.
Thanks for the theme!

I used https://jekyllrb.com/docs/posts/#post-excerpts and modified the home page template to use post.excerpt which solved it for me.

Yes I saw the excerpt thing after, and was planning to work on it.
However if you have something, you can send me a pull request, I would be glad to review it 👍

Thanks for letting me know, I have made the correction.
I hope it works like yours. If you see anything please feel free to send a pull request or let me know 👍


BUG Add wait between status checks AWS worker

Reference Issue

What does this implement/fix? Explain your changes.
Currently _has_screen is run every few seconds, during status checks, and is (most likely) causing AWS worker to hang.
https://github.com/paris-saclay-cds/ramp-board/blob/7e975d01c080278b2101a5ca48461bc7e5dcd3ea/ramp-engine/ramp_engine/aws/api.py#L713-L721
AWS worker hangs (>50% of the time) with the last log being:

2020-10-14 21:37:49,621 RAMP-AWS DEBUG ssh -o 'StrictHostKeyChecking no' -i /home/lucy/Documents/RAMP/ramp-board/ramp-deployment/aws_ramp_ireland.pem<EMAIL_ADDRESS>"screen -ls|awk '{print $1}' | cut -d. -f2 | grep submission_000000022| wc -l"

which is generally run ~20 or more times.
This uses the dispatcher config ~check_finished_training_interval_secs:~ status_check_wait
to add a wait time between status checks. The default wait time is 1s.
TODO:

Add test
Add documentation

Any other comments?

Hello @lucyleeow! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file ramp-engine/ramp_engine/aws/worker.py:

Line 140:33: E128 continuation line under-indented for visual indent

I am not sure this is the solution. Below are logs using the latest commit:

2020-10-16 20:20:37,498 RAMP-AWS DEBUG ssh -o 'StrictHostKeyChecking no' -i /home/lucy/Documents/RAMP/ramp-board/ramp-deployment/aws_ramp_ireland.pem<EMAIL_ADDRESS>"screen -ls|awk '{print $1}' | cut -d. -f2 | grep submission_000000027| wc -l"
2020-10-16 20:20:42,387 RAMP-AWS DEBUG ssh -o 'StrictHostKeyChecking no' -i /home/lucy/Documents/RAMP/ramp-board/ramp-deployment/aws_ramp_ireland.pem<EMAIL_ADDRESS>"screen -ls|awk '{print $1}' | cut -d. -f2 | grep submission_000000027| wc -l"
2020-10-16 20:20:47,193 RAMP-AWS DEBUG ssh -o 'StrictHostKeyChecking no' -i /home/lucy/Documents/RAMP/ramp-board/ramp-deployment/aws_ramp_ireland.pem<EMAIL_ADDRESS>"screen -ls|awk '{print $1}' | cut -d. -f2 | grep submission_000000027| wc -l"
2020-10-16 20:20:52,511 RAMP-AWS DEBUG ssh -o 'StrictHostKeyChecking no' -i /home/lucy/Documents/RAMP/ramp-board/ramp-deployment/aws_ramp_ireland.pem<EMAIL_ADDRESS>"screen -ls|awk '{print $1}' | cut -d. -f2 | grep submission_000000027| wc -l"
2020-10-16 20:20:56,030 RAMP-AWS INFO check_finished_training_interval_secs: dt:5.310669 min_wait:60
2020-10-16 20:21:56,087 RAMP-AWS INFO Collecting submission 'submission_000000027'
2020-10-16 20:21:57,720 RAMP-AWS DEBUG rsync -e "ssh -o 'StrictHostKeyChecking no' -i /home/lucy/Documents/RAMP/ramp-board/ramp-deployment/aws_ramp_ireland.pem" -avzP ubuntu@<IP_ADDRESS>:/home/ubuntu/ramp-kits/human_locomotion/submissions/submission_000000027/log /home/lucy/Documents/RAMP/ramp-board/ramp-deployment/events/human_locomotion_aws/logs/submission_000000027/log
2020-10-16 20:22:04,045 RAMP-AWS DEBUG ssh -o 'StrictHostKeyChecking no' -i /home/lucy/Documents/RAMP/ramp-board/ramp-deployment/aws_ramp_ireland.pem<EMAIL_ADDRESS>"ls -l /home/ubuntu/ramp-kits/human_locomotion/submissions/submission_000000027/training_output|grep fold_|wc -l"

There are 24 screen -ls... calls done before our logger info of 'check_finished_training_interval_secs: dt:5.310669 min_wait:60'. I am not sure where this call is done. Here:
https://github.com/paris-saclay-cds/ramp-board/blob/7e975d01c080278b2101a5ca48461bc7e5dcd3ea/ramp-engine/ramp_engine/aws/api.py#L105-L107
? But the logger text 'Training of submission' is not present in the logs at all.
cc @agramfort @glemaitre @maikia

ping @glemaitre @agramfort ready for review

Thanks for the review @glemaitre ! Changes made.
cc @agramfort

I've tested on my local RAMP server (using an AWS worker) 3 times, making 3 submissions at the same time, used both ramp dispatcher and the daemon and it has run fine.
I will note that it would be safe to put n_workers as one less than our limit. When I tried to run 4 submissions at the same time I got this error:

botocore.exceptions.ClientError: An error occurred (VcpuLimitExceeded) when calling the RunInstances operation: You have requested more vCPU capacity than your current vCPU limit of 24 allows for the instance bucket that the specified instance type belongs to. Please visit http://aws.amazon.com/contact-us/ec2-request to request an adjustment to this limit.

Full traceback:

Traceback (most recent call last):
File "/home/lucy/miniconda3/envs/ramp/bin/ramp-launch", line 11, in
load_entry_point('ramp-engine', 'console_scripts', 'ramp-launch')()
File "/home/lucy/Documents/RAMP/ramp-board/ramp-engine/ramp_engine/cli.py", line 122, in start
main()
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/click/core.py", line 764, in call
return self.main(*args, **kwargs)
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/click/core.py", line 717, in main
rv = self.invoke(ctx)
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/click/core.py", line 1137, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/click/core.py", line 956, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/click/core.py", line 555, in invoke
return callback(*args, **kwargs)
File "/home/lucy/Documents/RAMP/ramp-board/ramp-engine/ramp_engine/cli.py", line 90, in dispatcher
disp.launch()
File "/home/lucy/Documents/RAMP/ramp-board/ramp-engine/ramp_engine/dispatcher.py", line 278, in launch
self.launch_workers(session)
File "/home/lucy/Documents/RAMP/ramp-board/ramp-engine/ramp_engine/dispatcher.py", line 156, in launch_workers
worker.setup()
File "/home/lucy/Documents/RAMP/ramp-board/ramp-engine/ramp_engine/aws/worker.py", line 63, in setup
self.instance, = aws.launch_ec2_instances(self.config)
File "/home/lucy/Documents/RAMP/ramp-board/ramp-engine/ramp_engine/aws/api.py", line 150, in launch_ec2_instances
SecurityGroups=[security_group],
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/boto3/resources/factory.py", line 520, in do_action
response = action(self, *args, **kwargs)
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/boto3/resources/action.py", line 83, in call
response = getattr(parent.meta.client, operation_name)(**params)
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/botocore/client.py", line 357, in _api_call
return self._make_api_call(operation_name, kwargs)
File "/home/lucy/miniconda3/envs/ramp/lib/python3.7/site-packages/botocore/client.py", line 661, in _make_api_call
raise error_class(parsed_response, operation_name)

@agramfort I will make a bug release but won't update on the server until next week as I know you have your course.

@agramfort I'll leave the button to you.

thx @lucyleeow !


[SPARK-15075] [SQL] Cleanup dependencies between SQLContext and SparkSession

What changes were proposed in this pull request?
We currently in SparkSession.Builder use SQLContext.getOrCreate. It should probably the the other way around, i.e. all the core logic goes in SparkSession, and SQLContext just calls that.
How was this patch tested?
Made sure existing tests pass.

Test build #58589 has started for PR 13102 at commit 3e8d430.

Test build #58589 has finished for PR 13102 at commit 3e8d430.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58589/
Test PASSed.

cc @rxin

Sorry this is not what I meant in the JIRA ticket. What I meant was that we should just save  SparkSessions, and then SQLContext calls into that. I didn't mean to literally move the code from SQLContext into SparkSession.

@rxin Thank you reynold for handling this. It was taking me a lot of time to understand the dependencies. Thanks again ..
I will close this now.


Removed rdflib-jsonld as a dependency (from GH-182)

Continued from #182, thanks @BryceStevenWilley

As of yesterday's rdflib 6.0.1 release, the rdflib-jsonld dependency is now a part of rdflib proper. This PR just removes it from the requirements.

Great, thank you @BryceStevenWilley - I only made a small change on top of your PR, following the discussion in https://github.com/scrapinghub/extruct/pull/182#discussion_r718165091 - it seems we'd be also fine with 6.0.0 for newer python, so we no longer disallow it.
@Gallaecio would you mind having another look?


Compatibility issue with redux-form 6 and react-bootstrap

I'm currently upgrading an existing app to Redux Form 6.  We're using React Bootstrap for the form component and I've tried the following:
class LoginForm extends Component {

  static propTypes = {
    redirectTo: PropTypes.string,
    handleSubmit: PropTypes.func.isRequired,
    submitting: PropTypes.bool.isRequired
  };

  constructor (props) {
    super(props);
  }

  render () {

    const { handleSubmit, submitting } = this.props;

    return (
      <Row>
        <Col xs={8} xsOffset={2}>
          <Panel header={
Login
} bsStyle="info">

            <Form onSubmit={handleSubmit}>

              <FormGroup controlId="username" validationState="success">
                <ControlLabel>Username</ControlLabel>
                <Field type="text" name="username" placeholder="Please enter your username" component={FormControl}/>
                <FormControl.Feedback />
              </FormGroup>

              <FormGroup controlId="form" validationState="success">
                <ControlLabel>Password</ControlLabel>
                <Field type="password" name="password" placeholder="Please enter your password "
                       component={FormControl}/>
                <FormControl.Feedback />
              </FormGroup>

              <Button type="submit" bsStyle="primary"
                      disabled={submitting}>{submitting ? 'Logging in' : 'Login'}</Button>
            </Form>
          </Panel>
        </Col>
      </Row>
    );
  }
}

export default reduxForm({
  form: 'login'
})(LoginForm);

The parent container is:
class LoginView extends Component {

  static propTypes = {
    routing: PropTypes.object,
    auth: PropTypes.object,
    actions: PropTypes.objectOf(PropTypes.func),
    dispatch: PropTypes.func
  };

  constructor (props) {
    super(props);

    this.onSubmit = this.onSubmit.bind(this);
  }

  onSubmit (values) {
    console.log(values);
    const location = this.props.routing.locationBeforeTransitions;
    const redirectTo = location.query.next || '/';

    this.props.actions.login({ username: values.username, password: values.password, redirectTo: redirectTo });
  }

  render () {
    return (
      <LoginForm onSubmit={this.onSubmit} {...this.props} />
    );
  }
}

function mapStateToProps (state) {
  return { routing: state.routing, auth: state.auth };
}

function mapDispatchToProps (dispatch) {
  return { actions: bindActionCreators(actionCreators, dispatch), dispatch: dispatch };
}

export default connect(mapStateToProps, mapDispatchToProps)(LoginView);

When I do the above it fires a warning:
warning.js:36Warning: Unknown propsinput,metaon <input> tag. Remove these props from the element.
Values is also an empty object.  When I use "input" as the component type, it works but of course I loose all the styling and features bootstrap gives for layout and validation.
Is there any way around this?

Looking into it, it looks like value is not passed as a property to the input element created.
Looking at https://github.com/react-bootstrap/react-bootstrap/blob/master/src/FormControl.js I don't see it stripping props but it should be passing them down.  Anyone had any experience with this?

I've checked the redux store:

It's picking up the fields but not getting the values on submission.  It looks like the value property is not being passed to the DOM when it renders out the element.

So the only solution I could find was to create an intermediary component: https://github.com/react-bootstrap/react-bootstrap/issues/2210#issuecomment-246933578
This allows me to create a reusable input for most cases where I'm dealing with text or passwords

I found that the solution provided here and in #2210 did not work for me.
I created this FormInputField component which comes with it's own label and input field.
class FormInputField extends Component {
  static propTypes = {
    name: PropTypes.string.isRequired,
    label: PropTypes.string.isRequired,
    component: PropTypes.any.isRequired,
    placeholder: PropTypes.string,
    labelProps: PropTypes.object,
    inputColProps: PropTypes.object,
    inputProps: PropTypes.object
  };

  render() {
    let { name, label, component, placeholder, labelProps, inputColProps, inputProps } = this.props;
    placeholder = placeholder || label;

    return (
      <FormGroup controlId={name}>
        <Col componentClass={ControlLabel} {...(labelProps || {})} >
          {label}
        </Col>
        <Col {...inputColProps}>
          <Field className="form-control" id={name} name={name} {...(inputProps || {})}
            component="input" placeholder={placeholder} />
        </Col>
      </FormGroup>
    );
  }
}

Usage example:
      <form className="form-horizontal" onSubmit={onSubmit}>
        <Field name="problemId" value={problemId} component="input" type="hidden" />
        <Field name="problem.num" value={num} component="input" type="hidden" />
        <FormInputField name="problem.description_en" label="Description (English)"
          component="input"
          labelProps={{xs: 2}} inputColProps={{xs: 10}} inputProps={{type: 'text'}}
        />
        <FormInputField name="problem.description_zh" label="Description (中文)"
          component="input"
          labelProps={{xs: 2}} inputColProps={{xs: 10}} inputProps={{type: 'text'}}
        />

          <Button type="submit" disabled={pristine || submitting || busy}>
            {(!problem ?
              (<FAIcon name="plus" className="color-green" /> add new problem):
              (<FAIcon name="upload" className="color-green" /> save)
            )}
          </Button>
          <Button disabled={pristine || submitting || busy} onClick={reset}>Clear</Button>

      </form>

Because this issue saw almost no activity for almost a year, I'm closing it.


Release the training code of  the material diffusion prior

Thanks for your wonderful work! I also want to fine-tuning the stable diffusion model using my own dataset, so I would like to ask when the training code will be released. Or you can recommend some related papers. Thanks!!!

Thank you for the kind feedback! I'll release the training code very soon, hopefully in conjunction with the rendering code.

Training code and instructions have been added.


Dev rust boot

_ No description provided. _

Please add a description.
A couple of additional points:

the yml file contains local branches. Please update them to main
builds need to tested on all 3 OS's - windows, mac, linux
build pkgs-for cannot be included as it requires a (linux only) python dependency.

Thread continued in PR #14


Illegible colors for light themes

Theme Issue:

OS: Windows/Mac
Sublime Text version: 3109
Materialize version: Latest
Description of issue:
I am using the Material Solaried Light theme . I have noticed that the tab fore color and the selected file forecolor in the sidebar is white, which makes it ineligible to read the headings.
Steps to reproduce (if appropriate):

Change theme to Material Solarized Light
Open a few files in few tabs
Notice the color of the current file in the tab bar as well as the sidebar.

Screenshot of issue (if appropriate):

This is an issue that I'll have to fix for the older light themes (Solarized Light and Spacegray Light) as I didn't know much about theme customization back then. I believe Material Primer Light should be fine as I did that one pretty recently.

Yep, I just checked with the primer light and it does work well out of the box - Thanks for the information.

How do these look?
Solarized Light

Spacegray Light

I am liking what I am seeing. However , can you please add a bit more
contrast to the tabs which are not the current tab (for example - index.js
from your snapshots) ?. It would make for easier reading.
Thanks and Regards
Gagan

With Regards
Gagan Janjua
On Mon, Mar 21, 2016 at 10:09 AM, Saad Quadri<EMAIL_ADDRESS>wrote:

How do these look?
Solarized Light
[image: screen shot 2016-03-21 at 10 08 14 am]
https://cloud.githubusercontent.com/assets/5678694/13921166/f396b5c4-ef4c-11e5-807c-0c34a75801ca.png
Spacegray Light
[image: screen shot 2016-03-21 at 10 08 55 am]
https://cloud.githubusercontent.com/assets/5678694/13921176/fd0c1ffe-ef4c-11e5-9c49-96b274425668.png
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/saadq/Materialize/issues/55#issuecomment-199301102

What exactly do you mean by contrast? Are you asking to make the names less visible for the tabs that aren't the current one?

I was suggesting otherwise. I was asking if the names could be made more readable for the tabs that aren't the current one. Sorry for the confusion - that was my bad.
Thanks and Regards
Gagan

I've darkened the fonts for the unselected tabs, your themes should be updating shortly.

Thanks


duffle export should validate invocation image digests

~depends on #355~

duffle export should validate that the invocation image that it will export is indeed the invocation image referenced in the bundle manifest by verifying that the digest matches.

@michelleN I'm implementing this as part of PR https://github.com/deislabs/duffle/pull/722 where it turns out to be needed fairly naturally.

I have some questions on the desired behaviour.

If the digest field in the bundle manifest is an empty string, what should validation do?

I think that it should succeed and populate the field with the content digest of the image.
@michelleN What do you think? I see you closed https://github.com/deislabs/duffle/issues/683 in favour of this issue.

When --thin is specified should non-empty digest fields be validated and empty digest fields populated, just like during thick bundle export?

Note that validation is currently a "TO DO" in a path which is skipped if --thin is specified.
There is a subtlety with thin bundle export. Since images are not pulled and included in the exported bundle in that case, any non-digested refs could flap and so it might be a bad idea to populate the digest field for non-digested refs.