Terminal or Terminal Preview crashes when dragging a tab

Windows Terminal version
1.19.3172.0
Windows build number
11.22631.2506
Other Software
No response
Steps to reproduce

Install Windows Terminal or Windows Terminal Preview using Microsoft Store
After the instalation, the app will be works correctly
Close the app
Disable Windows Error Reporting (WER), running Disable-WindowsErrorReporting into Powershell
The application no longer opens

This problem started after disabling the WER service. I didn't update Windows or the app.
Expected Behavior
App working correctly.
Actual Behavior
The application no longer opens. I was enabled WER again (running Enable-WindowsErrorReporting and restarted Windows). But, the problem remains.

I generated the Debug ETL Trace.
terminal-trace.zip

Just to try and clarify: Does the terminal fail to launch? Or does it launch, and crash when you try dragging a tab/?

Would you happen to be running as admin (or with UAC entirely disabled)?
Do you have something like BeyondTrust running on this machine?

my own notes:

searching for 035b9761-fe39-4ecc-afc3-8b535e35cdc3 gave me failure ID 06b992a4-b446-d9d3-3a52-cd9f0ca6e82c which is MSFT:47683995
15b488c9-67cd-4b43-a1f0-73be90c169f3 actually pointed me at the same failure ID
That exception is STOWED_EXCEPTION_80070002_Microsoft.UI.Xaml.dll!event_base_event_source_winrt::Windows::Foundation::TypedEventHandler_winrt::Microsoft::UI::Xaml::Controls::NavigationView,winrt::Microsoft::UI::Xaml::Controls::NavigationViewBackR

Any chances that you've got the Terminal configured to open the tabs from a previous session, and those tabs included a Settings tab? That's my only thought as to why you might be seeing a nav view crash...

This for sure is a confusing one! Especially something like

But, I cannot open more tabs - the app closes unexpectedly

That's definitely not something that should be happening! Can you try filing /feedback with Feedback Hub, and make sure to "Recreate my problem"? Hopefully that'll give us a dump that's got more info in it

Hello @zadjii-msft!
I just reproduced the issue again and submitted it via the "Feedback Hub": https://aka.ms/AAny7ni
Here are some additional pieces of information that I believe are important:
I had to format my computer and reinstall Windows because other strange issues started occurring after a Windows Insider update failed to install.
After reinstalling Windows, I did not enable Windows Insider, and I did not attempt to disable Windows Error Reporting. Unfortunately, the problem I reported in this issue has recurred.
Details of my computer are as follows:
Windows 11.22621.2715
Terminal Preview 1.19.3172.0
Terminal 1.18.3181.0
Thank you!

Hello guys!
Is there an estimated timeline for the resolution of the Windows Terminal issue? Additionally, is any further information required to assist in the troubleshooting process?
Thanks,
Marcos

@zadjii-msft @carlos-zamora

Sorry no, we don't have any sort of timeframe for a fix here. As it stands, it kinda seems like you're the only one hitting this, and I couldn't figure it out just looking at the crash dumps we got. Ironically, with error reporting disabled, there's no real dumps or... seemingly anything I can find from your machine 🤷

Hello @zadjii-msft.
Thanks a lot for your help.
I will try to send new feedback.
About the error message in the Event Log:

Nome do aplicativo com falha: WindowsTerminal.exe, versão: 1.19.2402.26003, carimbo de data/hora: 0x65dce3ea
Nome do módulo com falha: Windows.UI.Xaml.dll, versão: 10.0.22621.3350, carimbo de data/hora: 0x01eae29f
Código de exceção: 0xc000027b
Deslocamento da falha: 0x0000000000872c90
ID do processo com falha: 0x7104
Hora de início do aplicativo com falha: 0x1DA7A240988A23A
Caminho do aplicativo com falha: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.19.10573.0_x64__8wekyb3d8bbwe\WindowsTerminal.exe
Caminho do módulo com falha: C:\Windows\System32\Windows.UI.Xaml.dll
ID do Relatório: 609ac06d-59e9-4191-b851-fe6deaa7ce8e
Nome completo do pacote com falha: Microsoft.WindowsTerminal_1.19.10573.0_x64__8wekyb3d8bbwe
ID do aplicativo relativo ao pacote com falha: App

Is there anything I can do to resolve this? For example: reinstall the XAML library, register the .dll again...
Thanks!

Hello @zadjii-msft! Thanks for your help.
If I launch the Terminal or the Terminal Preview from the Start Menu or Store, the app fails to launch and generates an item in the Event Viewer.
However, if I run "cmd" in the "Run" option (Win + R), the Terminal (or Terminal Preview) opens the prompt correctly. But, I cannot open more tabs - the app closes unexpectedly.
I don't have BeyondTrust installed on my machine. Before I tried disabling WER, the Terminal or Terminal Preview app was working correctly.
How can I check if my Terminal was configured to open the previous session? I don't remember enabling this feature...

This is my problem:

@zadjii-msft I'm using the default configuration, because I had uninstalled the Terminal last week and reinstalled it today. See my settings, now in English (previously in pt-br):

This issue has two behaviors:

If I launch the Terminal app from the Windows Store or Start menu, nothing happens - the Terminal does not open, and I get an error message in the Event Viewer.
If I run "cmd" using Winkey + R, the Terminal opens with the Command Prompt. Here, when I try to open a new tab, the app closes unexpectedly. The Terminal crashes with any profile - Azure, Prompt, WSL...

Well okay that does differentiate at least one part here. Connections that are created for "default terminal" ("defterm") invocations seem to work for you. But Terminal-initiated connections don't.
This vaguely makes me think of #16747 for whatever reason. From that thread:

Can you run conhost -- cmd from the Run dialog, then run set, and copy-paste the output here[^1]? Looks like there's some env var somewhere in there that's blowing up our env var loader/?
Alternatively, testing if this repros the same in 1.20 preview

Another idea: does wt -w 0 new-tab --inheritEnvironment work to open a new tab?
[^1]: this may have some amount of PII in it

Result of conhost -- cmd + set:

ALLUSERSPROFILE=C:\ProgramData
APPDATA=C:\Users\marcos.jahn\AppData\Roaming
CommonProgramFiles=C:\Program Files\Common Files
CommonProgramFiles(x86)=C:\Program Files (x86)\Common Files
CommonProgramW6432=C:\Program Files\Common Files
COMPUTERNAME=J05811
ComSpec=C:\Windows\system32\cmd.exe
DriverData=C:\Windows\System32\Drivers\DriverData
EFC_2388=1
FPS_BROWSER_APP_PROFILE_STRING=Internet Explorer
FPS_BROWSER_USER_PROFILE_STRING=Default
HOMEDRIVE=C:
HOMEPATH=\Users\marcos.jahn
IntelliJ IDEA Community Edition=C:\Program Files\JetBrains\IntelliJ Community\bin;
JAVA_HOME=C:\Work\Programas\openjdk-11.0.6-windows-x64
JENV_HOME=C:\Work\Programas\JEnv\latest
LOCALAPPDATA=C:\Users\marcos.jahn\AppData\Local
LOGONSERVER=\IRIR
M2_HOME=C:\Work\Programas\Apache\Maven\3.6.0
MAVEN_OPTS=-Xms2g -Xmx12g -XX:MaxDirectMemorySize=4096m
NUMBER_OF_PROCESSORS=8
NVM_HOME=C:\Work\Programas\nvm
NVM_SYMLINK=C:\Program Files\nodejs
OneDrive=C:\Users\marcos.jahn\OneDrive
OS=Windows_NT
Path=C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0;C:\Windows\System32\OpenSSH;C:\Program Files\RedHat\Podman;C:\Program Files\Git\cmd;C:\Work\Programas\JEnv\latest;C:\Work\Programas\Apache\Maven\3.6.0\bin;C:\Work\Programas\nvm;C:\Program Files\nodejs;C:\Users\marcos.jahn\AppData\Local\Microsoft\WindowsApps;C:\Users\marcos.jahn\AppData\Local\Programs\Microsoft VS Code\bin;C:\Work\Programas\nvm;C:\Program Files\nodejs;C:\Program Files\JetBrains\IntelliJ Community\bin
PATHEXT=.COM;.EXE;.BAT;.CMD;.VBS;.VBE;.JS;.JSE;.WSF;.WSH;.MSC
PROCESSOR_ARCHITECTURE=AMD64
PROCESSOR_IDENTIFIER=Intel64 Family 6 Model 140 Stepping 1, GenuineIntel
PROCESSOR_LEVEL=6
PROCESSOR_REVISION=8c01
ProgramData=C:\ProgramData
ProgramFiles=C:\Program Files
ProgramFiles(x86)=C:\Program Files (x86)
ProgramW6432=C:\Program Files
PROMPT=$P$G
PSModulePath=C:\Program Files\WindowsPowerShell\Modules;C:\Windows\system32\WindowsPowerShell\v1.0\Modules
PUBLIC=C:\Users\Public
SciTE_HOME=C:\ProgramData\SciTE
SciTE_USERHOME=C:\Users\marcos.jahn\AppData\Roaming\SciTE
SESSIONNAME=Console
SystemDrive=C:
SystemRoot=C:\Windows
TEMP=C:\Users\MARCOS~1.JAH\AppData\Local\Temp
TMP=C:\Users\MARCOS~1.JAH\AppData\Local\Temp
USERDNSDOMAIN=J1.LOCAL
USERDOMAIN=J1
USERDOMAIN_ROAMINGPROFILE=J1
USERNAME=marcos.jahn
USERPROFILE=C:\Users\marcos.jahn
windir=C:\Windows
ZES_ENABLE_SYSMAN=1

Running wt -w 0 new-tab --inheritEnvironment, the Terminal opens using the Default Profile. I tried changing the Default Profile to Powershell and WSL, which worked. But the Terminal crashes when I try to open a second tab.
Using the 1.20 preview, the problem also occurs.

Running wt -w 0 new-tab --inheritEnvironment, the Terminal opens using the Default Profile

WEIRD. I bet this IS the same thing!

What's the correct or recommended folders for these user environment variables: TEMP and TMP?

WEIRD. I bet this IS the same thing!

"8.3 paths"? Can you explain, please?

Sua resposta está clara, mas pode ser um pouco mais formal e educada para garantir uma comunicação respeitosa, especialmente se você está se comunicando em um fórum público ou com alguém que você não conhece bem. Aqui está uma sugestão de como você poderia reformular sua resposta:
Hello @DHowett!
Are you referring to this MacType? https://www.mactype.net/
No, I don't use it.

open a second tab.

Are you a MacType user per chance?

Hello @DHowett!
Are you referring to this MacType? https://www.mactype.net/
No, I don't use it.

"8.3 paths"? Can you explain, please?

Honestly, I don't even know. These predate my own familiarity with computers. But wikipedia's got a overview: https://en.wikipedia.org/wiki/8.3_filename

What's the correct or recommended folders for these user environment variables

tbh I would have thought that those paths should work. I'm more gluing together two half thoughts:

Your TMP variables are curious
https://github.com/microsoft/terminal/issues/16747#issuecomment-1981981063 tracked down some weird debug logs that pointed at TMP variables, but we didn't know why
disabling our env var generation (with --inheritEnvironment) did work to open a new tab

Hence why I'm thinking there's a bug here in the nexus of all this.
I honestly wouldn't expect the TMP vars to be 8.3 paths, I thought they'd just be normal paths? So switching them to C:\Users\marcos.jahn\AppData\Local\Temp instead might just work. But also, we shouldn't crash if we do encounter one of those...

@zadjii-msft Apparently, my user environment variables are configured correctly:

Hmm.

[x] tried this with a user with a very long name - that didn't repro locally.
[x] Tried one with dots: that didn't work either. weird weird weird.

Oh I had a thought: Not necessarily a root cause, but a way to mitigate the pain here.
You should be able to add "compatibility.reloadEnvironmentVariables": false to your settings (under each profile, or under profile.defaults in the JSON file). That should disable the environment variable regeneration entirely. There may be other unexpected things that act weird, but honestly, I'm sure it's better than crashing.

open a second tab.

Are you a MacType user per chance?

This was my issue! Thanks!


Cannot allocate 2MB and 1GB pages

There is no API to allocate 2MB or 1GB pages so user-land developers are forced to use minuscule 4KB pages everywhere.

The main obstacle here is making the memory allocator work well with this.  If you have a good solution, I'd love to hear it.


Only first img object is loaded

Perform all steps below and tick them with [x]

[ X] Check the related part of the Documentation
[ X] Update openHASP to the latest version
[ X] Reproduce the issue and describe all steps

Describe the bug
When adding multiple img's only the first one is loaded. It does not matter if the src is http:// og L:/
There is nothing wrong with the images. If I switch them around again only the first one is loaded.
Version | 0.7.0-dev 265f9c4
Build DateTime | Feb 25 2023 16:40:45 UTC
Environment | wt32-sc01-plus_16MB
To Reproduce
Provide a small, independent code sample that can be used to reproduce the issue.
Format the code like this:
{"page": 1, "id": 1, "obj": "img", "src": "http://<IP_ADDRESS>/openhasp/hk-Attic.png", "auto_size": 1,"h":240}
{"page": 1, "id": 2, "obj": "img", "src": "http://<IP_ADDRESS>/openhasp/hk-Bedroom.png", "auto_size": 1,"w":240, "y":240}

Expected behavior
Expected both images to load
Screenshots or video

[12:57:05.924][73716/89860 17][59244/59404  1] HASP: Loaded /pages.jsonl
[12:57:06.446][77812/92136 15][58276/58436  1] LVGL: Image draw cannot open the image resource
[12:57:06.448][77812/90540 14][58276/58436  1] LVGL: Image draw error
[12:57:06.602][77812/92136 15][58168/58328  1] LVGL: Image draw cannot open the image resource
[12:57:06.604][77812/90428 13][58168/58328  1] LVGL: Image draw error
[12:57:06.757][77812/92136 15][58168/58328  1] LVGL: Image draw cannot open the image resource
[12:57:06.759][77812/90540 13][58168/58328  1] LVGL: Image draw error
[12:57:06.913][77812/92136 15][58168/58328  1] LVGL: Image draw cannot open the image resource
[12:57:06.915][77812/90540 14][58168/58328  1] LVGL: Image draw error
[12:57:08.065][73716/90192 18][58276/58436  1] LVGL: Image draw cannot open the image resource
[12:57:08.068][73716/88608 16][58276/58436  1] LVGL: Image draw error
[12:57:08.259][73716/90192 18][58168/58328  1] LVGL: Image draw cannot open the image resource
[12:57:08.262][73716/88464 16][58168/58328  1] LVGL: Image draw error
[12:57:08.458][73716/90192 18][58168/58328  1] LVGL: Image draw cannot open the image resource
[12:57:08.461][73716/88496 16][58168/58328  1] LVGL: Image draw error
[12:57:08.678][73716/90192 18][58168/58328  1] LVGL: Image draw cannot open the image resource
[12:57:08.681][73716/88496 16][58168/58328  1] LVGL: Image draw error

Hi' @Asbjoern
Well, could be either one or perhaps even two things - out of two 😉
I've experienced exactly same issue and same log entries.
LVGL is not a happy camper, and you're straining the mcu by applying multiple png images simultaniously. Images that the mcu must process 'on the fly'

Convert the images to binary images instead and upload these. Use the online LVGL tool here

You might also experience that you actually run out of dedicated LVGL Memory handling/shifting between several medium/large sized bg images. I bumped my LVGL memory stack to 80kb (default 64kb) and haven't had any LVGL memory issues since then. Note that this will 'cost you' on the general device memory stack, so use with care !

Thanks for your reply.
Converting to binary did the trick and will work for me.

Out of curiosity, when sending images to a plate via the home assistant integration, is it automatically applying the binary transformation?
I am asking because I have implemented a kind of "camera view" which updates every couple of seconds. This works well on the last 0.6.x version, but does not work at all on 0.7.0-dev. I have not yet investigated this further and will probably open a new issue if it persists after 0.7.0 release (or earlier if desired).

Yep, Custom Components serves the images as binary images
Anything in the logs when you issue these push_image commands ?
Check both HA- and openHASP plate logs

Hi @htvekov ,
I have this same issue on a sc01 plus with 16Mb. I migrated a configuration from my wt32-sc01 to the sc01plus.
On the wt32-sc01 all works perfectly and on the sc01plus I have the no data issue.
I use 0.7.0-rc2 3ab88a3 version on both displays.
I thought these devices should have the same problem when running the same OpenHasp version

I thought these devices should have the same problem when running the same OpenHasp version

Devices have diffents specs. Most wt32-sc01's have 4MB flash where the plus model has 8 or 16MB.
Both have 2MB of PSram.
The plus model has an ESP32-S3 where the original version has an ESP32.
The difference you experience could have numerous causes. Difference in memory mapping, partition scheme etc.
The fact your on same fw does not neccesarily mean that fw is using same partition scheme. If you've OTA updated from 0.6.3 or 0.6.4 to 0.7.x the original partition scheme will remain. Only serial flashing with 0.7.x fw will use the new partition scheme.
I have not enough insight in this to give more than hints to the root cause.
We need @fvanroie insight on this 🙂

Hi @htvekov ,
I have this same issue on a sc01 plus with 16Mb. I migrated a configuration from my wt32-sc01 to the sc01plus. On the wt32-sc01 all works perfectly and on the sc01plus I have the no data issue. I use 0.7.0-rc2 3ab88a3 version on both displays.
I thought these devices should have the same problem when running the same OpenHasp version

Hard to say without the pages.jsonl file and images...

{"page": 1, "id": 1, "obj": "img", "src": "http://<IP_ADDRESS>/openhasp/hk-Attic.png", "auto_size": 1,"h":240}
{"page": 1, "id": 2, "obj": "img", "src": "http://<IP_ADDRESS>/openhasp/hk-Bedroom.png", "auto_size": 1,"w":240, "y":240}

Do the images show correctly when you upload the images to flash and load them from there?


Update multi-tenancy route to match name change

Signed-off-by: Craig Perkins<EMAIL_ADDRESS>Description
The route to multi-tenancy config was renamed in this PR https://github.com/opensearch-project/documentation-website/pull/1786 and broke existing links elsewhere in the documentation. This PR updates the existing links to account for the name change.
Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@cwillum This PR updates broken links to multi-tenancy documentation.

@cwperks All links are updated to point to the multi-tenancy index now.


More detailed example for creating client

I'm coming from https://github.com/amz-tools/amazon-sp-api to this project for the type script in this project.
But one thing I'm having trouble with is figuring out how to create the client. With amazon-sp-api, the documentation is very clear on how to create an access token. In fact, the client in that project, the client manages the access token itself, so I don't have to worry about expiration.
With this project it's less clear to me from the docs how to create the access token...some additional documentation would be very helpful.

Hi Kyle,
Thank you for reaching out.
There's an example of how to create an instance of a client class, which takes acesssToken as a prop.
But how you obtain this access token is up to you.
There are just too many options and permutations to create a neat abstraction at this point.
To get the token you need to follow the guide outlined in the SP API docs, or self authorize if it's an internal app.
Is this more clear?
Were you expecting this package to handle the OAuth flows as well?
Cheers!

Hi Kyle,
I've added more docs. Please take a look and let me know if you think this is more helpful.

It handles the request to get the access token and calling sts (to exchange a user's accessKeyId and secretAccessKey for an STS token) for me.

Yes, our package can do this too. I've added an example for this.
We are trying to keep it flexible, as there are many potential scenarios, and making the STS call inside the package is not always desirable, as it can be an overhead that is not necessary.
For example, our Lambda that makes the calls to the SP API, launches with the already assumed role and correct credentials (via standard AWS process), ready to make calls to the API. We do not need to make an additional STS call.

Could you please show any example on how to obtain the accessToken?  I'm currently stuck on this.

Could you please show any example on how to obtain the accessToken?  I'm currently stuck on this.

That's outside the scope of this package. It assumes you already have a token.

Could you please show any example on how to obtain the accessToken? I'm currently stuck on this.

@moltar, if you have a refresh_token, check kneumei's comment or the authentication functions under the test directory.

Could you please show any example on how to obtain the accessToken? I'm currently stuck on this.

Just passing alone so anyone new can start it:
`import { SellersApiClient } from '@scaleleap/selling-partner-api-sdk'
import axios from 'axios'
const credentials = {
accessToken: 'Atza|..',
refreshToken: 'Atzr|...',
roleArn: 'arn:aws..',
appClientId: 'amzn1...',
appClientSecret: 'amzn1..',
accessKeyId: 'XX..',
secretAccessKey: 'XX..',
region: 'na',
marketplaceId: 'ATVPDKIKX0DER',
sellerId: 'XX'
}
const response = await axios.post('https://api.amazon.com/auth/o2/token', new URLSearchParams({
grant_type: 'refresh_token',
refresh_token: credentials.refreshToken,
client_id: credentials.appClientId,
client_secret: credentials.appClientSecret
})
.toString(), {
headers: {
'Content-Type': 'application/x-www-form-urlencoded;charset=UTF-8'
}
})
const accessToken = response.data.access_token
const client = new SellersApiClient({
accessToken,
// Or use amazonMarketplaces.CA.sellingPartner.region.endpoint
// from @scaleleap/amazon-marketplaces package
basePath: 'https://sellingpartnerapi-na.amazon.com',
// Or use amazonMarketplaces.CA.sellingPartner.region.awsRegion
// from @scaleleap/amazon-marketplaces package
region: 'us-east-1',
roleArn: credentials.roleArn,
credentials: {
accessKeyId: credentials.accessKeyId,
secretAccessKey: credentials.secretAccessKey,
sessionToken: credentials.sessionToken
}
})
const marketplaceParticipations = await client.getMarketplaceParticipations()
console.log(marketplaceParticipations.data.payload)
`


Border on the left and top sides of the screen during full screen video playback on Firefox

I have verified that the issue is not related to the MicaForEveryone project.

The issue is not limited to YouTube, using Firefox in full-screen mode yields the same result.

I'll see what i can do.

This is a browser bug and it only happens under special conditions. I can't do anything here. Read issue #34 if you're interested in the details.


ClientGlobalContext.js.aspx deprecated again?

The "Dynamics 365 and Power Platform Monthly Update-May 2022" mentions that the ClientGlobalContext.js.aspx will be deprecated in October 2022. Is this true or outdated information in the Monthly Update?
Link to the Monthly Update: https://community.dynamics.com/crm/b/crminthefield/posts/dynamics-365-and-power-platform-monthly-update-may-2022

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

ID: 1a241829-ba79-0d2e-d061-cedb8978fbeb
Version Independent ID: 867a90ce-0a3e-0a82-0458-acf7e783e961
Content: Important changes (deprecations) coming in Power Apps and Power Automate - Power Platform
Content Source: power-platform/important-changes-coming.md
Service: power-platform
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

Thanks for your comment, @darshSWL. This issue has been assigned to the doc author for review.

@darshSWL It's outdated information. ClientGlobalContext.js.aspx isn't deprecated. Closing the issue.


[Platfrom]Add capability which uploading manifest.json to show analysis report on rsdoctor report platform.

Rsdoctor need a capability to  upload  manifest.json add show the analysis data on rsdoctor report platform.

The issue will be closed because the entire .rsdoctor folder cannot be uploaded via the browser. You can use @rsdoctor/cli to open the report：https://rsdoctor.dev/guide/start/cli


Ubuntu 22/04 version 20230924.1.0 does not generate androidx.databinding.DataBindingComponent (part 2)

Description
Last week I submitted issue https://github.com/actions/runner-images/issues/8409.
Because I could not give access to my project that ticket was closed.
I can not re-open it, but I did create a repo with the exact same problem. I can give access to this new repository to whomever picks up this issue.
Please let me know whom I should give access.
Description copied from previous ticket:
Since the update to version 20230924.1.0 the build actions for my project fail because the androidx.databinding.DataBindingComponent class is not generated. That is the only error I see, I can't find the reason for this class not being generated anymore.
Platforms affected

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

Runner images affected

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

Image version and build link
https://github.com/FDMediagroep/android-failing-build-20231004/actions/runs/6406656111
Is it regression?
Yes, it worked fine with version 20230917.1.0
Expected behavior
I expect my project to build successfully as it did with the previous version of the runner.
Actual behavior
Since the update to version 20230924.1.0 the build actions for my project fail because the androidx.databinding.DataBindingComponent class is not generated. That is the only error I see, I can't find the reason for this class not being generated anymore.
Repro steps
Execute the build action of my project

Hello @stspenke. We will try to find a person who can take a look. For now, you could give access to me.

Thanks a lot.
Without any code change, my build started to fail with the new runner.
It is quite possible that the fix needs to be made in my project, e.g. that my code contains some deprecated functionality, which the new runner no longer supports, but I can't find out what it is.

Hello @sergei-pyshnoi,
Do you already have any progress or findings for this issue?

Hello @sergei-pyshnoi, Do you already have any progress or findings for this issue?
Hi @stspenke . Thank you for your patience, We still on debug process and doesnt find nothing useful. Also look like it is project specific problem because we didnt receive similar reports. I will let you know as soon as we get any results.

It has been a week since I last heard from you...
Do you have any news?

@stspenke Glad to hear that problem is gone. I am not sure what exactly fix the issue, but I am sure it is related android sdks changes in last image update. I closing this issue as fixed. Feel free to ask if you have additional questions.

@sergei-pyshnoi I am very sad to see that after being on holiday for 3 weeks our builds are now again failing with the same problem... Can this ticket be reopened?
It seems that after every update to the pipelines there is a chance that the builds of our project start failing again, or are magically working again.

And since yesterday the build is succesful again...


make should accept object and not just array as parameters

While an array is sufficient for certain simple instantiations, it is not good enough for a more complex approach, in which your "factory" method needs to take action upon certain elements given to it.
Consider the following example, in which endpoints need to set default values. The work-around appears okay - but would had been much easier if an object was accepted, instead of an array.
        this.ioc.bind('myEndpoint', (ioc, params = []) => {

            // Work-around - why is this needed ?
            if(params.length !== 0 && typeof params[0] === 'object'){
                params = params[0];
            } else {
                params = {};
            }

            // Id
            let id = 'SharedEndpoint';
            if(params['id'] !== undefined){
                id = params['id'];
            }

            // Transporter
            let transporter = null;
            if(params['transporter'] !== undefined){
                transporter = params['transporter'];
            }

            // Address
            let address = null;
            if(params['address'] !== undefined){
                address = params['address'];
            }

            // Additional data
            let data = {};
            if(params['data'] !== undefined){
                data = params['data'];
            }

            return new Endpoint(id, transporter, address, data);
        });

Possible solution
Change the supported type to be an array or object. The build method just needs to take this into account, when creating concrete instances.


Cake.Recipe do not work with cake 0.21.0+

Because of the change for the dotnet aliases that was added in cake version 0.21.0, Cake.Recipe do not work with the latest version of cake.
So far, the only thing I noticed that are incompatible is the Verbose value set for the calls to the .NET Core aliases.

@AdmiringWorm do you have time to submit a PR? I know @gep13 is quite busy with other stuff right now.

@mholo65 I'm about to sign off for tonight, but I can see if I can't get some time before or after work tomorrow.

not sure what would be the best option though, drop the setting of the Verbose parameter altogether to keep it compatible with users running a lower version of cake, or setting it to the equivalent of what it's now and force users to the latest cake version.

@gep13 any thougths on this? Latest and greatest?

@mholo65 for now, I am inclined to say that we remove the property that is causing the problem just now.
In a future version, we can look to controlling the verbosity, which would force users to upgrade to specific version of Cake.


Update README.md

Detection, response, and recovery are just as important as prevention.

Updates from our friend in security!


Enforce "Split chains of dotted method invocations" using roslyn

Justification
Writing an analyzer for - Splitting long chains of dotted method/property invocations
Implementation

Added the analyzer.
Wrote tests for the new analyzer.

Testing
The tests that I wrote pass successfully.
Note: I am writing a fixer for this convention as well. Will post that in a follow-up PR.

Had an offline chat with Jayson.
This convention here does not mention lambdas anywhere (looks like it needs to be updated); so, I did not consider the lambdas.
I am going to mark this PR as WIP and rework the analyzer logic. Will mark it ready for review once it's ready again.

@tsharma-ni  There is another PR that will go in very soon that cleans up the ruleset files: https://github.com/ni/csharp-styleguide/pull/35
The NI.ruleset doesn't list your rule yet because it doesn't exist. As a part of this PR, can you please pull down those changes when they go in and add NI1017 to it? The default value should be whatever you set in your analyzer, so probably warning.


Fix the version of twine to 2.0

Fix #21250

grpc_build_artifacts_multiplatform

@veblush Alternatively, we might consider dropping 3.8 support for manylinux1?

It affected all python versions on manylinux1 x86 because twine is used to upload artifacts to pypi.


MM-15383 Adding RELEASE.md to cover the chart-releaser updates

Ticket -> https://mattermost.atlassian.net/browse/MM-15383

Can you also update the README.md to use the new instructions? Right now it tells you to clone the repo to install

@jwilander Thanks for the review. README updated to cover the chart releases.


[FEATURE] Loading release note on demand on download section

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like.
Adding more information like the release note, download count and file size on demand perhaps when clicking on More link. The release note is in markdown we need to convert it to html.
Application Version
v1.2.0
Screenshots
No response
Additional information
No response

@redet-G Do you want to work on this issue?

Sure

Sure

Great @redet-G


Adding new rows while using filters doesn't filter new rows.

When appendRows is used while there is already column filters enabled, the newly added columns are added as-is, they're not filtered. I think the list must be both filtered and sorted whenever new rows are added. or at least make it optional to sort. but they must obey the filtering criteria.

When a row is added with appendRows, it is being appended after the existing row, regardless of the grid filter state.
This is my intention, and I agree to treat it as optional as you said.

@bosskmk Is there way to implement this with available methods? Could you share an example? I'd like the filer to be enforced for new rows as well, and I haven't found a viable solution yet.


Broken link in "Lazy Computations"

https://msdn.microsoft.com/en-us/visualfsharpdocs/conceptual/lazy-computations-[fsharp]
at the end

Looks like @ReedCopsey fixed that with his PR. Thank you both!


Add API v1 documents that support swagger

Adding API v1 documents that support swagger.
Fix #2155 #2156 #2157 #2158 #2159 #2160 #2161
Signed-off-by: Cherry Wang<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2329   +/-   ##
=======================================
  Coverage   37.11%   37.11%
=======================================
  Files         158      158
  Lines       12847    12847
=======================================
  Hits         4768     4768
  Misses       7817     7817
  Partials      262      262

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 e6e0a20...aad1ab4. Read the comment docs.

Have not reviewed content. Will do so once documents are relocated as requested below.
For historical purposes, documents should be placed in a directory that includes the entire semver -- i.e. major, minor, and patch release numbers. If these documents are representative of the RAML used for Fuji (which is what their internal version numbers indicate), they should be placed in api/openapi/v1.1.1.

@michaelestrin Changed the folder name from v1 to v1.1.1

Changed the folder name from v1 to v1.1.1

@cherrycl I made a related mistake and didn't update this issue.  In a separate PR, I versioned the v2 documents in the same way that I requested you to rename the v1 documents.  I had some subsequent discussion during yesterday's certification working group call that lead me to revert my change.  I didn't update this issue to reflect that and I apologize for creating additional work for you.
I will pull your branch, revert the v1 ->  v1.1.1 change, push it, and get a review going.

@AnthonyMBonafide please review for completeness and to ensure it reflects the current state of the v1 API contracts.

@AnthonyMBonafide Those files are converted from the existing RAML files, so the content are the same as what we have right now.
May we merge this PR first to confirm the API file location and open issues to refine each one?
Or, we just leave V1 in RAML and use Swagger for V2 only.

Accepted and merged per:


Remove public site

Description

Keeps the migrations to delete the public tables
Updates and adds a global footer (admin configurable)
Update the layout of homepage

Fixes #3110 #3559
Test Steps

[ ] Remove the public site, go to "Wagtail Admin" > "Pages", set the "Apply Home Page" as the default page of the site. Remove the public site page
[ ] Do regression of the whole site

I have merged in all "Remove public…" PRs before this one now so this can be rebased of main.

I merged in the icon PR so this can be rebased.
I also see this error:
WARNINGS:
?: (urls.W003) Your URL pattern 'account/two_factor/setup/complete/' [name='two_factor:setup_complete'] has a name including a ':'. Remove the colon, to avoid ambiguous namespace references.
?: (urls.W005) URL namespace 'apply:projects:reports' isn't unique. You may not be able to reverse all URLs in this namespace

So nice seeing all the delete file statements in this PR!

I merged in the icon PR so this can be rebased.
I also see this error:
WARNINGS:
?: (urls.W003) Your URL pattern 'account/two_factor/setup/complete/' [name='two_factor:setup_complete'] has a name including a ':'. Remove the colon, to avoid ambiguous namespace references.
?: (urls.W005) URL namespace 'apply:projects:reports' isn't unique. You may not be able to reverse all URLs in this namespace

These two warning doesn't seem to b/c of changes in the PR. If it make sense I'll update them in this PR or have them resolved in another PR.

Unable to get the failing test reproduced locally. Will try it tomorrow.

@theskumar really curious what your fix is for the tests. When I tried to fix the fund previews back in Oct. I kept getting the same issue. It seemed like the unit tests were dependent on my local database for whatever reason and when I would do a full reset of that database I was start to see them recreated locally.

Once this issue is cornered we also should  probably add unit test refactoring to the roadmap for this year. All the tests should run isolated and unaffected by the database (if that is the case)

@theskumar This fixed all issues for me: https://github.com/HyphaApp/hypha/commit/6de8fff45ca52392ccf24e2fe96c960aba19301a. Might just be the start to a larger issue but it seems like there's some objects that are being initialized too early in the API app. Seems like wrapping them in functions does the trick? Curious to hear your thoughts or if you have another solution already cooked up.

Django did had start up issues in very old versions before they introduced the apps.py config option with ready() function in that would be excecuted after all the model classes are loaded. So that could be it.

@wes-otf you saved me hours. The solutions seems to be working and accessing the models before its initialized is a very bad idea in general. The loading of models and apps are very unpredictable.
That why Django introduced https://docs.djangoproject.com/en/5.0/ref/applications/#django.apps.AppConfig.ready to you can perform any on_load activities here.
I believe there are more code in the forms.py in certain apps that is initialized like this, instead of in the __init__.py looks like the fixes things for now. But we can re-visit and cleap all module level queries.

This doc need updating at some point as well:
https://github.com/HyphaApp/hypha/blob/50693eb9d95865925b24028911cf37ad4269a529/docs/getting-started/contributing/code-contributions.md#L19-L34

Reversing URLS

Updated.

@theskumar glad it worked!! Was a bit of a shot in the dark, weird stuff.

Can it be that the initial migrations created a public and apply site and set the public as default. When testing against this we get errors.

Yes. You are absolutely right. updated the code in https://github.com/HyphaApp/hypha/pull/3738/commits/b788eec4c591313cf4fbbafcd889e2d2c2b68c4d
Re-created by creating a fresh db, ran migration, without adding the sandbox db data

Deploying this to test now.


Consistently use quotes in metadata 'homepage' field

Motivation for this change
Things done

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

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

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

See discussion in https://github.com/NixOS/nixpkgs/pull/83909, should probably be handled there?

See discussion in #83909, should probably be handled there?

Thanks for the link, I didn't find that before! Indeed this PR is a subset of that one, and since that one's already rolling I'll close this one.


[BugFix] avoid backpack:build to generate interfaces for abstract models and others

Fixes https://github.com/Laravel-Backpack/Generators/issues/120.
WHY
BEFORE - What was wrong? What was happening before this PR?
backpack:crud was generating CRUDs for abstract models, and it was also skipping valid models that simply don't extend directly Illuminate\Database\Eloquent\Model.
HOW
How did you achieve that, in technical terms?

First it tries to guess the namespace by the file path
Then it also tries to find the namespace by the file contents.

Is it a breaking change or non-breaking change?
No.
How can we test the before & after?

Having an abstract Model on Models folder (should not generate CRUD)
Having a model that extends another Model or an abstract Model (should generate CRUD)

@tabacitu I think I'll ask @pxpm to take a quick look at this one, he is aware of this kind of stuff, requiring files on runtime, guessing namespaces ... he may find something that I should not have done 😅

Yes please 🙏 Take it away @pxpm

By the way, @pxpm, note that;
https://github.com/Laravel-Backpack/Generators/issues/105 < this a different thing 😅
Later we may discuss that one.

Ping! What's the status of this @pxpm & @promatik - can I merge it, do I have the green light?
Cheers!

@pxpm I applied your previous suggestion, I'll assign it to @tabacitu 👌

Thanks @promatik - I just tested a use case that I thought would be covered by this PR, but looks like not? Here's what I did:

created a new model in app\Models\Paper.php that extends the current Page:

<?php

namespace App\Models;

use Backpack\PageManager\app\Models\Page as OriginalPage;
use Illuminate\Database\Eloquent\Factories\HasFactory;

class Paper extends OriginalPage
{
    use HasFactory;
}

Now after I run php artisan backpack:build I expected to see a Paper CRUD... right? That's the point of this PR? Well... it's not there... 👀 Am I missing something?

@tabacitu I tried that, created that file, run the build, everything worked 👌

Can you try again?
Maybe your model was not loaded, or something in the use's was not correct, because the condition to validate is;
$reflection->isSubclassOf(Model::class)

Hi there @promatik i create "Paper" model who extend "Page" model.

On main totally ignore this model, but it this branch "load-models-by-namespace", created the CRUD

And update the model

Look ready to merge for me.
Cheers.


fixes docker error on windows: standard_init_linux.go:178: exec

Proposed changes:

Adds .gitattributes file, fixing a docker error on windows by preventing Git from replacing CRLF with LF for text files

Status:

[x] ready for code review
[ ] there are tests for the functionality
[ ] documentation updated
[ ] changelog updated

Thanks 👍


Update index.ts allow to pass transport

Expose createClientWithTransport to be able to use window.fetch instead

 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.


No data in kiosk mode since January 1st

It seems that the entities became unavailable since January 1st. I've got a notification about some maintanance from FusionSolar around that time too, so maybe it's related.

My inverter is online, I can see the data in FusionSolar app as well as in the web browser. My kiosk url is just around 6 months old, but just in case I've tried adding it again and it says no devices or entities, even though the kiosk url is reachable and working.

How do I debug? There are no logs that I'm aware of.

Not sure it's related to the kiosk mode. All the entities became 'unavailable' for me as well on January 1st, 4am, local time (EET).

Ok, I've got some debug logs. It seems that something's changed in the API:
2024-01-02 19:20:28.507 DEBUG (MainThread) [custom_components.fusion_solar.sensor] Finished fetching FusionSolarKiosk data in 0.446 seconds (success: False) 2024-01-02 19:30:28.060 DEBUG (MainThread) [custom_components.fusion_solar.fusion_solar.kiosk.kiosk] calculated API base url for AkvanzlDiD3tMKm994aQ2DiosoIewnG9: https://region02eu5.fusionsolar.huawei.com 2024-01-02 19:30:28.060 DEBUG (MainThread) [custom_components.fusion_solar.sensor] fusion_solar 2024-01-02 19:30:28.061 DEBUG (MainThread) [custom_components.fusion_solar.sensor] AkvanzlDiD3tMKm994aQ2DiosoIewnG9 2024-01-02 19:30:28.488 ERROR (MainThread) [custom_components.fusion_solar.sensor] Unexpected error fetching FusionSolarKiosk data: Expecting value: line 1 column 1 (char 0) Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 300, in _async_refresh     self.data = await self._async_update_data()                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 256, in _async_update_data     return await self.update_method()            ^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/config/custom_components/fusion_solar/sensor.py", line 69, in async_update_kiosk_data     data[f'{DOMAIN}-{kiosk.id}'] = await hass.async_add_executor_job(api.getRealTimeKpi, kiosk.id)                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run     result = self.fn(*self.args, **self.kwargs)              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/config/custom_components/fusion_solar/fusion_solar/kiosk/kiosk_api.py", line 37, in getRealTimeKpi     jsonData[ATTR_DATA] = json.loads(html.unescape(jsonData[ATTR_DATA]))                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/json/__init__.py", line 346, in loads     return _default_decoder.decode(s)            ^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/json/decoder.py", line 337, in decode     obj, end = self.raw_decode(s, idx=_w(s, 0).end())                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/usr/local/lib/python3.11/json/decoder.py", line 355, in raw_decode     raise JSONDecodeError("Expecting value", s, err.value) from None json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

I have tested it with my personal kiosk url and this provides me with the correct data.
Can you check if the kiosk url is still working?
And if so: could you try to remove an add the integration again?
If that is not working could you mail (fusionsolar-ha [at] verkoyen [dot] eu) the kiosk url, so I can try to replicate the issue.

@tijsverkoyen I have the same error, and the Kiosk URL does seem to have been invalidated judging by the Huawei interface.
I'm apparently not smart enough to find the configuration where the URL is stored in HomeAssistant to update the existing URL.
I'd rather not delete and re-create it, as that would probably delete data as well, could you point me at how to update the URL?
It might be worth adding this description to this section as well.
Thanks for the extension though! It's been running flawlessly for a year! :)

@tijsverkoyen I have the same error, and the Kiosk URL does seem to have been invalidated judging by the Huawei interface.
I'm apparently not smart enough to find the configuration where the URL is stored in HomeAssistant to update the existing URL. I'd rather not delete and re-create it, as that would probably delete data as well, could you point me at how to update the URL?
It might be worth adding this description to this section as well.
Thanks for the extension though! It's been running flawlessly for a year! :)

The configuration can be found in the configuration.yaml file (in my case stored in the secrets.yaml file, but that is up to personal preference):

platform: fusion_solar_kiosk
kiosks:

url: !secret fusion_solar_url
name: "Solcellsanläggning"

The configuration can be found in the configuration.yaml file (in my case stored in the secrets.yaml file, but that is up to personal preference):
- platform: fusion_solar_kiosk
  kiosks:
  - url: !secret fusion_solar_url
    name: "Solcellsanläggning"

Mine wasn't there, it didn't have it in any of the other config files.
I tried adding it on a lark, and HA says fusion_solar_kiosk is an unknown platform, is that maybe another extension? :)
I tried changing it to just fusion_solar which let it load, but there is still more errors. I'll try and dive more into it.

Some updates from my end:

I tried adding it on a lark, and HA says fusion_solar_kiosk is an unknown platform, is that maybe another extension? :)
I tried changing it to just fusion_solar which let it load, but there is still more errors. I'll try and dive more into it.

Was because fusion_solar_kiosk is the old extension.
It doesn't seem like this one reads from configuration.yaml, instead it seems to read from /config/.storage/core.config_entries.
I updated the Kiosk URL there, and now it works again. :)

The issue still persists with valid kiosk url. I see two relevant error in the logs, based on these I also think the kiosk API was changed:
`Logger: homeassistant
Source: custom_components/fusion_solar/fusion_solar/power_entity.py:64
First occurred: 31 December 2023 at 12:45:54 (564 occurrences)
Last logged: 10:37:28
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 229, in _handle_refresh_interval
await self._async_refresh(log_failures=True, scheduled=True)
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 374, in _async_refresh
self.async_update_listeners()
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 164, in async_update_listeners
update_callback()
File "/config/custom_components/fusion_solar/fusion_solar/power_entity.py", line 64, in _handle_coordinator_update
if self._data_name not in self.coordinator.data:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: argument of type 'NoneType' is not iterable`
`Logger: custom_components.fusion_solar.sensor
Source: custom_components/fusion_solar/fusion_solar/kiosk/kiosk_api.py:37
First occurred: 31 December 2023 at 12:35:54 (564 occurrences)
Last logged: 10:37:28
Unexpected error fetching FusionSolarKiosk data: Expecting value: line 1 column 1 (char 0)
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 283, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 242, in _async_update_data
return await self.update_method()
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/fusion_solar/sensor.py", line 79, in async_update_kiosk_data
data[f'{DOMAIN}-{kiosk.id}'] = await hass.async_add_executor_job(api.getRealTimeKpi, kiosk.id)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/fusion_solar/fusion_solar/kiosk/kiosk_api.py", line 37, in getRealTimeKpi
jsonData[ATTR_DATA] = json.loads(html.unescape(jsonData[ATTR_DATA]))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/json/init.py", line 346, in loads
return _default_decoder.decode(s)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/json/decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/json/decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
`

If that is not working could you mail (fusionsolar-ha [at] verkoyen [dot] eu) the kiosk url, so I can try to replicate the issue.

Right after midnight it started working again... I didn't update anything in my HA, so maybe it was really a kiosk API change?

@silfax

Not sure it's related to the kiosk mode. All the entities became 'unavailable' for me as well on January 1st, 4am, local time (EET).

Are you using Kiosk-mode? As I interpret your reply as you are not using kiosk mode. If so, there is another issue: https://github.com/tijsverkoyen/HomeAssistant-FusionSolar/issues/126

The configuration can be found in the configuration.yaml file (in my case stored in the secrets.yaml file, but that is up to personal preference):

platform: fusion_solar_kiosk
kiosks:

url: !secret fusion_solar_url
name: "Solcellsanläggning"

This is not correct!

It might be worth adding this description to this section as well.

I have added a separate entry in the FAQ: I need to change the Kiosk URL

Right after midnight it started working again... I didn't update anything in my HA, so maybe it was really a kiosk API change?

@enorkus, Can you check if the issue is resolved? If not: can you mail me your kiosk url?

I have not changed any configuration and it just started working today. I've also got another maintenance message from FusionSolar, so I'm guessing they had messed something up with the API previously and now fixed it. My kiosk url was accessible and working this whole time. Hopefully it doesn't happen again.
Thanks for your time - very useful integration :)


panic when using connect-to in client

quiche panics when trying to use the connect-to value in the client
https://github.com/cloudflare/quiche/blob/master/tools/apps/src/args.rs#L289-L293
env RUST_BACKTRACE=1 /home/username/Library/rust/quiche/tools/apps/target/x86_64-unknown-linux-musl/release/quiche-client --connect-to='www.google.com' https://www.google.com/
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: AddrParseError(())', /home/username/Library/rust/quiche/tools/apps/src/client.rs:68:22
stack backtrace:
   0: rust_begin_unwind
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/std/src/panicking.rs:517:5
   1: core::panicking::panic_fmt
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/core/src/panicking.rs:101:14
   2: core::result::unwrap_failed
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/core/src/result.rs:1617:5
   3: core::result::Result<T,E>::unwrap
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/core/src/result.rs:1299:23
   4: quiche_apps::client::connect
             at ./Library/rust/quiche/tools/apps/src/client.rs:68:9
   5: quiche_client::main
             at ./Library/rust/quiche/tools/apps/src/bin/quiche-client.rs:43:11
   6: core::ops::function::FnOnce::call_once
             at /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35/library/core/src/ops/function.rs:227:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Doesn't seem to matter what the value of connect-to is. Any ideas?

this argument takes an IP address not a hostname.


./satosa/backend.py:834:        # TODO: evaluate with UX designers if Jinja2 template

no more work to do, we close the task because not usefull anymore


Get ciphers

adds getCipher (which was erroneously put in as listCiphers) built on top of #95 to prevent merge conflicts, fixes #91

ACK

ACK
Fixes #91.

Ack?

Acknowledge, LGTM
On 5 Jan 2015 11:01 am, "Calvin Metcalf"<EMAIL_ADDRESS>wrote:

Ack?
—
Reply to this email directly or view it on GitHub
https://github.com/dominictarr/crypto-browserify/pull/96#issuecomment-68655918
.


[MRG] Add a oktest creator preprocessor

Add a preprocessor that can take a cell and create a "oktest" from that cell, replacing the cell content with check(path_to_test).
Experimental to the max.
This contribution is financed by the Earth data science lab at UC Boulder

Ready for review&merge.
The overall idea is to make authoring autograding tests easier. The proposed workflow here is to author a "instructor" notebook that contains cells which are tagged as containing tests for the autograder. The new create_oktests method will take the code contents of each of those cells, place it into a test file, and replace the contents of the cell with check('path/to/test.py'). Currently we assume a suitable check function has been imported at the start of the notebook already or will be injected by the autograder.

I plan on using this with a snippet like this:
def main():
    notebook = sys.argv[1]
    print('Processing', notebook)

    directory, nb_name = os.path.split(notebook)
    base_name, extension = os.path.splitext(nb_name)

    # create test files and notebook for the student
    nb = NotebookCleaner(notebook)
    # need to rewrite these cells as well, store results in
    # autograder directory as the students shouldn't be
    # able to see them
    nb.create_oktests(tag='private',
                      oktest_path=base_name,
                      base_dir='autograder')
    nb.create_oktests(tag='public',
                      oktest_path=base_name,
                      base_dir='student')
    nb.save(os.path.join('student', nb_name))

    # create test files for the autograder
    nb = NotebookCleaner(notebook)
    nb.create_oktests(tag='private',
                      oktest_path=base_name,
                      base_dir='autograder')
    nb.create_oktests(tag='public',
                      oktest_path=base_name,
                      base_dir='autograder')

Takes a master notebook with tests tagged as either private or public. Public tests will be available to the student to run locally and stored in the student subdirectory together with the re-rwitten notebook.
The autograder will have access to both private and public tests. These are stored in the autograder subdirectory. No notebook is stored there as that will come when a student submits their work. The autograder will then execute the student's notebook in the autograder directory and the check function can pickup the private tests as well.

this is a nifty workflow :-)
lemme think through the details a little bit and then I'll give the PR a review. We should think about how to document these workflows for people.

We should think about how to document these workflows for people.

It will be part of the "not-yet-named" package of creating and grading work. This is the current playground: https://github.com/earthlab/grading-workflow-experiments

the code looks good to me - one thing I'd like to flag is the hard-dependency we're creating on oktests. Why not call this function create_tests instead of create_oktests and then in the future we don't have to deprecate a function if we want to support some other format than the wonky >>> okpy structure? (e.g. you could have create_tests(..., kind='okpy') and create_tests(..., kind='markdown') or whatever

Good point. I renamed it to create_tests. Instead of adding an argument that doesn't do anything I put a comment in the docstring mentioning oktests. I'd hold off making the kind argument until it can do something. WDYT?

yep that's exactly what I was thinking - no need to add a kwarg now!

and thanks!!! 🎉  🎉  🎉  🎉


Preprocess failed for the WMT'14

This problem puzzled me a day.
I wanted to retrain the pre-trained model after pruning the model by prune.py.
I downloaded the parallel data set of English<->German from http://www.statmt.org/wmt14/translation-task.html, which are
commoncrawl.de-en.en europarl-v7.de-en.en news-commentary-v10.de-en.en commoncrawl.de-en.de europarl-v7.de-en.de news-commentary-v10.de-en.de
Then I concatenated commoncrawl.de-en.en, europarl-v7.de-en.en, news-commentary-v10.de-en.en together as train.en, and commoncrawl.de-en.de, europarl-v7.de-en.de,  news-commentary-v10.de-en.de together as train.de.  So that I can prepare the *.hdf5 files for training.  The train.en contains 4535522 sentences, so as train.de.
But than I met the error:
`
$> python preprocess.py --srcfile /home/data/wmt/wmt15-de-en/train.en --targetfile /home/data/wmt/wmt15-de-en/train.de --srcvalfile data/src-val.txt --targetvalfile data/targ-val.txt --outputfile data/wmt-deen
First pass through data to get vocab...
Number of sentences in training: 4294257
Number of sentences in valid: 2819
Number of additional features on source side: 1
* source feature 1 of size: 5
Source vocab size: Original = 1718846, Pruned = 50004
Target vocab size: Original = 2913125, Pruned = 50004
Traceback (most recent call last):
File "preprocess.py", line 524, in
sys.exit(main(sys.argv[1:]))
File "preprocess.py", line 521, in main
get_data(args)
File "preprocess.py", line 456, in get_data
max_word_l, max_sent_l, args.chars, args.unkfilter, args.shuffle)
File "preprocess.py", line 282, in convert
sources_features[i][sent_id] = np.array(source_features[i], dtype=int)
TypeError: 'NoneType' object has no attribute '__getitem__'
`
Did I do the right process to generate the training set of WMT' 14? Or is there any uploaded *hdf5 files for the pre-trained model? Further more, if I generate the hdf5 files successfully, can I use them to retrain the pre-trained model directly, since the contents of *.dict for the pre-trained model may be different from those in the new generated *.dict by preprocess.py?

Could you check that your data do not contain the word features separator?
https://github.com/harvardnlp/seq2seq-attn#using-additional-input-features
For the second question, you can't retrain a model with a different vocabulary.

@guillaumekln Yes, There are some feature separaor "|" in the data. What should I do to them? Should I directly delete them?

The separator is the sequence -|-. You should check those instead and remove or replace them.

@guillaumekln It works! I am now training the model by the processed data. Thank you very much.


500 internal server error when installing plugin

Hi there,
I'm getting a 500 internal server error when installing plugin. It seems that it has something to do with the composer packages but I haven't discovered the exact culprit.
I tried running composer update in the plugin folder but that did not resolve the problem either.

This is is resolved by updating the composer.json sentry dependency to 0.8.0.

Great @CptMeatball
Thanks a lot, i'll be pushing to October marketplace


Large table overflowing (hidden by right sidebar) without horizontal scrollbar

Large tables can overflow, but without visual scrollbar to indicate this and help scrolling. This is noticeable in our own example from the kitchen sink (https://pydata-sphinx-theme.readthedocs.io/en/latest/examples/kitchen-sink/tables.html#giant-tables):

(it seems it is already like this in several releases, I noticed that in 0.9 it actually just didn't hide the overflow: https://pydata-sphinx-theme.readthedocs.io/en/v0.9.0/demo/kitchen-sink/lists-and-tables.html#giant-tables)
From quickly playing with the browser dev tools, I noticed that changing the display to frim table to block made it add a horizontal scrollbar:
https://github.com/pydata/pydata-sphinx-theme/blob/48e4c012e070b2e5b2971eba3bc750d00fb8ba2c/src/pydata_sphinx_theme/assets/styles/content/_tables.scss#L5-L10

In some browsers, changing the display value of a table element to block, grid, or flex will alter its representation in the accessibility tree. This will cause the table to no longer be announced properly by screen reading technology.
‐ MDN - display - Table

This issue with display: table and the accessibility tree may or may not be true anymore on newer browsers. But the changing the display property has other consequences as well.
I think the most straightforward solution is to wrap the tables in a scrollable div, which is what I do in #1827.


Lollipop bounded zoom functionality

Thinking of implementing something like this: http://blog.scottlogic.com/2014/09/19/interactive.html
Seems to be a very robust chart as it looks like it works well both in touch and mouse input formats.

That looks beautiful!
On Mon, Dec 14, 2015 at 5:08 PM, Charles Markello<EMAIL_ADDRESS>wrote:

Thinking of implementing something like this:
http://blog.scottlogic.com/2014/09/19/interactive.html
Seems to be a very robust chart as it looks like it works well both in
touch and mouse input formats.
—
Reply to this email directly or view it on GitHub
https://github.com/BD2KGenomics/brca-website/issues/91#issuecomment-164609508
.


Remove padding for iframe container

This PR:

Closes https://github.com/gnosis/safe-react/issues/1580 by removing padding and letting developers define their own

Don't merge because I'm fixing the apps

Travis automatic deployment:
https://pr1937--safereact.review.gnosisdev.com/rinkeby/app

Travis automatic deployment:
https://pr1937--safereact.review.gnosisdev.com/rinkeby/app


Make Implementation#dup also dup @files

The default dup or clone behaviour is to make a shallow copy.
This provides a deep copy of the files instance variable as well so when they are modified
by merge_files the original will remain unchanged.
Thanks to @petertseng for bringing this to my attention and doing some detective work as to the cause.

Fixes: https://github.com/exercism/exercism.io/issues/3519

A better solution is to remove the ability to mutate files completely.
See: https://github.com/exercism/x-api/issues/155


Don't show QR code right away on dialog before fiat value retrieved if applicable

You might need to try it on a slower device. It's quite noticeable on my phone.


About Ego4d dataset

Hi there,
Thank you for your excellent work! I would like to inquire whether you used the full Ego4D dataset for training, or did you employ extracted features from the Ego4D dataset instead?
Best

Hi @lwpyh ,
We use the full Ego4D dataset (videos) for training.
Ego4D has its own license and we suggest following the steps in https://ego4d-data.org/#download to sign the license form and obtain the data. Once the raw videos are downloaded, you may follow the pre-processing steps in https://github.com/facebookresearch/LaViLa/tree/main/datasets#ego4d. The videos after pre-processing are significantly smaller.
Also, we would recommend the practices discussed in our followup work for faster training.
Best,
Yue

Thanks for the quickly reply, I will read it carefully.
Best


axis line color is wrong in legend

Fix this issue by adding:
this.color = color;

To this constructor:
https://github.com/swharden/ScottPlot/blob/c417e71b88b536d4127ea44aa47f70f943b9668d/src/ScottPlot/Plottables.cs#L467-L476

ca2d3553db5714b8ea7c39f873507768d019516c


Support traversal of git folders below the castle root

As discussed in #93 it can be very useful at times to fiddle with .git repos by using homeshick. To do that homeshick needs to stop skipping files and folders named .git when linking.
However, some things still need to be skipped:

The .git folder in the root of a castle, since that is a folder managed by git and not the user
The .git files in submodules of a repo
The .git folders in submodules of a repo for older git versions (if I remember correctly git switched to just managing all submodules in the .git folder of the root repository, so we need the git version where that changed)

If I understand this correctly, this feature will enable homeshick to track git project folders as well? That sounds great.
Personally I have 2 git working dirs in ~/.tmux/plugins/, which will get ignored when I start to homeshick track ~/.tmux/.
So if the main issue seems git's inability to track git working dirs inside git working dirs, then isn't this a solution? - http://stackoverflow.com/questions/4659549/maintain-git-repo-inside-another-git-repo
Especially that part about 'fake submodules' appears to be interesting.


Max HP is undefined in pf2e

Using the pf2e system the max hp of a character after character creation is showing as "undefined" but before character creation it works just fine.

This is caused by when a character wears armor, for some reason

the default setting uses system.attributes.hp.max as max hp path from the actor object, you could try changing that to hitPoints.max (pf2e specific), if it still borks idk, you could maybe see with the console command game.actors.filter(a => a.name === "Rek")[0] if you can figure out where the max hp is saved


MAINT: Update nbqa hook and github workflow

We should be able to directly run the pre-commit on .md notebooks, no need for the weird conversion.

Self-merging, hopefully shouldn't break anything.


Exclude new.min.css and lite.css from source

new.min.css and lite.css are auto-generated based on new.css so they should not be part of the source but provided as artifacts via releases.

Good point. In your opinion, would be the best way to accomplish this? Instead of using https://newcss.net/new.min.css, should a version-based URL be used, such as https://newcss.net/dist/1.0.0/new.min.css?

My initial thoughts were about the code repo, not about delivery.
Files which can be built from the source should not be maintained within in the source.
As we can already see there have been merge conflicts caused by the minimized css.
But also for delivery, it's typical to provide version-specific urls. Examples:

https://stackpath.bootstrapcdn.com/bootstrap/4.3.1/css/bootstrap.min.css
https://code.jquery.com/jquery-3.5.1.min.js

:+1: Was fixed in 1.1.0 (https://github.com/xz/new.css/commit/788be93b4a66c50f11f05f73d1db094159058046).
jsDelivr automatically auto-minifies the new.css file, so just pushing an update to npm makes it available.
lite.css is now redundant as fonts are no longer imported by default. The old CDN URLs on the newcss.net domain will stay the same, this change is only applicable for versions 1.1.0+.


Fixed problem with hidden files

TFColorWidget is throwing an error condition into MyBase when it tries to read hidden files in our share/palettes directory.

I think the error being logged is caused by a non-.tf3 file being read, for example a .DS_Store, not by a hidden file per se.


Fix bug when import large (>1GB) relations

Resolves #2097

use timeline modification's last_flush_lsn and timeline's last_record_lsn to determine the corresponding LSN to query data in DatadirModification::get
update test_import_from_pageserver to use a large number of rows (when in release build) to simulate the scenario described in the linked issue

update test_import_from_pageserver to use a large number of rows (when in release build) to simulate the scenario described in the linked issue

Can use BUILD_TYPE=release NEON_BIN=(pwd)/target/release ./scripts/pytest -k test_import_from_pageserver to run the test to confirm that it fails without this patch


NIFI-4642 updated tests to be more tolerant/variable to different sys…

…tem speeds.  Many of these should be integration tests and not unit tests
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

All looks good. +1 merged to master.


Set default output format

Is there a way to set --output table by default for the user or is json going to stay as the default format?

This is an in-progress feature being tracked under issue #657.


Microsoft Excel Multiple Signing, Document Versioning and Document Integrity

Question
I am using office 365 and found a scenario I think office 365 should be able to solve.
Consider I am an officer in a business and I have to issue out specific tools to at least 4 employees. I need them to accept receipt of the tools and digitally sign against an excel office 365 document I created online for future reference. I want to ensure document integrity once they sign until the day they have to handover the tools for clearance. Not having found a better solution online thought I should ask the developers on this platform.
My thoughts were that a versioning system like GitHub will help to track any changes as a solution or a digital blockchain solution would easily solve the problem.
I am looking forward to a solution that Bill Gates will definitely use. Thanks all.

Submission Guidelines
Thanks for your contribution! Sharing is caring.

@clarenznet - This is an interesting set of requirements, but it's pretty specific to your needs. I'd suggest posting in the Microsoft Tech Community, SharePoint General hub.
In this repo, we're trying to generate community-driven documents that help solve broad problems. I'm not saying your concept wouldn't be an excellent use case to document once you have it, but you won't get enough traction here to design that solution.


[AMBARI-23254] Mpack should have both displayName and description

Add displayName to mpacks API and registry API
(Please fill in changes proposed in this fix)
How was this patch tested?
GET http://{ambari_server}:8080/api/v1/registries/1/mpacks/
GET http://{{ambari_server}}:8080/api/v1/mpacks/
GET http://{{ambari_server}}:8080/api/v1/mpacks/
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1177/
Test FAILed.
Test FAILured.


Version 1.1.2 throws errors

After updating to 1.1.2 i get:
Failed to record analytics  [UpstashError: WRONGTYPE Operation against a key holding the wrong kind of value, command was: ["zincrby","mw:events:1713535200000",1,"{"identifier":"<IP_ADDRESS>","success":true}"]] {
name: 'UpstashError'
}
Downgrading makes the error go away.
I run analytics inside my middleware
const ratelimit = new Ratelimit({
  redis: redis,
  analytics: true,
  limiter: Ratelimit.slidingWindow(10, '10 s'),
  prefix: 'mw'
});

` if (request.method === 'POST') {
const ip = request.ip ?? '<IP_ADDRESS>';
const { success, limit, reset, remaining } = await ratelimit.limit(ip);
if (!success) {
  // Construct a response for blocked requests
  const blockedResponse = new Response('Too Many Requests', {
    status: 429, // HTTP status code for Too Many Requests
    headers: {
      'X-RateLimit-Limit': limit.toString(),
      'X-RateLimit-Remaining': remaining.toString(),
      'X-RateLimit-Reset': reset.toString(),
      'Content-Type': 'text/plain'
    }
  });

  return blockedResponse;
}

}`

Hi @ElectricCodeGuy,
This can happen the first hour you upgraded your version. This is because in analytics, we had a hashmap for every hour which will now be a sorted list after version 1.1.2
This means that when you upgrade to version 1.1.2, you won't be able to submit analytics for the first hour. Then you will be able to submit.

we are updating the error message to explain the transition to the new version in https://github.com/upstash/ratelimit/pull/103

Thanks for response. Yes totally fine :) You can close this!


App_Data flag isn't respected for Azure App Service Deployment:ARM

I'm using an Azure App Service Deployment:ARM task to deploy to an Azure web app. My build artifact is a zip file containing a web app and a few webjobs. It looks like this:

and the API.zip file's contents look like this:

In the deployment task, I have the flag to exclude the App_Data folder unchecked.

However, when my release runs, the App_Data folder never gets deployed along with the web app...

I apologize. This isn't a bug. I had the skip artifact download checkbox turned on in the release definition which I think caused this issue.


Verify topcount works in PEG

Description of change

Part of the solution for https://github.com/lanetix/issues/issues/10936. Verifies I didn't have to do anything for topcount before going into mapper.
Checklist

[x] npm test passes
[x] Code coverage has not been reduced in a meaningful way
[ ] Code has been reviewed by at least one person other than the submitter
[x] Relevant documentation has been updated or added

LGTM. More passing tests!


BUGFIX: Fix bugs documented in #242 (deployable without conflict to develop also, see #254)

This PR fixes some severe bugs. Service manager does not process preconfiguration provided by child classes. Please see #242 for details.
I'd highly appreciate to get some qualified feedback on this. Same for #254.
All it would take to invalidate all the work done and time spent here was to merge something else upfront, as happened before by merging replace-this-with-self-for-no-particular-reason-and-no-measurable-benefit before this one. I would not consider such proceeding as being serious and responsible.

Same resolution as https://github.com/zendframework/zend-servicemanager/pull/254#issuecomment-364026562


Add better network detection

Summary
This PR helps improve network detection by trying to connect to the client server to test the connection before reporting an online/offline state.

@jarredwitt this one is failiing

This appears to be causing issues with the unit tests
1) Actions.Views.Login "before each" hook: _callee for "handleLoginIdChanged":
     TypeError: Cannot read property 'isConnected' of undefined
      at networkConnectionListener (app/utils/network.js:34:2)
      at Object.detectNetwork (app/store/index.js:199:2)
      at configureServiceStore (node_modules/mattermost-redux/store/configureStore.dev.js:91:27)
      at configureAppStore (app/store/index.js:199:2)
      at Object._callee$ (test/app/actions/login.test.js:14:17)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at Generator.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
      at Generator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at invoke (node_modules/regenerator-runtime/runtime.js:155:20)
      at node_modules/regenerator-runtime/runtime.js:202:11
      at callInvokeWithMethodAndArg (node_modules/regenerator-runtime/runtime.js:201:16)
      at AsyncIterator.enqueue (node_modules/regenerator-runtime/runtime.js:224:13)
      at AsyncIterator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at Object.runtime.async (node_modules/regenerator-runtime/runtime.js:248:14)
      at Context._callee (test/app/actions/login.test.js:13:16)

  2) Actions.Views.SelectServer handleServerUrlChanged:
     TypeError: Cannot read property 'isConnected' of undefined
      at networkConnectionListener (app/utils/network.js:34:2)
      at Object.detectNetwork (app/store/index.js:199:2)
      at configureServiceStore (node_modules/mattermost-redux/store/configureStore.dev.js:91:27)
      at configureAppStore (app/store/index.js:199:2)
      at Object._callee$ (test/app/actions/select_server.test.js:13:23)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at Generator.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
      at Generator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at invoke (node_modules/regenerator-runtime/runtime.js:155:20)
      at node_modules/regenerator-runtime/runtime.js:202:11
      at callInvokeWithMethodAndArg (node_modules/regenerator-runtime/runtime.js:201:16)
      at AsyncIterator.enqueue (node_modules/regenerator-runtime/runtime.js:224:13)
      at AsyncIterator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at Object.runtime.async (node_modules/regenerator-runtime/runtime.js:248:14)
      at Context._callee (test/app/actions/select_server.test.js:12:34)

  3) Actions.Views.Thread handleCommentDraftChanged:
     TypeError: Cannot read property 'isConnected' of undefined
      at networkConnectionListener (app/utils/network.js:34:2)
      at Object.detectNetwork (app/store/index.js:199:2)
      at configureServiceStore (node_modules/mattermost-redux/store/configureStore.dev.js:91:27)
      at configureAppStore (app/store/index.js:199:2)
      at Object._callee$ (test/app/actions/thread.test.js:13:23)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at Generator.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
      at Generator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
      at invoke (node_modules/regenerator-runtime/runtime.js:155:20)
      at node_modules/regenerator-runtime/runtime.js:202:11
      at callInvokeWithMethodAndArg (node_modules/regenerator-runtime/runtime.js:201:16)
      at AsyncIterator.enqueue (node_modules/regenerator-runtime/runtime.js:224:13)
      at AsyncIterator.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
      at Object.runtime.async (node_modules/regenerator-runtime/runtime.js:248:14)
      at Context._callee (test/app/actions/thread.test.js:12:37)

@enahum @hmhealey tests are passing now, just needed to mock NetInfo


Support Framebuffer version 2, as seen on Android 8.1

This adds a new ColorSpace field to the middle of the struct.

/cc @bartsaintgermain


Deployed site gives 404

With the username change (love it), https://fabulousgk.github.io/fabulous-styleboard/ is no longer valid and returns a 404. It's used as an additional resource in TOP.

Ahh crud, woops

@MaoShizhong it is fixed, I created an org using the old username and transferred the repo to it.


Update terminology for blocked CIDR range

Merge after https://github.com/influxdata/kapacitor/pull/2649.

[ ] Tests pass (no build errors)
[ ] Rebased/mergeable

If the changes suggested in https://github.com/influxdata/kapacitor/pull/2649 are ever made, we'll want to update the docs with changes like the ones here. But we can close this for now.


Supervised or Unsupervised?

Hi, may I ask whether this work uses a supervised training?

@Beniko95J in the current version is supervised, but since it is modular you can define your own loss, and make it unsupervised.


Update tests to solc v0.8.22 and fix a few tests

Do not modify ns.diagnostics directly - ever. We may want to resolve an expression and drop all the diagnostics for it.
Ensure all variable errors are surfaced
Update solc testdata to v0.8.22

I am not sure why the coverage bot is complaining. I get an overall failure but in all the files the coverage has gone up or stayed the same.

I am not sure why the coverage bot is complaining. I get an overall failure but in all the files the coverage has gone up or stayed the same.

In src/sema/tags.rs, lines 101-104 were previously covered by test, but are not covered anymore. Please, check the indirect changes in the coverage report.

In src/sema/tags.rs, lines 101-104 were previously covered by test, but are not covered anymore. Please, check the indirect changes in the coverage report.

Good point. I've added a bunch of test and now the coverage does not decrease (in fact, it increases).
There are some areas which are still uncovered, but I have not decided how to fix that. For example for the dot tests I think we need a solidity file with all the syntax in it (or just a minimal set of files). That's not in this PR.


Cannot find module 'reduce-function-call'

I got this error when trying to install the starter template. I was able to resolve it by installing this dependency manually but it should probably be included in nuxt.
ERROR in ./~/css-loader!./~/vue-loader/lib/style-rewriter.js?id=data-v-b155f86e&scoped=true!./~/vue-loader/lib/selector.js?type=styles&index=0!./pages/index.vue
Module build failed: Error: Cannot find module 'reduce-function-call'
at Function.Module._resolveFilename (module.js:470:15)
at Function.Module._load (module.js:418:25)
at Module.require (module.js:498:17)
at require (internal/module.js:20:19)
at Object. (C:\dev\nuxt\node_modules\reduce-css-calc\index.js:5:26)
at Module._compile (module.js:571:32)
at Object.Module._extensions..js (module.js:580:10)
at Module.load (module.js:488:32)
at tryModuleLoad (module.js:447:12)
at Function.Module._load (module.js:439:3)
@ ./~/vue-style-loader!./~/css-loader!./~/vue-loader/lib/style-rewriter.js?id=data-v-b155f86e&scoped=true!./~/vue-loader/lib/selector.js?type=styles&index=0!./pages/index.vue 4:14-223
@ ./pages/index.vue
@ ./.nuxt/router.js
@ ./.nuxt/index.js
@ ./.nuxt/server.js

What's your version of node.js and npm please?

node: 7.5.0
npm: 4.1.2

I just tried with:
$node -v
v7.3.0
$npm -v
4.2.0

And it works fine, can you update npm with npm install -g npm @abelovic and try again?

@Atinux - Just tried your suggestion and I can no longer reproduce this error. Must have been something with my environment :/ Sorry for the inconvenience


brand-specific interface tests with routes

To catch merging a new CAN FD Hyundai port that doesn't have the message we override and send to disable LFA.
The new Ioniq 6 for example doesn't have the message, and not if not caught, the car could have been merged with LFA overriding lane changes, etc.

I didn't realize we forward CAM_0x2a4 which catches this, not needed for now!


entity-component system

objects that can have specialized lua code running on them based on freely addable components

there is some potential benefit to using ecs for the ui widgets as well
consider a hierarchical relation such as Widget→ScalableWidget→Console. this is fine by itself, but what happens when we want to encode a property other than "scalable", such as "draggable"? suddenly we are no longer able to use a strict hierarchy and it seems to no longer make sense to place "scalable" into a hierarchical relationship. even the relationship Widget→Console should be examined

there is some potential benefit to using ecs for the ui widgets as well
consider a hierarchical relation such as Widget→ScalableWidget→Console. this is fine by itself, but what happens when we want to encode a property other than "scalable", such as "draggable"? suddenly we are no longer able to use a strict hierarchy and it seems to no longer make sense to place "scalable" into a hierarchical relationship. even the relationship Widget→Console should be examined

instead of a full ecs we are going to use mixins for the ui classes instead. the ecs we end up implementing can be reserved for in-game entities only, in order to simplify some assumptions


:embed_sign vs. IdP SingleSignOnService Binding (HTTP Method)

Correct me if I'm wrong, but it appears the :embed_sign parameter is redundant with the IdP URL Binding. In other words:

if the Binding is urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST, then you MUST have embed_sign: true
else if the Binding is urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect, then you MUST have embed_sign: false

(Otherwise, your requests won't be signed, e.g. a POST with embed_sign: false will simply have no signature.)
Assuming this is the case, since embed_sign is a single value, it's not possible to mix SSO POST with SLO Redirect. Although not common in practice, it is theoretically possible that an IdP could require it that way in their metadata.
If this is the case, I wonder if there might be a better way to do this. For example, we could get rid of :embed_sign and introduce Settings.idp_sso_binding and Settings.idp_slo_binding parameters which would allow you to control which bindings you want to use for each endpoint.

'embed_sign' was an original parameter from the original ruby-saml gem.
If you review rest of OneLogin SAML toolkits, there are no 'embed_sign' there
I agree that supporting settings.idp_sso_binding and settings.idp_slo_binding would be better than the current 'embed_sign', but not sure if its implementation wont carry extra headache since we will need to keep supporting 'embed_sign' as well, until deprecate it.
`` Assuming this is the case, since embed_sign is a single value, it's not possible to mix SSO POST with SLO Redirect.```
Not really. You can modify the embed_sign parameter on the setting before initializing the logout_request/logout_response, based on idp data stored in a database setting or so, if you support multiple IdPs, or directly coded in the SSO and SLS service of the app if only supporting 1 IdP

I will raise a PR which does the following:

Add settings.idp_sso_binding and settings.idp_slo_binding with values :post and :redirect
embed_sign will be deprecated and will set the above two params if they are not explicitly set (the newly added params will take priority.)

Sound good?

Sound good, but rather than :post, :redirect
Better to put the real values:

https://github.com/onelogin/php-saml/blob/master/settings_example.php#L31
https://github.com/onelogin/php-saml/blob/master/settings_example.php#L57

Can we make the symbols :post/:redirect as aliases for the real values? It will be a lot more ruby-like.

Its ok to receive :post :redirect or a realvalue...but we should store the real value

PR raised


Report errors to sentry

This uses sentry's flask integration for reporting errors to
sentry. THis is a good first start, though we may need more reporting
in case an error doesn't happen during handling of a HTTP request or
we manually return an error code.
See https://github.com/trustlines-protocol/relay/issues/92

I'll just keep the original issue open.


Provide configuration to handle non-public repositories

🗣 Description
This pull request adds controls to manage interactions with non-public repositories. These controls manage whether or not non-public repositories should be processed and if non-public repositories are processed if their names should be masked in the GitHub Actions log output.

💭 Motivation and context
Blindly processing non-public repositories can lead to unintended information leakage. This also mirrors functionality added in https://github.com/cisagov/action-lineage/pull/38.

🧪 Testing
Automated tests pass.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All new and existing tests pass.

FYI, none of the checks appear to be required in this repo.


country and originaltitle for TV series as well?

Version: 2.9.13
Build: 2018-07-23 18:42
OS: Mac OS X 10.13.1
JDK: 1.8.0_161 x86_64 Oracle Corporation
What is the actual behaviour?
Though often present for TV as well, country and originaltitle are only scraped for movies ATM.
What is the expected behaviour?
Perhaps not a high priority, but it would be terrific if these fields could make it into the .nfo for TV. IMDb in particular has a standard of using the English title as the primary, though even most English speakers often don't know the work by that title...so while IMDb is arguably still the "reference standard," having the originaltitle in the metadata is extremely useful for searching purposes.
And country is just nice to have in general, regardless of medium.
Steps to reproduce:
Additional
Have you attached the logfile from the day it happened?

original title for shows and episodes is already done in v3, but for now we have barely sources for that data..
I hope you understand that imdb is our lowest prio for tv shows because that site is awful for scraping (no official API, endless site changes which needs work on our scrapers, inperformant site, ..), but I will add this to our v3 todo list

Thanks. I definitely understand the frustrations with IMDb. I also realized that their legacy flat files (the final versions with integrated IDs) are actually a pretty good fallback for some of this information, so for my application I'm planning to leverage their title.basics.tsv file for primary and original titles whenever I have an IMDb ID to work with. Of course, it's no longer being updated post-2017, but it's not bad in a pinch.

I've added both in v3
the TV show country is only available from TMDB and IMDB


SED spec updates

This PR makes a few minor changes to the SED spec:

Drops ph unit everywhere as requested in #81.
Clarifies that e_min and e_max can apply to differential representations (see #86).
Cleanup example files and add python script for regenerating these files automatically.

Looks good to me.


Added --multi option to infer operation to show a list of faces detected

What does this PR do?
A new option added to the infer operation called --multi. By default, infer prints the largest detected face. With --multi, a list of faces in the image are displayed.
Where should the reviewer start?
All changes are confined to the demo script.
How should this PR be tested?
Run the demo on an image with multiple face with infer --multi.
Any background context you want to provide?
None.
What are the relevant issues?
None.
[You can link directly to issues by entering # then the
number of the issue, for example, #3 links to issue 3]
Screenshots (if appropriate)
None.
Questions:

Do the docs need to be updated?
The help message explains the newly added option.
Does this PR add new (Python) dependencies?
No

Thanks @nma83 - this looks great!


Introduce toml standard library module

Description:

Need to convert[1] into a standard library module adhering to the standard library development process[2]
[1] https://github.com/ballerina-platform/module-ballerina-toml
[2] https://github.com/ballerina-platform/ballerina-standard-library/blob/main/docs/library-development-process.md

Closing since completed.


How to push next modal sheet without prior (enclosing) one? [GoRouter]

Discussed in https://github.com/fujidaiti/smooth_sheets/discussions/108

Originally posted by bqubique May  4, 2024
Based on this code block, if I want to directly go to Genre sheet, how would I go for it using GoRouter?
Normally I would expect this to work:
context.go('/intro/genre');

But I'm getting this error:
════════ Exception caught by rendering library ═════════════════════════════════
The following assertion was thrown during performLayout():
The sheet extent and the dimensions values must be finalized during the layout phase.
'package:smooth_sheets/src/foundation/framework.dart':
Failed assertion: line 125 pos 7: '_extent.hasPixels'

The relevant error-causing widget was:
    NavigationSheet NavigationSheet:file:///Users/blinqipa/Desktop/wolt_test/lib/main.dart:181:16

What would be the correct way to handle this?

Thanks for the quick fix on this @fujidaiti. Just tested it and it works perfectly!


Stack Corruption in mono (where code is fine on Windows)

If you take the following complete program and complie/run under mono on Ubuntu:
open System

let  Bits (n:int) =
    let rec look n cnt =
        //printfn "  ... %2d %2d" n cnt
        if n = 0 then cnt
        else look (n>>>1) (cnt+1)
    look n 0

[<EntryPoint>]
let main argv =
    for i in 0..7 do
        let bits    =  Bits i
        printfn "%2d = %2d" i bits
    0

You get:
0 =  0
0
* Assertion: should not be reached at tramp-amd64.c:396

Stacktrace:

Native stacktrace:

	mono() [0x49cf0c]
	/lib/x86_64-linux-gnu/libpthread.so.0(+0x10340) [0x7f82009c8340]
	/lib/x86_64-linux-gnu/libc.so.6(gsignal+0x39) [0x7f8200628bb9]
	/lib/x86_64-linux-gnu/libc.so.6(abort+0x148) [0x7f820062bfc8]
	mono() [0x62a329]
	mono() [0x62a537]
	mono() [0x62a686]
	mono() [0x4f368d]
	[0x4059c0ef]

Debug info from gdb:

Could not attach to process.  If your uid matches the uid of the target
process, check the setting of /proc/sys/kernel/yama/ptrace_scope, or try
again as the root user.  For more details, see /etc/sysctl.d/10-ptrace.conf
ptrace: Operation not permitted.
No threads.

=================================================================
Got a SIGABRT while executing native code. This usually indicates
a fatal error in the mono runtime or one of the native libraries
used by your application.
=================================================================

If you take the compiled executable (Test3.exe) and copy it over to Windows and try to run it, you get:
> Test3.exe
 0 =  0
 1 =  1
 2 =  2
 3 =  2
 4 =  3
 5 =  3
 6 =  3
 7 =  3

If you stay on Linux under mono and just uncomment the printfn it also runs perfectly.
.

Playing some more with it. If I replace the printfn with any inline computation or a call to a function defined in the same file, it still blows up... but, if I call anything external to my module, e.g.:
let _ = abs 2

It works perfectly. Looks like a tail-recursive code optimization gone bad.

You should raise this as a bug to mono, using the issue here to track it to completion.

I'll enter a mono bug

@dbwz8 Can you show us your mono version please? (I think that is mono --version)

Bug entered here: https://bugzilla.xamarin.com/show_bug.cgi?id=38250
@dbwz8 - we still need the exact mono version please, thanks

This has been resolved fix in mono


goapp: errors that suggest gcloud commands should point user to gcloud SDK

In experimenting with the Go App Engine SDK, the goapp command returned an error that helpfully suggested a gcloud command to resolve it.  Unfortunately, I did not have the gcloud command installed.  The goapp error would be somewhat more helpful if it also suggested where to get the gcloud command (download the Google Cloud SDK).
sameer-macbookpro2% goapp deploy -application ELIDED-154714 -version 1
09:57 AM Application: ELIDED-154714 (was: None); version: 1 (was: None)
09:57 AM Host: appengine.google.com
09:57 AM Starting update of app: ELIDED-154714, version: 1
09:57 AM Getting current resource limits.
09:57 AM Scanning files on local disk.
Error 404: --- begin server output ---
This application does not exist (project_id=ELIDED-154714'). To create an App Engine application in this project, run "gcloud beta app create" in your console.
--- end server output ---
error while running appcfg.py: exit status 1
sameer-macbookpro2% gcloud beta app create
zsh: command not found: gcloud
At this point, the user has to find their way to this page for instructions on how to proceed:
https://cloud.google.com/sdk/gcloud/#gcloud_and_the_sdk

This issue will be obsolete soon: goapp/appcfg.py will be (next month or so) be replaced with gcloud app deploy.

Closing assuming it's obsolete.