Reproducing the Problem

Execute regression tests in master branch
System Information

N/A
Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues

[x] I have provided detailed steps to reproduce the issue

This has been fixed by both https://github.com/QuantConnect/Lean/pull/2246 and https://github.com/QuantConnect/Lean/pull/2236

This has been fixed by both https://github.com/QuantConnect/Lean/pull/2246 and https://github.com/QuantConnect/Lean/pull/2236


assuming const global references for @jit decorated classes and functions

F = ...

@jit
def f(x):
   return F(x)

In order to get extreme performance gain, flags can be set to break CPython semantics so that F always referenced to the object we've seen, instead of allowing later mutations.

It is observed that in usual cases, global references wouldn't change at all.
If you want to use a mutable global variable in @jit functions, maybe writing globals()[name] is better?


establish testing against A+ API (A+ image in Travis build) Aalto-LeT…

…ech/intellij-plugin#47
so adding the scripts from @Mankro s repo to initiate dummy test data for pair submissions endpoint

Great!  Nice to have some Python too.
test-bench-course-mod.py: Did you write this by yourself or did you copy (part of) it? Only asking because of licensing, the code itself is fine. (In aplus-init.sh, you mention it was copied but those two MIT licensed script lines probably don't require any more official attribution than that, so that's ok.)

this is copied from @Mankro 's version of the repo: https://github.com/Mankro/course-templates/tree/test/tools (also MIT)


Implement authentication and authorization

Requirements:

Implement API authentication

Azure API Gateway is an option? ==> Ryan: For authorization, yes. But you still need a separate IdP for the authentication.
OAuth2 with access tokens and refresh tokens (do we need refresh tokens)? ==> Ryan: No for MVP; yes for proper solution.

Implement API authorization

APIM?

@kiran-k Re: 2022-12-13 Daily Scrum,
cc @boris-ning-usds
Logical separation between Upload team and FHIR team.

Upload team: Moving towards SAMS integration in APIM
FHIR team: Moving towards machine IdP integration in APIM

Glidepath for Upload team

Non-SAMS IdP: Basic auth with APIM
Non-SAMS IdP: Client credentials with APIM

Presumably shared secret

SAMS IdP: Client credentials with App Password

Presumably federated credential from SAMS

Machine IdP: Client credentials

Presumably either certificate (pre-registered pubkey) or federated credentials from Machine IdP

(1) is sufficient for FY23-Q1.


Broadcaster can't hear other users, but the others can hear him/her

In my application, once user take the Mic, we change the user role into broadcaster, sometime this user can be heard by everyone in the room, but he/she can't hear another broadcaster
I'm not sure if it related with my other issue #687, this happened without any logs from crashlytics or very hard to produce (it must be with user with slow connection) and it suddenly happend since last couple weeks, this happened on agora 5.2.0 - 5.3.0

agora_rtc_engine: 5.3.0 is based on the native SDK 3.7.0, I think it's better to raise a new ticket to the Agora Support.

I'll close it for now during open ticket, i'll reopen once i found something in logs


Add adapter that implements http.FileSystem for reading gzip files

Signed-off-by: Ashmita Bohara<EMAIL_ADDRESS>

Codecov Report

Merging #68 (55028b2) into master (9d4d4cd) will decrease coverage by 0.27%.
The diff coverage is 15.68%.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
- Coverage   96.48%   96.20%   -0.28%
==========================================
  Files         260      261       +1
  Lines       15215    15266      +51
==========================================
+ Hits        14680    14687       +7
- Misses        452      496      +44
  Partials       83       83

Impacted Files
Coverage Δ

pkg/embed/gzip.go
15.68% <15.68%> (ø)

cmd/collector/app/server/zipkin.go
68.29% <0.00%> (-2.44%)
:arrow_down:

cmd/query/app/server.go
94.11% <0.00%> (-1.48%)
:arrow_down:

cmd/query/app/static_handler.go
97.60% <0.00%> (+1.79%)
:arrow_up:

Continue to review full report at Codecov.

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


Lazy sparse param initialization for distributed training

Description
In the model wide&deep，a categorical feature 'sex' has three values '0,1,2'. Then feed it into SparseEmbedding layer, just like the code below.
embed_weight = mx.symbol.Variable('single_2_embed_weight', stype='row_sparse')
mx.symbol.contrib.SparseEmbedding(data=embed, weight=embed_weight , input_dim=3, output_dim=100000)

Environment info (Required)
Linux, mxnet-1.0.1,GPU,python-2.7
Detail
By checking the model's params， the 'single_2_embed_weight' has values in each row which is not sparse
# data is the model's parmas [checkpoint-0000.params]
>>> data = mx.nd.load('model/checkpoint-0000.params')
>>> data['arg:single_2_embed_weight']

<RowSparseNDArray 100000x32 @cpu(0)>
>>> data['arg:single_2_embed_weight'].indices

[    0     1     2 ... 99997 99998 99999]
<NDArray 100000 @cpu(0)>
>>> data['arg:single_2_embed_weight'].data

[[-0.00308852  0.00507648 -0.00339155 ...  0.00893246  0.00952273
   0.00661108]
 [ 0.00706272  0.00775839 -0.00312851 ... -0.00537404  0.0012673
  -0.00746952]
 [ 0.00495503  0.00214614  0.00021955 ...  0.00096997  0.00675734
  -0.00131243]
 ...
 [-0.00625316 -0.00615439  0.00471845 ... -0.00657647 -0.00150909
  -0.00539171]
 [-0.00908917 -0.00464959 -0.00893743 ... -0.00607294 -0.00969465
  -0.00399319]
 [-0.00455761  0.00734029 -0.00513125 ...  0.00258441  0.0042576
  -0.00019451]]
<NDArray 100000x32 @cpu(0)>

why the categorical input only has three values【or very few values】， it's embedding weight is dense【has values in each row which is not sparse】instead of 【few rows having values】?
Demo
https://gist.github.com/casscw/2e7a436704ead8804261f8b13e84f1a1

@eric-haibin-lin

@ZiyueHuang

Lazy initialization for each row in the row_sparse weight parameter is an optimization we haven't done yet. You're right - ideally the weight can be initialized only when a certain category is seen. Currently all rows are filled at one shot during initialization. What sparse embedding provides is the ability to only retrieve the parameters for the categories in the current mini-batch (instead of loading the full model).
For example, you might have a full model on CPU. For each mini-batch, only load the rows for the seen categories on GPU and perform forward backward.
Lazy initialization is definitely worth investigations though. What's your use case / application?

Thank you for explanation. Just wide&deep for recommendation system

in my application, categorical feature's input dim may be ten millions, but categorical feature actually has two millions. If lazy initialization is adopted, it will save much memory.

Will there be any development plan on this issue? Lazy initialization for large row sparse ndarray is crucial for use cases like sparse embedding on billions of features (with a minibatch only seeing a few thousand). @eric-haibin-lin

I'm not aware if anyone has spare time to work on this. If you'd like to contribute, I'm happy to discuss design and implementation

mark to come back soon

@mxnet-label-bot add[Distributed]


vs-select automatically selects the same value

I think this bug from indexOf method.
I have three products:

Label: Product A, Value: ABC
Label: Product B, Value: ABD
Label: Product C, Value: DEF

When i select Product A, vs-select show Label of both product like this : Product B, Product A .
                                    <vs-select placeholder="Select" color="success" class="mx-w-100" v-model="selectedProduct" v-if="getProducts.length > 0">     <vs-option v-for="(product, index) in getProducts" :key="index" :label="product.name" :value="product.short_name">         {{product.name}}     </vs-option> </vs-select>

Ubuntu 20.04
Node.js v14.15.4
Vuesax 4.0.1-alpha.25
Google Chrome Version 88.0.4324.146 (Official Build) (64-bit)
NPM

Any updates on this?

Well what are the product short_name ? They need to be unique if you set them as the value
It doesn't seem like they are

Well what are the product short_name ? They need to be unique if you set them as the value
It doesn't seem like they are

Here is my options:
[   {name: "Iphone 8 Plus", short_name: "ip8"},   {name: "Iphone X", short_name: "ipx"},   {name: "Samsung Galaxy A21", short_name: "samsunga21"}, ]
When i select Iphone 8 Plus option and also Iphone X selected too .

Hmm then probably another bug.. You can try installing fixed-vuesax instead, make sure to alias it in webpack


converge_if_changed ignore missing required properties

Description
The converge_if_changed method is only checking "properties specified by the user on the resource" to determine whether the block should be called or not. Meaning that if the user forget to pass my important property, my block protected by a converge_if_changed won't be called, and the requirement validation will never happen.
I already read #4596, but I think this issue is a more specific case, which I personally qualify as a bug what do you think? :)
Chef Version
12.13.37 & 12.19.36
Platform Version

CentOS 6.8, & 7.2
Windows Server 2012R2 & 2016

Replication Case
Create a resource
resource_name :my_resource

property :important_attribute, String, required: true

action :try_to_do_something do
  converge_if_changed(:important_attribute) do
    puts 'do something'
  end
end

And use it:
my_resource 'test'

Cc: @aboten

yeah so the problem here is that we should only be looking at the properties that the user cares about.  consider implementing a file provider using this api where the new_resource.mode is nil (the default) but the current_resource.mode exists and is 0755 because the file already exists.  we definitely don't want to do the comparison between the current_resource and new_resource on mode and decide that its 'different'.
so you can't fix this by enforcing that all the properties are checked, because they're not being checked for a good reason.
you might be able to simply run through all the properties given to converge_if_changed in a void context to get them and force the wave function to collapse first.

I'm not sure to understand your proposed solution/workaround. Are you saying I have to perform a validation before my converge_if_changed(:important_and_required_property)?
This seems very counter productive/intuitive.
Instinctively I would expect that the converge_if_changed method to compare the current value of the property to the provided one.
If the property validation is done in a lazy way, I don't care this is an implementation detail, but I would expect the requirement validation to be performed at this point, because I need the value!
My code is saying do something on this property implying please verify that it's provided first.
Am I weird to think like that?

no i'm saying that converge_if_changed could be modified to do a get of all the properties first (and throw the values away).

Instinctively I would expect the converge_if_changed method to compare the current value of the property to the provided one.

right, but that gets the case where the provided value is the default/unset/nil value wrong in many cases.
something more or less like this:
resource_name :file_perms

property :path, String, name_property: true
property :mode, Integer
property :owner, String
property :group, String

load_current_value do
  if File.exist?(path)
    mode File.stat(path).mode
  end
  [...]
end

action :doit do
  converge_if_changed(:mode) do
    FileUtils.chmod new_resource.mode, new_resource.path
  end
  [...]
end

if you call that like:
file_perms "/tmp/foo.xy" do
  owner "root"
end

then new_resource.mode will be nil, and you need to bypass the converge_if_changed check entirely -- because it is unset.  it very deliberately is skipping unset properties.
converge_if_changed could make sure that properties that were specifically marked required were not skipped, or it might be able to just go through all the properties and do a 'get' on them to force un-lazying all of them.  but it can't just compare all of them.

@Annih based on the last merged commit, is this bug fixed ?

no, i just made a comment in the PR that since this wasn't fixed the workaround used there had to be done, so this is still an open issue

Okay this bug is very confusingly written, and I think it is straight up a duplicate of #4596.  I think I confused myself before because I didn't read this correctly and thought it had something to do with the other converge_if_changed bug.
There's nothing particularly special about converge_if_changed here, we simply need to force validation on all required properties before calling the action or something like that.

Actually this looks like it is just fixed, might have been done by #6300
    Chef::Exceptions::ValidationFailed
    ----------------------------------
    important_attribute is a required property


Development

Fixed p4-Vega performance implementing Pixi.js;
Removed "listUsers" from Home page;
Attempting to fix "reload component" issue;


Dart2js problems when import prefix not followed by '.'

If the user declares an import prefix like so:
import '...' as prefix;

and then attempts to use prefix like a first-class object (i.e. not following it with '.'), the spec usually calls for a compile-time error.  There are two exceptions: prefix() should be treated as equivalent to this.prefix(), and prefix = ... should be treated as equivalent to this.prefix = ....
Dart2js currently fails to produce the expected compile-time error for the following patterns:
prefix?.identifier();
prefix?.identifier;
prefix?.identifier = ...;
prefix?.identifier += ...;
prefix?.identifier ??= ...;
f(prefix);
var x = prefix;
var x = prefix[...];

The following patterns cause Dart2js to crash:
prefix ??= ...;
prefix[...] ??= ...;
prefix = ...;
prefix += ...;
prefix[...] = ...;

The following pattern causes Dart2js to generate an improper warning at compile time, and throw a NoSuchMethodError at runtime:
class B {
  void prefix() {}
}
class C extends B {
  void f() {
    prefix();  // Should be equivalent to "this.prefix();"
  }
}

We will adjust the spec to follow lexical scope consistently for the other two cases.

Note that the spec has now been updated to follow lexical scope consistently, and commit 9c90a66d0d9f2dd6f3e7cd831f2a3e15ddeb8603 adjusted the tests accordingly.  So the two exceptions I noted in my initial bug report no longer apply.


[MRG, DOC] Fix contributing install instructions

Addresses https://github.com/jonescompneurolab/hnn-core/issues/311.

Did you check that it works?

Did you check that it works?

Yes, it works :) Just checked right after you asked ;)

Groovy! Thanks for improving the contributing guide @alexrockhill

Nice, thanks @alexrockhill !


bind hook only firing once with scan and aggregated map

Hi,
I'm trying to implement a stream that collect items from other streams into a map / object.
But when I then bind the stream for the collected map I do not receive all updates to the stream.
I made a simple example to showcase that:
const $counter = interval(1000);

export const [useCounterMap, counterMap$] = bind(
  $counter.pipe(
    scan((map, b) => {
      map[String(b)] = b;
      return map;
    }, {})),
  {}
);

Somewhere in a React component:
const counter = useCounterMap();

useEffect(() => {
  console.log(counter, '(counter)')
}, [counter]);

In the logs I receive the following:
{} (counter)
{0: 0} (counter)

And that's it ..
So the init value and the first timer event are received successfully, but then it stops.
I'm pretty new to rxjs, so maybe I'm doing sth wrong or overlooking sth,
but shouldn't the effect keep logging the counter-map with each second?
I do see the full map with every update when I directly subscribe to counterMap$

Oooh, of course right after creating this issue, I had the idea that the mutability of the map might be the problem.
So with this:
  scan((map, b) => {
    Obj.set(map, String(b), b);
    return {...map, [String(b)]: b};
  }, {})),

It works!
I'm not super happy creating a new map on every event though .. is there another way, or am I approaching this completely wrong?

Thanks for your quick answer!
It does make sense and made it clearer to me!
The issue can be closed.
Thank you :)


docs(datamodel): add many-to-many documentation

Add information about how to define a many-to-many relationship.

I would like to see how this works with the Mongo sample app first before we merge to see, but looks great for me

This doesn't work that nice on both postgress and mongo:

only 2 levels of client side queries are generated
on mongo we are getting complex queries that are not indexed properly
tons of extra crud methods on relation object - need to disable every single crud to not take it

While docs are ok.. This type of relation is not production ready
what we can do:

inspect indexes for performance.in separate issue
create flag for model to include it but do not generate crud methods for it

@craicoverflow we can mege.if you agree with suggested approach

We have plans to address this as part of the Mongodata provider that will give performance. I have created separate issue
https://github.com/aerogear/graphback/issues/1228


Setup a contract brokerage solution (contract testing)

For automated testing in CI/CD pipeline, we need a contract brokerage solution after being done with contract creation and mock testing.
Pact Broker ?

would have been great...


[vcpkg-tool][world rebuild] Instrument vcpkg.cmake to check for optional dependencies

Describe the pull request

What does your PR fix?
Fixes #27590

See the tagged issue for more information.
This PR adds a check to the vcpkg override of find_package to detect if it is called without REQUIRED and not explicitly handled using CMAKE_DISABLE_FIND_PACKAGE_ or CMAKE_REQUIRE_FIND_PACKAGE_.
It needs a PR on the tool as well.

Did you test this locally, with non-trivial ports/projects? I think this is too simple with regard to recursive find_package/find_dependency calls. In particular the latter must never use REQUIRED but has to check <Pkg>_FOUND.

With the limited resources of a Codespace I tested ports like sdl2 or OSG successfully. Can you recommend some ports that fin in 20 GB of storage and build in a reasonable amount of time?

qtbase. Probably already breaks with minimal features.... But from experience I can tell you that this is a waste of time. I tried to force CMAKE_REQUIRE_FIND_PACKAGE_ in the qt ports but qt seems to call find_package first with the CONFIG keyword and after that without that which is perfectly fine for CMake, e.g.
find_package(package CONFIG)
find_package(package REQUIRED)

and you cannot deal with that correctly. The same goes for COMPONENTS and OPTIONAL_COMPONENTS

Since find_package is a macro we might be able to record the package name between calls?
Even if some ports break I think the majority of ports is simple enough to work with this. We can set a flag to disable this check for some complex ports like qt.

Since find_package is a macro we might be able to record the package name between calls?

What about recursive and nested find_package calls ?

Even if some ports break I think the majority of ports is simple enough to work with this.

user code also interacts with vcpkg.cmake so it needs to be off by default. Also you could just inject cmake_language(DEFER) and check the property https://cmake.org/cmake/help/latest/prop_gbl/PACKAGES_NOT_FOUND.html

My new approach based on @Neumann-A's suggestions:

Define a variable outside vcpkg's find_package that stores the package names ans relevant arguments like CONFIG, REQUIRED, (OPTIONAL_)COMPONENTS
Add a package on each invocation of find_package to this list [nested/recursive calls shouldn't matter?]
Use cmake_language(DEFER) to process <PackageName>_FIND_REQUIRED and <PackageName>_FIND_REQUIRED_<c>: https://cmake.org/cmake/help/latest/command/find_package.html#package-file-interface-variables
Issue a warning if a package is inside PACKAGES_NOT_FOUND or _FIND_REQUIRED/FIND_REQUIRED` are false

Only enable this check for vcpkg internal usage.

Depends on #27950 and #27982

I believe this PR is replaced by https://github.com/microsoft/vcpkg/pull/27950 . Thanks for your contribution to vcpkg!


Adding a Js Modules

These are the list of things to check out to make sure everything is working correct:

[ ] Check the environment execution setup is done correctly or not.
[ ] Check out for the logger functions and test them whether they works as intended or not.

@behzodhalil
Leave a comment  below if it does not work as intended.

@behzodhalil

[ ] Review this pull request and check for the errors and make sure to  check for the build does not break !


Attachment - Unable to delete attachments and app area is greyed out after action

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.new.expensify.com
Log in with any account
Select any user
Click on Plus button and Add attachment
Attach any attachment
Select Delete option on the side

Expected Result:
User able to delete attachment
Actual Result:
Unable to delete attachments and app area is greyed out after action
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/173882444-2184232b-ca65-42b5-9960-c6c1fe0540fc.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

@madmax330 I broke this so I can fix it


data restriction and attribute masking

This PR includes changes for [Data access restriction]

Quota Limit Enforcement:
Limits data access based on daily API call count and data consumption.[For consumer role]

Attribute Masking Setup:
Controls access to specific data attributes.

Note: Attribute masking feature yet to be implemented for Redis(latest search)

TO DO:

[ ] Docs
[ ] Sql script
[ ] Data Update Python Script


[improve] [test] add end to end deduplication test.

Motivation
Message deduplication feature has been introduced into Pulsar long time ago, but the concrete problem it solve is vague.
The info i can get from the official document is that we can enable a switch in the conf file, and then we can enjoy the message deduplication feature.
But actually, there are a lot of point that we don't clarify.

How do we use it? Should user set the sequence id by api exposed manually, or just let the pulsar client mange it only?
Which concrete situation will we meet message duplication in? Can message deduplication feature handle these situations all?

Modifications
Add end to end deduplication test to clarify what problem message deduplication can handle, and what it can't.
Verifying this change

[x] Make sure that the change passes the CI checks.

This change is already covered by existing tests, such as (please describe tests).
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/thetumbled/pulsar/pull/62

Could you also shorten the method name? The test testProducerDuplicationWithReceiptLostDedupEnabledAndUserControlSequenceIdMultiPartitioned even has 90 characters.
Currently, the checkstyle plugin does not work for tests somehow (see #23079). So there is no code style check for tests.

Could you also shorten the method name? The test testProducerDuplicationWithReceiptLostDedupEnabledAndUserControlSequenceIdMultiPartitioned even has 90 characters.
Currently, the checkstyle plugin does not work for tests somehow (see #23079). So there is no code style check for tests.

Done, thanks for review.


What is the equivalent of fetch.max.wait.ms for testing kafka consumer?

Hi @mhowlett and team
I am tesing kafka producer and consumer using shell script :-
kafka-producer-perf-test.sh and
kafka-consumer-perf-test.sh
While testing producer I am using the command :-
bin/kafka-producer-perf-test.sh --topic topic_name --num-records 1000 --throughput -1 --producer-props bootstrap.servers=host:port batch.size=100000 acks=1 linger.ms=100 buffer.memory=33554432 compression.type=lz4 --record-size 1000
and while using consumer, I am using the command :-
bin/kafka-consumer-perf-test.sh --topic topic_name --bootstrap-server host:port --messages 1000 -fetch-size 100000 --timeout 300000
Like for producer there are parameters :- batch.size and linger.ms
and for consumer there are parameters :- --fetch-size
I want to know that through which parameter (to consumer) we can tell the kafka cluster to wait for a certain time (ms) and then send the message whenever a pull is wanted.
I did a kafka-consumer-perf-test.sh --help,
but could not find a parameter that does it for me?
Thanks for the help.

I'd guess you can set propertyname=value like the kafka-producer-perf-test, but I didn't check.
If you're interested in performance of .net in particular, you could use this:
https://github.com/confluentinc/confluent-kafka-dotnet/tree/master/test/Confluent.Kafka.Benchmark
(results will be different than java)


win-13 low on disk space for Temp directory.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
win-13 low on disk space for Temp directory.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Removing old files and folders off server in local Temp directory to free up disk space.


raygun4java-play2 1.6.0 can't be resolved

There is something weird going on with the raygun4java-play2 1.6.0 artifact; SBT fails to resolve it.
In an empty SBT 0.13.7 project, with only the following contents in build.sbt
libraryDependencies += "com.mindscapehq" % "raygun4java-play2" % "1.6.0"

It fails to resolve it with this error message:
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::              FAILED DOWNLOADS            ::
[warn] 	:: ^ see resolution messages for details  ^ ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: com.mindscapehq#raygun4java-play2;1.6.0!raygun4java-play2.play2
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::

Note that it tries to resolve a file raygun4java-play2.play2 instead of raygun4java-play2.jar. I have no idea why this happens.

I can confirm this.

Thanks for reporting this one, we'll be looking into it shortly and will let you know the results.

Just a note, the v1.6.0 JAR looks to be working corectly at  http://search.maven.org/#artifactdetails|com.mindscapehq|raygun4java-play2|1.6.0|play2, will have a fix for sbt resolving it incorrectly in the very near future.

Any progress on this? The culprit seems to be an incorrect packaging field in the POM:
<packaging>play2</packaging>

That should be
<packaging>jar</packaging>

Thanks for that, this has now been released as v1.6.1 and should be available from Maven Central within 24 hours.


Avoid increasing extra vpgrs for sbcc 64 inverse

resolves #SWDEV-287877
Fix: the 2 extra vgprs reduced the occupancy of sbcc 64 inverse from 2 to 1. Change data type from size_t back to int for sbcc.
Cherry-picked from develop branch

I updated the ticket and we'll need to wait for PMO approval before we merge this.


ShellExecution issue with env variables.

VSCode Version: 1.50.0
OS Version: macOS Catalina 10.15.5 and Windows 10

The code of an extension can affect other extension. For example PlatformIO modifies the process.env. In our extension we are trying to use a modified copy of process.env where we expand PATH with some additional paths. If PlatformIO is enabled, the env update doesn't work. If it is disabled, the env is updated without any issues.
Steps to Reproduce:
const newEnv = Object.assign({}, process.env);
newEnv.PATH = extendedPath;
const shellExec = new vscode.ShellExecution(`echo $Path`, { env: newEnv });
const newTask: vscode.Task = new vscode.Task(taskDefinition, scope, taskName, source, shellExec, problemMatchers);

Use the taskProvider API with the previously given code with PlatformIO extension activated.

Does this issue occur when all extensions are disabled?: No

@brianignacio5 I'm not sure I understand the problem. Are you saying that the newEnv doesn't always get used for the task?

Basically newEnv.PATH is not visible in the output of shellExec and remains with process.env (not updated).

I was not able to reproduce the issue. I made a task like this:
	const newEnv: { [key: string]: string } = <{ [key: string]: string }>Object.assign({}, process.env);
	newEnv.path = process.env.path + ';c:\\users\\alexr00\\TEST';
	const task = new Task(kind, folder, taskName, 'testTask', new ShellExecution('echo %path%', { cwd: cwd, env: newEnv }), matcher);

And when it ran, I could see that my additional path had been added:
> Executing task: echo %path% <

C:\Python27\;C:\Python27\Scripts; <lots more path> C:\Users\alros\AppData\Local\Programs\Microsoft
VS Code Insiders\bin;c:\users\alexr00\TEST

Terminal will be reused by tasks, press any key to close it.


Service workers are here

This adds a simple build step to add service workers using workbox.
The workbox's documentation is pretty bad and their rollup example doesn't work.
Notes

iD.js is pretty huge.(~4mb). Service workers get scared seeing such huge files. We need fix this sometime. ( Code splitting would help, but rollup doesn't support it yet ).
Some of API's that iD uses can have a runtime caching.
Need some input on how to make it work with openstreetmap.org website.
Figure out how to tell the user a new version is available. Currently it just does windows.alert.
Some nice points on offline first.
Currently to test if service worker is updating assets correctly I have setup an iD instance at https://languid-alley.surge.sh . To update it with some local changes I run surge -p dist -d languid-alley.surge.sh. (seems to work fine and also updates the assets properly)

What could go wrong.
Service worker doesn't actually interfere with the actual code, but it might keep sending the stale data if something goes wrong.

Need to make sure service worker doesn't eat up children URLs like openstreetmap.org/iD/xyz more on this here

Bright Side

Huge improvements in load time for users.
Offline support.
We can decide on a better caching strategy for APIs.
Would make iD more aligned with PWA

Hey @kepta just checking in on this.  Do you still think this is worth doing, or better to spend time on other things?
E.g. if service workers are kind of a non-starter with the code the way it is and rollup we can close, and keep in mind for iD v3.

Closing as stale.. Moving to a more modular iD would make this better.


Install instructions/Django configuration example

I cant use this because there isnt even a mention of what engine i would specify. Would be great to have!

OK, SETTINGS section in README.rst, I'll write this:
ENGINE name is 'django_redshift_backend'. You can set the name in your settings.py as::

   DATABASES = {
       'default': {
           'ENGINE': 'django_redshift_backend',
           'NAME': '<your database name>',
           'USER': '<your database username>',
           'PASSWORD': '<your database password>',
           'HOST': '<your database hostname>',
           'PORT': '5439',
       }
   }


improve the documentation of the action

So far the documentation was pretty minimal: a simple example, no explanation of the options, no real-world examples.

cc @jrbourbeau, I hope you don't mind me using dask as an example

Btw, I wrote a short blogpost about how we're using this over in the Dask / Coiled blogs that give you a shoutout https://blog.dask.org/2023/04/18/dask-upstream-testing. I can't remember if I sent you this already or not -- hope that's okay

I don't remember whether you pinged me regarding that, but I do remember reading that blog post and being happy that the action is useful for other people. In other words, yes, that's totally okay!


Add tap-netsuite variant

Checklist

[x] Add/update the file in the appropriate folder (/taps or /targets). The name of the file should match the name of the tap. If there is already one, add a descriptor to the name such as -search.
[x] Add/update the PNG logo image in /assets/logos/<taps or targets>. The image name must match the YAML file name.
[x] Tag @tayloramurphy or @pnadolny13 to flag it for review. Or post to the #hub channel on Meltano slack.

Reviewer Checklist

[ ] Validate file against JSON Schema. https://www.jsonschema.net/home is an option.
[ ] Build website locally and validate everything works.

Exciting to see! @pnadolny13 will you review and merge?

@pnadolny13 I agree, just didn't want to change the current name, but it would make things much clearer, given that there are three versions of NetSuite API, SOAP, REST and SuiteQL, and there is also a difference on the schema of the streams, so we cannot just replace SOAP with REST.

I also set the quality to silver, I know that it is a new tap, but I think that overall it is more organized than the others and is using the SDK, although it need to be tested with more accounts. Let me know if you think it would be better to set it to bronze.

@pnadolny13 I agree, just didn't want to change the current name, but it would make things much clearer, given that there are three versions of NetSuite API, SOAP, REST and SuiteQL, and there is also a difference on the schema of the streams, so we cannot just replace SOAP with REST.

This is a good point. Its a little weird to have a default variant in this case. I'm not very familiar with Netsuite, are the REST/SOAP/SuiteQL taps all attempting to extract somewhat similar data? Or is there totally different data coming from each different API.
It makes me wonder if we should split these out to their own standalone tap pages, we'd basically rename this tap on the hub to tap-netsuite-soap. We should merge this for now but if you think that makes sense I can open an issue to split them out eventually.


Wrong Warning for Permissions in Object (e.g. Report)

I have the following in my codeunit and receive  a wrong warning for permission declaration.

warning LC0003: Do not use an Object ID for properties or variables declaration. Use "tabledata "PRI-KA Entry" = R,
tabledata "Incoming Document" = RIM,
tabledata "Incoming Document Attachment" = RIMD" instead.

  report 50000 "PRI-KA Process Entries"
{
    Caption = 'Process ntries';
    UsageCategory = None;
    ProcessingOnly = true;
    Permissions = tabledata "PRI-KA Entry" = R,
     tabledata "Incoming Document" = RIM,
     tabledata "Incoming Document Attachment" = RIMD;

    dataset
    {
        dataitem(Entry; "PRI-KA Entry")
        {
            trigger OnAfterGetRecord()
            begin

            end;
        }
    }
}

Already mentioned in #4.
Is already in Progress

Should be fixed with version 0.7


[10.x] Test Improvements

fixes failing tests: https://github.com/laravel/framework/actions/runs/5514371113

Thanks!

Thanks, now I noticed why it was working on my local. I had to run more than just my tests to notice the context leak


[Relay] Remove DeviceMap from LowerTE

[Relay] This PR switches the graph, VM and AOT executors to use the merged
device_planner.cc from #9038, thus finally removing DeviceMap from the LowerTE Pass.

We retire analysis/context_analysis.cc and
transforms/device_annotation.cc (and their tests). That
includes the CollectDeviceInfo, CollectDeviceAnnotationOps and
ContextAnalysis entry points. These are all subsumed by the
PlanDevices pass and the device aware visitors.
The following passes now use the new 'Device Aware' visitors to
recover the device for every Relay sub-expression:

backend/aot_executor_codegen.cc (AOTOnDemandAllocator)
backend/graph_plan_memory.cc (StorageAllocaBaseVisitor etc)
backend/te_compiler.cc (LowerTensorExprMutator)
transforms/memory_alloc.cc (DialectRewriter)
backend/vm/compiler.cc (VMFunctionCompiler)

The following passes/utils must maintain the device information
encoded by the device planner within "on_device" annotations and
"param_device_types"/"result_device_type" function attributes:

backend/vm/lambda_lift.cc (LambdaLifter)
transforms/to_a_normal_form.cc (Fill)
ir/expr_functior.cc (Bind)

Remove a lot ad-hoc 'homogeneous' vs 'hetrogeneous' conditionals
in favor of just asking for the device. Also removed a lot of ad-doc
encodings of the 'default' device.
We no longer need to run device-planning twice (before and after
lowering). Device planning is also decoupled from memory planning.
The LowerTE Pass no longer needs an expression-to-device side table
(which was the problem which kicked this series of PRs off in the first place).

@jroesch, @electriclilies thanks for your patience, this is ready for review. Will probably have a few stragglers in unit testing to fix. And I could squeeze one more preparation PR out of this but unfortunately it wont shrink the size that much.

The CI failure in tests/python/unittest/test_meta_schedule_runner.py looks unrelated.

Thanks for the encouragement @Mousius, this one has been a bit of a long slog!

I am going to land this for sake of future pain, feel free to follow up if any other changes need to happen after this.

Thanks @jroesch. @Mousius I didn't push the changes for your comments before the merge but will get them in somehow.


Question about --enable-panel

Hi
just question if i understand correctly man page, I want to generate html report with two panels VISITORS and GEO_LOCATION
i tryied command
goaccess --output ../report.html ../nginx.access.log --persist --restore --config-file ../goaccess.conf --db-path ../db/ --geoip-database ../geoip/GeoLite2-City_20240412/GeoLite2-City.mmdb --ignore-status=404 --http-protocol=no --http-method=no --enable-panel=VISITORS,GEO_LOCATION --html-prefs='{"theme":"bright","layout":"vertical","showTables":false}'

or
goaccess --output ../report.html ../nginx.access.log --persist --restore --config-file ../goaccess.conf --db-path ../db/ --geoip-database ../geoip/GeoLite2-City_20240412/GeoLite2-City.mmdb --ignore-status=404 --http-protocol=no --http-method=no --enable-panel=VISITORS --enable-panel=GEO_LOCATION --html-prefs='{"theme":"bright","layout":"vertical","showTables":false}'

but there are all panels in generated report.html, is option --enable-panel working for html report?

--enable-panel=
Enable parsing/displaying the given panel. List of panels:
VISITORS
REQUESTS
REQUESTS_STATIC
NOT_FOUND
HOSTS
OS
BROWSERS
VISIT_TIMES
VIRTUAL_HOSTS
REFERRERS
REFERRING_SITES
KEYPHRASES
STATUS_CODES
REMOTE_USER
CACHE_STATUS
GEO_LOCATION
MIME_TYPE
TLS_TYPE

You might find the --ignore-panel option useful since panels are enabled by default. For example, --ignore-panel=REQUESTS or --ignore-panel=REQUESTS_STATIC. Alternatively, if you only need to visually hide them, you can toggle them on/off directly from the html report by clicking on the circle next to the panel name in the left-hand side navigation bar.

thanks, --ignore-panel is working, i missed that
toggle also working, is it possible to toggle panels off in --html-prefs ?

You can hide all of them, e.g.,
{"autoHideTables":true,"layout":"horizontal","perPage":7,"theme":"bright","hiddenPanels":["visitors","requests","static_requests","not_found","hosts","os","browsers","visit_time","vhosts","referrers","referring_sites","keyphrases","status_codes","geolocation","asn","mime_type","tls_type"],"geolocation":{"plot":{"chartType":"wmap"}}}

Closing this. Feel free to reopen it as needed.

Yes, I have similar hiddenPanels value, but STATIC_REQUESTS and TIME_DISTRIBUTION are not hidden


Don't assume fill for canvas polygon hit detection.

The canvas renderer will hit detect an unfilled polygon when hovering inside of the outline. This is undesirable and also inconsistent with the WebGL polygon hit detection, which only detects the polygon if the mouse is over the outline. Filled polygons will be hit detected as they were before, either by hovering the fill or outline.
Fixes #4846.

The Travis CI build failures appear to be caused by intermittent timeouts with the tests, unrelated to the PR changes. I see these locally (macOS / Chrome 64) from time to time as well, but they seem more common on Travis.

Thanks @schmidtk.  Apologies for the delay in getting it in.
If anyone wants to contribute rendering tests for this, they would be welcome.

Getting hit detection on polygons that are not filled can be now be achieved by adding a fill with zero opacity:
fill: new Fill({color: 'rgba(0, 0, 0, 0)'})

Should this logic also be applied to CircleStyle without fill?  Currently, for example, if a large box is draw over a small one the small box can be selected if fill is not defined, but if a large circle is drawn over a small one the small circle cannot be selected even if the CircleStyle has no fill.


LKAS without ACC

I was wondering if it would be possible for OpenPilot to maintain lane keep but disengage cruise control. Sometimes in traffic, I'd like to maintain the speed and braking, but have OpenPilot assist with lane keeping. Without OpenPilot, in my Honda CRV, if I press the LKAS button, the car starts assisting with lane keep, but it does not engage cruise. I was wondering if something similar would be possible with openpilot.

anything is possible with code

Please add it to a new version. Also that OP doesn't disengage when push the acceleration pedal.

Not gonna' happen.  @geohot has mentioned this many times.  You can try Arne's fork, though.

Arne to the rescue.
https://github.com/arne182/openpilot

Yeahhh!


NIFI-14115 Set Standard HTTP Headers for Framework Responses

Summary
NIFI-14115 Adjusts the framework Jetty Server configuration to set standard HTTP headers for all responses, regardless of web application.
The current implementation uses the Spring Security HeaderWriterFilter and applies standard Servlet Filters to configured web applications. This covers both framework REST API and custom extensions, but does not include requests to the root web server path.
The updated implementation replaces the Servlet Filter approach with a HeaderWriterHandler to set the same response headers using a Jetty Handler class. The change introduces a new StandardServerProvider to decouple Jetty Server instance initialization from lifecycle operations. The StandardServerProvider includes decoupled tests for basic operation, including returning expected response headers.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Changes look good.  Confirmed all headers previously supported are ported over.  Verifying build locally then will merge.

Local build failed.
[ERROR] Tests run: 3, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.125 s <<< FAILURE! -- in org.apache.nifi.web.server.StandardServerProviderTest [ERROR] org.apache.nifi.web.server.StandardServerProviderTest.testGetServerStart -- Time elapsed: 0.101 s <<< ERROR! java.io.IOException: HTTP/1.1 header parser received no bytes 	at java.net.http/jdk.internal.net.http.HttpClientImpl.send(HttpClientImpl.java:966) 	at java.net.http/jdk.internal.net.http.HttpClientFacade.send(HttpClientFacade.java:133) 	at org.apache.nifi.web.server.StandardServerProviderTest.getResponseHeaders(StandardServerProviderTest.java:118) 	at org.apache.nifi.web.server.StandardServerProviderTest.testGetServerStart(StandardServerProviderTest.java:105) 	at java.base/java.lang.reflect.Method.invoke(Method.java:580) 	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) 	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) Caused by: java.io.IOException: HTTP/1.1 header parser received no bytes 	at java.net.http/jdk.internal.net.http.common.Utils.wrapWithExtraDetail(Utils.java:388) 	at java.net.http/jdk.internal.net.http.Http1Response$HeadersReader.onReadError(Http1Response.java:590) 	at java.net.http/jdk.internal.net.http.Http1AsyncReceiver.checkForErrors(Http1AsyncReceiver.java:302) 	at java.net.http/jdk.internal.net.http.Http1AsyncReceiver.flush(Http1AsyncReceiver.java:268) 	at java.net.http/jdk.internal.net.http.common.SequentialScheduler$LockingRestartableTask.run(SequentialScheduler.java:182) 	at java.net.http/jdk.internal.net.http.common.SequentialScheduler$CompleteRestartableTask.run(SequentialScheduler.java:149) 	at java.net.http/jdk.internal.net.http.common.SequentialScheduler$SchedulableTask.run(SequentialScheduler.java:207) 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144) 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642) 	at java.base/java.lang.Thread.run(Thread.java:1583) Caused by: java.io.EOFException: EOF reached while reading 	at java.net.http/jdk.internal.net.http.Http1AsyncReceiver$Http1TubeSubscriber.onComplete(Http1AsyncReceiver.java:601) 	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$ReadSubscription.signalCompletion(SocketTube.java:648) 	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$InternalReadSubscription.read(SocketTube.java:853) 	at java.net.http/jdk.internal.net.http.SocketTube$SocketFlowTask.run(SocketTube.java:181) 	at java.net.http/jdk.internal.net.http.common.SequentialScheduler$SchedulableTask.run(SequentialScheduler.java:207) 	at java.net.http/jdk.internal.net.http.common.SequentialScheduler.runOrSchedule(SequentialScheduler.java:280) 	at java.net.http/jdk.internal.net.http.common.SequentialScheduler.runOrSchedule(SequentialScheduler.java:233) 	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$InternalReadSubscription.signalReadable(SocketTube.java:782) 	at java.net.http/jdk.internal.net.http.SocketTube$InternalReadPublisher$ReadEvent.signalEvent(SocketTube.java:965) 	at java.net.http/jdk.internal.net.http.SocketTube$SocketFlowEvent.handle(SocketTube.java:253) 	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.handleEvent(HttpClientImpl.java:1469) 	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.lambda$run$3(HttpClientImpl.java:1414) 	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596) 	at java.net.http/jdk.internal.net.http.HttpClientImpl$SelectorManager.run(HttpClientImpl.java:1414)

I tried the build on a different system (linux host) and the build was fine.  I suspect what I'm hitting is a local system/security configuration issue.  Still reviewing

Thanks for the feedback @joewitt.
Although I did not have the local build issue, the error points to a potential timing issue with the Jetty Server instance not being ready for the test sends the HTTP request.
To avoid potential timing issues, I simplified the test method to just check that the server start did not fail.

With timing addressed and header checks changed the test does now pass.  Running full clean build to confirm all the things.  Have all three commits


Bloom Should use Shift-Click to make multiple selections

Bloom uses Command-Click (on the Mac) to make multiple selection of Nodes and Relationships.
For the Mac, it's standard to use Shift-Click.

@gitclem Thanks for the feedback. Bloom's keyboard shortcuts are actually modelled after how things work in the MacOS Finder application. You'll notice that in Finder, Command-Click is used for multiple selection, whereas Shift-Click is used for selecting ranges. As such we don't have any plans to change from using Command-Click in the foreseeable future.

@gitclem Thanks for the feedback. Bloom's keyboard shortcuts are actually modelled after how things work in the MacOS Finder application. You'll notice that in Finder, Command-Click is used for multiple selection, whereas Shift-Click is used for selecting ranges. As such we don't have any plans to change from using Command-Click in the foreseeable future.


CODECHECKER_ANALYZE_OPTS not passed through to codechecker when using sysbuild

Describe the bug
When using Codechecker as an SCA tool with sysbuild, options like CODECHECKER_ANALYZE_OPTS are not passed through to the tool.
To Reproduce

west build -b nrf5340dk/nrf5340/cpuapp <app> --no-sysbuild -- -DZEPHYR_SCA_VARIANT=codechecker -DCODECHECKER_ANALYZE_OPTS="--ignore;/path/to/codechecker-skip.txt"
In the logs, we get "cd /home/xavier/alk/build && /home/xavier/.local/bin/CodeChecker analyze --keep-gcc-include-fixed --keep-gcc-intrin --output /home/xavier/alk/build/sca/codechecker/codechecker.plist --name zephyr --ignore /home/xavier/alk/boa/codechecker_skip.txt /home/xavier/alk/build/compile_commands.json || /usr/bin/cmake -E true"
west build -b nrf5340dk/nrf5340/cpuapp <app> --sysbuild -- -DZEPHYR_SCA_VARIANT=codechecker -DCODECHECKER_ANALYZE_OPTS="--ignore;/path/to/codechecker-skip.txt"
In the logs, we get "cd /home/xavier/alk/build_sysbuild/boa && /home/xavier/.local/bin/CodeChecker analyze --keep-gcc-include-fixed --keep-gcc-intrin --output /home/xavier/alk/build_sysbuild/boa/sca/codechecker/codechecker.plist --name zephyr /home/xavier/alk/build_sysbuild/boa/compile_commands.json || /usr/bin/cmake -E true". As we can see, no mention of the ignore option
When adding message("CODECHECKER_ANALYZE_OPTS ${CODECHECKER_ANALYZE_OPTS}") in zephyr/cmake/sca/codechecker/sca.cmake, we get that CODECHECKER_ANALYZE_OPTS is empty.

Expected behavior
Sysbuild should pass command line options through to zephyr/cmake/sca/codechecker/sca.cmake
Environment (please complete the following information):

OS: Linux
Version: v3.6.99-ncs2

A work-around for this, is to set it in the environment instead:
$ CODECHECKER_ANALYZE_OPTS="--ignore;/path/to/codechecker-skip.txt" west build -b nrf5340dk/nrf5340/cpuapp <app> --sysbuild -- -DZEPHYR_SCA_VARIANT=codechecker


Change footer UI to match with the rest of the theme

I would like to help

is this issue still open


docs(governance): add coc

Awesome! LGTM! 🚀

@all-contributors please add @krishguptadev for docs


Feature Request / Proposal: Duration type

Currently, there is no built-in type of UI5 for representing a Period or Duration of time.
My proposal would be to support ISO-8601 period formatted strings (e.g. P5Y3D) as a raw data representation format + some pretty-fication based on the LocaleData class (specifically to use displayNames to make the periods human readable and writable). A new model type could be created for this (e.g. sap.ui.model.type.Duration).
This ISO format is pretty popular (e.g. it is used in the new java.time.Period and java.time.Duration classes, there is a npm module which handles them, etc).
Of course, at some point some specialized controls could be made to help the user input these values.
If you guys think that it would be valuable to have such a type (i.e. to be able to easily model time duration / periods in UI5), I could contribute for the implementation as well :)
Serban

Hi @serban-petrescu ,
Thank you for your idea. This feature is already discussed and planned with internal record 1-1689.
Regards,
Diana

Hi all,
is there an update?
Best regards
Fabian

Hi Fabian,
no. The topic is still open. Also it is currently unclear when we will have time to work on it.
Best regards
Mathias.


Route not working

This URl is not picked up by the below Flight::route, it returns 404 route not found:
http://localhost/api/intune/hey?error=access_denied&error_description=AADSTS65004%3a+User+declined+to+consent+to+access+the+app.%0d%0aTrace+ID%3a+747c0cc1-ccbd-4e53-8e2f-48812eb24100%0d%0aCorrelation+ID%3a+362e3cb3-20ef-400b-904e-9983bd989184%0d%0aTimestamp%3a+2022-09-08+09%3a58%3a12Z&error_uri=https%3a%2f%2flogin.microsoftonline.com%2ferror%3fcode%3d65004&admin_consent=True&state=x2EUE0fcSj#
This URL IS picked up by the below Flight::Route:
http://localhost/api/intune/hey?error=access_denied&error_uri=https%3a%2f%2flogin.microsoftonline.com%2ferror%3fcode%3d65004&admin_consent=True&state=x2EUE0fcSj#
It looks like something in the query parameter error_description is causing flight not to pick up this route.
If you have any ideas I'd appreciate it :)
Flight::route('GET /api/intune/hey', function () { die("hey"); });

@SeanPatten , try replacing the "%0a" with "%20". Worked for me.

@SeanPatten , try replacing the "%0a" with "%20". Worked for me.

I'd love to, but unfortunately, it's a response from Microsoft Client Token workflow, so I have no control over the query parameters. I have a workaround, I am sending them directly to the Angular site, and posting the data to the endpoint.

huh, I actually am able to replicate this. I'll see if I can come up with a fix. Wonder why it's happening.

@SeanPatten I figured it out. The regex needed to be adjusted to account for multiple lines. Thank goodness for 100% unit test coverage to figure this out :smile:
https://github.com/flightphp/core/pull/510/commits/c7a143db0425b9b6a89e39f95a87ebfeff0f7e31


Question about loading pretrained_model in dcva.py and dcvaUnequalRowColumn.py

On line 138 : state_dictForLayer23={k: v for k, v in netForFeatureExtractionLayer23Dict.items() if k in state_dictForLayer23}
netForFeatureExtractionLayer23Dict.items() seems to be a dictionary of randomly initialized parameters, while state_dictForLayer23 is the real pretrained_model. "for k, v in netF...items()" retains the weight in the random parameter.
So this line of code seems to just load random parameters into the network.
Nevertheless, thank you very much for migrating the code in matlab to the python version for us to learn.

Thanks for pointing it, had found the issue but forgot to update here. Updated now.


Support for address / scriptPubKey descriptors, or expose the descriptor checksum helper

We use address descriptors downstream (because bitcoind doesn't support Miniscript descriptors yet), and we can't compute the address descriptor checksum without bitcoind short of copying the checksum code from here. (See https://github.com/revault/revaultd/pull/364.)
If rust-miniscript had support for address or scriptPubKey descriptors this would also solve the issue as we could just leverage their Display implem. It would be a preferable route if we think the checksum computation should be considered internal.

Can look into this once we begin the partial descriptors journey.

Sorry for the year+ late response. The checksum function is fairly stable, we can make this public for other uses for checksum algorithm too.

Made a PR to make checksum pub, keeping this open for more discussion.
I am not sure if we (ever) want to support addr descriptors. cc @apoelstra

Making checksum pub sounds good to me. This may even be useful for elements-miniscript.
+1 to revisiting the addr descriptor once we have partial descriptors. For now though we should not support it.


Adding version note in SVM tutorials

Right now, in [table of contents of ml tutorials] (http://docs.opencv.org/master/d1/d69/tutorial_table_of_content_ml.html#gsc.tab=0), both SVM tutorials show compatibility > 2.0. However, code inside such tutorials have been upgraded to version 3.0.
My suggestion in this PR is to keep compatibility > 2.0 as theory is valid for both versions but point out in a note that source code is meant for 3.0 (also give link to 2.4 tutorial code)

Please use links with "2.4" (http://docs.opencv.org/2.4/doc/...) instead of default

@alalek done!

:+1:


make slist add at head

The list should have an API that would add at the end of the list and this will be useful where sequential order matters. We can have a pointer that would track the tail of the list and method s_list_add_tail() and time complexity will be O(1). What will be the use cases of slist API's without s_list_add_head()?


[CLN]: make materialization function rather than struct

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

...

New functionality

...

Test plan
How are these changes tested?

[ ] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
Are all docstrings for user-facing APIs updated if required? Do we need to make documentation changes in the docs repository?

#3166
#3165  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 1, 7:14 PM EST: A user started a stack merge that includes this pull request via Graphite.


Pass context params from an execution to a workflow execution

Add invokingExecutionId to WorkflowRequestDTO to support context passing
Enable using arguments from another execution to use its execution level parameters.
This could be used when an assessment workflow gathers information and stores it in the context and then when invoking a follow-up workflow from the options of that assessment then the requester can pass the execution ID of the assessment.
The data flow looks like that:
    Start -> Assessment flow -> addParameter "foo": "bar" to WorkContext -> End with option 1

    Start -> Workflow from option 1 -> use Workcontext param "foo" -> work -> End

The invoking execution ID isn't stored in the DB, it is just a request argument passed in the POST request for invoking a workflow.
Signed-off-by: Roy Golan<EMAIL_ADDRESS>Signed-off-by: Moti Asayag<EMAIL_ADDRESS>FLPATH-458

/lgtm


targetSdk = 31 causes crash on Android

targetSdk was updated, which crashes because the activity isn't exported

Could you please clarify, what the issue is? Probably small reproducer would be the best.

Sorry for the terrible bug report. 🤦
If you run the android app in the multiplatform template in the repo then the app crashes on launch.

Already fixed

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Create VM Wizard - on basic settings step the cluster pull down should include only clusters with at least one assigned host

When creating a new VM on basic settings step, the options for the Cluster menu field are only those clusters that include at least one host (host status is not relevant). The VM can't run on a cluster without any host assigned, so those clusters should be blocked for vms creation..
Currently all existed clusters are displayed.

I think this issue is part of the https://github.com/oVirt/ovirt-web-ui/issues/1307


[Add]: A Card in the Resume Building.

Site URL
https://www.overleaf.com/latex/templates/tagged/cv
Description of the above provided site
A resume or CV is like a fancy paper that tells others about your work and skills. The website provides tools and templates to make this process easier. So, if you're looking for a job or applying to school, Overleaf can help you make a professional-looking resume or CV.
Few sample templates:

Category
Other
Sub Category
Resume Building
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

@the-shivam-gupta
Go ahead. You're assigned!

@the-shivam-gupta
Please add it here!


Make text on plugins page clickable/selectable

Is your feature request related to a problem? Please describe.
The text on the plugins page is currently not clickable or selectable, which makes it difficult to copy and paste information or click on links. This feature request is related to this problem.
Describe the solution you'd like
I would like to be able to click on and select text on the plugins page, as well as click on links within the text.
Describe alternatives you've considered
Currently, I have to manually type out information from the plugins page. This is time-consuming and inefficient.
Additional context
This feature would greatly improve the user experience on the plugins page and make it easier to access information and links.

I just realized this but I guess text selection is actually hand-written in lapce? I thought texts are just selectable by default.

We'd have to implement the selection logic on a widget that these views use for displaying markdown/text.
Though, unless someone decides to implement this, it would probably wait until the floem rewrite.


Output files in several styles

In most of the cases you will want to save crawled results in some DBs. However, it's nice to have a feature to export to a file in CSV, JSON and so on for experimental phase.

The following formats should be supported

JSON
JSON-line
CSV


Take parse_stdlib tests from 17 shards to 31 shards.

Experiment to see if it'll address rdar://problem/34771322, allowing us
to re-enable these tests on Linux CI in the short term.

@swift-ci please test Linux

@swift-ci pleas smoke test macOS

@swift-ci please smoke test macOS

Build failed
Swift Test Linux Platform
Git Sha - e50d9d0bbd9afc400d4e7bc9351b38e4c2765d6a

No silver bullet here.


Is this project dead?

I'm submitting a ... (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Is this project dead?
I cannot see any updates and a lot of open issues and pull Requests.
Thanks

what you are looking for?

Just saw some activity on #2111. So mabye there are still people trying to maintain?

and what will you do? @amelnikoff

@marjan-georgiev tagging you as it seems that you are still active at Swimlane.

@Maintainers Please reflect the (dead) status and archive it:
https://docs.github.com/en/repositories/archiving-a-github-repository/archiving-repositories


how to invoke hsimport using nvim coc

Hello -- Some of the hie features are quite obvious when using nvim with coc... others less so.  How would I invoke the hsimport plugin using coc?  Is there another way to invoke hie features from nvim outside of coc? (i.e., does coc provide access to all of the hie features?)

@EdmundsEcho Take a look at my coc.nvim port of @alanz's vscode plugin: https://github.com/expipiplus1/vscode-hie-server/
quickfixes and code actions to add imports have been working very well for me with this.

If you mean to invoke hsimport with your own input, not just the generated code-action, then I think there is currently no way.
But it would be a nice feature request to make this possible, if anyone wants to tackle it.

@fender @expipiplus1 Thank you both.
@fendor

not just the generated code-action

Can you tell me more about this?  My understanding of what hsimport enables is that it will either insert the import line for me when I use a missing def.  I assume doing its best with the db/hoogle information... or it may rely on my making sure the packages are included in my package.yaml config.  Even there, it might add them??
I don't see any of that capacity in my current coc/hie setup... but, maybe I'm missing something, and thus the question.
- E

Our hsimport plugin uses the local hoogle db. E.g., when you dont see a message in your lsp logs that there is no hoogle db, it should be working.
The Quickfix should be generated if there is a compilation error and we can find the identifier in hoogle. E.g. something like this:
main = print $ isJust Nothing

should generate a Quickfix action for isJust. However, I dont know the specifics for nvim.

When you say local, do you mean in my project?  I ask because during the install of hie it instructs the user to install the hoogle docs for the hie-ghc-version; a global install.

No, with local I mean on your device. So, no remote requests.

@fender Great.    However, in my project, I'm getting a "Not in scope" error.  Should the QuickFix not provide something?

Yeah, it should provide a quickfix, in vscode this little light bulb. Maybe a bug, can you provide a minimal example, please?


Transfer the repository to github.com/ebitengine/ebitengine

This should be in a little far future.

Confirmed that transferring a repository should not be problematic.
I tested this with https://github.com/hajimehoshitest/gomoduletest. This was https://github.com/hajimehoshi/gomoduletest until v2, and moved to hajimehoshitest at v3. v2.0.2 was tagged after transferring, but go get<EMAIL_ADDRESS>succeeded.

So now we call this engine Ebitengine, we have to rename the package. We should be able to transfer to v3 in the following way smoothly:
import (
    "github.com/hajimehoshi/ebiten/v2"
)

to
import (
    ebiten "github.com/ebitengine/ebitengine/v3"
)

I have several ideas:

Package Import Path
Package Name
Pros
Cons

github.com/ebitengine/ebitengine
ebitengine
Most consistent and conventional
Long

github.com/ebitengine/ebiten
ebiten
Backward compatible
Meaningless name?

github.com/ebitengine/core
core
Short
Confusing?


Problem with Ñ

When i print ñ its show another symbol

This plugin only handles printing and is not responsible for processing print content.  Perhaps you should check your part where the print content is being converted into print commands.


Add --silent option to omit the print of AWS credentials in the console

Hi,
Thank you for this repo, we're using it as our sso cli tool to access AWS accounts.
we had to modify it a bit to omit the print of the AWS credentials to the console while authenticating.
This MR contains the code change for that, it doesn't change the default behaviour.
Thanks.
Sadok

@matwhite can you review this PR, appreciate that :)
thanks

hey @sadok-f You have typo in some places: silent, slient and it seems like it is duplicating the verbose mode

Hey @maxtacu , thank you for checking, I've modified my PR.
now with the verbose option, the AWS credentials will be printed to the console.

@sadok-f Nice thanks! LGTM. Can you also please just increment the PATCH version and add changes to the CHANGELOG?

p.s. app version is set in oktaawscli/version.py. Please increment only PATCH version to 0.5.1

Version incremented and changes added to CHANGELOG


"TypeError: unsupported operand type" when refreshing initial access_token

Firstly, thanks for your excellent project! This is a rare find for restarting Compute Engines automatically.
Unfortunately, I can't run your code properly. When I open /vm/start using dev_appserver.py $PWD, I get a HTTP 500 error. This is the traceback
INFO     2017-07-22 10:24:57,971 discovery.py:273] URL being requested: GET https://www.googleapis.com/discovery/v1/apis/compute/v1/rest?userIp=<IP_ADDRESS>
INFO     2017-07-22 10:24:57,971 transport.py:157] Attempting refresh to obtain initial access_token
ERROR    2017-07-22 10:24:57,986 api_server.py:402] Exception while handling service_name: "app_identity_service"
method: "GetAccessToken"
request: "\n5https://www.googleapis.com/auth/devstorage.read_write"
request_id: "JUKaAPsqgy"
Traceback (most recent call last):
  File "/google/google-cloud-sdk/platform/google_appengine/google/appengine/tools/devappserver2/api_server.py", line 377, in _handle_POST
    api_response = _execute_request(request).Encode()
  File "/google/google-cloud-sdk/platform/google_appengine/google/appengine/tools/devappserver2/api_server.py", line 227, in _execute_request
    make_request()
  File "/google/google-cloud-sdk/platform/google_appengine/google/appengine/tools/devappserver2/api_server.py", line 222, in make_request
    request_id)
  File "/google/google-cloud-sdk/platform/google_appengine/google/appengine/api/apiproxy_stub.py", line 141, in MakeSyncCall
    method(request, response)
  File "/google/google-cloud-sdk/platform/google_appengine/google/appengine/api/app_identity/app_identity_defaultcredentialsbased_stub.py", line 200, in _Dynamic_GetAccessToken
    'expires': now + token.expires_in,
TypeError: unsupported operand type(s) for +: 'int' and 'NoneType'
INFO     2017-07-22 10:24:57,998 module.py:832] default: "GET /vm/start HTTP/1.0" 500 291

Steps to reproduce (inside Google Cloud Shell):
STARTVMDIR=~/src/app-engine-start-vm
git clone https://github.com/fivunlm/app-engine-start-vm.git $STARTVMDIR
cd $STARTVMDIR
mkdir lib
pip install -t lib/ -r requirements.txt
vim app.yaml
vim main.py
dev_appserver.py $PWD

(The first few steps, where I use environment variables, are copied from App Engine's "Hello World" Python tutorial.)
Do you know how to fix this?

HI @yuhui maybe it's related to you running this in dev_appserver. Have you tried on appengine?


Issue with fastboot and i18n initializer

When I load a ember-form-for component in fastboot I get: TypeError: Cannot read property 'lookup' of undefined in /initializers/ember-form-for-i18n.js. I think its because the container might not be available.

@PrinceCornNM It looks like it is the same issue as with ember-engines. Can you please take a look at relevant PR ?

I've just hit the same issue - attempted to use the add-on for the first time and seeing the same error:
TypeError: Cannot read property 'lookup' of undefined

Same issue here. Was this ever resolved?

I've faced with the same issue when run ember test.

@proteamer thanks for this test. I'm back from (really long) holidays, so I will fix that soon.

Fixed in #227, if not feel free to reopen this bug.

I'm running into this issue with Fastboot:
$ ember -v
ember-cli: 3.7.0
node: 8.3.0
os: darwin x64
$ ember new testing
...
$ cd testing/
$ ember install ember-form-for
$ ember install ember-cli-fastboot
$ ember generate component hello-world
$ ember s
[visit http://localhost:4200/]
TypeError: Cannot read property 'lookup' of undefined

@michaelrkn Which version of package do you have?

I installed the latest - package.lock says 2.0.0.

@michaelrkn can you please test the repository directly? This bug should be already fixed, if it will work, then I will create a new release next week

Yup, that seems to have fixed it. Didn't realize it had been so long since a release!

2.0.1 published


Add support for query parameters in loadbalancer health-check

spring.cloud.loadbalancer.health-check.path.default=/api/test?view=monitor
When I configured the path with a special character ? 。it was automatically converted to a URI encoding %3F,  causing the request to fail.
How to solve this problem?
thanks

It is probably because of how we construct the full URL in ServiceInstanceListSupplierBuilder assumes the path is just that, a path, not a path and request parameters (which are very distinct from a URI perspective).

Hello,
Is it fine if I take this issue and try to solve it ?

@njoysubho yes, please go ahead and submit a PR.

PR submitted. Thanks.

Fixed with https://github.com/spring-cloud/spring-cloud-commons/commit/de32d2b68c6f980c44855406a4c4fdd6e20f2fb6.


[DO NOT MERGE] Test GA Branch Against Pipeline

DO NOT MERGE

API change check
API changes are not detected in this pull request.


prefer Promise over Bluebird and WebSocket over ws modules

Coverage increased (+0.009%) to 96.61% when pulling f60adb616feba5388cd3766d453122cb161bdf28 on improve-browserify-support into 8393a2412b91c2f26191fba76bf7ba11c22ec53c on master.

Coverage increased (+0.009%) to 96.61% when pulling 14942ea5decd7a929a2f2d22ff84fc245cf5d06e on improve-browserify-support into 8393a2412b91c2f26191fba76bf7ba11c22ec53c on master.

Coverage increased (+0.009%) to 96.61% when pulling 14942ea5decd7a929a2f2d22ff84fc245cf5d06e on improve-browserify-support into 8393a2412b91c2f26191fba76bf7ba11c22ec53c on master.


Enquiry about canonical document distillation

Reading through your code, I found that Fuzz4ALL reads the document from config/documentation/xx/xx.md and stores it to dict_compat[‘docstring’], but no reference to dict_compat[‘docstring’] was found during prompting LLM.

Also, I would like to ask you how you handle document input that exceeds the maximum window limit of LLM?

Hi,

Reading through your code, I found that Fuzz4ALL reads the document from config/documentation/xx/xx.md and stores it to dict_compat[‘docstring’], but no reference to dict_compat[‘docstring’] was found during prompting LLM.

you can see docstring used in these lines to create the initial input to the LLM:
https://github.com/fuzz4all/fuzz4all/blob/13ecb201170a4e80fead9d936e9bf927072e1027/Fuzz4All/target/target.py#L254-L259

Also, I would like to ask you how you handle document input that exceeds the maximum window limit of LLM?

Yes currently there is no special considering to documents which exceeds the maximum token limit of LLMs, however if you use GPT-4 for example, the context window is actually quite large. But you are right that this could definitely be an issue, you may choose to truncate some of the documents in case it runs out

Thanks!


Support for 125 kHz Reader EM4100

Hi,
do you think that it's possible to add support for the 125 kHz Reader (EM4100/RDM6300/RDM630) AliExpress
I own this device and i would like to use these cards, because I'm already using a RFID Jukebox from this project: RPi-Jukebox-RFID
I think your solution will be more friendly for power consumption.
Thanks!
One more thing, could you please add a picture of you Webinterface? I'm still collecting and waiting for my chinabag for building the RFID Shelf.

Sorry for the late feedback. I guess 125kHz support should be possible, but nothing this project supports right now. Feel free to create a pull request.
I'll add a picture soon, right now it is just the main area I am working on and I didn't won't to have to update the screenshot everytime I change something (or have an outdated screenshot).


update text, resolves CAT-15062

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.377%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
615

Relevant Lines:
1002

💛  - Coveralls


Remove delay between coloring data and formulas

Sometimes the colors for data appear long before the formula colors. Tighten up this gap.

Fixed.


Added package + UI  improvement

UI improvements:

Table Of Contents can be collapsed on click, open by default, with on-hover tooltips.
Go back to Table Of Contents button, with on-hover tooltips.
Added link to Nim homepage when it says Nim on description.
Added some missing spaces.

Addition:

Added my packages.

:cat:

Thank you!


2023-07-15: Production Outage

Contact Details
https://aitok.us/ is down
What happened?
not sure yet
Steps to Reproduce

What browsers are you seeing the problem on?
Chrome
Relevant log output
<img width="1212" alt="image" src="https://github.com/aitok-ai/LibreChat/assets/1074685/6de6f147-81c4-43cc-ae74-f07027d27d4c">

Screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Restart Services
Via https://github.com/aitok-ai/LibreChat/actions/runs/5561350496

Problem Found: DNS
╭─   ~ ···················································································································  16:46:43
╰─❯ nslookup aitok.us
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Non-authoritative answer:
Name:	aitok.us
Address: <IP_ADDRESS>
Name:	aitok.us
Address: <IP_ADDRESS>
Name:	aitok.us
Address: <IP_ADDRESS>

╭─   ~ ···················································································································  16:46:51
╰─❯ nslookup libreChat-453514921.ap-southeast-1.elb.amazonaws.com
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Non-authoritative answer:
Name:	libreChat-453514921.ap-southeast-1.elb.amazonaws.com
Address: <IP_ADDRESS>
Name:	libreChat-453514921.ap-southeast-1.elb.amazonaws.com
Address: <IP_ADDRESS>
Name:	libreChat-453514921.ap-southeast-1.elb.amazonaws.com
Address: <IP_ADDRESS>

This is because AWS ALB changes IP addresses from time to time. We should have created CNAME for it.

https://chat.aitok.us/  works
In fact, anything works since we have *.aitok.us pointing to the CNMAE of libreChat-453514921.ap-southeast-1.elb.amazonaws.com

The DNS Problem: A CNAME cannot be placed at the root domain level,
https://ns1.com/resources/cname#:~:text=A CNAME cannot be placed,never to an IP address.

A CNAME cannot be placed at the root domain level, because the root domain is the DNS Start of Authority (SOA) which must point to an IP address.


CONTENT-51 - add put/patch for repositories

TODO:

[x] add tests
[x] more docs

opened a 2nd pr for the docs update i was intending to do: https://github.com/content-services/content-sources-backend/pull/27

https://issues.redhat.com/browse/14674583

⚠ This task currently requires qe-approval, but this PR does not fully resolve the task.  Please contact QE to determine appropriate testing required.

https://issues.redhat.com/browse/HMSCONTENT-71


Unable to submit python language server survey

Hi,
This is a duplicate of https://github.com/microsoft/vscode-python/issues/6996 but it hasn't been resolved and is closed for comments.
The bug might be coming from survey monkey but your users are the ones being impacted. I spent 20 minutes disabling tracker and cookie blocking, extensions, installing google chrome to try and check whether firefox was the culprit, looking for issues on the python-language-server repo, etc... That's not counting the time it took to fill the survey.
If people are going to take time out of their work day to give you feedback, they should be given functional tools to do so. If changing survey provider is not possible for business reasons, maybe a warning should be added at the top of the survey or in the survey prompt to notify users of the issue and let them know their answers will be recorded despite it.
Cheers

@EtienneDesticourt I'm sorry for this frustration, but please know that we truly appreciate the time you took to answer the survey. I'm looking into what may be causing this behaviour.

Language server is going away so this can be closed.


Function missing inside Function App

I have an azure function and I'm trying to deploy this to Azure. I followed this doc and created build/release to deploy the same. On checking Azure, I see function missing inside FunctionApp:

When I publish this function from Visual Studio to the function app, it is displaying correctly:

What build/release step am I missing to publish function to the function app? Please let me know.
Thank you!

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

ID: e8b12f39-46e7-7928-3384-ae8b99c67b6c
Version Independent ID: 2110797a-6604-0989-1504-3e4fe619f899
Content: Deploy a function app to Azure Functions - Azure Pipelines
Content Source: docs/pipelines/targets/azure-functions.md
Product: devops
Technology: devops-cicd-targets
GitHub Login: @pulkitaggarwl
Microsoft Alias: puagarw

@lipalath-ms Thank you for the feedback.
Users have previously reported intermittent when deploying of when redeploying functions with the same name.
Please refer to this alternate method or this post for help on diagnosing, collecting log info, etc.


Add build_info metric and use it in generated queries

Implements #25

WIP: Description forthcoming
TODO

[x] Check whether OTEL tracker should also use the name build_info
[ ] Update README regarding the environment variables we want to support for version/commit
[ ] Only call OpenTelemetry tracker set_build_info once

Update: OTEL support will be dependent on resolution of https://github.com/open-telemetry/opentelemetry-python/issues/3071 (I'm working on a PR)

I think we can treat the OTEL support as non-blocking, since I'm not sure how long it will take for my PR to get reviewed/merged/released
https://github.com/open-telemetry/opentelemetry-python/pull/3306


How to adjust the parameters to improve clarity and display colors correctly?

the left is encoded/decoded by H264Sharp
the right is direct draw from capture.
you can see the Buttone Border Color

        encoder = new();
        encoder.ConverterNumberOfThreads = 2;
        //decoder.EnableSSEYUVConversion = true;

        decoder = new H264Decoder();
        decoder.ConverterNumberOfThreads = 2;
        decoder.Initialize();
        //TagSVCDecodingParam decParam = new TagSVCDecodingParam();
        //decParam.uiTargetDqLayer = 0xff;
        //decParam.eEcActiveIdc = ERROR_CON_IDC.ERROR_CON_FRAME_COPY_CROSS_IDR;
        //decParam.sVideoProperty.eVideoBsType = VIDEO_BITSTREAM_TYPE.VIDEO_BITSTREAM_SVC;
        //decoder.Initialize(decParam);
        //decoder.Initialize(new TagSVCDecodingParam() { });

        var fps = 10;
        var factor = 0.5;
        var bitrate = (int)(xCapture.Width * xCapture.Height * fps * factor);
        ConfigType configNo = ConfigType.ScreenCaptureBasic;//Advanced
        encoder.Initialize(xCapture.Width, xCapture.Height, bitrate, 10, configNo);//200_000_000 4_992_000 1_500_000

this is my encoder/decoder setting

If I understood you well, It seems you are asking about high quality encoding because there are some quality loss.
This is related to implementation detail of Openh264.
ScreenCapture mode tend to have lower quality, it is optimized for low data transfer rate. CameraCapture mode is more accurate but uses more data, you can try that.
To clarify Advanced modes, CameraCaptureAdvanced does more processing and should produce better results with the cost of more CPU usage.
Now another point is the fps, this is another implementation detail of OpenH264. Fps number can effect quality, if you have bitrate control mode on. In general, you should match the fps number with the actual fps that is coming from your image source(i.e.Capture).
here you can see the parameters in detail in native side regarding ConfigType:
https://github.com/ReferenceType/H264Sharp/blob/254467eb716457472cebfd912a80bea4d8eb3496/H264SharpNative/Encoder.cpp#L140
Regarding the off-color I also noticed some blue off-coloring on text with white background. I only have it if the image is sourced from screen capture.  I will investigate this

I send you a sample of decoded image from screen share functionality of P2PVideocall application.
this is between two pcs
Original:

Encoded and decoded:

The color offsets are H264 encoder artifacts. its not lossless encoding.
I will check the color space conversions and get back here, but I'm almost sure this is from the encoder itself.
Ask cisco if you have further questions.

OK, thank you very much for your support.


Installing via Helm to custom namespace conflicts with CRD since 0.3.0

The dynatrace-operator Helm chart lets you install to a custom namespace, including deploying all the webhook configuration to the specified namespace: https://github.com/Dynatrace/helm-charts/tree/dynatrace-operator-v0.3.0/dynatrace-operator/chart/default/templates/Common/webhook
The CRD however hardcodes the expected namespace of the webhook to dynatrace: https://github.com/Dynatrace/dynatrace-operator/blob/master/config/crd/patches/webhook_in_dynakubes.yaml#L13. This causes the update to that version of the operator to fail.
I understand the CRD and the chart are decoupled but it may be worth documenting that this is a possible problem. Our solution is to deploy the CRD alongside the operator with our own Helm chart and modify the offending line to say {{.Release.Namespace}} instead.

Thanks for pointing this out.
For now, if you download the CRD from the latest release, it has this patch applied already and you could just replace the namespace with a different value.


Unsafe component lifecycle methods

https://github.com/ntkme/react-github-btn/blob/d210cecb47525d1a358698910300db995d382b3c/index.js#L15
This when used in apps will throw this error -
react-dom.development.js:12029 Warning: componentWillUpdate has been renamed, and is not recommended for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

* Move data fetching code or side effects to componentDidUpdate.
* Rename componentWillUpdate to UNSAFE_componentWillUpdate to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run `npx react-codemod rename-unsafe-lifecycles` in your project source folder.

Please update the following components: GitHubButton

You should update the lifecycle methods.

Fixed in v1.1.0.


enhancement: open template source file in editor

#72
example of usage :
leetcode show 1 -g -l cpp -e vim
and it can also be used like this
leetcode show 1 -g -l cpp -e "head -n5"
if someone is into that for some reason :D

done


Add eth_signTransaction endpoint

Added the eth_signTransaction endpoint according to the ETH JSON RPC standard (https://eth.wiki/json-rpc/API)

Please see https://github.com/PegaSysEng/ethsigner/pull/336 where pipeline is fixed


cherry picked CI update from main

What type of PR is this?
/kind failing-test

What does this PR do?

fixed ci workflow failed to run
Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

@duanmengkk @rxy0210
Does this PR introduce a user-facing change?

.github/ci.yaml  is  also need @village-way

.github/ci.yaml is also need @village-way

The modifications to .github/ci.yaml were not committed separately. I'll recommit based on the content of the main branch. https://github.com/kosmos-io/kosmos/pull/453/commits/6e1ad8e8fb173705d6caf645bfa7c3cb782d0b1c


set user after authentication

All default functions like Auth::user(), Auth::id() ... are not working because the user is null.

Please, consider pushing a PR with a test case.


What happens if a z-score is out of range?

What happens if a z-score is out of range of the upper limit for a "yes" response? Do we get an error? Is it possible to for this happen at all?

This is pretty much a non-issue, as it's pretty much impossible (in my samples) to even get z >= 2.0, but it does result in an error (if the score is not in any range, the return value is None) so it will need to be patched.


'zsh: command not found: Ok,' with keybase cli client

On Debian 9.2 (stretch); when running keybase prove <foo> in zsh I get
<snip>
▶ NOTICE Success!
zsh: command not found: Ok,

Seems like everything is registering just fine on my profile. I've tried taking a peak at the code but can't find what keybase is trying to invoke after Run is called on Prove. If someone minds telling me where I should be looking for the general flow I wouldn't mind trying to submit a PR if this is a known / confirmed issue, or if there's something I possibly don't have installed that keybase is trying to look for.
Thanks for all the hard work into a great experience, btw!

Closing this due to inactivity.


Setup end to end tests

Pupeteer tests of example implementation app.

Create demo app, basic html with ts support(?) Use rollup.


Looking for some advice

I've been experimenting more with jobqueue and I had a few questions (that probably also fall back to Dask itself).
Firstly, I've noticed scenarios where it seems like results are cached. The case I had was where I had a task that takes no arguments and more or less just sleeps for 5 seconds. If I started up 200 of these on an adaptive cluster, only 1 worker started and all the result seemed to come back in the time it took to run a single task. If I added an (unused) iterator argument to the task, the cluster scaled up and ran as expected. I'm guessing Dask is being clever in this respect somehow? Maybe you can point me somewhere that describes this (and how to turn it off)?
The second thing is I would like to use the adapt() method to control the resources but it seems that it's not possible to use this in combination with scale_up(). The use case is I want the possibility of the cluster dynamically scaling down (so I don't want to use minimum=) but if I have long-running tasks I don't want to wait for one to finish before adapt() decides to scale up (that seems to be how it's working).
Any tips would be very welcome.

@guillaumeeb Thanks for the help. The first point was exactly what I needed.
For the second case, the test setup we have doesn't really allow me to try your first point but from playing with the second case I can see that it would work. In the second case, tweaking wait_count (and interval) has an impact on shutting a worker down after it has executed something butadapt() still shut down some of my workers before I managed to queue an initial task for some of them.
One thing I noticed is that adapt() does not seem to respect minimum=0, it always leaves one worker running. That may be a side effect of our test setup, where we have 3 different clusters running (one for each hardware on the system), one worker of each type persists until they have all finished.

The best solution for the second point is probably your point 3, I'll see how we could implement that in our setup.

butadapt() still shut down some of my workers before I managed to queue an initial task for them.

Oh sorry, I didn't check the code of adaptive before saying that. Do not hesitate to look at it, it is not that big or complicated. https://github.com/dask/distributed/blob/master/distributed/deploy/adaptive.py. Taking a quick look at https://github.com/dask/distributed/blob/master/distributed/deploy/adaptive.py#L294-L307, this seems weird that adaptive is closing your workers before the wait_count, but there is probably something I miss.

The best solution for the second point is probably your point 3, I'll see how we could implement that in our setup.

Please post here once you have a solution!

Ok, when workers sit in the queue it's even more obvious, if no tasks have run, adapt() kills off the queued workers. Letting the scheduler know about (a fake?) task duration seems like it might be the only workable fix.

I tried the wait_count kwarg, what I did was:
cluster.scale(2)
cluster.adapt(minimum=1, maximum=10, wait_count=30)

As I expected, Adaptive only shut down one of my worker after 30 seconds. So that means I have 30 seconds to start tasks.
Anyway, do you still need this open @ocaisa, or did the previous conversation solved you issue?


Add Documentation to Examples

It's difficult to use the Arduino examples right now since you load the examples direct from the IDE and there is no documentation/diagrams, etc.
We should come up with a solution for this.
I would vote for a standard comment template at the top of sketches which includes author, description, controls, and link to breadboard diagram if necessary.
Here's what I had in mind:
// Title: Mincer
// Description: Granular Sampler
// Hardware: Daisy Seed
// Author: Andrew Ikenberry
// Controls:
// Knob 1: pitch
// Knob 2: speed
// Encoder: mode
// Diagram: https://github.com/electro-smith/DaisyExamples/blob/master/seed/Knob/resources/Knob_bb.png

via #16


Merge optional proto3 fields support into null_safety

Tried bringing travis to work here - sorry if it messes up this PR a bit.

Tried bringing travis to work here - sorry if it messes up this PR a bit.

this change was merged as part of https://github.com/dart-lang/protobuf/pull/463


Segfault starting DisplayingABitmap on Linux Mint 17.2 32-bit

haxe 3.2.1, openfl 3.6.1, openfl-samples 3.3.1, and lime 2.9.1.
I'm launching from the export /bin directory after compiling it on the same machine.  I also tried openfl run project.xml linux per https://github.com/openfl/openfl/issues/630#issuecomment-98473773 but with the same result.
~/DisplayingABitmap $ lime build project.xml linux
...
~/bin $ ./DisplayingABitmap
Segmentation fault

And then /var/log/syslog has this to say about it:
kernel: [116461.214165] DisplayingABitm[14990]: segfault at 74614472 ip b10c8d5c sp bff980d0 error 4 in i915_dri.so[b0e56000+526000]

I get the exact same error with PiratePig, and also with Defender's Quest on that machine.  In all three cases, a window is created and immediately closes.

Does openfl test linux --window-hardware=false work?

(Sorry for the delay, took a bit to recompile on my old netbook here)...yes!  It worked great!  What does this mean?  :)

It means that it builds without hardware acceleration, ie. opengl.
Might be a driver issue, or an incompatible card, you need opengl 2 (I think) to be able to run openfl app.
Could you print the output of the command glxinfo | grep 'version'

Ah, thanks.  That would do it, I guess, since the output is:
server glx version string: 1.4
client glx version string: 1.4
GLX version: 1.4
OpenGL version string: 1.4 Mesa 10.1.3

Yeah that's not going to be enough.
I wonder if we could have software as fallback in this case.

That would be wonderful.  That or at least a check, so that instead of segfaulting it can just state the version requirement.  :)  (Meanwhile I'm finding it a bit less than straightforward to upgrade OpenGL versions on Mint.)

I even upgraded to Mint 17.3, and also installed the "unsafe" only-available-showing-level-4 update for Mesa, and it still shows the same versions.  I wonder if that's normal...

Although, it seems that it may be impossible to upgrade past 1.4 since my graphics card is too old: "Intel Mobile 945GSE Express Integrated Graphics Controller"
Is there no way to support older hardware / GL versions for 2d applications?  (I have no idea what I'm asking, but there's a similar question about Godot out there, for example.)

We can't support opengl <2, missing too many features.
The only thing we could do is detecting it and falling back on software,
I'll see if that's possible.

Thanks, I'd appreciate that.  Right now I have two builds out, but for example, itch.io doesn't let you pick between builds for the same platform from their app, so a fallback would let everything work much smoother and save whichever users don't get the right build from exposure to errors and having to figure out which build they need.

I think we already fallback to software automatically for GL 1 drivers, same build

What about SwiftShader as an alternative fallback?
http://blog.chromium.org/2016/06/universal-rendering-with-swiftshader.html

Why close?

Perhaps because Josh has been doing cleanup, and since the issue has not been commented on for almost half a year, it can be safe to assume that the issue has been resolved in that time.

I'm not sure that's a safe assumption, but I hope it's true.  :)

Well, if not, and you get the same issue with this related case, there's no stopping you from reopening the issue :)

Hey guys :smile:
There is no segmentation fault when running openfl test DisplayingABitmap linux here using current versions of OpenFL and Lime. We should support both OpenGL and software renderer support, so even on machines with a poor graphics card, the issue should be resolved.
I understand that the issue exists for your in OpenFL 3.6.1 and Lime 2.9.1, but we have had more than a dozen releases since then, to resolve this and other issues, so to say it is fixed is not quite true from that perspective :smile:


Admin API markdown documentation

This is great and gets us pretty close to what we need. In a few weeks we can go through and talk about markdown formatting style, and some errors that need to be corrected from the Confluence export process. This is at least 80%. 👍

Will redo after last update to Docusaurus site


The use of Kubernetes, Nifi and Kafka to connect IOT devices

Proposed title of article
The use of Kubernetes, Nifi and Kafka technologies to link IOT devices
Proposed article introduction
Nifi, Kubernatives and Kafka are a set of technologies that can work with ranges of data and manage it. Nifi deals with the flow of processes, a technology that can be used to show the flow processes in a IOT device. Kubernatives is responsible for  application orchestration which will be used for orchestration of IOT devices with applications and Kafka is responsible for real time data analysis for real time data. I intent to combine the three technologies to enable the interaction of IOT devices and management of their data.
Key takeaways
By the end of the article, the learners should have an understanding of :

The use and operation of each technologies
The interaction of the KUBECTL
The interaction of the three technologies for the communication of IOT devices

Article quality
This article intends to provide an in-depth look into IOT devices integration using Kafka, Nifi and Kubernetes  by providing examples and use cases. The article is useful to beginners and more experienced learners who are aware of IOT devices and how they operate. I will offer solutions using  the three technologies to demonstrate how to use the technologies more intuitively.
References.
N/A

Could you highlight how the article differs from existing documentation and other blog sites?

I will mainly focus on the use and integration of kubernetes in its orchestration to the storage of IOT devices data and its processes transactions and management. Nifi will be employed to help in the better tracking of information that can play a major role in better tracking in cases of vulnerabilities. @WanjaMIKE

Good afternoon and thank you for submitting your topic to the EngEd program.
After some careful consideration it struck us that this topic may be a bit over saturated throughout other blog sites and official documentations. We're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts. Please feel free to suggest an alternate topic to explore. 🚀