Cannot debug unit tests - Failed to start debugger: "System.NullReferenceException: Object reference not set to an instance of an object

Issue Description
The project I am working on is based on .net 5.0. It uses NUnit (3.12.0) and NUnit3TestAdapter (3.15.1) packages. This issue happens when I try to debug generic NUnit unit tests.  It started recently and happens every time I try to click "Debug Test" link above generic unit test. Simple non-generic tests with and without parameters are working fine. I am using portable Visual Studio Code.
When I try to run such tests using "Run test" link, my tests won't run.

Steps to Reproduce

Create simple test class.
Create generic unit test using NUnit (look at example screenshots).
Try to debug the test by clicking "Debug test"

Expected Behavior
I should be able to debug the test
Actual Behavior
I cannot debug the test
Logs
OmniSharp log
[fail]: OmniSharp.Stdio.Host
        ************  Response (4029.1222ms) ************
{
  "Request_seq": 284,
  "Command": "/v2/debugtest/getstartinfo",
  "Running": true,
  "Success": false,
  "Message": "\"System.NullReferenceException: Object reference not set to an instance of an object\\n  at OmniSharp.DotNetTest.VSTestManager.DebugGetStartInfoAsync (System.String[] methodNames, System.String runSettings, System.String testFrameworkName, System.String targetFrameworkVersion, System.Threading.CancellationToken cancellationToken) [0x00113] in <562a322676604b9ba06bd663e080b5d2>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].GetFirstNotEmptyResponseFromHandlers (OmniSharp.Endpoint.Exports.ExportHandler`2[TRequest,TResponse][] handlers, TRequest request) [0x00099] in <97ecc425804b41999f2fc93560b238f8>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].HandleRequestForLanguage (System.String language, TRequest request, OmniSharp.Protocol.RequestPacket packet) [0x00163] in <97ecc425804b41999f2fc93560b238f8>:0 \\n  at OmniSharp.Endpoint.EndpointHandler`2[TRequest,TResponse].Process (OmniSharp.Protocol.RequestPacket packet, OmniSharp.Endpoint.LanguageModel model, Newtonsoft.Json.Linq.JToken requestObject) [0x0024b] in <97ecc425804b41999f2fc93560b238f8>:0 \\n  at OmniSharp.Stdio.Host.HandleRequest (System.String json, Microsoft.Extensions.Logging.ILogger logger) [0x000f3] in <08ec318a4de440fbb214735376fa2310>:0 \"",
  "Body": null,
  "Seq": 2316,
  "Type": "response"
}
C# log
No logs here.
.NET Test Log
NUnit Adapter <IP_ADDRESS>: Test discovery starting
NUnit Adapter <IP_ADDRESS>: Test discovery complete
System.ArgumentException: The specified argument cannot be empty. (Parameter 'tests')
   at Microsoft.VisualStudio.TestPlatform.ObjectModel.ValidateArg.NotNullOrEmpty[T](IEnumerable`1 arg, String parameterName)
   at Microsoft.VisualStudio.TestPlatform.ObjectModel.Client.TestRunCriteria..ctor(IEnumerable`1 tests, Int64 frequencyOfRunStatsChangeEvent, Boolean keepAlive, String testSettings, TimeSpan runStatsChangeEventTimeout, ITestHostLauncher testHostLauncher, TestSessionInfo testSessionInfo, Boolean debugEnabledForTestSession)
   at Microsoft.VisualStudio.TestPlatform.CommandLine.TestPlatformHelpers.TestRequestManager.RunTests(TestRunRequestPayload testRunRequestPayload, ITestHostLauncher testHostLauncher, ITestRunEventsRegistrar testRunEventsRegistrar, ProtocolConfig protocolConfig)
   at Microsoft.VisualStudio.TestPlatform.Client.DesignMode.DesignModeClient.<>c__DisplayClass23_0.b__0()
Environment information
VSCode version: 1.61.0-insider (1.60.2 too)
C# Extension: 1.23.15
Mono Information
        OmniSharp using built-in mono
Dotnet Information
.NET SDK (reflecting any global.json):
 Version:   5.0.203
 Commit:    383637d63f
Runtime Environment:
OS Name:     pop
OS Version:  21.04
OS Platform: Linux
RID:         linux-x64
Base Path:   [omitted]/5.0.203/
Host (useful for support):
Version: 5.0.6
Commit:  478b2f8c0e
.NET SDKs installed:
3.1.115 [omitted]
5.0.203 [omitted]
.NET runtimes installed:
Microsoft.AspNetCore.App 3.1.15 [omitted]
Microsoft.AspNetCore.App 5.0.6 [omitted]
Microsoft.NETCore.App 3.1.15 [omitted]
Microsoft.NETCore.App 5.0.6 [omitted]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

csharp
ms-dotnettools
1.23.15

vsc-community-material-theme
Equinusocio
1.4.4

The reason for this is that omnisharp does not consider any Attributtes to run or debug Nunit Tests:
src/OmniSharp.DotNetTest/TestFrameworks/NUnitTestFramework.cs
src/OmniSharp.DotNetTest/TestMethodPropertyProvider.cs
The Test Framework is "method-driven". The vscode extension tries to run and debug with the "method name", because that is how the codeLensProvider is implemented, instead, it should use the fully qualified name of the test.
fyi, you can list the tests with fully qualified names with:
dotnet vstest <path/to/test-project.dll> /ListFullyQualifiedTests /ListTestsTargetPath:test.txt

If you want to keep using omnisharp with nunit the way it is, I would recommend you to remove TestCase Attributes.. and create a Test method for each one of them.


chore(deps): downgrade eslint due to build error

Related Ticket(s)
None.
Description
Our dependency bot updated eslint to v8, which causes problems when running the server. This PR keeps eslint on version 8.6.0 like our current Web Components package has it to avoid any problems.
Changelog
Changed

downgraded eslint to 8.6.0

Closing, fixed in https://github.com/carbon-design-system/carbon-for-ibm-dotcom-web-components-template/pull/91.


How I can open the response in a browser tab?

Since the last update I can not see where I do.
Thanks.

Duplicate of #512


Respond by rebuilding just the modified file(s)

This might be related to #118, not sure. I don't really know how sphinx-autobuild works at the low level. Let me describe my scenario and someone more knowledgeable can decide.
I'm trying to help reduce dangling references in the main Python docs. I run make htmllive in the Doc directory and it does its thing. I can view the file I'm working on to see my changes in near realtime. As I work on a particular .rst file, it would be nice if, whenever I saved it, sphinx only reprocessed that one file. I'm running sphinx in nitpicky mode, and it appears to reprocess all files under the Doc directory, spewing thousands of nitpick warnings. Consequently, I can't easily see if the changes I've made to that one file have had a positive effect on the number of warnings for that file. That extra reprocessing seems to also make it take much longer for the generated HTML file to be available for viewing through the web.

Just to put it into perspective: I have a project where a full documentation build takes 30 minutes!
When automated rebuilding is triggered by incremental changes, it does not scale unless it rebuilds only the incremental changes.

[x] [DANGER ZONE] Yes, I am aware that a partial build may be inconsistent. I know what I do.

Big projects would benefit a lot from shorter feedback cycles. As a work-around, I have to add the desired file paths into conf.py include_patterns or edit all toctrees and remove all links that are not relevant for what I am currently working on.

This is the default (incremental builds not working in Python is a different issue due to last_updated_time)


[BUG] graphics setting isnt working

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[X] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
i started roblox and my settings are reseted sooo graphics setting that make 21 levels that doesnt working i deleted bloxtrap and roblox than i dowlonded again but its still not working
Bloxstrap Log
N/A

the fast flag got removed

Duplicate of #2812


drivers: pwm: implement fake-pwm driver

Based on fake can and fake gpio driver, this PR implements a fake pwm driver to be used with native_sim target for unit testing purposes
It only implements basic PWM functionality (no capture)

this looks like a great addition, @anangl can you please help review/approve?

Have you actually tried to build it for native_sim? Apparently CI did not do it as no build errors were reported.

I am actually using the fake_pwm module in a project I have.
When I try running the test I indeed see that it is not run. It seems to be filtered out. I'm not sure exactly how I can make the test run seeing the testcase.yaml. Should I add a new test just for native_sim or is there a way to make the filter accept native_sim ?

@nono313 looks like you now have CI errors :)

Thanks for the notice, I started a discussion about it on Discord in #twister


Travis: version 3.3 missing?

I've created a PR to add support for arangodb 3.3 for loopback-connector-arangodb connector.
Travis fail with
$ ./test/travis/setup_arangodb.sh
./setup_arangodb_3.3.sh: line 1: 404: command not found

https://www.arangodb.com/repositories/travisCI/

Hi,
we recommend using ArangoDB docker for travisCI tests.

Migrated! Thanks!


WIP: Replace organisation secrets with organisation variables

Reference: https://app.zenhub.com/workspaces/the-outlaws-6193fe815d79fc0011e741b1/issues/gh/energinet-datahub/team-the-outlaws/1199

Codecov Report

Merging #204 (a564b02) into main (cd2e9d5) will increase coverage by 2.04%.
The diff coverage is n/a.

:exclamation: Current head a564b02 differs from pull request most recent head 5540cb6. Consider uploading reports for the commit 5540cb6 to get more accurate results

@@            Coverage Diff             @@
##             main     #204      +/-   ##
==========================================
+ Coverage   71.08%   73.13%   +2.04%
==========================================
  Files         125       56      -69
  Lines        2639     1366    -1273
  Branches      288      135     -153
==========================================
- Hits         1876      999     -877
+ Misses        688      321     -367
+ Partials       75       46      -29

Flag
Coverage Δ

appcommon
?

jsonserialization
?

logging
?

schemavalidation
?

testcommon
73.13% <ø> (-0.15%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 70 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Auto indentation doesn't work (C#, mac)

VSCode Version: Code 1.11.2 (6eaebe3b9c70406d67c97779468c324a7a95db0e, 2017-04-13T07:56:42.517Z)
OS Version: Darwin x64 15.0.0
Extensions:

Extension
Author
Version

csharp
ms-vscode
1.9.0

mono-debug
ms-vscode
0.15.5

Auto indentation doesn't work after update. Code formatting hotkey sets it, but no real time auto indentation. Tried to reinstall, no result. Settings.json is default.
http://imgur.com/zdpjV3n

Which update are you referring to? C# for VS Code 1.9? (That update is a couple of weeks old, which is why I'm asking to clarify.)

@DustinCampbell, yes.

Strange thing is that indentation work with {}, but not without them.

@IvanSorokin: I'm not sure what you meant by that. Could you share some repro steps?

@DustinCampbell, I recorded this situation https://www.youtube.com/watch?v=pKxdup7cLgY

I'm not sure we've ever had that level of auto-indentation in VS Code. Are you sure it worked before the update?

@DustinCampbell, yes

I just downloaded an older version of C# for VS Code from our GitHub releases page (1.6), and the indentation is the same.
@jrieken: Do you know if anything has changed around indentation in VS Code? AFAIK, we do't do anything special to support it in C# for VS Code. Honestly, I'm not sure there's an indentation API that we could support.

Out of curiosity, is the problem that you need to turn on set the "edit.FormatOnType": true setting in VS Code? I don't think we've ever handled indentation in this way, but turning that option on causes formatting to occur when typing a ; or }

@jrieken: Do you know if anything has changed around indentation in VS Code? AFAIK, we do't do anything special to support it in C# for VS Code. Honestly, I'm not sure there's an indentation API that we could support.

Not that I know of. @alexandrudima will know

@DustinCampbell Thanks for workaround, it works! It seems that problem was with my settings.

Glad to hear this worked for you. I'd love to be able to add proper indentation support. Hopefully, we'll have a way to add custom language support for indentation in VS Code at some point to allow us to have parity with VS. However, until then, I don't think there's much we can do.
@jrieken, @alexandrudima: Are there any issues on the VS Code I should track for this issue? I found https://github.com/Microsoft/vscode/issues/9241 and https://github.com/Microsoft/vscode/issues/16998.


Missing libraries to compile qtgui applications in linux

inside the container stateoftheartio/qt6:6.6-gcc-aqt a suitable opengl library is missing
error from a cmake within the container:
-- Could NOT find OpenGL (missing: OPENGL_opengl_LIBRARY OPENGL_glx_LIBRARY OPENGL_INCLUDE_DIR)
-- Could NOT find WrapOpenGL (missing: WrapOpenGL_FOUND)
CMake Warning at /opt/Qt/Tools/CMake/share/cmake-3.27/Modules/CMakeFindDependencyMacro.cmake:76 (find_package):
  Found package configuration file:
    /opt/Qt/6.6.1/gcc_64/lib/cmake/Qt6Gui/Qt6GuiConfig.cmake
  but it set Qt6Gui_FOUND to FALSE so package "Qt6Gui" is considered to be
  NOT FOUND.  Reason given by package:
  Qt6Gui could not be found because dependency WrapOpenGL could not be found.
  Configuring with --debug-find-pkg=WrapOpenGL might reveal details why the
  package was not found.
  Configuring with -DQT_DEBUG_FIND_PACKAGE=ON will print the values of some
  of the path variables that find_package uses to try and find the package.
Call Stack (most recent call first):
  /opt/Qt/6.6.1/gcc_64/lib/cmake/Qt6/QtPublicDependencyHelpers.cmake:111 (find_dependency)
  /opt/Qt/6.6.1/gcc_64/lib/cmake/Qt6Quick/Qt6QuickDependencies.cmake:39 (_qt_internal_find_qt_dependencies)
  /opt/Qt/6.6.1/gcc_64/lib/cmake/Qt6Quick/Qt6QuickConfig.cmake:40 (include)
  /opt/Qt/6.6.1/gcc_64/lib/cmake/Qt6/Qt6Config.cmake:156 (find_package)
  src/client/CMakeLists.txt:2 (find_package)

I currently run:
sudo apt-get update
sudo apt-get install -y libgl1-mesa-dev

before using the container, but i would think that it should be part of the container

Oh, so you need to check #1 about the solutions.
If you need this additional lib for your build environment - then I would recommend to build a child container on top of this one. Otherwise the containers will have all sorts of additional libraries, will weight around a ton and will carry something that other people doesn't need. Multiple folks are just building everything from scratch to completely control the build environment and not rely on the dist libs, so this addition will cause them an arm and a leg to remove to ensure the env is clean.
So there are multiple angles and I prefer to stay slim rather then install every possible library I don't know about and make the image too fat. Hopefully you see the idea.

OK I understand minimalist aspect. But with that argument you could also remove the Qt6Gui package as well, since it has a hard dependency to any opengl package, this way you would save another bunch of megabytes. And instead keep the aqt binary this way we could easily use the same aqt you are using for your builds and extend the containers to our needs (or at least keep the AQT_VERSION variable alive so we could reuse it).

Yeah, you see those are a bit different buckets:

Why Qt6Gui package is here - because aqt (and qt installer) installs it by default. And it has the required libraries installed into the system, not the dev packages - because linking != compiling and does not require headers.
Unfortunately those additional dev packages installation can't be automated - I mean for libs it's relatively simple to find which ones are needed, but for dev packages - it depends. So I'd say it's on developer who needs those packages to install to fulfill the dependencies of the particular application.
 So I'm just showing a way to automate it for everyone and make a relatively similar interface across the platforms - here is value for me, not in installing some particular deps for particular apps.
Keeping AQT binaries means keeping all the python packages it depends on - and since it's not the necessary thing I'm removing it to keep the images clean from contamination. It's much harder to identify and remove something, then install something in addition - and btw it saves some space in the image which is a good benefit.

So minimal common images are a solid base for everyone. For sure there are some points to discuss - but I'm looking for constructive proposals or changes that are aligned with general vision and would be useful by everyone. For example: maybe leave AQT installed for root user and not contaminate the unprivileged build user environment, but as for now there was no PR's or proposals that would serve the particular goal, so hopefully you see the reasons behind my decisions and what I'm talking about in general.


Fix : (Security) Migitate Path Traversal Bug

bug fixes

DCO is missed.

@yeasy Fixed, Please Merge now *!

@yeasy you can merge now...

@yeasy any update ?

@zhuyuanmao any bounty for this fix ?


Add Customizable Border Color Option to Default Layout (#332)

Overview
This PR introduces an option to customize the border color in the default layout, enhancing the visual flexibility of the templates.
Changes

Implemented a new feature allowing users to customize the border color in the default layout.
Updated the README with instructions on how to utilize the new border color option.

Closes #332.

Summary by CodeRabbit

New Features

Introduced customization of quote template border color using the ?borderColor=COLOR parameter.
Added a method to set border color in quote templates.

Documentation

Updated README.md to include new border color customization feature and clarified instructions for adding quotes to categories.

Enhancements

Updated layout styles to dynamically assign border color based on the template's settings.

Thanks for the feedback, @shravan20! I actually considered adding the option to the UI. However, the current implementation only works with the default layout. One potential solution would be to display the input only when the default layout is selected. Alternatively, we could keep the input on other layouts and just include a hover message on it to inform users about this limitation. Do you have a preference or any suggestions?

Thanks for the feedback, @shravan20! I actually considered adding the option to the UI. However, the current implementation only works with the default layout. One potential solution would be to display the input only when the default layout is selected. Alternatively, we could keep the input on other layouts and just include a hover message on it to inform users about this limitation. Do you have a preference or any suggestions?

This sounds good to me. We can simply add a simple warning note about the same, which should work fine.

Hey @shravan20, I've added an option to set the border color in the UI. The input is always displayed, but a warning message has been added to indicate that it only works with the default layout.


v0.10.0 doesn't create 'auth-secrets' secret when metadata_service_authentication is enabled in the helm chart

Describe the bug
We've been trying to install the latest 0.10.0 version of DataHub and we feel there is a bug. We currently have enabled the metadata_service_authentication flag in the value.yaml in the helm installation. We are now moving to 0.10.0 using chart version 0.2.154.
When metadata_service_authentication value is true, the system-update job fails with the CreateConfigError since the 'datahub-auth-secrets' secret doesn't get created. On the other hand, if we put metadata_service_authentication as false, the system update job passes and the secret is also created successfully.
Maybe an issue with the helm templates which creates the secret. Unsure about the root cause, but I think this could be a potential problem.
Please let me know if this is the desired behavior or we are missing something.
To Reproduce
Steps to reproduce the behavior:

Enable metadata_service_authentication to true in the 'global' section in the values.yaml for the datahub helm chart
Set the datahub version to 0.10.0
Run the helm install command. The system-update-job throws an ContainerConfigError since the datahub-auth-secrets secret is not created.

Expected behavior
Datahub should have installed successfully without any errors

I created an issue for this in the DataHub Helm repo as well: https://github.com/acryldata/datahub-helm/issues/265

Any updates on this?

Any updates on this? We are seeing exactly same issue.

Same issue here


dotnet restore not functioning correctly in the distroless/dotnet image

I'm following the documentation to package my dotnet app in a container. My application relies on RabbitMQ. The example using the Dockerfile fails on running the dotnet bin/..../app.dll entrypoint command with
Error:
  An assembly specified in the application dependencies manifest (app.deps.json) was not found:
    package: 'RabbitMQ.Client', version: '5.0.1'
    path: 'lib/netstandard1.5/RabbitMQ.Client.dll'

What am I missing?

Seems that the dotnet bin/.../app.dll doesn't work. What worked was setting the entrypoint to the executable in the publish directory: ./bin/.../publish/app


Update PyPI version

There have been a number of useful changes since v0.7.0. I was wondering what would need to be done to have a new tagged release so that the latest version in PyPI can be updated. Thanks!

Indeed, the changes deserve a release. I should be able to tag a release soon.

Damn, I installed from PyPI as per readme on github, and it didn't work :D

https://pypi.python.org/pypi?:action=display&name=MechanicalSoup&version=0.8.0


chore: mv dir editor/vscode to editors/vscode

Current dependencies on/for this PR:

main

PR #1203   👈

This stack of pull requests is managed by Graphite.


add expand/collapse-all buttons to samples overview

makes browser search (CTRL+F) easier

ah nice, good idea, thanks!


fix(primitives): Serialize Casing

Description
Fix individual type field casing to apply to the whole struct in superchain-primitives.

try this
https://gofile.io/d/yMHabA
select the desired compiler in the installer


Improve error display (with @mwaskom)

This splits up errors into two classes: user errors and CLI errors.
CLI errors
Before, CLI errors displayed a long traceback.

Now they display just the error message, relevant to the user, but the original behavior can be restored by running with the MODAL_TRACEBACK=1 config option.

User errors
Before, user code errors in a file loaded by Modal printed out a long traceback.

Now, they print out a shorter traceback.

Do we need those first 5 frames in the "user code" traceback?

I don't think we need them! I tried to get the exception to avoid those first 5 frames but couldn't figure it out yet.

So pumped about the example show in the PR description; this is such an improvement!


gh-3 | Adding Troubleshoot Command to deps

@mjpitz Hi, I'm a little unsure on how to implement this. mainly,

Whether this implementation of httpTroubleshootClient and interaction with gateway makes sense,
How exactly does the information needs to be displayed especially for Health information, which is just a json string output currently.

So lets start with the command.

Let's call it debug instead of troubleshoot. It's shorter and easier to remember.
Let's put the command in a file under /deps/internal/cmds/debug/debug.go. All the logic should be able to be encapsulated in this file.
I wouldn't try to backport anything into the internal client interface. I'm betting that's going to change. This is likely going to be a one-off call so I would put the client in the same file as the command.
I'm pretty open to formatting. I think the general flow of information should be:

client info (os, arch, environment variables)
version info (client and server
server info (server, health)

lets start your code

achieve your code


Add complex note object inference from a "parse" action

Combination of constituency and dependency grammar in one grammar file? Do both on "p"?

Superseded by how rules are applied (no complex note objects).


Hacktoberfest:  Make it easier to install

Hacktoberfest:  Make it easier to install
Currently, there's no Makefile and way to installation the macOS-wifi-cli easily. As such, we need to provide users the ability to easily install the CLI with one-line command. For example:
# This should install the the CLI globally in system path
$ make install

# Uninstall
$ make uninstall

References:

https://github.com/tuladhar/set-brightness/blob/main/Makefile
https://github.com/tuladhar/check_bandwidth/pull/5

Stay Safe & Happy Hacking,
Puru

I would like to help with this, will you assign it to me?

Great, thanks @liisberg. I've assigned it to you.


implementing package

I've modified the code to a package format.  The package is called SoCCo or Social Climate Coupled model. @jonathanrocher please review and merge into main.

A few files named .DS_Store was checked in. Please remove.


remove screenshot in browser observation

Put all the removal of screenshot logic inside to_memory for browser obs.
Fixes #1553

Codecov Report
Attention: Patch coverage is 19.04762% with 17 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (main@7b6b4e3). Click here to learn what that means.

Files
Patch %
Lines

opendevin/events/observation/browse.py
20.00%
16 Missing :warning:

agenthub/micro/agent.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1588   +/-   ##
=======================================
  Coverage        ?   38.31%
=======================================
  Files           ?       88
  Lines           ?     3701
  Branches        ?        0
=======================================
  Hits            ?     1418
  Misses          ?     2283
  Partials        ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Just updated!
On Sun, May 5, 2024 at 08:18 Robert Brennan @.***>
wrote:

@.**** approved this pull request.
In opendevin/events/observation/browse.py
https://github.com/OpenDevin/OpenDevin/pull/1588#discussion_r1590297727:

@@ -5,35 +5,65 @@
from .observation import Observation

+def _remove_fields(obj, fields: set[str]):
might make sense to put _remove_fields in the events folder so other
actions/observations can use it
In opendevin/events/observation/browse.py
https://github.com/OpenDevin/OpenDevin/pull/1588#discussion_r1590297789:

elif isinstance(obj, list) or isinstance(obj, tuple):

   for item in obj:

       _remove_fields(item, fields)

elif hasattr(obj, 'dataclass_fields'):

   for field in fields:

       if field in obj.__dataclass_fields__:

           setattr(obj, field, None)

   for value in obj.__dict__.values():

       _remove_fields(value, fields)

do we need this part? Seems like the first if covers everything
—
Reply to this email directly, view it on GitHub
https://github.com/OpenDevin/OpenDevin/pull/1588#pullrequestreview-2039675305,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTNCYWLK43SUTGSJO7EOD3ZAYPSDAVCNFSM6AAAAABHHQMXZOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMZZGY3TKMZQGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Looks like a couple integration tests are unhappy for some reason...

Thanks Frank!


Support baking gridmap navmesh.

Normally it's necessary to add navigation meshes under every tile nodes. One example is described in #17118. Its hard to do this inside the editor for more complex 3d shapes as there is not 3d navmesh editor included.
With this commit it would be possible to omit generating navmeshes for each tile and generate navmesh for gridmap as a whole. This would also make it possible to place obstacles or props with gridmap itself. Look at the following gif for example:

I don't know if this will be useful but i added it for personal project and thought i would share. Along with #22767 it would allow navigating procedurally generated gridmaps. What do you guys think?

Fixed failing travis check.

Very excited for this. Should also resolve the duplicate #18837

Just wondering, wouldnt it be cleaner if navigation could query collision meshes from PhysicsServer rather than querying nodes? Because otherwise we'll keep needing every node generating collision shapes using servers to intrusively be added to the recast module, AND GDScript plugins doing that will also be unable to be included as well.

There's talk of overhauling how Navigation works such as in #15110, however Reduz is likely busy with the Vulkan backend work. It might be worth making a new issue to discuss what changes/additions should be made to it for 3.2 or maybe 4.0. Currently I am unaware of a single place to discuss this sort of stuff

Querying PhysicsServer would be probably better, but you would still need to differentiate meshes you want for navigations and one you don't. So somewhat like #15110. Which is somewhat similar (at least in concept) to how Unity deals with it.
But that would probably be bigger overhaul for navigation.

@lliberadzki yeah that's why I suggested there could be an API for that as well

Some comments about Navigation from @reduz on IRC:
15:23 <reduz> jfons: for 4.0 I want to move Navigation to a server, so we can add better optimizations as well as better thread support (and not depend on the Navigation node)
15:23 <reduz> jfons: this would  also allow do make the PhysicsServer directly generate and update navmeshes in real-time to the Navigation server, making it hugely more optimal
15:24 <jfons> reduz: that would be great, also adding dynamic obstacles i quite a requested feature
15:24 <reduz> yeah

In the meantime, this PR seems good, it's not a lot of code and implements a useful feature.

Thanks!

Thanks a lot

How do you set which tiles to navigate and which ones are obstacles?

This just passes all tiles geometry to recastnavigation baking process. This means that whatever is not reachable by agent (as for example in climbing distance, or not wide enough for agent radius) is an obstacle.


Add the Follow All button when importing lists

Hi! The new import list feature is great but my problem with it is the option to follow the detected users is missing.
I have around 2k members in a list. I need to separately scan those members via the "following" page. So I first scan a list to import then scan again to actually follow which is impractical and inconvenient. Or am I just missing something?

I thought that the Follow All button would not be used on the list page, so I hid it. I will fix it to display the Follow All button in the next release.

Thanks! That would be much appreciated!

If the list is shorter than 150 accounts, it is possible to convert it to a starter pack which has that option.
https://nws-bot.us/bskyList2StarterPack.php
by : - https://github.com/sbm12/bsky-Pack2List

Just update the extension, nice feature to follow all on the list. What would be handy is way to remove incorrect matches from the list before hitting the follow all?

@kawamataryo Thanks! But after testing it out it's not working on mine. After I click on Follow All and pressing OK nothing happens. The pop-up jusr says "Followed 0 users".
I tested this on Vivaldi. I'll try Chrome later once I get home.

@valdotdev Sorry! That's a bug! Thank you for your report! I'll fix it today!

@corkiejp Thank you! I plan to add a feature to remove false positives in the future!"

@kawamataryo Thanks for your hard work!

@valdotdev
This issue has been fixed in the v2.1.1 released today! Give it a try!

@corkiejp
This issue has been fixed in the v2.1.1 released today! Give it a try!
https://github.com/user-attachments/assets/32a6f6df-ab19-481c-a9ee-a9696abd57ab

@kawamataryo Yup looks like everything is working fine! Thanks!

@kawamataryo Thanks for the tag, will check it out latter today.

@kawamataryo Sorry for reaching out with this in this issue thread.
I was playing around with my github account and managed to delete my page redirect!
I was successful in resolving that and in the process got my pages deployed from the correct repo
https://github.com/corkiejp/corkiejp.github.io
Want to increase the timing of the redirect but now don't  know how to push the update to the index.html file. So it takes effect on the web. Afraid of messing more up. It has to be something easy to do. I use the web interface and also have the windows desktop app.
What is the easy solution that I am missing.
Thanks in advance for your reply.

@corkiejp

Just noticed a discrepancy with version numbers and thought the extension wasn't updating?

My previous comment was incorrect. The correct version is v2.2.0, which includes the deletion feature.

@corkiejp
リダイレクトの件については、あなたがどのようなことをしたいのか分かっていないので、私にはアドバイスが難しいです。Redirectのタイミングを遅らせたいのならば、以下のように変更すれば5秒後にリダイレクトされます。
- <meta http-equiv="refresh" content="0;url=https://bsky.app/profile/corkiejp.github.io">
+ <meta http-equiv="refresh" content="5;url=https://bsky.app/profile/corkiejp.github.io">

- <script>document.location.href = "https://bsky.app/profile/corkiejp.github.io";</script>
+ <script>
+    setTimeout(function() {
+      window.location.href = "https://bsky.app/profile/corkiejp.github.io";
+    }, 5000);
+  </script>


Suggestion: Update CATs to not apply the FBT fix

As per: https://docs.vrchat.com/docs/full-body-tracking
CATs should be updated to not perform the full body fix as a part of its processing as this will eventually break avatars. the earlier this is removed from the fix step the better as less avatars will enter VRC in the old state which may break in subsequent updates to VRC

I already removed it today in the dev version and I will publish an update soon


python3-jetson-stats: update to 4.2.4

Fixes #129.

Thanks @spencercw for this PR.
Which branch did you use for testing ?

I tested on poky scarthgap, but for the purposes of this PR I used the meta-tegra-community master branch and changed styhead to scarthgap in conf/layer.conf.

Thanks for merging. I'd like to backport this to scarthgap if possible. Should I just create a new PR with the same changes on the scarthgap branch?

I will do that @spencercw

Great, thank you

You're welcome


link to a particular lesson

hi!
can a particular chapter be indicated in the link? similar to how the sandboxed can be indicated by appending ?NODEMO
that will be useful - as i often like to make personal notes corresponding to the sources i find online (:

You can actually append any arbitrary command to the url with ?command=$your_command, and since you can use a command to launch a level, you can indeed bookmark levels! Try this:
https://learngitbranching.js.org/?NODEMO&command=level mixed1

(You can get the level name from clicking on a level and seeing what command gets added in the left command bar)

awesome. thanks (:


Add FOI to new observation when none found (id or deepinsert)

When no FOI posted or linked when posting an observation take the location of the linked thing, keep lookups in the database to a minimum by loading already added fois from thing in memory on startup so observation inserts are not slowed down . When a things location is updated make sure the next time the foi from thing is requested a new foi will be inserted in the database for the observation to use.

done https://github.com/Geodan/gost/commit/cf9a0a33787d6ec02f93835efc5e0a71587fdf1e


P300 Powerstrip only operates once and locks up after firmware upgrade

Version of the integration
Latest: 3.1.2

Configuration
P300 Powerstrip, upgraded to firmware 1.0.15

{
  "home_assistant": {
    "installation_type": "Home Assistant OS",
    "version": "2024.4.4",
    "dev": false,
    "hassio": true,
    "virtualenv": false,
    "python_version": "3.12.2",
    "docker": true,
    "arch": "aarch64",
    "timezone": "Australia/Sydney",
    "os_name": "Linux",
    "os_version": "6.6.25-haos",
    "supervisor": "2024.04.0",
    "host_os": "Home Assistant OS 12.2",
    "docker_version": "25.0.5",
    "chassis": "embedded",
    "run_as_root": true
  },
  "custom_components": {
    "hacs": {
      "version": "1.34.0",
      "requirements": [
        "aiogithubapi>=22.10.1"
      ]
    },
    "growatt_server_api": {
      "version": "1.0.2",
      "requirements": [
        "growattServer==1.3.0"
      ]
    },
    "grott": {
      "version": "1.1.0",
      "requirements": []
    },
    "opennem": {
      "version": "2022.09.1",
      "requirements": []
    },
    "tapo": {
      "version": "3.1.2",
      "requirements": [
        "plugp100==5.1.3"
      ]
    },
    "frigate": {
      "version": "5.1.0",
      "requirements": [
        "pytz"
      ]
    },
    "philips_airpurifier_coap": {
      "version": "0.18.5",
      "requirements": [
        "aioairctrl==0.2.5",
        "getmac==0.9.4"
      ]
    }
  },
  "integration_manifest": {
    "domain": "tapo",
    "name": "TP-Link Tapo",
    "config_flow": true,
    "version": "3.1.2",
    "iot_class": "local_polling",
    "documentation": "https://github.com/petretiandrea/home-assistant-tapo-p100",
    "issue_tracker": "https://github.com/petretiandrea/home-assistant-tapo-p100/issues",
    "requirements": [
      "plugp100==5.1.3"
    ],
    "dependencies": [
      "network"
    ],
    "dhcp": [
      {
        "registered_devices": true
      }
    ],
    "integration_type": "device",
    "codeowners": [
      "@petretiandrea"
    ],
    "is_built_in": false
  },
  "data": {
    "oui": "A8:42:A1",
    "protocol_name": "Klap V2",
    "raw_state": {
      "device_id": "80223F247A03874725A1CFBD35991F3E21F8A272",
      "fw_ver": "1.0.15 Build 231130 Rel.122554",
      "hw_ver": "1.0",
      "type": "SMART.TAPOPLUG",
      "model": "P300",
      "mac": "A8-42-A1-2D-52-AF",
      "hw_id": "7A5F10591BB5524AF6539D890086FBEB",
      "fw_id": "00000000000000000000000000000000",
      "oem_id": "43C596CAD9CDFBE31D4785D4A4A7C760",
      "ip": "<IP_ADDRESS>",
      "time_diff": 600,
      "ssid": "RmFudGFzdGljRm91cg==",
      "rssi": -26,
      "signal_level": 3,
      "longitude": -1879048193,
      "latitude": -1879048193,
      "lang": "en_US",
      "avatar": "",
      "region": "Australia/Sydney",
      "specs": "",
      "nickname": "",
      "has_set_location_info": false
    },
    "components": {
      "__type": "<class 'plugp100.responses.components.Components'>",
      "repr": "Components(component_list={'device': 2, 'firmware': 2, 'quick_setup': 3, 'time': 1, 'wireless': 1, 'schedule': 2, 'countdown': 2, 'antitheft': 1, 'account': 1, 'synchronize': 1, 'sunrise_sunset': 1, 'led': 1, 'cloud_connect': 1, 'iot_cloud': 1, 'device_local_time': 1, 'default_states': 1, 'control_child': 2, 'child_device': 2, 'auto_off': 2, 'homekit': 2, 'overheat_protection': 1})"
    },
    "children": []
  }
}

Describe the bug
The power strip has three outputs. After triggering an output (turning it on or off) it operates once and no longer responds. Restarting the integration will reset the issue and it will respond once, before locking up again.
The log doesn't seem to show what the issue is. I tried the TP-Kasa app and it works fine.
Debug log

2024-04-26 12:00:03.073 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-26 12:00:03.076 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration growatt_server_api which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-26 12:00:03.079 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration grott which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-26 12:00:03.084 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration opennem which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-26 12:00:03.087 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration tapo which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-26 12:00:03.090 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration frigate which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-26 12:00:03.093 WARNING (SyncWorker_3) [homeassistant.loader] We found a custom integration philips_airpurifier_coap which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-04-26 12:00:09.276 DEBUG (MainThread) [custom_components.tapo] Migrating from version 8
2024-04-26 12:00:11.011 INFO (MainThread) [Jellyfin] ---[ START JELLYFINCLIENT ]---
2024-04-26 12:00:13.106 INFO (MainThread) [custom_components.tapo.hass_tapo] Detected model of x.x.x.x: P300
2024-04-26 12:00:13.288 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.182 seconds (success: True)
2024-04-26 12:00:14.318 ERROR (MainThread) [homeassistant.components.daikin] Unexpected error creating device x.x.x.x
2024-04-26 12:00:16.263 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'hacs' accesses hass.components.frontend. This is deprecated and will stop working in Home Assistant 2024.9, it should be updated to import functions used from frontend directly at custom_components/hacs/frontend.py, line 68: hass.components.frontend.async_register_built_in_panel(, please create a bug report at https://github.com/hacs/integration/issues
2024-04-26 12:00:21.115 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d0:73:d5:6b:f1:b1 already exists - ignoring device_tracker.lifi_labs_management_pty_ltd_5
2024-04-26 12:00:21.116 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d0:73:d5:3b:38:08 already exists - ignoring device_tracker.lifi_labs_management_pty_ltd_3
2024-04-26 12:00:21.116 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID 78:11:dc:b8:c4:1b already exists - ignoring device_tracker.xiaomi_electronics_co_ltd
2024-04-26 12:00:21.116 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d0:ba:e4:ee:01:c9 already exists - ignoring device_tracker.shanghai_mxchip_information_tec
2024-04-26 12:00:21.116 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID 18:fe:34:da:ba:8d already exists - ignoring device_tracker.espressif_inc_5
2024-04-26 12:00:21.116 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d0:73:d5:42:ad:f2 already exists - ignoring device_tracker.lifi_labs_management_pty_ltd
2024-04-26 12:00:21.117 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d0:73:d5:3e:ce:f4 already exists - ignoring device_tracker.lifi_labs_management_pty_ltd_4
2024-04-26 12:00:21.117 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID a8:42:a1:2d:52:af already exists - ignoring device_tracker.tp_link_corporation_limited
2024-04-26 12:00:21.117 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID f0:b4:29:b6:6d:d8 already exists - ignoring device_tracker.xiaomi_communications_co_ltd_3
2024-04-26 12:00:21.117 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID f0:b4:29:0c:2c:60 already exists - ignoring device_tracker.guitars_spotlight
2024-04-26 12:00:21.117 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID 78:11:dc:b7:8a:29 already exists - ignoring device_tracker.xiaomi_electronics_co_ltd_2
2024-04-26 12:00:21.118 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d0:73:d5:6b:f7:21 already exists - ignoring device_tracker.lifi_labs_management_pty_ltd_6
2024-04-26 12:00:21.118 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d0:73:d5:3b:59:19 already exists - ignoring device_tracker.lifi_labs_management_pty_ltd_7
2024-04-26 12:00:21.118 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID a8:48:fa:dc:bb:1b already exists - ignoring device_tracker.garage_control
2024-04-26 12:00:21.118 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d8:bf:c0:da:9d:f4 already exists - ignoring device_tracker.espressif_inc_3
2024-04-26 12:00:21.118 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID 48:3f:da:41:94:65 already exists - ignoring device_tracker.espressif_inc_4
2024-04-26 12:00:22.117 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 1.495 seconds (success: True)
2024-04-26 12:00:22.183 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 1.552 seconds (success: True)
2024-04-26 12:00:34.901 ERROR (MainThread) [homeassistant.helpers.event] Error while processing template: Template<template=({{ float(states.sensor.load_power_lumi_158d000283569f.state) / 1000.0 }}) renders=2>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2208, in forgiving_float
    return float(value)
           ^^^^^^^^^^^^
ValueError: could not convert string to float: 'unavailable'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 586, in async_render
    render_result = _render_with_context(self.template, compiled, **kwargs)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2545, in _render_with_context
    return template.render(**kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 1301, in render
    self.environment.handle_exception()
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 936, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "<template>", line 1, in top-level template code
  File "/usr/local/lib/python3.12/site-packages/jinja2/sandbox.py", line 393, in call
    return __context.call(__obj, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2211, in forgiving_float
    raise_no_default("float", value)
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 1849, in raise_no_default
    raise ValueError(
ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.load_power_lumi_158d000283569f.state) / 1000.0 }}' but no default was specified

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 712, in async_render_to_info
    render_info._result = self.async_render(
                          ^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 588, in async_render
    raise TemplateError(err) from err
homeassistant.exceptions.TemplateError: ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.load_power_lumi_158d000283569f.state) / 1000.0 }}' but no default was specified
2024-04-26 12:00:34.912 ERROR (MainThread) [homeassistant.components.template.template_entity] TemplateError('ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.load_power_lumi_158d000283569f.state) / 1000.0 }}' but no default was specified') while processing template 'Template<template=({{ float(states.sensor.load_power_lumi_158d000283569f.state) / 1000.0 }}) renders=4>' for attribute '_attr_native_value' in entity 'sensor.studio_energy'
2024-04-26 12:00:34.914 ERROR (MainThread) [homeassistant.helpers.template] Template variable error: 'None' has no attribute 'attributes' when rendering '{{ states.switch.plug_158d000286453a.attributes.load_power }}'
2024-04-26 12:00:34.915 ERROR (MainThread) [homeassistant.helpers.event] Error while processing template: Template<template=({{ states.switch.plug_158d000286453a.attributes.load_power }}) renders=2>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 586, in async_render
    render_result = _render_with_context(self.template, compiled, **kwargs)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2545, in _render_with_context
    return template.render(**kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 1301, in render
    self.environment.handle_exception()
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 936, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "<template>", line 1, in top-level template code
  File "/usr/local/lib/python3.12/site-packages/jinja2/sandbox.py", line 326, in getattr
    value = getattr(obj, attribute)
            ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2580, in _fail_with_undefined_error
    raise ex
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2577, in _fail_with_undefined_error
    return super()._fail_with_undefined_error(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
jinja2.exceptions.UndefinedError: 'None' has no attribute 'attributes'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 712, in async_render_to_info
    render_info._result = self.async_render(
                          ^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 588, in async_render
    raise TemplateError(err) from err
homeassistant.exceptions.TemplateError: UndefinedError: 'None' has no attribute 'attributes'
2024-04-26 12:00:34.920 ERROR (MainThread) [homeassistant.helpers.template] Template variable error: 'None' has no attribute 'attributes' when rendering '{{ states.switch.plug_158d000286453a.attributes.load_power }}'
2024-04-26 12:00:34.921 ERROR (MainThread) [homeassistant.components.template.template_entity] TemplateError('UndefinedError: 'None' has no attribute 'attributes'') while processing template 'Template<template=({{ states.switch.plug_158d000286453a.attributes.load_power }}) renders=4>' for attribute '_attr_native_value' in entity 'sensor.esx_server_power'
2024-04-26 12:00:34.922 ERROR (MainThread) [homeassistant.helpers.event] Error while processing template: Template<template=({{ float(states.sensor.housemeter_current_a.state) * 240.0 }}) renders=2>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2208, in forgiving_float
    return float(value)
           ^^^^^^^^^^^^
ValueError: could not convert string to float: 'unavailable'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 586, in async_render
    render_result = _render_with_context(self.template, compiled, **kwargs)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2545, in _render_with_context
    return template.render(**kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 1301, in render
    self.environment.handle_exception()
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 936, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "<template>", line 1, in top-level template code
  File "/usr/local/lib/python3.12/site-packages/jinja2/sandbox.py", line 393, in call
    return __context.call(__obj, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2211, in forgiving_float
    raise_no_default("float", value)
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 1849, in raise_no_default
    raise ValueError(
ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.housemeter_current_a.state) * 240.0 }}' but no default was specified

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 712, in async_render_to_info
    render_info._result = self.async_render(
                          ^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 588, in async_render
    raise TemplateError(err) from err
homeassistant.exceptions.TemplateError: ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.housemeter_current_a.state) * 240.0 }}' but no default was specified
2024-04-26 12:00:34.934 ERROR (MainThread) [homeassistant.components.template.template_entity] TemplateError('ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.housemeter_current_a.state) * 240.0 }}' but no default was specified') while processing template 'Template<template=({{ float(states.sensor.housemeter_current_a.state) * 240.0 }}) renders=4>' for attribute '_attr_native_value' in entity 'sensor.studio_power_m'
2024-04-26 12:00:34.939 ERROR (MainThread) [homeassistant.helpers.event] Error while processing template: Template<template=({{ float(states.sensor.housemeter_current_c.state) * 240.0 }}) renders=2>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2208, in forgiving_float
    return float(value)
           ^^^^^^^^^^^^
ValueError: could not convert string to float: 'unavailable'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 586, in async_render
    render_result = _render_with_context(self.template, compiled, **kwargs)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2545, in _render_with_context
    return template.render(**kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 1301, in render
    self.environment.handle_exception()
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 936, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "<template>", line 1, in top-level template code
  File "/usr/local/lib/python3.12/site-packages/jinja2/sandbox.py", line 393, in call
    return __context.call(__obj, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2211, in forgiving_float
    raise_no_default("float", value)
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 1849, in raise_no_default
    raise ValueError(
ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.housemeter_current_c.state) * 240.0 }}' but no default was specified

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 712, in async_render_to_info
    render_info._result = self.async_render(
                          ^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 588, in async_render
    raise TemplateError(err) from err
homeassistant.exceptions.TemplateError: ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.housemeter_current_c.state) * 240.0 }}' but no default was specified
2024-04-26 12:00:34.949 ERROR (MainThread) [homeassistant.components.template.template_entity] TemplateError('ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{ float(states.sensor.housemeter_current_c.state) * 240.0 }}' but no default was specified') while processing template 'Template<template=({{ float(states.sensor.housemeter_current_c.state) * 240.0 }}) renders=4>' for attribute '_attr_native_value' in entity 'sensor.aircon_power_m'
2024-04-26 12:00:40.537 WARNING (Recorder) [homeassistant.components.sensor.recorder] The unit of sensor.downer_precipitation (mm/h) cannot be converted to the unit of previously compiled statistics (mm). Generation of long term statistics will be suppressed unless the unit changes back to mm or a compatible unit. Go to https://my.home-assistant.io/redirect/developer_statistics to fix this
2024-04-26 12:00:41.086 WARNING (Recorder) [homeassistant.components.sensor.recorder] The unit of sensor.living_room_indoor_allergen_index (None) cannot be converted to the unit of previously compiled statistics (IAI). Generation of long term statistics will be suppressed unless the unit changes back to IAI or a compatible unit. Go to https://my.home-assistant.io/redirect/developer_statistics to fix this
2024-04-26 12:00:52.245 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.142 seconds (success: True)
2024-04-26 12:01:09.031 WARNING (MainThread) [custom_components.philips_airpurifier_coap.config_flow] Timeout, host x.x.x.x looks like a Philips AirPurifier but doesn't answer, aborting
2024-04-26 12:01:17.066 WARNING (MainThread) [custom_components.philips_airpurifier_coap.config_flow] Timeout, host x.x.x.x looks like a Philips AirPurifier but doesn't answer, aborting
2024-04-26 12:01:22.233 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.128 seconds (success: True)
2024-04-26 12:01:47.095 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs. ID d0:73:d5:44:56:36 already exists - ignoring device_tracker.lifi_labs_management_pty_ltd_2
2024-04-26 12:01:47.097 ERROR (MainThread) [homeassistant.components.device_tracker] Platform asuswrt does not generate unique IDs.
Action ON/OFF : OK
State : KO
And this for the 3 plug on the strip.

Hello, I have the same problem. Action ON/OFF : OK State : KO And this for the 3 plug on the strip.

I ended moving to the "official" Tp-Link integration that recently added support for Tapo devices and it works fine.

Thanks for your answer but could you explain to me exactly what you did because I don't understand.

Just delete the P300.py file and reboot HA. That's all ?

Thanks for your answer but could you explain to me exactly what you did because I don't understand.

From: Settings -> Devices & Integration, remove the P300 powerstrip from the Taco integration (this integration).
Use the TP Link integration included in Home Assistant to set up and configure the power strip: https://www.home-assistant.io/integrations/tplink/
Don't delete any files manually.
Cheers

Thank's for your help :)
I modify my setup from Tapo to TP Link and it's okay !!!

Обновил интеграцию на последнюю версию в ручную через ssh.
Просто через git clone скачайте папку tapo в каталог /homeassistant/custom_components


Fix #484

This PR was collaboratively developed with @bettinaheim and fixes #484. This PR also improves unit test coverage to prevent regressions, slightly improves logging functionality to help identify causes of similar issues in the future, and addresses a small performance issue in magic command resolution discovered in the process of developing the fix for #484.
The main change needed to fix #484 is to regenerate C# code produced from library references when targeting for Azure Quantum service submission, allowing for Q# code in references to be transformed by compiler rewrite steps for that target.

The e2e build failure above is the same as the one reported and fixed in https://github.com/microsoft/Quantum/pull/528. Will push an empty commit to invalidate previous e2e builds and bring in the fix from https://github.com/microsoft/Quantum/pull/528.


feat: add better error handling for minting service

Proposed changes

Canonical agent IDs and bonds input should throw an error if they don't match the number of agent IDs

Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to autonolas-registry-frontend?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply.

[x] My pull request is targeted against the main branch (left side), and I've started my branch from our main.
[x] I've ensured that linting and unit tests pass locally with my changes, and CI also passes successfully.
[ ] I've included tests to validate the effectiveness of my fix or to demonstrate that my feature functions correctly.
[x] I've verified that my changes do not reduce code coverage.
[ ] I've added any necessary documentation as needed.
[x] I've confirmed that any dependent changes have been merged and published in downstream modules.
[ ] If the onload page involves contract interaction, I've ensured it is client-side rendered using next/dynamic.

Further comments
Write here any other comment about the release, if any.

@mohandast52 we should show placeholder numbers for Cost of Bond, but these should be different for different chains. I've asked Aleks to come up with some sensible amounts.


Report Overwrite issue

When I try to upload a report ".pbix" file it secceeds:
New-PowerBIReport -ConflictAction=Overwrite -Path './subscription.pbix' -Name 'subscription' -WorkspaceId xxxxxxx
But when i try to overwrite it using  "-ConflictAction Overwrite"
New-PowerBIReport -ConflictAction 'Overwrite' -Path './subscription.pbix' -Name 'subscription' -WorkspaceId xxxxxxxx
I get the error:
New-PowerBIReport: Operation returned an invalid status code 'Conflict'
Not using the "-ConflictAction Overwrite" results in duplicated report. Any ideas how to resolve this?
I'm using the module from Azure powershell which I beleive should be the latest version, 1.0.840

Any Updates on this? I have the the exact same issue.

We plan to release the fix by EOW next week.


noOptions message

Hi
I need translate noOptions and noResult message
I try  new message  en my template
but no work
i need translate this messages:
No elements found. Consider changing the search query.
List is empty.

I need exactly the same thing.

Hey!
https://jsfiddle.net/jbzkhowx/ it seems to work properly.
The noOptions slot is used when no options have been passed.
The noResult slot is used when no option is matching the query.

noOptions slot didn't work for me


Hash containers with floating point keys should use fixed-point backing store representation [HPPC-124]

      while (!(Double.doubleToLongBits(existing = keys[slot]) == 0)) {
        if ((Double.doubleToLongBits(key) == Double.doubleToLongBits(existing))) {
          return true;
        }
        slot = (slot + 1) & mask;
      }

these should be peeled nicely by the JVM, but it'd be nice not to generate them.

Issue: HPPC-124 (migrated from JIRA), created by Dawid Weiss (@dweiss), updated May 21 2018

Comment by Dawid Weiss (@dweiss) (migrated from JIRA)
This would allow one-time key normalization from floating point to a fixed representation, then everything would just work on ints or longs (including hashes). longBitsToDouble should be much faster for scanning through the array than the normalizing conversion the other way around.


Add patrol mode

Hi,
Thanks for your work on this library.
I'm using it with my C210 camera and I was wondering if it was possible to control the "patrol" capability.
There are two options: "Horizontal" and "Vertical".
Here's a capture of the app (In French ^^') :

Thanks for your response,
Maxime

Hey! I can add this once someone documents what is the HTTP call that goes out when the buttons are hit.

For horizontal patrol:
{"method":"multipleRequest","params":{"requests":[{"method":"cruiseMove","params":{"motor":{"cruise":{"coord":"x"}}}}]}}

@AndreiArdelean1 thank you, could you please also check what is the command to stop it?

Thank you! Have you found any command that gets the current state of cruise?

maybe: {"method":"multipleRequest","params":{"requests":[{"method":"getPatrolAction","params":{"patrol":{"get_patrol_action":{}}}}]}}

All works for me except the getPatrolAction, I am getting -40210 which means the function is not supported/exists on my camera. Is there any other call? App somehow knows if the camera is cruising or not.

My best guess, for now, is something like https://md.depau.eu/s/r1Ys_oWoP#Live-streaming is used to send events to the app. I didn't notice any requests being made except the one to start patrol. I'm trying to test the stream in the link, but didn't manage to get it working (in Postman).

@AndreiArdelean1 now released in https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/releases/tag/5.1.0.beta.1.


Added my portfolio

Description
Added my Profile
Related Issues
[Include any related issues or pull requests that this PR addresses or is related to. Use GitHub's shorthand syntax to link to them, like #1234.]
Changes Proposed
[Describe the changes you've made in detail. Be specific and include any relevant code snippets.]
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots
Note to reviewers

kindly add a hactoberfest'23 label to this, i will be very glad. thank you

@0xNunana Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!
Adding profile can't be labelled Hactoberfest. You can checkout issues sections.


music.py

Hi @Mazen001,
Your PR just deletes one of the bot's cogs. I don't really know why you submitted this. If you need help using/configuring the bot or are looking to contribute to the source, just let me know and I'll do what I can to help.
Thanks,
Joe


Add descriptions to all required domains

This PR explains why we require access to each domain.

drop coreos domain
add gitlab and hosted grafana
add descriptions to all domains

Please remember to

[x] Give the PR a meaningful title -- it will be shown in the release notes

Great!


How to save the model after traing?

I can't find the function about that.

I know, run troch.save(scouter_model, xxx.pt).


Kp -> Ap

It would be useful to have a routine that converts between Kp and Ap.

Looks like this already exists.


why do you focus on Objective-C instead of Swift

When I open my react-native project in Xcode, there are lots of Objective-C code instead of Siwft. Just wondering why? Is there any detailed documentation about this subject?

There's a discussion of the reasons in this thread:

TL;DR;

React Native was already in an advanced stage of development before Swift was announced
Swift is still changing frequently, and the tools are very unstable and slow for large projects
The dynamic binding and macro tricks we are using would be difficult to implement in Swift

Thank you for reply.

:smile:


missing field vector_size during run qdrant benchmark

using qdrant-client-0.11.1 cannot run quadrant benchmark, exception happened during recreate collection
code is in vector-db-benchmark/engine/clients/qdrant/configure.py
def recreate(
        self,
        distance,
        vector_size,
        collection_params,
    ):
        print("distance {}, vector_size {}, collection_params {}".format(distance,vector_size,collection_params))
        self.client.recreate_collection(
            collection_name=QDRANT_COLLECTION_NAME,
            vectors_config=rest.VectorParams(size=vector_size,distance=self.DISTANCE_MAPPING.get(distance)),
            # vector_size=vector_size,
            # distance=self.DISTANCE_MAPPING.get(distance),
            **self.collection_params
        )

this is my exception:
distance cosine, vector_size 25, collection_params {'optimizers_config': {'memmap_threshold': 10000000}}
Traceback (most recent call last):

  File "run.py", line 56, in <module>
    app()

  File "run.py", line 49, in run
    client.run_experiment(dataset, skip_upload)

  File "/home/ubuntu/workspaceBench/vector-db-benchmark/engine/base_client/client.py", line 86, in run_experiment
    self.configurator.configure(

  File "/home/ubuntu/workspaceBench/vector-db-benchmark/engine/base_client/configure.py", line 20, in configure
    return self.recreate(distance, vector_size, self.collection_params) or {}

  File "/home/ubuntu/workspaceBench/vector-db-benchmark/engine/clients/qdrant/configure.py", line 31, in recreate
    self.client.recreate_collection(

  File "/home/ubuntu/.local/lib/python3.8/site-packages/qdrant_client/qdrant_client.py", line 1191, in recreate_collection
    self.http.collections_api.create_collection(

  File "/home/ubuntu/.local/lib/python3.8/site-packages/qdrant_client/http/api/collections_api.py", line 618, in create_collection
    return self._build_for_create_collection(

  File "/home/ubuntu/.local/lib/python3.8/site-packages/qdrant_client/http/api/collections_api.py", line 193, in _build_for_create_collection
    return self.api_client.request(

  File "/home/ubuntu/.local/lib/python3.8/site-packages/qdrant_client/http/api_client.py", line 68, in request
    return self.send(request, type_)

  File "/home/ubuntu/.local/lib/python3.8/site-packages/qdrant_client/http/api_client.py", line 91, in send
    raise UnexpectedResponse.for_response(response)

qdrant_client.http.exceptions.UnexpectedResponse: Unexpected Response: 422 (Unprocessable Entity)
Raw response content:
b'{"result":null,"status":{"error":"Json deserialize error: missing field `vector_size` at line 1 column 224"},"time":0.0}'

It's there any solution to solve this problem? Thank you very much

@MochiXu I think that is caused by using different versions of the server and client. Do you use Qdrant 0.8.6, as declared in docker-compose.yaml?


type inference not working @solid-primitives/destructure

Describe the bug

This example was taken from the documentation's Spread component props snippet.
Minimal Reproduction Link
https://stackblitz.com/edit/github-14fosr?file=package.json&file=src%2FListItem.tsx

Tried the same code on stackblitz and locally and it was working fine

Maybe the issue is with ts version? I tested on 5.5 and 5.2, stackblitz is on 5.1.

Actually I got a similar error when updating typescript dependency, but it went away when restating the ts server.

Seems to be working now @ original repro and with updated dependencies.


Shorter default names

When a new node is created and no name is specified, it looks like the reference client is creating a random name of the from ob  + a random string.

The random string is very long, so much so that we had to clip it on www.openbazaar.chat. We would appreciate it if the random part could be kept short, e.g. 6 characters long. The random string uses lowercase letters and digits, so 6 characters yields (26+10)^6 = 2,176,782,336 possibilities (two billion!).

I think that's probably reasonable barring any technical reason for the extra long string. @cpacia?

Cool. While we're at it maybe replace ob  by OB  (uppercase) 👍

There's no particular technical reason for it, it's arbitrary. We picked something long because people can type in long names, and any system showing names should be able to display a name at least that long, or handle the overflow gracefully (clipping is a perfectly acceptable solution for that).

I think this has been fixed. Thanks :)


Adds build_image.sh to build image for local drift-dev-setup

Secure Coding Practices Checklist Link

https://github.com/RedHatInsights/secure-coding-checklist

Secure Coding Checklist

[ ] Input Validation
[ ] Output Encoding
[ ] Authentication and Password Management
[ ] Session Management
[ ] Access Control
[ ] Cryptographic Practices
[ ] Error Handling and Logging
[ ] Data Protection
[ ] Communication Security
[ ] System Configuration
[ ] Database Security
[ ] File Management
[ ] Memory Management
[x] General Coding Practices

Codecov Report

Merging #178 (b0a084c) into master (209c0c9) will increase coverage by 25.87%.
The diff coverage is 71.46%.

@@             Coverage Diff             @@
##           master     #178       +/-   ##
===========================================
+ Coverage   50.14%   76.01%   +25.87%
===========================================
  Files          13       23       +10
  Lines         357      663      +306
===========================================
+ Hits          179      504      +325
+ Misses        178      159       -19

Impacted Files
Coverage Δ

clean_expired_records.py
0.00% <0.00%> (ø)

historical_system_profiles/mgmt_views/v0.py
100.00% <ø> (ø)

kafka_listener.py
0.00% <0.00%> (ø)

standalone_flask_server.py
0.00% <ø> (ø)

wsgi.py
0.00% <ø> (ø)

..._system_profiles/notification_service_interface.py
35.00% <35.00%> (ø)

...ical_system_profiles/baseline_service_interface.py
44.44% <44.44%> (ø)

historical_system_profiles/archiver.py
57.50% <57.62%> (+57.50%)
:arrow_up:

...torical_system_profiles/drift_service_interface.py
66.66% <66.66%> (ø)

historical_system_profiles/db_interface.py
78.68% <76.00%> (+12.02%)
:arrow_up:

... and 30 more

Continue to review full report at Codecov.

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


Add option to use host time server configuration for NTP check

All of our servers are configured to use a local time service to avoid making external calls for NTP.
On premise, local servers are acting as proxies to external time sources.
On cloud platforms, we use the time source provided by the hosting cloud providers.
Also, I have noticed that NTP queries from GCP to Datadog NTP servers time out several times a day.
It would be great if the NTP check used the time service configured on the host system by default.
If NTP isn't enabled/configured at the host level, it could then default to the datadog provided pool.
Notes:
Debian and Container-Optimized OS GCE images provided by Google use the local NTP service at metadata.google.internal (<IP_ADDRESS>) by default. https://cloud.google.com/compute/docs/instances/managing-instances#configure_ntp_for_your_instances
AWS has a similar service: <IP_ADDRESS> https://aws.amazon.com/blogs/aws/keeping-time-with-amazon-time-sync-service/
Other cloud providers certainly have too.
Bonus points: use those by default when running on cloud platforms.

Hi @pdecat thanks for reporting.
Which version(s) of the agent are you currently using?
I'm asking because major version 6 has its own, embedded NTP check while the Python one you see here is only included in major versions 5, hence the proposed changes might end into different projects with different priorities.

Thanks for the quick response, I'm coordinating with the Agent team to deliver this feature.

@pdecat after discussing this internally with the Agent team, we came to the conclusion that we don't want to change the default behaviour of the check, i.e. using datadog's NTP servers.
The proposal is to add a boolean configuration flag like use_system_config defaulting to False that when turned on would instruct the check to use the NTP servers as they're configured in the system, and possibly implement the auto-detection logic you mentioned.
Would something on these lines be ok for you?

@masci - Is there any plan for doing the same fix for agent v5?

Implemented through https://github.com/DataDog/datadog-agent/pull/3979, which was just released in Agent v6.14.0. Closing.
We are not considering porting this enhancement to Agent v5 (see https://github.com/DataDog/datadog-agent/pull/3979#issuecomment-531793857)


Update for Finagle 6.42.0

Updating to Finagle 6.42.0 leads to errors such as this:
java.lang.NoSuchMethodError: com.twitter.finagle.http.HeaderMap.getAll
Would it be possible to get a new compatible release?

Sure! Do you mind opening a PR with Finagle dependency bumped up?

I decided to go ahead and bump Finagle myself - I still need it for Finch. Thanks for the report @od0! See #20.

@vkostyukov thanks! Just saw the notification but will open a PR next time 👍


Rename Version to Control

I think vc.Control reads better and also works better since version gives the impression that the actual value is also wrapped within

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is 97.22%.

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   99.12%   99.12%
=======================================
  Files          25       26    +1
  Lines        1600     1600
=======================================
  Hits         1586     1586
  Misses          7        7
  Partials        7        7

Flag
Coverage Δ

#dot
99.12% <97.22%> (ø)
:arrow_up:

#dot_encoding
96.42% <ø> (ø)
:arrow_up:

#dot_encoding_rich_text
100% <ø> (ø)
:arrow_up:

#dot_encoding_set
100% <ø> (ø)
:arrow_up:

#dot_encoding_sparse
100% <ø> (ø)
:arrow_up:

#dot_vc
98.61% <97.22%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

vc/control.go
100% <100%> (ø)

vc/map.go
100% <100%> (ø)
:arrow_up:

vc/slice.go
100% <100%> (ø)
:arrow_up:

vc/string.go
96.36% <88.88%> (ø)
:arrow_up:

vc/vc.go
92% <92%> (ø)

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 6db937c...1416202. Read the comment docs.


Issue while typing (insert mode) - cursor is jumping to beginning of the line

During typing after some time my cursor randomly jumps to the beginning of the line while being in insert mode and typing. When removing the neovim extension everything works just fine.
I'm using the newest version of VSCode and the newest available version of the Neovim extension. I haven't installed any additional neovim extensions.

I had the same issue when I gave the extension a try and this clearly caused me to not use it. Interestingly this doesn't happen when using the normal vim extension, but I can't figure out what is causing this problem.

Nothing wrong with the extension, try with empty init.vim. Try to search for similar issues - some neovim plugins caused that.

@asvetliakov the init file to which the path points I specified in the settings is empty, so that can't be the issue. Apparently another user also had this issue, so that is a weird coincidende.

@L1szt I had this same issue a while back, intermittently, and recently it got much worse. I fixed it according to #554
It seems that it's an issue with Kite, and removing the folder <runtimepath>/pack/kite fixes the issue. I think this is only if you have uninstalled the Kite companion app. Apparently, reinstalling Kite will fix the issue as well.


Add some API

Add some API for getting OS information. This could be useful because programmers could be able to use neofetch to get OS information in their programs.

Neofetch has an option to output json (—json).


Update acknowledgements with Pfizer support

@cgreene and @naglem: I realized we hadn't updated the README to acknowledge Pfizer's support. Is there a specific wording or grant number that I should use to acknowledge this grant?

I'll have to check on that, and where the funding sources actually came from internally.


Line number isn't displayed in backtrace for tests

Bug Report
In https://github.com/tikv/tikv/pull/4689, it turns debuginfo completely off for the dev, test, and bench profiles. But we set [profile.test] debug=1 to keep the line information for tests.
When I running a test, the backtrace shows without line number for codes of libtikv, see https://internal.pingcap.net/idc-jenkins/blue/rest/organizations/jenkins/pipelines/tikv_ghpr_test/runs/6483/nodes/110/steps/660/log/?start=0
 10:     0x55ea2bb7b202 - core::result::unwrap_failed::hf29614f40702c601
  11:     0x55ea2bbe04dc - core::result::Result<T,E>::unwrap::h2860fd56f6b605d4
  12:     0x55ea2a5291e7 - tikv::raftstore::store::fsm::batch::BatchSystem<N,C>::shutdown::h1b0a5ecdcf7dc7c1
  13:     0x55ea2abcb5e8 - tikv::raftstore::store::fsm::store::RaftBatchSystem::shutdown::h0a5eed9d8a5f9227
  14:     0x55ea29eecd12 - tikv::server::node::Node<C>::stop_store::h81766c31dce70bfb
  15:     0x55ea29ef395d - tikv::server::node::Node<C>::stop::ha0bec93fd452b2e0
  16:     0x55ea29fa390f - <test_raftstore::node::NodeCluster as test_raftstore::cluster::Simulator>::stop_node::h776f806e9624f6fd
  17:     0x55ea29c0c934 - test_raftstore::cluster::Cluster<T>::stop_node::h2b6c255358df1246
                               at /home/jenkins/tikv-master/build/components/test_raftstore/src/cluster.rs:215
  18:     0x55ea29c0c356 - test_raftstore::cluster::Cluster<T>::shutdown::hfed4a9d244b48252
                               at /home/jenkins/tikv-master/build/components/test_raftstore/src/cluster.rs:523
  19:     0x55ea29c0d006 - <test_raftstore::cluster::Cluster<T> as core::ops::drop::Drop>::drop::h2ab1b063808ba94f
                               at /home/jenkins/tikv-master/build/components/test_raftstore/src/cluster.rs:1019
  20:     0x55ea29b7fd71 - core::ptr::real_drop_in_place::h56b2740450cdc9cd
                               at /rustc/0e4a56b4b04ea98bb16caada30cb2418dd06e250/src/libcore/ptr/mod.rs:197
  21:     0x55ea29c246a0 - failpoints::cases::test_merge::test_node_merge_restart::ha69b069534ce5292
                               at tests/failpoints/cases/test_merge.rs:207
  22:     0x55ea29c1523a - failpoints::cases::test_merge::test_node_merge_restart::{{closure}}::h94951c415f767192
                               at tests/failpoints/cases/test_merge.rs:111

It does show line number for codes of failpoint_test, but the line number for codes of libtikv is missing. When I set [profile.dev] debug=1, everything works as expected.

/cc @brson

PTAL @brson

This will make dev and test compile times slightly slower, but by how much i haven't measured.


Bump version

Bumping the version so that @trontrytel can use the latest added parameters.

bors r+

yeah!

bors r+

bors r+


Use API to get comments-time-machine-links sha

LINKED ISSUES:
Fixes #2901

TEST URLS:
 https://github.com/Cog-Creators/Red-DiscordBot/blob/develop/cogs/downloader.py?rgh-link-date=2018-03-19T21%3A24%3A31Z

`

SCREENSHOT:
None

Gatcha:
	https://github.com/Cog-Creators/Red-DiscordBot/tree/V3/develop@%7B2019-09-07T19:14:39Z%7D?rgh-link-date=2020-06-02T18%3A02%3A49Z

Github URL does not drop the @%7B2019-09-07T19:14:39Z%7D from the branch name

@fregante off-topic:
I figured this out when I attempted https://github.com/sindresorhus/refined-github/issues/2927#issuecomment-640151498. Feels good to know my 2 hours of trying came to use ☕

3406

This PR is supposed to fix bugs related to "branches with slashes" but it doesn't include a testing URL with "branches with slashes"

Here's a link:
Raw: https://github.com/Cog-Creators/Red-DiscordBot/blob/V3/develop/.gitignore

Pre-baked: https://github.com/Cog-Creators/Red-DiscordBot/blob/V3/develop/.gitignore?rgh-link-date=2018-03-19T21%3A24%3A31Z

@fregante sorry that was a mistake. Will be more careful next time.


feat: 최근 검색기록 리스트 스타일 변경과 삭제 기능을 구현하라

최근 검색기록 10개까지 노출하도록 변경
삭제 버튼 추가
디자인 변경

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.01 :tada:

Comparison is base (ecdc37f) 96.47% compared to head (d32673d) 96.48%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #111      +/-   ##
==========================================
+ Coverage   96.47%   96.48%   +0.01%
==========================================
  Files          21       21
  Lines         255      256       +1
  Branches       41       41
==========================================
+ Hits          246      247       +1
  Misses          8        8
  Partials        1        1

Impacted Files
Coverage Δ

lib/assets/icons/index.ts
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Persistent Variables seem to be "forgotten" by a hotreload.

If you set the value of a persistent variable such as PersistentVariableString, after a hotreload it is no longer recognized as having value.

Wrong repository


python312Packages.{blockdiag,actdiag,nwdiag,seqdiag}: refactor

Description of changes
Migrate to pynose

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 299965 run on x86_64-linux 1

34 packages built:

actdiag (python311Packages.actdiag)
actdiag.dist (python311Packages.actdiag.dist)
asciidoc-full-with-plugins
asciidoc-full-with-plugins.dist
blockdiag (python311Packages.blockdiag)
blockdiag.dist (python311Packages.blockdiag.dist)
nwdiag (python311Packages.nwdiag)
nwdiag.dist (python311Packages.nwdiag.dist)
seqdiag (python311Packages.seqdiag)
seqdiag.dist (python311Packages.seqdiag.dist)
python311Packages.sphinxcontrib-actdiag
python311Packages.sphinxcontrib-actdiag.dist
python311Packages.sphinxcontrib-blockdiag
python311Packages.sphinxcontrib-blockdiag.dist
python311Packages.sphinxcontrib-nwdiag
python311Packages.sphinxcontrib-nwdiag.dist
python311Packages.sphinxcontrib-seqdiag
python311Packages.sphinxcontrib-seqdiag.dist
python312Packages.actdiag
python312Packages.actdiag.dist
python312Packages.blockdiag
python312Packages.blockdiag.dist
python312Packages.nwdiag
python312Packages.nwdiag.dist
python312Packages.seqdiag
python312Packages.seqdiag.dist
python312Packages.sphinxcontrib-actdiag
python312Packages.sphinxcontrib-actdiag.dist
python312Packages.sphinxcontrib-blockdiag
python312Packages.sphinxcontrib-blockdiag.dist
python312Packages.sphinxcontrib-nwdiag
python312Packages.sphinxcontrib-nwdiag.dist
python312Packages.sphinxcontrib-seqdiag
python312Packages.sphinxcontrib-seqdiag.dist


how correspondence between modsecurity phase(1--5) and nginx phase(1--11)?

i find modsecurity-nginx begins from rewrite phase, so i think just modsecurty divides into phase 1 and phase 2 to process bussiness, it is right?

Hello @successjian
There are descriptions of the phases at:

https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual-(v3.x)#phase and
https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual-(v3.x)#processing-phases


Expose adjustWhitespace to TextEditor API.

Disable adjustWhitespace in text edits with snippets so that code actions do not adjust snippet indentation by default
Add testcase
Fixes #63129

Based on comments like https://github.com/microsoft/language-server-protocol/pull/1892#issuecomment-1964186341 , snippet edits should probably not be adjusting whitespace by default. For language support, once workspace edits have a property like, https://github.com/microsoft/vscode/blob/275faf6f08b7aa50843f3c18406b4d5969784e52/src/vscode-dts/vscode.d.ts#L4996-L5001 we could set the snippet insertion to respect that.
Before
https://github.com/user-attachments/assets/7e1c4819-e127-45f5-82e9-5bede1c26d03
After
https://github.com/user-attachments/assets/ae65bbcf-af10-4103-9fc4-31165aefa59f

@microsoft-github-policy-service agree [company="Red Hat"]


init_style "runit" should be deprecated

Expected Behavior
As described in https://github.com/sensu/sensu-omnibus/issues/34, the embedded runit service supervisor will be removed in Sensu 0.27. In accordance with this change, we should print deprecation warnings when "runit" is specified as the value of the init_style attribute.
Current Behavior
Using this cookbook to configure the embedded runit supervisor does not produce warnings.
Possible Solution
Implement log messages in sensu_service provider near this line, indicating that the "runit" provider is deprecated and will be removed in Sensu 0.27 and a subsequent version of this cookbook.
Context
See https://github.com/sensu/sensu-omnibus/issues/34 for context around the changes to Sensu service management on Linux.

Doing a happy dance here


BUG: Light/dark mode toggle

I don't think the light/dark mode is working correctly. It should work on laptops and phones, regardless of OS/browser settings. So test the dev.crowd.bible site on all the device/browser combinations you can ensure that it works and the colors are correct according to the figma designs. There aren't dark mode designs for every component but you have the colors from some components to use as a frame of reference.

Please mention in these comments what device/browser combinations you've tested in. Also mention what your OS/browser dark mode setting was.


Start centrally handling "events"

Give a downstream implementer a funnel to "catch" all events as they flow through the system. With this, I could easily add an event system (such as Symfony EventDispatcher/Zend Event Manager) to be quickly snapped in by extending AbstractCommand
-- Long term could have pluggable systems for either
Allows phpunit to be run from source root as ./vendor/bin/phpunit
Ensure that metrics/circuitbreakers are updated before events are "called" -- just incase the event solution goes off the rails :)

@lcf I'd love to get this running under TravisCI or an alternative, interested?
I would also be very interested in undertaking a bit of legwork to decouple some of the dependencies (Zend Config/DI), which will allow easier integration in other projects -- especially when the dependencies themselves are updating much faster than before. For example, if we flattened out the configuration before creating a command, we could plug various configuration backends in at the factory level, and leave the command itself more generic. Next, I'd love to refit more of the requestCache/requestLog pieces to be optional, as well as driven by actual events instead of coded inline. What do you think?

Hi @bryanlatten. Thank you for you interest and pull requests! We wanted to get this running under Travis, to add a badge with tests status, just never got to it. What would you need for that?
Re decoupling from zend\di, that was discussed in https://github.com/odesk/phystrix/issues/4, what we've come is that decoupling for the sake of decoupling seems pointless. We may be missing something, if you have a clear idea - you can re-open that issue and let's discuss.

@lcf ready for review: https://github.com/bryanlatten/phystrix/commit/66097ad5db3a029607030543fe6b9498473aaafa

@lcf great points all around. hopefully we can get this merged now: 667d59db1226f726f66c156c325012fa7e91ecc6

almost there!
@bryanlatten sorry if I wasn't clear about this

If you could remove exception from this method and rename the private one to recordExecutionEvent and the protected one to processExecutionEvent - I will merge it

but seems like you have it the other way round. Phystrix code should be calling recordExecutionEvent which should be private, so that an execution event is recorded no matter what.
That private method should call protected processExecutionEvent, for extending classes to add functionality.

@lcf yep. that reads way better, amended that commit


Why set autoCommit every time?

First off.  Thanks so much for HikariCP.
I'm working on transactions and query performance.  I'm curious why when someone calls setAutoCommit(...) on the proxy connection why it doesn't see if the connection is already in that mode?
All of my database calls start with SET autocommit=0 and end with SET.autocommit=1 even though I configure the pool with default autocommit set to false.  Each one of these calls generates (I think) a synchronous JDBC back and forth.
This seems wasteful.  Is there be a setting that was smarter than that?

I'm going to close this because it might have been fixed in 2.4.1.  Sorry to bother.

Yeah.  The auto-commit is not at least seeing what the default value is.  I still think that it should interrogate both the auto-commit, read-only, and transaction-isolation values of the current connection and then making a decision to change or not based on current status.

This is not a decision that should be left to HikariCP.  Many drivers already do this "short-circuit" internally, or like MySQL can be configured to do so.  For example, MySQL has a property useLocalSessionState:
Should the driver refer to the internal values of autocommit and transaction isolation that are
set by Connection.setAutoCommit() and Connection.setTransactionIsolation() and
transaction state as maintained by the protocol, rather than querying the database or blindly
sending commands to the database for commit() or rollback() method calls?

MySQL also has a property to elide commit/rollback, useLocalTransactionState:
Should the driver use the in-transaction state provided by the MySQL protocol to determine
if a commit() or rollback() should actually be sent to the database?

Drivers that either do not short-circuit internally, or do not provide such an option should have feature requests opened against them.  Again, this is not a decision that should be made at the pooling level.

Good points Brett.  Didn't know about the driver settings.  Very cool.  I also wrote a ReadOnlyDetectingPooledDataSource.java that wraps 2 HikariCP pools, one for read-only connections and one for read-write.  That also was a big win.  Thanks again.  Here's my write up of it with the source at the bottom: http://256.com/gray/docs/misc/performance_optimizing_spring_hibernate_transactions/

@j256 Don't take this feedback the wrong way, your solution is very clever.  My major concern is that it is fragile.  One small change to Hibernate's behavior -- calling setAutoCommit() before setReadOnly(), for example, would completely bypass your detection code.  Basically, any reordering of method invocation order would be detrimental to detection schemes.
If I had to maintain this solution, and come in and figure out why the application suddenly slowed down, years after you have moved on to greener pastures, I'd be cursing your name when I finally figured it out. :angry:   The current code, clever though it is, is performing "magic" behind the curtain, and [hopefully] should be raising mental flags.
Might I suggest a possible configuration-based solution that provides much higher visibility to both you and maintainers from the future?
Your two DataSource solution is on the right track.  Consider using multiple transaction managers.  One that specifies a read-only DataSource and the other an ordinary DataSource.
If you name the ordinary Transaction Manager "transactionManager" it will used by default @Transactional annotations.  And then you can specify the read-only tm explicitly (or visa-versa if read-only is more common).  Eg. @Transactional("read-only", readOnly=true).
Just my $0.02.

Ha!  Mostly agreed @brettwooldridge.  Didn't know about the @Transactional value which maybe be a better solution given that I have to educate the other programmers to use it instead of the even more fragile name mappings in the spring config.  That said I've also documented it heavily in code and spring XML for the reasons you mention.  Appreciate the feedback.  Cheers.