Data classification: The un-checked algorithm can be saved successfully to mbconfig file, but still be selected on the "Trainer" tab page.

System Information (please complete the following information):
Windows OS: Windows-11-Enterprise-22H2
ML.Net Model Builder 2022: <IP_ADDRESS>0801 (Main Build)
Microsoft Visual Studio Enterprise: 2022(17.5.5)
.Net: 6.0, 7.0
Describe the bug

On which step of the process did you run into an issue:
The un-checked algorithm can be saved successfully to mbconfig file, but still be selected on the Trainer tab page.

TestMatrix
https://testpass.blob.core.windows.net/test-pass-data/wikipedia-detox-250-line-data.tsv
To Reproduce
Steps to reproduce the behavior:

Select Create a new project from the Visual Studio start window.
Choose the C# Console App (.NET Core) project template.
Add model builder by right click on the project.
Select Data classification scenario.
On Data page, choose the data source,
On the Dada page, click the "Advanced training options..." link.
Go to "Trainer" tab, un-checked any algorithm, for example: SDCA, then click "Save" button.

Click the "Advanced training options..." link again and go to "Trainer" tab.
You will see that the un-checked SDCA algorithm is still be selected.

Expected behavior
The un-checked algorithm should not be selected on the "Trainer" tab page.
Screenshot

Additional context

When training, the available Trainers don't contain the un-checked SDCA algorithm.
The un-checked algorithm can be saved successfully to mbconfig file.

The bug has been fixed on the latest main build <IP_ADDRESS>2301.


refactor(746): Refactor set default zoom logic

Description
Set default zoom level logic has updated.
**discord @puskuruk **
**closes #746 **
Please describe your changes. Also describe your aim and content. Do not forget to list the dependencies required caused by those changes.
## Things to check before creating a PR

[X] I have inspected my topic, checked.
[X] If it is a core feature, executed detailed tests.

Creating PR rules

[ ] PR must be created for an issue with approved tag. Otherwise PR will be rejected.
[X] Relevant issue number: The issue number related to PR must be attached to head of PR header, after prefix after prefix # must be attached in parenthesis. A header like this could be used "prefix(#issue_number): PR header" .
[X] A descriptive and understandable title: The PR title should clearly describe the nature and purpose of the changes. The PR title should be the first thing displayed when the PR is opened. And it should follow the semantic commit rules. For example, a title like "docs(#issueId): Add README.md" can be used.
[X] Related file selection: Only relevant files should be touched and no other files should be affected.
[X] Format and Lint Suitability : The code should be made in accordance with a certain format standard and examined according to the lint rules.
[X] Clean commit history: The commits where the changes are made should be clear and organized.
[X] Opening PR when job is completed: PR should be opened when job is completed and sent for review by other team members.

Changes

[ ] A new feature (a change that adds a new feature, not a breaking change)
[X] A new refactor (a change that is not a breaking change, that improves the readability or performance of the code)
[ ] A breaking change
[ ] Documentation change

How were these changes tested?
Please describe the tests you did to test the changes you made. Please also specify your test configuration.
Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Unfortunately the reference issue isn't approved yet, I've to close this PR until the issue is approved.

we have urgent problems. please keep this work in your locale. we will announce when we need this features.


Update lean-toolchain and CLI improvements

Description

This PR includes an update to the latest Lean 4 nightly toolchain and some smaller fixes and improvements.
This includes a fix for a wrong badge link in the README.md.
Additionally, I added a new CLI flag --prettify-output because the verbose output is too noisy for it being actually useful for running the test cases.

Thanks!


fix #263: show issued credentials with deleted connections

Fixes a small issue when a connection is deleted and there was a credential issued to that connection, the list of credentials would no longer show up.
#263 https://github.com/hyperledger/aries-toolbox/issues/263

I have reviewed the broken behavior and have verified that this change fixes the aforementioned behavior. Nice catch @ruzhukov, let's get this merged in.


New note title bug

Pointed out by @AM-Myrick - new notes will not display a title if the first character is a newline. The whole getNoteTitle function needs to be rewritten to fix this bug and display properly if the first character is header markdown.

Closed with #23


On error callback

Register a callback that is called every time a task returns an error.

My current use case for this I want to log the errors as they happen but I don't want to necessarily Stop the pool. Example would be:
func errorCB(ctx context.Context, err error) {
	if err != nil {
		log.Print(err)
	}
}

type handler struct {
	pool *pool.ContextPool
}

func (h *handler) hello(w http.ResponseWriter, req *http.Request) {
	h.pool.Go(func(ctx context.Context) error {
		time.Sleep(time.Second)
		return errors.New("oh no!")
	})
}

func (h *handler) close() error {
	return h.pool.Wait()
}

func main() {
	h := &handler{
		pool: pool.New().ContextPool().WithErrorCallback(errorCB),
	}
	defer func() {
		if err := h.close(); err != nil {
			log.Print(err)
		}
	}()

	http.HandleFunc("/hello", h.hello)

	http.ListenAndServe(":8090", nil)
}

Codecov Report

Merging #89 (fcd160c) into main (daaaa39) will decrease coverage by 0.91%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   99.28%   98.37%   -0.91%
==========================================
  Files          12       12
  Lines         419      431      +12
==========================================
+ Hits          416      424       +8
- Misses          3        7       +4

Impacted Files
Coverage Δ

pool/context_pool.go
87.87% <0.00%> (-12.13%)
:arrow_down:

pool/error_pool.go
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thinking out loud a bit here. Apologies if my comments below aren't clear -- I'm struggling to put words to my thoughts.
In general, I am a little concerned about this change because it encourages a pattern of long-lived pools. The problem with long-lived pools is that 1) errors are not propagated until the pool completes, and 2) panics are not propagated until the pool completes. This PR covers case 1, but it does not cover case 2.
In your example, you spawn a goroutine per request. However, the HTTP response writer is never actually used and the goroutine does not complete before the request completes, so the client will never see result or error. In practice, a HTTP server will parallelize requests, so a pool is not needed here. Additionally, handling requests should almost be done synchronously because you still need to write the response to the http.ResponseWriter when you complete the operation.
I know this probably sounds a bit pedantic, but I think adding streaming capabilities to pool might be a mistake because it suggests usage patterns that really aren't great. An ErrorPool handles a set of tasks where the result of all those tasks might be an error. Using the error early means that this is no longer true. Instead, I'd probably recommend using a Pool (rather than an ErrorPool) and handling the errors in a streaming manner. If you handling the errors as you go, the error return value isn't really useful anyways.

Excellent points, For the case of handling panics I think it wouldn't be a bad idea to add a PanicCallback as well and once the callback is invoked we can clear the recovered value from the Catcher and if no callback registered to propagate the panic as previously
func (p *Catcher) tryRecover() {
	if val := recover(); val != nil {
		rp := NewRecoveredPanic(1, val)
		if p.onPanicCallback == nil {
			p.recovered.CompareAndSwap(nil, &rp)
		} else {
		 	p.onPanicCallback(&rp)
		}
	}
}

In my example I've kept code short for the sake of brevity, think of the goroutine invocation within the HTTP request would just initiate a background process which shouldn't interrupt other invocations of the same task in case of error but the error should be dealt with in some way (reported/logged, etc.).
Could you please elaborate a bit more on "I think adding streaming capabilities to pool might be a mistake", since I believe the Error/Panic Callbacks would/should not really promote any streaming patterns (and why streaming patterns are bad in this case?)?
Moreover, I have been thinking what if we extend these callbacks to Pre/Post hooks which would be respectively invoked before and after the each task. Let me know what you think about it?

@camdencheek chiming in here on behalf of Cloud - we recently had a similar use case, where specifically we needed to do error handling after the pool becomes aware of the error (adds it to the underling errPool) - we were also tempted by the ability to have pool jobs error to add streaming handling of errors, and ended up getting bitten by the gotcha that cancellation will not happen until we actually return it. For that use case, we eventually switched to using a stream instead, and building our own error handling - this worked much better because stream offers a more explicit hook for handling the results of each job.

[...] I think adding streaming capabilities to pool might be a mistake because it suggests usage patterns that really aren't great. An ErrorPool handles a set of tasks where the result of all those tasks might be an error. Using the error early means that this is no longer true. Instead, I'd probably recommend using a Pool (rather than an ErrorPool) and handling the errors in a streaming manner. If you handling the errors as you go, the error return value isn't really useful anyways.

+1 on this - IMO if you want to expose "job lifecycle" concepts, it probably should live in stream instead. This also takes care of issues like "does my callback need to be concurrency-safe" (for stream, no).
(aside - now that I think of it, I think I like the old naming of lib/group better now, since it more directly implies a "bounded group of work" whereas pools in Go are often long-lived resource pools, in the context of which the attempted usage in https://github.com/sourcegraph/conc/pull/89#issuecomment-1426828622 kind of makes sense, even though the design of pool is not for this kind of long-lived-pool-of-resources use case)

since I believe the Error/Panic Callbacks would/should not really promote any streaming patterns (and why streaming patterns are bad in this case?)?

I think a callback that happens  as soon as something happens is what Camden means by "streaming", and as I note in my previous comment I think the intended model is the pool package lets you do a bounded set of work and collect the results, whereas the stream package lets you execute on those results as they come in (and this is where we might want to add extended lifecycle/streaming hooks, instead of having it in pool types)


Support for Hijri Calendar

Hello,
Great lib. But I want to change it to Hijri Calendar (Arabic calendar) format. How can I do that?
Something like this: https://github.com/sahaab/HijriCalendar-master

There's a fork of this library with this feature here.

Outstanding. Thank you for the help. Also do you know about the fork for Farsi - Persian/Iranian Calendar?

Outstanding. Thank you for the help. Also do you know about the fork for Farsi - Persian/Iranian Calendar?

Unfortunately, I don't. Sorry.


Sort completion items by relevance

Currently completion items are sorted alphabetically.
Ideally we would want them sorted by relevance, where relevance can be a combination of multiple factors, such as (but not limited too):

inheritance
visibility
locality

I would add usage if we can figure that out quickly.   If I happen to be in a project that uses scalaz.Future in nearly every file, then I am less likely to want to use scala.Future for example.


A way to get mini-starter to open in new empty tabs

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini-starter
Description
I would like to be able to have mini-starter be the start page when I create a new empty tab! If there's a way to get this to work custom that works too, but I think it could be useful.

Thanks for the suggestion!
This kind of customization is usually better to be left to the user who can implement this behavior using autocommands and dedicated events.
Here is one way of opening Starter buffer when entering new tabpage:
require('mini.starter').setup()
vim.api.nvim_create_autocmd('TabNewEntered', { callback = function() MiniStarter.open() end })

Closing as not planned.


@conda-forge-admin, please add user @diogomart

Additional comment:
Hi Hadrien,
Thanks a lot for putting meeko on conda-forge! I'm one of the developers of meeko. I'll create another issue to add @nbruciaferri who is also a developer of meeko.

See https://github.com/conda-forge/meeko-feedstock/commit/20c2aa8c9220148a3a199cccfef9ceafb4b19c83 for @ diogomart
(you can close here)


Improve error messages for downloading UF2s

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

Project coverage is 100.00%. Comparing base (27ef692) to head (ab2ffb3).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #83   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           32        32
  Lines         1388      1356   -32
=========================================
- Hits          1388      1356   -32

Flag
Coverage Δ

macos-latest-py-3.8
100.00% <100.00%> (ø)

ubuntu-latest-py-3.10
100.00% <100.00%> (ø)

ubuntu-latest-py-3.11
100.00% <100.00%> (ø)

ubuntu-latest-py-3.12
100.00% <100.00%> (ø)

ubuntu-latest-py-3.8
100.00% <100.00%> (ø)

ubuntu-latest-py-3.9
100.00% <100.00%> (ø)

windows-latest-py-3.8
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Accidentally Integrated twice

Hello,
I am currently running Alarmo 1.5.2.   While playing with integrations today, I accidentally clicked on the Alarmo integration.    It seems to have added a second device on the integrations page.

Comparing them, I can see the first is my original as it has an alarmo entity

and the second does not:

Unfortunately, I can't see how to delete the second entity.   Any ideas?

After looking at it more, I realized I couldn't arm the alarm any more.   I didn't save the log, but it noting that the alarm did not have a unique id and it was confused.
I ended up removing the integration and adding it back and starting fresh.    Everything is working again.
Seems like a check might be useful to prevent this from happening.

I don't know how to prevent the integration from being installed twice.
I will have to investigate.

Same here, 2 devices but can’t delete the unused one

The truth is that I don't know either.
My approach would be going to the .storage folder and remove the second integration from core.config_entries and core.device_registry.
But only touch these files if you know what you're doing (and when HA is not running).
I did not find a way yet to hide the button or block a second installation.

This issue has been fixed in the meantime.

The truth is that I don't know either.
My approach would be going to the .storage folder and remove the second integration from core.config_entries and core.device_registry.
But only touch these files if you know what you're doing (and when HA is not running).
I did not find a way yet to hide the button or block a second installation.

I just deleted the integration then re-installed, all settings were preserved.


Appstudio update test-component-pac-ucrn

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-ucrn-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
10 seconds
clone-repository

✅ Succeeded
9 seconds
appstudio-configure-build

✅ Succeeded
10 seconds
sast-snyk-check

✅ Succeeded
51 seconds
build-container

✅ Succeeded
24 seconds
deprecated-base-image-check

✅ Succeeded
23 seconds
sanity-inspect-image

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
19 seconds
clair-scan

✅ Succeeded
28 seconds
sbom-json-check

✅ Succeeded
12 seconds
sanity-label-check

✅ Succeeded
11 seconds
sanity-optional-label-check

✅ Succeeded
9 seconds
show-summary

Pipelines as Code CI/test-component-pac-ucrn-on-pull-request has failed.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

🏃 Running
---
clone-repository


fix(deps): bump

Applied mkpr on undefined
/tmp/mkpr.sh

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


blowfish: Implement Clone manually.

Fixes #336

I believe that the changes to Rust that caused this error were reverted and I'm not seeing the error on 1.6 stable, 1.7 beta, or 1.8 nightly. Please let me know if this is still needed, otherwise I will close this. Thanks and sorry for taking much to long to get back to you.

Hi. No problem with slow response. :)
Unfortunately I don't even remember what did I need this for. :D . So I'm closing, hopefully it works now, I'll reopen if I hit it again.


Suggested post-installation actions

I wanted to mention that the installation and configuration of the VM is well-documented, but this may leave new users a bit confused about what to do next. A good suggestion might be to have them go through a "default" workflow at least once, explaining this in the documentation.

Any update on this ?


fix(styles): import react for types

Description
This only affects flow type. Need to import * as React in order for flow to find React.Node
Scope

[x] Patch: Bug Fix
[ ] Minor: New Feature
[ ] Major: Breaking Change

thanks @kahwee, I've totally missed that!

Np! I noticed I couldn't pass flow in one of my projects that uses baseui. :)
Thanks.

Will any Flow experts be kind enough to chime in?
I changed React.Node to typeof React.Component to try to address the Flow type complaints.
It does seem to work.

Thanks Alex! Tested this change locally and found React.ComponentType<IconPropsT> to work well. Where IconPropsT is imported as import type {IconPropsT} from '../icon/types.js';

Ah, didn't realize there already was a PR


Quickscript.py doesn't create token.pickle anymore

Since January 5 2021, the quickstart.py script provided by Google started using JSON instead of Pickle to store the credentials.
I had to download the latest version from 2019 instead https://github.com/googleworkspace/python-samples/raw/aacc00657392a7119808b989167130b664be5c27/admin_sdk/reports/quickstart.py

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

ID: 48c3a530-31c0-7e57-c8ee-898a98479f84
Version Independent ID: 0dc6af4b-8d71-470f-5f3a-aaa548f9a156
Content: Connect Google Workspace (G Suite) data to Azure Sentinel
Content Source: articles/sentinel/connect-google-workspace.md
Service: azure-sentinel
Sub-service: azure-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

@guarismo
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Hi @guarismo, thank you for your comment on Microsoft docs! We'll have to check this with the team before updating, but once we've validated your comment, we'll get the page updated as needed.
I'm going to close this issue for now, but feel free to continue commenting here if you have anything to add. We value your contribution to docs!
#please-close


Solutions/Microsoft Entra ID/Analytic Rules/AuthenticationMethodsChangedforPrivilegedAccount.yaml

See the path for code in title of this issue.
This query generates duplicate incidents every 2 hours when analytic rule runs. please suggest the fix for it.

Hi @ksinghd09, Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 15-06-2024 Thanks!

Hi @ksinghd09,
We are working on reproduce the issue, we will update you

Hi @ksinghd09, We had a discussion with concern team for this issue, And team want to check, In your environment only this rule has been configured - https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Microsoft Entra ID/Analytic Rules/AuthenticationMethodsChangedforPrivilegedAccount.yaml And not this one, which having same name - https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Business Email Compromise - Financial Fraud/Analytic Rules/AuthenticationMethodChangedforPrivilegedAccount.yaml
Due to the name collision of both the rule it may have duplicate incident, but could you please confirm and let us know, so we can share this update with team. Thanks!

Hi @ksinghd09, Gentle Reminder: We are waiting for your response on this issue. If you still need to keep this issue active, please respond to it in the next 2 days. If we don't receive a response by 22-07-2024 date, we will be closing this issue.
Thanks!

Thanks for getting back, we have created custom analytic rule for our environment, It would trigger an alert by searching Audit logs table instead of IdentityInfo table.

Hey @ksinghd09, The audit logs table are used in rule -https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Business Email Compromise - Financial Fraud/Analytic Rules/AuthenticationMethodChangedforPrivilegedAccount.yaml, So could you please check and confirm if the created customer rule is using above rule, and not the one which you have mentioned into the issue title itself- https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Microsoft Entra ID/Analytic Rules/AuthenticationMethodsChangedforPrivilegedAccount.yaml
As mentioned in comment https://github.com/Azure/Azure-Sentinel/issues/10617#issuecomment-2225075304, if both the rule gets configured then, due to name collision the duplicates incident gets created. Thanks!

@ksinghd09, waiting for your response. Thanks!

Hi @ksinghd09, since we have not received a response in the last 5 days, we are closing your issue as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation.


Getting Error When Selecting Avatar Photo without photo

Describe the bug
A clear and concise description of what the bug is.
1.: Selected Male version
2.:
We failed to load the photo (we bet you look great, though)! Please try using a different file."}
https://api.readyplayer.me/v2/avatars
{"type":"AbsynthError","status":500,"message":"We failed to load the photo (we bet you look great, though)! Please try using a different file."}
https://api.readyplayer.me/v2/avatars

Hi, this is happening because base64image is new a required field and default avatars API has been changed. Its fixed in latest release v0.4.0.

Hi, this is happening because base64image is new a required field and default avatars API has been changed. Its fixed in latest release v0.4.0.

Thank you for the quick fix  @rYuuk , the v0.4.0 has already fix the problem, but the version number in the package still stays on 0.3.1.  ;)

The version number has been fixed.


Fail gracefully on different degrees in plonky3

We had missed some backends in the original PR

Looks like this PR has no diff anymore? Closing for now


Environments folders not deleted when removed from pixi.toml

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example

[project]
name = "test"
version = "0.1.0"
description = "Add a short description here"
authors =<EMAIL_ADDRESS>channels = ["conda-forge"]
platforms = ["linux-64", "win-64"]

[dependencies]
python = "3.11.*"
requests = "*"
numpy = "*"

[feature.test.dependencies]
pytest = "*"

[environments]
test = ["test"]

Issue description
Running pixi install with the above configuration results in 2 environments, default and test. Remove the test env configuration from pixi.toml and rerun pixi install. The envs folders are not updated and the deleted env's folder/s remain.
Current: .pixi/envs/test folder remains.
Expected behavior
.pixi/envs/test folder is removed.

I think I rather have a pixi clean command to clean on request. If you switch between branches this might already be a bigger annoyance than having the environment which doesn't break or interfere with other environments.
I could be convinced otherwise but this is the first idea I have on it.

agree that pixi clean makes the most sense here. In a similar vein, pixi global also gets a clean option that clears the rattler cache ?

There is now a pixi clean and pixi clean cache


[deploy] Provide additional information when reading public key

Reading a public key from a misconfigured or badly formatted source leads to an error such as
Did not find public key for signing

https://github.com/jreleaser/jreleaser/blob/1a6b7e6066b543714f96f9ba403425604df20d30/sdks/jreleaser-java-sdk-commons/src/main/java/org/jreleaser/sdk/commons/AbstractMavenDeployer.java#L348-L353
Proposed solution:
Update the value of the following keys in https://github.com/jreleaser/jreleaser/blob/main/api/jreleaser-resource-bundle/src/main/resources/org/jreleaser/bundle/Messages.properties:
ERROR_public_key_not_found
ERROR_secret_key_not_found

To include a message such as:
Ensure a value has been configured and properly formatted without indentation.

🎉 This issue has been resolved in v1.12.0 (Release Notes)


MS Purview and .Net

Hello,
I don't know if I am at the right place. I am not a developer, but an information specialist. Ik work for the Dutch government in the area of open data, data sets, metadata etc., with standards like SKOS, RDF, DCAT. These are the standards for government data exchange. Now I saw MS Purview and this sounds like a very promising direction for catalogs, assets and attributes.
My question is whether it is, or will be possible to serialize, create  from MS Purview / Azure .Net or other means to e.g. DCAT 2.0 datasets and/or JSON attribute properties.
Greetings and thanks in advance.
Huub

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

ID: f885f64d-0f86-a786-ef2f-446a847715d9
Version Independent ID: 441b4646-ac3b-c3b6-e2b1-9495992f5c80
Content: Quickstart: Share data using the .NET SDK - Microsoft Purview
Content Source: articles/purview/quickstart-data-share-dotnet.md
Service: purview
Sub-service: purview-data-share
GitHub Login: @jifems
Microsoft Alias: jife

@huubvanoers
Thanks for your feedback! We will investigate and update as appropriate.

Hi @huubvanoers
Thank you for reaching
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1

Hi,
Thanks for your answer. I will investigate it further.
Thanks!
Huub
Van: Naveenommi-MSFT @.>
Verzonden: donderdag 22 december 2022 08:18
Aan: MicrosoftDocs/azure-docs @.>
CC: huubvanoers @.>; Mention @.>
Onderwerp: Re: [MicrosoftDocs/azure-docs] MS Purview and .Net (Issue #103010)
Hi @huubvanoers https://github.com/huubvanoers
Thank you for reaching
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1 &dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1
—
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/103010#issuecomment-1362497648 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AOJP2LGOGR6O5A242V4CFVTWOP6C5ANCNFSM6AAAAAATFQRCQU .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AOJP2LDRMN6KFB3JYSUA2VTWOP6C5A5CNFSM6AAAAAATFQRCQWWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSRGYIHA.gif Message ID: @.*** @.***> >


Added breadcrumb map function to browse

In order to being able to really build a custom template for the browse
directive I have added another function to build even custom breadcrumb
paths. The other function LinkedPath is not that easy styleable as
this map function. That way we are able to build the breadcrumb path
matching different CSS frameworks like Bootstrap.
An example implementation should look like that, matching the Bootstrap structure:

  {{range $url, $name := .LinkedList}}

{{$name}}

  {{end}}

Signed-off-by: Thomas Boerger<EMAIL_ADDRESS>

Cool. I think the name is a bit misleading though: LinkedList is actually a map :wink: What about Breadcrumb or something?

Yeah sure, I just took this name because of the function LinkedPath

I'll merge this in, thanks again Thomas!
If you have a chance sometime, it'd be great to see a test for BreadcrumbMap. Maybe it'd even be possible to replace LinkedList with BreadcrumbMap? I'd love it if you could look into that, because I think the map is a better idea than the linked list, which is less flexible.

Errrr, that's weird, I definitely saw all 3 checks pass.


toolctl upgrade should skip, not abort when install directories don't match

❯ toolctl upgrade
[gh           ] ✅ already up-to-date
Error: aborting: golangci-lint is currently installed in /Users/patrick/go/bin, not in /usr/local/bin

Fixed by #56


mAP/NDS

Hello author, there seems to be some differences between the code and the paper. What are the results of mAP and NDS? Have you considered making the logs and checkpoints public?

Hi, we are sorry that we cannot access the previous logs and models. We will retrain our model at some point in the future. However, the provided configs should be able to reproduce the numbers in the paper.
May I ask if you could share your training logs and environment settings? Maybe I can help with your problem :)

Thank you so much! I use 4 3090GPUs, batch_size is 8, lr is 2e-4, and weight_decay is 1e-2. The following are my training results, logs and environment.

20240604_213640.log
environment.txt

hi,
We have fixed some bugs in the previous version; could you plz try again? I am sorry that I cannot try by myself because I do not have computing resources now  :(

I am glad to

hi,
We have fixed some bugs in the previous version; could you plz try again? I am sorry that I cannot try by myself because I do not have computing resources now :(

Hello author, the bug you fixed can indeed enable the model to obtain similar indicators to the paper.
Thanks！

L1bertad， Can you send me a log of your latest experiment? I would like to refer to it. Thank you.

widthformer.log
This is my experiment log, I hope it will be helpful to you.

thank you so much

L1bertad， Can you send me a weight(pth) of your latest experiment? I find that attention map of multiview pics cannot match. Thank you.

widthformer.log 这是我的实验日志，希望对您有所帮助。

Do you have the experiment log under BEVDet4D? Thank you very much


Call Android subprocesses with python3

Ensure that python 3 is used on systems in which python2 is the default
python executable, but which have both pythons installed.
Tested in Ubuntu 16.04.

Wait, I have a better idea, I'll try to make build.py both callable from build-all.py and runnable, this way the shseband python3 will be used automatically for it.


REACT_APP env not used in jest environment

Is this a bug report?
Yes, when running a test in a jest environment that uses REACT_APP environment variables, they don't get set.

Did you try recovering your dependencies?

N/A
Which terms did you search for in User Guide?

jest, environment, test, REACT_APP
Environment

System:
OS: macOS 10.14
CPU: x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
Binaries:
Node: 8.12.0 - ~/n/bin/node
Yarn: 1.9.4 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
Browsers:
Chrome: 69.0.3497.100
Edge: Not Found
Firefox: 61.0.1
Internet Explorer: Not Found
Safari: 12.0
npmPackages:
@storybook/react: 3.4.11 => 3.4.11
@types/react:  16.4.13
react: 16.5.2 => 16.5.2
react-dom: 16.5.2 => 16.5.2
react-scripts: 2.0.4 => 2.0.4
npmGlobalPackages:
create-react-app: Not Found
Steps to Reproduce

set environment variable in .env.local
write test checking the variable
see that it returns undefined

Expected Behavior

I'd expect the ENV vars to be there.
Actual Behavior

ENV vars are undefined
Reproducible Demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

Seems to be intended.
  // Don't include `.env.local` for `test` environment
  // since normally you expect tests to produce the same
  // results for everyone

Oh, I didn't realize they were using .local -- but yes this is to be expected then.
Move these variables out of the local version or put them in .env.test.local, thanks!


Allow Filling Outside RTH Algorithm Setting

Expected Behavior
Support for stop orders filling outside regular trading hours.
Actual Behavior
Some orders can't be configured to work outside RTH.
Potential Solution
Add a property to the Order Settings class which lets people configure it.
Checklist

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

We have implemented a property that can be used to address this issue:
InteractiveBrokersOrderProperties.OutsideRegularTradingHours

Solved by https://github.com/QuantConnect/Lean/blob/master/Common/Orders/InteractiveBrokersOrderProperties.cs#L54


Add Variables

I have just created a variable and used the event once.
We can do this with a function too. we just pass the text to it and it will use the same event again.
If I am going wrong please identify me.
Thanks

@allcontributors add @aliasgharkarani for code


Serialise body according to v4 spec

I added a wrapper object to the body to match the v4 body spec and tested it against the PACT-JVM-Provider Tests.
They can now validate the body - before that, the body was reported as missing.
Frankly, I'm not quiet sure if this is what you intended to do, or if you had other use cases in mind. I tried to preserve the contract-builder API, but had to drop the custom-json serialisation to get this to work. So please feel free to ignore this PR and have a go at it yourself if I misunderstood your intentions here!
Fixes #4

Let me think a bit about #4 and if needed we'll use this.

Fixed by downgrading do V3 in #7


Beta app description from Pilot is adding English (Australia) localization

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Every time I upload a new build to TestFlight with Pilot using the beta app description, it adds the description localized as English (Australia) instead of the expected English (U.S.). We don't use any languages other than en-US for any other metadata on iTunes Connect, so this causes an error when trying to submit the build for TestFlight Beta Review (as nothing else is localized to en-AU).
Complete output when running fastlane, including the stack trace and command used

[11:18:01]: -------------------
[11:18:01]: --- Step: pilot ---
[11:18:01]: -------------------
[11:18:01]: Login to iTunes Connect<EMAIL_ADDRESS>[11:18:03]: Login successful
[11:18:04]: Ready to upload new build to TestFlight (App: ********)...
[11:18:04]: Going to upload updated app to iTunes Connect
[11:18:04]: This might take a few minutes. Please don't interrupt the script.
[11:19:41]: iTunes Transporter successfully finished its job
[11:19:41]: --------------------------------------------------------------------
[11:19:41]: Successfully uploaded package to iTunes Connect. It might take a few minutes until it's visible online.
[11:19:41]: --------------------------------------------------------------------
[11:19:41]: Successfully uploaded the new binary to iTunes Connect
[11:19:41]: If you want to skip waiting for the processing to be finished, use the `skip_waiting_for_build_processing` option
[11:19:50]: Waiting for iTunes Connect to finish processing the new build (3.5.7 - 1537)
[11:20:23]: Waiting for iTunes Connect to finish processing the new build (3.5.7 - 1537)
[11:20:55]: Waiting for iTunes Connect to finish processing the new build (3.5.7 - 1537)
[11:21:27]: Waiting for iTunes Connect to finish processing the new build (3.5.7 - 1537)
[11:21:58]: Waiting for iTunes Connect to finish processing the new build (3.5.7 - 1537)
[11:22:31]: Waiting for iTunes Connect to finish processing the new build (3.5.7 - 1537)
[11:23:03]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:23:37]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:24:09]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:24:40]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:25:12]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:25:44]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:26:16]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:26:49]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:27:21]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:27:53]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:28:25]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:28:58]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:29:30]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:30:02]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:30:34]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:31:06]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:31:39]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:32:12]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:32:44]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:33:16]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:33:48]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:34:20]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:34:52]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:35:24]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:35:56]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:36:28]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:37:00]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:37:32]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:38:04]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:38:36]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:39:09]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:39:40]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:40:12]: Build doesn't show up in the build list anymore, waiting for it to appear again
[11:40:45]: Successfully finished processing the build 3.5.7 - 1537
[11:40:47]: Successfully set the beta_app_feedback_email and/or beta_app_description
[11:40:51]: Successfully set the changelog for build
[11:40:51]: Distributing new build to testers: 3.5.7 - 1537
[11:40:51]: Successfully distributed build to Internal testers 🚀

Environment

### fastlane gems

Gem
Version
Update-Status

fastlane
2.84.0
✅ Up-To-Date

Loaded fastlane plugins:

Plugin
Version
Update-Status

fastlane-plugin-versioning
0.3.4
✅ Up-To-Date

fastlane-plugin-badge
1.0.0
✅ Up-To-Date

@provTheodoreNewell That is super strange 🤔 I have never experience this myself and its kinda hard to look into based up that information you were able to provide.
Is your primary language still set to "English (U.S.)"?
Like...

Yes, primary language is still "English (U.S.)". It seems odd that it's a localizable field but Pilot doesn't need to know what language to pick.

I honestly had no idea that it was a localizable field.  I literally have no idea what could be causing this without digging way deep into it so just asking the questions first 😬

No worries – I'm located in United States.
Here's a screenshot of the field in question on iTunes Connect:


Scroll position often snaps to mid-window when typing

Environment
Windows Terminal version (if applicable): Windows Terminal (Preview)
Version: 0.9.433.0

Any other software? None applicable

Steps to reproduce
Run "dir" a few times to get a few hundred lines of output.
Resize the terminal window vertically (making it bigger). Then start typing. In many cases (I'm not exactly sure when) the current line will snap to somewhere in the middle of the window. It appears to be jumping to the bottom of the previous window position. Once a few commands are run and more output is printed it allows itself to go to the bottom
Expected behavior
When I start typing it shouldn't scroll the text unless that is necessary to make the cursor visible. Currently the terminal's notion of "visible" seems to be based on the previous window size.
Actual behavior
The content of the window scrolls up. This is quite disconcerting, and means that less relevant information is displayed.

I think we knocked out a bunch of these with #4741 and (to be merged) #4903, the first of which is definitely in for 0.10. I'm going to mark this one for feedback and hold it until 0.10 is out to see if it's still a problem.

This issue seams gone for me when I uninstalled the Windows Terminal and reinstalled version 0.10

Awesome, thanks for following up!

They still happen for me. Steps:

Make sure you're running the 0.10 version
Open the terminal and do something to generate some output, enough to more than fill the window
Size the window to larger. then use the mouse-wheel to move the entry line down to the bottom or below
Start typing - the entry line snaps back up to the bottom of the old Window size

It seems to correct itself more easily now, so the problem is less severe, but the problem is not gone.

I'm pretty sure we're not really going to be able to fix that one unfortunately. This is a complicated issue, so bear with me here.
When the Terminal is running commandline applications, it's running attached to an actual conhost instance that's hosting the Win32 console application. We need this so that Win32 APIs will still work in the Terminal. We call this conhost instance "conpty". This conpty has its own text buffer that's the size of the terminal's viewport. For the Terminal to properly display win32 applications, it's very important that the contents of the Terminal's viewport exactly matches the contents of conpty's buffer. If these two get out-of-sync, then the differences will become cumulative, and the contents of the Terminal might get horribly corrupted.
When you resize the Terminal, the terminal requests that conpty also resizes it's own buffer. When that happens, the terminal needs to react in a way that exactly matches how conpty reacts to the resize. As of #4741, the Terminal does a really good job of staying in sync with conpty. The Terminal moves it's viewport within the buffer so that the mutable part of the Terminal buffer still exactly matches the conpty buffer.
Unfortunately, conpty doesn't have any concept of a scrollback, so when it resizes, it can't magically summon lines back from the scrollback. If you just make conpty bigger, it's just going to add blank lines to the bottom of it's buffer. So when you make the Terminal taller, the conpty buffer looks like this:
  +------+       +------+
  |A     |       |A     |
  |B     | +-->  |B     |
  |C_    |       |C_    |
  |      |       |      |
  +------+       |      |
                 |      |
                 +------+

This then pairs with the Terminal snapping on input. When you type a character on the input line, the Terminal by default snaps back to the mutable viewport. Let's imagine that you had two lines of scrollback, with '1' and '2' in them. From the terminal's point of view, this is what's happening when you scroll up to make them visible, then increase the height of the buffer, then type a character.
          Conpty
                    +------+       +------+
                    |A     |       |A     |
                    |B     | +-->  |B     |
                    |C_    |       |C_    |
                    |      |       |      |
                    +------+       |      |
                                   |      |
                                   +------+
          +---------------------------------------+
         Terminal
                    +------+                   +------+                 +------+
          Visible+> |1     |         Visible+> |1     |     Visible     |1     |
                 |  |2     |                |  |2     |     = Mutable   |2     |
      Mutable+>  |  | ---- |     Mutable+>  |  | ---- |             +>  | ---- |
             |   |  |A     |            |   |  |A     |             |   |A     |
             |   |  |B     |            |   |  |B     |             |   |B     |
             |   +> |C_    |   +----->  |   |  |C_    |  +--->      |   |C_    |
             |      |      |  Increase  |   |  |      |  Type       |   |      |
             +>     +------+  Height    |   +> |      |  Input      |   |      |
                                        |      |      |             |   |      |
                                        +>     +------+             +>  +------+

You might be able to work around this with "snapOnInput": false in your profile settings, which will disable the snapping down on input.

I can live with it, although it's slightly worrisome when a real fix is impossible.


noticed inconsistency of generated newtype, in module Win32::Networking::Winsock: sa_family_t vs socklen_t

Suggestion
I was writing some code using Winsock, then I noticed the type sa_family_t is a struct with a single named field Value, while the type socklen_t is a tuple struct, which are added to the metadata at the same time:
https://github.com/microsoft/win32metadata/issues/1041
also, there're other inconsistency such as sa_family_t is annotated with #[repr(C)] while socklen_t is #[repr(transparent)], sa_family_t contains a doc string about the feature flags but socklen_t doesn't; socklen_t has a #[derive(...)] attribute but sa_family_t doesn't. here's the generated rust code in the windows crate:
https://github.com/microsoft/windows-rs/blob/891c469706d314fcf0e3a7913f45970ce11b13f0/crates/libs/windows/src/Windows/Win32/Networking/WinSock/mod.rs#L16828-L16832
https://github.com/microsoft/windows-rs/blob/891c469706d314fcf0e3a7913f45970ce11b13f0/crates/libs/windows/src/Windows/Win32/Networking/WinSock/mod.rs#L16901-L16903

side note
the `sa_family_t` probably should not be added, since there's `ADDRESS_FAMILY` in winsock2 already:
https://github.com/microsoft/windows-rs/blob/891c469706d314fcf0e3a7913f45970ce11b13f0/crates/libs/windows/src/Windows/Win32/Networking/WinSock/mod.rs#L4959-L4962

so why is this discrepancy? does the code generator not have defined rule in this case, or is it a corner case? does this need to be fixed?
btw, is there a recommended way to convert a windows-sys type alias to the corresponding windows crate newtype? should I transmute() directly, or can I assume the newtype wrapper would always be a tuple struct (and the sa_family_t is indeed an error in code generator)? or maybe we can add a new generic api similar to core::from_abi()?
I like the fact that libraries are using windows-sys in their public api, and I also like to use the windows crate for the convenience and safety features when writing non public facing code, and often feel the need to bridge the two crate together, however, I can't find in the documentation what's the "proper" way to do the convertion.
I wonder is this a  uncommon usage, or is there a well established solution and I am just missing the obiouvs? I have read #1393 and some other related issues, but I don't think they are relevant about mixing both windows-sys and windows crates.

sa_family_t and socklen_t were added to the metadata by request in: https://github.com/microsoft/win32metadata/issues/1041
The only relevant difference between the two is that socklen_t is defined in the Windows API headers (WS2tcpip.h) whereas sa_family_t isn't. This leads to having different attributes in the metadata (NativeTypedef vs. MetadataTypedef).
I recall NativeTypedef being used to distinguish real handle types from the ones that are manually added to the metadata. I can't quite remember the details but I think that explains the difference in codegen.

The only relevant difference between the two is that socklen_t is defined in the Windows API headers (WS2tcpip.h) whereas sa_family_t isn't. This leads to having different attributes in the metadata (NativeTypedef vs. MetadataTypedef).

ah, I didn't know sa_family_t was not in Windows API headers. it's the NativeTypedef (vs MetadataTypedef) that determines how typedefs are generated. now I understand.
I'll close this and open another one regarding the mixed usage of windows-sys and windows crates then. thank you for explaining.


FsDb : test_load_fail_from_invalid_path exception

Make test_load_fail_from_invalid_path pass. Means finding why the load() function is not seen as Throwing an except as it should.

I letted the load method raise indifferently IOError or JSONDecodeError wihtout catching it.
Test passes.


upload Image not working

I am using this for to upload images to my facebook page.
I can't upload images in electron.
same script working in puppeteer.

Events not 🔥, may be because upload is handling by Electron dialog


I know there is a issue  https://github.com/pixijs/pixi.js/issues/1755
But Is there a new way to do this  without mask ?

Yes, there is an addHole method which takes the current shape and sets it as a hole to the previous shape.
https://jsfiddle.net/ew6nyavx/
This only seems to work when using moveTo and then using line, bezier, quadratic line methods on Graphics class. The convenience drawing methods (drawRect, drawCircle, etc) don't seem to be working as holes.
https://jsfiddle.net/ew6nyavx/2/

Did some more digging on this issue, seems the holes can only be made if the hole and the source shape are both Polygons.
This works
new PIXI.Graphics()
  .beginFill(0x00ffcc)
  .drawPolygon([0, 0, 200, 0, 200, 300, 0, 300])
  .drawPolygon([10, 10, 190, 10, 190, 290, 10, 290])
  .addHole();

This doesn't work
new PIXI.Graphics()
  .beginFill(0x00ffcc)
  .drawRect(0, 0, 200, 300)
  .drawRect(10, 10, 180, 280)
  .addHole();

In case anyone needs a hole in his circle, I used this https://gist.github.com/markknol/5c5d48655ebac555a6eec41792acdfb6


[Feature Request] Extend use-recent-api-versions to AVM warnings or create new property for bicepconfig.json

Problem: (I did search previous issues but could not see this discussion)
Sometimes when you are using AVM in your Bicep templates, you can be using old versions without realising it. Either through copying an old template or the VS Code extension for Bicep sometimes auto-selects the incorrect version that isn't the latest, you are none the wiser.
My suggestion is that we need a way to get notifications on non-latest Azure Verified Module usage in templates. Either by extending the current use-recent-api-versions to include similar linting warning for older AVM usage like:
Can we either get the bicepconfig.json:
        "use-recent-api-versions": {
          "level": "warning",
          "maxAllowedAgeInDays": 0,
          "avm": true
        }

Or via an entirely new property within the bicepconfig.json to enable this, if technically possibly something like:
"use-latest-avm-versions": {
"level": "warning",
"enabled": "true" // false default
}

I think the benefits of having -something- that helps us catch older AVM usage, would be beneficial to everyone. AVM is gaining a lot of popularity also.
Hope that makes sense.

@StephenWeatherford can you respond to this?

@StephenWeatherford can you respond to this?

@StephenWeatherford
I'm keen to hear the teams thoughts on this one. It's a common pain point now we leverage AVM for the majority of our templates. Thanks!

@riosengineer Could you please enable the "use-recent-module-versions" rule in bicepconfig.json?  It's off by default:
{
  "analyzers": {
    "core": {
      "rules": {
        "use-recent-module-versions":{
          "level": "warning"
        }
      }
    }
  }
}

This was implemented a few months ago, please let us know if it's working for you.

@riosengineer Could you please enable the "use-recent-module-versions" rule in bicepconfig.json?  It's off by default:
{
  "analyzers": {
    "core": {
      "rules": {
        "use-recent-module-versions":{
          "level": "warning"
        }
      }
    }
  }
}

This was implemented a few months ago, please let us know if it's working for you.

Thanks for this, I had no idea this existed. Apologies!

@riosengineer No problem, I appreciate your taking the time to enter the request (and especially glad we already did it :-).
There's also currently no easy way to know what rules you're missing out on because they're not enabled.  I entered #15648 to help address this.

@riosengineer No problem, I appreciate your taking the time to enter the request (and especially glad we already did it :-).
There's also currently no easy way to know what rules you're missing out on because they're not enabled. I entered #15648 to help address this.

Thanks @StephenWeatherford. Those proposals look like a great addition to make things clearer!


Add build summary entry for enable-benchmarks

In our build summary (and netcdf_meta.h) it would be good if we included whether or not benchmarks were built.
Also it would be useful to have the HDF5 version.

Adding the HDF5 version is hard in the autoconf build. See http://git.savannah.gnu.org/gitweb/?p=autoconf-archive.git;a=blob_plain;f=m4/ax_lib_hdf5.m4 for ideas.


Automated cherry pick of #4724: wires: lift name length, charset constraint

Cherry pick of #4724 on release/2.14.
#4724: wires: lift name length, charset constraint

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.14@7e4dd80). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.14   #4745   +/-   ##
==============================================
  Coverage                ?   6.86%
==============================================
  Files                   ?     523
  Lines                   ?   89073
  Branches                ?       0
==============================================
  Hits                    ?    6119
  Misses                  ?   82446
  Partials                ?     508

Flag
Coverage Δ

#aFlag
6.86% <ø> (?)

Impacted Files
Coverage Δ

pkg/compute/models/wires.go
1.52% <ø> (ø)

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 7e4dd80...1e92214. Read the comment docs.

/lgtm

/approve


fallback to oldnames in API

if API dont return new "currentTemperature" use old "temperature" (same for humidity)

Thanks for your contribution ;)

I'll publish on NPM soon;)

Published as 0.0.36

thanks 👍


Update Screens.md

Just a small typo fix! This is how the docs rendered for me before: looks like this should all be the same code snippet.

Thanks @cycleseven !


[Metrics] Supply extra metrics to monitor the flows under prefer-leader mode.

Description
This pr is used to supply extra necessary metrics for monitoring the dataflows under prefer-leader mode, providing an intuitive understanding if the destination of dataflows meet the expectation.
In this pr, the report of dataflows' statistics is batched and scheduled to call prometheus API for better accessing performance.

/retest

@LykxSassinator is there anyone you prefer to review the PR?

@tonyxuqqi PTAL, thx

ping @disksing , PTAL, thx.


Write a Ruby program to find negative of a number

Description
Write a Ruby program to find negative of a number
Input  : 5
Output : -5

How to contribute

Save the solution in FindNegativeOfANumber.rb file
Add FindNegativeOfANumber.rb file in find-negative-of-a-number folder

!assign


Regression: the e-mail for a release no longer lists the release notes

What App Center service does this affect?
Distribution.
Describe the bug
When a release is made, App Center sends an e-mail. This e-mail used to contain the release notes. Now it doesn't.
To Reproduce
Steps to reproduce the behavior:
1. Publish a release

2. Wait for the e-mail

Expected behavior
The e-mail contains the notes to the release.
Screenshots

Desktop (please complete the following information):
* OS: macOS 10.14.3

* Browser Firefox 65.0.1

Hi @ddobrev
We couldn't repro the issue. Can you tell me a little more about the app or if you're still experiencing the issue?

Closing this for now. Feel free to create a ticket on intercom on appcenter.ms so we can help you further.


Validate labels in ANP/ACNP/CG

Fixes #3285.
This PR added some additional validations to ANP, ACNP and CG in
validating webhook. Antrea will validate labels used in ANP, ACNP and
CG under PodSelector, NamespaceSelector and ExternalEntitySelector.
Signed-off-by: wgrayson<EMAIL_ADDRESS>

LGTM, but do you think we should do the same for labelSelectors in ClusterGroups as well?

LGTM, but do you think we should do the same for labelSelectors in ClusterGroups as well?

Good catch. Added.

Codecov Report

Merging #3331 (cc03d67) into main (3f30d84) will increase coverage by 1.16%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3331      +/-   ##
==========================================
+ Coverage   60.85%   62.02%   +1.16%
==========================================
  Files         266      269       +3
  Lines       26520    26733     +213
==========================================
+ Hits        16139    16581     +442
+ Misses       8591     8340     -251
- Partials     1790     1812      +22

Flag
Coverage Δ

kind-e2e-tests
49.53% <33.33%> (+1.57%)
:arrow_up:

unit-tests
41.66% <100.00%> (+0.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/controller/networkpolicy/validate.go
48.32% <100.00%> (+7.12%)
:arrow_up:

pkg/controller/serviceexternalip/controller.go
66.47% <0.00%> (-5.12%)
:arrow_down:

pkg/controller/networkpolicy/status_controller.go
65.89% <0.00%> (-3.47%)
:arrow_down:

pkg/agent/nodeportlocal/k8s/npl_controller.go
58.70% <0.00%> (-2.05%)
:arrow_down:

pkg/agent/agent.go
51.34% <0.00%> (-0.36%)
:arrow_down:

...ster/commonarea/mock_remote_common_area_manager.go
20.83% <0.00%> (ø)

...multicluster/commonarea/mock_remote_common_area.go
30.23% <0.00%> (ø)

.../multicluster/mock_membercluster_status_manager.go
100.00% <0.00%> (ø)

pkg/agent/flowexporter/exporter/exporter.go
81.29% <0.00%> (+0.64%)
:arrow_up:

...ntroller/networkpolicy/networkpolicy_controller.go
71.34% <0.00%> (+0.91%)
:arrow_up:

... and 17 more

/test-all

/test-integration


About the pvalue.txt result

Hi, I see it described in the pvalue result for method2 in the notebook as, cell_a|cell_b: 1 if interaction is detected as significant, 0 if not.
But, in real output pvalue.txt result, contains values that are neither 0 nor 1. Is the value for the paired cell types in the pvalue.txt file the real p-value, or is it after processing based on the threshold？
Also, I saw that the pvaule.txt file was processed in the ktplot-py code, and found that the result of the pvalue was treated as the real pvalue. i'm a bit confused now

The notebook web site: https://github.com/ventolab/CellphoneDB/blob/master/notebooks/T01_Method2_with_subsampling.ipynb
and the ktplot-py code: https://github.com/zktuong/ktplotspy/blob/master/ktplotspy/plot/plot_cpdb.py line 263

Hi.
This is indeed wrong, this file contains the p-values derived from the random shuffling.
I've modified the notebook.
Thanks for noting the error.


Error generating .mjml

mjml -r test.mjml on mac command-line returns this error
Error: ENOENT, access 'test.mjml'
at Error (native)
mjml v. 1.2.1

Hey @kelvinaero !
can you paste the output of ls -l ?

-rw-r--r--  1 kelvinaero  staff    8271 Mar 16 11:32 body_promo.html
-rw-r--r--  1 kelvinaero  staff    2130 Mar 16 11:32 body_promo.mjml
-rw-r--r--@ 1 kelvinaero  staff    3697 Mar 15 15:24 btn-approved.jpg
-rw-r--r--@ 1 kelvinaero  staff    6606 Mar 15 15:23 btn-fb.jpg
-rw-r--r--@ 1 kelvinaero  staff   98319 Mar 15 16:19 contest-rules.jpg
-rw-r--r--@ 1 kelvinaero  staff  234177 Mar 15 16:19 enter-simple.jpg
-rw-r--r--  1 kelvinaero  staff     239 Mar 15 13:32 gulpfile.js
-rw-r--r--@ 1 kelvinaero  staff  164682 Mar 15 16:18 hero.jpg
-rw-r--r--@ 1 kelvinaero  staff   28388 Mar 15 16:18 img-logo.jpg
-rw-r--r--@ 1 kelvinaero  staff   79080 Mar 15 16:19 img-people.jpg
-rw-r--r--  1 kelvinaero  staff  599504 Mar 15 16:33 mar-1406-fb-promo.zip
drwxr-xr-x  5 kelvinaero  staff     170 Mar 15 10:34 node_modules
well I already created an .mjml file manually and copied some of the markup from one of your templates, just wanted to know why I cant generate a basic .mjml

You need to replace input.mjml by an mjml file in you case, I assume it would be
$ mjml -r body_promo.mjml

Well, I just parked inside this folder, but no matter how try it returns an error. Lets say I want to create version 2, I would use
$ mjml -r body_promo2.mjml
I still get:
Error: ENOENT, access 'body_promo2.mjml'
at Error (native)

mjml -r won't create the file, you need to 1) Create the file with your favorite Code Editor, then compile the file with mjml -r

Ok it looks like a documentation issue, in the end we ended up creating the file manually, but wanted to ask you.  just in case it was an error. It would be nice to be able to generate a skeleton file, but that is just a suggestion. Thank you.


fresh ought to respond to HEAD requests

currently it responds with 405
% curl -I https://fresh.deno.dev/
HTTP/2 405
accept: GET
vary: Accept-Encoding
date: Wed, 15 Feb 2023 08:54:06 GMT
via: http/2 edgeproxy-h
server: deno/gcp-asia-northeast1

We can / should write HEAD handler function in custom handlers, but not what you mean?
https://github.com/hashrock-sandbox/link-checker-test-fixture/blob/main/routes/page4.tsx#L13

Consider a case where a route has both HEAD and GET functions, for example:
export const handler: Handlers = {
  async HEAD(req, ctx) {
    console.log('HEAD')
    […]
  },

  async GET(req, ctx) {
    console.log('GET')
    […]
  },
};

In this case I would expect that curl --head for this route will run the HEAD function.
But currently in Fresh, a HEAD method request only runs the GET function. Any defined HEAD function is ignored. That seems like a bug to me.


Error while pulling image 2.0-sdk: Pulling from microsoft/dotnet

I get an error when i try to build iot edge module.
failed to register layer: re-exec error: exit status 1: output: remove \?\C:\ProgramData\Docker\windowsfilter\c4e27632c2a9d32834e6ba72d1ba7fe42ad3006662efdb182b7f332c3ac0de4d\UtilityVM\Files\Windows\System32\kerberos.dll: Access is denied.
i tried restarting my system and resetting docker does not help out. Am i missing something?

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

ID: 181d5562-f8d7-9c59-85ea-09efbe397f3e
Version Independent ID: dbedd952-a050-286e-c951-4529ec8e87b8
Content: Azure IoT Edge C# module
Content Source: articles/iot-edge/tutorial-csharp-module.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@guddu2112 Thanks for the feedback! We are currently investigating and will update you shortly.

@guddu2112 We were able to successfully build our IoT module using the doc. Since this seems to be an issue with your machine, we kindly request you to post your question to the MSDN forum. It would appear that there might be a permissions issue with that file or folder on your machine but it's tough to say.

I downgrades docker to 17.12 ce version and it worked. Could be problem
with latest docker 18 build not supported with iot edge?
On Tue, Apr 24, 2018, 03:48 Bryan Trach<EMAIL_ADDRESS>wrote:

@guddu2112 https://github.com/guddu2112 We were able to successfully
build our IoT module using the doc. Since this seems to be an issue with
your machine, we kindly request you to post your question to the MSDN
forum https://social.msdn.microsoft.com/Forums/en-US/home?forum=iotedge.
It would appear that there might be a permissions issue with that file or
folder on your machine but it's tough to say.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/7419#issuecomment-383740868,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ah12nD-CgqvVv-K7g6FfNBD2aY4t4QDkks5trlMbgaJpZM4TbKpd
.

dotnet new -i Microsoft.Azure.IoT.Edge.Module
A fatal error occurred, the required library hostfxr.dll could not be found at C:\Program Files\dotnet\

The library hostfxr.dll  is there


Maker service not stopping

As a user with a balance of zero, when I start the Maker service it shows the error Maker could not start, no coins. (request results in 409 Conflict).
However, the service starts anyway. When subsequently clicking "stop", it will display a success message The service is stopping. (request is even resulting in 202 Accepted).
Nevertheless, the service is not stopped and will stay this way indefinitely.
I do not know if we can do anything on the client, as this seems to be a server issue.
Just want to keep it here as reference - I will investigate the server code and open an issue in their repo if needed.

There's some more info in the issue I posted here: JoinMarket-Org/joinmarket-clientserver#1120

Fixed as https://github.com/JoinMarket-Org/joinmarket-clientserver/pull/1169 is merged. Eagerly awaiting v0.9.5 :rocket:


Doubt about the TLS certificates

Hi,
I don´t know much about certificates but I could not understand why you used "server.dc1.cluster.local" as CN and hosts. Where did you get this URL from? Wasn't it supposed to be the server URL? Something more like consul.$(NAMESPACE).svc.cluster.local where $(NAMESPACE) I should replace with the namespace?
Thanks in advance,
Paulo Leal

Hi,
The reason "server.dc1.cluster.local" is there, is because the "verify_server_hostname" option is enabled in the config.
verify_server_hostname - If set to true, Consul verifies for all outgoing connections that the TLS certificate presented by the servers matches "server.<datacenter>.<domain>" hostname. This implies verify_outgoing. By default, this is false, and Consul does not verify the hostname of the certificate, only that it is signed by a trusted CA. This setting is important to prevent a compromised client from being restarted as a server, and thus being able to perform a MITM attack or to be added as a Raft peer. This is new in 0.5.1.
https://www.consul.io/docs/agent/options.html#verify_server_hostname
The default name of the Consul datacenter is "dc1", the Kubernetes domain is "cluster.local".
Personally, I would include that name as a SAN in my SSL certificate, and not as a CN.

@jeroenjacobs1205 is right on the money.


sudo apt-get install rocm opencl-rocm does not work

Linux 16.04.02, AMD W9100 graphic card.
I follow the readme.md in ROCm project, on branch roc-1.5.0 .
I try to install opencl, but failed.

wget -qO - http://packages.amd.com/rocm/apt/debian/rocm.gpg.key | sudo apt-key add -
sudo sh -c 'echo deb [arch=amd64] http://packages.amd.com/rocm/apt/debian/ xenial main > /etc/apt/sources.list.d/rocm.list'
sudo apt-get update
sudo apt-get install rocm opencl-rocm
sudo apt-get install rocm opencl-rocm-dev

both 4 and 5 failed, can get the package.
I want to know why?

You only need to do 4 or 5
Greg
On Jun 16, 2017, at 4:38 AM, zhaojunfan <notifications<EMAIL_ADDRESS>wrote:
Linux 16.04.02, AMD W9100 graphic card.
I follow the readme.md in ROCm project, on branch roc-1.5.0 .
I try to install opencl, but failed.

wget -qO - http://packages.amd.com/rocm/apt/debian/rocm.gpg.key | sudo apt-key add -
sudo sh -c 'echo deb [arch=amd64] http://packages.amd.com/rocm/apt/debian/ xenial main > /etc/apt/sources.list.d/rocm.list'
sudo apt-get update
sudo apt-get install rocm opencl-rocm
sudo apt-get install rocm opencl-rocm-dev

both 4 and 5 failed, can get the package.
I want to know why?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/RadeonOpenCompute/ROCm/issues/130, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD8DuQPiA7x3ZqNhl2J5AH9lCwVHhbJ6ks5sEk0EgaJpZM4N8PnQ.

I did step 4, get the output like this:
mcw@mcw-All-Series-Invalid-entry-length-16-Fixed-up-to-11:~/fzj$ sudo apt-get install rocm opencl-rocm
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package opencl-rocm
I did step 5, get the output like this;
mcw@mcw-All-Series-Invalid-entry-length-16-Fixed-up-to-11:~/fzj$ sudo apt-get install rocm opencl-rocm-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package opencl-rocm-dev

Proper command is
sudo apt-get install rocm-opencl-dev

ROCm 1.6 is out now also we built new set of getting started instructions https://github.com/ROCm/ROCm.github.io/blob/master/ROCmInstall.md


Report improvements

The following reports are currently generated client-side from the applications collection. As such they are slow, or worse, with larger data sets (anything over a few hundred applications).
We should either make use of the more lightweight applicationRecords collection OR we should generate the report server-side as we do for Diversity and Qualifying Test Reports.

[ ] Reports. Reasonable Adjustments
[ ] Reports. Agency
[ ] Reports. Handover

Assigned to Halcyon in March's WO.

@JeremykJAC this task has some overlap with #1100. I'd suggest looking at the Diversity Report and the Exercises Export for possible ways to prepare the data server-side

Hi
As @warrensearle mentions above, the Handover and Reasonable Adjustments reports were covered in #1100, so I that just leaves the Agency report now.
Quick note though: The Agency report would show on-screen but the Export Button didn't actually do anything, it just stayed disabled and there was no code in place to construct the XLS data or download a file. It looks like the implementation for that hadn't been done yet that's all.
So... I have done the following:

Digital Platform - added a generateAgencyReport endpoint, which generates and stores the report in the database (under exercises.reports.agency)
Admin - Implemented the Export Data button.
Admin - Moved the Export Data button to be above the datatable, to match the design of other pages (i.e. Report - Reasonable Adjustments)
Admin - Added a Refresh button, also to match the design of the other pages.

If any of the above needs changing let me know! :)

Hi
One more thing...
I have got the the Auth / Login endpoint working in Postman now, so there will be no need to comment out the auth code temporarily anymore (which is what was doing in the screen-share the other day).
I will post a quick How To guide on the Slack channel.
Darrel

@JeremykJAC @HalcyonJAC We are yet to implement the export for the agency report as the plan was to export this data directly to particular angencies such as the Criminal Records Office, HMRC etc etc. That's a bigger piece of work that is in the roadmap for a later date.
As there is no code to enable any action here, am I right in thinking that testing only involves confirming that I can see the 'Export Data' button above the data table and see the newly added refresh button? I am able to see all of that.
Based on the fact that these buttons currently have no functionality, would it be possible to hide them on the UI for now? As this may cause confussion amongst users. When we complete the final stages of the work we can make them visible again.

@KateMJAC
I can confirm that this is working on 'develop' just fine. Here is a link to it, if it helps:
https://admin-develop.judicialappointments.digital/exercises/wdpALbyICL7ZxxN5AQt8/reports/agency#sra
I discussed this with Warren and he suspects that the problem is that the report depends on whether Character Checks part of the exercise has been completed or not and perhaps that part of the system isn't being used yet. i.e. the Agency report will be blank if not.

Thank you @HalcyonJAC I tested it on Develop and all works well.


Failed retries of failback should be judged first

What is the purpose of the change
related issue: #8715
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

I commit it on branch master in #8743


Add X-Forwarded-Port header in Zuul

Add X-Forwarded-Port header in Zuul PreDecorationFilter
even with X-Forwarded-Host header being absent.
This fixes #1807

Codecov Report

Merging #1808 into master will increase coverage by 0.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #1808      +/-   ##
==========================================
+ Coverage   75.35%   75.37%   +0.01%
==========================================
  Files         203      203
  Lines        6241     6241
  Branches      781      781
==========================================
+ Hits         4703     4704       +1
+ Misses       1198     1196       -2
- Partials      340      341       +1

Impacted Files
Coverage Δ

.../netflix/zuul/filters/pre/PreDecorationFilter.java
86.08% <50%> (ø)
:arrow_up:

...loud/netflix/hystrix/stream/HystrixStreamTask.java
75% <0%> (+0.46%)
:arrow_up:

Continue to review full report at Codecov.

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

Hello @ryanjbaxter ,
I added more unit tests :

setting http only (to see that port 80 is added)
setting port only (to see that only port is added)
setting port and proto.

Tell me is something missing !
Thanks,
Adrian


Fixed large json read corruption

GetSpan() worked when json is small
fixes #6683

Very well spotted ... Really great to have you as one of our contributors! If you post me your address and T-Shirt size via slack we will send you over a ChilliCream T-Shirt and some stickers.
slack.chillicream.com


Dataset Viewer issue for kietzmannlab/ecoset

Link
https://huggingface.co/datasets/kietzmannlab/ecoset
Description
The dataset viewer is not working for dataset kietzmannlab/ecoset.
Error details:
Error code:   ConfigNamesError
Exception:    ImportError
Message:      To be able to use kietzmannlab/ecoset, you need to install the following dependencies: boto3, botocore.
Please install them using 'pip install boto3 botocore' for instance.
Traceback:    The previous step failed, the error is copied to this step:  kind='/config-names' dataset='kietzmannlab/ecoset' config=None split=None---Traceback (most recent call last):
                File "/src/workers/datasets_based/src/datasets_based/workers/config_names.py", line 89, in compute_config_names_response
                  for config in sorted(get_dataset_config_names(path=dataset, use_auth_token=use_auth_token))
                File "/src/workers/datasets_based/.venv/lib/python3.9/site-packages/datasets/inspect.py", line 323, in get_dataset_config_names
                  dataset_module = dataset_module_factory(
                File "/src/workers/datasets_based/.venv/lib/python3.9/site-packages/datasets/load.py", line 1217, in dataset_module_factory
                  raise e1 from None
                File "/src/workers/datasets_based/.venv/lib/python3.9/site-packages/datasets/load.py", line 1188, in dataset_module_factory
                  return HubDatasetModuleFactoryWithScript(
                File "/src/workers/datasets_based/.venv/lib/python3.9/site-packages/datasets/load.py", line 907, in get_module
                  local_imports = _download_additional_modules(
                File "/src/workers/datasets_based/.venv/lib/python3.9/site-packages/datasets/load.py", line 221, in _download_additional_modules
                  raise ImportError(
              ImportError: To be able to use kietzmannlab/ecoset, you need to install the following dependencies: boto3, botocore.
              Please install them using 'pip install boto3 botocore' for instance.

Should we close since we will not support script based datasets?


Crash on server start

crash-2016-09-11_11.35.17-server.txt

We are aware I haven't pushed the fix to curseforge yet. I'll do it as soon as I am able to.
Thanks,
Dmodoomsirius

On Sep 11, 2016, at 11:30 AM, Jesse Smith-Rogers<EMAIL_ADDRESS>wrote:
crash-2016-09-11_11.35.17-server.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Might be Missing a check. I'll look at it soon.

Fixed.


Error when trying to build under Alpine (Musl)

Hi @etcimon.
Do you know if the lib should work on Alpine docker container?
I faced with this error:
/root/.dub/cache/libasync/~master/build/regular-release-RNnj5zspAaZtWLsA4BwBbA/libasync.a(libasync.posix.o): in function _D8libasync5posix13EventLoopImpl6notifyMFNbIiOCQBt6signal11AsyncSignalZb': posix.d:(.text._D8libasync5posix13EventLoopImpl6notifyMFNbIiOCQBt6signal11AsyncSignalZb+0x1d): undefined reference to pthread_sigqueue'

It seems like a linker issue, I think you need to link to a stub for pthreads to work in this docker.
https://github.com/alpinelinux/docker-alpine/issues/227
Have you tried linking with the supplied libpthread.a?

I've tried to use it. But I found the root of the problem (I suppose).
Must still doesn't have pthread_sigqueue.
And we don't have it for Musl in core as well.
https://github.com/dlang/dmd/blob/master/druntime/src/core/sys/posix/signal.d#L3056
So libasync is not compatible with Musl until someone implement it and upstream into main repo.

Btw BSD systems also don’t have this function. In the web people are mentioning that it is not very portable (only glibc probably has it).
For example, Microsoft rewrote their implementation with pthread_kill
https://github.com/dotnet/runtime/issues/4472


"font-size 16px" is way too small

Describe the bug
The generated 'style.css' has a font size of 16 pixels instead of 16 points, which is illegible even if you do not have a HDPI monitor.
To Reproduce
How can we reproduce the problem?  Please be specific.

What version of Python are you running?

3.7.3

What versions of what packages do you have installed? The output of pip freeze is very helpful.

4.5.4

What code are you running?  Give us a specific commit of a specific repo that we can check out.

n/a

What commands did you run?

I run this software via pytest-cov.
Expected behavior
I expect to get a coverage report in HTML, which I actually get. Just the styling is wrong.
The incriminating element is this:
 html>body {
-    font-size: 16px;
+    font-size: 16pt;
 }

But I can't seem to upload a patch to fix it everywhere.

You are right that the font-size should be defined in pt instead of px.  But unless my setup is very different from yours, 16pt seems too large.  I'm afraid there is no "right" answer to this question: it's going to depend on individual preferences.
You can define an extra_css file that will be included in your HTML report (https://coverage.readthedocs.io/en/v4.5.x/config.html#html) if you like, or you can use your browser controls to increase the size of the font.

I am currently (mostly) sitting in front of a 13" 1920x1080 machine, and there, the 16px amount to account for something like 2-4mm of text height (can't really tell, but it is very small). Yes, I can use browser controls, but I have to do that for every file I click on. I am not wedded to the 16pt, but please just make it something-point instead of always-wrong pixels (how about 14pt?). Having the font size in pt would allow the various desktops scale the fonts automatically, at least to some degree (I also have a HDPI laptop, where the font size would be probably like 1mm).

Do you mind attaching a screenshot of what you are seeing?
By way of reference, GitHub displays source code at the same size as coverage.py HTML reports (final size of 12px).  Does this page also look too small? https://github.com/nedbat/coveragepy/blob/master/coverage/html.py

Yes. The other fonts are also too small, esp. for permanent consumption. Effectively, the font size is about 1mm. Somehow, attaching a file does not seem to work, but I don't know why. I have to zoom in at ~180-200% for the text to be easily legible, which I can do by enforcing a minimum font size of 20pt - not something all websites can cope with. :|

@muellert I don't know why your experience is so different than other developers'.  I encourage you to set an extra_css stylesheet.  It will be automatically applied to your coverage.py HTML reports.

I've done so already. Thanks for the pointer...


Simplify modeling for builds and Sphinx context data

Considering our patterns for how we use model data in our builders, there are a few deficiencies:

The model code our builders use make use of the DONT_HIT_DB setting. But our builders should never hit the DB, they should always be API driven.
We hack up database modeling to mock out API Project objects, so that we can use some of the model methods. I feel like this should also be API driven, with all of these method instead becoming attributes, and the API returns these attributes hard coded
The context data is hodge podge mix of attributes. For instance, we don't pass on most of the project information, and we only pass on subproject slug and url, but no other fields. This should all be standardized, and perhaps, should just be data directly from our API
We pass a completely different set of data from the footer. This should be the same as the context data going in, and from the same source.

This will offer a few benefits:

If we standardize data, we can just drop the data as a dict directly into the Sphinx context. This allows us to also put the context data creation on a signal, to make this extensible.
Reduces confusion around builder being sometimes API driven, sometimes database driven. This is helpful for testing and replicating how we actually use the modeling
Allows for our stored context data, the data in the sphinx context, to match with the structure of the context we pass from the footer
The plan would be footer would be data driven, not just a blob of html. This is a separate thing. But data here would match the data already in the template

Solution

Convert methods without arguments to attributes on Project and return these selectively from the API.
Decide what to do with methods with arguments, if any. I lean towards making this API driven wherever we can. If this has to be a call on the API, i'm fine with that.
Consider what we need to return from the API for the build process (that is, full return) and what we return from the API for normal requests (for user data, and data into the Sphinx context). We can perhaps eliminate overlap
Make this a centralized class that both the footer data and Sphinx context rely on. Stop duplication of this code

This will take a larger audit of how we are using data from the API, and perhaps blends in with a larger API v3 project. I think we can keep this separate however.

Ping @ericholscher

Agreed 100%. We use a lot of the project modeling though in the build, which is why it's hard to break this out historically. This is a good starting point to completely breaking the builder code out, which seems like a useful end game.

I really want the builds to use other modeling instead of the Project, so it would be much more explicit what you can use that doesn't depend on hitting the DB.

This also relates to #3054, and I believe the full list of build data required is here: https://github.com/rtfd/readthedocs.org/pull/3054/files#diff-fab3f6c80eb0a5d20dc58fae29ac388dR28

Yeah, i was thinking the full build return Project API endpoint would just have the attribute fields appended there. I haven't done an audit to see how many argument methods we are using yet however.
I was also considering secondary modeling, though if we are able to make this 100% api driven, that might not be necessary. We'll just treat the objects as glorified dicts from Slumber. I can definitely see a case where we might want modeling though. I also started considering what a new API might look like, without getting too serious about it. I think the Way of The Future is to use OpenAPI and schema to auto generate a client. This would supplement the need for modeling on the client side -- again, if we're 100% api driven.


Convert wav to mp3 through ffmpeg

Since some Soundcloud tracks are downloaded as wav's, which means they can't be tagged. I always have to convert them to mp3 and then add their tags manually. So I'm just trying to add a way to auto-convert them and add their respective tags if so desired.
It currently uses just FFmpeg for the conversion (Might add Avconv at a later date, and allow the binary location's to be specified). I've tested this on both windows and Linux, and it seems to work fine on both.
If there is some python module for audio conversion out there, I'd switch to that. But, I personally don't know of any.
Edit: Added support for Avconv, and binary directory specification

I haven't been able to test it on OS X. But, I've given it a rough test on Linux, and a more in-depth test on windows.

Rather than add another dependency and a conversion step why not use the existing Mutagen, add a filetype check, then use APEv2 to tag the files?
Just curious.

I guess that would solve the problem of having un-tagged wav files. Though, another reason I wanted to convert, was because of how large wav files are compared to mp3, and because I wanted to try keep all the music in a single consistent format. But, that's more of a personal issue I guess.


import pathToRegexp from 'path-to-regexp'

I have a problem using this plugin and as soon as I use
import {Router} from 'marionette.routing';
Code builds but then I'm getting the error import pathToRegexp from 'path-to-regexp'
I'm guessing something was not setup correctly. I'm using brunch to build the app.

path-to-regexp is a dependency of cherrytreex which is a dependency of marionette.routing. The bundler should be able to resolve it.
There's one thing missing in cherrytreex: module field. I'll add to see if fix

Please update to version 1.1.0 and try again

Still no luck, but the funny thing is. The cherrytreex/cherrytree.js is transpiled and left with some import statement.
This is the beginning of the file
import pathToRegexp from 'path-to-regexp';

var toString = Object.prototype.toString;
var keys = Object.keys;
var assoc = function assoc(obj, attr, val) {
  obj[attr] = val;return obj;
};
var isArray = Array.isArray;

This is the intended for optimal output. Webpack, rollup handles that just fine


ADFS 2019 Support

This PR ports the changes in #1617 to msal-core and builds on #1582 and will close #99
Differences from AAD/B2C Authorities:

ADFS does not return ClientInfo, this PR builds uid from the subject claim on idToken instead
ADFS does not return preferred_username, this PR sets this to upn instead
ADFS uses .well-known/openid-configuration instead of 2.0/.well-known/openid-configuration

Coverage decreased (-0.3%) to 78.23% when pulling 520a46519e03a63f6c5ce77e2b09e3d4e20de3e4 on msal-core-adfs into 544414404bff325618644026ac5749646aea2e84 on dev.

LGTM, just a few nitpicks

So is this resolved or not?

@James-Randal ADFS 2019 is supported in<EMAIL_ADDRESS>If you have questions or find a bug please open a new issue. Thanks!

Is there an example of the configuration required to get this working with AD FS 2019? I'm trying to follow this tutorial but it's attempting to hit login.microsoftonline.com which returns a 400 error.
    MsalModule.forRoot(new PublicClientApplication({
      auth: {
        clientId: CLIENT_ID, // This is your client ID
        authority: ADFS_2019_ENDPOINT, // This is your tenant ID
        redirectUri: 'http://localhost:4200/' // This is your redirect URI
      },
      cache: {
        cacheLocation: 'localStorage',
        storeAuthStateInCookie: isIE, // Set to true for Internet Explorer 11
      }
    }), null, null)

ADFS_2019_ENDPOINT looks like 'https://staging.mydomain.com/adfs'
I would have expected that supplying a URL for the authority field would have overridden any references to login.microsoftonline.com, but am I doing something wrong?


Changed names of entities to better describe role and modified return…

Notes on this PR:
-changed names of StepManager --> ConfidenceScoreCalculator and
SuggestionManager --> ConfidenceScoreOptimizer to better satisfy their responsibilities as entities. We talked to Paul about this and he suggested names such as these.
-implemented in the ConfidenceScoreOptimizer a new condition that if the highest talk step's confidence score is less than 70%, then the top three results are returned instead. This makes sense because we are suggesting a variety of suitable talk steps when there is no clear one, and the user can make a informed choice accordingly.
I did some research on this and anything above and including 70% is a highly probable answer that will fulfill the user's query according to this Microsoft resource:
https://learn.microsoft.com/en-us/azure/cognitive-services/qnamaker/concepts/confidence-score
You can read my comments and documentations if you have any questions or concerns!

Fixed the branch issues


[FLINK-14776][Configuration] Migrate duration and memory size ConfigOptions in RocksDBConfigurableOptions

What is the purpose of the change
Migrate duration and memory size ConfigOptions in RocksDBConfigurableOptions
Brief change log

add new config BLOCK_SIZE_MEMORYSIZE, BLOCK_CACHE_SIZE_MEMORYSIZE, WRITE_BUFFER_SIZE_MEMORYSIZE, MAX_SIZE_LEVEL_BASE_MEMORYSIZE and TARGET_FILE_SIZE_BASE_MEMORYSIZE and mark as @PublicEvolving
deprecated the exist config
add new function and deprecated the exist function in DefaultConfigurableOptionsFactory.java which used the RocksDBConfigurableOptions

Verifying this change
This change added tests and can be verified as follows:

RocksDBStateBackendConfigTest#testConfigurableOptionsFromConfigWithMemorySize
RocksDBStateBackendConfigTest#testSetBothKeyAndDeprecatedKey

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (docs)

cc @dawidwys


Max bitrate typo

https://github.com/jhmaster2000/yet-another-discord-bot/blob/c94c86b5c0fa4ef8aaaba3ef4c7fb86c034dcbec/src/bot/commands/newchannel.js#L26

Fixed by https://github.com/jhmaster2000/yet-another-discord-bot/commit/1e8cf30b0533fd85d19e2eb795d7bf2b1bc35aab


How to run whisper-jax on CPU-only

Hello, I have installed jax and jaxlib but when I run the application with whisper-jax, it states:
is there a way to load whisper_jax so that is does not look for CUDA?
external/org_tensorflow/tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'cudart64_110.dll'; dlerror: cudart64_110.dll not found
2023-05-13 13:12:11.060560: W external/org_tensorflow/tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'cudart64_110.dll'; dlerror: cudart64_110.dll not found
Traceback (most recent call last):

Hello, Could you share the code that are you running?

@VincentRingstrom
I recommend you to start from python:buster docker.
docker pull python:buster

docker run --name jax_cpu -v <host_drive>:<docker_drive> -it python:buster bash
Under the docker you can run following installation commands and then you are ready to run under CPU. Good luck
pip install --upgrade pip
pip install --upgrade "jax[cpu]"
pip install torch
pip install git+https://github.com/sanchit-gandhi/whisper-jax.git


Add Travis CI

I tested this on my Travis account and the build passed, so this should be good to go.
(Again let me know if you have an alternative solution that you'd rather use.)

AFAIK a webhook isn't required to trigger builds. I just went here and threw the switch next to the repo and it seemed to work.


DrugBank: refactoring

Refactor DrugBank package so it doesn't freeze UI and works much faster.

Done in commit 17005337db5f78e08fa205bc6eee6b58fa0456ab.