Warnings/errors from "brew edit" commands being routed to standard output.

Description
This unusual routing prevents the normal output from being captured in a variable and used.
I see the problem doesn't occur on the macos-11 runner, but it does occur on every other macos runner I tried.
I see that the version of homebrew is older on the macos-11 runner (4.2.5) whereas the other runners use homebrew 4.3.0. But I don't think this is the whole problem, since I cannot reproduce this problem on my own machine running macos 14 and homebrew 4.3.0.
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 11
[X] macOS 12
[X] macOS 13
[X] macOS 13 Arm64
[X] macOS 14
[X] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
20240514.3
Is it regression?
yes
Expected behavior
When using a command brew edit --print-path [formula] the result should be printed on stdout, allowing you to capture the output for use in other commands. Any warnings should be printed on stderr.
    sdl_image_formula=$(brew edit --print-path sdl_image)
    echo sdl_image_formula: $sdl_image_formula

should result in something like:
Warning: edit is a developer command, so Homebrew's
developer mode has been automatically turned on.
To turn developer mode off, run:
  brew developer off

Warning: `brew install` ignores locally edited casks and formulae if
HOMEBREW_NO_INSTALL_FROM_API is not set.
sdl_image_formula: /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/s/sdl_image.rb

where you see the errors/warnings as the command is run, then you see the captured formula path afterward.
Here's a run showing that it used to work in macos-14-arm64:
https://github.com/btb/d2x/actions/runs/8953473131/job/24592024810
Here's a run showing it still works normally using the macos-11 runner: https://github.com/btb/d2x/actions/runs/9121219819/job/25080022032
Actual behavior
The warnings/errors are somehow being routed to the standard output, so they are improperly captured in the variable.
    sdl_image_formula=$(brew edit --print-path sdl_image)
    echo sdl_image_formula: $sdl_image_formula

results in:
Warning: edit is a developer command, so Homebrew's
developer mode has been automatically turned on.
To turn developer mode off, run:
  brew developer off

Warning: `brew install` ignores locally edited casks and formulae if
HOMEBREW_NO_INSTALL_FROM_API is not set.
sdl_image_formula: ::warning::`brew install` ignores locally edited casks and formulae if%0AHOMEBREW_NO_INSTALL_FROM_API is not set.%0A /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/s/sdl_image.rb

Here's the run where it stopped working in macos-14-amd64:  https://github.com/btb/d2x/actions/runs/9120125848/job/25077018332
Repro steps
Run the 'brew edit' command in a "run" shell, and capture its output to a variable
.

Hey @btb!
Good first issue. I cannot reproduce faulty behaviour on my macOS-14 too. Seems to be the real case. We'll check what might be done to resolve it.

Hello @btb . After investigation we found that problem could be in new runner version. Behavior also reproducible using self-hosted runner on local MacOS machine , so unfortunately we cannot do nothing from our side with it. You can create issue in runner repo for fixing it in a future. As workaround you can enable brew developer mode and set HOMEBREW_NO_INSTALL_FROM_API variable in runtime for avoiding warnings. I am closing issue but you if you have additional questions feel free to reach me out.


"Cannot find service module info file" Exception during training-stage

System Information (please complete the following information):

Model Builder Version: 16.0.2003.302
Visual Studio Version: 16.6.0 Preview 2.1

(Stack trace and csv-data at the bottom)
Describe the bug
When running the training in the Model Builder it crashes with a "Cannot find service module info file"
Detailed message:
Cannot find service module info file 'MicrosoftMLModelBuilderAutoML.servicehub.service.json' in 'C:\Program Files (x86)\Microsoft Visual Studio\2019\Preview\common7\ServiceHub\Services' (ErrorKind: ServiceModuleInfoNotFound)

To Reproduce
Steps to reproduce the behavior:

Start the Ml.net Model Builder
Select "Custom Scenario"
On "Add Data"-page add the file (bottom of this post, rename from txt to csv)
Select "Survived" as Label
Let Features be the four remaining columns
Go to "Train"-page and set task to "Multiclass-classification", and train time to 60 seconds
Get exception, (with little information about finding a solution and a link to report bug here)

Expected behavior
The training to finish without incident, (or the error message to be more helpful)
Screenshots
Data step:

Error message:

Train step, after error occurred:

Stack Trace
   at StreamJsonRpc.JsonRpc.<InvokeCoreAsync>d__115`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ServiceHub.Client.HubClient.RemoteServiceBrokerWrapper.<>c__DisplayClass9_0.<<RequestServiceChannelAsync>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.ServiceHub.Client.HubClient.RemoteServiceBrokerWrapper.<>c__DisplayClass9_0.<<RequestServiceChannelAsync>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ServiceHub.Utility.Shared.ServiceHubRetry.<ExecuteAsync>d__0`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.ServiceHub.Utility.Shared.ServiceHubRetry.<ExecuteAsync>d__0`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ServiceHub.Client.HubClient.RemoteServiceBrokerWrapper.<RequestServiceChannelAsync>d__9.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ServiceHub.Framework.RemoteServiceBroker.<GetPipeAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ServiceHub.Client.HubClient.<GetPipeAsync>d__34.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ServiceHub.Client.HubClient.<RequestServiceAsync>d__31.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ML.ModelBuilder.AutoMLServiceFactory.<CreateAutoMLServiceAsync>d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ML.ModelBuilder.TrainSession.<InitializeAutoMLServiceAsync>d__59.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ML.ModelBuilder.TrainSession.<StartAsync>d__48.MoveNext()

Data
full2.txt

@JakeRadMSFT Have you had the opportunity to investigate this?

@frankhaugen Thanks for reporting with details, it makes the investigation much easier!
As of VS version 16.6 Preview 2, we've started shipping Model Builder with VS instead of as an extension. Based on your version numbers above, it looks like you have the VS with Model Builder (16.6 Preview 2) and Model Builder installed as an extension (16.0.2003.302).  Could you try uninstalling Model Builder as the extension, and using the preview feature?
You probably know you can uninstall an extension via the Manage Extensions window -> Uninstall. You can then enable the VS Model Builder in the Preview Features window.

Could you please give this a try and see if it resolves the problem? Thanks!

@beccamc Thank you for a detailed reply, and your praise, and the training now works great
But evaluating on the other hand

   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
   at StreamJsonRpc.JsonRpc.<InvokeCoreAsync>d__94`1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ML.ModelBuilder.AutoMLServiceFactory.AutoMLServiceProxy.<PredictMultiClassClassificationAsync>d__16.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.ML.ModelBuilder.TrainSession.<PredictMultiClassClassificationAsync>d__49.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at Microsoft.ML.ModelBuilder.ToolWindows.EvaluateTabDataContext.<MulticlassficationAsync>d__143.MoveNext()

The bugs here is that the error message could be fleshed out to include "expecting X got Y", and that the default suggestion, (based on a random row), should never be able to fail due to a "format exception"
Should I file a new issue??

@frankhaugen I'm glad the work around fixed it!  Let's move to a new issue for the Try It Out issue. I've filed a new one here - #682
Thanks for finding and reporting these bugs!


Question about a lack of adaptability traits

Question forwarded from steam workshop
From: **Verdant Squire **
whats up with the lack of adaptability traits? Those were my favorite pick in the base game, and the "toughness" traits don't really stratch the same itch they did.

Toughness is literally the same as adaptability for all intents and purposes, just with some added army bonuses because if they can survive on a toxic world without a gas mask, they'd probably make slightly better soldiers too.


Changes Not Reflected on Solscan - (PR #14449)

Thanks @keone , I can see that the changes have been made on the Solana token list but my search on solscan for Kalia does not reflect the current changes - 8Wf4RCoeBsnjaWuSP8Z2uNKyBjjCVumLhAkpzDcTw175
Can you please advise

it looks okay now? i think there's just a delay
https://solscan.io/token/8Wf4RCoeBsnjaWuSP8Z2uNKyBjjCVumLhAkpzDcTw175


Using ISO pre-install template

Hi sh0rez,
I using another method to approach customize vSphere VM using pre-install ISO which more generic way.
Not sure if this is good way. Welcome to discuss if you are interesting.
Regards,
Elsvent

Hi!
I appreciate your work, but I believe this is out of scope here. But I would be happy to link it as an alternative way in the README. It looks like you deleted your repository, so would you mind restoring it from this PR?

Sorry, I must delete this accidentally. Fortunately, I got backup from my previous company.
I will Close and create a new PR.


Feature request: close or hide button

A simple (x) close button for hiding the overlays would be great. Right now they stay on top and I have no way to close them except doing a refresh of the page. When using pjax they also sometimes stay open on the next request or showing double ... old and new request windows will show on top of each other ...

+1


feat(#261): add new folder with deployment commands and delete deploy…

This is the issue #261.
Just modified and add some features for Slot CLI documentation that will bring great context with:
-deleted deployments page.
-added create, delete, update, describe, list, logs and help command pages.

@kariy.

Hi, @kariy I have an error, in this part, could you check it, I'll fix it as soon as possible

hey @juandiegocv27 thanks for the work!
can you also update the sidebar config so that the new pages would be accessible from the sidebar ?

Ok sure 🫡

Hi @kariy, I just fix it, and agreggate some features, hope for feedback

HI @kariy @milancermak @milancermak, is this PR is still relevant? Please let me know if there's anything else you’d like me to add or if you have any feedback. I'm happy to make any changes you need. Thanks

@juandiegocv27 could you fix the conflict

@ponderingdemocritus sure, but, talking in discord with @kariy told me to erase the file deployments.md, so I could return it, but all that I do was put more stuff and make deployments a folder

@ponderingdemocritus @kariy just fix the conflict

@ponderingdemocritus @kariy just fix the conflict, but new conflict appear, now the conflict dont tell me what to fix

@ponderingdemocritus @kariy @milancermak @ptisserand, just resolved conflicts

@ponderingdemocritus @kariy @milancermak @ptisserand Fix all the code and ready to merge:

waiting for more feedback 😎🚀

Thanks for the work here @juandiegocv27!
Some important points:

consistency: slot should be formatted with consistency as sometimes you use Slot and sometimes Slot.

formatting: I've left some commands on formatting, not on all the lines that require changes, you may do a general review of formatting.

@glihm thanks a lot for the feedback, I'll work on it as fast as possible ⚡

hI @glihm, @kariy, fix somethings in this issue, ready to merge🚀

@juandiegocv27 thanks for the work here! Some comments that also apply to all commands.
Once done, this should be ready for merging! Thank you for the iterations, let's know if you have any question!

Thank you for the feedback @glihm, I´ll make the changes as fast as possible

@juandiegocv27 thanks for the work here! Some comments that also apply to all commands.
Once done, this should be ready for merging! Thank you for the iterations, let's know if you have any question!

Thank you for the feedback @glihm, I´ll make the changes as fast as possible

No rush, thank you for your contribution. 👍

@juandiegocv27 thanks for the work here! Some comments that also apply to all commands.
Once done, this should be ready for merging! Thank you for the iterations, let's know if you have any question!

Thank you for the feedback @glihm, I´ll make the changes as fast as possible

No rush, thank you for your contribution. 👍

Already fix and put response and output in every command😎 @glihm

lgtm

thank you so much


IDE can't find sources

Hi there!
Using your plugin on a Google App Engine project, the IDE cannot find the generated sources:

The project is compiling fine though.
Any clue?

Ok, sorry for the noise, just figured out.


ci: Boost mem for integration test

unblock https://github.com/envoyproxy/envoy/pull/36899

/retest UNAVAILABLE


[FLINK-24704][table] Fix exception when the input record loses monotonicity on the sort key field of UpdatableTopNFunction

What is the purpose of the change
Fix the exception when the input record loses monotonicity on the sort key field of UpdatableTopNFunction
Verifying this change
This change is already covered by RankHarnessTest
Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): (no)
- The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
- The serializers: (no )
- The runtime per-record code paths (performance sensitive): (no)
- Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
- The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@flinkbot run azure

@flinkbot run azure


Adding 2 surround audio tracks with different bitrates?

Hey @donmelton,
I am working through some TV show Blurays that have commentary tracks that are already in ac3 640. I am encoding the main surround at 448, and would like to include the commentary tracks in surround format at 384. Is this possible in a single pass? I know I can easily add the commentary track as stereo at 224, but I'd like to keep surround if possible.
Is this a case of needing to separate the tracks, do the work, and remerge after to get my desired results?
Thanks as always!

@TehMaster Currently there's no way to specify different surround or stereo bitrates per track with other-transcode. That's intentional because I not only want to keep the option syntax simple and understandable but I also want users to approach transcoding without feeling compelled to twiddle things that, really, don't matter. I mean, seriously, just pick an overall bitrate and move on. :)
But, yes, you could certainly separate the tracks and remerge afterwards. Up to you.

@donmelton Sometimes I come here just for the sass. :) I really figured that is what you would say, and already laughed at myself for why I do this.. I mean really its only roughly MB difference.. Maybe i just need things to do and fiddle with? All i have is time these days... Ha.
Well fine. 640 across the board it is.

@TehMaster I'm here to entertain if nothing else! And choosing 640 Kbps is easy because that's the default. :)


User of Person not updated correctly

When you log in via UT, a User is always created. When the s-number or m-number matches a Person and the person has no bound account, we try to bind it or create it instead. However, when a student becomes an employee, the bind already exists, so the user of Person is not updated, see  here. Probably, we just need to check whether the passed username (the true login name) matches the bound User.

Is there any case where having the incorrect username stored in the User model is causing issues? Because if we check the username in the get_or_create_user function, then this will also update the user when someone logs in using their IA-account (get_or_create_user is also used in the IALoginBackend). So, this already was a 'problem' if someone has a student account and an Inter-Actief account and uses both.
If this desync causes problems somewhere, then we need to devise some kind of solution for only s- and m-accounts, or find a way to link multiple sign-in names to a single user (maybe take some inspiration from Alexia, which already supports that)

Had to think about what happened a bit again :D. The process is:

Student becomes member and logs in with s-account
Student becomes employee; this is correctly reflected in employee__number
Employee logs in the website

We now have:

Person(id=1, no=m12345)
User(id=1, uname=s54321)
User(id=2, uname=m12345)

where person#1.user == user#1. When you log in with m12345, the request user is set to user#2. On initial login person#1 is fetched by login name, get_or_create_user(m12345) is called on person#1. Since person#1 has a valid user it is not created or updated. Later the person middleware tries to fetch the person of the request user, user#2, but since person#1 was not updated, user#2 is not associated with a person, leading to "Hello, you are unknown in our administration" etc.
I don't think we need a multi-UT-login solution necessarily, I think it's fine to just force a user update if the user does not match (i.e. matched_person.user != request.user in the SAML backend ==> set the user). I think the only bad effect that could have is that an employee can't use their s-account anymore to log in (don't think that's a problem).

The difference with the IALoginBackend is that it returns a user, I guess that necessarily overrides the user.

Ah, I see the problem now. Should be pretty easy to fix.

@niomaster I think I've found an acceptable fix/workaround for the problem, which handles it in the same way that the old RADIUS logins handled this problem, and which the IALoginBackend still does. Do you agree?

Yes, let's keep it consistent with the IALoginBackend


!!omap support (or not?)

Describe the bug
I'm reading YAML's changelog, and according to it support for the !!omap type has been dropped in v1.2.
So I'd expect it to not be supported in a v1.2 compliant parser.
To Reproduce
The following code gives me an ordered map, as if !!omap never got dropped:
import yaml from 'yaml';

const result = yaml.parse(`--- !!omap
- Mark McGwire: 65
- Sammy Sosa: 63
- Ken Griffy: 58
`, {
  version: '1.2'
});

console.log ( result );

Expected behaviour
I'd probably expect that code to throw?
Versions (please complete the following information):

Environment: Node v18.19.0
yaml: v2.3.4

Additional context
Looking at the spec for v1.2.2 there's actually an example snippet of code that seems to imply that !!omap is still supported, but that's literally the only reference to "omap" in the entire document.

So I'm pretty confused about this. Should !!omap still be supported in v1.2 or not? And whatever the answer is how did we arrive at that? Because I'm seeing contrasting information.

If you'd rather not have the YAML 1.1 tags recognised when used explicitly, you may set resolveKnownTags: false when parsing: https://eemeli.org/yaml/#schema-options

I see. Is that the only other option needed to get a spec-compliant v1.2 parser?

The parser is by default 1.2 spec-compliant, and for all untagged content it follows the Core Schema, as recommended by the spec. For explicit tags, we follow the advice in the Other Schemas section,

None of the above recommended schemas preclude the use of arbitrary explicit tags.

and represent e.g. the JS Map as !!omap, Set as !!set, Date as !!timestamp, and Uint8Array as !!binary, using their YAML 1.1 definitions (given that those are well defined and more interoperable than any alternative). Tags for JS values that are not well represented by the YAML 1.1 type definitions are available separately via the yaml-types package.


feat: Groups menu overflow shadows

To resolve #7528 this adds a box-shadow to indicate that more scrolling is available. This is a common UX pattern, and it requires JavaScript.
Here's how it looks,

Light mode:
Dark mode:

There are a few other places where we have UI that has a scrollbar that could use this treatment - hopefully it generalizes and will be easy to apply in those spots (example, the various dropdowns on the search page).


Properly print path to config file to the screen in knife configure

config[:config_file] was nil so that was blank. Toss in some yard and shuffle things around while I'm in here.
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

Fixes #7303 #7304

Proxy tests are broken and unrelated to this change


Correctly Fill Auxiliary Vector

This pull request fills the auxiliary vector with values for AT_PHDR, AT_PHENT, AT_PHNUM and AT_ENTRY as well as minor cleanup of Elf.hh and Elf.cc.
Without this, programs compiled targeting RISC-V with gcc would not execute correctly as they did not know where the program headers were located.

Looks good to me


feat: changing jwt to paseto

A complete change was made to the application, from JWT, to Paseto, in which new unit tests were also carried out.

Replaced by #44


[Feature request] Add buttons for , , , ,  and  when writing comments

Please add buttons for
, ,
,
,
 and
 (in addition to the existing five), when writing / editing comments, as these also work well in both, Storeman 0.0.12 and OpenRepos' web-frontend.
Ideas:

[¶] button (codepoint U+00B6 or a uppercase "P") inserts
_

[c] button (bold lowercase "c") inserts _
[> ] button (greater sign, maybe followed by a space) inserts

_

[• ] button (codepoint U+2022 or a bold U+00B7, maybe followed by a space) inserts

_

[1.] button inserts

_

[CR] button inserts
_   (Edit: Not really needed, as
 is inserted on submission of a comment at every line break.)

Notes:

"_" denotes the cursor position after pressing the corresponding button.
It may make sense to reorder the buttons, e.g. [B] [I] [U] [c] [M] [¶] [> ] [• ] [1.] [🔗]

The  and  tags are not really needed as they are inserted automatically by OpenRepos.
And it's a question about the list tags. A list tag assumes that there are more than one list items inside it. So you insert a pair  or  tags and a pair of  tags inside it and what about next items? Add one more button for the  tag? Or try to detect if the cursor is inside list tags and insert list items tags on return press like in WYSIWYG editors?

The
 and
 tags are not really needed as they are inserted automatically by OpenRepos.

While I fully agree WRT the
 tag, paragraphs (
) are really hard to control properly.  I found setting paragraphs explicitly (in both, Storeman and the "plain text editor" in OpenRepos' web-frontend) necessary many times.
Somehow the behaviour with Storeman and OpenRepos' "plain text editor" appears to be slightly different: When using Storeman 0.0.12 often a
 is inserted, where I would have expected a

.
Also note that OpenRepos' web-frontend automatically inserts a
_
, when one starts with an empty message.  This may cause some of the perceived differences in behaviour.

And it's a question about the list tags. A list tag assumes that there are more than one list items inside it.

Well, as you chose to let users edit raw, HTML formatted commets (which is good IMHO, as easier to implement and providing the user with full control), users have to have some basic knowledge about HTML when using these tag-buttons.  (Users without such knowledge can still just type a message and it will be automatically formatted by OpenRepos, when submitted.  Only when these users try to edit a comment, they will have trouble understanding what happened to their message.)
In short: For editing raw HTML, one has to have some basic knowledge about HTML, anyway.
This is why I regard the buttons as little helpers to insert supported tags easily (i.e. without having to type them), but not more than that.

So you insert a pair  or  tags and a pair of  tags inside it and what about next items? Add one more button for the  tag?

I considered suggesting this yesterday, but did not have an idea for a symbol (and thought one may just cut&paste the inserted

).
Thinking about it again, an extra button absolutely makes sense.  You may use [· ] (codepoint U+00B7) for

, while [• ] (codepoint U+2022) for

, and order these buttons [• ] [1.] [· ]

Or try to detect if the cursor is inside list tags and insert list items tags on return press like in WYSIWYG editors?

IMHO, no: Once you start parsing HTML and try to react accordingly, things become complicated (e.g. multiple levels of nested tags) and you started the route to an full WYSIWYG editor (initially still rendering raw HTML).
IMO, you may consider this route as a separate WYSIWYG ("rich text editor") mode much later; for now the existing solution (editing raw HTML with helper buttons) is absolutely fine.

paragraphs
 are really hard to control properly

I found setting paragraphs explicitly ... necessary many times

As said on the manual page to insert a paragraph you should add an empty line just like here on GitHub. If you want to insert just a line break then start to type from a new line. Look at my screenshots:
This is how I've typed text

And this is how it was composed by OpenRepos

Yeah, I know.
While simple cases (as yours above) are working well, I regularly run into trouble when there are already a couple of tags (i.e. while editing comments).  I have the impression there is something wrong with the automatic tag insertion when submitting comments to OpenRepos (mostly with OpenRepos' web-frontend, as I am using this for years, but seemingly also with Storeman 0.0.12).  But I have never completely understood this phenomenon (i.e. the mechanisms behind it), otherwise I would know how to avoid it and / or be able to file a comprehensive issue description.
P.S.: This is why I ended up writing "plain text" formatted comments in OpenRepos' web-frontend (i.e. not HTML formatted ones using the "plain text editor"), when no markup, links etc. are needed.  But it is  extremely tedious to change that (to filtered HTML format) later on, this is where Storeman comes very handy, now.

Hmmm, I don't know if I should close this issue, as all major points raised in this issue are addressed in Storeman 0.0.13.
So I leave that up to you, how to proceed with the remaining points (e.g. defining them as irrelevant, hence closing this issue).
P.S.:
foo

`bar
foobar`
becomes

foo

bar

foobar

becomes

foo

bar

foobar

just by hitting "Edit", and then "Send" each time.  After that (with only a single paragraph left), it stays the same.

Sorry, I forgot about this issue.
Closing this, as this is working very well for most edits of existing comments, and supposedly is not an issue of Storeman, anyway.
Actually, I have the impression that this is improved already (both, in OpenRepos web-frontend and Storeman using its web API), but currently lack the time to properly test this.
But much more importantly than this: Storeman has become so nice, overall and the comment function specifically.
Thank you so much for your efforts, the result is really impressive.


Only allow HTTPS-enabled sites to be loaded

Currently, Internet Noise loads http and https. However, with HTTPS Everywhere enabled, it eventually stops working. For this reason --and also because it is more secure, I think we should only loads HTTPS-enabled sites.

Fixed
https://github.com/slifty/internet_noise/pull/11

One thought here -- if the purpose is to generate noise in a sense might we actually prefer the fake traffic to be as transparent as possible?
In other words, might we ironically want no HTTPS enabled sites to be loaded?

In other words, might we ironically want no HTTPS enabled sites to be loaded?

I don't know much about the bill being passed or how isps work, but doesn't HTTPS actually prevent any tracking by your isp anyway? There's no data they could sell, other than the IP you connect to. They neither know the URL nor the content of it.

Seems this is correct
So we DON'T want https :)

Indeed @Prinzhorn -- HTTPS means the ISP would be able to tell what site / IP you visit but not what content or what specific page you loaded

@laurenbenichou given the above do you still think HTTPS-only is something you would like to see given the goals around noise (vs around enforcing good HTTPS habits, which HTTPS Everywhere accomplishes)?
It seems that it could be supported at the very least as an option.

It's a one-character change.
inurl:https vs
-inurl:https

Two things:

When HTTPS Everywhere is enabled, Noise eventually stops loading sites.
Noise seems to only loads sites. It doesn't interact with said site --or does it? In that case, what data would the ISP get aside from the IP of the site you've visited? Also currently reading this

Best I can figure, the Google search query is encrypted, the dns lookups are not, and any request to the resulting site, if https, is encrypted. They won't get the search terms or the name of the page loaded, just the domain name lookup.
From the page you referenced: Encryption stops ISPs from simply reading content and URL information directly off the wire.
Maybe a better noise generator would be to do dns lookups?

That's interesting. Yes, maybe a way to make more noise is to look past loading random URLs, but actually loading the data the ISP is looking at.

So just to note: non-encrypted (e.g. HTTP) traffic means that the ISP has access to the actual content of the page. That means they know that someone is reading articles about "home birth" or "private school" or "the flying spaghetti monster" and therefore track life trends and preferences in the same way an advertiser might.  Then they can resell that analysis.
I agree that noise at DNS level would address the less invasive observation, but we shouldn't ignore the HTTP threat which most non tech savvy users are going to be victim to.

The raised issue is important though, and if we think random HTTP traffic is harmful then we should probably convert it to HTTPS! --> AGREED!
And you don't feel like it's harmful, then you should put a disclaimer about disabling HTTPS Everywhere for Noise to work.

I think that HTTPS sites should be removed from the noise. Especially the google searches. Since the isp cant see the actual content of the searches they are pretty useless at generating noise.

Hi, new here.
Instead of disabling HTTPS Everywhere to use Noise, would it be possible to change Noise code to pre-test the URL (cert?) to avoid privacy errors (which I presume is what is breaking the app). Alternatively, and I don't know if this is possible, but could the target tab be checked to see if it currently displaying a privacy error, in which case, close the tab and open a new one.
Again, I don't know anything about the code at this stage so these might be stupid / invalid suggestions.
Thanks again for putting this together though.


Getting ERROR: Challenge is invalid (returned: invalid) : Redirect loop detected

Hello,
I have 8 domains to secure, this was working great for 3 years under acme-v01. I've updated from acme-v01 to acme-v02 and now I get error in the challenge process: Redirect loop detected. It does not break at the same domain, seems random (sometime it's the first one, see log snippet, sometime it's the 3rd, etc.). Is it about my web servers?
When I request URIs prefixed by /.well-known/acme-challenge with curl or wget, I dont see any redirection : it serves a 200 or 404, depending if the file exists or not. There is no http => https redirection. In fact, I can see LE requests in my access logs (200/success) :
<IP_ADDRESS> - - [04/Mar/2021:10:50:05 -0500] "GET /.well-known/acme-challenge/xug085yTPYQlru9nWMymN_jpfLj7PC8n9qWiQ-XXXXX HTTP/1.1" 200 87 "-" "Mozilla/5.0 (compatible; Let's Encrypt validation server; +https://www.letsencrypt.org)" PID=124287 T=8858 XF=[<IP_ADDRESS>] H=<IP_ADDRESS>
<IP_ADDRESS> - - [04/Mar/2021:10:50:05 -0500] "GET /.well-known/acme-challenge/xug085yTPYQlru9nWMymN_jpfLj7PC8n9qWiQ-XXXXX HTTP/1.1" 200 87 "-" "Mozilla/5.0 (compatible; Let's Encrypt validation server; +https://www.letsencrypt.org)" PID=14646 T=2605 XF=[<IP_ADDRESS>] H=<IP_ADDRESS>

dehydrated log (domains/IP/token anonymized):
jeu mar  4 10:49:45 EST 2021
[...]
Processing [domain] with alternative names: [list of domains]
 + Checking domain name(s) of existing cert... unchanged.
 + Checking expire date of existing cert...
 + Valid till Apr  5 15:38:22 2021 GMT (Longer than 30 days). Ignoring because renew was forced!
 + Signing domains...
 + Generating private key...
 + Generating signing request...
 + Requesting new certificate order from CA...
 + Received 8 authorizations URLs from the CA
 + Handling authorization for domain1
 + Handling authorization for domain2
 + Handling authorization for domain3
 + Handling authorization for domain4
 + Handling authorization for domain5
 + Handling authorization for domain6
 + Handling authorization for domain7
 + Handling authorization for domain8
 + 8 pending challenge(s)
 + Deploying challenge tokens...
Calling deploy_challenge() hook for domain1
Calling deploy_challenge() hook for domain2
Calling deploy_challenge() hook for domain3
Copy challenge token on domaine3 via scp...
Calling deploy_challenge() hook for domain4
Calling deploy_challenge() hook for domain5
Calling deploy_challenge() hook for domain6
Calling deploy_challenge() hook for domain7
Calling deploy_challenge() hook for domain8
 + Responding to challenge for domain1 authorization...
 + Cleaning challenge tokens...
 + Challenge validation has failed :(
ERROR: Challenge is invalid! (returned: invalid) (result: {
  "type": "http-01",
  "status": "invalid",
  "error": {
    "type": "urn:ietf:params:acme:error:connection",
    "detail": "During secondary validation: Fetching /.well-known/acme-challenge/xug085yTPYQlru9nWMymN_jpfLj7PC8n9qWiQ-XXXXX: Redirect loop detected",
    "status": 400
  },
  "url": "https://acme-v02.api.letsencrypt.org/acme/chall-v3/113085970000/XXXXXX",
  "token": "xug085yTPYQlru9nWMymN_jpfLj7PC8n9qWiQ-XXXXX",
  "validationRecord": [
    {
      "url": "domaine1/.well-known/acme-challenge/xug085yTPYQlru9nWMymN_jpfLj7PC8n9qWiQ-XXXXX",
      "hostname": "domaine1",
      "port": "80",
      "addressesResolved": [
        "x.x.x.x"
      ],
      "addressUsed": "x.x.x.x"
    }
  ]
})

Using dehydrated v0.6.5
Any help to understand what is going on is very welcome.
Thanks

This can't be a dehydrated issue as dehydrated itself has no control over this, so I'm closing this issue.
I'm not sure what's going on here. From your logs it seems like you have some proxy setup going on, are you sure that all requests are going to the correct backend? Maybe you have a configuration issue with that? Otherwise I'd suggest running tcpdump on affected hosts port 80 while doing the renewal to catch the request and figure out what's going on. Maybe your 200 returns for whatever reason still have a Location header that might confuse the validation logic?

Thanks, i'll focus on our network then.
All the best


Model texturing

Describe the bug
I'm just curious to see if you can get some guidance on marker design. Since the texture of the model depends on the quality of the marker, I want to understand how to do it correctly. I attached screenshots of my marker and model here. There, for example, you can see that there is a black stripe on the back and other defects.
Screenshots

Marker here

I made a video guide on how to create 3D. Please refer to the link.


Update proposal  based on the doc. Update star count

Description

This PR fixes #
Notes to reviewers:
Checklist

[] DCO signed.
[] Title of the PR starts with area name, e.g. meshery:, mesheryctl:, ui:.

@kushthedude I made the proposed improvements. thanks for looking at this

Anything else holding this from being approved?

Anything else holding this from being approved?

Hi @antweiss, I personally feel like to hold it for a day or two more. As we are already at 288 stars and it won't be much amount of time when we will be at 300 stars 🤩 . We can directly update the proposal from your PR & prepare to get into CNCF's sandbox.
@leecalcote What do you suggest?

288! :)) when did that happen!? Sounds good to me.

First of all, I'll suggest that 🤩 I'm seeing nothing but stars. Way to be, @kushthedude! It takes a team here for sure.
And then, yes, I think we can merge here really at any point. There are a few notable items to clear out of the way before submitting this project proposal. We'll undoubtedly have to come back to update stats and make a few statements more boldly in the proposal.
Thanks for your help here. Looking good.

Gents, let’s go ahead and bring these changes in. No doubt we’ll have others in the future. Thanks for advancing this, @antweiss 👍


Add INN and KPP support for ru_RU locale

Add generation INN and KPP numbers for Russian locale.
INN and KPP are Tax identification information for all legal company in Russia

Thanks!


Change async modifier to defer in Dart scripts

An async script executes before the whole page is available (in chrome), but defer waits until the page has loaded. In cases where you want to make sure that all the DOM on the initial page is available from within your main function, you should use defer instead of async.
<script defer src="index.dart" type="application/dart"></script>
<script defer src="packages/browser/dart.js"></script>
Pull requests welcome.

I think this is now fixed


Skip tests that are failing without omnistaging (which will not be an…

… option shortly).
Enable tests on all platforms that were unnecessarily skipped.

Thanks Matt. Is there anything blocking this from being merged?

Thanks Matt. Is there anything blocking this from being merged?

No, sorry!

No, sorry!

I'm not sure, but I think you can start merges too, by applying the 'pull ready' label and then sending the resulting CL to<EMAIL_ADDRESS>If not, ping me on chat and we'll sort out the permissions so that you can.

I'm not sure, but I think you can start merges too, by applying the 'pull ready' label and then sending the resulting CL to<EMAIL_ADDRESS>If not, ping me on chat and we'll sort out the permissions so that you can.


thermald: allow ignoring cpuid check

Description of changes
thermald allows passing an option to ignore the cpuid check, which lets it run on "unsupported" platforms, like Lenovo ThinkPads, which sometimes have thermal control via the firmware.
But, certain Thinkpads do not have proper firmware support, yet it gets deactivated as the Lenovo kernel driver claims support. One example is T480(s).
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2771


DocumentTransform unit tests

Validates bugs #29, #32 as "expected"
These are unit tests for DocumentTransform.TransformAsync which, for a given "document" string validate if it generates document equal to "expected" document string.
In the current state they all pass. I think that it might be a better approach to first create a "working as-is" environment, and then add bugfixes separately.

I'll happily fix these bugs, although I'm wondering what behavior exactly would we expect; which of the trivia should we keep, and which should we drop?
IMHO, we should drop all except pragma, define if/else/endif; those should probably stay, at least for usings.
Second thing is, should namespace-nested usings also be dropped (as is), or should we leave them too?

That all sounds good, with one response:

Second thing is, should namespace-nested usings also be dropped (as is), or should we leave them too?

I don't care too much either way. I would tend toward including them. But as long as the code compiles I don't care too much. :)

After reconsidering the implications, I'm going to drop all trivia, including if/else directives, regions and pragmas. Because generation is expected to be run for each compilation, and a given compilation has a defined set of defines; and also because if-deactivated code parts are trivia nodes in Roslyn Syntax model.

Does this actually fix #29 and #32 now (as opposed to writing tests to expect the prior behavior)?

Yes, yes it does fix those two :) Glad to help.


Wish for your help

Hi @changwoolee ，I‘m VERY interested in your project, you've got a brilliant acheivement.
I also got a zynq-7020 board. But I am a newcomer in this area.
May you tell me how to run your projcet on the board.(in steps)
I'll really appreciate it.
Thanks in advance.

Hello @achiless123 .
You can set the project by using Xilinx SDSoC or SDx IDE. I used SDSoC 2016.4 version.
First, make the new project for your testing board and import the filesystem except Win10 Test App Release & Win10 Test App/*.
Second, set the hardware function  CONVOLUTION_ LAYER_ 1, CONVOLUTION_ LAYER_ 2, and CONVOLUTION_ LAYER_ 3 in lenet5/hw_layers/image_convolution.cpp as Top Hardware Functions at SDSoC.
Third, set the clock frequency as 100Hz for each hardware functions.
Now you can generate a bitstream file.
Additional information is included in https://github.com/changwoolee/lenet5_hls/issues/9
Thanks and have a nice day :)

Hi, @changwoolee
thanks for your adivice
more questions ：
How to use the app on win10
i just followed your introduction on readme.md  but i'm puzzlesd
where may i use the instuction ran .elf and ifconfig
on the ubantu system on the zynq7020 arm?
by the way, i tried to find the .elf file but i failed
thanks for your attention.

I also need to implement this project, but tried hard and failed several times, so I don't know what to do now. Can I help me? thank you very much.


Incorrect URL for MRQA SQuAD train subset

https://github.com/huggingface/datasets/blob/4ef4c8f8b7a60e35c6fa21115fca9faae91c9f74/datasets/mrqa/mrqa.py#L53
The URL for train+SQuAD subset of MRQA points to the dev set instead of train set. It should be https://s3.us-east-2.amazonaws.com/mrqa/release/v2/train/SQuAD.jsonl.gz.

good catch !


CI for QUIC stuff

We don't have any environment for CI builds that can run unit tests for QUIC stuff.
The environment requires ~Open SSL 1.1.1.~ a special OpenSSL made by tatsuhiro-t.

Update: The latest official OpenSSL is not enough. Our QUIC impl. requires the special version made by tatsuhiro-t.

@zwoop I really think we need a CI box that can run tests for QUIC. Otherwise it will follow the same path as BoringSSL support. Ideally, we should have it before merging QUIC branch for 9.0 so that nobody can break QUIC support.
For "quic-20" branch, which will be merged for 9.0, uses "quic-draft-20" branch on https://github.com/tatsuhiro-t/openssl

We are in the process of transitioning to a new CI environment. As a part of that process, we have configured the Centos CI job, which runs make check and the regression tests, to build against the openssl-quic library. For our current centos:8 new CI docker image, that version of openssl quic is:
repository: https://github.com/quictls/openssl.git
branch: OpenSSL_1_1_1k+quic
This is the branch that the quic version of curl presently builds against and what ATS currently supports.
Here is a link to that new CI job:
https://jenkins.trafficserver.org/view/Github/job/Github_Builds/job/centos/
In addition, the AuTest CI jobs will also test ATS built against this same version of openssl-quic. That is already done with the present CI as well.
Thus all the work for this issue is effectively done. However, I'll leave this ticket open until we finish migrating to the new CI system and this job is being run against new PRs.

I did an experiment with the new CI job to verify it functions like we expect it to. A week ago I made the following PR to fix the catch tests when compiled against openssl-quic: https://github.com/apache/trafficserver/pull/7944
I did the following run against master, which has the fix from that PR:
https://jenkins.trafficserver.org/view/Github/job/Github_Builds/job/centos/37/
The test passed, as expected.
I then did the following run on the commit before that PR got merged in:
https://jenkins.trafficserver.org/view/Github/job/Github_Builds/job/centos/38/
That job failed, as expected since there was a compilation issue with the openssl QUIC unit tests:
In file included from ./test/test_QUICFlowController.cc:27:
../quic/Mock.h:198:3: error: 'int MockQUICConnectionInfoProvider::select_next_protocol(SSL*, const unsigned char**, unsigned char*, const unsigned char*, unsigned int) const' marked 'override', but does not override
   select_next_protocol(SSL *ssl, const unsigned char **out, unsigned char *outlen, const unsigned char *in,
   ^~~~~~~~~~~~~~~~~~~~
../quic/Mock.h:516:3: error: 'int MockQUICConnection::select_next_protocol(SSL*, const unsigned char**, unsigned char*, const unsigned char*, unsigned int) const' marked 'override', but does not override
   select_next_protocol(SSL *ssl, const unsigned char **out, unsigned char *outlen, const unsigned char *in,
   ^~~~~~~~~~~~~~~~~~~~

This demonstrates that the new CI job is compiling the unit and regression tests against openssl-quic as we desire and is capable of finding issues with openssl-quic builds.
Again, as mentioned in the previous comment, I'll leave this issue open until we transition to the new CI system and new PRs are run against this job.

Thank you for setting up the CI environment.

In addition, the AuTest CI jobs will also test ATS built against this same version of openssl-quic. That is already done with the present CI as well.

Does this mean we can run autest test cases that requires curl that supports HTTP/3?
https://github.com/apache/trafficserver/blob/9e7cece7641b282a76fb95798f29e4cd39b51bc8/tests/gold_tests/timeout/active_timeout.test.py#L67-L70

Wow, I'm glad you asked this.
The answer is yes, we can run those tests. And indeed when I run it locally with our new CI's centos:8 docker image I run that test:
$ ./autest.sh --sandbox /tmp/sb/ --clean=none --ats-bin ~/build/ts_asf_master_test_quic_build/bin/ -f active_timeout
Python 3.6 or newer detected!
python3-dev/devel detected!
pipenv detected!
Using the pre-existing virtual environment.
Environment config finished. Running AuTest...
Running Test active_timeout:.... Passed

It's subtle, but notice the four dots. That fourth dot corresponds to the HTTP/3 test you describe.
I had done work to make that happen, and I thought it would carry over to the new jenkins server. But since you asked, I went to grab it's output to prove in this issue that it was running with the QUIC version of curl but noticed it had only three dots with the latest invocation. Working with @ezelkow1 , we got the PATH environment updated in the jenkins shell so that it builds with and runs the AuTests with the openssl-quic version of curl. Here's the latest run, with that now working:
https://jenkins.trafficserver.org/view/Github/job/Github_Builds/job/autest/72/console
Running Test active_timeout:.... Passed

Again, notice the four dots, indicating that the HTTP/3 test was run.
Thanks again for the thoughtful question.

Awesome, thank you.

Closing this since we are now using the new CI which is running with HTTP/3. And the latest autest jenkins run still shows the HTTP/3 test running:
https://ci.trafficserver.apache.org/view/Github/job/Github_Builds/job/autest/511/console
Running Test active_timeout:.... Passed


40 Node GPU Cluster required??

Exactly 8 GPUs are required due to the mesh shape assertion (1, 8). To run at native size, you will likely need 8x80GB GPUs (A100 80GB / H100GB). I think with HIP, you can also use AMD GPUs and translate the compiled CUDA to ROCm. The MI200 series will likely be needed to meet the VRAM spec for mixed precision.

Request you to move to this discussions thread here: https://github.com/xai-org/grok-1/discussions/183
Please close this issue and move to:
https://github.com/xai-org/grok-1/discussions
Reason:
https://github.com/xai-org/grok-1/issues/69
https://github.com/xai-org/grok-1/issues/108


Improve search engine preview

There's a message about enabling Javascript. Maybe we can add a generic description somewhere.

Google search in a new private window shows this for me:

Yep, looks good now!


Generate the plots and output

Generate the plots and show the top 5 players in both directions. Where do most players land and how big is the difference, i.e. compute mean and variance for the difference between the NBA 2k rating and our metric.

Added to the report notebook, possible future work noted in last paragraph


refactor: stricter linting rules

Closes #135

@rultor merge it.

@rultor merge it.

@rafamizes OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge it.

@rafamizes Done! FYI, the full log is here (took me 2min)


Respond to Iteration and Release or Milestone Filtered pages

This would be if the chart is placed on an iteration or release or Milestone filtered page, the data in the chart would also be filtered.

Wow, oops!  Almost all of the code is already written to support this.  I just fat fingered a function name: https://github.com/RallyCommunity/CustomChart/blob/master/CustomChartApp.js#L183
I'll get a new version pushed out soon!

Should be fixed now.  Please let me know if you're still having issues with it!

Thank you!


Replace the update ETAs example with an example to update RSAs

Since ETAs are now deprecated and RSAs are the suggested replacement.

Is the plan to add this example and also update https://developers.google.com/google-ads/api/docs/ads/mutate-ads?

Sure, I'll update the docs once this is merged.

Sounds good. Thanks for updating the docs.


exec sleep error

hi here,
while trying to analyze for testing the docker image traefik:latest, i had these strange bugs comes up while using dagda:
dagda     |
docker.errors.NotFound: 404 Client Error: Not Found for url:
http+docker://localunixsocket/v1.28/containers/10851416f569e8489dcc9006baec6751c68e76eeeafb7a233319ff37d6916f32/start ("oci runtime error: container_linux.go:247:
starting container process caused "exec: \"sleep\": executable file not found in $PATH"

anyone knows how to fix perhaps ?
thanks alot !

Traefik uses scratch as its Linux base image as far as I see in its Dockerfile.
At the moment this Linux base image is not supported by Dagda, so if you want contribute, the PR will be welcome.
Anyway, I have improved the error control system for avoiding that Dagda crashes when unexpected error occurs by not supported Linux base images.

thanks a lot !
seems better now
i'll see if i can contribute a PR :)

I'm glad to know that issue seems fixed. I close this issue then.
If you find a new issue, feel free for reporting it ;-)


Changes to support NSubstitute for mocking.

While OrleansTestKit team decides on real adaptability for supporting multiple mocking frameworks, I've added changes to support NSubstitute for mocking.
To build a package for NSubstitute:
dotnet pack -c Release -p:Flavor=nsubstitute

To build package for Moq:
dotnet pack -c Release

This is not best and elegant solution for maintenance preceptive, but I will try my best.

Sorry this didn't get traction. Unfortunately, I don't think this is something I can take on maintenance-wise. I hope we can revisit the mocking story in the not-too-distant future.


Trezor error message when sending from address.

I get this error message when trying to send BCC from a address on my Trezor (signing with my Trezor).(I'm trying to split my coins.)
I keep on getting this error -- 'NoneType' object has no attribute 'update_status'

are you running from source of binary?

Binary

plese try 2.9.3 (just out) and report back if you still have the issue

Still the same issue :(

The error ('NoneType' object has no attribute 'update_status') points to old python-trezor being used if I recall correctly, but I don't understand how that's possible when using binary. Any ideas?


Recipe list

STATUS: APPROVED 🟢
Hi @Ellyboi, your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉


Icon of the six-select gets selected when double clicking on the select

Six Webcomponents version
4.1.2
Steps to reproduce
double click on a six-select
What is expected?
Icon not being selected
What is actually happening?

System Info
No response
Any additional comments?
No response

@davidemarcoli Please check if it still happens with the newest version 4.2.2

Does not happen any more.


Wagtail 5.0 Upgrade

Maintainer has inquiry: https://github.com/neon-jungle/wagtail-schema.org/pull/20#issuecomment-1588645773
Waiting for his reply

This has been merged


CRD delete hangs forever if finalizers are hung

Running microk8s 1.20.1 - and seeing the EXACT same issue as https://github.com/kubernetes/kubernetes/issues/60538
when installing argo-cd - and then installing a "root" application..  trying to delete it hangs and then trying to remove the application CRD - which then hangs forever.
manually editing CRD and removing finalizers: made it get unstuck (exactly as specified workaround in kubernetes issue).

And when I then try to reinstall the CRD (helm install argo-cd) - the old hung "application" of this CRD - is back again.. so removing a CRD does NOT delete "instances of the CRD".. it really SHOULD :)

the CRD instance (an application) - also hung on deletion - until I removed finalizers from that as well.
It seems handling finalizers timing out properly is the central isue.

Indeed @KlavsKlavsen. I often find the order in which resources are deleted important.

I don't think there is something MicroK8s can do here.  I see this behavior too on other distro that I've used.  Im closing this unless we think there's more we can do.


Cache results from resolveEntityType

Signed-off-by: nikolay<EMAIL_ADDRESS>Description:

Related issue(s):
Fixes #853
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Base: 73.82% // Head: 74.01% // Increases project coverage by +0.19% :tada:

Coverage data is based on head (5aef87b) compared to base (9b23b7b).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #862      +/-   ##
==========================================
+ Coverage   73.82%   74.01%   +0.19%
==========================================
  Files          29       16      -13
  Lines        1780     1301     -479
  Branches      326      237      -89
==========================================
- Hits         1314      963     -351
+ Misses        372      274      -98
+ Partials       94       64      -30

Impacted Files
Coverage Δ

packages/relay/src/lib/clients/mirrorNodeClient.ts
96.26% <100.00%> (+0.18%)
:arrow_up:

packages/server/src/validator/types.ts

...s/server/src/koaJsonRpc/lib/methodConfiguration.ts

packages/server/src/koaJsonRpc/lib/RpcResponse.ts

packages/server/src/server.ts

packages/server/src/validator/utils.ts

packages/server/src/koaJsonRpc/lib/RpcError.ts

packages/server/src/validator/constants.ts

...kages/server/src/koaJsonRpc/lib/RpcInvalidError.ts

packages/server/src/rateLimit/index.ts

... and 4 more

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.


add target _blank to links

 Update content links

Nice 👍

This will happen to all link all the time?

Yea, double checked this is what we want going forward and the answer was yes.

Cool :+1:
On Mon, Nov 7, 2016 at 12:38 PM, Andrew Kos<EMAIL_ADDRESS>wrote:

Yea, double checked this is what we want going forward and the answer was
yes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/theonion/editor/pull/44#issuecomment-258922617, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAeZ4uzZu9_5nwYaJW1dn6UpLtuRysaks5q73BCgaJpZM4Krgps
.


initial Container implementation

part of #3183
implements the beginnings of the new Container schema:
container {
  from
  rootfs
  workdir
  withWorkdir
  variables
  withVariable
  exec
  exitCode
  stdout
  stderr
}

also: tweak the handling of scratch directories so it can be re-used for scratch containers. previously marshaling llb.Scratch() and then unmarshaling it back into a st llb.State would cause an error when calling st.Marshal. now we'll just set the definition to nil in the ID DirectoryID payload.

Hm, yeah, not sure what's going on with this timeout. It happened on re-run too, but it's not happening locally. I'll keep poking at it!

For continuity: it looks like the test timeout is genuine, and now that we have more tests we're exceeding the Go default 10 minute full-suite timeout. :sob:
This seems more to do with our runners having pretty limited parallelism than anything else, so I've just bumped the timeout.


npm run build error

Node version:
6.2.0
npm version:
3.9.2
Operating system:
windows 7
Command line used:
npm run build
Steps to reproduce:
All test were passed, but running "npm run build" or "npm run prebuild" throws the error and it can build the files. http://take.ms/qXh6D

Move the project to a directory that doesn't contain a slash. That should fix it.

Actually, after looking again, it appears the path you're running in doesn't contain spaces. I suggest downgrading to Node 5.11.1. Node 6 has issues on some Windows machines.

After downgrading to node 5.11.1, it's still having the same error.

even the most recommended version of node 4.4.5, throws the same error

Why are you running this project from the xampp folder? Try running this
from a different directory. Make sure there are no spaces in the path.
Also, try running just the lint task. Does it fail? If so, with what?
On Jun 22, 2016 3:04 AM, "gitlovenotwar"<EMAIL_ADDRESS>wrote:

even the most recommended version of node 4.4.5, throws the same error
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/coryhouse/react-slingshot/issues/170#issuecomment-227671886,
or mute the thread
https://github.com/notifications/unsubscribe/ABnFpUu1JvUXJFVAKFnfjpqIRrzCxW7dks5qOOyXgaJpZM4I5S05
.

It's not running on xampp, I just happen to place all my project into one folder, which is in xampp :) Also, after running lint, it throws the same error. So, I guess the linting gives this error.

@gitlovenotwar Can you please share your solution so I can try to reproduce?


[SPARK-40098][SQL][FOLLOWUP] Revert the pretty format of error messages in the Thrift Server

What changes were proposed in this pull request?
In the PR, I propose:

Output errors in the PRETTY format in the same way as before the PR https://github.com/apache/spark/pull/37520.
Do not output non-JSON elements in the MINIMAL and STANDARD formats.

Why are the changes needed?

To not break existing apps that might expect text errors in particular format.
Do not output extra text when the Thrift Server outputs errors in an JSON format.

Does this PR introduce any user-facing change?
Yes.
How was this patch tested?
By running the modified tests:
$ build/sbt -Phive -Phive-thriftserver "test:testOnly *ThriftServerWithSparkContextInBinarySuite"

Merging to master. Thank you, @HyukjinKwon for review.


Add register device cmd an alias

This PR adds a simple alias to the original --register_device. The alias --register-device complies to the Unix flag conventions.
Once again, thanks for the great software! :+1:

Just realized that the --register-device flag isn't anywhere in the README.  Would you mind adding a blurb about it for this PR?


Title buttons issue in Windows 11

This happens when you put the cursor on the title buttons
Info:
Firefox 101.0 (64-bit)
Windows 11 64-bit 21H2 (22000.708)

Hello. If you're talking about duplicate window controls, there's nothing I can do about it. The ones that appear on hover are the browser buttons. Others are rendered by the operating system itself. Although the system buttons are drawn, they are not clickable. You cannot interact with them. Only Mozilla can fix this.

Hey QNETITQ, I just wanted to contribute this.  I had to deal with this issue a long time ago and basically we just need to hide the buttonbox elements.  I'm not that experienced in CSS, but I managed to find a decent alternative to correct this problem.
In your Windows_11_10.css file, I modified opacity: 0 !important; and added color: transparent !important;
In doing this, it removes the hover effects (you may have a different solution), but it keeps the ugly double-elements from appearing.  The buttons remain functional.  Thank you for all of your very hard work!!
Here is the code I modified:
:root:is(:not(:-moz-lwtheme), [lwt-default-theme-in-dark-mode]) .titlebar-buttonbox:hover
            {
                opacity: 0 !important;
                color: transparent !important;

            }

            :root:is(:not(:-moz-lwtheme), [lwt-default-theme-in-dark-mode]) .titlebar-min
            {
                border-bottom-left-radius: 8px !important;
            }
        }
    }

For various reasons, it's better not to do this. There are problems in scaling mode.

I'm not sure why you feel that turning off opacity of an element would cause issues with scaling.  Especially since your own code presents issues with scaling.  I'm on a 4k monitor with 175% scaling and I have no issues whatsoever with the modification I made.  That said, it is your code, and I will just modify my own copy.  Thanks again for your hard work on this project.

If you find scaling issues, please report them. As for the window control buttons, I will think again about their implementation, but I do not promise anything.

Fix here for Windows 11; https://github.com/QNetITQ/WaveFox/issues/113#issuecomment-1474925989


update

[Enter feedback here]
it was not updating
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 9c82359b-d6a0-4f42-a6e1-a5e83f1edfeb
Version Independent ID: 9c82359b-d6a0-4f42-a6e1-a5e83f1edfeb
Content: Get support for Partner Center dashboard issues - Windows drivers
Content Source: windows-driver-docs-pr/dashboard/hardware-submission-support.md
Product: windows-hardware
Technology: dashboard
GitHub Login: @anaharris-ms
Microsoft Alias: anaharris

Not seeing any actionable feedback - please reopen with details as needed


drive redirection - raise an error if the directory can't be founde

Currently one can pass any path to /drive even invalid ones. The code handles those errors correctly however if the directory is invalid the redirected drive would just be empty on the server side. The parser, or channel, should at least print a warning when the redirected path is invalid.

Hi,
Allowing the /drive string to map to invalid directories (in our case it is /drive:'USB,/mnt/usbdevice') gave us the option for users to connect with or without a usb drive plugged in. Now they can't connect without a drive inserted. Is there a work around to allow access to /mnt/usbdevice regardless of whether there is a drive plugged in or not?


DATETIME column with sqlite driver doesn't save datetime in proper format

I am using Mikro-ORM 5.1.1 with sqlite, I have an entity with some DATETIME columns, the entity is like:
class Asset {
  @Property({ type: Date, columnType: 'DATETIME', nullable: true })
  modifiedDate?: Date;
  ...
}

When I create a new entity and save to database file, I have code like:
const asset = repository.create({modifiedDate: new Date()})
repository.persist(asset)
repository.flush()

I enabled debug and I can see the SQL query is like:
insert into `assets` (`modified_date`) values<PHONE_NUMBER>000)

The expect result should be something like:
insert into `assets` (`modified_date`) values ('2022-04-07 12:03:01')

I noticed that sqlite doesn't have a real datetime format, so it is actually a TEXT column. The question is, can I define my entity as Date so that mikro automatically convert the Date to the string format. Thank you!
Versions

Dependency
Version

node
v16.14.0

typescript
4.6.3

mikro-orm
5.1.1

your-driver
sqlite

Problem resolved by defining my own custom type.


Use debhelper v10

Hi Michael,
maybe we should Build-Depends on the latest debhelper compatibility level.
Regards,
Tobias

Thanks!


OverviewMap migration v5.3 --> v6.3

OverviewMap regression
Actually we migrate OL v5 to v6.3 but no layer showed in the overViewMap.
I fin in a note in release
The OverviewMap requires a list of layers.
Due to the constraint above (layers can only be added to a single map), the overview map needs to be constructed with a list of layers.

My question is how can display the layers in both Map and overViewMap in the sameTime,
N.B: I know from the doc and samples that we can create layers for each separately.
Man thanks.

You cannot use a layer in both maps, but you can use the same source in two layers, as in the example https://openlayers.org/en/latest/examples/overviewmap.html

Thanks for your response @mike-000
I know that we can use the same source but I need to create TileLayer, and I've  many layers and layerGroup.
Have we a way to clone all Map layers to use them in the OverviewMap ?
I already tried to use lodash _. cloneDeep  also the spread operation, but not working because


Better deprecation for vocabularies moving to rdf-vocab

In advance of a 2.0 release, better deprecate the use of vocabularies, such as RDF::DC by leveraging the autoload feature (with some filtering) to issue a deprecation whenever such a vocabulary is loaded intentionally.
Note that RDF::Vocab.each causes all vocabularies to be loaded, which should probably stifle such warnings.

Useful article on the topic: http://urbanautomaton.com/blog/2013/08/27/rails-autoloading-hell/.
Either implement const_missing? to do this and provide deprecation notices, or autoload an intermediate file which loads the actual vocabulary.

Reopened to ensure proper changes made to 2.0 branch too.

Integrated in b62201462eb8b6fdacaf8d366d5c2f7cb263ea6b on release-2_0 branch.


Add r2pm package

Created, but missing some dependencies:
sudo apt-get install dh-autoreconf

Thats debian specific. Add a chk in shellscript to ensure the program is in path, other wise suggest to install it with an echo msg

On 11 Sep 2017, at 12:17, Antonio Sánchez<EMAIL_ADDRESS>wrote:
Created, but missing some dependencies:
sudo apt-get install dh-autoreconf
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


RecyclerView — More Animations with Less Code using Support Library ListAdapter

Read this, it might be helpful.

Thank you @meness, but everything was already covered more than 2 years ago in this Adapter. DiffUtil was also implemented 1 year ago.


[BUG] WRITE_EXTERNAL_STORAGE permission need maxSdkVersion="29"

📝 Describe the Bug
Provide a clear and concise description of what the bug is.
📖 Library Version

PDF Viewer Version: [e.g., 2.0.0]

🚶 Steps to Reproduce
Please follow these steps to reproduce the issue:

Initial Setup/Library Method Used: [e.g., initWithUrl method]
Actions Taken: [e.g., Clicked on 'Load PDF']
Unexpected Behavior: [e.g., PDF not loading]
Relevant Code/Stack Trace: [Include code snippet or stack trace if applicable]

🤔 Expected Behavior
What did you expect to happen?
🖼 Screenshots/Videos
If applicable, add screenshots or videos to help explain your problem.
📱 Smartphone Info:

Device: [e.g., Samsung Galaxy S10]
OS: [e.g., Android 11]
Library Version: [e.g., 2.0.0]

💻 Code Snippets
Include relevant code snippets or links to code repositories.
📜 Log Output/Stack Trace
Include any error logs or stack trace for more context.
💬 Additional Context
Add any other context about the problem here. Mention if the issue is consistent, specific scenarios where the bug occurs, etc.

Please provide/add more detail to actual issue. Closing it for now, feel free to reopen.


Add interface to allow extracting remark information

Allows setting an environment variable to dump LLVM remark information for later analysis.

@apmasell Thanks for the patch. Was wondering if it would be preferable to expose remarks information in the function's per-signature metadata dictionary? I think it'd be possible to collect information from both the "cheap" and "full" module optimisation passes, as well as the function level optimisation passes. Perhaps something to discuss at the next public Numba meeting?

We could. The goal of this was to make something that could be fed into the external LLVM tools with little effort for our own debugging rather than something user friendly. Also, we don't know what the memory cost of these will be for larger programs.


running multiple bam files

Hello
I tried to run the snakemake as below but I didn't get the outpust as expected. any advice? thanks :)
snakemake -c 1 -s /ibseq/MMMVI/voc-identify/voc-identify.smk -d reports/wwc019 --configfile example-config.yaml --jobs 2
Creating specified working directory reports/wwc019.
Building DAG of jobs...
Using shell: /bin/bash
Provided cluster nodes: 2
Job counts:
count   jobs
1       all
1
[Tue Jun 22 22:04:19 2021]
localrule all:
jobid: 0
[Tue Jun 22 22:04:19 2021]
Finished job 0.
1 of 1 steps (100%) done
Complete log: /ibseq/MMMVI/reports/wwc019/.snakemake/log/2021-06-22T220419.690500.snakemake.log

You would see this result if you ran snakemake in a project directory the (the -d flag) that doesn't match the directory structure given in the readme under the heading Automated Workflow


use github actions to build injector image

Tested here: https://github.com/WanzenBug/piraeus/runs/881227632
(fails because registry user is not configured)

Forgot to put v in front of the docker tags... updated action log: https://github.com/WanzenBug/piraeus/runs/881272016

Also forgot to update the makefile... brb

I think it misses the --build-arg bits in the Makefile, if for whatever reason one builds it manually. This is one reason I prefer one source of truth, which in this case would be the Makefile

@rck please re-review. Basically moved everything into the makefile. The github workflow also just uses the makefile.

jop, I like that version, thanks for adapting.


Add NII-SRC Corpora

http://research.nii.ac.jp/src/en/list.html

Obtaining SRC corpora
Please send an e-mail to SRC secretariat with the information below.
Speech corpus/corpora requested
Name
Affiliation
Title (Professor, Researcher, Student, etc.)
E-mail address
Postal address
Telephone number
Purpose of corpus usage  (e.g. "Research in blind source separation and source localization", "To train a speech recognition algorithm".)

The application form and necessary information will be sent to you.
You will be requested to submit a Letter of Pledge for each corpus (an example).
Secretariat
Speech Resources Consortium, National Institute of Informatics
2-1-2 Hitotsubashi, Chiyoda-ku, Tokyo 101-8430, JAPAN
E-mail<EMAIL_ADDRESS>


App crashes when initial state for tabs is empty

Android
<EMAIL_ADDRESS><EMAIL_ADDRESS>Hello.
In DynamicExample, if set empty list to "tabs":
getInitialState() {
  return {
    tabs: [],
  };
}

and try to change state, app crashes with error:
setState(...): Can only update a mounted or mounting component.

Thanks!

Faced similar issue, workaround is
{ tabs.length ? <ScrollableTabView ...> : <ActivityIndicator ...> }


Filtering certain source results

https://www.reddit.com/r/neovim/comments/o5siul/is_there_a_preferred_way_to_disable_lsp/
I am basically trying to do what the OP in the above post is. However I wish to do that for all servers and not only lua.
I want to filter out the text type completions coming from the server.

I too am trying to do the same thing!

In this image, if I try and call console.log({ command }); the actual constant that I want is towards the bottom of the list, with a load of unnecessary Text values above it.
I don't have a very complex cmp setup, so I'm surprised that it's acting in this way.

I will seek assistance over there!

In my case, it was the fact that I had the "spell" source enabled through null-ls which was producing those suggestions.
Turning that off (at least in the file types where I don't want it) sorted my issues.

Take this javascript source:
import { html, render } from 'lit-html';

render(html`

I'm a little teapot

`, document.body);

With the cursor on the line below the
 tag, I'd like to receive emmet completions from emmet_ls, but when the cursor is on the line beneath the import statement, I certainly would not like to receive emmet completions.
I would like the ability to filter lsp snippets by treesitter context.


Add diff command

What is the purpose of this change? What does it change?

The commits in this PR adds the diff command that compares two snapshots and prints a line for each difference, similar to what the Unix diff utility does.
Was the change discussed in an issue or in the forum before?

https://github.com/restic/restic/issues/11
https://github.com/restic/restic/issues/1460
Checklist

[x] I have read the Contribution Guidelines
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's an entry in the CHANGELOG.md file that describe the changes for our users
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[ ] I'm done, this Pull Request is ready for review

Closes #11
Closes #1460

Codecov Report

Merging #1462 into master will decrease coverage by 0.42%.
The diff coverage is 4.16%.

@@            Coverage Diff             @@
##           master    #1462      +/-   ##
==========================================
- Coverage   52.74%   52.32%   -0.43%
==========================================
  Files         136      137       +1
  Lines       10859    10952      +93
==========================================
+ Hits         5728     5731       +3
- Misses       4257     4347      +90
  Partials      874      874

Impacted Files
Coverage Δ

internal/restic/node.go
41.95% <0%> (ø)
:arrow_up:

cmd/restic/cmd_diff.go
4.3% <4.3%> (ø)

internal/archiver/archiver.go
64.81% <0%> (-0.22%)
:arrow_down:

Continue to review full report at Codecov.

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

Sample output:
$ ./restic diff ea 2a
password is correct
comparing snapshot ea657ce5 to 2ab627a6:
 C   /restic/cmd_diff.go
+    /restic/foo
 C   /restic/restic
Files:           0 new,     0 removed,     2 changed
Dirs:            1 new,     0 removed
Others:          0 new,     0 removed
Data Blobs:     14 new,    15 removed
Tree Blobs:      2 new,     1 removed
  Added:   16.403 MiB
  Removed: 16.402 MiB

Codecov Report

Merging #1462 into master will decrease coverage by 0.99%.
The diff coverage is 2.17%.

@@           Coverage Diff            @@
##           master    #1462    +/-   ##
========================================
- Coverage   52.74%   51.75%    -1%
========================================
  Files         136      137     +1
  Lines       10859    11040   +181
========================================
- Hits         5728     5714    -14
- Misses       4257     4457   +200
+ Partials      874      869     -5

Impacted Files
Coverage Δ

internal/restic/node.go
41.95% <0%> (ø)
:arrow_up:

cmd/restic/cmd_diff.go
2.2% <2.2%> (ø)

internal/checker/checker.go
67.55% <0%> (-3.78%)
:arrow_down:

internal/archiver/archiver.go
64.81% <0%> (-0.22%)
:arrow_down:

Continue to review full report at Codecov.

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

Hi Alexander
tested the diff command with version v0.8.0-44-g06cb3f70 amd64
a new directory is shown with flag "+" but the new files inside are not recognized
Kind regards
Joerg

Yes, I thought it is sufficient to show that the directory has been added. Do you think it's valuable to traverse into the newly added dirs?

in my opinion? yes it is ;)
the new files in B are not part of A so they should be listed by diff
IMHO it is very useful to get a list of changed/new files after each backup run.

Ah, hm, you're right, and I think there's even a bug in the diff command: Files in added/removed subdirs aren't included in the stats. I'll have a look later today.

You were right, there was indeed a bug. So diff now descends into added/removed dirs and prints them, the PR is here: https://github.com/restic/restic/pull/1471


Disable metrics across tests and run in parallel

This disables metrics serving within tests which means they can then be run in parallel.
It then adds the p flag to ginkgo in make test to make tests run in parallel automatically using (I think) all but one thread on the machine.
Hoping this will make tests run faster within CI though not sure what the real impact would be, opening this PR as a sort of test of the performance

@JoelSpeed: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pull-wave-test-1.14
dbfcead999fcdce0a56834afb8d1ca59e6b8fa59
link
/test 1.14

Full PR test history

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JoelSpeed: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pull-wave-test-1.14
10f4f1da6b62524f49b3328df49505dd9c019e58
link
/test 1.14

Full PR test history

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JoelSpeed: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pull-wave-test-1.14
10f4f1da6b62524f49b3328df49505dd9c019e58
link
/test 1.14

Full PR test history

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Octokit.AuthorizationException in dotneteng-status

https://dev.azure.com/dnceng/internal/_build/results?buildId=2575453&view=results didn't open an issue. Investigate

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

I wasn't able to make much progress here. The only secret we've had to cycle recently is in https://dnceng.visualstudio.com/internal/_git/dotnet-helix-service?path=/.vault-config/helixprodservice.yaml&version=GBmain&line=170&lineEnd=177&lineStartColumn=1&lineEndColumn=1&lineStyle=plain&_a=contents. The secret specifies only the private key should be cycled, so that's what I've been doing.
It's possible that the secret was cycled successfully and this isn't related, but that particular secret secret seemed to be used for this particular endpoint so there might still be something there.

It does not repro always though

I think I found a clue
If we also summarize by operation_Name ( query) most of the failed requests are for operation GitHubHook/IssuesHook
Looking at which GitHub issues are mentioned in the traces
All the failed requests are processing issues in dnceng - query
The successful requests are for other repos - query

@dkurepa updated Maestro recently to use Octokit.Webhooks.AspNetCore - https://github.com/dotnet/arcade-services/pull/4111
@riarenas would you consider doing the same change in dnceng repo, we think it will help

Whatever fixes the issue sounds good to me

Could you summarize how the move to the new library would help with these exceptions?

https://github.com/dotnet/dnceng/issues/4299#issuecomment-2474336930 was posted today from the staging enviroment, which makes me question why a library update would be needed to solve this.
Are we sure it's not just a production secret that isn't working properly?

This is the issue opened in arcade-services mentioning that the current library is deprecated https://github.com/dotnet/arcade-services/issues/3527
I'd need more time to understand the change in authentication, looks like in Maestro there is a different secret used instead of installation and private key
The library handles payload parsing and integration with webhook events  - one of our theories is that the service might be getting incorrect payload

I used secret manager to cycle https://github.com/dotnet/dnceng/blob/main/.vault-config/shared/dotneteng-status-secrets.yaml#L1 and restarted the service.
I verified the alerting functionality is working again in https://github.com/dotnet/dnceng/issues/4449 and that the issue mirroring from azure devops to github is working in https://github.com/dotnet/dnceng/issues/4450.
I have a theory on what happened and I will add more details after I check that out tomorrow.

When cycling the secret used in https://github.com/dotnet/dnceng/issues/4405#issuecomment-2464853606 and the secret used in https://github.com/dotnet/dnceng/blob/main/.vault-config/shared/dotneteng-status-secrets.yaml#L1 both mention that older private keys should be deleted, but both secrets need to have an up to date private key. I deleted one key too many when I cycled the helix service secret.


Store original system UID in API

Discussed at the SME meeting 30Jul24:
It would be helpful to be able to store the original UID from the system sending/creating the information. The API now includes system and systemVersion. Can we add UID to that?

Could add a "system reference" attribute to top level in API alongside systen name and version?


Release note v3.0.0 sign off - Security

Looking for a sign off on the release notes for v3.0.0 in the area of Security

Release channel recommending closing the issues.


Custom IMAP Account : IMAP and SMTP throwing feilds throwing error when address has extra space

While creating a custom IMAP Account, Nylas is throwing errors when there is an extra space in end of the line. (--.com ) instead of (--.com)
Also, the error is too complex to understand. Attaching screenshot.

thanks for the report! this should be fixed in the next release

this should be fixed on 1.0.26 which was released today! Thanks for your patience!