Square maps with grid_size != tile_size

I would like to be able to use a tileset where the tile size differes from the grid size. However, it doesn't seem to be possible and I get the feeling that the grid_size is not taken into account when rendering.
I would expect the following example (tile_size=16x16, grid_size=8x8) to render sqaures with overlap but the tiles are aligned after the tile_size (see image) and not the grid_size. Setting grid size to 32 instead of 8 gives the same result.
    let mut settings = LayerSettings::new(
        MapSize(2, 2),
        ChunkSize(8, 8),
        TileSize(16.0, 16.0),
        TextureSize(96.0, 16.0),
    );
    settings.grid_size = Vec2::new(8.0, 8.0);
    let (mut layer_builder, _) = LayerBuilder::new(
        &mut commands,
        settings,
        0u16,
        0u16,
    );

    for i in 0..6 {
        let tile_pos = TilePos(i, i);
        let _ = layer_builder.set_tile(
            tile_pos,
            TileBundle {
                tile: Tile {
                    texture_index: i as u16,
                    ..Default::default()
                },
                ..Default::default()
            }
        );
    }

Can you re-test this using the latest from main? I believe this should be fixed now.

Hi, thanks for looking into it!
Strange enough it still doesn't seem to work properly. Changing the grid_size doesn't seem to do any change.
I've prepared an example for reproducing the issue if that would help (modified the basic.rs example)
https://github.com/wideberg/bevy_ecs_tilemap/tree/grid_size_issue

I'm also not sure how the three values work together, TilemapSize, TilemapTileSize, TilemapGridSize.
Having a tilemap size of 4x4 (tiles), tile size of 16x16 (px) and a grid size of 8x8 (px) would in fact fit a total of 8x8 tiles due to the overlap, but with the total map size in pixels being 72x72px since the final row and column would overflow the expected size of 64x64 px.

This happens because the square shader doesn't incorporate tilemap_data.grid_size in get_mesh like other shaders do.
I believe this is a simple fix, but I'm still really shaky on computer graphics math, so I'm not exactly sure how to do it.
I did try, and realized that for the game_of_life example with it's grid_size changed to vec2(16.0, 8.0), a naive multiplication against position just made things 16x farther apart on the x-axis and 8x apart on the y-axis.
Need to somehow translate it to a 1-0 range based on the tile size, I think? As that vector should actually result in tiles being spaced normally in the x-axis and overlapping by 8 pixels in the y-axis.

I would like to be able to use a tileset where the tile size differes from the grid size. However, it doesn't seem to be possible and I get the feeling that the grid_size is not taken into account when rendering.
I would expect the following example (tile_size=16x16, grid_size=8x8) to render sqaures with overlap but the tiles are aligned after the tile_size (see image) and not the grid_size. Setting grid size to 32 instead of 8 gives the same result.
    let mut settings = LayerSettings::new(
        MapSize(2, 2),
        ChunkSize(8, 8),
        TileSize(16.0, 16.0),
        TextureSize(96.0, 16.0),
    );
    settings.grid_size = Vec2::new(8.0, 8.0);
    let (mut layer_builder, _) = LayerBuilder::new(
        &mut commands,
        settings,
        0u16,
        0u16,
    );

    for i in 0..6 {
        let tile_pos = TilePos(i, i);
        let _ = layer_builder.set_tile(
            tile_pos,
            TileBundle {
                tile: Tile {
                    texture_index: i as u16,
                    ..Default::default()
                },
                ..Default::default()
            }
        );
    }

@wideberg can you try it again? We made some updates that allow it to work now. You'll have to use the main branch.

@wideberg @tigercat2000 thanks for pointing this out! A fix is on its way (see PR #243)

I made a quick test with tilesize 16x16 and gridsize 8x8 which gave the result I expected:

An asymetric gridsize of 4x8 also seems to work correct:

Thanks a lot! Great job!!

Fixed by: #243


Authentication Error on build 20180415.2

Updated Mac OS to Hi-Sierra 10.13.4 today, installed the newest version of Storage Explorer (build 20180415.2) and now storage explorer will not authenticate using credentials, it will only connect using a string or key not as the user. It displays an error

HI @MLeslie7 , thank you for reporting this issue. I have a couple of clarifying questions to ask:

Sign in (user name and password) worked for you prior to 1.0.0 and is now not working, yes?
Assuming #1, does the info bar pop up with any errors after you try to sign in? Does the auth screen stay blank? Basically, what are you seeing when you try to sign in?
Are you, or are you not able, to use SAS/name and key attach?

Thanks!

Thanks for the quick response

Yes it did before the OS update
A)No - I attached a screen shot of the login page. B)Yes always blank c) see the new screenshot
Update it did accept the conn string but would not allow for browsing the storage account an unauthorized popup, and have not tried a sas key yet.

This is my sups laptop we are on the road and might be offline this evening, but will be online tomorrow around 8 am cst to provide more info if needed.

Open Storage Explorer
Click - Add Account - Sign In
This window pops up and is blank for a minute then displays the pop up error (the screen shot from original post)

The screen shot of the error message was from after we had added an account with the conn string, it did not allow browsing the account. The login page error was the very first attempt to connect after installing the updated Azure Explorer and before adding the account with a conn string, thats why you can see nodes to browse

@MLeslie7 , I'm still a little confused. Let's start by focusing on sign in. So:

You click "Add Account", and then the connect dialog pops up to the initial page.
You click "Sign in..." and then you see this page:

At this point, I would expect you to see this pop up on top of the connect dialog:

But you're not seeing that, right? If so, I want you to try two things. First, three finger swipe up on your track pad. Let's make sure that the window from step 3 isn't hiding. If it is, doing a three finger swipe should make it visible. Second, if it isn't hiding, then close the Connect dialog, and launch the dev tools (Help -> Toggle Developer Tools, or f12). Then go to the console tab and filter by warnings.

If you see any (yellow or red text, can you screenshot what you see and post it here?

Thanks.

Will be offline until tomorrow morning.

From: Matthew Rayermann<EMAIL_ADDRESS>Sent: Wednesday, April 18, 2018 5:44:20 PM
To: Microsoft/AzureStorageExplorer
Cc: Mike Leslie7; Mention
Subject: Re: [Microsoft/AzureStorageExplorer] Authentication Error on build 20180415.2 (#47)
@MLeslie7https://github.com/MLeslie7 , I'm still a little confused. Let's start by focusing on sign in. So:

You click "Add Account", and then the connect dialog pops up to the initial page.
You click "Sign in..." and then you see this page:
[image]https://user-images.githubusercontent.com/21112697/38961602-19ff0f32-431e-11e8-8fc3-e4e9db399157.png
At this point, I would expect you to see this pop up on top of the connect dialog:
[image]https://user-images.githubusercontent.com/21112697/38961609-25bc663a-431e-11e8-8c9a-24fdb7375469.png
But you're not seeing that, right? If so, I want you to try two things. First, three finger swipe up on your track pad. Let's make sure that the window from step 3 isn't hiding. If it is, doing a three finger swipe should make it visible. Second, if it isn't hiding, then close the Connect dialog, and launch the dev tools (Help -> Toggle Developer Tools, or f12). Then go to the console tab and filter by warnings.
[image]https://user-images.githubusercontent.com/21112697/38961755-d2d6207c-431e-11e8-9318-39d9673013b3.png
If you see any (yellow or red text, can you screenshot what you see and post it here?

Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/AzureStorageExplorer/issues/47#issuecomment-382552384, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AM2S_0EnmdEORm2cPNMbokKUvdsjnQHxks5tp8HDgaJpZM4TavAd.

Hello,
Tried and the window was not hiding, here is the screen shot of the errors from the console.

If you need more info will be on a flight this afternoon (12 pm CST) until 5

Can you expand some of the arrows? We're probably most interested in the one next to "Object"s. Ideally just expand all of the arrows until you see some more information (stack traces, what the actual error is). Also, thank you for being willing to do this back and forth.

Ok that is no problem you need info to find the issue, talking more there have been 3 things updated in the last week
MacOS update was released this week
The signin page to Azure/Office 365 changed this week (I noticed the appearance is different)
Storage Explorer got a update as well
Between the 3 updates it looks like something is not correct, I can get any more info you need, thanks for the quick responses.
These are shots of the 3 different errors - different error codes, there are more errors but they are the same from the repeated attempts

Ah ok. I know who to reach out to now about this issue. Something is going wrong when we try to store your authentication token in your Mac keychain. I'll be back in a few hours with either some resolution steps or further diagnostic steps.
Until then, you can download the previous version of Storage Explorer at the bottom of this page if you need to be unblocked: https://azure.microsoft.com/en-us/features/storage-explorer/

Ok, here's what I'd like you to try:

Launch "Keychain Access" (cmd + space, type in keychain, hit enter)
In the filter at the top right, type in identityservice, you should now see an entry with the Name "Microsoft.Developer.IdentityService." If you don't, then stop here and let me know.
Right click on the entry, choose "Get Info", and go to the "Access Control" tab
There should be one program which has access to this password, ServiceHub.Host.CLR.x64

If you see don't see ServiceHub.Host.CLR.x64 in the list then:

Open finder, go to your applications, and find Microsoft Azure Storage Explorer
Right click on it, and choose "Show Package Contents"
Navigate to Contents/Resources/app/ServiceHub/Hosts/ServiceHub.Host.Core.CLR.x64/
Look for the file with the name "ServiceHub.Host.CLR.x64" and copy it to your desktop
Go back to Keychain Access, where the info window for IdentityService should still be open, click on the plus, and use the finder window that appears to select the file you copied to your desktop in step 4, and then click Add
Save changes
Try to sign in
If you can't sign in, try the steps below

If you see ServiceHub.Host.CLR.x64 in the list then:

Select the "Allow all applications to access this item" option, and then save changes
Try to sign in

If none of that works, then I'll need to do some more investigation.

Sorry for the delay travel...
It was set to allow all. I added the file as requested and saved changes but it doesn’t save.
I kept it on allow all but still no dice.

What do you mean by it won't save?

Also, can you do/attach the following?

From: Matthew Rayermann
Sent: Friday, April 20, 12:33 PM
Subject: Re: [Microsoft/AzureStorageExplorer] Authentication Error on build 20180415.2 (#47)
To: Microsoft/AzureStorageExplorer
Cc: Mike Leslie7, Mention
Also, can you do/attach the following?

Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/AzureStorageExplorer/issues/47#issuecomment-383168036, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AM2S_3ru3YchU8aQYVSle-aXsIYkiJGRks5tqhvSgaJpZM4TavAd.

Hey, I have the exact same problem with version 1.0.0
I created the log files you asked for. You can find them in attachment.
Also the link on the bottom of the page to download the previous version seems to link to the last version also..
If you have another link to previous version, that would be helpful.
thanks.
logs.zip

Hey y'all, can you try the following:

Close Storage Explorer
Open keychain
Select the "login" keychain on the right hand side
Click the padlock icon to lock the keychain (the padlock will animate to a locked position when complete, it may take a few seconds depending on what apps you have open)

Launch Storage Explorer
A pop up should appear saying something like "Service hub wants to access the keychain", enter your Mac admin account password and click Always Allow (or Allow if Always Allow is not available)
Try to sign in

That worked thanks!

From: Matthew Rayermann<EMAIL_ADDRESS>Sent: Monday, April 23, 2018 11:43:38 AM
To: Microsoft/AzureStorageExplorer
Cc: Mike Leslie7; Mention
Subject: Re: [Microsoft/AzureStorageExplorer] Authentication Error on build 20180415.2 (#47)
Hey y'all, can you try the following:

Close Storage Explorer
Open keychain
Select the "login" keychain on the right hand side
Click the padlock icon to lock the keychain (the padlock will animate to a locked position when complete, it may take a few seconds depending on what apps you have open)
[image]https://user-images.githubusercontent.com/21112697/39140751-6f08d00c-46da-11e8-8f8f-14fd25d1f5fb.png
Launch Storage Explorer
A pop up should appear saying something like "Service hub wants to access the keychain", enter your Mac admin account password and click Always Allow (or Allow if Always Allow is not available)
Try to sign in

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/AzureStorageExplorer/issues/47#issuecomment-383642758, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AM2S_6tcvTgxSz_A1DtH2EI8a4C4Xv68ks5trgS6gaJpZM4TavAd.

Awesome!!!! :D I'll get this added to our documentation.

I have exactly the same error with version 1.2.0,
but steps you described don't solve the problem for me.
The login popup after locking the Keychain does not opens.
I do not have in the Keychain the item "Microsoft.Developer.IdentityService"
here there is what i find in the logs of Service Hub:
`
alberto$ cat AadAccountProviderService-50236-*
07/06/2018 12:57:17 : Error : SecKeychainFindGenericPassword, for serviceName Microsoft.Developer.IdentityService and accountName AdalCache
Status: '-25300'
Failed, item not found
07/06/2018 12:57:17 : Error : SecKeychainFindGenericPassword, for serviceName Microsoft.Developer.IdentityService and accountName AdalCache
Status: '-25300'
Failed, item not found
07/06/2018 12:57:17 : Error : SecKeychainFindGenericPassword, for serviceName Microsoft.Developer.IdentityService and accountName AdalCache
Status: '-25300'
Failed, item not found
07/06/2018 12:57:17 : Error : SecKeychainFindGenericPassword, for serviceName Microsoft.Developer.IdentityService and accountName AdalCache
Status: '-25300'
Failed, item not found
`


Remove obsolete DeferNested config switch

The switch is intended to control a core feature. The feature is never disabled in any shipping or testing scenario, and the switch probably no longer has the expected effect.

@dotnet-bot test Windows x64_debug

@dotnet-bot test Windows x86_debug

@boingoing Can you review? (Thanks.)


cmd/go2go: nonsensical recursive type compiles

Here are two types (playground):
type A[type X] interface{ a(X) }
type B[type X A[B]] interface{ b(X) }

The type A[B] is semantically invalid; B needs to be instantiated.  But this seems to be allowed currently.
In fact if we modify A a bit we can even supposedly implement B (playground):
type A[type X] interface{ b(B[X]) }
type B[type X A[B]] interface{ b(X) }
type S struct{}
func (_ S) b(A[int]) {}
var _ B[A[int]] = S{}

This code now reports a proper cycle error (playground). Closing.


SalesChannel analytics code cannot be deleted in administration after it is set once

1. Why is this change necessary?
You can't remove the google anayltics id in the saleschannel settings in administration because the sales_channel_analytics table doesn't allow a null value for the column tracking_id.
2. What does this change do, exactly?
When the trackingId is not set, it will delete the analytics table entry and removes the analyticsId FK in saleschannel.
3. Describe each step to reproduce the issue or behaviour.
Add new saleschannel
Add an anayltics code in analytics tab
save
Remove it and save
The request won't be successful
4. Please link to the relevant issues (if any).
5. Checklist

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

Code fixed.


[CALLSTACK] ALlow callstack retrievel through a RESTFul call (Next de…

…livery will be the JSONRPC stack :-))

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 3 files pending identification.

Protex Server Path: /home/blackduck/github/Thunder/588/rdkcentral/Thunder

Commit: b0b91361a1f2a39e64a5010135a4d095869c4232

Report detail: gist


[PE-D][Tester D] Error message for invalid add command could be more specific about which compulsory field is missing

Considering how long the add command can get due to the large number of fields, it can be hard to spot which compulsory field is missing. The error message could be more specific about which compulsory field is missing to aid the user.

Labels: severity.Medium type.FeatureFlaw
original: zrei/ped#9

Closed as a feature flaw that we're not allowed to fix in v1.4


Volatile error recovery output?

Over at https://github.com/PRQL/prql/pull/2002, I'm seeing that our tests are producing different output on each run.
So running cargo insta test --accept on that branch gives:
diff --git a/prql-compiler/src/test.rs b/prql-compiler/src/test.rs
index 6958f7e..9036579 100644
--- a/prql-compiler/src/test.rs
+++ b/prql-compiler/src/test.rs
@@ -2966,7 +2966,7 @@ fn test_errors() {
        │
      1 │ Mississippi has four S’s and four I’s.
        ·                                       ┬
-       ·                                       ╰── identifier expected * or an identifier, but found
+       ·                                       ╰── identifier expected an identifier or *, but found
     ───╯
     "###);

...and running it again will give a slightly different result.
Is this expected?
(I'm not asking anyone to debug PRQL's tests... Just from the Chumsky perspective, is this expected / is there some way of handling?)

If you're using the built-in Simple error type, this is expected - it stores expected tokens as a HashSet, which has non-deterministic ordering. That, in turn, means error display may place tokens in a random order each run.

Thanks for the swift reply.

If you're using the built-in Simple error type, this is expected - it stores expected tokens as a HashSet, which has non-deterministic ordering. That, in turn, means error display may place tokens in a random order each run.

Is there a reasonably close way of having it be deterministic? We'd be happy to do something hacky. Or would it require a fairly deep change in Chumsky?
The non-determinism is difficult to test around — it would be tractable if it were just a few instances, but this would mean all our error messages will be volatile.

For a project as large as PRQL, I'd recommend having a custom error type anyway: it gives you more precise control what errors are reported, what information they contain, and how they're combined.
The non-determinism is indeed a little annoying, I'll look into using a static RNG for the HashSet.

Great; it sounds like we create the message ourselves and so can sort the items before displaying.
Thanks for the help! I'll close


Gmail Update

Added new features, colors fixes and revamped code.

I'm seeing this:

Is this correct? By looking at the code it looks like it should be inbox, snoozed, done.
How does it look at your end?

Bug: The Compose floating action button disappears when the left menu is hidden.

Is this correct? By looking at the code it looks like it should be inbox, snoozed, done.
How does it look at your end?

For me, It's called "All Mail" instead of "Done" as well, although I've dragged it up out of the More section.

Style issue: The current Gmail left menu, as well as the real Inbox's left menu, animate left-right when being displayed, rather than suddenly appearing.

@boukestam I'd recommend taking another look at this, since the recent commits.

Thansk guys, I'm testing right now
I have found a small styling issue with the box shadow being cut off

Also in the left sidebar it's still Inbox, Starred, Snoozed. I thought the reorder code was supposed to make it Inbox, Done, Snoozed? Maybe I just don't understand it correctly, but looking at the code that is what is looks like. Also All Mail is not being renamed to Done, which it also looks like it should do.
Is this right?

@boukestam I don't think it should be called Done, because All Mail contains ... all mail 😄both archived and unarchived. But I agree, if it's possible, it should be moved up next to Inbox and Snoozed.
@g60madman let me know if you're working on that already, or if I should.

These new commits should fix this issue.

To actually turn All Mail into Done, we could point it to 'in:archived'.

Oh, I see, @g60madman your code attempts to add a link to a search for '-is:inbox' which is similar to 'in:archived'.
Do you know why that code's not working?

I made some progress on 'Done'. It's not renamed to Done yet, but it goes to /#archive. Also, it moves up next to Inbox and Snoozed... but only if you click the More/Less button in the inbox/folder list.

This change for some reason seems to break the Trash and Spam links. They're still there, but clicking them does nothing. Investigating now...

Alright, I figured out why more had to be clicked:
The code making this change waits to find all of the menu nodes. If they're hidden in More, it can't proceed, because it would encounter errors trying to attach events to an element/node that doesn't exist. My simple 'fix' has one problem: When clicking away from Done, it will only be un-highlighted if you click on Inbox or Snoozed. If you're looking at Done, and switch to a different folder that's not Inbox/Snoozed, it will stay highlighted along with the new one you've clicked.

And... fixed! Please check this out and test it @boukestam @g60madman

Looks good! The sidebar is now almost exactly like inbox. I've tested it a bit and found no more problems. If you guys can also test a bit, then if you agree I will publish a new version to the store and merge with the main repository. Btw, the extension is doing great:

We are also getting a lot of feedback which it amazing to read

Awesome :) I haven't encountered any issues so far. The only thing I'd want to change is: If the user didn't have All Mail dragged into the top area already, there will be another All Mail in the More section, in addition to the Done up top. The one below correctly points to /#all, the only issue is that the icon is the Done checkmark.
I'm not sure this issue is important important enough to delay release, but up to you!

If that is a simple fix then I can wait for that. It could annoy a few people who have expanded the more section. If it is more difficult than I will just publish since the sidebar is overall much better I think, even with that issue.

@boukestam Fixed the Done/All Mail icon problem.

@boukestam I say push it up is up! Also might be helpful to list the new github and maybe list the developers as well ;)

@g60madman I just set the homepage link to this repo in the manifest.json. That's where you go when you right click the extension and click the first option, 'Inbox in Gmail'.
@boukestam should add it as the homepage on the Chrome Web Store as well

@russeldc Nicely done!

I found a small issue. When you star an email all the way to red star it shows this

I got the red button fixed but it's showing the stupid red box  on top of it
/* Red stared with redband */
td.apU.xY .T-KT.xg::before {
background-image: url('https://i.ibb.co/pvycvpK/ic-mark-red-bang-clr-24dp-r4-2x.png') !important;
background-size: 24px !important;
}

@g60madman I don't use stars, how would I get that to show to test this?

@russelldc you will need to just check mark an email it will go blue, if you check mark it again it will show red.

I am going to work on pulling all the images down and host them external. I am worried Google may kill the links eventually to the static images.

@g60madman Why another random image host, and not just in the icons folder of the extension?

We can actually store the images in an images folder in the extension root. Then we can refer to those images with chrome.runtime.getURL

@boukestam Nice job responding the same exact time as me, but nothing can stop @g60madman  :P

Fail...I will work on moving them into the folder

OK I am failing on getting the images to pull from the local folder. Can some one take care of that. Here is the images.
images.zip

I want to get that favicon pull integrated asap... after I added the styling to hide checkboxes when not checked or hovering, the emails look so sad and plain.

I found this link to fix the images to load locally, but still can't figure it out
https://stackoverflow.com/a/32130426

We could use the Base64 method, considering these are pretty small, simple images.
For example, here's the Pin icon:
url('data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAADAAAAAwCAQAAAD9CzEMAAAAVUlEQVR4Ae3WsRHAIAxDUY3KZhqNUUiZCl+KyJcc/7vXK2gQER3Rqu8HgMt5S0nC6XcQAAAAQBMwNLfA1MiNv4Ksh/ddYAfGAgAAaAB8fxhzhEXU3AXHwUDMUEVsEgAAAABJRU5ErkJggg==')

A little unwieldy... I'll look into fetching from the images folder when I get back to my desk later.

Oh right, nice job remembering to bring over the empty inbox Sun 👍
There's a very thin line of blue at the bottom of it, though.
And about changing the title on the header to Inbox, I really want to programmatically change that depending on the inbox folder you're in, see: Issue #10

@russelldc  Check out this commit
https://github.com/kallepersson/inboxtheme/commit/d649cf7c7aa61da1a219e63e64318b227e262422
There is some java that goes along with to also change the tool bar color when switching between, snoozed, etc
https://github.com/kallepersson/inboxtheme/blob/master/src/menu.js#L28-L49

@russelldc here is the plugin source code. I already ran the sh script they have so you can just load it and see they have the top bar working properly based on switching from section to section i.e. inbox, sent, etc.
inboxtheme-master.zip

@g60madman Hmm.. as far as I know, that code is what switches the highlight on the left menu. And it's in our project already.

@russelldc got ya, well they have code to switch the top bar and also the title in their code

@russelldc here is the working one
inboxtheme-master.zip

No need to compile and post that, we can just look at the source code. I'd get this done now, but I've gotta jet to an event.
Looks like this css is relevant:
/* Toolbar background */
._in header { background-color: #898984 !important; }
._in[data-hash*="#inbox"] header { background-color: #4285f4 !important; }
._in[data-hash*="#starred"] header { background-color: #FDB500 !important; }
._in[data-hash*="#snoozed"] header { background-color: #ef6c00 !important; }
._in[data-hash*="#archive"] header { background-color: #0f9d58 !important; }

(I changed the #search/is-inbox to #archive)

Keep in mind, if we're adapting this code, as per the MIT license, @boukestam needs to add a LICENSE file to this repo as well.

@boukestam @russelldc I already pulled some of their code so wee need to add the MIT license.

@russelldc  here are more icons I added for all the pin colors
images.zip

@russelldc any luck getting the colors to change when clicking on ingox, snoozed, etc? It's still open on the https://github.com/boukestam/inbox-in-gmail/issues/10

@g60madman No, but I'm pretty sure I know how to get it done. I've been sick the past 2 days, unfortunately. I might be able to tomorrow.

@russelldc unplug brother and rest. Like Hogan says


I need the country mask format for force number validation

I use react-hook-form and to force the user to enter a correct number, I need the mask format for validation
Usage example for solution:
      <PhoneInput
        defaultCountry="br"
        onChange={(phone, country, mask) => {
          console.log(phone, country, mask);
        }}
      />

Is it possible?

@matheusmirandaferreira Hey
You can get country mask right now using this workaround:
onChange={(phone, country) => {
  const format = getCountry({ field: "iso2", value: country }).format
}}

NOTE: Ability to get mask from callback argument will be added in next major release.

@goveo Thanks !

@matheusmirandaferreira v4.0.0 has been released.
Check out the migration doc:
https://react-international-phone.vercel.app/docs/Migrations/migrate-to-v4
Now you can use getActiveFormattingMask helper function:
https://react-international-phone.vercel.app/docs/Usage/GetActiveCountryMask


[BUG] gradients of Operations with multiple consumers

Describe the bug
Creating operations with multiple consumers causes a NULL gradient to be assigned in build grad see here.
To Reproduce
Define an operation with multiple consumer Operations.
Expected behavior
The gradients to be summed over these consumers.
Environment:
All installs.

Fixed in a5b45c5ffbded1103116f3db12c5f99def85d55a.
Soon this will make its way into dev and once the tests pass into master.


fuse mount losing sticky bit

# mkdir r; dd if=/dev/random bs=32 count=1 of=p status=none; ./restic-0.16.0 version; ./restic-0.16.0 -q init -r r -p p; ./restic-0.16.0 -q -r r -p p backup /tmp; ./restic-0.16.0 -q -r r -p p mount /mnt
restic 0.16.0 compiled with go1.20.6 on linux/amd64
Now serving the repository at /mnt
Use another terminal or tool to browse the contents of this folder.
When finished, quit with Ctrl-c here or umount the mountpoint.

# rm -rf foo; cp -a /mnt/snapshots/latest/tmp foo; ls -lahd /tmp /mnt/snapshots/latest/tmp foo
drwxrwxrwx 14 root root 4,0K Sep 27 10:18 foo
drwxrwxrwx  2 root root    0 Sep 27 10:18 /mnt/snapshots/latest/tmp
drwxrwxrwt 14 root root 4,0K Sep 27 10:18 /tmp

I expected the sticky bit to be present on the /tmp directory in the fuse mount, but it wasn't.

Clearly a defect in restic mount. The tarball from restic dump does have the sticky bit on the directory.

I would like to help with this issue
@greatroar can you tell me how did you check with restic dump ? I don't see the sticky bit even with dump command
batman@fedora:~/fun/restic$ ./restic -r ../test dump latest tmp > dumpp.tar
enter password for repository:
batman@fedora:~/fun/restic$ ls -lahd dumpp.tar
-rw-r--r--. 1 batman batman 247K Dec 24 01:34 dumpp.tar
batman@fedora:~/fun/restic$ rm -rf tmp/
batman@fedora:~/fun/restic$ tar -xf dumpp.tar
batman@fedora:~/fun/restic$ ls
build.go      cmd              doc     dumpp      go.mod         helpers   Makefile   restic  VERSION
changelog     contrib          doc.go  dumpp.tar  go.sum         internal  README.md  steps
CHANGELOG.md  CONTRIBUTING.md  docker  foo        GOVERNANCE.md  LICENSE   res        tmp
batman@fedora:~/fun/restic$ ls -lahd tmp/
drwxr-xr-x. 1 batman batman 1.1K Dec 24 01:35 tmp/
batman@fedora:~/fun/restic$

I did RCA of this issue

This issue existed because in<EMAIL_ADDRESS>sticky bit was silently just being dropped (SUID and SGID were maintained just fine). This issue was fixed in the commit - https://github.com/anacrolix/fuse/commit/d711f1e6391d975060229ebcd53168d6c012e74a
Restic is currently using v2 of this fuse library but the fix above is there in v3
I tried upgrading to v3 but it seems that it had issues working on linux (NOTE: The latest master of fuse works fine - https://github.com/anacrolix/fuse/commit/c6729d8bb2af377b5df04d4efdc425526cb764f3)
With the latest master, I can see the sticky bit persisting in the mounted directory

I've taken a closer look at the changes in v0.3 of the fuse library and found a few problems that are fixed in https://github.com/anacrolix/fuse/pull/6. These changes untested so far, as I don't have a mac anymore. Once the bugfixes (in particular the crash fix) have been merged, we could just go forward and bump the dependency.

Restic 0.17.0 will include version 0.3.1 of anacrolix/fuse, which includes the bugfix for the sticky bit.


Localrules are ignored for grouped jobs

I am using the latest versions of snakemake (8.25) and am trying to use the executor plugin (0.11.1) for slurm for my pipeline.
However, when I use the command-line to run snakemake with the slurm executor all rules are considered SLURM-rules. That is to say, neither localrules: a, b, c nor the per-rule localrule: True directive have any effect. After some digging I found that this only occurs if the group directive is used on the respective rules. I.e. if I remove the grouping then everything runs fine and localrules are recognized properly.
Minimal Snakefile if you want to try it
The rule a belongs to group local

DIRECTORY = "test"

rule all_a:
    input:
        [f"{DIRECTORY}/test_file-a-{i}.txt" for i in range(10)],
    localrule: True

rule a:
    input:
        directory(DIRECTORY)
    output:
        DIRECTORY + "/test_file-a-{i}.txt"
    localrule: True
    group:
        "local"
    shell:
        "touch {output[0]}"

Running all_a reveals that both all_a as well as a are considered normal rules rather than localrules
$ snakemake --executor slurm -R all_a

Job stats:
job      count
-----  -------
a           10
all_a        1
total       11

Select jobs to execute...
Execute 10 jobs...

[Fri Nov  1 13:57:51 2024]
rule a:
    input: test
    output: test/test_file-a-4.txt
    jobid: 5
    reason: Missing output files: test/test_file-a-4.txt
    wildcards: i=4
    resources: mem_mb=<TBD>, disk_mb=<TBD>, tmpdir=<TBD>, slurm_account=<TBD>

[Fri Nov  1 13:57:51 2024]
rule all_a:
    input: test/test_file-a-0.txt, test/test_file-a-1.txt, test/test_file-a-2.txt, test/test_file-a-3.txt, test/test_file-a-4.txt, test/test_file-a-5.txt, test/test_file-a-6.txt, test/test_file-a-7.txt, test/test_file-a-8.txt, test/test_file-a-9.txt
    jobid: 0
    reason: Forced execution
    resources: mem_mb=<TBD>, disk_mb=<TBD>, tmpdir=<TBD>, slurm_account=<TBD>

on the other hand if we remove the group clearative and run the same command in the terminal we get:
$ snakemake --executor slurm -R all_a

Job stats:
job      count
-----  -------
a           10
all_a        1
total       11

Select jobs to execute...
Execute 10 jobs...

[Fri Nov  1 14:05:23 2024]
localrule a:
    input: test
    output: test/test_file-a-4.txt
    jobid: 5
    reason: Missing output files: test/test_file-a-4.txt
    wildcards: i=4
    resources: mem_mb=1000, mem_mib=954, disk_mb=1000, disk_mib=954, tmpdir=/tmp, slurm_account=bczk-delta-gpu

[Fri Nov  1 14:05:23 2024]
localrule all_a:
    input: test/test_file-a-0.txt, test/test_file-a-1.txt, test/test_file-a-2.txt, test/test_file-a-3.txt, test/test_file-a-4.txt, test/test_file-a-5.txt, test/test_file-a-6.txt, test/test_file-a-7.txt, test/test_file-a-8.txt, test/test_file-a-9.txt
    jobid: 0
    reason: Forced execution
    resources: mem_mb=1000, mem_mib=954, disk_mb=1000, disk_mib=954, tmpdir=/tmp, slurm_account=bczk-delta-gpu

indicating that now the rules are indeed localrules.
PS: as you can probably see from the copy/paste output, the actual commands used  also specified some resources and partitions which is actually how I found the bug in the first place as I was trying to run simple CPU stuff on a GPU partition since SLURM is only supposed to be used by gpu-using rules in my pipeline...
Thanks a lot already,
Cheers,
Noah  :sunny:

Yes, I realized that myself - and in all my stress, I did not open an issue report :-(. Thanks for doing so. It's on my to-do list.


Adding Timezone, Page, and Screen to Outbound Integration

@stevenmiller888 @f2prateek @amillet89

@f2prateek addressed!

Is this considered a breaking change? Will customers be upset if they suddenly start seeing page/screen calls show up in Outbound?

@f2prateek no. They have been asking for it.


Replace RocksEngine with KvEngine type parameters in tikv::server

What problem does this PR solve?
Making the tikv crate generic over storage engines.
Part of https://github.com/tikv/tikv/issues/6402
What is changed and how it works?
Just replacing instances of RocksEngine with KvEngine-bound type parameters in the tikv::server module.
I discovered that KvEngines need to be Unpin here, so I went ahead and made all KvEngines Unpin so engine authors can find that out early.
Related changes
Check List
Release note

No release note.

@Connor1996 I think you've wanted in the past to not add more engine type parameters here, but I don't know how to avoid them. Could you review?

/lgtm

/merge

/merge


bug: commit confirm window in neogit shows nothing

Did you check docs and existing issues?

[X] I have read all the noice.nvim docs
[X] I have searched the existing issues of noice.nvim
[X] I have searched the exsiting issues of plugins related to this issue

Neovim version (nvim -v)
0.8.3
Operating system/version
6.2.2-arch1-1
Describe the bug
When the commit confirm window of neogit shows up, it was blank.

Steps To Reproduce

commit something in neogit
:wq
Confirm window pops up and it shows nothing

Expected Behavior
Yes and no options should appear on the window.

Here is my config:
local lazypath = vim.fn.stdpath('data') .. '/lazy/lazy.nvim'
if not vim.loop.fs_stat(lazypath) then
    vim.fn.system({
        'git',
        'clone',
        '--filter=blob:none',
        'https://github.com/folke/lazy.nvim.git',
        '--branch=stable',
        lazypath,
    })
end
vim.opt.rtp:prepend(lazypath)

require('lazy').setup({
    {
        'TimUntersberger/neogit',
        dependencies = {
            'sindrets/diffview.nvim',
            'nvim-lua/plenary.nvim'
        },
        config = function()
            require('neogit').setup {
                disable_signs = false,
                disable_hint = false,
                disable_context_highlighting = false,
                disable_commit_confirmation = false,
                auto_refresh = true,
                disable_builtin_notifications = false,
                use_magit_keybindings = false,
                kind = "tab",
                console_timeout = 2000,
                auto_show_console = true,
                popup = {
                    kind = "vsplit",
                },
                signs = {
                    -- { CLOSED, OPENED }
                    section = { ">", "v" },
                    item = { ">", "v" },
                    hunk = { "", "" },
                },
                integrations = {
                    diffview = true
                },
                sections = {
                    untracked = {
                        folded = false
                    },
                    unstaged = {
                        folded = false
                    },
                    staged = {
                        folded = false
                    },
                    stashes = {
                        folded = true
                    },
                    unpulled = {
                        folded = true
                    },
                    unmerged = {
                        folded = false
                    },
                    recent = {
                        folded = true
                    },
                },
            }
        end,
    },
    {
        'folke/noice.nvim',
        dependencies = {
            'MunifTanjim/nui.nvim',
        },
        config = function()
            require('noice').setup({
                cmdline = {
                    view = 'cmdline',
                },
                lsp = {
                    override = {
                        ['vim.lsp.util.convert_input_to_markdown_lines'] = true,
                        ['vim.lsp.util.stylize_markdown'] = true,
                        ['cmp.entry.get_documentation'] = true,
                    },
                },
                presets = {
                    bottom_search = true,
                    command_palette = false,
                    long_message_to_split = true,
                    inc_rename = true,
                    lsp_doc_border = false,
                },
            })
        end,
    },
})

Hi, is there any update on this?

 I don't use neogit, so not planning to look into this.

I dont know how to do it, can anyone please help?

I think this is a duplicate of #232, which is fixed by TimUntersberger/neogit#441. As mentioned there, you can use disable_commit_confirmation = true in the meantime, so the pop-up won't appear at all.


#1430 Avoid exceptions when running in the headless browser mode

Hi @marijnh,
I've added some checks for undefined values to prevent exceptions occuring when partial browser functionality used during headless Edge runs.
These checks don't change any of the existing logic. My test running smoothly with these fixes.
I would greatly appreciate it if these changes could be released asap.
Thank you!
https://github.com/codemirror/dev/issues/1430

Sorry, no. I'm willing to add workaround for popular browsers, but not for obscure test runners.

Sorry, no. I'm willing to add workaround for popular browsers, but not for obscure test runners.

Sorry to hear.
I would like to try to insist to check this again while this is not obscure browsers and this direct usecase when people are performing testing in the ci/cd pipelines.
The function that is fixed used for the printing without any check is deprecated , and this is really obscure.
Please make a great tool usable.

The function that is fixed used for the printing without any check is deprecated , and this is really obscure.

It's only used when the platform doesn't support MediaQueryList.removeEventListener, which is standard and supported on all browsers that have MediaQueryList.


Add check to avoid applying cover to viewer token

https://ptb.discord.com/channels/915186263609454632/1149317226655129600/1167204010370007060

Should not be an issue with the improvements in upcoming v0.6.0


No implementation found for void.com.unistyles

Description
https://github.com/jpudysz/react-native-unistyles/assets/51382079/708a1e92-f8ce-4fce-a708-d5494f4e667c
Steps to reproduce

Init a fresh project with official expo template
add react-native-unistyles
Build using EAS
Launch app and swith theme

Snack or a link to a repository (optional)
No response
Unistyles version
^2.0.0-rc.4
React Native version
0.72.6
Platforms
Android
Engine
Hermes
Architecture
Paper (old)

I think I probably know what it is. Can you confirm that by EAS you bundled in release mode?

I'm gonna need more details as I'm not able to reproduce with my demo app.
eas build --platform android --local

https://github.com/jpudysz/react-native-unistyles/assets/9088288/f47cd048-f380-4032-b184-c8f89fe2cfc2

Also:
Here is my official repo for expo router:
https://github.com/jpudysz/react-native-unistyles-expo-router
Steps to reproduce:

Setup project

clone repo
upgrade unistyles to RC4
yarn expo prebuild

setup eas.json

Build project

eas build --platform android --local

https://github.com/jpudysz/react-native-unistyles/assets/9088288/570e0ea0-c9da-4a84-aa69-42e37468e4fb
Feel free to open with more details or steps to reproduce/repo.


Multi class species detection model suggestions

Dear all,
This is not an issue reporting but rather a request for guidance pertaining to image sizes, number of classes, data quantity and training on a new multi class model training.
The image files I am using for training are quite large in size ranging from 12563512 pixels (4516 × 2782) to 83678686 (12523 × 6682). Several such image files are used with a single csv file for annotations. There are a total of 2903 annotations across 13 classes.
In general, the code works for me but the predictions are very poor even on the file used for training. The scores for predictions are very low ~0.01 and the bounding boxes don't look right. I am presently unable to train the model for larger epochs due to hardware limitations. Any suggestion this would also be welcome. There is a chance that we might be able to get access to google cola pro or azure sagemaker and that may solve some of these problems.
In general, looking to get an advice on the approach and the architecture. Here is the code.
Dictionary labels are taken from data frame but here is standalone creation
dict_labels = {'Sitaphal': 0, 'Other': 1, 'Mango': 2, 'Lemon': 3, 'Tree': 4, 'Chinch': 5, 'Coconut': 6, 'Jambhul': 7, 'tree': 8, 'Coconut tree': 9, 'chinch': 10, 'coconut': 11, 'w': 12, 'Orange': 13}
Initializing a multi class model
multi_class_m = main.deepforest(num_classes=len(dict_labels), label_dict=dict_labels)
deepforest_release_model = main.deepforest()
deepforest_release_model.use_release()
multi_class_m.model.backbone.load_state_dict(deepforest_release_model.model.backbone.state_dict())
multi_class_m.model.head.regression_head.load_state_dict(deepforest_release_model.model.head.regression_head.state_dict())
Setting configuration for the multi class model
multi_class_m.config["train"]["csv_file"] = get_data("automated_all_annot.csv")
multi_class_m.config["train"]["root_dir"] = os.path.dirname(get_data("automated_all_annot.csv"))
multi_class_m.config["train"]["fast_dev_run"] = False
multi_class_m.config["batch_size"] = 5
multi_class_m.config["train"]['epochs'] = 10
multi_class_m.config["workers"] = 2
Training
multi_class_m.create_trainer(log_every_n_steps=1)
multi_class_m.trainer.fit(multi_class_m)
assert multi_class_m.num_classes == len(dict_labels)
Prediction: score_threshold is kept 0 to see whatever is predicted
test_predict = get_data("AMD-SRD-123456002-PTV-EP01.jpg")
multi_class_m.config["score_thresh"] = 0.0
boxes = multi_class_m.predict_image(path=test_predict, return_plot = True, thickness=10, color = (0, 165, 255))
predictions_df = multi_class_m.predict_image(path=test_predict, return_plot = False, thickness=10, color = (0, 165, 255))
Here is a google drive link to download the folder containing all image files and the annotation csv.
Thank you Ben and all the development team for making this available for us! Much much appreciated. Looking forward to your comments and suggestions.

Let's close this and make a discussion. Easier to track.  https://github.com/weecology/DeepForest/discussions/458


First party CRDT support

Probably implemented as a Syncable wrapper around a Store.

Coming in https://beta.tinybase.org/guides/releases/#v5-0 - how exciting!


In the described scenario, after the moment of committing an entry on the leader, it takes HeartbeatInterval + 3/2 * RTT until the follower learns this entry is committed.
This is suboptimal, and could take HeartbeatInverval + 1/2 * RTT if the leader didn't cut the commit index at Progress.Match before sending it to the follower.
Informs #138

@nvanbenschoten @ahrtr PTAL.

This is suboptimal, and could take HeartbeatInverval + 1/2 * RTT if the leader didn't cut the commit index at Progress.Match before sending it to the follower.

Why do we care? Follower reads?

Why do we care? Follower reads?

Yes, and also the general "convergence" of the system. For other kinds of reads, like quorum reads, this also seems relevant.

@ahrtr Good to merge?


Keep VSI filenames in sync between write and read

As @rbuffat suggested, we will use driver-supported file extensions when creating a Collection in the context of a default, unnamed MemoryFile.

Coverage decreased (-0.9%) to 85.448% when pulling c16b7f9820a2ce55da465b8197f7aa053d69b09c on vsi-file-extensions into 7559619bef3e2e3a2d712b674c3c7c358551ccae on maint-1.8.


Max pixel text contents too close

First bug: At 200px on my iPhone the text contents have no padding so when two pages are adjacent the words run into each other.

@james-xli can you take a look at this?

Yes, thanks! I see the issue. Will try to get to this in the next evening or two if I can, before I have to travel for the holidays.


Include @types/react as a dependency when running backstage CLI create-plugin command

Feature Suggestion
@types/react is not included as a dependency by the create-plugin backstage CLI command, but I believe it is needed.
However, if it is not really needed, please close this issue.
Possible Implementation
Include @types/react v. 16.9 as a dependency in the package.json generated by the create-plugin command in the backstage CLI.

Closing since it isn't required for all plugins, and should thus just be added manually. https://github.com/backstage/backstage/pull/6509#issuecomment-882354759

@rubenLindstrom can the issue be closed as well?


move docs from nebula-java to nebula-docs-cn

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: Amber1990Zhang:white_check_mark: oldLady344:x: wfystxYou have signed the CLA already but the status is still pending? Let us recheck it.

will open another pr to handle this.


PatchSeries: add a signed-off-by line for the original author

I don't have a local setup to test this, but in theory it ought to work.

Seems my settings were wrong. I didn't get a message about your other PR from today either.


Update dependencies, enable DB backed conversation logs

Required DB migrations for the new tables.

Pull Request Test Coverage Report for Build 3400

7 of 36 (19.44%)  changed or added relevant lines in 3 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.2%) to 66.847%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

go/sql/user_conversation.go
3
32
9.38%

Files with Coverage Reduction
New Missed Lines
%

src/routes/commandpedia.ts
1
66.67%

src/model/entity.ts
2
76.92%

Totals

Change from base Build 3390:
-0.2%

Covered Lines:
7237

Relevant Lines:
10201

💛  - Coveralls


Bug: Double click on  -> double push to history (no change location.path)

react-router v4

go double click a normal link on a page without JS. same thing :)

I have same problem.


loc loss nan

When I train the mobile0.25 on widerface, sometimes the loc loss is nan.

https://github.com/biubug6/Pytorch_Retinaface/issues/10#issuecomment-567029509

when i get this error, i print the values of loss_t,  loc_t and loc_p. and i found the min(loc_t) in smooth_l1_loss is -inf, this result in inf in loss_t, but i dont know how -inf produced in loc_t?

I just discard the batch like this..., and have not made in-depth analysis to it.

The error may caused bydata_augment.py and box_utils.py ,data_augment.py  maybe pass  gt_box with the area of zero ,and data_augment.py may pass it to calculate log() or other function.In data_augment.py.The reason is that multi gt_box matching the same anchor , after the function "match" in box_utils.py (especially in line 143 of "data_augment.py"),may lead one anchor match a new wrong gt_box but class is not set to zero because of original overlap.


Not Seeing QuestProOpenXRTrackingModule

I'm running Windows 11.
I do see "Quest Pro OpenXR" in modules though.
When I install it with Air Link open, it just displays like a missing image and says Loading...
I see my Quest Pro on the home tab with an "Outgoing" rate, but the "Incoming" rate is always 0 m/s.
Avatar Status is always just "Loading..." even with VRC open through Steam VR.
Log after launching VRCFaceTracking:
[VRCFaceTracking.Services.ActivationService] Information: VRCFT Version <IP_ADDRESS> initializing...
[VRCFaceTracking.Services.ActivationService] Information: Initializing OSC...
[VRCFaceTracking.Services.ActivationService] Information: Initializing main service...
[VRCFaceTracking.Services.ActivationService] Information: Checking for updates for installed modules...
[VRCFaceTracking.Services.ActivationService] Information: Initializing modules...
[UnifiedLibManager] Information: Starting initialization tracking
[UnifiedLibManager] Information: Tearing down all modules...
[UnifiedLibManager] Information: Initializing runtimes...
[UnifiedLibManager] Information: Initializing VRCFT - Quest OpenXR, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[UnifiedLibManager] Information: Loading External Module VRCFT - Quest OpenXR, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null
[QuestProTrackingModule] Information: [QuestOpenXR] OpenXR Runtime init success.
[UnifiedLibManager] Information: Tracking initialized via VRCFT_Module___QuestOpenXR.QuestProTrackingModule

The module initialized correctly and you're getting face tracking data. the Incoming rate is OSC messages (from VRChat). There is some OSC networking related problem with the game, there appears to be nothing wrong with the module. Next time raise the issue in the Discord first.

Stale


Routing to view a given source

At the moment I can go to
http://gammapy.github.io/web-experiments/
and select a source via the "source name" field and then the info for that source is displayed.
But the URL is still
http://gammapy.github.io/web-experiments/
@kepta - Is there a way to do routing, so that it's possible to hit e.g.
http://gammapy.github.io/web-experiments/?cat=3fgl&source=42
and have the info for that source displayed?
Is this a lot of work or something that can be implemented relatively quickly?
Context: This is something I would like to have for
https://github.com/cdeil/hgps-webpage/blob/master/src/js/map.js#L81
there the tooltip displays the Fermi catalog name, and I'd like to link to this webpage (and in the future gamma-sky.org) to have the catalog info for this source listed.

sure, I will try to add a PR fixing this by today.

@cdeil the source name is generally of the format  2FHL J0213.9-6949.
how do you want me to interpret  cat=3fgl&source=42

I think using the row index number for the source would be simplest, no?
Alternatively, we could do things like mapping "2FHL J0213.9-6949" to "2FHL_J0213.9m6949" for the URL.

I have a question concerning routing from Javascript (which I've never implemented).
Does it have to be BASE_URL?cat=3fgl&source=42 or is it also possible to have BASE_URL/3fgl/42?
This is only possible if the routing is done on the server, right?

No it can be done via router implementations.
One example would be to use https://github.com/reactjs/react-router .
However, since I lack time ... I would some hot fix code, this won't be the best way to implement routing. But we can change it later.

This has been implemented in #10.
Closing this feature request.


handler/oauth2: set JWT ExpiresAt claim per TokenType from the session

Fixes https://github.com/ory-am/fosite/issues/120. The JWT strategy now uses the ExpiresAt map introduced in 0.5.0 to set expirations by fosite.TokenType. Previously the expiration was the same for all token types based on the JWTClaims.ExpiresAt field.
Signed-off-by: Cristian Graziano<EMAIL_ADDRESS>

Awesome, thanks!


Add linux node selector to bundleUnpacker job

Signed-off-by: Michelle Nguyen<EMAIL_ADDRESS>
Description of the change: Added default nodeSelector "kubernetes.io/os": "linux" to the bundle unpacker job's pod template.
Motivation for the change: For the most part, we've noticed that OLM does a good job of adding the "kubernetes.io/os": "linux" node selector by default to most of the K8s resources it deploys. However, we've noticed the bundle unpacker job does not have any nodeSelector applied to it. In multi-OS environments, this can be a problem if this job starts up on a non-linux nodes.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive
[ ] Tests marked as [FLAKE] are truly flaky
[ ] Tests that remove the [FLAKE] tag are no longer flaky

@joelanford, @njhale , would either of you be able to take a look? Thank you!

/ok-to-test

/lgtm

@aimichelle could you please update fix the failing unit test? FYI, I've rebased from this side.

Thank you! Just pushed changes to the tests. Let me know if there's anything else you need. :)


Filtering documentation needs some clarification and a running example

What does It is important to understand that when using filters in your request, the filters you specify will only be applied to the bottom-level columns. actually mean?
What is the bottom-level, what are columns? Are you talking about the top level entries in the JSON object (unlikely, at least given my tests, though that might just be broken)? Are you talking about the most deeply nested entries in that JSON, and if yes, what if there are multiple entries (with different paths) at the same depth?
It would be nice to be able to specify exactly what one wants to filter. So something like ?name=data.*.id:1234, so one can do more specific filtering.
At the very least some actual test data and examples and responses for those in the documentation would help.

Agreed and this explanation will be improved. What it meant was that you can only filter by the bottom level parameters is that nested objects in the response cannot be filtered by at the moment. As an example, this is what can be filtered by:
{
  "data": [
    {
      "id": 2, // filterable
      "game_id": 2, // filterable
      "submitted_by": {
        "id": 1,
        "name_id": "xant",
        "username": "XanT",
        "date_online":<PHONE_NUMBER>,
        "avatar": {
          "filename": "masterchief.jpg",
          "original": "https://media.mod.io/images/members/1/1/1/masterchief.jpg"
        },
        "timezone": "Australia/Brisbane",
        "language": "en",
        "profile_url": "https://mod.io/members/xant"
      },
      "date_added":<PHONE_NUMBER>, // filterable
      "date_updated":<PHONE_NUMBER>, // filterable
      "date_live":<PHONE_NUMBER>, // filterable
      "logo": {
        "filename": "IMG_20170409_222419.jpg",
        "original": "https://media.mod.io/images/mods/1/1/2/IMG_20170409_222419.jpg",
        "thumb_320x180": "https://media.mod.io/cache/images/mods/1/1/2/thumb_320x180/IMG_20170409_222419.jpg",
        "thumb_640x360": "https://media.mod.io/cache/images/mods/1/1/2/thumb_640x360/IMG_20170409_222419.jpg",
        "thumb_1280x720": "https://media.mod.io/cache/images/mods/1/1/2/thumb_1280x720/IMG_20170409_222419.jpg"
      },
      "homepage": "https://www.rogue-hdpack.com/", // filterable
      "name": "Rogue Knight HD Pack", // filterable
      "name_id": "rogue-knight-hd-pack", // filterable
      "summary": "It's time to bask in the glory of beautiful 4k textures!", // filterable
      "description": "
About

Rogue HD Pack does exactly what you thi...", // filterable
      "metadata_blob": "rogue,hd,high-res,4k,hd textures", // filterable
      "profile_url": "https://rogue-knight.mod.io/rogue-knight-hd-pack", // filterable
      "modfile": {
        "id": 2,
        "mod_id": 2,
        "date_added":<PHONE_NUMBER>,
        "date_scanned":<PHONE_NUMBER>,
        "virus_status": 0,
        "virus_positive": 0,
        "virustotal_hash": "f9a7bf4a95ce20787337b685a79677cae2281b83c63ab0a25f091407741692af-1508147401",
        "filesize": 15181,
        "filehash": {
          "md5": "2d4a0e2d7273db6b0a94b0740a88ad0d"
        },
        "filename": "rogue-knight-v1.zip",
        "version": "1.3",
        "changelog": "VERSION 1.3 -- Changes -- Fixed critical castle floor bug.",
        "download_url": "https://mod.io/mods/file/2/c489a0354111a4d76640d47f0cdcb294"
      },
      "media": {
        "youtube": [
          "https://www.youtube.com/watch?v=dQw4w9WgXcQ"
        ],
        "sketchfab": [
          "https://sketchfab.com/models/ef40b2d300334d009984c8865b2db1c8"
        ],
        "images": [
          {
            "original": "https://media.mod.io/images/mods/1/1/2/IMG_20170409_222419.jpg",
            "thumb_320x180": "https://media.mod.io/cache/images/mods/1/1/2/thumb_1020x2000/IMG_20170409_222419.jpg",
            "filename": "IMG_20170409_222419.jpg"
          }
        ]
      },
      "rating_summary": {
        "total_ratings": 1230,
        "positive_ratings": 1047,
        "negative_ratings": 183,
        "percentage_positive": 91,
        "weighted_aggregate": 87.38,
        "display_text": "Very Positive"
      },
      "tags": [
        {
          "name": "Unity",
          "date_added":<PHONE_NUMBER>
        }
      ]
    },
    {
        ...
    }
  ],
  "cursor_id": 60,
  "prev_id": 30,
  "next_id": 160,
  "result_count": 100
}

I agree with your suggestion that filtering nested objects by the * notation would be ideal, as mentioned in another thread this is not currently supported however we are investigating implementing it.

Ah, I'd have called that "properties of the data elements", or top-level (though not quite), but certainly not bottom level.
Thanks for the explanation, that cleared things up.

How does this sound:
mod.io has powerful filtering available to assist you when making requests to the API. You can filter on all columns in the parent object only. You cannot apply filters to columns in nested objects, so if a game contains a user object you cannot filter by the username column, but you can filter by the games name since the column resides in the parent object.

Somewhat better. Though "nested objects" is still slightly fuzzy. I guess an example with comments // can be filtered on for everything on the correct level would help.


Model with List field of self type adds "List" under definitions in spec

Given a model:
@ApiModel(description = "Foo class")
public class Foo {

    @ApiModelProperty(required = true, value = "some content")
    private List<Foo> foos;

    public List<Foo> getFoos() {
        return foos;
    }

    public void setFoos(List<Foo> foos) {
        this.foos = foos;
    }
}

The expected definition in the spec is:
"definitions" : {
    "Foo" : {
      "required" : [ "foos" ],
      "properties" : {
        "checks" : {
          "type" : "array",
          "description" : "some content",
          "items" : {
            "$ref" : "#/definitions/Foo"
          }
        }
      },
      "description" : "Foo class"
    }
  }

However, we are seeing:
"definitions" : {
    "Foo" : {
      "required" : [ "foos" ],
      "properties" : {
        "checks" : {
          "type" : "array",
          "description" : "some content",
          "items" : {
            "$ref" : "#/definitions/Foo"
          }
        }
      },
      "description" : "Foo class"
    },
    "List" : {
      "properties" : {
        "empty" : {
          "type" : "boolean",
          "default" : false
        }
      }
    }
  }

Note the additional List definition. The code works just fine (as expected) if the field contains List of non Foo:
@ApiModelProperty(required = true, value = "some content")
private List<Bar> bars;

Up on some initial debugging: I noticed that this issue occurs when Foo model is in the process of being resolved (ModelConverterContextImpl.processedTypes has an entry but ModelConverterContextImpl.modelByType doesn't have an entry yet). ModelResolver at Line 123 requests for the Model of the inner type of the List (in this scenario, the unresolved Foo) but a null is returned. This leads to treating the List as a "complex type".
It is very likely this issue will re-appear if Bar Class has a List ofFoo (cyclic reference)
@ApiModel(description = "Foo class")
public class Foo {

    @ApiModelProperty(required = true, value = "some content")
    private List<Bar> bars;

    public List<Bar> getBars() {
        return bars;
    }

    public void setBars(List<Bar> bars) {
        this.bars = bars;
    }
}

@ApiModel(description = "Bar class")
public class Bar {

    @ApiModelProperty(required = true, value = "some content")
    private List<Foo> foos;

    public List<Foo> getFoos() {
        return foos;
    }

    public void setFoos(List<Foo> foos) {
        this.foos = foos;
    }
}

I must also mention that this is based on observation in the master and tags/1.5.1-M2 branches.

Can you try the develop_2.0 branch? It's available as a snapshot, but would require a little extra work as we've repackaged the project as io.swagger instead of com.wordnik. Snapshot version is 1.5.0-SNAPSHOT.

Okay. I'll take a look at develop_2.0.

cloned develop_2.0 and ran mvn clean install. Got a build failure related to code coverage:
[INFO] --- jacoco-maven-plugin:<IP_ADDRESS>409121644:check (default-check) @ swagger-jersey-jaxrs ---
[INFO] Analyzed bundle 'swagger-jersey-jaxrs' with 1 classes
[WARNING] Rule violated for bundle swagger-jersey-jaxrs: complexity covered ratio is 0.00, but expected minimum is 0.90
[WARNING] Rule violated for bundle swagger-jersey-jaxrs: classes missed count is 1, but expected maximum is 0
[WARNING] Rule violated for class io.swagger.jersey.SwaggerJerseyJaxrs: lines covered ratio is 0.00, but expected minimum is 0.90
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 14.440s
[INFO] Finished at: Fri Jun 05 11:11:45 PDT 2015
[INFO] Final Memory: 32M/327M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.jacoco:jacoco-maven-plugin:<IP_ADDRESS>409121644:check (default-check) on project swagger-jersey-jaxrs: Coverage checks have not been met. See log for details. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.

The code install fine with mvn clean install -DskipTests.
Travis-CI seems to be building fine. But I see swagger-jersey-jaxrs/target/jacoco.exec which Travis can not find:
[INFO] --- jacoco-maven-plugin:<IP_ADDRESS>409121644:check (default-check) @ swagger-jersey-jaxrs ---
[INFO] Skipping JaCoCo execution due to missing execution data file:/home/travis/build/swagger-api/swagger-core/modules/swagger-jersey-jaxrs/target/jacoco.exec

@sudohippie - heh, we don't really have jacoco installed so it never fails for us, but since it's code coverage, that doesn't affect functionality. We're looking into 'fixing' it, but for now, please skip it and check the original issue.

Seems to work for self references but not for circular references - i.e. as in the Foo / Bar classes above - in that case the "top level" reference works but the reference to the contained class that refs back to the top-level class gets a List definition. I'll continue digging...

Looks like fixed at the develop_2.0 branch after the #1130 PR.

Thanks, I'll confirm later today!

confirmed - seems to work in develop branch - closing!

@olensmar please check this related issue, https://github.com/swagger-api/swagger-core/issues/1178 i think it's related but not completely solved


Clear Chat function problems

Throws an error when /clear command is called from chat
Permaban doesnt do anything in the database.

On the second bullet point, see #882 and my review comment:

The only data we store for each user is a timeout_end, as you can imagine, you can't fit permabans into that sort of model, which is why we just clear the timeout_end field on permaban.
Reason is: There's just currently no use to knowing a user is banned (so why bother implementing it/storing it?). the only thing we use timeout_end for is the paid timeout module (so you can stack timeouts).


Fix the flakiness of testNFASerialization

What is the purpose of the change

Running org.apache.flink.cep.nfa.NFATest.testNFASerialization with NonDex tool would discover flaky test.
NFATest.java has been changed to solve the flakiness.

Reproduce step

Test Environment

Apache Maven: 3.6.3
Java: <IP_ADDRESS>
OS: Ubuntu 20.04.6 LTS
Linux kernel: 5.4.0-164-generic

mvn -pl flink-libraries/flink-cep edu.illinois:nondex-maven-plugin:2.1.1:nondex -Dtest=org.apache.flink.cep.nfa.NFATest#testNFASerialization -DnondexMode=ONE

Root cause

assertEquals() at NFATest.java L308 would fail because the returned array elements of PriorityQueue.toArray() at NFAState.java L124-125 are in no particular order.
NonDex would shuffle "the returned array" in the same permutation on the first invocation of an object when nondexMode = ONE.
As a result, the returned array must be sorted to ensure that the comparison is correct.

https://github.com/NDR0216/flink/blob/b14411e7062c235cb28582a37e37decc7c425476/flink-libraries/flink-cep/src/test/java/org/apache/flink/cep/nfa/NFATest.java#L308
https://github.com/NDR0216/flink/blob/b14411e7062c235cb28582a37e37decc7c425476/flink-libraries/flink-cep/src/main/java/org/apache/flink/cep/nfa/NFAState.java#L115-L126
Brief change log

Sort the returned array of PriorityQueue.toArray()
Add the breakeven condition by using hashCode() because the original comparator COMPUTATION_STATE_COMPARATOR only compares the timestamp and ID

Verifying this change
This change is already covered by existing tests, such as org.apache.flink.cep.nfa.NFATest.testNFASerialization.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)

No

The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)

No

The serializers: (yes / no / don't know)

No

The runtime per-record code paths (performance sensitive): (yes / no / don't know)

No

Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)

No

The S3 file system connector: (yes / no / don't know)

No

Documentation

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

No

If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Your fix looks good to me. Provided a brief description of the issue, cause and good fix!

LGTM. It's good to have the testing environment stated, and the explanation for root cause is clear too!

The fix looks good to me.


Whoops

I'm still new at this whole Git thing, and I left out a file from my last pull request. Sorry.

And a new feature, too. :)

Yeah, I can't handle git :). Fixing.

Closing, new one coming up.


SCCM SCAP extension pre-release availability

The page says This version of the tool is a pre-release feature that's only available in version 1806. This version isn't certified by NIST. Is it true that it was only available in 1806? https://github.com/MicrosoftDocs/SCCMdocs/blob/master/sccm/core/get-started/capabilities-in-technical-preview-1803.md#scap-extensions suggests it was available in 1803 TP and later, but I didn't find it for 1902.

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

ID: 7a689124-c901-d2e0-508f-998b39852315
Version Independent ID: 6f6dc930-ad34-e747-f03f-93c46a3e237c
Content: SCAP extensions - Configuration Manager
Content Source: sccm/compliance/plan-design/scap/about-scap.md
Product: configuration-manager
Technology: configmgr-compliance
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

https://github.com/MicrosoftDocs/SCCMdocs/blob/master/sccm/compliance/plan-design/scap/install-configure-scap.md also mentions the same path of cd.latest\SMSSETUP\TOOLS\ConfigMgrSCAPExtension\ConfigMgrExtensionsForSCAP.msi without  the 1806 only caveat being mentioned anywhere.

Yes. It was first available in 1803 technical preview branch. Then it should only be available in current branch version 1806. It was removed from later current branch versions. I don't know if it's still in the technical preview branch. The feature team made the decision to remove it, but wanted to keep the documentation.
If you want more information on the status of the tool, see the Product feedback section in our docs to file feedback to the engineering team.
I'm closing this issue as there's nothing actionable for docs at this time.


Question: Replying to callback queries with text

How do i get the bot to reply to callback queries with text?
bot.command("", (msg, reply, cmdNext) => {
bot.callback((query, cbNext) => {
const data = JSON.parse(query.data)
if (data.action === "yes")
bot.text((msg, reply, textNext) => reply.text("You clicked yes"))
})
})

From the most of the examples using callback queries, there's this line here
https://github.com/botgram/botgram/blob/bbca7dff9a7a93d7f47f3544b53dc414adf9e143/examples/actions.js#L67
You can replace the .action method with any reply method in your case .text.
So your code would look something like
bot.command("<command>", (msg, reply, next) => {
  reply.inlineKeyboard([ /* buttons go here */ ]);
});

bot.callback((query, next) => {
  // parse data ...
  const reply = bot.reply(query.message.chat);

  if (data.action === "yes") {
    reply.text("You clicked yes").then(err => {
      if (err)
        return query.answer("<error message>");
      query.answer();
  });
  }
});

Hope that helps 🙂

Nice nice. Nibebe laptop chuo?
On Wed, May 30, 2018, 08:12 Willie M.I.K<EMAIL_ADDRESS>wrote:

From the most of the examples using callback queries, there's this line
here
https://github.com/botgram/botgram/blob/bbca7dff9a7a93d7f47f3544b53dc414adf9e143/examples/actions.js#L67
You can replace the .action method with any reply method in your case
.text.
So your code would look something like
bot.command("", (msg, reply, next) => {
reply.inlineKeyboard([ /* buttons go here */ ]);
});
bot.callback((query, next) => {
// parse data ...
const reply = bot.reply(query.message.chat);
if (data.action === "yes") {
reply.text("You clicked yes").then(err => {
if (err)
return query.answer("");
query.answer();
});
}
});
Hope that helps 🙂
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/botgram/botgram/issues/20#issuecomment-393032558, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Acrq0AwWM-YbKaFWujAJI998x7-CLW0Uks5t3iorgaJpZM4URWag
.

Closing this due to inactivity, if you've got any more questions I'm happy to answer :)


Bug about rename_channels after add_channels

Description of the problem
Hello, this this Runkai. I recently found a problem with applying raw.rename_channels to newly added channels.
I use raw.add_chennels to add to new channels called 'PZ’ and 'FZ' for consistency in number of channels (you know in machine learning research, it is often to keep the number of channels same). After adding the two channels, I want to rename them to 'Pz' and 'Fz' using raw.rename_channels. However, there is a Keyerror occured. After checking the source code, I found it is because raw._orig_units do not have 'PZ' and 'FZ' in their keys. Finally, I fix the problem with directly assignment values to raw._orig_units.
So, my questions are as follows:

does "renaming operation" to newly added channel not supported?
if _orig_units is protected and should not be changed after initializatoin?

Steps to reproduce
tmp_arr = np.zeros([len(chns_need_added), len(raw)])
tmp_info = mne.create_info(ch_names=chns_need_added， sfreq=raw.info['sfreq'], ch_types='eeg)
tmp_raw = mne.io.RawArray(tmp_arr, tmp_info)
raw = raw.add_channels([tmp_raw], force_update_info=True)

# error occur
channel_renaming_dict = {'PZ':'Pz', 'FZ':'Fz'}
raw.rename_channels(channel_renaming_dict)

# fix
for ch in chns_need_added:
  if ch not in raw._orig_units.keys():
     raw._orig_units[ch] = 'uV'
channel_renaming_dict = {'PZ':'Pz', 'FZ':'Fz'}
raw.rename_channels(channel_renaming_dict)

Link to data
No response
Expected results
I think rename to newly_added_chns should be supported.
Actual results
KeyError: 'PZ'
Additional information
mne: 1.0.3

Hello，I tried to fix the bug today. And I found it is fixed and no error occur in a newer version. I came across the problem because I was using a older version. So, I think it is not a bug now.
Also, I think _orig_units should not be changed during processing, because it is directly read from header files. Thus, for added channels from RawArray, they have no _orig_units.

Can not reproduced on main with:
import numpy as np
import mne

raw = mne.io.RawArray(np.zeros([5, 1000]), mne.create_info(5, 1000, ch_types="eeg"))
chns_need_added = ["PZ", "FZ"]
tmp_arr = np.zeros([len(chns_need_added), len(raw)])
tmp_info = mne.create_info(
    ch_names=chns_need_added, sfreq=raw.info["sfreq"], ch_types="eeg"
)
tmp_raw = mne.io.RawArray(tmp_arr, tmp_info)
raw = raw.add_channels([tmp_raw], force_update_info=True)

# error occur
raw.rename_channels({"PZ": "Pz", "FZ": "Fz"})

Or with:
import numpy as np
import mne

raw = mne.io.RawArray(np.zeros([5, 1000]), mne.create_info(5, 1000, ch_types="eeg"))
for ch in raw.ch_names:
    raw._orig_units[ch] = "uV"
chns_need_added = ["PZ", "FZ"]
tmp_arr = np.zeros([len(chns_need_added), len(raw)])
tmp_info = mne.create_info(
    ch_names=chns_need_added, sfreq=raw.info["sfreq"], ch_types="eeg"
)
tmp_raw = mne.io.RawArray(tmp_arr, tmp_info)
raw = raw.add_channels([tmp_raw], force_update_info=True)

# error occur
raw.rename_channels({"PZ": "Pz", "FZ": "Fz"})

As you were using 1.0.3 and also report the bug as fix, I'll close.


Add option to model-server to initialize database schema

In this way the user does not have to manually run the SQL code on the database.
Fix #34

Did you check if Apache Ignite is capable of doing this?

Good point. I tried searching the documentation, but I could not find anything related to the automatic initialization of the database. I read that Ignite support a subset of DDL and I read it has problems with schemas (apparently it always create tables in the PUBLIC schema, according to some bug reports). However I confess I am a bit confused and I could have miss it a feature to do this


Fix for ZEPPELIN-478

Steps to repro the issue
The following is in case of FIFO scheduler
1.Run a  para(say para1-> %sh sleep 100). The status of the para is "RUNNING".
A new last para(say para2) is created
2.Now run some thing from para2.(eg: %sh ls )(para1 should be "RUNNING")
Expected: The status of para2 should be "PENDING"
Current: The status of para2 is "READY"

I have tested and it works really well.
Thanks for the nice description about the problem and good fix.
Looks good to me.

Can we merge this, if there is no concerns?

Merging if there is no more discussions

Its Merged (However the Mirror update seems slow today)

@corneadoug In this case, you can resolve the problem by file an issue to INFRA (e.g. https://issues.apache.org/jira/browse/INFRA-9974)

@Leemoonsoo thank you, just created https://issues.apache.org/jira/browse/INFRA-10912


Missing dispose VirtualDisplayController

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
VirtualDisplayController.dispose() is missing when dispose Android view.
Proposal
PR: https://github.com/flutter/engine/pull/43807

Close as fixed


Run template/templater from autocomplete

I feel like it would be a great addition to be able to quickly run templates by typing their name and selecting the auto complete entry, or at least have this as a toggle-able option
Example
I have a template called "Info Callout.md" that has

[!info] Title
Body

In a different note that I want to insert, I type in
In
In the autocomplete field Info Callout is shown for auto complete, when I press enter, instead of inserting [[Info Callout]], it runs command templater-obsidian:insert-templater or the command for inserting a template from default obsidian (not sure what it is)
I don't often insert templates as links to files however in case there are people who do the setting could be toggle-able

Hi, @Shyclops :)
How about using the functionality of the Custom Dictionary complement to insert templates (e.g., callouts)? Launching templater from autocomplete seems to go beyond the role of this plugin, so I believe it’s unlikely to be implemented.
FYR
https://tadashi-aikawa.github.io/docs-obsidian-various-complements-plugin/6. Showcase/📓 Custom dictionary examples/#callout


Refactoring

Is your feature request related to a problem?
No response
Describe the solution you'd like
Use freezed
Additional context
No response

Please add your planning poker estimate with Zenhub @Chralu

Done with v3.0.0


Create design-and-contribution.md

New page to replace former "contribution" page
Note: navigation needs to be updated to reflect new page title.

We need to hold on merging this until https://github.com/carbon-design-system/carbon-website/issues/323 is fixed.


Add Htop to the Webui

Hi,
since we are doing NN there could be heavy datasets which explode the ram and thus it would be nice to see the current ram load/cpu gpu/load  in the webui.
What would you like to be added:
see top
Why is this needed:
It is a comfort feature
Without this feature, how does current nni work：
It does not show it
Components that may involve changes:
Brief description of your proposal if any:
Since its standard html you could easily add it.

Thanks @MaximilianArnold  for raising the new feature request.

@Lijiaoa may you suggest how big the effort for prototyping this request? would like to discuss this with team based on the prototype if it is simple one.

Maybe 3-5 days. First we should find this message(cpu/gpu), and then UI should design where to show this message. Also, we should add rest api for this.
Best wishes,
Lijiao
From: Scarlett Li @.>
Sent: 2021年3月17日 10:53
To: microsoft/nni @.>
Cc: Lijiao Guo (Inspur) @.>; Mention @.>
Subject: Re: [microsoft/nni] Add Htop to the Webui (#3441)
@Lijiaoahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FLijiaoa&data=04|01|v-ligu%40microsoft.com|05dc2727516b45446f3d08d8e8efd676|72f988bf86f141af91ab2d7cd011db47|1|0|637515464103449231|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=tVBqfhd8hlUAU%2Fwu84QNazE7inGuVzj17sSQSxcBc7c%3D&reserved=0 may you suggest how big the effort for prototyping this request? would like to discuss this with team based on the prototype if it is simple one.
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fnni%2Fissues%2F3441%23issuecomment-800752271&data=04|01|v-ligu%40microsoft.com|05dc2727516b45446f3d08d8e8efd676|72f988bf86f141af91ab2d7cd011db47|1|0|637515464103459188|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2B8mm4dLV51x0cNqJnmr2u%2FOCh54H%2F%2FkvkzucMfY%2BtsQ%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOUOGKXRS7HAXYB6TEVFDV3TEAKSNANCNFSM4Y5LDXSA&data=04|01|v-ligu%40microsoft.com|05dc2727516b45446f3d08d8e8efd676|72f988bf86f141af91ab2d7cd011db47|1|0|637515464103459188|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lUVnnmN5ZRfG6R0wSFxhLKTW%2F%2F3i9gfd614yCbTKNEY%3D&reserved=0.

@MaximilianArnold , this is a valid request. There will be a lot of work to support this feature when there are many different training services. For training services like AML, OpenPAI, they have collected such information (CPU/GPU/Mem/Network utilization) and displayed on their WebUI. Following "separate of concern", it is a good design choice for NNI to link each trial job to their corresponding web page in the training service. This has been supported by NNI for training services such as OpenPAI. For the training services (i.e., local, remote) which are implemented by NNI, it might be good to collect those information and display on NNI WebUI. We will put this feature in our backlog, and encourage external contribution on this feature


[UX] How display a large database schema ?

In the FédéSCoT instance, @bruhnild create a schema with tables and a lot of fields :

I find this schema hard to read, and the user need to zoom a lot before knowing on which table he's clicking.
Maybe we could search a way to display schema only with table names instead of table + field names ?
This could be a display option ?
What do you think @bruhnild @DarkVisenya @jcougnaud ?

I agree with you, it's a good idea to display a lighter version of this schema.
I also think that displaying only the table names can be a good solution. We could have a switch button to switch from one mode to the other.
Another mode could be to display only the first n columns (the other columns would not be displayed at all or via a scroll bar, however, in this last case, I do not know if an overflow option is available with svg).


Use 'in' operator instead of has_key.

It's not available in Python 3 and in is recommended.

Travis failure appears to be due to external resources...
PS, does OWSLib really claim to support Python 2.5? I see mention of it in places, but then again the tox.ini does not have an env for it.

Travis failure appears to be due to external resources...
PS, does OWSLib really claim to support Python 2.5? I see mention of it in places, but then again the tox.ini does not have an env for it.

Thanks @QuLogic.  Maybe we should remove any refs to 2.5 in the codebase to be safe?

Thanks @QuLogic.  Maybe we should remove any refs to 2.5 in the codebase to be safe?


Re-evaluate the IMessageContext interface

There's a lot of stuff on that thing that's not relevant to users

Couple different roles going on here:

Methods for the handler to publish other messages or send responses back to the original sender
Methods for enrolling in outbox transactions
Methods for the callbacks and continuations


Change RLMSyncPermissionResults notification blocks to match other …

…collection notifications
Breaking change.
Requires https://github.com/realm/realm-object-store/pull/483
Fixes https://github.com/realm/realm-cocoa/issues/5076
Not sure yet how to test this. None of our existing permissions tests rely on the ordering of permissions or how they're inserted into the collection, and the underlying APIs make no guarantees about this.

Superseded by #5168


Prisma2 - No createMany - Workaround?

I am sadly aware that there is no createMany ability currently with Prisma2 (or Prisma). So, I am wondering if there is a better approach for my situation. Initially, I planned on uploading an array like so (from within GraphQL playground):
mutation{
  createOneProposal(data:{
    email<EMAIL_ADDRESS>    name: "Ecommerce"
    types:{
      create:{
        model: PURCHASE
        name: "e-commerce"
        cost: 600
        services:{
          create:{
            service: "Responsive web design"
          }
          create:{
            service: "Another service!"
          }
          create:{
            service: "And yet another service!"
          }
          }
        }
      }
  }){
    created_at
    proposal_id
    types{
      type_id
      services{
        service_id
      }
    }
  }
}

As I learned, this is not allowed. You can only have one create function.
So, how exactly could I go about this? Stringify the array, and then break it back apart on the Query side when I want to access it? Doesn't seem like a great solution...
Any ideas would help. I genuinely just assumed createMany was a thing ;)
Given Model:
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "mysql"
  url      = env("MYSQL_URL_PRISMA2")
}

model Post {
  content      String    @default("")
  created_at   DateTime  @default(now())
  post_id      Int       @default(autoincrement()) @id
  published    Boolean   @default(false)
  published_at DateTime?
  title        String    @default("")
  author       User
}

model Profile {
  bio        String?
  profile_id Int     @default(autoincrement()) @id
  user_id    User
}

model Proposal {
  email       String   @unique
  name        String?
  proposal_id Int      @default(autoincrement()) @id
  created_at  DateTime @default(now())
  types       Type[]
}

model Type {
  cost        Int?
  name        String?
  model       Model?    @default(SUBSCRIPTION)
  services    Service[]
  type_id     Int       @default(autoincrement()) @id
  proposal_id Proposal
}

model Service {
  service_id Int     @default(autoincrement()) @id
  service    String?
  type_id    Type
}

model User {
  email    String    @default("") @unique
  name     String    @default("")
  password String    @default("")
  role     Role      @default(USER)
  user_id  Int       @default(autoincrement()) @id
  posts    Post[]
  profiles Profile[]
}

enum Role {
  USER ADMIN
}

enum Model {
  SUBSCRIPTION PURCHASE CUSTOM
}

You can do it like so:
mutation {
  createOneProposal(
    data: {
      email<EMAIL_ADDRESS>      name: "Ecommerce"
      types: {
        create: {
          model: PURCHASE
          name: "e-commerce"
          cost: 600
          services: {
            create: [
              { service: "Responsive web design" }
              { service: "Another service!" }
              { service: "And yet another service!" }
            ]
          }
        }
      }
    }
  ) {
    created_at
    proposal_id
    types {
      type_id
      services {
        service_id
      }
    }
  }
}

Create take in an array of items.

Also, it issue belong to

Thank you @pantharshit00 ! You're right, will do! There's not much activity there, I see. Hopefully simple questions will get responses :) I'm trying to figure out how to write the gql tag for the same mutation here, so hopefully someone responds!

Can we reopen this issue for more discussion?
deleteMany and findMany are available, it seems like createMany is an easy-to-include addition.

@AnandChowdhary Please open an issue about this in https://github.com/graphql-nexus/nexus-schema-plugin-prisma as it belongs there. This repository solely contains issues about Prisma Client.


Bad indenting of function decl inside an extern block

-                pub fn freopen(filename: *const c_char, mode: *const c_char,
-                               file: *mut FILE)
-                               -> *mut FILE;
+                pub fn freopen(filename: *const c_char,
+                           mode: *const c_char,
+                           file: *mut FILE)
+                           -> *mut FILE;

Seems to happen inside an inline mod too, so not specific to extern blocks


Small.updateManifest(info.manifest, false) 找不到符号

C:\ZY\AndroidStudioProjects\Small\Android\Sample>gradlew.bat buildBundle -q
[Small] building bundle 1 of 5 - app.detail (0x67)
        [app.detail] split library res files...                         [  OK  ]
        [app.detail] slice asset package and reset package id...        [  OK  ]
        [app.detail] split library R.java files...                      [  OK  ]
        [app.detail] split R.class...                                   [  OK  ]
        [app.detail] split aar classes...                               [  OK  ]
        -- output: armeabi/libnet_wequick_example_small_app_detail.so (6739 bytes = 6.6 KB)
[Small] building bundle 2 of 5 - app.home (0x70)
        [app.home] split library res files...                           [  OK  ]
        [app.home] slice asset package and reset package id...          [  OK  ]
        [app.home] split library R.java files...                        [  OK  ]
C:\ZY\AndroidStudioProjects\Small\Android\Sample\app.home\src\main\java\net\wequick\example\small\app\home\MainFragment.java:230: 错误: `找不到符号`
                            if (!Small.updateManifest(info.manifest, false)) {
                                      ^
  符号:   方法 updateManifest(JSONObject,boolean)
  位置: 类 Small
1 个错误

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app.home:compileReleaseJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log

Seems app.home conflicts with small-0.9.0,
so Sample for user will build failed with cmd ./gradlew buildBundle -q ,
as method updateManifest() not existing in small-0.9.0 jar.

Use DevSample first, fix it later.


Add Launcher

Add a launcher and show it by default on startup.
This will make it easier to select a kernel when https://github.com/jtpio/p5-notebook/issues/50 is implemented.
TODO

[x] Add simple launcher
[x] Stage p5 logo
[x] Add the example to the launcher

It might actually be possible to drop this plugin after: https://github.com/jupyterlab/jupyterlab/pull/9305


Update the map when unit moves to 2 squares away from edge

Like in this case below, currently in CivONE if the cruiser moves to within 1 tile from edge the map only then updates:

It should update when it moves down to within 2 tiles in such a way, that the map first updates, and only then the unit moves.

This is only true for sea units with a view range of 2.


[BUG] Some CRDs has wrong names.kind

What is the bug?
All CRDs follow the name convention OpensearchEntity like

OpensearchRole
OpensearchIndexTemplate

But Cluster and ISM have different format like:

OpenSearchCluster
OpenSearchISMPolicy

The name of cluster CRD is documented correctly here https://github.com/opensearch-project/opensearch-k8s-operator/blob/main/docs/userguide/main.md#using-custom-object
However ISM kind is wrong here https://github.com/opensearch-project/opensearch-k8s-operator/blob/main/docs/userguide/main.md#managing-ism-policies-with-kubernetes-resources

In my case there are even some CRDs missing, I can only find:

OpensearchActionGroup
OpenSearchCluster
OpensearchRole
OpensearchTenant
OpensearchUser
OpensearchUserRoleBinding

So confirmed missing are at least (these are the ones I use):

OpensearchComponentTemplate
OpensearchIndexTemplate
OpensearchISMPolicy

In my case there are even some CRDs missing, I can only find:

OpensearchActionGroup
OpenSearchCluster
OpensearchRole
OpensearchTenant
OpensearchUser
OpensearchUserRoleBinding

So confirmed missing are at least (these are the ones I use):

OpensearchComponentTemplate
OpensearchIndexTemplate
OpensearchISMPolicy

I experienced the same issue. Those CRDs are fairly new and it appears they are not yet bundled in a release. If you deploy the helm chart from source those will be included. I included my output below when built from source. This brings up a bigger question of when a new version will be released.
opensearchactiongroups.opensearch.opster.io
opensearchclusters.opensearch.opster.io
opensearchcomponenttemplates.opensearch.opster.io
opensearchindextemplates.opensearch.opster.io
opensearchismpolicies.opensearch.opster.io
opensearchroles.opensearch.opster.io
opensearchtenants.opensearch.opster.io
opensearchuserrolebindings.opensearch.opster.io
opensearchusers.opensearch.opster.io

If you want to test a temporary release you can use mine: https://github.com/opensearch-project/opensearch-k8s-operator/issues/662#issuecomment-1889429874

If you want to test a temporary release you can use mine: #662 (comment)

Thank you! This was able to prove that the next release will solve our issue!


Cache Pyxis GraphQL schema

Fetching Pyxis GraphQL schema is time consuming, this can be a performance bottelneck when Freshmaker creates many PyxisGQL instances.

It's expected the tests won't be scheduled (due to deprecated ubuntu image is used, it's fixed in main branch) in this branch, since we're going to merge this branch to main branch soon, I'm not going to pick some commits from main branch. I ran the unit tests locally and all passed.


[storage] Update to the latest @autorest/typescript

@EmmaZhu We have logged #15813 which tracks the work needed to not only move to the latest version of the code generator, but also use the new Azure Core v2
Are you looking to move to the latest version of the code generator, but without moving to the new Azure Core v2? If no, then you can close this issue in favor of #15813

doesn't apply now that #15813 has been completed.


Add ircu2/snircd/irc2 controllers + fix tests to support them

Note: the testKeyValidation change depends on https://github.com/ircdocs/modern-irc/pull/107
But the existing code wasn't really compliant with any spec so far, because RFC2812 didn't provide any error code in case its specified key format wasn't followed...

(merged despite this not being documented in Modern yet, because I don't want to wait too long before adding IRCds)


Keeping Alloy up to date

Commit-Boost currently uses an outdated version of Alloy 0.5.4, while the newest one is 0.8.0. This outdated dependency is causing compatibility and maintenance challenges for projects like bolt that depend on commit-boost-client.
Are there any plans to update Alloy to the latest version and keeping it up to date?

ty for fast response!


travis: add .travis.yml for shellcheck

Summary

Enable Travis CI testing.
Implementation details
Added a .travis.yml file.
Testing
New tests cover the changes: yes
Licensing

This contribution is under the terms of the Apache 2.0 License: yes

This PR is very intentionally failing right now. This should be rebased once the changes are merged in from #45; there may be some follow up edits to get this passing.


Updated CONTRIBUTORS.md

I am updating the contributors page with my name in it. Here are the PRs that were created by me and have been merged. Please review and add my name to the contributor's list:

https://github.com/kyverno/kyverno/pull/5484
https://github.com/kyverno/kyverno/pull/5272
https://github.com/kyverno/kyverno/pull/5228
https://github.com/kyverno/kyverno/pull/5143
https://github.com/kyverno/kyverno/pull/5106
https://github.com/kyverno/kyverno/pull/4996
https://github.com/kyverno/kyverno/pull/4855
https://github.com/kyverno/kyverno/pull/4733

Hi @realshuting could you please review this PR?

Congrats @shahpratikr and thanks for the contributions!

I was off last week. Congratulations @shahpratikr 🎉