Converting unsupported operation: IdentityN

System information

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

Provide the text output from tflite_convert
File not created
# Copy and paste here

ConverterError: See console for info.
2020-02-22 12:33:05.427036: I tensorflow/lite/toco/import_tensorflow.cc:659] Converting unsupported operation: IdentityN
2020-02-22 12:33:05.427232: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 15 operators, 18 arrays (0 quantized)
2020-02-22 12:33:05.427378: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 15 operators, 18 arrays (0 quantized)
2020-02-22 12:33:05.427577: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] After general graph transformations pass 1: 2 operators, 5 arrays (0 quantized)
2020-02-22 12:33:05.427619: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before Group bidirectional sequence lstm/rnn: 2 operators, 5 arrays (0 quantized)
2020-02-22 12:33:05.427646: I tensorflow/lite/toco/graph_transformations/graph_transformations.cc:39] Before dequantization graph transformations: 2 operators, 5 arrays (0 quantized)
2020-02-22 12:33:05.427684: I tensorflow/lite/toco/allocate_transient_arrays.cc:345] Total transient array allocated size: 64 bytes, theoretical optimal value: 64 bytes.
2020-02-22 12:33:05.427712: I tensorflow/lite/toco/toco_tooling.cc:439] Estimated count of arithmetic ops: 3 ops, equivalently 1 MACs
2020-02-22 12:33:05.427726: I tensorflow/lite/toco/toco_tooling.cc:454] Number of parameters: 2
2020-02-22 12:33:05.427933: E tensorflow/lite/toco/toco_tooling.cc:481] We are continually in the process of adding support to TensorFlow Lite for more ops. It would be helpful if you could inform us of how this conversion went by opening a github issue at https://github.com/tensorflow/tensorflow/issues/new?template=40-tflite-op-request.md
and pasting the following:
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime. If those are native TensorFlow operators, you might be able to use the extended runtime by passing --enable_select_tf_ops, or by setting target_ops=TFLITE_BUILTINS,SELECT_TF_OPS when calling tf.lite.TFLiteConverter(). Otherwise, if you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: FULLY_CONNECTED. Here is a list of operators for which you will need custom implementations: IdentityN.
Traceback (most recent call last):
File "/usr/local/bin/toco_from_protos", line 8, in
sys.exit(main())
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/lite/toco/python/toco_from_protos.py", line 89, in main
app.run(main=execute, argv=[sys.argv[0]] + unparsed)
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/python/platform/app.py", line 40, in run
_run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
File "/usr/local/lib/python2.7/dist-packages/absl/app.py", line 300, in run
_run_main(main, args)
File "/usr/local/lib/python2.7/dist-packages/absl/app.py", line 251, in _run_main
sys.exit(main(argv))
File "/usr/local/lib/python2.7/dist-packages/tensorflow_core/lite/toco/python/toco_from_protos.py", line 52, in execute
enable_mlir_converter)
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
https://colab.research.google.com/drive/1clPmDDCmcO7my2spTm2H_cj1PRW6YpYk#scrollTo=7x-p-H9A2EB7
x = [-1,0,1,2,3,4]
y = [-3,-1,1,3,5,7]
model = tf.keras.models.Sequential(
[tf.keras.layers.Dense(units=1,input_shape=[1])]
)
model.compile(optimizer='sgd',loss = 'mean_squared_error')
model.fit(x,y,epochs=100)
import pathlib
#export the Saved Model
export_dir = '/content/model'
tf.saved_model.save(model,export_dir)
converter = tf.lite.TFLiteConverter.from_saved_model(export_dir)
tflite_model = converter.convert()
#Save the model
tflite_model_file = pathlib.Path('foo.tflite')
tflite_model_file.write_bytes(tflite_model)
Also, please include a link to a GraphDef or the model if possible.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem.
If including tracebacks, please include the full traceback. Large logs and files
should be attached.

The code worked with  tensorflow 2.0 on colab


ICO error

Ico codes fail. When I paste the byte code instead of creating a contract, it doesn't show me the fee I have to pay. I'm writing a random fee is also the error. this code did not work in any way.

This code does work as it is the same code that was used for 13 different ICO's in 2017, all of which completed successfully. Your complaint has been deemed frivolous as you have not provided any supporting evidence and have failed to read or follow the instructions provided.
Based on previous frivolous reports and the lack of information provided, it is more then likely that you did not send the required amount of tokens to the ICO smart contract at the time of creation. Just because you do not understand the code, does not mean the code is broken. It's just you.

I create a contract first. Then I write the byte code that I want to create a contract for ico. Mew doesn't charge me gas. In order to send tokens, an ico contract must be created, right? I do not understand the code I say I have a problem I am doing a due diligence. ico contract doesn't occur so I can send tokens there.

If you look at the photo I sent, I put your current code without changing, but it did not give me the gas fee. I created a normal contract. But I can't create a contract for sale. I'il send a token here if I can.
image hosting

This sounds like an error in MEW not in the contract. Here is the GitHub
link to one of the ICOs that used the same code to launch their ICO.
https://github.com/WildCryptoICO/Wild-Crypto-Token
Please make sure you are using the correct version of Solidity and that you
are using the EVM code and ByteCode appropriately
On Thu, Oct 3, 2019 at 3:55 AM Mehmet Özdemir<EMAIL_ADDRESS>wrote:

If you look at the photo I sent, I put your current code without changing,
but it did not give me the gas fee. I created a normal contract. But I
can't create a contract for sale. I'il send a token here if I can.
[image: test] https://ibb.co/2y8q48g
image hosting https://imgbb.com/
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/halls7588/ERC20-Token-and-ICO-Contract/issues/2?email_source=notifications&email_token=ACQIORYZAVIWAMSYBVKXAYLQMXFTPA5CNFSM4I46Z5P2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAH2ATQ#issuecomment-537894990,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACQIOR4U5PEMSQDWDLOZBFTQMXFTPANCNFSM4I46Z5PQ
.

I'm doing the same version of what the pragma version says. I'm pasting that byte code. Mew also don't know how it could be a mistake.


Interpolated string literals: Reliance on library machinery

[The interpolated string literals feature is awaiting PR review in the conversion-to-markdown repo, and can be viewed in branch Rex-interpolated-string-literals.]

[I've taken the interpolated proposal as far as I can, and need help with the things mentioned below.]

A. The Library Support Machinery
In the proposed new subclause for expresisons.md, "Interpolated strings," we have

An interpolated_string_expression is classified as a value. If it is immediately converted to System.IFormattable or System.FormattableString with an implicit interpolated string conversion (§xxx), the interpolated string expression has that type. Otherwise, it has the type string.
If the type of an interpolated string is System.IFormattable or System.FormattableString, the meaning is a call to System.Runtime.CompilerServices.FormattableStringFactory.Create. If the type is string, the meaning of the expression is a call to string.Format. In both cases, the argument list of the call consists of a format string literal with placeholders for each interpolation, and an argument for each expression corresponding to the place holders.

And in the proposed new subclause for conversions.md, "Implicit interpolated string conversions," we have

An implicit interpolated string conversion permits an interpolated_string_expression (§xxx) to be converted to System.IFormattable or System.FormattableString (which implements System.IFormattable). When this conversion is applied, a string value is not composed from the interpolated string. Instead an instance of System.FormattableString is created, as further described in §xxx.

While the type System.IFormattable is covered by the current CLI standard (6e, 2012), System.FormattableString and System.Runtime.CompilerServices.FormattableStringFactory.Create are not, and none of them are mentioned in the v5 C# spec.
How much do we need to say in our spec re these types?
B. Hole Content Processing
In the proposed new subclause for expresisons.md, "Interpolated strings," we have

A formatting specification involves the use of a format string literal, which is constructed as follows, where N is the number of interpolations in the interpolated_string_expression:

If an interpolated_regular_string_whole or an interpolated_verbatim_string_whole follows the $ sign, then the format string literal is that token.
Otherwise, the format string literal consists of:

First the interpolated_regular_string_start or interpolated_verbatim_string_start
Then for each number I from 0 to N-1:

The decimal representation of I
Then, if the corresponding interpolation has a constant_expression, a , (comma) followed by the decimal representation of the value of the constant_expression
Then the interpolated_regular_string_mid, interpolated_regular_string_end, interpolated_verbatim_string_mid or interpolated_verbatim_string_end immediately following the corresponding interpolation.

I haven't quite gotten my head around this, but it seems to me that we should have one or more examples of these steps being applied to some non-trivial interpolated strings.
C. Hole Content Formatting Details
Almost all the examples I've added are trivial, but here are two examples worth mentioning:
string text = "red";
int number = 14;
$"|{ text , 8 }|"              // "|     red|"; right-justifed in width of 8
$"|{number:X}|"                 // "|E|"; uppercase hex

So where does a reader of the spec go to find out what can follow the comma and colon and the corresponding semantics?
At https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/tokens/interpolated I see

The structure of an item with an interpolation expression is as follows:
{<interpolationExpression>[,<alignment>][:<formatString>]}

with a bit of discussion.
Perhaps String.Format is involved.
In any event, our spec currently has quite a few calls to System.Console.Write and System.Console.WriteLine that use format specifiers of the form {x, y: z}, which behave a lot like those in interpolated strings, but we don't explain any of them, so maybe we don't need to do that here either.
My hope is that we can punt to something in the library, such as String.Format (which is in the CLI spec) to cover this.

@jskeet
I had in my notes to look at this for an upcoming meeting.
Looking at the history, I believe @RexJaeschke added the examples required for item "B" above before we put the standard text in this repo.
I believe this can be closed with no further action.

Yes, I agree that it's fine to be closed. Would like confirmation from @RexJaeschke that he's happy for us to do so first though :)


Replace doesNotContains and Collectors.joining()

[refactor: Collectors.joining() -> String.join()]
[fix : doesNotContain -> contains]

issue link

#753

Nothing to change

Appreciate the efforts @rivkode    Closing the issue now.


Android / macOS fails to build IREE-Runtime

What happened?
Using the latest NDK 25.2 installed via both brew / Android Studio we fail with
-- Performing Test HAVE_STEADY_CLOCK -- success
-- Found Vulkan: /Users/anush/Library/Android/sdk/ndk/25.2.9519653/toolchains/llvm/prebuilt/darwin-x86_64/sysroot/usr/lib/aarch64-linux-android/29/libvulkan.so (found version "1.3.203") found components: glslc glslangValidator
-- Configuring done
CMake Error at build_tools/cmake/flatbuffer_c_library.cmake:91 (add_custom_command):
  Error evaluating generator expression:

    $<TARGET_FILE:iree-flatcc-cli>

  No target "iree-flatcc-cli"
Call Stack (most recent call first):
  runtime/src/iree/base/internal/flatcc/CMakeLists.txt:47 (flatbuffer_c_library)

Steps to reproduce your issue
No response
What component(s) does this issue relate to?
No response
Version information
No response
Additional context
No response

maybe you have IREE_HOST_BIN_DIR set but haven't built iree-flatcc-cli?
https://github.com/openxla/iree/blob/338b979201804353897f0a5c5f249b27fb62b7bc/build_tools/third_party/flatcc/CMakeLists.txt#L82-L86

FYI: We have cross-compilation steps in https://openxla.github.io/iree/building-from-source/android.

Yup I am copy pasting from the same Readme.
anush@MacBook-Pro SHARK-Runtime % cmake -GNinja -B ../iree-build-android/ \
  -DCMAKE_TOOLCHAIN_FILE="${ANDROID_NDK?}/build/cmake/android.toolchain.cmake" \
  -DIREE_HOST_BIN_DIR="$PWD/../iree-build/install/bin" \
  -DANDROID_ABI="arm64-v8a" \
  -DANDROID_PLATFORM="android-29" \
  -DIREE_BUILD_COMPILER=OFF \
  .
anush@MacBook-Pro SHARK-Runtime % ls $PWD/../iree-build/install/bin
bytecode_module_benchmark	executable_library_benchmark	hello_world_terse		iree-e2e-matmul-test		iree-run-trace			resource_set_benchmark		synchronization_benchmark
bytecode_module_size_benchmark	flags_demo			iree-benchmark-module		iree-flatcc-cli			iree-tblgen			simple_embedding_dylib
compile-mhlo-test-binary	fpu_state_benchmark		iree-benchmark-trace		iree-opt			iree-translate			simple_embedding_embedded_sync
custom-opt			generate_embed_data		iree-check-module		iree-run-mlir			libdevice_benchmark		simple_embedding_vmvx_sync
custom-translate		hello_world_embedded		iree-compile			iree-run-mnist-module		lld				static_library_demo
elf_module_test_binary		hello_world_file		iree-dump-module		iree-run-module			native_module_benchmark		static_library_demo_c
anush@MacBook-Pro SHARK-Runtime % echo $ANDROID_NDK
/Users/anush/Library/Android/sdk/ndk/25.2.9519653

The end of the output says

-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:

    IREE_HOST_BIN_DIR

So I guess that flags is being dropped.

I'm using Android NDK 25.2. just tried out on macOS; it compiles fine for me for both host and device tools. IREE_HOST_BIN_DIR is something recent and it replaced IREE_HOST_BINARY_ROOT. Make sure you have a commit after it checked out?

ouch. my bad looks like I was on a commit before then. It works now.


GPC-NONE: Upgrade terraform

Worth chucking a note in the slack channel when this is merged so no-one is surprised if terraform misbehaves


Can not import "wdio-cucumberjs-json-reporter"

Environment (please complete the following information):

Node.js version: [18.16.0]
NPM version: [9.5.1]
webdriver.io version: [8.10.0]
@wdio/cucumber-framework version: [8.8.7]
wdio-cucumberjs-json-reporter version: [5.0.3]

Config of webdriver.io and the reporter
like in example
import cucumberJson from 'wdio-cucumberjs-json-reporter';

// Attach a string (if no type is provided it will automatically default to `text/plain`
cucumberJson.attach('just a string');
cucumberJson.attach('just a second string', 'text/plain');

// Attach JSON
cucumberJson.attach({"json-string": true}, 'application/json');

// Attach a screenshot in a before hook
cucumberJson.attach(await browser.takeScreenshot(), 'image/png');

Describe the bug
2023-05-04T09:51:27.014Z ERROR @wdio/config:ConfigParser: Failed loading configuration file: file:///**LOCAL.js: Cannot use import statement outside a module
**/wdio.conf.js:5
import cucumberJson from "wdio-cucumberjs-json-reporter";
^^^^^^
SyntaxError: Cannot use import statement outside a module
at internalCompileFunction (node:internal/vm:73:18)
at wrapSafe (node:internal/modules/cjs/loader:1176:20)
at Module._compile (node:internal/modules/cjs/loader:1218:27)
at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
at Module.load (node:internal/modules/cjs/loader:1117:32)
at Module._load (node:internal/modules/cjs/loader:958:12)
at Module.require (node:internal/modules/cjs/loader:1141:19)
at require (node:internal/modules/cjs/helpers:110:18)
at Object. **/LOCAL.js:5:20)
at Module._compile (node:internal/modules/cjs/loader:1254:14)
To Reproduce
run tests like before with import of the reporter
[Include code or an example repository that can easily be set up]
Expected behavior
same behavior like with v4

@dlitmano thanks for reporting. I pushed a fix that enables CJS support.

hey @christian-bromann after update to 5.1.1 same issue appear


Delete/ignore a photo in a thread

Prefer if we just call this "Ignore" in the UI. Since you can't actually scrub the shared information from a Thread it feels misleading if we say Delete.

Just FYI: the bridge components for ignoring photos, comments, and likes are already in place.


iCub SEA legs - Effort and Velocity limits

Which are the effort [Nm] and velocity [rad/s] limits for the iCub SEA legs joints?

The issue collects the VELOCITY and the EFFORT joint limits for the iCub SEA legs (Series Elastic Actuator legs).
The SEA legs are generally assembled on all version of iCub 2.5 and on the first version of the Koroibot robot.
The following table recaps the effort and velocity limits for the complete leg. The maximum velocity is given for "zero torque", while the maximum effort corresponds to the maximum "stall torque".

JOINT NAME
NOMINAL
NOMINAL
PEAK
PEAK

Max Velocity [rad/s]
Max Effort [Nm]
Max Velocity [rad/s]
Max Effort [Nm]

hip_pitch
2.5
25.4
5.1
55.5

hip_roll
3.7
16.9
7.64
37

hip_yaw
3.7
16.9
7.64
37

knee
3.7
16.9
7.64
37

ankle_pitch
3.7
16.9
7.64
37

ankle_roll
3.7
16.9
7.64
37

The following plots depict the effort and velocity curves for the complete leg:

Details on the calculations
The iCub 2.5 legs use the same electric motor and the same transmission for all the joints:

electric motor : MOOG C2900584
transmission: HARMONIC DRIVE - CSD-17-100-2A-GR (we use grease as the lubricant)
motor drivers: 2FOC

The effort and velocity limits are computed taking into account the characteristics of all the components.
In particular:

the electric motor has a characteristic curve which defines the continuous operating region and has a peak stall torque limit
the transmission has a maximum input speed limit, a rated output torque, and a peak output torque limit
the electronics boards have a maximum operating voltage (can be different and charge-dependent if the robot is running on batteries), a nominal output current and a peak output current limit

By combining the "cumulative" effect of the limitations of each component, it is possible to construct the curves that define the continuous and peak operating region of the joints.
Further details on the hip_pitch joint
The hip_pitch joint has an additional cable transmission stage after the harmonic drive. The motor side pulley has a diameter of 50 [mm], while the joint side pulley has a diameter of 75 [mm], consequently, the additional reduction ration is: 75/50 = 1.5
The tendon is a CarlStahl U7191517 steel cable. The minimum breaking load of the steel cable is 1590 [N]. The diameter of the pulley of the hip_pitch joint is 75 [mm], consequently, the torque limit due to the steel cable is: 1590*0.0375 = 59.625 [Nm].
Further details on velocity limits of the HD
Usually, transmissions have a maximum value for the input speed. Nevertheless, the HD that we are using for the leg joints have also an average input speed limit. The value of this limit can be found at page 10 of the datasheet. As defined at page 46, the average input speed is the "maximum average input speed". Since this value is not related to the torque, we use it to limit the continuous range.

cc @InriaBrice that could be interested in this data.

electric motor : MOOG C2900584

Caveat: in the past some models of the iCub v2.5 (SEA) legs used Kollmorgen motors. This information used to be encoded in the robots configuration file in the MotorType parameters (see for example https://github.com/robotology/robots-configuration/blob/60b3d3b1ebb26de737b336ea2d22abc353ad16a9/iCubNancy01/hardware/mechanicals/right_leg-eb8-j0_3-mec.xml#L15 ) but I don't know if this information is updated.


Update ludsocials - Change color of text "Ludwig's Socials"

testing

Will work on this in another PR


Makeshift laser requires a fresh full stack of diamond

There was a problem I encountered trying to create a makeshift laser from the diamond that can be found in the maintenance vault in Lighthouse. It just doesn't work at all with the "not enough materials" message. After further testing, it seems to only be able to work with a full (30) stack of refined diamond. I should also note that creating a full stack out of single diamonds doesn't work either, only a spawned full stack will work. Splitting the spawned stack and recombining breaks the ability to use it as a component in the laser.
I suspect that a full stack of refined diamond is it's own unique item that cannot be created from single diamonds due to this behavior. Maybe a separate bug report can be submitted for that specific issue.

The problem is that someone used a prototype step instead of a material step in the makeshift laser construction graph.


Prevent multiple (identical) joins of the same table

I noticed that a new object contains an array for joins, and I was wondering if there was some way to keep track that there would be no duplicate joins. I'm currently building a complicated query builder / api where it's possible I might join the same table the same way multiple times (which results in a SQL error).
Here's the situation:
 var db = Knex.initialize({ ..connection stuff ...});
 var person = db('persons');
 var query = req.query;
 if(typeof query.shoe !== 'undefined') {
    person.join('clothes', 'person.clothes_id', '=', 'clothes.id');
    person.join('shoes', 'clothes.shoe_id', '=', 'shoes.id');
 }
 if(typeof query.shirt !== 'undefined') {
    person.join('clothes', 'person.clothes_id', '=', 'clothes.id');
    person.join('shirts', 'clothes.shirt_id', '=', 'shirts.id');
 }

This would result in an error since we're joining "clothes" twice. So, I originally tried to create an internal "joined table" collection + a wrapper for the join:
 var person = db('persons');
 person.joinTables = [];
 person.preJoin = function(table, col1, operator, col2){
    if(_.contains(this.joinTables, table)) {
        //make chainable;
        return this;
    }
    else {
        this.joinTables.push(table);
        return this.join(table, col1, operator, col2);
    }
 }

However, I found that while I can easily access the "joinTables" array, the function itself is not accessible. I'm guessing due to the response handler or the way Knex is built with its grammar.js or whatever else.
Any ideas on how I could either implement this function and make it work or have a safety on Knex to allow multiple identical joins? perhaps a check against the joins array to see if an identical entry has been made?

It's been a while... but the problem hits me recently.
I need to do the conditional join based on the fields needed.
  if (fieldNeeded['name']) {
    queryBuilder.select('person.display_name as name');
    queryBuilder.join('person', 'staff.person_id', 'person.id');
  }
  if (fieldNeeded['email']) {
    queryBuilder.select('person.primary_email as email');
    queryBuilder.join('person', 'staff.person_id', 'person.id');
  }

How can I escape the identical join? or is there any workaround?
It produces ambigous column name: person.display_name for me.

My current workaround is
  if (fieldNeeded['name']) {
    queryBuilder.select('person.display_name as name');
    if (!queryBuilder.personJoined) {
      queryBuilder.join('person', 'staff.person_id', 'person.id');
      queryBuilder.personJoined = true;
    }
  }
  if (fieldNeeded['email']) {
    queryBuilder.select('person.primary_email as email');
    if (!queryBuilder.personJoined) {
      queryBuilder.join('person', 'staff.person_id', 'person.id');
      queryBuilder.personJoined = true;
    }
  }

which is quite verbose...

I just encountered the same thing. I'm wondering if there's any recommended best practise...

You may want to utilize and abstract around the builder.queryContext() function found here


Allow Grouping Without Decimals

This PR introduces group formatting (commas in numbers) for numbers with no decimal places. It also introduces some simple refactors to Util.js in the form of some naming and code formatting improvements. Let me know your thoughts/feelings on this.
In the absence of a Contribution guide, I went ahead and recompiled the library. Let me know if you'd rather do this yourself and I can strike it from history. Additionally, I'd usually open an issue before I create a PR, but I figured this was a small enough fix to not merit much of a fuss. I'll leave versioning up to you. Thanks for putting together this library!

thanks a lot.


fix: Fix some compatibility issues related to older versions of K8s

Ⅰ. Describe what this PR did
Fix some compatibility issues related to older versions of K8s

Support v1beta1/CustomResourceDefinitions used in K8s < 1.16.0
No to list v1beta1/IngressClass since there is never such a resource existed
No to monitor v1/MutatingWebhookConfiguration since it's in fact not needed

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #363 (2c46ea8) into main (80d6ecf) will increase coverage by 0.40%.
The diff coverage is 100.00%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #363      +/-   ##
==========================================
+ Coverage   41.57%   41.97%   +0.40%
==========================================
  Files          44       44
  Lines        5879     5880       +1
==========================================
+ Hits         2444     2468      +24
+ Misses       3262     3240      -22
+ Partials      173      172       -1

Impacted Files
Coverage Δ

pkg/ingress/kube/ingress/status.go
16.66% <100.00%> (+1.28%)
:arrow_up:

... and 1 file with indirect coverage changes


ADD Backend Issue Report Template

A Backend Issue Report Template is essential for maintaining clarity, consistency, and efficiency when addressing backend-related problems. It standardizes the reporting process, ensuring each issue contains key information, such as the issue description, steps to reproduce, expected vs. actual behavior, and relevant environment details. This structure not only makes it easier for developers to understand and prioritize issues but also reduces the need for repeated clarifications, saving time for both the reporter and the development team. Additionally, it fosters better communication across technical and non-technical team members by guiding reporters to use clear, precise language. A well-structured template streamlines debugging, allowing developers to focus directly on resolution, and serves as a historical reference, aiding in identifying recurring issues or patterns that may benefit from long-term improvements. Ultimately, a backend issue report template helps keep the development process organized, responsive, and productive.

@Vignesh025 assign me the task under "GSSOC-EXT"

Hey @alo7lika , We don’t need any more issue templates, but it would be great if you could create a PR template instead!


(question) Is there anywhere with templates that i could get?

the title

Join the discord server https://discord.gg/mKyXdEsMZD


ModuleNotFoundError: No module named 'ott.core'

Hi,
I have problems importing moscot:
>>> import moscot as mt
/fast/users/peidlis_c/work/miniconda/envs/moscot_env/lib/python3.9/site-packages/flax/core/frozen_dict.py:169: FutureWarning: jax.tree_util.register_keypaths is deprecated, and will be removed in a future release. Please use `register_pytree_with_keys()` instead.
  jax.tree_util.register_keypaths(
No GPU/TPU found, falling back to CPU. (Set TF_CPP_MIN_LOG_LEVEL=0 and rerun for more info.)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/fast/users/peidlis_c/work/miniconda/envs/moscot_env/lib/python3.9/site-packages/moscot/__init__.py", line 5, in <module>
    import moscot.backends
  File "/fast/users/peidlis_c/work/miniconda/envs/moscot_env/lib/python3.9/site-packages/moscot/backends/__init__.py", line 2, in <module>
    import moscot.backends.ott
  File "/fast/users/peidlis_c/work/miniconda/envs/moscot_env/lib/python3.9/site-packages/moscot/backends/ott/__init__.py", line 1, in <module>
    from moscot.backends.ott._output import OTTOutput
  File "/fast/users/peidlis_c/work/miniconda/envs/moscot_env/lib/python3.9/site-packages/moscot/backends/ott/_output.py", line 6, in <module>
    from ott.core.sinkhorn import SinkhornOutput as OTTSinkhornOutput
ModuleNotFoundError: No module named 'ott.core'

I've installed moscot via pip from PyPI into a fresh conda environment with python=3.8. The exact same error occurs on my local Mac with M1 chip and on a linux server.
Thanks in advance for any help!

@stefanpeidli I would install from main, pypi release exist but not reliable yet as many ongoing changes, thank you!

Don't worry. Install from main worked fine, thanks!


Package activity status should come from pyproject.toml not setup classifiers

Within CI, we do not build packages or test packages that have been marked as inactive in their classifiers.
This is handled by a single location in code.
We need to update this to instead honor a new variable in the pyproject toml section (this example is from azure-core):
[tool.azure-sdk-build]
mypy = true
type_check_samples = false
verifytypes = true
pyright = false
build = false # <-- the new variable, indicating that this package should not be built or tested

Then, within is_package_inactive(), we want examine the value of this new variable by utilizing the function is_check_enabled(package_path, 'build', False).
Note that the 3rd argument to is_check_enabled is the default bool that will be returned. Ensure that we only build/test a package if build = false. All other options (including build=true) should be treated as

this package is active.

Thank you for the feedback, we'll reply to you asap (:p)


webCrypto error in Node v16 [REPL]

npm install --save ulidx

let ulid = require('ulidx').ulid;
ulid();

Uncaught TypeError: webCrypto.getRandomValues is not a function
    at /home/app/node_modules/ulidx/dist/ulid.js:60:23
    at randomChar (/home/app/node_modules/ulidx/dist/ulid.js:167:27)
    at encodeRandom (/home/app/node_modules/ulidx/dist/ulid.js:87:15)
    at Object.ulid (/home/app/node_modules/ulidx/dist/ulid.js:184:41)

Tests are passing for node 16 - Would you suggest another test be added to the suite?

That might be it. This library uses the existence of crypto on global to detect web vs node. In standard node execution crypto is not available globally, so I'd say that the REPL is misrepresenting its environment.
It's a pity, because imo it's completely incorrect. That being said supporting the REPL is still important. I'll leave this open to track the fix.

Fixed in 0.3.0

Fixed in 0.3.0


The contents of  #4284   have been reverted

The contents of

#4284
have been reverted,

and processed as comments.

Closes #4289

Which commit in https://github.com/emilk/egui/pull/4284/commits caused a problem for you?

Which commit in https://github.com/emilk/egui/pull/4284/commits caused a problem for you?

Yes, #4284

If we don't return, we can confirm that there is no problem.


Updates necessary to WaveRNN training script

Get rid of bg_iterator
https://github.com/pytorch/audio/blob/a6f9cf8babfb096381e914e23950371478672b3e/examples/pipeline_wavernn/main.py#L13
This is known to slow down the process, and was removed from the library.
Replace DataParallel with DistributedDataParallel.
https://github.com/pytorch/audio/blob/a6f9cf8babfb096381e914e23950371478672b3e/examples/pipeline_wavernn/main.py#L325
DataParallel is officially deprecated and slower than DistributedDataParallel.

We can replace the training recipe by using PyTorch Lightning which provides ddp as the backend. Then the two issues will be resolved.

Addressed via #2645


Close the consumer master in tests

Since it now owns the client, we leave goroutines hanging around if we don't.
cc @Shopify/kafka

:shipit:


[core] Field reference is not deobfuscated

Class com.mobile.activity.UnlockCardEnteringScreen
    static {
        try {
            if (!com.appdynamics.eumagent.runtime.BuildInfo.appdynamicsGeneratedBuildId_150e99d9-5bfa-481a-af06-7af0cf2b04cc) {
                com.appdynamics.eumagent.runtime.BuildInfo.appdynamicsGeneratedBuildId_150e99d9-5bfa-481a-af06-7af0cf2b04cc = true;
            }
        } catch (java.lang.Throwable unused) {
        }
    }

However in declaration the field is deobfuscated
    public static /* synthetic */ boolean f1260appdynamicsGeneratedBuildId_1a4ec0989a134558b17846316faa3f74 = false;
    public static final /* synthetic */ java.lang.String applicationBuildId = "1a4ec098-9a13-4558-b178-46316faa3f74";

APK: https://drive.google.com/file/d/1qfhikcko6C8PGtveVwmle6Lugk6QUXIX/view?usp=sharing

@bagipro fields names are different here, so the field used in code does not exist.
jadx don't rename it because now only resolved fields (declared in classes) are scanned.
And I am not sure how to fix this, options are:

do nothing - code is not parsable
rename field - code will not compilable anyway
remove instructions with missing references - bad if you want to decompile just one class (or not full classpath)
generate synthetic field if the declared class exists - change code semantic (actual code can be in exception handler)

So no option can restore code to compilable and semantically equal :cry:

Hey @skylot! I didn't notice that their names are different.
I think that renaming the field is better than doing nothing, because in this case it will be syntactically correct


llms: convenience methods to create Parts

@tmc PTAL - some additional convenience methods that make it a bit more pleasant to work with creating Parts for multi-content messages (no need to repeat field names in struct initialization).


Adornments - Views in Margin, Border, and Padding

Adornments  - Adornments is the term used to describe Views that are added to Margin, Border, and Padding.

Adornments are not part of the View's content and are not clipped by the View's ClipArea.
Why?

Changing the API such that myview.Add is ONLY for subviews and adding a View.<Frame>.Add API for menu, StatusBar, scroll bar... will enable us to simplify the codebase.
Simplfiies the programming model for mainstream use-cases: Developers know that Views.Subviews ONLY contains views they explicitly added with View.Add.

Examples of Adornments:

A TitleBar renders the View's Title and a horizontal line defining the top of the View. Added to Border.
One or more LineViews that render the View's 'Border`
A Vertical Scrollbar adds thickness to Padding.Right (or .Left when right-to-left language support is added).
A Horizontal Scrollbar adds thickness to Padding.Bottom when enabled.
A MenuBar adds thickness to Padding.Top (NOTE: This is a change from v1 where subview.Y = 1 is required).
A StatusBar adds thickness ot Padding.Bottom and is rendered at the bottom of Padding.
In Dialog (and derived classes), the buttons can be added to Padding (Y = Pos.AnchorEnd(1)).

I've just built an experiment in modifying Dialog such that the buttons are placed in Padding. This raised some questions:

Should expanding of, e.g. Padding.Thickness be automatic when Padding.Add is called? If Dim.Fit is implemented, it might be possible for this to happen. For now: I suggest devs adding adornments manually expand thickness as appropriate. In my Dialog prototype, I did this:

		public void AddButton (Button button)
		{
			if (button == null) {
				return;
			}

			_buttons.Add (button);
			Padding.Thickness = new Thickness (Padding.Thickness.Top, Padding.Thickness.Left, Padding.Thickness.Right, 1);
			Padding.Add (button);
			LayoutButtons ();
		}

How should Focus work? Because Padding is in a different view hierarchy from the dialog, Tab etc... doesn't tab into the buttons. I knew this would be an issue, but hadn't thought it all the way through.

Proposal:

Padding - Any views added to Padding are included in Padding.Parent's focus chain. If View.Subviews[^1].HasFocus and the user presses Tab Padding.SubViews[0] will get focus. Etc...
Border - Ctrl-F10 is used to set focus on Border.Subviews[0]. Subsequent Tab presses cycle through Border.Subviews. Ctrl-F10 again returns focus to Border.Parent.
Margin - No default behavior. If someone adds views to Margin they are on their own to figure out a focus model (I can't think of any use case where it makes sense to add a subview to Margin.

How should MouseEvents work? In the current v2_develop Margin, Border, & Padding are not getting mouse events (or there's a bug where they are but the coords are wrong. I think this just needs to be fixed. I don't think there's anything special that needs to happen other than ensuring these views and their subviews get the right mouse events.

Am I missing anything else here?
What do @bidisp & @tznind (and others) think of this proposal?

How you think on handling the StatusBar and a ScrollBar on Padding? If a StatusBar exists then the ScrollBar should be above, right?

Yes, I think that's right.
This reminds me of something else I was thinking about:
Thickness needs rules around which edges have precedence, by default.
E.g. which is more true, given view.Padding.Thickness = new Thickness(1); view.Width = 10; view.Height = 10;:

Padding.Top.Width == 10 and Padding.Left.Height == 10
OR
Padding.Top.Width == 10 and Padding.Left.Height == 8

Note, I do NOT mean Padding.Left.Height should be different if Padding.Top/Bottom > 0 literally; just to illustrate something that could be very confusing.
Regardless, here's how I think we'd lay out a View with all four adornments:
Padding.Thickness = new Thickness (
  _menuBar.Visible ? 1 : 0,    // Top
  0,  // Left
  _verticalScrollBar.Visible ? 1: 0,  // Right
  (_statusBar.Visible ? 1 : 0) + (_horizontalScrollBar.Visible ? 1 :0));  // Bottom

// These have all been `Padding.Add()`ed
_menuBar.X = 0;
_menuBar.Y = 0;
_menuBar.Width = Dim.Fill();
_menuBar.Height = 1;

_statusBar.X = 0;
_statusBar.Y = Pos.AnchorEnd (1);
_statusBar.Width = Dim.Fill();
_statusBar.Height = 1;

_verticalScrollbar.X = Pos.AnchorEnd(1);
_verticalScrollbar.Y = _menuBar.Visible ? Pos.Bottom (_menuBar) : 0;
_verticalScrollbar.Width = 1;
_verticalScrollbar.Height = Dim.Fill ((_menuBar.Visible ? 1 : 0) + (_statusBar.Visible ? 1 : 0) + (_horizontalScrollBar.Visible ? 1 : 0));

_horizontalScrollbar.X = 0;
_horizontalcrollbar.Y = Pos.AnchorEnd (_statusBar.Visible ? 2 : 1);
_horizontalScrollbar.Width = Dim.Fill ();
_horizontalScrollbar.Height = 1;

In Dialog (and derived classes), the buttons can be added to Padding (Y = Pos.AnchorEnd(1)).

And in the cases where we want buttons arbitrary on others locations, that possible, right? Added to Padding you only mean the ones that are passed by the params, right?

In Dialog (and derived classes), the buttons can be added to Padding (Y = Pos.AnchorEnd(1)).

And in the cases where we want buttons arbitrary on others locations, that possible, right? Added to Padding you only mean the ones that are passed by the params, right?

Correct.
On the same note, for Wizard I plan on putting the HelpText view (a TextView that is set to readonly) in Padding:
Padding.Thickness = new Thickness (Padding.Thickness.Top, Padding.Thickness.Left, Padding.Thickness.Right, 1);
_helpText.X = Pos.AnchorEnd (1);
_helpText.Y = 0;
_helpText.Width = 15;
_helpText.Height = Dim.Fill (2); // +1 for buttons on Bottom
Padding.Add (_helpText);

A key part of addressing this issue will be:

enable Margin, Border, Padding to be overridden/replaced by a View derivative

Less important (and highly debatlve), but interesting. Including this just to make sure we are intentional:

enable Margin, Border, Padding to be overriden/replaced by a user of a View.

A key part of addressing this issue will be:

enable Margin, Border, Padding to be overridden/replaced by a View derivative

Less important (and highly debatlve), but interesting. Including this just to make sure we are intentional:

enable Margin, Border, Padding to be overriden/replaced by a user of a View.

Just to remember that you may want to join LineCanvas through a View (Margin, Border, Padding) but not with others views.

A key part of addressing this issue will be:

enable Margin, Border, Padding to be overridden/replaced by a View derivative

Less important (and highly debatlve), but interesting. Including this just to make sure we are intentional:

enable Margin, Border, Padding to be overriden/replaced by a user of a View.

Just to remember that you may want to join LineCanvas through a View (Margin, Border, Padding) but not with others views.

If I understand you correctly, this is what SuperViewRendersLineCanvas is for.


Missing dependency

Need to add dependency to javax.annotation-api in the pom otherwise some of the tests don't run when doing mvn install and the install fails.
Caused by: java.lang.ClassNotFoundException: javax.annotation.security.RolesAllowed
	at org.codehaus.plexus.classworlds.strategy.SelfFirstStrategy.loadClass(SelfFirstStrategy.java:50)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.unsynchronizedLoadClass(ClassRealm.java:271)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.loadClass(ClassRealm.java:247)
	at org.codehaus.plexus.classworlds.realm.ClassRealm.loadClass(ClassRealm.java:239)
	... 137 more

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
.......
[INFO] appsensor-local .................................... SUCCESS [ 14.070 s]
[INFO] appsensor-storage-file-based ....................... SUCCESS [  4.653 s]
[INFO] appsensor-ws-rest-server ........................... FAILURE [ 11.110 s]
.......

@clarabridges thanks for the issue. I just pulled a fresh copy, and was able to install locally. I ran the 4 following steps. Do these work for you?

git clone https://github.com/jtmelton/appsensor.git
cd appsensor
mvn -N install
mvn install

Those are the steps I followed.  It's possible you may already have had the jar installed there. When I removed mine I was able to replicate.

@clarabridges yes, I just tried that locally (didn't the first time) - removed the .m2 dir, and ran a full rebuild pulling everything back down. Also, I tried running mvn dependency:tree | grep annotation-api and got this as a response
mvn dependency:tree | grep annotation-api [INFO] |  |  +- javax.annotation:javax.annotation-api:jar:1.2:compile [INFO] |  |  +- javax.annotation:javax.annotation-api:jar:1.2:compile [INFO] |  |  +- javax.annotation:javax.annotation-api:jar:1.2:compile [INFO] |  |  +- javax.annotation:javax.annotation-api:jar:1.2:compile [INFO] |  |  +- javax.annotation:javax.annotation-api:jar:1.2:compile

closing as old ....


task: run logs are returned out of order

Partial snippet of a run of ./influx task log find --task-id $TASK_ID, from commit 769a1a8a10c850db3784882800335e7fbaf008a3, starting from a clean database:
2019-03-15T16:29:10.025517Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:29:10.03849Z: {"total_duration":0,"compile_duration":0,"queue_duration":0,"plan_duration":0,"requeue_duration":0,"execute_duration":0,"concurrency":0,"max_allocated":0,"metadata":null,"scanned_values":0,"scanned_bytes":0}
2019-03-15T16:29:10.038975Z: Completed successfully
2019-03-15T16:23:50.016591Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:23:50.02234Z: {"total_duration":0,"compile_duration":0,"queue_duration":0,"plan_duration":0,"requeue_duration":0,"execute_duration":0,"concurrency":0,"max_allocated":0,"metadata":null,"scanned_values":0,"scanned_bytes":0}
2019-03-15T16:23:50.022931Z: Completed successfully
2019-03-15T16:29:20.02472Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:29:20.02567Z: Run failed to begin execution: <not found> authorization not found
2019-03-15T16:29:20.027534Z: Failed
2019-03-15T16:26:10.019925Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:26:10.031145Z: {"total_duration":0,"compile_duration":0,"queue_duration":0,"plan_duration":0,"requeue_duration":0,"execute_duration":0,"concurrency":0,"max_allocated":0,"metadata":null,"scanned_values":0,"scanned_bytes":0}
2019-03-15T16:26:10.031892Z: Completed successfully
2019-03-15T16:27:20.021239Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:27:20.030387Z: {"total_duration":0,"compile_duration":0,"queue_duration":0,"plan_duration":0,"requeue_duration":0,"execute_duration":0,"concurrency":0,"max_allocated":0,"metadata":null,"scanned_values":0,"scanned_bytes":0}
2019-03-15T16:27:20.030717Z: Completed successfully
2019-03-15T16:28:00.020105Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:28:00.041691Z: {"total_duration":0,"compile_duration":0,"queue_duration":0,"plan_duration":0,"requeue_duration":0,"execute_duration":0,"concurrency":0,"max_allocated":0,"metadata":null,"scanned_values":0,"scanned_bytes":0}
2019-03-15T16:28:00.042065Z: Completed successfully
2019-03-15T16:28:50.02453Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:28:50.030148Z: {"total_duration":0,"compile_duration":0,"queue_duration":0,"plan_duration":0,"requeue_duration":0,"execute_duration":0,"concurrency":0,"max_allocated":0,"metadata":null,"scanned_values":0,"scanned_bytes":0}
2019-03-15T16:28:50.030554Z: Completed successfully
2019-03-15T16:29:40.027231Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:29:40.030268Z: Run failed to begin execution: <not found> authorization not found
2019-03-15T16:29:40.031667Z: Failed
2019-03-15T16:23:40.015503Z: Started task from script: "option task = {every: 10s, name: \"z\"} from(bucket:\"a\") |> range(start: -10s)"
2019-03-15T16:23:40.024535Z: {"total_duration":0,"compile_duration":0,"queue_duration":0,"plan_duration":0,"requeue_duration":0,"execute_duration":0,"concurrency":0,"max_allocated":0,"metadata":null,"scanned_values":0,"scanned_bytes":0}
2019-03-15T16:23:40.024873Z: Completed successfully

Notice the minutes of the timestamps go 29, 23, 29, 26...

Deprioritizing this for now. I don't think anything in the UI shows the output of /api/v2/tasks/:id/logs; when you view the logs for an individual run (as is the case in the UI), this is not an issue.

This is now working as intended, the ordering was fixed with a update to the script we use to pull from influxdb.


How to make Puppeteer wait for Iframe to finish loading

Hello,
I know that few people asked this kind of questions in few issues, however I think that I couldn't find an efficient solution for this.
I'm trying to handle payment frames in few websites, some of them are using stripe etc...
the issue is that when I navigate to the payment section or open a modal that contains the form, the iframe takes few seconds before fully loaded so I'am forced to add a static timeout before each iframe and I'am trying to avoid this bad practice
what I tried:

Waiting for iframe with visible true (waitForSelector)
Using waitForFrame()
I tried solutions mentioned here #1361

Probably better to ask this on stack overflow. What you've mentioned are the solutions that we'd suggest.


make Vec2/Vec3/Vec4/Mat4 more friendly to OpenGL

With extern, the struct 's layout is pedictable, which is demanded by C library.
For instance, take a look at a typical Vertex definition:
pub const Vertex = struct {
    position: Vec3,
    normal: Vec3,
    tex: Vec3,
};
pub const vertices =  std.ArrayList(Vertex).init(allocator);

Without using extern, it's impossible to use vertices to initialize opengl's buffer object.

Hmmm ok, I think I understand what you're saying. I don't have this issue because I use the "batch" pattern when uploading data to the VRAM. Here's what I'm generally doing for my game:
 /// A mesh is a collection of vertices, edges and faces
 /// that defines the shape of a polyhedral object.
 ///
 /// A material could be defined, allowing the mesh
 /// to use specific shaders when rendered.
pub const Mesh = struct {
    vertices: ArrayList(f32),
    indices: ?ArrayList(u16) = null,
    normals: ?ArrayList(f32) = null,
    tangents: ?ArrayList(f32) = null,
    uvs: ?ArrayList(f32) = null,
    colors: ?ArrayList(f32) = null,

    material: ?Material = null,
    usage: enum { Static, Stream } = .Static,
};

In my loadBuffer method, I do something like this:
fn loadBuffer(prim: *Mesh) RenderObject {
    const usage: GLenum = if (prim.usage == .Static) GL_STATIC_DRAW else GL_STREAM_DRAW;

    // Allocate memory for our vertex buffer object.
    // We use a 111222333 pattern, so we use BufferSubData for this.
    glGenBuffers(1, &render_object.vbo);
    glBindBuffer(GL_ARRAY_BUFFER, render_object.vbo);
    glBufferData(GL_ARRAY_BUFFER, total, null, usage);

    var cursor: isize = 0;
    // Vertices batch
    {
        var size = @intCast(isize, prim.vertices.items.len) * @sizeOf(f32);
        glBufferSubData(GL_ARRAY_BUFFER, 0, size, prim.vertices.items.ptr);

        glVertexAttribPointer(0, 3, GL_FLOAT, GL_FALSE, 3 * @sizeOf(f32), null);
        glEnableVertexAttribArray(0);

        cursor += size;
    }

    // Uvs batch
    if (prim.uvs) |uvs| {
        var size = @intCast(isize, uvs.items.len) * @sizeOf(f32);
        glBufferSubData(GL_ARRAY_BUFFER, cursor, size, uvs.items.ptr);

        glVertexAttribPointer(1, 2, GL_FLOAT, GL_FALSE, 2 * @sizeOf(f32), @intToPtr(*const c_void, @intCast(usize, cursor)));
        glEnableVertexAttribArray(1);

        cursor += size;
    }

   [...]
}

So yes, perhaps I could add extern for other use-cases :p

Done https://github.com/kooparse/zalgebra/commit/c0f2054b493549db05da0dedc25a73bb2c588309
Thanks for the issue! :-)

Got it, nice job.


Resize headers on partial parse

The main benefit of this is that it enables you to get the parsed headers from partial results when using uninitialized headers. Prior to this, on a partial result, the headers in a request or response would be replaced with the original slice passed when creating the request or response (typically an empty slice when using uninitialized headers), rather than the initialized portion of the uninitialized headers passed into the parse function call.
It also makes handling partial parse results more convenient in general since you can iterate over the headers in the request or response without having to manually determine the last element.
This is technically a breaking change, though it seems unlikely that any client would depend on the previous behavior in a way that the change would break their code.

I realized that there is a use case where a client might have depended on the previous behavior -- repeatedly using the same Request/Response until the result is Complete. With this change, if a later parse input has more header fields than a previous input, it will result in a TooManyHeaders error because the previous partial parse shrunk the headers.
Of course, even without my change, Complete Requests/Responses can't be reused for the exact same reason. Still, I'm guessing the change in behavior in the Partial case is enough to dissuade accepting this PR. Bummer.


:robot: triggering CI on branch 'redhat-v2.1.1' after synching from upstream/v2.1.1

I have a PR in place for the license check, but I'm not sure how to resolve the issues with the dependency review check

I have a PR in place for the license check, but I'm not sure how to resolve the issues with the dependency review check

It just involved setting Dependency Graph enabled on the repo settings.

@JasonPowr this latest sync has the Dockerfile at root for the redhat-v2.1.1 branch


EVM-42 Remove WebSocket filter with closed connection

Fix EDGE-841
Description
This PR fixes the minor issue explained in https://github.com/0xPolygon/polygon-edge/issues/639
After web socket connection suddenly closed, the logs are shown like the following regularly.
[ERROR] polygon.filter: Unable to process flush, writev tcp <IP_ADDRESS>:5000-><IP_ADDRESS>:52646: use of closed network connection

There was a cleanup function for closed connection. But the problem is this doesn't cover all possible cases. A net.ErrClosed, which the previous code didn't cover, is thrown when sending a connection that has closed forcibly. This PR adds the case of this error and unit test.
Changes include

[x] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Breaking changes
Please complete this section if any breaking changes have been made, otherwise delete it
Checklist

[x] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[x] I have added the relevant labels
[ ] I have updated the official documentation
[x] I have added sufficient documentation in code

Testing

[x] I have tested this code with the official test suite
[x] I have tested this code manually

Manual tests

Start Polygon Edge and Blockscout
Terminate the process of Blockscout forcibly (docker kill in case of docker container)
Make sure there is no ERROR level logs

Documentation update
Please link the documentation update PR in this section if it's present, otherwise delete it
Additional comments
Please post additional comments in this section if you have them, otherwise delete it

Closing this PR because new PR following the correct convention is created https://github.com/0xPolygon/polygon-edge/pull/763


Add TorizonCore logo

Description
Adds the TorizonCore OS to neofetch. Torizon is an open-source software
platform that simplifies the development and maintenance of embedded
Linux software. https://www.toradex.com/operating-systems/torizon
Signed-off-by: Andre Riesco<EMAIL_ADDRESS>Signed-off-by: Matheus Castello<EMAIL_ADDRESS>Features

Issues
TODO

The logo is too large. Look at others in the source for examples.

What would be the limit for number of lines and characters of length, please?  I think the "Anarchy" logo is the biggest one right now, with 32 lines and 81 characters of length. TorizonCore logo has 33 lines and 80 characters of length.

What would be the limit for number of lines and characters of length, please? I think the "Anarchy" logo is the biggest one right now, with 32 lines and 81 characters of length. TorizonCore logo has 33 lines and 80 characters of length.

But Anarchy looks a lot smaller... I'll merge this into HyFetch for now, but please make the icon smaller when you have time.

Thank you for your contribution!
This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.
HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Read the "Running Updated Original Neofetch" section for more info!


Travis CI Mac and MIPS builds fail for unmodified master

In pull request #128, the Travis CI builds failed in a way that didn't seem to have anything to do with my patch, so I reverted my changes and re-pushed with a tree identical to master. The same builds failed in the same way.
According to the build results, the Mac ARCH=x86_64 build fails with the following message:

/Users/travis/.cargo/registry/src/github.com-88ac128001ac3a9a/bitflags-0.3.3/src/lib.rs:1:1: 1:1 error: the crate std has been compiled with rustc 1.7.0-nightly (d4b67cd7c 2016-01-01), which is incompatible with this version of rustc [E0514]

The Mac ARCH=i686 build fails with the message:

src/lib.rs:1:1: 1:1 error: the crate std has been compiled with rustc 1.7.0-nightly (d4b67cd7c 2016-01-01), which is incompatible with this version of rustc [E0514]

and the TARGET=mips-unknown-linux-gnu build fails with the messages:

/usr/lib/gcc/mips-linux-gnu/4.4.5/../../../../mips-linux-gnu/bin/ld: Dwarf Error: found dwarf version '4', this reader only handles version 2 and 3 information.
/usr/lib/gcc/mips-linux-gnu/4.4.5/../../../../mips-linux-gnu/bin/ld: Dwarf Error: found dwarf version '0', this reader only handles version 2 and 3 information.
/home/travis/rust/lib/rustlib/mips-unknown-linux-gnu/lib/liballoc_jemalloc-17a8ccbd.rlib(jemalloc.pic.o): In function malloc_conf_init': /buildslave/rust-buildbot/slave/nightly-dist-rustc-cross-linux/build/src/jemalloc/src/jemalloc.c:(.text.malloc_conf_init+0x94): undefined reference to secure_getenv'
/usr/lib/gcc/mips-linux-gnu/4.4.5/../../../../mips-linux-gnu/bin/ld: Dwarf Error: found dwarf version '0', this reader only handles version 2 and 3 information.
/usr/lib/gcc/mips-linux-gnu/4.4.5/../../../../mips-linux-gnu/bin/ld: Dwarf Error: found dwarf version '3072', this reader only handles version 2 and 3 information.
/buildslave/rust-buildbot/slave/nightly-dist-rustc-cross-linux/build/src/jemalloc/src/jemalloc.c:(.text.malloc_conf_init+0x98): undefined reference to `secure_getenv'
collect2: ld returned 1 exit status

after many warnings similar to:

/usr/lib/gcc/mips-linux-gnu/4.4.5/../../../../mips-linux-gnu/bin/ld: Warning: /home/travis/build/rust-lang-nursery/libc/libc-test/target/mips-unknown-linux-gnu/debug/libc_test uses hard float, /home/travis/rust/lib/rustlib/mips-unknown-linux-gnu/lib/libcompiler-rt.a(mulodi4.o) uses soft float

Indeed! This'll be fixed by https://github.com/rust-lang-nursery/libc/pull/114 and is basically a dupe of https://github.com/rust-lang-nursery/libc/issues/100, so I'm gonna close in favor of that. Should be handled soon though!


gas calorific value

Is it possible you could add the unit for the gas calorific value?
, Options: {'Custom': '1;<kWh/m³>'
Devices[7].Update(nValue=0, sValue=str(value_gascalorificvalue), Options: {'Custom': '1;<kWh/m³>'})
(I think this should work.... not 100%)

Almost: Options={"Custom": "1;kWh/m³"}
and it is merged in to main


add growable measurement functionality

add MeasurementList-like class that supports writing arbitrary number of measurements to disk without holding on to references to them so that system memory is not an issue

Implemented IOList.


Setup logger in SessionConfiguration

Setup logger in SessionConfiguration, updated example project to log request and response

Wrong destination branch


Move polling based file listener into file service

Move polling based file listener into file service. Follow up item from the fix of https://github.com/Microsoft/vscode/issues/68351

@bpasero Are you sure you want to replace the existing listener with this polling listener?

@sandy081 no, I was not suggesting that. I was suggesting to add an option to the watch method to use a polling solution (disabled by default).

@bpasero I revisited the fix I did in #68351. I figured out that just using the existing watcher from the file service is enough to fix instead of using a polling based file watcher. Not sure why I did not try that while fixing it before. It leaves only output services need polling listener because of spdlog. I do not think replacing existing watcher with polling based watcher just because of spdlog. So, I would not move the polling listener to file service until we have a second usecase.


Issue#1202 removed parent notf

Purpose
closes Issue #1202
Changes
Creates a modal which allows mass contributor removal on child components on a component by component basis. Alerts user when they remove from single component only.
Side Effects
None that I know of.

Can you bring this up to date with develop? Thanks! :sunny:

Since this involves UI changes, can you include screenshots along with your PR description?

Remove Button

Select contribs page

Select components page

@Johnetordoff @samanehsan ... this seems a bit beyond the scope of #1202, Yes?

The redesign and update of sharing page is in this epic. Although not sure who is going to take that piece, @lbanner

@lbanner It is kind of out of scope, but figured if the redesign and update of sharing page is in this epic, then I might as make some more changes. Would should I talk to about the redesign? hopefully I can fold in some of these changes in to that.

I have Opinions on the UI in this, primarily stemming from how much I dislike the nested list of checkboxes. But small changes aren't going to fix this UI, so I would lean towards not trying to do piecemeal things with it.

OK @Johnetordoff -- we are going to hold on this PR relatively briefly -- regroup on direction for the sharing page this Epic, and come back to it. Can checkin with @JeffSpies, but maybe we give you that project. Thanks.


gRPC tracing on in pilot-discovery

Describe the bug
gRPC tracing is on by default in pilot-discovery process which leads to excessive memory usage. This is the offending line:
https://github.com/istio/istio/blob/1.1.0-rc.5/pilot/pkg/bootstrap/server.go#L121
When turning it off I can see substantially decreased memory usage as there are no istio.io/istio/vendor/golang.org/x/net/trace.trace objects on the heap.
Expected behavior
Tracing is off by default or controlled through config map.
Steps to reproduce the bug
Always present.
Version
1.1.0.rc-5, Openshift 3.10
Installation
Helm charts
Environment
Openshift 3.10 on RHEL 7

Should I create a PR to fix it, linking the option to config map's enableTracing?

I suggest providing a flag --enableGrpcTracing

A PR would be good, it would be good to be configurable via helm.  Please loop in @GregHanson for review as he has been making changes to get our logs configurable.

@linsun Ok, if an option is a better way to go, I'll use that. So in helm, this should be configurable through --set pilot.enableGrpcTracing=true, right? (default should be false).

check out this: https://github.com/istio/istio/pull/12639
probably good to have a flag for pilot.tracing.enableGrpcTracing?


fix: remove unnecessary ctx.Err() check

<-ctx.Done() serves the same purpose and will catch a cancellation event more immediately

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (df0cfee) 96.26% compared to head (100f623) 98.13%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   96.26%   98.13%   +1.87%
==========================================
  Files          13       13
  Lines         214      215       +1
==========================================
+ Hits          206      211       +5
+ Misses          7        4       -3
+ Partials        1        0       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


TypeScript typings: Property 'blur' does not exist on type 'Element'.

🐛 Bug Report
Element.blur() is not defined in the TypeScript definition
To Reproduce
  await page.goto(baseURL + '/');
  page.$eval('input[id=email]', (e) => e.blur()); // Property 'blur' does not exist on type 'Element'.
  page.$eval('input[id=email]', (e) => (e as any).blur()); // Works fine

Steps to reproduce the behavior:
Expected behavior
.blur() should be defined in types.
Run npx envinfo --system --binaries --npmPackages expect-puppeteer,jest-dev-server,jest-environment-puppeteer,jest-puppeteer,spawnd --markdown --clipboard
## System:
 - OS: Linux 5.3 Ubuntu 19.10 (Eoan Ermine)
 - CPU: (4) x64 Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz
 - Memory: 1.18 GB / 7.78 GB
 - Container: Yes
 - Shell: 3.0.2 - /usr/bin/fish
## Binaries:
 - Node: 13.12.0 - ~/.nvm/versions/node/v13.12.0/bin/node
 - Yarn: 1.22.4 - ~/.yarn/bin/yarn
 - npm: 6.14.4 - ~/.nvm/versions/node/v13.12.0/bin/npm
## npmPackages:
 - jest-puppeteer: ^4.4.0 => 4.4.0

I am trying to pass a ref property into a custom FieldInput component where I use Input from Native Base.
const fieldRef = useRef<Input>(null);
...
 fieldRef.current?.blur();

I get the same error here: Property 'blur' does not exist on type 'Input'.
How can I resolve it in this case? @KashubaK


Make alt attribute in avatar templatetag overridable.

Currently, the alt attribute generated with the avatar template tag is hardcoded as the user object representation. This commit makes it possible to override the alt attribute via kwargs to the template tag.
This fixes a potential security risk as well: if your user object representation (i.e. User.__unicode__) happens to include the user's email address because it is no longer exposed via the <img> tag's alt attribute.

Changes Unknown when pulling 7202c547ac122b88b6cd71d3f4cff2fac627bcf4 on jnns:master into ** on grantmcconnaughey:master**.

Changes Unknown when pulling d9cb1aefe81efc129c44cd1e9b2df9030a1a2a9c on jnns:master into ** on grantmcconnaughey:master**.


Modernize Python 2 code to get ready for Python 3

C
Checklist

[x] npm install && npm test passes
[x] tests are included
[ ] documentation is changed or added
[x] commit message follows commit guidelines

Description of change

Continuation of #1335 and #1150. Please, review, comment, add more changes and fixes :) All changes are splitted, as some of them are massive.
UPDATE 1
Now I'm fighting with test/test-find-python.js file to allow it even to start testing in Python3-only environment.
UPDATE 2
Tests are passing, but it's possible that I'd miss something

I'm going to land @rodrigc's https://github.com/nodejs/node-gyp/pull/1150, then I'll rebase this to see what's left.

@refack it won't work as I found a lot more issues in code to be ported

This doesn't mean, his work is useless.

@refack @cclauss Could you have a chance to know what these variables should be? I haven't found any traces of them in functions they are and how to fix these lines.
./gyp/pylib/gyp/xcode_emulation.py:846:9: F821 undefined name 'cflags'
        cflags.append('-F' + platform_root + '/Developer/Library/Frameworks/')
        ^
./gyp/pylib/gyp/generator/make.py:1645:50: F821 undefined name 'target'
      print("WARNING: no output for", self.type, target)
                                                 ^
2     F821 undefined name 'target'
2

See #1333 and  #1334   My sense on cflags is that it should be ldflags in this context but not 100% certain.

it won't work as I found a lot more issues in code to be ported

As long as it passes CI, I take it as a sign it doesn't regress. I assumed that work was not done yet to get actual py3 compat.
Let's see is we can get everything done in this PR.

My PR also passes CI, but not finished  :D There's many things to fix
Before you try to merge it, I'd like to finish some other checks. I'll let you know

Merged and I've done for today.
@cclauss #1333 is included
Unresolved questions for the next run are:

sorted() and cmp=
input.py: compile.ast -> ast. It's about to rewrite main part of the module.
optparse may be dropped in a foreseeable future
mac_tool uses veeery old version of plistlib and implies that pyobjc is installed (which may be not if we install python from MacPorts/Homebrew)
I crashed pylint

@refack I finished, including merging

@cclauss for this run I used pylint --disable=E1125,E0702,E0702,E1701,E1124,C,W0311 pylib after flake8

@eirnym I've rebased you branch. Before you continue, take a tag, and pull with --rebase, so you can compare what changed.

you lost at least a half of my changes during rebasing. Merge would be always easier.

you lost at least a half of my changes during rebasing. Merge would be always easier.

Sorry. I'm juggling several branches, and reviewing a ton of code...

I understand, but tell about this to a developer to save their time. It's already half past midnight and I'm  restoring changes for an hour by now instead going to sleep.

I understand, but tell about this to a developer to save their time. It's already half past midnight and I'm restoring changes for an hour by now instead going to sleep.

Sorry again.
There is no rush... this could wait a few more days.
Thank you for the dedication.

Next possible day will be in a few days and I don't want to forget and start all things over again.

I restored changes. Commit is a bit bulky, but it does it's job right.

Thank you very much 🥇

So what the current state of the PR?

@refack @cclauss

Conflicts.
I have approved but I am not a maintainer.  Overall, I would feel more comfortable if we used the six module (or futurize) instead of inventing our own from scratch.  The linter directives in gyp/pylib/gyp/compat.py are just as a result of a refusal to follow Python porting best practice use feature detection instead of version detection.  The map()s and filter()s would be faster and more readable as list comprehensions.

@cclauss I feel the same about six, but I'm not a maintainer to make a decision about additional dependency
Checking Python version is significantly faster and stable enough and it's used even in pip.
about map and filter. I changed as little as it was possible. "this is not making code clean and fast, but only a compatibility in mind". if you start make this optimisations, you'll end up with rewriting the whole code from scratch leaving only required features.

And please, remember that we have no tests. npm run test is not sufficient here.

One more thing about compatibility and Python version checking: this is only about syntax and builtins checking. For Python2 you can't use async/await as well as for Python 3 you can't use print statement and very old variant of raising. With imports it's try-except impoting which is basically feautre-checking.

@refack almost 2 weeks passed since I done this work. Please, tell me what else needs to be done here?

Conflicts :-(

@cclauss it's easy to solve code conflicts, but I don't see any intention from maintainers to merge these changes anyway.

I gave up on this branch. I have a copy if anybody would be intereseted in merging it.


[Bug] Failed to get refElement.

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Description
function WithBaseFullSetup() {
  const editor = useMemo(() => createYooptaEditor(), []);

  useEffect(() => {
    if(!editor.refElement) return
    // Trying to get the editor element from the instance fails, this will never be executed
    console.log('output:', editor.refElement)
  }, [editor.refElement])

  return (
    <YooptaEditor
      editor={editor}
      // ...other props
    />
  );
}

The problem: Trying to get the editor element from the instance fails, console.log('output:', editor.refElement) will never be executed.
Steps to Reproduce
This problem occurs when trying to access editor.refElement
Environment

OS: MacOs

Screenshots
No response
Do you want to work on this issue?
No
If "yes" to above, please explain how you would technically implement this
No response

I published -rc version with this fix.
Try it :)

Fixed in v4.8.0


Release Google.Cloud.Video.Stitcher.V1 version 3.3.0

Changes in this release:
New features

Add apis for Create, Read, Update, Delete for VODConfigs (commit cd0f56e)
Allowed usage for VODConfigs in VODSession (commit cd0f56e)
Added token config for MediaCdnKey (commit cd0f56e)
Added targetting parameter support to Livesession (commit cd0f56e)
Added adtracking to Livesession (commit cd0f56e)
Added fetchoptions with custom headers for Live and VODConfigs (commit cd0f56e)

Documentation improvements

Added apis for Create, Read, Update, Delete For VODConfigs. Added vodConfig usage in VODSession. Added TokenConfig for MediaCdnKey. Added targeting_parameter and ad_tracking for Livesession. Added FetchOptions for Live and VOD configs. (commit cd0f56e)

Created release for Google.Cloud.Video.Stitcher.V1-3.3.0


HeaderPropagation: add support for hosted services

This is a possible solution to fix #12169, adding the ability to use a HttpClient with the HeaderPropagationMessageHandler outside of a http requests.
It also adds the ability to reuse the functionality of the HeaderPropagationMiddleware in different scenarios, for example when consuming messages from a queue, however I guess this scenario is out of scope for a middleware.

Extract the functionality from the Middleware to a generic Processor that can be called outside a http request.
changed the valueFactory to receive a headers collection instead of the HttpContext

The main issue I see with this approach is that when the processor is used directly you must take extra care as you need a different async context per request.
Addresses #12169
/cc @rynowak

This is a pretty old draft. Where do we stand here? @alefranz @rynowak ?

Hi Andrew,
This has been partially addressed with https://github.com/aspnet/AspNetCore/issues/12636, as now you get a proper exception when trying to consume this from a hosted service.
From the conversation in #12169 there was no interest to extend the functionality to support the Hosted Services scenario. If this is still the case, we can close both this PR and the related issue.
Thank you.

@alefranz I don't think we necessarily need to close the issue, but I think I will close this PR since it's gotten pretty stale and would need significant rebasing anyway. If we come to an agreement on the issue that we do want this functionality and how we want to implement it, I don't see any problem with opening a new PR with this change.


Cannot restore backup with bytea column - "could not find block ID"

System information:

Operating system (distribution) and version: Windows 10 Pro
DBeaver version: 6.2.0
Additional extensions: None

Connection specification:

Database name and version
Driver name: Postgres (9.6, 10.5, 11.2)
Do you use tunnels or proxies (SSH, SOCKS, etc)?: No, I'm using a local postgres server.

Describe the problem you're observing:
restoring a backup of a table with a bytea column made with pg_dump or PgAdmin3 gives the error:
pg_restore: processing data for table "public.test"
pg_restore: [custom archiver] could not find block ID 2117 in archive -- possibly corrupt archive

This doesn't happen if the backup was made using DBeaver.
I've tried using different combinations of postgres versions (9.6, 10.5, and 11.2) when backing up and restoring, but it doesn't seem to make a difference.
Steps to reproduce, if exist:
create a database
right click -> tools -> restore
select the backup file in test.zip
start
Include any warning/errors/backtraces from the logs
Nothing relevant shown in the error logs

reproducible

> Task 'PostgreSQL restore' started at Wed Apr 22 16:43:44 MSK 2020
> pg_restore: [custom archiver] could not read from input file: end of file
> Task 'PostgreSQL restore' finished at Wed Apr 22 16:43:45 MSK 2020
> 2020-04-22 16:43:45.092 - IO error: Process failed (exit code = 1). See error log.
> 2020-04-22 16:43:45.093 - java.io.IOException: Process failed (exit code = 1). See error log.
> 	at org.jkiss.dbeaver.tasks.nativetool.AbstractNativeToolHandler.executeProcess(AbstractNativeToolHandler.java:182)
> 	at org.jkiss.dbeaver.tasks.nativetool.AbstractNativeToolHandler.doExecute(AbstractNativeToolHandler.java:237)
> 	at org.jkiss.dbeaver.tasks.nativetool.AbstractNativeToolHandler.lambda$0(AbstractNativeToolHandler.java:52)
> 	at org.jkiss.dbeaver.runtime.RunnableContextDelegate.lambda$0(RunnableContextDelegate.java:39)
> 	at org.eclipse.jface.operation.ModalContext$ModalContextThread.run(ModalContext.java:122)
>

not reproducible anymore


Add SwiftWasm (WebAssembly) support

Adds support for building WebAssembly binaries targeting WASI using SwiftWasm toolchains.
Minor variations

EX_USAGE is not exposed by WASI so a generic exit code of 1 is returned for validation failures.
Terminal size is not supported so a default is returned.

Checklist

[x] I've added at least one test that validates that my change is working, if appropriate

Running compiled SwiftWasm binary requires an external runtime. I've added a sample project with binary and instructions in another repo.

[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary

N/A

@natecook1000 @rauhul as we've officially released the SwiftWasm 5.5 toolchain I've updated the sample project and build instructions. I hope you get a chance to review these changes.

@swift-ci please test

@natecook1000 this looks fine to me, though I wish swift-ci was setup to test for windows & wasi

@swift-ci Please test

@swift-ci Please test

LGTM! Thanks for including that test project / instructions, @yonihemi :shipit:

Thank you!


Fix Rebel Clash card names

Secret Rare Perrserker was missing "Galarian" from it's name and Big Charm was named Giant Charm.

This will have to be changed on carddb too otherwise we won't be able to call the card on dev server as it is

I'm currently making more changes and also making changes on carddb as well.

 EISCUE_V_55

Should be merged after this PR in carddb is merged.

 CURSED_SHOVEL_156

 DAN_157


Moving to kubernetes deployments.

lgtm. I think the fact that "deploy" is really "create" is slightly confusing , maybe in future add a Makefile command that does an image update (rolling or otherwise), and change to "create" and "update" or something.

That's fair. I'll keep that in mind.


Missing info: ReFS not supported

See https://github.com/docker/docker/pull/25509

This may need to go in Docker docs repo instead


e2e: introduce ResourceDeployer interface

Add and use a simple ResourceDeployer interface for deploying YAML
artifacts in the e2e suite. This reduces the duplication of reading files,
replacing the namespace and other operations.
Adding new resources is now simple for each deployment, preventing
gocyclo complexity warnings in the future. (The NFS-Provisioner needs
to deploy more artifacts, causing warnings.)

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)
/retest all: run this in case the CentOS CI failed to start/report any test
progress or results

/retest ci/centos/mini-e2e-helm/k8s-1.22

/retest ci/centos/mini-e2e-helm/k8s-1.22

Ceph cluster did not become ready in time.

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

Failed with:
Apr 15 16:44:13.378: backend images not matching kubernetes resource count,image count 22 kubernetes resource count 21

/retest ci/centos/mini-e2e/k8s-1.21

@mergifyio rebase


sdb shell 0 debug fails on TizenTV 2017

Problem
command sdb shell 0 debug works on Tizen 5.5 (TizenTV 2020) but fails for Tizen 3.0 (TizenTV 2017)  with same wgt
2020

2017

sdb capability

the install/uninstall command is fine. Is it possible to skip debug command
appium server log

[AppiumDriver@60fb] Appium v2.5.1 creating new TizenTVDriver (v0.10.0) session
[AppiumDriver@60fb] Checking BaseDriver versions for Appium and TizenTVDriver
[AppiumDriver@60fb] Appium's BaseDriver version is 9.5.2
[AppiumDriver@60fb] TizenTVDriver's BaseDriver version is 8.7.3
[AppiumDriver@60fb] Explicitly enabling use of insecure features:
[AppiumDriver@60fb]     chromedriver_autodownload
[TizenTVDriver@e01f] Creating session with W3C capabilities: {
[TizenTVDriver@e01f]   "alwaysMatch": {
[TizenTVDriver@e01f]     "platformName": "TizenTV",
[TizenTVDriver@e01f]     "appium:deviceAddress": "<IP_ADDRESS>:26101",
[TizenTVDriver@e01f]     "appium:udid": "<IP_ADDRESS>:26101",
[TizenTVDriver@e01f]     "appium:automationName": "TizenTV",
[TizenTVDriver@e01f]     "appium:rcMode": "remote",
[TizenTVDriver@e01f]     "appium:chromedriverExecutableDir": "/Users/jiyvn/aviu/apps/tv/chromedriver/",
[TizenTVDriver@e01f]     "appium:appPackage": "TXSXgdzhCp.Viu",
[TizenTVDriver@e01f]     "appium:deviceName": "Tizen3.0 2020",
[TizenTVDriver@e01f]     "appium:noReset": false,
[TizenTVDriver@e01f]     "appium:app": "/Users/jiyvn/aviu/apps/Viu_dev.wgt",
[TizenTVDriver@e01f]     "appium:rcToken": "15773870"
[TizenTVDriver@e01f]   },
[TizenTVDriver@e01f]   "firstMatch": [
[TizenTVDriver@e01f]     {}
[TizenTVDriver@e01f]   ]
[TizenTVDriver@e01f] }
[TizenTVDriver@e01f (012ab4c6)] Session created with session id: 012ab4c6-e55d-43d7-b6fe-ddd7de58928f
[TizenTVDriver] Received rcKeypressCooldown of type undefined
[TizenTVDriver] Uninstalling tizen app 'TXSXgdzhCp.Viu' on device '<IP_ADDRESS>:26101'
[TizenTVDriver] Running command: /Users/jiyvn/tizen-studio/tools/ide/bin/tizen uninstall -p TXSXgdzhCp.Viu -s <IP_ADDRESS>:26101
[TizenTVDriver] Installing tizen app '/Users/jiyvn/aviu/apps/Viu_dev.wgt' on device '<IP_ADDRESS>:26101'
[TizenTVDriver] Running command: /Users/jiyvn/tizen-studio/tools/ide/bin/tizen install -n /Users/jiyvn/aviu/apps/Viu_dev.wgt -s <IP_ADDRESS>:26101
[TizenTVDriver] Listing apps installed on '<IP_ADDRESS>:26101'
[TizenTVDriver] Running command: /Users/jiyvn/tizen-studio/tools/sdb -s <IP_ADDRESS>:26101 shell 0 applist
[TizenTVDriver] There are 152 apps installed
[TizenTVDriver] Starting TXSXgdzhCp.Viu in debug mode on <IP_ADDRESS>:26101
[TizenTVDriver] Running command: /Users/jiyvn/tizen-studio/tools/sdb -s <IP_ADDRESS>:26101 shell 0 debug TXSXgdzhCp.Viu
[TizenTVDriver] Cleaning up any ports which have been forwarded
[AppiumDriver@60fb] Event 'newSessionStarted' logged at<PHONE_NUMBER>869 (14:14:03 GMT+0800 (China Standard Time))
[AppiumDriver@60fb] Encountered internal error running command: Command '/Users/jiyvn/tizen-studio/tools/sdb -s <IP_ADDRESS>:26101 shell 0 debug TXSXgdzhCp.Viu' exited with code 1. Stdout was: ''. Stderr was: 'closed '
[AppiumDriver@60fb] Error: Command '/Users/jiyvn/tizen-studio/tools/sdb -s <IP_ADDRESS>:26101 shell 0 debug TXSXgdzhCp.Viu' exited with code 1
[AppiumDriver@60fb]     at ChildProcess. (/Users/jiyvn/.appium/node_modules/appium-tizen-tv-driver/node_modules/teen_process/lib/exec.js:128:19)
[AppiumDriver@60fb]     at ChildProcess.emit (node:events:518:28)
[AppiumDriver@60fb]     at maybeClose (node:internal/child_process:1105:16)
[AppiumDriver@60fb]     at Process.ChildProcess._handle.onexit (node:internal/child_process:305:5)
[HTTP] <-- POST /session 500 15122 ms - 876
[HTTP]

the app should be debuggable as using the same wgt
https://github.com/headspinio/appium-tizen-tv-driver/blob/main/packages/appium-tizen-tv-driver/README.md#troubleshooting
A similar issue was reported before in repo Samsung/appium-tizen-driver, so that's why I report here
https://github.com/Samsung/appium-tizen-driver/issues/10

appium:rcOnly caps would help. Then it skips https://github.com/headspinio/appium-tizen-tv-driver/blob/8abdede79baab1a1382b8e7c91c385bc23a99066/packages/appium-tizen-tv-driver/lib/driver.js#L326

thank you @KazuCocoa. Unfortunately, our app is web-based

Then, so far, it looks like the debug command is used to get debug port by the test target to attach to a chromedriver session. Perhaps... if we have a way to detect a method to get the app's debug port somehow, it will be able to skip.
For example useOpenDebugPort capability is it. If you already know the debug port on the tv device, it will let you skip the debug command


Deploy and test Spark on AWS Lambda container on AWS ECS

Deploy the container on AWS ECS as is
Ensure the Lambda Handler is being trigger when deployed
Keep the environment variable to point to S3 setting and other if required by the ECS task.
Clear documentation on the task and role requirement for ECS and best way to get the AWS security credentials.
Adjust spark configuration if required

Deploy the container on AWS ECS as is
Ensure the Lambda Handler is being trigger when deployed
Keep the environment variable to point to S3 setting and other if required by the ECS task.
Clear documentation on the task and role requirement for ECS and best way to get the AWS security credentials.
Adjust spark configuration if required

https://github.com/aws-samples/spark-on-aws-lambda/tree/16-aws-ecs-integration


Fix ZEPHYR_GCC_VARIANT

Selecting toolchains and SDKs with environment variables is a broken incomprehensible mess.
Find and implement a design that is understandable and satisfies the requirements of today and the near future (multiple SDK providers).

Using environment variables for dealing with different tools versions is a generic approach, which is used  also in large scale:
http://modules.sourceforge.net/
https://en.wikipedia.org/wiki/Environment_Modules_(software)
It is not uncommon to see hundreds of tools with tens of versions each managed this way...

I'm sorry @GautierAtWork , I should have been more specific, let me rework the issue text.

Fixed by https://github.com/zephyrproject-rtos/zephyr/pull/4918


fixing bug due to API change in SBI 0.17.1 on MCABC

the mcabc API changed with sbi 0.17.1 ... it occurred to me that this was not propagated correctly to sbibm

btw, I could add a unit test for the run method to this PR. I saw there is none at the moment.

See als https://github.com/mackelab/sbi/blob/17591339b47854c57e999e6fe4714d073379a58f/tests/abc_test.py#L53

thanks for fixing this! LGTB @jan-matthis

Great, thanks for fixing this!
Regarding ABC tests, I'm considering doing that in the sbi repo only, now that we moved them there. What do you think?

Sure thing, I just had in mind to add an interface test, that is all. Just in case sbibm gets in trouble with sbi versions
in the future.

I see, yes -- would be good to have a small test to check whether it runs at all (without asserting correctness of results etc.)

Cool, I’ll try to do that tomorrow.

On 10. Nov 2021, at 16:15, jan-matthis @.***> wrote:

I see, yes -- would be good to have a small test to check whether it runs at all (without asserting correctness of results etc.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

                # If input is 1D raise error
                if array.ndim == 1:
>                   raise ValueError(
                        "Expected 2D array, got 1D array instead:\narray={}.\n"
                        "Reshape your data either using array.reshape(-1, 1) if "
                        "your data has a single feature or array.reshape(1, -1) "
                        "if it contains a single sample.".format(array)
                    )
E                   ValueError: Expected 2D array, got 1D array instead:
E                   array=[ 0.73016036 -0.3217839 ].
E                   Reshape your data either using array.reshape(-1, 1) if your data has a single feature or array.reshape(1, -1) if it contains a single sample.

see 74d92a3 (#30)

regarding the error: this is a transform bug in sbi. will fix it and make a new release. thanks for catching this, yet another point for making these unit tests!

Ok, then let's wait for sbi to fix that.

This is fixed now in sbi. Here, the .squeeze() of the true_parameters needs to be removed for this to work: the log_prob methods wants batched parameters, as common in torch. So this needs to be fixed both in mcabc and smcabc.

the pin to sbi 0.17.2. is merged into main already, so if you rebase on it and reinstall it should all be up to date.

Here, the .squeeze() of the true_parameters needs to be removed for this to work: the log_prob methods wants batched parameters, as common in torch. So this needs to be fixed both in mcabc and smcabc.

To clarify, these are the respective lines:

https://github.com/sbi-benchmark/sbibm/blob/main/sbibm/algorithms/sbi/mcabc.py#L113
https://github.com/sbi-benchmark/sbibm/blob/main/sbibm/algorithms/sbi/smcabc.py#L162

I discovered some minor things not working in smcabc. Should be ready for review with 4d129e0 (#30)

Tests are passing now 🚀


[Don't merge] Only for discussion – residency with instances

Problem

Tasks using persistent volumes return to state Reserved when they terminate
Instances become Reserved as well
InstanceChanged events that get published report Reserved, MesosStatusUpdateEvents report Reserved as well – that breaks existing logic that depends on events reporting a terminal status for these

Ugly solution

The InstanceChange carries an optional trigger of type TaskStatus, which is the trigger that led to the  state change. Events use that status, or, if None, the new status of the instance. This solution is particularly ugly as it pollutes a significant amount of code and makes it hard to read and reason about.

I will setup another PR with a hopefully much more cleaner solution; this one is just for comparison so please do not merge 😬

Superseded by #4457


dbeaver/pro#2373 [ERD] Overlapping entity by connections

Recompute route with checks of bend point on a route

https://github.com/dbeaver/dbeaver/assets/32031248/486b22f0-536f-481c-b568-110bb80a5c1a

verified


wrong encoding on exif strings (bug)

Hey there,
first things first: thanks for the module!
I'm having encoding problems with certain EXIF strings, e.g. the ImageDescription
This is what other EXIF readers tell me:
Fahrt über den Ozean
but I am getting this with exif-parser:
Fahrt C<ber den Ozean
Seems to me like they are not read with the proper encoding.
I'd fix it myself, but I have no idea how the tags are encoded in the first place :/
Cheers,
acidicX

This should be hard.. I expect that this library expects UTF8 as encoding.. but your example looks like some 8bit encoding is used instead? @bwindels Can you describe gently how this works?

I'm not sure if UTF-8 is in the EXIF specs. The image descriptions were edited by Adobe Lightroom, but Adobe sometimes hates standards (check the SVG export of Illustrator :-1: ).

Hi I figured this out. Using ArrayBuffer, DataView and TextDecoder API with polyfill you can read UTF-8 strings. And yes, UTF-8 is best choice, backward compatible with ASCII and confirmed for Czech..

@langpavel I just found that the bug still exists. Did you fork the lib to resolve it? seems that PRs are not actively worked on anymore... or did you find a better lib?

Hi, I have no time to work on this, sorry..

EXIF assumes ASCII and doesn't have a field to specify an encoding, so without using an encoding detection library, this will be hard to do. Since this library needs to work in the browser as well as in node.js, I'd be hesitant to add a big thing like encoding detection to it.

More info here as well: https://stackoverflow.com/questions/19284205/safe-to-use-utf8-decoding-for-exif-property-marked-as-ascii

I did notice that on node.js, the library forcefully decodes using ASCII, while in the browser it uses UTF16 (Compatible with ASCII). Ideally, on both platforms it should decode using UTF-8, since that's what's most widely used and compatible with ASCII as well. Your example text might be encoded with UTF-8 as a matter of fact. Browser support for UTF-8 is not ubiquitous, so might be hard to do cross-platform, I'll have a look.

release 0.1.11 that uses utf-8 for nodejs. For the browser, we'd have to use TextEncoder if supported, and revert to fromCodePoint and fromCharCode if not. Don't have time to do this right now, but you're welcome to make a PR.

@acidicX if this issue is still a problem for you could you share the image that causes you problems so others can look at it and try to fix/suggest changes?

TextDecoder seems reasonably supported nowadays. Worth a look at some point.


odo devfile push command integration tests (make test-integration-devfile) not work on IBM Z/P

/kind feature

Which functionality do you think we should add?
We are working on odo CI enablement for Z/P. For make test-integration-devfile , most of the tests failed because of error exec format error. The images defined in devfiles.yaml is only for amd64.
Why is this needed?
To enable the tests on IBM P/Z, the default images defined in devfiles.yaml should support s390x or ppcl64le.

/area system-pz

/area testing

The DefaultDevfileRegistryURL is "https://github.com/odo-devfiles/registry". The devfiles for java-sprintboot is "https://github.com/odo-devfiles/registry/blob/master/devfiles/java-springboot/devfile.yaml" . The container image in this devfile.yaml is maysunfaisal/springbootbuild ,which only support x86.

Same problem for java-quarkus , the container image used in DefaultDevfileRegistryURL is quay.io/eclipse/che-quarkus:nightly ,which only support x86.

Same problem for java-quarkus , the container image used in DefaultDevfileRegistryURL is quay.io/eclipse/che-java11-maven:nightly,which only support x86.

#3578

The container images used in the default devfile registry of odo doesn't support P and Z. Actually, the images used in default devfile registry is from personal repo and quay.io/eclipse repo. The images also don't support multi-arch. I think the images used in odo should change to use RH offically built images or RH offically registry (registry.access.redhat.com/) like S2I which support multi-arch.

@rgarg1 Is there any plan to support devfiles container image in RH offical registry? Thanks.

@mohammedzee1000 Hope to get your comments, whether or when it is possible to replace container image in devfiles to support P and Z. Thanks

Hi @yselkowitz , Is there any plan to support devfile container images for multi-arch ? The images are quay.io/eclipse/che-quarkus:nightly and quay.io/eclipse/che-java11-maven:nightly and so on in repo quay.io/eclipse/ .  Those images are required by odo using devfiles to create applications. Thanks.

@mohammedzee1000 @kadel For your visibility.

This issue is applicable for Power as well.

same issue with https://github.com/devfile/api/issues/244

Since the issue have been solved with the private registry . I would like to close this issue and move the discussion to
Duplicate of #4228

Since the issue have been solved with the private registry . I would like to close this issue and move the discussion to
Duplicate of #4228


模型测试

你好，为什么你的模型需要对原图放大三倍后做测试检测率才会高一点？这样对1080p来说确实是一个挑战，请问有什么好的改进建议吗？

Hi @hsl20130659, depending on your use cases, FaceBoxes may or may not be good enough. Enlarging the image would scale up the size of small faces thus improving the performance if your image contains many small faces. You could also modify the network and explore improvement along the line of utilising detection layers at early blocks (stride 8,16 for example) such as those commonly seen in SOTA face detection model.


error[E0599]: no function or associated item named new found for struct backends::winrt::windows::media::playback::MediaPlaybackItem in the current scope

Thanks for all the quick fixes in this library!
I'm attempting to use MediaPlaybackItem. Here's roughly what I have:
use winrt::*;

import!(
    dependencies
        os
    modules
        "windows.media.core"
        "windows.media.playback"
        "windows.media.speechsynthesis"
);

...
        let stream = self.synth.synthesize_text_to_stream_async(text)?.get()?;
        let content_type = stream.content_type()?;
        let source = MediaSource::create_from_stream(stream, content_type)?;
        let item = MediaPlaybackItem::new(source)?;
        self.player.set_source(item)?;
        Ok(())

Only, I get the error in the title. MediaPlaybackItem seems to have a constructor that accepts a source. I've been successfully assigning my source to a player, so I do seem to have one, and I'm not sure why this API call isn't resolving. I can provide a repro if needed, but am filing this in case I'm hitting some issue that already exists.
Thanks again.

The new function is reserved for the "constructor" without any arguments (called a default constructor in some languages). There is a static create method that accepts a MediaSource. You can try something like this:
let item = MediaPlaybackItem::create(source)?;

This should get easier once we have code completion tracked by #107.


Indentation

Hello, how can I use sourcer for indent line/file?

Sourcer is not meant to be a code formatter. It just keeps track of the existing formatting.

Thanks!