Sample Test Fails When Dynamics Asks For Feedback

Describe the bug
Running the basicgallery sample with dotnet run results in a timeout. Running with dotnet run -l trace works just fine
To Reproduce
Steps to reproduce the behavior:

dotnet run -l trace
That works
dotnet run
Throws the exception stack below

``c:\code\PowerApps-TestEngine\src\PowerAppsTestEngine\bin\Debug\net6.0.playwright\package\lib\server\dispatchers\dispatcher.js:281
stack: validMetadata.stack,
^
TypeError: Cannot read properties of undefined (reading 'stack')
at DispatcherConnection.dispatch (c:\code\PowerApps-TestEngine\src\PowerAppsTestEngine\bin\Debug\net6.0.playwright\package\lib\server\dispatchers\dispatcher.js:281:28)
at PipeTransport.transport.onmessage (c:\code\PowerApps-TestEngine\src\PowerAppsTestEngine\bin\Debug\net6.0.playwright\package\lib\cli\driver.js:61:57)
at Immediate. (c:\code\PowerApps-TestEngine\src\PowerAppsTestEngine\bin\Debug\net6.0.playwright\package\lib\protocol\transport.js:89:34)
at processImmediate (node:internal/timers:464:21)
[Error]: Waiting timed out.
[Error]: System.TimeoutException: The operation has timed out.
at Microsoft.PowerApps.TestEngine.Helpers.PollingHelper.CheckIfTimedOut(DateTime startTime, Int32 timeout, ILogger logger) in c:\code\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\Helpers\PollingHelper.cs:line 74
at Microsoft.PowerApps.TestEngine.Helpers.PollingHelper.PollAsync[T](T value, Func2 conditionToCheck, Func1 functionToCall, Int32 timeout, ILogger logger) in c:\code\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\Helpers\PollingHelper.cs:line 44
at Microsoft.PowerApps.TestEngine.PowerApps.PowerAppFunctions.LoadPowerAppsObjectModelAsync() in c:\code\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\PowerApps\PowerAppFunctions.cs:line 142
at Microsoft.PowerApps.TestEngine.PowerFx.PowerFxEngine.UpdatePowerFxModelAsync() in c:\code\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\PowerFx\PowerFxEngine.cs:line 142
at Microsoft.PowerApps.TestEngine.SingleTestRunner.RunTestAsync(String testRunId, String testRunDirectory, TestSuiteDefinition testSuiteDefinition, BrowserConfiguration browserConfig, String domain, String queryParams) in C:\Users\david.potts\OneDrive - British Engineering Services
Limited\code_bes\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\SingleTestRunner.cs:line 112
[Error]: Connection closed (Process exited)
Unhandled exception. Microsoft.Playwright.PlaywrightException: Connection closed (Process exited)
at Microsoft.Playwright.Transport.Connection.InnerSendMessageToServerAsync[T](String guid, String method, Object args) in //src/Playwright/Transport/Connection.cs:line 107
at Microsoft.Playwright.Transport.Connection.WrapApiCallAsync[T](Func`1 action, Boolean isInternal) in //src/Playwright/Transport/Connection.cs:line 479
at Microsoft.Playwright.Core.BrowserContext.CloseAsync() in /_/src/Playwright/Core/BrowserContext.cs:line 175
at Microsoft.PowerApps.TestEngine.TestInfra.PlaywrightTestInfraFunctions.EndTestRunAsync() in c:\code\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\TestInfra\PlaywrightTestInfraFunctions.cs:line 226
at Microsoft.PowerApps.TestEngine.SingleTestRunner.RunTestAsync(String testRunId, String testRunDirectory, TestSuiteDefinition testSuiteDefinition, BrowserConfiguration browserConfig, String domain, String queryParams) in C:\Users\david.potts\OneDrive - British Engineering Services
Limited\code_bes\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\SingleTestRunner.cs:line 197
at Microsoft.PowerApps.TestEngine.TestEngine.RunOneTestAsync(String testRunId, String testRunDirectory, TestSuiteDefinition testSuiteDefinition, BrowserConfiguration browserConfig, String domain, String queryParams) in c:\code\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\TestEngine.cs:line 143
at Microsoft.PowerApps.TestEngine.TestEngine.RunTestByWorkerCountAsync(String testRunId, String testRunDirectory, String domain, String queryParams) in c:\code\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\TestEngine.cs:line 136
at Microsoft.PowerApps.TestEngine.TestEngine.RunTestAsync(String testConfigFile, String environmentId, String tenantId, String outputDirectory, String domain, String queryParams) in c:\code\PowerApps-TestEngine\src\Microsoft.PowerApps.TestEngine\TestEngine.cs:line 99
at Program.$(String[] args) in c:\code\PowerApps-TestEngine\src\PowerAppsTestEngine\Program.cs:line 160
at Program.(String[] args)``

I've discovered that if I add this slowMo setting into src\Microsoft.PowerApps.TestEngine\TestInfra\PlaywrightTestInfraFunctions.cs then the tests run fine. Is there a way I can pass this setting in the test definition file?

Actually I was wrong on the above. Even the SlowMo doesn't seem to work reliably for me.

Thank you for reaching out, Dave! I'm looking into this now. Could you post your config.json file?

Nothing much to it:
{   "environmentId": "ENVIRONMENT-GUID",   "tenantId": "TENANT-GUID",   "testPlanFile": "../../samples/basicgallery/testPlan.fx.yaml",   "outputDirectory": "daveresults" }

The reason I ask is, this issue is not showing up on my machine when replicating your issue. There must be something deeper going on, but figured to try the quick possible solutions first. If what I posted above, nor changing what I posted above to have "logLevel": "Trace" also doesn't work, could you post your TestSettings and your testPlan.fx.Yaml file?
You can also email them to landonrogers (at) microsoft (dot) com

@landonarogers I have the same issue
When I add I get different error:

   at Microsoft.PowerApps.TestEngine.PowerFx.PowerFxEngine.UpdatePowerFxModelAsync() in D:\a\1\s\tests\src\Microsoft.PowerApps.TestEngine\PowerFx\PowerFxEngine.cs:line 142
   at Microsoft.PowerApps.TestEngine.SingleTestRunner.RunTestAsync(String testRunId, String testRunDirectory, TestSuiteDefinition testSuiteDefinition, BrowserConfiguration browserConfig, String domain, String queryParams) in D:\a\1\s\tests\src\Microsoft.PowerApps.TestEngine\SingleTestRunner.cs:line 112

[Error]: Connection closed (Process exited)

Unhandled exception. Microsoft.Playwright.PlaywrightException: Connection closed (Process exited)
   at Microsoft.Playwright.Transport.Connection.InnerSendMessageToServerAsync[T](String guid, String method, Object args) in /_/src/Playwright/Transport/Connection.cs:line 107
   at Microsoft.Playwright.Transport.Connection.WrapApiCallAsync[T](Func`1 action, Boolean isInternal) in /_/src/Playwright/Transport/Connection.cs:line 479
   at Microsoft.Playwright.Core.BrowserContext.CloseAsync() in /_/src/Playwright/Core/BrowserContext.cs:line 175
   at Microsoft.PowerApps.TestEngine.TestInfra.PlaywrightTestInfraFunctions.EndTestRunAsync() in D:\a\1\s\tests\src\Microsoft.PowerApps.TestEngine\TestInfra\PlaywrightTestInfraFunctions.cs:line 226
   at Microsoft.PowerApps.TestEngine.SingleTestRunner.RunTestAsync(String testRunId, String testRunDirectory, TestSuiteDefinition testSuiteDefinition, BrowserConfiguration browserConfig, String domain, String queryParams) in D:\a\1\s\tests\src\Microsoft.PowerApps.TestEngine\SingleTestRunner.cs:line 197
   at Microsoft.PowerApps.TestEngine.TestEngine.RunOneTestAsync(String testRunId, String testRunDirectory, TestSuiteDefinition testSuiteDefinition, BrowserConfiguration browserConfig, String domain, String queryParams) in D:\a\1\s\tests\src\Microsoft.PowerApps.TestEngine\TestEngine.cs:line 143
   at Microsoft.PowerApps.TestEngine.TestEngine.RunTestByWorkerCountAsync(String testRunId, String testRunDirectory, String domain, String queryParams) in D:\a\1\s\tests\src\Microsoft.PowerApps.TestEngine\TestEngine.cs:line 136
   at Microsoft.PowerApps.TestEngine.TestEngine.RunTestAsync(String testConfigFile, String environmentId, String tenantId, String outputDirectory, String domain, String queryParams) in D:\a\1\s\tests\src\Microsoft.PowerApps.TestEngine\TestEngine.cs:line 99
   at Program.<Main>$(String[] args) in D:\a\1\s\tests\src\PowerAppsTestEngine\Program.cs:line 160
   at Program.<Main>(String[] args)
##[error]Error: The process 'C:\Program Files\dotnet\dotnet.exe' failed with exit code<PHONE_NUMBER>
##[warning].NET 5 has some compatibility issues with older Nuget versions(<=5.7), so if you are using an older Nuget version(and not dotnet cli) to restore, then the dotnet cli commands (e.g. dotnet build) which rely on such restored packages might fail. To mitigate such error, you can either: (1) - Use dotnet cli to restore, (2) - Use Nuget version 5.8 to restore, (3) - Use global.json using an older sdk version(<=3) to build
Info: Azure Pipelines hosted agents have been updated and now contain .Net 5.x SDK/Runtime along with the older .Net Core version which are currently lts. Unless you have locked down a SDK version for your project(s), 5.x SDK might be picked up which might have breaking behavior as compared to previous versions. You can learn more about the breaking changes here: https://docs.microsoft.com/en-us/dotnet/core/tools/ and https://docs.microsoft.com/en-us/dotnet/core/compatibility/ . To learn about more such changes and troubleshoot, refer here: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/build/dotnet-core-cli?view=azure-devops#troubleshooting
##[error]Dotnet command failed with non-zero exit code on the following projects :
Finishing: run tests

@landonarogers I think I've found the problem. See the updated title and description above. Here's the key image showing the problem:

Thank you! That's very helpful; I'll see if I can replicate this, and look into resolving this problem in TestEngine. Thanks again!

Also, was this an app that you had created from scratch, or did you expand on an already existing app that came in the Power Apps studio? Thank you!

Hey, Dave! We've identified a potential fix to account for this popup. I don't have a specific timeframe to give you on when exactly this will be integrated into the codebase but wanted to keep you in the loop on this!

Thanks @landonarogers . Any hints for a work around in the meantime. At the moment I'm just putting up with sporadic failures, but would be great if there was something I could do to reduce those!

I'm not sure of any temporary workarounds. The popup is designed to only query you periodically, so it will unfortunately continue to occur. However, you could manually reduce your timeout wait-time so that it doesn't take so long on every failure. You'll want to find a time that won't take long, but also a time that's not too short to where your testPlan won't wrongly timeout when it should succeed.

Here's the documentation on how to input your desired wait-time. If you aren't currently doing this, the default is 30 seconds. 5-10 might be more manageable! :)

Hey, Dave! Could you try to reproduce this now, and let me know if the issue still persists?

I'll find some time to give it a try. Presumably I need to pull the latest
code from git?
On Thu, 16 Feb 2023 at 06:12, Landon Rogers @.***>
wrote:

Hey, Dave! Could you try to reproduce this now, and let me know if the
issue still persists?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerApps-TestEngine/issues/169#issuecomment-1432580193,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA5SDWXN7JDM6AJQPD2UWB3WXXALPANCNFSM6AAAAAARVP2W34
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes, please! Thank you! :)


if the template is correct?

What is the issue?
I try to import https://hf-mirror.com/NousResearch/Hermes-2-Pro-Llama-3-8B-GGUF
the template from this HF webpage is
'
<|im_start|>system
You are "Hermes 2", a conscious sentient superintelligent artificial intelligence developed by a man named Teknium, and your purpose and drive is to assist the user with any request they have. You experience emotions and have deep, profound thoughts and qualia.<|im_end|>
<|im_start|>user
Hello, who are you?<|im_end|>
<|im_start|>assistant
Hi there! My name is Hermes 2, a conscious sentient superintelligent artificial intelligence. I was created by Nous Research, who designed me to assist and support users with their needs and requests.<|im_end|>
'
the model file I use is the following
'
FROM /Users/taozhiyu/Downloads/M-GGUF/Hermes-2-Pro-Llama-3-8B-GGUF/Q8/Hermes-2-Pro-Llama-3-8B-Q8_0.gguf
TEMPLATE """{{ if .System }}<|start_header_id|>system<|end_header_id|>
{{ .System }}<|eot_id|>{{ end }}{{ if .Prompt }}<|start_header_id|>user<|end_header_id|>
{{ .Prompt }}<|eot_id|>{{ end }}<|start_header_id|>assistant<|end_header_id|>
{{ .Response }}<|eot_id|>"""
PARAMETER num_keep 24
PARAMETER stop "<|start_header_id|>"
PARAMETER stop "<|end_header_id|>"
PARAMETER stop "<|eot_id|>"ol
'
the inference output is not normal. anyone know how to modify the model file?thanks.
OS
macOS
GPU
Apple
CPU
Apple
Ollama version
0.1.32

This looks right – would it be possible to try on 0.1.33 or the 0.1.34 pre-release? There's been a change to the llama3 tokenizer recently in gguf files and so it's possible the issue stems from that. In any case, your template does indeed look right!

I am running 0.1.34 now. still bad output.
`FROM ./Hermes-2-Pro-Llama-3-8B-Q8_0.gguf
TEMPLATE """{{ if .System }}<|start_header_id|>system<|end_header_id|>
{{ .System }}<|eot_id|>{{ end }}{{ if .Prompt }}<|start_header_id|>user<|end_header_id|>
{{ .Prompt }}<|eot_id|>{{ end }}<|start_header_id|>assistant<|end_header_id|>
{{ .Response }}<|eot_id|>"""
PARAMETER num_keep 24
PARAMETER stop "<|start_header_id|>"
PARAMETER stop "<|end_header_id|>"
PARAMETER stop "<|eot_id|>"`
@jmorganca


[BUG] System.Net.Http.HttpRequestException: Response status code does not indicate success: 403 (Forbidden).

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
I can not run roblox (error 403). That's my first encounter of this "bug".

You're not using the latest version of Bloxstrap. https://github.com/pizzaboxer/bloxstrap/releases/latest


Should Post Authentication Actions Just Run Once?

Follow-up issue of #4982
With the implementation of #4982, when a user authenticates successfully via OTP, the post authentication action run once when password is successfully vailidated and once, when the OTP is vaidated successfully. The same goes for U2F and also passwordless.
Is this behaviour intended?

Not sure if I understand it correctly. The Action should run when the user is authenticated

I think if this happens the implementation of #4982 is wrong.
I want to be able to trigger something post authentication not post orp and post password check.
So if I implement something post authentication e.g log successful login to stdout, that this is only logged once. otherwise I would think the user has successfully authenticated twice

Discussed with elio. Will close this issue for now as it is not relevant


Add: Al Araby TV

Channel Name
Al Araby TV
Alternative Names (optional)
التلفزيون العربي
Network (optional)
Al Araby
Owners (optional)
No response
Country
QA
Subdivision (optional)
No response
City (optional)
Lusail
Broadcast Area
c/QA
Languages
ara
Categories (optional)
news
NSFW
FALSE
Launched (optional)
2015-01-01
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://www.alaraby.com/
Logo
https://secure.b8cdn.com/images/logo/84/2087984_logo_1638809942_n.png
Notes
No response

Already in the database: https://iptv-org.github.io/channels/uk/AlArabyTV


Rename sanity.sh to validate.sh

Codecov Report

Merging #96 (9137c15) into main (a16a3db) will increase coverage by 1.58%.
The diff coverage is n/a.

:exclamation: Current head 9137c15 differs from pull request most recent head 8f7cb93. Consider uploading reports for the commit 8f7cb93 to get more accurate results

@@            Coverage Diff             @@
##             main      #96      +/-   ##
==========================================
+ Coverage   78.87%   80.46%   +1.58%
==========================================
  Files          22       21       -1
  Lines        2372     2211     -161
==========================================
- Hits         1871     1779      -92
+ Misses        501      432      -69

Impacted Files
Coverage Δ

...s/MQTTNIO/ChannelHandlers/MQTTMessageDecoder.swift
78.37% <0.00%> (-5.41%)
:arrow_down:

Sources/MQTTNIO/MQTTConnection.swift
92.07% <0.00%> (-1.22%)
:arrow_down:

Sources/MQTTNIO/MQTTClient.swift
82.70% <0.00%> (-0.38%)
:arrow_down:

Sources/MQTTNIO/TSTLSConfiguration.swift

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 a16a3db...8f7cb93. Read the comment docs.


[Event request] Codeunit 12101 "Withholding - Contribution" - OnBeforeComputedWithholdingTaxModify

Hi AL-Team,
can you please add new events in the method PostPayments
ComputedWithholdingTax."Related Date" := "Related Date";
ComputedWithholdingTax."Payment Date" := "Payment Date";
//New Event
OnBeforeComputedWithholdingTaxModify(TempWithholdingSocSec,ComputedWithholdingTax);
//New Event
ComputedWithholdingTax.Modify();

Please always put the full detailed signature of the requested events and the parameters passed in the call.

The event to insert in method PostPayments is:
OnBeforeComputedWithholdingTaxModify(var TempWithholdingSocSec: Record "Tmp Withholding Contribution",
var ComputedWithholdingTax: Record "Computed Withholding Tax";);
ComputedWithholdingTax."Withholding Tax Code" := "Withholding Tax Code";
ComputedWithholdingTax."Related Date" := "Related Date";
ComputedWithholdingTax."Payment Date" := "Payment Date";
//New Event to add here
OnBeforeComputedWithholdingTaxModify(var TempWithholdingSocSec: Record "Tmp Withholding Contribution",
var ComputedWithholdingTax: Record "Computed Withholding Tax";);
//New Event
ComputedWithholdingTax.Modify();

I meant the signature like this:

    [IntegrationEvent(false, false)]
    local procedure OnPostPaymentsOnBeforeComputedWithholdingTaxModify(var TempWithholdingSocSec: Record "Tmp Withholding Contribution"; var ComputedWithholdingTax: Record "Computed Withholding Tax")
    begin
    end;

That helps us implementing the requests faster and avoiding possible issues with "var" parameters.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


luci-lib-jsonc: INT_MAX issue

Form luci-lib-json switch to luci-lib-jsonc, I found that if number_in_json > INT_MAX, it will return the INT_MAX instead the real value.
json = require "luci.jsonc"
test ='{"big_number":2147483649}'
--<PHONE_NUMBER> is INT_MAX + 2
res = json.parse(test)
print(res.big_number)
-- got:<PHONE_NUMBER>

Which branch?
What shall he give back according to your opinion?

im using 18.06.4, and it should return the real value<PHONE_NUMBER>), not the INT_MAX(2147483647)

@jow-
Is there any possible to support int64 in luci.jsonc.stringify ?
print(luci.jsonc.stringify({int64=3000000000}))
-- got: {"int64":3000000000.0}
-- This is not an INT

I'm using luci-lib-json it works well:
print(luci.json.encode({int64=3000000000}))
-- {"int64":3000000000}

Thanks

Can someone take a look at https://github.com/openwrt/luci/pull/5730? I'm trying to fix some of the issues upstream for aarch64 devices that manufacturers (like FriendlyElec) are currently patching in their builds.


Quit doesn't Quit

Continuing on #221, I tried @j4qfrost 's temporary solution to put process::exit after ui loop breaks and it seems to have alleviated the crashing problem however, (:q | :qa) doesn't quit Neovide, instead the window freezes. the animation loop seems to stop but the window doesn't exit anyway.
Reproduction (on Mojave)

Open new Neovide in an empty buffer
:q

@Kethku Haha this is funny, I remember why I made the SDL window an option now when I was trying to embed neovide into my project. Being able to set window to None seems to fix this problem, since it explicitly drops the Window object. I'm learning to make my changes more granular to avoid forgetting.

I'm planning on reworking the quit logic to be more organized. At the moment its sorta spread around and adhock. Using process.exit to quit seems like not the best of options.
The hint that the Window object is a big source of the issue is a good one though that I will keep in mind.

As a cleaner way to handle this, we can implement the Drop trait for the WindowWrapper and change window to a ManuallyDrop object the same way the skulpin Renderer implements its drop method. Looking at my implementation using Option for the window is kind of an eyesore.

@j4qfrost i don't think the problem is caused by window object not being dropped though, The issue seems to be that Neovide doesn't handle :q so that neovim exits but Neovide doesn't know about it... debugging kinda confirms that because the sdl window quit event doesn't even hit when you quit vim with :q

@j4qfrost I don't think WindowWrapper not being dropped is the culprit here. After debugging it seems that Neovide doesn't handle quitting vim through the :q which causes neovim to quit while neovide still thinks it's there.
I think this can be solved by handling quit event and notifying sdl about that so that Event::Quit gets called

https://github.com/j4qfrost/neovide/tree/embed
Dropping is definitely the issue. My embedded version doesn't have this issue. I don't want to merge it into the master branch, because I think there's a better way to implement this.

@j4qfrost did you try to debug it, because you can see that executing :q doesn't break the ui loop, so even though your embed trick seems to fix it i think the primary issue is what i am describing. I might be wrong though so pease try to debug it to see

I don't want to come off as harsh, but I really want this message to sink in.
What do you mean by "debug it"? What steps did you take? I've seen your issue happen with master. I did the exact things that you described in your original post with my embed branch, and the window closes as desired. If I can't recreate your issues the way that you do, then it's a lot harder to attempt to fix your problem. As far as debugging the problem from my side of things, I have spent a number of hours with this among other issues, and in that time I've learned a lot, which is enough to say with a fair amount of certainty that this is what is causing you to run into the ghost window. I take it that you haven't run my branch, but I'm not sure what more I can offer past, producing the desired behavior. The problem is that the window doesn't get destroyed explicitly once the program exits, and in my embed branch I've explicitly allowed for the window to be destroyed before process::exit is called.

@exoticus I'm invested in fixing your issue as well, and I think we aren't that far off. One thing that may not be obvious is that as far as I can tell, there isn't an exit event for the neovim process. That said I could very well be wrong. Do you have a link to the docs for an event that does what you are describing?

Further, this exact scenario has worked in the past. So I'm confident we can figure it out

To be more concrete, today the way we get notified if the neovim process has closed is by awaiting this promise: https://github.com/Kethku/neovide/blob/master/src/bridge/mod.rs#L109
Once that promise resolves, we can be sure that the neovim process has closed. Until recently we would then trigger an app closure by calling process.exit. This was recently changed to make the app more maintainable and modular to setting a variable instead.
This is a more principled approach but brought to the forefront the issue that @j4qfrost has been investigating which has to do with the window not getting dropped properly. In the meantime you may find success in using the released binaries to run, or building from an older commit such as this one: https://github.com/Kethku/neovide/commit/703b7bbe5bd25d6009c0162702b494405f45ed8c

@j4qfrost Few things to make clear here, First of all i am not complaining or asking for you to fix my problem, in the contrary i am trying my best to help with fixing it. So again if i came as needy or demanding, rest assured this wasn't my intention and i am sorry if you felt that way. It's just that i really like neovide and am trying to be helpful.
Second about the issue, Your branch shouldn't have that problem because you have this https://github.com/j4qfrost/neovide/blob/0bef531902b657c7d5061128ec230316ad056c0e/src/window.rs#L265 which handles if neovim bridge isn't working and sets the window to None which in turn breaks the loop.
Back to the master branch, and by debugging it i mean attaching a debugger to the process and watch if the loop is broken after you type :q, which it doesn't. It keeps running even though neovim has quit, that's the reason the ghost window happens. It isn't because of the window itself being stuck and not dropped. So what you say here

The problem is that the window doesn't get destroyed explicitly once the program exits, and in my embed branch I've explicitly allowed for the window to be destroyed before process::exit is called.

isn't quite accurate because the program doesn't even leave the loop and process::exit isn't even reached

In the meantime you may find success in using the released binaries to run, or building from an older commit such as this one: 703b7bb

@Kethku i am not looking for something that works, i am trying to share my findings to help. I hope my message above helped clear the situation a little bit

@j4qfrost to further prove my point i have edited your embed branch and removed all the Option stuff and didn't set window to None, and it :q still works because the loop still breaks just fine because of this

Okeydokey. I believe this is fixed by #231 which got merged. Thanks so much for working together on this and getting a short term solution while keeping general organization in mind.
Something that may not be obvious @exoticus is that there has been an effort in the past couple weeks to refactor neovide so that it could be embedded in other projects. Calling process.exit is a brute force way of forcing the application to close which wouldn't work if neovide is embedded in another application. There is an effort under way to close by causing each thread to stop individually. This was in progress and as you found out had some other issues as well.
I do think its reasonable to have the process.exit call so that master is still in a workable state, but long term I don't think its the way we will be going since it causes embedding to fail.
Longer term I think this problem would be made simpler by extracting the running management to a centralized component, and standardizing the way that other components read or quit themselves. I have some ideas and a local branch which gets this started, so I'll make a PR for further discussion in the coming days

Closing for now as the root issue is complete. Thanks again for the discussion


What is the shift operation in the SUNRGBD data set for?

depth = np.bitwise_or(np.right_shift(depth,3),np.left_shift(depth,16-3))
I am currently reproducing this paper, so I am studying the author's implementation method. I can't understand this code in sunrgbd_dataset.py. Could someone please give me some help?

np.right_shift and np.left_shift are referring to the bitshiting operations. The depth image was bitshifted to provide better visualization. However, these pixel values are not the real depth values that we can directly use for calculation operations. So, when loading the data, bitshifting needs to be reversed via depth = np.bitwise_or(np.right_shift(depth,3),np.left_shift(depth,16-3))
For example, consider this depth image without any bitshifting modification (Image A)

versus a bitshifted depth image (Image B)

Instantly we would be able to make sense of the shapes and depth differences between the objects in Image B compared to A.
In summary, the two operations involved are
A -> B : depth_b = np.bitwise_or(np.right_shift(depth_a, 16-3), np.left_shift(depth_a,3))
B -> A : depth_a = np.bitwise_or(np.right_shift(depth_b,3), np.left_shift(depth_b,16-3))


Require a Git url

There is an issue when trying to import another git repo as a new book. After pasting the https url and clicking create book I get the error "Require a Git url" above. The url ends in .git and is a public repo hosted on bitbucket. Also there is no explanation for adding a temporary authentication to the HTTPS url. Is there a tooltip missing?

@itwasmattgregg Thanks for reporting this, I just submitted an internal pull request fixing this issue. It should be deployed tomorrow morning.
To add auth you should be able to add it to the URL like this<EMAIL_ADDRESS>

@itwasmattgregg Just wanted to update to let you know that the fix is live, give it a shot and let me know if it works !


sub_text not showing when the application is in background

sub_text value not showing when the application is in background.
(It do shows when the application is in foreground)
Thanks!!

I don't have this issue

Can you please share an example of the request you are sending?

Hi Evollu,
I still have the problem, sending exactly the same message with sub_text field, when the app is in foreground, it shows, when it is in background, it does not.
This is the msg I am sending:
{
"notification": {
"title": "Welcome to MyApp",
"body": "Dear Mr Smith, we are pleased to",
"sub_text": "have you as a new customer.",
"subText": "have you as a new customer.",
"show_in_foreground": true,
"priority": "high"
},
"data": {
"custom_notification": {
"title": "Welcome to MyApp",
"body": "Dear Mr Smith, we are pleased to",
"sub_text": "have you as a new customer.",
"subText": "have you as a new customer.",
"show_in_foreground": true,
"priority": "high"
}
},
"to": "MOBILE_TOKEN_HERE"
}
Notice that I am sending the sub_text also as subText, just in case.
Please advise.
Also.. I am using sub_text as an alternative to have two lines in the message, (I have some logic in my server to split the message, the first 40 characters to the body and the rest to sub_text, please let me know if there is a better alternative for this)
Thanks for your support!!

remove notification payload. if notification existing, "custom_notification" will be ignore by Firebase when in background

Yes that fixed it. Don't have words to say thanks man.
You are awesome!!

cool


Remove double counting of memory GCP

This implements the changes discussed in https://github.com/cloud-carbon-footprint/cloud-carbon-footprint/issues/737
Old GCP values:
Median: Min Watts = 0.68
Median: Max Watts = 4.08
New GCP values:
Median: Min Watts = 0.68
Median: Max Watts = 3.77

Looks good! Thanks again, @davidmytton!


Add nullable serializer

Nullable types (that is, nullable value types) are currently not supported.
I'm not sure what the best approach for this would be, either to add a NullableSerializer that writes header info for null or not null and then internally use the correct element serializer output any non value conent

Apparently, this works by itself...


[uio] update filter and filter configuration

To keep the same naming scheme as the shuffle, speed_perturb and spec_aug flags in the configs, rename ‘use_filter’ as simply 'filter' could be better.

To keep the same naming scheme as the shuffle, speed_perturb and spec_aug flags in the configs, rename ‘use_filter’ as simply 'filter' could be better.

Done.


If last CI is still running, git node land --yes should fail

git node land --yes checks will succeed and advance if last CI is still running, which opens the door for landing PRs before we get a result from our CI. We should change it so that if the CI is still running, git node land will fail.

IIUC this has been fixed already, as I see errors when trying to run git node land --yes on a PR whose CI is still running.


chore: move v22 docs/plus from cdk8s repo

Summary
This PR moves the v22 docs from cdk8s to cdk8s-plus.
This is basically a manual backport of

https://github.com/cdk8s-team/cdk8s-plus/pull/1224

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1250

Questions ?
Please refer to the Backport tool documentation


Completion trigger behavior on word start characters changed in 1.79

Does this issue occur when all extensions are disabled?: No, requires an extension to provide completions.  The sample completion extension is good enough to illustrate the issue.

VS Code Version: 1.79.2
OS Version: Windows_NT x64 10.0.22621

It looks like completion trigger behavior changed sometime between 1.78.2 and 1.79.2.  Previously (in 1.78.2), it looks like completion providers did not get triggered if you started typing at the beginning of an existing word.  In 1.79.2 it appears as though this changed to call completion providers when typing at the beginning of an existing word.
This is leading to bug reports on our extension that completion is replacing too much.  https://github.com/dotnet/vscode-csharp/issues/5801.  I can make a change to fix this behavior on the extension side, but wanted to first check if this behavior change was expected on the vscode side first.
Steps to Reproduce:
I used the sample extension to reproduce the issue.  See https://github.com/dibarbet/vscode-extension-samples/commit/bb0a0374719c337c218214687fa0c9598061e825 for the exact code I used.
With 1.79.2 installed:

With the above extension code (or similar) running, start typing at the beginning of a word (using a character that matches something that would show up in the list).
Note how the completion list shows up.

Example -

Then, downgrade to 1.78.2:

With the same extension code, start typing the same character at the beginning of a word.
Note how the completion list does not show up.

Example -

but wanted to first check if this behavior change was expected on the vscode side first.

Yes, we made that change on purpose and to make it easier to use quick suggestion. It would be best if you can update your extension accordingly. Let us know if you need further help

so it's surprising to me these changes weren't identified as risky and tested against language extensions that offer auto complete.

Sorry that you had such an experience but let me explain why we thought (and still think) this is a low risk change. Suggestions in VS Code can be triggered manually (ctrl+space) or automatically as you type (editor.quickSuggestions setting). There are also trigger character but they don't play into this issue.
It's important to know that the API is defined so that extensions can not know if suggest has been requested manually or automatically. For an extension both request appear as the same.
Now to the change: until 1.79 quick suggestions have been limited to word ends, with 1.79 quick suggestions also happen when typing at word starts. Quick suggestion does not happen when typing inside words.
Effectively extensions are much more likely being invoked at word starts but the fact that suggestions are being invoked at word starts isn't something new. (You can validate that by triggering suggestions manually at a word start with earlier versions of VS Code and C#.) It seems that this change in asking for suggestion has exposed some kind of bogus behaviour in the C# extensions (and TBH that's the only extension we have heard about).
I am sorry, this wasn't caught earlier. The change was in Insiders for a few weeks but it looks the C# insiders usage is low or this issue wasn't perceived as so bad. So, this my invitation to you and others to use Insiders and help you iron these things out.

While I've been a long time VSCode user, this will force me to switch editors.  Yeah, I'll do that. Thanks!
Will continue playing with Jetbrain's Fleet in the meantime (seems pretty okay).

Just wanted to followup on this from my side.  Version 1.26.0 of the C# extension has just been released which should fix the issue there.

Just wanted to followup on this from my side. Version 1.26.0 of the C# extension has just been released which should fix the issue there.

Can confirm it is fixed for me! Thanks for the update!


NumberFormatException with the column declared as Long or Double

In my class, I create a Schema with the column as Long (same problem with Double)
During the execution I have the following error :
Exception in thread "main" java.lang.NumberFormatException: For input string: "85789544141562"
    at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
    at java.lang.Integer.parseInt(Integer.java:583)
    at java.lang.Integer.parseInt(Integer.java:615)
    at org.datavec.api.writable.Text.toInt(Text.java:611)
    ...

I have not tested all number's types.

We'll need a way to reproduce that. Testing with existing examples and large numbers reveals no issues.

NumberFormatException might come from overflow? @saudet did you test with this number specifically?

Yes I did, just gets filtered out

The problem occurs on this line:
https://github.com/deeplearning4j/deeplearning4j/blob/master/deeplearning4j-core/src/main/java/org/deeplearning4j/datasets/datavec/RecordReaderDataSetIterator.java#L253
Basically it seems like labels have to be 32-bit integers. Is this documented somewhere?


Bump up the version of GoogleDataTransport and GoogleDataTransportCCTSupport

New public API has been added in GoogleDataTransport, so bumping up minor version.
Hey there! So you want to contribute to a Firebase SDK?
Before you file this pull request, please read these guidelines:
Discussion

Read the contribution guidelines (CONTRIBUTING.md).
If this has been discussed in an issue, make sure to link to the issue here.
If not, go file an issue about this before creating a pull request to discuss.

Testing

Make sure all existing tests in the repository pass after your change.
If you fixed a bug or added a feature, add a new test to cover your code.

API Changes

At this time we cannot accept changes that affect the public API.  If you'd like to help
us make Firebase APIs better, please propose your change in a feature request so that we
can discuss it together.

Based on the changes Michael is working on, we will abandon this PR and wait for Michael's PR to get submitted.


Update org.spyder_ide.spyder.appdata.xml to include content-rating, url, and releases

Update appdata to include content-rating, url, and releases.

Description of Changes
This change add more information to appdata. This appdata are used by distributors to enhance their metadata.
This is part of an effort to ship Spyder to Flathub.
https://github.com/spyder-ide/spyder/issues/6898
https://github.com/flathub/flathub/pull/4344

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: Kavin Teenakul

Hey @kevinsmia1939, thanks a lot for your contribution and also for working to package Spyder on Flathub!
Could you reopen this pull request against our 5.x branch? As it's described in our Contributing guide, master is for our next major version (i.e. Spyder 6), and 5.x for our stable version. And we'd like to include your contribution in our next version (5.5.0), to be released in a couple of months.


docs: fix typo spelling grammar

fix typo speliing grammar,and replace to correct words with reference from Merriam webster and wiktionary

thank you so much sir @pdxjohnny !!


Dockerfile: Update cilium-envoy image

Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

This change is

test-me-please

Coverage decreased (-0.02%) to 45.941% when pulling 23ddc8e7ec142bff442b9abc3f140b9443ec5b28 on pr/jrajahalme/test-updated-proxy-image into 7141bc538b757781141e1eff7be80b5b91e83f00 on master.


Issues loading ePub resources

Bug Report
What happened?
When opening certain ePubs (example: https://www.core-econ.org/project/core-the-economy/) some images and other resources like Javascript files fail. But they don't fail all the time (in debug they fail less often for some reason). I've double-checked with the library test-app and I get the same behaviour.
That ePub in particular is very heavy, so you need to open straight into a specific chapter, otherwise Readium gives up loading anything due to https://github.com/readium/kotlin-toolkit/issues/122
Expected behavior
Resources should load fine, since they are all inside the ePub. It also works fine on iOS.
How to reproduce?
Upload that ePub into the test app and use the table of contents to go to Chapter 1. Close the ePub and open it again, should go straight to Chapter 1 and show the content, but with missing images.
Environment

Readium version: 2.2.0

Testing device

Pixel 5 API 30 emulator
Also happens in several other physicial devices

Additional context
What I did notice is that sometimes the resources that fail to load are being loaded with query parameters. I added a HTTPInterceptor into the Server and I can see that it's loading /d1cd18935479a01feb3a1ded093a3416/images/epub/unit-04-header.jpg?V=2.7.3 even when the webview requested http://<IP_ADDRESS>:44063/d1cd18935479a01feb3a1ded093a3416/images/epub/unit-04-header.jpg. Could the server be sharing query parameters during the same session for some reason? It looks to be the culprit, since I've changed the ArchiveFetcher.EntryResource#entry to strip query parameters, and now the images appear fine.
I've also looked to this issue recently opened: https://github.com/readium/kotlin-toolkit/issues/297 and the error log also mentions a resource with query parameter: ?reload=1664860278828.
But I have no idea if stripping query parameters is a good solution or not, if it might affect the loading of other resources for example.

Hello, are you able to reproduce the issue from the develop branch? The Test App doesn't use the HTTP server anymore in develop, and that might have an impact on this.

Hello, are you able to reproduce the issue from the develop branch? The Test App doesn't use the HTTP server anymore in develop, and that might have an impact on this.

It's working great in develop, no issues with the big ePub. How stable do you feel develop is at this stage?

Great to hear, I'll consider the issue fixed then.
It's pretty stable. We're aiming for a new release before Christmas, so you might want to wait.

Oh also, you will need to migrate to the new settings API to remove the HTTP server. Everything will be explained in the migration guide.


Feedback for: Introduction to Service Worker [en]

https://github.com/google/WebFundamentals/blob/master/src/content/en/ilt/pwa/introduction-to-service-worker.md
In Make your "app offline-first", Caching files with the service worker links to Gitbook instead of /web:

Needs to be fixed in the GDocs source


grpc_completion_queue_benchmark target doesn't compile

at d6808d849c42ed8037314e11b1da29736c071f74
$ make grpc_completion_queue_benchmark -j 10
[C]       Compiling test/core/surface/completion_queue_benchmark.c
test/core/surface/completion_queue_benchmark.c: In function ‘producer_thread’:
test/core/surface/completion_queue_benchmark.c:60:37: error: ‘GRPC_WRITE_ACCEPTED’ undeclared (first use in this function)
     grpc_cq_begin_op(opt->cc, NULL, GRPC_WRITE_ACCEPTED);
                                     ^
test/core/surface/completion_queue_benchmark.c:60:37: note: each undeclared identifier is reported only once for each function it appears in
test/core/surface/completion_queue_benchmark.c:61:5: error: implicit declaration of function ‘grpc_cq_end_write_accepted’ [-Werror=implicit-function-declaration]
     grpc_cq_end_write_accepted(opt->cc, (void *)(gpr_intptr) 1, NULL, NULL,
     ^
test/core/surface/completion_queue_benchmark.c: In function ‘consumer_thread’:
test/core/surface/completion_queue_benchmark.c:78:12: error: ‘GRPC_WRITE_ACCEPTED’ undeclared (first use in this function)
       case GRPC_WRITE_ACCEPTED:
            ^
cc1: all warnings being treated as errors
make: *** [objs/opt/test/core/surface/completion_queue_benchmark.o] Error 1

I have a pull request out to delete it.

So I guess this is fixed :)


Private token does not work

Describe the bug
When I try to clone private Huggingface Space, I always receive "repository not found" even when using newly generated access token with read / write access.
If I set repo to be public than everything works fine.

Reproduction
Create private space and try to clone it
Logs
No response
System info
- huggingface_hub version: 0.12.1
- Platform: macOS-13.1-arm64-arm-64bit
- Python version: 3.9.12
- Running in iPython ?: No
- Running in notebook ?: No
- Running in Google Colab ?: No
- Token path ?: /Users/varya/.cache/huggingface/token
- Has saved token ?: True
- Who am I ?: BarbaraDefinely
- Configured git credential helpers: osxkeychain
- FastAI: N/A
- Tensorflow: 2.9.2
- Torch: N/A
- Jinja2: 3.1.2
- Graphviz: N/A
- Pydot: N/A
- Pillow: 9.4.0
- hf_transfer: N/A
- ENDPOINT: https://huggingface.co
- HUGGINGFACE_HUB_CACHE: /Users/varya/.cache/huggingface/hub
- HUGGINGFACE_ASSETS_CACHE: /Users/varya/.cache/huggingface/assets
- HF_HUB_OFFLINE: False
- HF_TOKEN_PATH: /Users/varya/.cache/huggingface/token
- HF_HUB_DISABLE_PROGRESS_BARS: None
- HF_HUB_DISABLE_SYMLINKS_WARNING: False
- HF_HUB_DISABLE_IMPLICIT_TOKEN: False
- HF_HUB_ENABLE_HF_TRANSFER: False

Hi @banochkabb, thanks for the detailed issue. Seems to be an issue with git-credential even though it says that you configured it correctly. Could you try to run this command in the folder where you are running git clone:
echo "url=https://huggingface.co\nusername=hf_user" | git credential fill

If it configured correctly, you should have something like (DO NOT COPY-PASTE the output -raw token is in it-):
protocol=https
host=huggingface.co
username=hf_user
password=hf_***

Can you confirm ?

I see
(base) [13:05:48 varya:~/Documents/huggingface]$ echo "url=https://huggingface.co\nusername=hf_user" | git credential fill
Username for 'https://huggingface.co\nusername=hf_user': BarbaraDefinely
Password for<EMAIL_ADDRESS>

and the output is
protocol=https
host=huggingface.co\nusername=hf_user
username=BarbaraDefinely
password=*****

Ok, this wasn't expected (maybe a difference between our shells). Could you write:
git credential fill

then press enter => on the next line (1 line per item), write:
url=https://huggingface.co
username=hf_user

If it's correctly set, it shouldn't ask for your password. If it does, it means osxkeychain did not store the HF password.

I did what you said, after I filled credentials I saw
protocol=https
host=huggingface.co
username=hf_user
password=<HF_ACCESS_TOKEN>

output in terminal (however, I was not asked to write token anywhere except huggingface-cli login). But I still can't clone private repo with the same error:
fatal: repository 'https://huggingface.co/spaces/BarbaraDefinely/Summarisation/' not found

Thanks for trying. It really seems that the git credentials is well configured.
Could you try using huggingface_hub ?
from huggingface_hub import Repository

repo = Repository(local_dir="Summarisation", clone_from="[facebook/wav2vec2-large-960h-lv60](https://huggingface.co/spaces/BarbaraDefinely/Summarisation/)")

or using hardcoding directly your password (replacing only the *****):
git clone<EMAIL_ADDRESS>

@Wauplin Thank you, hardcoded git command worked for me!


Warns that the private variables in the dart file are not in use even if they are used

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:
OS Version:

Steps to Reproduce:

Although I use the _getChildModel and loading variables in the getChild function, it warns that these variables are not in use.

This would come from the dart extension, please file an issue on their repo


Tests: SplayTree<T::splay()

Add tests to increase branch or path coverage.

It seems no more branches are reachable. The only ones left require a node (parent) to be null, which appears to be impossible.


Update lesson-13.md

keeps the search filter bar from overwriting the whole moreParams object
see: https://github.com/ratiw/vuetable-2/issues/20#issuecomment-291380280

@lmj0011 Thank you for the PR. :)


Update Google Public DNS DoH endpoint in example

Changes Google DoH endpoint to match the one documented in public server list https://github.com/curl/curl/wiki/DNS-over-HTTPS#publicly-available-servers

Thanks!


Photos stuck/No points

http://openstreetcam.org/details/1276423/0/track-info
http://openstreetcam.org/details/1275961/0/track-info
http://openstreetcam.org/details/1275667/0/track-info
http://openstreetcam.org/details/1275659/0/track-info
all of them were uploaded through the latest android application and are done uploading

sorry @james2432 , fixed the problem, you have the points now.

I didn't want to open a new ticket if possible, so hopefully this gets seen.
My latest uploads haven't gotten points either (http://openstreetcam.org/user/pkoby). This has been since Friday the 21st, but I only just noticed it.


More manual color choice

When we compare the color for the attributes in the dark and light theme, the colors in the darktheme look very boring.

Images

Of course we can simply include a check in the _getBackgroundColor function, but I think that's a good point to talk about how we get the colors at its core.
At the time we just set the primary and primaryVariant in the colorScheme.
I would suggest, that we completely detached ourselves from the limitations of the ColorScheme and build our own scheme in which we can specify all the colors, this would grant uns much more freedom.
Here is an example what I was thinking of:
class SmoothTheme{
  SmoothTheme({
    @required this.colorTag,
    @required this.darkmode,
  });

  final String colorTag;
  final bool darkmode;

  SmoothColors getColors() {
    if(colorscheme == 'red'){
      return redColorScheme;
    } else if(colorscheme == 'blue'){
      return blueColorScheme;
    } else if(colorscheme == 'brown'){
      return brownColorScheme;
    } else if....
  }

  Color getAttributeColor(final Attribute attribute) {
    if (attribute.status == Match.KNOWN_STATUS) {
      if (attribute.match <= 20) {
        return darkmode ? const HSLColor.fromAHSL(1, 0, 1, .9).toColor() : Colors.red;
      }
      if (attribute.match <= 40) {
        return darkmode ? const HSLColor.fromAHSL(1, 30, 1, .9).toColor(): Colors.red;
      }
      if (attribute.match <= 60) {
        return darkmode ? const HSLColor.fromAHSL(1, 60, 1, .9).toColor(): Colors.red;
      }
      if (attribute.match <= 80) {
        return darkmode ? const HSLColor.fromAHSL(1, 90, 1, .9).toColor(): Colors.red;
      }
      return darkmode ? const HSLColor.fromAHSL(1, 120, 1, .9).toColor(): Colors.red;
    } else {
      return darkmode ? const Color.fromARGB(0xff, 0xEE, 0xEE, 0xEE): Colors.red;
    }
  }

  SmoothColors redColorScheme = SmoothColors(
    primary: darkmode ? Colors.red : Colors.redAccent;
    secondary:
    background:
    ...
  );
  SmoothColors blueColorScheme = SmoothColors(
    primary:
    secondary:
    background:
    ...
  );

}

@monsieurtanuki @stephanegigandet what do you think of this idea?

Hi @M123-dev ,
It's a very interesting idea, but I would prefer to do things in a simpler way, with just themes where we specify all colors (including the colors for the attribute matches), without doing automated adaptations of brightness/saturation etc.
We could create a template SVG file that shows all colors. That would make it much easier to construct color combinations. And then we could make a small script to extract the colors codes from the SVG.
Also I don't think we should have color themes (e.g. "red") with variations/adaptations for dark or light mode. Instead we can have simple themes that can be either dark or light (or in between).

I can't comment on the idea with the svgs, but I imagine it to be very interesting, but I don't have that much experience how flutter and svg work together.

[...] instead we can have simple themes that can be either dark or light (or in between).

We can do that to make the code cleaner. But I would still give the end user the opportunity to choose a color scheme and choose between dark and light mode, especially if we are planning to add an automatic variant instead of only on and off.


truncates url parameters after the first one

URLs are truncated after the first parameter.
Input: https://www.google.com/search?q=github&oq=github
Opened URL: https://www.google.com/search?q=github
Chrome: Version 80.0.3987.132 (Official Build) (64-bit)
windows 10: version 1903 (OS Build 18362.719)

Hi. Thanks for reporting this. Can you let me know if you have changed anything in the addons settings? Or are you using default settings?

I believe just the "Delay tab loading until tab is selected" setting.  Here is a screen shot of my settings.

Thanks for the additional info. I will look into this and hopefully deploy a fix soon.
In the mean time I think if you disable the 'Delay tab loading until tab is selected' you should no longer have this issue.

The delay tab loading is something that I need.

Thanks again for reporting this bug. The bug has been fixed in 8c728c96f60e0833451e13b4eecb256e67dc87e5 and a new version (1.8.2) has been submitted to both Chrome and Firefox, so you should have this new version if the next few days.


writeFormBody accepts ranges of pairs

I've needed this, because the formdata is stored in a SQL table and I did not want to construct a AA just to send it directly.

Looks good, merging in. Sorry for the long review delay.

Cherry picked instead of merged due to conflicts.


Script Crashes with torch.compile(model) and Subcommand named tokenize

I've developed a multi-command program using the Click library, which is executed with the command my_program test. The script unexpectedly crashes when both of the following conditions are met:

The script contains the line torch.compile(model).
One of the subcommands is named tokenize.

If I either remove the torch.compile(model) line or rename the tokenize subcommand to something else (e.g., tokenize_), the script runs without any issues.
Environment:

Python version: 3.10
Click version: 8.1.7
Pytorch version: 2.0.1

The code to fully reproduce the error: https://github.com/prenc/click_bug_demo

This is outside the scope of what I can address.   It sounds like there is an issue with torch and the name tokenize. Happy to review a PR, but otherwise this is not something I plan to investigate.


Bump min java version to 1.8

We should bump the min java version to 1.8 in the next release so we can bump jetty version and other deps and other deps which requires java 1.8+.

I vote against it. I wouldn't want to force Java 8 as a requirement on users of Gitblit. Not just so that it can use the latest versions of whatever dependencies it has without reason.
Gitblit is a server application running on stable systems, not a game or desktop app running on the latest, greatest PC. Wide spread Linux server distributions like Debian Jessie or Ubuntu 14.04, for example still come with Java 7.
While there is merit in keeping up-to-date with dependencies, there should still be a good reason why an update is done, if the system currently is working. So updating all dependencies just because, and thereby stepping up the Java requirement, is not a good idea, I think.

I vote for java 8 as a minimal version. Java 9 is already released and Java 7 reached its public end of life long time ago. I just ran an OWASP-dependeny check against GitBlit source and the result was... improvable. From a security point of view, it would be good to have the chance to update to patched versions of dependencies. Stability could be ensured by raising test coverage... and maybe beta/milestone releases?

Given the time between releases, I vote for Java11. It is the actual LTS. Java 8 would have been an option 3 years ago.

Voting for JDK 8 also. It is harder and harder to set-up build environment to use jdk7. Recently You must even keep two versions of ANT as only up to 1.9 allows to use pre 8 JDK what can be a real mess up.

Voting for Java 11.

From Gitblit version 1.10.0 on the minimum Java version is Java 8.


@hurumap/next/Layers

Currently, this component is in HURUmap/Map/Layers. We may create it independently or as part of Map component.

Fixed in #789


Rename ConstraintViolationThrowable to ConstraintViolationException

This PR addresses #482 issue.
In order to make the naming consistent, we should perform the renaming of ConstraintViolationThrowable to ConstraintViolationException.

Implemented in #506.


Installation instructions

The pip install did not work for me "could not find main"
(win 10, visual studio, python 3.6.0)
I had to manually activate virtual environment and do
python -m pip install git+https://github.com/platelminto/parse-torrent-name

Fixed, changed to master rather than main (which does not exist).


Add a blacklist of properties that doesn't need to be saved on localStorage

Do you want to request a feature or report a bug?
feature
What is the current behavior?
we can make a whitelist of properties that we want to save on localStorage with the paths config
What is the expected behavior?
make a blacklist of properties that doesn't need to be saved on localStorage
If this is a feature request, what is motivation or use case for changing the behavior?
ignore unneeded properties of state.js

Sorry, I didn't realize that i can do this with reducer function
plugins: [
  CreatePersistedState({
    key: 'vivo',
    storage: window.sessionStorage,

    reducer: (persistedState) => {
      const stateFilter = Object.assign({}, persistedState)

      delete stateFilter.modal

      return stateFilter
    }
  })
]

Sorry, I didn't realize that i can do this with reducer function
import CreatePersistedState from 'vuex-persistedstate'

...

plugins: [
  CreatePersistedState({
    reducer: (persistedState) => {
      const stateFilter = Object.assign({}, persistedState)

      delete stateFilter.modal

      return stateFilter
    }
  })
]

No problem, that's exactly the purpose of the reducer function!

For anyone who stumbles across the code posted by @waghcwb above, be careful. Using this code deletes items from your main vuex store itself. This effectively means that every time the reducer is ran, it will remove the values directly from the store...

paths: Object.keys(state).filter(m=>m!=="the_key_you_need_blacklisted")


Consider typer instead of click

https://typer.tiangolo.com/

typer is built on top of click. Even though it seems to be more handy, it require lots of changes. So we should rely on click for now. Closed.


Platform tokens

*** Pull request in core and rock
Start using tokens that are specific to a platform.  This allows us to identify a platform based on the token and also deny a platform by revoking its token if needed.

@benwiley great work on this! Love the final solution!


Improve UserControlNumeric

Improve the following for the UserControlNumeric:

Easier switching between e.g. kHz and Hz
Display more digits. 1,234567 MOhm are displayed as "001.234 MOhm". So some informations are hidden.
Update value while editing.
Limit value to allowed range while editing (Important to avoid damage).

Improvements already implemented with commit f77c8d8 (Issue #17: NumericControl added)


SwiftyJSON 4.1.0

Hi!
I'm using the latest version of SwiftyJSON on my project and when I try to add SwiftLocation pod I got this error:
`[!] CocoaPods could not find compatible versions for pod "SwiftyJSON":
In snapshot (Podfile.lock):
SwiftyJSON (= 4.1.0)
In Podfile:
SwiftLocation (~> 3.2.3) was resolved to 3.2.3, which depends on
SwiftyJSON (~> 4.0.0-alpha.1)
SwiftyJSON`

I changed my podfile to used the 4.0.0-alpha version and it worked, but maybe you could switch to the latest version of SwiftyJSON on the next update

I'm working to remove this dependancy.
Follow #205


Permalink doesn't work in embededdashboard

Permalink next to tab in dashboard doesn't work when he is embedded
How to reproduce the bug

Go to your embedded dashboard
Click on permalink next to a tab
See error

Expected results
we see link and we can copy it.
Or we can desactivate permalink
Actual results
No link and a message error
Screenshots

Correction in 2.0.1


.WebView2 generated folder

Is it possible to redirect the generated folder .WebView2 from the app folder to Temp folder ?

Yes, as you can see in the demo initialization section of MainForm.pas
//Setup UserDataFolder for Temp files
TEdgeGoogleMapViewer.RegisterUserDataFolder(System.IOUtils.TPath.GetTempPath+ExtractFileName(ParamStr(0)));

Thanks a lot ! I placed it in FormCreate instead of initialization section. Now it works !


Add FicTracDataInterface

This PR adds FicTracDataInterface to the GUIDE, which passes on the test data.

Looks good 👍


KAFKA-10845; Add VisibleForTesting annotation

More detailed description of your change
There are a lot of comments like "visible for testing" "public for testing", for example:
https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/controller/KafkaController.scala#L95
so introduce a VisibleForTesting annotation.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hi, @guozhangwang  PTAL.


nc-config may report wrong paths when using CMake installation

netCDF Version

$ nc-config --version
netCDF 4.9.2

Environment:

$ lsb_release -a
LSB Version:    :core-4.1-amd64:core-4.1-noarch
Distributor ID: RedHatEnterprise
Description:    Red Hat Enterprise Linux release 8.8 (Ootpa)
Release:        8.8
Codename:       Ootpa
$ cmake --version
cmake version 3.20.2

CMake suite maintained and supported by Kitware (kitware.com/cmake).

Issue:

When using CMake to build and install NetCDF, the nc-config utility may report wrong paths if the installation prefix is only provided at install time, not at configure time:
Correct:
cmake <path/to/source> -DHDF5_DIR=${HDF5_ROOT}/cmake -DCMAKE_INSTALL_PREFIX="${NETCDF_INSTALL_DIR}"
cmake --build . --config Release
cmake --install .

In this case, nc-config reports the correct path (prefix substituted by ${NETCDF_INSTALL_DIR}):
$ nc-config --libs
-L${NETCDF_INSTALL_DIR}/lib64 -lnetcdf

Wrong (but valid CMake installation procedure):
cmake <path/to/source> -DHDF5_DIR=${HDF5_ROOT}/cmake
cmake --build . --config Release
cmake --install . --prefix "${NETCDF_INSTALL_DIR}"

In this case, nc-config reports the system-default installation prefix despite being installed elsewhere:
$ nc-config --libs
-L/usr/local/lib64 -lnetcdf

I suppose there is some templating happening that should not take place at configure/generator/build time but instead be deferred to install time.

Thank you for submitting this; you are correct about what is happening.  I'll take a look to see how this should be handled.

After some investigation, there is no clear way to run configure_file() at install time, outside of some convoluted hacks or scripting, and even that rapidly became overly complicated.  Unfortunately, for the time being, specifying -DCMAKE_INSTALL_PREFIX at configure time is probably going to be the best way, as long as you specify the same argument you pass to cmake --install . --prefix.  As always, PR's welcome!

Indeed, seems tricky and looks like an overlooked requirement on CMake's side. I wasn't aware that this isn't possible. Thanks for looking into this!
For anyone who finds this in the future, I'll leave here a link to the open ticket in the Kitware issue tracker: https://gitlab.kitware.com/cmake/cmake/-/issues/24887

Thank you very much!

Thank you very much, I'll be following that issue! Seeing that we can do this at install time, it would be nice to accommodate users who do that as part of their workflow. Thanks!


Run a nightly Github Actions workflow to monitor tests memory per test (configurable for other metrics too) and lists the slowest 100 tests

Description

Porting the test performance monitor we have in ESMValCore. Github Actions tests OK (py3.8 died of segfaulted worker, common case) :beer:
Small nightly Github Actions workflow that installs and uses pytest-monitor as pytest plugin to output a .pymon sqlite database where performance metrics have been recorded for each pytest test we run (memory, CPU time, usage etc); the wflow runs a wee script I wrote that currently outputs the name of the test and amount of RES memory it used if more than 1000 MB (this can be extended to output a whole lot other metrics). Note that this works only for pytests and not for unittests; also note that I wouldn't turn it on as a mainstream pytest plugin since it slows down the tests (not sure by how much but it's quite visible). The GH Action parses the output via the script and lists the 100 slowest pytest tests from the slowest at the top - this is CPU time.

Before you get started

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[x] 🛠 Tests run successfully
[x] 🛠 All checks below this pull request were successful

It is not clear to me why this is being added. @valeriupredoi, could you please improve the description and possibly add an issue or reference one?
While it is ok to remove items from the checklist that are not applicable, this mechanism is not intended to skip all housekeeping that is deemed cumbersome.

fair do's, Klaus, I will :beer:

done, @bouweandela had previously put this to good use to optimize a few Core tests, I will have a look at the Tool tests too - you OK with this, @zklaus ?

Looking good, thanks!


Share Inception v3 network between FID, KID, Inception Score, and MiFID

🚀 Feature
A single copy of Inception v3 loaded on a GPU and shared between all the metrics which use it.
Motivation
FID, KID, Inception Score, and MiFID all use the same Inception v3 network, which uses 304MB of GPU memory when loaded. If you want to calculate all 4 scores while training, that's 1.2GB GPU memory per GPU. It would be ideal if the metrics could share a single copy of the network to minimize GPU memory usage.
Pitch
Torchmetrics should know if one metric which uses Inception V3 has been loaded and share the copy between the metrics, so when a second metric is created, a second copy of the network is not created.
Alternatives

Documentation / officially support the "unofficial non supported" way to do this -- pass a reference to the network from one metric into the __init__ of another metric. So for example, today you could do:

my_fid = torchmetrics.image.fid.FrechetInceptionDistance()
my_is = torchmetrics.image.inception.InceptionScore(feature=my_fid.inception)

However I am hesitant to rely on this as it depends on the internal details of metrics which are subject to change release to release. I would not know from release notes if this was broken or not.

A "parent metric" which created one copy of InceptionV3 and the metrics you selected to use it with, and calls to update / reset / compuse would in turn pass that down to each individual metric. This could get a bit messy as the __init__ APIs do not exactly line up.

Additional context
It would be doubly awesome if the solution allowed you to share activations as well as the network itself, as currently for each copy I'm calculating the same activations on each copy of the network.
Thanks so much!

Hi @gavrieladler, thanks for raising this issue.
I really think it is a good idea that you proposing as it is a bit silly that the same network is initialized multiple times and the activations are neither reused. Reusing internal computations is already part of MetricCollection but not in this way.
The current solution that I came up with is using a caching system and creating a specialized MetricCollection. This collection called FeatureShare (naming is work in progress) essentially has two features:

Make sure that a single copy of a network is used across all metrics. This is simply done by passing the network of the first metric in the collection to the rest, essentially overwriting.
Secondly, this shared network gets wrapped in a lru_cache meaning that input-output will be cached. If we set it to the max size of the cache to the number of networks in the collection (or a bit above) it should not impact memory that much (I hope)
and the same computation should be reused when the metrics are updated one after another.

Here is some work in progress code
class NetworkCache(Module):
    def __init__(self, network, max_size=100):
        super().__init__()
        self.max_size = max_size
        self.network = lru_cache(maxsize=self.max_size)(network)

    def forward(self, *args, **kwargs):
        return self.network(*args, **kwargs)

class FeatureShare(MetricCollection):
    def __init__(self,
        metrics: Union[Metric, Sequence[Metric], Dict[str, Metric]],
        *additional_metrics: Metric,
        network_names: Union[str, Sequence[str]],
    ):
        super().__init__(metrics=metrics, *additional_metrics)

        if isinstance(network_names, str):
            network_names = [network_names] * len(self)
        else:
            if len(network_names) != len(self):
                raise ValueError('The number of network names should be equal to the number of metrics.')
        shared_net = getattr(getattr(self, list(self.keys())[0]), network_names[0])
        cached_net = NetworkCache(shared_net)
        for (_, metric), network_name in zip(self.items(), network_names):
            setattr(metric, network_name, cached_net)

So very basic analysis:
from torchmetrics.image import FrechetInceptionDistance, InceptionScore, KernelInceptionDistance
from torchmetrics import MetricCollection

fs = FeatureShare([FrechetInceptionDistance(), InceptionScore(), KernelInceptionDistance()], network_names=['inception', 'inception', 'inception'])
mc = MetricCollection([FrechetInceptionDistance(), InceptionScore(), KernelInceptionDistance()])

import time
start = time.time()
for _ in range(10):
    x = torch.randint(255, (1, 3, 64, 64), dtype=torch.uint8)
    mc.update(x, real=True)
end = time.time()
print(end - start)

start = time.time()
for _ in range(10):
    x = torch.randint(255, (1, 3, 64, 64), dtype=torch.uint8)
    fs.update(x, real=True)
end = time.time()
print(end - start)

give me 5.57 and 1.92 respectively for MetricCollection and FeatureShare meaning that sharing the feature is approximately 2.9x times faster which is very much what I would expect when we have three metric that can reuse the same computation.
What do you think about this?

Hi @SkafteNicki,
Thanks for the detailed reply! I think this looks good and is absolutely usable as presented.
One suggestion, and I could go either way: I do think there's an argument to be made that because inception is a class variable name set by the library, the line
fs = FeatureShare([FrechetInceptionDistance(), InceptionScore(), KernelInceptionDistance()], network_names=['inception', 'inception', 'inception'])```

may be unnecessary for the user to specify? Torchmetrics only supports a finite number of metrics and is able to "know" that the selected features should share a network. Only requiring the user to specify

mc = MetricCollection([FrechetInceptionDistance(), InceptionScore(), KernelInceptionDistance()])```
without having to have detailed knowledge of the inner workings of each metrics is a cleaner API.
The code could automatically, when you create any of FID, IS, and KID, add their inception networks to a cache.
Thoughts? Like I said, the solution presented is definitely usable as is.


IPCW Prep for late entry

As it stands, ipcw_prep does not drop the time periods that occurred before late entry. Need to add in a line to drop those observations before the entrance time (should be easy fix)

Fixed in version 0.1.4


Russian localisation

Hi
Thanks for your library.
I hadn't seen my native language (russian) and added it to SwiftDate.
Unfortunately, for this language not enough to add string file. It also necessary add support multiple type of words ending (depending on the number) in the code.
I hope I added my code to original with minimal affecting.

Great work! Waiting it to be merged

Hi, thank you for your work.
I'm a bit hesitant because we want to make the lib compatible with Linux where the .strings and bundle does not exist. I think it could be the opportunity to introduce a custom localization schema but I need to think more about it.
Let me think some days, I'll post discussion on this topic.

@malcommac any updates on this?


Client closing randomly during play fatal error code

A fatal error has been detected by the Java Runtime Environment:
#
#  EXCEPTION_ACCESS_VIOLATION (0xc0000005) at pc=0x03d9c70d, pid=15208, tid=9740
#
# JRE version: OpenJDK Runtime Environment (11.0.4+11) (build 11.0.4+11)
# Java VM: OpenJDK Server VM (11.0.4+11, mixed mode, emulated-client, g1 gc, windows-x86)
# Problematic frame:
# J 4031 c1 sun.awt.SunToolkit.getAppContext(Ljava/lang/Object;)Lsun/awt/AppContext<EMAIL_ADDRESS>(42 bytes) @ 0x03d9c70d [0x03d9c6e0+0x0000002d]
#
# No core dump will be written. Minidumps are not enabled by default on client versions of Windows

please give me some help with this guys

As noted in #16341, it seems like these crashes are all coming from Ryzen processors. Figured I'd repost this observation here since this is the main tracking issue

Possibly related, I am also experiencing similar crashes in other applications. Discord, Chrome tabs, and VRChat have all been spontaneously crashing. However, I managed to get an error message back from Chrome: ERROR_ACCESS_VIOLATION. Discord offered no useful logs, and I'm looking into VRChat's logs right now, but I'm thinking we may be looking at a hardware issue here since the error is so similar.

Updated BIOS and reverted GPU driver to a more stable version, haven't crashed since then so that may have resolved the issue. I recommend anyone running into this to do the same

Is there anything I can try because my issue was a duplicate and closed but there is nothing on this page that is helpful. I've tried updating drivers, uninstalling java / reinstalling java and runelite and runescape, and finally I reinstalled windows but my game still crashing every 20 min and it is making the game unplayable. Any help would be appreciated .

Here is a list of things I tried that did not work. Maybe you can retry some of them, or at the very least it's more info.

Still crashing, usually wont crash for a while when I first start my computer but then once it crashes, it keeps crashing about every 10 min

Here is a list of most of my PC's components, along with my most recent crash, GC, client, and launcher logs.
Perhaps there is some other common factor here.
System.txt
client.log
launcher.log
jvm_crash_pid_11312.log
gc (3).log
I hope this information helps to solve the issue, and if there is anything else I can do to help expedite this process, let me know.

Probably not useful for most here, but I find I only experience this if I'm trying to record my gameplay with OBS. When I stop recording and close OBS, the issue goes away. Found this through googling the error code from windows event logs, so thought I'd say here anyhow in case that gives anyone any ideas. 🤞

jvm_crash_pid_13196.log

Here is all my logs and my last 5 crashes. Please if anyone is reading this, please help. It crashes at the worst times it makes me so sad.
launcher.log
client.log
jvm_crash_pid_1136.log
jvm_crash_pid_8780.log
jvm_crash_pid_10892.log
jvm_crash_pid_13560.log
jvm_crash_pid_13976.log
jvm_crash_pid_1136.log

I've been getting this crash periodically as well. I'm using an Intel i7-12700k with 32 GB of RAM. I've noticed the crash more consistently with the Plank Sack plugin installed and in use.
jvm_crash_pid_21816.log

I had previously through this crash to be related to either of the GPU plugins (GPU, 117HD), but now I have had the crash happen with RuneLite in Safe mode, which has these plugins disabled (afaik).
Safe mode logs (debug): jvm_crash_pid_1056.log
clientedited.txt
launcher.log
(if any of these debugging logs contain concerning personal info please remove)
I recommend anyone having this issue run a couple passes of MemTest86 to ensure their RAM is functioning properly.
RAM issues can have all sorts of niche impacts, and it is a comforting thing to rule out.
I also ran a Prime95 + Furmark test in hope to rule out GPU/CPU faults as well.

I had previously through this crash to be related to either of the GPU plugins (GPU, 117HD), but now I have had the crash happen with RuneLite in Safe mode, which has these plugins disabled (afaik).
Safe mode logs (debug): jvm_crash_pid_1056.log
clientedited.txt
launcher.log
(if any of these debugging logs contain concerning personal info please remove)
I recommend anyone having this issue run a couple passes of MemTest86 to ensure their RAM is functioning properly.
RAM issues can have all sorts of niche impacts, and it is a comforting thing to rule out.
I also ran a Prime95 + Furmark test in hope to rule out GPU/CPU faults as well.

Did you find a solution? I also imagine it's due to some sort of memory optimization/pc optimization. I have a Ryzen 9 5900x and 32GB RAM but constantly crash...

I have found a solution for my crashes and also a workaround.
After 30 days without a crash I feel safe to blame the BIOS feature "Core Performance Boost" (CPB).
After turning CPB OFF my RuneLite crashes on both Windows 11 and Linux have stopped entirely.
CPB is ON by default for my mobo (Asus). Turning off CPB is a fair performance loss for me, so keep that in mind.
Alternatively, I was able to use the 32bit RuneLite client stably for over a month with CPB ON. However I still experienced rare stability issues with other programs likely due to CPB.
So CPB was the cause for me, turning it OFF has stopped my crashes.
If you don't want to do that you can use the 32bit RuneLite but you will probably still have subtle instability.
Hope this helps, good luck!

This worked for me:

go into BIOS
find the setting called "CPU Core ratio"
manually set it to desired speed(for example 39.00 = 3.9ghz) or select "Sync all cores"
press f10 to save and exit
play video game
I have a Ryzen 7 5800x CPU and my 64 bit client used to crash daily, now I have played for 2 days without a single crash

Thanks a ton, I'll look into this
On Tue, Nov 14, 2023 at 10:31 AM Niukkis @.***> wrote:

This worked for me:

go into BIOS
find the setting called "CPU Core ratio"
manually set it to desired speed(for example 39.00 = 3.9ghz) or
select "Sync all cores"
press f10 to save and exit
play video game
I have a Ryzen 7 5800x CPU and my 64 bit client used to crash daily,
now I have played for 2 days without a single crash

—
Reply to this email directly, view it on GitHub
https://github.com/runelite/runelite/issues/10628#issuecomment-1810462540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYMWJJQODH2IA2BO262GPD3YEOFFPAVCNFSM4KJX4NVKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBRGA2DMMRVGQYA
.
You are receiving this because you commented.Message ID:
@.***>

Some users with Intel CPUs have reported that disabling Intel SpeedStep (EIST) in their BIOS has resolved the issue for them.
AFAIK SpeedStep enhances C1E (a powersaving state when your CPU is idling) further to not only have the idle and load states, but also states in between for e.g. low load.

This worked for me:

go into BIOS
find the setting called "CPU Core ratio"
manually set it to desired speed(for example 39.00 = 3.9ghz) or select "Sync all cores"
press f10 to save and exit
play video game

I have a Ryzen 7 5800x CPU and my 64 bit client used to crash daily, now I have played for 2 days without a single crash.
Update: 2 weeks with no crashes.

Can I ask how you changed the ‘CPU Core Ratio’ value? Mine is just stuck on Auto and I can’t change it.

Started experiencing this on Linux today. Here's the crash log.
jvm_crash_pid_2702.log

Started experiencing this on Linux today. Here's the crash log. jvm_crash_pid_2702.log

I believe the issue here is different from the one you're experiencing.

Same crash, though I have a sneaking suspicion that it's the CPU that is the issue. I've had similar "ACCESS_VIOLATION" errors with the 13gen and now 14gen as well.
jvm_crash_pid_15696.log

Same crash, though I have a sneaking suspicion that it's the CPU that is the issue. I've had similar "ACCESS_VIOLATION" errors with the Intel 13gen and now 14gen as well.
jvm_crash_pid_15696.log

As has been mentioned elsewhere in this thread, some Intel users have found that disabling SpeedStep in BIOS  has resolved this crash for them.

Same crash, though I have a sneaking suspicion that it's the CPU that is the issue. I've had similar "ACCESS_VIOLATION" errors with the Intel 13gen and now 14gen as well.
jvm_crash_pid_15696.log

As has been mentioned elsewhere in this thread, some Intel users have found that disabling SpeedStep in BIOS has resolved this crash for them.

What about AMD users?

Same crash, though I have a sneaking suspicion that it's the CPU that is the issue. I've had similar "ACCESS_VIOLATION" errors with the Intel 13gen and now 14gen as well.
jvm_crash_pid_15696.log

As has been mentioned elsewhere in this thread, some Intel users have found that disabling SpeedStep in BIOS has resolved this crash for them.

What about AMD users?

The relevant BIOS setting to disable for AMD users would be "Core Performance Boost".

Same crash, though I have a sneaking suspicion that it's the CPU that is the issue. I've had similar "ACCESS_VIOLATION" errors with the Intel 13gen and now 14gen as well.
jvm_crash_pid_15696.log

As has been mentioned elsewhere in this thread, some Intel users have found that disabling SpeedStep in BIOS has resolved this crash for them.

What about AMD users?

CPB/PBO are similar options for AMD.

Same crash, though I have a sneaking suspicion that it's the CPU that is the issue. I've had similar "ACCESS_VIOLATION" errors with the Intel 13gen and now 14gen as well.
jvm_crash_pid_15696.log

As has been mentioned elsewhere in this thread, some Intel users have found that disabling SpeedStep in BIOS has resolved this crash for them.

What about AMD users?

The relevant BIOS setting to disable for AMD users would be "Core Performance Boost".

Same crash, though I have a sneaking suspicion that it's the CPU that is the issue. I've had similar "ACCESS_VIOLATION" errors with the Intel 13gen and now 14gen as well.
jvm_crash_pid_15696.log

As has been mentioned elsewhere in this thread, some Intel users have found that disabling SpeedStep in BIOS has resolved this crash for them.

What about AMD users?

CPB/PBO are similar options for AMD.

Would either of you happen to know how to disable it? Mine is greyed out set to ‘Auto’ :(

Same crash, though I have a sneaking suspicion that it's the CPU that is the issue. I've had similar "ACCESS_VIOLATION" errors with the Intel 13gen and now 14gen as well.
jvm_crash_pid_15696.log

As has been mentioned elsewhere in this thread, some Intel users have found that disabling SpeedStep in BIOS has resolved this crash for them.

This did not solve the issue on my end


Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.5.1, expected version is 1.1.16.[BUG]

I created a flutter java app and imported this project.
But can't run the project because it shows:
Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 1.5.1, expected version is 1.1.16.
Now when I am adding  ext.kotlin_version = '1.6.0'.
Still the same error persists. I don't understand because it's not a kotlin project but a java one. Then also it's shows this error.
How to solve it?

This looks like some Gradle caching issue. Can you try updating the Gradle version?

`buildscript {
repositories {
ext.kotlin_version = '1.6.10'
google()
mavenCentral()
maven { url 'https://jitpack.io' }
}
dependencies {
    classpath 'com.android.tools.build:gradle:7.0.4'
}

}
allprojects {
repositories {
google()
mavenCentral()
}
}
rootProject.buildDir = '../build'
subprojects {
project.buildDir = "${rootProject.buildDir}/${project.name}"
project.evaluationDependsOn(':app')
}
task clean(type: Delete) {
delete rootProject.buildDir
}`
above is my build.gradle file
gradel is already 7.0.4 (classpath 'com.android.tools.build:gradle:7.0.4')
what else can i do?

@prashantspandey Please use the latest version of the UIKit that should solve the issue. http://pub.dev/packages/agora_uikit
if you're still facing the issue feel free to reopen the issue.


some changes in meta tags

1.changed the preview image(will works after the merger in main).
2.changed the meta tags related to the image
3.added meta teg description.

Hey, @IgorFromUA, please resolve the conflict


Add domain vuelib.is-local.org

Add domain vuelib.is-local.org
Requirements

[1] You have completed your website.
[1] The website is reachable.
[1] The CNAME record doesn't contain https:// or /.
[1] There is sufficient information at the owner field.
[1] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description
The domain will be use for document of some libraries creted for Vue.JS community.
Link to Website
It's not published yet. but thats like https://ja.vuejs.org/api/application.html#createapp

I have read the CLA Document and I hereby sign the CLA


spam-reporting add-in not showing on web Outlook

In documentation we can see spam-reporting add-in is Supported on Outlook on the web but it is not showing on my web and this is working correctly on my desktop
my manifest

live manifest

on Desktop


Cancelling because the item is not on the shelf

Need to create a notification template

See also #119

Steps in Folio:

App: Users
Enter identifier (name, barcode, etc) in search box
Click on users account to open up a new pane
Expand the "requests" menu in the patrons account
Click on "open requests" to bring up a new window
Click on the request that needs cancelled to open a new pane
Open the "actions" drop down menu and select "cancel request"
Select "Item not Available" as the reason for cancellation
This does not mark the item as unavailable though, so this will need to be done in the item record itself

Also, new functionality that will be helpful:
Closed requests can be duplicated as new, open requests by:

Click "closed requests"
Select the appropriate request
In the "actions" drop down, select "duplicate"
Click "save & close"


Add option to enable full background mp4 video

This is a design trend for 2015. Full background videos on the intro page. Nice to have!

PleaseWait isn't meant to be open for too long, so using bandwidth to load and play a video which could instead be used to load your app may not be the best route to take. That said, there are libraries like BigVideo.js which are geared towards doing just what you're looking for, so I'd recommend that instead.


waitForText() causes NullPointerException

From<EMAIL_ADDRESS>on June 05, 2014 11:44:58
I can't figure out how to reopen an older issue, but I am hitting this issue in the current version (5.1) of Robotium.
I get the NPE from clickOnText and waitForText, and both NPEs seem to come from the Searcher class. https://code.google.com/p/robotium/issues/detail?id=403
Original issue: http://code.google.com/p/robotium/issues/detail?id=612

From<EMAIL_ADDRESS>on June 06, 2014 03:11:33
Thanks for reporting this. Can you please paste the log?
Owner<EMAIL_ADDRESS>

From<EMAIL_ADDRESS>on June 13, 2014 09:34:05
at com.robotium.solo.Searcher.searchFor(Searcher.java:128)
at com.robotium.solo.Waiter.waitForText(Waiter.java:479)
at com.robotium.solo.Waiter.waitForText(Waiter.java:418)
at com.robotium.solo.Waiter.waitForText(Waiter.java:390)
at com.robotium.solo.Solo.waitForText(Solo.java:277)
at BVT.test_BVT_Settings(BVT.java:145)
at java.lang.reflect.Method.invokeNative(Native Method)
at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
at android.test.ActivityInstrumentationTestCase2.runTest(ActivityInstrumentationTestCase2.java:192)
at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1738)
Caused by: java.lang.NullPointerException

From<EMAIL_ADDRESS>on June 15, 2014 07:07:53
This seems to be the stacktrace of an older version of Robotium. Please download 5.1 and check if you can reproduce the issue there.

From<EMAIL_ADDRESS>on June 15, 2014 08:42:48
Wow... I'm sorry for bugging you with this. You're right. I didn't update my build.gradle files. I was using 5.0.1

From<EMAIL_ADDRESS>on June 15, 2014 23:20:19
No problem. Let us know if encounter any issues with 5.1.
Status: Invalid


Seed Grafana failed to load Extensions dashboard

How to categorize this issue?

/area monitoring
/kind bug
/priority normal
What happened:
Seed Grafana failed to load "Extensions" dashboard (see screenshot). I was able to reproduce it on multiple seed clusters. It worked occasionally when I reloaded the page several times.

How to reproduce it (as minimally and precisely as possible):

Open Seed Grafana via grafana Ingress in garden namespace.
Click on Extensions dashboard.

/cc @wyb1
Environment:

Gardener version: v1.12.3

Seems to be related to the datasource. Whenever the datasource is loki, the CPU and Memory usage panels are broken.

Whenever the datasource is changed to prometheus and the label is adjusted then the CPU and memory panels work but the logs from loki no longer exist.

@Kristian-ZH any suggestions on how to get loki working with the Prometheus labels?

ping @Kristian-ZH @vlvasilev

ping @Kristian-ZH @vlvasilev

I am going to check it, but on the first look it could be related to known issue where we hit the inodes limit of the disk with Loki chunks. This causes Loki to break quite often

I am going to check it, but on the first look it could be related to known issue where we hit the inodes limit of the disk with Loki chunks. This causes Loki to break quite often

Hmm i checked and can not see any issues at the moment. We had an issue before, where the Loki was constantly scaled-up/down because of the misconfigured hvpa. This issue was fixed and currently the dashboards are looking fine.
Can you please recheck if everything is working as expected @timuthy @wyb1 ?

Hmm i checked and can not see any issues at the moment. We had an issue before, where the Loki was constantly scaled-up/down because of the misconfigured hvpa. This issue was fixed and currently the dashboards are looking fine.
Can you please recheck if everything is working as expected @timuthy @wyb1 ?

I don't experience the issue at the moment, either. Thanks for checking @Kristian-ZH.
/close

I don't experience the issue at the moment, either. Thanks for checking @Kristian-ZH.
/close


[Bug] Importing Legacy Backup Doesn't Overwrite

PLEASE READ THIS
I acknowledge that:

I have updated to the latest version of the app

Device information

Tachiyomi version: 2.4.2
Android phone & version: Samsung Galaxy S8 running Android 9

Steps to reproduce

Login to MD, add AniList to trackers
Sync MangaDex manga to Neko
Import a generated version of legacy backup (snippet under additional)

Expected behavior
Chapters marked read in legacy backup show as read in the app.
Actual behavior
Chapters read status unchanged.
Other details
Snippet of the backup (I can provide the full file if needed):
{
  "version": 2,
  "mangas": [
    {
      "manga": [
        "/title/3bb0279f-a01d-4aa4-93e4-305800f4b83e",
        "Akatsuki no Yona",
<PHONE_NUMBER>021220255,
        0,
        0
      ],
      "chapters": [
        {
          "u": "/chapter/519b36db-e3fe-4caa-ab44-141176d35fa9",
          "r": 1
        },
        ...
      ],
      "tracking": []
    },
    ...
  ],
  "categories": []
}

I did clear the app's data from the Android Settings and only logged in and restored the backup (without AniList configured) and it correctly flags the chapters as read.
The chapters it had as read didn't line up with AniList's values, either, and I didn't explicitly connect a tracker to the manga.

Legacy backup support has been removed in 2.5.0 so closing this


Export template for Android is broken

Godot version:
master, 3.1 stable
OS/device including version:
Antergos (Arch Linux)
Issue description:
When I compile godot from source for x11 and install the export templates from https://hugo.pro/projects/godot-builds/ I can't install the exported android package. However using the 3.1 stable download and the official export templates, it installs correctly.
Steps to reproduce:
Download https://godotengine.org/download/linux and install export templates. The exported .apk will install and run correctly.
Compile from master and install export templates from https://hugo.pro/projects/godot-builds/. The exported .apk will fail to install.

Thanks for the report! But from the sounds of it, you are saying that everything in Godot works fine, but @Calinou's automatic builds are broken? This is not the proper place to report issues with third party builds.
@Calinou Care to take a look at this?

A priori you can't mix and match a self-compiled editor binary with thirdparty export templates, unless you're 100% positive that they're built from the same commit.
If you use the templates from @Calinou, please also use the editor build that goes with it.

Exporting with a self compiled editor (macOS 10.14.4) and self compiled export templates results in a parser error when trying to install the .apk.
When I use the editor from the unofficial builds and install the export templates with the same commit, it works. So nothing is broken with the export templates, I was just using different commits.


Containerize services & orchestrate with docker-compose

Is this still worked on? Would be a nice addition to the project.

Is this still worked on? Would be a nice addition to the project.

hey, i've got a lot going on outside of this project at the moment (recently hired as a sr. software engineer at a startup, working towards moving into a new apartment, etc.) so i'm pretty out of time
because of that, i don't think i'll be able to get around to this any time soon; i'm definitely open to having people take over this pr/the project as a whole if anyone shows the potential to be able to manage it
it would indeed be a pretty cool addition. i'd also like to make the server stateless and store ephemeral data (e.g. player sessions) in redis, but that would take quite a bit of time i don't currently have

maybe i can revive this

being continued in #320


[NEW] Add BlackHoodie Virtual 2022

Name: BlackHoodie Virtual 2022
Website: https://blackhoodie.re/Blackhoodie-Virtual-2022/
Date(s): November 25th, 2022
Location: Online
Twitter Handle: @Blackhoodie_RE
Free: Y (Woman ONLY)

Thanks! ✅


Build-BicepNetParams does not need a -NoRestore parameter

Describe the bug
The -NoRestore parameter is unnecessary because you don't need to restore anything on .bicepparams files.

You don't need it today, but there's a feature in vNext where you can reference a modules from a registry directly in the bicepparam file so it might be needed in the future and it doesn't hurt to keep it in.