[AIRFLOW-3213] Create ADLS to GCS operator

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-3213
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.

Description

This PR adds a new operator, AdlsToGoogleCloudStorageOperator, which copies files from Azure Data Lake to Google Cloud storage, and can optionally "sync" between the two, copying only files that don't exist in the destination. It functions much like the existing S3 to GCS and GCS to S3 operators.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
test_init
test_execute

Commits

[x] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes flake8

Codecov Report

Merging #4134 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4134      +/-   ##
==========================================
- Coverage   77.49%   77.49%   -0.01%
==========================================
  Files         199      199
  Lines       16233    16246      +13
==========================================
+ Hits        12580    12590      +10
- Misses       3653     3656       +3

Impacted Files
Coverage Δ

airflow/settings.py
80.41% <0%> (-0.74%)
:arrow_down:

airflow/jobs.py
77.09% <0%> (-0.28%)
:arrow_down:

airflow/operators/python_operator.py
95.03% <0%> (ø)
:arrow_up:

airflow/operators/http_operator.py
93.54% <0%> (ø)
:arrow_up:

airflow/sensors/s3_prefix_sensor.py
100% <0%> (ø)
:arrow_up:

airflow/sensors/sql_sensor.py
100% <0%> (ø)
:arrow_up:

airflow/operators/s3_to_hive_operator.py
94.01% <0%> (ø)
:arrow_up:

airflow/sensors/s3_key_sensor.py
100% <0%> (ø)
:arrow_up:

airflow/operators/s3_to_redshift_operator.py
100% <0%> (ø)
:arrow_up:

airflow/operators/docker_operator.py
97.67% <0%> (+0.02%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

@bkvarda Can you rebase on master as well?


Modul throws UnhandledPromiseRejectionWarning

Hello,
i´m just getting started with MagicMirror modules so maybe i´m missing something really simple.
I checked out the git repo, ran 'npm install' inside the folder, added the snippet to the config.js and changed the language to italian but when i try to start with npm start i get this output
`[10.10.2023 12:50.08.505] [ERROR] (node:11147) UnhandledPromiseRejectionWarning: Error: Cannot find module 'request'
Require stack:

/home/schotti/Desktop/MagicMirror/modules/MMM-learnlanguage/node_helper.js
/home/schotti/Desktop/MagicMirror/js/app.js
/home/schotti/Desktop/MagicMirror/js/electron.js
/home/schotti/Desktop/MagicMirror/node_modules/electron/dist/resources/default_app.asar/main.js
at node:internal/modules/cjs/loader:1084:15
at Function. (node:electron/js2c/browser_init:2:117419)
at Module._resolveFilename (/home/schotti/Desktop/MagicMirror/node_modules/module-alias/index.js:49:29)
at node:internal/modules/cjs/loader:929:27
at Function._load (node:electron/js2c/asar_bundle:2:13327)
at Module.require (node:internal/modules/cjs/loader:1150:19)
at require (node:internal/modules/cjs/helpers:110:18)
at Object. (/home/schotti/Desktop/MagicMirror/modules/MMM-learnlanguage/node_helper.js:8:15)
at Module._compile (node:internal/modules/cjs/loader:1271:14)
at Object..js (node:internal/modules/cjs/loader:1326:10)
at Module.load (node:internal/modules/cjs/loader:1126:32)
at node:internal/modules/cjs/loader:967:12
at Function._load (node:electron/js2c/asar_bundle:2:13327)
at Module.require (node:internal/modules/cjs/loader:1150:19)
at require (node:internal/modules/cjs/helpers:110:18)
at loadModule (/home/schotti/Desktop/MagicMirror/js/app.js:180:19)
(Use electron --trace-warnings ... to show where the warning was created)
[10.10.2023 12:50.08.506] [ERROR] (node:11147) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
[11147:1010/125009.070170:ERROR:object_proxy.cc(590)] Failed to call method: org.freedesktop.portal.Settings.Read: object_path= /org/freedesktop/portal/desktop: org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.portal.Desktop was not provided by any .service files
MESA-LOADER: failed to retrieve device information
MESA-LOADER: failed to retrieve device information
MESA-LOADER: failed to retrieve device information
`

Thanks for your help in advance.

Hi, thanks for trying out this module. Am not able to reproduce this yet. Will investigate a bit more.

The problem is that the module want's to import the package 'request' and since 'request' isn't part of MagicMirror anymore, you have to install it (on fresh systems) manually by running npm install request in the folder of this module.
I fixed this in PR #7. With this no manual installation will be necessary.


build: cmake: Add support for GNU/Hurd

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Needed a rebase to pass the test.  I rebased it myself.  Hopefully this passes.


fix: Fix integration test counter and trigger

Fixes #1271
Test Plan

How do we know the code works?

Integration tests run only if have 2  or more approves
Checklist

[x] Fix integrations test counter
[x] Make it as separate job
[x] Move all integration tests( except WSL) to same file

It works properly on the fork, however. Here it seems that it does not count approves.
I must investigate it more

It works properly on the fork, however. Here it seems that it does not count approves.
I must investigate it more

fixed using custom api call


Implement ScaleGestureRecognizer

Spec: https://material-motion.github.io/material-motion/starmap/specifications/primitives/gesture_recognizers/ScaleGestureRecognizer

🎊 My diff is ready for review at http://codereview.cc/D1964


Add hubert robust

Add a new upstream model.
HuBERT base continually trained with Libri 960 hr with Musan noise, Gaussian noise, and Reverberation.

Tested by
import torch
import s3prl.hub as hub
model = getattr(hub, 'hubert_base_robust_mgr')()
device = 'cuda'
model = model.to(device)
wavs = [torch.randn(160000, dtype=torch.float).to(device) for _ in range(16)]
with torch.no_grad():
    reps = model_0(wavs)["hidden_states"]

Should I modify the README of upstream?

Thanks @nobel861017 ,
Sure please update the upstream/README

Thanks @nobel861017 ,
Sure please update the upstream/README

I added a row at the bottom of the table.

You added the same row as the hubert/hubert large ?
Isn't it some kind of robust hubert?

I'm sorry, I forgot to save the file lol.

You added the same row as the hubert/hubert large ? Isn't it some kind of robust hubert?

fixed


Solve the issue when multiple entity has the movement to the same position

I think we need to figure out the speed and the priority of every type of movement (jump/fall/regular move)
Since there will be still some control that require precise input, we can implement a key to do slow motion and with a small indicator that shows when the turn start, similiar to beat meter in necrodancer See reference


feat(footer): update footer links

Update the footer to include two new links and update existing one:
Car finance - /car-finance
Hire purchase (HP) - /car-finance/hp
Personal contract purchase (PCP) - /car-finance/pcp

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Published nightly binaries have inconsistent test&debugimage filenames compared to jdk&jre filenames

It seems inconsistent naming on github published binaries, see: https://github.com/AdoptOpenJDK/openjdk8-binaries/releases/tag/jdk8u-2021-03-20-04-39
jdk and jre are all the same timestamp.
debug and test are different

Seems this is intentional: https://github.com/AdoptOpenJDK/openjdk-github-release/blob/435bc9d2175f596c380de3a9f0bbbdf0cdb19368/sbin/Release.sh#L21
only renames -jdk and -jre files.

Intentional - or a consequence of debug and testimages starting to be produced at a later date than the jdk / jre images?


Ensure that images render correctly in doc

The {imagesdir} attribute is added automatically when using the image reference. The current docs are not showing any images referenced this way.

@jenmalloy I've updated the PR with the ifdefs

@jenmalloy fixed


azopy does not support NFS

Need a mention here that AzCopy doesn't support NFS Azure File Shares.

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

ID: d1bdc29f-175d-09b9-771d-cd84dbffbb49
Version Independent ID: f5fe11b9-8224-aa19-602d-bcc3ed7c9452
Content: azcopy copy
Content Source: articles/storage/common/storage-ref-azcopy-copy.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@javester Thanks for the question! We are investigating and will update you shortly.

@javester The following table shows the current level of support for Azure Storage features in accounts that have the NFS 4.1 feature enabled.

https://github.com/Azure/azure-storage-azcopy/issues/1648
It been already added to the backlog for now. We don't currently have any plans so far as I know.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Expand Binary-Only section to document companion OME-XML

As we are reviewing the companion sample files for the OME schema, this PR updates the corresponding section of the OME-TIFF specification to improve our description of the partial metadata blocks to document both available approaches: either a master OME-TIFF file or a companion OME-XML file.
/cc @dgault @hflynn

Staged at https://www.openmicroscopy.org/site/support/ome-model-staging/ome-tiff/specification.html

One minor nitpick but otherwise renders fine and links are all good. I leave @dgault to comment on the correctness of the XML block.

The XML block and accompanied description looks pretty straightforward and easy enough to understand from my perspective.

Thanks both, merging and releasing today - unless there is a reason to hold off this going live @sbesson ?

@hflynn: no. Amended samples are live so there is no reason not to release the formats documentation.

Great, will build and promote now then, thanks.


Error granting roles on tf helper

The tf helper doesn't grant the appropriate roles in the following sections:
https://github.com/terraform-google-modules/terraform-google-forseti/blob/b7f70a6b63a894ea60ab73834e8b02ed58b260c0/helpers/forseti-setup.tf#L48-L52
https://github.com/terraform-google-modules/terraform-google-forseti/blob/b7f70a6b63a894ea60ab73834e8b02ed58b260c0/helpers/forseti-setup.tf#L54-L58

We dropped the helpers/forseti-setup.tf Terraform configuration in the v1.0 release of this module; in addition to the issues that you mentioned that configuration had bugs around managing IAM bindings vs members, and so on. Because of this we decided to drop this in favor of the scripts/setup.sh. Having a Terraform based setup configuration would be nicer than a shell script but we have no immediate plans to add that configuration.
If you'd be willing to add an updated version of the Terraform setup script we'd be happy to help out, just comment on this issue and we can continue this conversation.


[DOC] Minor documentation improvements and remove glossary entry for array-like

A couple of minor formatting fixes, and I removed the entry for array-like in favor of the one defined in the numpy glossary.
Also looking if I can figure out which CSS variable to override to format terms in the same way as py:obj.

Spectrum class: removes the whitespace before the [k] reference, add the missing footbibliography directives, and format the argument method with double backticks.

before: https://mne.tools/dev/generated/mne.time_frequency.Spectrum.html?highlight=spectrum#footcite-welch1967
after: https://output.circle-artifacts.com/output/job/c6f49603-b97d-4aef-bece-78550cf5b6d5/artifacts/0/dev/generated/mne.time_frequency.Spectrum.html#mne.time_frequency.Spectrum

Same for EpochsSpectrum class.

before: https://mne.tools/dev/generated/mne.time_frequency.EpochsSpectrum.html?highlight=epochspectrum
after: https://output.circle-artifacts.com/output/job/c6f49603-b97d-4aef-bece-78550cf5b6d5/artifacts/0/dev/generated/mne.time_frequency.EpochsSpectrum.html#mne.time_frequency.EpochsSpectrum

Add references and footbiliography to psd_array_welch and psd_array_multitaper functions:

welch: https://output.circle-artifacts.com/output/job/c6f49603-b97d-4aef-bece-78550cf5b6d5/artifacts/0/dev/generated/mne.time_frequency.psd_array_welch.html#mne.time_frequency.psd_array_welch
multitaper: https://output.circle-artifacts.com/output/job/c6f49603-b97d-4aef-bece-78550cf5b6d5/artifacts/0/dev/generated/mne.time_frequency.psd_array_multitaper.html#mne.time_frequency.psd_array_multitaper

Good to merge on my end, I haven't figured out which CSS variable had to be changed to format the terms in the same way as py:obj. I'll look into it again in another PR.


Generalized Pool

Closes #1331
What was wrong?
There is alot of duplicated functionality between OrphanBlockPool and OperationPool.
How was it fixed?
I've moved the OperationPool class into a new class HashableContainerPool, which can be used to create pools for HashableContainer objects. By default, the containers are keyed in the pool's _pool_storage dict by the container's hash_tree_root value. This behavior can be changed though by overriding the  _get_root method in a HashableContainerPool child class.
In the case of OrphanBlockPool, we want to key pool values using the root found in block.message, so we override the _get_root method like so:
def _get_root(self, block: BaseSignedBeaconBlock) -> Root:
  return block.message.hash_tree_root

To-Do

[x] Clean up commit history

[ ] Add entry to the release notes

Cute Animal Picture

also i'm not entirely sure we need to override the key functionality, i.e. direct to the inner message on a "signed" operation -- in which case we can drop that for now -- we would need to double-check the spec but so far we have been getting along fine working directly w/ the signed operation and deferring to the inner operation as needed...

Hey Alex, I was able to tighten this up quite a bit.

Someone with write access will need to merge this for me :smile:


iOS warnings when loading app (require cycles)

I am getting a lot of these warnings when opening the app.
Xcode 10.2 and iOS 13.3.2 simulator (iPhone 8).

I'm not an iOS developer, but I can try to fix it if needed.

I see this under Android too.  If you can investigate, please do.  I don't think this is dangerous, so I've been ignoring it.  But it would be nice to eliminate if possible.

It is related to a known bug of rn-fetch-blob, left unfixed for a couple of years.
https://github.com/joltup/rn-fetch-blob/issues/183#issuecomment-450826541
I have fixed it manually with MIT's patch-package.


Add directors to WWCode Birmingham

Mail from Jodi

Hi Pia,
Could you please add the following 2 Directors to the WWCode Birmingham OC?
Rebeccah...
Nicole...
Thanks!

More info on email.

My mistake! :/


Add support for leading comma

Not everyone likes to use trailing comma. Most of the time users' eyes behavior would focus more on the left-hand side of the monitorsand therefore it is very easy to spot on whether it is leading comma or other rust operators, such as =>, +, ., etc.
    match io::stdin().read_line(&mut buffer_input) {
        Ok(buffer_input_size)
            => println!(
                "Hello, {}! Your name size is {}."
                // call .trim() because .read_line also read `\n` character at the end of input
                , buffer_input.trim()
                , buffer_input_size
            )
        , Err(error) => println!("[Error] Cannot read input. Reason: {}.", error)
    };

Thanks for reaching out with your request!
As you may already be aware, rustfmt's main job is to reformat input code according to the style codified in the Style Guide, and that's why the closing braces are formatted they way they are.
rustfmt is still configurable nevertheless, largely to support some of the formatting decisions that cover the style topics large portions of the developer community find contentious (spaces vs. tabs, etc.). Generally speaking we're open to the inclusion of new configuration options, provided the default value still aligns with the Style Guide, but we do also have to consider whether the impact on the codebase and associated maintenance, bug surface, etc. is tolerable in the context of option and style it produces. For a wildly hyperbolic example, we wouldn't accept a new option that doubled the size and complexity of the codebase for a bespoke style that'd only be used by one or two developers because such a change would be far too much of a burden on rustfmt's main job.
I'm not adamantly opposed to supporting this, but it's also not something that the rustfmt team is ever likely to work on due to the sharp divergence from our core style goal and comparatively minimal appetite for this custom styling option. However, if you or anyone else feels strongly enough about wanting to have this and are willing to submit a PR with an implementation then we'll certainly review and take into consideration!


track time usage

Let's track time usage per user

@rultor release, tag is 0.2

@rultor release, tag is 0.2

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.2

@yegor256 Done! FYI, the full log is here (took me 4min)

Oops! Job gh:yegor256/threecopies#3 was not in scope


[#3368] feat(jdbc-doris): support pre-assign partition for Doris table

What changes were proposed in this pull request?
Support pre-assign partition for Doris table.
Why are the changes needed?
Fix: #3368
Does this PR introduce any user-facing change?
No.
How was this patch tested?
UT & IT

@mchades @zhoukangcn , would you help review this when you are free?

@mchades, this PR is ready for review, thx.

Overall LGTM. @yuqi1129 Do you want to take another look?

I'm okay with the current changes.


[ci] playground tests - show live output from the tests

Microsoft Reviewers: Open in CodeFlow

@radical should we update the xunit runner settings to enable live output too?

@radical should we update the xunit runner settings to enable live output too?

Is this sufficient to show the output in VS? If so, this sounds like a great idea.

Is this sufficient to show the output in VS? If so, this sounds like a great idea.

Unfortunately I don't think it works in VS @bradwilson

Also, if you want to see live output with dotnet test, you need to pass --logger "console;verbosity=normal":
https://github.com/user-attachments/assets/c7af5cc7-c899-400e-827b-ec4aba3ee86a


docs: remove comment stating FIFO SQS Queues aren't supported

Description of changes:
Remove comment stating FIFO SQS Queues aren't supported
Description of how you validated changes:
https://aws.amazon.com/blogs/compute/new-for-aws-lambda-sqs-fifo-as-an-event-source/
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1367 into develop will increase coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1367      +/-   ##
===========================================
+ Coverage    94.42%   94.42%   +<.01%
===========================================
  Files           78       78
  Lines         4554     4555       +1
  Branches       912      912
===========================================
+ Hits          4300     4301       +1
  Misses         119      119
  Partials       135      135

Impacted Files
Coverage Δ

...nslator/model/preferences/deployment_preference.py
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


refactored PostProcessor

This is a major refactoring of PostProcessor. Some method names need to be replaced, I'm unsure what they currently do, and others could probably be cleaned up (method_name18 and method_name19). I've also replaced regex checks by direct calls to .split (other than method_name13, I couldn't get that to work). Finally, instead of chaining flat_map it will now reduce over procs. This looks scary initially, but one can get used to it easily.
Please check if the two calls of reducing over procs can be merged - the logic of post_process, method_name3, full_stop_separated_tokens and method_name16 confuses me, and I'm not sure if there's a specific order or if everything could be run at once.
All tests pass.

Thanks! I really appreciate it. My only concern is that this seems to slow things down. This spec goes from 25s to 28.5s on my computer: https://github.com/diasks2/pragmatic_tokenizer/blob/master/spec/performance_spec.rb#L35-L47
Readability is important but in my opinion I think performance is possibly more important for this gem as I think a lot of use cases will be processing large amounts of text. So for changes that don't change the underlying functionality but slow things down I have some concern.
What are your thoughts?

I'm unsure what the reason for this is - either it's the procs (possible, but I doubt it), or it's the change of various regex. E.g. I went from:
flat_map { |t| (t[0] == '‚' || t[0] == ',') && t.length > 1 ? t.split(/(,|‚)/).flatten : t }
```
to:
````ruby
flat_map { |t| t.split(/^(,|‚)+/) }

While I believe the latter is the better solution, it will increase the necessity of optimized regex. E.g. quite a few methods would not even need to run if the token length is 1, and checking for string length might be much faster than running a regex.
What I've not tried yet is using Regexp#union to merge multiple regex into one larger regex and using this for token.split. I guess it could help a lot, I'll give it a shot.
But besides the speed issue (which we will tackle in the upcoming days and weeks) I strongly think that such a library should be maintainable. The original code was so complicated that even you might not be able to easily change/fix/add logic in a few months from now, and certainly noone else will. I'd therefor suggest to keep the code as understandable as possible, as it will allow other contributors to help.

Thanks, makes sense. Once we get the refactoring done let's focus on improving the speed. Thanks for all your help!

I tried to clean up the naming some: https://github.com/diasks2/pragmatic_tokenizer/commit/06bb017dd231365f97df77622f67dfdc8721f605

Great, that makes it a bit more understandable. I'm still unsure about how post_process, method_name3 , unified1  and full_stop_separated_tokens  work together, can these be cleaned up or rewritten? Due to the calls to external classes one has to jump around quite a bit between these methods.
Oh, and do you manage to remove the if/else ternary from split_emoji (and then move it to the regex constants)? I failed when trying so. And: if you could clean up unknown_period1?and unknown_period2?, I'd be grateful. Can some of their regex be extracted or merged? Does it make sense to extract domain?, email_address?, ip_range? and use these instead? In case a e.g. check for a domain is used multiple times (looks as if), one could save the returned boolean as instance variable so the token is only checked once. That should increase speed.
PS: if ABBREVIATIONS and STOP_WORDS were sets instead of arrays, it should be much faster to check against them.


Use IDs instead of indexes to identify the Runner

The IDs are guaranteed to be unique and stay constant, no matter what you do to the pane or window.
This diff is based on the idea in #81, by Roberto Agostino Vitillo (@vitillo), but rewritten to:

apply to the latest version of the source
use IDs for both windows and panes, not just panes

I have tested panes and windows as runners, both with VimuxUseNearest and without it. VimuxTogglePane() also works just fine.
This change is incredibly useful because once you have the runner, you can do whatever you want: spawn new panes, move the pane to a different window, join it back, reorder the panes and windows, change layouts, — it does not matter, your runner will remain set.

sigh @benmills

:+1:

Hey guys, this is exactly what I was looking for.
What about update and merge this PR?
What do you think @benmills?

This is absolutely brilliant, I'm moving to fork for now. Thank you @yunake

i have no idea why @benmills didn't even show up here to comment. if he dislikes the idea, a clear communication would help clear things up and i could move on. i still use this every day and would love to see this merged. Ben, do you still plan to actively maintain vimux?

Hey @yunake, don't think @benmills is actively ignoring this :) life gets in the way sometimes.
Either way, this is something I've been meaning to look into myself -- an easy way of keeping a map between open panes and runners so that rearranging a tmux layout doesn't mess things up.
I'll be reviewing your PR promptly and we can take it from there, how does that sound? 😄

Hey @ostera I've seen you being given write access and this has renewed my interest in getting this merged. The conflicts look trivial to resolve, but I've been using my own fork for three years now and have no idea what else might've changed in the way the plugin operates. Please review and let me know what you think, this might no longer be the best approach?

I've resolved the conflicts

i'm quite busy with other stuff, i will re-test this changeset on top of current master in a couple of days and let you guys know. i think it should work :)

@yunake ping

6 years later hope dies last

All home is not dead. As of this last week Ben turned up and we got the repository migrated where we now have three maintainers and can add more. I'm happy to consider adding more contributors with permissions to work directly based on contribution activity.
Looking through the outstanding PRs there seems to be several that are going to conflict, but this looks like one of the more widely desired ones that people have switched to forks just to get, so I'll probably prioritize this one.


New tooling bits, next preview release?

I have a critical question about the timeline of the next tooling bits, as the guys at dotnet-docker are stuck on a CLI version that's completely broken for linux 4.6+ kernels.
Related:
https://github.com/dotnet/core/issues/171
https://github.com/dotnet/dotnet-docker/issues/61
https://github.com/dotnet/cli/issues/3681
and many more (ubuntu 16.04 issues etcetera)
As more and more distro's move to 4.6+ kernels and a lot of CI's  using these in turn have as well (including the CI we use) there are more and more people grinding to a halt with this CLI version inside docker containers.
On top of that there is no obvious fix but to compile the CLI from source and then include this in your own docker build. Which is really quite an embarrassing answer for a company like Microsoft to point to as the only fix for now.
It seems this would require just a tiny bit of coordinated efforts to get resolved and I would love to know an estimate timeframe of the next tooling release with this fix.
EDIT: This is of course not exclusive to docker containers but to all dotnet cli instances running on 4.6+ kernels, I have the luck that my dev env is OSX and not an up-to-date linux distro.

@richlander @leecow can you comment on this please?

@NinoFloris we're tracking a fix for this here: https://github.com/dotnet/coreclr/issues/6604


More default channel colors

We currently have a few default channel colors.  We should add more based on channel names.
We should add: gray/grey => #fff, red/green/blue => #f00/#0f0/#00f
We can also add various fluorescence channels.  In general, these colors should be the closest color we have in the current present list to the color of the emission wavelength of that fluorescence method.  For instance, the CD* channels are listed here: https://www.abcam.com/primary-antibodies/human-cd-antigen-guide and here: https://en.wikipedia.org/wiki/List_of_human_clusters_of_differentiation.  Their emission wavelengths can be looked up by searching for them via something like "CD4 emission wavelength".  I found some of them here: https://www.aatbio.com/fluorescence-excitation-emission-spectrum-graph-viewer .  Besides the CD antigens, we've seen a few others (see some of the example files).
We want to only have fairly "primary" colors, rather than very specific colors per channel; the reason for this is to attempt to avoid near-duplicates.

When someone gets around to working on this, we can get a list of all the channel names we've extracted from our various test files as a reference of names we might want to research.


Crash WorkManager

Hi,
I've been using insight 1.0.2 in my App and 4% of my users experienced some crashes while the app was running in the background.
I turned out to be caused by android.arch.work:work-runtime-ktx:1.0.0-alpha02
This issue is known and solved in work-runtime-ktx:1.0.0-alpha05, is it possible to update this dependency? (https://developer.android.com/jetpack/docs/release-notes#july_24_2018)
Antoine

Hello Asalliou,
Thank you for your feedback.
You can now use the updated com.algolia.instantsearch-android:insights:1.0.6 dependency, which includes work-runtime-ktx:1.0.0-alpha05.
Let us know if you encounter any issues.

Hello @q-litzler
Thank's for your feedback. I will restore "insight" in my app as soon as a stable version of this dependency is released and included. I'll watch for a new version.


Tensorflow setup on clound9

I am doing the project using Tensorflow. I set up it on Clound9 since It run on Ubuntu. I chose python 2.7. However, It kept showing an error: enter image description here . Does anyone have any idea why it keep happening? By the way, I clone tensorflow github in my repository. Thanks
(tensorenv)ngoduyvu:~/workspace (master) $ pip install --upgrade https://storage.googleapis
.com/tensorflow/linux/cpu/tensorflow-0.8.0rc0-cp27-none-linux_x86_64.whl
Collecting tensorflow==0.8.0rc0 from https://storage.googleapis.com/tensorflow/linux/cpu/tensorflow-0.8.0rc0-cp27-none-linux_x86_64.whl
Downloading https://storage.googleapis.com/tensorflow/linux/cpu/tensorflow-0.8.0rc0-cp27-none-linux_x86_64.whl (22.2MB)
100% |████████████████████████████████| 22.2MB 40kB/s
Collecting six>=1.10.0 (from tensorflow==0.8.0rc0)
Downloading six-1.10.0-py2.py3-none-any.whl
Collecting protobuf==3.0.0b2 (from tensorflow==0.8.0rc0)
Downloading protobuf-3.0.0b2-py2.py3-none-any.whl (326kB)
100% |████████████████████████████████| 327kB 1.8MB/s
Requirement already up-to-date: wheel in ./miniconda/envs/tensorenv/lib/python2.7/site-packages (from tensorflow==0.8.0rc0)
Collecting numpy>=1.8.2 (from tensorflow==0.8.0rc0)
Downloading numpy-1.11.0-cp27-cp27mu-manylinux1_x86_64.whl (15.3MB)
100% |████████████████████████████████| 15.3MB 60kB/s
Collecting setuptools (from protobuf==3.0.0b2->tensorflow==0.8.0rc0)
Downloading setuptools-20.7.0-py2.py3-none-any.whl (508kB)
100% |████████████████████████████████| 512kB 1.8MB/s
Installing collected packages: six, setuptools, protobuf, numpy, tensorflow
Found existing installation: setuptools 20.6.7
Cannot remove entries from nonexistent file /home/ubuntu/workspace/miniconda/envs/tensorenv/lib/python2.7/site-packages/easy-install.pth

Can you try the pip with with --ignore-installed?

Thanks, I find out what is the problem. I already install tensorflow, but what I did were install again so It collapsed.


The kargo version command doesn't work when not logged in

Checklist

[x] I've searched the issue queue to verify this is not a duplicate bug report.
[x] I've included steps to reproduce the bug.
[x] I've pasted the output of kargo version.
[x] I've pasted logs, if applicable.

Description
Running kargo version when you're not logged in returns an error
Example
$ kargo version
Error: get version info from server: unavailable: unsupported protocol scheme ""

Steps to Reproduce
First:
rm -rf ~/.config/kargo

Then
kargo version

Expected behavior
There should be a --client flag that returns the version of the client when not logged into a Kargo server
Example:
kargo version --client

It's not just version. It's actually all commands that fail with this if you don't have an existing config file. Even login fails.
It appears to have been introduced in #1179
cc @devholic

Hi @krancour when do you plan/think to release kargo cli v0.3.1 to fix this issue?
It's quite critical as it's not possible to test the quickstart guide because not possible to create any project.
Is there an alternative way to create a Kargo Project? maybe YAML?

@bygui86 I expect v0.3.1 to be released today.


google-java-format doesn't work on Java 16+

See https://github.com/google/google-java-format#jdk-16

I was unable to find the correct syntax for Coursier to fix this issue so I opened a question on Stackoverflow.

I was unable to find the correct syntax for Coursier to fix this issue so I opened a question on Stackoverflow.

Fixed in #8


Calling conventions in function types aren't parsed properly

Set calling convention with Edit function properties dialog
Try editing function with y
It no parse :(

Does this happen with any specific calling conventions? I can't reproduce with thiscall / stdcall / fastcall / linux-syscall / regparm (which shows up as __fastcall?) / cdecl

if regparm shows up as fastcall, then https://github.com/Vector35/binaryninja-api/issues/1741 has regressed.

Clang supports __convention("name") parsing now, and I'm pretty sure the core one did too? Regardless, as of 3.0.3413-dev clang will do this.


Step order difference between code and algorithm in paper

Hello @hojonathanho,
does the code here swapped the order of step 4. and step 5. of the algorithm presented in the paper?

I've experimented with both orderings, and I found no significant difference in performance.


What does the value in MemoryRegionKind::UnknownBios(u32) mean?

What does the number contained in it mean? Is this the e820 memory type? I think it's worth documenting.

Yes, it contains the e820 memory type. Feel free to open a PR to document this otherwise I'll try open one over the week-end.


RLP-840: RIF Comms base

This is the first PR for issue RLP-840, which is about implementing the RIF Comms transport layer.
Changes
This PR:

moves raiden.network.transport to transport.
implements skeleton classes for the RIF Comms transport layer implementation.
applies Reformat Code + Optimize Imports to all affected files.

It looks good for now, just some comments:
1. The methods that are not implemented; maybe we can add a raise not implemented error instead of leaving them empty. Node, MessageParams and Layer classes have empty methods.

this is a great idea :+1:
i've gone ahead and done it for the RifCommsLayer class. however, i was not able to find any empty methods in the MessageParams class.

2. The file  transport/udp/**init**.py is really necesary?

yes. we need to have udp as a package to import code from udp/transport.py, udp/utils.py and udp/healthcheck.py elsewhere in the codebase (which is currently happening in several places).

unit test results here


add(cmd/ghz): expose --lb-strategy

Fixes #258 (at least the MVP)

Hello, thanks for the PR. Unfortunately load balancing in gRPC is a complicated topic. The reason #242 was API-only change is because even when using the included round robin LB strategy, additional setup might be (usually is) needed by the user. My understanding is that specifically a resolver would need to be registered via init side effect (See example). By default grpc-go uses a a passthrough resolver, so even setting the balancing strategy like this, depending on the configuration and hostname used, it would likely end up resulting in picking the first instance. The workaround is to use dns:///... scheme in passed hostname, or alternatively set the default resolver to "dns" (see docs).
So i think there is a little bit of additional work needed in this PR. in main() after we set up the config, where we create the logger, before Run() we should check if LB flag was set, and if it is round-robin we should set the resolver to dns using resolver.SetDefaultScheme("dns").

We can simply document that if you use client load balancing, the user should specify the endpoint with the dns:/// prefix. That's what we do internally.

Hello, thanks for the PR. This has been published in 0.91.0.


[firebase_auth] Need better examples when used with google_sign_in

There are no examples on how to do below 2 things in the firebase_auth plugin.

A detailed example for how to listen to the firebase_auth [onAuthStateChanged] stream, especially when signed in using google_sign_in.

An example to show how to utilize [signInSilently] in the google_sign_in plugin when using the [firebase_auth] plugin.

@cyanglaz
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/1112. Any further collaboration will be done there.


Use enum package for constants

instead of re-inventing the wheel, let's port the constants in this package to the enum package (available in py>=3.4 and a backport for py2).

If we do use enum, how do you envisage unpacking the .value numeric value?
Either we would need to do something like the following everywhere:
self.command(const.const.DISPLAYOFF.value,  ...)

or modify the command method to sense whether an enum or integer is supplied and call .value appropriately.
[I think I prefer the second option, but open to suggestions]

Yes the 2nd option sounds good, and then maybe add a deprecation warning if an integer is passed in, or just add a comment and remove int support at some point.

The command needs to be able to take integers - see https://github.com/rm-hull/ssd1306/blob/master/oled/device.py#L132-L147:
            self.command(
                const.DISPLAYOFF,
                const.MEMORYMODE,
                const.SETHIGHCOLUMN,      0xB0, 0xC8,
                const.SETLOWCOLUMN,       0x10, 0x40,
                const.SETCONTRAST,        0x7F,
                const.SETSEGMENTREMAP,
                const.NORMALDISPLAY,
                const.SETMULTIPLEX,       0x3F,
                const.DISPLAYALLON_RESUME,
                const.SETDISPLAYOFFSET,   0x00,
                const.SETDISPLAYCLOCKDIV, 0xF0,
                const.SETPRECHARGE,       0x22,
                const.SETCOMPINS,         0x12,
                const.SETVCOMDETECT,      0x20,
                const.CHARGEPUMP,         0x14)

The non-const values are parameters to the preceding command

Once all the test related tickets are resolved I'll take a look at this.

Having constants for the various display and interface types would also be useful, my app (and the demo_opts) now have them hardcoded, e.g.:
if args.display in ('ssd1306', 'ssd1331', 'sh1106'):
    if args.interface not in ('i2c', 'spi'):

Not being able to subclass Enum and stuff is annoying, let's skip this idea.


Fixed issues in Spatial Awareness System

XRTK - Mixed Reality Toolkit Change Request
Overview
Updated spatial awareness system to work with updated camera system changes in #517
Target of the change:
Is this enhancement for:

Core (core framework, interfaces and definitions)

Changes:

Fixed spatial awareness system transforms
Updated wmr and lumin observers to work with updated camera system changes
Fixed #506 additional component inspector

Submodules

https://github.com/XRTK/SDK/pull/157
https://github.com/XRTK/WindowsMixedReality/pull/68
https://github.com/XRTK/Lumin/pull/64

Gonna wait to merge this until after the camera system PR is merged into development


Is this still maintained?

Hi, Is this repo still maintained? Can I use this safely?

Yes, this library is still maintained, but I haven’t done much too it in quite some time.
I am planning a major version bump at some point in the near-ish future, and contemplating merging it into my @ethersproject mono repo, but still not sure of that yet. I’d open an issue on this repo to discuss that last point before I do.

Thank you, Richard, for the update.


no-restricted-paths/ How to set except zone for the relative paths. Please can you help me 🙏🙏🙏

Good day! I have a folder structure:
my-project
 |----- src
 │         |----- main
 |                    |----- api.1
 |                              |----- ConstService.js
 |                    |----- api.2
 |                              |------ AccountService.js
 |                    |----- components
 |                             |------ ScreenLogin.js
 |                    |------ service
 |                    |------ workers

And I need to restrict the import from './src/main/api.1' to './src/main', but except one zone './src/main/api.2. It means only possible to import the entities from './src/main/api.1' to './src/main/api.2. Means to get the error if import the entities from './src/main/api.1' to './src/main/components', './src/main/service', and './src/main/workers/'.
My rule:
'rules': {
    'import/no-restricted-paths': [
      'error',
      {
        'zones': [
          { 'target': './src/main', 'from': './src/main/api.1', 'except': ['./src/main/api.2'] },
        ],
      }
    ],
  },

The linted file is './src/main/components/ScreenLogin.js:
// IN THIS CASE GET THE ERROR NO-RESTRICTED-PATHS:
import { ConstService } from '../api.1'

The linted file is './src/main/api.2/AccountService.js:
// IN THIS CASE I DON'T NEED GET THE ERROR, BUT I HAVE THE ERROR:
import { AccountService } from '../api.1'

Please can you tell me how can I set the exception for './src/main/api.2' in my case. According the documentation: An optional except may be defined for a zone, allowing exception paths that would otherwise violate the related from. What does it means the exception path only can be relative to from zone. Please can you tell which way I can set in my case 🙏🙏🙏

Assuming your eslint config is in the root, this seems like it should work.
If you'd like to make a PR with a failing test case, I can try to get it working.

Assuming your eslint config is in the root, this seems like it should work.
If you'd like to make a PR with a failing test case, I can try to get it working.


Swap to using std's OnceCell once stablized

OnceCell is set to be partially stablized in Rust 1.70. (https://github.com/rust-lang/rust/pull/105587).
If it releases as a part of Rust 1.70 and it's acceptable to bump the MSRV of this crate to it, it may be possible to replace the once_cell dependency of this crate with the stdlib's implementation.

I don't think there is a strong motivation to do that, and there will be a strong pushback against increasing the MSRV of this crate.


🐛 [firebase_auth]  FirebaseAuthException: [firebase_auth/missing-client-identifier] on OnePlus8Pro

Bug report
Describe the bug
An exception of the following type is being thrown Only on OnePlus8Pro while I try to use FirebaseAuth.instance.verifyPhoneNumber method while doing mobile number verification on a flutter app.
FirebaseAuthException: [firebase_auth/missing-client-identifier] This request is missing a valid app identifier, meaning that neither SafetyNet checks nor reCAPTCHA checks succeeded. Please try again, or check the logcat for more details.
Steps to reproduce
Steps to reproduce the behavior:

Calling this FirebaseAuth.instance.verifyPhoneNumber is the single-step replication procedure. the user is being redirected to Recaptcha page even though the safety net is being enabled. This doesn't seem to occur with any other device.  verificationFailed callback has been hit as well.

Expected behavior
The ReCaptcha page should not occur and the OTP is expected to be sent to the mobile number.
Sample project
Providing a minimal example project which demonstrates the bug in isolation from your main App greatly enhances the chance of a timely fix.
Please link to the public repository URL.

Additional context
This behavior is seen while the app is running on emulators (which is expected). but it is not the desired behavior for a real device and through the error tracking system it is evident that this particular issue happens on OnePlus8Pro device alone.

Model
OnePlus8Pro (QKR1.191246.002)

Model Id
QKR1.191246.002

Name
OnePlus8Pro

---
Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel unknown, 2.10.5, on Microsoft Windows [Version 10.0.22000.795], locale en-IN)
[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2020.3)
[√] VS Code
[√] Connected device (1 available)
[√] HTTP Host Availability

! Doctor found issues in 1 category.

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.16.2
Flutter SDK 2.10.5

dependencies:
- animated_bottom_navigation_bar 0.3.3 [flutter]
- cached_network_image 3.2.0 [flutter flutter_cache_manager octo_image cached_network_image_platform_interface cached_network_image_web]
- connectivity_plus 2.2.0 [flutter connectivity_plus_platform_interface connectivity_plus_linux connectivity_plus_macos connectivity_plus_web connectivity_plus_windows]
- contacts_service 0.6.3 [flutter collection quiver]
- crypto 3.0.2 [typed_data]
- cupertino_icons 1.0.4
- dio 4.0.6 [http_parser path]
- file_picker 4.4.0 [flutter flutter_web_plugins flutter_plugin_android_lifecycle plugin_platform_interface ffi path]
- firebase_analytics 9.1.1 [firebase_analytics_platform_interface firebase_analytics_web firebase_core firebase_core_platform_interface flutter]
- firebase_auth 3.3.8 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 1.12.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_dynamic_links 3.0.2 [firebase_core flutter]
- firebase_messaging 10.0.9 [firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web flutter meta]
- flutter 0.0.0 [characters collection material_color_utilities meta typed_data vector_math sky_engine]
- flutter_app_badger 1.4.0 [flutter]
- flutter_inappwebview 5.4.3+7 [flutter]
- flutter_local_notifications 9.5.3+1 [clock flutter flutter_local_notifications_linux flutter_local_notifications_platform_interface timezone]
- flutter_localizations 0.0.0 [flutter intl characters clock collection material_color_utilities meta path typed_data vector_math]
- flutter_markdown 0.6.9+1 [flutter markdown meta path]
- flutter_screenutil 5.5.3+2 [flutter]
- flutter_segment 3.9.0 [flutter flutter_web_plugins meta]
- flutter_sim_country_code 0.1.2 [flutter]
- flutter_slidable 1.3.0 [flutter]
- flutter_typeahead 3.2.7 [flutter flutter_keyboard_visibility]
- fluttertoast 8.0.9 [flutter flutter_web_plugins]
- freshchat_sdk 0.9.7 [flutter]
- google_fonts 2.3.0 [flutter http path_provider crypto]
- google_sign_in 5.3.1 [flutter google_sign_in_android google_sign_in_ios google_sign_in_platform_interface google_sign_in_web]
- image_picker 0.8.5+3 [flutter image_picker_android image_picker_for_web image_picker_ios image_picker_platform_interface]
- intl_phone_field 3.1.0 [flutter]
- jovial_svg 1.1.2 [xml args quiver vector_math jovial_misc http meta flutter]
- local_auth 1.1.11 [flutter flutter_plugin_android_lifecycle intl platform]
- lottie 1.2.2 [archive flutter path vector_math]
- markdown 4.0.1 [args charcode meta]
- package_info 2.0.2 [flutter]
- permission_handler 8.3.0 [flutter meta permission_handler_platform_interface]
- pinput 1.2.2 [flutter]
- provider 6.0.3 [collection flutter nested]
- reactive_forms 10.7.0 [flutter intl]
- sendbird_sdk 3.1.12 [flutter collection uuid json_annotation encrypt shared_preferences http logger mime connectivity_plus meta http_parser web_socket_channel universal_io]
- sentry_flutter 6.5.1 [flutter flutter_web_plugins sentry package_info_plus meta]
- shared_preferences 2.0.15 [flutter shared_preferences_android shared_preferences_ios shared_preferences_linux shared_preferences_macos shared_preferences_platform_interface shared_preferences_web shared_preferences_windows]
- shimmer 2.0.0 [flutter]
- sign_in_with_apple 3.3.0 [flutter meta sign_in_with_apple_platform_interface sign_in_with_apple_web]
- snapping_sheet 3.1.0 [flutter]
- sqfentity 2.2.0+6 [flutter sqflite_sqlcipher synchronized path sqfentity_gen sqflite_common sqflite_common_ffi meta]
- sqfentity_gen 2.2.0+12 [build source_gen meta analyzer]
- story_view 0.13.2 [flutter flutter_cache_manager rxdart video_player collection]
- syncfusion_flutter_charts 19.4.56 [flutter intl vector_math syncfusion_flutter_core]
- syncfusion_flutter_treemap 19.4.56-beta [flutter syncfusion_flutter_core]
- timelines 0.1.0 [flutter]
- timer_count_down 2.2.1 [flutter]
- uni_links 0.5.1 [flutter uni_links_platform_interface uni_links_web]
- url_launcher 6.1.2 [flutter url_launcher_android url_launcher_ios url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_launcher_windows]
- uuid 3.0.6 [crypto]

dev dependencies:
- build_runner 2.1.11 [args async analyzer build build_config build_daemon build_resolvers build_runner_core code_builder collection crypto dart_style frontend_server_client glob graphs http_multi_server io js logging meta mime package_config path pool pub_semver pubspec_parse shelf shelf_web_socket stack_trace stream_transform timing watcher web_socket_channel yaml]
- flutter_launcher_icons 0.9.2 [args image path yaml]
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher material_color_utilities meta source_span stream_channel string_scanner term_glyph typed_data]

transitive dependencies:
- _fe_analyzer_shared 31.0.0 [meta]
- analyzer 2.8.0 [_fe_analyzer_shared cli_util collection convert crypto glob meta package_config path pub_semver source_span watcher yaml]
- archive 3.3.0 [crypto path]
- args 2.3.1
- asn1lib 1.1.0
- async 2.8.2 [collection meta]
- boolean_selector 2.1.0 [source_span string_scanner]
- build 2.3.0 [analyzer async convert crypto glob logging meta path]
- build_config 1.0.0 [checked_yaml json_annotation path pubspec_parse yaml]
- build_daemon 3.1.0 [built_collection built_value http_multi_server logging path pool shelf shelf_web_socket stream_transform watcher web_socket_channel]
- build_resolvers 2.0.6 [analyzer async build crypto graphs logging path package_config pool pub_semver stream_transform yaml]
- build_runner_core 7.2.3 [async build build_config build_resolvers collection convert crypto glob graphs json_annotation logging meta path package_config pool timing watcher yaml]
- built_collection 5.1.1
- built_value 8.3.2 [built_collection collection fixnum meta]
- cached_network_image_platform_interface 1.0.0 [flutter flutter_cache_manager]
- cached_network_image_web 1.0.1 [flutter flutter_cache_manager cached_network_image_platform_interface]
- characters 1.2.0
- charcode 1.3.1
- checked_yaml 2.0.1 [json_annotation source_span yaml]
- cli_util 0.3.5 [meta path]
- clock 1.1.0
- code_builder 4.1.0 [built_collection built_value collection matcher meta]
- collection 1.15.0
- connectivity_plus_linux 1.2.0 [flutter connectivity_plus_platform_interface meta nm]
- connectivity_plus_macos 1.2.2 [connectivity_plus_platform_interface flutter]
- connectivity_plus_platform_interface 1.2.1 [flutter meta plugin_platform_interface]
- connectivity_plus_web 1.2.1 [connectivity_plus_platform_interface flutter_web_plugins flutter]
- connectivity_plus_windows 1.2.1 [connectivity_plus_platform_interface flutter]
- convert 3.0.2 [typed_data]
- cross_file 0.3.3+1 [js meta]
- csslib 0.17.2 [source_span]
- dart_style 2.2.1 [analyzer args path pub_semver source_span]
- dbus 0.6.8 [args ffi meta xml]
- encrypt 5.0.1 [args asn1lib clock collection crypto pointycastle]
- fake_async 1.2.0 [clock collection]
- ffi 1.2.1
- file 6.1.2 [meta path]
- firebase_analytics_platform_interface 3.1.0 [firebase_core flutter meta plugin_platform_interface]
- firebase_analytics_web 0.4.0+7 [firebase_analytics_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- firebase_auth_platform_interface 6.2.0 [firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 3.3.8 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core_platform_interface 4.2.4 [collection flutter meta plugin_platform_interface]
- firebase_core_web 1.6.0 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_messaging_platform_interface 3.2.0 [firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_web 2.2.8 [firebase_core firebase_core_web firebase_messaging_platform_interface flutter flutter_web_plugins js meta]
- fixnum 1.0.1
- flutter_blurhash 0.6.4 [flutter]
- flutter_cache_manager 3.3.0 [clock collection file flutter http path path_provider pedantic rxdart sqflite uuid]
- flutter_keyboard_visibility 5.2.0 [meta flutter_keyboard_visibility_platform_interface flutter_keyboard_visibility_web flutter]
- flutter_keyboard_visibility_platform_interface 2.0.0 [flutter meta plugin_platform_interface]
- flutter_keyboard_visibility_web 2.0.0 [flutter_keyboard_visibility_platform_interface flutter_web_plugins flutter]
- flutter_local_notifications_linux 0.4.1+1 [flutter flutter_local_notifications_platform_interface dbus path xdg_directories]
- flutter_local_notifications_platform_interface 5.0.0 [flutter plugin_platform_interface]
- flutter_plugin_android_lifecycle 2.0.6 [flutter]
- flutter_web_plugins 0.0.0 [flutter js characters collection material_color_utilities meta typed_data vector_math]
- frontend_server_client 2.1.3 [async path]
- glob 2.0.2 [async collection file path string_scanner]
- google_sign_in_android 5.2.5 [flutter google_sign_in_platform_interface]
- google_sign_in_ios 5.2.5 [flutter google_sign_in_platform_interface]
- google_sign_in_platform_interface 2.1.3 [flutter quiver]
- google_sign_in_web 0.10.0+5 [flutter flutter_web_plugins google_sign_in_platform_interface js]
- graphs 2.1.0 [collection]
- html 0.15.0 [csslib source_span]
- http 0.13.4 [async http_parser meta path]
- http_multi_server 3.2.0 [async]
- http_parser 4.0.1 [collection source_span string_scanner typed_data]
- image 3.1.3 [archive meta xml]
- image_picker_android 0.8.4+13 [flutter flutter_plugin_android_lifecycle image_picker_platform_interface]
- image_picker_for_web 2.1.8 [flutter flutter_web_plugins image_picker_platform_interface]
- image_picker_ios 0.8.4+11 [flutter image_picker_platform_interface]
- image_picker_platform_interface 2.5.0 [cross_file flutter http plugin_platform_interface]
- intl 0.17.0 [clock path]
- io 1.0.3 [meta path string_scanner]
- jovial_misc 0.8.4 [pointycastle intl async collection convert meta]
- js 0.6.3
- json_annotation 4.5.0 [meta]
- logger 1.1.0
- logging 1.0.2
- matcher 0.12.11 [stack_trace]
- material_color_utilities 0.1.3
- meta 1.7.0
- mime 1.0.2
- nested 1.0.0 [flutter]
- nm 0.4.4 [dbus]
- octo_image 1.0.2 [flutter flutter_blurhash]
- package_config 2.0.2 [path]
- package_info_plus 1.4.2 [flutter package_info_plus_platform_interface package_info_plus_linux package_info_plus_macos package_info_plus_windows package_info_plus_web]
- package_info_plus_linux 1.0.5 [package_info_plus_platform_interface flutter path]
- package_info_plus_macos 1.3.0 [flutter]
- package_info_plus_platform_interface 1.0.2 [flutter meta plugin_platform_interface]
- package_info_plus_web 1.0.5 [flutter flutter_web_plugins http meta package_info_plus_platform_interface]
- package_info_plus_windows 1.0.5 [package_info_plus_platform_interface ffi flutter win32]
- path 1.8.0
- path_provider 2.0.10 [flutter path_provider_android path_provider_ios path_provider_linux path_provider_macos path_provider_platform_interface path_provider_windows]
- path_provider_android 2.0.11 [flutter path_provider_platform_interface]
- path_provider_ios 2.0.7 [flutter path_provider_platform_interface]
- path_provider_linux 2.1.6 [ffi flutter path path_provider_platform_interface xdg_directories]
- path_provider_macos 2.0.6 [flutter path_provider_platform_interface]
- path_provider_platform_interface 2.0.4 [flutter platform plugin_platform_interface]
- path_provider_windows 2.0.6 [ffi flutter path path_provider_platform_interface win32]
- pedantic 1.11.1
- permission_handler_platform_interface 3.7.0 [flutter meta plugin_platform_interface]
- petitparser 4.4.0 [meta]
- platform 3.1.0
- plugin_platform_interface 2.1.2 [meta]
- pointycastle 3.6.0 [collection convert js]
- pool 1.5.0 [async stack_trace]
- process 4.2.4 [file path platform]
- pub_semver 2.1.1 [collection meta]
- pubspec_parse 1.2.0 [checked_yaml collection json_annotation pub_semver yaml]
- quiver 3.1.0 [matcher]
- rxdart 0.27.4
- sentry 6.5.1 [http meta stack_trace uuid]
- shared_preferences_android 2.0.10 [flutter shared_preferences_platform_interface]
- shared_preferences_ios 2.0.9 [flutter shared_preferences_platform_interface]
- shared_preferences_linux 2.1.1 [file flutter path path_provider_linux path_provider_platform_interface shared_preferences_platform_interface]
- shared_preferences_macos 2.0.4 [flutter shared_preferences_platform_interface]
- shared_preferences_platform_interface 2.0.0 [flutter]
- shared_preferences_web 2.0.4 [flutter flutter_web_plugins shared_preferences_platform_interface]
- shared_preferences_windows 2.1.1 [file flutter path path_provider_platform_interface path_provider_windows shared_preferences_platform_interface]
- shelf 1.2.0 [async collection http_parser path stack_trace stream_channel]
- shelf_web_socket 1.0.1 [shelf stream_channel web_socket_channel]
- sign_in_with_apple_platform_interface 1.0.0 [flutter plugin_platform_interface meta]
- sign_in_with_apple_web 1.0.1 [flutter flutter_web_plugins sign_in_with_apple_platform_interface js]
- sky_engine 0.0.99
- source_gen 1.2.2 [analyzer async build dart_style glob meta path source_span yaml]
- source_span 1.8.1 [collection path term_glyph]
- sqflite 2.0.2 [flutter sqflite_common path]
- sqflite_common 2.0.1+1 [synchronized path meta]
- sqflite_common_ffi 2.1.0+2 [sqlite3 sqflite_common synchronized path meta]
- sqflite_sqlcipher 2.1.1 [flutter synchronized path sqflite_common]
- sqlite3 1.5.1 [collection ffi meta]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- stream_transform 2.0.0
- string_scanner 1.1.0 [charcode source_span]
- syncfusion_flutter_core 19.4.56 [vector_math flutter]
- synchronized 3.0.0+2
- term_glyph 1.2.0
- test_api 0.4.8 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph matcher]
- timezone 0.8.0 [path]
- timing 1.0.0 [json_annotation]
- typed_data 1.3.0 [collection]
- uni_links_platform_interface 1.0.0 [flutter plugin_platform_interface]
- uni_links_web 0.1.0 [flutter flutter_web_plugins uni_links_platform_interface]
- universal_io 2.0.4 [collection crypto meta typed_data]
- url_launcher_android 6.0.14 [flutter url_launcher_platform_interface]
- url_launcher_ios 6.0.14 [flutter url_launcher_platform_interface]
- url_launcher_linux 3.0.1 [flutter url_launcher_platform_interface]
- url_launcher_macos 3.0.1 [flutter url_launcher_platform_interface]
- url_launcher_platform_interface 2.0.5 [flutter plugin_platform_interface]
- url_launcher_web 2.0.7 [flutter flutter_web_plugins url_launcher_platform_interface]
- url_launcher_windows 3.0.1 [flutter url_launcher_platform_interface]
- vector_math 2.1.1
- video_player 2.2.17 [flutter video_player_platform_interface video_player_web html]
- video_player_platform_interface 5.1.1 [flutter plugin_platform_interface]
- video_player_web 2.0.10 [flutter flutter_web_plugins video_player_platform_interface]
- watcher 1.0.1 [async path]
- web_socket_channel 2.2.0 [async crypto stream_channel]
- win32 2.3.6 [ffi]
- xdg_directories 0.2.0+1 [meta path process]
- xml 5.3.1 [collection meta petitparser]
- yaml 3.1.1 [collection source_span string_scanner]

@Dinesh-DK
Have you created SHA1 key with debug key info and adding the same to Firebase project ?
Also, try to activate  Android device verification from the console and make sure the SHA1 key is provided there.
The error you are getting seems to be occuring due to missing configuration rather than a bug in firebase_auth.
Also, check this SO link and see if it helps.
Closing from here, as, the plugin is correctly catching the exception and throwing missing-client-identifier, that means something is amiss from your integration point of view.


multiple vaults via hierra file

This is not a bug - just a feature request if possible :)
allow to specify multiple vaults via hierra yaml file.

Hey @marcelmitto ,
Would you be able to describe in detail how you would expect this to work along with how you would expect this to look like in the hiera.yaml file?

A small example from my side. Or Puppet server now search in 5 keyvaults but this gives a lot of underlying 404's. As a workarround it is possible to include the name of the keyvault in the search/secret name. But due to our company policy it is not possible to include the keyvault name in the secret name. It would therefore be ideal if the keyvault name is stripped during the search for the secret. Another nice solution is also welcome.
---
lookup_options:
  corp1:
    lookup_key: azure_key_vault::lookup
    options:
      vault_name: 'CORP1'
      vault_api_version: '2016-10-01'
      metadata_api_version: '2018-04-02'
      key_replacement_token: '-'
      confine_to_keys:
        - '^corp1_.^user_unix_hash$'
  corp2:
    lookup_key: azure_key_vault::lookup
    options:
      vault_name: 'CORP2'
      vault_api_version: '2016-10-01'
      metadata_api_version: '2018-04-02'
      key_replacement_token: '-'
      confine_to_keys:
        - '^corp2_.^user_unix_hash$'

corp1_user1_unix_hash: "%{alias('azure_key_vault::lookup')}"
corp2_user1_unix_hash: "%{alias('azure_key_vault::lookup')}"

Keyvault secrets:
  corp1:
    user1_unix_hash (secret)
  corp2:
    user2_unix_hash (secret)

Search for the users without the keyvault name in the key name.


NPCLib cannot run in two different plugins

Describe the bug
[LeakAuctions] Task #52306 for plugin generated an exception
26.04 12:35:52 [Server] INFO java.lang.LinkageError: net/jitse/npclib/NPCLib
NPCLib JAR name
E.g. npclib-plugin-2.7.jar
Server version
1.15.2
To Reproduce
run two plugins using NPClib and see the error after trying to spawn both of them
Expected behavior
The Library will be managed to run on two or more separated plugin

Task #1674 for  generated an exception
26.04 14:10:09 [Server] INFO java.lang.LinkageError: loader (instance of  org/bukkit/plugin/java/PluginClassLoader): attempted  duplicate class definition for name: "net/jitse/npclib/NPCLib"
26.04 14:10:09 [Server] INFO at java.lang.ClassLoader.defineClass1(Native Method) ~[?:1.8.0_242]
26.04 14:10:09 [Server] INFO at java.lang.ClassLoader.defineClass(ClassLoader.java:757) ~[?:1.8.0_242]
26.04 14:10:09 [Server] INFO at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:142) ~[?:1.8.0_242]
26.04 14:10:09 [Server] INFO at org.bukkit.plugin.java.PluginClassLoader.findClass(PluginClassLoader.java:163) ~[Spigot-1.15.2.jar:git-Spigot-a03b1fd-fc318cc]
26.04 14:10:09 [Server] INFO at org.bukkit.plugin.java.JavaPluginLoader.getClassByName(JavaPluginLoader.java:202) ~[Spigot-1.15.2.jar:git-Spigot-a03b1fd-fc318cc]
26.04 14:10:09 [Server] INFO at org.bukkit.plugin.java.PluginClassLoader.findClass(PluginClassLoader.java:107) ~[Spigot-1.15.2.jar:git-Spigot-a03b1fd-fc318cc]
26.04 14:10:09 [Server] INFO at org.bukkit.plugin.java.PluginClassLoader.findClass(PluginClassLoader.java:96) ~[Spigot-1.15.2.jar:git-Spigot-a03b1fd-fc318cc]
26.04 14:10:09 [Server] INFO at java.lang.ClassLoader.loadClass(ClassLoader.java:419) ~[?:1.8.0_242]
26.04 14:10:09 [Server] INFO at java.lang.ClassLoader.loadClass(ClassLoader.java:352) ~[?:1.8.0_242]

Hi, thanks for creating this issue ticket!
I never intended NPCLib to be used by multiple plugins at the same time. The easiest workaround right now is to strip the library from your plugin and run the NPCLib plugin (which loads all classes for you) in your server.
In the meantime I'll see if I can get the system to figure out if the classes are already loaded into the JVM.

Another easy way you could resolve this, is by repacking the code of NPCLib and port it to a different package. Therefore it can't get duplicate classes.
Still looking for a way to register NPCLib as a service, similar to ProtocolLib.

Yep, I found a way to make it work. But it'll require some work on my end. For now I recommend one of the upper solutions. I'll publish the real solution in a major release which I'll start working on.

I'll add the solution to the README on the main page as well.
Closing issue. Ticket may be reopened at any time.

https://github.com/MinecraftLibraries/NPCLib/blob/master/README.md#repacking-the-library

Thanks for the suggestions!
and a fast response!

Not a problem! Best of luck with your plugin. :)


Sort By Date Missing from Sort Dropdown

The option to sort the query by date is missing from the dropdown box. Typing "sort:created" still works.
Also, a minor thing related to this, can it be changed so the default sort is descending (newest -> oldest)? Currently, it is ascending (oldest -> newest), but I think it makes more sense to see the newest combos first.

Solved in next release (already available for preview)


[$250] Trip - Trip details RHP opens instead of not here page when opening invalid trip link

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

Version Number: 9.0.78-1
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp https://github.com/Expensify/App/pull/52823
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Device used: Windows 11
App Component: Other
Action Performed:

Go to staging.new.expensify.com
Go to any report.
Paste the following link https://staging.new.expensify.com/r/6732929228344392/trip/2905990518128580779/0
Click on the link.

Expected Result:
Not here page will open.
Actual Result:
Trip details RHP opens with Modify Trip and Trip support button.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[x] iOS: Standalone
[x] iOS: HybridApp
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/063a62d9-bcfd-4483-a1b4-c97105acaa37
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021871712471050888781
Upwork Job ID:<PHONE_NUMBER>050888781
Last Price Increase: 2024-12-25

Issue OwnerCurrent Issue Owner: @s77rt

This feature is locked behind a beta, so there's no need to block deploy on it. @blazejkustra please fix this since you worked on the PR introducing trip details.

Thanks for triaging @jasperhuangg ! Been working all day on a couple other DBs 🙏

You can't access the trip details page unless you're on the beta so there's no need to block deploy on this.

Oh, I missed this comment: https://github.com/Expensify/App/issues/54537#issuecomment-2561403522
Looks like this is going to be fixed by @blazejkustra , so no need for proposals.

Still waiting for input from @blazejkustra


Feature Request: AWS Integration

It looks like the Datadog API supports configuring the AWS integration. It would be nice if this was updated to provide support for that.

Thanks for requesting. With the other integrations already there as examples, I think it's pretty straight-forward to do this. As such, I'll tag it as a good first issue.

Fixed in #162


Padrino doesn't respect Net::HTTP's set_debug_output($stderr) setting

I'm trying to debug my Net::HTTP calls but I can't get it to show up in the console output. I tried http.set_debug_output(Padrino.logger), but that doesn't seem to do it either.
Any tips? Or is this a real bug?

Can you show us a sample of how you use net/http with debug logging? Heres a example, running in the padrino console. It definitely logs to the Padrino logger:
require 'net/http'
uri = URI('http://google.com')
Net::HTTP.start(uri.host, uri.port) do |http|
     http.set_debug_output Padrino.logger
     request = Net::HTTP::Get.new uri.request_uri
     response = http.request request
end

#=> Conn keep-alive

It does log to the log stream directly without formatting though, as net/http uses "<<", which works like that in Padrino::Logger.

 Closing.

No problem, we're happy to assist.


Fix Typo: passwort → password

Hi,
I just a very simple PR to fix a typo (passwort → password).
I saw it while starting QLever UI using qlever ui.

Thanks for spotting and fixing that typo :+1:


opensmtpd: 6.6.4p1 -> 6.7.0p1

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/opensmtpd/versions.
meta.description for opensmtpd is: "A free implementation of the server-side SMTP protocol as defined by\nRFC 5321, with some additional standard extensions\n"
meta.homepage for opensmtpd is: "https://www.opensmtpd.org/"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 6.7.0p1 with grep in /nix/store/14730j4q4561w3x9lzf6h8v3wyk6ijck-opensmtpd-6.7.0p1

directory tree listing: https://gist.github.com/2c08105d7e930a754eb906d703af18f1

du listing: https://gist.github.com/1519b178bd0a84e7cb5b452d54607cd8

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
opensmtpd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/14730j4q4561w3x9lzf6h8v3wyk6ijck-opensmtpd-6.7.0p1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A opensmtpd https://github.com/r-ryantm/nixpkgs/archive/cd79923c142152b7c068c26f54865e1e9e11a673.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/14730j4q4561w3x9lzf6h8v3wyk6ijck-opensmtpd-6.7.0p1
ls -la /nix/store/14730j4q4561w3x9lzf6h8v3wyk6ijck-opensmtpd-6.7.0p1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- opensmtpd

Maintainer pings
cc @rickynils @obadz @ekleog for testing.

Seeing the changelog, I'm not sure whether this should be backported — I've asked #opensmtpd to know whether the security vulnerabilities listed in the middle of the bugfixes are qualified or just preventive fixes; if they're just preventive I'd say the backport breakage risk is probably too high, but if there's a security vulnerability then there's no choice but to do the backport.

I've received confirmation that the vulnerabilities were also fixed in 6.6.4p1, so there's no need for backport :)

After this update error work smtpctl:
smtpctl: this program must be setgid smtpq

Hmm… that looks weird, it wasn't in the release notes, and making it setgid smtpq sounds like it'd allow every user on the system to run it, which we definitely don't want to do. I've just asked on the opensmtpd IRC whether that's expected as well as the correct way forward on this matter :)

I've just asked on the opensmtpd IRC whether that's expected as well as the correct way forward on this matter :)

Have you got any answer?

There is an issue for this in the archlinux bug tracker.

Indeed, sorry for having forgotten to report here. yesterday, I got the answer that the setgid is currently required for offline enqueuing, and unfortunately smtpctl is currently not split from sendmail so there's little running around it.
Meaning that we'll have to add a setgid wrapper for smtpctl, I'll try to get to it when I get some spare time, but if someone beats me to it (and I'd love someone to beat me to it :p), feel free to ping me for review :)


[Android] Fix first element changing place with background image

Related Issue
Fixes #3869
Description
This is just a cherry-pick from this https://github.com/microsoft/AdaptiveCards/pull/3393
Microsoft Reviewers: Open in CodeFlow

@almedina-ms go ahead and publish this please. Any version updates needed for Android at all (custom.props already updated to 1.2.8) ? Once done, we can then go ahead publish the 1.2.8 release on https://github.com/microsoft/AdaptiveCards/releases. Thanks


xen-dom-mgmt: domU isn't able to load without passthrough node

Some of DomU may have empty or deleted passthrough node, but 'copy_node_by_path' doesn't handle it.

I am fine with the diff. But please, align the commit message with the standard.
https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes
Pay attention to the "imperative mood" part.

done

added ack

@lorc, can you please re-check a new commit message?

added ack


Incorrect union type inferring when properties match

TypeScript Version:  3.4.4 and<EMAIL_ADDRESS>Search Terms:
Code
type someType = {
  name: 'first',
  first: string,
} | {
  name: 'second',
  second: string,
}

function someFunction(arg1: someType, arg2: someType) {
  if (arg1.name === arg2.name) {
    if (arg1.name === 'first') {
      // Here compiler shows an error
      console.log(arg2.first)
    }
  }
}

Expected behavior:
Since arg1 and arg2 have same names property first should also exist on arg2
Actual behavior:
Property 'first' does not exist on type 'someType'.
Property 'first' does not exist on type '{ name: "second"; second: string; }'.ts(2339)
Playground Link:
https://www.typescriptlang.org/play/#src=type someType %3D {
  name%3A 'first'%2C
  first%3A string%2C
} | {
  name%3A 'second'%2C
  second%3A string%2C
}

function someFunction(arg1%3A someType%2C arg2%3A someType) {
  if (arg1.name %3D%3D%3D arg2.name) {
    if (arg1.name %3D%3D%3D 'first') {
      %2F%2F Here compiler shows an error
      console.log(arg2.first)
    }
  }
}

Duplicate of #12184, I think.

if (arg1.name === 'first') {

This can only narrow arg1.  TS only narrows variables directly involved in the conditional expression and doesn't track relationships with other variables.


Address PyYAML vulnerability

See https://github.com/performancecopilot/ansible-pcp/security/dependabot/ansible_pytest_extra_requirements.txt/PyYAML/open.

Hmm, I recall this issue (and associated fix) being considered incompatible with something else, but for the life of me I cannot recall details now.  I notice the linux-system-roles projects have dismissed this alert though.
@richm @nhosoi do you remember all the details of this ... ?  Thanks!
https://github.com/linux-system-roles/metrics/security/dependabot/ansible_pytest_extra_requirements.txt/PyYAML/closed

Hmm, I recall this issue (and associated fix) being considered incompatible with something else, but for the life of me I cannot recall details now. I notice the linux-system-roles projects have dismissed this alert though.
@richm @nhosoi do you remember all the details of this ... ? Thanks! https://github.com/linux-system-roles/metrics/security/dependabot/ansible_pytest_extra_requirements.txt/PyYAML/closed

Do you support EL6 or other platforms that use python 2.6?  If not, then you can get rid of the line with python_version < "2.7" - otherwise - we dismissed errors like this in system roles because that code was only used in unit testing for python modules.

Why not just merge the commit as is and move on?  Is there some harm to it?

Why not just merge the commit as is and move on? Is there some harm to it?

No, then you can follow up with removing the line or removing the file entirely if it is not needed i.e. if you just copied/pasted this from system roles, and you don't need to do unit testing, then you can just remove the file (and remove the reference to it in pytest_extra_requirements.txt) with a subsequent PR.

Do you support EL6 or other platforms that use python 2.6? If

We do support EL6 in these roles.  However, the roles have no python code at all - I suspect we can remove this line entirely (it was indeed cargo culted from the system roles template repo, from memory).  Let me try that... hmm, and probably we don't need the 'idna' line above it either right?  (idna -> https://pypi.org/project/idna/ I guess)

Do you support EL6 or other platforms that use python 2.6? If

We do support EL6 in these roles. However, the roles have no python code at all - I suspect we can remove this line entirely (it was indeed cargo culted from the system roles template repo, from memory). Let me try that... hmm, and probably we don't need the 'idna' line above it either right? (idna -> https://pypi.org/project/idna/ I guess)

You don't need any of the *_extra_requirements.txt files then.  You can just make them 0 length files.


Update pq-crypto's readme with instructions on how to add another KEM

Issue # (if available):
https://github.com/awslabs/s2n/issues/904
Description of changes:
This covers the high level steps involved with getting a new PQ KEM add to s2n for hybrid key exchanges.
This depends on changes in https://github.com/awslabs/s2n/pull/1070 and https://github.com/awslabs/s2n/pull/1084.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (pq_readme@773ec8e). Click here to learn what that means.
The diff coverage is n/a.


Fix syntax error

There is a missing part of init function.

fwiw, this problem was introduced in commit c956f98: https://github.com/richardpenman/whois/commit/c956f98478b9572db6ef8778e75dfcca644080c5


Use $store for store and store for value in docs

Seems like we need to start using this convention in docs.

Fixed edce30c


jpeg:sampling-factor option not honored

According to the changelog, the jpeg:sampling-factor option is supported since v0.3.4:
https://github.com/dahlia/wand/blob/master/docs/changes.rst#version-034
However, it does not seem to be taken into account. The following program generates the same output file as the convert command without the sampling-factor option:
from wand.image import Image
with Image(filename="page-1.png") as img:
    img.format = "jpeg"
    img.compression_quality = 70
    img.options["jpeg:sampling-factor"] = "4:4:4"
    img.save(filename="page-1-wand.jpg")

This program produces a 174kb file, equal to the one created by the following command:
convert -quality 70 page-1.png page-1.70.jpg

Whereas the following command with the sampling-factor option produces a 208kb file of much better quality:
convert -quality 70 -sampling-factor 4:4:4 page-1.png page-1.70-444.jpg

I know, that this issue is almost 2 years old, but I've struggled with this recently. It turns out, that C method for setting options (MagickSetOption) doesn't  work with "jpeg:sampling-factor", and MagickSetImageProperty should be used. Maybe it's only problem with old libraries, but update wasn't an option in my case.
You can use it directly in your code:
library.MagickSetImageProperty(img.wand, 'jpeg:sampling-factor', '4:4:4')

This has been addressed in #369, and should be part of the next release.
with Image(filename='input.png') as img:
    img.artifacts["jpeg:sampling-factor"] = "4:4:4"
    img.compression_quality = 70
    img.save(filename="output.jpg")


[LakeFSOutputCommitter] Support all writing modes

The relevant modes are:

ErrorIfExists (default)
Append
Overwrite
Ignore

tl;dr: not fast enough, and we shut down the project.


Fix ToVariableName

Fixes https://github.com/Azure/autorest.csharp/issues/1937
Description
Previously the ToVariableName function or the ToCleanName function with the camelCase=false will only change the first letter to lower case, make the letter after a separator (such as _) upper case, do not do anything else. We found some issues in that implementation that if we have the correct casing in our properties, such like OSProfile, this function will only give us the result oSProfile because it does not identify the full first word.
This PR changes this function to make sure it could convert all letters in the first word to lower case, where the "first word" refers to the word before the last upper case letter or before a separator, whichever comes first
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

/azp run autorest.csharp - ci

/azp run license/cla


Rethink Automatic Niconico Mylist Generator

At the request stage, only register.
EC2 instance status is managed in a new table.
Auto-registration runs periodically in batches.
EC2 instances stop in another batch.

[ ] Update Diagram to include the Mylist Create.


use safe cast in propagate_constraints

Which issue does this PR close?

Closes #11252
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

seems like a arrow-rs related problem, but we can use safe cast to avoid it temporarily

Thanks @Lordworms 🙏


[C-1463] [C-1462] CTA to search member name on LinkedIn

requested by Kestra
Problem
When we encounter this error or when the GitHub profile doesn't display an associated LinkedIn account, we have to manually search for it using the name on LinkedIn, and often we are able to find it.
Solution
Could you please enable a pre-search on LinkedIn within the member enrichment feature, so that we can save time?
→ Jonathan: e.g. for my name: https://www.linkedin.com/search/results/all/?keywords=jonathan reimer I think thats a great idea.
From SyncLinear.com | C-1462
From SyncLinear.com | C-1462

@nuno any idea how that could look like? Perhaps another item in the member profile dropdown? "Search Member on LinkedIn"?


anti magnetic element

added element that runs away when trying to be interacted with

Excellent, thank you for this!


As a cif user, I want to cancel changes to a project, so that I can undo my work

Description:
A user should be able to revert any changes they have made while a project revision is in draft.
Reverting committed changes should not be possible. Since those changes were approved by an admin and committed, a new revision should be done at that point.
The user should be able to revert all changes related to the current revision.
Acceptance Criteria:
Given that I have made changes to a project revision
When I want to undo my changes
Then I can revert the changes related to the current revision.
Development Checklist:

[x] The form_change and project_revision tables have delete mutations to populate the deleted_at / deleted_by columns in the database
[x] The project_revision delete mutation should cascade to set the deleted_at / deleted_by columns of all related form_change records
[x] A Discard Changes button is shown on the edit project page as per the wireframe
[x] https://github.com/graphile-contrib/pg-omit-archived is used to automatically omit deleted records
[ ] Clicking the Discard Changes button shows a list of all changes related to the current revision to be discarded
[x] Confirming the discard reverts the changes using the mutations described above
[x] Wireframe completed and linked to ticket
[x] Meets DOD

Personas:

Program Development Team
Operations Team
Tech Team

Notes:

Adding applicable Personas to the ticket
Program Development Team - CIF Persona
Operations Team - Persona
Tech Team - CIF Persona

see buttons at the bottom of the wireframe

@dleard I think that the description as-is is a bit advanced for an MVP. I'd only have a single Discard Changes button, as shown in Raj's wireframe. If there's a need we can add some more granular controls later

Thank you @pbastia ! I tested in Dev by:

Entering some project details then selecting the 'discard changes button'.
After selecting button, user is brought back to the project lists table.
My discarded project was not included in list (as expected) and the 'Add a Project' button displayed (vs. resume project draft).

I did notice that when I discarded the changes, I did not get a list of what was being discarded, but my assumption is this will be a new user story to be done during the new design.
Good work!

That's right, we just redesigned how this page is going to look like after this morning's discussion, but the functionality remains the exact same, so 100% this code is going to be reused there


Two questions about QField features/functionality - automatisation and sync with DB

Hi,
could you help me with these questions:
a) is it possible to send changes you made in the QField to the QFieldCloud automatically? Button "Push changes" requires action from the user - is it possible to create a scheduler or install some plugin which would push changes automatically on preset time, say once per day?
b) If we use PostgreSQL DB for storing layers in QFieldCloud, how to push changes from the DB to the project on mobile device? Option "Synchronize" upload the whole layer/project - we just need to get small updates made in the DB.
Thanks.

Regarding automatic push of changes - is it?
https://docs.qfield.org/get-started/tutorials/get-started-qfc/#enabling-automatic-pushing-of-changes-to-qfieldcloud

Hey @DanasKaz .

As for b), this is not supported yet.
We have b) in the backlog of issues with no immediate plans for development. If you are interested in speeding up such development, please drop us an email at sales [at] qfield.cloud.

Hi,
thanks for reply.
Regarding b) - so the only way to get the changes made in DB on QField is to sync the whole project/layer, isn't it?
Thanks.

You can always make a direct connection from QField to your PG layers, but you lose the offline editing. Other than that, QField always gets a full offline copy of your PG layers.


rkt: Import trusted pub keys for ACI verification.

In order to enable the ACI signature verification in rkt, we need to provide the rkt runtime a list of trusted public keys. I am thinking maybe we can do this during the creation of the runtime?
For example, provide a flag to specify a trusted_keys.json file, which looks like:
{
    {
        "prefix": "coreos.com/etcd",
        "pubkey": "PATH_TO_PUBKEY",
        "root": false
    },
    {
        "prefix": "fluentd.org/fluentd",
        "pubkey": "PATH_TO_PUBKEY",
        "root": false
    },
    ...
}

/cc @dchen1107 @vmarmol @bakins @philips

Or maybe we can use the Secret to pass the pubkeys and prefix, then let kubelet/rkt to write those keys in the /etc/rkt/trustedkeys/

That would preclude use of images for file based pods when the master is
unavailable (during bootstrap or a failure).  Needs to be per-node at
least, and maybe also per container.
@pmorie because you wrote #7627
This is sort of like a pull secret, but not so secret, since it is a public
key.
@deads2k because you wrote  #7894
@lavalamp because you are reviewing  #7894
On Fri, May 15, 2015 at 5:09 PM, Yifan Gu<EMAIL_ADDRESS>wrote:

Or maybe we can use the Secret to pass the pubkeys and prefix, then let
kubelet/rkt to write those keys in the /etc/rkt/trustedkeys/
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/8351#issuecomment-102552173
.

Yes, it sounds like this could be another type of pull secret. (see also #8517)

Is the trusted_keys.json based on an existing format or is it a representation you're working on now?  If it's the latter, then I think it would be useful to allow the file to contain the public key itself, similar to how we do with .kube/config files today.  That makes it very easy to ensure that the information is fully portable.
Since this information isn't used to authenticate the kubelet to the repository, I think I like the idea of having this be per-node.  The public keys will be the same for every image pull attempt against the particular prefix and it's not privileged information.

Also by now this is a one-time setup per node, we only need to receive the pubkeys once and write it down on the config file. So I think I am not very worried when master is unreachable.

/cc @jonboulle

After a little bit more thought and playing with secrets, I think I am going to take the original approach, which provides the trusted_keys.json. Because it's easier to let rkt do the trust during the bootstrapping.

This hasn't been touched in a while, but I'm interested in getting this in.  Yifan: is your PR still reasonable to get merged?  If you aren't planning on actually getting it merged in I will work on this issue.

@runningwild I am not able to go back to this in a week or two. Feel free to take the work :)


Boolean properties on iOS

Hello,
I noticed that only on iOS if I use boolean properties then they are mapped to numbers on MP, so for example, if I set a bool property "is used: true" then on MP I see "is used: 1" from iOS

hi @rMozes , thanks for bringing it up. It's been fixed in v1.2.1. https://github.com/mixpanel/mixpanel-flutter/releases/tag/v1.2.1


Docker support

Hey guys!
I tried to pack the project into docker to make it easy to deploy and faced some problems, maybe some one could help here. Changes published to docker branch
Requirement
Make it easy to deploy Identity Server Admin to use it in different apps as a part of micro service architecture. I saw somewhere in the feed that @skoruba planned to pack the project into NuGet but I think docker another way to make it portable
Solution
Pack the project (Admin and STS) into docker images and run them at any time with required port inside your cluster
Implementation
I added two Dockerfiles in target project folders, docker-compose.yml in root plus several powershell scripts in the folder with same name. It works but made dummy. To start run containers - run 'run-docker.ps1' script or execute commands from it one by one (for non-Windows platforms).
SQL server is also packed into an image and run as a part of compose with persistent external volume (skoruba-identityserver-sql). All three containers (SQL, STS, Admin) work within same docker network (skoruba-identityserver-network) and can interact with each other
Note

Admin project port 80 is mapped to host's port 5001 instead of 9000
MS SQL container's port 1433 is mapped to host's 14333! This made for compatibility with MS SQL instance on the host

Problems

Dependency projects are not packed as NuGet and cannot be restored with 'dotnet restore'. Because of that instead of using single multi step Dockerfile I build it on host then just copy to aspnetcore-runtime image. This can be solved with modified Dockerfile which copies 'bin' folder with dependencies but prebuild on host is required anyway. Also projects use 'npm install' which is not preinstalled into default aspnetcore-sdk build image.

As dotnet CLI calls required before 'docker build' they're packed into build-docker-*.ps1 scripts

I've added DockerRelease config file with overriden parameters but they don't apply so duplicated in docker-compose.yml and passed with 'environment'
While STS and Admin containers can use short docker-compose.yml service names when reference to sql container successfully I can't access to sts/.well-known/openid-configuration from within Admin project
Don't know how to apply EF migrations from within container. Before building docker image DbContext reference to host DB, after running a container not sure how to do that. Maybe something like 'EXEC dotnet ef database update'? Need to try this way
Both projects are configured to run in 'Development' environment because certificate for HTTPS is not set up

Accidentally I pushed changes into branch in my fork. I guess there should be a way to cherry pick them here?

Hi @NTurakulov
Thank you for your effort to run this admin in docker. I don't have any experience with docker yet.
How can I help you with better preparing this project to running in docker?
Thanks!

@Seaear - add support for docker - #132 - please take a look.
Thanks!

So it looks like this functionality was merged. Should the main readme be updated to check off the Docker Support box and add instructions to run the project with Docker?

For sure, I am working on it.

Please let me know if you would like any help on that.

Hi, I'm trying to run a demo from source using docker compose on Visual Studio but I can't login because login on database failed.
Isn't this supposed to work without changing configuration?
If not maybe I can help to complement the documentation for that.

So far I can see that there is connectivity between the 3 docker instances and that the login settings all match up.

Well, I tried again today and without any change all is working!


Introduce onClick hover/modal

So far TGVE has been relying on hover. Having both options will be useful to variety of projects.

So, here's a summary of my understanding of first steps, following the discussion on 16/3/22:

Create a new component, called "Popup", modelled closely on "Tooltip"
Make a new function in the "Welcome" component, based on the _renderTooltip function, that is called from onClick and sets the state variable "popup" (null if the object from onClick is undefined, or the JSX for the Popup component if there is an object).
Add {popup} in Welcome's render function (as is done for tooltip).
Think about what css class to use for the popup, to ensure that it appears on top of the map.
... (edit this comment as things come to mind)


Drop current database (select other database automatically)

When trying to delete the currently active PostgreSQL database, an error message appears:

Can't delete object 'foodb'
Ursache:
Cannot drop the currently open database.
Switch to another database and try again
(Note: enable 'Show non-default databases' option to see them).

This message indicates DBeaver knows what to do, so it should to it itself instead of asking the user to do so.

This persists in 21.1.0 (although the error message now starts with "Cant delete object {0}").
At least, if DBeaver requests the user to switch to another database, DBeaver should explain how that can be done.

still actual against 21.2.4


Ensure that a block of lines starting with "pattern" and ending with "pattern" contains a given block of text

From https://tickets.opscode.com/browse/CHEF-3290.
Start trying to implement ideas from https://github.com/chef/chef/issues/7043

If you're at the community summit again this year We should probably get together and talk about patterns etc :+1:

I'll get in late  Monday.  I'll be there for the hack day on Friday.

Fixed by #101. Waiting review and merge. Many of the ideas and capabilities proposed by CHEF-3920 have been implemented.


Look into using JWT encryption

In order to help hide the claims text, consider using asymmetric encryption such as HMAC.

The api would have the private key to sign a new "encrypted JWT"
The app would have the public key to decrypt it. Verification that the JWT is valid from the API is already possible due to the JWT signature.
Perhaps use a new set of keys for each user or time period.

Note that "encrypted JWT" is not really how this should be called.

No longer relevant. Closed before repository gets archived.