Adding a new operation to get a list of VMImages in an edgezone by Ed…

…gezone
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that label "WaitForARMFeedback" will be added automatically to begin ARM API Review. Failure to comply may result in delays to the manifest.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[x] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@kjilla also add another word to customWords for this https://github.com/Azure/azure-rest-api-specs/pull/18980/checks?check_run_id=6357424872
also please resolve https://github.com/Azure/azure-rest-api-specs/pull/18980/checks?check_run_id=6357423990

@kjilla03 lastly, please provide here why you are adding to an existing version/why you missed the train for the existing version release and why you can't wait until the next release

/azp run

@kjilla03 lastly, please provide here why you are adding to an existing version/why you missed the train for the existing version release and why you can't wait until the next release

We cant wait for the next release because the marketplace is going to use this api to filter the images and this got delayed because ARM team suggested some changes and I was busy with other high pri tasks. We absolutely need this change to go in this release.

@kjilla also add another word to customWords for this https://github.com/Azure/azure-rest-api-specs/pull/18980/checks?check_run_id=6357424872
also please resolve https://github.com/Azure/azure-rest-api-specs/pull/18980/checks?check_run_id=6357423990

Fixed the case mismatch

@kjilla03 lastly, please provide here why you are adding to an existing version/why you missed the train for the existing version release and why you can't wait until the next release

We cant wait for the next release because the marketplace is going to use this api to filter the images and this got delayed because ARM team suggested some changes and I was busy with other high pri tasks. We absolutely need this change to go in this release.

@JeffreyRichter, this is Kusuma's breaking change (adding to existing version) justification. Could you review/approve?

@ArcturusZhang could you check on the PoliCheck?

Any update on this
From: Theodore Chang @.>
Sent: Monday, May 16, 2022 03:26 AM
To: Azure/azure-rest-api-specs @.>
Cc: Kusuma Jilla @.>; Mention @.>
Subject: Re: [Azure/azure-rest-api-specs] Adding a new operation to get a list of VMImages in an edgezone by Ed… (PR #18980)
@ArcturusZhanghttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FArcturusZhang&data=05|01|Kusuma.Jilla%40microsoft.com|3741839639a6475d490908da3726707a|72f988bf86f141af91ab2d7cd011db47|1|0|637882935488886121|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=8uDCIUy14ZY3Ot3At%2BSJN%2FMdI5myne1FmyDp4DoIdfE%3D&reserved=0 could you check on the PoliCheck?
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F18980%23issuecomment-1127495458&data=05|01|Kusuma.Jilla%40microsoft.com|3741839639a6475d490908da3726707a|72f988bf86f141af91ab2d7cd011db47|1|0|637882935488886121|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=PJ7W2oxnnHBUTEW%2FSnxSXjhYtzFe4xXszCrUZhis%2F9w%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAHVQH52CZRDHW6XMQ6YVLZTVKIPCTANCNFSM5VPDB6XA&data=05|01|Kusuma.Jilla%40microsoft.com|3741839639a6475d490908da3726707a|72f988bf86f141af91ab2d7cd011db47|1|0|637882935488886121|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=PoqZvmVE1Y5%2B6OuhNPxyDWEZfLLV8qHllak9XAe4fvI%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@ArcturusZhang could you check on the PoliCheck?

@ArcturusZhang could you check on the PoliCheck?


Document what happens when using the resource plugin w/ hotrod

We should add some details to the demo in https://github.com/signadot/hotrod/tree/main/resource-plugins/mariadb so it's clear what the plugin does in this case.

This is not relevant anymore


Create index.html

Create an index.html file and insert the provided HTML snippet into it. The snippet includes a Unicode code for an emoji that displays "Hello World!" with a smiling face. Ensure the file is properly structured with the correct HTML5 doctype, charset, and title.

Test


SysGnuWin issues introduced in #271

In reviewing pull request #271, which was merged over the weekend, I found a few issues in changes that were made to SysGnuWin.f90:

gfortran's FSTAT documentation says its arguments need to be INTEGER(4), but they were changed to default integers using INTEGER (see lines 80-81). This means that the size of the integer now depends on compiling options. Is there a reason this was changed?

The make_command on line 223 now creates a directory called -p and will have issues if windows path names include '/' in their path names. I had fixed this syntax in a previous pull request (#194), but it's been overwritten with linux syntax again.

I made these changes to that sys file in order to fix some bugs with CygWin, but the eventual fix was to use the linux file for CygWin. I'll look again at all the changes that were made to SysGnuWin.f90 as much of that might actually have been unnecessary.

It looks like the SysGunLin.f90 file should also use INTEGER(4) for FSTAT. The Intel version(s) of the function can take default integers.

In wondering why this didn't cause a problem sooner, I found that this function (FileSize) is only used in the nwtclib tests. Am I missing it somewhere?
In any case, does this line handle the integer size issues you pointed out?
   Status = FSTAT( INT( Unit, B4Ki ), StatArray )

No, that doesn't fix the issue. The StatArray needs to be INTEGER(4) in gfortran: https://gcc.gnu.org/onlinedocs//gfortran/FSTAT.html#FSTAT. That documentation also says Unit is a default integer, so I am not sure why Unit would be converted to INTEGER(4) in the SysGnuLin version of the routine.
I think this routine was called in some of the other NWTC code that that Marshall wrote that isn't in the OpenFAST repository. NWTC_Library was part of a lot of different codes.
You would only notice a problem if you were building with default integers set to be INTEGER(8), which you would probably never do. But if you're going to be changing the data types in a routine, it should follow the documentation.

closed by #279


ensemble error on aws ec2

Hi Brad,
I am trying to run platinum sample on aws installation and I am having issues with ensemble. Can you please hepl?
subprocess.CalledProcessError: Command 'set -o pipefail; export PATH=/mnt/bcbio/anaconda/bin:$PATH &&  /mnt/bcbio/anaconda/bin/bcbio-variation-recall ensemble --cores=4 --numpass 1 --names gatk-haplotype,freebayes,vardict --nofiltered /mnt/testrun/work/bcbiotx/tmpxAkvAs/NA12877-ensemble.vcf.gz /mnt/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa /mnt/testrun/work/gatk-haplotype/NA12877-effects-annotated-filterSNP-filterINDEL-noeff-nomultiallelic.vcf.gz /mnt/testrun/work/freebayes/NA12877-effects-annotated-filter-noeff-nomultiallelic.vcf.gz /mnt/testrun/work/vardict/NA12877-effects-annotated-noeff-nomultiallelic.vcf.gz
2018-10-02 21:21:06 ip-172-31-31-90 INFO [bcbio.run.itx] - Note: -w option not given, printing list of sites...
2018-10-02 21:21:09 ip-172-31-31-90 ERROR [bcbio.variation.recall.main] -
java.lang.Exception: Problem retrieving reference variant for {:chr "2", :start 127413614, :refa "AG", :alta ["."], :end 127413616, :vc-indices (0)}: []
bcbio.variation.ensemble.intersect/get-rep-vc/fn       intersect.clj:   58
clojure.core/comp/fn            core.clj: 2438
clojure.core/map/fn            core.clj: 2624
clojure.lang.LazySeq.sval        LazySeq.java:   40
clojure.lang.LazySeq.seq        LazySeq.java:   49
clojure.lang.RT.seq             RT.java:  507
clojure.core/seq            core.clj:  137
clojure.core/map/fn            core.clj: 2616
clojure.lang.LazySeq.sval        LazySeq.java:   40
clojure.lang.LazySeq.seq        LazySeq.java:   49
clojure.lang.Cons.next           Cons.java:   39
clojure.lang.RT.next             RT.java:  674
clojure.core/next            core.clj:   64
bcbio.variation.variantcontext/write-vcf-w-template  variantcontext.clj:  189
clojure.lang.RestFn.invoke         RestFn.java:  573
bcbio.variation.ensemble.intersect/ensemble-vcfs/fn       intersect.clj:   88
bcbio.variation.ensemble.intersect/ensemble-vcfs       intersect.clj:   86
bcbio.variation.ensemble.intersect/-main       intersect.clj:  140
clojure.lang.RestFn.applyTo         RestFn.java:  137
clojure.core/apply            core.clj:  630
bcbio.variation.recall.main/-main/fn            main.clj:   34
bcbio.variation.recall.main/-main            main.clj:   33
clojure.lang.RestFn.applyTo         RestFn.java:  137
bcbio.variation.recall.main.main                    :
' returned non-zero exit status 1

Running below returned empty (no results).
[ec2-user@ip-172-31-31-90 work]$ tabix ./gatk-haplotype/NA12877-effects-annotated-filterSNP-filterINDEL-noeff-nomultiallelic.vcf.gz chr2:127413614-127413619
[ec2-user@ip-172-31-31-90 work]$ tabix /mnt/testrun/work/freebayes/NA12877-effects-annotated-filter-noeff-nomultiallelic.vcf.gz chr2:127413614-127413619
[ec2-user@ip-172-31-31-90 work]$ tabix /mnt/testrun/work/vardict/NA12877-effects-annotated-noeff-nomultiallelic.vcf.gz chr2:127413614-127413619

Diya;
Thanks for the detailed report and apologies about the issue. I'm not positive what is happening here, the ensemble method should be getting calls to look up from the input files and it's strange there are no variants in any file in those regions. My guess is that there's some unexpected/problematic variant in one of those files that we're not correctly dealing with.
My best suggestion would be to try expanding your tabix query to see if you can pick up any variants that are triggering the issue. I think if you decrease the start of these you'll pick up a deletion in one of the callers and we might have more information to work with. Thanks so much for the help debugging.

Diya;
Ivan reported a similar issue on the biovalidation mailing list and provided some example problem calls that appear to demonstrate the issue (https://groups.google.com/forum/#!msg/biovalidation/LcYsAkWebIs/tVywrb7DBAAJ).It appears as if MuTect2 is outputting some
problematic calls: reference only calls without an ALT allele. Since these
overlap the VarDict called variants it causes a lookup failure. I pushed a fix
to remove these calls prior to doing the ensemble call, which should resolve
the issue. If you update with:
bcbio_nextgen.py upgrade -u development

remove the problem ensemble directory (rm -rf ensemble) and re-run, it
should re-normalize the inputs and hopefully work cleanly for you. Let us know if
you run into any other issues.


Metaverse light wallet issue

Metaverse light wallet at www.myetpwallet.com/ is legal.

Please allow ~10minutes for caches to refresh.


Evaluation processing time

Hello
I'm testing the evaluation stage, but after downloading the checkpoint for Torch, it waits here and does nothing?
Is there a problem or should I really wait?

How much time is needed to do the evaluation?

I had similar issue during evaluation. You can try reducing number of nodes by change torchpack -np 8 to torchpack -np 1 It works for me.
So it appears that the number of nodes must match the number of GPUs.

thanks so much


Some UI features requirements

Metrics Service

[x]  UI should scale better. (#1499)
[ ]  Allow switch charts to show current metrics and historical metrics (#1411)
[ ] Provide summary path information for any two points selected in the DAG (#875)

Usability

[ ]  Allow user to submit a storm jar by UI. (#1648)
[ ]  Allow user to submit multiple jars when composing the DAG. (#1450)
[x]  Allow user to change the transition time when changing the parallelism. (#1682)
[ ]  Allow user to submit config when replacing a processor. (#1366)
[x]  Textual Changes: (#1691)

Change "Receive Throughput Skew" to "Task Receive throughput", or some better
Still show the above graph if only one task is present.
Change "Task Selector" to "Pick one task to see its detail metrics..."or some better
Summary page, use (s): 2 processor -> processors ,  executor -> executors
Add a document link at the right corner. Along with the source code link.

AppMaster

[ ]  Able to see history job status.
[ ]  Show error status in Application page. (#1681)

Security

[ ]  Authentication/authorization.
[ ] Add a security configuration options to disable all HTTP POST, DELETE, and PUT, so that we can host gearpump demo on public server.

Cluster managerment

[ ] Allow admin to add/remove a worker, add/remove a master.

Some minor suggestions.

Could we have configs shown in a three-column table (name, value, origin) ?
On the submit DAG console, submit is placed to the left of cancel. Is it more user friendly (more common) to place submit right to cancel (just like the Close and comment and Comment buttons here).
On dynamic change, do we show which executors are alive and which are not ?

@manuzhang I've 2 comments on your suggestions:

Akka's configuration is HConf format. Currently there is no frontend tool to highlight such format. If backend can convert it into a nested structure as you described, we can update the configuration viewer as you expected.
I followed the classic UI design language, that "Cancel" button is at right. The reason is that submission can be complex, you need to input many things before click "Submit" button. But if you want to close a dialog quick, the more intuitive position is always the most right side.

about no.2. I don't know about the design language but I've seen @huafengw added a jar file and then quickly closed the console although he meant to submit. It's a minor issue though.

@stanleyxu2005
Please refer to the "Iteration" section of Config API
Config.root.unwrapped

Recursively unwraps the object, returning a map from String to whatever plain Java values are unwrapped from the object's values.

@manuzhang
I disagree with these two columns key-value style display. I think tree-like display style looks better.

Actually it is key, value, default value, description. If it config can be retrieved in this way, yes. we can provide a read-only configuration viewer.

Add two new requirement:

Add a security configuration options to disable all HTTP POST, DELETE, and PUT, so that we can host gearpump demo on public server.
Allow admin to add/remove a worker, add/remove a master.

If I understand new feature 1 correctly, that disabling particular operations means to have a guest account that can only view cluster information.

All features here have been migrated to Apache Jira as below.

feature
Apache Jira

Able to see history job status
https://issues.apache.org/jira/browse/GEARPUMP-92

Allow admin to add/remove a worker, add/remove a master
https://issues.apache.org/jira/browse/GEARPUMP-93

Could we have configs shown in a three-column table (name, value, origin) and allow users to search ?
https://issues.apache.org/jira/browse/GEARPUMP-94


[Weighted Shard Routing] Disallow search requests with preference parameter with strict weighted shard routing

Is your feature request related to a problem? Please describe.
With weighted shard routing, requests are distributed across data nodes in the ratio the weights are assigned. Since search requests with preference parameter runs the request only on the specified nodes, this won't honour weighted shard routing.
ie for a 3 az domain with weights (1, 2, 4) preference searches on nodes with weight 1 can cause more request on nodes with weight 1.
Describe the solution you'd like
Allow support for strict weighted shard routing using cluster setting.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@anshu1106 Are you actively working on this? can I assign it to you?

@anshu1106 Are you actively working on this? can I assign it to you?

Yes


SAML2 client not working if IdP is in different time zone

Turns out the function that detemines whether a SAML2 authentication request is still valid does not take time zones into consideration.
SAML2DefaultResponseValidator.isDateValid() returns false if the IdP is in a different time zone and leads to an invalid authentication request.

as workaround until this is fixed, set the setMaximumAuthenticationLifetime to 24h:
((SAML2DefaultResponseValidator) saml2Client.getResponseValidator()).setMaximumAuthenticationLifetime(24 * 3600 * 1000);
it would probably be nice to expose that setter method on the interface as well in future...

Thanks for the report. Looks like the time received is handled via joda time which should consider time zones, but the sp time is handled by system milliseconds which does not. I think both time instances should be based on jodatime to mitigate the problem.

Thanks for reporting and the workaround. Would you mind proposing some PR to mitigate the problem?

I'll put together a PR for this issue today.

Thanks.


Live Deployments

This started off as #125 but should also cover #126 as it was quicker to do both at the same time.
Deployments are now automatically created for every branch at
$branch.corewar.shonk.software
see: deployment for this branch
As soon as this is in main, deployments should also be deleted automatically for any branches that are deleted.

It's available for me?

Seems to be on my end


HIVE-26800 : Backport HIVE-21462 to branch-3: Upgrading SQL server backed metastore when changing data type

JIRA link : https://issues.apache.org/jira/browse/HIVE-26800
This Backport was already done in branch-3.1 using this HIVE-21755 so we need to backport the same to branch-3.

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@zabetak @abstractdog Can you please review and merge this


SVG download - too jumpy

This is just personal preference, but I don't like that the page refreshes when I click on the SVG button. I understand why this is the case.
but I don't like it.

since #71 the actual download of the SVG is outside viewer,
as far as i know there is no purely client side solution that works on desktop macs, see https://github.com/MICommunity/interaction-viewer/issues/71#issuecomment-157691145
the viewer just provides the SVG when asked: https://github.com/MICommunity/interaction-viewer/blob/master/src/controller/Controller.js#L1094-L1102
Max has added the following code to Complex Portal for download:
function exportSVG() {
    var xml = xlv.getSVG();
    var xmlAsUrl = 'data:image/svg;filename=xiNET-output.svg,';
    xmlAsUrl += encodeURIComponent(xml);
    window.open(xmlAsUrl, 'xiNET-output.svg');
}

I think changes to this land on the Complex Portal issue list,
closing...


adding docker support

awesome :) will look at this a little later

I suggest to add a volume also for mongo so you won't lose the data if you restart zenbot, example ./data/ and put it in gitignore

@grigio good point, added separate container for mongodb-data.
p.s. you won't lose data as long as you don't remove mongodb container

moved Dockerfile to main directory
added environment check for docker
added more comments

@egorbenko cool, I hope it will be merged, Is there a reason to check be in the in docker?
To me this is enough
c.mongo_url = "mongodb://" + (process.env.MONGODB_PORT_27017_TCP_ADDR || "localhost")+ ":27017/zenbrain"

instead of
https://github.com/egorbenko/zenbot/blob/6a7e34124db0480fd539540b5ec3c2b8f9a91cbb/config_sample.js#L6-L9

@grigio that works too :), thank you for the comments

FYI, on my docker image i've made some launch scripts so that the user can decide exactly which components to run and what parameters to provide.  I'm not sure if you think it will be valuable here:
https://github.com/BarnumD/docker-zenbot

@BarnumD you can already do that with docker-compose.
 docker-compose up mongodb reducer btc-usd server

Personally I like more docker-compose because it is a more standard way to deal with multiple processes in docker.
Another way with less overhead is with GNU parallel, a single command for multiple and connected processes
❯ parallel --ungroup ::: "ping google.com" "ping github.com"                                               ⏎
PING google.com (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: icmp_seq=0 ttl=56 time=18.626 ms
PING github.com (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: icmp_seq=0 ttl=53 time=111.576 ms
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=56 time=20.574 ms
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=53 time=111.746 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=56 time=20.605 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=53 time=111.267 ms

I didn't test it inside a docker container but it should work.

You should pin node to at least a major version.


UI: Convert DataOptions and RuntimeOptions steps to use generic FormComponent

If the generic FormComponent in PR #75 is acceptable, the DataOptionsForm and RuntimeOptionsForm should be adjusted to use FormComponent instead of the switching logic that is currently implemented.

PR #85 is pending

Closing - PR was merged.


[Bug]: User can add multiple host address/ports for the same datasource

Is there an existing issue for this?

[X] I have searched the existing issues

Description
In the create/edit datasource form, the user can add multiple host address/ports for a single datasource.
Since a given datasource can only have a single address + port combination, having the option to enter multiple values is not correct.
Steps To Reproduce

Go to the create datasource form/
Next to the host address and port fields, you will see a "+" icon. When you click on this icon, you can add another set of address and port fields. You can keep doing this indefinitely.

Public Sample App
No response
Issue video log
No response
Version
Cloud

This is actually a valid use case where a user might have multiple replicas of their database.
In cases where one of the replicas is not reachable, Appsmith will try and connect to other replicas which the user has specified.


DebuggerWatch all closed/open by default feature request.

Sometimes I get a ton of variables being expanded in my DebuggerWatch window. And of those variables I'd like to only see or watch a few.   Is there a option to automatically close all expanded variable trees by default? If not could we add one please?
Otherwise, we'll be trying to implement hacky things like this:
https://stackoverflow.com/questions/25495869/vdebug-expand-all-variable-trees/42860032#42860032
Which I almost got working.
But it would be great to have it as an option.
Or even better, is there a way to just watch singular variables?

This is available in the latest versions of vdebug :VdebugTrace


import.meta not supported inside wxt.config.ts file

Describe the bug
Try using import.meta.dirname inside wxt.config.ts file.
✖ Command failed after 96 ms                                                                                5:14:37 PM

 ERROR  Cannot use 'import.meta' outside a module                                                                        5:14:37 PM

  import.meta.dirname;
  ^^^^

  SyntaxError: Cannot use 'import.meta' outside a module
  at new Script (node:vm:99:7)
  at createScript (node:vm:255:10)
  at Object.runInThisContext (node:vm:303:10)

Reproduction
Use import.meta.dirname inside wxt.config.ts file.
Steps to reproduce
No response
System Info
System:
    OS: macOS 14.6.1
    CPU: (8) arm64 Apple M1
    Memory: 77.20 MB / 8.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.11.1 - ~/.proto/shims/node
    Yarn: 1.22.22 - ~/.proto/shims/yarn
    npm: 10.2.4 - ~/.proto/shims/npm
    pnpm: 9.6.0 - ~/.proto/shims/pnpm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 127.0.6533.120
    Chrome Canary: 130.0.6669.0
    Edge: 127.0.2651.105
    Safari: 17.6

Used Package Manager
npm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of #297. The file is loaded as this weird CJS/ESM hybrid, and it doesn't support all ESM features, like import.meta.dirname.
Use __dirname instead.


Prepare for AI UK

Summary
A 25 min research talk on participatory methods. Georgia will introduce what participatory methods are and their importance, then two usecases will be presented:

Connected by Data will explain their work
AutSPACEs contributor (remote) will talk about their experience as a participant, why they chose to participate and keep doing it and what it has meant to him. Georgia will make 3 brief questions to prompt the answers and bring a more conversational tone to the talk.

0-5'
Intro to Participatory methods

5 - 15
Connected by Data

15 - 25'
AutSPACEs

Deadline
Our slot is: Tues 19 March at 10:25 - 10:50am on the Future Stage.
What needs to be done?

[x] Invite co-presenters
[ ] Plan session
[ ] Feedback and finalise
[ ] Practice run
[ ] Present

Who can help?
Any suggestions welcome in the comments below.
Updates
David has done a fab outline here: https://hackmd.io/ujfSSW4PQ5ywvhdT-nckgA

might be worth to send a reminder for this one at some point to have enough time to plan the session?

Perfect, thanks! 🙏


Fix issue where upload notification shows up on app start

Description (required)
Fixes #3981
Tests performed (required)

Tested that app doesn't show this notification when there are no pending uploads
when an upload is started, the notification appears

Codecov Report

Merging #4099 (53165c3) into master (7d8e09f) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #4099      +/-   ##
============================================
- Coverage     10.59%   10.59%   -0.01%
  Complexity      476      476
============================================
  Files           343      343
  Lines         12887    12896       +9
  Branches       1001     1001
============================================
  Hits           1366     1366
- Misses        11453    11462       +9
  Partials         68       68

Impacted Files
Coverage Δ
Complexity Δ

...ree/nrw/commons/contributions/ContributionDao.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...java/fr/free/nrw/commons/upload/UploadService.java
0.81% <0.00%> (-0.04%)
1.00 <0.00> (ø)

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 7d8e09f...53165c3. Read the comment docs.


@conda-forge-admin, please update circle

@conda-forge-admin, please update circle.
ref: https://circleci.com/gh/conda-forge/cquadpack-feedstock/7

@conda-forge-admin, please update circle.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I updated the circle-ci deploy key and followed the project.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I updated the circle-ci deploy key and followed the project.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I updated the circle-ci deploy key and followed the project.

Here's the new build.
ref: https://circleci.com/gh/conda-forge/cquadpack-feedstock/11


added support of RTF body

Status

[ ] In Progress
[ ] Ready
[x] In Hold - (Reason for hold)

Related Issues
fixes: https://github.com/demisto/etc/issues/26951
Description
A few sentences describing the overall goals of the pull request's commits.
Screenshots
Paste here any images that will help the reviewer
Must have

[ ] Tests
[ ] Documentation

@moishce Are we reverting this?


Initial Rails helpers

[WIP] @enderahmetyurt and I paired on some initial helpers for Rails today, though I’m not entirely sure how we should best include them.

I don't want Phlex to be dependent on Rails, all the Rails integration should be optional.
I’m not sure if we should have a Railtie or Rails Engine, but we'll likely want to extend the application with some controller methods and custom helpers.

render_page
The idea is that this method would be available in a rails controller so if you used a consistent naming convention for your page components, you could render the correct page automatically.
class ArticlesController < ApplicationController
  def index
    render_page articles: Article.all
  end
end

component
The component method is designed to be used in ERB templates so you can render a Phlex component from your existing Rails views.

@joeldrapper, maybe we could have two different gems:

phlex could contain the logic for creating components. This gem would be independent of any framework.
phlex-rails would contain the Rails-specific modules and the Engine/Railtie.

It could be just like factory_bot and factory_bot_rails. What do you think?

@alexandreruban that's a really good option. The alternative is to have a rails.rb file that can be optionally required. I think you could just do this in the Gemfile.
gem "phlex", require: "phlex/rails"

I think it would be easier to extract a phlex-rails gem later than it would be to merge a separate gem back into the main one, so my instinct is to see how far we can get with just one gem while always keeping that door open. I don't feel strongly about this though.


Create simple template class. Fix DRY in view and mailer modules

in view:
https://github.com/lotus/view/blob/master/lib/lotus/view/template.rb
in mailer:
https://github.com/lotus/mailer/blob/master/lib/lotus/mailer/template.rb

@davydovanton Thanks for this PR, but lotus-utils shouldn't have any runtime dependency, unless it's for a feature that we want to offer. This proposal just introduces a way to DRY code across frameworks, which doesn't justify the add of this gem.
This framework can be used as inflector, or for service objects from other Ruby apps, which don't need to install tilt for pluralize some strings.
Closing this :wink:

no problem, thanks for review! :blush:


Fix some minor memory leaks

This addresses some minor memory leaks that get picked up by the address sanitizer.
fixes #8263

Having had a look at the conflicts in the development branch it looks like 3715e57 fixes this. I'll close the PR.

Closed by 3715e57


Set anchors on all headers and method names

I'd like to be able to link to specific section in the docs, or a specific method. For example, I'd like to link to the Usage section on the alert path with: http://ionicframework.com/docs/v2/api/components/alert/Alert/#Usage and the link would jump the user directly to Usage section.
Same for methods. I'd like to be able to link to the setTitle method with: http://ionicframework.com/docs/v2/api/components/alert/Alert/#setTitle

Additionally, like github markdown files that automatically create an anchor for each header, the header should have a link icon next to it when the user hovers:

Done!


Animations

Changes:

Updated .NET MAUI and Maui Graphics versions.
Apply animations in drawn controls changes (Example: Toggle the Switch control).

Ripple Effect


Text area focus color does not pass contrast ratio for a11y

Describe the bug
We have an a11y bug reported on our product that the text area focus color does not have enough contrast (2.0 where 3.0 is needed).
The texarea focus color in Clarity v2-v4 is hard coded to #69C0E2. I am wondering why the focus color is brighter and different from the input field (which uses #0072A3). Would make sense to have the text area using the same color as all other controls, e.g. input, checkbox, radio, select.
Further on, I think we should have fixed focus color for buttons too. We have multiple a11y issues reported where users have have hard time seeing button focus (especially primary buttons).
How to reproduce
Steps to reproduce the behavior:
Focus any text area
Focus buttons
Expected behavior
Same focus color as input
Versions
App

Clarity: 2, 3 and 4

Device:

Browser [chrome]
Version [85]

@chlane lets review this together for compliance and strategy.

See our recommendations for visual focus indication for all elements that get focus at https://confluence.eng.vmware.com/display/A11Y/Visual+Focus+Indication

@chlane
While not visible to the open-source community, I've noted deficiencies in that recommendation. We cannot move forward with that.
Once the recommendation is vetted across all use cases and signed off on by design/ux we should be able to implement it. Until then, we're stuck.

My intent in suggesting  we need to review for strategy is as Scott says 'vetted across all use cases' - we have several approaches across the library and given the current color system work this is an excellent opportunity to develop a comprehensive strategy.

Clarity cares deeply about accessibility, and we work with our accessibility team to identify and triage issues. We have an internal tracking system to identify and verify accessibility issues, which has captured this already. Some issues with Clarity Angular are difficult to solve without causing side effects or breaking changes, and we have to balance concerns with fixing issues and breaking changes. As we build out Clarity Core, we are able to address these root cause accessibility issues and recommend adopting the Clarity Core implementations as they become available. We are closing this issue as we have the issues tracked internally. Please follow our development and releases to see when we release relevant Clarity Core components to make this possible.


Smpl chunk: play loops

Hello,
I've come to your library thanks to rodio and cpal. I need to play wave files that contain 'smpl' chunk and play forever the loop stored in this chunk (forever, ie, until the user wants to stop the sound).
I've figured out how to parse this chunk (it's pretty easy) but don't know how to perform the loop. I don't know it you're interested in implementing this kind of features, but can you help me and just say where I should start?
Thanks in advance,
Philippe
NB: my informations about the smpl chunk comes from here: http://www.piclist.com/techref/io/serial/midi/wave.html

Thanks for taking the time to open an issue. I’m not sure I understand your question correctly. Hound itself only reads wave files, it doesn’t handle playback. If you want to loop some audio, you can read the samples into a buffer, and repeatedly send that same buffer to the audio playback library. It looks like the smpl chunk contains some metadata about the audio data that is needed by a sampler if you want to use the audio as the basis of a virtual instrument. Parsing that chunk is something that could be a nice feature for Hound, but to then use that information, for example if you want to transpose the audio data to a certain note, you’ll need a DSP or resampling library.

Thanks for your quick answer. I'm sorry that my question wasn't very clear and indeed she was confusing. When I said 'play', I was thinking 'read'.
The smpl chunk contains at its ends a list of loops, particularly the start and the end of the loop, as two byte offsets. I want my program to read the chunk data but, when it reach the loop endpoint, goes back to the startpoint. My problem is that I don't know which part of the code to change.

I want my program to read the chunk data but, when it reach the loop endpoint, goes back to the startpoint.

I think what you want to do instead, is read the audio data into a buffer, and then do the looping in the piece of code that is sending the samples to your audio playback library; not in the code that is reading the samples. Something like this:
struct Loop {
    samples: Vec<i16>,
    begin: usize,
    end: usize,
    pos: usize,
}

impl Loop {
    pub fn new<R>(reader: WavReader<R>) -> Loop {
        // Read samples here, and read begin/end manually from SMPL chunk.
        // Support for parsing SMPL could possibly be added to Hound.
    }

    // Called periodically by the audio playback system.
    pub fn fill_buffer(&mut self, buffer: &mut [i16]) {
        for x in buffer.iter_mut() {
             *x = self.samples[self.pos];
             self.pos += 1;
             if self.pos >= self.end {
                 self.pos = self.begin;
             }
        }
    }
}

The smpl chunk contains at its ends a list of loops, particularly the start and the end of the loop, as two byte offsets.

Hmm, in order to convert those byte offsets to sample indices, you need to divide by the number of bytes per sample. Usually the number of bytes per sample is the reader.spec().bits_per_sample / 8, but the format does allow for other combinations in some cases. The number of bytes per sample is available in the WavReader internally, but at the moment it is not exposed.

Thanks for your answer. I'll try to modify Hound to implement your suggestion. Thanks again!


Can't show list client on tab service.

Hi @renatoalmeidaoliveira !
I have tried to install nbservice with netbox 3.4.5 on docker environment.
The installation did not give any error but when I declared the service for the nbservice plugin, I had the problem that the client item was not displayed.

Can you explain this to me or show me how to check.
Thank you

Hi @congto, the Clients field is a Multi Select Field filled by Tenants, so you need to create some Tenant before creating a service.

Hi @renatoalmeidaoliveira

I followed the instructions and it worked. Thank you, hope you continue to develop it, I will follow and learn how to use it.


SIGHUP kills process instead of reloading logging.json

https://github.com/EOSIO/eos/pull/6791 was supposed to add the ability to reload the logging.json on a SIGHUP. It appears that SIGHUP signal_set is never setup via start_sighup_handler since start_sighup_handler is never called so SIGHUP kills the process.

Released in 1.8


Dev Resources Cleanup

Pull requests into this repository require the following checks to be completed by the submitter and reviewers.
Changes in this PR:
Added mechanism to the CHPL endpoint list parsers, to the Endpoint Resources List web scrapers, and to the query_endpoint_resources.sh script (for endpoint lists queried using the curl command) to now also automatically save each endpoint list in the dev resources folder, reducing any files that contain more than 10 endpoint entries down to 10 entries. Also updated the dev resources and prod resources folders with the most recent endpoint list data.
Submitter:

[x] Administrative tasks are complete.

This pull request describes why these changes were made.
The JIRA ticket links to this PR.
The JIRA ticket for this PR is: Lantern 537 JIRA Ticket

[x] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

[x] Code review has been performed.

Code diff has been done and checked to ensure only expected code is being committed.
A linter for the language has been run.

Primary Reviewer:

[ ] Code works.

Code has been run locally and the basic functionality affected by the PR has been checked.
Edge cases have been checked to ensure behavior is as expected.

[ ] Code review has been performed.

Code is maintainable, reusable, efficient, and correct.
Code accomplishes the tasks purpose.
Code follows style guidance appropriate for the language, including passing linter checks.
Code is well commented (which does not mean verbosely commented).

[ ] Tests are complete.

Tests are included and test edge cases.
Tests have been run locally and pass.

Merger:
Prior to merging, ensure that all checks have been completed. Delete the branch after the merge
is complete.

also needs a rebase!


chip-tool can't pairing to chip-lighting-app

Problem
chip-tool can't not pairing to chip-lighting-app.
log is below because of invalid endpoint during networking commissioning.
CHIP:EM: ec pos: 1, id: 17768, Delegate: 0xdd6e1b70 CHIP:SVR: Packet received from Node:1b669: 40 bytes CHIP:ZCL: APS frame processing success! CHIP:ZCL: RX len 29, ep 0, clus 0x31 CHIP:ZCL:  FC 1 seq 0 cmd 2 payload[ CHIP:ZCL: 04 63 68 69 70 08 71 77 65 72 61 73 64 66 00 00 00 00 00 00 00 00 70 17 00 00 CHIP:ZCL: ] CHIP:ZCL: Drop cluster 0x31 command 0x2 CHIP:ZCL:  due to invalid endpoint: CHIP:ZCL: 0x0 CHIP:ZCL: Data model processing failure!
Proposed Solution
chip-tool use endpoint 0 for networking commissions .
but example endpoint is 1.
We need to modify to change endpoint 0 to 1.

Can you leave steps to reproduce this issue?

We're in progress of reviewing older issues. Is this still an issue given how old the SDK this was tested against? Please re-open if so. Thanks!


Any chance to use a span insted mark tag

Hi there! The lib works realy well, but I need to highlight some text inside a textarea with React. This is possible? Tnks for read.

Duplicate of #94


Changed Skia OpenGL bitmap origin to bottom left

What does the pull request do?
Changes Skia OpenGL bitmap origin to bottom left, which is the default for OpenGL.
What is the current behavior?
Skia OpenGL bitmap origin is top left, so existing apps have the Y axis flipped.
What is the updated/expected behavior with this PR?
Using OpenGlControlBase renders with bottom left, as expected by anyone using OpenGL.
Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes
It's a behavior breaking change without warning (kinda bad), but should be easy to notice and fix though (multiply Y scale by -1).
Fixed issues
Related discussion: #6952

You can test this PR using the following package version. 0.10.999-cibuild0023034-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0023066-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]


Style/GuardClause false negative

Code
def foo
  if true
    1
  else
    2
  end
end

Expected behavior
The cop should have been triggered to suggest this change
def foo
  return 1 if true

  2
end

This is especially important if the else branch has lots of lines.
Actual behavior
No trigger.
If I remove the else branch then the cop is triggered.
def foo
  if true
    1
  end
end

If I add a return to each branch then the cop is triggered.
def foo
  if true
    return 1
  else
    return 2
  end
end

Steps to reproduce the problem
Code above
RuboCop version
bundle exec rubocop -V
1.34.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.21.0, running on ruby 3.1.1 x86_64-darwin21)

That behavior seems correct. Style/GuardClause cop is offense for conditional branches with if-else if the body of the condition contains return or break or next or raise or fail.
Therefore, the following code is not an offense:
def func
  if something
    work
  else
    test
  end
end

def func
  unless something
    work
  else
    test
  end
end

However, the following code is an offense:
def func
  if something
    work
  else
    test || raise('message')
  end
end

def func
  if something
    work and return
  else
    test
  end
end

Ok. I was just thinking that in this case the if is at the end of the function so both branches have an implicit return.
Also I don't understand why having or not having an else branch should make any difference.
If this is not intended or if you don't think this would be useful then feel free to close the issue.

That's by design indeed - sometimes an if/else just reads better, so we wanted to leave people the option to chose what go for. I recall originally this cop was always enforcing the style you prefer, perhaps we left this as an option (or can add an option). At any rate - I want this cop to be relatively relaxed out-of-the-box.

Btw, I see that the cop examples are clearly incorrect - e.g. we have this very same code listed as something that would be marked as an offense and most of the listings are not wrapped in the defs. We'll need to fix this.


test pr

This PR is to test the workflow. No need to review it. -Nico

The docs build workflow works as expected when triggered by PR from external contributors.


x1211 core—As TS (Katy), I want to add Xenium kit costings to the probe hybridisation xenium operation to reflect and accurately bill for xenium runs

Xenium probe hybridisation operation should have a drop down list (Faculty or SGP) for the whole slide.
Dropdown list to appear between barcode and SGP number columns on table.
Specify a costing per labware.
The column should appear in the probe_hybrid_view
A patch will be need for old data.
Size: medium

View update: https://github.com/sanger/stan-sql/blob/devel/views/x1211_kit_costing.sql


updated FAQ to include workaround for RelatedObjectDoesNotExist exception

Problem
Added documentation to FAQ to clarify workaround for RelatedObjectDoesNotExist exception (see #1556).
Solution
FAQ entry
Acceptance Criteria
N/A

Coverage: 100.0%. Remained the same when pulling fb45c77e04f50ed5aaea3d4b435335097b2345df on matthewhegarty:RelatedObjectError-faq-entry into 54bc71662050a2a57da8fd6e31c8afd87f4dd48f on django-import-export:main.


Does it can search post?

search posts by keyword, return a list of post id, post url, post title

RTFD
-> We won’t be focusing on anything else than chat-related things. This API is called fbCHAT, after all ;)


Tabs role="presentation" isn't a11y compliant

When running a11yAudit(); I am seeing the following issue:
Error: Assertion Failed: The page should have no accessibility violations. Violations:
[serious]: <ul> and <ol> must only directly contain <li>, <script> or <template> elements
Violated 1 time. Offending nodes are:
<ul id="ember553" class="nav nav-tabs nav-justified ember-view">

Here is the "offending" markup.

I asked Mel about this on Discord and she pointed out that the role="presentation" changes the li so that it's not li anymore.
I looked to see if this role was added by bootstrap itself but it doesn't appear to be:
https://getbootstrap.com/docs/4.3/components/navs/#tabs
Then I looked that the ember-bootstrap tabs demo page:
https://www.ember-bootstrap.com/#/components/tabs
And inspected the tab examples which showed:

So it appears that ember-boostrap adds the role="presentation".

Thanks @BitBrit for reporting! This is a relict from the BS3 examples: https://getbootstrap.com/docs/3.4/components/#nav
@jelhan was super quick to provide a PR: #782, see my comment there, and feel free to chime in if you have thoughts!


[stable/Jenkins] Merge Master.SidecarContainers: into Master.Sidecars

Re:
https://github.com/helm/charts/blob/9084ce54adc1cbb4988175935ffbb3d01b57edfe/stable/jenkins/values.yaml#L238
I deliberately added Master.Sidecar in the JCasC PR rather than Master.JCasC.Sidecar, so that other sidecars could be placed here.  Can't see any logic behind having both Master.SidecarContainers and Master.Sidecar.  @torstenwalter , do you have any objection if I move your commented example to Master.Sidecar and then the configAutoReload one will be underneath it?
My PR referred to this: "Hierarchy of sidecar values allows further sidecars to be added for other purposes in future" (I accept this PR became a bit wordy at times!).   For clarity, I'll also rename this key to Master.Sidecars.

No objections

Merging both is a good idea, but we need to pay attention that people know what they are configuring. e.g. .Values.Master.Sidecar.image becomes confusing once there are more sidecars.
This is how the yaml looks at the moment:
  Sidecar:
    image: shadwell/k8s-sidecar:0.0.2
    imagePullPolicy: IfNotPresent
    resources:
    configAutoReload:
      enabled: false
      sshTcpPort: 1044
      label: jenkins_config
      folder: /var/jenkins_home/casc_configs

and in addition the SidecarContainers property I introduced.
What about renaming "Sidecar" to "Sidecars" and moving the configAutoReload settings below that config key. SidecarContainers could become others. So the result would be:
  Sidecar:
    configAutoReload:
      enabled: false
      image: shadwell/k8s-sidecar:0.0.2
      imagePullPolicy: IfNotPresent
      resources:
      sshTcpPort: 1044
      label: jenkins_config
      folder: /var/jenkins_home/casc_configs
    others:

That way it becomes clear which sidecar you are configuring. I could also add another one (in a separate PR) for the GitHub webhook proxy. e.g:
  Sidecar:
    configAutoReload:
      ...
    webhookproxy:
    ....
    others:

others would still provide people with the option to configure whatever other sidecar containers they would like to have,
Issue with renaming would be that it's a breaking change. No clue how you would deal with that in this chart.

All sounds good Torsten.  Yes image, imagePullPolicy, & resources belong under configAutoReload.  Also agreed leaving Sidecar and not renaming it to Sidecars will be less disruptive.

Just seen my mistake. I forgot to rename Sidecar to Sidcars. Corrected that now. I think that's clearer and we face backward incompatibilities anyhow when moving the other properties.
@holmesb Do you want to create the PR or should I? Just want to avoid double work while both of us start working on this independently.
@maorfr Any opinions how to deal with these breaking changes?

These are new features (as you know), and its probably better to do now rather then later.
I say Break away ;)

PR is ready. @holmesb would be great if you could test your part and provide your feedback.
Thanks for pointing out that we ended up with different sidecar config fields.

LGTM @torstenwalter, nice work.  No problems according to my testing.  Thanks for this.


⚠ Spryr has degraded performance

In 1bd2093, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3506 ms

Resolved: Spryr performance has improved in 8a6228b after 10 minutes.


config: better secret detection in bash scripts

Description:
in bash, these assignments are equivalent (below), so the apostrophe and quotation marks should be optional
also, very often passwd variable name is used
PASSWORD="secret"
PASSWORD='secret'
PASSWORD=secret
Checklist:

[ ] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[ ] Have you lint your code locally prior to submission?

The gitleaks config should handle this now


R tensorflow example

This is an example of using a TensorFlow model with a GPU in R.
it uses the new saturn-rstudio-tensorflow GPU image, which won't exist until this PR gets through the system

(cc @forana )


improving fib.cpp to run to a certain limit

Now it can run to a particular  fib number

@A-ee can i join this org?


Added URL to skip test

Task linked: QF-2515 Add invoices admin to QFieldCloud admin

business logic for running test to be changed

Added QFIELDCLOUD_TEST_SKIP_VIEW_ADMIN_URLS and QFIELDCLOUD_TEST_SKIP_SORT_ADMIN_URLS, will allow to extend the configuration of the skipped tests in case the QFieldCloud app is used as dependency.


proxy: add tests for ustats

I think this is good enough? I might've tested changing individual counters but I think if any of them are changeable it's going to be fine.

merged to next

Released in 1.6.20


Use metadeps to specify pkg-config dependencies declaratively

This makes it easier for distribution packaging tools to generate
appropriate package dependencies.

The single appveyor error here seems completely unrelated and pre-existing; it occurs on current master as well. Everything else seems to work fine.

Closing for now as I think the metadeps story is still in the works


Can't train in colab

Hi thank you for sharing
I'm trying to train in colab but a have problem, somebody can help me pls?
I'm sure the folder structure as author.
I have tested in colab the following versions :
CUDA: 11.2
PyTorch 1.6.0
Python 3.8.5
!CUDA_VISIBLE_DEVICES="0" python tools/trainval.py configs/trainval/tinaface/tinaface_r50_fpn_bn.py

vedadet - WARNING - EvalHook is not in modes ['train']
vedadet - INFO - Loading weights from torchvision://resnet50
vedadet - WARNING - The model and loaded state dict do not match exactly

unexpected key in source state_dict: backbone.fc.weight, backbone.fc.bias
missing keys in source state_dict: neck.0.lateral_convs.0.conv.weight, neck.0.lateral_convs.0.bn.weight, neck.0.lateral_convs.0.bn.bias, neck.0.lateral_convs.0.bn.running_mean, neck.0.lateral_convs.0.bn.running_var, neck.0.lateral_convs.1.conv.weight, neck.0.lateral_convs.1.bn.weight, neck.0.lateral_convs.1.bn.bias, neck.0.lateral_convs.1.bn.running_mean, neck.0.lateral_convs.1.bn.running_var, neck.0.lateral_convs.2.conv.weight, neck.0.lateral_convs.2.bn.weight, neck.0.lateral_convs.2.bn.bias, neck.0.lateral_convs.2.bn.running_mean, neck.0.lateral_convs.2.bn.running_var, neck.0.lateral_convs.3.conv.weight, neck.0.lateral_convs.3.bn.weight, neck.0.lateral_convs.3.bn.bias, neck.0.lateral_convs.3.bn.running_mean, neck.0.lateral_convs.3.bn.running_var, neck.0.fpn_convs.0.conv.weight, neck.0.fpn_convs.0.bn.weight, neck.0.fpn_convs.0.bn.bias, neck.0.fpn_convs.0.bn.running_mean, neck.0.fpn_convs.0.bn.running_var, neck.0.fpn_convs.1.conv.weight, neck.0.fpn_convs.1.bn.weight, neck.0.fpn_convs.1.bn.bias, neck.0.fpn_convs.1.bn.running_mean, neck.0.fpn_convs.1.bn.running_var, neck.0.fpn_convs.2.conv.weight, neck.0.fpn_convs.2.bn.weight, neck.0.fpn_convs.2.bn.bias, neck.0.fpn_convs.2.bn.running_mean, neck.0.fpn_convs.2.bn.running_var, neck.0.fpn_convs.3.conv.weight, neck.0.fpn_convs.3.bn.weight, neck.0.fpn_convs.3.bn.bias, neck.0.fpn_convs.3.bn.running_mean, neck.0.fpn_convs.3.bn.running_var, neck.0.fpn_convs.4.conv.weight, neck.0.fpn_convs.4.bn.weight, neck.0.fpn_convs.4.bn.bias, neck.0.fpn_convs.4.bn.running_mean, neck.0.fpn_convs.4.bn.running_var, neck.0.fpn_convs.5.conv.weight, neck.0.fpn_convs.5.bn.weight, neck.0.fpn_convs.5.bn.bias, neck.0.fpn_convs.5.bn.running_mean, neck.0.fpn_convs.5.bn.running_var, neck.1.level_convs.0.0.conv.weight, neck.1.level_convs.0.0.bn.weight, neck.1.level_convs.0.0.bn.bias, neck.1.level_convs.0.0.bn.running_mean, neck.1.level_convs.0.0.bn.running_var, neck.1.level_convs.0.1.conv.weight, neck.1.level_convs.0.1.bn.weight, neck.1.level_convs.0.1.bn.bias, neck.1.level_convs.0.1.bn.running_mean, neck.1.level_convs.0.1.bn.running_var, neck.1.level_convs.0.2.conv.weight, neck.1.level_convs.0.2.bn.weight, neck.1.level_convs.0.2.bn.bias, neck.1.level_convs.0.2.bn.running_mean, neck.1.level_convs.0.2.bn.running_var, neck.1.level_convs.0.3.conv.weight, neck.1.level_convs.0.3.bn.weight, neck.1.level_convs.0.3.bn.bias, neck.1.level_convs.0.3.bn.running_mean, neck.1.level_convs.0.3.bn.running_var, neck.1.level_convs.0.4.conv.weight, neck.1.level_convs.0.4.bn.weight, neck.1.level_convs.0.4.bn.bias, neck.1.level_convs.0.4.bn.running_mean, neck.1.level_convs.0.4.bn.running_var, bbox_head.cls_convs.0.conv.weight, bbox_head.cls_convs.0.bn.weight, bbox_head.cls_convs.0.bn.bias, bbox_head.cls_convs.0.bn.running_mean, bbox_head.cls_convs.0.bn.running_var, bbox_head.cls_convs.1.conv.weight, bbox_head.cls_convs.1.bn.weight, bbox_head.cls_convs.1.bn.bias, bbox_head.cls_convs.1.bn.running_mean, bbox_head.cls_convs.1.bn.running_var, bbox_head.cls_convs.2.conv.weight, bbox_head.cls_convs.2.bn.weight, bbox_head.cls_convs.2.bn.bias, bbox_head.cls_convs.2.bn.running_mean, bbox_head.cls_convs.2.bn.running_var, bbox_head.cls_convs.3.conv.weight, bbox_head.cls_convs.3.bn.weight, bbox_head.cls_convs.3.bn.bias, bbox_head.cls_convs.3.bn.running_mean, bbox_head.cls_convs.3.bn.running_var, bbox_head.reg_convs.0.conv.weight, bbox_head.reg_convs.0.bn.weight, bbox_head.reg_convs.0.bn.bias, bbox_head.reg_convs.0.bn.running_mean, bbox_head.reg_convs.0.bn.running_var, bbox_head.reg_convs.1.conv.weight, bbox_head.reg_convs.1.bn.weight, bbox_head.reg_convs.1.bn.bias, bbox_head.reg_convs.1.bn.running_mean, bbox_head.reg_convs.1.bn.running_var, bbox_head.reg_convs.2.conv.weight, bbox_head.reg_convs.2.bn.weight, bbox_head.reg_convs.2.bn.bias, bbox_head.reg_convs.2.bn.running_mean, bbox_head.reg_convs.2.bn.running_var, bbox_head.reg_convs.3.conv.weight, bbox_head.reg_convs.3.bn.weight, bbox_head.reg_convs.3.bn.bias, bbox_head.reg_convs.3.bn.running_mean, bbox_head.reg_convs.3.bn.running_var, bbox_head.retina_cls.weight, bbox_head.retina_cls.bias, bbox_head.retina_reg.weight, bbox_head.retina_reg.bias, bbox_head.retina_iou.weight, bbox_head.retina_iou.bias
Traceback (most recent call last):
File "tools/trainval.py", line 65, in
main()
File "tools/trainval.py", line 61, in main
trainval(cfg, distributed, logger)
File "/content/drive/My Drive/Colab Notebooks/vedadet/vedadet/assembler/trainval.py", line 86, in trainval
looper.start(cfg.max_epochs)
File "/content/drive/My Drive/Colab Notebooks/vedadet/vedacore/loopers/epoch_based_looper.py", line 29, in start
self.epoch_loop(mode)
File "/content/drive/My Drive/Colab Notebooks/vedadet/vedacore/loopers/epoch_based_looper.py", line 15, in epoch_loop
for idx, data in enumerate(dataloader):
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 359, in iter
return self._get_iterator()
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 305, in _get_iterator
return _MultiProcessingDataLoaderIter(self)
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 944, in init
self._reset(loader, first_iter=True)
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 975, in _reset
self._try_put_index()
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 1209, in _try_put_index
index = self._next_index()
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/dataloader.py", line 512, in _next_index
return next(self._sampler_iter)  # may raise StopIteration
File "/usr/local/lib/python3.7/dist-packages/torch/utils/data/sampler.py", line 226, in iter
for idx in self.sampler:
File "/content/drive/My Drive/Colab Notebooks/vedadet/vedadet/datasets/samplers/group_sampler.py", line 39, in iter
indices = np.concatenate(indices)
File "<array_function internals>", line 6, in concatenate
ValueError: need at least one array to concatenate

I test so there is  one folder image and annotations

self.group_sizes = []
('self.flag', array([], dtype=int64))
in   File "/content/drive/My Drive/Colab Notebooks/vedadet/vedadet/datasets/samplers/group_sampler.py", line 41, in iter


TypeError: Cannot redefine property: when shimming

From https://github.com/aws-observability/aws-otel-lambda/issues/99
It seems that based on certain build systems like esbuild it's possible to create a bundle which doesn't work well with shimmer. Is this something that can be worked around or is inevitable and a symptom of these build systems making the result closer to a type-safe language than a dynamic one?

Looking at the code (https://github.com/othiym23/shimmer/blob/master/index.js#L14), it seems the object is enumerable but not updatable. I think this is a case that should be handled by shimmer but that surely means that instrumentation would not work.


Fix semantics_tester

Description
Semantics_tester was broken.  If a node had multiple children, it was checking the first and then bailing.
Fixing that has revealed other bugs.
I talked with @GaryQian and I think the test_text tests now have the correct fixes applied in paragraph.dart.  Gary to review.
I'm not as confident on the other changes - DatePicker seems like it may be right now, but not sure why we get all those empty nodes. /cc @goderbauer and/or @darrenaustin we get them because there are empty container placeholders there.
Sliver tests I think are right - they were just missing implicit scrolling flags and not caught previously due to this bug.
Semantics traversal test seems like it may be a different bug.  It's possible the 64 bit rect stuff impacted this and we didn't catch it because of the brokeness of the tester.
This regression was introduced in #16253, where a return true that used to live in a closure was moved to a for loop.
Related Issues
Fixes https://github.com/flutter/flutter/issues/34363
Tests
I added the following tests:
Added a sanity check for the tester.  This test fails without these changes.  This is similar to a test I was trying to write, and I was confused about why it was passing when it was so clearly wrong.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

Good catch! Thanks for fixing!

we get them because there are empty container placeholders there.

What does this mean? Empty containers should not produce a semantics node?

Paragraph changes LGTM!


Missing a queue name upon receiving a task event

Hi there!
Context
I am facing the issue with missing values for queue label for the Celery tasks metrics. Basically, I don't see any other value except the default one: celery

So I tried to run the exporter locally, and here is what I've found:

track_task_event only considers queue property upon receiving a task, but the queue name is not always available in this way ( I assume it's available when a queue name is passed directly to the call).

To improve lookup capabilities of the exporter, I've added another way of getting a queue name which must solve the aforementioned issue.

Can you add a test?


Migrating spring data cosmos from sdk/cosmos to sdk/spring

Description
We are migrating azure-spring-data-cosmos and azure-spring-data-cosmos-test from sdk/cosmos to sdk/spring.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - spring - tests

/azp run java - cosmos - tests

Please update ci.yml and test.yml related contents.

@kushagraThapar do you think we want to do this? Don't we still want the azure-spring-data-cosmos stuff to run with the rest of our Java code?

Please update ci.yml and test.yml related contents.

@kushagraThapar do you think we want to do this? Don't we still want the azure-spring-data-cosmos stuff to run with the rest of our Java code?

@trande4884 - yes that's correct. We want to keep this as it is.. so that we can run spring data cosmos CI with azure-cosmos code changes automatically.

/azp run java - cosmos - tests

Hi, @trande4884 , could you please resolve the conflicts?

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

/azp run java - cosmos - tests

/azp run java - cosmos - tests

Hi, @trande4884 .
Could you please resolve the conflicts and fix check failures?

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/check-enforcer override

Pipeline/test/matrix configs all looking good to me.

/azp run java - spring - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/check-enforcer override


ucc: init at 1.1.0

Description of changes
Collective communication operations API for high performance computing applications.
The soon-to-come openmpi upgrade to version 5.0.0 will make use of this package.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 195543 run on x86_64-linux 1

1 package built:

ucc

Thank you!


Overwrite names of dependecies of subcharts

I am encountering the following problem:
I want to deploy an umbrella chart which holds two subcharts, each subchart is a microservice which uses mysql as database. So mysql is a dependency in each subchart. Now i do want to deploy two seperate mysql charts in my system. When i try to install the umbrella chart, i get the message that {Release}-mysql templates are already deployed. Error: release test failed: secrets "test-mysql" already exists
./
└── charts
    ├── microservice-1
    │   ├── charts
    │   │  └── mysql
    │   └── templates
    └── microservice-2
        ├── charts
        │   └── mysql
        └── templates

Therefore, i need a way to deploy both mysql charts of the subcharts with a different name, only when deployed via umbrella chart. Overwriting the name of the mysql charts of the subcharts is not possible via values.yaml of the umbrella chart:
## Overwrite
microservice-1:
    mysql:
        name: microservice-1-mysql
microservice-2:
    mysql:
        name: microservice-2-mysql

Stating aliases in requirements.yaml files of the subcharts does not take effect in this scope.
What can i do?

charts/stable/mysql/templates/_helpers.tpl
{{/*
Create a default fully qualified app name.
We truncate at 63 chars because some Kubernetes name fields are limited to this (by the DNS naming spec).
*/}}
{{- define "mysql.fullname" -}}
{{- $name := default .Chart.Name .Values.nameOverride -}}
{{- printf "%s-%s" .Release.Name $name | trunc 63 | trimSuffix "-" -}}
{{- end -}}

@cgebe but this overrides the app name not the release name. Do you know how to override the release name of a sub chart ?

@tonyg4864 If you change the name of the subchart as @cgebe mentioned, you don't have conflicts. You cannot override the release name since that is your umbrella release. So if you have mysql1 and mysql2, both will be under "some-release" release (which is basically your umbrella).


Incorrect 'CAA record is not "letsencrypt.org"'

Getting this issue for the domain appwise.be. Doesn't seem to be related to #11.
The CAA records are there, there are even multiple (one for wildcards, and also one for iodef (reporting issues).
The call to dns.google/resolve results in:
{
  "Answer":
  [
    {
      "name": "appwise.be.",
      "type": 257,
      "TTL": 21600,
      "data": "0 issuewild \"letsencrypt.org\""
    },
    {
      "name": "appwise.be.",
      "type": 257,
      "TTL": 21600,
      "data": "0 issue \"letsencrypt.org\""
    },
    {
      "name": "appwise.be.",
      "type": 257,
      "TTL": 21600,
      "data": "0 iodef<EMAIL_ADDRESS>    }
  ]
}

I think the code is tripping up because of this, because it assumes there should only be one 257 record with the expected CAA contents. It should instead check in the list of 257 records if any is for issue, and if so, it must contain the value letsencrypt.org.
The spec for CAA records allows the flag field to be something else than 0, although I've yet to see something besides the value 0.

Potential solution would be changing the for loop here to something like:
const issueRecord = data.Answer.find( item =>
  item.type == 257 && typeof item.data === 'string' && item.data.startsWith('0 issue ')
);

if (issueRecord.data !== "0 issue \"letsencrypt.org\"") {
  return issueRecord.data;
} else {
  return null;
}

That last if test could even be a simple issueRecord.data.includes('letsencrypt.org'). I think the quotes (") are optional around a CAA value.
Note: I don't usually write JS code, so please ignore my formatting if it looks weird 😅

Actually, there may be multiple issue records (to handle multiple cert issuers). So it should be something like:
const issueRecords = data.Answer.filter(x =>
  x.type == 257 && typeof x.data === 'string' && x.data.startsWith('0 issue ')
).map(x => x.data);

// check if any record allows LetsEncrypt (or no record)
if (issueRecords.length == 0 || issueRecords.some(x => x.includes('letsencrypt.org'))) {
  return null;
} else {
  //
  return issueRecords;
}

Synced for r.forwarddomain.net. Is it work for you?

Seems like I missed something when copy pasting code for the PR, so made a quick fix in #14, sorry! 🙏

Synced. Please try again.

Ugh, now it's issueRecords not defined, because the variable is called issueRecord 🤦

Want me to create another patch?

Np. Let me check thoroughly this time.

Anyway you can try run tests locally. It's not that hard. Simply follow setup from CI, and replace any occurences of r.forwarddomain.net with yours in test folder.

I've reverted r.forwarddomain.net to before your PRs. take your time.

Took a bit to set up in a Docker container, but got it working eventually.
After fixing the typo, I also had to tweak the tests for the "target" domain and the response code (301 instead of 302). But that's to be expected, the config of that domain is just different.

Synced. Thank you!
Btw, can you share your Dockerfile? Many people wanted it but I haven't got time to setup it.

Oh uhm I didn't actually Docker-ify it, I grabbed a generic ubuntu image and installed the necessary binaries manually.
I'll take a look tomorrow about actually writing one. Would this be for running the service itself, or running the tests? (or both?)

I opened #16 for this. Only for service needed now.


topics can't install if they need other topics

If a topic needs another topic there is no way to configure the repo to do this besides changing the alphabetical order of the topics.
This can be seen in https://github.com/20k-ultra/dotfiles/commit/2fd9b802303d42aeb960cd9c60837c8eb62d692a by the addition of z in front of topics that relied on xorg being installed first.

This only happened because the config script for nvm wanted to install nvm but needed to source zshrc thus requiring zsh be installed. This issue was mitigated by just calling the code that sourcing zshrc was doing. See https://github.com/20k-ultra/dotfiles/commit/c89dd70507a86725649b9c45846f0d3b9553ebcc.
All examples of such a problem have been removed and if software needs a certain order to be installed then consider moving that software inside that topic it depends on.


Connecting to Rancher Redis Cluster?

Since Rancher since 1.0.1 already creates it's own Redis cluster. Could I simply connect my application to this and avoid having to setup and manage a cluster instance myself?

Nothing physically stops you, but this is a bad idea for a variety of reasons.  It's not a cluster, it's independent standalone instances.  We don't persist anything so there is no persistence configured, and the default won't go to a volume that will be saved across upgrades.  And it can be restarted, reconfigured, or removed whenever we feel like because it's an internal implementation detail, not a public service.

@ronag If you have further questions, I'd suggest using the forums to ask them to get feedback from community members.


meta.json file is not generating

What did you do?
I have attached the mapr to the Prometheus on two instances at one instance it is working good coming to second one its not creating the meta.json file finally throwing an error
This is the status of mount
posix                  1.0T     71.1G    952.9G   7% /prometheus
What did you expect to see?
wal corruption need to resolve
What did you see instead? Under which circumstances?
Environment

Prometheus version:
insert output of prometheus --version 2.22.0

Logs:

level=error ts=2020-10-29T03:29:33.202Z caller=main.go:764 err="opening storage failed: block dir: \"/prometheus/01ENRA5TM45DGMR9PG0NF5R3M6\": open /prometheus/01ENRA5TM45DGMR9PG0NF5R3M6/meta.json: no such file or directory"

I have checked on all compacts there is no JSON file

You can't run to Prometheus instances on the same directory, is that what is happening here? Which filesystem are you using?

NO I have deployed the application on Kubernetes it is inside the pod and the instance is from an operator, I just added the PV to the Prometheus instance I didn't change anything there and I am using mapr-fs system

We look at this in the bug scrub and networked filesystems such as mapr-fs are not supported, as they often do not provide full POSIX semantics. If you can reproduce this on a local filesystem please let us know, otherwise we shall presume the issue is with your filesystem.


[5.3] Some of the GenerateXcodeprojTests are hanging on pull request testing

This is a 5.3 nomination of a test-only fix that is already in the main branch.
Some of the Xcode project generation tests were using in-memory file systems and others not.  But the Xcode project generation code does not currently support non-local file systems (which should itself perhaps be addressed), and so this could cause tests to start traversing large parts of the actual file system, including over NFS mounts.
Ideally the Xcode project generation should support the in-memory file system, but since this functionality was primarily useful before Xcode supported loading packages natively, it's not clear whether it's worth the effort to fix that.
rdar://66386071
(cherry picked from commit 409f52f2b56f73d53cb6c0cdf5fc783d95c5f832)

@swift-ci please test


Documentation describes different behaviour

Describe the bug
It could be that the msgraph documentation is incorrect, but the behaviour does seem incorrect.
I deleted the Graph Explorer Enterprise Application from our AAD (de8bc8b5-d9f9-48b1-a8ad-b748da725064) to ensure the permissions were also reset. I then consented to the default permissions (openid profile User.Read email). I was then able to execute the query https://graph.microsoft.com/v1.0/users/<anotheruser_email>/?$select=id,interests,pastProjects,skills,schools,responsibilities and get a valid JSON response.
To Reproduce
Steps to reproduce the behavior:

As a user in an AAD tenant with no previous access to Graph Explorer (or having cleared the existing application), go to Graph explorer, sign in and consent to the default permissions.
Enter the above query, changing <anotheruser_email> to a user's email address in the same directory.
Observe a 200 response

Expected behavior
I would have expected that with User.Read, that I should only be able to to view my own profile, but not another user's
Screenshots
Graph Explorer result:

Current JWT scopes:

Desktop (please complete the following information):

OS: Windows 11
Browser: Chrome
Version 115

Smartphone (please complete the following information):
n/a
Additional context
I don't know if there's possibly something else at play (in which case it would be good to note this on the ms graph permission reference). At present, the documentation clearly says that User.Read.All is required to view "extended properties" that fall outside the User.ReadBasic.All permission, such as the interestes and responsibilities, etc.

Hi, @neilpendlington,
Not sure where this behavior is specified in the docs. But based on this https://learn.microsoft.com/en-us/graph/api/user-get?view=graph-rest-1.0&tabs=http, you should be able to query other users in the organization

Hi @ElinorW  - apologies, I should have included the link to the documentation I was referencing.
See https://learn.microsoft.com/en-us/graph/permissions-reference#userread

User.Read
Allows users to sign-in to the app, and allows the app to read the profile of signed-in users. It also allows the app to read basic company information of signed-in users.

User.Read.All
Allows the app to read the full set of profile properties, reports, and managers of other users in your organization, on behalf of the signed-in user.

My confusion is that I'm specifically requesting the User.Read scope, which I would have expected to then not be able to select another user's "delve" profile fields interests,pastProjects,skills,schools,responsibilities


Closes: #1 updated validations.py python script

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@dewansh-59  made it.


feat(LinearAlgebra/Finsupp): Make Finsupp.total computable

This also removes the noncomputable section at the top of the file, so as to prove that this really does enable computation.
Previously this was noncomputable because it was constructed as the forwards direct of an equiv whose reverse direction wasn't computable.
This makes DFinsupp.coprodMap and Basis.sumCoords computable too.

On second thoughts I think this might just create churn; my original thought was "we'll want these separate if we every make finsupp fully computable, as only one direction needs DecidableEq"; but if we make the DFinsupp version of the refactor, then both directions need this assumption anyway.


Propagate metadata to derived datasets

See if we have the information to do this at index time.

When the provenance includes multiple ancestors, each ancestor might have a different value for some field, and so when they are combined we may get multi-valued fields.
examples: (currently in QA)

https://portal.hubmapconsortium.org/search?mapped_data_types[0]=scATAC-seq (SNARE-seq) [Lab Processed]&entity_type[0]=Dataset
https://portal.hubmapconsortium.org/search?mapped_data_types[0]=snRNA-seq (SNARE-seq) [Lab Processed]&entity_type[0]=Dataset
https://portal.hubmapconsortium.org/browse/dataset/b698d686b5eff28cac2c7f34d70893ac


add follow links to windows curl command

-L options was omitted for windows links

Nice catch thanks a lot @levequej 🙏🏻


Fix august token refresh

I had a refactoring error when I convert this to async.
Fixes https://github.com/home-assistant/core/issues/39526

@snjoetw Can you take a look ?

@snjoetw not sure if you are seeing these or super busy and can't get to this?

@snjoetw Any chance you have a moment to take a look at this ?

Thanks @bdraco, merged

Thanks.  Can you push out a new release so I can bump home assistant?

Just released 0.25.1: https://pypi.org/project/py-august/0.25.1/

@bdraco please use 0.25.2 instead: https://github.com/snjoetw/py-august/releases/tag/0.25.2

Thanks updated https://github.com/home-assistant/core/pull/40109


ssr no support

How did you import vue-notification?
For SSR, I did it like below and it worked!
import Notifications from 'vue-notification/dist/ssr.js'


luci-proto-yggdrasil: show peer latency

Show latency measurement among other peering information. Latency field was added to yggdrasil admin api in v0.5.6.
Depends on: openwrt/packages#24411
cc: @wfleurant

Decent. Ping when the other PR is merged.

@systemcrash Router update PR is merged (openwrt/packages#24411).

merged. Thanks @one-d-wide


Logs with timestamps

Currently the log file of the SidechainNode doesn't contain timestamps for the printed data.
We should be able to add such possibility.
Probably:
-Dorg.slf4j.simpleLogger.showDateTime=true -Dorg.slf4j.simpleLogger.dateTimeFormat="yyyy-MM-dd HH:mm:ss:SSS"****
may help.

I changed the log format also by placing a file here (in the sidechain app):
/src/main/java/resources/simplelogging.properties
with a content like this:
org.slf4j.simpleLogger.defaultLogLevel=debug
org.slf4j.simpleLogger.showDateTime=true
org.slf4j.simpleLogger.dateTimeFormat=yyyy-MM-dd HH:mm:ss:SSS Z
and was able to append log messages from the sidechain app with this
private static Logger log =  LoggerFactory.getLogger(MyApp.class);
log.debug("...."):

( should be useful to add these to the docs!)

Done in 0.3.0


Trills and jitters in wav synthed by mb-melgan

@kan-bayashi  I train mb-melgan with baker or LJ speech dataset, but synthed wavs seem to have distinct trills and jitters. Any ideas or advices to solve this probem? thanks

@xiaoyangnihao Did you ever solve this problem? I am having this problem with StyleMelGAN as well.


Removes partner follow count, fixes partner page for partners without profiles

The type of this PR is: Bugfix

This PR resolves MX-474 && MX-478
Description
A bug came up in the Viewing Rooms QA session today, and related to a ticket MX just added to its sprint. I thought I'd take care of them both.
Test Plan
n/a
Screenshots

(oop meant to add a #mergeongreen label)


Publish git log since last release with every commit

I've pulled this branch and trying to figure out how to run this. Is there a gradlew command for this, or is it strictly through GH actions? How would I generate this locally?

Update: bash .github/scripts/gitlog.sh  runs it, and I get a docs/unreleased.log file out with commit info in it as expected, but it seems to struggle on figuring out the tags
fatal: No tags can describe '126fa2031195b3fd9f9f83548d5e049240e9abac'.
Try --always, or create some tags.

docs/unreleased.log:
126fa203 Publish git log since last release with every commit
16cf9a08 SW-673 Add refresh token support to test scripts (#756)
3632c0be SW-1885 Add readyByDate to nursery withdrawals (#755)
c1ba5cad Add imports for JUnit assertion methods (#740)
77c3420e SW-1875 Add batch version to search API (#739)
fa7c6ee1 SW-1939 Remove v1 accession payload classes (#733)
...

Update: bash .github/scripts/gitlog.sh  runs it, and I get a docs/unreleased.log file out with commit info in it as expected, but it seems to struggle on figuring out the tags
fatal: No tags can describe '126fa2031195b3fd9f9f83548d5e049240e9abac'.
Try --always, or create some tags.

That would be because some of the tags in the server repo are on a branch.  I didn't realize that git describe would fail in that case.  Will look at an alternative.

Update: bash .github/scripts/gitlog.sh  runs it, and I get a docs/unreleased.log file out with commit info in it as expected, but it seems to struggle on figuring out the tags
fatal: No tags can describe '126fa2031195b3fd9f9f83548d5e049240e9abac'.
Try --always, or create some tags.

That would be because some of the tags in the server repo are on a branch. I didn't realize that git describe would fail in that case. Will look at an alternative.

We could probably skip the git describe.. list the tags and grab the latest.


Lazy building scrolling tree

Use case
Currently when trying to implement tree UIs (for example: comments section with nested comments - see Instagram's comments for reference) there is no way to do it but to flatten the list and use one SliverList to show the data in the UI.
Proposal
Add a way to add a SliverList inside of a SliverList.
This will allow the CustomListView to scroll with all the data while also perform nicely with all the Slivers benefits.
In my use case, this is critical in SliverAnimatedList.
If you have any other solutions I will be happy to discuss.
Solutions I tried

Using a ListView.Builder inside the SliverList builder function with shrinkWrap: true.  - Poor Performance
As pointed above - Normalizing the nested list with the list.expand function and using this list as the data source. - Poor Developer Experience - because everytime new data arrives we need to manage the flattened list which is not ideal.

Code example
CustomScrollView(
    slivers: [
      SliverList(
        delegate: SliverChildBuilderDelegate(
          (context, index) {
            final data = widget.comments[index];
            return SliverList(
              delegate: SliverChildBuilderDelegate(
                (context, index) {
                  if (index == 0) {
                    return CommentWidget(data);
                  } else {
                    return CommentWidget(
                        data.comments[index - 1],
                        isNested: true);
                  }
                },
                childCount: data.comments.length + 1,
              ),
            );
          },
          childCount: widget.comments.length,
        ),
      ),
    ]
)

Not quite SliverList, but this would be called something like SliverGroup. I don't believe we've writen it yet. FYI @Piinks if we have a bug for this already

Thanks for the quick response @jonahwilliams,
Just to clarify, we need a widget that gets a builder function in order to get the most performence out of this feature.
Adding a widget that gets a children array is not quite helpful here since we can have plenty of nested items.

Thanks @Piinks for the detailed information.
I think that SliverList supporting both box and slivers as children actually simplifies the API, don't you think?
It is more friendly for new flutter devs and also more intuitive in my opinion..
I prefer building my own solutions than using a widget with limited customizations.
I have set in my project an infra for scrolling (with plenty of customization - animations, states and more) and it would really complicate the code to add a new widget..
That is why I suggested to nest a SliverList inside a SliverList, just because it is logical and also easy to use and understand as a developer.
Nevertheless, I look forward to see your solutions!
Hope to hear from you soon,
Thanks

any update on this? it can help solve many lagging issues.


Colorize

Add "set highight {dark|light|plain}" to colorize source-code listing.
Require linecache2 from gemcutter.org. And line caching is a good thing anyway.

Definitely up for adding colors!
Here I'd need:

All tests passing (skip is cheating).
Remove ruby version conditionals.
Remove line caching again. If you want that, please add a different PR with tests showing which enhancements you're adding.

After that we can start discussing the feature. I think I would prefer to just colorize if we can, instead of adding a command. What do you think?

In my experience there are cases where you don't want to colorize output. Some examples that come to mind is when the colors look awful as might happen if you are using a dark background for a light background theme, or there is an interface outside the debugger that handle showing source in some other way.
In my other debuggers where I use pygments, it also allows setting the colors via some sort of color style that people can select. See http://python2-trepan.readthedocs.io/en/latest/commands/set/style.html and also http://python2-trepan.readthedocs.io/en/latest/commands/set/highlight.html
As for working on the code, I will not have the time to do this in the foreseeable future. I you don't understand how the code works, I can try to explain that, but otherwise, this will have to wait for someone interested enough in this to see it done in the way that makes you happy.

@rocky Thanks for your input, I'll consider it when I have a look at the code. First time I checked, this PR wouldn't even get the tests passing.

Leaving this WIP PR opened in case someone wants to take it from here.


docs: update readme and types to reflect alternate wasm loading path

For some servers, the loading script will not be in the same location as the WASM module file.
Update documentation and types to reflect how to load rdkit-js from an alternate path.
Additional context from this discussion: https://github.com/rdkit/rdkit-js/discussions/360#discussioncomment-6478625

LGTM @sgoodrow-zymergen , thanks for this!


Build: suppress progress infos for apt-get

Current situation
When calling apt-get, currently the progress is printed:
...
Selecting previously unselected package libann0.
(Reading database ...

Preparing to unpack .../0-libann0_1.1.2+doc-7build1_amd64.deb ...

This is not wanted for non-interactive actions.
Solution
Add --quiet option to apt-get calls.

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=539617 is still open


Subliminal cannot handle files with multi episodes with OpenSubtitles provider

When using subliminal in combination with the latest guessit version, there is a problem when the series contains multiple episodes.
Guessit returns a list of episodes, while an int is expected in the Video class.
Example file: Marvels.Agents.of.S.H.I.E.L.D.S05E01-E02.720p.HDTV.x264-AVS.mkv
guessit Marvels.Agents.of.S.H.I.E.L.D.S05E01-E02.720p.HDTV.x264-AVS

For: Marvels.Agents.of.S.H.I.E.L.D.S05E01-E02.720p.HDTV.x264-AVS
GuessIt found: {
    "title": "Marvels Agents of S.H.I.E.L.D.",
    "season": 5,
    "episode": [
        1,
        2
    ],
    "screen_size": "720p",
    "format": "HDTV",
    "video_codec": "h264",
    "release_group": "AVS",
    "type": "episode"
}

When searching subtitles with subliminal in combination with opensubtitles provider:

Stack trace:
2017-12-21 15:43:32,500 ERROR Unexpected error in provider u'opensubtitles'
Traceback (most recent call last):
  File "C:\Tools\GitHub\Auto-Subliminal\lib\subliminal\core.py", line 134, in list_subtitles_provider
    return self[provider].list_subtitles(video, provider_languages)
  File "C:\Tools\GitHub\Auto-Subliminal\lib\subliminal\providers\opensubtitles.py", line 228, in list_subtitles
    query=query, season=season, episode=episode, tag=os.path.basename(video.name))
  File "C:\Tools\GitHub\Auto-Subliminal\lib\subliminal\providers\opensubtitles.py", line 183, in query
    response = checked(self.server.SearchSubtitles(self.token, criteria))
  File "C:\Tools\GitHub\Auto-Subliminal\lib\subliminal\providers\opensubtitles.py", line 295, in checked
    raise OpenSubtitlesError(response['status'])
OpenSubtitlesError: 408 Invalid parameters; episode is not number

Addic7ed does not seem to have a problem with it. I did not test other providers.

@opensubtitles hey, how should we handle this? API expect a number but video file is multi-episode, so the parameter episode is a list.

we are strictly following IMDB numbering. I think it is this: http://www.imdb.com/title/tt2364582/episodes?season=5&ref_=tt_eps_sn_5
There is not possible to add there "1,2". I think correct solution is to query lower number for episode, so 1. The best would be using moviehash of course.


DBZ-2413 Added support for MySql8 backup_admin, create role, and drop role privileges

@rgibaiev thanks a lot! In fact I believe the test failure is related, as it's parser failure related to a statement containing the ROLE keyword, and this PR also is touching on this. I suppose your change introduced some kind of ambiguity in the grammar.
Re your question on adding other privilege keywords: the parser need to be able to parse any statement a user might run, so if there's grammar changes needed in order to achieve that, let's do this, there's no way around :)

Closing this PR in favor of a bigger one https://github.com/debezium/debezium/pull/1745


Bug: MQTT block can't load saved project

Please find attached two saved scratch workspaces using the music and MQTT snippets
Archive 2.zip
This is really cool. I Just think maybe the kids will want to be able to save and load the projects as well.

Sorry about this
I can see that the zip contains two Scratch files - both with the MQTT blocks, and one of them loads fine and the other doesn't.
Can you give me any clues about what differs between the two Scratch projects? Did you create them differently? Can you think of anything that you did differently between them that might explain why one loads and the other doesn't?

Does one of them load for you? Neither load for me. I had just created the second as a test.

https://github.com/IBM/taxinomitis/assets/1444788/6cde70d7-0bb7-4810-9fb9-a02c9501fe50

Oh, how weird, it did load the second time. I might have to break out a debugger on this. Maybe an exception escapes and can tell me what is wrong. Thanks, and sorry for using your time to show me that. I was unaware one loaded. It didn't this AM, but then maybe I loaded the wrong file...

Traceback
gui {"validationError":"Could not parse as a valid SB2 or SB3 project.","sb2Errors":[{"keyword":"required","dataPath":"","schemaPath":"#/required","params":{"missingProperty":"objName"},"message":"should have required property 'objName'"}],"sb3Errors":[{"keyword":"type","dataPath":".targets[0].variables['`jEk@4|i[#Fk?(8x)AV.-my variable'][1]","schemaPath":"#/definitions/stringOrNumber/oneOf/0/type","params":{"type":"string"},"message":"should be string"},{"keyword":"type","dataPath":".targets[0].variables['`jEk@4|i[#Fk?(8x)AV.-my variable'][1]","schemaPath":"#/definitions/stringOrNumber/oneOf/1/type","params":{"type":"number"},"message":"should be number"},{"keyword":"oneOf","dataPath":".targets[0].variables['`jEk@4|i[#Fk?(8x)AV.-my variable'][1]","schemaPath":"#/definitions/stringOrNumber/oneOf","params":{"passingSchemas":null},"message":"should match exactly one schema in oneOf"},{"keyword":"type","dataPath":".targets[0].variables['`jEk@4|i[#Fk?(8x)AV.-my variable'][1]","schemaPath":"#/oneOf/1/type","params":{"type":"boolean"},"message":"should be boolean"},{"keyword":"oneOf","dataPath":".targets[0].variables['`jEk@4|i[#Fk?(8x)AV.-my variable'][1]","schemaPath":"#/oneOf","params":{"passingSchemas":null},"message":"should match exactly one schema in oneOf"}]}

So it is a schema validation error. Hmm.

I can't recreate this and am fairly sure that load/save is working correctly so I'm closing this


how link  MSSQL2000

how link  MSSQL2000
FOUND EOF ERROR

SQL Server 2000 is not supported. Look into ODBC golang drivers.