[SQA] [Access-Control-Cluster] Facing issue while executing all ACL test case combinedly

From chip-test-scripts created by Survensa: CHIP-Specifications/chip-test-scripts#511
Description:
When all the test case in Access-Control-cluster is executed using PICS in UI, the below mentioned test cases are gets failed.

TC-ACL-2.3
TC-ACL-2.4
TC-ACL-2.7
TC-ACL-2.8
TC-ACL-2.9

If the same test cases are executed separately it passed with same PICS uploaded in UI.
Scenario 1:
Executing all test-cases at same time : TC-ACL-2.7,TC-ACL-2.8 in UI is got stuck individually.
Logs:   ACL- All tc's_stuck up 2.7.log , ACL- without 2.7_stuck up 2.8.log
Report:  ACL- All tc's_stuck up 2.7.pdf ACL- without 2.7_stuck up 2.8.pdf
Scenario 2:
Executing all other test cases except the above one : TC-ACL-2.3, TC-ACL-2.4 and TC-ACL-2.9 are failed.
Log:  ACL-without 2.7,2.8.log
Report:   ACL- without 2.7, 2.8.pdf
Scenario 3:
Executing all the above test cases individually, the same is passed.
Logs:  ACL- 2.3.log, ACL- 2.8.log, ACL- 2.9.log, ACL- 2.4.log, ACL- 2.7.log
Report:   ACL- 2.7.pdf, ACL- 2.8.pdf,  ACL- 2.9.pdf , ACL- 2.3.pdf, ACL- 2.4.pdf

@Survensa We will fix this in upcoming PR.

@Survensa  fixed in PR 25712 please verify it and close the ticket.

@manjunath-grl @kvikrambhat
Verified with latest released image(Version: v2.8-official Sha: 3a3e9b3).
Issue is not resolved. When all the semi-automated test case in Access-Control-cluster is executed using PICS below mentioned test case fails.
TC-ACL-2.4
TC-ACL-2.7
TC-ACL-2.8
TC-ACL-2.9
TC-ACL-2.10
Scenario 1:
Executing all semi-automated test-cases at same time :TC-ACL-2.4, TC-ACL-2.10 gets failed and TC-ACL-2.7 in UI  got stuck.
Logs: ACL_All semi-automated.txt,
Report: ACL_All semi-automated.pdf,
Scenario 2:
Executing all other semi-automated test-cases except TC-ACL-2.7 at same time :TC-ACL-2.4, TC-ACL-2.10 gets failed and  TC-ACL-2.8 in UI  got stuck.
Logs:  ACL_without 2.7.txt
Report:  ACL_without 2.7.pdf
Scenario 3:
Executing all other test cases except TC-ACL-2.7 and TC-ACL-2.8 the above one : TC-ACL-2.10 and TC-ACL-2.9 are failed.
Logs: ACL_without 2.7 and 2.8.txt
Report: ACL_without 2.7 and 2.8.pdf
Note: TC-ACL-2.7, TC-ACL-2.8 and TC-ACL-2.10 gets failed even when the test case is executed individually or combined raised issue for the same
Issue link for TC-ACL-2.7 and TC-ACL-2.7: https://github.com/CHIP-Specifications/chip-certification-tool/issues/771
Issue link for TC-ACL-2.10: https://github.com/CHIP-Specifications/chip-certification-tool/issues/768

@raul-marquez-csa did you get a chance to look into this ?

Verified with latest released image(Version: TH Fall2023 Sha: 22538eb7).
Issue is not resolved. When all the semi-automated test case in Access-Control-cluster is executed using PICS below mentioned test case fails.

TC-ACL-2.6
TC-ACL-2.7
TC-ACL-2.8

Logs: ACL_Semi-automated_2023_09_06_14_58_29.log
Report: ACL_SEMI-AUTOMATED.pdf

Verified with latest released image(Version: TH Fall2023 Sha: 3f84bff5).
Issue is not resolved. When all the semi-automated test case in Access-Control-cluster is executed using PICS below mentioned test case fails.

TC-ACL-2.7
TC-ACL-2.8

Logs: ACL_all-semi-automated_2023_10_19_13_03_00.log
Report: ACL_ALL_SEMI-AUTOMATED.pdf
PICS XML file used: PICS for ACL.zip

@raul-marquez-csa Can you take a look at this please ? Looks like I forgot to tag you after I moved it to the open-source repo


Fix parent-child link resolution if seeded from bundle on diff pages

Fixes #47

Coverage increased (+1.6%) to 82.617% when pulling 8c23492aa14b5b20f7f862838b5f9fef29dca950 on bugfix/multipage-preseed-link-resolution into 6bffad2674741126c88b66c7facccf046832ff2f on master.

Coverage increased (+1.6%) to 82.617% when pulling 8c23492aa14b5b20f7f862838b5f9fef29dca950 on bugfix/multipage-preseed-link-resolution into 6bffad2674741126c88b66c7facccf046832ff2f on master.


Create design for Optimus

[ ] Landing Page
[ ] Dashboard
[ ] Photography Template
[ ] Logistics and Fashion Landing Page

Optimus landing page
https://www.figma.com/file/4T9P5cUlMpWiw4XCqt6CVH/Team_19_MyCMS?node-id=42%3A2

Optimus dashboard
https://www.figma.com/file/4T9P5cUlMpWiw4XCqt6CVH/Team_19_MyCMS?node-id=1028%3A3365

Photography portfolio
https://www.figma.com/file/4T9P5cUlMpWiw4XCqt6CVH/Team_19_MyCMS?node-id=297%3A124

Fashion landing page
https://www.figma.com/file/4T9P5cUlMpWiw4XCqt6CVH/Team_19_MyCMS?node-id=100%3A1766

Logistics landing page
https://www.figma.com/file/4T9P5cUlMpWiw4XCqt6CVH/Team_19_MyCMS?node-id=18%3A20


Dynamic content_url?

Could it be that I have to define the content_url in my cordova-hcp.json? How can I set it dynamically?
I have the case that this URL is dynamic (the same for config-file) and could change at runtime. In your Wiki I saw that I can fetch app updates with the JavaScript API:
chcp.fetchUpdate({
  'config-file': 'http://path/to/chcp.json'
});

Unfortunately I can only set config-file but not content_url. Is this as expected or do I miss something here?

Forget it. I have a fallacy in the whole process. chcp.json is located on the server side so I can manipulate it there and can switch the URL's as I want.

Yep, you got it right :)


Iterator improvements

Nullability safety for increment/decrement
Style improvements

These are minor fixes and mostly style but also ensures iteration doesn't throw segfault when iterating past the end, this matches how it is done in HashMap which makes sense to apply everywhere IMO
Can limit it to just null fixes if desired 🙂

Looks good. I'd only suggest using likely().


[WIP] just to show that tabs plugin seems to be broken with lists and indentation

just to show that tabs plugin seems to be broken with lists and indentation
Did you sign your commit? Instructions
Have you read Contributing guidelines?

@slonka yes, in order for the liquid tags to be properly rendered inside lists, you'll need to capture them in a variable and then indent them, you can see an example here.


Support for Razer Blade 15" - 2019 Advanced

Device/Specs:
RZ09-03017E01
i7-9750H
RTX 2070 Max-Q
Windows 11 24H2 (IoT Enterprise LTSC) (26100.712)
PID: 0x0245

Found this project after considering having a go at my own, Synapse is sh*t so kind of need an alternative.
I've been able to get razer-cli enumerate to show the RazerDevice instances, and with manually specifying, or compiling librazer with the PID for my laptop, I can get every function working as far as I know (Fan Control, Performance Modes, Logo Control, Keyboard Backlight).
However trying to run razer-cli info ends up spitting out a Error: Command not supported or Error: Command failed with unknown status: 01, which are the same errors returned by attempting to use functions that don't exist on my model (Battery Care) (There may be more but I'm not sure).
razer-cli itself works (excl info), and I expect razer-tray may be failing due to not getting a valid response from an info check?
Any help on getting this SKU supported would be appreciated, thanks for all your work so far :)

I expect razer-tray may be failing due to not getting a valid response from an info check?

As jank as it is, my temporary workaround is basically commenting out all references to battery care in the tray app, and it seems to run perfectly fine. With all functions working.

Hi @rainyskye, thanks for opening the issue, and I'm glad to see you managed to make it work for you. I merged today #2 which adds some infrastructure to support more devices.
Different laptops have different sets of features, e.g. battery health, boost, and overclock might not be present on some models.
Folks here  came up with compatibility spec and I'll need to come up with something similar.
Let me think over the weekend about how to integrate such functionality. It touches both cli and tray apps.

@rainyskye , could you please share a bit more about manually specifying the PID in code? I was able to find some lines related to this, but due to a lack of experience with rust, I'm not sure how to get things to work as you did.

@rainyskye , could you please share a bit more about manually specifying the PID in code? I was able to find some lines related to this, but due to a lack of experience with rust, I'm not sure how to get things to work as you did.

Manually specifying the PID in code is essentially the same as -p 0xPID0, in my case, if I run razer-cli enumerate, the output will look something like this
RazerDevice { pid: 0x0245, path: \\<path goes here>} }

++ more instances repeated

So if all you want to do is attempt to make your machine work with no other changes, you can use razer-cli -p 0x0245 <command> (replacing 0x0245 with your PID.

Hi @rainyskye, thanks for opening the issue, and I'm glad to see you managed to make it work for you. I merged today #2 which adds some infrastructure to support more devices.
Different laptops have different sets of features, e.g. battery health, boost, and overclock might not be present on some models.
Folks here  came up with compatibility spec and I'll need to come up with something similar.
Let me think over the weekend about how to integrate such functionality. It touches both cli and tray apps.

Sounds good, I think specifying a feature set in the PID list is the easiest way, obviously implementation isn't as simple, but let me know if I could perhaps help at all. :)
Would make future support as easy as potentially an auto detect for which are supported, then they can be strictly defined, that is assuming Razer haven't used the same PID across multiple models with unique feature sets.

@rainyskye , oooh, got it. Though, the cli-only implementation is a bit clunky. Initially, I thought that you recompiled the whole -tray app with the necessary PIDs adjusted to yours.
Nevertheless, tried using only the CLI and it worked just fine:
PS C:\Users\Vladimir\Downloads> .\razer-cli.exe -p 0x023a fan manual
PS C:\Users\Vladimir\Downloads> .\razer-cli.exe -p 0x023a fan rpm 5000
Thanks! Now I need to figure out how to get the tray app to work with my PIDs.

Now I need to figure out how to get the tray app to work with my PIDs.
tray app relies on autodetect and uses hardcoded PIDs from this list https://github.com/tdakhran/razer-ctl/blob/69817b18dcf0a360d6644f0c8e1ae186ed621bed/librazer/src/device.rs#L12

pub const SUPPORTED: &[DeviceInfo] = &[
    DeviceInfo {
        name: "Razer Blade 16 2023",
        pid: 0x029f,
        path: None,
    },
    DeviceInfo {
        name: "Razer Blade 14 2023",
        pid: 0x029d,
        path: None,
    },
];

However, it still might not work, because it queries all information that is hardcoded now. As discussed here https://github.com/tdakhran/razer-ctl/issues/12#issuecomment-2156023690, PID is not sufficient to differentiate between laptop models and I plan to move to Razer Model IDs.

my temporary workaround is basically commenting out all references to battery care in the tray app, and it seems to run perfectly fine. With all functions working.

If the aforementioned changes made razer-cli info work, then adding PID to the table and removing battery health-related stuff from razer-tray should make it work.


Splitting existing EKS terraform script into modules

What's about splitting the script parts into descriptive modules, either per Jenkins X functional group similar to here https://github.com/jenkins-x/terraform-google-jx/tree/master/jx/modules or by cloud "resource" type, eg service-account, storage, secret.

I think in an ideal world we should just have main|output|variables.tf in the root level of the terraform directory (together with the README).

This is a good idea, I'll do this for the EKS scripts.

This is a good idea, I'll do this for the EKS scripts.

We probably want to align between the different cloud provider scripts then.


[Shared model] Update namings in host config

Related Issue
Part of #4005
Description
Updates host config namings and removes unused variables from rendering to reflect the dev spec
Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run AdaptiveCards-SharedModel-CI, AdaptiveCards-.NET-PR, AdaptiveCards-NodeJS-Daily, AdaptiveCards-UWP-Daily-PRs, AdaptiveCards-Android-CI, AdaptiveCards-iOS-CI

all errors are fixed by other branches to be merged into feature/inputEvolution, merging


PLTCONN-3883: Implement changes to support Direct permission config in connector spec that will show up in UI

Description
What is the intent of this change and why is it being made?
Add permissions to entitlements
How Has This Been Tested?
What testing have you done to verify this change?
Manual testing

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


Support overwrite tag

Please answer these questions before submitting pull request

Why submit this pull request?
[ ] Bug fix
[x] New feature provided
[ ] Improve performance

Coverage increased (+0.009%) to 14.421% when pulling d91a2a90700828f3c2d34a3182fd38424dbb8e7a on ascrutae:feature/support-overwrite-tag into aa9a7c443774d96385f308f2ec1f425f47719958 on apache:master.

Merged. Let's fix webflux and run tests to review all existing cases.


Fix integer CLI arguments

The jest-validator PR broke this and we now have to be explicit about them.

👍


Can u use flow messages?

Are u using flow messages? like this messages

Hi, thanks for opening issue. Currently we don't implement flow messages. It's need verified business whatsapp account and we cannot do that atm. Actually this wrapper just for internal purpose, and we doesn't use it for now, bu there is possibility that we will add the flow messages in the future

Ok, thx


Fix alert timer by adding controller

Should resolve https://github.com/jpochyla/psst/issues/500

Just tested, works perfectly for me :P


[bug] Set onion as default FF search engine - do a URL bar search, searx redirects to .local

This is unacceptable for remote use for obvious reasons

Fixed by: 5966933d5219b471c63027e2b19c07d97b3f7ab2


148 hyperfine use f levels

Fixes #148.

Pull Request Test Coverage Report for Build 154

133 of 134 (99.25%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.3%) to 90.719%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

maxwellbloch/hyperfine.py
29
30
96.67%

Totals

Change from base Build 150:
2.3%

Covered Lines:
1300

Relevant Lines:
1433

💛  - Coveralls


feat: add saya service

This PR aims at adding the basis for Saya service.
Saya service is responsible of the settlement of a Katana instance.
Here's a recap for Saya's needs:

Data availability: to publish the state update. Currently only celestia is supported.
Trace generation: using SNOS in rust, Saya must generate the trace corresponding to the state transition. This is the next step that should be covered.
Facts registry: Once the trace is generated, we can compute the fact for the state transition, which can be optimistically registered on the settlement layer.
Proof generation: The trace and the facts are then sent to a prover in order to generate the proof related to them. Once generated, the proof is sent to the settlement layer verifier with the facts.

This PR adds the skeleton of Saya, where the DA part is first implemented.
Example of usage:
cargo run --bin saya -- --rpc-url http://localhost:5050 --da-chain celestia --celestia-node-url http://<IP_ADDRESS>:26658 --celestia-namespace glihm --celestia-node-auth-token eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.....

For the other steps, we still depend on external dependencies. Once they are all updated, we will be able to update Saya to support them.
I will open some issues to keep improving Saya's service, like:

Add a all-in-one toml file to configure all Saya service (DA, prover, etc...).
Improve options CLI.
Add a RPC server to query data from Saya and current progress.
Add some parallelism when it's possible, as Saya will inevitably be lagging due to the settlement layer being slower than Katana.
Add testing on args and CLI side of things.

If had to modify the Cargo.toml to ensure we use a very specific version of cairo-lang related stuff, due to breaking changes in the 2.5.0.
@kariy I've started to migrate to 2.5.0 locally to evaluate all the required changes, we should sync on this to prepare the bump + preparation of the RPC 0.6.0. :+1:

Also we may want to exclude building saya binary from the release ci considering it's only the basic implementation atm.

As discussed, it is not included in the archive, but only in the build. Thanks for pointing this out anyway @kariy. :pray:


Tagged Queries

Add user supplied comments to the generated SQL. Used for tracing SQL back to the code that generated it.

Duplicate of #267


-[AFSecurityPolicy evaluateServerTrust:forDomain:] crash

Hi there,
I recently tried to add certificate file to my iOS project. I'm getting crash.
It seems that it can find the correct .cer file, but it's failing to add a NSData object to a mutable array.
 for (NSData *certificateData in self.pinnedCertificates) {                 [pinnedCertificates addObject:(__bridge_transfer id)SecCertificateCreateWithData(NULL, (__bridge CFDataRef)certificateData)];             }
* Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '* -[__NSArrayM insertObject:atIndex:]: object cannot be nil'
*** First throw call stack:
(
0   CoreFoundation                      0x000000010f2e234b exceptionPreprocess + 171
1   libobjc.A.dylib                     0x000000010e92621e objc_exception_throw + 48
2   CoreFoundation                      0x000000010f21338f -[__NSArrayM insertObject:atIndex:] + 1375
3   AFNetworking                        0x000000010a2713a2 -[AFSecurityPolicy evaluateServerTrust:forDomain:] + 1378
4   AFNetworking                        0x000000010a290bb1 -[AFURLSessionManager URLSession:didReceiveChallenge:completionHandler:] + 1089
5   Foundation                          0x000000010e42a2cd __NSBLOCKOPERATION_IS_CALLING_OUT_TO_A_BLOCK + 7
6   Foundation                          0x000000010e429faf -[NSBlockOperation main] + 101
7   Foundation                          0x000000010e4286ac -[__NSOperationInternal _start:] + 672
8   Foundation                          0x000000010e4245ef __NSOQSchedule_f + 201
9   libdispatch.dylib                   0x000000010fe760cd _dispatch_client_callout + 8
10  libdispatch.dylib                   0x000000010fe53e6b _dispatch_queue_serial_drain + 236
11  libdispatch.dylib                   0x000000010fe54b9f _dispatch_queue_invoke + 1073
12  libdispatch.dylib                   0x000000010fe573b7 _dispatch_root_queue_drain + 720
13  libdispatch.dylib                   0x000000010fe5708b _dispatch_worker_thread3 + 123
14  libsystem_pthread.dylib             0x0000000110225736 _pthread_wqthread + 1299
15  libsystem_pthread.dylib             0x0000000110225211 start_wqthread + 13

Could it be that your cer file is broken?

make sure your certificate is in DER format

It turned out that the certificate was not in DER format. After I converted the format, it doesn't crash now.

Same issue here. We were using a PEM encoded cert and the app crashed. We fix it converting the PEM encode to DER:
`PEM to DER
openssl x509 -in cert.crt -outform der -out cert.der`
More info: https://support.ssl.com/Knowledgebase/Article/View/19/0/der-vs-crt-vs-cer-vs-pem-certificates-and-how-to-convert-them


feat(sidenav): emit event when backdrop is clicked

This allows clients to distinguish between close events caused by calling close() and those caused by the backdrop being clicked.
Closes #1427

I resigned the CLA for my additional email address.

Rebased onto current master branch to get alpha-10 goodness and resolve a merge conflict.


Add reference to end-to-end serverless sample in main README.md

Added a new entry in the main README.md to reference the End-to-End/Serverless sample.
@kaverma, can you please merge this one. With this we close the circle!
Thanks.

Merged, thanks Raúl for completing this change!
From: Raúl Alarcón @.>
Date: Friday, 11 March 2022 at 8:34 PM
To: Azure/actions-workflow-samples @.>
Cc: Kanchan Verma @.>, Mention @.>
Subject: [Azure/actions-workflow-samples] Add reference to end-to-end serverless sample in main README.md (PR #97)
Added a new entry in the main README.md to reference the End-to-End/Serverless sample.
@kavermahttps://github.com/kaverma, can you please merge this one. With this we close the circle!
Thanks.

You can view, comment on, or merge this pull request online at:
https://github.com/Azure/actions-workflow-samples/pull/97
Commit Summary

7c54b84https://github.com/Azure/actions-workflow-samples/pull/97/commits/7c54b8408abe6a0c307ca4d6a86c750045bde789 Add reference to end-to-end serverless sample in main README.md

File Changes
(1 filehttps://github.com/Azure/actions-workflow-samples/pull/97/files)

M README.mdhttps://github.com/Azure/actions-workflow-samples/pull/97/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5 (2)

Patch Links:

https://github.com/Azure/actions-workflow-samples/pull/97.patch
https://github.com/Azure/actions-workflow-samples/pull/97.diff

—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/actions-workflow-samples/pull/97, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGELAKL4UBVAM42Q4NNT2ZTU7NOHDANCNFSM5QP3ZGLA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>


docs: update index.md tutorial-overview buttons

Description
This PR updates the misalignment of the Tutorial-Overview Buttons
Related issue(s)
Fixes #1099

Excellent job making these PRs to fix the prev/next buttons, @thulieblack! Thank you for showing initiative and helping us fix our mistake 🙌🏽

/rtm

/rtm


misc: remove related images/contents/snapshots in cleanup progress

The cleanup process exits before completion. In addition to snapshots, we also need to clean up related content and images.
Depends on Ryan's PR #605.

cc @ryansavino

I fixed a few things in #605 per review comments. You may want to re-cherrypick the commits.

I fixed a few things in #605 per review comments. You may want to re-cherrypick the commits.

Done.

LGTM


Interfaces

https://github.com/ratfactor/ziglings/issues/153

@chrboesch Okay, see what you think of the added text. I hope you don't mind that I moved the printing loop into main(). I thought it would make the demonstration a little simpler.

@ratfactor Really great, I love it!

Awesome. Thanks for making the exercise!


Add 'Of Ships & Scoundrel' to Showcase Gallery

Part of #304, results can bee seen here:
https://github.com/crest-ocean/crest/tree/readme/ships-and-scoundrel#showcase-gallery

Great!


feat: support wrap in code block

Summary
New Feature
Add a wrap icon button to toggle the wrapLongLines feature provided by react-syntax-highlighter, enhancing the readability of long code lines.
Not Enabled:

Enabled:

The two wrap icons come from tabler-icons, which are under the MIT License.
New Config
Add a new config named defaultWrapCode in markdown section, decide whether enable wrap by default.
Related Issue

Checklist

[ ] I have added changeset via pnpm run change.
[x] I have updated the documentation.
[ ] I have added tests to cover my changes.

Nice!

Please resolve the conflict.

Conflict has been resolved and changeset has been added.


Logging via syslog

I'd like to be able to log behavior that happening on selfauth so there's some amount of visibility on my system around authentication. If login attempts are being made, I want to be able to get OSSEC alerts which can be picked up if selfauth writes to syslog.
Since not everyone needs this, I'm thinking a few new (completely optional, for backwards compatibility) constants:

LOG_SUCCESSFUL -- defaults to true, logs when the user authenticates successfully.
LOG_FAILED -- defaults to true, logs when an authentication attempt is rejected because the password is incorrect, or another problem occurred.
LOG_FAILED_PASSWORDS -- defaults to false, allows logging the password attempt when the login fails.

Logging would be enabled by default for existing installations and for new ones, although with passwords omitted. The setup script could emit these constants to allow people to configure logging should they wish to.
When LOG_SUCCESSFUL is true, the following is then written to syslog via the syslog() function:
IndieAuth login successful from {remote-ip}.

Should the User Agent be included?
Logs would write at the Notice level for a successful login and the Error level for a failed login.
When LOG_FAILED is true, the following is written to syslog. The terminology is intentionally designed to trip up OSSEC:
IndieAuth: login failure from {remote-ip}.

When LOG_FAILED_PASSWORDS is also true, then this is appended to the end of the previous syslog entry:
Password refused: {password}.

Please let me know what you think. If you're happy with this idea, I can start work on an implementation.

Hi,
This shouldn't conflict with shared hosting. If syslog is disabled or otherwise unsupported on the host OS, syslog() might return false when unable to successfully write to syslog, so that return value could be ignored. In the future, perhaps another method like mail() could be tried.
If the syslog() function is disabled through the disable_functions INI setting, then AFAIK, the function won't exist at all, so wrapping it in function_exists() should be enough to detect that.
Checking the documentation, syslog() apparently never raises any PHP notices, warnings, or errors, doesn't throw any exceptions, and has been available since PHP 4.0, so should be available on a shared hosting platform -- although I can imagine the logs may not be accessible.
Perhaps during setup we could expose a small checkbox to control the default value so it may be completely disabled?

This totally slipped me by, so here we go. I do like the idea of logging things, and syslog() is probably the best solution unless we want to pull in something like PSR-3. More thoughts:

I would not turn any logging on by default. I do think logging IPs with authentication requests makes sense, and I would simply never want to log any IPs by default. Especially when people running this on shared hosts might be feeding it into logs they themselves cannot clear.
LOG_FAILED_PASSWORDS sounds like a nice-to-have that needs massive disclaimers around it. We can’t work on the assumption that everyone is using a password manager. This means people are typing their passwords, and typos happen. This option sounds good, but if you over time fill logs with deviations of your real password, you better be making sure you are purging those logs real good. (Of course again with the problem that syslog() may be out of reach to the user who unwittingly turned this on.)

I can almost see us strategically dropping these into the source code, but commented. Anyone who understands syslog() and wants to use it to trip up other alarm bells on a server, will probably be OK uncommenting a couple of functions. Even if they aren’t well versed with PHP. This will at least keep it out of the hands of users who cannot see the possible side-effects.
Like the idea, just not sure how to execute it without giving users some flags in the config with huge warning disclaimers. And I don’t like warning disclaimers in what is supposed to be a simple single-purpose thing.

Hi,
Good points being made here, I hadn’t thought about typos or shared hosting. That’s what years of using a password manager and dedicated hosting will do!
I like the idea of having some commented out code, although I wonder if that is going to be problematic every time there is an update and I git pull as there would be changes on the local filesystem.
I’d be happy to write this, likely on the weekend when I have more time. I’ll send a pull request when I can.
Perhaps longer term, there could be a small plugin system for self auth so any additional features that may need to carry warning labels can be developed outside self auth? I could then continue to develop it to have, e.g. Pushover support (which is quite a bit more complex as I’d probably need ext/curl support).
Would you be open to me coming up with a proposal for a plugin system in another issue?

I like the idea of having some commented out code, although I wonder if that is going to be problematic every time there is an update

This is actually a really good point to raise. And something I forgot because I have been operating out of my own private branch and merging changes into it as we go anyway. Everyone is so used to their own setups that it is sometimes hard to think of other setups out there :wink:
So maybe not hide it behind a comment but behind a constant anyway? And just not include the constant in the example config at all, so people can only discover it by looking at the code and seeing what it does?
if (defined('SYSLOG_FAILURES') && SYSLOG_FAILURES === 'I understand') {
    syslog(sprintf(
        'IndieAuth: login failure from %s for %s',
        $_SERVER['REMOTE_ADDR'],
        $me
    ));
}

For future reference, logging passwords to system logs …
@TwitterSupport:

We recently found a bug that stored passwords unmasked in an internal log. We fixed the bug and have no indication of a breach or misuse by anyone. As a precaution, consider changing your password on all services where you’ve used this password.

Oopsie.


Fwup error fix

Don't actually merge this.
What i did
Wrote a tiny script that wraps a command allowing it to successfully complete before closing file descriptors.
why
There seems to be a race condition in Erlangs :port implementation where the file descriptor closing message gets dispatched before the final IO data message. This causes the Port to be closed before receiving the error message from FWUP.

Please don't judge me

Not judging. I think this is great. You dug into this enough to get me ideas on how to fix this more cleanly and also to understand what Erlang is doing behind the scenes. I think that I have more respect for Erlang now but I'm going to double check my hunch tomorrow when my brain is fresh.
Creative use of dd to simulate a sleep... It made me remember than I hadn't enabled it in Busybox.

It worked pretty reliably without the simulated sleep. But with about 1 or 2 seconds it's battle ready. :|

Thanks! I added a handshake with fwup that fixes the fundamental problem and the change is in #12. Your debug was critical for me to see this.


ExportFormatter: Add missing book description and remove extra newlines in markdown export

Hi,
This pull request includes two updates to the app/Entities/Tools/ExportFormatter.php file regarding the markdown export:

Added the missing book description in the bookToMarkdown() method.
Added a check for an empty chapter description, preventing unwanted extra lines (\n\n) between the chapter name and the page content when the description is empty.

Thanks for these changes/fixes @czemu!
They are now merged to be part of the next feature release.
I followed this up with 0f9957bc036e40d068dfb0958560485f7456964a to handle the wider range of potential formatting in descriptions via HTML to MD conversion, and to also update testing to cover the changes made.

Thanks @ssddanbrown!


probleme installation python_epson_printer

Bonjour,
j'ai un problème sur un raspberry 2 Raspbian Jessie 4.4:
après lancement de la commande "sudo python setup.py install"
Il semble y avoir un problème avec atlas
Merci d'avance
running install
running bdist_egg
running egg_info
writing requirements to python_epson_printer.egg-info/requires.txt
writing python_epson_printer.egg-info/PKG-INFO
writing top-level names to python_epson_printer.egg-info/top_level.txt
writing dependency_links to python_epson_printer.egg-info/dependency_links.txt
reading manifest file 'python_epson_printer.egg-info/SOURCES.txt'
writing manifest file 'python_epson_printer.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-armv7l/egg
running install_lib
running build_py
creating build/bdist.linux-armv7l/egg
creating build/bdist.linux-armv7l/egg/epson_printer
copying build/lib.linux-armv7l-2.7/epson_printer/init.py -> build/bdist.linux-armv7l/egg/epson_printer
creating build/bdist.linux-armv7l/egg/epson_printer/test
copying build/lib.linux-armv7l-2.7/epson_printer/test/init.py -> build/bdist.linux-armv7l/egg/epson_printer/test
copying build/lib.linux-armv7l-2.7/epson_printer/test/printable_image.py -> build/bdist.linux-armv7l/egg/epson_printer/test
copying build/lib.linux-armv7l-2.7/epson_printer/epsonprinter.py -> build/bdist.linux-armv7l/egg/epson_printer
copying build/lib.linux-armv7l-2.7/epson_printer/testpage.py -> build/bdist.linux-armv7l/egg/epson_printer
byte-compiling build/bdist.linux-armv7l/egg/epson_printer/init.py to init.pyc
byte-compiling build/bdist.linux-armv7l/egg/epson_printer/test/init.py to init.pyc
byte-compiling build/bdist.linux-armv7l/egg/epson_printer/test/printable_image.py to printable_image.pyc
byte-compiling build/bdist.linux-armv7l/egg/epson_printer/epsonprinter.py to epsonprinter.pyc
byte-compiling build/bdist.linux-armv7l/egg/epson_printer/testpage.py to testpage.pyc
creating build/bdist.linux-armv7l/egg/EGG-INFO
copying python_epson_printer.egg-info/PKG-INFO -> build/bdist.linux-armv7l/egg/EGG-INFO
copying python_epson_printer.egg-info/SOURCES.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
copying python_epson_printer.egg-info/dependency_links.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
copying python_epson_printer.egg-info/requires.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
copying python_epson_printer.egg-info/top_level.txt -> build/bdist.linux-armv7l/egg/EGG-INFO
zip_safe flag not set; analyzing archive contents...
creating 'dist/python_epson_printer-1.8.0-py2.7.egg' and adding 'build/bdist.linux-armv7l/egg' to it
removing 'build/bdist.linux-armv7l/egg' (and everything under it)
Processing python_epson_printer-1.8.0-py2.7.egg
Removing /usr/local/lib/python2.7/dist-packages/python_epson_printer-1.8.0-py2.7.egg
Copying python_epson_printer-1.8.0-py2.7.egg to /usr/local/lib/python2.7/dist-packages
python-epson-printer 1.8.0 is already the active version in easy-install.pth
Installed /usr/local/lib/python2.7/dist-packages/python_epson_printer-1.8.0-py2.7.egg
Processing dependencies for python-epson-printer==1.8.0
Searching for numpy>=1.11.0
Reading https://pypi.python.org/simple/numpy/
Best match: numpy 1.11.1
Downloading https://pypi.python.org/packages/e0/4c/515d7c4ac424ff38cc919f7099bf293dd064ba9a600e1e3835b3edefdb18/numpy-1.11.1.tar.gz#md5=2f44a895a8104ffac140c3a70edbd450
Processing numpy-1.11.1.tar.gz
Writing /tmp/easy_install-gDC2rg/numpy-1.11.1/setup.cfg
Running numpy-1.11.1/setup.py -q bdist_egg --dist-dir /tmp/easy_install-gDC2rg/numpy-1.11.1/egg-dist-tmp-Lxv7Cz
Running from numpy source directory.
/tmp/easy_install-gDC2rg/numpy-1.11.1/setup.py:327: UserWarning: Unrecognized setuptools command, proceeding with generating Cython sources and expanding templates
warnings.warn("Unrecognized setuptools command, proceeding with "
Warning: distutils distribution has been initialized, it may be too late to add a subpackage compatWarning: distutils distribution has been initialized, it may be too late to add an extension _dummyWarning: distutils distribution has been initialized, it may be too late to add a library npysort/tmp/easy_install-gDC2rg/numpy-1.11.1/numpy/distutils/system_info.py:1646: UserWarning:
Atlas (http://math-atlas.sourceforge.net/) libraries not found.
Directories to search for the libraries can be specified in the
numpy/distutils/site.cfg file (section [atlas]) or by setting
the ATLAS environment variable.
warnings.warn(AtlasNotFoundError.doc)
/usr/bin/ld: ne peut trouver -lcblas
collect2: error: ld returned 1 exit status
/usr/bin/ld: ne peut trouver -lcblas
collect2: error: ld returned 1 exit status
Warning: distutils distribution has been initialized, it may be too late to add an extension multiarrayWarning: distutils distribution has been initialized, it may be too late to add an extension umathWarning: distutils distribution has been initialized, it may be too late to add an extension umath_testsWarning: distutils distribution has been initialized, it may be too late to add an extension test_rationalWarning: distutils distribution has been initialized, it may be too late to add an extension struct_ufunc_testWarning: distutils distribution has been initialized, it may be too late to add an extension multiarray_testsWarning: distutils distribution has been initialized, it may be too late to add an extension operand_flag_tests/bin/sh: 1: svnversion: not found
Warning: distutils distribution has been initialized, it may be too late to add a subpackage coreWarning: distutils distribution has been initialized, it may be too late to add a subpackage commandWarning: distutils distribution has been initialized, it may be too late to add a subpackage fcompilernon-existing path in 'numpy/distutils': 'site.cfg'
Warning: distutils distribution has been initialized, it may be too late to add a subpackage distutilsWarning: distutils distribution has been initialized, it may be too late to add a subpackage doc/bin/sh: 1: svnversion: not found
Warning: distutils distribution has been initialized, it may be too late to add a subpackage f2pyWarning: distutils distribution has been initialized, it may be too late to add an extension fftpack_liteWarning: distutils distribution has been initialized, it may be too late to add a subpackage fftWarning: distutils distribution has been initialized, it may be too late to add a subpackage lib/tmp/easy_install-gDC2rg/numpy-1.11.1/numpy/distutils/system_info.py:1548: UserWarning:
Atlas (http://math-atlas.sourceforge.net/) libraries not found.
Directories to search for the libraries can be specified in the
numpy/distutils/site.cfg file (section [atlas]) or by setting
the ATLAS environment variable.
warnings.warn(AtlasNotFoundError.doc)
Warning: distutils distribution has been initialized, it may be too late to add an extension lapack_liteWarning: distutils distribution has been initialized, it may be too late to add an extension _umath_linalgWarning: distutils distribution has been initialized, it may be too late to add a subpackage linalgWarning: distutils distribution has been initialized, it may be too late to add a subpackage maWarning: distutils distribution has been initialized, it may be too late to add a subpackage matrixlibWarning: distutils distribution has been initialized, it may be too late to add a subpackage polynomialWarning: distutils distribution has been initialized, it may be too late to add an extension mtrandWarning: distutils distribution has been initialized, it may be too late to add a subpackage randomWarning: distutils distribution has been initialized, it may be too late to add a subpackage testingWarning: distutils distribution has been initialized, it may be too late to add a subpackage numpyCould not locate executable gfortran
...

il semblerait que ce soit uniquement des warnings ?

Je vous met le log complet.
Merci d'avance
pi.txt


Self related model findMany query fails

I have this model, Item, that can be linked to other Items.
The connect/disconnect API works great, but when I go to actually query for items related to this item Prisma fails with this error:
Error occurred during query execution: ConnectorError(ConnectorError { user_facing_error: None, kind: QueryError(Error { kind: Db, cause: Some(DbError { severity: "ERROR", parsed_severity: Some(Error), code: SqlState("42601"), message: "syntax error at or near \")\"", detail: None, hint: None, position: Some(Original(374)), where_: None, schema: None, table: None, column: None, datatype: None, constraint: None, file: Some("scan.l"), line: Some(1134), routine: Some("scanner_yyerror") }) }) })
Here is a screenshot of my model:

Versions:
"prisma2": "2.0.0-preview024"
"@prisma/client": "2.0.0-preview024"
"nexus": "^0.12.0-rc.9"
"nexus-prisma": "^0.11.1"

Hey @tylerjbainbridge,
Can you please share a minimal reproduction of this? The current information is not enough to triage this and identify the bug. It would be also great if you can paste the datamodel instead of a screenshot so that I can easily copy it.

Closing due to inactivity.


Error Failed to compile. Module not found

I get the following error when I try to run npm start.
    Failed to compile.

    Error in ./src/App.js
    Module not found: ./modules/world/components/country-list-fdt in /home/mana/node-apps/mana/mana-hapi2/src
    resolve file
      /home/mana/node-apps/mana/mana-hapi2/src/modules/world/components/country-list-fdt doesn't exist
      /home/mana/node-apps/mana/mana-hapi2/src/modules/world/components/country-list-fdt.js doesn't exist
    resolve directory
      /home/mana/node-apps/mana/mana-hapi2/src/modules/world/components/country-list-fdt doesn't exist (directory default file)
      /home/mana/node-apps/mana/mana-hapi2/src/modules/world/components/country-list-fdt/package.json doesn't exist (directory description file)
    [/home/mana/node-apps/mana/mana-hapi2/src/modules/world/components/country-list-fdt]
    [/home/mana/node-apps/mana/mana-hapi2/src/modules/world/components/country-list-fdt.js]
     @ ./src/App.js 19:22-76

Not sure why. I migrated this application and the file does exist. The application worked fine before.  I hit Control-B in WebStorm on the import in App.js and it goes right to the file. I import the file as follows:
import CountryList from './modules/world/components/country-list-fdt';

Where is the file located?
The error message says:
/home/mana/node-apps/mana/mana-hapi2/src/modules/world/components/country-list-fdt.js doesn't exist

Are you sure it exists there?

Figured it out. My files have a .jsx extension. I need to change the extensions of all my files that use .jsx to .js.

👍 Maybe we can warn about this in the future. We don’t see any benefits to .jsx extension so we’d rather use this chance to stop encouraging people to follow this convention. But this indeed could be surprising if you’re used to it.


MadClowns Support options

This may be a tad complex (as with all modular packs):
MadClowns Processing:

Adds materials and is essentially a smelting/petrochem extension

MadClowns Extended Materials:

clowns-ore# (1-9 for raw ores)  --these are essentially like angels ores
IF PURE ANGELS (special vanilla mode only get ore1,4,5,7)
clowns-resource# (special vanilla only gives 2, with both 1 and 2 active without) --these are essentially like oil patches but use regular miners

The processing of these unlock directly after angels ones in the tech tree...
resource1 unlocks in "water-washing-2", while resource2 processing unlocks with "oil-gas-extraction".
Clowns nuclear (both) go into mostly hybernation with angels industry active but don't generally impact ore generation stuff.

All Helmod plans done with basic (non burner) machines.

All Helmod plans done with basic (non burner) machines.


Add support for TypeScript

Explicitly ignore any type declaration files. While not explicitly a problem, this causes issues when running TypeScript via ts-node. See https://github.com/TypeStrong/ts-node/pull/207 for more details.

Thank you! 🙏

What about adding an index.d.ts file to the root directory of the repo so that it is possible to import * as requireDir from 'require-dir'?

@brandonros Haven't seen that done before, don't think that is necessary

It is. Try to import * as requireDir from 'require-dir' from any TypeScript project. It will throw an error about a lacking module declaration. You can either add it to the repo like https://github.com/axios/axios/blob/master/index.d.ts or create @types/require-dir (like https://www.npmjs.com/package/@types/express)

@brandonros Yep typings I understand - I would rather go that route. I don't use typescript, but if somebody else wants to do them it should be a pretty simple API to model.


Create TypeScript typings for angular-ui-tree

I'm working in TypeScript and currently have no type safety when writing code using angular-ui-tree. Create a typings file (.d.ts) for the control and submit it to the DefinitelyTyped repository.

This was added with borisyankov/DefinitelyTyped#5540 with borisyankov/DefinitelyTyped@cf1f872


Update project installation def to pyproject.toml

The roject should follow the same structure as cellarium-ml does


Issue with the GPTResponsder + quota of chatgpt

Hi, can you please resolve this issue:
When I run main.py via powershell [as administrator] then i get following error:
PS C:\WINDOWS\system32\ecoute> python main.py
Traceback (most recent call last):
File "C:\WINDOWS\system32\ecoute\main.py", line 3, in
from GPTResponder import GPTResponder
File "C:\WINDOWS\system32\ecoute\GPTResponder.py", line 2, in
from keys import OPENAI_API_KEY
ModuleNotFoundError: No module named 'keys'
I do have keys.py where i have entered the secret openai key:

When I run main.py directly by double clicking on it [from the location it is stored using windows explorer], after checking mic and speaker then ecoute app opens, when i speak then i get the following error message, i haven't exceeded my limit and using free version of chatgpt3.5. how this issue can be resolved? thanks
[INFO] Adjusting for ambient noise from Default Mic. Please make some noise from the Default Mic...
[INFO] Completed ambient noise adjustment for Default Mic.
[INFO] Adjusting for ambient noise from Default Speaker. Please make or play some noise from the Default Speaker...
[INFO] Completed ambient noise adjustment for Default Speaker.
[INFO] Whisper using GPU: False
READY
You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.
You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.

Same problem, I have no idea how to deal with it.

Problem is

Now Torch, Cuda, Python are not compatible with each other, you would need to reinstall everything with compatibility check
whl wheel also has unsupported versions
The CUDA version compatibility depends on the specific version of PyTorch you're installing.
--extra-index-url https://download.pytorch.org/whl/cu117

This line suggests that you're trying to install a PyTorch version compatible with CUDA 11.7. However, as of now, PyTorch officially supports up to CUDA 11.3.
If you're using CUDA 11.7, you might need to downgrade your CUDA version to 11.3 or wait until PyTorch releases a version compatible with CUDA 11.7.
Once you do it, do
pip uninstall torch torchvision torchaudio
pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cu117 // 117 or 113, still unsure
Check python version
python --version
Check the CUDA version:
nvcc --version (If CUDA Toolkit is installed)
OR
nvidia-smi (To check driver's CUDA capability):
Check Torch and Cuda compatibility
python
import torch
print(torch.version.cuda)
OR
you can also use
import torch
print(torch.__version__)
print(torch.cuda.is_available())  # Optional: Check for CUDA support
Error Output: If you get an ImportError: No module named 'torch', then PyTorch is not installed.
Output : True , everything is working fine
Output : False , Retry, you are in trouble

It still can't work : (


wellbeing: A couple more small changes for Friday's round of usability testing

Changes

Put "Review your answers" in an expandable
Moves the grouping select menu closer to the category radios for the combined histogram

Testing

Pull branch
Run server
Visit http://localhost:8000/consumer-tools/financial-well-being/results/?45
Observe the "review your answers" table now inside an expandable
Observe the select menu for the specific grouping of the histogram now floating 30px to the right of the radio buttons to select the category, rather than floating all the way to the right of the available space

Screenshots

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Assignee tool :arrow_right:

@keelerr I've only done the expandable for the combined results/histogram page. Let me know as early as you can if you also want the answer review for the other results page (the one with the three-part answer review and link to separate histogram page).

@scotchester, nope! Just the change to one page is great. Thanks.


Ability to spend unconfirmed utxos with sendcoins RPC

Background
Add ability to spend unconfirmed outputs when making onchain transactions.
When opening channels it is possible to spend unconfirmed outputs by passing spend_unconfirmed as part of the openChannel call.
Suggest adding the same spend_unconfirmed flag for sendcoins RPC.
Your environment

version of lnd: 0.11
which operating system (uname -a on *Nix) : mac
version of btcd, bitcoind, or other backend: bitcoind

Steps to reproduce
Deposit coins onto a fresh node, then try to make a couple of onchain payments. They can fail when change outputs are unconfirmed.
Expected behaviour
I should be able to spend the coins from my wallet.
Actual behaviour
I can't spend them because some are unconfirmed.

In this case, would output fanning or batching be a replacement to spending unconfirmed change? Or perhaps RBF of a base transaction to update the set of outputs?

I don't think that would work for our current use case - we need to be able to do instant one-time sends using unconfirmed outputs.


Help function on new Matlab version

Calling "help" for any function does not return any documentation. In the meantime, you can still access the documentation that would normally be returned from the help function by calling "edit".

Hello Sam,
Thank you for raising the issue.
I checked, so for some reason in the MATLAB 2022a version help is not getting displayed.
However, I checked on MATLAB 2020a, I could see the help for the functions.
I am not sure why it happens as of now, but I will get back to you with the reason and try to solve it. Meanwhile, you can look at the documentation online, https://phild001.github.io/biomechZoo-help/ .
Best regards,
Vaibhav


Docs isn't clear enough on how to run dsw

just run again the dsw command line on your project

I'm following the docs and then suddenly I get the instruction above.
I'm confused because at this step I never used the dsw command yet.
Also, I don't get yet why do we need the dsw globally :confused:

Just added some better description on how to use dsw.
It is installed globally, so you can run and re-run it easily from whenever you are.


passAuthInstallOnly=true builds docs still mentioning PKI auth

If you build skilltree docs with the build flag passAuthInstallOnly=true  PKI authentication is still mentioned in second bullet in the authentication documentation information
/skills-client/auth.html
The text is
"In PKI Mode: User's id is derived implicitly from the certificate, an authentication endpoint is not required"

PR: https://github.com/NationalSecurityAgency/skills-docs/pull/142


NE-1071: Default HAProxy maxconn value to 50000 for OCP 4.12

This change modifies the Go doc comment to reflect that in OpenShift 4.12 the default value for maxconn will now be 50000.
See: https://github.com/openshift/router/pull/417

/label px-approved

/label docs-approved

/label qe-approved

Seems like a straightforward change
/lgtm

/approve


rustc: Provide unpacked source tree derivation for use in tools

Racer doesn't function when RUST_SRC_PATH points to the source tarball.
I chose unpackedSrc instead of src to avoid any possibly breakage, but perhaps it's better to replace src?  It could also be made an additional output of the rustc derivation instead.
Any thoughts?
Motivation for this change
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I'm not really happy with the unpacking derivation.  It seems like something that would have existed elsewhere by now, but I couldn't find anything to share it with.

Fixed in master.


fix empty messages when sending custom payload to socketio

Proposed changes:

Fixes socketio channel to avoid sending empty messages when a bot utterance contains a custom payload

Fixes #7344
Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[X] reformat files using black (please check Readme for instructions)

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

Thanks for submitting a pull request 🚀 @kalkbrennerei will take a look at it as soon as possible ✨

@kalkbrennerei do you have time to take a look at this PR?

@santteegt are you up for wrapping up this PR?

Hi @m-vdb,
Yeah, however, I'm not sure if I should write unit tests for the changes I made as there are no existing unit tests for socketio channel. I was waiting for some direction here https://github.com/RasaHQ/rasa/pull/7373#discussion_r547562142

I don't know much about this part of the codebase, but I think it'd be great if we could have a test that covers the change your introducing, even if there's none today. If it requires a reasonable amount of work (that I can't tell for certain)


Week 3 Review Ticket

Focus and habits
boolean
deployment

2.7+/3
Review by: Kian Pasokhi

Did everything necessary for the week. Made a Boolean age ranger which determines if you are child or adult. Deployed java website successfully and took effective notes for focus and habits; has good goals in mind.  As for the + goes, he helped me deploy which makes him deserve any extra points possible. He went out his way to register the domain for me when my account wasn't working.


No metoffice

Questions user whether they have a metoffice account, and if not, tells them how to obtain one, and does not set up the metoffice credentials.
Related to issue #6

Looks good, thanks Holger


Card Action button  is not working with hero card in facebook messenger

Bot info

SDK Platform: .NET
SDK Version: 3.0.10.
Active Channels: WebChat,  Facebook
Deployment Environment: Azure App Service

Issue Description

I created a  HeroCard. It have a button (CardAction) to select the item. If the user clicks on the button (the button title should be "click me"), I expect the user to see the response of "click me" intent. After clicking on button in messenger it gives error like "Object Serialized to String. JObject instance expected".
In the emulator it works perfectly. but in messenger it is not getting value, it is getting title as text
and giving the above mentioned   error.

Code Example
HeroCard
[HeroCard
title = Lets Begin
subtitle = Microsoft Bot Framework
text = Build and connect intelligent bots to interact with your users naturally wherever they are, from text/sms to Skype, Slack,   Office 365 mail and other popular services.
image = https://greatsmilesbirmingham.com/wp-content/uploads/2017/09/canstockphoto30606317.jpg
buttons = @{cardActionTemplate( 'imBack',  'click me', 'click me intent' )}
]
cardActionTemplate( type,  title, value)
[CardAction
Types = @{if(type == null, 'imBack', type)}
Title = @{title}
Value = @{value}
Text = @{value}
]

@v-kydela can you you please assist?

@swaraj27 - It looks like you're using declarative dialogs. I imagine the dialog system must be performing some kind of transformation that doesn't account for the differences in card actions between channels. Can you tell me if there's any particular sample or tutorial you've followed so I can try to reproduce this?
Also, I would like to know what you mean when you say:

After clicking on button in messenger it gives error like "Object Serialized to String. JObject instance expected".

What do you mean by "it gives the error"? Where do you see the error? Is it thrown by the client or the connector service or the bot?

We have not followed any tutorial. We are referring the composer documentation. When we integrate our bot with Facebook Messenger, the response received on messenger after clicking the action button is "Object Serialized to String. JObject instance expected".

@swaraj27 - Sorry to ask again, but this information will be very important to solving the problem. Are you saying the error message is found in the activity's text and value properties? What debugging tool are you using to see these properties? Can you post a screenshot?

This is my code for  Action Button. I am  testing this in bot framework emulator it is working fine. But when I m integrating this with facebook messenger and after clicking on the button it is giving error like this.

@swaraj27 - Thank you for the screenshot because this explains what's actually going on. When a user clicks on a card, the channel client sends a message to the bot that says it's from the user, so it's a user-to-bot message. Since you said you're having problems with a card, I had thought you were saying the error message was in this user-to-bot message, but now I see the error is showing up as a bot-to-user message. This means problem is not with the card, but in the bot code that handles the message that the user sends to the bot by clicking the card. Since you're not following any tutorial or template, can you provide the bot code (or just show the Composer action) that executes on the turn after the card is sent?

Actually I can reproduce this using the "Responding with Cards" template. This is a bug in the Composer bot.
@cwhitten - Please have a look at this. The Composer bot is unable to respond to card actions correctly on all channels. You may be interested in the code I've written for the cards library to adapt incoming payloads.
This issue should probably be reported in the Composer repo: https://github.com/microsoft/BotFramework-Composer

@cwhitten I cannot transfer this issue to the composer repo. Can you please triage and transfer if appropriate?


can't use Enum in unions yet

Hi, thanks again for all your work on Crystal!
I'm hoping to write an object like this one, where a value in the attributes hash may be an Enum:
class Entity
  alias AttributeName = String
  alias AttributeValue = String | Enum # | Bool | ...
  alias Attributes = Hash(AttributeName, AttributeValue)

  def initialize
	@attributes = Attributes.new
  end

  def set(attr_name, attr_value)
    @attributes[attr_name] = attr_value
  end

  def get?(attr_name)
    @attributes[attr_name]?
  end
end

However, it fails with:
Error in line 3: can't use Enum in unions yet, use a more specific type

https://carc.in/#/r/10ng

Is there another way to accomplish this?
Is there another issue to track progress on this behavior? (If so, feel free to close this one)
How "fixable" is this? (If I understood the problem, could I contribute to the solution? Or is it a "big picture"-type issue?)

Umm... do you need it to be any enum, or one enum in particular?

In my dreams ... any enum :)

Closed in favor of #2733

Thanks, I'll keep an eye on it!


Spec documentation  for action includes and excludes is missing

See implementation: https://github.com/apache/openwhisk-wskdeploy/pull/991
The action spec: https://github.com/apache/openwhisk-wskdeploy/blob/master/specification/html/spec_actions.md#actions

@shazron Thanks for pointing this put.  Will see what I can do to add/expose this documentation once I return from holiday.  If you want to make an attempt, please feel free to submit a PR.


Update with new Event APIs

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

LGTM this is an easy one :P


chore: Add RCMP branding

Summary | Résumé

Adds RCMP branding option
Updated Logo dropdown to alpha sort the Logo options (default remains at the top)

Test instructions | Instructions pour tester la modification

Test new RCMP branding option

Pull Request Checklist
Please complete the following items in the checklist before you request a review:

[x] Have you completely tested the functionality of change introduced in this PR? Is the PR solving the problem it's meant to solve within the scope of the related issue?
[x] The PR does not introduce any new issues such as failed tests, console warnings or new bugs.
[ ] If this PR adds a package have you ensured its licensed correctly and does not add additional security issues?
[ ] Is the code clean, readable and maintainable? Is it easy to understand and comprehend.
[ ] Does your code have adequate comprehensible comments? Do new functions have docstrings?
[ ] Have you modified the change log and updated any relevant documentation?
[ ] Is there adequate test coverage? Both unit tests and end-to-end tests where applicable?
[ ] If your PR is touching any UI is it accessible? Have you tested it with a screen reader? Have you tested it with automated testing tools such as axe?

The alt text for the logo seems to only display in English but I may be missing something.

Was this for the preview / test screen?

The alt text for the logo seems to only display in English but I may be missing something.

Was this for the preview / test screen?

Yup, exactly that


Release URL scheme changed with v0.11.0

As commented by @ChineduUzoka over in #365, the release URL scheme changed with v0.11.0
Is this new scheme what will be used henceforth, or was it a mistake?

This was a mistake. I'll rectify it. Thanks!

fixed.


Setting up the Rage on Berserker

Looks like there's not a way that I can set up the amount of Rage that I stacked on Berserker.

The option won't currently show up if you only take War Bringer; only the other path enables it. That's fixed in the next update.


Add client auth

Add CLI flags to supply client-side authorisation files.
Example behavior:
vault-sidekick can't access vault without client authorisation
$ export VAULT_ADDR=https://${vault_addr}:${vault_port}
$ export VAULT_AUTH_METHOD=kubernetes
$ export VAULT_SIDEKICK_ROLE=example
$
$ bin/vault-sidekick

[error] unable to create the vault client: Post https://${vault_addr}:${vault_port}/v1/auth/kubernetes/login: remote error: tls: bad certificate

Supplying the key + cert, we can now talk to vault (hitting an expected error)
$ bin/vault-sidekick \
    -client-cert vault-client.crt \
    -client-key vault-client.key

[error] unable to create the vault client: Error making API request.

URL: POST https://${vault_addr}:${vault_port}/v1/auth/kubernetes/login
Code: 400. Errors:

* missing client token

Error if you try to supply only the key or cert
$ bin/vault-sidekick \
    -client-cert vault-client.crt

[error] invalid options, you are supplying the client certificate, but not the private key

The build is failing because you haven't formatted the go files. Run go fmt on the scripts


Refactor the permission system

Currently the zien permission system is based on the the original deployer address never being compromised.  That is not expected to remain true longer term.
We need to update the permissioning to system to allow the deployer to resign as the owner and set a new owner

Some ideas on a better system.

Multi sig for changes to the management addresses
Deployer address resigns all rights once the contract has been deployed.
Creator role to generate new NFTs. Limited to creating, can not hold NFTs
Treasurer account to hold the created NFTs, The account has not special access

Moved to Trello


[address-resolver] resolve several issues

This PR includes four individual commits associated with the address resolver module. It should resolve #4535. It's the easiest to review them one-by-one.
[address-resolver] accept multicast Address Error Notification
This commit enables processing of Address Error Notification sent as non-confirmable CoAP multicast message. The code change relaxes the verification of CoAP type.

[address-resolver] fix last transaction time of entry created by packet inspection
For cached entries created based on snooping message, the last transaction time was incorrectly set.

[address-resolver] dequeue a message from resolving queue before invalidate the entry
During code inspection, I found that in two places, a cache entry may be invalidated in the query state. That may result in a packet permanently being stuck in resolving queue in the Mesh Forwarder layer.
The fast test, to emulate erroneous behavior without this PR:
Topology: Leader <--> Router
Leader:
> reset

Router:
> ping Leader_ML-EID
> state leader   (emulate moving to another partition, command needs to be executed within 3 seconds)
Done
> bufferinfo
...
arp: 1 1 (stays forever)
...

[address-resolver] introduce MoveAt functionality
This commit introduces generic MoveCacheEntryAt functionality. This also helps to address @abtink request to put cache entries which are based on snooping messages, right after cached entries.

I run some manual and stress tests on this PR. However, I think we should have more system/unit tests for address resolver/address error functionalities. Unfortunately, I don't have the bandwidth to add them now.

Thank you for review. I've updated two commits to resolve comments.


NeoVim migration

You probably know but neovim is quite stable now and the plugin api has a java wrapper, although WIP. As far as I know this would eliminate the need of VimL and maybe make development easier.
What are your thoughts on this?

i think viml will not be eliminated ,neovim will not replace vim ,because it still is a text editor

Looks like it's not official wrapper. Anyway now we need VimL java-parser because javaparser java library still cannot parse unfinished classes, that's why for now we depend on VimL parser.

i think viml will not be eliminated ,neovim will not replace vim ,because it still is a text editor

I don't quite understand what you want to say. But I definitely think neovim is a replacement or successor. Maybe not in the sense that vim ceizes to exist, but in the sense that it is an improvement. Next version if you want.

Looks like it's not official wrapper. Anyway now we need VimL java-parser because javaparser java library still cannot parse unfinished classes, that's why for now we depend on VimL parser.

Well.. it is mentioned in the related projects of neovim.
But why do you actually need to parse unfinished classes?

@tepf because when completion is executed we should know, what class, variable, is related to. We should find variable declaration, classname import and etc...
And by the way, plugin should work in neovim, too. May be when we change parser engine later, we can move in that direction.

So you mean the java-parser can't build an AST when there are unfinished lines like LinkedLi? Anyway doesn't matter. I just wanted to see if you know about the java wrapper.
Yes it does work in neovim. I just thought development would be easier that way..

if the JC2's will change it's tools,i think JDT  is better choice,as the YCM's author says

Anyway, thank you for mention. I'll keep it on mind. Close for now.


couch_peruser creates databases only sporadically

Description
With couch_peruser=true  databases are created only sporadically. I have created 10 users and only 1 of them has received its own database. I don't know if there is a naming convention that must be followed?
Steps to Reproduce

setup local couchDB on Windows
set couch_peruser=true
generate some users with admin account

I used:
PUT http://<IP_ADDRESS>:5984/_users/org.couchdb.user:78946
with:
{
    "name": "78946",
    "type": "user",
    "roles": [],
    "password": "pass"
}

I used 10 different numbers as username.
The users were created perfectly and I have 10 users in the _users database.
Expected Behaviour
I expected 10 private databases - one per user... But I only got one database.
Your Environment

CouchDB version used: 3.1.1 and 3.0.0 (tried both)
Browser name and version: Chrome 86
Operating system and version: Windows 10

Additional Context

Was this a single node? I think couch_peruser needs all nodes to be up to operate properly, and will fail if any of the nodes are down (or are restarting). I think eventually the dbs are created though, once the cluster is stable.

Yes it was a single node. No cluster.

It worked for me
CouchDB Version: 3.1.1
POST: http://username:password@localhost:5984/_users
Header: content-type: application/json
body:
{

    "_id": "org.couchdb.user:john",

    "username": "john",

    "type": "user",

    "name": "john",

    "password": "john@123",

    "roles": [

        "user"

    ]

}


mv'ing files within Gdrive

Hello, should I be able to "mv" files between folders within Gdrive.
When I try I get:
mv: cannot move 'file.jpg' to '../snapArchive/2016-12/file.jpg': Input/output error
At the moment I am working round it by copying locally then back to Gdrive.

Yes, it was caused by a breaking change in Drive API v3. It should be fixed in version 0.6.6

Hi, many thanks for the fantastically quick response. I was almost going to report that it had not worked after installing from the beta repository but then realised I hadn't remounted :-)
MUCH quicker than the 0.6.4 version with cp/rm script that I had been using (3x + faster)
Many thanks again.


Phone Number Validation (without two-factor authentication)

I am trying to validate that the user entered a valid email address and phone number. I don't want to use them for two-factor authentication, just to confirm that we have both valid email and mobile.
The email part is working fine. My issue is with the phone part.
This is what I am doing:

The user register and I insert the data as follow:

var user = new User {
    FirstName = model.FirstName,
    LastName = model.LastName,
    UserName = model.Email,
    Email = model.Email,
    PhoneNumber = model.Mobile
};
var result = await _userManager.CreateAsync(user, model.Password);

Email verification happen.

When the user login, I check if the user.PhoneNumberConfirmed is true (all fine continue) if false (the user didn't verify the phone number) i have this code:

if (!user.PhoneNumberConfirmed)
{
    var code = await _userManager.GenerateChangePhoneNumberTokenAsync(user, user.PhoneNumber);
    await _smsSender.SendSmsAsync(user.PhoneNumber, "Your security code is: " + code);
    return RedirectToAction(nameof(VerifyPhoneNumber), new { PhoneNumber = user.PhoneNumber });
}

I do get an SMS to the registered phone number, but the security code seems as if it is for the email verification not the phone. Sample of the sms:
Your security code is: CfDJ8KHDKnJMUoZDqp45kWWHlYUmeKZC/NdiL6Qssar+Yj00N+CK09pftakRBcU+KXDYrXLbngH5v+5FHrHMUvHiYZke5WeJw2Pefp2tr6RlmwVX3fJ124av9WkPiY0CfK9NMH9njWcEfe7oMSF.....
Any idea how to fix this?
I am using .NET 2.0.0-preview2-006497

Got the same issue with 2.0.0 final. I migrated to the new way of 2FA but I still need SMS for urgent user communication and phone verification is required for obvious reasons.
I suggest to either change this back to 6 digits code which are more reasonable in this case (imho) or provide a way to configure the behavior.
Thanks!

Dupe of https://github.com/aspnet/Identity/issues/1388


Some questions about evaluation.

Hi author, I congrats and appreciate your work! But I have some questions about the evaluation of this model.
I noticed that the data format is:
{'target_l': [54, 31], 'target_th': [3, 10], 'loc': [[27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 29, 31, 42, 43, 44, 38, 45, 46, 47, 48, 49, 50, 51, 52], [53, 54]] ......

where cur_loc (loc[1]) has overlapped the target_l in location 54, when I read your codes I found that target_l 54 is used for calculating the recall. Could you please tell me if I am right about this? Is it a bug or did I miss something? I hope to get your reply, thanks!

Hi, I add these lines in utils.py:
for batch_idx in batch_idx_list:
    uid, sid = data_queue[batch_idx]
    uid_batch.append([uid])

    # add here:
    data_input[uid][sid]['loc'][0].extend(data_input[uid][sid]['loc'][1][:])
    data_input[uid][sid]['tim'][0].extend(data_input[uid][sid]['tim'][1][:])
    data_input[uid][sid]['cat'][0].extend(data_input[uid][sid]['cat'][1][:])
    data_input[uid][sid]['loc'] = [data_input[uid][sid]['loc'][0]]
    data_input[uid][sid]['tim'] = [data_input[uid][sid]['tim'][0]]
    data_input[uid][sid]['cat'] = [data_input[uid][sid]['cat'][0]]
    data_input[uid][sid]['target_l'] = [data_input[uid][sid]['target_l'][-1]]
    data_input[uid][sid]['target_c'] = [data_input[uid][sid]['target_c'][-1]]
    data_input[uid][sid]['target_th'] = [data_input[uid][sid]['target_th'][-1]]
...

which should process the data
{'target_l': [54, 31], 'target_th': [3, 10], 'loc': [[27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 29, 31, 42, 43, 44, 38, 45, 46, 47, 48, 49, 50, 51, 52], [53, 54]] ......}
to something like:
{'target_l': [31], 'target_th': [10], 'loc': [[27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 29, 31, 42, 43, 44, 38, 45, 46, 47, 48, 49, 50, 51, 52, 53], [54]] ......}
which should address the label leakage problem by only calculating the last target location's recall and the last category's recall. (Same as the problem definition)
Then I try to rerun the experiments and got the following results:

NYC

NYC

TKY

TKY

category

location

category

location

R@1
R@5
R@10
R@1
R@5
R@10
R@1
R@5
R@10
R@1
R@5
R@10

0.293
0.502
0.575
0.231
0.391
0.428
0.424
0.653
0.720
0.204
0.350
0.403

The results have dropped a lot compared to the results reported in the paper. If I have misunderstood your operation, I hope you can help me understand.
If the bug I mentioned above does exist, do you think it is a reasonable way to modify your code as the code block displayed above? I look forward to your reply!

Hi,
Thanks for reaching out!
This is not a bug. Actually, you have some misunderstandings about our settings.
Assuming data = {'target_l': [54, 31], 'target_th': [3, 10], 'loc': [[27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 29, 31, 42, 43, 44, 38, 45, 46, 47, 48, 49, 50, 51, 52], [53, 54]] ......},

data['loc'][0] is the historical session and data['loc'][1] is the current session. We divide the trajectory into sessions with "week" strategy. This means you cannot add 53 to data['loc][0]. Moreover, in the data samples that we don't see, there are also cases where the lengths of their data['loc'][1] are larger than 1 or 2. Your processing operations reduce a large number of test samples.
The actual process is: when we want to predict 54, the model only knows the historical session and 53; when we predict 31, the model knows the historical session and [53, 54]. This form of data organization is very common in our baselines, such as DeepMove and LSTPM.
Thanks again.

Hi author, thanks for your timely reply!
I understand the meaning of a session and thanks for the explanation. May I know which part of your codes masked out location 54 in data['loc'][1] when the model predicted location 54 in data['target_l']? I understand three masks (target_mask, his_mask, cur_mask) are used to mask those paddings.
I see the progress that generate_batch_data(.) in utils.py input all locations in current sessions (e.g., [53, 54]) into the model for the forward propagation to generate outputs (th_pred, c_pred, l_pred) that match the target data format.
So I still don't understand which part of your codes ensures the second claim (The actual process) of your response.

Hi,
Maybe this picture can help you understand. I think this is an advantage of the RNN series, compared to the Transformer (attention series) which requires an explicitly defined mask.

Thanks for your time author, I think I see what you mean here...
But I think that when passing hidden states like hc_t and hc_l to capturer_c and capturer_l will still cause the label leakage problem since they are the final hidden states of the previous GRU. I'm confused... Am I right about this?

These codes in model.py:
cur_t_rnn, hc_t = self.capturer_t(rnn_input_his_concat, rnn_input_cur_concat, his_mask, cur_mask, mask_batch[1:])
        if self.cat_contained:
            cur_c_rnn, hc_l = self.capturer_c(rnn_input_his_concat, rnn_input_cur_concat, his_mask, cur_mask, mask_batch[1:], hc_t)
            cur_l_rnn, _ = self.capturer_l(rnn_input_his_concat, rnn_input_cur_concat, his_mask, cur_mask, mask_batch[1:], hc_l)

indicates that cur_c_rnn and cur_l_rnn are containing ground-truth features introduced by hc_t. If I didn't misunderstand the reason you explain RNN structure to me, I can tell cur_t_rnn is unbiased and without the ground-truth leakage problem. However, the main results reported in the paper are recall of category and location, I believe the final hidden states (which correspond to hc_t and hc_l) actually have learned ground-truth information which means that the predictions cur_c_rnn and cur_l_rnn are biased.
I'm just trying to understand your work. I just hope I won't be misled as well as make mistakes in my own work in the future, again I think the contribution of your paper is not so much about getting much prediction accuracy but using human logic to help make next-hop predictions. So I hope to get your proper help to understand this work better together, and I hope you can continue to help me and not misunderstand my intention to open this issue, thanks again!

I see what you mean. We will check the code and update this repository as soon as possible. Thank you for your issue.

We have revised the code and the new experiment is underway. It also requires new parameter searching.
Let's expect promising results.
We will update the new results as soon as possible.
Thanks again.
PS: the newest results for location prediction are<EMAIL_ADDRESS>on NYC at Epoch 9 and<EMAIL_ADDRESS>on TKY at Epoch 7, when I write this comment. These results are still competitive with the baselines.  (These outperform all of the baselines except LSTPM, whose settings require filtering more sparse users.)

Hi author,
It seems that your newest results to some extent consistent with my rerun experiments' results above, while your newest results compute recall for more samples. I think numerical results are not that important actually, your causality contribution and this small bug leave some room for future work.
Thank you for your timely revisions and all your replies.


Exports with mono and C# scripts can corrupt the build results

Tested versions

Reproducible in: 4.4.dev [3cad849b1] and later
Not reproducible in: 4.4.dev2 [97ef3c837]

By git bisect I found the first bad commit is [0dde931bc95e].
System information
Godot v4.4.dev.mono (3cad849b1) - Windows 10.0.22631 - Vulkan (Forward+)
Issue description
The exported project with mono throws many System.TypeInitializationException, and there are many native method bindings that can't be found, such as Object.tr or Control.get_theme_icon.
[0dde931bc95e] deleted some .inc files and changed the hashes of some functions, and those old hashes need to be fetched in a new way. The exported project was using the old hashes, resulting in not getting the target function pointers correctly.
One of the errors from exported projects:
ERROR: System.TypeInitializationException: A type initializer threw an exception. To determine which type, inspect the InnerException's StackTrace property.
 ---> Godot.GodotObject+NativeMethodBindNotFoundException: Unable to find the native method bind. (Method 'Control.get_theme_icon')
   at Godot.GodotObject.ClassDB_get_method_with_compatibility(StringName type, StringName method, UInt64 hash) + 0x100
   at Godot.Control..cctor() + 0x3240
   at System.Runtime.CompilerServices.ClassConstructorRunner.EnsureClassConstructorRun(StaticClassConstructionContext*) + 0x160
   --- End of inner exception stack trace ---
   at System.Runtime.CompilerServices.ClassConstructorRunner.EnsureClassConstructorRun(StaticClassConstructionContext*) + 0x247
   at System.Runtime.CompilerServices.ClassConstructorRunner.CheckStaticClassConstructionReturnGCStaticBase(StaticClassConstructionContext*, Object) + 0x1c
   at Godot.Control.InvokeGodotClassMethod(??? method, NativeVariantPtrArgs args, godot_variant& ret) + 0x64
   at Godot.Label.InvokeGodotClassMethod(??? method, NativeVariantPtrArgs args, godot_variant& ret) + 0x55
   at ExportBrokenExample.CustomLabel.InvokeGodotClassMethod(??? method, NativeVariantPtrArgs args, godot_variant& ret) + 0xf9
   at export_broken_example!<BaseAddress>+0x79fd9a
   at: void Godot.NativeInterop.ExceptionUtils.LogException(System.Exception) (:0)

Steps to reproduce

Create a simple Label node with some scripts in C#.
Export project.
Exception will occurred when running.

Sample Label Scene:
[gd_scene load_steps=2 format=3 uid="uid://dhkseg6v08mew"]

[ext_resource type="Script" path="res://CustomLabel.cs" id="1_ig7tw"]

[node name="Main" type="CanvasLayer"]

[node name="Label" type="Label" parent="."]
anchors_preset = 15
anchor_right = 1.0
anchor_bottom = 1.0
grow_horizontal = 2
grow_vertical = 2
theme_override_font_sizes/font_size = 64
text = "Native invoke failed."
horizontal_alignment = 1
vertical_alignment = 1
script = ExtResource("1_ig7tw")

Sample Script:
using Godot;

namespace ExportBrokenExample;

public partial class CustomLabel : Label
{
    public override void _Ready()
    {
        base._Ready();

        Text = Tr("Native invoke successful.");
    }
}

Minimal reproduction project (MRP)
export_broken_example.zip

I tried building the export templates and GodotSharp assemblies from commit 3cad849b1f47747f69081086faa2db190706a829 and I'm unable to reproduce the issue.
It looks like you are using a custom build. I suspect the GodotSharp assemblies you are using were compiled using a different version of the Godot source code than the export templates, which is not supported.
Changing the hashes shouldn't break compat for C# as long as you use the GodotSharp assemblies that match your export templates.

I complied the editor and export template in the same commit of the code with the commands below:
bash -c "openssl rand -hex 32 > bin/godot.gdkey"
$env:SCRIPT_AES256_ENCRYPTION_KEY = (cat ./bin/godot.gdkey)
$env:BUILD_NAME = "dctewi"

$buildFlags = "production=yes", "debug_symbols=yes", "module_mono_enabled=yes", "p=windows", "arch=x86_64"

& scons $buildFlags target=editor
& scons $buildFlags target=template_debug
& scons $buildFlags target=template_release

./bin/godot.windows.editor.x86_64.mono.exe --headless --generate-mono-glue modules/mono/glue
python ./modules/mono/build_scripts/build_assemblies.py --godot-output-dir=./bin --godot-platform=windows

./bin/godot.windows.editor.x86_64.mono.exe --doctool

I don't know why the issue can be reproduced in my machine. I'll try to reproduce it with another PC without Godot installed or built.

When executing the build_assemblies.py script, you are not using the --push-nupkgs-local option. Make sure the GodotSharp packages are pushed to a local NuGet source; otherwise, your project may be using different packages than the ones you just built.

I installed the nuget packages later in bin/GodotSharp/Tools/nupkgs manually and I changed the status variable to custom in version.py so that my custom build nuget package had a version like 4.4-custom to avoid using a wrong version package. I'm not sure what went wrong.

You can list the registered NuGet sources with dotnet nuget list source. It should contain the local NuGet source where you are placing the NuGet packages, and it should be [Enabled]. Check if any of the other sources listed there also contains packages with the same version (4.4-custom), if so it may be getting the packages from the wrong source.
Also, the --push-nupkgs-local option takes care of clearing the cache for you. Since you are not using it, make sure to remove the GodotSharp packages from the NuGet cache; otherwise, it may be getting the cached packages instead of the new ones. You can list the cache locations with dotnet nuget locals all --list (look for global-packages).

After git clean -dfX, the hashes in the C# code files generated in modules\mono\glue\GodotSharp\GodotSharp\Generated\Objects successfully changed to the new version of the hashes.
I thought perhaps some old compilation cache was affecting the code generation process. However, rebuilding Godot with a completely new repo does avoid the problem. It's time to close this issue.


feat: pass tool call id to tool runs

Passes tool_call_id to the tools run during agent's execution. Without this, due to this check, one can't access to the artifacts (if provided) from the result of the tool run.

@ha3 we recommend that new code that involves agents be written using langgraph.
We'll fix bugs in the given agent executor if they arise, but generally reluctant to add new functionality, as the langchain agent executor is not the best way to write agent code.
If you'd like to get this PR merged, could you please:

explain why we need to enhance the agent executor instead of using a langgraph agent?
please add a unit test that covers the relevant scenario


Copy & Paste not working on Chrome 75 for OSX

Copy paste is not working for me on OSX Chrome.
Chrome Version 79.0.3945.117 (Official Build) (64-bit)
OSX 10.14.6 Mojave
Works fine on same system with FireFox

Is that still an issue?

Let's close this. If you still experience any issues, comment here or open a new issue please.


Update ios-device-enrollment-program-for-hybrid.md

Changed "Windows Intune Subscriptions" to "Microsoft Intune Subscriptions" in two places.

@tycastMSFT Nice catch and nicely done PR. Thanks.


Improve error messaging for modeled errors that provide their own message

Describe the feature
Many AWS service errors have a modeled message property, that the service can use to provide detailed information about the error.  Because modeled error properties are namespaced under properties, this error message is accessible as error.properties.message on an error named error.
However, many modeled error types don't provide a message; presumably the type of the error is sufficient to inform the caller of the nature of the error in this case.
Every service error also has a message property that they inherit through the ServiceError protocol.  This message property is intended to provide a human-readable error message, even for errors that don't explicitly provide one as a modeled field.  This property is exposed on an error named error as error.message.
So, for service errors that provide their own modeled message property, they actually have two message properties: error.message and error.properties.message.  It's not clear to the customer why there are two properties, nor what one might contain versus the other.  It should be presumed, though, that when the service provides a modeled error message, that it should contain the most useful and relevant message.
Use Case
It should be clear to customers how to get the most useful error message for any error that is returned by an AWS service.
Proposed Solution
A message accessor should be provided that selects the best option between a modeled error message and a ServiceError protocol-provided message, and returns the most appropriate of the two.
This will likely have to be done using code-generation, based on whether the error models its own message or not.
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Moved to 2157


fix: adjust release message

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


[Stress] Write logs periodically while tests are running

Change the Stress framework to write to log files periodically while the test is running (default write once per minute, configurable on command line).

Given the age of this issue and the shift to the new stress framework, I'm going to close this out as stale.   Please feel free to reopen if further discussion is needed.


Add blog post about pull from usptream

Fixes #496
I think there are too many screenshots, but I can't decide what to remove :D (maybe the issue in issue_repository doesn't need to be mentioned)... waiting for some feedback on that
Preview:

@majamassarini @jpopelka @FrNecas @lachmanfrantisek thanks for all the suggestions!!


Support to Null Safety

Dart migration guide: https://dart.dev/null-safety/migration-guide

Thanks, migrated to Null Safety in v1.0.0.


Feature/sg 34 regseg transfer learning

Added example code for Regseg transfer learning, which will later be used for an example notebook.
This PR holds several necessary features for it to run:

BCE with Dice loss.
Binary IOU metric object (I.e IOU only for target class).
Binary segmentation visualisation callback.
Supervisely dataset interface.
Different lr assignment for head and backbone for RegSeg.

Nice work.
We have some required fixes inline...


chore(ci): exclude CodeQL hardcoded-credentials check in tests

Changes proposed in this pull request

Context

fixes #750

Checklist

[x] Related issues linked using fixes #number
[ ] Tests added/updated
[ ] Documentation added
[ ] Make sure that all checks pass

Tested here:
https://github.com/wilsonianb/rafiki/pull/171
https://github.com/wilsonianb/rafiki/pull/172


BW-176 Add Apache Ignite as Spring jCache/Hibernate provider

@ottenhoff & @ern ?

merged: @jonespm & @buckett ?


fix: better error messages

[X] I have added automatic tests where applicable
[X] The PR title is suitable as a release note
[X] The PR contains a description of what has been changed
[X] The description contains manual test instructions

Changes
I was trying to use the awesome and brand new Schema Generator (https://github.com/ftrackhq/ftrack-ts-schema-generator).
However, I got the following error:
Failed to perform request.  CustomError [ServerError]: fetch failed
    at Session.getErrorFromResponse (/Users/Documents/src/ftrack-javascript/source/session.ts:336:19)
    at Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:561:22)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:534:5)
    at async Session.query (/Users/Documents/src/ftrack-javascript/source/session.ts:738:23) {
  errorCode: undefined
}
Server reported error in unexpected format.  CustomError [ServerError]: fetch failed
    at Session.getErrorFromResponse (/Users/Documents/src/ftrack-javascript/source/session.ts:336:19)
    at Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:561:22)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:534:5)
    at async Session.query (/Users/Documents/src/ftrack-javascript/source/session.ts:738:23) {
  errorCode: undefined
}
/Users/Documents/src/ftrack-javascript/source/session.ts:336
    const error = new ErrorClass(response.content, response.error_code);
                  ^
CustomError [ServerError]: fetch failed
    at Session.getErrorFromResponse (/Users/Documents/src/ftrack-javascript/source/session.ts:336:19)
    at Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:588:20)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:534:5)
    at async Session.query (/Users/Documents/src/ftrack-javascript/source/session.ts:738:23) {
  errorCode: undefined
}

This wasn't very useful, so I made this pull request. After that pull request, the error was instead way more helpful:
Failed to perform request.  CustomError [ServerError]: Error: unable to verify the first certificate
    at Session.getErrorFromResponse (/Users/Documents/src/ftrack-javascript/source/session.ts:336:19)
    at Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:561:22)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:534:5)
    at async Session.query (/Users/Documents/src/ftrack-javascript/source/session.ts:738:23) {
  errorCode: undefined
}
Server reported error in unexpected format.  CustomError [ServerError]: Error: unable to verify the first certificate
    at Session.getErrorFromResponse (/Users/Documents/src/ftrack-javascript/source/session.ts:336:19)
    at Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:561:22)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:534:5)
    at async Session.query (/Users/Documents/src/ftrack-javascript/source/session.ts:738:23) {
  errorCode: undefined
}
/Users/Documents/src/ftrack-javascript/source/session.ts:336
    const error = new ErrorClass(response.content, response.error_code);
                  ^
CustomError [ServerError]: Error: unable to verify the first certificate
    at Session.getErrorFromResponse (/Users/Documents/src/ftrack-javascript/source/session.ts:336:19)
    at Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:588:20)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Session.call (/Users/Documents/src/ftrack-javascript/source/session.ts:534:5)
    at async Session.query (/Users/Documents/src/ftrack-javascript/source/session.ts:738:23) {
  errorCode: undefined
}

This allowed me to truly figure out that it was a company-related VPN thing that caused the issue, and that I needed some certificate changes.

Test
I tried the following test - it provides a much better error message now:
import { Session } from '@ftrack/api';

const client = new Session();
client.query("select id from Project").then(console.log);

Yes - I'll fix that Monday. But feel free to add it already! 😄

@gismya actually, I remember now why I didn't do it. I tried it at first, but the cause also seems to include the message and/or the stack trace?
I do at least remember that the error looked duplicated when I included both.
Not sure if that's still what we want. But up to you! 🙏

I just hadn't approved the new release - should be out now.


Fix: NoMethodError (undefined method `confirmations=' for #<RPC::IncomingTransfer...

Hi,
I have an update. There was an error caused by 0.13, so the new Monero version.
The new monero version now includes "confirmations" and "suggested_confirmations_threshold". Both have been added to the module class "IncomingTransfer".
Methods "confirmed?" and "pending?" checked are still working, as they should.

Hello @langesi , thanks for your contribution. I had no time at all to have a look at 0.13.
if confirmations is coming from the TX itself, then we don't need it and can remove it. nice!

@langesi  you should be able to accept merge requests yourself


Allow change of user agent and use GET request

I don't know why it is saying 204 additions and 195 deletions. I can re do this PR if u want.
#3

i think its because hard tabs VS. soft tabs, no problem, i'll merge it.
I am curious though why you need a GET instead of a HEAD request?
A HEAD is much like a GET only without the response body, so it requires less resources.
On the other hand with the GET in there we could eventually implement a checksum system.

I would like to implement a screen shot but checksums sound way easier. GET request is for a "typical users" request, some apps may offer something different for HEAD (I may be wrong about that)
Really it could be easily be a config option, or a gui dropdown. That would be sweet!