Clarification for " Superseded update" resolution

Hi,
The steps described in the Scenario "Superseded update indicated as missing in Update Management" do not make complete sense in the context of the Azure Portal.
In step 1 - It directs to the Automation Account, and select update management.
In Step 2 - it prompts to verify the update is truly superseded
In step 3 - it says to mark the update as "declined". However there is no option I can see to change the status of the update, or to change the field "approved" to false. There is no reference in the docs. And clicking on the update sends us to a query in the log analytics.
Could you please amend to give more details on how to set the update as "declined"?
Thanks,

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

ID: c53fae91-9d02-f626-dd03-7c532bcffe38
Version Independent ID: 10756920-a188-fd6f-c9c9-910d6ce27673
Content: Troubleshoot Azure Automation Update Management issues
Content Source: articles/automation/troubleshoot/update-management.md
Service: automation
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@attig78 Thank you for your query, we will review and get back to you at the earliest.

@attig78 - What is documented as resolution for this scenario is outdated and inaccurate. I believe the original author of this scenario was trying to specify that if your machines are configured to report to WSUS to get update information and download/install updates, and superseded updates are reported as missing, you need to go into WSUS to decline those updates.  Additionally, if I remember correctly, UM doesn't allow you to specify updates that have been superseded for inclusion in the update deployment.
Please let me know if this explanation helped. In the meantime, I'm going to bug this article.
Thanks.

Hi Matt,
Yes,  I presumed this scenario was for WSUS rather than Azure Update. It would be great to update this article to at the very least mention it relates to WSUS. Even better, include a solution, if it exists, for customers that are exclusively using Azure Update, via GUI or powershell.
Thanks
Emir Attig
Get Outlook for Androidhttps://aka.ms/ghei36

From: Matt Goedtel<EMAIL_ADDRESS>Sent: Thursday, September 17, 2020 3:19:25 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Emir Attig<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Clarification for " Superseded update" resolution (#62768)
@attig78https://github.com/attig78 - What is documented as resolution for this scenario is outdated and inaccurate. I believe the original author of this scenario was trying to specify that if your machines are configured to report to WSUS to get update information and download/install updates, and superseded updates are reported as missing, you need to go into WSUS to decline those updates. Additionally, if I remember correctly, UM doesn't allow you to specify updates that have been superseded for inclusion in the update deployment.
Please let me know if this explanation helped. In the meantime, I'm going to bug this article.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/62768#issuecomment-694268543, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AN2YSD346MFGJZQEUJBQ3RDSGILG3ANCNFSM4RPNVHDQ.

This article has been updated and should go live later today or tomorrow AM.  #please-close


repo downlevel  says wyoming 1.2, uplevel componentts can't connect

trying to test config of the snd-external service, my test client using wyoming 1.5.2 cannot connect

fixed in #10


Update CodeQL bundle to 20210907 / 2.6.1

Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[x] Confirm the changelog has been updated if necessary.

https://github.com/github/codeql-action/pull/733/checks?check_run_id=3538712488 looks transient, let's rerun.


Can't create git repo in macOS

Hi
Error in CLI and API.
Seems that I can't chat anymore with people I added before I reinstall my Mac.
I tried Keybase long time ago, delete it, and then I try to use it to test git and teams features.
My log id: 270c0b1cffb26c81d2bdaa1c
Thanks!

Can you send a screenshot?

yeah, it's because you reset your account and have all new keys, it's cryptographically impossible for you to gain access to your old data until you are rekeyed by your collaborators. We warn pretty heavily about this on reset.

I understand about the chat/files part.
But why I have the same problem with git?

Git is build on top of the file system. Sorry for any confusion there

Ok, thanks.
What can I do to fix this?
I need a new account?

No, i think the people you're working with need to come back online and rekey for you.  @strib do you know the specifics of how a git repo should work with a reset user?

@JonaDupIO: from your logs, it looks like you're trying to access a team repo, not a multi-user repo (like your screenshot above implies).  That should work, assuming the repo has been created via the GUI or command line, and your current, post-reset account is a member of that team.
What output are you getting on the git command line?

➜  test keybase git create test
libkb.NeedSelfRekeyError{Tlf:"", Msg:"This device does not yet have read access to directory /keybase/private/jonathandupuich, log into Keybase from one of your other devices to grant access: --- is not readable by jonathandupuich (uid:---)\ngithub.com/keybase/kbfs/libkbfs.isReadableOrError\n\t/Users/build/go/src/github.com/keybase/kbfs/libkbfs/md_util.go:74\ngithub.com/keybase/kbfs/libkbfs.(*KBFSOpsStandard).getMaybeCreateRootNode\n\t/Users/build/go/src/github.com/keybase/kbfs/libkbfs/kbfs_ops.go:480\ngithub.com/keybase/kbfs/libkbfs.(*KBFSOpsStandard).GetOrCreateRootNode\n\t/Users/build/go/src/github.com/keybase/kbfs/libkbfs/kbfs_ops.go:518\ngithub.com/keybase/kbfs/libgit.getOrCreateRepoAndID\n\t/Users/build/go/src/github.com/keybase/kbfs/libgit/repo.go:214\ngithub.com/keybase/kbfs/libgit.CreateRepoAndID\n\t/Users/build/go/src/github.com/keybase/kbfs/libgit/repo.go:328\ngithub.com/keybase/kbfs/libgit.(*RPCHandler).CreateRepo\n\t/Users/build/go/src/github.com/keybase/kbfs/libgit/rpc.go:172\ngithub.com/keybase/kbfs/vendor/github.com/keybase/client/go/protocol/keybase1.KBFSGitProtocol.func2\n\t/Users/build/go/src/github.com/keybase/kbfs/vendor/github.com/keybase/client/go/protocol/keybase1/kbfs_git.go:58\ngithub.com/keybase/kbfs/vendor/github.com/keybase/go-framed-msgpack-rpc/rpc.(*callRequest).Serve\n\t/Users/build/go/src/github.com/keybase/kbfs/vendor/github.com/keybase/go-framed-msgpack-rpc/rpc/request.go:76\ngithub.com/keybase/kbfs/vendor/github.com/keybase/go-framed-msgpack-rpc/rpc.(*receiveHandler).handleReceiveDispatch.func1\n\t/Users/build/go/src/github.com/keybase/kbfs/vendor/github.com/keybase/go-framed-msgpack-rpc/rpc/receiver.go:121\nruntime.goexit\n\t/usr/local/Cellar/go/1.9/libexec/src/runtime/asm_amd64.s:2337"}

Ohh, I see, this is just your personal private folder, I was confused, sorry.
If you want us to reset your folder, please run this command on an currently-valid keybase device, and substitute the current date and time where indicated:
keybase sign -m "<DATE_AND_TIME>: Please reset folder /keybase/private/jonathandupuich because all the devices that have access have been revoked."

and post the results here.

BEGIN KEYBASE SALTPACK SIGNED MESSAGE. kXR7VktZdyH7rvq v5weRa0zkKNwdzf kcTHO5jd0qLyPC2 g6qzhri8kzFi6wZ 5xZh4ZXRXGrLFou vsjGfynTJ6fQiYj INvMe7EpJGuL5DP qv23Puy6Nm5PSnY SKc9baeAybVdEHh NrNG1cPcS5xn7qE yesQaR5kEmiNiN8 tEJx0BWjc5Eqi38 6qc0YhaFjaJ9Da4 dLqrbxZ4zH2QMOb Iygf47f5LThCulN 7tpmOVAMX9a27TL ZhHGSVH7FfEGN0o 6qoP0WFLYdzPDZl spfLRDRhGe89vmN r8DkdwE2uQmDkFH oPdgswPyquuTH19 XCOdFa3P110wTzF 3tOqauDe2nWZtwC FMilwOy5VjrLDbZ T0Y702A. END KEYBASE SALTPACK SIGNED MESSAGE.
Thanks a lot!

I reset the folder.  Please check it out and close this if everything seems ok.  (Might require a Keybase restart.)

Hi @strib!
I have the same problem! Please reset my folder:

BEGIN KEYBASE SALTPACK SIGNED MESSAGE. kXR7VktZdyH7rvq v5weRa0zkIj8oen zlvptRsys9Rlh4o YLGptVFKC9qrxq0 l0iPiQWO1QWYbUu Q2UgJizsd4W9rdA 3Up8yWZSMot420z KBHXguASs0IdXLJ prEsonY9QJe6Gyb FcdnAZRiyFaang5 iMMuhCWueoRLu0n GAnaRCzdVNIirGV V3XlKamG3oABxQn TN2O12W2uKkJdnZ RVvLEJ4OQO97AIo XwSyMjs05DruYA7 4w8Jw8OlGxs29KU pj7S4RdVknTaSDO IeZbAYPUjlHzMb7 I9ZGwo4kZEF5LFC qNo2Lvhn5JaNDLB wX7MWE0Po1DGxzH i6aSrK3SAItsrF9 t1GFjoZG7lhFRm. END KEYBASE SALTPACK SIGNED MESSAGE.

@maksim77: I reset that folder too.

@strib
I've the same problem with my private folder. Could you please reset my folder?

BEGIN KEYBASE SALTPACK SIGNED MESSAGE. kXR7VktZdyH7rvq v5weRa0zkTKTw0I IX1um4NXneemjTh F0NZGRDQ2OyNoCr eQ5QTyI6MqpbCrm qsu6D7LmGkb1FfV Esn4gsANCn2NO04 GlovQ1cW4yFIWKL 5YtpcpVDIbMx8kG HJXNNcEANTeetP1 VGVMAfG8XNnL0Lm DmLQ7oa00WPV5Ty GHfBCLlGLTxg9Vv uYWPYmDhwO5A7FQ Wjx37Jl6bBLgTLH AO1D5zaVdlKqNpL BbgP4TbubSJfepe oKLSmbWfBwEksAP 1mMwCIjQgSrHX3t yG4bEq0mccytZW2 H7g3sRaBJPKPWaN G2wwVjg1xA7pZXA 78yg95zY5VoUoVG QSLE1Z0paOpCaO0 E01sxRGe1XC. END KEYBASE SALTPACK SIGNED MESSAGE.

@samkhawse: done.

I'm having the same problem after I reinstalled my mac. I'm trying to create a new personal git repo. Any help is much appreciated!


[HUDI-4314] Improve the performance of reading from the specified ins…

…tant when the Flink streaming read application starts
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

@hudi-bot run azure


check for getEntriesByName in performance before calling

Resolves https://github.com/GoogleChrome/web-vitals/issues/159
Fixes the web-vitals package breaking applications in Opera Mini.
getEntriesByName is not supported in some version of Opera.
First reported here https://github.com/bbc/simorgh/pull/9153

Thanks for the report! Can you sign the CLA so I can merge? Otherwise I'm happy to just make the change myself. Either way :)

@googlebot I signed it!

Thanks!


Refactor GAPIC's BUILD.bazel files

The BUILD.bazel files are fairly cookie cutter. They are generated as scaffolding, then maintained by hand. Making sweeping changes across them (e.g. defining public headers) can be cumbersome. Factoring out the definitions of these components will help reduce future maintenance costs.

Any library that is using our CMake helper can probably benefit from this helper. To get a list of libraries, I used:
grep -r "gapic_library" --include "CMakeLists.txt" -l | xargs dirname | sort

On first glance, I only see nuance in:

bigquery, iam: handwritten samples that need a dep on the mocks.
securitycenter: Not compatible on Windows, although @devbww tells me it will be compatible when we pick up Protobuf v27

Closing.
Only securitycenter remains. It is blocked externally on Protobuf v27. There is a v27-rc3 out now. I will just keep track of this in my head.


added back removed chainId

admins merged a pr which deleted our chainId
https://github.com/moraswap/tokens/commit/ba49789bb5efdb5d90b04b16378167d2118662ff

Our mistake, so sorry about that.


Is all the Supplementary Data getting used?

Describe the bug/issue
Supplementary data folder is still using naming conventions from old folder structure
Check if the data is still necessary, and either delete it or update paths

I think it is - those numbered folders refer to the numbered notebooks in the Beginners guide, not the older numbered folders we used to have. 🙂 https://github.com/GeoscienceAustralia/dea-notebooks/tree/develop/Beginners_guide


枚举实例‘DOTDOTDOT’，与其name属性表意不一致

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

这个PR基于旧版，不能通过合并消除版本冲突了，重写一个 #5617


Misc fixes (.mp4 play back, upload flv/jpeg, validator fix)

Little fixes - removal of undefined variables, fix .mp4 media display, allow more things to be uploaded media.

I'm going to close this as it's been over a year now so suspect things have been fixed elsewhere or versions have moved on...


Bump version to 0.3.24

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Document how to use Ruby interface

See the discussion in https://github.com/ignitionrobotics/ign-math/issues/134#issuecomment-659048663.
We have Ruby bindings, but it's not easy to find out how to use them.
Things that need to be documented:

Requirements (i.e. ruby-dev and swig)
Usage
How to test

Planning to start on this issue today but I'm little lost on how to do it. Must I put the documentation in README using master branch?

Thanks, @luccosta . The README sounds good to me.
I think you can mention the required dependencies under source install prerequisites.
And I recommend targeting the ign-math6 branch, and we'll merge it forward afterwards.

Thanks for the guidance @chapulina . I dont know if the style used was the expected, but love to hear advices to improve documentation.


[Feature] Custom Serialization code needs performance tests

What is the bug?
The Custom Serialization logic was added to the security plugin codebase in an effort to improve serialization speed in different scenarios; however, a scenario was found where there was a significant regression [1], there should be a suite of tests that could catch and/or add check to ensure this doesn't happen in the future.
How can one reproduce the bug?
Inspect the gradle tasks and pull request checks for tests that measure the runtime performance of the serialization code in the Security Plugin - don't find any.
What is the expected behavior?
There should be a set of performance test cases that can validate the baseline and the behavior with the feature enabled to be sure no significant latency increases occur.
Do you have any additional context?

[1] Coming from https://github.com/opensearch-project/security/issues/3776
POC for a performance test https://github.com/opensearch-project/security/compare/main...peternied:security:dls-perf

I think irrespective of custom serialization, it will be good to have JMH.
I've a commit which I can help to productionize: https://github.com/opensearch-project/security/compare/main...mgodwan:security:jmh

[Triage] Thank you for filing @peternied. Having performance tests in general for optimizations will help measure the benefit of PRs that provide optimizations w/o any end user change.


fixed config.yml file

fixed config.yml file

👷 Deploy Preview for quiet-pie-34a01b processing.

Name
Link

🔨 Latest commit
5759571c79e0a397aa58c2e5f7edcc5e13557c2c

🔍 Latest deploy log
https://app.netlify.com/sites/quiet-pie-34a01b/deploys/65461bc62444fa0008cc3a17


bug in Shape.size for dim == NumDimensions

System information
This does not matter.
Describe the problem
when dim equals s.NumDimensions(), the function should return -1, instead it panics.
Source code / logs
if dim < 0 || dim > s.NumDimensions() {

should be:
if dim < 0 || dim >= s.NumDimensions() {

Thanks for the report, please specify the filename (link is even better) in the future to make it clear what you are talking about.  Thanks!


Update clarity-protobuf

In the latest patch, the protobuf was updated, and the hero_id was changed from uint32 to int32, which caused the hero_id in the parser to become incorrect.

According to the protobuf documentation under https://protobuf.dev/programming-guides/encoding/ there should be no difference in encoding here:

As you saw in the previous section, all the protocol buffer types associated with wire type 0 are encoded as varints. However, varints are unsigned, so the different signed types, sint32 and sint64 vs int32 or int64, encode negative integers differently.

The intN types encode negative numbers as two’s complement, which means that, as unsigned, 64-bit integers, they have their highest bit set.

Are you getting negative numbers since that patch?

Confirmed. All replays are currently broken.

@spheenik
All IDs are multiplied by 2, I assume this is related to storing the number in binary code.
11 -> 110 (3 -> 6)

Hmm.
I don't know how to change this.
As per protobuf spec, unsigned and signed integers are encoded in exactly the same way.
Which means even if I change the protobufs, you will still get the wrong value.
@tsproskurnin Have you checked that the binary encoding has changed?
Protobuf encodes at least 8 bits for each signed or unsigned int, where 7 bits are payload.
Both 3 and 6 can easily be encoded in 7 bits.
Valve always do stuff like this.

No, have not checked binary encoding.
So, you right about encoding signed and unsigned numbers. Then i have no idea why it encode wrong in calrity.

But I found that the hero_Id of picks_bans in Demo.CDemoFileInfo info = Clarity.infoForFile(args[0]) is correct. I'm not sure if this helps

The thing is there could be two things leading to this:

clarity decodes wrong value
CS client encodes wrong value, which is decoded by clarity correctly

I suspect it to be 2. Maybe the lower bit (which causes multiplication by 2) encodes something new? Do you see the lowest bit to be 1 in any case?

But I found that the hero_Id of picks_bans in Demo.CDemoFileInfo info = Clarity.infoForFile(args[0]) is correct. I'm not sure if this helps

But isn't this the only place where the CHeroSelectEvent is used? @tsproskurnin: Can you confirm that you also use infoForFile()?

No, im not using infoForFile()

Then how do you get at a CHeroSelectEvent? Is it an embedded message?

Well, so we are talking about 'm_nSelectedHeroIDinCDOTA_PlayerResource? Because this seems to have been encoded as a IntVarSignedDecoder, and Valve seem to have changed this to be a HeroID_t`.
That way, I could just implement a different encoder to fix this.
@tsproskurnin Can you confirm this is what you also mean?

Yes, thats it.
Also it may be same problem with "m_pGameRules.m_BannedHeroes and m_pGameRules.m_SelectedHeroes in CDOTAGamerulesProxy

Any quick fix I can make to get parsing running for STRATZ?

@howardchung Hey, how is odota still parsing when clarity is not.

Any quick fix I can make to get parsing running for STRATZ?

you can get real hero names by getting entity of "SelectedHero" and checking its class name so you can map it with your own list of heroname=>id

I have not investigated yet, seeing elevated errors but not 100% failure. It's possible we're only using the hero ID in a non-critical way

I'm unsure about the fix. Doesn't this break old replays?
I will have to test first, but I do not have time at the moment.
As a workaround, you could check if the type is HeroID_t and divide by 2?

I have not tested an old replay, but it works for new replays.

So, I finally came aroung to testing this:
I created two messages:
message TestInt {
  optional int32 value = 1;
}

message TestUInt {
  optional uint32 value = 1;
}

and created the following code to test this:
import skadistats.clarity.wire.csgo.s2.proto.CSGOS2ClarityMessages;
import java.util.Arrays;
public class Test {
    public static void main(String[] args) {
        var intMsg = CSGOS2ClarityMessages.TestInt.newBuilder()
                .setValue(987654321)
                .build()
                .toByteArray();

        var uintMsg = CSGOS2ClarityMessages.TestUInt.newBuilder()
                .setValue(987654321)
                .build()
                .toByteArray();

        System.out.println(Arrays.equals(intMsg, uintMsg));
    }
}

The output is true, which means the problem is that Valve decided to change the semantics of the field, which is something I will not address in clarity.

fixed in 3.1.0


Update ArgumentParser dependency in Package.swift

1.2.2 is the version of ArgumentParser used by the toolchain, according to the checkout config: https://github.com/apple/swift/blob/277674a642b058fe95c37e24d5c861c5b81b6a82/utils/update_checkout/update-checkout-config.json#L103
Sticking to the old versions like 1.0.x and 1.1.x has negative downstream effects, since tools using SwiftSyntax (like swift-format) are restricted to these versions too, which can easily conflict with other packages.
A more general solution could be to rely on SemVer and use .upToNextMajor instead of .upToNextMinor.

@swift-ci cit

@swift-ci test

@swift-ci test macOS


Add helpers for adding and removing recipients from a group DM.

These endpoints are missing a helper function:
https://discord.com/developers/docs/resources/channel#group-dm-add-recipient
https://discord.com/developers/docs/resources/channel#group-dm-remove-recipient

These are OAuth2 endpoint. DD currently does not support that.

These are OAuth2 endpoint. DD currently does not support that.

The 'Group DM Add Recipient' is already used in the getDmChannel() helper.

No they dont


SA1009 conflicts with range operator

If you have something like:
array[(index - 1)..]

SA1009 wants it to be:
array[(index - 1) ..]

Duplicate of #3064


llama 2 70b gptq use too much cpu memory

I want to test https://huggingface.co/TheBloke/Llama-2-70B-Chat-GPTQ. I can run the following code in an a100 40GB gpu:
from transformers import AutoModelForCausalLM, AutoTokenizer, pipeline

model_name_or_path = "TheBloke/Llama-2-70B-chat-GPTQ"
# To use a different branch, change revision
# For example: revision="main"
model = AutoModelForCausalLM.from_pretrained(model_name_or_path,
                                             device_map="auto",
                                             trust_remote_code=False,
                                             revision="main")

tokenizer = AutoTokenizer.from_pretrained(model_name_or_path, use_fast=True)

prompt = "Tell me about AI"
prompt_template=f'''[INST] <<SYS>>
You are a helpful, respectful and honest assistant. Always answer as helpfully as possible, while being safe.  Your answers should not include any harmful, unethical, racist, sexist, toxic, dangerous, or illegal content. Please ensure that your responses are socially unbiased and positive in nature. If a question does not make any sense, or is not factually coherent, explain why instead of answering something not correct. If you don't know the answer to a question, please don't share false information.
<</SYS>>
{prompt}[/INST]

'''

print("\n\n*** Generate:")

input_ids = tokenizer(prompt_template, return_tensors='pt').input_ids.cuda()
output = model.generate(inputs=input_ids, temperature=0.7, do_sample=True, top_p=0.95, top_k=40, max_new_tokens=512)
print(tokenizer.decode(output[0]))

But when I use the following command to test:
python main.py     --model hf-causal     --model_args pretrained=/nas/lili/models_hf/Llama-2-70B-chat-GPTQ/    --tasks hellaswag     --device cuda:0

It don't use gpu, but it use cpu and cause OOM of cpu.

I am also observing the same issue. My CPU is being used rather than GPU
python main.py --model hf-causal-experimental --model_args pretrained=openlm-research/open_llama_3b,dtype=float16 --tasks truthfulqa_gen --batch_size 4 --output_path "out/results.json" --device cuda:0

Does this issue persist on the big-refactor branch?

Closing as stale.


Screen reader does not read documentation link

Issue Type: Bug
Testing https://github.com/microsoft/vscode/issues/89320
Create an extension that uses the new diagnostic link API:
export function activate(context: vscode.ExtensionContext) {
	const fooErrors = vscode.languages.createDiagnosticCollection('foo');
	let disposable = vscode.commands.registerCommand('extension.helloWorld', () => {
		fooErrors.clear();

		const activeDocUri = vscode.window.activeTextEditor!.document.uri;

		fooErrors.set(activeDocUri, [
			{
				range: new vscode.Range(0, 0, 0, 5),
				message: 'You should always use semicolon;',
				source: 'eslint',
				code: '@typescript-eslint/semi',
				code2: {
					value: '@typescript-eslint/semi',
					link: vscode.Uri.parse('https://eslint.org/docs/rules/semi')
				},
				severity: vscode.DiagnosticSeverity.Error,
			},
			{
				range: new vscode.Range(1, 0, 1, 5),
				// message: 'Ensures that double quoted strings are passed to a localize call\nto provide proper strings for\ndifferent locales',
				message: 'Ensures that double\nquoted strings are passed to a localize call to provide proper strings for different locales',
				source: 'eslint',
				code2: {
					value: 'code-no-unexternalized-strings',
					link: vscode.Uri.parse('https://github.com/microsoft/tslint-microsoft-contrib/blob/master/src/noUnexternalizedStringsRule.ts')
				},
				severity: vscode.DiagnosticSeverity.Error,
			}
		]);
	});

	context.subscriptions.push(disposable);
}

Turn on NVDA.
In the problems panel, when navigating through the diagnostics, nothing gets read to indicate that there is a link that could be followed.
In the problems peek view (F8), the "value" text is repeated three times
VS Code version: Code - Insiders 1.42.0-insider (c4b6a6b537de49a734d0eedea06152ea37dfdb6a, 2020-01-28T06:36:30.779Z)

I will take care of this.

Actually in Problems view, when you tab on the message it takes to the link.
@isidorn Can you help me in understanding what is the right thing to do here. May be can we get validated with some accessibility expert?

I just tried this out, and what is the actual link?
From trying out @RMacfarlane sample what I see wrong is that you have this expand control to expand multi line elements. Please set the aria-expanded property to reflect if the multi line element is expanded.
Is the link no rendered for me, here's what I see

fyi @jvesouza since I know you use the problems view and you might have additional feedback on it

When I access the list of problems and press the tab key, the focus is moved to the show fixes button. I press the enter key but nothing seems to happen.

@jvesouza thanks, that sounds like a bug. @sandy081 can you reproduce this?

I can't and enter key opens the quick fixes for me.

Please set the aria-expanded property to reflect if the multi line element is expanded.

Sure
Please try with following extension sample
export function activate(context: vscode.ExtensionContext) {

	const collection = vscode.languages.createDiagnosticCollection('test');
	const uri = vscode.workspace.workspaceFolders![0].uri.with({ path: path.join(vscode.workspace.workspaceFolders![0].uri.path, 'package.json') });
	collection.set(uri, [createDiagnostic(uri, new vscode.Range(new vscode.Position(3, 4), new vscode.Position(3, 10)))]);

	vscode.window.onDidChangeActiveTextEditor(e => {
		if (vscode.window.activeTextEditor) {
			collection.set(vscode.window.activeTextEditor.document.uri, [createDiagnostic(vscode.window.activeTextEditor.document.uri, new vscode.Range(new vscode.Position(3, 5), new vscode.Position(3, 10)))]);
		}
	});
}

function createDiagnostic(uri: vscode.Uri, range: vscode.Range): vscode.Diagnostic {
	return {
		message: 'this is a sample error',
		range,
		severity: vscode.DiagnosticSeverity.Error,
		source: 'abcd',
		code: {
			value: '@typescript-eslint/semi',
			target: vscode.Uri.parse('https://eslint.org/docs/rules/semi')
		},
	};
}

@sandy081 then it is probably fixed with our other improvements in the markers panel.
Feel free to close once you add the aria-expanded. Thanks!


Making an optimization in the Signature Pad View

Found that the GetImage methods were spending most of the time in the
Points property building the array.
The Points property has been improved to not built several temporary arrays.
getCroppedRectangle has been changed to only call Points once and reuse the
result.
IsBlank property should not have to calculate the entire array, only find one
point and return false.
This fix is in response to case 121796

This looks good.


Hyundai FPv2: reduce number of ECU queries

Splitting out, will merge support first with no fingerprinting changes.
This was inferred by cross referencing which fw versions changed here when adding LONG https://github.com/commaai/openpilot/commit/b8cc31f888d4301b7db59ff9419db9215868a3f7 and which were changed when removing SHORT: https://github.com/commaai/openpilot/pull/22344
Assumptions were wrong, updated based on result from Optima here: https://github.com/commaai/openpilot/pull/24733
TODO:

[ ] Figure out which request engine responds to for Hyundai, it also responds to Nissan and Mazda's requests
[ ] Some cars' fwdRadar do not respond at all to any of our requests (service not supported)

[ ]  Some cars' fwdRadar do not respond at all to any of our requests (service not supported)

If you get local or remote hands on one, try first sending a session change to extended (from default).

Checked the first segment of all qlog routes in the last 14 days, there were no instances where a VW ECU returned with a fw version that wasn't what were were expecting in this PR 🥳:
Checked platforms:
{'VOLKSWAGEN PASSAT 8TH GEN': 720, 'VOLKSWAGEN TIGUAN 2ND GEN': 155, 'SKODA SUPERB 3RD GEN': 55, 'VOLKSWAGEN JETTA 7TH GEN': 135, 'VOLKSWAGEN ATLAS 1ST GEN': 65}
Checked dongles:
{'8dbccfa899c0d1b8': 550, 'c439baf36d9b6107': 150, '7291d51974eb2d1d': 55, '46e8fc71a2f4298d': 165, '4bd71e1011d03cb0': 135, 'bbc2fa573509b5ff': 5, '8d09dff0af485a2c': 65, 'c78c1972adfa10df': 5}


Canvas

How to use canvas in Jaspr? A light Canvas will be good for website.
Thanks

Sorry, I don't have a good answer for this one right now.
Jaspr does not implement any canvas functionality right now. But you can use dart:html to manually work with a canvas.


missing reference for Plugin.CurrentActivity

missing reference when i install via Nuget for Plugin.Activity v 1.0.1 for xamarin android project
Bug Information
Version Number of Plugin: v1.0.1
Device Tested On: will not compile
Simulator Tested On: will not compile
Version of VS: 2017 pro
Version of Xamarin: <IP_ADDRESS>6
if i down grade to a previous version reference is included.  assembly is not included in packages directory

Works for me just fine, haven't ever had any issues. maybe uninstall and re-install, should be in lib folder

well its not on mine and I don't have your computer.  don't be so quick to make assumptions.  as you can see from the screen shots its missing in my packages folder.  I uninstalled, reinstalled, down and upgraded, nothing pulled it down.

started a new project and same thing.  only nuget package installed was Xam.Plugins.GeoLocator

That is what I just did:

Did you just try manually adding the dll in the lib folder?

at first I thought the same thing.  the assembly is missing from the lib folder.  just the xml file is there.  I am using VS 2017 if that matters
are there android specific settings that may effect this?  something else I may have missed?

Are you correctly compiling against API 25+?

Seems like a nuget issue.

What if you just try to install the Current Activity Plugin separate?

I created a new project and installed just the CurrentActivity plugin and had the same issue

installed a pre version of CurrentActivity first then Geolocator, now the assembly is included

hmmmm there are no pre-versions... https://www.nuget.org/packages/Plugin.CurrentActivity/
Something is wrong with your NuGet for sure.. glad you go tit working though....

cleared nuget cache and that solved the problem

Hello guys, I am getting the same issue after doing all workaround that you have followed also after clearing the NuGet cache. Please suggest if you have an alternative approach to resolve the issue.
Error details.
Plugin.CurrentActivity.CrossCurrentActivity.Current.Activity
The type 'Activity' is defined in an assembly that is not referenced. You must add a reference to assembly 'Mono.Android, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=84e04ff9cfb79065'

Same for me. I went crazy... then I restarted VS and it was gone... :expressionless: :rage:

@ryanharding how did you manually added the reference? The nuget package of System.Collection has only . files in it?

@reinierkops I just browsed to my solutions nuget packages folder location (  ....\packages\System.Collections.4.3.0\ref\netstandard1.3 ) in the add references window on my android project.
Hope that helps you.

I have rolled back this updated for now.


WMDP-216 CD should fail if ecs isn't stable

Ticket
https://wicmtdp.atlassian.net/browse/WMDP-216
Changes

Added a Wait for ECS stable job in the deploy workflow

Context for reviewers
In order to apply these changes, this PR has changes in three places:

in the eligibility screener and the mock-api repo an additional CD step has been added
in the infra repo, the IAM role has been updated to allow ecs:DescribeClusters and ecs:DescribeServices

Testing
Successful run

@rocketnova still trying to find a way to put an explicit timeout. According to the documentation the ecs wait services-stable command polls an underlying API. Here is the default timeout:

It will poll every 15 seconds until a successful state has been reached. This will exit with a return code of 255 after 40 failed checks.

Searching around seems to suggest adding the DeploymentCircuitBreaker attribute to the tasks themselves is what we're looking for. However, I haven't been able to find a way to configure the circuit breaker at all (there's just booleans to enable it). I can add this by either adding a flag to the update service command like this or add the deployment_circuit_breaker block to the ecs services in the infra repo. The terraform way seems more manageable and I'll add a supplemental PR to navapbc/wic-mt-demo-project#14
cc @kyeah

@aplybeah Hmmm if it defaults to failing out after 10m, then it's probably not worth it to put in the effort to reduce that time. 10m is generally reasonable. Thank you for researching this!


Simplify recruiter API and convert to long-poll register

The goal is to close the known "holes" in the recruiter system by only allowing a worker to be recruited when it is holding open an HTTP long-poll with the recruiter. This will allow us to scale the K8s worker deployment up and down without getting faulty registration data.

Changing branches, so closing this


Are there any plans to expose the skip_special_tokens parameter in the future, driven by serving configuration?

https://github.com/deepjavalibrary/djl-serving/blob/d3ecca3ed0cccead885f6b68c2c5b977198a2446/engines/python/setup/djl_python/deepspeed.py#L367C32-L367C51
For developers working on relevant applications, the truncation of certain endoftext tokens is helpful for the extraction of answers.

@extraleo
If you are interested this feature, would you mind create a PR to add skip_special_tokens support?

deepspeed handler is deprecated. Closing this issue.


Detected

I got Detected, but it was a warning to remove any software modifying my skins or textures, something along those lines, after I removed the software, repaired, cs go I was able to play again

i got this too
Disconnected by VAC: You cannot play on secure servers
Modifying game files and/or in-game textures may cause this error.
Please remove any custom textures, skin changers, or software that modifies game files and restart your computer. We also recommend verifying your game files after making these changes.

@agshaidun u prob use the onetap.crack so u got these. U changed the depot

@agshaidun u prob use the onetap.crack so u got these.  U changed the depot

how do i change the depot back ??

Just verify ur game's integrity

Use @danielkrupinski VAC Bypass.

dont use antiaim or rage aimbot over 40fov and u wont get vac kick, if u do get vac kick just change account / sign out and back in and wait for like 10-20 mins

This is because of nSkinz. Been using nSkinz (by namazso) and after atleast 5 rounds I get VAC kicked

I mean what did you expect?

Cheat ain't detected, if your newcomer and don't know how to protect cheat from vac don't paste.
Only maximally for us what have some knowledge, is possible vac error.


Better Armor Piercing config cost not working correctly for difficulties

The configured cost stays the same on every difficulty, and doesn't increase with higher difficulties.

I'm just dumb, nvm


[Bug] No delete button after mirror build failure

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened

As of now, this problem belongs to an accidental occurrence, but after it appeared, I found that the image was never built successfully and I could not delete this task. At the moment I can only re-execute the build process by creating a new task, but the task in question will only remain on the page forever.
StreamPark Version
2.0.0
Java Version
No response
Flink Version
1.14.5
Scala Version of Flink
2.11
Error Exception
null

Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

cc @MonsterChenzhuo Does this issue still exist in the latest code? Please test it

@wolfboys  I haven't encountered it for the time being.

The code is up to date, and now the mirroring is built successfully.There has been no failure to build the image for a long time.

fixed in new version


Adding Author and Company field

Adding the ability to set the author and company for a pattern.
Do not merge, more work needs to be done.

This is an automated message:
You can preview this docs PR at http://415.docs-pr.validatedpatterns.io
Note that they get generated every five minutes, so please wait a bit.


changed the xcode version from 9 to 9.3

@hkellaway i got this to work and the travis-Ci has executed well. FYA now.


Add serverpath preference

Adds a configurable path to the server URL. Nice to have in some scenarios like reverse proxying.

If we want to allow users to specify path, it's probably better to just have a single URL parameter instead of all separate values (host, port, path etc).

I inferred a deliberate design pattern from the effort put into separating domain/address, port and scheme (aka encryption), thus added the path as another distinct option. Combining these in a single Server URL would be great for tech-savvy users but might be more difficult for users that do not care about protocol scheme and portnumber. Also, having the port as separate preference seems reasonable due to the default value. I am incertain regarding the encryption preference, however.
Maybe we should just broaden the address preference to allow for scheme, address and path while disregarding port and sync the entered scheme with the encryption preference?

Most users don't set up encryption, so probably default "http" prefix will work for them just fine. Port is always a problem. They are always trying to change it even in a separate field. I am thinking that it would be nice to have some special warning message if user selects port 8082. Otherwise default port 5055 in the URL should just fine.
I think most users are familiar with URLs. It might be even easier for them to put whole URL than separate fields. I often hear complaint that they can't save server address. Turns out they are trying to put whole URL there instead of just IP or domain name.

https://github.com/tananaev/traccar-client-android/pull/269/commits/1ecd8ed454f9a2b3a194e00f2bba92c1e9c251e7 replaces separate settings of host, port & encryption with a single Server URL preference.
Open Points:

Currently a toast message is used as a warning if port was set to 8082, but I could not find a proper wording for the message.
A safe upgrade path is missing - users would lose their settings on app upgrade. Not sure how to prevent that.
I kept a check of the entered hostname against your DOMAIN_NAME pattern. I think this check could be omitted?

Two big things that need fixing:

Why do you need parse URL and format it again? You just need to verify that it's a valid URL and check port.

You should not change any localisations other than default English.

Why do you need parse URL and format it again? You just need to verify that it's a valid URL and check port.

Several reasons why I went that way:

get rid of user provided credentials, query-string or fragment-identifier
replace otherwise 'valid' protocols  like ftp:// with http://
add the default port so the user can at least see what is used if no port was manually set

Regarding your 2nd point: Sorry, I thought obsolete strings should be removed from locales as well, will fix it later on.

I think all that is unnecessary. Why would you replace "ftp" with "http"? If anything, we should disallow user entering anything other than "http(s)", but even that is too much validation IMHO.
What we should definitely implement instead is migration from previous version.

Finally got around to adjust the commit to your recommendations. Not sure about the migration, though. The approach works but is rather naive and simply checks for existence of a port preference on creation.

Thanks for your feedback!
I think I addressed your concerns with the latest commits. Please let me know if you have any further remarks or want me to squash the commits.

Merged, thanks.


cryoCARE_predict.py Check error on V100 and A100 GPUs

I get "Check failed: narrow == wide (-1305608192 vs.<PHONE_NUMBER>)checked narrowing failed; values not equal post-conversion" error when using cryoCARE_predict.py on V100 and A100 GPUs, but works fine on RTX 2080 Ti. Using cuda 11.3, keras-gpu 2.3.1 and tensorflow 2.4.1.

Having the same issue. Seems related to the input tomogram size. If you can reduce the size sufficiently (e.g. use imod trimvol) it will run.

Thanks. Normally cryoCARE_predict.py tries different tile size until it finds one that fits into the memory available (see below). In the case I described, the script is not going through this optimization. I have been using RTX 2080s for the prediction step and it works fine although the GPU memory available is significantly smaller (12GB vs 32GB/42GB).

Its probably the cudatoolkit version that leads to that problem.
Could you type
conda list cudatoolkit
I guess you have installed version 10. This version is not compatible with the chipset in V100 and A100.
Try to create a environment with this command:
conda create -n cryocare_c11 -c conda-forge -c anaconda python=3 keras-gpu=2.3.1 cudatoolkit=11
Install cryocare in this env and you should be good to go.

Thanks anyways. I will try a fresh install.

Hi @rahelwoldeyes
I found out that the cuda 11 instructions had more problems. I've updated them:
https://github.com/thorstenwagner/cryoCARE_mpido#installation
If you want, give it a try :-)
Best,
Thorsten

@rahelwoldeyes After more problems, I've updated them again ^^ Hopefully they now work for you as well. Installting keras-gpu 2.31 via conda does not work for the CUDA 11 setup.

Thank you for keeping me updated @thorstenwagner. I appreciate it!

I have this error as well, using CUDA 11.0 (as per the instructions) with A6000 cards. Should I try a more recent CUDA, though TF 2.4 recommends 11.0?


feat: add OpportunityDefiType

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3186

PR #3183

PR #3190   👈

This comment was auto-generated by Graphite.


padding in canvas and  download image resolution

Thanks for the great library.
I have a question.

as you can see in the image I can't able to give a margin in the downloaded .png.  QR is visible in the corner of the image too.
how can I achieve this?
2nd  Question:  The image has the same resolution as the canvas has. Can we somehow download images with a better resolution without increasing the size of the canvas?

Anyways I solved the second issue with this.
   qrStylingObj._canvas.getCanvas().style.height = '20rem'
  // add getCanvas getter to QRCodeStyling class so it can easily accessible. like qrStylingObj.getCanvas().style.height

Anyways I solved the second issue with this.
   qrStylingObj._canvas.getCanvas().style.height = '20rem'
  // add getCanvas getter to QRCodeStyling class so it can easily accessible. like qrStylingObj.getCanvas().style.height

The library doesn't currently have an option to add a margin to the QR code itself.

2nd Question: The image has the same resolution as the canvas has. Can we somehow download images with a better resolution without increasing the size of the canvas?

You can achieve this by creating a second QRCodeStyling object with a larger width and height:
new QRCodeStyling({
  /* Your config */
  width: 2000,
  height: 2000,
}).download();

The library doesn't currently have an option to add a margin to the QR code itself.

2nd Question: The image has the same resolution as the canvas has. Can we somehow download images with a better resolution without increasing the size of the canvas?

You can achieve this by creating a second QRCodeStyling object with a larger width and height:
new QRCodeStyling({
  /* Your config */
  width: 2000,
  height: 2000,
}).download();

Hi @justraman

About margins. The lib doesn't support it, but it is a good idea for improvement.
The only hack I see to add some margin is to count horizontal or vertical dots (in your case is 25) and define the height and width of the canvas that will have the remainder of the division of 25. For example for height and width 324 % 25 === 24. So the margin will be 12 (24 divided by 2). Let me know if you need a better explanation.

You can init QRCodeStyling with any width and height, (2000 for example). Then change the width and height of canvas via css. So the downloaded file will have 2000x2000.

Hi @justraman

About margins. The lib doesn't support it, but it is a good idea for improvement.
The only hack I see to add some margin is to count horizontal or vertical dots (in your case is 25) and define the height and width of the canvas that will have the remainder of the division of 25. For example for height and width 324 % 25 === 24. So the margin will be 12 (24 divided by 2). Let me know if you need a better explanation.

You can init QRCodeStyling with any width and height, (2000 for example). Then change the width and height of canvas via css. So the downloaded file will have 2000x2000.

@kozakdenys
Thanks for your explanation

The only hack I see to add some margin is to count horizontal or vertical dots (in your case is 25)

It is 25 in this case but the data can be anything that's why the numbers of dots will possibly change.
also adding a small margin is not enough.
#35  I have created a PR for this seems like it's working for me. can you check yourself?

@kozakdenys
Thanks for your explanation

The only hack I see to add some margin is to count horizontal or vertical dots (in your case is 25)

It is 25 in this case but the data can be anything that's why the numbers of dots will possibly change.
also adding a small margin is not enough.
#35  I have created a PR for this seems like it's working for me. can you check yourself?

Thanks, @justraman
Published your feature in version 1.3.4

Thanks, @justraman
Published your feature in version 1.3.4


Initial record filtering for C4 testing

Fills in the C4 task filtering and creates a new class to filter patient records. Provider filtering criteria as defined in the final rule are still awaiting clarification and have not been implemented yet

I'm going to merge this in though we need to add a task to address the corner case issues with the date filtering however.

What's the corner case issue?

Rob gives a good example above: if the filter is 18 year at Measure Period then due to the way date time logic is performed in the measures someone who is born on 01/01/xxxx 18 years from the MP still meets the measure logic even though the absolute value for the birth time compared to the start of the measure period would state otherwise.
The code here is considering age as of the start of the measure period (i.e., the first second of the first day) but per the above it should really be the end of the first day. So this filter could count someone with a birthdate on the first day of the measure period incorrectly.

The corner case issue arises for birthdays on 01/01/xxxx . The way that the
calculation works is that the AgeAt qdm function does not take
hours,minutes and seconds into account.  So for instance if there is a
constraint like this AgeAt measure period >= 18.     Now say there is a
patient whos birthday is  01/01/xxxx 3:00 PM and they are turning 18 on the
start of the MP.  With a straight timestamp based filtering approach this
patient would not be included because the hours, min.... would cause them
to look as if they are still 17 at the start of the MP.  However , the date
time logic calculation does not take the smaller units into account so what
will be compared is the year,month and day.  Taking that into account the
patient would be included because they will 18 at the start of the MP.
On Wed, Nov 4, 2015 at 8:56 AM, Samuel Sayer<EMAIL_ADDRESS>wrote:

What's the corner case issue?
—
Reply to this email directly or view it on GitHub
https://github.com/projectcypress/cypress/pull/232#issuecomment-153725522
.


ref #232: correct shipping, voucher, discount and payment cost on cur…

Q
A

Branch
6.2.x

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tests pass?
yes

Fixed issues
chameleon-system/chameleon-system#232

License
MIT

Added issue to mark extensions as deprecated: https://github.com/chameleon-system/chameleon-system/issues/243

Thanks @pSpeck


Potentially more efficient way to filter virtual aps

Please take a look at this possible better way to handle filter_virtual_aps()
Rather than waiting till the workspace is full of aps, before starting to filter them (which requires a search across all possible combinations of the beacons in the workspace, we can check each one added (the way we already remove identical duplicates) as it is added.
This is a little more expensive if the number of aps added is less than the number we can hold in the workspace, but is much more efficient for every beacon added there after,
Overall, I think it is worth doing. I think it simplifies the logic of the filtering.

I think the general idea is reasonable, although we should verify that it gives the same result as the previous method. Approved, with some commenting and renaming suggestions.

The results are not identical with this approach. The old algorithm search for virtual APs after adding the new AP in rssi order and after the workspace was full. So there could be multiple matching virtual APs the first time the search happened. With the new approach, no virtual matches ever make it into the workspace.
However, for the two approaches, the actual beacons that are included in the request to the server for a given scan, should be the same, if any virtual matching APs are ignored.

However, for the two approaches, the actual beacons that are included in the request to the server for a given scan, should be the same, if any virtual matching APs are ignored.

Not sure I follow. But I guess I'll amend my remark by saying instead that we should verify that the results that the new algo gives are at least as "desirable" as the those given by the previous algo.

Not sure I follow. But I guess I'll amend my remark by saying instead that we should verify that the results that the new algo gives are at least as "desirable" as the those given by the previous algo.

Yes, agreed. I'm making the point that the old algorithm could leave some virtual aps in the request, but the new approach will always remove them.


upgrade dependencies

Hi,
Currently this project is using several outdated dependencies and therefore likely vulnerable for attacks using CVE-2024-4068.
This pull request upgrades the outdated dependencies.
Regards,
Adrian

I also create a issue for that: https://github.com/nrwl/nx-set-shas/issues/170


Fix Uninitialized Variable Errors in GoogleTest's gtest-death-test.cc

Summary
This pull request addresses compilation errors encountered in GoogleTest's gtest-death-test.cc due to potentially uninitialized usage of the dummy variable. These errors were preventing successful compilation of the ninja using GoogleTest.
Issues Addressed
The errors being thrown were as follows:
error: ‘dummy’ may be used uninitialized [-Werror=maybe-uninitialized]
This error was caused by the dummy variable in the StackGrowsDown() function potentially being used before it was initialized.
/ninja/build-cmake/_deps/googletest-src/googletest/src/gtest-death-test.cc:1305:24: error: ‘dummy’ may be used uninitialized [-Werror=maybe-uninitialized]
1305 |   StackLowerThanAddress(&dummy, &result);
Changes Made
Initialized the dummy variable to zero within the StackGrowsDown() function to ensure it is never used uninitialized.
This change resolves the compilation errors and does not alter the intended functionality of the tests.
Impact
This fix ensures that ninja can be compiled successfully with GoogleTest, specifically addressing the issues with strict compiler settings regarding uninitialized variables. It enhances the stability and reliability of the build process.
Testing
After making these changes, I successfully compiled ninja  with GoogleTest without encountering the previous errors.
All tests related to the modified functionality have been executed to ensure they pass and behave as expected.
Additional Notes
This fix is critical for environments where compiler settings treat warnings as errors, which is common in various CI/CD pipelines and strict development setups.

Please do not add files that belong to CMake-generated directories, this is not the way to solve the issue. See PR #2360 for a better way to solve the issue. Another one is simply to use GTest 1.11 or above with -DGTEST_ROOT=<path>  to point to its installation path.


CLI updates to support new login flow

This updates the CLI to use the new CLI login endpoints.
Easiest to review commit by commit because of some renaming in the first commit.

@paulgb Squashed in some changes (mostly in the login command) that has the CLI listen for the "login success" message before showing a prompt for the confirmation code. Might be worth looking at that last commit again.

@paulgb Squashed in some changes (mostly in the login command) that has the CLI listen for the "login success" message before showing a prompt for the confirmation code. Might be worth looking at that last commit again.

LGTM


Url of diftool can now be set up

1/2 done, 2./2 to do - if the url is not present, the track and others must not be show at all  (or show infgo about diff tool)
Would be nice to have more easy way to run and deploy diff tool.


Resolved Add Users button overlaps with the user table.

Signed-off-by: samshinde<EMAIL_ADDRESS>:nut_and_bolt: Description: What code changed, and why?
The Add Users button overlaps with the user table on the add-users-to-team modal and changed the height of the table to get a space between them.

:chains: Related Resources
#2752
:+1: Definition of Done
Add user button overlaps with the user table so added space between them.
:athletic_shoe: How to Build and Test the Change

start Automate + UI
navigate to https://a2-dev.test/settings/users and create several users
navigate to https://a2-dev.test/settings/teams/editors/add-users
you should be able to see the overlap of the buttons with the last row of users

STEP 1
inside the hab studio

[default:/src:0]# build components/automate-ui-devproxy/
[default:/src:0]# start_automate_ui_background
[default:/src:0]# start_all_services

STEP 2
open new window
go to automate UI path

$ cd components/automate-ui
and run the command

:white_check_mark: Checklist

[ ] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[ ] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable

Have we checked that this does not occur for any other similar pages?

Have we checked that this does not occur for any other similar pages?

@msorens Yes, I have checked the settings tab in which we have the same pop-up in these pages: Projects-> Create Rule, Policies-> Add Member, and there is no overlapping issue.


refactor trackers to be customizable

Acceptance criteria: as a user I can add or remove time trackers.
This adds a requirement to have a database connection.

pulling this update will show user a migration page with info

actions to do if any
where sensitive data is located and how stored
where to delete


Added volatile_organic_compounds_parts as VOC metric

Home Asssitant's BLE integration for AirThings has renamed the VOC metric. Added support for the new name.

Was merge with other pull request


feat: bump pyroscope library to support adhoc mode.

There's a bug in clib, preventing native integrations from working, https://github.com/pyroscope-io/pyroscope/pull/602 needs to be merged first and then the commit updated again.


more keyboard interactions

i am missing two types of keyboard interactions:

key pressed event for special key combinations e.g. alt-a, ctrl-f1
key released event

These are largely limitations of the terminal interface.
On the first point, the maintainer of curses explains how support of modified keys in terminals is limited and so fragmented as to be impossible to handle them all: https://invisible-island.net/ncurses/ncurses.faq.html#modified_keys
Then there's a pretty good blog post about key release events that here: https://blog.robertelder.org/detect-keyup-event-linux-terminal/.  Asciimatics is designed to run in a true terminal environment (i.e. the SSH connection case in that blog post), so cannot support release events.
The best you can do there is use the keyboard package and run as root if you really need that level of control.  That should give you access to everything, but at the cost of running as root and no longer being able to run over a terminal connection.

Longer term, there may be a new standard for handling modifiers (as debated in https://github.com/microsoft/terminal/issues/8719).  However, that is still far from universal and would currently require moving away from curses (e.g. to termkey), so not something that would be worth implementing yet.

Net is that I think I'll just have to document the current limitations.

Now covered by https://github.com/peterbrittain/asciimatics/commit/613a4ddc67cb808b45dc4b90f1b93f0b78bfc9df


refactor: jsdoc formatted comments

This is Epic!
Did you generate the comments, or brute force it!?

No automation used in this PR. Lots of copy+paste.


add diff txt

Description:

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

Codecov Report

Merging #43 (0776d5d) into main (5a6208e) will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   58.82%   58.89%   +0.07%
==========================================
  Files         154      154
  Lines        6992     6992
==========================================
+ Hits         4113     4118       +5
+ Misses       2864     2859       -5
  Partials       15       15

Flag
Coverage Δ

unittests
58.89% <ø> (+0.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

cmd/cli/env.go
0.00% <ø> (ø)

pkg/configurator/validating_webhook.go
80.00% <0.00%> (+3.44%)
: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 9807ae8...8a77021. Read the comment docs.


Feedback for "SAP Mobile Application"

https://help.sap.com/webcomponents/products/SAP Business Application Studio/9d1db9835307451daa8c930fbd9ab264/95f86cceb6f1441d81c2b562766719ed.html?locale=en-US&version=Cloud
Regarding Mobile Development Kit reference, updated cotent is available at this page https://help.sap.com/doc/f53c64b93e5140918d676b927a3cd65b/Cloud/en-US/docs-en/guides/getting-started/mdk/bas.html

Hi,
Thank you for the update. I have changed the link to the one you provided.
You will be able to see the new link with our next release.

https://people.sap.com/jitendrakumar.kansal


the total container height is not equal to content height

why if I set the height of the container h: 20, and rowHeight: 1, then the total block height with the class "react-grid-item" is not 200px, but 210 ??

I guess because there is a space between the blocks and this space equals 10px

It is probably due to a margin being set on the grid item. Search the issues tab for "margin". This has been discussed many times. https://github.com/STRML/react-grid-layout/issues/978

Я думаю, потому что между блоками есть пробел, и это пространство равно 10px

I guess because there is a space between the blocks and this space equals 10px
if the content is 128 pixels, the total height is 131px )

I guess because there is a space between the blocks and this space equals 10px

if the content is 128 pixels, the total height is 131px )

It is probably due to a margin being set on the grid item. Search the issues tab for "margin". This has been discussed many times. #978

I do not use other variables to set the height. only number h:20. rowHeight:1


Libhandler system

This PR adds libhandler as a system.

Looks good to me. Since this already has changes from #26 I'll prefer to merge that one first.

Ah, that's a mistake. I probably branched on pipes...

My latest commit reroots the branch to main.

I've pushed the Docker images to dockerhub effecthandlers/effect-handlers.

Looks good, thanks!


Changed {HOME} to Relative "."

Fix for #13
Tested: Kafka, Postgres, airflow

I have updated the main branch to use ./data/service/persist directory to persist all data. Thanks for raising the issue.


fix inconsistencies

fix inconsistencies in \id tag and rename \mt1 tags in some places to \mt

Thanks for fixing the \id tag issues. Reg. the \mt tag, I'd prefer to keep it as \mt1 instead of \mt. Although \mt = \mt1, the latter is preferred by many. I'm going to merge the PR and then will change the \mt to \mt1.

I changed \mt1 to \mt just because some books had the \mt tag and some others had \mt1 tag. And the books which had \mt1 tag didn't have the next lines (\mt2, \mt3, etc). If that's what people prefer, great!

Our approach is to keep the text as close as how it appears in the scanned
copies. So in situations like this, it is easier for us to make a decision.
So we don't need to add \mt2 or \mt3 if they are not in the scanned copy. I
just skimmed through the pages and found that this version has an English
title alongside the Tamil text. Hence we need to add it.
[image: image.png]
Since it is English I can do it, but if you're willing please feel free to
send a PR and I'll merge it.
TIA
On Sun, 23 Jan 2022 at 22:38, Berin Aniesh @.***> wrote:

I changed \mt1 to \mt just because some books had the \mt tag and some
others had \mt1 tag. And the books which had \mt1 tag didn't have the next
lines (\mt2, \mt3, etc). If that's what people prefer, great!
—
Reply to this email directly, view it on GitHub
https://github.com/tfbf/Bible-Tamil-Sathiyavedam-1957/pull/2#issuecomment-1019527131,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFOPK5HDAIKGW4AAWR4UMLUXQY2HANCNFSM5MTMYXJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Our approach is to keep the text as close as how it appears in the scanned copies. So in situations like this, it is easier for us to make a decision. So we don't need to add \mt2 or \mt3 if they are not in the scanned copy. I just skimmed through the pages and found that this version has an English title alongside the Tamil text. Hence we need to add it.  Since it is English I can do it, but if you're willing please feel free to send a PR and I'll merge it. TIA
…
On Sun, 23 Jan 2022 at 22:38, Berin Aniesh @.> wrote: I changed \mt1 to \mt just because some books had the \mt tag and some others had \mt1 tag. And the books which had \mt1 tag didn't have the next lines (\mt2, \mt3, etc). If that's what people prefer, great! — Reply to this email directly, view it on GitHub <#2 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABFOPK5HDAIKGW4AAWR4UMLUXQY2HANCNFSM5MTMYXJQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you modified the open/close state.Message ID: @.>

I can submit a pull request soon, what tag should i use for the English title?

\mt3 should do.

Here's the link to the scan...
https://archive.org/details/Tamil-Bible-BSI-OV-1957/page/n881/mode/2up?view=theater


Feature to manually create/edit the Train paths

Is your feature request related to a problem? Please describe.
When you create a train route, you don't actually make the path for where the train goes. You just tell the train which platform it needs to go to before driving to the next assigned platform, and the actual path the train takes between these stops is determined by some sort of pathfinding AI, and unfortunately, this AI can get annoying at times. This is not a problem when your train network is small, but when you have a city-sized train network(In my case, a tram network), you can start to see what I mean. Let me explain.
So I'm trying to create a tram route and want the tram to get from Station A to Station B(see picture for full detail). So in this situation, you would expect the pathfinding AI to choose the quickest way to get from Platform A to Platform B, right? Well, nope. Instead of just going straight towards the station, the tram decides to take the branching path and proceeds to drive around the ENTIRE CITY before coming back to the station it was supposed to go to, like an hour later.
From what I see, instead of choosing the shortest path to the next assigned platform, the AI just chooses the first path it finds, even if there were way shorter and quicker routes it could've taken. This is frustrating because there's no way to change this. The best solution now would be to just delete all the tracks it shouldn't use to restrict where the pathfinding AI can go and then place them down again after the train route has been created, but it's a pain to do this every single time you create a new route or need to edit an old one.
Describe the solution you'd like
The first solution would be to make the pathfinding AI better, but I think it would be better to just give us the option to create the paths ourselves. That way, we would always have the option to do it manually in case the AI, for any reason, fails again.
Additional Information
In case you're asking, all of the tracks are the same speed.

Perpetrator of the crime:

It does Take Track Speed at Junctions into Account

I know, but all of the tracks in this example are the same speed(40 km/h)

I know, but all of the tracks in this example are the same speed(40 km/h)

Try Fiddling around with the Speed

I tried that, and the tram route in question did make the right turn. But now every OTHER tram that goes through that junction also started taking that exact turn, even the ones that aren't supposed to. There are too many routes going through that junction for the speed trick to work.

I tried that, and the tram route in question did make the right turn. But now every OTHER tram that goes through that junction also started taking that exact turn, even the ones that aren't supposed to. There are too many routes going through that junction for the speed trick to work.

ok... Try having the Original Route go to new Platforms Via a Separate Right-Of-Way

you can add some useless platforms and set the sleep time to 1 to flush AI's annoying

Eh I'm trying to avoid having to make direct changes to the track layout. You see, the network I'm making is actually a recreation of a real-life city network, and I want to keep it as accurate to the real one as possible. Also, that wouldn't really look visually appealing. I prefer to do this only as a last resort. Can't there be another way to do this?

Eh I'm trying to avoid having to make direct changes to the track layout. You see, the network I'm making is actually a recreation of a real-life city network, and I want to keep it as accurate to the real one as possible. Also, that wouldn't really look visually appealing. I prefer to do this only as a last resort. Can't there be another way to do this?

I have no idea.

Thanks for your feedback, I'm working on a better pathfinding algorithm. Waypoints might come in the future as well, haven't planned that yet.


Re-activate gorelease SemVer check

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
397

Relevant Lines:
443

💛  - Coveralls


In need of Lazy Loading update

Unless I'm misunderstanding the libraries being used or referenced.  Lazy loading is now supported which is in contradiction with what this article states:
[quote]Lazy loading. EF Core doesn't currently support lazy loading.[/quote]
https://docs.microsoft.com/en-us/ef/core/querying/related-data#lazy-loading

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

ID: d0026f69-7e5d-0d3e-5586-16ecc0656ff0
Version Independent ID: 3cb264da-2ff1-1256-572f-aa4ebca6e255
Content: Razor Pages with EF Core in ASP.NET Core - Read Related Data - 6 of 8
Content Source: aspnetcore/data/ef-rp/read-related-data.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@tdykstra 	please investigate.

Lazy loading was added in 2.1.  I made a PR to correct the remark about lazy loading not being supported.  We should add how-to instructions in the tutorial, I'm updating the title to make that clear.
Thanks @akousmata  for bringing that to our attention!


app.xr.input add event does not fire in pc.XRTYPE_AR (immersive-ar) mode

Description
Changing instances of pc.XRTYPE_VR to pc.XRTYPE_AR on this example: https://playcanvas.github.io/#/xr/vr-hands cause hands to no longer be detected.
Steps to Reproduce

Visit https://playcanvas.github.io/#/xr/vr-hands
Ctrl+F for XRTYPE_VR (there are two instances)
Change them to XRTYPE_AR and click the reload button
Observe that the hands are no longer there - the app.xr.input.on('add', ...) handler does not fire.

Thank you for report. This has been fixed recently in this PR: https://github.com/playcanvas/engine/pull/6069
It will be released to everyone with next engine release likely.

Nice, thanks!


Temperature effect on Select

Hi there,
I have some doubts about the process behind the select method.

Is there any detailed explanation about what happens under the hood while using select and gen?
I mean, can select(['joke', 'poem']) perform actions that are equivalent to those performed by gen(regex='(joke|poem)', temperature=_)? If not, in what they differ?
Is there any impact due to temperature on the select? If yes, which is the default temperature used by the select?

Thanks!

(First of all - I'm unsure the specific method guidance uses for these functions. But from what i know about constrained decoding, I think below would still be valid)
I recently came across this article from @AidanCooper that I really like: https://www.aidancooper.co.uk/constrained-decoding/
He's got a great example in there (using another constrained decoding library), which I've abbreviated and pasted below.
import sglang as sgl

@sgl.function
def us_president_choices(s):
    s += sgl.user("Name a US president.")
    s += sgl.assistant(
        "An example of a US president is " +
        sgl.gen("president", choices=["Donald Duck", "Millard Fillmore"])
    )

@sgl.function
def us_president_regex(s):
    s += sgl.user("Name a US president.")
    s += sgl.assistant(
        "An example of a US president is " +
        sgl.gen("president", regex=r"(Donald Duck|Millard Fillmore)")
    )

state_choices = us_president_choices.run()
state_regex = us_president_regex.run()
print(state_choices["president"])  # >>> Millard Fillmore
print(state_regex["president"])  # >>> Donald Duck

Takeaway:

The greedy algorithm used by the regex implementation is short-sighted, and cannot resist choosing the "Donald" option, despite ultimately landing on an incorrect answer.

Not having studied the guidance source code in great detail, if something else is happening behind the scenes that would make the above example invalid, I'd love to be corrected here!

I second the first question -- it would be useful to understand, because I very often see counterintuitive behaviour between unconstrained generation through raw transformers and constrained generation through guidance. I need to debug further, but it seems to me there's a few possibilities: top_p/top_k filtering; temperature (although I think this shouldn't matter because guidance is not random, i.e. always decodes greedily); (perhaps most insidiously) tokenization issues that are prompt-/use case-specific.
To answer your question @parkervg, a quick test would suggest that both select and gen (with regex) in guidance are greedy:
from transformers import AutoModelForCausalLM, AutoTokenizer, PretrainedConfig

import guidance

model_name = "unsloth/llama-3-8b-bnb-4bit"
device = "cuda"

tokenizer = AutoTokenizer.from_pretrained(model_name)
model = AutoModelForCausalLM.from_pretrained(model_name)
lm = guidance.models.Transformers(model=model, tokenizer=tokenizer)

# lm += "An example of a US president is the current president, "
lm += "The US president who freed the slaves with his Emancipation Proclamation was "
# choices = ["Abraham Lincoln", "Donald Duck"]
choices = ["Abraham Ozler", "A. Lincoln"]
lm += guidance.select(choices)
# lm += guidance.gen(regex=rf"({choices[0]}|{choices[1]})")
print(lm)
# Output: The US president who freed the slaves with his Emancipation Proclamation was Abraham Ozler

Funny enough, for my use case, I actually prefer this behaviour, but I'm having trouble reliably getting it with my model/prompt, so some insight as to the internals of gen, select, and any differences between raw transformers would still be useful.

Funny enough, for my use case, I actually prefer this behaviour, but I'm having trouble reliably getting it with my model/prompt, so some insight as to the internals of gen, select, and any differences between raw transformers would still be useful.

I've made a bit of progress on understanding the root of my issue, and I think it's something to do with tokenization rather than top_p or top_k - see #876 if you're curious. (I don't want to hijack this issue, but it relates to question (1) in the OP.)

Hey all, thanks for the great discussion here! I'll first answer the second question, then bounce back to the first :).
In terms of the guidance API, the challenge with sampling parameters in general -- temperature, top_p, top_k, etc. -- is that we really need them to be part of our grammar tree so that we can selectively apply them across different ranges of generations in a model without needing to make fresh API calls (i.e. govern them as part of a stateless interaction).
The current best way to do this is to wrap ranges of generations in the with_temperature function. The tricky business here is that we want to let users override it with specific temperature values set lower in the stack, so that you can do things like a global temperature across a giant function, but surgically update a single gen inside of that to use a different one. The idea is that sampling parameter settings that are nested more deeply would take precedence.
Some ideas here in terms of API:

Setting them globally on the entire LM object (like we used to do in the model init)
Use a with Sampler(temperature=...): context manager, for a range of code inside a function. This makes it more convenient than a single function wrapper, but adds a layer of indentation.
with_sampling_params(grammar) wrapper functions, like a generalized with_temperature today
sampling parameter settings on each atomic library function (as kwargs to gen, select, json, etc.

Would appreciate thoughts from this group on what API feels ergonomic here!
Here's a code example of setting temperature on select in guidance today:
import guidance
from guidance import models, gen, select
from guidance.library import with_temperature

lm = models.Transformers("gpt2", device_map="mps", echo=False)
lm += "1+1=2. 2+2=4. 3+3="

# picks '6' every time
for i in range(10):
    lm_temp0 = lm + select(['6', '7'], name="num")
    print(lm_temp0['num'])

# switches between options
for i in range(10):
    lm_temp2 = lm + with_temperature(select(['6', '7'], name="num"), temperature=2)
    print(lm_temp2['num'])

On how select works today -- we do enforce constraints greedily (i.e. one token at a time), which I totally appreciate has its challenges. This is how the Donald Duck/A. Lincoln style artifacts can creep up.
One thing we need to stress better that constraints are more like guardrails and don't fundamentally remove the requirement to prompt properly. If you change your prompts to tell the model what its choices are ahead of time -- even abstractly -- you'll generally get much better performance and avoid these artifacts of greedy decoding for most capable models. This is a broadly applicable principle (e.g. you'll get way better performance out of your JSON Schemas if you simultaneously pass the schema to the model as part of the prompt AND enforce constraints on generation, vs. just doing one or the other). Proper prompting and constrained decoding really are complementary.
Closed model providers who run constrained decoding in production often do silent prompt modifications/extensions to "prime" the model ahead of constraining. In guidance, we don't want to do silent modifications to user prompts -- part of the beauty is that you can always see and inspect exactly what's going to the model, and when. 'm trying to think of the best way to encourage this "best practice" -- perhaps more educational material, or a kwarg on some of our syntactic sugars like json() which append the schema to your prompt (which you can always disable)?
On the scientific side, we think there are non-greedy ways of addressing this too, that might come at the cost of more inference time compute. Greedy constraint enforcement is nice because it essentially doesn't slow the model down, but if you're willing to add more latency/compute, we can likely do much better. I think we'd want to do these tricks as "opt-in" though, because they're almost all linear in terms of the size/length of the select options.

have to run for a bit but happy to answer more questions in the AM

Thanks for the thorough explanations @Harsha-Nori ! Really interesting stuff.
I like the emphasis of constraints as guardrails, and not a silver bullet that allows you to bypass prompt refinement. As for the temperature API point - in the spirit of being as explicit as possible, I'm in favor of pattern 4 (sampling parameter settings on each atomic library function). I could imagine the following context manager pattern could get unwieldy:
with assistant():
    with Sampler(temperature=temperature):
        ....

Being an argument of the atomic function, this would

Remove the need for a distinct Sampler import
Allow for more explicit parameter-passing patterns
Give the new developer hints for the recommended interaction pattern in the form of named arguments from their PyCharm/VSCode/etc. IDE, rather than searching through the entire from guidance import ... module to find the function/class they need

If I wanted to effectively set a more 'global' parameter, I might do something like below:
from functools import partial
zero_temp_gen = partial(gen, temperature=0.0)
lm += zero_temp_gen("Say something factual")

Thanks for that insight! I agree, I think greedy decoding is a very sensible and intuitive default (although sadly does not seem to quite hold for all models as of now, see #876). Beam search or similar could be a nice-to-have for certain applications, but I don't really need it for any of my current use cases from what I can tell. Similarly, I like the simplicity of the low-level guidance interface -- I wouldn't want it to default to modifying my prompt in potentially opaque ways.
For temperature settings, I think the per-atomic function approach is best, too. A fifth option would be to allow configuration-until-configuration-updated, i.e., setting temperature/other params on the fly in a separate atomic function:
lm += set_generation_opts(temperature=0.2)
# temperature is now 0.2 until it's changed again

Even as it stands now, a section on how temperature, top_*, etc. are used in the docs would be great for folks coming from raw transformers. (I spent quite a bit of time wondering whether #876 could be caused by these parameters.) Either way, very grateful to see you guys thinking about user experience to the extent that you are! Cheers!

Thanks for linking me to this interesting thread, @parkervg!

If you change your prompts to tell the model what its choices are ahead of time -- even abstractly -- you'll generally get much better performance and avoid these artifacts of greedy decoding for most capable models.

This is certainly true. However, I have found for complex tasks with less-capable models, greedy select behaviour can produce poor results, even when providing full context on the constraints.
I think SGLang's approach of evaluating the full set of choices in their entirety makes a lot of sense here (although I have observed some strange behaviour with their implementation — possibly some sort of token healing artefact). The latency penalty should be pretty minimal for realistic sets of select options. For select specifically, only supporting greedy token selection is quite prohibitive for smaller models that aren't so great at instruction following, unless the options are very predictable (task-wise and first-token-wise).
API idea (4) would be a great way to support other sampling strategies beyond the greedy default for individual select constructs!


com.alibaba.dubbo.rpc.RpcException: Unsupported client type: httpclient, supported client type is grizzly mina netty netty4

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: v2.6.5
Operating System version:windows 7
Java version: v1.8

Steps to reproduce this issue

<dubbo:service id="helloService" interface="com.alibaba.hello.api.HelloService" version="1.0.0" protocol="dubbo,hessian" />
<dubbo:reference id="helloService" interface="com.alibaba.hello.api.HelloService"/>

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
hope success
Actual Result
com.alibaba.dubbo.rpc.RpcException: Unsupported client type: httpclient, supported client type is grizzly mina netty netty4

Would you pls provide a demo to reproduce the problem ?

demo:
provider:
<dubbo:protocol name="dubbo" server="netty4" port="8010" accepts="100"/>
<dubbo:protocol name="hessian" server="jetty" port="8011" accepts="100"/>
<dubbo:service id="helloService" interface="com.alibaba.hello.api.HelloService" protocol="dubbo,hessian" />
consumer:
<dubbo:consumer client="httpclient" >
<dubbo:reference id="helloService" interface="com.alibaba.hello.api.HelloService"
registry="leaseReference"/>
</dubbo:consumer>

Would you pls provide a demo to reproduce the problem ?

demo:
provider:
<dubbo:protocol name="dubbo" server="netty4" port="8010" accepts="100"/>
<dubbo:protocol name="hessian" server="jetty" port="8011" accepts="100"/>
<dubbo:service id="helloService" interface="com.alibaba.hello.api.HelloService" protocol="dubbo,hessian" />
consumer:
<dubbo:consumer client="httpclient" >
<dubbo:reference id="helloService" interface="com.alibaba.hello.api.HelloService"/>
</dubbo:consumer>

The value of the consumer side configuration client of dubbo should be one of grizzly mina netty netty4.

The value of the consumer side configuration client of dubbo should be one of grizzly mina netty netty4.

I want use the HTTP protocol.The exception was 'Unsupported client type: httpclient'

The value of the consumer side configuration client of dubbo should be one of grizzly mina netty netty4.

I want use the HTTP protocol.The exception was 'Unsupported client type: httpclient'


Change branch alias tag in composer.json file.

Coverage remained the same at 100.0% when pulling a1bb2c4f0ef895569efa446a726cce0790411ce3 on 4.2 into e910cacbd57680800956ce8687e0613a0c6fd2fc on master.


deno: make update script independent from CWD

Changes the deno update script to be independent of where it is executed

@06kellyjac are you the right person to review this change?

@amarshall you touched these files last in #308949 - I am unsure about the rules on who reviews this and how to mention them? Is it all automated?


Selecting a display for Intel RealSense D435 camera so there would be no lag

I am using an Ubuntu 22.04 system (open to use Windows if it may change anything).
I am using a secondary display to show the real-time feed of RealSense D435 camera on the display.
I would like to know if the secondary display does need to have certain criteria for the RGB feed of this camera to meet no lag and be exactly real-time.
For example, does the display need to be 60Hz refresh rate? Does it need to have response time <16ms?
I ask since it looks FPS of RGB feed is 30 FPS.
Any recommendation on this end is really appreciated.
What hardware specs do I need to consider for the display I am planning to buy if I want to make sure the video stream shown from Intel RealSense D435 is not laggy at all and indeed real-time? Assume, budget is not a concern.
P.S.: Have you seen people facing this lag problem?

Hi @monajalal  You should not need a special display screen.
The D435 model has a fast global shutter on its depth sensors but a slower rolling shutter on its RGB sensor. This can cause the RGB to lag behind depth if fast motion is being observed.
A workaround is to set depth to 30 FPS and set RGB to 848x480 resolution and 60 FPS.  When RGB is 60 FPS, the resultant lag reduction compensates for the slow shutter to equalise the performance of the two shutter types.
Another method of reducing lag on the RGB sensor is to disable RGB auto-exposure and set a manual RGB exposure value of 78 and an RGB FPS speed of 6 (six, not sixty).
The D455 type cameras (D455, D456, D457, etc) have a fast global shutter on both the RGB and depth sensors if you want to consider that option in future so that you do not need to compensate for a slow shutter on the RGB sensor,

We need to use the 1920x1080 resolution for the RGB camera. Also, for this application, we don't need the depth. Do you think if I only need to use 30FPS RGB feed in that resolution, it's gonna be laggy?

It depends on how fast the camera will be moving or, if the camera is stationary, how quickly people and objects observed by the camera are moving.  If there is motion faster than a wave of the hand then there may be lag / blurring on the RGB.  The D415 camera model also has the rolling shutter on its RGB and is affected by high speed motion in the same way.  https://github.com/IntelRealSense/librealsense/issues/3554 has more information about this.
If the camera will be stationary or observing motion slower than a hand wave then RGB lag will likely not be an issue.  The computer itself should be well specced though because streaming 1920x1080 at 30 FPS will have more of a processing burden than using a lower resolution.


Fix variables shadowing

Found with "gometalinter --vendor --disable-all --enable vetshadow ./..."

Overall I'm not a fan of compiler/linter directives sprinkled throughout the code, so most of my comments were in hopes of removing as many as possible.
What are your thoughts for gometalinter usage? In my opinion we either need to add it to the pre-commit hook or CI for the source code pollution to be justified. Would it introduce considerable overhead in the pre-commit hook?

In my opinion we either need to add it to the pre-commit hook or CI for the source code pollution to be justified.

I'll work on that.

Would it introduce considerable overhead in the pre-commit hook?

Some of the checks are quite fast, so maybe we can add them to the pre-commit hook.

I'll work on that.

Do you want to include the pre-commit update in this PR?

No, it's unrelated to this change so it'll come in its own PR. :)


pangea-node-sdk: replace node-rs/crc32 (GEA-11594)

Replace node-rs/crc32 with a pure-JS implementation of CRC32C so that pangea-node-sdk can more easily be added to Next.js projects without requiring the addition of a native addon loader like node-loader.
Verified working locally on @snpranav's Next.js App Router test app.

Rebased onto latest main and fixed merge conflict.


Support for OpenID Connect

Seems like OpenID 2.0 is not supported by google anymore:
https://developers.google.com/identity/protocols/OpenID2Migration#shutdown-timetable
Do we have an upgraded Zuul release available? On a fork?
Regards,
Sukrut

@mcantrell  - Using Zuul with http://techblog.netflix.com/2012/06/annoucing-archaius-dynamic-properties.html would do wonders.


More clean up, Use groovy syntax

Current coverage is 56.75% (diff: 66.66%)

No coverage report found for master at 7b9f542.

Powered by Codecov. Last update 7b9f542...5868b90

Current coverage is 56.75% (diff: 66.66%)

No coverage report found for master at 7b9f542.

Powered by Codecov. Last update 7b9f542...5868b90

Current coverage is 56.75% (diff: 66.66%)

No coverage report found for master at 7b9f542.

Powered by Codecov. Last update 7b9f542...5868b90

@vanniktech This is confusing. Locally, Jacoco says 51%. Using the "coverage" plugin in Intellij, it says 64%.

Current coverage is 56.75% (diff: 66.66%)

No coverage report found for master at 7b9f542.

Powered by Codecov. Last update 7b9f542...5868b90

@vanniktech All fixed.

Current coverage is 56.75% (diff: 66.66%)

No coverage report found for master at 7b9f542.

Powered by Codecov. Last update 7b9f542...6e564c6

@vanniktech Could we switch to using coveralls?


Please rename the project

Hey,
We're asking 3rd party code not to use the spring-boot prefix. Could you please rename it?
Let me know when you're done so that I can add it to the list of 3rd party starters.

Done.


Unable to create session

When trying to start my gulp task using gulp-protractor I get the following error:

Using the selenium server at http://localhost:4444/wd/hub
[launcher] Running 1 instances of WebDriver
/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/atoms/error.js:108
var template = new Error(this.message);
^
SessionNotCreatedError: Unable to create session from org.openqa.selenium.remote.NewSessionPayload@12a45fd2
Build info: version: '3.9.1', revision: '63f7b50', time: '2018-02-07T22:42:28.403Z'
System info: host: 'PC-1705', ip: '<IP_ADDRESS>', os.name: 'Linux', os.arch: 'amd64', os.version: '4.4.0-112-generic', java.version: '1.8.0_131'
Driver info: driver.version: unknown
at new bot.Error (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/atoms/error.js:108:18)
at Object.bot.response.checkResponse (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/atoms/response.js:109:9)
at /home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:160:24
at wrappedCtr.invokeCallback_ (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/goog/../webdriver/promise.js:1337:14)
at wrappedCtr.execute_ (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/goog/../webdriver/promise.js:2776:14)
at wrappedCtr.executeNext_ (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/goog/../webdriver/promise.js:2758:21)
at goog.async.run.processWorkQueue (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/goog/async/run.js:124:15)
at
at process.tickCallback (internal/process/next_tick.js:188:7)
From: Task: WebDriver.createSession()
at Function.webdriver.WebDriver.acquireSession (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:157:22)
at Function.webdriver.WebDriver.createSession (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:131:30)
at Builder.build (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/selenium-webdriver/builder.js:445:22)
at HostedDriverProvider.DriverProvider.getNewDriver (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/protractor/lib/driverProviders/driverProvider.js:38:7)
at Runner.createBrowser (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/protractor/lib/runner.js:186:37)
at /home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/protractor/lib/runner.js:276:21
at _fulfilled (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/q/q.js:797:54)
at self.promiseDispatch.done (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/q/q.js:826:30)
at Promise.promise.promiseDispatch (/home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/q/q.js:759:13)
at /home/nlie/dev/ts-gui/nms-conf-gui/node_modules/gulp-protractor/node_modules/q/q.js:525:49
[launcher] Process exited with error code 1
{ [Error: protractor exited with code 1]
message: 'protractor exited with code 1',
showStack: false,
showProperties: true,
plugin: 'gulp-protractor',
__safety: { toString: [Function: bound ] } }

When manually running 'protractor' from command line this one works but it seems it does not work when gulp-protractor is used. Also the webdriver-manager is started and already tried to update etc.

Is this Ubuntu ? I had similar issue and seems to be related to permissions on node_modules directory in your workspace. Can you try to manually run webdriver-manager update for the first time?

Yes this is Ubuntu. Running webdriver-manager update is no problem. I can even run my protractor tests when webdriver-manager is started but only when using the command 'protractor' and not when I use the command 'gulp protractor' (which uses gulp-protractor)


Uncomment part of the volume discounts

This PR simply uncomments the section of the volume discounts

LGTM w/ some minor suggestions.

Updated


Add setup and usage instructions for Raspberry Jam in README

Once hardware is finalised this can be done.

This also applies to generally fleshing out the documentation for the project.

Added basic update commands for rpi and git install instructions

This now includes all needed steps to set up RPI for use with program including creating a device tree overlay if required.

Done


Resend RI and NY test events to Universal Pipeline

Description
RS wants test events resent to the Universal Pipeline for NY and RI for the dates of July 3-7, 2024.
Proposed solution
Resend test events from the following queries:
Test events from RI/NY facilities
Test events for RI/NY patients
Please notify RS when this is complete via the #shared-simple-report-universal-pipeline channel
Open question: should we resend test events and their corrections? maybe we actually have to, since a corrected message will refer to a prior message

Completed and informed ReportStream on 07/17/2024


[Baggage] History

Add baggage history ordered by time with statistics about which time periods have more congestion in order to organize better next edition.

Basic history added.

TODO: Show ho registered the baggage!


Improve spearmanr

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

Project coverage is 100.00%. Comparing base (44b560f) to head (85bd6ed).
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #447   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          103       103
  Lines         2606      2608    +2
  Branches       159       159
=========================================
+ Hits          2606      2608    +2

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