Cherry pick logging refactor from primedev

Honestly I only half like this refactor, I would prefer if we used the {} style of formatting stuff among other things.
Anyway this PR isn't about my opinion so whatever

cant the {} formatting be archived with std::make_format_args and std::vformat we are on cpp 20 anyways

cant the {} formatting be archived with std::make_format_args and std::vformat we are on cpp 20 anyways

Yeah, this was just me porting the refactor over without any changes

cant the {} formatting be archived with std::make_format_args and std::vformat we are on cpp 20 anyways

The {} formatting comes from fmtlib which spdlog uses so you can very easily port to it if you want.

If i was working on refactoring logging now I'd probably have Msg, DevMsg, Warning, Error, FatalError functions where DevMsg would be stubbed in release.
Also having logging levels behind a general convar is a good idea.

If i was working on refactoring logging now I'd probably have Msg, DevMsg, Warning, Error, FatalError functions where DevMsg would be stubbed in release.

I definitely agree about the separation between Error and FatalError, because having to pass NO_ERROR to every error log to not have the program terminate is annoying

I would prefer if we used the {} style of formatting stuff among other things

100% agree on that ^^"

So how do we approach this? Change %s back to {} before merge or merge first and then in a second PR revert back to {}?

So how do we approach this? Change %s back to {} before merge or merge first and then in a second PR revert back to {}?

Id say changing it back would be beyond the scope of the cherry pick.
Also changing it back now would make future primedev integrations more annoying

I'd so, stick with %s and then change it back later once we got primedev updated?
I'm fine with whatever gets us further faster, even if that means having a temporary "regression" [^1]
Could test late tonight. If someone else could give it a try earlier than that, would be perfect so that we can get it merged ASAP. ^^
[^1]: Obvious remark that this is not a "regression" in the sense of that it breaks anything but it in the sense that it's style/behaviour that we might not necessary be satisfied with.

I'd so, stick with %s and then change it back later once we got primedev updated? I'm fine with whatever gets us further faster, even if that means having a temporary "regression" 1

Yup works for me, Fifty might even make some more logging refactors between now and then anyway on primedev given the conversation thus far

Man i wish i could compile locally right now instead of having to do everything inside github

Game still seems to function fine from the testing I did
Was able to launch the game to the menu fine, as well as play a small amount of gun game
A log from successful menu launch:
message.txt
A log from having an error appear (purposefully, to test it):
error.txt
Things I have feedback on (though I understand this is simply taking from primedev, and not necessarily nitpicking stuff)
It should also be noted that I have the perspective of a helper and explaining to end users for this, and less of a development outlook
Milliseconds for the timer in the logs is most likely unnecessary
Milliseconds for the timer in the folder for the logs is most likely unnecessary
(should be noted that neither of these are bad, however it feels like it isn't needed and just takes up more space on screen. in the log, I suppose the milliseconds could matter, but it becomes a "will end users understand why there's more numbers" scenario)
The errors not logging the mods really makes it not useful in a lot of scenarios however. A lot of the compile issues are because of missing dependencies, or conflicting mods. While some you can guess, having the mods in there at all makes the process 100x faster, especially accounting for explaining to the end user. Needing two logs to diagnose one issue is a little annoying. If the errors were still inside the full log, I would say it doesn't matter as much and just ask for the full log, but they seem to be omitted from the full log (or at least, from what I see. I'll attach the log with an error i created for testing, the same as in the previous error log)
message.txt
Apart from the issues I have (which, again, I realize is likely for a later date, easier to write them out here and reference/copy paste later than think of the same things twice), this pr does seem to work completely properly in game. No idea code wise
I believe the only public log reader is Spectre, which I'm already rewriting due to the code being bad, so I'll rewrite with this in mind. Logging the mods in error.txt (or logging the errors in message.txt) would make this much easier for Spectre and for helpers

I believe the only public log reader is Spectre, which I'm already rewriting due to the code being bad, so I'll rewrite with this in mind. Logging the mods in error.txt (or logging the errors in message.txt) would make this much easier for Spectre and for helpers

I don't remember the old logging printing out the installed mods on a script compile error? Although i guess your point is that people will start sending just the error log, which wont contain any mod information.
I don't see why we couldn't log the installed mods on a compile error, other than it maybe drowning out the actual error in a sea of mod prints.

@itscynxx could you give it a test with a proper crash? like an access violation or something. There are some changes that i'm not 100% sure on in the crashhandler which would be nice to test

@itscynxx could you give it a test with a proper crash? like an access violation or something. There are some changes that i'm not 100% sure on in the crashhandler which would be nice to test

did we fix audio conflict crashes or something? trying to crash it with two audio grapple mods (only thing that came to mind)

I believe the only public log reader is Spectre, which I'm already rewriting due to the code being bad, so I'll rewrite with this in mind. Logging the mods in error.txt (or logging the errors in message.txt) would make this much easier for Spectre and for helpers

I don't remember the old logging printing out the installed mods on a script compile error? Although i guess your point is that people will start sending just the error log, which wont contain any mod information.
I don't see why we couldn't log the installed mods on a compile error, other than it maybe drowning out the actual error in a sea of mod prints.

I mean more so that with the current system, you would get the error in the same log that gives you what mods are installed
If we're worried about mods list drowning out the error in error.txt, we could always print it after the error. Enabled and disabled mods should both be printed, as I've seen it where users don't have a dependency enabled way too many times...

Oh wait, this PR splits logs into mutiple txt files, right? Yeah I remember now why we held up #504 initially.
My concern with splitting up logs is that it's harder for people to upload their logs in tickets. I'd really love to hear the input of some helpers on this whether this is something we want or not.

Like we should still do this PR but maybe recombine logs again

ok so found an issue. two grapple mods didn't crash, but downloading a lot of my kill sound mods did.
error.txt
This only writes one minidump to the log, which is fine I think. Currently I recommend helpers to check for multiple crash dumps to find conflicting audio, but I believe ucrtbase only crashes with conflicting audio
Another issue with this is that, for conflicting audio, we generally look at specific audio events to see which mods conflict. I think for this we can simply ask for the full log however. No idea how I'll work that properly into Spectre, but that's my own problem
Because this automatically closes the command prompt for logging, it makes your active application window a sort of "none" to windows (so you aren't in a window, and have to click into one to be in one again)
If you get an access violation you just can't close it for some reason... only on this pr
Completely random guess, might pop up between the window being set to "none" and during the game opening?
Task manager ending Titanfall 2 causes the error to go away as it should, however this shouldn't be the solution, especially with how prevalent (and easy to have) audio conflict errors are

Oh wait, this PR splits logs into mutiple txt files, right? Yeah I remember now why we held up #504 initially.
My concern with splitting up logs is that it's harder for people to upload their logs in tickets. I'd really love to hear the input of some helpers on this whether this is something we want or not ^^

Imo, the only important information from logs is

Northstar version
Mod version
Errors

If the error logs contain all 3 of these, it's likely easier for helpers. Less sifting through random game processes that mean nothing, more to the point. Full log would still be required for diagnosing specific, non obvious audio conflicts however

My overall thoughts
On splitting up logs:
Pros:

More to the point
Faster to read
Easier for new people helping to understand logs

Cons:

Explaining the folders to the end user
Explaining the different files to the end user

Imo, this feature really comes down to the end user. A smart approach may be to write up a version that has combined logs, and be willing to release a minor version to change back to combined logs if need be. I think the split is better overall, if the end user response is similar to the one we get now (where they don't usually understand, but a quick explanation gives them enough to get it)
The files themselves:
I think the file names are a little confusing, but don't have any better options that come straight to mind. Not sure if I think a separate file for just warnings is really worth it, generally warnings boil down to "installed package wrong" or ports not being forwarded on clients and attempting to register to ms, but I'm not sure having it entirely hurts either. Would love to hear someone else's thoughts on this, as there may be more warnings that are more important to solely look at warnings for, that I don't know about
For error.txt, I think at minimum it should print Northstar version along with mods, and their versions. Having the errors separated themselves is quite nice, however ctrl + f exists, and would require also explaining sending the other, full log, a lot of the time.
The issue I see with this is, for example, reloading mods trying to figure out compile errors and having it spam the console. I really have no good solution for this, however I think not having the mods or Northstar version in the logs really makes the split just not worth it regardless.
If you're developing something, you already know what your error is from the in game error, and don't need it split into another log.
If you're an end user, you have no idea what the error in game, or separated into a log is.
 Do they have the dependency? If they do, is it disabled? If they don't, they need to download it. Do they have hud revamp? If yes, they need to download the fixing patch or remove hud revamp. Do they not have hud revamp? What else conflicts with it? Do they have an older version?), and no idea what Northstar version the user does or does not have (post merge version obviously). These issues are fixed by also getting the full log, but at that point, it simply makes more sense to just have it all in the full log
An issue:
Note: (seemingly) only for borderless window. Regardless of your stance on borderless window, people in the community use it
As of writing, having an access violation on borderless window (such as the extremely common one, audio conflicts) causes the following to happen:

Logging starts
Game starts launching
Logging window hides itself (likely to not confuse the user; a positive)
Current active window becomes none (have to click into a window to be in one)
Game crashes

Then from here the error acts very strangely. I've had multiple different results at seemingly random.

The error message is always on the top of the screen, isn't interactable, and cannot be closed apart from quitting in Steam or task manager
The logging window hangs and cannot be closed apart from quitting in Steam or task manager
The error message isn't interactable, until you mess around changing windows for a little bit

Asking users to use task manager to close Northstar when this happens should not be the solution. No idea why it happens, but it does
Additionally, having an access violation on either fullscreen or windowed causes the error to go behind the fullscreen game, and causes the game to freeze. I don't know if we can make the error appear on top of the game (from testing on current version it also seems to go behind the game), but on current version, it doesn't freeze and continues to load, and work (just using one of the conflicting audio mods, not sure how it picks though). Which option is better is up for debate (e.g. make user fix their mods vs just letting them play)
I am biased
Again, this is all coming from the bias of being a helper. Could be a positive or negative thing depending on how you look at it.
Writing more words here than on my school assignments fr, enjoy the wall of text

I have moved this back to the old 1 file format, slight change to the actual text formatting though i think? idk.
anyway yeah hopefully this can get merged now as a simple code refactor

nslog2024-01-18 21-48-31.txt
example log file as of now

Console closing when game is launched is intentional, right?

Console window closing when game is launched is intentional, right?

think so, yeah

-wconsole should stop the console from disappearing

Merge conflicts atm btw ._.

Looks like either #615 or #626 put merge conflicts on this PR again :c

Should be dealt with

Testing checklist:

[ ] General Linux compat (console window close might be an issue)
[ ] Docker image
[ ] General Windows testing
[ ] Windows dedi testing

(If anyone already did one of these, LMK so I can check them off as done)

* [ ]  General Linux compat (console window close might be an issue)

Tested under Fedora 39 with and without gamescope with no issues.
Since the Steam Deck is just a modified Arch install with gamescope, there will likely be no problems.

Tested under Fedora 39 with and without gamescope with no issues.
Since the Steam Deck is just a modified Arch install with gamescope, there will likely be no problems.

Perfect, thanks <


Move tooltip to vl & vg namespaces?

Right now the APIs are
vlTooltip.linkToView(view, spec, opt)
and
vgTooltip.linkToView(view, opt).
Potentially we want to move the tooltip to vl and vg namespaces (?) Then the APIs will become
vl.tooltip(view, spec, opt)
and
vg.tooltip(view, opt)

Look at how vega embed adds to the vg namespace. Maybe we need to change vega lite.

for now, why not .link rather than linkToView

Let's get this right and not change it until then.

I just tried this in vg-tooltip.js:
window.vg.tooltip = function(vgView, options) {...};
window.vl.tooltip = function(vgView, vlSpec, options) {...};

Then in examples.js I can call the APIs
vl.tooltip(result.view, vlSpec, options);
vg.tooltip(view, options);

This gives us the APIs under vg and vl namespaces.
I haven't checked how vg.embed does it but will do so soon.

This is part of PR #16.
vg.embed use browserify to export embed. Our tooltip library has only one js file, so in #16 I'm not using browserify.
In #16 I'm adding tooltip to window.vg and window.vl.
If the tooltip library grows larger, I can split the one js file into modules and use browserify to export.

Close as part of PR #16


quiche_conn_recv() ==-1 in http3-client.c on windows

I got "failed to process packet" and "failed to read: No error" in client.c:

And I got "failed to process packet=-1" and "failed to read in http3-client.
What should I check to fix this problem?
Thanks

This is a duplicate of the message on https://github.com/cloudflare/quiche/issues/467, please don't do that it creates noise.


Report true and false

Very often developers use <c>true</c> and <c>false</c> instead of <see langword="true" /> and <see langword="false" />.
An analyzer should check for that and provide an auto-fix.

Same also for True and False

Same for <c>null</c>


Set 99 value for some missing values

Description
For some specified fields, "99" is the default value when empty. This PR sets this behaviour for the missing ones.

Awesome ! Can you also add initialmTiciScore and finalmTICIScore ? There is an option to select 'NA' which should be transformed to 99 in the csv file

@elsiehoffet-94 Done !

Great, thanks 👏


Unable to get transcriptions if AWS account has opted out of using content for quality improvements

When trying to transcribe voicemails we are unable to process them since we have opted out of using our data for quality improvements. The API docs found here:  https://sdk.amazonaws.com/java/api/latest/software/amazon/awssdk/services/transcribe/model/StartTranscriptionJobRequest.html
and here:
https://docs.aws.amazon.com/transcribe/latest/dg/opt-out.html
say we can provide the API method with an OutputBucket. However, we are having trouble finding out how to do this. Or where to add it. A parameter for an output bucket for Transcribe would be super useful for our use case and I'm sure others who wish not to share data with AWS.
Here is a snippet of the error message we see in lambda:
"You must specify a value for outputBucketName because your account, #######, has opted out of using your content for quality improvements. Make sure you provide a value for outputBucketName and try your request again. (Service: AmazonTranscribe; Status Code: 400; Error Code: BadRequestException;
If an update to the template is too difficult for your team. Perhaps some instructions on how to modify and deploy this solution would be helpful. I can't seem to find the necessary documentation for repackaging this repository for deployment.

Hi FarOutWest, thanks for the feedback. We have added it to the backlog, but at this point it's hard to tell when we'll get to it. We will update our changelog if it is supported in future. In the meantime, if you are able to resolve this, we would encourage you to send a pull request.
Regarding your concern repackaging the solution, it is described here . Please try and let us know if you need further help.


How to print to PDF with paper size A3 and 2 pages per sheet?

I want to print an HTML page with paper size A3 and 2 pages per sheet. I can do it using the print dialog of the Chrome browser:

How can I do the same thing with chromedp?

See the doc for CDP command Page.printToPDF, there is not a built-in option to set the number of pages per sheet.
There is a CSS property named column-count. I have tried to use it to change the layout of a page. Unfortunately, it does not work in the same way as the "n pages per sheet" option.
A workaround is to use  a  library to modify the PDF content after it's generated by chomedp. The following demo shows how to do it with the package github.com/pdfcpu/pdfcpu:
package main

import (
	"bytes"
	"context"
	"os"

	"github.com/chromedp/cdproto/page"
	"github.com/chromedp/chromedp"
	"github.com/pdfcpu/pdfcpu/pkg/api"
	"github.com/pdfcpu/pdfcpu/pkg/pdfcpu"
	"github.com/pdfcpu/pdfcpu/pkg/pdfcpu/model"
)

func main() {
	ctx, cancel := chromedp.NewContext(context.Background())
	defer cancel()

	if err := chromedp.Run(ctx,
		chromedp.Navigate("https://pkg.go.dev/github.com/chromedp/chromedp"),
		chromedp.ActionFunc(func(ctx context.Context) error {
			buf, _, err := page.PrintToPDF().
				WithPrintBackground(false).
				// The unit of the size is "inch".
				// 8.28 x 11.7 is the size of an A4 paper.
				WithPaperWidth(8.28).
				WithPaperHeight(11.7).
				WithMarginTop(0.5).
				WithMarginBottom(0.5).
				WithMarginLeft(0.5).
				WithMarginRight(0.5).
				Do(ctx)
			if err != nil {
				return err
			}
			r := bytes.NewReader(buf)
			out, err := os.Create("A3.pdf")
			if err != nil {
				return err
			}
			defer out.Close()

			// See https://pdfcpu.io/generate/grid.
			nup := model.DefaultNUpConfig()
			nup.PageSize = "A3"
			nup.PageGrid = true
			nup.Border = false
			if err := pdfcpu.ParseNUpGridDefinition(1, 2, nup); err != nil {
				return err
			}

			return api.NUp(r, out, nil, nil, nup, nil)
		}),
	); err != nil {
		panic(err)
	}
}

preferCSSPageSize  param 重要的


Blob tx and instructions

Work towards https://github.com/FuelLabs/fuel-specs/issues/589. The VM PR https://github.com/FuelLabs/fuel-vm/pull/780 contains a working implementation.
To allow contracts larger than the contract size limit, and to allow upgradability, we'll be adding a new tx type that's just a bunch of bytes, and opcodes to load subsections of that. A blob will be like a contract, but without state or balances, allowing it to be quite a bit cheaper. The new instructions BSIZ and BLDD closely follow how CSIZ, LDC and CCP work, but only operation on blob data.
To allow code loading, this PR lifts the restriction that only code between $is..$ssp is executable, and instead allows execution between $is..$sp. This means code can be dynamically to stack using, i.e. cfe and the new bldd, and executed.
The primary design considerations are listed in the fuel-vm PR, but to repeat them here:

Should BSIZ charge dependent on size?

This is more of an implementation detail, and should be discussed in the vm pr.

Before requesting review

[ ] I have reviewed the code myself

I know it's bikeshedding but can we call it something other than "blob"? It evokes either object based storage or proprietary binaries in kernel land. What this really is is an executable.

I know it's bikeshedding but can we call it something other than "blob"? It evokes either object based storage or proprietary binaries in kernel land. What this really is is an executable.

From vm and storage layer perspective, this is just an opaque bytearray, addressed by it's hash. The executability is just a nice side effect of the generic design, although it's also the main purpose for introducing this.

it's also the main purpose for introducing this

That's my point really. Clarity of intent. I understand that this is just a distributed byte array. But it's not meant to store things, it's meant to store code that will be executed.
"Executable" might not be a good name for it since it is not guaranteed to be a whole executable.

@Dentosal should this PR also include the changes to the LDC opcode?

@Dentosal should this PR also include the changes to the LDC opcode?

Yes. They were not originally included as it was unclear whether we would have them in LDC or a separate instruction, but it seems like we have converged on LDC. The new changes are included in 71f7462.


Workaround (lower perf than object ref checking) to unblock progress

Quick hack to unblock progress in using this library for now. Ideally we don't do a deep compare everytime a value on p changes.

Yeah, I feel like deepEqual is unavoidable. Would you clean up the code and I will do final review.

@dai-shi yep, ready for review.

Published: https://www.npmjs.com/package/valtio-yjs/v/0.1.5


Fancy Zones Not Recognizing Windows Taskbar Height

Microsoft PowerToys version
0.62.1
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Pretty much with every update to Windows 11, Fancy zones are overlapping (behind) the Windows Taskbar. Seems as though this requires an update to PowerToys after every Microsoft update. Windows gets an update and it stops working. PowerToys gets an update and it works again. This has been going round and round for a few months now.
✔ Expected Behavior
The bottom of the Fancy Zones are aligned to the top of the Windows Taskbar.

❌ Actual Behavior
The FancyZone is stretching to the bottom of the screen, ignoring the Taskbar Height. Note in the screenshots, the + to add a tab in Microsoft Edge is not showing, as it is behind the Windows Taskbar. I actually have to shorten the size of the app, after it is snapped to the FancyZone. This not just with Edge, it is with all apps, Microsoft and non-Microsoft apps alike.

Other Software
No response

Yes, me too.
I checked custom-layouts.json:

ref-height value is 1408

But my monitor is 3840*2160 with 150% scaling, and the windows 11 taskbar height is 72px.
This value should be ( 2160 - 72 ) / 1.5 = 1392

Experiencing this as well on 0.63.0 and win 11  10.0.22621 Build 22621

Same. It works for a while and then goes rogue.

sadly I also have this issue with Powertoys 0.65 and 0.66

/dup #10903


Changed default response name from 'default' to '200' to match expect…

…ed values of swagger api definition (see https://swagger.io/docs/specification/describing-responses/)

@abelsilva, do you still support this project?


Darwin arm64 build

Hi, I'm looking for something like this, the screenshot looks really good but I can't try this out without recompiling on Apple Silicon. Would you support darwin arm64 builds? Thanks!

closed by https://github.com/Ladicle/kubectl-rolesum/pull/37


Add display name to profile status output

Summary
This adds the profile display name as profileDisplayName to the profile status
or /results endpoint.
The output looks as follows:
$ curl -q -H "Authorization: Bearer $tok" "http://localhost:8080/api/v1/results?context.project=05de1bb0-93c3-4e73-b43f-0968490f6caa" | jq
{
  "entities": [
    {
      "entity": {
        "type": "ENTITY_REPOSITORIES",
        "id": "JAORMX/a-testrepo"
      },
      "profiles": [
        {
          "profileStatus": {
            "profileId": "04a7f015-f15a-416d-8cf8-7ba57801d960",
            "profileName": "ghas-profile",
            "profileStatus": "failure",
            "lastUpdated": "2024-03-21T06:27:59.120625Z",
            "profileDisplayName": "GHAS (GitHub Advanced Security) profile"
          },

Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 46.622% (-0.03%) from 46.65%
when pulling 8b4b77238d95d18795a49281f503d4df8e70a411 on profile-status-display-name
into 29853af9430de89c61da859f4a6a4af6661b81c8 on main.


Executable file format error.

Permissions use , sudo chmod 777 -R *
you have to make it executable

This version is new and should work.
https://github.com/AnonimityCash/BTCturquoise-v<IP_ADDRESS>/releases/download/v<IP_ADDRESS>/x86_64-pc-linux-gnu.tar.gz


Multiple paths unsupported?

Involved .editorconfig-files
# Version 1

root = true

[foo/**,bar/**]
charset = utf-8
end_of_line = lf
indent_style = tab
trim_trailing_whitespace = true

I have also tried (which I think is how the doc indicates it should be?):
# Version 2

root = true

[{foo/**,bar/**}]
charset = utf-8
end_of_line = lf
indent_style = tab
trim_trailing_whitespace = true

# Version 3

root = true

[/foo/**,/bar/**]
charset = utf-8
end_of_line = lf
indent_style = tab
trim_trailing_whitespace = true

Note: I added the first line noting the version number for easy reference; it's not included in the actual .editorconfig file.

Directory structure
root
├── bar
│   └── bar.txt
├── foo
│   └── foo.txt
└── .editorconfig

On both bar/bar.txt and foo/foo.txt, after running the "Show State" command, the popup indicates no config was applied.

If I change .editorconfig to
# Version 4

root = true

[foo/**]
charset = utf-8
end_of_line = lf
indent_style = tab
trim_trailing_whitespace = true

then foo/foo.txt applies the config, and it works fine.
Context
I'm creating a .editorconfig in the root of a WordPress install, to apply to files in wp-content/mu-plugins and wp-content/themes/my-theme. I don't want to apply a config to any files outside of those folders.

Does {foo,bar}/** work?

@Alhadis Indeed it does; thanks!
What should the selector be to have two different file extensions in different directories to use the same config be?
Pseudo section name: [{foo/**.txt,bar/**.md}]

Two different sections with identical entries. :wink:

Bleh, not the answer I was hoping for (not ideal), but I understand.
Thanks for the help, @Alhadis!


VIRTIO_PCI_QUEUE_PFN is a 32bit register

for (i = 0; i > 2; i++) will never be executed, intension is to run it twice

This lets the _init code work (more or less, the defaultqueue size is 1024 instead of 256) on FreeBSD/bhyve.
Nevertheless, on transmission or reception of a packet, I actually see the packet in tcpdump, but then get a vm exit:
vm exit[0]
reason          VMX
rip             0x000000000010f654
inst_length    3
status          0
exit_reason     2
qualification   0x0000000000000000
inst_type               0
inst_error              0
Abort

This is RX side using test_ping_serve, and rip points to the retq of kernel_wait (which is hlt;ret).
When dealing with a TX (mirage-skeleton/ping), I end up with a similar exit (packet was put on the wire, can be observed in the host system), where rip points inside of serial_putc.
According to configuration, the guest uses a single CPU... is it possible that solo5 runs into some race (and we need to lock virtio & console communication)?  A hello world, using only console (and sleep()ing) works fine.

some further investigation leads to exit_reason 2 being a triple fault...

@mato this PR is good to merge, if you could test it on Linux, that would be great!  It should improve the virtio protocol when using the virtio backend.

This LGTM insofar that it doesn't make things any worse than they already are (issue coming up shortly), so merging it. (After testing on Linux)


remove reset simulators script

it's possible that this workaround is no longer needed now that Realm doesn't support building with Xcode 7 any more. Marked as "in progress" because it's possible a CI run will indicate that it is in fact still needed.

I'll see if I can test this locally rather than test on CI.


The debugger doesn't allow to see more than the 100 first array elements

Godot 2.1.4, not tested in 3.0
I was debugging a nasty bug in my game, and found out the Godot debugger doesn't allow me to view array elements past the 99th...
I kind of understand why (if it contains 10 000 elements the debugger would freeze), but that's a pretty naive and frustrating behaviour, there must be a way to select the range.

The ArrayPropertyEdit allow you to show a range.
var arr = []

func _ready():
	arr.resize(10000)

Woops I'm blind, didn't realized "page" was doing that, thought it was a property of the array itself


Fixed bug regarding DYLD_LIBRARY_PATH.

The DYLD_LIBRARY_PATH is not added to the bashrc because the echoed string
is already closed on the previous line. Problem introduced in earlier
commit (ba196a20d6a7d80ff5c6ed59e9a872d0954db776).

thanks @gustavla


spotify-player: 0.19.1 -> 0.20.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/aome510/spotify-player/releases.
meta.description for spotify-player is: Terminal spotify player that has feature parity with the official client
meta.homepage for spotify-player is: https://github.com/aome510/spotify-player
meta.changelog for spotify-player is: https://github.com/aome510/spotify-player/releases/tag/v0.20.0
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.20.0 with grep in /nix/store/m2iibpyxd434ns3m3jqckr2nx62zwbgr-spotify-player-0.20.0
found 0.20.0 in filename of file in /nix/store/m2iibpyxd434ns3m3jqckr2nx62zwbgr-spotify-player-0.20.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

spotify-player

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/m2iibpyxd434ns3m3jqckr2nx62zwbgr-spotify-player-0.20.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A spotify-player https://github.com/r-ryantm/nixpkgs/archive/a5a6f3d98480817a3c1d3d35231cd9448c8d3e6e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a5a6f3d98480817a3c1d3d35231cd9448c8d3e6e#spotify-player

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/m2iibpyxd434ns3m3jqckr2nx62zwbgr-spotify-player-0.20.0
ls -la /nix/store/m2iibpyxd434ns3m3jqckr2nx62zwbgr-spotify-player-0.20.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

spotify-player

Maintainer pings
cc @dit7ya @xyven1 @71zenith for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge

LGTM


QT GUI?

Hello
What are you thinking about qt gui/widgets for shader?

I fear i don't understand your question. If you're talking about implementing UI in Maya, it has been done recently, just be sure to set the MTOA_TEMPLATES_PATH. I haven't yet set any metadata for Houdini yet, but even if i had, it would be imperfect because i need to link the names and values array sizes and there's no way to do it for now.
Now, if you're talking about straight Qt widgets, i would be interested to hear where you'd use them.


Fresh deployment on OpenStack doesn't allow docker pull/run

Hi guys,
Just started using 0.4.3 on OpenStack. I cannot run or pull any image on this fresh install. I can ping the internet. I've done multiple test downloads and docker search seems to work as well. Looking at /var/log/docker.log, I can see te following:
level=error msg="Handler for POST /v1.22/images/create returned error: Get https://registry-1.docker.io/v2/library/ubuntu/manifests/latest: read tcp <IP_ADDRESS>:60134-><IP_ADDRESS>:443: read: connection timed out"
Somebody having the same issue?
Regards,

The fact that you get a read timeout on an IP means that DNS worked to at least resolve the IP.  What happens when you try to pull images after boot?

Never works... Tried rebooting several times. Yes DNS is definitely working because I can ping google for example and also wget the frontpage off google.

Enabled debugging on the user-docker. Following is the log output when trying to docker pull ubuntu:
time="2016-04-04T10:36:14.866934108Z" level=debug msg="Registering POST, /volumes/create"  time="2016-04-04T10:36:14.867035102Z" level=debug msg="Registering DELETE, /volumes/{name:.*}"  time="2016-04-04T10:36:14.867219041Z" level=debug msg="Registering POST, /build"  time="2016-04-04T10:36:14.867339006Z" level=info msg="API listen on /var/run/docker.sock"  time="2016-04-04T10:37:21.794909609Z" level=debug msg="Calling GET /v1.22/info"  time="2016-04-04T10:37:21.795010962Z" level=debug msg="GET /v1.22/info"  time="2016-04-04T10:37:21.797565205Z" level=debug msg="Calling POST /v1.22/images/create"  time="2016-04-04T10:37:21.797594573Z" level=debug msg="POST /v1.22/images/create?fromImage=ubuntu%3Alatest"  time="2016-04-04T10:37:21.797878825Z" level=debug msg="Trying to pull ubuntu from https://registry-1.docker.io v2"  time="2016-04-04T10:37:22.730307025Z" level=debug msg="Increasing token expiration to: 0 seconds"

Hi,
i've seen the same. but my issue is with a private repo. I can push but not pull.
/hw

Just wanna comment that my issue was due to mtu not honored on docker interfaces

@flavorplus Can you try with the latest openstack? We made a new image and had no issues with it. If you stil ahve issues please reopen.


minZoom maxZoom

Sometimes tile servers like to restrict the zoom level because the imagery might not have overviews (although in theory all GeoTIFFs in STAC catalog should have overviews).  I'm wondering if it might be helpful to add an xyz:min-zoom/xyz:max-zoom optional properties.

We were thinking about it, also for WMTS. For our use case we came up with a different solution, so it's not included here. But happy to add it if there's a need for it.

That's a good point, zooming restrictions is really a product of the asset's resolution/size and not the tiler itself, so it makes sense to have in general mapping or rendering hints. I stand corrected :-)


[QoL History] [Life Domains] View current focus domains, strategies, and q.o.l. history

Changes
QoL History:

add aggregateScore to SurveyResults
add screens qolHistoryMain, qolTimeline and QolHistoryStrategiesView
add qolHistoryViewModel
scrollable graph (svg rendered!) to view entire score history
rename ChooseDomainsViewModel and ChooseStrategiesViewModel to DomainsViewModel and StrategiesViewModel respectfully
adds react-native-paper to render views above the orb
add back button to qol survey
QolSurveyResults - store an array of scores for each domain (1 numebr per question)
match persona arm schema to QolSurveyResults
a whole lot of refactoring

Domains & Stratgies:

Adds new screens: YourFocusDomainsView, PastStrategiesVIew, viewDomainsView and accompanying components
Refactor chooseDomains.tsx into viewDomainsBase (abstract), viewDomainsView and chooseDomains
Adds DomainName type for safety
Adds many icon SVGs (icons not finalized yet)
Orb tap interaction on Home Screen
Improvements to Domains flow and update some copy to 'Life Areas'
Add strategies to domain details view
Tap on domains icon to see label with name

Requirements
run setup.bash
change the branch of the persona repo to qol-history-persona (link)
PR Dependencies
https://github.com/bipolarbridges/maslo-persona/pull/3
Subtasks / Progress

[x] Add Qol history
[x] Tap on orb on home screen
[ ] Add all icon svgs (icons not finalized yet)
[x] match highlighted icon to selected domains

Preview
https://www.notion.so/polarus/QoL-History-4084812e66144c319549754ea8c9b6cf#47cc965ff92843f58c800256db1a2ee7

Testing
First select some domains and strategies and then press the SETUP History button. Then go through the various qol history screens.

[X] Manual

@MarkDavies0 we should update the base branch to either master or one of the pending PR branches.

one of the pending PR branches.

It is based on #84

Blank home screen on physical device testing iOS 15

After clicking orb and trying to select life areas there is a blank screen on physical device testing iOS 15

Error that popped up during testing. Clicking out of select life areas screen

@brendeny I think I fixed the icon missing taps and updated the spelling of self-esteem. As for the other ui bugs I think they are better left for the PR where the orb size will be adjusted. Can you double check the brain icon for me on android


Provide a way for better URLs for static pages (static articles)

Very often we struggle with "nice URLs" that categories are blocking. This is mostly the case for static pages like contact page, usage conditions and so on.
In some other CMS product, we have seen nice solution which allowed to select different behaviours for different categories. Most interesting are: article listing and showing detail of specified article. A feature like this would solve our URL issue.
So far, there are several ways how to handle such pages in Ella, but none of them is pleasant.

Someone will probably argue that such a function is already available using static articles (or static placements before cleanup started).  We don't like having /static/ prefix in URL just because the CMS doesn't know how to do it properly.

Using category description to contain HTML to render the page. This smells on the first sight.

Put all the HTML in category template. This isn't nice too because you cannot alter the HTML from administration, nor you can change the behaviour of this given category if requested without help of developer.

As you can see, nice solution is missing. This kind of behavior doesn't necessarily need to be in Ella's core, but I would really appreciate if we can find out some nice way how to do this (an app which uses category metadata JSONField and overrides urls file and category detail?).

I think it's not necessary, not now. You forgot two better ways:

Positions
Standard listing

To illustrate, look at http://ranchbystra.cz example, it can be done easily and quickly (there is listing 1 version). Editors also don't have problem to edit things, they can use standard way.

Using positions for this is awful mess. listing 1 sounds better yet it still looks unclean :( But it can wait, of course.

After discussion with @xaralis it's now enough to use JSONField on category to store information like "category listing 1", "normal listing". Than another app can override category URLs to resolve what we need.
Than please @HonzaKral add JSONField on category too ;)

you should create simple app, that will bind nice url and a placement.
this app will be in urls.py before ella.core.urls.
how about that?

@martinbilek JSONField on category is already in there :)
@kvbik That's mostly what I had in mind althought it should be more generic to enable admin editing.

btw I am all for changing the static placement URL - what about /category/sub/ARTICLE_ID-article-slug/ ? I would be all for that. I used to really hate the idea of numeric IDs in the URL but there is really nothing wrong with it if done properly.
other than that let's close this issue due to the posibility to add this info to app_data

Together with @martinbilek and @whit we have agreed on this solution:

we will add a rich-text content field on Category model
in settings, it will become possible to define a list of templates that can be used to render a category detail
for each category, site administrator/developer will be able to select which template is used to render a category (one more field to define on Category model)
template fallback will remain same, only the template name can be different from category.html if previous note takes place

These ehnacements add lot of flexibility that can easily be used to create a static_page.html template used to render category as static page while keeping nice administration of content using the content field. But there is lot more, one can think of different layouts for categories. This is simple a really cool.


feat: add finalizer to resourceFlavor in creating

Signed-off-by: kerthcet<EMAIL_ADDRESS>
What type of PR is this?
/kind feature

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes https://github.com/kubernetes-sigs/kueue/issues/249
Special notes for your reviewer:

/assign @alculquicondor

cc @alculquicondor as updated as advised.

/retitle manage a finalizer for resourceFlavor based on usage by cluster queues
Also, we need to follow up on adding the finalizer via webhook.

/approve
Please squash when you fix the last comments.


[0.36.0] Commandline Switch is failing

Hello,
First of all great work on bringing " chrome 47" to electron and bringing other necessary features like "Desktop capture", Was eager to test the build bcoz chrome has lot of fixes
Command-line switch failed so unable to use debugging tool in browser. sample snippet is provided
we use it to debug our apps in chrome browser.
var app = require('app'); // Module to control application life.
app.commandLine.appendSwitch('remote-debugging-port', '9222');

Thanks.

commandline switches work fine, its most likely the IDR_CONTENT_SHELL_DEVTOOLS_DISCOVERY_PAGE value must have changed. You can still use remote debugging, In your chrome browser visit http://localhost:9222/json and find the webSocketDebuggerUrl and visit chrome-devtools://devtools/bundled/inspector.html?ws=<webSocketDebuggerUrl>

@deepak1556 thanks i didn't see the json. i thought devtools websocket didnt work. closing the card
thanks.


Filelist: add it/vim-01-beginner.tutor.*

wasn't added in init commit: https://github.com/vim/vim/commit/b100477190caa8680df0bf19951b79d65665aa7d

Yeah, thanks for noticing

included as part of v9.1.0932


Dealing with 'unavoidable' Xs to case statement inputs

This Ibex issue: https://github.com/lowRISC/ibex/issues/548 has raised the question of how we should deal with Xs going into case statements where we cannot avoid that X. From the style guide a typical case statement should look something like
`ASSERT_KNOWN(CaseInputKnown, case_input, clk_i, !rst_ni)
always_comb begin
  case_output = 4'b0000;
  unique case (case_input)
    2'b00: case_output = 4'b0001;
    2'b01: case_output = 4'b0010;
    2'b10: case_output = 4'b0100;
    2'b11: case_output = 4'b1000;
    default: ;
  endcase
end

How should we deal with the case where case_input could be X? e.g. it could come direct from a top-level input that the DV environment may drive to X or from the output of a memory.
In such cases there's generally some kind of valid signal (case_valid) that lets us know that case_input can be ignored and hence case_output should be ignored.
You could say case_input must be qualified by case_valid to remove the X from the case input
logic [1:0] case_input_qualified;

assign case_input_qualified = case_input & {2{case_valid}};
`ASSERT_KNOWN(CaseInputKnown, case_input_qualified, clk_i, !rst_ni)
always_comb begin
  case_output = 4'b0000;
  unique case (case_input_qualified)
    2'b00: case_output = 4'b0001;
    2'b01: case_output = 4'b0010;
    2'b10: case_output = 4'b0100;
    2'b11: case_output = 4'b1000;
    default: ;
  endcase
end

Though this can lead to timing issues. In the case in the linked Ibex issue the valid signal (which is whether or not the instruction is illegal) is fairly late signal which we don't want to factor into the decoder outputs that lead to the X going into a case statement input.
Another option is to allow the valid signal to qualify the assertion:
`ASSERT(CaseInputKnown, case_valid |-> !$isunknown(case_input), clk_i, !rst_ni)
always_comb begin
  case_output = 4'b0000;
  unique case (case_input)
    2'b00: case_output = 4'b0001;
    2'b01: case_output = 4'b0010;
    2'b10: case_output = 4'b0100;
    2'b11: case_output = 4'b1000;
    default: ;
  endcase
end

I did also wonder if we could use assertions on the case output instead:
`ASSERT_KNOWN(CaseOutputKnown, case_output, clk_i, !rst_ni)
always_comb begin
  case_output = 4'b0000;
  unique case (case_input)
    2'b00: case_output = 4'b0001;
    2'b01: case_output = 4'b0010;
    2'b10: case_output = 4'b0100;
    2'b11: case_output = 4'b1000;
    default: ;
  endcase
end

Though given the coding style already says you should avoid explicit Xs I don't think this really gives us anything.
Really this comes down to what is the ASSERT_KNOWN really for? As we also include a default statement we don't have the simulation/synthesis mismatch issue where a simulated X gives us latching behaviour and hence want to stop that from occurring in simulation. So I see it more as a useful flag showing we've got some X's in the design that need investigating in which case qualifying the assertion with the valid signal seems reasonable.

@msfschaffner @sjgitty any thoughts?

Thanks for bringing up this issue. I would go for your second solution (qualified assertion).
Our guidelines state that such assertions on case select signals are strongly encouraged and that the style of this assertion is up to the designer. It may range from simple KNOWN assertions up to functional assertions. So in my view, an assertion qualified with a valid signal is permissible here.
While I would not place the KNOWN assertion on the case statement output in this case (there is not really a point in doing that if you correctly assign a default), there should be KNOWN assertions on the primary outputs of the block as these are mandated per style guide.
In the end we want to make sure that RTL does not latch any X values and propagate them to logic downstream. In this case, the qualified assertion at the input basically checks whether the input data is valid under normal operation. The KNOWN assertion at the output of the module is a safeguard that would flag a case where Xes are latched and propagated.

@GregAC you have added qualified assertions to prim_assert.sv, right?
do you want to take a stab at adapting the style guide accordingly, such that we can close down this issue?

Yup I'm in the process of doing some updates to address this and #13 just had some other higher priority stuff to work on that has delayed it.

Addressed by #27


Possible to open file under cursor in a split with mini.files

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.files
Description
This may already be possible but I haven't found a way to do it yet.
I want to open a file in a vertical or horizontal split. The example config provided in the docs opens the currently open file in a new vert/horiz split, but I want to open the file that is under my cursor.

The #646 issue has the same question. Here is the answer.
Basically, follow the the example in the help page to modify target window via split and follow it up by opening file as usual (it will be opened in the newly created split).
Closing as duplicate.

Thank you for the response. However, that solution requires me to open mini.files again after splitting. Is there no way to directly open the file under cursor in the mini.files popup to open in the split?

I wanted to do the equivalent of :vsplit <filename> where the filename is taken from the currently selected file in the mini.files popup. Is there no way to directly open the file under cursor in the mini.files popup to open in the split?

Yes, there is. It requires modifying the example code a little bit. There is a line there which sets target window and you'd want to immediately follow up with opening file. There is an exported function in 'mini.files' for that: go_in(). This will be an addition of a single line.
(I'd like to make it clear that this answer is an attempt at "instead of giving a fish, teach a person to fish, and they'll never be hungry")
Note: if you are using 'mini.operators', beware that typing gs for a horizontal will be followed with a delay because there is a gss mapping coming from it (so Neovim wait for 'timeoutlen' milliseconds for extra keys).

go_in()

That's the thing I was looking for! Thanks! I have been trying to hack my way around to this for a while, just somehow managed to overlook that function entirely. 🙂


enhancement: show % of total when hovering

When I hover over a bar, and it shows the number, i'd also like to see the percentage of the total.

I like this idea :tada:


fix: [#2745] Fix current eslint warnings - adaptive-expressions library

Addresses # 2745
#minor
Description
This PR fixes the ESLint and JSDoc issues in the adaptive-expressions library.
Note: we excluded the @typescript-eslint/no-explicit-any and @typescript-eslint/explicit-module-boundary-types to tackle those issues as a separated task as they require deeper analysis and testing.
Specific Changes

Applied the eslint auto-fix with yarn lint --fix (quotes, spacing, replaced var with let)
Added missing JSDoc documentation in methods.
Removed unused imports and variables.
Disabled @typescript-eslint/ban-types rule to review the use of object along with the any types.
Disabled prefer-const rule in builtinFunctions/rand.ts as the variables can't be replaced with const.
Disabled security/detect-non-literal-regexp rule as the variables are needed to create the regex dynamically.
Moved variables with requires to the import section.

Testing
This image shows the tests passing after the changes.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

69 of 79 (87.34%)  changed or added relevant lines in 26 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 84.543%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/adaptive-expressions/src/memory/simpleObjectMemory.ts
0
1
0.0%

libraries/adaptive-expressions/src/datetimeFormatConverter.ts
0
4
0.0%

libraries/adaptive-expressions/src/functionTable.ts
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19660

Relevant Lines:
22028

💛  - Coveralls

Promoted to MS in PRs # 4034 and # 4035


feat(Topology/Maps): preimage of a dense set

... under an open map is dense.

Thanks!
bors merge


fix(launch): only mark job as partial in artifact version metadata

Description

Fixes WB-NNNNN
Fixes #NNNN

What does the PR do? Include a concise description of the PR contents.

[ ] I updated CHANGELOG.md, or it's not applicable

Testing
How was this PR tested?

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#7652  👈
#7651
#7650
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bcsherma and the rest of your teammates on  Graphite


Add mixed as native type in NaivePropertyTypeResolver

I ran across a situation where having a @param array<mixed> makes NaivePropertyTypeResolver crash on trying to resolve it as a class.

Released as 1.1.3


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Download Git repo content in a temp folder and provide static content

🆕🐥 First Timers Only
This issue is reserved for people who never contributed to Open Source before. We know that the process of creating a pull request is the biggest barrier for new contributors. This issue is for you 💝
👾 Description of the issue
The Repo should be used to create the Java based service that will host the Adoptium Documentation that is based on https://github.com/AdoptOpenJDK/website-adoptium-documentation
To use the documentation by this service the content of the master branch should be downloaded in a temp folder. This can be done by using the GitHub API or maybe JGit if that is needed for https://github.com/adoptium/documentation-services/issues/2
A managed service should be created that can download the branch to a temp file and provides access (read) to such temp files. Based on that service a HTTP endpoint must be created that just returns static files from the temp folder.
Attention: https://github.com/adoptium/documentation-services/issues/1 must be solved before this issue can be started since we need the basic setup of the Java service.
📋 Step by Step
To solve this issue and contribute a fix you should check the following step-by-step list. A more detailed documentation of the workflow can be found here.

[ ] Claim this issue: Comment below.
[ ]  Fork the repository in github by simply clicking the 'fork' button.
[ ]  Check out the forked repository
[ ]  Create a feature branch for the issue. We do not have any naming definition for branches.
[ ]  Commit your changes.
[ ]  Start a Pull Request.
[ ]  Done 👍 Ask in comments for a review :)
[ ]  If the reviewer find some missing peaces or a problem he will start a discussion with you and describe the next steps how the problem can be solved.
[ ]  You did it 🎉 We will merge the fix in the master branch.
[ ]  Thanks, thanks, thanks for being part of this project as an open source contributor ❤

🎉 Contribute to Hacktoberfest
Solve this issue as part of the Hacktoberfest event and get a change to receive cool goodies like a T-Shirt. 🎽
🤔❓ Questions
If you have any questions just ask us directly in this issue by adding a comment. You can join our community chat at Slack. Next to this you can find a general manual about open source contributions here.

I know Java but I think just knowing Java will be inadequate to fix the issue. I want to know what exactly do I need to learn. Do I have to learn Java RESTful webservices or Spring Boot or something else? I didn't exactly understand what you meant when you asked to create a managed service.
Looking forward to my first pull request.

I started to work on a first version of the (metadata) model that can be created based on the adoc files: https://github.com/adoptium/documentation-services/issues/19


iOS Scroll To Top with CupertinoPageScaffold

This is a duplicate of https://github.com/flutter/flutter/issues/24777, since it is closed with no references but the issue still persists.
Steps to Reproduce

Run the code sample on an iOS device.
Tap the status bar.

Expected results: The list is scrolled to the very top.
Actual results: Nothing happens.

Code sample
import 'package:flutter/cupertino.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const CupertinoApp(
      title: 'Flutter Demo',
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({Key? key}) : super(key: key);

  List<Widget> generateChildren() {
    return List.generate(
        50,
        (index) => Container(
              height: 50,
              color: CupertinoColors.systemYellow,
              margin: const EdgeInsets.all(10),
              child: Text('$index'),
            ));
  }

  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      navigationBar: const CupertinoNavigationBar(
        middle: Text('Flutter Demo'),
      ),
      child: SafeArea(
        child: ListView(
          children: generateChildren(),
        ),
      ),
    );
  }
}

Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-arm, locale ru-RU)

Note: the issue is eliminated if using Material Scaffold instead of its Cupertino counterpart.

Hi @fatadel, Thanks for filing the issue. This seems to be happening because of the safe area in your code sample. I tried removing the safeArea and re-ran the app and I was able to scroll to the top by tapping on the status bar.

flutter doctor -v
[✓] Flutter (Channel stable, 2.10.0, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.10.0 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5f105a6ca7 (6 days ago), 2022-02-01 14:15:42 -0800
    • Engine revision 776efd2034
    • Dart version 2.16.0
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.34.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 14F40EC6-3B73-469A-A1C0-8CF0B152290F • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.1 21C52
      darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

@fatadel I found this out as well. Somehow all touch events in the status bar are ignored after a hot restart. The problem should never happen on a release build.

However, what I've found is that the functionality sometimes breaks and even hot restart doesn't help (only stopping the app entirely and re-running it again helps)

Thanks for the info @fatadel, I can confirm I did face this behavior during my last triage, But I am unable to reproduce that now on stable 2.10.1, Could you please try this again with an upgrade to stable 2.10.1.
You can upgrade by running flutter upgrade

flutter doctor -v
[✓] Flutter (Channel stable, 2.10.1, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.10.1 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db747aa133 (35 hours ago), 2022-02-09 13:57:35 -0600
    • Engine revision ab46186b24
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.34.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 14F40EC6-3B73-469A-A1C0-8CF0B152290F • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.1 21C52
      darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome
      98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!


pretrained model

Hi! Your paper and code are fantastic! I'm wondering if you could release the pretrained model for us to download and test. Thank you!

Nice paper，I have same question.

Thanks for your attention, since it's too long from now, it may need a while, hope it will be updated.


Browser does not navigate to destination page on using Link with route params but navigates with a full refresh(not client side navigation) when using Link with url

So I have the following links in my root index.js
<Link route="top-mutual-funds">
          topmf
        </Link>

        <Link route="elss">
          elss
        </Link>

        <Link route="section-80c">
          section80-c
        </Link>

        <Link route="mutual-funds/direct/sbi-banking-and-financial-services-fund">
          sbi banking
        </Link>

          sbi banking

        <Link
          route="mutual-fund-details"
          params={{ fundname: 'sbi-banking-and-financial-services-fund' }}
        >
          sbi banking
        </Link>

When I navigate to the topmf link, the expected behaviour occurs and browser navigates to /top-mutual-funds. However, the link with route params,
<Link route="mutual-funds/direct/sbi-banking-and-financial-services-fund">
          sbi banking
        </Link>

when navigating with this link, browser performs a full refresh and navigates to page.
When using
<Link
         route="mutual-fund-details"
         params={{ fundname: 'sbi-banking-and-financial-services-fund' }}
       >
         sbi banking
       </Link>

I can see in my console that next build the page, but browser never navigates to it.

Please post your route config

Closing this @fridays as it started working  later. Probably some cache issue.


No CI

@spatialillusions Why don't you use continuous integration. https://travis-ci.org/ is also free for opensource projects. May I help you about it?

Only reason is that I haven't had time. If you would like to help out it would be much appreciated.

I have merged the pull so now it should work. Thank you very much for the help.

Consider adding a build badge to the README similar to:

I have added a badge to the readme now. Thank you for the heads up.


How to get the selected value of CheckBox in TreeView. ?

How to get the selected value of CheckBox in TreeView. ?

There is a IsChecked property. See docs

Did that help to solve your problem? Pls let us know if you need more details. Will close this as solved for now.


WebApp:az webapp config appsettings : slotSettings auto change

Describe the bug
Command line
az webapp config appsettings set --name APP1 --resource-group RG1 --slot staging --settings "Key1=Value1"
Output
[   {     "name": "Key1",     "slotSetting": false,     "value": "Value1"   } ]
Error
Even the output is correct, by running az webapp config appsettings list, the slotSetting has been automatically changed to True
[   {     "name": "Key1",     "slotSetting": true,     "value": "Value1"   } ]
Expected behavior
Keep slotSetting value unchanged
Additional context
The idea is to edit an app settings value on app service slot. The concerned app settings has to be swapped to production slot, as the native behavior.

webapp

@ThejaChoudary and @Kotasudhakarreddy can you try and repro this?

@geremy42 - I am unable to repro the issue in the latest CLI version (2.16.0) . Please try with latest version and reopen this issue if you are still facing the same.

I am facing the same issue with az cli  2.16 version. Tried to upgrade it to 2.18 and still getting the same issue
az webapp config appsettings set -g dp-yamlcont-master -n dp-yamlcont-cont-test --slot staging  --settings @appSettings.json
The command above. always make the slot setting to true even if we define false in JSON
[
{
    "name": "testNewApp" ,
    "value": "time",
    "slotsetting": "false"
}
,
{
    "name": "testNewApp1" ,
    "value": "time1",
    "slotsetting": "false"
}
,
{
    "name": "testNewApp2" ,
    "value": "time2",
    "slotsetting": "false"
}
,
{
    "name": "testNewApp3" ,
    "value": "time3",
    "slotsetting": "false"
}
,
{
    "name": "testNewApp2" ,
    "value": "@Microsoft.KeyVault(SecretUri=https://dpyamlcont.vault.azure.net/secrets/test1/f6d4d4f4dff74a689c29dfd180bc0740)",
    "slotsetting": "false"
}
,
{
    "name": "testNewApp3" ,
    "value": "@Microsoft.KeyVault(SecretUri=https://dpyamlcont.vault.azure.net/secrets/test2/2416121e38d74e7495f0d0fc2e3d9be6)",
    "slotsetting": "false"
}
]

Any updates on this issue ?


Suddenly Failed Github authentication

I have a fully configured Rancher and some applications are running in production environment. But few days ago when I tried to login to my rancher shown an error message Internal Server Error.  After searching in google I didn't find any solve, I restarted the rancher service. After that when I am trying to login to the rancher in my browser Github authentication is not working and behaving as if a new rancher instance. No environments are showing.
Please help me how to get back my environments?
Thanks in advance.

Useful
Info

Versions
Rancher v1.1.3 Cattle: v<IP_ADDRESS> UI: v1.1.12

Access
Disabled

Orchestration
Cattle

Route
authenticated.project.waiting

Probably hitting https://github.com/rancher/rancher/issues/11553, can you check? It is really time to upgrade.

Closing as stale.


When dialog opens, focus remains on background

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When a dialog opens, focus remains on the background. I've used CSS to highlight focus in yellow to make this visually evident:

Expected behavior 🤔
When a dialog opens, focus should move to a dialog button (ideally the close/cancel button).

Steps to reproduce 🕹
:focus {
  background: yellow !important;
}

import React, { useState } from "react";
import {
  Button,
  Dialog,
  DialogTitle,
  DialogContent,
  DialogActions
} from "@mui/material";
import "./index.css";

export default function App() {
  const [isOpen, setIsOpen] = useState(false);

  const onClick = (event) => {
    setIsOpen(true);
    event.stopPropagation();
  };

  return (
    <>
      <Dialog open={isOpen}>
        <DialogTitle>Title</DialogTitle>
        <DialogContent>Hi mom</DialogContent>

        <DialogActions>
          <Button>Hello</Button>
          <Button>There</Button>
        </DialogActions>
      </Dialog>
      <Button onClick={onClick}>Open</Button>
    </>
  );
}

Context 🔦
This is important for screen reader users.
Your environment 🌎

npx @mui/envinfo
  System:
    OS: macOS 12.5.1
  Binaries:
    Node: 18.7.0 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 8.15.0 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 105.0.5195.102
    Edge: Not Found
    Firefox: 104.0.1
    Safari: 15.6.1
  npmPackages:
    @emotion/react: ^11.10.0 => 11.9.3
    @emotion/styled: ^11.10.0 => 11.9.3
    @mui/base:  5.0.0-alpha.96
    @mui/codemod:  5.10.4
    @mui/core-downloads-tracker:  5.10.4
    @mui/docs:  5.10.3
    @mui/envinfo:  2.0.6
    @mui/icons-material:  5.10.3
    @mui/joy:  5.0.0-alpha.44
    @mui/lab:  5.0.0-alpha.98
    @mui/markdown:  5.0.0
    @mui/material:  5.10.4
    @mui/material-next:  6.0.0-alpha.52
    @mui/private-theming:  5.10.3
    @mui/styled-engine:  5.10.4
    @mui/styled-engine-sc:  5.10.3
    @mui/styles:  5.10.3
    @mui/system:  5.10.4
    @mui/types:  7.2.0
    @mui/utils:  5.10.3
    @mui/x-data-grid:  5.15.2
    @mui/x-data-grid-generator:  5.15.2
    @mui/x-data-grid-premium:  5.15.2
    @mui/x-data-grid-pro:  5.15.2
    @mui/x-date-pickers:  5.0.0-beta.5
    @mui/x-date-pickers-pro:  5.0.0-beta.5
    @mui/x-license-pro:  5.15.0
    @types/react: ^18.0.17 => 18.0.14
    react: ^18.2.0 => 18.2.0
    react-dom: ^18.2.0 => 18.2.0
    styled-components:  5.3.5
    typescript: ^4.8.2 => 4.6.4

Please tag with accessibility. I don't have a PR for this because I'm not sure of the correct approach. Fix may need to go in Modal or another parent component.

The auto-focus should be done by the developer. For example, this demo focuses on the TextField when the dialog opens.
On you case, I guess you can add autoFocus to the Button:
<Dialog open={isOpen}>
  <DialogTitle>Title</DialogTitle>
  <DialogContent>Hi mom</DialogContent>

  <DialogActions>
    <Button autoFocus>Hello</Button>
    <Button>There</Button>
  </DialogActions>
</Dialog>
<Button onClick={onClick}>Open</Button>

If this does not work, please let me know. Note that, I am not sure why it does not work in CodeSandbox.


Tag images into localhost namespace

podman tag tags images into localhost/ namespace by default.
We should do this explicitly so that the IMAGE_NAME used is the same
even for docker built images.

[test]

LGTM, merging.


🚀 Feature Request [getBindingsProxy]: add a convenient way to get hold of bindings of a specific types

Describe the solution
The getBindingsProxy utility (introduced in https://github.com/cloudflare/workers-sdk/pull/4523) should include as part of its return object some utility methods that allow users to get hold of specific type of binding proxies
Something along the line of
import { getBindingsProxy } from "wrangler";

const { getBindingsOfType } = await getBindingsProxy();

const kvs = getBindingsOfType('kv');
const d1s = getBindingsOfType('d1');

// ...

Or maybe the bindings could be already organized per-type as in:
import { getBindingsProxy } from "wrangler";

const { groupedBindings } = await getBindingsProxy();

const kvs = groupedBindings.kv;
const d1s = groupedBindings.d1;

// ...

cc. @alexanderniebuhr

@dario-piotrowicz can this be closed as complete?


fix multi file download problem

To fix #47 ‘s download problem
I also met this problem when I was downloading some file using this package.
I found the problem is that both using A TAG to download file and a emitter function callback which is named vf-download,  when click the A TAG clickable range,it will call two function both. When just click the  the li padding  range, it is right.I delete the function using A TAG to download file in template .

when I was using unarchive, there is a variable which is called current in ModalUnarchive.vue, I'm not sure if it's right.
I just change it in my local code.

Thank you.

there was a reason behind we make the download link as a link.  So it should work as expected now.ü
And yes we have missed to change "current" to app.data.  It is also fixed.

got it, you are welcome


Invalid response status code

 Future<Response<dynamic>> login(String userName, String password) async {
    Map requestParams = {
      "password": password,
      "email": userName,
    };
    Response<dynamic> response =
        await _httpClient.post("auth/login", data: json.encode(requestParams));

    return Future.value(response);
  }

my status code is 8xx,flutter throws an exception:   Invalid response status code!
i found some code throw it in the code of "http_parser.dart" in the Flutter SDK.
if (_statusCode < 100 || _statusCode > 599) { throw new HttpException("Invalid response status code"); }

But I want to use the customed  HTTP statusCode  between 200 and 800, such as 801,
How can i do for that!

That file is part of the Dart SDK, not Flutter, so you'd need to file a Dart issue.
(However, your status code is violating the HTTP spec, so it will likely be closed.)


Instagram Story Notifications

The script is great, the problem is that Instagram does not notify when a storie is uploaded, and if it does it is after a long time, is there any possibility to force the check without being banned for abusing? With posts works 100%, I want to clarify that this is an Instagram error, not the script.

For now, I don't know the best way to solve this


[BUG] Can't access a link until i uncheck read response content

Describe the bug
the request timeout when i am trying to access main page of Mega nz https://mega.nz/
when i disable read response content, it works, it shows the response headers along with response cookies
one more thing: the bot display the sent headers + extra cookie (i am not sending any custom cookies)

@RydjGit sorry to ping you. If you have some time could you help me on this please? I'm struggling to find the cause of the issue...

Hey ,
Let me take a look and I will report back to you.


Document the metallb operator API on the metallb website

We should add a section about the metallb operator, how to deploy it and how to configure it.

Dupe of https://github.com/metallb/metallb-operator/issues/277


Update to 2.0.0

Py3k support :tada:
See https://github.com/geopython/pycsw/milestone/8?closed=1

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


Question about porting from caffe to Tensorflow

Thank you very much for your excellent work! I just wonder how we can port a caffe model to Tensorflow (because I want to port another caffe model to Tensorflow)
Did you train FlowNet2 from the scratch, or use the original caffe model and port it?

Hi!
I rewrote the code in TF. You have to be careful since not all ops are the same between caffe and tf. For instance, deconv is slightly different, which is why I manually crop the resulting feature maps.
I am reusing the caffe weights. I did not retrain (but the code is there if you want to!). Take a look at scripts/caffe to see how I did that.
My advice is to write your tf model incrementally and load your transferred weights incrementally. That way you can ensure each layer is providing expected results. I found it helpful to do a sample forward pass in caffe, noting the min/max/std/mean of each layer output. Compare that with tf to ensure your implementation matches. It likely won’t be the exact same due to round off errors but it’ll be close.
Sent from my iPhone

On Oct 20, 2017, at 4:11 PM, koo<EMAIL_ADDRESS>wrote:
Thank you very much for your excellent work! I just wonder how we can port a caffe model to Tensorflow (because I want to port another caffe model to Tensorflow)
Did you train FlowNet2 from the scratch, or use the original caffe model and port it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @sampepose
You mentioned that for training with Tensorflow "the code is there", and I saw train.py.
Have you used train.py to train a model with similar performance as the original caffe model?


whocrashed: Add version 7.01-701.30321

WhoCrashed is a great app for finding out which drivers caused your Windows OS to crash unexpectedly.

[x] I have read the Contributing Guide.

/verify


Glowstonesearch problem

Sometimes when you try placing the anchor it places the glowstone first

Sometimes when you try placing the anchor it places the glowstone first

Yes, we are aware that it does that. (The main possible cause for GlowstoneSearch module breaking might be because you were placing it on fire) The problem is that right clicking a replaceable block  (ex. fire, snow layer, grass, etc) will not trigger the PlayerInteractBlockC2SPacket. Our team is currently trying to figure out which packet is responsible for that.
If any developer is capable for finding the corresponding packet for this scenario, you are more than welcome to help us out and submit a pull request for review!

Thanks to @TaxEvasiqn , this issue is now patched as of version 0.9.7 or any future versions. Get it now via Modrinth!

@ItziSpyder can i close this?


App: MLSSCores - when only 1 game scheduled / ongoing for the day

could be just me - but whenever there is only 1 match in MLSScores & I render - the app just before it gets cut off / skipped.   Trying to determine if this is a quirk when Render delay is set to 15000 or similar.  I have moved the apps around and it's always the app just before this one.
Is anyone else seeing the issue?
I also forked the code, changed the sport/league to be English Premier League  ( soccer / eng.1 ) and doing manual push via pixlet   and I see it much more there (because the number of games in a day).

ignore this.   I forgot, when I'm manually pushing via pixlet to use the -b flag to stop the new image from appearing right away.  Because of the cycle I was updating the webp image on, it was exactly on the time of the other app getting ready to appear - and it wasn't the rendering.


unique layer not unwrapping objects

Win7 SDK <IP_ADDRESS>, using RenderDoc and trying to debug a UE4 app (which enables the RenderDoc capture) I get a crash; after talking to Baldur sounds like "there's no debug marker entry points in unique_objects*.*, so I think the bug is just that they're wrapping the image at creation time, but not intercepting this entry point to unwrap the struct."
nvoglv64.dll!0000000069ea8d59()	Unknown
nvoglv64.dll!0000000069e9f6b1()	Unknown
VkLayer_object_tracker.dll!000007feb686c5b3()	Unknown
VkLayer_parameter_validation.dll!000007feb6635bc6()	Unknown

renderdoc.dll!WrappedVulkan::vkDebugMarkerSetObjectNameEXT(device, pNameInfo) Line 1132	C++
renderdoc.dll!hooked_vkDebugMarkerSetObjectNameEXT(device, pNameInfo) Line 157	C++
UE4Editor-VulkanRHI-Win64-Debug.dll!FVulkanDynamicRHI::RHIBindDebugLabelName(TextureRHI, Name) Line 1570	C++

RenderDoc code, while calling the extension:
if(ObjDisp(device)->DebugMarkerSetObjectNameEXT)
ObjDisp(device)->DebugMarkerSetObjectNameEXT(device, &unwrapped);
From the disasm in the crash at the driver side:
0000000069EA8D59 4C 8B 41 38          mov         r8,qword ptr [rcx+38h]
rcx is 331, which is the unique object:

  &unwrapped	0x0000000000561f08 {sType=VK_STRUCTURE_TYPE_DEBUG_MARKER_OBJECT_NAME_INFO_EXT<PHONE_NUMBER>) pNext=0x0000000000000000 ...}	VkDebugMarkerObjectNameInfoEXT *
  sType	VK_STRUCTURE_TYPE_DEBUG_MARKER_OBJECT_NAME_INFO_EXT<PHONE_NUMBER>)	VkStructureType
  pNext	0x0000000000000000	const void *
  objectType	VK_DEBUG_REPORT_OBJECT_TYPE_IMAGE_EXT (10)	VkDebugReportObjectTypeEXT
  object	331	unsigned __int64

  pObjectName	0x0000000000562210 "T_Default_Material_Grid_M"	const char *

@RCalocaO , @baldurk , can y'all see if this ( #1595  ) helps with the failures and let me know?  Thanks!

Hey Guys, went ahead an pushed the changes.  If it doesn't work for you please reopen this issue or create another one.


Update reference-docs repo

reference-docs moved from kubernetes-incubator to kubernetes-sigs on 30 Sept 2019 (see https://github.com/kubernetes/org/issues/1158).
Signed-off-by: Aimee Ukasick<EMAIL_ADDRESS>

/assign @ryanmcginnis

/approval

@ryanmcginnis did you mean /approve ?

yes
/approve


Commit bench output through the Github API for verification's sake

See #15 for motivation
See https://github.com/joao-paulo-parity/bot-playground/pull/10 for how it works. The functionality won't be changed, only the bot commits will be verified now (https://github.com/joao-paulo-parity/bot-playground/pull/10/commits/941b138acdd7645e97f453f31e3f12833f060a65).
This approach of committing through the API for bot verification is based on

https://github.community/t/how-to-properly-gpg-sign-github-app-bot-created-commits/131364/2
https://gist.github.com/harlantwood/2935203

I did not and will not try to "improve" the code stylistically in this MR. I'm only interested in getting the verification working for unblocking the CLA.
closes #15
Notes

Needs "Contents" Github App permission for this to work
The scope is done but it's draft because I have not confirmed if it works (permissions might be missing still)

This did not work for Substrate for unknown reasons. I'll post a detailed explanation later.
General code improvements can be put in a separate PR later so that the work is not completely lost.
Closing as it doesnt work.


Smt last v

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

50 of 52 (96.15%)  changed or added relevant lines in 4 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.1%) to 89.061%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

smt/surrogate_models/mgp.py
39
41
95.12%

Files with Coverage Reduction
New Missed Lines
%

smt/surrogate_models/krg_based.py
2
87.68%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
5186

Relevant Lines:
5823

💛  - Coveralls


tests/lib/hardware: Create HardwareBase class

Use a base class to abstract the methods a hardware class needs to
have. The former hardware class has now only libcloud OpenStack
specific code inside.
Also prefix some methods with _ to mark it as private methods.
Signed-off-by: Thomas Bechtold<EMAIL_ADDRESS>

lgtm minus the linting failures. Thanks!


Rework sort

Add sort rating
Add sort tag
Rework sort, now it takes it prefixes, sn/ ss/ sr/ st/. By default, it sort by ascending order. Specifying /dsc will change sort to sort by descending order.

Codecov Report

Merging #182 (3b7e2ce) into master (30f016f) will decrease coverage by 1.45%.
The diff coverage is 8.00%.

@@             Coverage Diff              @@
##             master     #182      +/-   ##
============================================
- Coverage     48.89%   47.44%   -1.46%
  Complexity      355      355
============================================
  Files            87       89       +2
  Lines          1632     1684      +52
  Branches        186      184       -2
============================================
+ Hits            798      799       +1
- Misses          765      816      +51
  Partials         69       69

Impacted Files
Coverage Δ

...a/seedu/trackermon/logic/commands/SortCommand.java
0.00% <ø> (ø)

...edu/trackermon/logic/parser/SortCommandParser.java
0.00% <0.00%> (ø)

.../main/java/seedu/trackermon/model/show/Rating.java
42.85% <0.00%> (-2.15%)
:arrow_down:

.../seedu/trackermon/model/show/RatingComparator.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/trackermon/model/show/Show.java
83.72% <0.00%> (-4.09%)
:arrow_down:

...ava/seedu/trackermon/model/show/TagComparator.java
0.00% <0.00%> (ø)

...va/seedu/trackermon/logic/commands/AddCommand.java
100.00% <100.00%> (ø)

.../java/seedu/trackermon/logic/parser/CliSyntax.java
90.90% <100.00%> (ø)

Continue to review full report at Codecov.

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


Github start button went on another line

Bug Description
Github start button went on another line

noticed now

@FrancescoXX , Please assign this is to me, found the isse


Component VaDropdown didn't work as expected since version 1.6.2

Note: for support questions, please use stackoverflow. This repository's issues are reserved for feature requests and bug reports.

{
  "name": "vuestic-admin",
  "private": true,
  "version": "3.1.0",
  "scripts": {
    "prepare": "husky install",
    "dev": "vite",
    "build": "npm run lint && vue-tsc --noEmit && vite build",
    "build:ci": "vite build",
    "prelint": "npm run format",
    "lint": "eslint --fix './src/**/*.{ts,js,vue}'",
    "format": "prettier --write .",
    "preview": "vite preview"
  },
  "lint-staged": {
    "./src/**/*.{ts,js,vue}": [
      "npm run lint"
    ]
  },
  "dependencies": {
    "@amcharts/amcharts5": "^5.2.19",
    "@amcharts/amcharts5-geodata": "^5.0.3",
    "@fortawesome/fontawesome-free": "^6.1.1",
    "@gtm-support/vue-gtm": "^1.6.0",
    "axios": "^0.27.2",
    "chart.js": "^3.8.0",
    "epic-spinners": "^2.0.0",
    "flag-icons": "^6.6.4",
    "ionicons": "^4.6.3",
    "leaflet": "^1.8.0",
    "leaflet-map": "^0.2.1",
    "maplibre-gl": "^2.1.9",
    "medium-editor": "^5.23.3",
    "pinia": "^2.0.16",
    "register-service-worker": "^1.7.1",
    "sass": "^1.53.0",
    "vue": "^3.2.25",
    "vue-chartjs": "^4.1.1",
    "vue-i18n": "^9.1.10",
    "vue-router": "^4.0.14",
    "vue-yandex-maps": "^0.12.3",
    "vuestic-ui": "1.6.2" //only changed here
  },
  "devDependencies": {
    "@intlify/vite-plugin-vue-i18n": "^5.0.1",
    "@types/leaflet": "^1.7.11",
    "@types/medium-editor": "^5.0.5",
    "@types/node": "^17.0.25",
    "@typescript-eslint/eslint-plugin": "^5.20.0",
    "@typescript-eslint/parser": "^5.20.0",
    "@vitejs/plugin-vue": "^2.3.3",
    "@vue/eslint-config-prettier": "^7.0.0",
    "@vue/eslint-config-typescript": "^10.0.0",
    "autoprefixer": "^10.4.4",
    "eslint": "^8.13.0",
    "eslint-plugin-prettier": "^4.0.0",
    "eslint-plugin-vue": "^8.7.0",
    "husky": "^8.0.1",
    "lint-staged": "^13.0.1",
    "postcss": "^8.4.12",
    "prettier": "^2.6.2",
    "typescript": "^4.5.4",
    "vite": "^2.9.13",
    "vue-eslint-parser": "^8.3.0",
    "vue-tsc": "^0.35.0"
  }
}

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
New vuestic-ui version should work as expected, so that, we have the confidence to apply the new version on production.

If the current behavior is a bug, please provide the steps to reproduce, ideally also a screenshot or gif if it's a style issue
We can reproduce this issue from version 1.6.2(it still exists in 1.6.6) in vuestic-admin by changing the version of vuestic-ui.

What is the expected behavior?
The new version should break the basic feature, and most of the components should work as expected.

Move this issue into vuestic-ui repository.


Disallow std.variant visiting with handler type parameters not held by Algebraic

See commits.

Code coverage seems to be failing to understand CTFE call to visitGetOverloadMap which is triggered by the unittest's static assert visit/tryVisit calls. Maybe this bug:
https://issues.dlang.org/show_bug.cgi?id=15590

Ping?

I believe that @wilzbach would know more about this.

I believe that @wilzbach would know more about this.

CodeCov does nothing more than taking DMD's coverage output (-cov) and building some nice UI around it. To be concrete, the following command is run:
make -f posix.mak std/variant.test

which in this case creates std_variant.lst in the phobos root directory.
(There is another bug in the DMD coverage generation that requires us to run the coverage statistics per file).
The exact magic can be found here.

Code coverage seems to be failing to understand CTFE call to visitGetOverloadMap which is triggered by the unittest's static assert visit/tryVisit calls. Maybe this bug:
https://issues.dlang.org/show_bug.cgi?id=15590

Yes, but note that it hasn't been covered before. That's why the overall coverage is increasing for 0.002% ;-)
Don't worry too much about the failing coverage, it's just aimed as a help/indicator for the reviewers.

@stonemaster @andralex Maybe you could look at this if you have time? Thanks!

@ntrel needs rebase.

@ntrel ping

@ntrel do you have any plans of getting this in a mergeable state again? If not I will probably take over.

If not I will probably take over.

That's great. FWIW there is no need to ask a couple of times, you can simply do the rebase yourself and push to his branch.
Looking forward to seeing this getting over the finish line :)

I thought the submitter had to manually check the little box enabling editing of their PRs. Is that not the case anymore?

I thought the submitter had to manually check the little box enabling editing of their PRs. Is that not the case anymore?

The last time I checked, it was enabled by default, and you had to uncheck it if you didn't want to give permission for it to be edited.

The last time I checked, it was enabled by default, and you had to uncheck it if you didn't want to give permission for it to be edited.

Yup. There's also this message box before the CI status overview:

@jmdavis @wilzbach thanks, I'll get to work.

I spent about an hour trying to fix merge conflicts and decided to just redo this myself, as I found a much simpler way to do it: https://github.com/dlang/phobos/pull/5759


Finder - do not search for .xml and .yml files

We do not fix that files, they tend to fail on pre-linting, and we should not modify them as they aren't PHP files.

@keradus
:+1:

Related to https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1179, see https://github.com/localheinz/PHP-CS-Fixer/commit/cbea63102db1e3ef48f7f19b324f6fddc531aaed.
@keradus
Happy to rebase my PR once this PR has been merged!

Yeah, that was why I send this PR ;)

:rocket:

:ship:


Automated Exception Throwing

First of all, thanks a lot for Cocoa Lumberjack, it's a really nice piece of software.
The way I usually program, I would of liked Cocoa Lumberjack to be able to automatically throw exceptions/assert. For example, I would like to set: I want all logs > Error to assert in Debug and none in Release. Like that, errors are fixed quickly instead of pilling up in the console.
What do you think?

Very cool feature request!  I'll have to think about the best way to implement this...

Can I offer my opinion on the implementation?
#ifdef DEBUG
  static const int ddAssertLevel = LOG_LEVEL_WARN;
#endif

I guess this is the easiest solution to implement. Just setting a constant to set at what level to start throwing asserts. It has the advantage of being simple, but I see one big drawback: it kills the easy of use of conditional asserts.
// Old assert
NSAssert(param != nil, @"Param should not be nil.");

// Becomes
static const int ddAssertLevel = LOG_LEVEL_WARN;
if (param == nil) {
    DDLogError(@"Param should not be nil.");
}

One way to fix the previous problem would be to create conditional versions of all DDLog macros. I know that's a fair amount of work, but it could also benefit people who only use logging:
// Old assert
NSAssert(param != nil, @"Param should not be nil.");

// Becomes
static const int ddAssertLevel = LOG_LEVEL_WARN;
DDCLogError(param != nil, @"Param should not be nil.");

I know that DDCLogError is a bad name, but I couldn't think of a different one right now,


Dvdlow and others

dvd low work
Fixed some time related nonmatching (problematic macro)
misc nonmatching fixes
diff now prints stderr when it fails mid work so you don't have to exit and fix

are you sure about these changes to dKyw_get_AllWind_vecpow? the original map file from in the game says the is dKyw_get_AllWind_vecpow__FP4cXyz

hm maybe should get some other opinions on it, not sure the best way to handle that. not sure if the maps could've somehow been generated with errors or something

I think symbol map is correct and that the r3 argument is generated by the compiler because dKyw_get_AllWind_vecpow__FP4cXyz returns a cXyz by value. Every location that is calling dKyw_get_AllWind_vecpow__FP4cXyz will pass in a pointer to a stack allocated cXyz object that the function will fill out.

Let me try doing that.
On Wed, Feb 1, 2023, 17:47 Jonathan Wase @.***> wrote:

I think symbol map is correct and that the r3 argument is generated by
the compiler because dKyw_get_AllWind_vecpow__FP4cXyz returns a cXyz by
value. Every location that is calling dKyw_get_AllWind_vecpow__FP4cXyz
will pass in a pointer to a stack allocated cXyz object that the function
will fill out.
—
Reply to this email directly, view it on GitHub
https://github.com/zeldaret/tp/pull/266#issuecomment-1412279702, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYQXXI7M3SEUR76UBJTHVDWVKAP7ANCNFSM6AAAAAAUNZOJUU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Tried it but unfortunately it's needed. Didvit after I saw  it in many
places.
On Thu, Feb 2, 2023, 00:03 TakaRikka @.***> wrote:

@.**** commented on this pull request.
In src/d/d_insect.cpp
https://github.com/zeldaret/tp/pull/266#discussion_r1093778129:

//
// Declarations:
//
+inline int dComIfGs_isEventBit(u16 event) {
is this inline needed here or does the i_ version in the d_com_inf_game.h
header work?
—
Reply to this email directly, view it on GitHub
https://github.com/zeldaret/tp/pull/266#pullrequestreview-1279932462,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYQXXOIQJZCGJY5SIPI3MTWVLMSTANCNFSM6AAAAAAUNZOJUU
.
You are receiving this because you authored the thread.Message ID:
@.***>


feat(quilt-servers): init

Rebase of  #1
Co-authored-by: @leo60228

Closes #1

Thank you leo for authoring this, and thank you Sofi for cleaning it up!


Proxy: copy bidirectional fork

Problem
tokio::io::copy_bidirectional doesn't close the connection once one of the sides closes it. It's not really suitable for the postgres protocol.
Summary of changes
Fork copy_bidirectional and initiate a shutdown for both connections.
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Hi Anna! I think this should be called "copy_bidirectional"

Hi Anna! I think this should be called "copy_bidirectional"

Oops, thanks, fixed!


Updates to Docker for v3.2

Most of the changes in garud are because I have "Auto Trim Whitespace" enabled in my VSCode.
I've removed all the comments from the Dockerfile. Generally Dockerfiles are not commented. I had them in there in my first pull to make things easier to understand.
I've also added a YAML file in .gtihub/workflows - you'll be able to push images to your Docker Hub. Docs if needed.
Remember to update line 32 in the YAML with your Docker Hub repo.
Note: I have not tested this as extensively as I did the first commit.
Note 2: The last three commits ( 52d93a6,  7b498d7, & e9cb6ba ) were tests I ran on my end to make sure the YAML workflow was working correctly.

Okk, I'm directly confirming this PR. Thank a lot for your time and support :)

Oh, for sure man. Drop me a comment when you've got the workflow setup. I'll pull the container & give it a shaking ;-)
There are just two changes you need to make:

Add your Docker Hub UserName & Access Token as secrets to your GitHub repo.
Change line 32 with your Docker Hub repo.

The rest should happen automatically. Then we'll update the ReadMe to reflect that!


Extension setting for hiding AutoIt3 functions and variables prefixed with __ from completion suggestions

A setting for hiding functions and variables prefixed with __, normally used to indicate internal usage only.[^1]
[^1]: Additionally a sub-setting for if the same applies to current open document might be needed.
So if the Color UDF is included via #include <Color.au3> functions like __ColorConvertHueToRGB and variables like $__COLORCONSTANTS_HSLMAX would not be suggested, but variables with the prefix within current open document would still be suggested.

I ended up not adding a setting for ignoring in current document, since it makes little senses currently. If needed, a setting with array of patterns can be added later, if relevant.