Color capability warning

In powerlevel9k_init() there's this bunch of code :
Display a warning if the terminal does not support 256 colors
local term_colors
term_colors=$(tput colors)
if (( term_colors < 256 )); then
print -P "%F{red}WARNING!%f Your terminal supports less than 256 colors!"
print -P "You should put: %F{blue}export TERM="xterm-256color"%f in your ~/.zshrc"
fi
Unfortunately, on FreeBSD, "tput colors" is the same as "tput col" and returns the number of columns of the terminal. So, I get this :
$ echo $TERM
xterm-256color
$ tput colors
80
// resizing window
$ tput colors
93
So I get the warning every time I open a new shell... even if the environment is nicely configured.
To get the number of colors, the command is :
$ tput Co
256
I suggest to use the command "echotc Co", which is a zsh builtin, to get the color capabilities of the terminal. That command is much more portable than tput.

@sdupille great catch! Could you submit a pull request for that?

Hey, I try to submit a pull request, but I can't manage to do it. Sorry, it's my first pull request using github...
Here's the patch, anyway :
index 0d4b3e7..b039f0d 100755
--- a/powerlevel9k.zsh-theme
+++ b/powerlevel9k.zsh-theme
@@ -947,7 +947,7 @@ function zle-keymap-select {
powerlevel9k_init() {
Display a warning if the terminal does not support 256 colors
local term_colors

term_colors=$(tput colors)

term_colors=$(echotc Co)
if (( term_colors < 256 )); then
print -P "%F{red}WARNING!%f Your terminal supports less than 256 colors!"
print -P "You should put: %F{blue}export TERM="xterm-256color"%f in your ~/.zshrc"

OK, pull request created. Hope that's OK.

As @dritter said, great catch, and thanks for the PR, @sdupille!


Use route directive in Caddyfile

This PR changes the logic within the Caddyfile to properly implement ordering logic for serving the different endpoints for Chibisafe. Some of the header lines have also been removed to reduce redundant lines, as X-Forwarded-For and X-Forwarded-Proto are removed from the request header by Caddy unless trusted_proxies is used.
The biggest change is the use of the route directive, which allows us to order directives as we wish, rather than having Caddy automatically re-order them. This enables the usage of pass_thru for file_server so that Caddy will always check for an existing file first before attempting to proxy to the backend services.

Too much to rebase, I commited with credit 👍
https://github.com/chibisafe/chibisafe/commit/fffd55a6960745e236823ee634c5ce9c5d677a89


refactor: create kv client once and update to structured logging

Signed-off-by: Anish Ramasekar<EMAIL_ADDRESS>
Reason for Change:

Keyvault name is per SecretProviderClass. So switching to just getting the vault URL and kv client once instead of recreating the client for every single object in the array.

Requirements

[x] squashed commits
[x] included documentation
[x] added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

[ ] Yes
[x] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

Codecov Report

Merging #634 (e82a33f) into master (385f9d4) will increase coverage by 0.11%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #634      +/-   ##
==========================================
+ Coverage   64.55%   64.66%   +0.11%
==========================================
  Files           7        7
  Lines         584      583       -1
==========================================
  Hits          377      377
+ Misses        173      172       -1
  Partials       34       34

/azp run pr-e2e-azure


Add float support to libsodium custom ops

Hey @mortendahl,
Here is what I have right now. It's working with floats. I will have to use generics or create new classes to make it work with uint8 as well.
I have created this PR in case you have time to provide preliminary feedback. However we probably prefer to merge libsodium-ops to master then merge this pr to master.
Thank you!

Hey @mortendahl ,
I think I have addressed most of your comments, except one. I wasn't able to add tensorshape checks in MakeShape:
inline TensorShape makeShapeSeal<float>(const Tensor& ciphertext, OpKernelContext* ctx){
  auto ciphertext_shape = ciphertext.shape();
  ciphertext_shape.AddDim(sizeof(float));
  int last_dim = ciphertext_shape.dims() - 1;
  OP_REQUIRES(context, ciphertext_shape.dim_size(last_dim) == sizeof(float),
      errors::Internal("Last dim of ciphertext_shape should match sizeof(float)"));
  return ciphertext_shape;

With this function, I will be getting:
In file included from external/primitives/tf_encrypted/primitives/sodium/cc/easy_box_kernels.cc:13:
external/primitives/tf_encrypted/primitives/sodium/cc/easy_box_kernels_utils.h:27:15: error: use of undeclared identifier 'context'
  OP_REQUIRES(context, ciphertext_shape.dim_size(last_dim) == sizeof(float),
              ^
external/primitives/tf_encrypted/primitives/sodium/cc/easy_box_kernels_utils.h:27:15: error: use of undeclared identifier 'context'
external/primitives/tf_encrypted/primitives/sodium/cc/easy_box_kernels_utils.h:27:3: error: non-void function 'makeShapeSeal' should return a value [-Wreturn-type]
  OP_REQUIRES(context, ciphertext_shape.dim_size(last_dim) == sizeof(float),
  ^
bazel-out/darwin-opt/bin/external/local_config_tf/include/tensorflow/core/framework/op_kernel.h:1780:7: note: expanded from macro 'OP_REQUIRES'
      return;

I understand that the problem is that makeShapeSeal expect to return a TensorShape, when OP_REQUIRES return void. just not sure yet how to handle this situation. Any idea are welcomed :). thank you.
Note I also add to add plaintext_type to sodium_easy_box_open_detached to handle uint8 and float32 otherwise the compiler was complaining that it wasn't an input parameter.
Thank you!

BTW, why not add support for int32, uint32, uint64, int6, double while you're at it? Given the current code it should be straight forward!

plaintext_type as been renamed to plaintext_dtype. We will parametrize test and support to additional types in a separate PR. Thanks @mortendahl for your help and review!


Display clearer error messages when Wikibase instance is unreachable

Currently running OpenRefine directly from the master branch and reconciling with an independent Wikibase instance doesn't fully work on Chrome on MacOS Catalina.
To Reproduce
Steps to reproduce the behavior:

First, run OR in terminal from cloned master branch
Then, add reconcilitation service and corresponding manifest
Finally, try adding some data and building a schema to prepare to upload the data to Wikibase

Current Results
Once you get to the statement level for a new item – it is not possible to add values to the statement, the value field does not appear.
Expected Behavior
A value box should appear, once a property is selected. At the moment I can select a property, but then nothing happens.
Screenshots

Versions

Operating System: MacOS Catalina 10.15.6
Browser Version:  Google Chrome Version 84.0.4147.135 (Official Build) (64-bit)
JRE or JDK Version: JRE 1.8.0_261
OpenRefine: OpenRefine master branch as of 28.08.2020

Thanks for the report. Are there any error messages on the Javascript console or the terminal that the server is started from? Does the problem only occur with that single reconciliation service?

There are some messages around missing constraint configs, but they don't seem vital (attaching a file copy of the terminal session below). And yes, the problem is only with this service, Wikidata reconciliation and upload works fine within the same session in Chrome. Also I should say, the reconcilitation and upload to Rhizome's Wikibase works fine in Firefox on my machine, so it seems to be just a Chrome-specific issue.
Terminal Saved Output.txt

The terminal log seems fine. Could you provide the error messages (if any) of the js console?

Ah, yes, sorry, forgot to check the console messages. Here are screenshot of what I'm getting:

Sorry, I'm a researcher, not a proper developer so let me know if there is anything else that will be helpful to see, but I'm missing to send.

Thanks for the log. Could you click on "project-bundle.js:10439" to see the script executing and provide a screenshot of it?

Does this help:

I just realized that the project-bundle.js on my computer should be the same as yours. So providing the line numbers should be enough :)
Could you click on the last error message to see the stack just like you did with the one above it?
From the message you provided, some ajax calls were blocked by your browser (ERR_BLOCKED_BY_CLIENT). Do you use AdBlockers? Maybe this could help: https://stackoverflow.com/questions/23341765/getting-neterr-blocked-by-client-error-on-some-ajax-calls.

Ah, yes, good call – blockers. So it's not the AdBlock extension I use, that one doesn't make any difference when it's turned on or off; it was Privacy Badger. Interestingly, I didn't think that could be the problem because Wikidata instance was working, it was only when I switched to the Rhizome custom Wikibase that the schema got broken. I'm attaching the other screenshot you requested just in case. Perhaps the solution is to just indicate in the documentation that users should disable any blockers / privacy extensions when using the service?

So Privacy Badger is the problem in your case. But I still cannot reproduce it on Windows.
I've enabled both Privacy Badger and AdBlocker, but I could still edit the schema as usual.

The 404 error for "rhizome-artbase.reconci.link/search?filter=..." seems unrelated here though it's quite annoying:) I'll try fixing it.
@tfmorris @wetneb Could you reproduce the issue on your computer (with Privacy Badger enabled)?

According to Privacy Badger's documentation, this add-on learns websites to block gradually: it blocks a given domain if it has been reached from at least three sites. So, with a fresh install of the app, it is normal that it does not block anything, whereas for @lozanaross who has been using it for longer (and has interacted with the Wikibase instance more), the domain is blocked.
On our side, I think we should just display a clear error message saying that the target Wikibase is unreachable (perhaps hinting at privacy extensions/ad blockers, although this is unlikely to happen with other extensions which use a fixed list of blocked domains).

Thanks, yes this sounds like a good solution – just providing a bit more clues to users as to what might be the issue will be very helpful.

So, the task here is to review where we are calling the Wikibase instance and make sure we have proper error handling in place to display an error message when the service is unreachable.


Unable to install via Windows 11 Store

VS Code Version: -
OS Version: Windows 11 Pro, build: 22000.318

Unable to install from the Windows Store, other apps on the store work fine.
The connection was lost during download
Code: 0x80072EE2

@bnkamalesh, is this reproducing today?

@sbatten does not look like an issue anymore! Thanks for the help.


Change attachments filename boundaries

I changed the addAttachment method on SPService to do not exclude "&" and "-" characters from filename

Q
A

Bug fix?
[ x]

New feature?
[ ]

New sample?
[ ]

Related issues?
fixes 526

What's in this Pull Request?
Update the SPService to manage more characters adding attachments

Thank you @Gregghis for the fix!


Kernels die in Julibox.org

Hello everyone,
I have the current problem that some kernels in JuliaBox are dying all the time. At first I was using Julia 0.4.2. After some time, the kernels of all my files wouldn't connect anymore. Then I switched to 0.5.0-dev. It worked but after some time this kernel also stopped working. Now I had to switch to the current older version of Julia (0.3.12). But I fear that after some time it will stop working too.
I read some comments here of some people who seem to have the same problem as me.
Pkg.update() didn't seem to fix this.
I think I added some packages for plotting before it stopped working.
A detailed solution to this problem would be appreciated.

0.3.12 is unlikely to stop working, because it has a different architecture. It would be useful to have a detailed set of steps to reproduce this.
Does resetting make 0.4.2 work again?

At first, thanks for the quick answer.
Hmm, resetting to the system with installed packages only seemed to fix the problem.
0.4.2 and 0.5.0-dev are working again. It seems there where some package conflicts that corrupted my kernels.
The only packages I added where PyPlot and Gadfly via Pkg.add("PyPlot") and Pkg.add("Gadfly"). By doing this, my package system seemed to get corrupted.
But now I have the issue that I am unable to add any new packages due to missing space. Here's the error message:
INFO: Initializing package repository /home/juser/.julia/v0.4
INFO: Cloning METADATA from git://github.com/JuliaLang/METADATA.jl
fatal: cannot create directory at 'Timestamps': No space left on device
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry the checkout with 'git checkout -f HEAD'
LoadError: failed process: Process(git clone -q -b metadata-v2 git://github.com/JuliaLang/METADATA.jl METADATA, ProcessExited(128)) [128]
while loading In[2], in expression starting on line 1
in run at ./process.jl:531
in anonymous at pkg/dir.jl:52
in cd at ./file.jl:22
in init at pkg/dir.jl:50
in cd at pkg/dir.jl:28
in add at pkg.jl:23

Nevermind, I see where the problem is. Time to delete some files....
Again, thanks for the help.


Request class hides connection.

org.scribe.model.Request class hides connection, so that it's impossible to modify it's settings (e.g. I'm not able to set followRedirects.

This is true...
I'm not 100% sold on the idea of making a getter for the connection object though. Perhaps making it protected so you can subclass and modify it at will, perhaps adding your own followRedirectMethod?
Pablo

If you are sure, that some parameters in connection are a must, then maybe in createConnection() execute empty, protected method configureConnection(connection) and after that, set those parameters.
private void createConnection() {
Connection connection;
...
configureConnection(connection);
connection.setRequestMethod(this.verb.name());
if (connectTimeout != null)
{
connection.setConnectTimeout(connectTimeout.intValue());
}
}
Thx anyway
Lukas

I don't want to provide an easy access to the connection object. I want to make it difficult on purpose so people don't mess with it.
It should be possible for guys who know what they are doing (like yourself) to modify the connection.
Creating the configureConnection method to me is just like a setter, but a bit more confusing.


Add ADOPTERS.md to KubeClarity

Please add ADOPTERS.md file, so we could better track usage of the APIClarity.
Example: bank-vaults, logging-operator

Marking as won't fix as this issue will not be addressed for now.


Cannot install Infer in a Redhat-based machine: fail at step updating OCaml

Hi,
I was trying to install Infer using the script build-infer.sh but it failed at the first steps when updating OCaml. Below is the error message which is thrown during the installation.
Could you advise me what happens and how to fix it? I ran the command build-infer.sh under a Linux environment (Redhat-based)
Thank you!
=-=- Updating package repositories =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
[default] synchronized from https://opam.ocaml.org

=-=- Synchronizing development packages -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
[lock] https://github.com/rgrinberg/opam-lock already up-to-date
Exception:
Reqs_error
 (Process_error
   ("opam install -y ANSITerminal.0.7 atd.1.2.0 atdgen.1.10.0 bin_prot.113.33.00+4.03 biniou.1.0.12 camlp4.4.04+1 camlzip.1.07 camomile.0.8.5 conf-autoconf.0.1 conf-m4.1 conf-pkg-config.1.0 conf-which.1 core.113.33.02+4.03 core_kernel.113.33.02+4.03 cppo.1.4.1 ctypes.0.11.3 easy-format.1.2.0 extlib-compat.1.7.0 fieldslib.113.24.00 javalib.2.3.3 jbuilder.1.0+beta6 lambda-term.1.10.1 lwt.2.7.1 menhir.20170101 merlin-extend.0.3 ocaml-migrate-parsetree.0.7 ocamlbuild.0.11.0 ocamlfind.1.7.1 ounit.2.0.0 ppx_assert.113.33.00 ppx_bench.113.33.00+4.03 ppx_bin_prot.113.33.00+4.03 ppx_compare.113.33.00+4.03 ppx_core.113.33.01+4.03 ppx_custom_printf.113.33.00+4.03 ppx_deriving.4.1 ppx_driver.113.33.02+4.03 ppx_enumerate.113.33.00+4.03 ppx_expect.113.33.01+4.03 ppx_fail.113.33.00+4.03 ppx_fields_conv.113.33.00+4.03 ppx_here.113.33.00 ppx_inline_test.113.33.00+4.03 ppx_jane.113.33.00 ppx_let.113.33.00+4.03 ppx_optcomp.113.33.01+4.03 ppx_pipebang.113.33.00+4.03 ppx_sexp_conv.113.33.01+4.03 ppx_sexp_message.113.33.00+4.03 ppx_sexp_value.113.33.00+4.03 ppx_tools.5.0 ppx_tools_versioned.5.0alpha ppx_type_conv.113.33.02+4.03 ppx_typerep_conv.113.33.00+4.03 ppx_variants_conv.113.33.00+4.03 re.1.7.1 react.1.2.0 reason.1.13.4 reason-parser.1.13.4 result.1.2 sawja.1.5.2 sexplib.113.33.00+4.03 topkg.0.8.1 typerep.113.24.00 utop.1.19.3 variantslib.113.24.00 xmlm.1.2.0 yojson.1.3.3 zed.1.4 --color=never",
   Unix.WEXITED 3)).

Which version of infer are you trying to build?

I am using a modified version of Infer, which is forked from revision eb477b771ed095b21e889be306269b73d4d29900 of the original Infer, back in April 27, 2017.
I just try the newest Infer and this issue doesn't occur.
But before, I also didn't any issue with my modified version of Infer from the revision eb477b771ed095b21e889be306269b73d4d29900

@tddsg This might be due to a change in opam, which can sometime break external projects, as discussed eg on this PR: https://github.com/ocaml/opam-repository/pull/10047
Sometimes it can be worked around by using ./build-infer.sh --no-opam-lock (or record the changes with make opam.lock, then ./build-infer.sh) but sometimes that's not enough either and it's necessary to adjust the dependencies in the opam file manually and run make opam.lock.


Add edge extraction features from polyhedron

This PR allows extracting the list of edges from the output file of qconvex.
It makes also the radii of STP BV accessible after the object has been built

I should not have created this PR directly from Ana's repository, I cannot push to it. I will close it and reopen it from my repository. Thank you for the quick reaction, I will fix everything in the new PR


Player#setTransformSafely

.setLocation has .setLocationSafely
.setLocationAndRotation has .setLocationAndRotationSafely
.setTransform has nothing.
I would like a .setTransformSafely() method.

You have a good point, I'll look into adding this soon.


BC - Launch Page, App Logo, App Name

What

[x] Launch page is displayed while app is loading
[x] Works with both dark and light mode
[x] Added logo and name to app
[x] #40

#Evidence

looks good


Text not colored in hover state

What happened?
Text is not colered in hover state in light mode (create).
Please reproduce in steps

go to create
hover on collections of NFT

Expected Behavior
Text should be pink same as the line under the section.
What browsers are you seeing the problem on?
Chrome
At which address did you encounter bug?
kodadot.xyz
Are you logged in?
Yes
Which wallet you are using?
SubWallet
At which chain did you encounter bug?
No response
Screenshots

Relevant log output
No response
Payment link for reward
https://kodadot.xyz/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

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

:wave:


AttributeError: 'SummaryMetadata' object has no attribute 'display_name' ( OS:windows, Python36, tensorflow (1.3.0rc0) tensorflow-tensorboard (0.1.7)

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug or a feature request.
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

@SusieZ Sorry, I don't understand the question.
Please follow the instructions in the New issue template.
And please note: if this is a tensorboard question, please file it here:
https://github.com/tensorflow/tensorboard/issues


Added Arch Linux package recipe

added new directory for not clobbering /src while building the arch linux package.
added lines to .gitignore for cleaner maintenance after building.

Thanks!


Add a converter that protects agains log forging and xss injection

The existing Logback CRLFConverter protects agains log forging by injecting newline characters in the logs.  However it does not protect agains XSS injection attacks.  This adds a new CRLFAndXSSConverter that escapes angle brackets to prevent the execution of scripts when the logs are processed with a tool that processes html.

In my opinion, this is first of all a very weak XSS defense, and second HTML escaping before logging is the wrong place to do it. I respectfully suggest this is not integrated into the project.

In my opinion, this is first of all a very weak XSS defense, and second HTML escaping before logging is the wrong place to do it. I respectfully suggest this is not integrated into the project.

stijns, thanks for your effort to contribute to the project; Did you see jim manico's remarks ? i think he makes sense and would iinvite you to rethnk your changes. Please let me know if need help.

For those that love logging please take a look at our new cheatsheet:
https://cheatsheetseries.owasp.org/cheatsheets/Application_Logging_Vocabulary_Cheat_Sheet.html

I can certainly see the first point of this being a weak XSS defence, the approach I took is probably a bit naive.
But I don't get the second point, I believe that this is referring to the practice of sanitising input at the point of entry instead of sanitising when logging.  Which is certainly the preferred approach, but then it seems to me that the same logic could be used against the existing CRLFConverter.  This converter protects against log forging not at the point of entry but at the log level.

The CRLF Converter sanitizes the input for the context in which it will be output, which in this case is a line-delimited log file. It makes sense in that case for the input to be sanitized of CRs and LFs because those could lead to injections in the log itself. If you later read the log file and display its contents in an HTML page, it would make sense only at that point to sanitize it for the HTML context by escaping characters dangerous in HTML such as < > etc.
You would not want to do HTML sanitization before you output it to HTML because you can't be sure of what other contexts the log information might be consumed in. For example, the log could also be written to a database, or ingested by a bot, or viewed by another viewer with its own set of encodings. In any of those cases the pre-existing HTML encoding could cause problems, including a loss of integrity of what was originally intended to be written to the log in the first place.

Thanks for the clarification.


Getting home-manager standalone plus neovim plus themes.

This PR will not be merged as-is. It will be split into multiple PRs and later updated to only include HM.

Closing in favor of https://github.com/SoxinOS/soxin/pull/37.


Issues with Bitfinex

I'm currently using the Xchange library to stream data from various Bitcoin exchanges, including Bitfinex. I'm having a problem with some of the Bitfinex ticker data I'm receiving, namely, some samples have an execution timestamp that comes after in time from the received timestamp, which doesn't make much sense. I ran some tests by querying Bitfinex's API directly, and I didn't seem to receive any data samples where the execution timestamp was after the received timestamp. I've attached a histogram showing the difference in seconds between the received and execution timestamps (positive is what I want, as it means the received timestamp came later in time). I've also contacted Bitfinex directly, and when they ran their own tests, they didn't seem to have this problem either.
bitfinex_ticker.pdf

I don't see both "execution timestamp" and "received timestamp" on the Ticker from bitfinex so I'm not sure what you're trying to compare. Here's the adapter method. One thing is that it looks like the time resolution on the timestamp from bitfinex is seconds and not milliseconds. that could be your issue.
public static Ticker adaptTicker(BitfinexTicker bitfinexTicker, CurrencyPair currencyPair) {
BigDecimal last = bitfinexTicker.getLast_price();
BigDecimal bid = bitfinexTicker.getBid();
BigDecimal ask = bitfinexTicker.getAsk();
BigDecimal high = bitfinexTicker.getHigh();
BigDecimal low = bitfinexTicker.getLow();
BigDecimal volume = bitfinexTicker.getVolume();

Date timestamp = DateUtils.fromMillisUtc((long) (bitfinexTicker.getTimestamp() * 1000L));

return new Ticker.Builder().currencyPair(currencyPair).last(last).bid(bid).ask(ask).high(high).low(low).volume(volume).timestamp(timestamp)
    .build();

}

Apologies, I meant to say "receivedAt" instead of received timestamp and "timestamp" instead of execution timestamp. Essentially what's going on is that the value for "receivedAt" (when I get the data) is often lower (earlier in time) than the value for "timestamp," which doesn't make sense as this is saying I'm getting ticker values for data from the future. You can see from the histogram that these values are pretty significant, too (the numbers are all in seconds).
Could this issue be reopened?

What is receivedAt? I cannot find what your referencing. What object is it contained in?

receivedAt is a field for a TickerWrapper object we constructed as a wrapper around the Xchange Ticker object. When we receive a new Ticker, we set receivedAt = new Date(). I can send you the code if you would like.

Well that's the answer. new Date() is your computer's time. timestamp is Bitfinex's time they send.

I understand. Since new Date() is my computer's current time, I'm setting receivedAt = new Date() to show when I get the ticker while timestamp is just the ticker timestamp. The ticker timestamp should be much earlier than when I get the ticker value, but my issue is that the ticker timestamp is often higher in value than receivedAt.

Your issue is not in the code in XChange, so we cannot help you here. XChange is unmarshalling the JSON correctly. Without full control and insight into your and Bitfinex's servers, how are you expecting to figure out your problem? It's simply a matter of unsynchronized clocks and it's not in your control, so better not to worry about it and design your system with that fact in consideration.


abaplint.json "skipGeneratedFunctionGroups": true, not ignoring Function group with namespace

I have an issue where a generated function group is checked and various syntax issues are picked up.
The function group is not in the z namespace.

Is there a way to exclude a function group from any abaplint checks? or do you have to specify all the *.prog.abap files under each individual check?

currently, the logic looks for a TOBJ object corresponding to the function module, do you have this in the repository?
or which kind of generated function group is it?

The generated function groups is for view maintenance. The TOBJ xml does exist in the repository and is serialised.

thanks, I think there is a bug in the regex when getting the AREA from the TOBJ object, the "/" should be escaped(https://github.com/abaplint/abaplint/issues/1226#issuecomment-679926949)
https://github.com/abaplint/abaplint/blob/master/packages/core/src/objects/maintenance_and_transport_object.ts#L24

also, can you try running abaplint on command line, to check that the results are the same? do npm install @abaplint/cli -g to install/upgrade and then run abaplint in the directory
note that I dont always keep the vscode extension up to date

Thanks @larshp ,
I'll test this and the macro dash(-) issue via the command line first thing in the morning.
W.r.t the vscode extension, you have mentioned this to me previously, but I thought I would give it a go anyway seeing as there was an update available. If the update didn't address the issue I would have defaulted to the cli.
Thanks again for all your assistance!

The issue is not present in the cli, thus this is only a vscode extension issue.

the fix will be inside abaplint, as I will be moving everything to URIs sometime
reopening

issue might be the same as https://github.com/abaplint/abaplint/issues/1230

@habrahams can you help trying again? might have been the same issue as https://github.com/abaplint/abaplint/issues/1230

Issue resolved, thanks! Do you still want to keep the issue open or can it be closed now?

closing


TextInput placeholder is always multiline on Android

Description
On Android, the <TextInput>'s placeholder property always renders to multiple lines, whereas on iOS it trails the ending (like ellipsizeMode="trail").

React Native version:
info Fetching system and libraries information...
System:
    OS: macOS 10.15.6
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 5.61 GB / 32.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 14.8.0 - /usr/local/bin/node
    Yarn: 1.22.4 - /usr/local/bin/yarn
    npm: 6.14.7 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.9.3 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 13.6, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK:
      API Levels: 25, 28, 29
      Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.0, 30.0.0, 30.0.0, 30.0.1
      System Images: android-29 | Google APIs Intel x86 Atom, android-29 | Google Play Intel x86 Atom
      Android NDK: 21.2.6472646
  IDEs:
    Android Studio: 4.0 AI-193.69<IP_ADDRESS>26763
    Xcode: 11.6/11E708 - /usr/bin/xcodebuild
  Languages:
    Java: 1.8.0_265 - /usr/bin/javac
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: 0.63.2 => 0.63.2
  npmGlobalPackages:
    *react-native*: Not Found

Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Create <TextInput> component
Set placeholder to anything long

Expected Results
The placeholder should leave dots at the trail, so no line wraps.
Snack, code example, screenshot, or link to a repository:
<TextInput
	style={textInputStyle}
	placeholder={"This is a test placeholder which doesn't ellipzise with trailing dots"}
	placeholderTextColor="gray"
	value=""
/>

Is there any progress？ I met the same problem.

maybe it's strange, but use
flex: 1
and placeholder will in one line

I am also still seeing this issue on 0.63.4

I created a PR for this here: https://github.com/facebook/react-native/pull/29664

Is there any progress？ I met the same problem.

Still the problem here...

Current workaround I can find for now is add maxHeight: <fontsize + padding> to style prop.

I got similar problem, up vote for a new prop ellipsizeMode

Same issue here

Still the problem here too...

same issue here

same issue here

any solution or workaround this? facing same issue

Is the issue resolved yet?

I am also facing same issue. Any solution?? Waiting for this issue to be fix.
@mrousavy Any update on your PR??

this issue is still there in 10/10/2023 !

same issue here

Same issue here, especially when changing the device font size

This is still happening on RN 0.73.4


Bug 1185678 - Create Parse Context object for L20n Parsers

https://bugzilla.mozilla.org/show_bug.cgi?id=1185678

@Pike f?

Not a whole to add, besides the nit on state exposed on the returned AST.Node children. Which would weirdly jump if you parse with and without pos?

@stasm f?

Has this landed in https://bugzilla.mozilla.org/show_bug.cgi?id=1185678 ?

yup


Function-like preprocessor macro .DefTok includes the opening '('.

Also, it's a IDENTIFIER_LPAREN instead of IDENTIFIER as in the case of object-like macros.
@xlab The fix may/probably will break cgogen, sorry, but it's a bug and have to be fixed.

Function-like macros are not handled by cgogen yet, but thanks for the alert.


Fix issue with PhoneNumber regexp - start and end  of line (^ and $)

Tests to demonstrate issue #79
WIP

This corresponds to the behaviour of all other validators.


Add Orion Multi-tenancy Information

Add information about Orion multi-tenancy. Link to Besu docs.

Closed duplicate.


Publication strategy

Options
There are three research outputs involved:

The paper
The data
The shinyapp

I have looked online for advice but what I have found has just confirmed the options available rather that help me decide which one to go for. (eg https://community.rstudio.com/t/can-i-publish-a-shiny-app-in-a-scientific-journal-how-where/6306/5)The possible options I can see are:

Publish everything as one DOI.
Publish the paper & app under one DOI and the data in ORDA.
Publish all 3 under separate DOIs (the app and dataset in ORDA). However, this seems like a duplication as it would result in data being published as part of both the app and on it’s own although once the data is published I could probably set the app up to source the data from ORDA)
Publish the paper separately and the app and data as one DOI but tweak the app to allow for downloading the data through it.

Strategy Decision

Code & Data are already published in ddm_sims repository and already have a Zenodo DOI
To archive app, we'll submit to ORDA when finalised (see issue #15).

Reserved DOI 10.15131/shef.data.8109161 (not published yet) at https://figshare.shef.ac.uk/s/11f65856db28308644a4

Regarding the Categories and keywords I've added to the ORDA entry, happy with those @tomstafford ?

How do I see these please?
On Wed, 15 May 2019 at 15:27, Anna Krystalli<EMAIL_ADDRESS>wrote:

Regarding the Categories and keywords I've added to the ORDA entry, happy
with those @tomstafford https://github.com/tomstafford ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/annakrystalli/xspl_power_analyser/issues/14?email_source=notifications&email_token=AAMS7OFR2KSCQAVZQ4OZMADPVQMVDA5CNFSM4HMCQ6R2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVO2YPY#issuecomment-492678207,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMS7ODVQSELECZ3ZSJ7BN3PVQMVDANCNFSM4HMCQ6RQ
.

If you click on the link above 👆, the categories and keywords should be on the right-hand panel.

Here it is again: Reserved DOI 10.15131/shef.data.8109161 (not published yet) at https://figshare.shef.ac.uk/s/11f65856db28308644a4

I see it now, apologies
Let's change "power analyses" to "power analysis" since that is more common
And add "drift diffusion model", "speed-accuracy trade-off", "reaction times" as keywords please


Advanced debugging support: time travel debugging

Is there an etc for this?
This would be really useful, especially if it can be leveraged inside the browser as well, to create a nice UX around it.


[Meta] Issue 23+ Client Support

Update as of August 2018
Current consensus is that developing a custom client may actually prove to be faster, and a better use of resources than reverse-engineering the I23+ client. Preliminary work has begun on AuthExerciser (#237), MapViewer (#238), and Client Gfx Patch (#450) which lay a foundation for a custom client. Additional work on a QT based GUI resources should be followed here: Segs/Segs-Data#6
The below is left here for posterity, and to provide a sort of loose roadmap for feature parity.
Summary
This is a meta issue, for proposed Issue 23+ Client support. As the term "issue" is vague and may refer to either a GitHub issue, or a City of Heroes Client Release Milestone, I'll standardize the usage here by referencing all CoX Client release milestones as such: I## or I24 or I7 or by referring to them as client patches
The Goal here is to provide information and resources should a desire rise to support later clients, or should a developer come to the table interested in forking current development to a later client patch, such as I23+
This meta issue should identify and reference other issues or concerns that may prevent, or create hurdles for development towards I23+ support. Additionally, this should support general discussion revolving around late-game client support viability.
When submitting new issues related to this meta, please tag with the (proposed) Issue23 label for sorting.
Known release-blocker Issues:

[ ] issues to go here

Issues needing more research:

[ ] Blowfish encryption likely changed every time the client was patched, which means that packet calls would have to be rewritten to work with the new client.
[ ] Is there a way to configure the server for a specific CoX patch period? Something like server-issue: 11
[ ] The server should check client version and output a message for versions incompatible with the server issue
[ ] Should the scripting system be able to target a specific release version?
[ ] Target: CoH release (I0)
[ ] Target: CoV release (I6)
[ ] Target: Going Rogue release (I18)
[ ] Target: Sunset Client (I24beta)

Additional Game Subsystems that will need development

[x] Tailor System (this may be handled mostly client side)
[ ] Exemplaring & Sidekicking
[ ] Respec'ing
[ ] Personal Badges, Exploration Badges, Accolades, and Supergroup Badges
[ ] PvP
[ ] City Info Terminals & Server/Player Stats
[ ] Enhancement Diversification
[ ] Arenas and Arena Combat
[ ] Villian Hero Type, Classes, and Powersets
[ ] Mission Architect
[ ] Wentworth's Auction House
[ ] SuperGroup Bases
[ ] Rogue/Vigilante Hero Types
[ ] Holiday Events (Halloween, Winter, etc.)
[ ] Ski Chalet ski challenge
[ ] Ouroboros (Flashback System)
[ ] Salvage and Invention System
[ ] Incarnate System
[ ] Day Jobs

Closing this, as SEGS Engine will support I24 assets at runtime!


Customize columns in WebUI's history pane

Hi,
After many, many workflows runs have populated the WebUI's history pane, where some flows have failed, and some succeeded, it get kinda hard to tell the different runs apart.
It would be nice if it was possible to customize the number of columns visible in the view, and what data these columns should present.
That means that we e.g. could chose to show the unique workflow execution ID in the table, and e.g. in our specific use cases it would be nice to be able to show one of our input parameters to the workflow (which is, internally for us, a unique identifier for each flow, as it represents data coming from an external system).
/Johan

Hi Johan,
As for number of columns, you can already do that using drop down menu behind eye icon in the top right corner of the list view.

It's a little bit more complicated with adding new customizable columns. I'm personally afraid it might clutter already complex enough view and turn quite a lot of effort into too little value at the end.
We have plans to redesign the list view and make it look more like in rules. We're also investigating into more sophisticated solution for searching and filtering executions and planning to support our Trace API.
Another 'cheap' solution for "be able to show one of our input parameters to the workflow" would be to sort param line by position property making your unique ID always appear first. If you think it might help you, we should probably create another issue specifically for this change.

Sorry for the late reply.
Yes, I know about the possibility to enable/disable the columns in your screenshot. But it is missing some kind of column for me to identify the runs from each other.
We've started to implement the Trace features in our workflows, so to be able to search and filter executions and runs on those would help a lot.
Not exactly sure what you mean by "sort param line by position property making your unique ID always appear first" though.

Hi Johan,
we were on the path of adding search on Trace-tags to execution list, but it didn't make it to the upcoming v1 release - sparing details, we are thinking of a special trace-based views, and return to designing once the current release behind. Your voice counts for search on trace tag, we may reconsider.
What are the other columns / data you are missing on the list? Is it an ability to search for them, or display in the list, or both?

We have started using trace tags for our workflows, that way we can group together various executions for logging, review and debugging purposes.
Usually a set of raw data from the sequencing machines is associated with one unique tag name. This raw data might be re-processed a couple of times by the same workflow before we deem the end result to be satisfactory (after which the output data from the workflow can be delived to the customers/researchers).
At the moment we have a small script (https://github.com/arteria-project/arteria-packs/blob/master/scripts/trace_runfolder.py) that could be run with e.g. python scripts/trace_runfolder.py --tag <tag_name> | xargs -n1 st2 execution get which helps us find the relevant traces for a specific tag, and then the associated executions.
Being able to view/filter/search executions based on these things in the WebUI would greatly help.


Created event interface

#6

Created an interface for a general event
Implemented classes for specific Event types: Instant Event + Counter Event

@jessieAnhNguyen As you can see in the PR, there are a lot of automated checks that have failed because tsdx helpfully installed a GitHub Action that runs a linter across our codebase, and detected that your code is not following the code style conventions.
Instead of fixing this manually, you should run Prettier on your code locally, and that will fix all the issues for you.
If you use VSCode, you should install the Prettier for VSCode extension, and turning of Format on Save in VSCode Preferences > Settings.
If you don't use VSCode, it's probably supported in your preferred editor too.


Re-organize interface (grouping method)

Coverage remained the same at 70.43% when pulling 164518cfa840e96c36cec53c4b82110146d72468 on organize into 5207f8d9f6a2dc1728dd591701714a1133456dd3 on master.


I need mixed file-finder & buffer-finder, like Sublime Text

Just remember one shortcut, I can search files with keywords or select recent(sorted) opened files. just like Sublime Text's Command + P.
That's why every time I try to use the Atom, I go back to using Sublime Text.

vs-code has the same thing. If you have no text in the search input it shows recently opened files.

@scottwio thx for reply.

If you have no text in the search input it shows recently opened files.

In my environment, Atom also shows recently opened files, but after a few seconds("reindexing" text at top gone), it only keeps last one recent file. maybe it's reordered?

Thanks for reaching out!
We noticed that this is a duplicate of https://github.com/atom/fuzzy-finder/issues/214. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


DataTable should re-render when cell content updates

In #740 we made it so that rows would re-render when one was added/deleted. We should also support the case of cell content updates triggering a state change as well for the rows. This issue is for adding a check to verify that cell content has been updated (or not) and updating state accordingly.

Hi @joshblack we're stuck on an old release until this is fixed :( so wondering if I can help at all? In componentWillReceiveProps() couldn't we just compare the entire rows, e.g.:
if (!isEqual(this.props.rows, nextProps.rows)) {
  this.setState(state => getDerivedStateFromProps(nextProps, state));
  return;
}

Definitely could @asfordmatt! Would be the fastest solution in the interim, although would be worried about deep equality checks given that this component will want to support pagination in the future. Might be a premature optimization though 🙂

@joshblack please see the referenced PR. I've updated the storybook to add a button that increases the port number in the sample row data. I've also added a test for the same.
Do you think it's worth keeping the earlier test that just compares the row IDs? Or just leave it to the full compare to catch everything?


SHould the Tag Manager code be into two templatetags?

It looks like Google no recomend the Tag Manager code is split with one part in the head and one in the body https://developers.google.com/tag-manager/quickstart there is a write up of the changes here http://dmitriilin.com/where-to-place-google-tag-manager-code-v-2/
Do you have any intention or interest in including two templatetags, one for the body and one for the head? This could be a backwardly compatible change by creating two new templatetags, leaving the old one two output both tags.

Sure, I'm not currently using this code, but I'll update that. Or do you have any proposed code for this?

Someone already did this: I opened a PR for myself to see if there's anything else that needs to be done:
https://github.com/Lacrymology/django-google-tag-manager/pull/4

Wow - I was going to offer to do this. I will take a look at the PR

Fixed with #5 and pushed v0.0.3 to PyPi

Thanks @Lacrymology and @olegserov


Suppress -Wimplicit-fallthrough warnings

Signed-off-by: Alf-Rune Siqveland<EMAIL_ADDRESS>

Well, I guess this isn't happening...


bug: react animations CreateAnimation component not working and wrongly documented

Bug Report
Ionic version:
[x] 4.x
Current behavior:
CrateAnimation tag expects only one element in fromTo attribute
Expected behavior:
fromTo?: PropertyValue; is expected to be fromTo?: PropertyValue[]; in CreateAnimationProps based on the documentation
Steps to reproduce:
follow this documentation example found in https://ionicframework.com/docs/utilities/animations:
Related code:
<CreateAnimation
  duration={1500}
  iterations={Infinity}
  fromTo={[
    { property: 'transform', fromValue: 'translateX(0px)', toValue: 'translateX(100px)' },
    { property: 'opacity', fromValue: '1', toValue: '0.2' }
  ]}
>
  ...
</CreateAnimation>

Other information:
Also this code didn't work, I was expecting a simple animation and nothing happened when the component loaded
<CreateAnimation
  duration={1000}
  fromTo={{
    property: 'opacity',
    fromValue: '1',
	toValue: '0'
  }}
>

example animation

</CreateAnimation>

Ionic info:

Ionic:

   Ionic CLI       : 6.2.0 (/usr/local/lib/node_modules/@ionic/cli)
   Ionic Framework : @ionic/react 5.0.5

Capacitor:

   Capacitor CLI   : 1.5.1
   @capacitor/core : 1.5.1

Utility:

   cordova-res : not installed
   native-run  : 0.3.0

System:

   NodeJS : v10.15.3 (/usr/local/bin/node)
   npm    : 6.4.1
   OS     : macOS Mojave

Thanks for the issue. The first issue just looks like a type error on our end. You can work around it for now by typecasting your fromTo array as any.
The second issue is not a bug. Since Ionic Animations are programmatically controlled you need to explicitly set play={true} on CreateAnimation otherwise the animation will remain paused.

Thanks @liamdebeasi !
I'll just wait for an update on the first issue, and please also check the import line in the examples
import { createAnimation, Animation } from '@ionic/react';
should be
import {CreateAnimation } from '@ionic/react';
About the second one, thank you very much, my bad, didn't find that on the docs. Now it's working :)
Thank you!

Thanks! I will update the docs so the play requirement is clearer (also will fix the import).
For the first issue, here is a dev build you can use for now npm i<EMAIL_ADDRESS>

You're my today's favorite person. Thanks again

Thanks for the issue! This has been resolved via https://github.com/ionic-team/ionic/pull/20775 and will be available in an upcoming release of Ionic Framework.


Feature | Added dataUrl for Response

Hey, @guanguans thank you for submitting this PR - I think this is a cool addition to Saloon! I'm going to have a quick look over the PR but I think the implementation is solid.


patch: Change NPM build workers to Ubuntu

Getting errors with alpine during CI build, hence changing workers OS to Ubuntu
Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

@balena-ci I self-certify!


OAuth2.0 - client_credentials error: The request body must contain the following parameter: 'grant_type'

When I use OAuth2 with client_credentials, I get the following error:
{
  "error": "invalid_request",
  "error_description": "AADSTS900144: The request body must contain the following parameter: 'grant_type'. Trace ID: 325b873b-08a3-4b23-b002-650f1f00f700 Correlation ID: 8d8f1825-b12e-468b-8c67-f686e9b46383 Timestamp: 2024-03-27 09:22:23Z",
  "error_codes": [
    900144
  ],
  "timestamp": "2024-03-27 09:22:23Z",
  "trace_id": "325b873b-08a3-4b23-b002-650f1f00f700",
  "correlation_id": "8d8f1825-b12e-468b-8c67-f686e9b46383",
  "error_uri": "https://login.microsoftonline.com/error?code=900144"
}

This is replicated if I click on "Get Access Token" or when sending the request.
Same details work absolutely fine with postman.

Also testing against Microsoft Azure AD / Entra Id and get a similar error with version 1.12.3 of Bruno
AADSTS900144: The request body must contain the following parameter: 'client_id'.
Likely caused by Bruno sending the request body payload for client_credentials with content-type application/json.
RFC6749 - The OAuth 2.0 Authorization Framework expects the content type to be application/x-www-form-urlencoded  ref https://datatracker.ietf.org/doc/html/rfc6749#section-4.4.2
MS Azure AD/Entra ID follows the standard only and expects application/x-www-form-urlencoded as content-type
ref https://learn.microsoft.com/en-us/entra/identity-platform/v2-oauth2-client-creds-grant-flow#first-case-access-token-request-with-a-shared-secret
Likely needs adjustment in https://github.com/usebruno/bruno/blob/1349a7975082e2afa64f4cc3b1c2d0d884991b4b/packages/bruno-electron/src/ipc/network/index.js#L216

Also testing against Microsoft Azure AD / Entra Id and get a similar error with version 1.12.3 of Bruno AADSTS900144: The request body must contain the following parameter: 'client_id'.
Likely caused by Bruno sending the request body payload for client_credentials with content-type application/json.
RFC6749 - The OAuth 2.0 Authorization Framework expects the content type to be application/x-www-form-urlencoded ref https://datatracker.ietf.org/doc/html/rfc6749#section-4.4.2
MS Azure AD/Entra ID follows the standard only and expects application/x-www-form-urlencoded as content-type ref https://learn.microsoft.com/en-us/entra/identity-platform/v2-oauth2-client-creds-grant-flow#first-case-access-token-request-with-a-shared-secret
Likely needs adjustment in
https://github.com/usebruno/bruno/blob/1349a7975082e2afa64f4cc3b1c2d0d884991b4b/packages/bruno-electron/src/ipc/network/index.js#L216

Yes, this seems to be the issue. I tried replicating these scenarios directly. Can +1 this.

This will go out in the 1.13.0 release. Please subscribe to Discord announcement channel to stay notified on the releases.

Using the 1.13.0 version, it still seems to be giving an error. Even though I click the "Get Access Token" button, still the Timeline shows the data being sent to the Request URL (Not the Access Token URL) as json, but with the client ID and secret instead, which should be sent as a GET to the Access token URL and then the token generated should be sent to the Request URL instead of what is seemingly happening.

This has been added. Closing the ticket


Restoring CC3 backup fails with error

Hi @alexreinert,
I'm migrating my pivccu from a Pi4 to a Pi5 (fresh install).
I now tried to export a backup from the old CCU3 and import it in the new CCU3 and get the following error:

Do you have any idea what might be wrong?
Thanks,
Florian

The radio module is needed when restoring the backup, but it is not found.
Please provide the output of sudo pivccu-info on the Pi 5 before restoring the backup.

Thanks, using the hardware resolved the issue


Node18 support

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #115
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[ ] Revision of an existing capability
[x] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 3.15.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


OOM not detected with scan_pessimistic_batches_for_oom

Hi,
I tried the recipe of Librispeech "pruned_transducer_stateless7_streaming" on my own dataset which was working fine with Kaldi.
But with Icefall/K2 I got an OOM problem with --max-duration = 750 , 600  and was not detected with  "scan_pessimistic_batches_for_oom".
I filter out utterances longer than 50 seconds with "./pruned_transducer_stateless7_streaming/train.py" instead of 20 used in Librispeech because in my dataset statistics I see than 20 s is too small for me.
So my questions are :

is it a good strategy to keep decreasing  --max-duration until there is no OOM or it will decrease accuracy in some way ?
it is OK to have utterances way longer to 20 seconds or it will always get difficult to handle and it better to find a way to split them ?
can you give some details on the error message because it did not understand the pytorch documentation on "max_split_size_mb"  and  "PYTORCH_CUDA_ALLOC_CONF" ? Do you  have suggestions on how to set these ?

Thanks for your help. Here is the log :
2023-02-13 16:43:08,625 INFO [train.py:1198] features shape: torch.Size([12, 4688, 80])
2023-02-13 16:43:08,627 INFO [train.py:1202] num tokens: 3635
Traceback (most recent call last):
File "/home/usera/git/icefall/egs/ge/./pruned_transducer_stateless7_streaming/train.py", line 1265, in
main()
File "/home/usera/git/icefall/egs/ge/./pruned_transducer_stateless7_streaming/train.py", line 1258, in main
run(rank=0, world_size=1, args=args)
File "/home/usera/git/icefall/egs/ge/./pruned_transducer_stateless7_streaming/train.py", line 1136, in run
train_one_epoch(
File "/home/usera/git/icefall/egs/ge/./pruned_transducer_stateless7_streaming/train.py", line 841, in train_one_epoch
scaler.scale(loss).backward()
File "/home/usera/miniconda3/envs/icefall/lib/python3.9/site-packages/torch/_tensor.py", line 487, in backward
torch.autograd.backward(
File "/home/usera/miniconda3/envs/icefall/lib/python3.9/site-packages/torch/autograd/init.py", line 197, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 10.25 GiB (GPU 0; 47.53 GiB total capacity; 9.45 GiB already allocated; 9.67 GiB free; 23.12 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

I think you could try to filter the uneven-sized minibatch, as we did in https://github.com/k2-fsa/icefall/pull/843.

If you’re using DynamicBucketingSampler try setting quadratic_duration to sth between 30 and 50. You will need to tune max_duration later (typically I see it can be increased about +50%). You have extremely long utterances in training and that should help balance out the GPU memory usage across mini batches of different cut durations.
https://github.com/lhotse-speech/lhotse/blob/527d96453b9ad119729ddef1a9613bcae536e6c7/lhotse/dataset/sampling/dynamic_bucketing.py#L75

BTW,  this quadratic_duration thing is a very new feature in Lhotse.
The uneven-sized minibatch problem turned out to only be an issue in quite old versions of Lhotse (zengwei: we could actually remove this extra code, should be unnecessary in general.)
The batch length seems to be 12 * 4688 / 100, about 555 seconds.
HOWEVER, this:
CUDA out of memory. Tried to allocate 10.25 GiB (GPU 0; 47.53 GiB total capacity; 9.45 GiB already allocated; 9.67 GiB free; 23.12 GiB  reserved in total by PyTorch
makes me think that the issue may be sharing the GPU with another process, maybe from a different training run, since 10.25 plus the 9.45 GiB already allocated is much less than the 47.53GiB total.
Unlikely to be a fragmentation issue since even the "23.12 GiB reserved in total by PyTorch" plus the 10.25GiB request is much less than the 475.53GiB total capacity.
Use nvidia-smi to see what's using the GPU

OK, sorry for late reply.
Yes I trained with DynamicBucketingSampler.
Indeed an other process (not mine) was sharing the GPU.
Do you suggest that the GPU memory used by k2 during training will be always less that the one used for the first batch and that the problem comes from the other process which grew in memory ?

Oom search is not a guarantee.
On Wed, Feb 22, 2023, 7:24 PM iggygeek @.***> wrote:

Closed #910 https://github.com/k2-fsa/icefall/issues/910 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/icefall/issues/910#event-8579977560, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO2QLN2GFOPTXNKPCP3WYXZNVANCNFSM6AAAAAAU4S7L7A
.
You are receiving this because you commented.Message ID:
@.***>


Use go modules, remove vendor, update code & fix SSH

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Updated to latest go and go modules

@kubernetes/kube-deploy-reviewers

Self-merging to support code-health.


Suggestion: Documentation in Markdown instead of Word docs

I suggest to use Markdown instead of Word for all documentations and instructions. Word/Binary is really not great in Git.
Best wishes & stay healthy!
Christian

Thank you! Great suggestion! We are actually looking at moving the documentation and instructions to docs.microsoft.com soon. I'll close this comment for now but will update here with a link once we've moved to docs!

That's great! I think I also have a good solution for the issue with the basic auth. How can I contribute?

We have just updated the repo with a new solution to work with MFA: https://github.com/microsoft/powerapps-tools/tree/master/Administration/CoEStarterKit/Audit Log (MFA)
Would be great to get your feedback on this!

Hey Manuela,
it works - good job! :)
Suggestion for an security improvement. For the current CoE setup you only need the following permissions App Permissions from Management API:
ActivityFeed.Read (delegated)

Best wishes
Christian


Add cli option to run launch script

It would be nice if the code cli could accept an argument to run a launch script.  For example...
code --launch "Launch Program"
...where "Launch Program" is a pre-configured command in launch.json.
A one-line command like code --launch "Launch Server" ./api-server &&  code --launch "Launch Client" ./api-client" could then be used to orchestrate dev environments for microservice-based projects.

/findDuplicates

Dup of https://github.com/Microsoft/vscode/issues/10979


feat(Accordion): Create Accordion Component

TODO

[x]  Create a Collapsible Component
[x]  Create a Collection Component
[x]  Create an Accordion Component
[x]  Make it accessible / performant
[x]  Test on web/mobile platforms
[ ]  LGTM

closes #604

Fine to have collapsible as well here.

Unfortunately we run into the same problem as a few other components currently in progress, where native doesn't have an idea of querying the DOM to find the order.
What we could do.... is maybe have a wrapper that uses onLayout or something, calls back through context to the parent with it's reference and dimensions.
Then the parent can listen for all the dimensions, then compare them based on flexDirection to determine their order.

Unfortunately we run into the same problem as a few other components currently in progress, where native doesn't have an idea of querying the DOM to find the order.
What we could do.... is maybe have a wrapper that uses onLayout or something, calls back through context to the parent with it's reference and dimensions.
Then the parent can listen for all the dimensions, then compare them based on flexDirection to determine their order.

do you want to elaborate more on this, actually, I didn't get what's the issue? why do we need to know the order of the elements?

@TheEhsanSarshar not sure actually but it's what Collection is doing with the querySelector parts. Maybe it's only for accessibility and we can leave it out on native.

@natew I guess you're right it's only for Accessibility, at least with Accordion

Is there anything I can do to help move this along? Would love to be able to use a Tamagui Accordion component

Is there anything I can do to help move this along? Would love to be able to use a Tamagui Accordion component

Today, I'll try to finish the remaining parts.
In meantime if you test it on native devices that would be great 😊

I've been developing with the code from this PR copied into my local repo and it's been working beautifully in both iOS and Android emulators

@josephto nice! That's a great endorsement. @TheEhsanSarshar want me to fix up merge conflicts, is this ready for re-review?

@natew yeah please do that

It's ready for review

@TheEhsanSarshar could you add a docs page for it? Looking at it now.

Ok playing with it, lets get the demo to be closer to radix:

Lets show animations (let me know if there's troubles there)
Can we show the chevron/arrow indicator like the do
The "focus" border on the headers when you click them

I made a few fixes on my branch TheEhsanSarshar-create-accordion-component you can merge back in here.

Also just saw this in Framer Motion:
https://www.framer.com/motion/layout-animations/
Notice LayoutGroup. Not sure if necessary here but seems relevant for Accordions. We don't have layout animations ofc but maybe still applies? Not sure.

thanks @natew for your review.
sure I'll check all those things that you said.

My only concern about animation part is that it should work with all the drivers that we have.
or maybe for layout animations we might need to have some sort of a reusable mechanism that we might later need for other components as well.

Really looking forward to this component. Any estimate for when it's ready? 😁

because we have some layout animation ( height animation when the accordion expands or collapses). we aren't super confident to merge this PR because of some API changes that might happen later.
right now I'm working on a tiny layout animation library, hopefully, I can finalize it soon.
other parts of the Accordion are ready, if it's possible you can use this branch 🙂
till we finalize that part

I would only implement animations in the accordion for web-use components. Still, for the native devices, I would remove the styling feature because React Native does not truly support animations unless you use a third-party library. I honestly believe that this will become too problematic to maintain. I understand that for user experience, it's a nice to have feature, but it's not a critical one.

Hi, I'm a bit confused, so technically Accordion can be used (via PR's code), but it will execute without collapsible animations? is that true? thanks @natew @TheEhsanSarshar and the rest of the team and contributors for making cross-platform styling easier.

@ammarfaris we'll try to merge that this week


squeryl-record doesn't honor StringFields as objects

squeryl-record doesn't work with StringFields created as so:
object f extends StringField(this, 36)
instead requiring:
val f = new StringField(this, 36)

This issue / PR is being closed because the Lift community has decided to decomission the Lift Persistence group of libraries including Mapper, Record, et al. We encourage folks to check out using some alternatives for their persistence needs. In particular the following libraries may be of interest:

ProtoQuill
slick
scalasql

There are likely others, as well! Please feel free to reach out on the Lift Mailing List if you have any questions or concerns.


Adress issues raised by staticcheck and add it to workflow

Staticcheck is a wonderful tool for performing static analysis of Go programs. This PR addresses all the issues (3 to be exact) that it mentioned and also adds it to the workflow for further checks in the future. I also updated the checkout action to v2 which most importantly has some performance work :)

I also added gosec for finding security issues in the code (there are none currently fyi) :)

Love this contribution. Thank you. Just a couple comments before we merge


Foot routing doesn't use crossing and ignores foot=no

https://graphhopper.com/maps/?point=54.325611%2C10.115705&point=54.32631%2C10.114567&locale=de&vehicle=foot&weighting=fastest&elevation=true&use_miles=false&layer=Omniscale
The routing algorithm doesn't consider the crossings at the sides of the intersection. The middle of the intersection is even tagged as "foot=no" but this seems to be ignored.
Correct routing would be like this:
https://graphhopper.com/maps/?point=54.325611%2C10.115705&point=54.326004%2C10.115259&point=54.32631%2C10.114567&locale=de&vehicle=foot&weighting=fastest&elevation=true&use_miles=false&layer=Omniscale

https://graphhopper.com/maps/?point=54.325597%2C10.115694&point=54.326263%2C10.114616&locale=de&vehicle=foot&weighting=fastest&elevation=true&use_miles=false&layer=Omniscale
Well.... at least it doesn't route over the foot=no anymore. But I don't understand why it doesn't route here at all now.

Wörthstraße is not connected to the paths.

OK, so this was the problem the whole time. Sorry.


Psalter support?

Are these issues auto-fixable?
Atm I get:

Psalm doesn't know how to fix issue(s): NotStrictEqualSign

Yeah they are, but Psalm does not allow plugin to register fixable issues like that.
Technically, as long as you are in psalter mode, this plugin should start fixing issue, so if you try to fix another issue, this will get fixed along the way. Otherwise, if you only want to fix this issue, the command should look like vendor/bin/psalm --alter --plugin=vendor/orklah/psalm-strict-equality/src/Plugin.php
This should be improved in Psalm itself but I couldn't find time to do that


Improve Resources CARD UI

I plan to use react glassmorphism to make cards for resources section and add animations to it for better UI

@Abhirup-02 please have a look

Not needed


Generate documentation when building so that it is included in nuget package

Generate documentation xml during build so that it can be included in nuget package so that intellisense works better for developers.

@andreas-eriksson Thank you!

Included in version https://github.com/Jinjinov/Hardware.Info/releases/tag/v<IP_ADDRESS>


Fix {class,static}method-decorators for dotted names

Summary
This brings ruff's behavior in line with what pep8-naming already does and thus closes #8397.
I had initially implemented this to look at the last segment of a dotted path only when the entry in the *-decorators setting started with a ., but in the end I thought it's better to remain consistent w/ pep8-naming and doing a match against the last segment of the decorator name in any case.
If you prefer to diverge from this in favor of less ambiguity in the configuration let me know and I'll change it so you would need to put e.g. .expression in the classmethod-decorators list.
Test Plan
Tested against the file in the issue linked below, plus the new testcase added in this PR.

I went back and forth on this a bit but ultimately I think it makes sense in the interest of pragmatism and matching user expectation.


addition of snake game

@Priyanshi662 i would like to add the snake game. plz assign it to me under jwoc'24

Mam, I have read the issue and I would like to work on this issue.
Please assign it to me under JWOC'2024.

@vishal2005025 assigned.

@Priyanshi662 i have already given a pr of this issue and its merged as well many days ago. u can check.

@Priyanshi662 this was my pr from 5 days ago


substitution for extras should ignore unknowns

When passing a string, e.g. a shell command to be executed, as extra argument when using filter=formatter(), the string substitution replaces unknown keywords with {} instead of leaving them untouched.
Example:
test = 'echo {dynamic_message} > {some_file}'
with any task like transform(... filter=formatter()..., extras=[test]) results in echo {} > {} after substitution.
Suggestion:
Ruffus should ignore unknown keywords during string substitution.
Peter

Surprised this does not just blow up!
I think there is some over cautious Exception handling here.
Is it not just easier to use double {{}} to prevent substitution?
As in regular python:
>>> "{{test}}".format()
'{test}'

I can ignore unknown keywords but that probably would require catching the keyerror exception in a loop and progressively either dynamically adding the double braces or faking the missing keywords
>>> "{{test}}".format(test="{test}")
'{test}'

Either way this seems a lot of overhead to add to a process which is just going to hide missing replacements.
I would prefer to make sure that an Exception is thrown.
Let me know what you think.

You are right, I hadn't thought about the {{}} option. This might indeed be easier, I'll check it tomorrow in my code and get back to you.
I am still a bit fuzzy about all the string replacements in Ruffus, so I am probably off the right track here, but I was just thinking in the direction of using the __missing__ hook in an object sub-classed from dictionary and then return "{" + key + "}" in these cases. But again, no clue if this is at all applicable to Ruffus.

You mean the equivalent of:
>>> class dict_ignoring_missing(dict):
...      def __missing__(self, key):
...          return '{' + key + '}'
...
>>> import string
>>> string.Formatter().vformat("{found} {missing}", args = [], kwargs=dict_ignoring_missing(found=True))
'True {missing}'

Does anyone still use vformat()? :-)
I have a feeling that this would be even more surprising to python users!
At the moment I am inclined to make sure that Ruffus throws a hissy fit (Exception) when there are missing keys, though hopefully with a clear Error message. That is always the challenge.

Well, it probably depends on prior expectations: Ruffus makes some assumptions (input files turned into output files) and for these assumptions to hold, there should be an error when the string replacement fails, e.g., due to missing keys, for inputs/outputs.
But what are the assumptions concerning the extras=? In my understanding, there aren't any - Ruffus replaces in a best effort principle what can be replaced, otherwise it just passes the arguments to the function. Consequently, it should not change the user's arguments.
However, right now, I think it all comes down to a "design decision" that needs to be properly documented and that's about it (so, maybe, this is not a bug, but a suggested enhancement for the documentation?). As you suggested, the built-in escaping mechanism {{}} to avoid changes by Ruffus' string replacement actions is likely the way to go, because... well, it's a built-in feature ;-)

Yes. These sorts of calls are the trickiest part of Ruffus. I certainly
regret some of the (now obviously excessive) syntactic flexibility I
allowed earlier in the design but that will always be retained for
backwards compatibility.
Leo
On 24 March 2015 at 08:17, Peter Ebert<EMAIL_ADDRESS>wrote:

Well, it probably depends on prior expectations: Ruffus makes some
assumptions (input files turned into output files) and for these
assumptions to hold, there should be an error when the string replacement
fails, e.g., due to missing keys, for inputs/outputs.
But what are the assumptions concerning the extras=? In my understanding,
there aren't any - Ruffus replaces in a best effort principle what can be
replaced, otherwise it just passes the arguments to the function.
Consequently, it should not change the user's arguments.
However, right now, I think it all comes down to a "design decision" that
needs to be properly documented and that's about it (so, maybe, this is not
a bug, but a suggested enhancement for the documentation?). As you
suggested, the built-in escaping mechanism {{}} to avoid changes by
Ruffus' string replacement actions is likely the way to go, because...
well, it's a built-in feature ;-)
—
Reply to this email directly or view it on GitHub
https://github.com/bunbun/ruffus/issues/45#issuecomment-85392264.

A failed substitution now fails the entire job. With verbose>=4, will result in an error message like:
Job Warning: Input substitution failed:
          Missing key = {dynamic_message} in 'echo {dynamic_message} > {some_file}'.
                input =  '',
               filter = formatter(['test_transform_formatter/a.tmp'])..


indentation

indented the stuff yeow

Hi Brady,
Some possible issues for your consideration:

Can you clean up the whitespace in the  tag's in lines 85-91 (after the "by")
There are a few places where you've indented same level elements making them look nested (they're not) i.e  tag with an indented  tag on line 22-23
You have also indented the  tag after the  tag (lines 41-43) though it isn't nested inside it and doesn't need to be indented. There are many occurrences of this so if you decide to make these changes, please make it consistent throughout file
Cheers,
Lisa


Vertical anchoring of Popover broken

Problem description
If you set the vertical value of the anchorOrigin prop for the Popover component to anything other than 0 or center, you do not get the desired result. I believe this line is the culprit:
https://github.com/callemall/material-ui/blob/1a9b612c02cd606a3ae31c8a241505a2e8e67264/src/Popover/Popover.js#L306
Looks like the expressions of the ternary operator are reversed.
Steps to reproduce
Create a Popover or other component that uses a Popover (such as a Menu) and set a anchorOrigin: vertical value of something other than 'center' or 0.
<Popover anchorOrigin={{vertical: 'bottom'}} ... />
Versions

Material-UI: v1.0.0-beta.6
React: 15.6.1
Browser: Any

What's the desired result? The behavior seems ok to me:

Is that from v1.0.0-beta?

That is what I'd expect. I'm not seeing that behavior from v1.0.0 however.

This demo is coming from #7927. The live documentation hasn't been updated yet.

I don't know what that demo is supposed to be showing, but the following code does not work with v1.0.0-beta.6:
import React, { Component } from 'react';
import Button from 'material-ui/Button';
import Menu, { MenuItem } from 'material-ui/Menu';

class SimpleMenu extends Component {
  state = {
    anchorEl: undefined,
    open: false,
  };

  handleClick = event => {
    this.setState({ open: true, anchorEl: event.currentTarget });
  };

  handleRequestClose = () => {
    this.setState({ open: false });
  };

  render() {
    return (
      <div>
        <Button
          aria-owns={this.state.open ? 'simple-menu' : null}
          aria-haspopup="true"
          onClick={this.handleClick}
        >
          Open Menu
        </Button>
        <Menu
          id="simple-menu"
          anchorEl={this.state.anchorEl}
          open={this.state.open}
          onRequestClose={this.handleRequestClose}
          anchorOrigin={{vertical: 'bottom', horizontal: 'right'}}
        >
          <MenuItem onClick={this.handleRequestClose}>Profile</MenuItem>
          <MenuItem onClick={this.handleRequestClose}>My account</MenuItem>
          <MenuItem onClick={this.handleRequestClose}>Logout</MenuItem>
        </Menu>
      </div>
    );
  }
}

export default SimpleMenu;

That code is copied verbatim from the current v1 docs, with the addition of the anchorOrigin property. When I run it, the menu correctly drops down from the right side of the button, but not from the bottom.

Perhaps it is a bug with the Menu component instead, but when I look at the code for Menu, it passes the props onto Popover, and I am 99% sure the line of code I referred to above is the problem...

Could you provide a reproduction test case? That would help a lot 👷 .
You can use codesandbox.io for instance.

By bad, you have provided one.

The observed result is the expected one. However, I think that we should be adding a warning when the anchorOrigin.vertical is set at the same time of the getContentAnchorEl property. We can't have both at the same time.

Add getContentAnchorEl={null} and you should be good.

Is this going to work for the Menu component? Looking at the source, it appears that Menu sets the getContentAnchorEl prop of the Popover directly. It does not get passed through.

I haven't tried, but the source code makes me think that users can override the getContentAnchorEl property provided by the Menu to the Popover.

👍  You're right. I see now that if you set the getContentAnchorEl prop on Menu, it will override the one the Menu sets on Popover because of {...other}. Tested it out and it worka here. Probably should add something to the documentation for Menu.
Optionally, you could just set getContentAnchorEl to null from Menu if an anchorOrigin prop is passed to Menu with a value for vertical. It can be easily done from the getContentAnchorEl function already present in Menu.

I was really confused about why I was getting that error just from adding anchorOrigin to a Menu. It should really just ignore getContentAnchorEl if anchorOrigin is there, or at least the warning should instruct users to null out getContentAnchorEl.

@CorayThan You are right. We need to:

Add a description for getContentAnchorEl
Improve the warning error message to explain what's going on and with an actionable message.

We need to:

How about

set getContentAnchorEl to null when the caller isn't doing anything with it, and just want to provide the `anchorOrigin to the menu.

@smeijer This sounds like a good idea, do you want to submit a pull request?

I'm sorry - just to clarify - why that issue was closed? 4.0.0 alpha7 does not seem to have getContentAnchorEl={null} documented.

@dy You need to follow the props cascading. getContentAnchorEl is documented on the Popover.

We will improve the demos in #10804.

@oliviertassinari ,

@dy You need to follow the props cascading. getContentAnchorEl is documented on the Popover.
Doesn't it undermine entire concept of component encapsulating the implementation? When using component, should developers think about its deeper implementation, rather then just rely on documented component's API?

thanks

getContentAnchorEl is documented on the Popover.

That is correct, it is mentioned there. But if I am looking correctly, that page contains no explanation or warning about the necessity to override it to null when using anchorOrigin prop with a value for vertical.


Release/0.29.1

This is an intermediate release to fix some show stoppers and introduce some small improvements.

Coverage decreased (-0.07%) to 35.692% when pulling 87fee6e1cd43d246a63d022b6668dbea09e61781 on release/0.29.1 into 20abeacc904e587fac0fb4e04da518fc9fb359d2 on master.


Microsoft.Identity.Client conflict when using PnP and EXO v2 cmdlets within same script

I'm using the EXO v2 cmdlets (v 2.0.4, latest) to create and apply retention label policies to sites. There is also some requirements to use PnP and lookup some info in SharePoint first.
PnP has version <IP_ADDRESS> of Microsoft.Identity.Client.dll (in both 1.0.4. and latest nightly 1.4.14). But the Exchange cmdlets have version <IP_ADDRESS> of Microsoft.Identity.Client so I get an error 'Could not load file or assembly 'Microsoft.Identity.Client, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0a613f4dd989e8ae'. Could not find or load a specific file.'
Is there an easy way to resolve this mismatch so I can use both modules (I have limited knowledge here if there's an obvious solution), or possibility PnP will be upgraded to the later version at any point?
Any advice much appreciated!

It looks like the 2.0.3 version of ExchangeOnlineManagement doesn't use Microsoft.Identity.Client.dll. If you can do what you need with that version you could add import-module ExchangeOnlineManagement -MaximumVersion 2.0.3 at the beginning of your script to force that version to load. If the old version isn't installed on your machine you can install it with Install-Module ExchangeOnlineManagement -MaximumVersion 2.0.3.

Thanks for the reply. I did try that, but I am using these in an Azure Durable Function, running on PowerShell 7. EXO only works from 2.0.4 with PowerShell 7 as they only recently made that update!
For now I managed to work around the issue by importing the modules manually into the function via import statements, and not allow either managed dependencies or a standard Modules folder to auto load them. If I load both in manually before using either, it seems to force using the later version of Identity.Client. Only importing PnP and running cmdlets, then later trying to import EXO and connect will result in the error as the older assembly is then in memory I believe. I'm not an expert in this area by any stretch but this sort of makes logical sense (I think!).
Bit weird though as within our Dev team we had mixed results replicating this locally, some of us would get the conflict and others not (even when not having installed the modules and just importing via the psd1 manually to mimic what should be happening in Azure).

It sounds like you have a pretty good handle on it. I'm not an expert on the order PowerShell goes through when it autoloads modules, but it would make sense that it would load PnP first if your script uses that first, and EXO later. I can see how it would break if PnP autoloads first with an older version than works with EXO.

This issue can be resolved by including the Microsoft.Identity.Client assembly in the alternative AssemblyLoadContext that is  used for Application Insights.

We have updated the MSAL.NET to 4.36.1, nearly the latest one.
Can you give it a try with the latest nightly and check ?

Closing this now ! Not sure if we can do anything from our side to fix. We try to keep the MSAL in sync with Az modules but way too many modules use different versions.

@gautamdsheth This can be resolved by moving the reference to MSAL into the PnP.PowerShell.ALC project/assembly.
https://docs.microsoft.com/en-us/dotnet/core/dependency-loading/understanding-assemblyloadcontext

@gautamdsheth could we re-open this issue? As interoperability with other modules might be a good thing to have. @pschaeflein has a good suggestion btw.

@pschaeflein is correct, and was correct two years ago. The AssemblyLoadContext as implemented in PnP.PowerShell is only isolating Microsoft.ApplicationInsights, and PnP.PowerShell has a lot more dependencies (namely Microsoft.Identity.Client) that conflict and need to be isolated in the same manner.

This fix need to happen.

This amazing powershell module is just painfull to use.
Connect-PnPOnline -url $AdminSiteURL -ClientId $AppID_PnP -Te …
ERROR: |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
ERROR: | Could not load file or assembly 'PnP.PowerShell.ALC, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. Uncaught
ERROR: | exception during type initialization. (0x80131534)


usb: web USB sample fails Chapter9 USB3CV tests.

Describe the bug
Web USB sample (./samples/subsys/usb/webusb/) fails TD 9.21: LPM L1 Suspend Resume Test
To Reproduce
Build and flash ./samples/subsys/usb/webusb/ and perform USB3CV tests on the device.
Expected behavior
The test should have passed.
Logs and console output

Environment (please complete the following information):

OS: Tests are run under Windows OS as they are developed for it.
Toolchain zzephyr-sdk v0.11.3
72d5dee2e337bf98458f28c26bc7c707e87d1e55

Impact
The impact is very low as I don't think there are many customers willing to use WebUSB.
The sample deserve to be rewritten. Firsts insights: the BOS descriptor lacks in some parts
described in USB Link Power Management ECN - a part of USB 2.0 spec.

WebUSB sample does not provide relevant descriptor. I've added implementation and the test is passing. Will post asap.


Deleted

Describe the bug

Additional Files
No response
Environment
WandB version:
OS:
Python version:
Versions of relevant libraries:
Additional Context
No response

Sorry, must have spammed my keyboard, see #6870. Don't think I can permanently delete but closing.

Sorry, must have spammed my keyboard, see #6870. Don't think I can permanently delete but closing.


Create espm.txt

Add Escola Superior de Propaganda e Marketing
São Paulo, SP, Brazil
This domain is used by teachers, as described in the document:
DOC036.pdf

@thiagoandf Pull request merged. Thank you!


[BUG] The specified configuration does not have a telemetry channel. (Parameter 'configuration')

Describe the bug
When creating SDK clients in parallel in integration tests, each test recreates an instance of DefaultAzureCredential grabbing an Azure CLI credential. In a few percent of the cases, this fails with an exception.
Actual behavior (include Exception or Stack Trace)
System.ArgumentException: The specified configuration does not have a telemetry channel. (Parameter 'configuration')
at Microsoft.ApplicationInsights.TelemetryClient..ctor(TelemetryConfiguration configuration)
at Microsoft.ApplicationInsights.DependencyCollector.Implementation.EventHandlers.DiagnosticsEventHandlerBase..ctor(TelemetryConfiguration configuration)
at Microsoft.ApplicationInsights.DependencyCollector.Implementation.AzureSdkDiagnosticsEventHandler..ctor(TelemetryConfiguration configuration)
at Microsoft.ApplicationInsights.DependencyCollector.Implementation.AzureSdkDiagnosticListenerSubscriber.GetEventHandler(String diagnosticListenerName)
at Microsoft.ApplicationInsights.DependencyCollector.Implementation.DiagnosticSourceListenerBase`1.OnNext(DiagnosticListener value)
at System.Diagnostics.DiagnosticListener.AllListenerObservable.OnNewDiagnosticListener(DiagnosticListener diagnosticListener)
at System.Diagnostics.DiagnosticListener..ctor(String name)
at Azure.Core.Pipeline.DiagnosticScopeFactory..ctor(String clientNamespace, String resourceProviderNamespace, Boolean isActivityEnabled)
at Azure.Core.Pipeline.ClientDiagnostics..ctor(ClientOptions options)
at Azure.Identity.CredentialPipeline..ctor(TokenCredentialOptions options)
at Azure.Identity.CredentialPipeline.GetInstance(TokenCredentialOptions options)
at Azure.Identity.DefaultAzureCredentialFactory..ctor(TokenCredentialOptions options)
at Azure.Identity.DefaultAzureCredential..ctor(DefaultAzureCredentialOptions options)

To Reproduce
Steps to reproduce the behavior (include a code snippet, screenshot, or any additional information that might help us reproduce the issue)
            services.AddApplicationInsightsTelemetry(o =>
            {
                o.RequestCollectionOptions.TrackExceptions = true;
            })
                .AddSingleton<TokenCredential>(s =>
                    new DefaultAzureCredential(new DefaultAzureCredentialOptions
                    {
                        ExcludeVisualStudioCodeCredential = true,
                        ExcludeSharedTokenCacheCredential = true,
                        ExcludeInteractiveBrowserCredential = true,
                        ExcludeEnvironmentCredential = true,
                        ExcludeVisualStudioCredential = true,
                        ExcludeManagedIdentityCredential = true,
                    }))
                .AddSingleton(s => new DigitalTwinsClient(new Uri(Configuration["ADT:InstanceUrl"]), s.GetRequiredService<TokenCredential>()))

Then create several integration tests (with IClassFixture<WebApplicationFactory<Startup>>)
Environment:

"Azure.DigitalTwins.Core" Version="1.2.0"; "Azure.Identity" Version="1.3.0"
dotnet 3.1

.NET SDK (reflecting any global.json):
Version:   5.0.101
Commit:    d05174dc5a
Runtime Environment:
OS Name:     Mac OS X
OS Version:  10.15
OS Platform: Darwin
RID:         osx.10.15-x64
Base Path:   /usr/local/share/dotnet/sdk/5.0.101/
Host (useful for support):
Version: 5.0.1
Commit:  b02e13abab
.NET SDKs installed:
2.1.811 [/usr/local/share/dotnet/sdk]
3.1.404 [/usr/local/share/dotnet/sdk]
5.0.101 [/usr/local/share/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.All 2.1.23 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.23 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.10 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.1 [/usr/local/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.23 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.10 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.1 [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]
could also reproduce on Azure Devops Ubuntu agent.

Hi @algattik:   Is there a reason why you're using DefaultAzureCredential in the tests rather than the Credential member that your TestEnvironment instance offers?

Hi @jsquire, at the moment we use the same Startup class used in the runtime code, but I could try that.

TestEnvironment seems to be part of azure-sdk-for-net's internal test framework. We are building an enterprise project and need to work with standard WebApplicationFactory. Otherwise please advise on using TestEnvironment  externally.
Also note that we use AzureCliCredential for developer convenience, which apparently TestEnvironment doesn't support.

@algattik:  Apologies; I overlooked that "customer" part of the text in your contact card and assumed based on your Azure membership that the integration tests that you referred to were part of the SDK itself.  You're correct in that the TestEnvironment isn't the way to go here.   I'll need to defer to @christothes and @schaabs as the Identity experts for better guidance.

Hi @algattik This appears to be an issue with Application Insights. Would you mind opening an issue here - https://github.com/microsoft/ApplicationInsights-dotnet ?

@christothes @jsquire
:wave: I'm also seeing such outcome, but with a different call stack. Creating a repro case in my context may actually be non trivial.
This issue has been closed and I'm a little lost as to how this issue should be handled.
Any guidance would be greatly appreciated. Thoughts?
References:

Azure.Storage.Blobs: 12.8.0
Microsoft.ApplicationInsights.AspNetCore: 2.17.0

  Message:
    System.ArgumentException : The specified configuration does not have a telemetry channel. (Parameter 'configuration')
  Stack Trace:
    TelemetryClient.ctor(TelemetryConfiguration configuration)
    DiagnosticsEventHandlerBase.ctor(TelemetryConfiguration configuration)
    AzureSdkDiagnosticsEventHandler.ctor(TelemetryConfiguration configuration)
    AzureSdkDiagnosticListenerSubscriber.GetEventHandler(String diagnosticListenerName)
    DiagnosticSourceListenerBase`1.OnNext(DiagnosticListener value)
    AllListenerObservable.OnNewDiagnosticListener(DiagnosticListener diagnosticListener)
    DiagnosticListener.ctor(String name)
    DiagnosticScopeFactory.ctor(String clientNamespace, String resourceProviderNamespace, Boolean isActivityEnabled)
    ClientDiagnostics.ctor(ClientOptions options)
    StorageRequestValidationPipelinePolicy.ctor(ClientOptions options)
    StorageClientOptions.Build(ClientOptions options, HttpPipelinePolicy authentication, Uri geoRedundantSecondaryStorageUri)
    StorageClientOptions.Build(ClientOptions options, Object credentials, Uri geoRedundantSecondaryStorageUri)
    BlobClientOptions.Build(Object credentials)
    BlobContainerClient.ctor(String connectionString, String blobContainerName, BlobClientOptions options)
    BlobContainerClient.ctor(String connectionString, String blobContainerName)
    BlobsDownloader.BuildBlobContainerAccessor(IConfiguration configuration, String connectionStringKey, String name) line 96
    PublicFilesFixture.ctor(PublicFilesWebServerSetupFixture wss) line 27

Hi @nulltoken.   I'd advise opening a new issue for the Application Insights team, as this looks like it is most likely an issue in that library.  https://github.com/microsoft/ApplicationInsights-dotnet

@jsquire Thanks. I will do this.


YouTube (and html 5 video) playback 100% broken across all browsers (likely since Ventura 13.3.1 update)

Before opening the issue, have you...?

[X] Searched for existing issues
[X] Looked through the wiki
[X] Updated MonitorControl to the latest version (if applicable)

Describe the bug
Complete writeup on the issue is on the Reddit YouTube forum.
Issue started approximately 2-3 weeks ago (as of 1-May), probably tied to the last Venture 13.3.1 update, but can not confirm directly.  With MonitorControl running and audio output directed at controlled monitor, attempting to play any web based video via video html tag (YouTube, Reddit, or other standard embedded videos) will result in an entirely broken experience with video audio de-synced, video/audio length corrupted, and tracks jumping or skipping forward at high rate of speed (100x speed or greater).
This is a 100% failure case in my situation across all browsers (Safari, Chrome, Edge tested so far) and failure is immediate even with a fresh install of a new browser.  No login to YouTube required.  Impacts Ads, Shorts, and Videos, and seemingly any sources that use the video html tag for playback.
In contrast video html tag is set to muted will play normally, or if audio is routed to a non-DDC/CI output then the video will play normally.
Issue does not occur if you disable Monitor Control on Startup and reboot.
Major video services such as Netflix, HBO, and Prime Video do not experience this issue when played directly from their sites.
I did briefly have glitches with QuickTime Player jumping to the end of a video.  However, it was very rare.
Steps to reproduce

Install Monitor Control (latest version as of 1-May-23)
Update to Ventura 13.3.1 (assuming this is source of issue)
Direct output to external monitor (maybe reboot for good measure)
Attempt to play YouTube videos (or any video 5 html test site can work).
Audio with bug out immediately, Video will start misbehaving within a few seconds, or skipping ahead on the scrubber will eventually trigger the behavior...or just be patient and it will start freaking out.  Most of the time it breaks within fractions of a second, but rarely a video will play through for a bit with audio broken.
Repro's on all videos tested and ads, whether signed in or not or even fresh install of new browser.
Mute browser and fast forward, videos will play normally; or use HTML 5 test sites which usually run tests in muted mode which will work fine.  So audio specifically seems to be the breaking issue.
Disable "Start on Login" and Reboot for videos to play normally with stable audio and video.

Expected behavior
Expect YouTube videos and other web based videos to play normally.
Related Videos:

Edge Recording - https://youtu.be/IpHQXmWJmVQ
Recording of several failure behaviors - https://youtu.be/9WHRW0jE7B4
See Video Notes in each description for more context of recording.

Anything else?
Full write-up of the situation available here: https://www.reddit.com/r/youtube/comments/13449c7/youtube_streaming_broken_100_repro_across_safari/
Environment Information (please complete the following information)
- macOS version: Ventura 13.3.1
- Mac model: Macbook Pro M1 Max (Max specification)
- MonitorControl version: Current as of 1-May
- Monitor(s): (2 monitors) LG OLED and QLED Ergo Brand 27"
- Apple Silicon/M1 (yes or no): Yes

Do you have the same issue with BetterDisplay? Do you have still have this issue with 13.4? I could not reproduce the issue.
Audio routing does not happen through DDC/CI, it is only volume control - but the issue in hand should not be related to that as it clearly seems to be some sort of software issue (maybe something interferes with MonitorControl or pushes MC into a continuous configuration loop which takes away resources briefly? I am not sure).

Same issue here!
Macbook Air M2.
It happens when I try to use my LG Monitor as the audio output.