No notification for new messages anymore fdroid

Since ~2 months i dont get any messages anymore on riot fdroid build unless i open the app (and am connected to the inet) evendough the riot banner is there and telling syncronisation all the time. - it does not notify me about anything.
This has like always worked before and its the same problem with  a freind so we cannot communicate..

https://github.com/vector-im/riot-android/issues/3190

hi, if you use microG, make sure the app is excluded from battery optimizations!

I'm also having problems with notifications.
Neither my girlfriend or me get any notifications. I read all threads about this problem but there is no solution.
A few pointers:

2 different phones from different companies (One Plus 3t and Huawei P20 lite) both running android 9 bit didn't work with 8 either
Latest riot version from google play store
No notifications are displayed until the app is opened
The "find notification errors" in the app returns all green
No battery optimization on either devices enabled
Latest synapse docker container. tried a few older versions too (none before 1.0 though)

Reverted to 0.8.26 which is working.
Also tried miniVector which ships 0.9.1 same problem there.
Something has been seriously broken since the 0.8.28 releases :/
No playing with battery or other stuff is working.

Btw, any reason #3155 does not get more attention?

@akallabeth thanks your the response.
I can't find 0.8.26 (or any 0.8.x for that matter) on the docker repo and it seems they jumped versions from 0.34 to 0.99 about 6 months ago.
What's up with that

@chrisiaut I´m installing using f-droid directly. Have all versions there (for Riot.im, miniVector only back to 0.8.15)

@chrisiaut and the docker repo you reference is for synapse, the matrix server part or do I miss something?

I'm experiencing the same issue with Android app notifications. I can't use the Google Play store version because I NEVER, EVER sign my phone into google cloud, it takes over the whole phone and is a big invasion of privacy. I have no problem with using GCM for notifications (without the message content), but signing my phone in is a big no-no for me.
So I was absolutely falling in love with Matrix/Riot until some recent testing showed that my own phone was not getting notifications when asleep / app swiped away. Then I researched it and discovered the Play store version uses GCM, which requires Google account signin.
After trying the latest version in F-droid (0.8.28a) I am unable to receive notifications at all. Upon reading this issue, I tested installing 0.8.26 (F-Droid supports installing older versions). I now get notifications when:
1.) App is out of view.
2.) Phone is in sleep mode (screen off).
3.) App is swiped close in multi-tasking (but still running in background).
4.) App is swiped close and phone is in sleep mode.
The only issue now is that the notifications take one to two minutes to come through. This is not ideal, but for me, I can live with it.
I noticed that in 0.8.26 I have a pinned notification (always running in the background). I'm sure this is a background service, Conversations (my go-to XMPP client) also does this.
My biggest concern is that I'm stuck on an older Android version on F-Droid, with no workable update path. This would be a show-stopper for me. If I can't use Riot without a Google account sign-in then I'll stick with Conversations and XMPP. But I really want to move to Matrix because there is no good iOS app for XMPP  (at least not in my experience). I have no problem installing from Play store via Aurora / Yalp or similar.
For the last year or two now, Conversations has been delivering my messages / notifications flawlessly. I get them quite fast too. And with XEP-0280 (Message Carbons) and XEP-0313 (Message Archive Management) I appear to get all my message history in both directions on all devices at all times.
Please let me stay on the Matrix, Riot.im is so sweet, and the video calling is often equal to Facetime (the gold standard for video call quality in my experience).

^ I forgot to mention: I'm running my own home server (Synapse), which I believe is setup correctly since other users are getting their notifications just fine. And I am running my own ejabberd domain for XMPP.

OK, so most of what I said above may no longer apply. I did the same tests on my spare phone (Samsung Note 3 running LineageOS 14.1 [Android 7.1.2]). Using Riot.im version 0.9.3 from Play store the notifications seem to work well without signing in to google account. Although sometimes I get a sound, sometimes I don't (no DND configured), I always get a notitification and my LED flashes blue, so good enough I guess.
So my primary phone that has the issue I described above is a Nokia 6 running the stock ROM (currently Android 9). I beleive all existing supported Nokia phones have moved over to "Android One", so no funny business from likes of Huawei/Samsung.
I think it looks like it's an Android 9 issue, and if it effects Android One (pure Android) then we need to fix it ASAP.

Yesterday deinstalled riot and installed 0.8.26 again. Signed in asked a friend for a test message and restarted the phone.
I got a push message pretty intant (+- 1 minute) after he sent his message.
so guess the old version works for me.

FIXED! I'm really sorry to blame the latest version of Riot.im on Play store. It was Android 9 battery optimization after all, which if not buggy is very counter-intuitive.
So I'm used to going into "Battery Optimization" preferences in Android 7+ and setting my background apps to "Not optimized". This section in settings is still there , but it behaves differently, I noticed after a reboot my customizations (setting key apps to Not optimized) are lost / reset. I had previously turned off "Adaptive Battery" alltogether by disabling "Use Battery Manager", or I thought I had. So actually we need to go into "Background activity manager" and set Riot.im app to BOTH "Auto activate" and "Lock screen".
So, to recap, I took the following steps on Android 9:

Re-enabled "Use Battery Manager" in "Adaptive Battery".
Enable "Auto Activate" and "Lock Screen" for riot.im in "Background activity manager".
Ignore the old "battery optimization" settings we used in Android 7+.

Thanks Google, keeping me on my toes. I think I know what PIE stands for now; Pain In Elbow, coz after updates, I no longer know my arse from my elbow.
So, I can say that at my end, with exhaustive testing, the latest version of Riot.im (0.9.3) on Play store has notifications working almost flawlessy. We just need to use the appropriate battery restrictions mechanism for the version of Android in use. I say 'almost' flawless, because sometimes I get no sound with my notifications, sometimes I do. But as long as I get the blinking LED and a lock screen notification all is good here.
I'm suggesting that we close this issue. It's confusing with out of date versions of Riot. A new issue could be made instead for any ongoing. I'm an IT manager and so I test things very thoroughly, I can say notifications on Android are working well.
Useful resource that helped me: https://get.slack.help/hc/en-us/articles/360001562747-Known-issues-with-Android-notifications

@ethoms ok, good that it works with playstore, but this issue is specifically for the f-droid version (which still does not work)

Confirming bug using 0.9.4 from f-droid on Android 9.

@ethoms ok, good that it works with playstore, but this issue is specifically for the f-droid version (which still does not work)

Yes, you are correct, and I apologise for suggesting we close this issue. I was getting confused between this issue and the above mentioned and related issue #2512. It was that other issue that I meant we should close, and indeed it has since been closed.
I just wanted to restate that I DO NOT actually need to use the F-Droid version as I previously thought I did. The Play Store version (which I install via Aurora) is just fine because it DOES NOT require a google account sign-in for FCM (previously named GCM) to work. I do not mind my push notifications going through Google FCM servers as long as the message content is not disclosed, which is how I understand Riots' use of FCM to work.

Confirming bug using 0.9.4 from f-droid on Android 9.
Should have read that more closely ^^  - also tested 0.9.4 from fdroid - still not working.

Galaxy S4, LOS 14.1, Andoid 7.12
Downgrade again to 0.8.26

0.9.4 was working well for me on Android 8, but like others, I don't seem to get notifications once the f-droid version of Riot has been in the background for a certain amount of time. Battery optimizations have been disabled.
Would it be better to do like other apps and have a permanent notification so the server can push messages rather than periodically waking in the background and polling?

0.9.7 solves the problem for me.

I am having the same issue on my new phone (running Android 9).
I have Riot 0.9.8, installed via Aurora Store (Google Play).
Same issue goes for Riot.im and RiotX btw.

0.9.7 solves the problem for me.

I can't find 0.9.7 in the F-droid store

Download links: https://f-droid.org/en/packages/im.vector.alpha/
0.9.8: https://f-droid.org/repo/im.vector.alpha_40090800.apk

I had a device there notifications where working (Riot 0.9.9 F-Droid).
After a reboot, the notifications are working, but without any sound (still Riot 0.9.9 F-Droid).
I checked every setting that eventually could change it, but no success.
I restarted the app and then the smartphone: no success.
Then I gave up and install RiotX from F-Droid (main repo) on the same device: now both, RiotX 0.16.0 from F-Droid and Riot 0.9.9 from F-Droid are working!
After a reboot, still everything is working.
Android version: 7.0

myself and my brother had the same problem!

riot updated.
You can download remember Notification or   Notification history from Google play. these are definitely do  working and it was confirmed.
which F-Droid  is using ext? apk? rpm?


m365 entra user get sends incorrect query to Graph API

Priority
(Urgent) I can't use the CLI
Description
When running the command
m365 entra user get --id {user_id} --properties 'id,userPrincipalName,jobTitle,aboutMe' --withManager --debug
It fails with the error

The expression "id in ('{user_id}')" is not valid.

In debug I see that the query sent by CLI is
https://graph.microsoft.com/v1.0/users?$filter=id eq
'{user_id}'&$select=id,userPrincipalName,jobTitle,aboutMe&$expand=manager($select=displayName,userPrincipalName,id,mail)

When calling /users endpoint there is a limitation on the Graph API side that some properties like aboutMe cannot be requested. Those properties can be read only when get user endpoint is called.
Check the doc (remark below the table)
The correct query generated by m365 entra user get should be
https://graph.microsoft.com/v1.0/users/{user_id}?$select=id,userPrincipalName,jobTitle,aboutMe&$expand=manager($select=displayName,userPrincipalName,id,mail)

Steps to reproduce
m365 entra user get --id {user_id} --properties 'id,userPrincipalName,jobTitle,aboutMe' --withManager --debug
Expected results
Command returns info about the specific user even if aboutMe, birthday, hireDate, interests, mySite, pastProjects, preferredName, responsibilities, schools, skills, or mailboxSettings property is specified in the option properties.
Actual results

Error: The expression "id in ('{user_id}')" is not valid.

Diagnostics
node.exe : Executing command entra user get with options
{"options":{"id":"{user_id}","properties":"id,userPrincipalName,jobTitle,aboutMe","withManager":true,"debug":true,"output":"json"}}
At C:\Program Files\nodejs\m365.ps1:16 char:5
+     & "$basedir/node$exe"  "$basedir/node_modules/@pnp/cli-microsoft3 ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (Executing comma...utput":"json"}}:String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

- Running command...
Existing access token xxx still valid. Returning...
Request:
{
  "url": "https://graph.microsoft.com/v1.0/users?$filter=id eq
'{user_id}'&$select=id,userPrincipalName,jobTitle,aboutMe&$expand=manager($select=displayName,userPrincipalName,id,mail)",
  "method": "get",
  "headers": {
    "Accept": "application/json;odata.metadata=none",
    "user-agent": "NONISV|SharePointPnP|CLIMicrosoft365/7.4.0",
    "accept-encoding": "gzip, deflate",
    "X-ClientService-ClientTag": "M365CLI:7.4.0",
    "authorization": "Bearer xxx"
  },
  "responseType": "json",
  "decompress": true
}
Request error:
{
  "url": "https://graph.microsoft.com/v1.0/users?$filter=id eq
'{user_id}'&$select=id,userPrincipalName,jobTitle,aboutMe&$expand=manager($select=displayName,userPrincipalName,id,mail)",
  "status": 400,
  "statusText": "Bad Request",
  "headers": {
    "cache-control": "max-age=0, private",
    "transfer-encoding": "chunked",
    "content-type": "application/json",
    "vary": "Accept-Encoding",
    "strict-transport-security": "max-age=31536000",
    "request-id": "87ed1266-f68b-456c-a69b-233b291f459f",
    "client-request-id": "87ed1266-f68b-456c-a69b-233b291f459f",
    "x-ms-ags-diagnostic": "{\"ServerInfo\":{\"DataCenter\":\"Germany West
Central\",\"Slice\":\"E\",\"Ring\":\"5\",\"ScaleUnit\":\"002\",\"RoleInstance\":\"FR3PEPF00000442\"}}",
    "x-ms-resource-unit": "2",
    "date": "Mon, 22 Jan 2024 14:19:44 GMT"
  },
  "error": {
    "error": {
      "code": "-1, Microsoft.SharePoint.Client.InvalidClientQueryException",
      "message": "The expression \"id in ('{user_id}')\" is not valid.",
      "innerError": {
        "date": "2024-01-22T14:19:44",
        "request-id": "87ed1266-f68b-456c-a69b-233b291f459f",
        "client-request-id": "87ed1266-f68b-456c-a69b-233b291f459f"
      }
    }
  }
}
Timings:
api: 363.7576ms
core: 8.9974ms
command: 393.3846ms
options: 0.1301ms
total: 403.2618ms
validation: 0.409ms
Error: The expression "id in ('{user_id}')" is not valid.

CLI for Microsoft 365 version
v7.4.0
nodejs version
v20.8.1
Operating system (environment)
Windows
Shell
Windows PowerShell
cli doctor
{
  "os": {
    "platform": "win32",
    "version": "Windows 11 Pro",
    "release": "10.0.22621"
  },
  "cliVersion": "7.4.0",
  "nodeVersion": "v20.8.1",
  "cliAadAppId": "b79e0f02-29c7-4d90-b3ef-f4d7817d467f",
  "cliAadAppTenant": "single",
  "authMode": "Secret",
  "cliEnvironment": "",
  "cliConfig": {},
  "roles": [
    "User.Read.All"
  ],
  "scopes": {}
}

Additional Info
No response

Seems like a solid explanation @MartinM85, thanks for bringing this to our attention. Let's get this fixed asap!

I can take it, because it's something I need.

Awesome!


Recreate model loader class

load verticies

rewrote the constructor for geometry loader to accept just vao
try to create the scene node structure from the model
use it as option to load as scene node or as single vao

load materials

overload the constructor to accept the materials loaded from the model
alternativly create setters for different materials

donce


Mention to install brew for Mac

@mtorabi59
What is the best resource regarding install brew on Mac?

@Remi-Gau There is the official installation instructions on their own website and also another page that I found helpful!


Allow listing recipes in submodules with --list-submodules

@crdx Do you prefer a nested view, like this:
Available recipes:
  foo
  bar

  abc:
    xyz
    wer

  hjk:
    yui
    asd

Or a flattened view, like this:
Available recipes:
  foo
  bar
  abc::xyz
  abc::wer
  hjk::yui
  hjk::asd

If I may add my preference? I'm in favor of the nested view as I'm not that comfortable showing :: anywhere 😅
The way we've used the "module-like" just scripts so far has relied on the separator being a space character and I'd like to avoid seeing the :: in any output 😅
I can see us using --list-submodules in some inner module, maybe already with the just devices case I pointed in https://github.com/casey/just/issues/929#issuecomment-2132780998  — but not at the top-level just case.
The :: separator feels like some Rust style preference to me and not something I've seen in any other command line tool we use.

The issue with spaces in a flattened view is that it wouldn't be possible to tell what was a submodule/recipe and what was an argument:
Available recipes:
  foo
  bar
  abc xyz
  abc wer
  hjk yui
  hjk asd

abc could be a recipe with an argument named xyz. The ::-separated notation is also necessary in few places. I chose it because :: is commonly used to separate module paths, e.g., in Rust, C++, and I think others.
The output of --summary uses spaces to separate recipe names, and completion scripts split on spaces, so in order to include recipes in submodules, they must be ::-separated. Because of this, :: must also be accepted on the command line, so that a recipe from the output of --summary can be passed to just.
When, eventually, you can reference variables and recipes in other modules, you'll have to specify module paths using ::. This is just because the grammar would otherwise be ambiguous between submodule recipes and arguments:
mod foo

bar: foo::baz

So the command-line interfaces will all accept :: as an alternative to spaces, and when module paths eventually appear in justfiles themselves, they'll be space-separated. I do agree that spaces are nicer!
I'll probably add some sort of option to list submodules nested, as in this PR, and flattened, in another PR, using ::.
As far as a depth limit, I'd be happy to add that in the future, feel free to make an issue if/when you want it!

Yeah to clarify my opinion — I don't have any objections of using :: inside Justfiles themselves. I do think it makes a lot of sense to use :: inside Justfiles.
I'm mostly against showing that in the output of any just commands when ran on the command line.
I also do agree that the flattened view as in your example would be confusing and would not want that either.
It is indeed a tricky case!

The output of --summary uses spaces to separate recipe names, and completion scripts split on spaces, so in order to include recipes in submodules, they must be ::-separated. Because of this, :: must also be accepted on the command line, so that a recipe from the output of --summary can be passed to just.

I'm not sure I fully follow this train of thought, so let me try to understand what you mean with this example:
Available recipes:
  foo
  bar

  abc:
    xyz
    wer

  hjk:
    yui
    asd

I would image just <TAB> would show in autocompletion:
foo
bar
abc
hjk

And then I'd do just abc <TAB> and see:
xyz
wer

But it sounded like what I'd have instead is this when running just <TAB>?
foo
bar
abc::xyz
abc::wer
hjk::yui
hjk::asd

Do correct me if I'm wrong! I'm mostly worried that the tab autocompletions would get useless for us if all the submodule recipes would also be shown in there with the :: separating them instead of needing "diving deeper" like we now have since https://github.com/casey/just/pull/2108

@casey The original nested output is my personal preference. I wouldn't use the flattened view.
Note: I don't use the concept of a default recipe, i.e. I don't have just on its own run anything. Instead, the first recipe is always:
[private]
help:
    just --list --unsorted

Because I prefer to see what's available and pick from the list rather than have a default. So, I can easily add a new flag --nested (or whatever) to this recipe everywhere.
@valscion I believe that the output of --summary is correct to show fully-qualified recipes. Technically a submodule recipe is still a recipe that can be run from the top-level context so completions should show it. And it needs to be ::-separated because of completion word-splitting characters.
The current shell completions are generated by applying simple replacements to the clap-generated default completions:
https://github.com/casey/just/blob/f5bb82dea3edde07b5aca4303e2d73a300f45d5e/src/completions.rs
I wouldn't personally like to try to implement more intelligent submodule recipe completion handling like this. This is the point where hand-crafted completion scripts would be better.

Technically a submodule recipe is still a recipe that can be run from the top-level context so completions should show it.

Yeah I do agree — but this "technically correct" vs. what I'm in favor of as "intuitive behavior" are different.
For us, it would make the autocompletion cumbersome and the first tab hit after entering just <TAB> would be useless as we'd see hundreds of potential autocompletions. At that point, we would not see the rest of the commands available at all.
For example, the aws CLI (which is huge!) also doesn't show "submodules" but instead lists the outermost modules first:

Then I can tab to just the aws s3 scripts:

This is how I'd expect "top level modules" to first work and then submodules showing only after I've explicitly tabbed to show the submodules.

The original nested output is my personal preference. I wouldn't use the flattened view.

Okay, nice. I'm going to merge this, if someone wants the flat view, they can open an issue.
The completion scripts are extremely hacky. just uses clap to parse arguments, and clap generates the basic completion scripts. We then modify them to make them complete recipe names with the output of just --summary, which includes ::-separated recipe paths in modules.
I would prefer to use space-separated recipe paths, and I would prefer to complete them one at a time, but this just isn't possible with the current approach, since the completion scripts have no idea which arguments are present on the command line so far, so they can't incrementally complete recipes and modules as the user descends into modules.
Fixing this is blocked on https://github.com/clap-rs/clap/issues/1232, which is a proposed improvement to claps's completion engine. I suppose someone could try modifying the completion scripts, but that gets messy fast.

I would prefer to use space-separated recipe paths, and I would prefer to complete them one at a time, but this just isn't possible with the current approach, since the completion scripts have no idea which arguments are present on the command line so far, so they can't incrementally complete recipes and modules as the user descends into modules.
Fixing this is blocked on https://github.com/clap-rs/clap/issues/1232, which is a proposed improvement to claps's completion engine. I suppose someone could try modifying the completion scripts, but that gets messy fast.

Oh wow what a rabbit hole — I understand now that this isn't such an easy feat 😅


Remove 	proxy.config.http.cache.vary_default_*

This includes
	proxy.config.http.cache.vary_default_text
	proxy.config.http.cache.vary_default_images
	proxy.config.http.cache.vary_default_other

Blocks #5457

I could be wrong, but I don't think you can do that much with Vary in header_rewrite.
https://github.com/apache/trafficserver/issues/2925

But that's multi-value headers, right ? Vary: would be a single value ?

Yeah, got all the things mixed up. Nothing to see here!


null is not an object (evaluating 'state.hasOwnProperty')

I think my code already same with example but I am ussing connectMeteor instead createContainer. And it produce above error. Here are my code:
  getMeteorData(){
    const sub = subscribeCached(MeteorStore, 'account', this.selector(), this.options());

    return {
      // ready: handle.ready()
    }
  }

  selector(){
    const search = this.state.search;
    return {
      'name': {$regex: search, $options: "i"},
    };
  }

  options(){
    return {limit: this.state.limit, sort: {createdAt: -1}};
  }

here is the complete error:

edit:
I have tried using createContainer but still same error, so I think thats not the cause
Note:
I am not really good at Redux stuff.
please help

Did you do the initialization part? I.e.
// myReduxStuff.js
import initMeteorRedux from 'react-native-meteor-redux';
import {AsyncStorage} from 'react-native';
import {persistStore, autoRehydrate} form 'redux-persist';

const MeteorStore = initMeteorRedux(null, autoRehydrate());

// Pick your storage option, I used AsyncStorage which makes sense for react-native
persistStore(MeteorStore, {storage: AsyncStorage}, () => {
  // Callback tells minimongo to use MeteorStore until connectivity is restored
  MeteorStore.loaded()
});

export {MeteorStore}

of course I did with the same syntax as above

any info about this @JulianKingman ?

Ok, obvious question, are you importing MeteorStore on your component? It looks like for some reason the persister is not recognizing the redux store.

of course I did something basic like that. I am using RN 0.40. All just the same with your example. if I don't import MeteorStore, ofc the error will be undefined MeteorStore in line x.

I also have to follow the instructions as in demonstrated, but got the same error as above, please check back in the your example

Ah, try passing undefined instead of null for the first param in initMeteorRedux

well done! thanks @JulianKingman .. its fixed now.
I'll try dig deeper using this package, if I found something strange again in more advance usage, I'll inform you..

btw the error had gone using undefined, but it encountered warning like this.

is this normal or we can remove this warning? @JulianKingman

Hmm, can you trace it to anything particular? I don't think I have any promises in the code.


fix: Modify log location and regular expressions

It looks like I forgot to click squash so this two-line modification resulted in several commits in the commit history.
Sorry I don't know how to change it to squash...


buymecoffee button(yellow button) on right bottom is not responsive

This is a(n):

[ ] Update to an existing Animated Button
[ ] Any Error
[ ] Proposal to the Repository
[ ] Documentation / README.md changes

Details:

please assign this issue to me

@abansal15 You can work on it and create a PR but it's a iframe tag popup by default of buymeacoffee.
All the best!

sir i am not able to locate the code of that iframe  in vs code but i have located in the browser inspect sir can you pls help

Hi, are you still working on this issue? If you've already made a PR with the changes that have been merged then please close the issue. If you have an open PR, please mention this issue number in your PR in order to link them.


Dev

18 Apr. 2020: It looks like this is the first pull request for this repo in a really long time. dev branch now contains all the notebooks and figures for the INSTA paper. I've renamed this repo to insta_paper. (It was fish_annotation_paper before.)

18 Apr. 2020: It looks like this is the first pull request for this repo in a really long time. dev branch now contains all the notebooks and figures for the INSTA paper. I've renamed this repo to insta_paper. (It was fish_annotation_paper before.)


Huge amount of RAM usage and Raspberry Pi support

Hey all!
I was wondering why kite uses so much ram it can get up to above a gig?!
I was also wondering if it will ever have raspberry pi support
:smiley:

@metalogical Would you mind chiming in here please?

@metalogical Would you mind chiming in here please?

See https://help.kite.com/article/107-large-memory-usage regarding memory.
Regarding Raspberry Pi: it's unlikely that we will ever support Raspberry Pi devices due to the limited memory and CPU resources available. We intend to support the new Apple Silicon chips some time this year (see https://github.com/kiteco/issue-tracker/issues/491), which will get us closer to supporting Linux for ARM, but Raspberry Pi will likely never work well.

See https://help.kite.com/article/107-large-memory-usage regarding memory.
Regarding Raspberry Pi: it's unlikely that we will ever support Raspberry Pi devices due to the limited memory and CPU resources available. We intend to support the new Apple Silicon chips some time this year (see https://github.com/kiteco/issue-tracker/issues/491), which will get us closer to supporting Linux for ARM, but Raspberry Pi will likely never work well.


Using bestvideo+bestaudio with certain websites gives an error

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running yt-dlp version 2021.06.09
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] Command-line config: ['-R', '15', '-v', '-i', '-o', 'z:\\video\\%(title)s (%(id)s) [%(height)sp].%(ext)s', '--cookies', 'Z:\\cookies.txt', '-f', 'bestvideo+bestaudio', '--fixup', 'detect_or_warn', '--add-metadata', 'https://www.metacafe.com/watch/11636649/dancer4/']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] yt-dlp version 2021.06.09 (exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] exe versions: ffmpeg 2021-05-19-git-2261cc6d8a-full_build-www.gyan.dev, ffprobe 2021-05-19-git-2261cc6d8a-full_build-www.gyan.dev
[debug] Proxy map: {}
[debug] [metacafe] Extracting URL: https://www.metacafe.com/watch/11636649/dancer4/
[metacafe] 11636649: Downloading webpage
[metacafe] 11636649: Extracting information
[metacafe] 11636649: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
ERROR: Requested format is not available
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1133, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1165, in __extract_info
  File "yt_dlp\YoutubeDL.py", line 1202, in process_ie_result
  File "yt_dlp\YoutubeDL.py", line 2158, in process_video_result
yt_dlp.utils.ExtractorError: Requested format is not available

Description

This error is not only happening with certain videos nor with only metacafe, it happens also with xvideos and xhamster. IIRC this didn't happened on youtube-dl, please fix this.

Metacafe doesn't have a bestvideo or a bestaudio. Just don't use any -f

It is happening also with xvideos and xhamster, try with any url. Isn't there any way you guys could make it to automatically ignore betvideo+bestaudio if it's missing from the supported formats? I'm downloading all URL in an automated way just giving any URL I found so editing the command line for certain videos will break the automation.

Use bv+ba/b then...

Isn't there any way you guys could make it to automatically ignore betvideo+bestaudio if it's missing from the supported formats

No. The app downloads exactly what you request it. Just use the correct options

Thank you, didn't thought about that, I was of course using the wrong commands.

@pukkandan Hey, one question. I'm using bv*+ba/b now and I'm getting this with several metacafe videos:
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] yt-dlp version 2021.06.09 (exe)
[debug] Python version 3.8.10 (CPython 64bit) - Windows-10-10.0.19042-SP0
[debug] exe versions: ffmpeg 2021-05-19-git-2261cc6d8a-full_build-www.gyan.dev, ffprobe 2021-05-19-git-2261cc6d8a-full_build-www.gyan.dev
[debug] Proxy map: {}
[debug] [metacafe] Extracting URL: https://www.metacafe.com/watch/3354402/lorena_orozco_besame/
[metacafe] 3354402: Downloading webpage
[metacafe] 3354402: Extracting information
ERROR: the JSON object must be str, bytes or bytearray, not NoneType
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1133, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1154, in __extract_info
  File "yt_dlp\extractor\common.py", line 567, in extract
  File "yt_dlp\extractor\metacafe.py", line 230, in _real_extract
  File "yt_dlp\extractor\common.py", line 950, in _parse_json
  File "json\__init__.py", line 341, in loads
TypeError: the JSON object must be str, bytes or bytearray, not NoneType

Is that a bug? Video link is working fine.

That video doesn't play for me in browser either

I can't access metacafe anymore now, it seems the website is suffering some problems. I got a bad gateway and I can't even access my profile, is giving me errors.
Will try later and see if it happens again because some videos downloaded correctly.

I can access the site, and other videos play correctly, but this one doesn't. For now, lets assume its an issue on their side. If the video becomes playable in browser but still doesn't download, open a new issue


Resized map chart overflows

Version
3.4.2
Development Environment
OSX, Chrome
Current Behavior
Using the map chart, if I use the resize method, then zoom in to the resized map, the map overflows over other elements, seemingly to the original size.
Here's a demo: https://jsfiddle.net/cmhdwabx/
Just zoom in to the map and see it overflow over the adjacent elements.

Expected Behavior
Zoomed in maps should crop to the size they are resized to

It will be fixed v3.6.0.


Prompt - check if vcs folder exist (.git, .svn, .hg) before getting vcs repo status

This improves responsiveness of this "Prompt" since it prevents unnecessary repo checks ie. hg summary is verys slow (1s) regardless so it will slow down the shell.
https://github.com/gnhuy91/dotfiles/commit/d5b237f930e00d1eccfd3c1b128027edcec90871

But what if you're in a subdirectory of your repo?

Ah, I missed that, thanks for pointing it out.


Public note crashes status page

Describe the bug
Once I add a public note to an active incident and subsequently refresh the status page, it fails to load.
There is an exception thrown in the console about a Date being undefined.
This result in the status page (preview) showing a blank page.
react-dom.production.min.js:189 Error: Invalid date
    at OneUptimeDate.fromString (Date.ts:490:15)
    at OneUptimeDate.isAfter (Date.ts:329:26)
    at eval (Detail.tsx:112:16)
    at Array.sort (<anonymous>)
    at getIncidentEventItem (Detail.tsx:111:14)
    at eval (Overview.tsx:571:41)
    at Array.map (<anonymous>)
    at Overview_Overview (Overview.tsx:566:43)
    at Xh (react-dom.production.min.js:167:135)
    at dj (react-dom.production.min.js:197:256)

it's from the line throw new BadDataException('Invalid date');
To Reproduce
Steps to reproduce the behavior:

create a monitor
create an incident for that monitor
create a status page
display the monitor on the status page
navigate to the status page preview -> works
create a public note for the incident
reload the status page preview -> exception

Expected behavior
I would expect the incident note to be displayed.
Desktop (please complete the following information):

OS: MacOS
Browser Chrome
Version: Version 114.0.5735.133 (Official Build) (arm64)

Additional context
The incident I'm testing with: https://oneuptime.com/dashboard/c327b326-bd80-4b88-ad71-4543948c283a/incidents/22f72448-a65f-48c4-aa61-0d203cea46c4/public-notes

Fixed https://github.com/OneUptime/oneuptime/commit/70478bd1fa6b24ad18993798cf9305800eb4172a. Will release as soon as the build passes. Approx 1 hour.


Update LOCALIZATION_CONTRIBUTORS

Summary
I like to participate in the German translation of the Qiskit Docs.
Details and comments

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


Debugger not closing eveb after clicking disconnect

After running the debugger (connect to remote) option, when I am clicking on disconnect it doesn't close.

What debugger are you using?

What debugger are you using?

the default one which is provided by VS Code

/jsDebugLogs
Please provide reproducible steps and the js-debug logs


How can I switch the keyboard from English to Russian?

Hey. Installed Xrdp on Xubuntu, with terminal connection there is no possibility to change keyboard layout (only English layout), keyboard settings can not help. How can I switch the keyboard from English to Russian?

I'm not sure anything about Russian but it seems to be working for me.
When I typed "qwerty", "йцукен" is shown on the screen. Is that correct?
The client is FreeRDP and I specified keyboard with /kbd:Russian option.

If the local server says locally setxkbmap -layout ru, then at terminal connection on the login screen and password only russian layout is available without the possibility to switch to the english layout.
XRDP version 0.6.1-2, how to update to the current version?

Which Debian do you use? Debian already has v0.9.1.

I use Xubuntu 16.04,( Xenial Xerus).
XRDP Installed using apt-get install xrdp.
Installed version 0.6.1-2.

I installed Xubuntu 17.10 and xrdp 0.9.1, with terminal connection only one english keyboard layout is available. When will it be possible to have several layouts in terminal mode?

Can't use russian layout on Linux Mint 18.2 (Ubuntu 16.04).
Also there is a problem with arrow keys - they didn't work.

@F1nZeR xrdp 0.6.1 doesn't support Russian layout and "won't fix".


unixPB: Remove xorg packages to avoid dragging in gdm

Signed-off-by: Stewart X Addison<EMAIL_ADDRESS>
Checklist

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[x] playbook changes run through VPC or QPC (if you have access) VPC
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

Please put the name of the software product (and affected platforms if relevant) in the title of this issue
The Adoptium AQAvit tests require Xvfb to be started on Linux in order to test GUI capabilities. Our playbooks currently install both Xvfb and Xorg - the latter of which shouldn't be necessary since the machines themselves are all either virtual or sitting in data centres, so I believe we can remove it (Perhaps it predates Xvfb?). On some Linux distributions (Certainly Ubuntu, possibly others) the xorg package, when installed via ansible, pulls in gdm3 which results in it starting a GUI greeter on the "console" of the machine, chewin up resources unnecesarily.
ALTERNATE OPTION: We could continue to let it be installed, but ensure the greeter service is disabled on startup, although I'd rather not force that on everyone in case the playbooks are being run on people's desktop systems :-)
I've already mentioned this to @smlambert in the #testing channel but it would be good to hear from @AdamBrousseau to understand if he sees any problem with removing these packages.

If it's a test package and it is confirmed and tested that it isn't needed then I'm fine to have it removed. Thanks for the heads up.

1383 is also green, so this should ben good to merge when approved.

LGTM, I assume this doesn't harm TCK manual testing?

TCKs do not directly use these playbooks, but it should be safe there if if it was :-) Xvfb and VNC are the ones that are important there.

If it is good to go - what are you waiting for? "Lean is Mean" (aka better)


Add threadSafe annotation to fmpp Maven Plugin

Without this annotation Maven assumes the plugin is NOT threadsafe, and any parallel builds spews log warnings

I wasn't able to run the tests locally, (they just hung before getting to the fmpp module)
Assuming CI passes, and this is a valid change, I can create an issue

@bdemers It looks like CI has passed.  Could you please create an issue and we will merge this.

@bdemers It looks like CI has passed. Could you please create an issue and we will merge this.

Sorry... I should have been more clear.   I meant a JIRA issue, but don't bother.. This is a minor update.


Improve pose searching performance

Pose searching uses a naive implementation right now, and is overall the slowest version it could be. A good first step to improve this is to change the search algorithm by using something like kNN. I'll leave this issue vague enough to leave the optimization decisions up to the implementer.

Done in https://github.com/GuilhermeGSousa/godot-motion-matching/pull/73


Explainability on cora citeseer and pubmed dataset

@a-lucic  thanks for opensourcing the work can this be extended to opensource dataset like cora citeseer and pubmed which are used for nodeclassification task ? if so what are the code changes to be made

Hi, yes you can try any of those datasets by modifying lines 47-51 of main_explain.py:

Replace "adj" with the adjacency matrix from one of those datasets
Replace "features" with the feature matrix from one of those datasets
Replace "labels" with the labels
Adjust your train and test splits


Switch repo to a metapackage

Ref:

https://github.com/roots/wordpress-packager/pull/507
https://github.com/roots/wordpress/issues/32

Closes https://github.com/roots/wordpress/issues/32
Closes https://github.com/roots/wordpress/issues/3
Closes https://github.com/roots/wordpress/issues/20
Closes https://github.com/roots/wordpress/issues/12
Closes https://github.com/roots/wordpress/issues/10

@retlehs All the mentioned issues can be closed I guess, a little spring cleaning 😊

Entirely unsure what's the right place to put this, so just noting it here:
This change breaks composer patches. It requires one to change from:
"patches": {
      "roots/wordpress": {
      }
    }

to:
"patches": {
      "roots/wordpress-no-content": {
      }
}


[Doc] Update the CorrettoJDK 8 and AdoptOpenJDK 8 compatibility with WSO2 IS 5.7

Background
Update the Tested Operating Systems and JDKs to indicate the compatibility of CorrettoJDK 8 and AdoptOpenJDK 8 with WSO2 Identity Server 5.7.

The doc was updated by stating that WSO2 IS 570 is compatible with CorrettoJDK 8 (Ubuntu 18.04
and CentOS 7.5) and AdoptOpenJDK 8 (Ubuntu 18.04 and CentOS 7.5).


Implement function "DeleteAllReports"

Description of the feature
Would be nice to have the equivalent function of:
"oModule.DeleteAllReports()"
for Maxwell 2D and 3D.
Steps for implementing the feature
No response
Useful links and references
No response

Hi @GiadaVent ,
I have implemented a new feature in the method: delete_report, the option to delete all reports if the user does not pass any name.
For example:
aedtapp.post.delete_report()
I will close this issue once the branch is merged in the main.
Thanks


Bulk Add Users to "Manage senders for impersonation protection"

I do not see anything listed in the documentation to bulk add users to "Manage senders for impersonation protection". Is this possible?

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

ID: f0b8f651-9d8e-a34b-8dde-2b6a9448f875
Version Independent ID: 20e96426-627d-9973-8303-a43a114c7b61
Content: Anti-phishing policies - Office 365
Content Source: microsoft-365/security/office-365-security/anti-phishing-policies-about.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@HarrisburgUSysAdmin, you're in the "about" topic. The "how to" topic contains PowerShell instructions to create and [modify(https://learn.microsoft.com/microsoft-365/security/office-365-security/anti-phishing-policies-mdo-configure#use-powershell-to-modify-anti-phish-policies) anti-phishing policies.
Those instructions don't specifically contain information about adding multiple users for impersonation protection, but they do link off to the corresponding New-AntiPhishPolicy and Set-Anti-PhishPolicy, where the TargetedUsersToProtect parameter is identified as "MultiValuedProperty" and the rather complex syntax for it is explained.
So, to add multiple users for user impersonation protection to an existing anti-phishing policy, use the following syntax:
Set-AntiPhishPolicy -Identity "<Policy Name>" -TargetedUsersToProtect "DisplayName1;EmailAddress1","DisplayName2;EmailAddress2",..."DisplayNameN;EmailAddressN"

OK?

Sorry, I should have also included the EnableTargetedUserProtection parameter in the previous command:
Set-AntiPhishPolicy -Identity "<Policy Name>" -EnableTargetedUserProtection $true -TargetedUsersToProtect "DisplayName1;EmailAddress1","DisplayName2;EmailAddress2",..."DisplayNameN;EmailAddressN"

Thanks for the feedback.
Closing


feat: Scalar ElysiaJS integration

👋 hey everyone, i'm really excited for this PR and everyones feedback!
I think it would be pretty cool to expand the options of the swagger integration, so people can choose between Scalar & Swagger UI.
Video Walkthrough

Some big benefits of Scalar:

Full customization
an amazing ElysiaJS theme
Integrated REST API Client for testing
Offline search using FUSE.js
and many more small features

We would be beyond honored to have Scalar as the default, and I think i can create the integration in a way that its seamless and backwards compatible.
Also, what a joy it was working in ElysiaJS, the codebase is really clean and getting started was as simple as bun dev 🤯
Stoked to hear what y'all think.
I still need to:

[x] add full configuration of scalar object
[x] tidy up the theme
[x] add to the documentation
[ ] iterate based on everyones feedback

@SaltyAom thank you so much for your response, and time in writing out that message. I pushed up the conflict fixes, and this is ready to get merged :) I can help write out any documentation in the core ElysiaJS repository :)
We are incredibly grateful for all the kind words, and how grateful we are for the community being excited for this PR. We are dedicated to making scalar the best, so we will always be here to fix and improve things fast!
Appreciate your feedback, i created issues for both of these https://github.com/scalar/scalar/issues/735 https://github.com/scalar/scalar/issues/737
I will look into these soon, incredibly excited to be integrated and to help out on the next steps once merged!

That feedback is so great! I’ve already added fetch support to our very own request example lib, we just need to add it to the API reference package. 😇 https://scalar-snippetz.netlify.app/


ARROW-6371: [Doc] Row to columnar conversion example mentions arrow::Column in comments

A comment in rowwise conversion example is updated to reflect removal of arrow::Column.

Codecov Report

Merging #5219 into master will increase coverage by 1.6%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5219      +/-   ##
==========================================
+ Coverage   87.67%   89.27%    +1.6%
==========================================
  Files        1025      742     -283
  Lines      147008   106136   -40872
  Branches     1437        0    -1437
==========================================
- Hits       128886    94753   -34133
+ Misses      17760    11383    -6377
+ Partials      362        0     -362

Impacted Files
Coverage Δ

cpp/src/arrow/vendored/xxhash/xxhash.c
73.6% <0%> (-0.38%)
:arrow_down:

js/src/util/fn.ts

go/arrow/memory/memory_avx2_amd64.go

rust/datafusion/src/execution/filter.rs

rust/arrow/src/csv/writer.rs

rust/datafusion/src/bin/main.rs

go/arrow/ipc/file_reader.go

js/src/builder/index.ts

rust/parquet/src/arrow/converter.rs

go/arrow/array/builder.go

... and 275 more

Continue to review full report at Codecov.

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

I think the cython output is generated from the python directory to avoid circular dependencies?  @pitrou?

@emkornfield This is right.


Remove the top two feedback links in the Public View

Leaving only 'Provide general site feedback'

If it's okay with you, @ocrasborn , I removed all three links, because the 'general site feedback' link is already available at each page in the footer

Good, @Woseseltops !


Always unset the required-version option during ecosystem checks

Uses our existing configuration overrides to unset the required-version option in ecosystem projects during checks.
The downside to this approach, is we will now update the config file of every project. This roughly normalizes the configuration file, as we don't preserve comments and such. We could instead do a more targeted approach applying this override to projects which we know use this setting 🤷♀

There's already the infrastructure for overriding config file options for this purpose, I'd rather not change the public API without stronger justification.

Oh also... the work Alex was doing on config overrides via the CLI would suffice for that approach. Seems nice to switch over once we have it.


Support xs:list inline lists, take 2

This is a redo of PR #37
I have addressed all comments expect the larger one about HeterogeneousSerializerFactory, I will create a separate PR for that

Rebased on dev so things should be in order now.

Nice to see that 0.6.0 was released, though I'm sad I couldn't get this finished in time.
Going to rebase this branch with respect to the 0.6.0.
Regarding pre-commit:
I haven't used pre-commit in any projects before and I run into the following issue when running pre-commit install:
An unexpected error has occurred: CalledProcessError: command: ('/home/jacob/.cache/pre-commit/repolmlbeusl/py_env-python3/bin/python',
 '-mpip', 'install', '.')
return code: 1
stdout:
    Processing /home/jacob/.cache/pre-commit/repolmlbeusl
      Installing build dependencies: started
      Installing build dependencies: finished with status 'done'
      Getting requirements to build wheel: started
      Getting requirements to build wheel: finished with status 'done'
      Preparing metadata (pyproject.toml): started
      Preparing metadata (pyproject.toml): finished with status 'error'

stderr:
      error: subprocess-exited-with-error

      × Preparing metadata (pyproject.toml) did not run successfully.
      │ exit code: 1
      ╰─> [14 lines of output]
          Traceback (most recent call last):
            File "/home/jacob/.cache/pre-commit/repolmlbeusl/py_env-python3/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/
_in_process.py", line 351, in <module>
              main()
            File "/home/jacob/.cache/pre-commit/repolmlbeusl/py_env-python3/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/
_in_process.py", line 333, in main
              json_out['return_val'] = hook(**hook_input['kwargs'])
            File "/home/jacob/.cache/pre-commit/repolmlbeusl/py_env-python3/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/
_in_process.py", line 152, in prepare_metadata_for_build_wheel
              return hook(metadata_directory, config_settings)
            File "/tmp/pip-build-env-b6bsl2th/overlay/lib/python3.10/site-packages/poetry/core/masonry/api.py", line 40, in prepare_met
adata_for_build_wheel
              poetry = Factory().create_poetry(Path(".").resolve(), with_groups=False)
            File "/tmp/pip-build-env-b6bsl2th/overlay/lib/python3.10/site-packages/poetry/core/factory.py", line 57, in create_poetry
              raise RuntimeError("The Poetry configuration is invalid:\n" + message)
          RuntimeError: The Poetry configuration is invalid:
            - [extras.pipfile_deprecated_finder.2] 'pip-shims<=0.3.4' does not match '^[a-zA-Z-_.0-9]+$'

          [end of output]

      note: This error originates from a subprocess, and is likely not a problem with pip.
    error: metadata-generation-failed

    × Encountered error while generating package metadata.
    ╰─> See above for output.

    note: This is an issue with the package mentioned above, not pip.
    hint: See above for details.

Something poetry-related given the logs but I don't use poetry for my own projects so it's hard to tell what's wrong.

(This is @ajoino, accidentally commented using my work account again...)
...
Something poetry-related given the logs but I don't use poetry for my own projects so it's hard to tell what's wrong.

Hi. It is a bug in isort pre-commit hook. I fixed that here https://github.com/dapper91/pydantic-xml/commit/a24d8a234386f3e28b1308399fd87675eb476e34, so you could rebase to that commit or run pre-commit autoupdate

I've updated to be up-to-speed with latest dev, but I rebased instead of merging and that goofed the history.

Codecov Report

Merging #38 (09159f5) into dev (2754abd) will decrease coverage by 0.43%.
The diff coverage is 83.72%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##              dev      #38      +/-   ##
==========================================
- Coverage   93.70%   93.28%   -0.43%
==========================================
  Files          22       22
  Lines        1033     1072      +39
==========================================
+ Hits          968     1000      +32
- Misses         65       72       +7

Flag
Coverage Δ

unittests
93.28% <83.72%> (-0.43%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pydantic_xml/serializers/factories/homogeneous.py
89.69% <83.72%> (-5.14%)
:arrow_down:

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

is this going to be merged? support for xs:list is much needed

@saharwenrix Hi,
MRs to version 1 are no longer accepted, only critical bug fixes.
In version 2 xs:list can be handled using custom xml serializers: https://pydantic-xml.readthedocs.io/en/latest/pages/misc.html#custom-xml-serialization


Azure chatbot does not display full (multiturn) prompt text in Facebook Messenger embedding

Library name and version
Azure
Describe the bug
I have built a bot in Azure portal that functions as a replacement for the FAQ of my website. I chose to also integrate the bot in Facebook messenger chat and to implement this messenger chat button on my website.
The problem now is that the multiturn prompts that I added into the knowledge base, don't display the full text of the questions that I am proposing to my users (see picture). If you click on one of the options though, the bot understands the question and reacts accordingly.
Expected behavior
It should display the full sentence/question I am proposing.
It does show the full prompts in 'test in webchat' though.
Actual behavior
It cuts of the question after 11 characters and fills the rest with '...'.

Reproduction Steps
I've built my Bot in the Azure portal and connected it to my Facebook page. Then activated the Facebook Chat Plugin on my Wordpress 6.0 website.
Environment
Azure Appservice, Azure portal, Plugin on Wordpress 6.0 website.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Service Bus:0.17952341,Functions:0.11907785,Websites:0.047807105'
Label prediction was below confidence level 0.6 for Model:CategoryLabels: 'Mgmt:0.49624133,Client:0.26850235,Service:0.2250338'

Hi @michiel98fjhsad.  Thank you for reaching out and we regret that you're experiencing difficulties.  Unfortunately, this repository is focused on the Azure SDK for .NET - we're unable to assist with other Azure issues.  Since what you're describing appears to be related to the Azure Portal and not one of our SDK packages, your best path forward would be to open an Azure support request.
I'm going to close this out; if I've misunderstood what you're describing, please let us know in a comment and we'd be happy to assist as we're able.


Update liboqs-codeowners per liboqs PR #1943

See https://github.com/open-quantum-safe/liboqs/pull/1943.

Unconditionally, changes to config.yaml must

[ ] be approved by 2 members of the OQS TSC
[ ] not violate permissions documented in GOVERNANCE.md files for sub projects where such files exist

The following goals apply to changes to the file config.yaml with exceptions possible, as long as the rationale for the exception is documented by comments in the file:

[ ] all sub projects should be treated identically wrt roles & responsibilities as per the detailed list below
[ ] teams/team designations are to be used wherever possible; using personal GH handles should only be used in team definitions
[ ] Admin changes to the file must be documented by comments as to the rationale of the change

All the following conditions hold for permissions set in config.yaml:

sub project maintainers have admin rights on the sub projects
OQS and sub project release managers have maintainer rights on the sub projects but can themselves set/reset branch protection rules limiting write access to sensitive branches
sub project committers have write rights on all branches of the sub projects but can request branch protection rules limiting this
sub project contributors (incl. code owners) have write rights on all branches except main on those sub projects
OQS and sub project triage actors have triage rights on all branches of the sub projects
OQS maintainers and LF admins have admin rights on the organization (e.g., org-wide secret management) as well as maintenance rights on the team configurations

@open-quantum-safe/tsc The corresponding CODEOWNERS changes have been approved and are ready for merge in https://github.com/open-quantum-safe/liboqs/pull/1943 pending approval here. Per process, we need two approvals by TSC members here to proceed.

As per request from @christianpaquin, these are the people behind the affected GitHub handles for this PR:

crockeea is Eric Crockett
brian-jarvis-aws is Brian Jarvis
cothan is Duc Tri Nguyen
Martyrshot is Jason Goertzen
praveksharma is Pravek Sharma
ashman-p is Norman Ashley


🐛 [BUG]: Wrong props in the controlled flow guideline

Is there an existing issue for this?

[X] I have searched the existing issues and this is a new bug.

Current Behavior
Currently, the props is called apply-changes, it does not work
Expected Behavior
Remplace apply-changes by apply-default in the guideline documentation
Steps To Reproduce
No response
Relevant log output
No response
Anything else?
No response

Thanks, it's corrected now (will be visible in a couple mins).


[BUG] Transition often jumps to next card instead of smoothly animating

Describe the bug
When swiping through pages quickly, I notice that sometimes it animates smoothly and other times it seems to not animate and just jump to the next card
Video:
https://user-images.githubusercontent.com/120388/109396137-f8fae180-78e4-11eb-8f80-fe286d3ad2d9.mp4
Environment:

iOS
iPhone 12 XS Max
2.0.2

Additional context
I tried it on the simulator and on the device, with example projects and I'm able to reproduce the issue consistently.

I installed some older versions and it seems like the issue was introduced in 2.0.1

Hi @chrysb thanks a lot for your the feedback. I'll give it a look ASAP. In the meantime please use 2.0.0.
The video shows a manual transition. Is this the scenario where you detected the bug. I'm basically asking if your sample code used a manual transition (hit a button to go to next page) or it was some scenario where the user dragged the elements

@chrysb what are you doing exactly? I'm on develop and the example seems fine. This is iPhone 12 Pro Max.
https://user-images.githubusercontent.com/11335612/109400073-7868cf00-793e-11eb-85e3-c5441059882b.mov

I've been experiencing this on 14.4 and 14.5 (beta) when swiping too fast sometimes on tags/2.0.2 and iPhone 12 Pro Max. It also seems that onPageChanged isn't being called either in these instances.

I've created a new branch where I'm dealing with some minor issues. I pushed some changes in case you guys wanna gimme some feedback @vanities @chrysb . The branch is called fix/ios13-issues.
I'll see if I can give it a bit more testing before making a beta release

2.1.0

Sorry, I just saw this! I will test right now!

hey @fermoya. I have just updated to 2.1.0 but it seems like this issue still occurs on my end.
Tested this on my iPhone 12 Pro 14.4.2.
Am I missing something?
https://user-images.githubusercontent.com/9655723/113012993-9747c480-917b-11eb-857c-954cc86d69d0.MP4

Hi @beeirl are you testing on a simulator or real device?

@fermoyadrop on a real device

I can confirm this is still happening. Thanks for working on this! SwiftUIPager will be powering a core interaction in our new app.

Hi @chrysb , I’ve left a WIP in #207  if you want to take a look

Hi @chrysb , I’ve left a WIP in #207 if you want to take a look

Would love to, but can't seem to load the dragging-animation branch via SPM. Sorry for the dumb question, but do you know how?

@fermoya I tried it out and still experienced the issue, although it feels less pronounced. I also noticed a drop in FPS.
FYI — I'm using an interactiveSpring animation.

Hi @chrysb , what do you mean by FPS? interactiveSpring is a faster animation than .default, is that what you mean?
Also, can you send a video? I've been trying .draggingAnimation(.custom(,interactiveSpring)) and it seems fine. I don't think using a spring there is the right choice, at least one with a big response.

@fermoya By FPS I mean the actual framerate, it appears "laggy" as it seems to be dropping frames (a performance issue) that goes away when I revert back to 2.1.0.
Unfortunately I can't take a video because it's an unreleased product, wish I could be more helpful there.
Also, I was wrong, my animation is an .interpolatingSpring

@chrysb I see. There's some stuff to have in mind here. There's two animations in place:

onDragChanged
onDragEnded

Version 2.0.0 has both as .default and lets you changed onDragEnded animation. Version 2.1.0 changed onDragChanged animation from .default to .linear(duration: 0.1). This was fit to the fact .default animation made the animation delay while dragging (see #157). However, when the drag is fast enough, the last animations (onDragEnded seems ignored by SwiftUI) and that's why I suggested using the same animation on both onDragChanged and onDragEnded. This solution that you just tried doesn't work well with springs if the response is too high. I don't thing interpolatingSpring would work well. interactiveSpring would work better but it might still be too long of an animation. I'm thinking of enabling different animations for both events and let the user decide of what's best for them because I'm out of ideas here 🤷♂

@fermoya Thanks this is helpful to know! Did you build this using SwiftUI's DragGesture or is it UIKit under the hood?

@chrysb this is a DragGesture from SwiftUI. If you take a look to the documentation,they're very specific about springs. See interactiveSpring here:

A convenience for a spring() animation with a lower response value, intended for driving interactive animations.

@chrysb this is a DragGesture from SwiftUI. If you take a look at the documentation,they're very specific about springs. See interactiveSpring here:

A convenience for a spring() animation with a lower response value, intended for driving interactive animations.

Interesting! I'll give that a shot and see. Thank you :)


keepass-keefox: add myself as a maintainer

Keefox plugin for Keepass currently has no maintainer. Adding myself, as I've been the one bumping version so far.

That's great. Thank you


Prevent autopairs when next char is not whitespace (or when the next char is a letter)?

Is there a generic way to prevent autopairs to insert when the next char is not whitespace (or maybe better, the next char is a letter)? Or is this something that is configured for each rule individually? And if the latter, can I then add something to the config_internal_pairs block? Thanks!

{
  extensions={alpha={after=true}},
}

Thanks ❤


normal pop growth on EGTA

standard def
for PMID:30925937
we only have sensitive at present.

normal growth on EGTA FYPO:0006854
edit file: 771ddc6a790a6ebe6b86dea23a41583aca8b0b1c
release: 9cb110e7652dad44b54e9ba751cb393db4b03d13


GregTech ore metadata

GregTech ore blocks use the metadata value to indicate the hardness of ores. If the metadata is not set then all ores will have the lowest hardness and be breakable by a mere wooden pickaxe.

Isn't this what you whant?
https://github.com/lawremi/CustomOreGen/wiki/Documentation#tile-entity-support

While GregTech ores do use the NBT field, the point is that they also use the regular metadata field.
For example <OreBlock block='gregtech:gt.blockores' nbt='{m:500, n:1, id:"GT_TileEntity_Ores"}'> should actually be <OreBlock block='gregtech:gt.blockores:3' nbt='{m:500, n:1, id:"GT_TileEntity_Ores"}'>. Notice the :3.

Thanks for pointing this out. The actual meta value to use is an open question. Should it be randomized, e.g. based on the perlin noise field within a cloud? Or just set to a fixed value, on a per-material basis?

Ok, after some investigation, it looks like it is material-dependent. That's easy to replicate in the config.

Also, small ores should use a meta value 1 less than the meta for a regular ore.

I didn't precisely adhere to Greg's hardness values, mostly to maintain balance given that we are changing the generation patterns. However, I did follow the rule for small ores.


"Write after end" error

Every time when I made changes and deploy the node at that time I am getting the below error

At run time i.e. by clicking on inject node I am getting "done is not a function"

Hi hemangikhakhi, thank you for your report :)
can you please provide some more context?

What versions of node and node-red are you running?
What does your full flow look like? Can you please export it and paste it here?

Hi hemangikhakhi, thank you for your report :)
can you please provide some more context?

What versions of node and node-red are you running?
What does your full flow look like? Can you please export it and paste it here?

[{"id":"d207a3d8.ce07e","type":"http in","z":"e4605825.a089e8","name":"GET /api/subscribe","url":"/api/subscribe","method":"get","upload":false,"swaggerDoc":"","x":420,"y":480,"wires":[["508ee66.92e2618"]]},{"id":"508ee66.92e2618","type":"sse-server","z":"e4605825.a089e8","nodeId":"@tq-bit/node-red-contrib-server-sent-events","nodeVersion":"1.0.6","name":"Send message server event","event":"message","data":"","x":750,"y":560,"wires":[]},{"id":"a20b2f14.d15d1","type":"sse-client","z":"e4605825.a089e8","nodeId":"@tq-bit/node-red-contrib-server-sent-events","nodeVersion":"1.0.6","name":"Subscribe to message server event","url":"https://testnodeml.interplay.iterate.ai/api/subscribe","event":"message","headers":"","x":370,"y":660,"wires":[["b34774bd.a582d8"]]},{"id":"b34774bd.a582d8","type":"json","z":"e4605825.a089e8","name":"Parse JSON","property":"payload","action":"","pretty":false,"x":630,"y":660,"wires":[["68978f30.2a3d7"]]},{"id":"68978f30.2a3d7","type":"console","z":"e4605825.a089e8","name":"","active":true,"tosidebar":true,"console":false,"tostatus":false,"complete":"false","x":860,"y":660,"wires":[]},{"id":"7afe112c.7c206","type":"function","z":"e4605825.a089e8","name":"","func":"var require = context.global.get('require'); \n\n// msg.payload = {\n//   hello: 'world'\n// }\n    let i = 0;\n\n\n let timer = setInterval(write, 1000);\n\n\n    function write() {\n      i++;\n\n      if (i == 4) {\n        // res.write('event: bye\\ndata: bye-bye\\n\\n');\n        msg.payload = { \n            \"event\": \"bye\",\n            \"data\" : \"bye-bye\"\n        }\n        clearInterval(timer);\n       // res.end();\n       node.send(msg);\n        return;\n      }\n\n      msg.payload = {\n          'data:': i\n          \n      }\n      \n      node.send(msg);\n\n}\n\n    write();\n// return msg;","outputs":1,"timeout":10000,"newThread":false,"noerr":0,"x":390,"y":560,"wires":[["508ee66.92e2618"]]},{"id":"42bcdf12.3f8d5","type":"inject","z":"e4605825.a089e8","name":"","topic":"","payload":"","payloadType":"date","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":120,"y":560,"wires":[["7afe112c.7c206"]]}]

Unfortunately, I'm not able to reproduce your issue. Here's my setup:

I've put in a conditional check for done and moved the unsubscribe functions further down the lane.
Can you please check if your issues still persist?

Where to put a conditional check for done?

https://github.com/tq-bit/node-red-contrib-server-sent-events/commit/7a72f360155e48337035f10f860a0772e4bc5659
I also published it to npm and node-red, please update the node and check again

Thanks now it's working.

@hemangikhakhi
This issue is possibly fixed by this pull request.
Newest version 1.2.1 is on npm. You're welcome to check on this again


Adding readiness_checks

Added readiness checks which allow for much better control of the status of the different containers (load balancer will not send new requests to busy instances)
https://cloud.google.com/appengine/docs/flexible/java/configuring-your-app-with-app-yaml#updated_health_checks

Good PR, thank you @cedricbellet, I will run few tests before approving it.

Apparently the path argument does not work as described in the documentation. I deployed to my own projects and the paths for health checks was still the default liveness_check and readiness_check endpoints. I had to manually override the paths. That being said, moving to updated health checks is still the right thing to do.

Disregard previous comment - although the path shown in the UI does not match the YAML file, it seems to be working. Manually overwriting the paths messed everything up :) I think the commit works well as is (doing some more experiments)


Csv import siteIDs front

added isImported to know where the user come from -> used in back
added 'auto activation at import' to headers to send to BE + checkbox
added siteIDs to optional properties in modal
i18n messages to match Y/N dialog message

import tag pop up optionnal fields modified 👌🏻


WIP: Fixed grammar in example

What does this implement/fix?
Fix grammar and writing in the fNIRS example

Thanks Maureen, looks great.

Thx @mshader

#7057


Backport "HBASE-25739 TableSkewCostFunction need to use aggregated deviation (#3067)" to branch-2.3

Signed-off-by: Michael Stack<EMAIL_ADDRESS>Reviewed-by: David Manning<EMAIL_ADDRESS>

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 23s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
4m 13s
branch-2.3 passed

+1 :green_heart:
compile
1m  3s
branch-2.3 passed

+1 :green_heart:
shadedjars
5m 49s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 43s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 54s
the patch passed

+1 :green_heart:
compile
1m  4s
the patch passed

+1 :green_heart:
javac
1m  4s
the patch passed

+1 :green_heart:
shadedjars
5m 55s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 41s
the patch passed

_ Other Tests _

-1 :x:
unit
7m 25s
hbase-server in the patch failed.

33m 50s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3172

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux eebd88b39308 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / 545330ecbd

Default Java
AdoptOpenJDK-11.0.10+9

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/testReport/

Max. process+thread count
987 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 42s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
3m 58s
branch-2.3 passed

+1 :green_heart:
compile
3m 12s
branch-2.3 passed

+1 :green_heart:
checkstyle
1m 17s
branch-2.3 passed

+1 :green_heart:
spotbugs
2m 10s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 36s
the patch passed

+1 :green_heart:
compile
3m 13s
the patch passed

+1 :green_heart:
javac
3m 13s
the patch passed

-0 :warning:
checkstyle
1m 15s
hbase-server: The patch generated 1 new + 83 unchanged - 17 fixed = 84 total (was 100)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
18m 45s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
2m 21s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

49m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3172

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux 0bcc7e6beeb3 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / 545330ecbd

Default Java
AdoptOpenJDK-1.8.0_282-b08

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
86 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 47s
Docker mode activated.

-0 :warning:
yetus
0m  9s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
5m  3s
branch-2.3 passed

+1 :green_heart:
compile
1m  9s
branch-2.3 passed

+1 :green_heart:
shadedjars
6m 16s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 44s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 30s
the patch passed

+1 :green_heart:
compile
1m  9s
the patch passed

+1 :green_heart:
javac
1m  9s
the patch passed

+1 :green_heart:
shadedjars
6m 34s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 49s
the patch passed

_ Other Tests _

-1 :x:
unit
226m 48s
hbase-server in the patch failed.

257m  0s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3172

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 5270e079692c 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / 545330ecbd

Default Java
AdoptOpenJDK-1.8.0_282-b08

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/testReport/

Max. process+thread count
2266 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3172/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


#define statements in config.h can lead to collision with other libraries

The file include/oauth2/config.h generated by autoconf uses non specific #define statements, like #define PACKAGE_NAME.
As far as I can see, this file is used in include/oauth2/apache.h only.
If a program includes liboauth2 and another library with the same type of config.h file, the build will probably fail.
A solution is to prefix the #define statements in config.h to lower the risk of collision. What do you think?

I think that is kind of expected when using autoconf and I don't think there's even a way to do it differently (and still use authconf), see also https://www.gnu.org/software/autoconf/manual/autoconf-2.69/html_node/Configuration-Headers.html

I'm not familiar with autoconf so I've made some tests and it looks like the file config.h isn't even required for the library to work properly.
Here is the changes I've made for my test:

Remove AC_CONFIG_HEADERS([include/oauth2/config.h]) from file configure.ac
Remove config.h* references from file Makefile.am
Remove #include <oauth2/config.h> from the files include/oauth2/apache.h and test/check_version.c

After these steps, the library builds without any error and the tests are all successful (including memchached and redis tests), because autoconf defines all the config variables during compilation, example:
libtool: compile:  gcc -DPACKAGE_NAME=\"liboauth2\" -DPACKAGE_TARNAME=\"liboauth2\" \
-DPACKAGE_VERSION=\"1.3.0\" "-DPACKAGE_STRING=\"liboauth2 1.3.0\"" \
<EMAIL_ADDRESS>-DPACKAGE_URL=\"\" -DSTDC_HEADERS=1 \
-DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 \
-DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_DLFCN_H=1 \
-DLT_OBJDIR=\".libs/\" -I. -Wall -Werror -Iinclude -Isrc -g -O2 -MT src/cache/liboauth2_memcache_la-memcache.lo -MD -MP \
-MF src/cache/.deps/liboauth2_memcache_la-memcache.Tpo -c src/cache/memcache.c  -fPIC -DPIC \
-o src/cache/.libs/liboauth2_memcache_la-memcache.o

Don't think I'm trying to convince you to remove the file config.h because of my investigation, far from it, it's your project and your call, and I might have missed some important stuff.
But I think it was worth giving you the result of my tests...

After looking in to it, I believe this was a combination of a leftover from an early experiment plus a way to avoid ap_auto_config inclusion errors, so I have removed it now indeed; thanks.


deprecated does not work across modules without caches, nor within single module

This is very likely a more general issue wrt attributes.
Take these two files, where we expect to get a warning in B since we're using x and it's marked as deprecated.
module A

val x : int

[@deprecated]
let x = 2

module B

open A

let y = x

If we just check each file, we don't get any warning. From what I could tell, the attributes of x are empty when it's resolved during the desugaring of B.
$ ls A.fst* B.fst*
A.fst  B.fst
$ ./bin/fstar.exe A.fst
Verified module: A
All verification conditions discharged successfully
$ ./bin/fstar.exe B.fst
A.fst(0,0-0,0): (Warning 241) Unable to load A.fst.checked since checked file A.fst.checked does not exist; will recheck A.fst (suppressing this warning for further modules)
Verified module: B
All verification conditions discharged successfully

However we do get a warning if we use checked files:
$ ls A.fst* B.fst*
A.fst B.fst
$ ./bin/fstar.exe A.fst --cache_checked_modules
Verified module: A
All verification conditions discharged successfully
$ ./bin/fstar.exe B.fst --cache_checked_modules
B.fst(5,8-5,9): (Warning 288) A.x is deprecated
Verified module: B
All verification conditions discharged successfully

I also noticed that one should attach it to the let-binding instead of the signature for it to work here. Do the attributes of a let-binding take precedence over a val when both are present?
It also does not seem to work within a single module:
module C

val x : int
[@deprecated]
let x = 2

let y = x

$ ./bin/fstar.exe C.fst
Verified module: C
All verification conditions discharged successfully
$ ./bin/fstar.exe C.fst --cache_checked_modules
Verified module: C
All verification conditions discharged successfully

(I tried putting the attribure in the signature, and in both, with no difference.)

Tentative fix in https://github.com/FStarLang/FStar/commit/00fa3dac7fec1198982fadfd9079f2ef6dd1a729


Sketch in Windows

Hello,
Which program can open "mui-cssjs.sketch" in Windows OS?

"Sketch" is a design program for OS X. It might be possible to export the files to SVG. Would that help?

On May 31, 2015, at 4:24 PM, Sajad Dehshiri<EMAIL_ADDRESS>wrote:
Hello,
Which program can open "mui-cssjs.sketch" in Windows OS?
—
Reply to this email directly or view it on GitHub.

It's good idea!
I can use SVG files in Illustrator

Ok, great. I can look into it tonight.

On May 31, 2015, at 4:48 PM, Sajad Dehshiri<EMAIL_ADDRESS>wrote:
It's good idea!
I can use SVG files in Illustrator
—
Reply to this email directly or view it on GitHub.

svg files are in the 'svg' directory in master:
https://github.com/muicss/mui-design-files
Let me know if it works for you!

thanks :)
all files worked but windows LTR users need to install this font: "Helvetica Neue";


STORM-3379: Fix intermittent NPE during worker boot in local mode

Contains https://github.com/apache/storm/pull/2996, please review that first.
This issue is caused by a race. When Nimbus assigns executors to a supervisor, it writes them into Zookeeper. The supervisor is also told via Thrift (in local mode via a handle directly to Nimbus). When the worker starts and initializes WorkerState, it normally uses Thrift to ask the supervisor about the assignment. If that fails it tries Zookeeper. The attempt to read from Zookeeper is just a best-effort attempt to start the worker, it is inherently racy.
In local mode, the supervisor Thrift server isn't running, so the WorkerState just skips straight to reading from Zookeeper. Since the supervisor is told by Nimbus about assignment directly, there is no guarantee that the assignment becomes visible in Zookeeper before the WorkerState tries reading it. This causes the NPE.
The fix is to provide the Worker with a handle to the supervisor Thrift interface in local mode, similar to how the supervisor gets a handle to Nimbus. WorkerState can then ask the supervisor directly about the assignment.
I ran the tests in TestPlanCompiler (which runs a local cluster) a few hundred times, and they appear stable with this fix and STORM-3376.

Will also put +1 once everything is sorted out in #2996.

Squashed and rebased on top of STORM-3376. Will merge shortly. Thanks for the reviews.


Button: add parameter for stop propagation

Feature Type
[ ] Clarity Port
[ x ] Enhancement
Description
We have a scenario where clicking a button hosted in a datagrid column is selecting the row, which is undesired.
Need a way to set @onclick:stopPropagation="true"

Fixed in v2.0.0, released today


Issue setting up auth functionality manually

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [x ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

SUCCESS: Your application was packaged for Azure in 18 seconds.
Checking if authentication should be setup...
Loading azd .env file from current environment...
AZURE_USE_AUTHENTICATION is set, proceeding with authentication setup...
Creating Python virtual environment "scripts/.venv"...
Installing dependencies from "requirements.txt" into virtual environment (in quiet mode)...
Setting up authentication...
Checking if application 48427c9d-99a........... exists
Application already exists, not creating new one
Traceback (most recent call last):
File "/workspaces/azure-search-openai-demo/./scripts/auth_init.py", line 209, in
asyncio.run(main())
File "/usr/local/lib/python3.11/asyncio/runners.py", line 190, in run
return runner.run(main)
^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/asyncio/runners.py", line 118, in run
return self._loop.run_until_complete(task)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
return future.result()
^^^^^^^^^^^^^^^
File "/workspaces/azure-search-openai-demo/./scripts/auth_init.py", line 184, in main
server_object_id, server_app_id, _ = await create_or_update_application_with_secret(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/workspaces/azure-search-openai-demo/./scripts/auth_init.py", line 64, in create_or_update_application_with_secret
await update_application(auth_headers, object_id, app_payload)
File "/workspaces/azure-search-openai-demo/scripts/auth_common.py", line 32, in update_application
raise Exception(response_json)
Exception: {'error': {'code': 'Authorization_RequestDenied', 'message': 'Insufficient privileges to complete the operation.', 'innerError': {'date': '2024-03-20T21:42:30', 'request-id': '45d0a377..............', 'client-request-id': '45d0a...............'}}}
ERROR: failed running pre hooks: 'preprovision' hook failed with exit code: '1', Path: './scripts/auth_init.sh'. : exit code: 1
ERROR: error executing step command 'provision': failed running pre hooks: 'preprovision' hook failed with exit code: '1', Path: './scripts/auth_init.sh'. : exit code: 1
Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful
I'm trying to set up the auth functionality, but dont have access / permissions to Azure AD for my organization. I had somebody with permissions configure the two apps as outlined in the loginAndAclSetup.md file. I then set the environment variables with the app ids on my end but get the error outlined above.
Any thoughts what might be causing this /how to solve?
Thanks in advance!

Thanks! We'll be in touch soon.

Yep - i see the permission error, i guess I'm wondering what is the minimum permission I'd need to get this working. Can these apps be set up by an administrator manually ?

Yep - i see the permission error, i guess I'm wondering what is the minimum permission I'd need to get this working. Can these apps be set up by an administrator manually ?

Yes. There are instructions here.

Thanks, just wanted to make sure: Is there a way to create the apps separately by an administrator and run the authentication functionality on my script?  I can't have application administrator for my organization, what would be the workaround ?

Thanks, just wanted to make sure: Is there a way to create the apps separately by an administrator and run the authentication functionality on my script? I can't have application administrator for my organization, what would be the workaround ?

Not easily. You would have to make some code changes in both the front and back end. If you can’t get the application registration created then perhaps you could continue without the authentication component?

Yep - that's what they did and i was running into the above error. Took a slightly different approach:
Set up authentication with my administrator by going to the app service, and having them create the app vis that wizzard. That effectively added a login page for the application. However, does this mean that the blob storage resource is also secure? I'm taking a look at it , it says it is accessible through public domain. Any thoughts here would be helpful. Sorry if this is a noob question.

By default the storage account is public as in, accessible to the internet. But it still requires a key or SAS to access the blobs.
Unfortunately, this really isn’t the place to get technical support, sorry.


update jems. Implements #99

Codecov Report

Merging #100 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #100   +/-   ##
=========================================
  Coverage     65.55%   65.55%
  Complexity      484      484
=========================================
  Files           147      147
  Lines          1710     1710
  Branches        115      115
=========================================
  Hits           1121     1121
  Misses          532      532
  Partials         57       57

Continue to review full report at Codecov.

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


SortedRange.empty should be const, .front should be inout

s.naarmann (@SimonN) reported this on 2020-09-02T20:51:44Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=21216
Description
Example code that fails to compile with DMD 2.093.0 on Linux 64-bit:

    import std.range;

    class C { }

    struct A {
    private:
        SortedRange!(C[]) _backend;

    public:
        inout(C) front() inout {
            return _backend.front();
        }

        bool empty() const {
            return _backend.empty();
        }
    }

    void main() { }

This generates two errors: Can't call front() on an inout _backend, and can't call empty() on const _backend. The compiler even suggests annotating SortedRange's methods with const/inout.

Expected instead: The above code compiles.

Workaround: Replace "SortedRange!(C[]) _backend;" with "C[] _backend;", and design struct A to ensure that _backend is always sorted.

In Phobos's source for std.range.SortedRange, empty() is declared non-const, but has a comment "//const". This has been untouched since at least 2014.

---

Related bug:
https://issues.dlang.org/show_bug.cgi?id=9792
length field of a const SortedRange

s.naarmann commented on 2020-09-08T15:34:37Z
Discussion on the D Forums:
<EMAIL_ADDRESS>
Gist: Instead of adding const/inout, which might not fit every wrapped range, we should use `template this` to let the compiler figure out the appropriate annotations for empty/front/... individually for each wrapped range.


doc(*): initial spec draft

Initial spec draft.
Did some changes to README.md. Build instruction in README.md.

Not quite sure what to do with the logo. I just copied it from the the project root.


Generate pre-signed S3 PUT URLs that the frontend can use to upload to S3

This is the recommended AWS approach for uploading to S3: the backend generates the S3 pre-signed PUT URL with the required condition and fields. And then the frontend uses that information to upload from the frontend.
Please see the first answer here: https://stackoverflow.com/questions/54076283/how-to-upload-a-file-to-s3-using-presigned-url-with-react-js for an example of what needs to be done on the frontend side (using axios).
I have tested this locally using curl and it works 😄  Can upload either image or asset, can also overwrite/update if you provide
S3 key. This handler will serve as THE handler for all uploads to S3.
Happy to show a demo during the work session if needed.

Nice work!
When a user is uploading an asset, we also want them to create a new record in the asset table. Do we need to fire another request for that or can we bundle them into one asset POST request? Maybe this would be for another ticket (because we don't need this behaviour for picture upload) but curious about it

When a user is uploading an asset, we also want them to create a new record in the asset table. Do we need to fire another request for that or can we bundle them into one asset POST request? Maybe this would be for another ticket (because we don't need this behaviour for picture upload) but curious about it

I agree, in my opinion we do want that happening within this POST. However, we can do that later unless Jay is feeling up for it for this particular PR. At the very least, let's put a "TODO" within the type == asset if statement so we know where it'll need to be added.

No, this handler won't include (now or in the future) creating a new record in the asset table, so that will need to be a separate POST request once FE uploads the file to S3. We already have a route for handling that (https://github.com/uwblueprint/cdc-backend/blob/master/app/routes/admin/asset.py#L14) and there's no point in duplicating the validation and creation logic from there to here. If we have that here, we would have to add conditional validation on the JSON body of this request, and this handler could throw errors that are totally unrelated to AWS. Furthermore, if we add the record in asset table here, we would be adding an s3_key in asset table that is currently not valid. So if the upload fails for some reason, we would have to add logic for deleting the asset. If we added logic for POST function of the asset, then we would also have to handle PUT of the asset table here because this function also allows for updating a file in S3.
This handler's function is supposed to be completely agnostic to Postgres and other logic (like asset creation/updating). This handler's function is simple: you want to upload something to S3, this function will serve as a proxy so FE can upload to S3 without needing AWS credentials.


Add nav for pages

Adds nav and header
Hooks up no_header option

closes https://github.com/axodotdev/oranda/issues/43
related https://github.com/axodotdev/oranda/issues/24

Based on the element discussion, I think the changes I would recommend for this PR are:

putting nav under title
rebasing to get new CSS
adding header CSS to the theme.css in fringe
++ maybe to address https://github.com/axodotdev/oranda/issues/69, but that's a maybe

(anyways that's the bits I captured)

CSS was added here and removed from oranda
https://github.com/axodotdev/turbo-axo/pull/177

This looks good to me, but I think we should bring back the Big Header style. The two h1s look unbalanced otherwise.


Pytest Failing "where 1 = <Result TypeError("object of type 'int' has no len()")>.exit_code"

with updates where we pass an int as ID instead of a string, test_success_download, test_fail_download, and test_metadata are failing because
assert 1 == 0
+  where 1 = <Result TypeError("object of type 'int' has no len()")>.exit_code

I'm really confused about where this is coming from since the only place i use len() is on a list. Any advice on how to look more into the mocks in order to see where this could be coming from?
When I add print statements they don't show up in pytest

pytest -s at least shows the stdout (i.e. the print statements).

@cbarnes7
Wow, that was frustrating to debug! The issue is that the invoke function always expects a list of strings, even when you would be passing numbers at the command line. Should be good to go if you make that change. I also did a code review on your PR: https://github.com/PHM-Data-Hub/lib/pull/15

The typer docs mention this, but their example never passes a number, so it was ambiguous: https://typer.tiangolo.com/tutorial/testing/#call-the-app

The type annotations confirm: https://github.com/tiangolo/typer/blob/master/typer/testing.py#L13
args: Optional[Union[str, Iterable[str]]] = None,