Possible XSS issues

Hi there, our site gets occasionally audited for security purposes. We currently are being flagged with the following errors.
window.location.search and passed to window.location.replace() via the following statement:
window.location.replace(window.location.pathname+window.location.search+"#"+e)

Is it possible that you could comment on  how secure history is especially in regards to XSS issues in the README?

Does your audit say why passing window.location.search to window.location.replace is a bad idea? In that code, all we're trying to do is preserve the existing query string when navigating around using hash history. Seems harmless. Also, you're not even running that code unless you're using hash history.

We've had this code flagged before by other security scanning tools. For instance this is what we get when we run BURP scans on it.
DOMbased vulnerabilities arise when a clientside script reads data from a controllable part of the DOM (for example, the URL) and processes this data in an unsafe way.
DOMbased open redirection arises when a script writes controllable data into the target of a redirection in an unsafe way. An attacker may be able to use the vulnerability to construct a URL that, if visited by another application user, will cause a redirection to an arbitrary external domain. This behavior can be leveraged to facilitate phishing attacks against users of the application. The ability to use an authentic application URL, targeting the correct domain and with a valid SSL certificate (if SSL is used), lends credibility to the phishing attack because many users, even if they verify these features, will not notice the subsequent redirection to a different domain.
Note: If an attacker is able to control the start of the string that is passed to the redirection API, then it may be possible to escalate this vulnerability into a JavaScript injection attack, by using a URL with the javascript: pseudoprotocol to execute arbitrary script code when the URL is processed by the browser.
Burp Suite automatically identifies this issue using static code analysis, which may lead to false positives that are not actually exploitable. The relevant code and execution paths should be reviewed to determine whether this vulnerability is indeed present, or whether mitigations are in place that would prevent exploitation.

Issue remediation
The most effective way to avoid DOMbased open redirection vulnerabilities is not to dynamically set redirection targets using data that originated from any untrusted source. If the desired functionality of the application means that this behavior is unavoidable, then defenses must be implemented within the clientside code to prevent malicious data from introducing an arbitrary URL as a redirection target. In general, this is best achieved by using a whitelist of URLs that are permitted redirection targets, and strictly validating the target against this list before performing the redirection.

I think the fear is, the attacker could craft a URL (in a public forum or whatever) that may cause a redirect when a user wasn't expecting one. It's not saying that history is actually doing this, it's just saying that it could happen . The recommended fix is to verify that the URL matches known patterns or white listed redirect locations. Our app gets this code via a react-router dependency so it very well be protected, but I need documented proof that this kind of attack is not possible.
I'm surprised that this hasn't come up before being how widely used this code is.
PS. whether the code's used or not these security scanners just pick this code up via static analysis and pattern matching.

I think this part is key:

Burp Suite automatically identifies this issue using static code analysis, which may lead to false positives that are not actually exploitable. The relevant code and execution paths should be reviewed to determine whether this vulnerability is indeed present, or whether mitigations are in place that would prevent exploitation.

Our usage of hash history (you're probably using history version 2 or 3) boils down to this:
window.location.replace(window.location.pathname + window.location.search + '#' + hash)

In this code we're not just passing window.location.search directly to window.location.replace. We're actually putting the existing pathname first. So if someone did try to craft a malicious query string, like
http://example.com/the/path?javascript:alert("hi")

we would actually be doing
window.location.replace('/the/path?javascript:alert("hi")')

which is safe. I think what the suite is trying to avoid is doing stuff like
window.location.replace('javascript:alert("hi")')

which is obviously not safe.
So in this case I'd say that "mitigations are in place that would prevent exploitation". Would you agree?

@srobertson Can you please comment? Otherwise, let's close. I don't think we have a security problem here.

@mjackson I think that's a reasonable explanation regarding  window.location.pathname.


Client enables all the exit nodes and network routes after restart

I'm having this issue on several devices: Windows, MacOS, Linux. After a restart, the client enables all of the available exit nodes and every single available network route. I can't imagine any case where I would want this by default. The client should remember whatever was toggled and those settings should remain the same, even after a restart, until changed.

This is the current behavior, we will introduce persistent selection in the coming releases.

@mlsmaycon do we have an ETA?
For me it's especially an issue if at home and the home route or exit node is still selected which causes network issues...

This hasn’t been fixed yet? I had this issue in July and ended up switching to another software as this broken feature was causing too many issues.


Split application configuration into separate modules and add tests

Move command line parsing into config/argparse
Move context initialization into config/default_context (add ContextBuilder class)
Move logging into config/logging
Add related unit tests, plus tests for conductor and dispatcher

I get an error running the agent with the updates from this PR:
tob-agent_1    | 2019-06-30 14:38:38,033 aries_cloudagent.conductor ERROR Unable to start administration API
tob-agent_1    | Traceback (most recent call last):
tob-agent_1    |   File "/home/indy/aries_cloudagent/admin/server.py", line 169, in make_application
tob-agent_1    |     f"{protocol_module_path}.routes"
tob-agent_1    |   File "/home/indy/aries_cloudagent/classloader.py", line 102, in load_module
tob-agent_1    |     mod = import_module(mod_path)
tob-agent_1    |   File "/home/indy/.pyenv/versions/3.6.8/lib/python3.6/importlib/init.py", line 126, in import_module
tob-agent_1    |     return _bootstrap._gcd_import(name[level:], package, level)
tob-agent_1    |   File "", line 994, in _gcd_import
tob-agent_1    |   File "", line 971, in _find_and_load
tob-agent_1    |   File "", line 955, in _find_and_load_unlocked
tob-agent_1    |   File "", line 665, in _load_unlocked
tob-agent_1    |   File "", line 678, in exec_module
tob-agent_1    |   File "", line 219, in _call_with_frames_removed
tob-agent_1    |   File "/home/indy/src/subdir/credential-registry/message_families/issuer_registration/indy_catalyst_issuer_registration/routes.py", line 13, in
tob-agent_1    |     from .manager import IssuerRegistrationManager
tob-agent_1    |   File "/home/indy/src/subdir/credential-registry/message_families/issuer_registration/indy_catalyst_issuer_registration/manager.py", line 8, in
tob-agent_1    |     from aries_cloudagent.messaging.util import send_webhook
tob-agent_1    | ImportError: cannot import name 'send_webhook'
tob-agent_1    |
tob-agent_1    | During handling of the above exception, another exception occurred:
tob-agent_1    |
tob-agent_1    | Traceback (most recent call last):
tob-agent_1    |   File "/home/indy/aries_cloudagent/conductor.py", line 184, in start
tob-agent_1    |     await self.admin_server.start()
tob-agent_1    |   File "/home/indy/aries_cloudagent/admin/server.py", line 211, in start
tob-agent_1    |     self.app = await self.make_application()
tob-agent_1    |   File "/home/indy/aries_cloudagent/admin/server.py", line 173, in make_application
tob-agent_1    |     self.logger.error(
tob-agent_1    | AttributeError: 'AdminServer' object has no attribute 'logger'


Fix units and colorscales for CMIP6 data

Currently we're just using FOREST defaults and the units as they come in from the catalogue. We may want to review what units and scales we use. For example

Kelvin or celsius for temp
mm.h^-1 for precip. This is also average monthly precip currently, so we may need to incorporate this information somewhere.
wind - km/h, knots m.s^-1 ?

and many others as well

@stevehadd does this kind of thing relate to @CarolineBain's issue #168?


Gradle plugin: generate logback config even if there's no gatling resource directory

Current issue

Logback generation happens at doLast closure of processGatlingResources task. When there's no gatling resource, i.e. default folder src/gatling/resource missing - the logback.xml doesn't get generated.

Scope of changes

LogbackConfigTaskAction should be called as doLast closure of another task.
proposal is to call it after successful compilation of Gatling simulation
task name is gatlingClasses

Closing in favor of #3952

Just for the record, this seems to be the same issue as:

https://github.com/lkishalmi/gradle-gatling-plugin/issues/110
https://github.com/lkishalmi/gradle-gatling-plugin/issues/111
https://github.com/lkishalmi/gradle-gatling-plugin/issues/115

@slandelle is smth wrong with the implementation ? Or just updating the issue parameters ?

The latter, everything's fine.

Ok, sorry for bothering.

No pro


Cache installed solvers check in tests

@rostyboost, thanks for the PR. @rileyjmurray is currently refactoring the conic solver tests. I've asked him to make a change analogous to yours in PR #910.


Implement TODO tasks in inventory

Context (Problem, Motivation, Solution)

Prevent card deck from being modified in the inventory during a game
Limit maximum number of card copies in a deck to three

Link related issues!
https://github.com/norswap/0xFable/blob/master/docs/architecture.md#cards-and-inventory
Describe Your Changes

Added a modifier notInGame() to prevent players from modifying card deck while participating in a game
Implement checkDeck() to check if a player possesses more than three copies of  a card in a deck

Checklist

[x] I have performed a self-review of my code
[x] I ran make lint and fixed resulting issues
[x] I ran the relevant tests and they all pass
[x] I wrote tests for my new features, or added regression tests for the bug I fixed

Testing
basic tests here

🚀🚀🚀


HERE’S!!* WAY TO WATCH Ipswich Town vs Chelsea LiVe 𝐒𝐭𝐫𝐞𝐚𝐦𝐬 ON Premier League TV CHANNEL

About the match
Ipswich Town is going head to head with Chelsea starting on 30 Dec 2024 at 19:45 UTC at Portman Road Stadium stadium, Ipswich city, England. The match is a part of the Premier League.
🔴WATCH📺📱👉 Ipswich Town vs Chelsea Live
🔴WATCH📺📱👉 Premier League 2024-25 Live
🔴WATCH📺📱👉 Ipswich Town vs Chelsea Live

Currently, Ipswich Town rank 19th, while Chelsea hold 3rd position. Looking to compare the best-rated player on both teams? Sofascore's rating system assigns each player a specific rating based on numerous data factors.
On Sofascore livescore you can find all previous Ipswich Town vs Chelsea results sorted by their H2H matches. Sofascore also provides the best way to follow the live score of this game with various sports features. Therefore, you can:
Find out who scored in a live match
Get real-time information on which team is dominating the match using the Attack Momentum
Follow detailed statistics such as ball possession, shots, corner kicks, big chances created, cards, key passes, duels and more
Track all home and away games for each team in the Premier League
Check out how Sofascore community votes on which team is more likely to win this match.
All of these features can help you decide on Ipswich Town vs. Chelsea game prediction. Even though Sofascore doesn't offer direct betting, it provides the best odds and shows you which sites offer live betting. Live U-TV odds are viewable on Sofascore's Football live score section.
Where to watch Ipswich Town vs. Chelsea? Under TV Channels section you can find the list of all channels that broadcast Ipswich Town – Chelsea live match. You can also live stream this match through our betting partners or click on any links on Sofascore for a legal live stream.
Event details:
NAME: Ipswich Town - Chelsea
DATE: 30 Dec 2024
TIME: 19:45 UTC
VENUE: Portman Road Stadium, Ipswich, England
More details:
Ipswich Town live score, schedule and results
Chelsea live score, schedule and results
Sofascore livescore is available as iPhone and iPad app, Android app on Google Play and Windows phone app. You can find us in all stores on different languages as "Sofascore".


Fix #2409: add enable_each_rank_log to deepspeed/launcher/runner.py

Hi, deepspeed team.
This pr(https://github.com/microsoft/DeepSpeed/pull/2409) missed enable_each_rank_log in deepspeed/launcher/runner.py.

hi, any comment? thanks.


Add my first file

I have created a new branch, created a file and made a commit!

Completed Merge!


Hybrid-cloud support: static nodepools

Recent PoC by Adam showed that we're already ready to support on-prem scenarios where Claudie runs within a single private datacenter, despite none of the machines having a public IP address.
Thus, we can already add hybrid-cloud support that would satisfy vast majority of basic use cases.

Storage can work as it worked previously, i.e. it's not a concern.
Nodepools will be the "static" nodepools as are shown WIP here: https://github.com/berops/claudie/blob/master/docs/input-manifest/input-manifest.md#nodepools . Note that the definition of a static nodepool should contain a list of IPs, and the credentials will be just the SSH key that we'll use to bootstrap the machine.
LB needs DNS, and historically we've had an idea to have the user to preconfigure the DNS record on their own, and just tell Claudie about it, see https://github.com/berops/claudie/issues/32#issuecomment-1286809690. Alternatively, I suppose we could just force the users to use one of the already supported DNS providers. E.g. cloudflare, but the issue there is that it is unlikely to work with private IPs.

Child of https://github.com/berops/claudie/issues/32

https://github.com/berops/infra/issues/343

The E2E tests for "on-prem" nodepools can be done by spawning pods within the dev cluster that'll pretend to be on-prem machines to Claudie. KubeVirt could help


WDS Component in Table Columns types: PlainText

Closing as will be doing all required column types at once ( plaintext, URL, number and button )


Add Meson option to control tests

This makes use of Meson's feature option, which gives fine control to packagers on whether to require tests or not, but also gives  a comfortable defaults for average users.

Good call on the dev docs, that seems in scope for this PR so I’ll send a an update in a bit

@dcbaker Where are we on this PR? Do you want another approval?

I think I just need to rebase and merge. I was thinking about this though, and made a PR to Meson to fix at least some of the issues we ran into: https://github.com/mesonbuild/meson/pull/12950 So the eventual Meson 1.5 release, haha

@bretbrownjr rebased, I noticed that in your last refactor (and I missed in review) that part of the name of the define had become part of the function for config.hpp, so there's a patch to fix that as well.


More logging

Codecov Report

Merging #11360 (80bd299) into main (522e914) will decrease coverage by 0%.
The diff coverage is 87%.

:exclamation: Current head 80bd299 differs from pull request most recent head 0f66868. Consider uploading reports for the commit 0f66868 to get more accurate results

@@          Coverage Diff           @@
##            main   #11360   +/-   ##
======================================
- Coverage     63%      63%   -1%
======================================
  Files        476      476
  Lines      33936    33941    +5
  Branches    5499     5496    -3
======================================
- Hits       21544    21492   -52
- Misses     10343    10387   +44
- Partials    2049     2062   +13

Impacted Files
Coverage Δ

...c/kernels/execution/cellExecutionMessageHandler.ts
78% <87%> (+<1%)
:arrow_up:

src/notebooks/controllers/kernelSelector.ts
28% <0%> (-15%)
:arrow_down:

...notification/pythonExtensionRestartNotification.ts
90% <0%> (-10%)
:arrow_down:

.../notebooks/controllers/controllerDefaultService.ts
65% <0%> (-9%)
:arrow_down:

src/kernels/installer/productInstaller.node.ts
72% <0%> (-7%)
:arrow_down:

src/kernels/execution/helpers.ts
56% <0%> (-5%)
:arrow_down:

src/kernels/kernelDependencyService.node.ts
80% <0%> (-5%)
:arrow_down:

src/standalone/api/kernelApi.ts
72% <0%> (-5%)
:arrow_down:

src/notebooks/notebookCommandListener.ts
49% <0%> (-4%)
:arrow_down:

src/kernels/kernel.ts
80% <0%> (-3%)
:arrow_down:

... and 11 more


DSA Codes

Programming codes

Put DSA code inside in proper folder.
For reference follow the DSA folder in master branch.

Updated all the changes please merge this PR.


Fixing #530 - Adding support for SILKY_EXPLAIN_FLAGS

Adding support for additional flags that can be passed as part of the "analyze" query, for more explicit analysis.

Merged.  Thanks for the PR.


⚠ Mobitopup has degraded performance

In de02218, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 4610 ms

Resolved: Mobitopup performance has improved in e7ae369.


Add ChaCha and Salsa intrinsics.

Added @macaba intrinsics implementation #58 for ChaCha20. Salsa20 has since been added to NaCl, so I created Salsa20BaseIntrinsics based on macaba's work. At minimum both got a 4x speed up.

Added ChaCha20 & Salsa20.
Added benchmarks for Salsa20 and XSalsa20.
Added checks for intrinsics support and failovers.
Added VariableLengthCiphers tests, to ensure that the intrinsics methods work when using length specific methods.

This pr will require a way of testing with and without intrinsics, I'm not sure how to do this but I think COMPlus_EnableAVX is related?
A future update could speed up ProcessKeyStreamblock by using the ProcessStream method.
private static byte[] _zeros = new byte[BLOCK_SIZE_IN_BYTES];

public override void ProcessKeyStreamBlock(ReadOnlySpan<byte> nonce, int counter, Span<byte> block)
{

    if (block.Length != BLOCK_SIZE_IN_BYTES)
        throw new CryptographicException($"The key stream block length is not valid. The length in bytes must be {BLOCK_SIZE_IN_BYTES}.");
#if INTRINSICS
    if (Sse2.IsSupported)
    {
        ProcessStream(nonce, block, _zeros, counter);
        return;
    }
...

Codecov Report

Merging #128 (288ae8c) into master (ed3bf52) will decrease coverage by 2.40%.
The diff coverage is 98.69%.

@@            Coverage Diff             @@
##           master     #128      +/-   ##
==========================================
- Coverage   99.78%   97.37%   -2.41%
==========================================
  Files          14       16       +2
  Lines         455     1294     +839
  Branches       57       86      +29
==========================================
+ Hits          454     1260     +806
- Misses          1       29      +28
- Partials        0        5       +5

Impacted Files
Coverage Δ

src/NaCl.Core/Base/Snuffle.cs
57.89% <75.00%> (-39.25%)
:arrow_down:

src/NaCl.Core/Base/Salsa20BaseIntrinsics.cs
98.48% <98.48%> (ø)

src/NaCl.Core/Base/ChaCha20BaseIntrinsics.cs
99.06% <99.06%> (ø)

src/NaCl.Core/Base/ChaCha20Base.cs
100.00% <100.00%> (ø)

src/NaCl.Core/Base/Salsa20Base.cs
88.05% <100.00%> (-11.95%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thanks for the contribution and for adding Salsa20 intrinsics too!
I wonder if we can reduce the method complexity of those intrinsics ones by breaking them apart. Afaik, the initial code was grabbed from another repository which I currently don't recall.
There's also XSalsa20 over there but I haven't promoted those to stable since I felt it needed more tests. The target was to also support XSalsa20Poly1305.
Regarding testing of intrinsics turned on and off, yes, it was something around COMPlus_EnableAVX, I'm not sure since I looked at this a while ago. For the CI, we might need to add tests for running with and without intrinsics I guess.
Btw, do you have some benchmark numbers you can share to see the perf increase?
I might need to take a look at why the Azure CI failed with Windows only.

I wonder if we can reduce the method complexity of those intrinsics ones by breaking them apart.

I've split Intrinsics up by chunk size and altered ChaCha64 and Salsa64 to add HChaCha/HSalsa support. Aside from that I haven't changed the other methods. I guess I could extract some of the Xor/Store/Unpack methods and give them nicer names. I'll add ProcessKeyStreamIntrinsics, the changes to the 64 chunk methods should make it painless.

There's also XSalsa20 over there but I haven't promoted those to stable since I felt it needed more tests. The target was to also support XSalsa20Poly1305.
Regarding testing of intrinsics turned on and off, yes, it was something around COMPlus_EnableAVX, I'm not sure since I looked at this a while ago. For the CI, we might need to add tests for running with and without intrinsics I guess.

I'll take a look, I've never used env variables in unit tests before, hopefully Google will help.

Btw, do you have some benchmark numbers you can share to see the perf increase?

My numbers are from some benchmarks I did yesterday. Sorry about the quality, my laptops about to die 😅
Salsa

ChaCha

I've split Intrinsics up by chunk size and altered ChaCha64 and Salsa64 to add HChaCha/HSalsa support. Aside from that I haven't changed the other methods. I guess I could extract some of the Xor/Store/Unpack methods and give them nicer names. I'll add ProcessKeyStreamIntrinsics, the changes to the 64 chunk methods should make it painless.

Sounds good, thanks a lot for the effort you are putting into this.

I'll take a look, I've never used env variables in unit tests before, hopefully Google will help.

If you run the benchmarks from Powershell or cmd via dotnet run, you can set the variable using set VARIABLE=VALUE before the dotnet command.

My numbers are from some benchmarks I did yesterday. Sorry about the quality, my laptops about to die 😅
Salsa

ChaCha

Thanks for sharing those, the numbers look nice :)

I've experimented with a refactor that treats ChaCha20Base and Salsa20Base as a proxy for the internal ...Core and ...CoreItrinsics methods. It removes the intrinsics checks and preprocessor spam but arguably makes the code harder to read, I don't think I changed any externally visible apis.
I'm planning on creating some general tests for intrinsics code and relying on the CI with different flags enabled to do a thorough check. Ideally I'd configure XUnit to run the tests/test classes again with different flags but it doesn't seem supported 🤷

@TimothyMakkison my apologies for the late follow-up, it has been hectic over here.
Thanks a ton for pulling this big task off, I added a few comments to the PR; most of them are minor stuff.
I also noticed a PowerShell script but I didn't see it being called from DevOps, is it something you intended to do? The build defined in this project uses Cake, we could set the environment variable in there afaik; I will need to investigate further into that. Perhaps we can use that and just define tests that should be running with specific environment variables without the need to update the GitHub actions.
Last but not least, I wonder if you could work on the coverage a little bit more since we dropped quite a lot with this PR, you can check codecov from above.

Thanks for the review. I got sidetracked and forgot about this PR so it's unfinished.

I wonder if we do need the preprocessor directive since we are checking if the hardware supports it

System.Runtime.Intrinsics.X86 is only supported for .NET 3.0+. Intrinsics code won't compile for libraries that multi target versions unless we use a preprocessor.

Could you make this access modifier implicit?

Not sure what you mean by implicit, do you mean private?

We could switch to scoped namespace declaration here since we already using it on the other files =)

Should I convert all the files in the solution to scoped namespaces?

I also noticed a PowerShell script but I didn't see it being called from DevOps, is it something you intended to do? The build defined in this project uses Cake, we could set the environment variable in there afaik;

Sorry I was using the benchmark code and powershell script for testing and forgot to remove them. I'll modify the cake file.
Do you have an opinion on the refactor?
The logic up to 202ff15 was fairly simple; the intrinsics code was added with pre processing and an IsSupported check. This has the advantage of not touching the api, keeping the code/code flow the same and adding minimal additional code and files.
Commit 4d68791 separates the intrinsic and non intrinsic code from SalsaBase/ChaChaBase creating individual files for them, SalsaBase/ChaChaBase act as a facade calling the internal IChaChaCore interface for all methods. This has the disadvantage of adding a lot of code, altering a lot of code to keep the api the same, duplicating code and arguably being overly complex.
I'll probably undo the refactor unless you prefer it, I thinks it's too complicated, harder to maintain, and isn't needed if the CI will test SIMD and non SIMD versions. Should also improve the code coverage and reduce duplication.

Thanks for the review. I got sidetracked and forgot about this PR so it's unfinished.

I wonder if we do need the preprocessor directive since we are checking if the hardware supports it

System.Runtime.Intrinsics.X86 is only supported for .NET 3.0+. Intrinsics code won't compile for libraries that multi target versions unless we use a preprocessor.

Could you make this access modifier implicit?

Not sure what you mean by implicit, do you mean private?

We could switch to scoped namespace declaration here since we already using it on the other files =)

Should I convert all the files in the solution to scoped namespaces?

I also noticed a PowerShell script but I didn't see it being called from DevOps, is it something you intended to do? The build defined in this project uses Cake, we could set the environment variable in there afaik;

Sorry I was using the benchmark code and powershell script for testing and forgot to remove them. I'll modify the cake file.
Do you have an opinion on the refactor?
The logic up to 202ff15 was fairly simple; the intrinsics code was added with pre processing and an IsSupported check. This has the advantage of not touching the api, keeping the code/code flow the same and adding minimal additional code and files.
Commit 4d68791 separates the intrinsic and non intrinsic code from SalsaBase/ChaChaBase creating individual files for them, SalsaBase/ChaChaBase act as a facade calling the internal IChaChaCore interface for all methods. This has the disadvantage of adding a lot of code, altering a lot of code to keep the api the same, duplicating code and arguably being overly complex.
I'll probably undo the refactor unless you prefer it, I thinks it's too complicated, harder to maintain, and isn't needed if the CI will test SIMD and non SIMD versions. Should also improve the code coverage and reduce duplication.

Yeah, I agree with that. It added a lot of maintainability complexity, if we limit the scope of the supported .NET versions we can stick with the first logic you proposed.
I think we can drop .NET 3 and just stick with the supported versions going forward. Supporting obsolete versions of .NET just adds a lot of maintainability headaches.

The "refactor" was added to make the code testable, with the CI and cake file this thankfully isn't needed. This pr is compatible with all the different versoins of .NET it just requires a couple of pre processor checks.

Updated the cake file, NET 7 RC 2 has broken my sdk so I can't install the cake tool and test it. Hopefully it works 🤞
Reverted the changes to Salsa20Base and ChaCha20Base.
Deleted intrinsic/scalar tests.
Optimized the shuffle function for Salsa64, this used a different instruction set so I updated the guard checks.
To reduce the scope of this pr I haven't updated all the namespaces. I'm tempted to create a code cleanup pr changing the namespaces, exception parameters, empty arrays and unused fields etc.

To reduce the scope of this PR I haven't updated all the namespaces. I'm tempted to create a code cleanup pr changing the namespaces, exception parameters, empty arrays and unused fields etc.

Yeah, sounds great!

Do you think it would be worth getting someone who knows intrinsics to double check this? This was my first time using intrinisics/unsafe code in this way.

That would be nice but I really don't know anyone from my side. :/

I had the cunning idea of submitting a pr to the bouncycastle project. Give them a performance bump for large cipherstreams and have another set of eyes look at the implementations. Once I get a grasp of IDigest I'll make a pr


created issue template

corresponds to issue-> #31

@kaiiyer  Please add swoc2021 and a level tag to this issue #31

@kaiiyer  Please add swoc2021 and a level tag to this issue #31


no margin on collection items

when burger menu is not opened, the cards dont have bottom margin

I'm not sure if I understand correctly... but on my side, there is always a margin bottom regardless of whether the burger menu is opened or not:
https://user-images.githubusercontent.com/128157824/230907582-676f68e3-2542-4bb8-94ab-ffd148f455b3.mp4

I'm not sure if I understand correctly... but on my side, there is always a margin bottom regardless of whether the burger menu is opened or not:
https://user-images.githubusercontent.com/128157824/230907582-676f68e3-2542-4bb8-94ab-ffd148f455b3.mp4

Go to collection

👋


wolfram-engine: unable to log in unless with root account

Describe the bug
After installing and registering on the website, it prompts you to run wolframscript to log in. When logging in with a correct user name and password, it fails to activate unless run as root.
The Wolfram Engine could not be activated using your Wolfram ID.

This may have occurred because WolframScript has already activated a different
 installation of the Wolfram Engine using your account's limited
 free activation keys. Only two keys are generated
 when you create a free Wolfram Engine license using
 https://www.wolfram.com/engine/free-license/.

For additional Wolfram Engine keys, contact Wolfram Research at
 https://www.wolfram.com/contact-us/.

Steps To Reproduce
Steps to reproduce the behavior:

install wolfram-engine
register for wolfram engine
run wolframscript and log in

Expected behavior
Logs in successfully for all accounts no matter if it is root or not
Notify maintainers
@fedeinthemix
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.70, NixOS, 22.11 (Raccoon), 22.11pre411613.7e52b35fe98`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.0`
 - channels(root): `"home-manager-22.05.tar.gz, nixos"`
 - channels(beef): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

This could be worked around by moving /root/.WolframEngine to /home/user/.WolframEngine


Set the font-size with default to 16px, so that the font size can be set outside.

So when importing the css in mailer.scss, we can do something like this
   $font-size-base: 12px;
   @import 'bootstrap-email';

Sorry this took a while, thanks for the contribution, merging!

Pushed to rubygems: https://rubygems.org/gems/bootstrap-email

No worries, thanks for open sourcing this project :).


Ruff support

Changes in this PR:
Trello card / Rollbar error (etc)
Screenshots of UI changes:
Before
After

[ ] Requires env variable(s) to be updated

Codecov Report

Merging #1148 (6a4e072) into main (13b8472) will decrease coverage by 0.16%.
Report is 12 commits behind head on main.
The diff coverage is 88.23%.

@@            Coverage Diff             @@
##             main    #1148      +/-   ##
==========================================
- Coverage   91.17%   91.02%   -0.16%
==========================================
  Files          49       49
  Lines        1530     1548      +18
  Branches      196      199       +3
==========================================
+ Hits         1395     1409      +14
- Misses        122      125       +3
- Partials       13       14       +1

Files Changed
Coverage Δ

judgments/tests/test_document_delete.py
100.00% <ø> (ø)

judgments/tests/test_document_history.py
100.00% <ø> (ø)

judgments/tests/test_metadata_panel.py
100.00% <ø> (ø)

judgments/urls.py
100.00% <ø> (ø)

judgments/utils/paginator.py
100.00% <ø> (ø)

judgments/views/results.py
80.00% <0.00%> (-5.72%)
:arrow_down:

judgments/views/unlock.py
89.65% <0.00%> (-3.21%)
:arrow_down:

judgments/views/button_handlers.py
48.83% <20.00%> (+1.10%)
:arrow_up:

judgments/utils/link_generators.py
90.32% <62.50%> (-9.68%)
:arrow_down:

judgments/tests/factories.py
100.00% <100.00%> (ø)

... and 19 more

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

For now, consider using stages: manual to not have this be part of the standard run?


Match lerped values using closeTo

Description
Several of our tests make use of numbers without an exact floating point representation (frequently 0.x where x!=5) which, when scaled, also scale the error. The end result is that some of these tests currently implicitly rely on an implementation detail of floating point math and are sensitive to differences in the ~15th decimal place.
This patch reduces the sensitivity of some of these tests, checking values up to the precisionErrorTolerance from the foundation layer rather than requiring en exact match.
Related patches
flutter/engine#20879 fixes a bug in lerpDouble such that when the a, b endpoint parameters differ significantly in value, 0.0 is always returned rather than an intermediate value. As part of that patch, we change the implementation of the calculation to an algebraically-equivalent form which reduces the floating point error, but which produces small differences in the least-significant bits of the double representation, particularly when the arguments in the test have no exact floating-point representation (such as 0.2, 0.3, 0.6, 0.8 in several of these particular tests).

An alternative is that we change some tests to use values with exact floating point representations, such as 0.25, 0.5 or other numbers that can be represented as sums of inverse powers of two, but that feels like a bit of an unfair constraint to place on test authors.
In general, we should probably be using closeTo() or similar comparisons rather than absolute equality checks for cases where doubles are being scaled or similar.

We normally use "moreOrLessEquals" which abstracts out the epsilon.

TIL. Done!


StencilJS ct definition

hi @lmiller1990 , I'm trying to write a definition file for stenciljs. However, stencil isn't a framework, and comes with its own bundling. I had to pick webpack or vite, so went with vite and used all vite's defaults. Never used vite before, so was a leap of faith.
However, I am getting a rather cryptic error message  of Unexpected identifier 'development'
There is an example stencil project in my fork of your cypress-ct-solid repo: https://github.com/junelau/cypress-ct-stenciljs
would you mind taking a look please?

I tried your project, but I got:

What is the minimal steps to reproduce?
PS: I do not work on Cypress anymore, but I will still try my best to reply and help you out. I might not reply immediately, though.

thanks for looking into it. i fear this isn't the best way to test stencil components with cypress, since stencil is its own server and wrapping it under vite would obscure things even further.


Hints duplicate text of card fields in Images section

Hints duplicate the text of the card fields in the image section, so they are redundant.
Discussed with PO.

Test on:
tamazlykar/1070-redundant-tooltips
Regression:
image_card

@tamazlykar
Tested on: tamazlykar/1070-redundant-tooltips


NEXT-00000 - Changed typo in twig block of contact-form

1. Why is this change necessary?
Its a typo
2. What does this change do, exactly?
Correcting the word phome to phone
The block isnt referenced anywhere
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
#3403
NEXT-21321
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

🤖 Generated by Copilot at 1a6469f
Fixed a spelling error in a twig block name for the contact form element. Added a changelog entry and preserved backward compatibility for the old block name.

🤖 Generated by Copilot at 1a6469f

Fix spelling error in twig block name for phone input in contact form template (link)
Add changelog entry for the fix (link)

@shyim Hi, Is there a problem with my PR? Have I forgotten something or do I still need to do something? I'm just wondering because it seems like it's incomplete because it's not scheduled or merged.

@shyim Hi, Is there a problem with my PR? Have I forgotten something or do I still need to do something? I'm just wondering because it seems like it's incomplete because it's not scheduled or merged.

Hi @AEYCEN, sorry for the delay. No to me it all looks fine so far. We will now import and process your PR. We just need to change the deprecation to 6.7. Because of the already ongoing 6.6 phase we are no longer allowed to add more deprecations for 6.6. But we will do this for you during the import. 👍

Hi @AEYCEN, thank you for your patience. Your changes have been merged to trunk: https://github.com/shopware/shopware/commit/022a707c53b5b5d3e674786e6d41e67dc46930cc


Missing container for homeassistant's raspberrypi3

Information about the Issue
(HTTP code 404) unexpected - {"message":"manifest for homeassistant/raspberrypi3-homeassistant:latest not found"}
Steps to reproduce the behavior

query for the container in kitematic and select create
or, run the pull command specified by the author "docker pull homeassistant/raspberrypi3-homeassistant"

Go to https://hub.docker.com/r/homeassistant/raspberrypi3-homeassistant/tags/
Observe no latest tags were created for that image
Select a different tag in Kitematic (if one exists)
🎉  enjoy container running.

Ideally you open an issue with the repo owner to let them know how frustrating it is that they don't have a latest image tagged

Thanks for the quick reply.

On Apr 19, 2018, at 9:04 PM, French Ben<EMAIL_ADDRESS>wrote:
Go to https://hub.docker.com/r/homeassistant/raspberrypi3-homeassistant/tags/
Observe no latest tags were created for that image
Select a different tag in Kitematic (if one exists)
🎉 enjoy container running.
Ideally you open an issue with the repo owner to let them know how frustrating it is that they don't have a latest image tagged
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Update step_four.md

Windows users --like me-- may miss the period, and docker will not find Dockerfile.txt.

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Thanks, @PlatonicWannabe :)


soc: apollo3: add ambiq apollo3 support

Hello,
I'm creating this pull request pretty early, mostly to get some help. I hope that's ok. The Ambiq Apollo3  SOC seems to be pretty different and I'm struggling with some parts. If there is someone who has experience with Ambiq's SOCs I'd be greatful to help me with the dtsi setup.
https://cdn.sparkfun.com/assets/learn_tutorials/9/0/9/Apollo3_Blue_MCU_Data_Sheet_v0_9_1.pdf

Watchdog WIP

SDK implementation - https://github.com/sparkfun/AmbiqSuiteSDK/blob/e280cbde3e366509da6768ab95471782a05d2371/mcu/apollo3/hal/am_hal_wdt.c


"Approve" option and consent don't work well together

I login to an OIDC application with "approve" and giving consent for all claims
I logout  from the application
Now I login again.

Expected behavior:
I need to consent all the claims again.
Actual Behavior:
I need to just select either "approve" or "approve always" and continue
The above behavior doesn't have any meaning to me. Either the consent for all claims must be asked again or the approval step should be skipped for specific time. I don't understand current behavior.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!

I didn't observe this behavior in IS 7.0 fresh pack.
The "approval always" option is no more available in IS 7.0.


Fixed #45: custom messages for custom validators

I added a test for the 'allow the use of custom messages on failed validation', too.
Unfortunately the test-setup seems not to work well with the promise-based interface of Checkit. Even failing tests are counted as passed and you only see the errors by scanning the test-list for thrown errors. I added a done-callback in a final then(done, done) to the Checkit-Promise of my test to make it work. All other tests should be updated, too. But I'll open an issue for that.
There seemed to be some un-built changes of former commits. So lines 718 + 897 of dist/checkit.js are unrelated to this fix and contain new messages for the field-type integer.
I also added webpack to package.json and changed the npm-scripts build and test to use local versions of mocha and webpack.

Hi Rhys! Are there any suggestions on this?


BigDecimal support

BigDecimals are directly supported by the JsonGenerator, so trivial to
add support from cheshire.

Awesome, thanks!


Quote Functionality

Added Quote functionality. Mods can add a quote (or remove all quotes).
Everyone can request a quote.
Commands:
!addquote This is the quote.
!removeallquotes
!quote
Quotes are stored in quotes.txt in the default directory.

So this is just randomised text? Shouldn't that already be possible? If not it would make sense to add something more general instead.

Not sure what you mean by randomized text. The idea is that the streamer says something funny or ridiculous or whatever, and the mods have the ability to capture the funny quote. All the quotes are then stored in a master list, and when someone requests a !quote, botnak picks a random one from that list and displays it.

You just explained what I meant by randomised text. Generally the funny things get made into sounds anyway. Text quotes would just flood the chat and wouldn't capture much of the contextual hilarity. I don't know what Gochai thinks though.

Oh gotcha. I guess if the chat was being flooded, you could just allow subs to request a quote.

I just took a look at the patch and it looks like you've actually made it a mod command anyway. I tale it this is a mistake?

Ahh, yes, the !quote command should have been PERMISSION_ALL.


Provide dummy versions of e.g. .bashrc so they can be updated externally

Basic idea:
Have two files:

dot/bashrc
dot/real-bashrc

$HOME/.bashrc links to dot/bashrc which, in turn, simply sources dot/real-bashrc. This allows .bashrc to be altered by other tools, and not affect the contents of dot/real-bashrc. dot/bashrc can be added to .gitignore to ensure any altered state is not checked in by mistake.

Problem is being able to link to the real-bashrc file from $HOME, where .bashrc is linked from...


add support for server-side hyderation

Why?
There are some cases when the application is getting hydrated on the server side. The cssInjector utility throws an error because the document is unavailable on the server side.
What?
Support for server-side hydration is added. Made style sheet creation conditional and limited to client-side on the browser.
Also, the redundant style sheet creation of other files is removed.

Codecov Report
Base: 100.00% // Head: 99.66% // Decreases project coverage by -0.34% :warning:

Coverage data is based on head (660837e) compared to base (d889620).
Patch coverage: 90.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##              main       #2      +/-   ##
===========================================
- Coverage   100.00%   99.66%   -0.34%
===========================================
  Files           11       12       +1
  Lines          585      597      +12
  Branches        88       90       +2
===========================================
+ Hits           585      595      +10
- Misses           0        2       +2

Impacted Files
Coverage Δ

src/beacons/fill/index.tsx
100.00% <ø> (ø)

src/utils/cssInjector.ts
95.55% <86.66%> (-4.45%)
:arrow_down:

src/utils/hydration.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Feed names in/after OPML import are non-deterministic

Hi All!
For a project I've been working on (https://github.com/alecmuffett/they-search-for-you) I have been creating/curating RSS feeds which point to a political database site.
The feeds are given a topic title in the OPML file* but upon import of the OPML file, some or all of the feeds names in the sidebar, are inherited from the RSS Channel-Title element of the feed as-retrieved; but if I delete the entries in the sidebar and re-import the OPML, then most (perhaps all?) feed names are taken from the OPML file instead, presumably because caching has occurred behind the scenes.
I suspect that there's a timerace happening to choose/select/discover the name of a newly imported feed, and that — again, for a new feed — sometimes the OPML wins, and sometimes the initial channel retrieval wins.
I feel that the OPML should win in these circumstances.  What do you think, please?

example OPML: https://raw.githubusercontent.com/alecmuffett/they-search-for-you/main/ALL-RSS-FEEDS.opml

I’m not sure what the current behavior is or what it should be. Thinking required.


Examples for -Service?

I can't find any information on the -Service option. Could an example be provided?

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

ID: 35517b3c-d00e-c08d-c274-8b606ad87bfa
Version Independent ID: 3428c627-8703-752e-fbdf-52526f281f57
Content: Enable-BitLocker (bitlocker)
Content Source: docset/windows/bitlocker/enable-bitlocker.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

@officedocsbot assign @e0i

@eeegs  Thank you for your feedback. The -service parameter is used as a switch to Indicate that the system account for that computer can unlock the encrypted volume. The system account is local to the computer similar to an administrator accounts. The system account is used by the operating system and by services that run under Windows.
Hope this helps!
Thanks
Sri

Hello @eeegs  ,
The issue is being closed since your question seems to be answered. Feel free to re-open this issue if you feel that it hasn't been answered or that there are further suggestions to improve the documentation itself.
Thanks.


Ninja build : Redirect error output to a log file

I am using ninja build for compilation , it works fine and is quite fast but i am not able to redirect outputs. I am migrating from gnu make and would like to redirect the error output to a log file and exit if there are any compilation errors.
Below is the snippet:
cc = gcc
cflags= -c
rule compile
command = $cc $cflags  $in -o $out &> log.err || ( cat log.err; exit 1; )
build ninja/hello.o: compile hello.c
It produces below error:
gcc: error: &>: No such file or directory
The command works fine when its used in gnu make/shellprompt but fails when the command is invoked by ninja build via cygwin. Please can someone tell me how can I solve this problem, as there are huge number of files that needs to be compiled and i just want to display errors on screen and not informaton messages and warning messages. Any pointers to solve this will be really appreciated.
Thank You in advance!

On windows, ninja uses CreateProcess API to invoke command. It does not support bash specific notation.
What happen if you write command like below?
cc = gcc
cflags= -c
rule compile
command = bash -c "$cc $cflags $in -o $out &> log.err || ( cat log.err; exit 1; )"

Thanks it works.
But it does reflect proper output when it enters the || condition.
I added a colour output in || condition, but it did not display any colour.
command = bash -c " $cc $ccflags $in.k -o $out &> log.err || ( echo -e '\e[0:31m' "compile fail "'\e[0m'; cat log.err; exit 1 ; )"
This does not give coloured red output which I would expect. Please can you provide pointers on it.

Thanks it works.
But it does reflect proper output when it enters the || condition.
I added a colour output in || condition, but it did not display any colour.
command = bash -c " $cc $ccflags $in.k -o $out &> log.err || ( echo -e '\e[0:31m' "compile fail "'\e[0m'; cat log.err; exit 1 ; )"
This does not give coloured red output which I would expect. Please can you provide pointers on it.

ANSI output on Windows under cygwin doesn't work, https://github.com/ninja-build/ninja/pull/1319#issuecomment-439188115
Closing this since the original issue was addressed.


Java 13.3: Importing Packages Audio

Description 📜
I worked on Audio of Importing Packages in Java
Fixes #2624

Type of change 📝

[x] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)

Domain of Contribution 📊

[x] Java_Domain

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@garvitraj can you review my PR


Use always strings to represent the port

before it a string when result of parse_url() and a int when the
default was used.

I recommend it the other way round …force always integer.

@bwoebi adjusted per feedback


fix: replace color-adjust by print-color-adjust

Replace deprecated color-adjust by print-color-adjust.
Thx autoprefixer for the warning
autoprefixer: Replace color-adjust to print-color-adjust. The color-adjust shorthand is currently deprecated.

See https://caniuse.com/?search=print-color-adjust

Thanks @belgattitude for your contribution!


Text is blurry

I am developing a game in Piston and need to display text in order to show the score. I followed the instructions given here https://github.com/PistonDevelopers/piston-examples/blob/master/src/hello_world.rs and was able to render the text successfully, but it looks blurry. I have tried the solution provided here https://github.com/PistonDevelopers/piston_window/issues/179, but it didn't work for me.

Try swap the graphics backend to opengl-graphics: https://github.com/PistonDevelopers/opengl_graphics/blob/master/examples/text_test.rs

I tried using opengl-graphics, but it didn't make a difference. You can have a look at my code here. The blurriness is more obvious with a light background, so finally I switched to a dark background.

Yeah seeing this on macOS 10.14.6 on the retina screen.

Same here. Could it be that it is not taking the pixel ratio of the screen when building the glyphs? Which might explain it being blurry on Retina screen? I don't have a non-retina screen to check that assumption. I'm happy to look into it, would love some guidance as to where the glyphs are built.

I believe you can fix the blur using a scale transform and larger test. Ideally this should be detected and handled by the text rendering. Ideas?

I'm also interested in rendering sharp text. If someone finds a solution (on online guide) please post a it (or a link) here.

I confirm that rendering twice the size but with scale(0.5) on the transform works. So for the time being I'll put that problem aside ;)


Fix a potential race condition regarding localisation

Prior to this PR, the localisation flag existed on the client object and resources were querying it. This leaves up the potential for subsequent operations to change the flag before all resources which depend on it have been fully processed. This change injects the localisation flag at construction time instead, so that it is immutable for a given resource.

While I was unable to reproduce the race condition myself, we had a customer who could reproduce it and verified this PR did indeed fix it for them (see https://contentful.zendesk.com/agent/tickets/6776).

Coverage increased (+0.1%) to 76.11% when pulling 6c9704b9b9de360d38674a3810db281a2966621a on fix-potential-race-condition into 42ad9dbbe183a92f0082d696b1d81974eecb0d6b on master.

:shipit:


Nexus Issues for Mac

@defilova
Having ran the Nexus cli comfortably on the Ubuntu OS, i discovered some issues running it on the Mac OS.
error: failed to run custom build command for nexus-network v0.4.0 (/Users/gwilldan/.nexus/network-api/clients/cli)`
Caused by:
process didn't exit successfully: /Users/gwilldan/.nexus/network-api/clients/cli/target/release/build/nexus-network-b9098be9342c089f/build-script-build (exit status: 1)
--- stdout
cargo:warning=Failed to run protoc: No such file or directory (os error 2)
--- stderr
Error: Os { code: 2, kind: NotFound, message: "No such file or directory" }`
This is the error message given and i've searched through the discord. I've seen multiple persons with this same issue unattended to. Kindly look into it thank you

install protoc


Sync from master

Codecov Report

Merging #534 (42068b6) into dev (50cdb9d) will increase coverage by 0.63%.
The diff coverage is 27.94%.


chore: update commit message to use conventional commits

This PR resolves []
Description

Our query map updates in metaphysics were failing because they did not adhere to conventional commit format 🙃
This just updates the message to conform.
https://github.com/artsy/metaphysics/pull/5827
PR Checklist

[ ] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

update query map pr to use conventional commits - brian

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Dev changes (update query map pr to use conventional commits - brian)

  Generated by :no_entry_sign: dangerJS against 4b4702d7fbb6fd1ff4913f1ef759379908b964fd


less secure apps

I love this project. Its great. However sometimes it says log in error and it doesn't allow the use of less secure apps. Even when i go my account and enable less secure apps i still get alot of messages saying log in errors now and then. Any way i can make this code up to security standers or just always enable google voice to use this library?

@Davidthecoolsmartguy
It appears that less secure apps would be required, unless and until google enables OAUTH2 login for voice.


uninitialized constant FormtasticBootstrap running a jruby application.

Hello, I have a rails application and I'm using jruby-1.6.6.
I've put in config/initializers/formtastic.rb
Formtastic::Helpers::FormHelper.builder = FormtasticBootstrap::FormBuilder
Then I create the war file and put under tomcat but when I run the application it says:
uninitialized constant FormtasticBootstrap.
It's critical from me because if I use formtastic bootstrap I can't run my applications.
The same problem if I run rake db:migration RAILS_ENV="production"

Sorry was my mistake, I've put the gem formtastic-bootstrap under group assets in my Gemfile.


[BEAM-10656] Enable bundle finalization within the Java direct runner.

This is towards making all UnboundedSources execute as splittable dofns within the direct runner using the SDF unbounded source wrapper since it relies on bundle finalization to handle checkpoints.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @amaliujia @ibzib
CC: @boyuanzz This should enable testing unbounded SDFs using the direct runner with bundle finalization callbacks.

retest this please

Run Java Precommit

Run Java PreCommit

retest this please

Run Spark ValidatesRunner

Run Flink ValidatesRunner

Run Spark ValidatesRunner

Run Flink ValidatesRunner

Run Spark ValidatesRunner

Run Flink ValidatesRunner

Run Spark ValidatesRunner

The test failure is known issue, failing in postcommit https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/7773/testReport/ and https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/7805/

Expanded existing JIRA: https://issues.apache.org/jira/browse/BEAM-8460, opened #12503 to disable failing test category for spark and flink

Thanks @lukecwik
Will take a look at this PR soon.


TUHAbnormal Needs to be updated in braindecode/examples/benchmark_lazy_eager_loading.py

In the file braindecode/examples/benchmark_lazy_eager_loading.py. The TUHAbnormal class initialization needs to be updated
ds = TUHAbnormal(
        TUH_PATH, subject_ids=subject_ids, target_name='pathological',
        preload=preload)```

In the latest TUHAbnormal version there is no **subject_ids** argument

Hi @kuwar81523,
thanks for reporting.
Would you mind creating a PR to fix this?
Thanks.

Hi @kuwar81523,
I was wondering if you had the opportunity to consider @gemeinl' proposal.

Yeah sure @bruAristimunha :)

Sure @bruAristimunha

We look forward to your pull request, @kuwar81523  =) If you need any help please let me know.


Remove local SHA1 from online repository #1485, #1509

In this commit we remove the local SHA1 and all its references from
ZCL_ABAPGIT_REPO_ONLINE. SHA1 is also removed from repo persistency.
SHA1 insn't needed anymore. All where-used places were either migrated to
remote_SHA1 or removed.
Needs further testing.
#1485
#1509

tested ok, merging


When is this happening?

Sorry @paulschwarz ... I've been really busy last few weeks. I'll try pushing this repo forward next couple of days.

I'm also interested ;)

@paulschwarz @bradoyler I think I wrapped up the project. Here is an official page. There is test suite (in node + browser) and everything


How do I add eslint-plugin-react to the CLIEngine

I wondering how I can add this plugin to the CLIEngine. I looked in the docs and it says I need to pass a plugin object but I do not know what does that mean.
That is what I've so far:
import * as ESLintPluginReact from "eslint-plugin-react";
const cli: CLIEngine = new CLIEngine(cliOptions);
cli.addPlugin("eslint-plugin-react", ESLintPluginReact);

Not sure how this would look like in typescript, but in javascript I think it should be
import ESLintPluginReact from "eslint-plugin-react";

import * as from CJS Modules is TS’ broken module system; you’d have to enable synthetic imports to get it to work properly.

Fixed this by using the build in dependencies manager from the system I wanted to create a plugin for. But thanks for the fast response!
From the programm Unibeautify
const ESLintPluginReact = dependencies.get<NodeDependency>(
        "ESLintPluginReact"
      ).package;

@ljharb I'll keep that in mind for other projects!


Add support for @option tag (partially close #51)

This PR add support for @option tag for short and long options.

@option supports short and long options, separated by pipes. Options can have an argument specified between < and >.
If @option format is not recognized by the somewhat complex regex, it is processed as a "bad" option, and outputted in a unstyled list. A warning is displayed on stderr.
@arg without $[0-9] or $@ is processed as an @option tag, and a warning is displayed on stderr.

It also revamp description and titles processing to limit the use of "\n" in code.

Hey, thank you for your contribution and your patience. I have no idea how I missed this great work. Thanks again for your time!


Grub should check if kernel/initrd/init exist while generating the grub.cfg

It can happen that install-grub.pl generates invalid grub entries and results into unbootable system. We should sanity check if referenced files exist upon install.
Fixes https://github.com/NixOS/nixpkgs/issues/14999

cc @wkennington @edolstra

It might be worth just copying the kernel always.
On Fri, Aug 5, 2016, 3:40 AM Domen Kožar<EMAIL_ADDRESS>wrote:

cc @wkennington https://github.com/wkennington @edolstra
https://github.com/edolstra
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/issues/17531#issuecomment-237819589,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABsO_aaQQhEr6AdP_m6saNV0-_hFlmoCks5qcxMWgaJpZM4JdkMX
.

grub-mkrelpath and grub-fstest sound like they could be useful here.

@dezgeg are you willing to take this on? :)

Well, I'll try to debug this further today as it's really annoying on servers.

I wanted to implement this using findmnt, but it's tricky to filter our /nix/store bind mount:
$ findmnt --target /nix/store
TARGET     SOURCE                FSTYPE OPTIONS
/nix/store /dev/dm-0[/nix/store] btrfs  ro,noatime,ssd,space_cache,subvolid=5,subvol=/nix/store

@wkennington I think it's best if we just symlink the kernels if /boot/ is the same as filesystem as /nix/store. And otherwise copy.

On second thought, should grub-install really inspect current state of mounts to determine the booting logic? That really sounds error prone as configuration may be different than the state.

Here's how the logic should go, but it requires a substantial refactoring:

determine if /nix/store and /boot/ are the same filesystem
if yes, use symlinks, otherwise copy the kernels
figure out what the device for boot mountpoint is
check if boot device + path really exist
generate the grub menu

There is a comment: 240     # FIXME: should use grub-mkconfig.. I think that could might help in the long term, as GRUB probably has more battle-tested code e.g. for the "figure out what the device for boot mountpoint is" part.

What's the simplest way to recover from this? I'm in the process of doing so but the process I have taken is pretty messy (potentially destructive?).
I booted the system from some live media I could find (non-NixOS, incidentally), mounted the system in a chroot (as in the install from linux instructions, changed my configuration to require a new kernel so it would actually build the thing, ran nixos-rebuild switch and rebooted. (Not yet sure if this will work, still in progress.)
Is the best way to do nixos-install again without changing anything on the filesystem?

I edited the grub manually to boot and then redeployed nixos cofrectly.

Closing while triaging, I don't think this specific 2016 issue is an issue anymore.
@domenkozar: feel free to re-open if you are hitting what you think is the same failure. I've never heard of anything like that since I've begun using NixOS in early 2017. I believe the scripts and our handling of boot files also have changed a lot since.
Anyone else: feel free to cross-reference this issue in a new issue if you think it is relevant to any issue you may be encountering related to GRUB and the way we handle boot entries.


[BUG]  Not able to create opensearch index via terraform in AWS cloud, seems there is issue with provider Version 2.3.1

What is the bug?
terrafrom: OpenSearch index throws error below.
[BUG] Error: HEAD HealthCheck failed: This is usually due to network or permission issues. The underlying error isn't accessible, please debug by disabling health checks.
Files:
provider.tf
terraform {
required_providers {
aws = {
source = "hashicorp/aws"
version = "~> 5.48"
}
opensearch = {
source = "opensearch-project/opensearch"
version = "~> 2.3.1"
}
}
}
Main.tf
resource "opensearch_index" "kb" {
name               = "bedrock-name_of _index"
number_of_shards   = "2"
number_of_replicas = "0"
index_knn          = true
mappings      = <<-EOF
{
"properties": {
"bedrock-knowledge-base-default-vector": {
"type": "knn_vector",
"dimension": 1536,
"method": {
"name": "hnsw",
"engine": "faiss",
"parameters": {
"m": 16,
"ef_construction": 512,
"ef_search": 512
},
"space_type": "l2"
}
},
"AMAZON_BEDROCK_METADATA": {
"type": "text",
"index": "false"
},
"AMAZON_BEDROCK_TEXT_CHUNK": {
"type": "text",
"index": "true"
}
}
}
EOF
force_destroy = true
}
How can one reproduce the bug?
Plan + apply an index using v2.3.1 of the opensearch provider
What is the expected behavior?
Should be able to create index in AWS OpenSearch.
What is your host/environment?
Terraform Cloud
Do you have any screenshots?
No
Do you have any additional context?
trying to create knowledge base in aws bedrock with the help of below link. getting stuck 2 places. One is OpenSearch index creation.
https://dev.to/aws-builders/how-to-manage-an-amazon-bedrock-knowledge-base-using-terraform-2688

Hi @AvneeshB , can you please try again with this example.


feat(microfrontends): use npx over pnpm dlx for turbo ignore

use the same turbo ignore as the deployment button

lgmt!


PC Requirements

I was using a PC with with GTX 980ti GPU and of course it's too old and does not support many modules (including perception module of course). I want to use Apollo 3.5 or 5.0 with LGSVL simulator. I managed to build and run both of them and test the routing as mentioned in the installation guide of LGSVL.
Now I was looking to buy the recommended PC requirements which include the GTX 1080 GPU. This GPU stopped being produced. Therefore, my question is what other GPU could I buy that is also supported by Apollo.
I read in other issues that people are having trouble building apollo using RTX 2070 - 2080.
Any help would be appreciated.
System information

OS Platform and Distribution: Linux Ubuntu 18.04

Thanks

I have just tried to run Apollo with an Nvidia GTX 1050 GPU with 2GB.
It does not pass the GPU tests (about 17 failures out of 558 tests in total).  For the vast majority of the failures (perhaps even all of them) it would seem to be due to insufficient GPU memory.
bash apollo.sh test_gpu (test run from within docker container).
I hoped that I could get away with that low spec card to at least test if my SW setup worked before purchasing a more expensive card.

After adding another GTX 1050Ti with 4GB to the system I passed a few more tests but 6 are still failing due to memory allocation.

too much tests run at the same time. just add"exclusive" tag to (bazel)  the failed gpu test.

I have same question. Does the RTX2080 can build apollo now?

Does Apollo support RTX now?

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


Build: Added Coveralls integration (fixes #9)

Changes Unknown when pulling cd951a73c544235e09e3629a03ae147f841145ce on coveralls into ** on master**.


Write a Ruby program to divide two numbers

Description
Write a Ruby program to divide two numbers
Input  : 2 4
Output : 0.5

How to contribute

Save the solution in DivideTwoNumbers.rb file
Add DivideTwoNumbers.rb file in divide-two-numbers folder

!assign


Feature Request: add get_doc function

    def get_doc(self, service, method):
        docpath = "/data/doc/" + service + "/" + method
        return self._get_resource(docpath)

I think this would be more a resource for the iracingdataapi developer rather than the final user instead.
What it could be better maybe is to add doscstrings to all the API methods to ease the final user implementation

Generally, I think this could be usefull. But it doesn't need to have service & method arguments. The response data of the main /data/doc endpoint includes all docs, unless iRacing changed something recently.

I think I'm with @musantro on this one at the moment. It would be great to add docstrings everywhere, this is definitely one of the areas where it's clear that this was originally thrown together in an afternoon for something we needed behind the scenes at NOMAD.

Use-case is/was indeed to ease/help with further dev/bugfixing of iracingdataapi, not anything else...

Generally, I think this could be usefull. But it doesn't need to have service & method arguments. The response data of the main /data/doc endpoint includes all docs, unless iRacing changed something recently.

see.... i didn't know.  would have made my life ,uch simpler :)

I still don't see why it should be a public method. But if I were to put this method, maybe I would set it as private method using Python conventions.


Fix Lint/Syntax behavior when Enabled: false of Lint department

Here's another fix I found in work on #11688.
Ruby code has a syntax error and .rubocop.yml with Lint department is disabled:
def foobar
  t = {
    a: "foo"  # missing "," syntax error
    b: 'bar'
  }
  return t
end

# .rubocop.yml
Lint:
  Enabled: false

But Lint/Syntax must not be disabled:

There is one exception from the general rule above and that is Lint/Syntax, a special cop that checks for syntax errors before the other cops are invoked. It cannot be disabled and its severity (fatal) cannot be changed in configuration.

https://docs.rubocop.org/rubocop/1.46/configuration.html#severity
Before
The syntax is invalid but an offense is not registerd:
$ bundle exec rubocop
(snip)

Inspecting 1 file
.

1 file inspected, no offenses detected

After
Lint/Syntax detects invalid syntax as expected:
$ bundle exec rubocop
(snip)

Offenses:

example.rb:4:5: F: Lint/Syntax: unexpected token tIDENTIFIER
(Using Ruby 3.2 parser; configure using TargetRubyVersion parameter, under AllCops)
    b: 'bar'
    ^

1 file inspected, 1 offense detected

The Lint/Syntax behavior for DisabledByDefault: true configuration is also fixed.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Nice catch!


Unable to get language support

I'm a little bit confused with how to enable language support (in web browser).
I've tried to add the following:
  var idx = lunr(function () {
         this.use(lunr.multiLanguage('en', 'fr'));
  });

But I get the following warning:
 Overwriting existing registered function: lunr-multi-trimmer-en-fr
 Function is not registered with pipeline. This may cause problems when serialising the index.

I tried different things like adding lunr.Index.load(idx); but I get another warning (version mismatch: current 0.7.1 importing undefined) and I can't even build the index...
Any suggestion would be really appreciated

Sorry, this is a new feature that was not written by me. There are unit tests that show how it work; you can look at those and see if they provide insight.


Don't share container UTS namespace with pod

kubernetes doesn't share the UTS namespace
Signed-off-by: Mrunal Patel<EMAIL_ADDRESS>

LGTM

why kubernetes doesn't share the UTS namespace?
is it reasonable?
hypercontainer's pod(sandbox) also shares the UTS with the containers.

That's what I thought too. There must be a use case like apache multi host. I will look in the issues.

On Oct 18, 2016, at 6:48 PM, Lai Jiangshan<EMAIL_ADDRESS>wrote:
why kubernetes doesn't share the UTS namespace?
is it reasonable?
hypercontainer's pod(sandbox) also shares the UTS with the containers.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


nil check when looking for the type of a value

Fixes #2465
Somehow the value of a k8s attribute is nil, and we panic on this. The fix is to check for nil before we look for the type of the attribute.

im not sure this is fully fixed. I still can't start any containers that have 0.69.0.
   line: panic: runtime error: invalid memory address or nil pointer dereference
   source: stderr
   tag: splunk-otel-collector/081389b4048d

@jamie1911 please send the whole error message including the stacktrace so we can see where the panic originates, it may be later in the code. I'll reopen #2465 with your comment.


Ability to add arguments for a config file

Allow defining arguments that can be used in the config file, e.g.

bin/aerys -c config.php -- --port 80 --host localhost or,
bin/aerys -c config.php --args port:80 --args host:localhost

Pending issue upstream: https://github.com/thephpleague/climate/issues/73

I guess we can just recommend to use environment variables as well.

I think this should be solved with a configuration for the app instead of process arguments.


Create a String Merge function

It would be nice to be able to take two generated values and merge them together into a single field.

See the new stringMerge function in the README


Code running problems

When I train the model under experiments/cityscapes/744/ours and I type 'sh train.sh 4' in the terminal, it reports the error
tee: 'log/seg_20230706_104127'$'\r''.txt'$'\r': No such file or directory
/train_semi.py: not found.
how to fix this

You should cd experiments/cityscapes/744/ours first.


minio: 2019-02-26T19-51-46Z -> 2019-10-12T01-39-57Z

Motivation for this change
Update to latest version: https://github.com/minio/minio/releases/tag/RELEASE.2019-10-12T01-39-57Z
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @eelco

I tested this manually on CentOS 7.4. It seems to work fine.

@GrahamcOfBorg test minio

@GrahamcOfBorg build minio

@mmahut The test seems to fail because of the python minio package. I don't see a direct relation but after updating the packages and adding the future dependency the test also passes on master now.
/cc @peterromfeldhk as the maintainer of the python minio package

@GrahamcOfBorg test minio

@ofborg build minio python37Packages.minio python38Packages.minio

Can you please rebase this pull request?

@mmahut done


docs: add glossary of terms

Description
It would be helpful to clearly define some of the Shorebird terms on our docs site, including but not limited to:

Updater (Our addition to the Flutter engine, the code that downloads and applies patches. Written in Rust, source at https://github.com/shorebirdtech/updater)
Release (and how a shorebird release differs from a play store release)
Patch (they apply to specific releases, are downloaded on the first app launch after they've been published, are visible on the next app launch after that)
Collaborator (an app has 1-N collaborators. A collaborator is a Shorebird user who can publish releases and patches to a given app.)

Done with https://github.com/shorebirdtech/docs/pull/50


[meta] I'd like to help

Is this project still alive? I would like to contribute since I'd like to use Mycroft on my phone without having to self-host mycroft-core. What's the state of the art, i.e. what are the things already implemented, and what are the things that need to be implemented before anything else? Could you give me some directions?
Thank you :-D
Note: I tried to ask this question on chat.mycroft.ai but nobody answered, so I posted it here

Hi, Sorry so slow in writing. Fought off a cold. So for the core project there was a seperate project started to host what another person was working on. In terms of the core, everything. The project has stalled as the person mainly working on it is, I am assume off overseas on deployment (military) or otherwise incommunicado. So there are two things to start with. Go over the core project code here. Once you get a grip of that, then I have a fork with the "latest" on the alternative core. Sorry, it is kind of vague.  But such is life right now.  If you would like to jump in, the project and users will be thrilled.

Ok, I will follow your suggestions. Thank you for the answer! :-D

Hello! I am the overseas/deployed developer, and I am still here working on it! I've rewritten the core application in Kotlin, re-implemented the STT with Vosk instead of PocketSphinx, and have ported most of Padatious (intent parser) to Android. I will be going underway again soon in the near future, but I should be able to keep in better touch this time. I recommend following the aforementioned advice, but I am also happy to answer any questions about the alternative core.


Stale reads documentation and samples are outdated

Version v3.0.0 introduced a breaking change where timestamp bounds now expect milliseconds precision but in the sample code there is a comment that staleness is specified in seconds. Same issue is also present in the documentation  for the client library.
Sample should have following options to perform 15 seconds stale read:
const options = {
    // Guarantees that all writes committed more than 15 seconds ago are visible
    exactStaleness: 15000,
};

Hi @kosanin, thanks for pointing it out. I have raised a PR for this.


release: v0.20.0

See CHANGELOG

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Apparently, this missed a bunch of files that need to be updated. Will fix.

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Added the missing files.

AWS CodeBuild CI Report

Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


flunkeyism, electricans (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


pre-go-live Checklist link gives error

Could someone let me know what the customer needs to do to access this link successfully?  I am the delivery manager from the Partner and I get the same error message.
https://mbs.microsoft.com/customersource/Global/365Enterprise/learning/documentation/installation-setup-guides/fasttrack-checklist-fin-and-ops

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

ID: 50392961-60d5-ee55-613a-1128a25ef11a
Version Independent ID: c163ee84-669b-137a-a610-9e88c91709ad
Content: Prepare for go-live - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-and-ops/imp-lifecycle/prepare-go-live.md
Service: dynamics-ax-platform
GitHub Login: @ClaudiaBetz-Haubold
Microsoft Alias: chaubold

I forgot to paste in the actual text of the error message...
Sorry, something went wrong
An unexpected error has occurred.
Web Parts Maintenance Page: If you have permission, you can use this page to temporarily close Web Parts or remove personal settings. For more information, contact your site administrator.
TECHNICAL DETAILS
Troubleshoot issues with Microsoft SharePoint Foundation.
Correlation ID: d1ffc39e-b3ca-e028-d4eb-66b4506e65b0
Date and Time: 2/27/2019 7:16:58 AM

@ClaudiaBetz-Haubold, is there something customers/partners need to do to access the link?
@AvaSueGould, the link opens a CustomerSource page that says this:
Microsoft Dynamics 365 for Finance and Operations customers should complete a Pre-Go-live review with the Microsoft FastTrack team before requesting their production environment.
If you are not familiar with Microsoft FastTrack, see the Microsoft FastTrack for Dynamics 365 overview for more information.
This checklist lists the tasks that the project team must complete before requesting a production environment.
The project manager or a key project member must complete this checklist during the Pre-Go-live phase of the project, generally 4–6 weeks before the proposed Go-live date, when user acceptance testing (UAT) is completed or almost completed.
When you have completed this checklist:
If you have over 150 seats and are working with a Microsoft solutions architect assigned to your project, send it to them.
If you have 20-149 seats, send it to<EMAIL_ADDRESS>

Yes, there was an issue within Customer Source that prevented users from accessing it. This was resolved within the same day. Please try again.
Sorry for the inconvenience!
Kind regards,  Claudia
Outlook für Androidhttps://aka.ms/ghei36 herunterladen

From: Sally Erickson<EMAIL_ADDRESS>Sent: Wednesday, February 27, 2019 4:55:56 PM
To: MicrosoftDocs/dynamics-365-unified-operations-public
Cc: Claudia Betz-Haubold; Mention
Subject: Re: [MicrosoftDocs/dynamics-365-unified-operations-public] pre-go-live Checklist link gives error (#963)
@ClaudiaBetz-Hauboldhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FClaudiaBetz-Haubold&data=02|01|Claudia.Betz-Haubold%40microsoft.com|224a949fe49d4411aa0708d69cd4729c|72f988bf86f141af91ab2d7cd011db47|1|0|636868833602183064&sdata=uBJyuXWcK1fQBbGZg330OWzI4z1LS1cTBqz5hDqMN34%3D&reserved=0, is there something customers/partners need to do to access the link?
@AvaSueGouldhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAvaSueGould&data=02|01|Claudia.Betz-Haubold%40microsoft.com|224a949fe49d4411aa0708d69cd4729c|72f988bf86f141af91ab2d7cd011db47|1|0|636868833602193072&sdata=gKnk215MIEGf1dcENWOWYxpT7dJ4rvYLSENemJAxVXs%3D&reserved=0, the link opens a CustomerSource page that says this:
Microsoft Dynamics 365 for Finance and Operations customers should complete a Pre-Go-live review with the Microsoft FastTrack team before requesting their production environment.
If you are not familiar with Microsoft FastTrack, see the Microsoft FastTrack for Dynamics 365 overviewhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fdynamics365%2Funified-operations%2Ffin-and-ops%2Fget-started%2Ffasttrack-dynamics-365-overview&data=02|01|Claudia.Betz-Haubold%40microsoft.com|224a949fe49d4411aa0708d69cd4729c|72f988bf86f141af91ab2d7cd011db47|1|0|636868833602193072&sdata=RGYX50GALJHC5McqfjFq7yENasCm7EBIomPf8PXCwog%3D&reserved=0 for more information.
This checklist lists the tasks that the project team must complete before requesting a production environment.
The project manager or a key project member must complete this checklist during the Pre-Go-live phase of the project, generally 4–6 weeks before the proposed Go-live date, when user acceptance testing (UAT) is completed or almost completed.
When you have completed this checklist:
If you have over 150 seats and are working with a Microsoft solutions architect assigned to your project, send it to them.
If you have 20-149 seats, send it to go-live<EMAIL_ADDRESS>—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-unified-operations-public%2Fissues%2F963%23issuecomment-467942180&data=02|01|Claudia.Betz-Haubold%40microsoft.com|224a949fe49d4411aa0708d69cd4729c|72f988bf86f141af91ab2d7cd011db47|1|0|636868833602203076&sdata=7NF7yN7KFYR%2B%2Bzuko%2Btj29NAmc2%2FhMRudG4256C96g0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAbPrf63D3Mlyok53TyIAePu0eO6ZV0Pmks5vRribgaJpZM4bT3RQ&data=02|01|Claudia.Betz-Haubold%40microsoft.com|224a949fe49d4411aa0708d69cd4729c|72f988bf86f141af91ab2d7cd011db47|1|0|636868833602213084&sdata=a4LPFXLk2PMpOM1L2Vm2gmvvijA1X%2BZzAc4EN3e8vGQ%3D&reserved=0.


Suspect argument on FileNotFoundException in DistributedApplicationOptions.ResolveAssembly()

When DistributedApplicationOptions.ResolveAssembly() fails to find the assembly with the given AssemblyName, the following exception gets thrown:
https://github.com/dotnet/aspire/blob/8aa6c1822a8400db5772e563b54847e239d7cc88/src/Aspire.Hosting/DistributedApplicationOptions.cs#L61
I suspect this is not the intended message. I'd rather have this piece of code instead:
throw new FileNotFoundException($"No assembly with name '{AssemblyName}' exists in the current AppDomain.");

Thanks. Would you like to offer a PR?

Sorry for the noise, I messed up by initially referencing the issue in a merge request done on my fork, rather than on the official repository. I hope my second attempt will be fine.


Add link for PHP

Closes #130

I have similar thoughts. The contents section is really to navigate the current site. We could call out "external link" or something like that, but it's really just meant to be a table of contents.
I'd really like to have a PHP section in the docs but we haven't had any contributors step up. I'd suggest moving this into a different/new section for external links. Maybe the existing Links and other IBM i Open Source resources?

What I preceived is that the other languages were there (Python, Node). People would expect PHP to be there in the same category and not know about IBM's distinction between the types of documentation.


Removed ErrorPage

Removed references to ErrorPage so it can be split off in to a separate module. Most of this happens in the CMS (https://github.com/silverstripe/silverstripe-cms/pull/1849)
Related RFC: https://github.com/silverstripe/silverstripe-framework/issues/4149

Don't merge until CMS + errorpage is done.

Very cool, great effort and persistence, Jono! :) Does this need to be in beta1 (= is it an API change?). I can't see any documented in the upgrading guide here in this PR. If it's an API change, you've just got around a day to get this over the line - we're trying to stabilise master now for a beta1 release in the next few days.

Aaargh, pushed some changes to my branch but they weren't getting picked up here so I closed and now can't reopen due to force push. New PR at https://github.com/silverstripe/silverstripe-framework/pull/7061.
@chillu yes it's an API change in CMS due to removal of ErrorPage class. I've added some notes to the changelog. This should be all good to go I think, module set up is done, just awaiting review and merge of PR. Primary discussion and overview of work is at https://github.com/silverstripe/silverstripe-errorpage/issues/1


"nse" (no side effect) format specifier is missing

I've just learned from a video of a Microsoft Build 2018 talk that there is a "nse" format specifier that prevents side effects from debugger evaluations. Seems like that specifier is missing in the documentation.
References:

Video of the talk where it is shown: https://youtu.be/yAZLibpil_c?t=11m30s
Written documentation: https://github.com/Microsoft/vs-diagnostics-demo-recipe-app/blob/master/tips/NoSideEffects.md

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

ID: 92ab3309-0c6a-5150-64d8-0339a03f3238
Version Independent ID: e29ede27-d73c-7294-cdcc-c1227353bbb9
Content: Format specifiers in the debugger (C#) - Visual Studio
Content Source: docs/debugger/format-specifiers-in-csharp.md
Product: visual-studio-dev15
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@cremor Thanks! I'll look into this.

Adding "nse" to this doc


New Adapter: neotest-ctest - A neotest adapter for C/C++ using CTest as a test runner

Hello,
I've been meaning to create this ticket for quite some time now, but I never got around to "completing" my adapter. Now, however, I'm quite happy with the result, which is why I'm reaching out with a request to add it to your list of adapters.
You'll find it here: https://github.com/orjangj/neotest-ctest
It's primarily targeting C/C++ test frameworks using CTest as a test runner. It currently has basic support for GoogleTest and Catch2. I've put some effort into the implementation to ensure that extending it with new test frameworks will be fairly easy. I'm planning on adding support for more frameworks such as doctest and CppUTest. Maybe more if there are feature requests.
It also parses test results and displays errors as diagnostics.
It does not handle compilation of source/tests (and it probably never will), but I've found that using https://github.com/Civitasv/cmake-tools.nvim as a companion plugin works like a charm.
It does not support the debug/dap strategy yet, but I have plans to support that as well.
Hope to hear from you soon.

I will create a PR to include it in the README if you accept my request :)


Odom seems not been published and the map cannot load in RVIZ.

Thanks for sharing your great work.
Some thing strange happens. There is no error or warning in the terminator when i launch the code "python3 train_velodyne_td3.py"  in ''TD3" dir.
And I modified the "false" to " true" for the "gui" in empty_world.launch to see if the gazebo works.
It turned out the gazebo was working well.
But the Rviz still cannot load the map.
Here are my outputs  and  resuts.
I would like to ask that could you please give me some suggestions or advice?

Hi
Can you specify what are the displayed errors in the Rviz?
I see you are using ROS Melodic. Did you clone the Melodic version of this repository?

Hi
Can you specify what are the displayed errors in the Rviz? I see you are using ROS Melodic. Did you clone the Melodic version of this repository?

Thanks for your reply. I found the question is the rp version i used was not compatible with my ROS_Version, i checked it out, and after changing the version of the rp version, this problem has been solved and the rviz runs perfectly.
Thank you again for your time and your promising work sincerely.

Hello, I can run normally in the noetic version, but I won't be able to switch to the melodic version, I want to know how to clone the Melodic version of this repository,I am not very familiar with this operation


Kconfig: add SHELL selection for XEN_SHELL config

Previously, there was and error during builds for samples, that have not enable SHELL option during build. XEN_SHELL module used Zephyr shell infrastructure, but had no dependency to it. Now it will be selected in case user wants to use shell cmds for Xen domain managment.

+1

Acked-by: Leonid Komarianskyi<EMAIL_ADDRESS>


Font rendering in zoomable stages

Oh dear vahe, merge conflicts everywhere

Tom merge, until you didn't push smth


Force disable Vulkan overlays in the editor and project manager

MangoHud would be shown on every popup in multi-window mode, and seemed
to cause rendering issues for various users.
There are also issues with RTSS, and VkBasalt also doesn't seem
particularly good to use in the editor.
All these stay available to enable when running the project itself,
including running from the editor (so running the editor after e.g.
export MANGOHUD=1 in the environment will ignore it for the editor,
but use it when playing the project).
Fixes #57403.
Fixes #57937.

Before:

After:

At the same time, we can also force ENABLE_VKBASALT=0 on the editor/project manager. Yyou generally don't want injected SMAA/CAS from influencing the editor UI (it'll negatively impact font rendering quality).

At the same time, we can also force ENABLE_VKBASALT=0 on the editor/project manager. You generally don't want injected SMAA/CAS from influencing the editor UI (it'll negatively impact font rendering quality).

Done. Like the others, it also supports a DISABLE_VKBASALT override to allow disabling it without overriding the user-defined ENABLE_VKBASALT.

Thanks!

This approach doesn't appear to be working on Windows with RTSS, both on Vulkan and Direct3D 12 when using https://github.com/godotengine/godot/pull/70315. I've also tried setting DISABLE_RTSS_LAYER to 1 in the command line before running the editor to no avail.
Using Windows 11 and RTSS 7.3.4 Beta 3 (bundled with MSI Afterburner).

Can you check the RTSS JSON file to see if what defines it relies on to skip it?
As a example for MangoHud on Linux:
$ cat /usr/share/vulkan/implicit_layer.d/MangoHud.json
{
    "file_format_version" : "1.0.0",
    "layer" : {
      "name": "VK_LAYER_MANGOHUD_overlay",
      "type": "GLOBAL",
      "api_version": "1.3.0",
      "library_path": "/usr/$LIB/mangohud/libMangoHud.so",
      "implementation_version": "1",
      "description": "Vulkan Hud Overlay",
      "functions": {
         "vkGetInstanceProcAddr": "overlay_GetInstanceProcAddr",
         "vkGetDeviceProcAddr": "overlay_GetDeviceProcAddr"
      },
      "enable_environment": {
        "MANGOHUD": "1"
      },
      "disable_environment": {
        "DISABLE_MANGOHUD": "1"
      }
    }
}

Should probably be discussed upstream then, if it doesn't work when you define DISABLE_RTSS_LAYER manually in the environment yourself. I assume you can reproduce it with other applications too.

RTSS 7.3.4 has an interesting entry in its changelog (ReadMe.pdf):

Improved compatibility with Vulkan applications, using multiple coexisting swapchains (e.g. Godot v4 editor)


Print mutli-line doc comments before recipe

@kranurag7 re your comment in #2090, this prints out multiline doc comments:
[doc('multiline
comment')]
foo:

Like this:
Available recipes:
    # multiline
    # comment
    foo

How does that strike you?
I was thinking that:
Available recipes:
    foo # multiline
          comment

Could get awkward to read, and would mean that a long doc comment would be squashed to the right.

Thanks for the fix.
With 1st, one have to re-think that the comment is for the recipe that is listed down and one may get confused if the comment is for the recipe listed down or above.
I think keeping it on the right is more flexible for users because then they'll read from left to right.
Details

$ ./target/debug/just -f justfile.foo --list
Available recipes:
    # multiline
    # comment
    foo
    hello
    moo
    # this is a comment
    # for test recipe
    # comment
    test
    work
    zoo

Could get awkward to read, and would mean that a long doc comment would be squashed to the right.

we can auto-wrap after few characters, something like git commit messages where auto-wrapping is recommended after 72 characters.

Auto-wrapping would require word-splitting, which is error prone. For example, if word-splitting is done on spaces, the splitter would split commands like echo foo in the middle. Whatever we do, I'm inclined to leave the lines as the author wrote them, and they can adjust the line-length how they want.

I'm going to merge this, since it's definitely an improvement over the current behavior. I'm still undecided about trying to print multi-line doc-comments after recipes with alignment. I think it's probably not a simple win, since I think things like this look pretty awkward:
Available recipes:
    a # this is
        a long comment
    b # another comment
```

I'm going to merge this, since it's definitely an improvement over the current behavior. I'm still undecided about trying to print multi-line doc-comments after recipes with alignment. I think it's probably not a simple win, since I think things like this look pretty awkward:

Please, please do the alignment! We have lots of recipes with a lot of docs and the output of just --list is just a mess.
It wasn't well aligned previously, especially when there is a recipe with long list of arguments (3 and more), but with moving documentation under the recipe name while for other recipes it is right aligned make everything very hard to read!

@zekefast I'm happy to consider it! Could you open a new issue with an example of the current output you're getting how you'd like it to be improved?