Duplicate object property and getter/setter should generate an error

let a = {
  a: 1,
  get a() {}
};

This should generate an error.

No it shouldn't. This is allowed in ES6.

@RReverser I recall the semantics changing for duplicate properties. Was this also changed for { a: 1, a: 2 }?

Yup. The latest one takes effect.

@RReverser Acorn still throws an error for that case then.

In ECMAScript 5 strict mode code, duplicate property names were considered a SyntaxError.  With the introduction of computed property names making duplication possible at runtime, ECMAScript 6 has removed this restriction. (c) https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Object_initializer
P.S. I'd reference official spec here but it's way too big to open from mobile browser.

Acorn still throws an error for that case then when it shouldn't.

That's weird, I remember fixing it for {ecmaVersion: 6}. Care to open separate issue / send PR?

For reference, the duplicate property checks were dropped in rev 26, here's an overview:  https://github.com/rwaldron/tc39-notes/blob/46d2396e02fd73121b5985d5a0fafbcdbf9c9072/es6/2014-07/jul-29.md#41-review-latest-draft

P.S. I'd reference official spec here but it's way too big to open from mobile browser.

Here you go: https://people.mozilla.org/~jorendorff/es6-draft.html#sec-additions-and-changes-that-introduce-incompatibilities-with-prior-editions

<IP_ADDRESS>: In Edition 6, it is no longer an early error to have duplicate property names in Object Initializers.

@RReverser

That's weird, I remember fixing it for {ecmaVersion: 6}.

Yup you were right, apologies.

Good, thanks @rwaldron for reference and @sebmck for checking it.

Thanks you guys. Learned a lot from the thread.
Should check the spec carefully next time. (traceur throws error for this case, so I thought it was the norm)

@RReverser no problem :)


Bump etcd client sdk to latest commit 65ac859a1b613a3b1b509cd80400f9f…

…cbeae97d6
So that it can automatically bump google.golang.org/protobuf to v1.33.0 to resolve CVE-2024-24786

This PR was reverted.


Keep tags naming consistent

..else F-Droid can't autoupdate

https://gitlab.com/fdroid/fdroiddata/-/merge_requests/6428/diffs#diff-content-0fea63baa1ec1efe202dd609e3645937d9588aca

Apparently, 1.2.3/4 were inconsistent, I'm sorry, but 1.2.5 is already correct again. This happens when switchting between different projects a lot, I'll try to be careful :man_shrugging:


NRK scrobble

I have one bug, but can't seem to find the cause. The "now scrobbling" is never cleared, even when pausing, stopping, or navigating away. It only seems to go away when I close the tab. Since I can't figure this out, I was planning to merge this to master.

I have one bug, but can't seem to find the cause. The "now scrobbling" is never cleared, even when pausing, stopping, or navigating away. It only seems to go away when I close the tab. Since I can't figure this out, I was planning to merge this to master.

Well, this should be handled in the checkForChanges method. Have you tried adding some logs to make sure that the pause and stop methods are being called as expected?

I have one bug, but can't seem to find the cause. The "now scrobbling" is never cleared, even when pausing, stopping, or navigating away. It only seems to go away when I close the tab. Since I can't figure this out, I was planning to merge this to master.

Well, this should be handled in the checkForChanges method. Have you tried adding some logs to make sure that the pause and stop methods are being called as expected?

My impression is that since Nrk is not an single-page-app, the data needed is unloaded or gone by the time I stop. Not sure though, since scrobbles seems to be sent correctly to trakt.

That's weird, if it's being sent correctly to Trakt it should be working, 'cause that's all done in the same function... I can take a look later.

Hm, I tested and it seems to be working fine. I navigated away and it stopped the scrobble and removed the current scrobbling item. It's normal for it to appear while paused, because the scrobbling item isn't removed when pausing. Can you double check?

Strange. This happens every time, both if the playback is paused, or still playing. The icon itself is however reset.
Not really related to this PR, but I notice that the scrobble-background doesn't completely fill the popup.

Ok, should be fixed. It's indeed because NRK isn't a single-page app, but this could also technically happen on other streaming services if you refresh the page too fast and the extension can't stop the scrobble in time, so I made it so that every time a content page is loaded it checks for a scrobble and stops it if the tab matches the scrobbling tab.

Thanks for helping out!


Add arabic language support

I want to add arabic support for minimal app .

I want to add arabic support for minimal app .

would you mind if I do it ?


alertmanager/alertmanager-main's config-volume is called cm/alertmanager-alertmanager-main

After running deploy:
Volumes:
  config-volume:
    Type:       ConfigMap (a volume populated by a ConfigMap)
    Name:       alertmanager-alertmanager-main
...
...: timeout expired waiting for volumes to attach/mount for pod "monitoring"/"alertmanager-alertmanager-main-0". list of unattached/unmounted volumes=[config-volume]

This happens, because alertmanager/alertmanager-main expects its config map to be named cm/alertmanager-alertmanager-main, while it is actually created as cm/alertmanager-main by kube-prometheus/.../deploy.
Might be implicitly fixed by #33, even though it is already broken right now and the behavioural change was not introduced by prometheus-operator-0.5.

This is a change introduced by the v0.3.0 release of the operator, which is why current master of this repository is pinned to the v0.2.x release series. Yes indeed #33 is upgrading to the latest Operator version and fixes all the incompatibilities and also upgrades other components such as kube-state-metrics. I'm expecting #33 to land soon, but will probably have to wait for the v0.5.1 release of the Operator.

Maybe my setup was confused from an earlier installation of the Prometheus operator 0.5.  Thus closing this as invalid.


Add a path parameter to the setSocketUrl methods

Problem
When using nginx as a reverse proxy only a direct redirection work.
I mean this work:
location / {
                proxy_pass http://<IP_ADDRESS>:8080/;
                proxy_http_version 1.1;
                proxy_set_header Upgrade $http_upgrade;
                proxy_set_header Connection 'upgrade';
                proxy_set_header Host $host;
                proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
                proxy_cache_bypass $http_upgrade;
        }

this will not work (for signaling server, accessing to the files work):
location /something/ {
                proxy_pass http://<IP_ADDRESS>:8080/;
                proxy_http_version 1.1;
                proxy_set_header Upgrade $http_upgrade;
                proxy_set_header Connection 'upgrade';
                proxy_set_header Host $host;
                proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
                proxy_cache_bypass $http_upgrade;
        }

Error in the console:
CONNECT_ERR Unable to reach the easyRTC signalling server
How I use setSocketUrl:
this.$easyrtc.setSocketUrl(':443/something'); // not work
this.$easyrtc.setSocketUrl(':443/something/'); // not work
this.$easyrtc.setSocketUrl('/something'); // not work
this.$easyrtc.setSocketUrl('/something/'); // not work
this.$easyrtc.setSocketUrl(':9080'); // work

Why I want to do that:

This allow to pass different service to the 443 port on the same server
This allow me to do that on my html file without hosting the files on my web server:

<script src="/something/socket.io/socket.io.js"></script>
<script src="/something/easyrtc/easyrtc.js"></script>

PeerJs worked with it
I was using PeerJs before and for the connection I used something like that:
this.peer = new Peer({host: \<host\>, port: '', path: 'something'});

Let me know if it possible somehow or if you can consider doing that in the future.
Thank you for your time.

Le me check I had this working in the past.

This is the way, like that we don't have to maintain socket.io basic if you need custom provide your own socket io client instead.
See : https://github.com/priologic/easyrtc/pull/67


Invalid File URL

After migration if i add a file via the upload on my instance of the parse dashboard it uploads successfully however the url it generates when i click on the file points to the localhost and not the api endpoint url. All my server urls configs in parse-server and in parse-dashboard DO NOT point to the localhost and point to the absolute api endpoint. The end of the url for the file location is correct. Below is the screen shot of the url and of course there is an error since safari can't resolve localhost on a remote machine.

Tried it on a heroku (parse-server) install and still the same issue. Any ideas?

in that case please wait till 2.2.5 as it's gonna be fixed by #1287

Updated to 2.2.6 and still the same issue...

You'll also need to set your publicServerURL, can you check that setting and see if it works?

Working now. Thanks! The documentation wasn't clear as that option was under the email verification and password reset section and nothing in the file handling wiki or section


Provide an optional Index of apps on MM

Not MM Meta Pages
Provide an optional index of apps on MM, sort of like a directory listing of what content is hosted as child apps on a particular site.
This should be configurable within the script such that a child app can be hidden through the Index, and it has a name and description through meta data or comments or something similar.

Closed in  697de1b

Closed in v0.2.0 8457019
https://github.com/nikhgupta/middleman-apps#listing-child-apps-on-a-middleman-page


Faster version "World.Get(Entity entity)"

The new version is 30% faster than the original.
    public ref T Get<T>(Entity entity)
    {
        ref var slots = ref EntityInfo.EntitySlots[entity.Id];
        ref var slot = ref slots.Slot;
        return ref slots.Archetype.Chunks[slot.ChunkIndex].GetArray<T>()[slot.Index];
    }

Unity2022.3.20f1

Can you create an official benchmark in arch to verify this for the repo?

Sorry. My bad. This difference only exists in Unity, they are not different in the benchmark.


build ios error

I got this error when build ios device.
ld:symbol not found for atchitecture arm64
xocde: 13.0
macos: macOs Big Sur, intel core i5

Could you upgrade to XCode 15 to see if the issue persists? Unfortunately building iOS apps using the iOS 17 SDK is a requirement enforced by Apple (since iOS 17 is the current version of iOS) and that requires XCode 15.

Thanks for reply。My computer does not support xcode15.
Maybe I should use another library

Unfortunately, you might run into other issues with App Store Connect then.
See https://developer.apple.com/news/upcoming-requirements/?id=04292024a#:~:text=Apps uploaded to App Store,10 starting April 29%2C 2024.


[BUG] g.Cfg()  set  value can't support hierarchical data access by char separator

1. go and gf version
github.com/gogf/gf v2.0.0-alpha   and   v2.0.0-alpha
go1.16 darwin/amd64
2. step
ctx := context.TODO()
g.Cfg().Set(ctx, "logger", map[string]interface{}{})
g.Cfg().Set(ctx, "logger.level", "debug")
testVar, _ := g.Cfg().Get(ctx, "logger")
fmt.Println(testVar)

3. What did you expect to see?
It supports hierarchical data access by char separator, which is '.' in default.

@tianbocheng It makes confusion when using gcfg.Config.Set, so it will be removed in latest version.


MAC randomness, deanonymization and NM/DHCP bug

Using the ranges of <48:> for MAC adress as a reservation in some places reduces the entropy, as network admins can just use regex in a list to easily find the owner, even if the adress was changed multiples times, making his work easier.
Using the script does not make the user invisible on WIFI. Using Airodump-ng with a good wireless adapter, the MAC of the user can be found very quickly.
The script won't work inside a virtual machine, the connection will just hang until i disable the script (using nmcli and the graphical nm is the same result).
Why did you comment hostnamectl, and went back to using hostname ?
Edit: There's also the fact that graphical applications that might require root(such as bleachbit) cannot run after the change of hostname. It seems that one way to fix that is by adding the .Xauthority entry, but doing that will create problems at reboot when trying to log in session, and a way to fix that would be to add a line that remove the .Xauthority file after using OFF on the script(which is really bad), or using chown on the .Xauthority file

this is a part of a previous issue #6
you can always spoof your own MAC, I choosed HP vendor because there are lot of them, so you should be well disguised and making a blacklist of HP vendors isn't a viable solution in an enterprise
speaking about WiFi, you can indeed find a user physically, as I mentioned, if you're saying that the original MAC can be retrieved over distance, then I will require some more details on how exactly you would perform that
I personally tested it on multiple VMs along with other users and apparently we have managed to solve some issues regarding NetworkManager #4 #7 , but if there is still some bugs, I would greatly appreciate if you will inform me more about it

You know more than me about stealth subjet so i will trust you about MAC.
Yes i did read your post on your blog about triangulation, no i can't retrieve the original MAC, i just meant if one was to infiltrate a wifi network, someone that has connection passing the same gateway can detect the one using your script with Airmon-ng. It will show the spoofed MAC sure, but the other user will be aware of the presence.
About the VMs, i found out that the connection will hang indefinitly if the user has the 'auto-reconnect' feature on any AP, and the solution is to disable that feature and then relaunch the script and connect manually. So maybe adding to the script a line that back up the known hosts to avoid reconnection, and set them back when turning the script OFF.

you mean just to detect that somebody has connected to a WiFi network regarding the beacons/data? yes, I agree, perhaps I wasn't mentioning it explicitly by saying "broadcast" in the readme, so I'll update it then
note that the same thing can be done with DHCP if using ethernet, even if you specify manually the addressing
a solution would be to spoof someone else's MAC for even better stealthing
so, you're using a WiFi adapter for your VM? well I'll admit that such tests were out of my scope, thus I'll investigate deeper into that
it seemed to me, however, that the issue #7 mentioning .Xauthority was solved by chowing it on line 122, perhaps you're speaking about the case that the X session was launched by root originaly? I'll look up more at it

Yes i did not express myself correctly, i was indeed talking about beacons. And yes your script is most efficient when using with WAN. Totally agree with you about spoofing else's MAC, using ingeniosity can fix the few weaknesses the script has.
As for .Xauthority well i dont understand what is going on then, you might want to check https://github.com/g0rbe/duppy  it's a script similar to your but written in C++, the owner managed to fix the issue of .Xauthority by chowning it and cleaning the content after each time the user turn OFF the script. I tested it multiple times and it's working.

@cryptolok Last things, i think the sleep time is too long and can hang when it says 'If not connected or taking too long - reconnect manually', especially if one does not one to connect afterward, or can the user CTRL+C at this point without worry ?
And sometimes the hostname does not change though i don't know why(i use nmcli), but noticed the static hostname does not change at all, even when the hostname change successfully.

you can send kill signal at any time, for sure, but it's probably a smart idea to ask user before performing DHCP
by static name you mean /etc/hostname ? that's normal, I won't perform any persistent actions, note that you suppose to launch the script in order to affect the environment, hence the "source" command before the script

I understand, thanks for the answers, it's much stable that way for the system. Though i still dont understand why my hostname does not change, actually it changes to some random numbers while i'm not connected yet, but revert back to the default one the moment i get a connection to an AP/Ethernet.

random number is a random hostname assigned
in this case, try to execute the script within a root shell, if hostname will still change after a network connection, then NM can be the issue,  probably will trace its syscalls to determine the problem

I've just commited some changes to usage README, baiscally adding "source/bash" before executing the script on both activation and deactivation, this should force hostname changes
and added user confirmation for DHCP
I'll see for the NM/Xauth/VM/WiFi a little bit later
speaking about @g0rbe work, seems like he/she/it tried to do the same thing in C++, which is totally free to do since he/she/it wrote own code, but I still found C++ somehow atrocious for such task (almost thousand lines of code) and a little bit egocentric of his/her/its part for not mentioning encountered issues for my project...

I've patched a little the way hostname is changed, this would defenitely solve the problem and even perhaps normalize NM comportment
waiting for your confirmation

cool
try to add "hostname $RANDOM" at line 128, abefore NM restart

Before trying that, i tried as usual because i had an update yesterday, and the hostname is changing now, so i can't tell if adding the line would have fixed it.
Also the script is way faster now thanks, but did you notice each time you run it, it create a new connection for the AP ?
Example: if your ap is "Wifi", the next time you connect to it with the script, it creates "Wifi 1", then next time "Wifi 2". Maybe dropping the known AP would be better ?

nice to know about hostname
but not so about AP...
try to delete the line 136 (nmcli)
but it may be normal due to MAC spoofing since the AP is binded to an interface with a specific MAC

Deleting line 136 did the trick, it does not create a new AP anymore thanks. Dropping the know AP i think still would be nice, making the the auto reconnect feature of NM disabled, but i don't know if it can cause issues(it shouldnt?).

awesome :)
thanks for contribution and feel free to discuss any subject regarding the project

@cryptolok, thanks for being open to suggestion, would it be possible to save the known AP at each startup (/tmp), and get them back when turning off ? "/etc/NetworkManager/system-connections" is the file to look at for AP. I'm suggesting this because it's annoying and not discreet when booting the computer to get directly connected to known AP.

well, the initial point was to automate ethernet connecion, but it's true that I didn't deeply think about the WiFi ...
I'll see about it

I didn't mentioned you either for the very same reason, I didn't know about you or your work in the first place
it is strangely true that I implemented 'hostnamectl' after 11 days of your initial release, but I abandoned the idea since, I don't want permament changes, which hostnamectl does
anyway
my license is widely persimmive so you're free to do anything you like with my work, it's just looks very similar to mine ... but those things happens I guess
I dont care of mentioning, I do my work and try to improve the work of the others and this thread isn't about the subject anyway
so if you have some suggestions about NM issues, I'll gladly consider them as a valauable addition
as of my understanding, I don't see a point of disabling ICMP since, it uses ARP in the first place
however, you totally forgot about IPv6 NDP broadcasting that will spoil your stealth

Yeah, i totally forget ipv6.
I noticed at the arm version of parrot, that change the mac automatically, because of the Network Manager .conf file. So i can overwrite the default config with my conf file. If i want random mac i use the random.conf and with preserve.conf  i can spoof a selected mac and i can avoid the mac changes of NM (if any).
man nm-system-settings.conf could be a good reading about that.
(english is not my native language, so sorry if i explained myself wrong)


Regex for detecting language codes incorrect

System Info
Ubuntu, PHP 8.1.2
PHP Version
8.1.2
Environment/Platform

[ ] Command-line application
[X] Web application
[ ] Serverless
[ ] Other (please specify)

Description
In the Codewithkyrian\Transformers\PretrainedTokenizers\NllbTokenizer class this regex is used to detect language codes: /^[a-z]{3}_[A-Z]{3}$/
However some models, like Xenova/nllb-200-distilled-600M use a format like eng_Latn (full list)
I would suggest something like /^[a-z]{3}_[a-zA-Z]{3,4}$/
Is there a big penalty to false positives here? Is this check required?
Reproduction
$trans = pipeline('translation', 'Xenova/nllb-200-distilled-600M');
$trans('Translation test', srcLang: 'eng_Latn', tgtLang: 'deu_Latn');

You're right, the current regex does not accommodate the formats provided by this particular model. Didn't get to test with it so thank you for bringing this to my attention.
In this context, I don't see any significant penalty for false positives, so sure, your suggested regex /^[a-z]{3}_[a-zA-Z]{3,4}$/ would be more inclusive for different language code formats.
I appreciate your contribution and will incorporate this improvement. Thank you!

Thanks so much for your work! <3


Windows compilation

Hi,
version 0.1.3 does not compile under Windows(rustc v1.45.2, stable-x86_64-pc-windows-msvc toolchain).  Compiler is complaining about trait FromRawSocket is not in scope and sugests to import std::os::windows::prelude::FromRawSocket.
Greetings.
fix_windows_compilation.zip

Sorry for the late reply, this is my mistake, Actually I did not build and test on window. Thanks again for your PR.

#3


Programmatic way to run Vitest

Clear and concise description of the problem
Is there a way to run Vitest programmatically. For StrykerJS we are investigating support StrykerJS. Ideally we would like to be able to set an global environment right before each test run and be able to recover a global environment variable.
Vitest support is currently the most upvoted issue for us. https://github.com/stryker-mutator/stryker-js/issues/3465
Suggested solution
// Code below is pseudo-code to illustrate our needed API.
import { Vitest } from 'vitest';

const vitest = new Vitest();

// runInBand in order to share global variables with the test environment
await vitest.readConfig({ overrides: 'here', runInBand: true, concurrency: 1 });

// register global hook: beforeEach, needed to determine coverage per test. We can also say a test file is a test instead (and use the filename as test name).
vitest.beforeEach((test) => {
  global.__stryker__.currentTest = test.name;
});

// Load the test files without running the tests. Or run the test once, but be able to rerun them without reloading the files
await vitest.loadFiles();
const results = await vitest.run();
// Results include the test names or test filenames including the test results (passed or failed).

// Grab coverage from a global variable
const { mutationCoverage } = global.__stryker__;
// Determine test coverage per mutation.

// Test mutant 1:
global.__stryker__.activeMutation = 1;
const resultMutantOne = await vitest.run({ testFilter: ['..array of test names to run for mutant 1'] });

// Test mutant 2:
global.__stryker__.activeMutation = 2;
const resultMutantOne = await vitest .run({ testFilter: ['..array of test names to run for mutant 2'] });

Alternative
No response
Additional context
We are okay with using a non public API for now and we are willing to do the work 😅
https://github.com/vitest-dev/vitest/issues/524
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

There is non-documented (because it's not stable) way to programatically run Vitest:
import { startVitest } from 'vitest/node'
await startVitest('test', ['filename'])

I think this is how WallabyJS uses it. Currently it returns only true/false, if run was successful. But it will return Vitest (also available from vitest/node) in #2072

You can also use createVitest that will return you Vitest context:
import { createVitest } from 'vitest/node'
const ctx = await createVitest('test')
await ctx.start(['filename'])

Thank you for the fast reply. We are working on a prototype! We'll get back to you for any questions.

There is a small page describing the public API in the documentation: https://vitest.dev/advanced/api.html


Usage quota bandwidth request and/or response?

It is not clear to me if the usage quota bandwidth applies to request payloads and/or response payloads.  Thank you for your help.

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

ID: 248a0753-0fa7-1cc0-d13a-d378ab182e95
Version Independent ID: 7efa247a-7184-6e7f-c1f5-3ff2531df357
Content: Azure API Management access restriction policies
Content Source: articles/api-management/api-management-access-restriction-policies.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

@dapathy Thank you for your feedback! We will review and update as appropriate.

Hey @dapathy - Usage quota by bandwidth applies to the requests made to the APIM, not the requests/responses that APIM makes. @miaojiang could you confirm?

@dapathy Since this doesn't require a doc change, I'm going to close the issue but we can continue our discussion if you had any further questions.

Thank you @mike-urnun-msft

We're still seeing evidence suggesting responses are considered in the bandwidth quotas.  Can we confirm this behavior?  Thank you


[hotfix][azure] fix ArchUnit flag

What is the purpose of the change
This mistake actually prevented the CI from failing when violations have been fixed.
(I expect this to fail the CI the first time as there are fixed violations that haven't been removed)
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure

CI failed correctly again, pushed the removal of the violation and will merge once CI passes.


fix: executeSelect now use provided credentials instead of GOOGLE_APP…

…LICATION_CREDENTIALS
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #2531, #2234 ☕

Thanks for the review.


Upgrade to IP BOM 6.0.0.CR31 + related changes

+1

Working successfully on master.
Merged. Thanks! @psiroky


about the total frames

the waymo-open-dataset has 1150 records with 20s in each record, namely 23w frames totally。in training set,there are 20w frames。you train for 36 epochs. so i want to know if i downsample the training set with 10:1 to 2w frames, do i need to train for 36 epochs?

yes.


Alert for outdated malware client

From the section above, "You can now view Antimalware Client Version and Endpoint Protection Deployment Status by going Assets and Compliance > Overview > Device Collections > All Desktops and Server Clients."
Followed this navigation in my test lab admin console, but could not find the antimalware client  version and protection deployment status.

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

ID: 0eabddcb-8049-59c7-7784-abe31633d274
Version Independent ID: e31eb614-07aa-4085-9a17-50a4e0bf7587
Content: Configure Endpoint Protection alerts - Configuration Manager
Content Source: sccm/protect/deploy-use/endpoint-configure-alerts.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @jocasti for the feedback, I agree that's poorly worded. What it should say is that from any device collection you can add a column to the list for those two attributes. Right-click the column header and select those columns to add. (Although the latter column name is Endpoint Protection Deployment State.)

@aczechowski Create PR#884 using your recommended changes.

Moved to PR#884


Webhook CountryCode fix due to API change

The COVID-19 API now includes country code (CountryCode) in addition to country (Country). This change breaks the chatbot tutorial as it is currently implemented: https://developer.ibm.com/tutorials/crisis-communication-chatbot-watson-assistant-webhook-integration-discovery-covid-data/.
The US Infection Level dialog passes "US" which is now the country code. Modified the webhook to use CountryCode instead of Country.
This could also be fixed by changing the dialog to pass "United States of America" which would not require a change to the webhook.

Thanks @mullerj for the investigation and patch!   We will accept it.  I'd like @moficodes to do a quick review.

@mullerj thanks for catching that.
I think we should update the dialog instead of the web-hook. since most other countries will be the country name.
https://github.com/Call-for-Code/Solution-Starter-Kit-Communication-2020/blob/master/starter-kit/assistant/skill-CDC-COVID-FAQ.json#L2014
This is the line where the country name "US" is passed. Let's change that instead of the web-hook function.

@moficodes No problem, makes sense to update the dialog instead.

@moficodes Updated pull request based on your suggestion to change the dialog instead of the webhook.
I saw another pull request with the same fix so I closed my pull request without merge to cancel it. Reopened my pull request after I noticed that the other pull request was also closed without merge.


fix: not round amounts

Remove custom format functions on the math lib, and replace them with simple helpers using bn.format functions.

@luizstacio if you prefer can do this changes later together https://github.com/FuelLabs/fuels-wallet/issues/198
just wanna make sure to inform @tomiiide

@luizstacio if you prefer can do this changes later together #198 just wanna make sure to inform @tomiiide

No is ok, already fixed


Countdown timer: python version

Make sure that you get this subproject assigned to you before working on it
create a countdown timer using PyQt5 or tkinter. Listed below are the goals that you need to achieve.

[ ] User can see an event input box containing an event name field, an
date field, an optional time, and a 'Start' button.
[ ] User can define the event by entering its name, the date it is
scheduled to take place, and an optional time of the event. If the time is
omitted it is assumed to be at Midnight on the event date in the local time
zone.
[ ] User can see a warning message if the event name is blank.
[ ] User can see a warning message if the event date or time are incorrectly
entered.
[ ] User can see a warning message if the time until the event data and time
that has been entered would overflow the precision of the countdown timer.
[ ] User can click on the 'Start' button to see the countdown timer start
displaying the days, hours, minutes, and seconds until the event takes place.
[ ] User can see the elements in the countdown timer automatically
decrement. For example, when the remaining seconds count reaches 0 the remaining
minutes count will decrement by 1 and the seconds will start to countdown from 59. This progression must take place from seconds all the way up to the remaining days position in countdown display.

Hey! I'd love to work on this for hacktoberfest (I'm experienced with Python, but an absolute beginner at contributions). Would be great if you could assign it to me. Let me know if there's anything else I have to keep in mind while working on this :)

Hey! I'd love to work on this for hacktoberfest (I'm experienced with Python, but an absolute beginner at contributions). Would be great if you could assign it to me. Let me know if there's anything else I have to keep in mind while working on this :)

sure go ahead.
the program directory is 1-Beginner->countdown_timer, create a python folder here and add your python files here.
Add a README.md file if you want to

Hey, I need to ask for clarification on goal5:

User can see a warning message if the time until the event data and time that has been entered would overflow the precision of the countdown timer.

What do you mean by the "precision of the countdown timer"? @kana800

#47: added PR :)


SQL injection vulnerability in field filter

The query API provides the field and value parameters to support query by field.
It inserts the user's input into the raw SQL expression which can lead to a SQL injection vulnerability.
session = session.And(fmt.Sprintf("%s like ?", util.SnakeString(field)), fmt.Sprintf("%%%s%%", value))

And the /api/get-organizations is a public route that everyone can visit.
Proof of concept:
https://door.casbin.com/api/get-organizations?p=1&pageSize=10&value=e99nb&sortField=&sortOrder=&field=(updatexml(1,concat(0x7e,(select%20user()),0x7e),1))%20%20and%20name

@wuhan005 thanks for reporting! Do you have any suggestions for resolving it?

Strict match given field is exactly one in the table, then using prepared statement to avoid SQL injection. (谷歌翻译的，将就着看吧

@seriouszyx can you fix this issue for all occurrences?

@hsluoyz I'll work on it.

@wuhan005 thanks for reporting! Do you have any suggestions for resolving it?

In my opinion, the input field should be filtered by an allowlist, and make sure we have set the Beego's run mode to runmode=prod in production to avoid leaking debug information.

CVE-2022-24124 assigned.


remove True/False arg in UOps.RANGE

will fix the failling tests

Please fix tests and make process replay pass if possible

This misses the point! The point is to remove the need for the True/False, not encode it in a different way


New steps

@hancush, this PR does the following:

consolidates City Council and Plan Commission approval into one step
reorders steps (placing City Council approval before certifying applicant free-of-debts)
conceptualizes steps 9, 10, and 11 as actions happening in the "past"
updates the tally view accordingly

@hancush - made some changes in the tests! For the "past" vs. "future" tense...here's how Jeanne and I talked about it:
Steps 2-7 indicate that we or the applicants need to do something ("future") – i.e., an applicant on Step 2 is waiting for someone to review their deed, an applicant on Step 7 needs to submit an EDS, etc....
But with Step 8, we start to indicate that an action is complete ("past") – i.e., an applicant on Step 8 completed their EDS.
With the final steps, we continue to indicate "past" actions:
Step 9 - Applicant approved by City Council...
Step 10 - Applicant certified...
Step 11 - City sold lot to applicant...

@reginafcompton Thinking on past / future actions, I can see how that would be helpful for Jeanne's staff in the front end. Functionally, though, they're all just plain ApplicationSteps, so I think it would be least confusing to minimize comments differentiating them in the back end, like this one and these.
What do you think?

@hancush.....hmmmmmm. This distiction affects some of the template logic. So, the comments can help shed light on the nature of the front end.

If the only difference is the template logic, and the template logic uses future_steps to determine what text to use, what if we move the comment in question up next to future_steps so it's clear what's informing what we see on the front end? i.e., it's not any function of the ApplicationStep but rather this hard-coded list of steps.

Cool, I added a clarifying comment for this list.


Update integration tests documentation and possibilities

Coverage remained the same at 71.017% when pulling bbc811e769f694494f929195b4fe2479c5bfb83c on tyll:integration_automation into 230d53f282d848291b85b79d7badb4d6772469e0 on nmstate:master.


Slack Channel Does Not Include Team Id

I am using the Azure Bot Service to connect to Slack, but it does not appear to be sending the Slack Team Id in the activity to our Bot. We can obtain the Team Id from the channel data directly, but that is not a fool-proof solution, since the contents of that Channel Data are not consistent between messages. For example, for a simple text activity, it contains a SlackMessage property, but for button press events, it has a Payload property, and the structure of those two properties are different as well. We also have the option of parsing it out of the account id's, but this feels like a dangerous assumption as well.
It would be really great if the Azure Bot Service could place the Team Id in the Conversation.TenantId property which seems like a pretty good analog for what the Team Id is, the uniquely identifying Slack Tenant Id. Based on the ability to generate account id's using the Team Id, it seems the Bot Service has the Team Id available to it, it just doesn't expose it as a stand-alone property. For reference, this is what Microsoft Teams seems to do with it's messages to the Bot.
However it would be passed, it would be great to have some conventional way to retrieve that value for Slack without having to parse out message data. We're trying to minimize as much channel specific code in our bot as possible, and needing to parse out the Slack Team Id is a big pain point.
Below is an abbreviated JSON snippet of the activity data from the Azure Bot Service Slack channel for reference.
{
  "type": "message",
  "serviceUrl": "https://slack.botframework.com/",
  "channelId": "slack",
  "from": {
    "id": "U0123456789:T0123456789"
  },
  "conversation": {
    "id": "B0123456789:T0123456789:D01RM387LUU",
    "tenantId": null
  },
  "recipient": {
    "id": "B0123456789:T0123456789"
  },
  "text": "hi",
  "channelData": {
    "SlackMessage": {
      "team_id": "T0123456789"
    }
  }
}

Hi @kluhman, it's pretty hard for us to do this reliably, for the reasons you stated yourself. If we get it wrong, it will be hard to change without breaking bots dependent on it.
Can you try a middleware approach similar to this and see if it works for you: https://github.com/microsoft/botbuilder-dotnet/blob/main/libraries/Microsoft.Bot.Builder/BotFrameworkAdapter.cs#L1689

@arturl Yep, we can do an approach like that. I was just hopeful it would be possible to do this on the ABS side to minimize channel specific code in our implementation. It seemed to me that the ABS connector already has some mechanism of parsing the Team Id from the various types of Slack requests because it is used in the <user-id>:<team-id> recipient and from account object ids. I'm pretty sure Slack user id's are not returned in that format, so to me it implied the ABS connector was doing the concatenation before sending it to our bot.

Hi @kluhman  - just wondering if Artur's solution will suffice for now. If so, we would like to close this issue.

Yes, we have a workaround in place for this. But it would be great in the future to have support from the ABS for this

Closing with suggestion to provide future support from us for this feature.


meteor plugin as installed package. See http://ternjs.net/doc/manual.html#plugin_third_party

This PR gives you the capability to avoid setting meteor.js inside tern/plugin folder but setting tern-meteor folder inside your node_modules. More once you will publish tern-meteor with npm publish, you will able to install it with npm install. See http://ternjs.net/doc/manual.html#plugin_third_party for explanation how tern loads tern plugin.

+1
This PR is trivial enough.  If @Slava isn't inclined, though, I would agree that you ( @angelozerr ) should just publish tern-meteor to NPM yourself.  Just fork and publish.
Worst case, if you don't want to do it, I'll be happy to do so.

IMHO, I think it should be better that @Slava publish it, because he knows when tern-meteor can be released.

I would not disagree; on the other hand, it's been a year.


Pipewire rather than pulseaudio?

I am seeing discussions about this all over, and arch itself defaults to pipewire as well now.
But here in the packages I saw that pulseaudio is still the default.
Have you looked into this topic already?

Hm, what packages of pulse are remaining?

So, the state is and has been: we're all in for pipewire (installed the manjaro-pipewire group). But you're right, pulseaudio-ctl was present and presumably unused.

re-opening: due to using pulsemixer in waybar, we still depend on pulse compatibility layers. we could probably get rid of it, if we found a pipewire native replacement for it. I don't know of one.

^^ googling this produces mainly hints that one should do exactly what we are doing: install the compatibility layer for pulse or alsa and go with their tooling.


Windows placement memory in Monitor plug and play situations.

Provide a description of the new feature / enhancement
I have a laptop with a docking station that has two monitors connected.
Whenever I unplug my laptop from my docking station, all my windows are moved to the laptop screen, so far so good. When I plug my laptop back into my docking station, the windows are not returned to where they were before. (They seem to randomly go somewhere at just as random sizes).
If I understand the functions of PowerToys correctly, it already does placing windows very well. If it would remember where windows were in different monitor configurations, it would make the usability of plugging and unplugging my laptop from my docking station a LOT more user friendly. Initially, it could be an independent module, it could be rolled out to FancyZones later.
In my personal environment (and this is less standard usage but more power user environment), I use a DisplayPort KVM to switch the monitors to another (linux) computer. The effect is the same: the monitors disappear and reappear often. Not remembering where the windows were makes it a real pain to switch.
Should it matter, Linux GNOME does this perfectly, it returns the windows to the same place when the monitors reappear.

duplicate of: https://github.com/microsoft/PowerToys/issues/10621

going to close against #10621,


DevUI: Updates in Beta2

Beta2 contains a major refactor of DevUI I am creating this placeholder to fix our Dev UI card here when its released.

OK CR1 fixed the issues and I made the changes in the Card to work for CR1 .


Feature Request: Update markdown paths when using git mv

Hi,
I love the new feature "markdown.updateLinksOnFileMove.enabled". I was wondering if it could be expanded to work with git mv or Git: Rename in the command palette.
Apologies if this is a duplicate - I could not find anything when searching.
-Chris

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

Unfortunately I don't think this would be easy to add. For performance reasons, we decided to only have the Markdown path updater listen to file changes made in VS Code (and not those made directly on the filesystem)
The git commands in VS Code simply execute git commands on the command line, so any change look like normal file system changes
I don't think we can address this without negatively impacting performance or causing the rename flow to be triggered in unexpected cases

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Reorganize distributed tree files

I had a lot of trouble navigating ArborX_DetailsDistributedTreeImpl.hpp file while working on APIv2 change, and understanding how parts fit together, and what is used by the nearest and by the spatial search.
This PR splits ArborX_DetailsDistributedTreeImpl.hpp into several files. There is almost no new code, just things moving around. One minor change is making DistributedTreeImpl non-templated, and moving the utility functions out of the struct.
New files:

ArborX_DetailsDistributedTreeSpatial.hpp containing the spatial queryDispatch
ArborX_DetailsDistributedTreeNearest.hpp containing the nearest queryDispatch
ArborX_DetailsDistributedTreeUtils.hpp containing the helper routines (forwardQueries, communicateResultsBack, etc)
ArborX_DetailsDistributedTreeImpl.hpp containing only the friend struct DistributedTreeImpl to be a friend of DistributedTree

One minor change is making DistributedTreeImpl non-templated, and moving the utility functions out of the struct.

Make it a separate commit/pull request.

Make it a separate commit/pull request.

No. It would require me to almost redo the whole thing for little gain.

Rebased on master to fix conflicts.

Had to rebase on #1005 to fix conflicts.


Update Hero Section of the Website

Instead of using solid colors and gradient. I just want to add images to the hero section for better engagement to user and add better relevant effects.
@aditya-bhaumik Please assign this issue to me.

@rajatsinghal02  sure go ahead make sure to star the repo and join the discord server
also make sure to complete in the next 2 days or it will be assigned to someone else

it is mentioned in contributing.md

please lable this issue with hacktoberbest also.

@rajatsinghal02 issue does not matter
Your pull request is the main thing i have added over there


Difference between directory Bundles and dist?

Since I wanna write some my own logic in custom web pack file, I need to know the basic file flow of original web pack config
I found out that there will be a abc.json and also a abc.json, in Bundles and Dist directory separately. take "abc" is a module for example. What's their difference? Thanks!

bundles has files the different entrypoints in your app (basically pages) which is loaded by the browser dynamically.
dist directory has those pages for the use of SSR.

Thanks! @arunoda
I have two last question:

Whicn lines of code of webpack.js in build directory are to config the compilation of the xxx.json file in Bundles directory?
If I want to write my own webpack.js with original function , Do I have to copy the original content of webpack.js?

That's this
Not sure about it.

Thanks so much!  @arunoda


Extend dotenv options and read schema and defaults

I've been missing the ability to declare files like: .env.schema and .env.defaults like in dotenv-extended
TLDR;

These files are allowed to be commited serve as reference, and starting point for 'unaware' people spin up a project with zero or less effort.

Schema: Sometimes an contributor update an required ENV var and unaware of that, i start the project and it breaks.
a Schema will help figuring out what it is, and show in git history what has been changed.

Defaults: Already are achieved on CRA like defaulting to localhost:3000, or PublicPath directory. but the pain is thats not visible, it depends to be documented. and are not expandable by the user to set new defaults automatically to the ENV.

Hi @itsmelion, that sounds interesting - however, I would expect that you'd already be able to see in your git history which variables were changed, and when?
The biggest issue with adding something like this is that it would be static, as people often bump their react-scripts version on existing projects, and the schema and defaults files would not update.

I was going to propose the same thing, but using dotenv-defaults instead of dotenv-extended - IMHO .env.schema adds extra complexity for no real benefit - as @mrmckeb said, those changes are visible in the git history.
https://www.npmjs.com/package/dotenv-defaults

Indeed, .schema is stupid.
the shema can be in the defaults file, but empty or even commented.
Anyway, the manner that Environment Variables works in CRA we can already achieve all of this.
i was mislead with a paradigm that .env is always ignored in source control. but in the Frontend universe it doesn't apply..
TLDR;
we should commit the .env and use it as a defaults file normally.
overrides should go in the .env.* according to the docs.


Timestamp problem when using wait for frames

Required Info

Camera Model
{D435i }

Firmware Version

Operating System & Version
{Win (10)

Kernel Version (Linux Only)

Platform
PC

SDK Version
{ legacy / 2.23.0 }

Language
{python}

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I want to capture consecutive frames using pipeline.wait_for_frames() and depth_frame = frames.get_depth_frame(), and I recorded the timestamp of each obtained depth_frame. The problem is some of the neighboring frames have same timestamp, why would this happen? Following are my code and plot of timestamp:
import pyrealsense2 as rs
import numpy as np

#Configure depth and color streams
pipeline = rs.pipeline()
config = rs.config()
config.enable_stream(rs.stream.depth, 1280, 720, rs.format.z16, 30)
config.enable_stream(rs.stream.color, 1280, 720, rs.format.bgr8, 30)

#Declare pointcloud object
pc = rs.pointcloud()
#Declare point object
points = rs.points()

#Start streaming
profile = pipeline.start(config)
device = profile.get_device()
depth_sensor = device.first_depth_sensor()

timestamp = []
deltaTime = [0]

for j in range(100):
    # Wait for a set of frames
    frames = pipeline.wait_for_frames()
    # Retrieve the first depth frame, if no frame is found, return an empty frame instance.
    depth_frame = frames.get_depth_frame()
    # record the timestamp of retrieved frame
    timestamp.append(depth_frame.timestamp)
    # delete variable
    del depth_frame,frames

pipeline.stop()

@yangliu995 , the pipe object of the SDK is designed to provide an aggregation container (frameset) for all the enabled stream (Depth+RGB). And when only one of the streams (e.g. RGB) is updated on time then the generated frameset will include the new RGB and the previous Depth frames (i.e. repetitions).
In case you comment out the RGB config line then the duplications shall disappear from the resulted graph.

#config.enable_stream(rs.stream.color, 1280, 720, rs.format.bgr8, 30)

@ev-mp Thanks a lot for your reply, that's very helpful. So that means if I enable both RGB and Depth streams, sometimes the depth stream can not be updated on time? If I want to keep RGB stream, is there any way to make both streams update synchronously?

Hi @yangliu995,
Since the hardware sync between depth and color is applied for D415 only, I think you need to manipulate the frame data by your own thru frame number or timestamp.

@yangliu995 , streaming RGB+Depth synchronized can be achieved when you follow certain rules:

RGB and Depth should be set to identical FPS.(15/30)
The Exposure priority control shall be set to prefer FPS over AE  (and prevent auto-exposure from affecting the FPS). Or in case of the manual exposure - the value must be lower than the expected frame interval.

Adhering to those rules would allow D435i to activate and effectively employ the internal RGB->Depth sync with the RGB sensor acting as Master. It is still possible though, for the firmware to temporally deactivate the internal sync due to additional constrains, but in most cases you'll get HW-synchronized depth and RGB frame pairs.

@yangliu995 , do you need it further clarified ? Please update

Thanks a lot. This helped me solve my problem.


[BUG] Do not work with lazy?

I use windows 11 , and neovim > 0.9
I use lazy.nvim to load the goto-preview and have loaded( from the lazy log), but i can not use goto-preview.
have set the default config to true,

Sorry, that's not enough information to go on. Please set debug = true in your config and paste the logs here to start.


Crash when player dies while having a power flower

Crash log: https://mclo.gs/VdK3Tjm
Had the server crash two times when a player died. The crash log seems to indicate it's caused by Project Expansion's power flowers. Not sure if it matters but the player has roughly a dozen final power flowers next to four watches of flowing time on dark matter pedestals and both times he was one-shot by another player. The server immediately crashed the moment they died and I don't think they died any other times since they built the power flowers.

This looks like more of an issue with ProjectE, we're calling methods that should be safe

Confirmed:
Reproduced in 1.18, killed by player, zombie & lightning
Can't reproduce in 1.14 & 1.15

I can also attest to this issue. Died to radiation poisoning. Can no longer get into my world for the time being.
crash-2022-06-04_19.25.39-server.txt

Fixed in 1.0.6


ArgoCD Version

I know I can set the ArgoCD version under .spec.version.
But it would be nice to bump the default version up to 1.6.1 since it's been out for a while.

Perfect timing! v0.0.11 just reased yesterday with 1.6.1 as the default version. I am submitting the PRs for the Operator Hubs and should show up there as soon as those can get merged.

The PR was merged, so expect v0.0.11 to land in the Hub soon.


Style correction

Do the style correction of the files galaxy-chop.py and utils.py

this issue closes with issues #29


Install component does not work

I was going to try this out, but it refuses to install--
I had rust nightly so after reading the other messages here I switched to stable(by making it the default with rustup).
Same result as before.
It brings up the Install button.
I click it.
It says: could no install one of the required rls components
vs2017 15.9.6

Interesting, I'll look into it. This might be due to the renaming of rls-preview to rls.

I just pushed an update that should have fixed this. Would you mind downloading the latest version (0.4.6) and testing?

I grabbed the update, but it still does the same thing
Is there a log file that maybe shows more info about what is going wrong?

I have the same issue

I have the issue as well.
Visual Studio 2017 15.9.7 with the latest plugin from the marketplace.

I'm guessing this has something to do with it.

@v3Monsta unless they changed rustup that shouldn't actually be the issue. installing rls-preview still works from my testing but in newer versions of Rust the module is reported at "rls". When I get home tonight I'll test it just to make sure.

It could also be caused by the installed Nightly build missing the RLS component as described here. Perhaps more verbose errors on attemping to install the components could help with this.

That seems far more likely than my guess and I'll see if downgrading to a build of nightly that includes the RLS fixes my issue. However, while the current rustup does autocorrect rls-preview to rls, it might still be worth changing to avoid breaking if rustup changes in the future.

The extension by default explicitly tries to use nightly, so you'd need to change that under Tools > Options > Rust.  Of course, that dialog was broken for me in VS 2017 15.9.12, possibly due to a VS bug or breaking change - I've created a PR to fix that:  https://github.com/dgriffen/rls-vs2017/pull/24
I've been switching to stable and that seems to work.

I've been switching to stable and that seems to work.

And then the same thing/problem/error in the VS2017.

@mouse07410 that is very strange, if it happens in VS Code as well it may be better to file a bug there since they have more resources to triage the bug and I can then look into what their fix was.

Let me double-check the VSCode.


cSAI are displayed as cDAI

https://twitter.com/davecraige/status/1284930019786014726?s=21

From what I am checking looks like cDAI and cSAI are described properly on the map. Can we close this issue @danfinlay ?


Missing Unittests added for MetricStatusManipulationMSRMTActivationStateNOTRDY

invalidate test run when setMetricStatus precondition fails, added unit tests for MetricStatusManipulationMSRMTActivationStateNOTRDY
Checklist
The following aspects have been respected by the author of this pull request, confirmed by both pull request assignee and reviewer:

Adherence to coding conventions

[x] Pull Request Assignee
[ ] Reviewer

Adherence to javadoc conventions

[x] Pull Request Assignee
[ ] Reviewer

Changelog update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

README update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

config update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

SDCcc executable ran against a test device (if necessary)

[x] Pull Request Assignee
[ ] Reviewer

Please fix build.

Please fix build.

done


OpenPilot Unable to See Lane Markers Causing Lane Departure

Describe the bug

While at slight curve in the highway, there was a change in road type and OpenPilot no longer saw the lane lines and caused my car move into another lane.
How to reproduce or log data

Have OpenPilot engaged at highway speeds and drive on pavement that has different material (imagine cement / asphalt change) while on a curve.
Expected behavior

If I can see the lane lines or markers on the ground, OpenPilot should also see the lane markers so the vehicle does not veer into vehicles. (Maybe hardware limitation to the low resolution of the mobile phone's camera?)
Device/Version information (please complete the following information):

Device: comma two
Dongle ID: bdad50b8a1f9fa8e
Route: bdad50b8a1f9fa8e|2021-02-28--12-26-03--13
Timestamp: bdad50b8a1f9fa8e|2021-02-28--12-26-03--13
Version: master-ci
Car make/model: Toyota Rav4 Hybrid 2021

Additional context

Thanks for report. Should be significantly improved with 0.8.3, due to the new segnet and better laneless performance.


Rendering resolution is higher in WebXR AR than not

Project: https://playcanvas.com/project/998364/overview/webxr-ar-camera-high-dpr
When in AR, it looks like we are rendering in a higher resolution than when we are not. Device pixel ratio is disabled
The 'i' icon in the bottom left is an example of this. Expectation is that the resolution should be the same between the two
Normal

In AR

Framebuffer is created for AR session by underlying system.
When starting XR session, it is possible to provide base layer for rendering, with frame buffer scaler! https://www.w3.org/TR/webxr/#xrwebgllayer-interface
Important to ensure how things like Depth Sensing and few others will be affected.
Also, I've avoided Layers all together from integration, as it is large and complex, and requires well thought API and integration design.

@Maksims I'm looking at the XR manager here: https://github.com/playcanvas/engine/blob/main/src/framework/xr/xr-manager.js#L675
And it changes the resolution of the graphics device when we first enter via the framebufferWidth/Height.
I guess my question here is whether it should be user controllable on what the resolution scale in XR should be? Or should it be left up to the device via getNativeFramebufferScaleFactor?

I was looking at _onSessionStart and changing the creation of the layer to:
        this._baseLayer = new XRWebGLLayer(session, this.app.graphicsDevice.gl, {
            alpha: true,
            depth: true,
            stencil: true,
            framebufferScaleFactor: adjustedScaleFactor
        });

For a test and to match the scale of resolution that is used in the non XR part of the app but the framebuffer being created is still at full resolution with getNativeFramebufferScaleFactor returning 1 instead of the value set above 🤔


Added Novia UAS

Institution/School Name
Novia University of Applied Sciences
Instiution/School Website
https://www.novia.fi/
Email Users
Please place an x between the square brackets if yes

[x]   Students
[ ]   Academic/Teaching Staff
[ ]   Other/Support Staff
[ ]   Alumni

Education Levels
Please place an x between the square brackets if yes

[ ]   Post-graduate research
[ ]   Graduate School
[X]   College (University) or Undergraduate School or equivalent
[ ]   Community College or equivalent
[ ]   High School or equivalent
[ ]   Elementary School or equivalent

Teachers use novia.fi and students use edu.novia.fi


Legacy certs should say "legacy" on profile page

Describe the Issue
The legacy certs all have the word legacy in front of them - e.g.
On the map (There's an open issue to add it to the legacy JS cert)
On the settings page
On the certifications themselves
But we don't have "Legacy" in front of the cert names on the profile page - e.g. example 1, example 2
image

Affected Page
https://www.freecodecamp.org/mot01
Expected behavior
These should all say "View Legacy "

This should be good to go to open up for contribution right?
Or was there something else needed to discuss?

I think changing the key values in the translation.json file will fix this bug.

@moT01  I want to work on this issue can you please assign me?

Feel free to make a PR @shanmukh-sai2003. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues.
If you would like to contribute and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum


Suggestion: Constructor and Class

type Constructor<T> = new(...x: any[]) => T;

type Class<T> = Constructor<T> & {
    prototype: T;
}

The use case for these should be fairly obvious. I'm separating them because, according to MDN, a constructor could choose to return an object that doesn't actually have the constructor's .prototype as its prototype -- but this doesn't actually work in Chrome for me; is that a bug in Chrome or an inaccuracy in MDN? I can't follow the spec well enough to tell either way.
It's worth noting that Constructor<T> should be revised if #5453 or #12265 get added, to the effect of
type Constructor<T, ...U> = {
    new(...U): T;
}

type Class<T, ...U> = Constructor<T, ...U> & {
    prototype: T;
}

but backwards compatibility there requires type argument defaults (I swear I've seen a proposal for that, but I can't find it, or I'd link it too); maybe it would be better to prefix these versions with Vague or something to that effect?

We limit the number of additional declarations added to the library to avoid namespace pollution, and to limit back-compat liabilities. You are welcome to define these type aliases as noted above in your own project and leverage them as needed.


Fix swift-symbolgraph-extract search paths

Fixes and architecture and implementation bug where the include paths
for swift symbolgraph extract were being determined inside build plan
instead of by the target build description. This bug resulted in the
wrong set of paths being determined.
The new code moves the path determination logic into the proper target
build descriptions and releverages the already computed include paths.
The result of this is symbol graphs can be generated for Swift and C
targets properly now without duplicate module in the search paths and
with correct -fmodule usage.

builds on top of #7610

@swift-ci test

@swift-ci test windows

@swift-ci test

@swift-ci test windows


Startup Error

Hello Team,
Thank you for this great effort. I only just installed mail-for-good and go the errors below after running "sudo docker-compose up":
(node:46) DeprecationWarning: Using the automatically created return                                                                                value from client.query as an event emitter is deprecated and will be removed in pg                                                                                @7.0. Please see the upgrade guide at https://node-postgres.com/guides/upgrading
postgres_1  | ERROR:  relation "users" does not exist at character 112
postgres_1  | STATEMENT:  SELECT "id", "googleId", "picture", "token", "email", "na                                                                                me", "sentEmailsCount", "createdAt", "updatedAt" FROM "users" AS "user";

@badruayomaya Hey, this is just an error that occurs for the first run, but everything should work fine - you can check the browser http://localhost

I cannot make it past this spot on AWS. Freezes up every time.

@jeremycady You mean your VM freezes ?

@karuppiah7890 Correct.
Right before:

postgres_1 | STATEMENT: SELECT "id", "googleId", "picture", "token", "email", "na me", "sentEmailsCount", "createdAt", "updatedAt" FROM "users" AS "user";

mentioned above.
I was required to start/stop my instance several times through the process. I ended up being able to see application and login via Google, but I could not add a user, a list, or add my SES information. It would just sit and think and then ended up crashing again.
Not sure what the issue is. I'll try it on my local machine today and see if I can get working there before attempting on AWS again.

I think it's a resource usage issue. What is your AWS EC2 specs ? And yes,
lemme know how it works in your local machine
On Oct 19, 2017 4:26 PM, "jeremycady"<EMAIL_ADDRESS>wrote:

@karuppiah7890 https://github.com/karuppiah7890 Correct.
Right before:
postgres_1 | STATEMENT: SELECT "id", "googleId", "picture", "token",
"email", "na me", "sentEmailsCount", "createdAt", "updatedAt" FROM "users"
AS "user";
mentioned above.
I was required to start/stop my instance several times through the
process. I ended up being able to see application and login via Google, but
I could not add a user, a list, or add my SES information. It would just
sit and think and then ended up crashing again.
Not sure what the issue is. I'll try it on my local machine today and see
if I can get working there before attempting on AWS again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/mail-for-good/issues/227#issuecomment-337873035,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMNw6M9bflvLlJ7E5w4ZxJzqwmw08U2Iks5styrBgaJpZM4P6qTV
.

I had setup a t2.micro instance per the instructions.

Okay. I have tried it only on DO servers actually, with 1GB RAM and it worked quite well. I see that t2.micro also has 1GB RAM. Not sure why the issue. Most probably a bug which is leading to too much resource usage. Can you create a separate issue ?

And yes, do try it in a separate t2.micro instance again, after you are done setting up in your local machine. That way you can ensure the issue is reproducible

I had a huge spike during my first attempt, I think taking up to 75% of CPU. I later removed docker and attempted again, but could have been an issue with it not being a "clean" install.
I believe my initial issue was connecting to the Google API (put in https where it wanted http).
Yep, I'm wanting to do local just so I know have the process down correctly. I'll give it another try on a fresh instance afterward.

75% CPU is too much. And yes, try again later and let me know how it worked

After the recent changes, I'll need you to fill up a separate issue if you encounter any problem on deploy/first launch


Unable to Scroll on VS Code Terminal

VS Code Version: 1.53.2
OS Version: Windows 10 - 1909

Steps to Reproduce:

Open VS Code
Run anything in terminal window producing greater than 25 lines of output. Output appears to be restricted to 25 lines. Terminal buffer is set to 999 and I have checked all shell windows for possible restrictions.

Does this issue occur when all extensions are disabled?: Yes/No  Yes

Can you try to reproduce this in Insider's? I'm unable to

Can you try to reproduce this in Insider's? I'm unable to

Hi! I am not familiar with "Insider's".  It's actually 19 or 20 lines best I can tell.
settings.json:
{
"editor.suggest.maxVisibleSuggestions": 1,
"terminal.integrated.scrollback": 999,
"terminal.integrated.shell.windows": "C:\WINDOWS\System32\WindowsPowerShell\v1.0\powershell.exe",
"terminal.integrated.automationShell.windows": "",
"terminal.integrated.automationShell.osx": ""
}

Some further info here. It doesn't actually seem to be limited to lines, but rather limited to the amount of window available. If I open the screen further it will capture the output to the height of the screen, but there is no scrollback buffer to speak of.
I should also mention that I had my system re-imaged with fresh windows install and the problem still exists. I've added two more images. The first is the output before scrolling, and the second is when I pull up on the scroll bar. Note that it is empty. No buffer captured at all.

What happens if you delete this setting that you have?
"terminal.integrated.scrollback": 999,

No change. I also updated to pdk 2.0.0 and the issue is still present. Not sure if it's related to pdk or not, but other commands such as "tree" show the complete output. I have a similar case open with puppetlabs just in case.
https://github.com/puppetlabs/pdk/issues/829

what happens if you type yes in the teminal and then ctrl c after about 30+ lines have been printed? can you scroll then?

The behavior is just... odd. If I make the screen smaller, the output is limited to about 1 line less than is available in the window. Ie. I can scroll up, but it only shows one additional line of output...

If I expand the terminal window by pulling up on the upper border, I can see more lines that would normally be in the buffer. I can't see these additional lines in the smaller window, ie. I cannot scroll back through them as you normally would be able to.

I just discovered something else. "Clear" is only clearing the visible amount of the terminal. If I type "clear" and then scroll up, the lines that were not visible are still there. I did cntrl-c 50 times, can only scroll through about 25 lines. Typed "clear" and about 25 lines remained.

Can you try changing your renderer type?
"terminal.integrated.rendererType"

Tried all found of the rendering types, but there is no change.
More oddness here. I type cntrl-c 50 times and I had all 50 lines in my buffer and could scrollback through them all the way back to the top of the screen. I ran the 'pdk validate' command after that, and what it appears to be doing is wiping out the scrollback buffer by as many lines as are being output to the screen by the pdk command.
The screenshot here shows the cntrl-c lines missing after running the command.

@meganrogge @Tyriar Just an update on this. Turns out is actually an issue with PDK. Please see the link below for details and thanks for looking into it!
https://urldefense.com/v3/__https://github.com/puppetlabs/pdk/issues/829*issuecomment-790983778__;Iw!!Ojd1I5wBFw!5hIV2_N9XjlBp86SgzNvRbxIdQNzEHRGf6yl66VQl_3FqibMxCVytfHAmR7p4co$

Great, thanks for letting us know!


Site footer

Footer components:

Short about us with link
list of all the site pages includes
license

#80


[meta] Add GSP consumer

Detailed Description
Add config file for GSPs
Should be able to use the same framework, to pull all the gsp data
Need to think how to pull data from yesterday, as the GSP results are updated roughly around midnight

[ ] load GSP data
[ ] add infrastructure terraform and deploy to AWS
[ ] add datamodel GSP yields

Thought was to add this to the same database server as PV data


Fixes bugs in xml docs inclusion

Fixes bug #913 and #914.

Wow that's a shocker of a regression! Thanks for jumping on it so quickly, I'll try get a Nuget out in the next hour or so ...

Just released v5.5.2 with the fix

Thanks for the quick pull. I needed this for a feature I'm implementing :)

Just a heads up, but I think this latest change may have undone the performance improvement that was introduced with 5.5.0 changes. if I use the 5.5.0 version it takes 2 seconds to render the json file, but with 5.5.2 it has gone back to 9 seconds again. I've tried using the file path version of IncludeXmlComments and the new XPathDocument version and both are taking the same amount of time to render.

Yep - I see it. The xml doc creation was separated out again. Will fix and publish later today.

@jreames9 - I just pushed a commit 46d50b that re-introduces the perf. optimization. Before I push a new Nuget, is there any chance you could pull down and test it out?

Sure. Give me a bit to pull the source code and get it set up so I can test
it on my system.
On Tue, Nov 1, 2016 at 11:20 AM, domaindrivendev<EMAIL_ADDRESS>wrote:

@jreames9 https://github.com/jreames9 - I just pushed a commit 46d50b
that re-introduces the perf. optimization. Before I push a new Nuget, is
there any chance you could pull down and test it out?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/domaindrivendev/Swashbuckle/pull/915#issuecomment-257612134,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AON1-7-PqK57Bo6Eel3N3lY-4QM3cPRkks5q52a0gaJpZM4KjZp3
.

--
James Reames Jr.

I pulled the change and reran the call and it dropped the time from almost
10 seconds to about 1.2. So for my implementation it's working great again.
thanks!
On Tue, Nov 1, 2016 at 11:29 AM, James Reames Jr<EMAIL_ADDRESS>wrote:

Sure. Give me a bit to pull the source code and get it set up so I can
test it on my system.
On Tue, Nov 1, 2016 at 11:20 AM, domaindrivendev<EMAIL_ADDRESS>
wrote:

@jreames9 https://github.com/jreames9 - I just pushed a commit 46d50b
that re-introduces the perf. optimization. Before I push a new Nuget, is
there any chance you could pull down and test it out?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/domaindrivendev/Swashbuckle/pull/915#issuecomment-257612134,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AON1-7-PqK57Bo6Eel3N3lY-4QM3cPRkks5q52a0gaJpZM4KjZp3
.

--
James Reames Jr.

--
James Reames Jr.

Looking at the code, it should be fine for mine as well...


Add some path, file, and URL utilities.

Codecov Report

Merging #1 (2a25e0e) into main (c672492) will increase coverage by 0.48%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main        #1      +/-   ##
===========================================
+ Coverage   99.51%   100.00%   +0.48%
===========================================
  Files          17        19       +2
  Lines         825       867      +42
===========================================
+ Hits          821       867      +46
+ Misses          2         0       -2
+ Partials        2         0       -2

Impacted Files
Coverage Δ

filez/filez.go
100.00% <100.00%> (ø)

internal/utils.go
100.00% <100.00%> (ø)

jsonz/jsonz.go
100.00% <100.00%> (ø)

rsaz/rsaz.go
100.00% <100.00%> (+7.14%)
:arrow_up:

templatez/templatez.go
100.00% <100.00%> (ø)

urlz/urlz.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Endpoint for OI for entire chain?

Hi, is there a tastytrade endpoint to get OI for the entire chain instead of having to subscribe to summary for every option?

There is not! You'll need to subscribe to candles or summary events for each individual option unfortunately.

Was afraid of that. Such a terrible design since there's already an endpoint for fetching the entire chain and OI is static every 24 hours.

Do you think you could add an input for choosing which fields to subscribe to in the future to only call for certain fields like OI? Napkin code:
    async def _channel_setup(self, event_type: EventType, event_fields: List[str] = ['eventSymbol']) -> None:
    ......
        if len(event_fields) > 1:
            message['acceptEventFields'] = event_fields
        else:
            message['acceptEventFields'] = accept

Do you think you could add an input for choosing which fields to subscribe to in the future to only call for certain fields like OI?

I must have missed this, sorry!  I'm not sure I see the utility, you can always discard the info you don't need.


The new Angular support not working properly

I just saw the new release, and I wanted to try out the new Angular support. I created a new Angular v14 application, tried the steps in the documentation, and I am getting this error:

package.json
{
  "name": "autoanimate",
  "version": "0.0.0",
  "scripts": {
    "ng": "ng",
    "start": "ng serve",
    "build": "ng build",
    "watch": "ng build --watch --configuration development",
    "test": "ng test"
  },
  "private": true,
  "dependencies": {
    "@angular/animations": "^14.1.3",
    "@angular/common": "^14.1.3",
    "@angular/compiler": "^14.1.3",
    "@angular/core": "^14.1.3",
    "@angular/forms": "^14.1.3",
    "@angular/platform-browser": "^14.1.3",
    "@angular/platform-browser-dynamic": "^14.1.3",
    "@angular/router": "^14.1.3",
    "@formkit/auto-animate": "^1.0.0-beta.2",
    "rxjs": "~7.5.6",
    "tslib": "^2.4.0",
    "zone.js": "~0.11.8"
  },
  "devDependencies": {
    "@angular-devkit/build-angular": "^14.1.3",
    "@angular/cli": "~14.1.3",
    "@angular/compiler-cli": "^14.1.3",
    "@types/jasmine": "~4.0.3",
    "jasmine-core": "~4.3.0",
    "karma": "~6.4.0",
    "karma-chrome-launcher": "~3.1.1",
    "karma-coverage": "~2.2.0",
    "karma-jasmine": "~5.1.0",
    "karma-jasmine-html-reporter": "~2.0.0",
    "typescript": "~4.7.4"
  }
}

I'm not an angular dev myself, but maybe @Waterstraal who contributed the angular content can weigh in. One thing to keep in mind is AutoAnimate is ESM only — meaning if your build environment requires Common JS modules you might have issues.

I'm not an angular dev myself, but maybe @Waterstraal who contributed the angular content can weigh in. One thing to keep in mind is AutoAnimate is ESM only — meaning if your build environment requires Common JS modules you might have issues.

I think the default Angular app uses ESM, so that shouldn't be a problem, but from what I know compiling Angular libraries is not that straightforward :\

Ah yes, @PointSingularity you're right. We would probably need to set up ng-packagr to create a valid Angular Package Format package. This is something that's usually abstracted away when using the Angular CLI, but in this repository that's of course not the case... Sadly, I don't have much time to investigate this. Can you pick this up @PointSingularity?

@Waterstraal Sadly I don't have any free time I can devote to this :(
I think this would be really hard to do with the current project setup. It would require to basically add all angular dependencies + ng-packagr, which would bloat the whole project.
It really is unfortunate that the if you want to build a proper angular library, you need additional tooling.
Another way would be converting this to a monorepo, with multiple publishable libraries, but that would require a lot of restructuring.
The easiest solution IMO would be to just show how to create your own directive, and how to import it, which from what I've seen @Waterstraal has already done.

Any updates on this?

Hey folks, just chiming in. I got this to work in my own project implementing it as a directive similar to what @Waterstraal did.
2 ways this can go without major change this repository setup.

We drop the angular support in this package and have a guide to implement this directive instead.
We drop the angular support in this package and create a new repo to have an angular package that conforms to the angular package requirements using ng-packagr

@Amrhub I just copied the implementation in this repository
https://github.com/formkit/auto-animate/blob/master/src/angular/index.ts

Hi, I have published my library for this since setting up Angular CLI here without significant hiccups is not trivial. I made an Nx mono-repo workspace for this and published a standalone version of the directive on npm.

NPM package: https://www.npmjs.com/package/ng-auto-animate
Demo Angular app: https://ng-auto-animate.netlify.app
Source Code: https://github.com/ajitzero/ng-auto-animate

v0.0.1 is essentially the same code as committed in this repo, except that I do not use options on the directive, which is too common of a name and can cause conflicts. Instead, I will conditionally handle options via the directive name itself.
Example:

...

...

There are currently no docs, but I will try to update this.
Also, I plan to support global default values, such as a custom animation duration which can be configured at the module level. I will look into how that would work with standalone directives.

Great... I understand this is only for Angular 16?

Great... I understand this is only for Angular 16?

It should work with any version of Angular, but the inject() function being used is available from v14 onwards only. Converting it to constructor-based imports will make it work with any project version (9+ at least).
You can also copy the source code from this repo itself and maintain it to use in your projects. The only issue in this repo is the lack of a build step.


Recording fails immediately if recording process fails

Hello,
This might not be a issue or a bug, unless "retry" functionality is already implemented.
Is chinachu supposed to give up immediately if the tuner process fails?
I would believe that it could retry with another free tuner of the same type if it fails to avoid broken recordings.
Example
29 Jan 03:09:00 - PREPARE: 2014-01-29T03:10:00 [テレビ朝日] ももクロＣｈａｎ
29 Jan 03:09:00 - WRITE: /home/chinachu/chinachu/data/recording.json
29 Jan 03:09:55 - RECORD: 2014-01-29T03:10:00 [テレビ朝日] ももクロＣｈａｎ
29 Jan 03:09:55 - LOCK: PT2-T1 (n=2)
29 Jan 03:09:55 - SPAWN: recpt1 --device /dev/pt1video2 --b25 --sid 1064,epg 24 - - (pid=14327)
29 Jan 03:09:55 - WRITE: /home/chinachu/chinachu/data/recording.json
29 Jan 03:09:55 - STREAM: ./recorded/[140129-0310][GR24][PT2-T1]ももクロＣｈａｎ.m2ts
29 Jan 03:09:55 - #recpt1: using device: /dev/pt1video2 using B25... pid = 14327
29 Jan 03:09:59 - #recpt1: Cannot tune to the specified channel: /dev/pt1video2
29 Jan 03:09:59 - UNLOCK: PT2-T1 (n=2)

The end result is a 0 byte sized file.
In this particular case it seems the pt2 driver failed in some way (It was able to tune after a reboot), but I did lose a recording even if /dev/pt1video3 was still working and free for chinachu to use.
What do you think?

now you can use Mirakurun and Rivarun.


Add new APIs to allow efficient retrieval of receipts

What was wrong?
Our current APIs do not allow to implement getTransactionReceipt in Trinity (without reaching into Py-EVM internals).
That's because the expected response includes information that we can not retrieve if the APIs aren't changed or new APIs are added.
Exampe response
```json
{
  "jsonrpc": "2.0",
  "id": 1,
  "result": {
    "blockHash": "0xf64a12502afc36db3d29931a2148e5d6ddaa883a2a3c968ca2fb293fa9258c68",
    "blockNumber": "0x70839",
    "contractAddress": null,
    "cumulativeGasUsed": "0x75d5",
    "from": "0xc80fb22930b303b55df9b89901889126400add38",
    "gasUsed": "0x75d5",
    "logs": [
      {
        "address": "0x03fca6077d38dd99d0ce14ba32078bd2cda72d74",
        "topics": [
          "0x24bcf19562365f6510754002f8d7b818d275886315d29c7aa04785570b97a363"
        ],
        "data": "0x0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000000a4861636b65726e65777300000000000000000000000000000000000000000000",
        "blockNumber": "0x70839",
        "transactionHash": "0x1e2910a262b1008d0616a0beb24c1a491d78771baa54a33e66065e03b1f46bc1",
        "transactionIndex": "0x0",
        "blockHash": "0xf64a12502afc36db3d29931a2148e5d6ddaa883a2a3c968ca2fb293fa9258c68",
        "logIndex": "0x0",
        "removed": false
      }
    ],
    "logsBloom": "0x00000000000000000000000000000400000000020000000000000000400000000000000000004000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000020000800000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
    "root": "0xc659845f1ac4e899ff1b0666dbac5deeda33a4a5d85da71f617f352824146e40",
    "to": "0x03fca6077d38dd99d0ce14ba32078bd2cda72d74",
    "transactionHash": "0x1e2910a262b1008d0616a0beb24c1a491d78771baa54a33e66065e03b1f46bc1",
    "transactionIndex": "0x0"
  }
}

With only a transaction_hash we have no way of telling the block or the index of the transaction within that block which we need to give a qualified response.
How was it fixed?
I tried to change the APIs in a way that would prevent us from unnecessary database access as much as possible (I think).
Therefore I've added the following new APIs to Chain:
def get_canonical_transaction_index(self, transaction_hash: Hash32) -> Tuple[BlockNumber, int]:

:point_up: Allows to make subsequent requests that would not re-read the same information again.
def get_canonical_block_header_by_number(self, block_number: BlockNumber) -> BlockHeaderAPI

:point_up: We have a similar API for full blocks but it seems wasteful to call that when retrieving a block header by number would do it.
def get_canonical_transaction_by_index(self, block_number: BlockNumber, index: int) -> SignedTransactionAPI:

This API is to bypass get_canonical_transaction which would involve another db lookup that for the block number and index that we already know at this point.
def get_transaction_receipt_by_index(self, block_number: BlockNumber, index: int) -> ReceiptAPI:

This API is to bypass get_canonical_transaction which would involve another db lookup that for the block number and index that we already know at this point.
To-Do

[ ] Clean up commit history

[ ] Add entry to the release notes

Cute Animal Picture

This might be best reviewed in the context of https://github.com/ethereum/trinity/pull/1337 that uses these new APIs.

@pipermerriam This is the successor of #1886


Need more cocktail recipes

 Issue by jbkieffer
Mon Nov 19 22:13:52 2018
Originally opened as https://github.com/jbkieffer/github-training/issues/2

Please add one or more cocktail recipes to the file cocktail-recipes.md.

I can find some more cocktail recipes.


Misleading return value when registering duplicate value

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
If a duplicate shortcut is registered, the library logs an error to the console and resolves with an observable holding null (https://github.com/ngneat/hotkeys/blob/master/projects/ngneat/hotkeys/src/lib/hotkeys.service.ts#L160). This leads to unexpected behaviour, because the callback to execute when the shortcut has been pressed, is still executed but the event is null. This becomes especially confusing, because of the of(null) the callback is immediately executed, even if the key has not been pressed.
Expected behavior
The observable should error, if an error occurs, thus not executing the callback.
Minimal reproduction of the problem with instructions
Just register a callback twice.
Environment

Angular version: 15.x

Browser:
- [x] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

You're welcome to create a PR.


option for only code blocks, not inline?

Thank you for the great extension! I've thought of doing this myself a few times but have never gotten to it.
Would it be possible to get a setting to toggle so it would not show up on inline code references? I use inline code references a lot to talk about variables and other things that I wouldn't want to copy, just to get the style of the text to look different.
(also one more note, there is a typo in the roam depo description, clock instead of block: Adds a button to copy the contents of a code clock to the clipboard)

Hmm I'll have to think on that, may have to restructure a bit but it should be possible

If you would be interested in a PR I could give it a shot.

Would definitely welcome a pr! I'm thinking you'd probably have to put have a check for a new extensionAPI settings panel toggle within the mutationObserver

closed via #3


Fix postal_code -> address_zip API mismatch between Native SDK and Stripe JS

Firstly, thanks for the great plugin! 👍
Now, There seem to be some differences in the expected fields of the object used to create a card token between the Native SDK and StripeJS.
postal_code is used in the native SDK, but will cause a 400 (Bad Request) if used with StripeJS while attempting to create a card token in the browser. In lieu of postal_code, StripeJS expects address_zip.
This commit just sets up a function that can be used to map any other differences between the Native and StripeJS card token object (I don't know if there are any more), and fixes the issue with postal_code. With this, the same API can be used in the browser and on the native device.

Thanks!


Move DART-specific constraints into a subdirectory

@cdellin pointed out that we put the DART-specific state spaces in the aikido/statespace/dart subdirectory, but make no such distinction for aikido/constraint. We should consider the same directory structure for aikido/constraint for consistency.

Does it make sense to invert this grouping?  I.e. move these into:
aikido/dart/statespace
aikido/dart/constraint
This would make the DART-specific code all be in one namespace which could be a component (which many other components would probably depend on).  I am not sure if this is useful or not, but it may make some of the use-cases @cdellin was proposing easier by decoupling DART as a hard dependency.
Specifically, now that statespace is its own thing, it's more likely that planners and trajectory representations are be decoupled from interfacing directly with DART.

I would prefer not to have a subdirectory for DART-specific objects because it's AIKIDO's mandatory dependencies. The general rule of AIKIDO I believe is to have a subdirectory for an optional component that depends on optional dependencies.


Exporting the operator 'aten::scaled_dot_product_attention' to ONNX opset version 17 is not supported.

hi,first of all,thanks for sharing the project for us! but when i try to run the "export.py" code,i get the error
torch.onnx.errors.UnsupportedOperatorError: Exporting the operator 'aten::scaled_dot_product_attention' to ONNX opset version 17 is not supported. Please feel free to request support or submit a pull request on PyTorch GitHub: https://github.com/pytorch/pytorch/issues. i referred to the issue #19 and #32，and my environment is
WSL2 Ubuntu 22.04 Python 3.10.13 CUDA 11.8 CUDNN 8.9.4 TensorRT 8.6.1 numpy==1.24.1 onnxruntime-gpu==1.16.0 opencv-python==<IP_ADDRESS> matplotlib==3.8.0, i tried to change opeset value,but it did not work.
is there any suggestions ?thanks

Hi @Albert337, thank you for your interest in LightGlue-ONNX.
What's your PyTorch version? Version 2.1.0 is required. Please refer to requirements-export.txt:
https://github.com/fabio-sim/LightGlue-ONNX/blob/85d7050930cbfc72ce9e806bced05eab6b56732c/requirements-export.txt#L1-L8

Can the model be exported using Torch version 2.1.0 and subsequently utilize the TRT engine on a different version of CUDA and Torch on the same device?


[BUG] Undefined string in german world clock

Describe the bug
undefined string in d/m/y in world clock
To Reproduce
Steps to reproduce the behavior:

start a new world and activate simple calendar
add modules pf2e german and babele
open world clock

Expected behavior
correct display of d/m/y without the undefined string
Screenshots

Foundry Information

Foundry Version: 0.8.9
Game System: PF2E german
Simple Calendar Configuration: PF2e Calendar

Browser (please complete the following information):

OS: Win10
Browser Edge / Chromium
Browser Version 93.0.961.38

Hey based on your screenshots the issue is with the built in world clock of pathfinder 2e not simple calendar.
That dialog isn't simple calendars and I am pretty certain that the PF2E world clock does not pull any information from simple calendar.
Your best bet would be to bring it up to the PF2E system developers.

sorry for the mixup!


Use positive wording when suggesting reviewers

Reading the word "blame" on the first comment of a pull request
could create a negative impression.
By changing "blame information" to "annotation information" we can
keep up the positive spirit of our contributors.

Good call!
What about "history" instead of "annotation"?

@vjeux I was trying to stay close to the actual git commands (https://git-scm.com/docs/git-annotate), but I'm also fine with changing it into something else :)

Sounds good! I had no idea that git annotate was a thing :)


Opportunity FMV Workflow Rule does not account for when Amount is $0 and FMV has not been populated

Thread
Currently, the criteria on the workflow rule is:
((Opportunity: Amount EQUALS 0) OR (Opportunity: AmountEQUALS null)) AND (Opportunity: Fair Market Value NOT EQUAL TO 0)
The way it currently is, if an Opportunity of any record type is created with a $0 Amount, it is being blanked out by the workflow rule.
The criteria needs to be changed to:
((Opportunity: Amount EQUALS 0) OR (Opportunity: AmountEQUALS null)) AND (Opportunity: Fair Market Value GREATER THAN 0)

**lurch: add

Tracking W-029460

Included in beta release 3.145 (Beta 7)


Remove --enable-aad parameter on az aks update command

--enable-aad on az aks commands will fail because --enable-aad can only be used while creating the cluster. @mlearned can you verify?

@TomGeske : Thanks for your contribution! The author(s) have been notified to review your proposed change.

correct  #sign-off


fix loading screen text

*Issue:*https://github.com/bcgov/entity/issues/24518
Description of changes:
Added translation
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun