Connecting to Team POs to build Personas profile

Describe the issue
We have worked on drafting user personas for Developers and SecOps. We need to connect to Product owners to start drafting/creating their user persona.
Which Sprint Goal is this issue related to?

Note: 'Milestone' is a ZenHub term that we use synonymously with 'Sprint'.

The 'Milestone' description (created in ZenHub) should clearly list Sprint Goals. This section should indicate which 'goal' this issue is related to.
Additional context
After attending the PO community of practice i connected with Aaron U, Mary C after they gave they agreed to connect with them.
Definition of done

Schedule a interview meeting with couple of POs
Interview PO
Take rough notes
Flex goal: Compile insight from interview and create a one pager Persona profile

finished the draft and sharing it with the PO for  feedback.


test(plugin-htlc-eth-besu): fix failed to resolve env var ETHERSCAN_KEY

Description
In the CI job called cactus-test-plugin-htlc-eth-besu we have a failure due to some API key missing.
Logs
Run cd packages/cactus-plugin-htlc-eth-besu && forge test -vvvvv
Compiling 21 files with 0.8.19
Solc 0.8.19 finished in 1.46s
Compiler run successful with warnings:
Warning (2018): Function state mutability can be restricted to pure
   --> src/test/solidity/integration/PrivateHashTimeLock.t.sol:153:5:
    |
153 |     function calculateHashSecret(uint256 base, uint256 exponent, uint256 modulus)
    |     ^ (Relevant source part starts here and spans across multiple lines).

Error:
Failed to resolve env var `ETHERSCAN_KEY` in `${ETHERSCAN_KEY}`: environment variable not found

Context:
- environment variable not found
Error: Process completed with exit code 1.

Acceptance Criteria

Tests in cactus-test-plugin-htlc-eth-besu are passing.
We did not pay for an API key - if that's the only way tests can be made passing then we should re-design the test cases.

@RafaelAPB Do you happen to remember any context about this etherscan API key? I'm assuming these tests were passing originally but wondering how.

fixed by: #3300 , the code change for ci.yaml file with regards to ETHERSCAN_KEY fixed this issue. Thank you.


Serialized charts have time field in them

closed by https://github.com/thomasboyt/bipp/commit/8d1e8b49934305163d27056cc7066203a7c8baf1


change debug behaviour

Hello,
is it possible to change debug behaviour in unit testing ?
this is default :
Launching debug configuration:
{
"type": "lldb",
"request": "launch",
"name": "run card_dispenser_test",
"program": "${workspaceFolder}/target/debug/card_dispenser_test",
"args": [],
"cwd": "${workspaceFolder}",
"sourceMap": {},
"sourceLanguages": [
"rust"
],
and i need change to this behaviour :
"configurations": [
    {
        "sourceMap": {
            "/project": "${workspaceFolder}"
            },
        "name": "Remote launch",
        "type": "lldb",
        "request": "launch",
        "program": "${workspaceFolder}/target/armv7-unknown-linux-gnueabihf/debug/test, //.
        "initCommands": [
            "platform select remote-linux", // Execute `platform list` for a list of available remote platform plugins.
            "platform connect connect://<IP_ADDRESS>:1234",
            "settings set target.inherit-env false", // See the note below.
        ],
    }

is that possible ? i cant find any config file with default config
Thanks
Jan

If i put it into my settings.json its fine, But i have this error:

does anybody know, which property in build config i have forgotten ? I am compiling with debugging info.


Pod does not compile: Value of optional type 'Selector?' not unwrapped in file "NSObject+KVO.swift"

Xcode 9.0
Bond (6.4.2)
/Pods/Bond/Sources/Bond/Shared/NSObject+KVO.swift:335:47: Value of optional type 'Selector?' not unwrapped; did you mean to use '!' or '?'?
line that is breaking:
unsafeBitCast(imp, to: _IMP.self)(me, selector)

Same problem

Here too in 6.4.0, I'm back to 6.3.0 for now.

Can you try 6.4.3? Thanks :)

To fix this I had to go into my project and manually set my Bond pod to use Swift 4.0. When I did that it started working.

it's still the same

@tadasz thanks for checking it out. Is your project written in Swift 4?

yes it is. once i force selector unsafeBitCast(imp, to: _IMP.self)(me, selector!)  it compiles fine

Doesn't help here either, I'm trying to run my current Swift 3 project in Xcode 9.

It's not a code problem. The project compiles without any warning with Carthage or as standalone. It's the CocoaPods integrations that is messing things up and I'm not yet sure where the problem is.

Try adding this in your pod file:
post_install do |installer|     installer.pods_project.targets.each do |target|         if             target.name == ‘Bond’             target.build_configurations.each do |config|                 config.build_settings[‘SWIFT_VERSION’] = ‘4.0’             end         end     end end

This should be working now without any scripts.

Got this error just now with version 6.6.0 swift 3 xcode 9.2.

Went back to 6.3.0 and seems to work but it would be nice to use the latest version.

I had the same issue right now...

What version of CocoaPods do you use? Bond does not support Swift 3 anymore.

I use cocoapods-1.4.0
Set everywhere Swift 4.0
The same problem.

Hey guys! I found solution! Remove the lines like below:
post_install do |installer|
installer.pods_project.targets.each do |target|
target.build_configurations.each do |config|
config.build_settings['SWIFT_VERSION'] = '3.0'
end
end
end
This enforce your library to use swift 3.0.


Fixed #31805 -- Fixed SchemaTests.tearDown() when table names are case-insensitive.

ticket-31805

@lothemar Can you confirm this fix?

Many thanks Carlton! :rocket:

Fix works on my end


Godot 4.1.2 stable windows 64 crashes or screen goes black when loading a project

Godot version
4.1.2 stable win64
System information
OS: Windows 10 22h2, Graphics: intel UHD graphics 630, CPU: intel core i5-8300h @ 2.30 ghz, RAM:  8gb
Issue description
I just installed godot for the first time. When I made a new project with the forward+ renderer the project generated just fine but it would crash even before the godot logo showed. When I tried the same thing with the Mobile renderer it loaded but nothing rendered. I could tell it loaded because my mouse cursor would change when I hovered over where a text box would be but not even the editor gui wouldn't render. The screen was just black. It only worked when I used the compatibility renderer. I have used unity and blender for a while and it works fine on this device.
I tried reinstalling godot, restarting my PC, updating everything from the windows settings, running from windows terminal with the flag -v verbose and the error with forward+ renderer was: StringName: 688 unclaimed string names at exit. the error when using the mobile renderer was the same except it had a warning: WARNING: Blend file import is enabled in the project settings, but no Blender path is configured in the editor settings. Blend files will not be imported.
at: _editor_init (modules/gltf/register_types.cpp:73). The exact same error and warning appears when using the compatibly renderer but the editor and viewport rendered just fine.  The mobile renderer works fine on my android tablet but not my laptop.
Steps to reproduce
Create a new godot project using the forward+ or mobile renderer. But the problem may be with my CPU or graphics
Minimal reproduction project
Mobile.zip
ForwardPlus.zip
compatibility.zip

Related to https://github.com/godotengine/godot/issues/83680.

the error when using the mobile renderer was the same except it had a warning

This is an unrelated warning, which you can ignore.

@Calinou  Hi. Sorry I didn't respond for a while. I forgot. If I were to update my intel UHD graphics 630 driver, witch version should I update to. I currently have version <IP_ADDRESS>38. My CPU is the Intel Core i5-8300H @ 2.30 GHz. Should I update to the version 25.20?


Missing assets

When I create a new Rails app using the v2 branch, I am not able to start the app due to the following assets missing:
app/assets/images/mobile-icons/startup.png
app/assets/images/mobile-icons/touch-icon.png
app/assets/images/previews/default.jpg
Once these have been added to my app everything works correctly.

What version of Rails are you running?

Rails 5.1 and Ruby 2.3.1

Fixed!


Updated automatic provider registration to handle additional errors

Fixes the issue with Resource provider not registered automatically for Microsoft.Security/pricings #22563.

Extends the check on http code 409 to include 404 and 400, the latter related to issue #20823.
Obtains the applicable resource provider (e.g., Microsoft.Security, Microsoft.Quota) from the error.message field in the http response if not available in the error.details.target field
Maintains the requirement to also match on error code/unregisteredRPCodes (e.g., "MissingSubscriptionRegistration", "Subscription Not Registered")

[x] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to module CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

Expecting the service to return BadRequest is odd. Can you link to the ARM guidelines specifiying this is a valid design.


ProjInfo sets environment variables for the current process.

Here environment variables are set for the current process.
This means that spawning an unrelated child process after using ProjInfo is affected by those.
In my case, I have to reset them after, otherwise child dotnet processes that I run myself don't work as expected.
Ideally ProjInfo would not change current process's environment, and if it does, it should at least revert any changes it makes (which doesn't help if other things are using the Environment in parallel, but is better than the current state).
If the environment is only set for the benefit of child processes, could ProcessStartInfo be utilised to set them for child processes only?

Instead of environment variables, we could return a collection of global properties, but then callers would be forced to use those global properties at all times or else something might be incorrect. The SDK and MSBuild tooling rely on the values of those properties being set.

It's fine to use environment variables, but instead of changing the current environment, they should only be applied to child processes. Otherwise the user is forced to figure out what they need to undo manually, which is not user-friendly.
One way to do this would be for functions like Init.init to return toolsPath * EnvironmentChanges, and things like WorkspaceLoader.Create accept such a tuple.
A backwards-compatible way would be for these functions to have overloads that do that on top of current versions.


Only use properties when required

Properties really only are meant for 'properties' of a class that can't be readily derived from other instance attributes.
Consider changing these to regular methods. Would need to change all code that uses these properties since you can invoke them as dq.ros_pose instead of dq.ros_pose().

Will probably do this by deprecating the properties for a while and introducing the preferred regular methods


docs(contributing): add instructions for creating a branch (#653)

docs(contributing): add instructions for creating a branch (#653)
Fixes Issue
My PR closes #653
👨💻 Changes proposed(What did you do ?)
I've added the step by step process to create a branch, commit changes to that branch and push the changes to the that particular branch.
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

Your pull request is rejected, reason 👉🏾is here👈🏾
Thanks for wanting to contribute, I hope you continue to do so properly.


Add testing for async use case

I've made sure that all tests run across both sync and async storage use cases, after seeing issues #19 and #20. A few tests are failing, without any change to the async implementation. I'd suggest merging this PR and creating a new issue to track the failing tests.

@rhys-saldanha Good point. Thank you for contribution!


How to use L.TimeDimension.prepareNextTimes(a, b, c)

Could someone who understands how this works explain how exactly to use this please.
I basically want to animate through some WMS layers and would like to preload 4 frames ahead of the current frame shown so that they are cached once the animation gets to those frames.
Any help gratefully received.

Hi @mo-ian-watkins,
the easiest way would be to use the option minBufferReady (see L.TimeDimension.Player options and example 2).
If you set this option to 4, the player will call prepareNextTimes automatically.

Thank you.
We are not using the player as we will using our own time scrubber control.
But through trial and error, I got to the bottom of it, e.g.:
L.TimeDimension.prepareNextTimes(1, 4, true)
will load 4 frames ahead of current frame (plus wrap around if needed). If we call that as every frame is displayed, there will four frames preloaded or at least in the process of arriving.
I think the biggest confusion was what the first parameter actually did <Number> numSteps


Temporarily remove PlayerManagement groups implementation

We will temporarily remove the groups implementation in playermanagement, but retain the code in a separate branch for reference/use when we are ready to add it back in and complete it.
The group functionality in playermanagement is currently in an unfinished state (e.g. no security model) and isn't currently on any of the roadmap work. Since the functionality is currently not complete and subject to change, and as we start to lock down on versionable code/releases it will be easier to add APIs than to break existing APIs.

Closed in #455
Original code is in the with-groups branch


remove unnecessary space in the command.

Signed-off-by: Tomoya.Fujita<EMAIL_ADDRESS>

@clalancette just removing the space in the command line, could you review and merge?

as far as i check, galactic and foxy can have backport.

@Mergifyio backport foxy galactic

@ahcorde thanks for taking care of this 😄


Port check looking for opened udp ports

The port healthcheck is now checking for opened ports
on /proc/net/udp
This solves #81 issue.

@orimarti Looks awesome! I'll test soon. A next step may be having two additional checks, one just for TCP and one for UDP.

Passes automated tests and looks like it does what it says!


Eval triggers

This is a feature request for Genie's eval-triggers, or something like them.  Basically, I want to make a trigger that fires when $stunned=1.

I wrote 2 scripts: 1 to monitor your position (standing, sitting, kneeling, prone) and one to monitor hidden/unhidden.  they just sit and monitor the variables on a regular basis and send updates to windows that display status.  Here is the position one, something like this might work for you:
oto set_cur_status
changemon_standing:
put #echo >stand STANDING
waiteval $standing = 0
put #window clear stand
goto set_cur_status
changemon_sitting:
put #echo >stand SITTING
waiteval $sitting = 0
put #window clear stand
goto set_cur_status
changemon_kneeling:
put #echo >stand KNEELING
waiteval $kneeling = 0
put #window clear stand
goto set_cur_status
changemon_prone:
put #echo >stand PRONE
waiteval $prone = 0
put #window clear stand
goto set_cur_status
set_cur_status:
if $kneeling = 1 then gosub changemon_kneeling
if $standing = 1 then gosub changemon_standing
if $sitting = 1 then gosub changemon_sitting
if $prone = 1 then gosub changemon_prone
goto set_cur_status


UI - Transparent expand button allows to accidentally click on external links in descriptions

Affected module
UI
Describe the bug
Expanding button in descriptions doesn't cover enough text. If you have an external link in description field which happens to be just in line with the expand button, you will accidentally click on this link instead of expanding the field. For more details, please see screenshots attached.
To Reproduce

Mis-clicked somewhere near the dropdown button

Got email service opened instead

Expanded the field and saw that I have an email mentioned here

Expected behavior
Perhaps, a more dense transparency would make do.
Version:

OS: [e.g. iOS]
Python version:
OpenMetadata version: 0.11.5
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

@Sachin-chaurasiya @devyani-kaushik I thought we changed this control to "Read More" link?

@harshach @Tonilyn, This has been fixed in 0.12.0.
https://user-images.githubusercontent.com/59080942/190949001-f80de5c9-e0cb-4ebb-96c6-1d58d44d646b.mov


Docker image with Flash Attention

Not intended for merge, just using this to build a Docker container using Jenkins, had a lot of trouble doing so on my laptop.

Discussed offline, might merge for agent requirements. Waiting on some follow-up conversations tmrw before opening PR for review

Merging because we're going to need for agent


Docker?

Hi @grokzen, I'm not sure if this is off-topic in regards to the redis-cluster-py, but this has been nagging me for a week now. In the Redis Cluster Specification, it said that Docker only supported if using host network only, means that Redis Cluster cannot be deployed using Docker in multiple servers. Is this true? Or do you know a way to solve this? I already read the redis github issue about this matter, looks like they want to support announce-ip (so they can use the internal Docker Container IP) but this have not been implemented yet. I'm very sorry if this is out of place, but I'm really curious whether you have valuable thoughts about this. Thanks!

@aldycool Hi. Here is my 2 cents on the docker problem.
The TLDR version is the following few points

Yes docker is supported
This problem is something that redis & docker them self has to solve (This is my personal opinion)

The long answer.
Okey so lets start with some internals on how the client works internally and how cluster discovery works.
When you create a client instance it will issue a command to one of the nodes that you specify in the list of  startup_nodes that you send into the client class (See nodemanager.py#L178). That command will return a data structure that describes the structure of the cluster. It defines what nodes that exists, master/slave relations, how to reach all them and the slots layout. This data is then used to populate the internal data structures used by the client so that when you issue a command, it knows where to try to send each command into the cluster.
So lets talk about how the cluster is setup and why this is what is "breaking" the docker support. When you are creating a cluster and you want all the nodes to meet and start to build the cluster you probably execute this command ruby /redis/src/redis-trib.rb create --replicas 1 <IP_ADDRESS>:7000 <IP_ADDRESS>:7001 ... for all nodes so that they will join up in a working cluster. The problem with this approach is that redis is kinda stupid/simple where it will blindly look at the ip addresses that you specefied as arguments to that command and use them as the locations where all the other cluster nodes is. It do not try to do some lookup of your public ip or anything in that sense becuase in one way it is not up to redis to understand or try to figure out what exact scenario you want. Becuase there is 1000+ ways of setting up a cluster, redis should know about all cases and behave exactly like you want out of the box. This is not something that you can expect redis to do becuase it is not reasonable to put that kind of logic into any system. So, the solution has to be done on your side and exactly how you setup your cluster.
So in order to fix the problem with docker, you have to fix the parameters you send into redis-trib.rb becuase that is the only way to solve the problem for real in all cases. So there is a few different kinds of solutions to this problem that you can use to solve the IP problem, but it all depends on your specific setup and how you operate your cluster.
If you look for example here https://github.com/Grokzen/docker-redis-cluster/blob/master/docker-data/start.sh#L4 and how i do it in my docker image, i use a small ifconfig | grep ...command to determine the public IP address that docker uses on my own machine in order to determine the IP address that i should send into redis-trib.rb. What this does is that when you later in your client sends the cluster CLUSTER SLOTS command, it will return the IP addresses of all nodes no matter in what container i have them inside and my client that i run outside docker (on my host machine) will be able to connect to all nodes in the cluster. The downside of this approach is that it do not work if you want to run the client instance on another machine over the network. The reason for this is that if your cluster tracks the IP address <IP_ADDRESS> and the machine you host it on has <IP_ADDRESS> and the machine that runs the client has ip <IP_ADDRESS> then the .23 machine will never understand what/where/how to reach the address <IP_ADDRESS> because that is running locally on the .123 machine.
The only known way to solve the case above is to somehow in your setup find the IP address that you use on your host and send that into redis-trib.py so that it can discover all the nodes on all other machines you run. This problem however is not something that is easily solved in a general way becuase you have to adapt to your own infrastructure and needs.
What you can do is to use DNS records to help track the IP addresses of machines and you send that into a shell script that does a lookup of the IP and that IP can be the public address of your host machine so that even from inside your docker container you can reach to another docker container on another host just based on public ip of the other host machine.
One thing you can't do right now (unless it was added recently) is to send DNS names into redis-trib.rb instead of having to send in IP addresses and then you only need to have the DNS working proerly and redis would handle looking it up and trying to connect to each node and then the problem would go away rather easily (i think atleast O.o)

Closing this one because it is not a real issue per say. We can still continue talk about it in this issue or you can ping me over at gitter.im (free chat, see link in readme)

It looks like we are running into this issue. How can we be sure that this is the issue and not something else? The script seems to hang at rc.Ping() and sometimes rc.StrictRedisCluster().
Here's a stack trace:
File "import_size_fit_accuracy_data_into_redis.py", line 86, in <module>
    start()
  File "import_size_fit_accuracy_data_into_redis.py", line 83, in start
    init()
  File "import_size_fit_accuracy_data_into_redis.py", line 11, in init
    initRedisCon()
  File "import_size_fit_accuracy_data_into_redis.py", line 42, in initRedisCon
    rc = rediscluster.StrictRedisCluster(startup_nodes=startup_nodes,decode_responses=True)
  File "/usr/local/lib/python2.7/dist-packages/rediscluster/client.py", line 181, in __init__
    **kwargs
  File "/usr/local/lib/python2.7/dist-packages/rediscluster/connection.py", line 141, in __init__
    self.nodes.initialize()
  File "/usr/local/lib/python2.7/dist-packages/rediscluster/nodemanager.py", line 228, in initialize
    need_full_slots_coverage = self.cluster_require_full_coverage(nodes_cache)
  File "/usr/local/lib/python2.7/dist-packages/rediscluster/nodemanager.py", line 270, in cluster_require_full_coverage
    return any(node_require_full_coverage(node) for node in nodes.values())
  File "/usr/local/lib/python2.7/dist-packages/rediscluster/nodemanager.py", line 270, in <genexpr>
    return any(node_require_full_coverage(node) for node in nodes.values())
  File "/usr/local/lib/python2.7/dist-packages/rediscluster/nodemanager.py", line 267, in node_require_full_coverage
    return "yes" in r_node.config_get("cluster-require-full-coverage").values()
  File "/usr/local/lib/python2.7/dist-packages/redis/client.py", line 715, in config_get
    return self.execute_command('CONFIG GET', pattern)
  File "/usr/local/lib/python2.7/dist-packages/redis/client.py", line 667, in execute_command
    connection.send_command(*args)
  File "/usr/local/lib/python2.7/dist-packages/redis/connection.py", line 610, in send_command
    self.send_packed_command(self.pack_command(*args))
  File "/usr/local/lib/python2.7/dist-packages/redis/connection.py", line 585, in send_packed_command
    self.connect()
  File "/usr/local/lib/python2.7/dist-packages/redis/connection.py", line 484, in connect
    sock = self._connect()
  File "/usr/local/lib/python2.7/dist-packages/redis/connection.py", line 529, in _connect
    sock.connect(socket_address)
  File "/usr/lib/python2.7/socket.py", line 224, in meth
    return getattr(self._sock,name)(*args)
KeyboardInterrupt

@ashhar-hasan do that happen every time or just some of the time?

Everytime, the only difference is in whether is stops at rc.StrictRedisCluster() or at rc.Ping().

Running this locally on a redis cluster poses no issues, only when docker is involved this happens. I will test it another environment and see I can reproduce there.
The important part is that the script was working fine until some time ago and that's what makes me think this is an issue caused by a config change in the redis cluster.
How can I know what IP is redis using to address nodes? Using CLUSTER NODES, CLUSTER INFO?

Cluster discovery is done here with CLUSTER SLOTS as used command https://github.com/Grokzen/redis-py-cluster/blob/unstable/rediscluster/nodemanager.py#L178
If you issue that command to your redis cluster inside docker, you will get the IP addresses that this client uses to connect to the nodes. If your client can't reach the IP addresses then it will fail with similar exceptions you have above. This is a classic problem with docker installations that either your network is bad, or you cluster on the inside is bad.


'NuGet Authenticate' not working with 'dotnet tool install'

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: NuGet Authenticate, .NET Core
Environment

Server - Azure Pipelines or TFS on-premises?

Azure Pipelines, account msasg, project Shared Data, release definition id: 279 (Already reverted, you can check release id 39209)

Agent - Hosted or Private:

Hosted agent, agent queue 'Azure Pipelines, vs2017-win2016'
Issue Description
dotnet tool install <pkg name> --add-source <internal source> will fail due to Auth error (401) when trying to install an tool package from NuGet feeds within same Azure DevOps account/project, even after NuGet Authenticate task is added & executed before this step.

@yhvicey yes this is expected. To fix this issue add an explicit dotnet restore task and specify arguments: '--no-restore' for the dotnet tool install. For explaination you can refer this doc  .

Do I need to add the internal feed globally, before running dotnet restore?

@yhvicey you need to add the feed reference in nuget.config file. You can take the reference of this doc for any other information related to this

Looks like dotnet restore requires a project file, which does not suit our case (run some internal tool package without any project file). I'll stick to authenticate nuget feed using PAT but still really appreciate your information, thanks!


fix(java): make Run with Args work for Java in Dap

Description
When Java and Dap extras are installed, the command "Run with Args" does not work as expected: after inserting the arguments to pass to the Java program, you get
Debug adapter didn't respond. Either the adapter is slow (then wait and ignore this) or there is a problem with your adapter or `java` configuration. Check the logs for errors (:help dap.set_log_level)

As discussed here https://github.com/LazyVim/LazyVim/discussions/4604, this happens because the Java Dap adapter expects the config.args' to be a string, while the default implementation in dap core, (see https://github.com/LazyVim/LazyVim/blob/v12.43.0/lua/lazyvim/plugins/extras/dap/core.lua#L34 and https://github.com/LazyVim/LazyVim/blob/v12.43.0/lua/lazyvim/plugins/extras/dap/core.lua#L2) passes the config.args` as a table.
Alternative to https://github.com/LazyVim/LazyVim/pull/4673
as suggested in https://github.com/LazyVim/LazyVim/pull/4673#issuecomment-2452925460
Related Issue(s)
Fixes #4672
Screenshots

Checklist

[x] I've read the CONTRIBUTING guidelines.

@folke as you suggested here https://github.com/LazyVim/LazyVim/pull/4673#issuecomment-2452925460 I implemented the check in dap.core directly.
I hope I did the right things in Lua, which I don't know much.
I tested this solution with a few Java projects, both with the configurations created automatically by Java Dap and with custom .vscode/launch.json configurations, possibly with existing args; everything works.

Why not just chaning the return statement in the config.args method depending on the config.type?
I don't really like duplicating code for no reason, that's why I suggested to make the changes in core.dap as opposed to having two different get_arsg functions.

@folke it's not just a matter of returning a string or a table: an existing Java configuration would have args as a string and the original code would throw an error when it prepares the string for the dialogue, because it would treat it as a table.

Can you check https://github.com/LazyVim/LazyVim/pull/4689 ?

Can you check #4689 ?

@folke I added my review; thanks! :)


Allow openfortivpn to be used on Darwin

Motivation for this change
I use a variation of this derivation on Darwin for a long time. The key difference is the platforms metadata which restricts the software to Linux only (which is not true).
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

Problem with this is, although merged: the package depends on ppp which is not supported on macOS? The "Software" openfortivpn still works with the system ppp daemon(as it is designed to).


python37Packages.tomlkit: 0.5.5 -> 0.5.7

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.7-tomlkit/versions.
meta.description for python37Packages.tomlkit is: '"Style-preserving TOML library for Python"'.
meta.homepage for python37Packages.tomlkit is: '"https://github.com/sdispater/tomlkit"

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.5.7 with grep in /nix/store/i57ss4ls5s2zz61f6qvqmd5y40c8v1r7-python3.7-tomlkit-0.5.7

directory tree listing: https://gist.github.com/4606a756e819cf385b60919e69be26c0

du listing: https://gist.github.com/b7f351c39ebd586b884f0610f5d8d290

Rebuild report (if merged into master) (click to expand)

24 total rebuild path(s)
6 package rebuild(s)
6 x86_64-linux rebuild(s)
6 i686-linux rebuild(s)
6 x86_64-darwin rebuild(s)
6 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
google-music-scripts
poetry
python27Packages.poetry
python27Packages.tomlkit
python37Packages.poetry
python37Packages.tomlkit

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i57ss4ls5s2zz61f6qvqmd5y40c8v1r7-python3.7-tomlkit-0.5.7 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A python37Packages.tomlkit https://github.com/r-ryantm/nixpkgs/archive/23bef849e1ac11c6699741abde3f71b05ac1c76e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/i57ss4ls5s2zz61f6qvqmd5y40c8v1r7-python3.7-tomlkit-0.5.7
ls -la /nix/store/i57ss4ls5s2zz61f6qvqmd5y40c8v1r7-python3.7-tomlkit-0.5.7/bin

cc @jbaum98 for testing.

@GrahamcOfBorg eval

@GrahamcOfBorg eval


Customizing the search result format

Is there any way to customize the result format of gigablast. It shows lots of unuseful data in every result, like respider - spider info - sections - page info - ip address , etc... Can we customize the format of the result in our style?
Thanks in ADV

Hi have you had any luck with this request

The format is in the C code.

If you want to edit the c++ code then the printing for each result starts here:
https://github.com/gigablast/open-source-search-engine/blob/master/PageResults.cpp#L3959
But the real solution is to request the results in json (&format=json) and then customize the results display in a front end framework.

Hi Vonbetz  thank you have now sorted that issue      Could you tell me where to find the code to change font size in returned results on results.cpp have tried every page i can think of to do this Dave

https://github.com/gigablast/open-source-search-engine/blob/4a943f1c79770d6a511889bec00baa194a5a9d55/PageResults.cpp#L230
rewrite the style tag.

Thanks vonbetz missed that dave

Hi okay have tried  that one is beyond my current knowledge  could you possibly show me how to do that
please with abit of code or explanation Regards Dave

hello mr/miss vonbetz
your 2nd anser to call the results from json, using the paramter &format=json is for the API call not the original source code
correct me if i am wrong
also how can we change the start page and name and title and etc?
what is the file to change plz if possible?
also plz see this
hello did you get the software to run on AWS?
if so at what URL IP address does it run at plz?
is it the default localhost:8000?
i dont think that will be live on the web internet?
does the AWS cloud give you an IP and you would have it run at like
IP CLOUD ADDRESS:8000?
hello i dont know if you can answer this but i want to run from source steps on a linux cloud server ver 20 and so if it compiles will it run on localhost;8000 in a live web page or will it detect the IP that my cloud server gives to me? so like IP CLOUD ADDRESS:8000?
Lisa
hope you can let me know cuz i am trying to deploy on my cloud provider on linux ubuntu ver 20


since 2 days, tapo smart plug plugin don't work with octoprint

hi,
i use this super plugin since long time and last night mi pi start in safe mode i search , serch, search, re install all octoprint, and finally it seems tapo smart plug crash octoprint 1.9.0.
i must use psu control now but is it possible to re use your plugin, i prefer! it is more easiest to set and there is more options very usefull.

This seems to be related to #140
In my setup even reinstalling the plugin did not work, as a cached older version of the TapoP100 plugin was used.
For me I had to access the venv
source ~/oprint/bin/activate
and then run
pip uninstall pyp100 && pip install git+https://github.com/almottier/TapoP100.git@main
Hope it helps.


Wrong outputs after applying the demo mode on the GOPRO_Large dataset

Hi Seungjun,
I am trying to run your PyTorch code in the demo code. I have downloaded your trained models(GOPRO_L1 and GOPRO_L1_amp) and unzip into the corresponding directory and run the main.py in the demo mode successfully. However, I found some wrongs while checking the outputs of your model after applying on the test dataset of the GOPRO_Large. The output picture looked like:

Could you please give me some tips?
Thanks.

Hi @MezereonXP,
You seem to be experiencing the same issue as #1.
What was your command exactly?
It should be like:
python main.py --precision half --save_dir GOPRO_L1 --demo true --demo_input_dir /home/seungjun/Research/dataset/GOPRO_Large/test/GOPR0384_11_00 --demo_output_dir temp

My command likes:
python ./src/main.py --save_dir GOPRO_L1_amp --demo true --demo_input_dir ./gopro/blur/ --demo_output_dir ./result

Here is the structure of my directories:
├── experiment
│   ├── GOPRO_L1
│   └── GOPRO_L1_amp
├── gopro
│   └── blur
├── LICENSE
├── README.md
├── src

Can you try executing python main.py inside src directory?
I haven't tried python ./src/main.py.
Some paths are relative to the executing directory and it may not work if executed outside src.

Can you try executing python main.py inside src directory?
I haven't tried python ./src/main.py.
Some paths are relative to the executing directory and it may not work if executed outside src.

Yeah，I really appreciate your help！
I got the right output after I executed inside the src！


Add unit tests

This package has 0 unit tests :(
Adding tests is very important now that so many people and apps rely on this module. The current thinking is to use Mocha for unit tests.

Closing this because I haven't done any tests yet, maybe some day…


New topic Update STO Images STO-7078

Preview: Update your STO images

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65bbe6cd40b0d7161ba052c8--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65bc386209fcd01214bb3ed8--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65bc3b9d3ab67a15f82485e0--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65be909533787dc696ef3260--harness-developer.netlify.app

Closing this ticket, I've implemented the changes in a new PR: https://github.com/harness/developer-hub/pull/5272


Update build-tools-container.md to default to 64-bit builds

As a novice when it comes to building on Windows, it took a significant amount of research to find out why I was getting missing symbols when compiling basic extension modules for Python.
https://learn.microsoft.com/en-us/cpp/build/how-to-enable-a-64-bit-visual-cpp-toolset-on-the-command-line#remarks

When you install a C++ workload in the Visual Studio installer, it always installs 32-bit, x86-hosted, native and cross compiler tools to build x86 and x64 code.

I couldn't find out the proper flag for 64-bit so I used the dedicated batch file.

@anandmeg
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@ofek, thanks for this feedback. We're consulting with the product team and will update the docs per their input. Thanks again!

Can you please explain a little bit further? Alternatively, it would perhaps be better to keep this open and then close when the change happens otherwise it appears externally that the feedback is simply ignored.

@ofek, per input from the product team, vcvars64.bat is the old version of the developer command prompt.  Also, VSVars64.bat file might not be there if the user is only building .NET stuff.  For VsDevCmd to target 64-bit, you can use the flag
-arch=amd64. We'll revise and update the docs accordingly.
Thanks!

I updated, thanks! Does this look like how you imagined now?

@ofek, thanks for your contribution & feedback. We've updated the docs per product team's input. The changes should be live later today. Thanks!


Bash prompt errors when mounting /hostroot

What happened:
I am using the ops-toolbelt image with the host volume "/" mounted under "/hostroot".
When trying to change the root directory of my current shell process by executing chroot /hostroot - it works, but it seems the shell prompt is misconfigured.
$ chroot /hostroot
/bin/sh: 1: get_kube_ctx: not found
/bin/sh: 1: get_git_status: not found
\[\033]0;ip-10-250-8-164.eu-north-1.compute.internal\007\]\n\[\033[1m\]\[\]\u\[\033[0m\] at \[\033[1m\]\[\]ip-10-250-8-164.eu-north-1.compute.internal\[\033[0m\] in \[\033[1m\]\[\]\w\n\[\033[0m\]\[\033[1m\]$ \[\033[0m\]

subsequent commands:
$ docker ps
.....
80c-910a-0eabd6637adb_0
f82f0c731108        gcr.io/google_containers/pause-amd64:3.1                                           "/pause"                 2 hours ago         Up 2 hours                              k8s_POD_node-exporter-qmqw9_kube-system_d0b0e8f7-68e3-4227-aab0-72b798439bdd_0
/bin/sh: 1: get_kube_ctx: not found
/bin/sh: 1: get_git_status: not found

What you expected to happen:
No errors when executing command when mounting the host root directory.
How to reproduce it (as minimally and precisely as possible):
Create a pod that mounts the host root directory:
apiVersion: v1
kind: Pod
metadata:
  annotations:
    kubernetes.io/psp: gardener.privileged
  labels:
    app: root
#    networking.gardener.cloud/to-dns: allowed
  name: rootpod-test
#  namespace: kube-system
spec:
  containers:
    - command:
        - sleep
        - "10000000"
      image: eu.gcr.io/gardener-project/gardener/ops-toolbelt
      imagePullPolicy: Always
      name: root-container
      resources: {}
      securityContext:
        privileged: true
        runAsUser: 0
      stdin: true
      terminationMessagePath: /dev/termination-log
      terminationMessagePolicy: File
      volumeMounts:
        - mountPath: /hostroot
          name: root-volume
  dnsPolicy: ClusterFirst
  enableServiceLinks: true
  hostNetwork: true
  hostPID: true
  #nodeName: ip-10-250-4-151.eu-north-1.compute.internal
  priority: 0
  restartPolicy: Always
  schedulerName: default-scheduler
  securityContext: {}
  serviceAccount: default
  serviceAccountName: default
  terminationGracePeriodSeconds: 30
  tolerations:
    - effect: NoExecute
      key: node.kubernetes.io/not-ready
      operator: Exists
      tolerationSeconds: 300
    - effect: NoExecute
      key: node.kubernetes.io/unreachable
      operator: Exists
      tolerationSeconds: 300
  volumes:
    - hostPath:
        path: /
        type: ""
      name: root-volume

exec into the pod
execute chroot /hostroot

Anything else we need to know:
There are no errors when using the busybox image. That is e.g. what gardenctl shell uses.
This is where the promp is configured
Environment:

I could also reproduce it if I start a new shell (sh) process. It does not occur with bash
root at term-13733848215326145417 in / on cli-gcp.core.example.com [default]
$ sh
sh: 1: get_kube_ctx: not found
sh: 1: get_git_status: not found
\[\033]0;term-13733848215326145417\007\]\n\[\033[1m\]\[\]\u\[\033[0m\] at \[\033[1m\]\[\]term-13733848215326145417\[\033[0m\] in \[\033[1m\]\[\]\w\n\[\033[0m\]\[\033[1m\]$ \[\033[0m\]bash
                     _                            _          _ _
  __ _  __ _ _ __ __| | ___ _ __   ___ _ __   ___| |__   ___| | |
 / _` |/ _` | '__/ _` |/ _ \ '_ \ / _ \ '__| / __| '_ \ / _ \ | |
| (_| | (_| | | | (_| |  __/ | | |  __/ |    \__ \ | | |  __/ | |
 \__, |\__,_|_|  \__,_|\___|_| |_|\___|_|    |___/_| |_|\___|_|_|
 |___/

Run ghelp to get information about installed tools and packages

root at term-13733848215326145417 in / on cli-gcp.core.example.com [default]
$

please assign this to me and i will have an look when available, thanks!

done, thanks @neo-liang-sap for having a look

the issue is in https://github.com/gardener/ops-toolbelt/blob/master/dotfiles/.bash_prompt#L100-L118
the code unset PROMPT_COMMAND and set PS1 to achieve color effect in bash and display git status / kube context, and this PS1 is not recognized by sh
after i commented out https://github.com/gardener/ops-toolbelt/blob/master/dotfiles/.bash_prompt#L100-L118 , sh is back to normal
will investigate whether it is possible for sh to use PS1

the root issue is /bin/sh can't translates \nnn sequences in PS1 into their octal counterpart. from doc here it can't consume these \nnnn (color etc) like bash
i didn't find an approach to using sh meanwhile keep the color style etc.
Is there any possible suggestion except i comment out https://github.com/gardener/ops-toolbelt/blob/master/dotfiles/.bash_prompt#L100-L118 ?

but what is the root cause for the two errors? have you checked this?
/bin/sh: 1: get_kube_ctx: not found
/bin/sh: 1: get_git_status: not found

the .bash_prompt is unable to be loaded in /bin/sh which causing two function (get_kube_ctx get_git_status)
.bash_prompt is able to be loaded normally only in /bin/bash

Hello @danielfoehrKn , would you consider @petersutter 's suggestion chroot /hostroot /bin/bash as an acceptable solution for this issue?
Thanks.
-Neo

Sure np  if it works :)

/close

i also added an PR here to add know issues part in README.md to log issues like this one


Remove rocm5.2 warnings from builds

Updates to remove rocm5.2 file-deprecation warnings from hip/host/ocl builds

@r-abishek Seems like one more warning is seen from CI for cmake. No warnings seen on make
cmake -DBACKEND=OCL -DCMAKE_BUILD_TYPE=Release ../..

-- The C compiler identification is GNU 9.4.0

-- The CXX compiler identification is GNU 9.4.0

-- Detecting C compiler ABI info

-- Detecting C compiler ABI info - done

-- Check for working C compiler: /usr/bin/cc - skipped

-- Detecting C compile features

-- Detecting C compile features - done

-- Detecting CXX compiler ABI info

-- Detecting CXX compiler ABI info - done

-- Check for working CXX compiler: /usr/bin/c++ - skipped

-- Detecting CXX compile features

-- Detecting CXX compile features - done

-- Radeon Performance Primitives (RPP) Version -- 0.95

-- RPP install path set to -- /opt/rocm/rpp

-- CMake minimum version required: 3.5

-- CMake version found: 3.23.0-rc2

CMake Warning (dev) at /opt/cmake-3.23.0-rc2/share/cmake-3.23/Modules/FindPackageHandleStandardArgs.cmake:438 (message):

  The package name passed to `find_package_handle_standard_args` (OPENCL)

  does not match the name of the calling package (OpenCL).  This can lead to

  problems in calling code that expects `find_package` result variables

  (e.g., `_FOUND`) to follow a certain pattern.

Call Stack (most recent call first):

  cmake/FindOpenCL.cmake:52 (find_package_handle_standard_args)

  CMakeLists.txt:114 (find_package)

This warning is for project developers.  Use -Wno-dev to suppress it.

-- Found OPENCL: /opt/rocm/opencl/lib/libOpenCL.so

-- ROCm OpenCL Found - Set CL_TARGET_OPENCL_VERSION=220

-- Added custom command for rpp_kernels

-- OpenCL kernels added!

-- OpenCL include directories -- /var/jenkins_home/workspace/ipclang_Short-GPUOpen_rpp_PR-116/gYwNxhPKM/rpp/src/include/cl/;/var/jenkins_home/workspace/ipclang_Short-GPUOpen_rpp_PR-116/gYwNxhPKM/rpp/src/include/common/;/opt/rocm/include/

-- amd_rpp set to build with GPU_SUPPORT using OpenCL backend

-- amd_rpp set to build with ENABLE_SIMD_INTRINSICS

-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.71.0/BoostConfig.cmake (found version "1.71.0") found components: filesystem system

-- Found OpenMP_C: -fopenmp (found version "4.5")

-- Found OpenMP_CXX: -fopenmp (found version "4.5")

-- Found OpenMP: TRUE (found version "4.5")

-- Configuring done

-- Generating done

@LakshmiKumar23 Interesting, don't observe that in my build. Building on 5.2.


spanner: recurring session deletion error

The error below has occurred in multiple test runs, appears unrelated to other changes. The specific error extracted below is (re)tried a total of 35 times in the linked build log.
From go1.13 build log in multiple test runs of #1070:
=== RUN   TestSample
2019/11/19 18:46:44 Failed to delete session projects/golang-samples-tests/instances/golang-samples-tests/databases/test-golang-samples-tests-7/sessions/AJSwhAQAAZtS04M65t1ZbONyLquqF4NCB76VjuLCamp9h7eh4OTesRqxc2ZsWw. Error: rpc error: code = NotFound desc = Database not found: projects/golang-samples-tests/instances/golang-samples-tests/databases/test-golang-samples-tests-7
[...]
--- PASS: TestSample (79.23s)

Changing this to a bug because the sample may not be cleaning up properly.

I think this is because the database is dropped before the data client tries to delete session. So the error says "Database not found".
https://github.com/GoogleCloudPlatform/golang-samples/blob/2eb88b14be2dca75536c709aceccadd68400c9b9/spanner/spanner_snippets/snippet_test.go#L45-L46
https://github.com/GoogleCloudPlatform/golang-samples/blob/2eb88b14be2dca75536c709aceccadd68400c9b9/spanner/spanner_snippets/snippet_test.go#L78-L85
For the above code, I believe deleting database will happen first before client can close all of their sessions.


poetry add unable to find black

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name: MacOS Mojave
Poetry version: Poetry 0.12.10
Link of a Gist with the contents of your pyproject.toml file: https://gist.github.com/rajeev/703bd6b8b96c7b2e375aff8cc8b06079

Issue
I'm trying to add black as a dev-dep for my project. However, poetry does not "find" black, either as a dev nor as a regular dependency. poetry add also fails with the same result.
I tried poetry add black [1], and tried adding it via the pyproject.toml. However, doing a poetry search black shows black in the results.
[1]: poetry add -vvvv
Using virtualenv: /Users/kane/Library/Caches/pypoetry/virtualenvs/test-py3.7
PyPI: 0 packages found for black *
[ValueError]
Could not find a matching version of package black
Exception trace:
/Users/kane/.poetry/lib/poetry/vendor/py3.7/cleo/application.py in run() at line 94
status_code = self.do_run(input, output_)
/Users/kane/.poetry/lib/poetry/console/application.py in do_run() at line 88
return super(Application, self).do_run(i, o)
/Users/kane/.poetry/lib/poetry/vendor/py3.7/cleo/application.py in do_run() at line 197
status_code = command.run(input, output_)
/Users/kane/.poetry/lib/poetry/console/commands/command.py in run() at line 77
return super(BaseCommand, self).run(i, o)
/Users/kane/.poetry/lib/poetry/vendor/py3.7/cleo/commands/base_command.py in run() at line 146
status_code = self.execute(input, output_)
/Users/kane/.poetry/lib/poetry/_vendor/py3.7/cleo/commands/command.py in execute() at line 107
return self.handle()
/Users/kane/.poetry/lib/poetry/console/commands/add.py in handle() at line 69
packages, allow_prereleases=self.option("allow-prereleases")
/Users/kane/.poetry/lib/poetry/console/commands/init.py in _determine_requirements() at line 230
requirement["name"], allow_prereleases=allow_prereleases
/Users/kane/.poetry/lib/poetry/console/commands/init.py in _find_best_version_for_package() at line 266
"Could not find a matching version of package {}".format(name)
add [-D|--dev] [--git GIT] [--path PATH] [-E|--extras EXTRAS] [--optional] [--python PYTHON] [--platform PLATFORM] [--allow-prereleases] [--dry-run] [--]  ()...

Apparently, allows-prereleases works! That's weird because I searched many times for this issue, but didn't see the one named "Can not install black".


Delete encryptEngine and optimize encryptRule

Fixes #2732.

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Fix handling of empty strings (fixes #51)

Should be merged in conjunction with https://github.com/PullJosh/leopard/pull/69

Is this the intended behavior for comparison operators?

Those genuinely are empty inputs in the test project, not zeroes:

This is more correct than the old version.


Help? pls [Question] [Need Answer]

I know this is duplicate but no one answer :(
i close this if i got the right answer :3

How can make the server gives 10 minions to the AI Bots when i do addbot 25
Means the The 25 AI Bots will have 10 bots
How to make umm delay bots splitting feeding , etc?

@MastaCoder @Megabyte918 @AlexHGaming

Apparently it won't add minions to AI bots. You can change it to add minions to AI bots by not checking if the player is an AI. Then, configure serverBots to 10. Or, if you don't want players to have minions, you can check if the player is a real player, and then not add bots.

@XtremePlayzCODE Its possible lol , AI  bots to have minions :3


Collapse isn't compatible with HTML 5.0

windows 10 Pro 1903
google chrome
link : https://mathiasbynens.be/notes/html5-id-class#:~:text=The HTML 4.01 spec states,there is no such limitation.
Hello,
I have id's auto generated then, I note if id start with numbers collapse doesn't work, then event like show.bs.modal are never called. Perhaps is the same with other component like tabs.
Thank you.

Does this need a docs mention? I feel like this it outside our wheelhouse.

I think, yes, because you can build collapse, tabs... only based on HTML.

We already have a mention but improvements are welcomed.
https://getbootstrap.com/docs/4.5/getting-started/javascript/#selectors

Closing per last comment then—already have something to point folks to for now and can revisit if others find room for improvement here.


windows are not resized after insert (from fullscreen mode)

Open any window in fullscreen mode
yabai -m window --toggle zoom-fullscreen
Create another window and exit fullscreen mode
The new window is placed on top, old window hasn't been resized

When you say

Create another window and exit fullscreen mode
Do you switch focus back to the old window before toggling fullscreen mode?

Also, does the new window end up in fullscreen at this point?

The new window is placed on top, old window hasn't been resized

Sorry, I'm a bit confused about what you're saying. Could I clarify whether the following steps match what you are doing? In the steps, I make reference to an attached screen recording of me going through the same steps.

open an app (in the video I open the terminal)
toggle full screen of the app
open another app (in the video the Reminders app)
switch focus back to the first app and toggle full screen for that app

https://user-images.githubusercontent.com/97084099/195508184-1cd177eb-1f66-464b-a195-706f4ef9d7f2.mp4
If I am doing the same steps as you, could you explain how the behaviour on your system differs from the recording?

https://user-images.githubusercontent.com/98094840/195994174-b67579fc-3c56-4e2f-b97c-434d9cff3b5e.mp4

Have 2 open windows
Fullscreen window on the right
open another window (I'm using a keyboard shortcut)
new window is placed on top, grabbing focus, and a fullscreen window is automatically toggled back

I am also seeing the same behaviour that @nikita-orca recorded after upgrading from 4.x.x (don't remember which one exactly, sorry) to 5.0.0

@nikita-orca Could I check what's the behaviour you're expecting instead in your recording? Is it one of the following?

Are you expecting the fullscreen terminal window to be resized to the top right corner (taking up quarter of total space) when you open the second terminal window? If that's the case, I'm afraid that's no longer the expected behaviour (please the reply to @dimitarvdimitrov below).
Otherwise, are you expecting the fullscreen terminal window to remain fullscreen and the second window to be placed on top? This is what happens when I do what you did in the video with 3 windows instead of 2 so I think there's a bug here which can be fixed by adjusting the following line
https://github.com/koekeishiya/yabai/blob/c888d80540a580c61e68ee3f6682fb0099962f0e/src/view.c#L254
to  : node->zoom == node->parent && node->zoom != view->root.

@dimitarvdimitrov In 4.0.4 opening a new window when there's a window currently in fullscreen would cause the fullscreen window to revert back to normal. In 5.0.0, the behaviour has changed to keeping the fullscreen window in fullscreen and placing the new window on top. #864 looks like the specific change in 5.0.0 responsible.

so it looks like it's working as intended. What do you think about making this un-fullscreen behaviour a config option - e.g. unzoom_on_new_window?

@koekeishiya could you do this fix, please?
https://github.com/koekeishiya/yabai/issues/1481#issuecomment-1281938635

I looked into this

So it would be nice to fix the bug at least

a bit more and realised that the change I suggested here

to  : node->zoom == node->parent && node->zoom != view->root.

would also cause unexpected behaviour: if you had two windows open and --toggle zoom-parent one of them then open a third window, the zoom-parent window would be zoomed fullscreen instead parent (so basically the opposite of the current issue happens).
It looks like the underlying cause is that whenever it comes to a situation where there are two windows in a space, zoom-parent and zoom-fullscreen fold into one and subsequently we are unable to determine whether the zoomed window is supposed to be fullscreen or parent (currently it is taken to be parent).
Maybe there needs to be a separate boolean field in window_node to distinguish between the two?

@nikita-orca As for this

I would like yabai to turn off fullscreen when a new window is added But I see #864, so that's not possible currently

The following signal should be able to do it if an unzoom_on_new_window option suggested by @dimitarvdimitrov doesn't get added.
yabai -m signal --add event=window_focused action='if [[ $(yabai -m query --windows --window | jq ".\"is-floating\"") == "false" ]]; then
win_id=$(yabai -m query --windows --space | jq "map(select(.\"has-fullscreen-zoom\" and (.\"has-focus\" | not)))[0].id")
if [[ $win_id != "null" ]]; then yabai -m window $win_id --toggle zoom-fullscreen; fi
fi'

I've been using this signal to automatically unzoom windows even when switching window focus (in yabai v4.x.x the unzooming would happen when adding windows but not switching). However, this

@koekeishiya could you do this fix, please? #1481 (comment)

still needs to be addressed before the signal is able to achieve the behaviour you desire in your screen recording

screen.mp4

up

thanks @nikita-orca, this signal works well as a workaround for me

It's not working for me :(

The signal kind of works but it's janky. It takes a moment for it to realise something's changed and then everything kind of rejiggles into the right place.

Made zoom persistence configurable; yabai -m config window_zoom_persist on|off


Add nested Ordered and Unordered lists

Thanks @bryancasler (and @hartwork)! Merging this as it seems like a good idea and everything looks fine to me.


feat(RequestPoolManager): Added custom request type and custom priority

What kind of change does this PR introduce?
Added a custom request type to the addRequest in the requestPoolManager. The custom request with controllable priority is useful when caching the studies in advance for the next case
What is the current behavior?
Only interaction, thumbnail, prefetch can be used as the request type, with a fixed priority value.
What is the new behavior?
A custom request type can be defined with a custom priority. In addition, the priority of the  interaction, thumbnail, prefetch can be changed if needed.
Does this PR introduce a breaking change?
No, it will only add a new feature to enable custom priorities for the requests
Other information:
None

This will be requested again from a cornerstonTools branch


tidy routes: adjusts web routes as follows:

Commit message(s)

1st) splits web.php into web.auth.php (login, logout), web.tidy.php (main routes), web.debug.php (the 500 route demo);
2nd) modifies RouteServiceProvider to load the routes; note the original mapWebRoutes() remains intact, but commented;
3rd) leverages invokable and resource controllers fully; adds custom routes for the '@restore' actions not included in resource controllers;
4th) registers the resource routing with the qualified controller class (ie UsersController::class); '@restore' routes still mapped with strings;
and made the ImagesController effectively an invokable controller, like the single-method, invokable Dashboard and Reports controllers.

Notes
I do realize these routes have been called into question previously, and that similar efforts in another PR were ultimately discarded (grouping routes declined in #22), with the express desire and intent being to keep the routes "super explicit". If that remains the case, then it is understood this will also not be merged...
But, I wanted to open the PR anyway, to say that this week is my first good look at Inertia.js and, after reading the "how does it work" and "protocol" pages in the documentation, one of the first things I did was open this reference implementation and examine the routes. And when I did see this routes file I nearly closed all things Inertia and planned to forget about it on the basis of "if my routes need to look like that, I'm not using it" ...
But I looked closer and realized that with the exception of the route name attached to the index action, and the additional restore action, these were in fact just resource routes and controllers at work.
I empathize with 'super explicit' routes for the benefit of newcomers to the (laravel) framework, I also think the web.tidy.php is worthy of consideration, as contrasted to the original web.php routes definitions, and which offers routing examples of:

Invokable (single-action) controllers
Resource (CRUD) controllers, using the qualified MyController::class syntax, and
Custom action routes, left in-tact for e.g. 'ContactsController@restore' actions/methods, and still using the familiar string syntax to register the action.

Summary
So, this pull request is actually more lines of code than the original, spread across multiple files, but it does also demonstrate the multiple ways to register routes in laravel along with the utility and expressiveness of that routing system. And to those already familiar with laravel, it becomes immediately evident that Inertia does in fact work with any/all options.

Thanks
Thanks for looking it over and for the consideration. And Thanks for Inertia :)

Closing remarks
No need to leave this dangling :)
I did not open this PR with any strong expectation that it would be embraced or accepted; rather to bring attention to the point that the relevant routes could be made much more terse and tidy. That much is demonstrated in the enclosed discussion thread here, for anyone interested enough.
And thank you again for this reference implementation of Inertia in a Laravel project.


The LogStream API should support reading data when TTY is enabled.

When TTY is not enabled the first 8 bytes of a stream have the form :
{STREAM_TYPE, 0, 0, 0, SIZE1, SIZE2, SIZE3, SIZE4} where STREAM_TYPE is
0 (STDIN), 1 (STDOUT), or 2 (STDERR) and SIZE1, SIZE2, SIZE3, SIZE4 is
the size of the payload as a 4-byte big endian integer. When TTY is
enabled the stream is simply the raw data with no formatting.
Fixes #353

Current coverage is 36.14%

Merging #354 into master will increase coverage by +0.46% as of 824af85

@@            master    #354   diff @@
======================================
  Files           66      66
  Stmts         3197    3204     +7
  Branches       775     776     +1
  Methods          0       0
======================================
+ Hit           1141    1158    +17
+ Partial        110     109     -1
+ Missed        1946    1937     -9

Review entire Coverage Diff as of 824af85

Powered by Codecov. Updated on successful CI builds.

@rgrunber Thanks for the PR.


Support for relative positioning/auto layout

Yes, Autolayout integration is the missing piece from the puzzle. What about at least giving a roadmap so developers considering Pixate will be encouraged to hear that you are planning to "address" the issue?


arm64 package build (Ubuntu bionic deb)

Is your feature request related to a problem? Please describe.

I see there are tests and a CI pipeline for ARM64.  Is there a package built for Ubuntu Bionic ARM64 like "td-agent"?  Also, is there an ARM64 docker image?
Describe the solution you'd like

Either a package built for Ubuntu Bionic or instructions to create one.
Describe alternatives you've considered

I can use fluent-bit if I have to as I've compiled that for ARM64 but the rest of our cluster uses fluentd (td-agent).
Additional context

This is very useful. :)

is there an ARM64 docker image?

You can see arm related tags in docker hub.
Use it: https://hub.docker.com/r/fluent/fluentd/tags

Is there a package built for Ubuntu Bionic ARM64 like "td-agent"?

This is beta. See https://github.com/treasure-data/omnibus-td-agent/issues/209#issuecomment-568633733
This repository is not for docker image or td-agent. close.

Hi,
Thank you for showing me where these are.  There are 2 issues that you can see here: https://hub.docker.com/r/fluent/fluentd/tags

None of these tags look to be ARM64.  (They look to be armhf which was often used on 32 bit Raspberry Pis. That’s an Arm v7 architecture.). It is possible they will function on the Arm v8 core used by the ARM64 or aarch64 platforms.  That’ll require some testing.

All the Arm tags have their architecture set to linux/amd64 which can preclude them from being found automatically.

Thanks again!

None of these tags look to be ARM64

I can see arm64 images...

All the Arm tags have their architecture set to linux/amd64 which can preclude them from being found automatically.

How to set arm64? We use this script to push images: https://github.com/fluent/fluentd-docker-image/blob/master/post_push.erb
If you have a patch, it is great.

I confirmed that v1.8-arm64-debian image can run on RaspberryPi 3 AArch64 Wayland/Weston image:

What is the shortage of running on your ARM64 docker environment?


Tuple calls into DeepEquals

Call Equals on two Tuples that contain a class that throws a exception on one or more of it's properties, throws these exceptions.
https://deck.net/0f43cf562bd77cbf8c794131bf9be4f4
Expected Result
new Tuple(new A()).Equals(new Tuple(new A())) returns False.
https://repl.it/@Triogap/WiltedGraciousOutliers
Actual Result
new Tuple(new A()).Equals(new Tuple(new A())) throws NotImplementedException.
Tuple should not call into DeepEquals for the objects it contains, but should call Equals for every property. If all are true, Equals should return true.

Thanks for reporting the issue, @Triogap! We got it fixed and just merged it upstream. The fix will be included next Bridge release (see attached milestone).


Adding SASS & Styling Updates

The pull request will introduce the following changes:

The CSS files added in https://github.com/GitHugop/AnimeScreen/commit/80450718ec7a95964fc25ac934e6bb2e13fb0dd7 have converted to SASS files in the /assets/sass folder.
Said CSS files have been restructured for improved consistency and maintainability.
Styling has been tweaked a bit, including the shadow's opacity being reduced.

The registration form has been refined. The following image compares the old style with the new style.

@lap00zza
The code for the registration form have been separated into partials which are reusable across the website. I've further integrated them BEM convention into the code. Regarding autoprefixing, that's usually something which gets taken care off automatically during deployment.
Let me know what you think of the updates! Any suggestions will be taken into consideration.

The Facebook registration button has been taken out as per @GitHugop's request.

A few people are suggesting that the form should be more compact. What is your input on this?

Looks great. We probably should remove the "Name" field, since it might confuse users, and set username as default display name. They can change it later.

@GitHugop the Name textfield has been removed!
Other commits have been... committed... to make the form:

Smaller in height/more compact. This should keep (most) people happy.
Flexible with adding/removing elements by discarding the fixed height property. It expands/shrinks based on the contents.

Oh dear, we're really going over the line with the objective of this pull request.

@yumiris Remove the "Invalid password/email/username" part

The suggested changes seem to make the form prone to being hard to read.
Additionally, the compact layout makes it fit much more easily on mobile screens.
However, if you've got an additional design in mind, raise a new Issue and I'll take a look! :)


Feature/small changes to fixed width

Modified FixedWidthReader so that we can feed a line to a reader independently.
The modification were necessary for integrating fixed width files on Hadoop: https://github.com/datacleaner/DataCleaner/issues/1343

I feel a bit like I am missing a unit test for this kind of usage.
An I wonder if that will reveal a slight abstraction issue... The reader today takes a Resource as a constructor argument. Would you have any resource to provide it with when you are yourself providing the lines of data?

I am providing the lines of data. I need to be able to use this class without the BufferReader. I will add a unit test.

I need to wait for this PR: https://github.com/apache/metamodel/pull/103 if I am to avoid conflicts.

+0
Code is acceptable from me, but I think it would be better to separate "parser" from "reader" here. A reader is a stateful thing IMO - it has a pointer/cursor to some data that it is reading through and it is also closeable. The parsing part however can have a different scope or maybe even be completely stateless. So I would prefer if we make two different classes for it (the reader will probably use the parser, but not the other way around).

OK let's merge this and maybe some day improve it as per my remarks.


[Release] 2017.07.19 - Fix scroll behaviour on features page

You can release this. Socket changes are only on socket page which is not visible on production.


feat: Expand serde deserialize impl

Fixes #1338
The config now supports a list of integers and a single integer as the accept value.

Seems like the lychee::cli::test_suggests_url_alternatives test is flaky again.

Awesome PR. Lifesaver!
Yup, sorry about that. That happens when archive.org takes too long to send a response. We have exponential back off in place, but it might be better to disable the test altogether. 😕
I re-ran it and now the error seems to be relevant to the changes. Can you take a look?

I re-ran it and now the error seems to be relevant to the changes. Can you take a look?

Yes, will do.

The test ran fine multiple times on my local machine. The description states that it is flaky:

Unfortunately, LinkedIn does not always return 999, so this is a flaky test. We only check that the cache file doesn't contain any invalid status codes.

LinkedIn never fails to deliver. 😆 Merging. Thanks for the quick fix!🥇

LinkedIn never fails to deliver.

Lol!


Feat: set_hostname - Add fqdn capability

Add FQDN capability for set_hostname.
Request made for an HPC cluster. Seems reasonable.

ok for me, good feature  @oxedions

Good point :-)
On it

Thanks for reviewing.


make fm add beliefs resample data

The API endpoint for sending sensor data already resamples. That logic should also be available in the CLI command for reading in data from CSV.

Support is on its way: https://github.com/SeitaBV/timely-beliefs/pull/98


01_original.py not running

I tried to run the 1st code under dqn_speedup by running the command python 01_original.py and get the following error.
Traceback (most recent call last):   File "01_original.py", line 32, in <module>     exp_source = ptan.experience.ExperienceSourceFirstLast(env, agent, gamma=params['gamma'], steps_count=1)   File "/home/daksh/anaconda2/envs/torch/lib/python2.7/site-packages/ptan/experience.py", line 168, in __init__     super(ExperienceSourceFirstLast, self).__init__(env, agent, steps_count+1, steps_delta, vectorized=vectorized) TypeError: super() argument 1 must be type, not classobj
Please look into it and let me know.

I then tried to use the exact same versions of python modules as given in your requirements.txt and then get this error -
Traceback (most recent call last):   File "01_original.py", line 23, in <module>     env = ptan.common.wrappers.wrap_dqn(env)   File "/home/daksh/anaconda2/lib/python2.7/site-packages/ptan/common/wrappers.py", line 235, in wrap_dqn     env = ProcessFrame84(env)   File "/home/daksh/anaconda2/lib/python2.7/site-packages/ptan/common/wrappers.py", line 133, in __init__     self.observation_space = spaces.Box(low=0, high=255, shape=(84, 84, 1), dtype=np.uint8) TypeError: __init__() got an unexpected keyword argument 'dtype'


Question: How to disable file/inode id while searching?

First off, thanks for the awesome plugin, love it!
Now, I have this issue where oil shows a leading indicator of what I suspect is the file/inode id when I search in the buffer:

This behavior also shows in the screencast on the README, so I think it must be an intended feature.
I have no idea what to do with this information, so is there a way to turn it off? It is a bit irritating for me since it moves the text while searching.

You're looking for :help concealcursor. If you look at the config options, it defaults to "n". You probably want "nc" at least.

Thanks, that helped a lot! I did not realize what this was!

Just to save anybody who lands here and makes the same mistake I just did, concealcursor is local to window, so setting it directly in your init.vim will not work! oil.nvim has an win_options.concealcursor opt that will set this option specifically for oil.nvim buffers.


Gem is loaded before rails environment and it overrides the ActiveSupport #stringify_keys

Hello! Since it looks like this gem is not dependent on rails gem it gets loaded before the rails and overrides the default #stringify_keys method from rails which is causing very weird and almost untraceable issues especially in test environment!
Can we please add that dependency and ensure that it's not just overriding rails default behaviour or remove this monkey patch and switch to using #transform_keys(&:to_s) and #trasnform_values in one place where it's being used in the SDK. Here: https://github.com/livekit/server-sdk-ruby/blob/main/lib/livekit/grants.rb#L91
Using refinement should also limit the scope of the change of #stringify_keys

I've opened a PR https://github.com/livekit/server-sdk-ruby/pull/57 not sure if this would be acceptable solution to everyone there should be no functionality change to the gem 😄


Fix missing parenthesis in the documentation

Cosmetic and style PRs such as minor code reorganizations, adding comments, fixing a minor typo, etc. will not be accepted as first contribution (see https://github.com/godotengine/godot/issues/66666)

Um, okay. But can someone, who is not a first-time contributor, fix this typo?

Consider grouping such fixes together in a single commit/PR so that it's easier to review and merge

So, in the period of hacktoberfest, you should do several changes (probably in different files) in order such PR to be accepted.

Got it.
(Though I'd like to point out that I'm not participating in any kind of activities—I'm participating in an open source project. Well, even if it means just a single byte fix.)

Alright, @akien-mga told me that this can be merged cuz it's an actual bugfix as the documentation is wrong. He also said it's preferable if such fixes are grouped together in a single commit, as we want to avoid contributors who open multiple PRs just for a single typo each.

Thanks, congrats with first contribution, and sorry for this little misunderstanding.


Configuration contains keymap conflicts

Configuration contains keymap conflicts

The ':checkhealth' output contains the following WARNINGs:
which-key: require("which-key.health").check()

WhichKey: checking conflicting keymaps ~
- WARNING conflicting keymap exists for mode **"n"**, lhs: **"gb"**
- rhs: `<Plug>(comment_toggle_blockwise)`
- WARNING conflicting keymap exists for mode **"n"**, lhs: **"gc"**
- rhs: `<Plug>(comment_toggle_linewise)`

To Reproduce

:checkhealth

Desktop

OS: MacOS 13.3.1a
Terminal: iTerm2 3.4.19

Neovim Version

NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/opt/homebrew/Cellar/neovim/0.9.1/share/nvim"

Hi there!
Thanks for the report. Please feel free to file an issue against the upstream repository as this is the default for that plugin.


AbstractBlockProcessor | failed persisting block

Description
As a staker, I want the execution client to run and recover from errors so that I can stake successfully.
Acceptance Criteria

Besu gracefully recovers from a storage error, or restarts when it encounters a fatal error

Steps to Reproduce (Bug)
I have not been able to reproduce this issue. I reported it on the #besu-mainnet discord channel and @siladu suggested I file an issue.
Expected behavior: [What you expect to happen]
Besu recovers and continues to operate after an error.
Actual behavior: [What actually happens]
Besu continues to attempt the failed storage operation and never recovers. After a restart of Besu, operation returns to normal.
Frequency: [What percentage of the time does it occur?]
I have encountered this once on 5/6/23.
Edit: It happened again on 5/7/23.
Logs (if a bug)
The following is the error message and stack trace that repeated for 15 minutes until I restarted Besu:
May 05 20:58:28 whopper besu[435498]: 2023-05-05 20:58:28.359-04:00 | vert.x-worker-thread-0 | ERROR | AbstractBlockProcessor | failed persisting block
May 05 20:58:28 whopper besu[435498]: org.hyperledger.besu.plugin.services.exception.StorageException: org.rocksdb.RocksDBException: While fdatasync: /var/lib/besu/database/416929.sst: Input/output error
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.plugin.services.storage.rocksdb.segmented.RocksDBColumnarKeyValueStorage$RocksDbTransaction.commit(RocksDBColumnarKeyValueStorage.java:391)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.services.kvstore.SegmentedKeyValueStorageTransactionTransitionValidatorDecorator.commit(SegmentedKeyValueStorageTransactionTransitionValidatorDecorator.java:59)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.services.kvstore.SegmentedKeyValueStorageAdapter$1.commit(SegmentedKeyValueStorageAdapter.java:114)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.bonsai.storage.BonsaiWorldStateKeyValueStorage$Updater.commit(BonsaiWorldStateKeyValueStorage.java:482)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.bonsai.worldview.BonsaiWorldState.persist(BonsaiWorldState.java:408)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.mainnet.AbstractBlockProcessor.processBlock(AbstractBlockProcessor.java:181)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.mainnet.BlockProcessor.processBlock(BlockProcessor.java:79)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.MainnetBlockValidator.processBlock(MainnetBlockValidator.java:207)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.MainnetBlockValidator.validateAndProcessBlock(MainnetBlockValidator.java:135)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.MainnetBlockValidator.validateAndProcessBlock(MainnetBlockValidator.java:83)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.consensus.merge.blockcreation.MergeCoordinator.validateBlock(MergeCoordinator.java:516)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.consensus.merge.blockcreation.MergeCoordinator.rememberBlock(MergeCoordinator.java:546)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.consensus.merge.blockcreation.TransitionCoordinator.rememberBlock(TransitionCoordinator.java:157)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.api.jsonrpc.internal.methods.engine.AbstractEngineNewPayload.syncResponse(AbstractEngineNewPayload.java:244)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.ethereum.api.jsonrpc.internal.methods.ExecutionEngineJsonRpcMethod.lambda$response$0(ExecutionEngineJsonRpcMethod.java:73)
May 05 20:58:28 whopper besu[435498]:         at io.vertx.core.impl.ContextBase.lambda$null$0(ContextBase.java:137)
May 05 20:58:28 whopper besu[435498]:         at io.vertx.core.impl.ContextInternal.dispatch(ContextInternal.java:264)
May 05 20:58:28 whopper besu[435498]:         at io.vertx.core.impl.ContextBase.lambda$executeBlocking$1(ContextBase.java:135)
May 05 20:58:28 whopper besu[435498]:         at io.vertx.core.impl.TaskQueue.run(TaskQueue.java:76)
May 05 20:58:28 whopper besu[435498]:         at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
May 05 20:58:28 whopper besu[435498]:         at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
May 05 20:58:28 whopper besu[435498]:         at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
May 05 20:58:28 whopper besu[435498]:         at java.base/java.lang.Thread.run(Thread.java:833)
May 05 20:58:28 whopper besu[435498]: Caused by: org.rocksdb.RocksDBException: While fdatasync: /var/lib/besu/database/416929.sst: Input/output error
May 05 20:58:28 whopper besu[435498]:         at org.rocksdb.Transaction.commit(Native Method)
May 05 20:58:28 whopper besu[435498]:         at org.rocksdb.Transaction.commit(Transaction.java:208)
May 05 20:58:28 whopper besu[435498]:         at org.hyperledger.besu.plugin.services.storage.rocksdb.segmented.RocksDBColumnarKeyValueStorage$RocksDbTransaction.commit(RocksDBColumnarKeyValueStorage.java:385)
May 05 20:58:28 whopper besu[435498]:         ... 22 more
May 05 20:58:28 whopper besu[435498]: 2023-05-05 20:58:28.363-04:00 | vert.x-worker-thread-0 | INFO  | MainnetBlockValidator | Invalid block 17198146 (0x198887087bec51add61a58d3d7f329b71b730a7a8402f50ff96e6e76227cb2f3): Optional[org.rocksdb.RocksDBException: While fdatasync: /var/lib/besu/database/416929.sst: Input/output error], caused by org.hyperledger.besu.plugin.services.exception.StorageException: org.rocksdb.RocksDBException: While fdatasync: /var/lib/besu/database/416929.sst: Input/output error

Full logs from Besu and Teku from the beginning of the day when the failure occurred:
besu.log
teku.log
Versions (Add all that apply)

Software version: 23.4.0
Java version:

openjdk version "17.0.6" 2023-01-17
OpenJDK Runtime Environment (build 17.0.6+10-Ubuntu-0ubuntu122.04)
OpenJDK 64-Bit Server VM (build 17.0.6+10-Ubuntu-0ubuntu122.04, mixed mode, sharing)

OS Name & Version: Ubuntu 22.04.2 LTS
Kernel Version: 5.15.0-70-generic #77-Ubuntu SMP
Consensus Client & Version if using Proof of Stake: Teku 23.4.0

Additional Information (Add any of the following or anything else that may be relevant)

Besu setup info

Environment="JAVA_OPTS=-Xmx8g"
ExecStart=/usr/local/bin/besu/bin/besu \
  --network=mainnet \
  --sync-mode=X_SNAP \
  --data-path=/var/lib/besu \
  --data-storage-format=BONSAI \
  --engine-jwt-secret=/var/lib/jwtsecret/jwt.hex \
  --metrics-enabled=true \
  --rpc-http-enabled

System info - AMD EPYC 3251 8-Core Processor, 64GB memory, 4TB SSD (2.6TB available)

It may be worth noting that I got another storage error in Teku about ~9 hours later, which caused Teku to shut itself down. After it started back up, it resumed operating normally.
FATAL - Exiting due to fatal error in RetryingStorageUpdateChannel tech.pegasys.teku.infrastructure.exceptions.FatalServiceFailureException: java.util.concurrent.CompletionException: org.iq80.leveldb.DBException: IO error: /var/lib/teku/beacon/db/3055058.ldb: Input/output error (See log file for full stack trace)

Thanks for the report. Root cause seems like a hardware issue, but I think that besu should be able to recover better from this.

Hi @superlemon0, we've done some analysis of the issue you've reported and based on all the logs you've provided we've come to the conclusion that the block validation failure you get is due to a faulty hardware which ends up affecting RocksDB, the database used by besu. At that stage there isn't much besu can't do rather that what we're already doing, which is retry. Looking at the RocksDb github it seems that there isn't much more they can do about it as well (https://github.com/facebook/rocksdb/issues/1403#issuecomment-254612859).


Treat unused-parameter warnings as errors in CMake

Search before asking

[X] I had searched in the issues and found no similar issues.

Motivation
See https://gcc.gnu.org/onlinedocs/gcc-3.2/gcc/Warning-Options.html.
Solution
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Recently I happened to learn about the project structure through CMakeLists.txt.
I'd be happy to try to resolve the issue. Of course, if he (#2267 ) didn't choose this issue
At the same time, I am a beginner and am happy to accept other tasks to understand the project faster.

/assign


Removing as Xen VMs were never implemented

cc @agrare

In the future, please ensure cross repo is run on model and factory removals (see issues opened by @himdel )


[Feature Request]: Add new argument to inject PydanticOutputParser into utils.get_program_for_llm()

Feature Description
In core.program.utils.get_program_for_llm(), only default PydanticOutputParser would be used.
Might you add a new argument to send customized output_parser in?
Reason
In current codes, the PydanticProgram constructed from get_program_for_llm() could only use default PydanticOutputParser
Value of Feature
With this feature, it would be easy to modify the prompt in the PydanticOutputParser object, so the PydanticProgram could use different prompt for output_parser other than default prompt

Hey,
I open the bug on this issue with a possible fix


simplify the ctest command line

Fixes #12

I am not sure what was happening, but removing "things that I don't know what they are doing" from ctest line seems to fix things on my machine.

11: ==3909==LeakSanitizer has encountered a fatal error.
11: ==3909==HINT: For debugging, try setting environment variable LSAN_OPTIONS=verbosity=1:log_threads=1
11: ==3909==HINT: LeakSanitizer does not work under ptrace (strace, gdb, etc)

ok, I am going to try that

I merged in your commit from #13 and looked what ASAN prints while the tests run. And it gives an error from testCorrelationId. This is something to look into (later). Without the commit from 13, asan dies at the end, without userful error message. So there is a lot to investigate in this. We do need working sanitizers to ensure we don't have memory bugs in the binding.
https://github.com/jiridanek/qpid-proton/pull/14/checks?check_run_id=2622446949#step:14:53
11: testCorrelationId (test_message.AccessorsTest) ... =================================================================
11: ==4005==ERROR: AddressSanitizer: heap-use-after-free on address 0x606000003cb0 at pc 0x7f1f2d27207d bp 0x7ffcc444d0c0 sp 0x7ffcc444c868
11: READ of size 16 at 0x606000003cb0 thread T0
11:     #0 0x7f1f2d27207c  (/lib/x86_64-linux-gnu/libasan.so.5+0x4e07c)
11:     #1 0x7f1f2780d8a4 in b_string c/_cffi_backend.c:6730
11:     #2 0x7f1f2cff070e in _PyCFunction_FastCallDict Objects/methodobject.c:231

Got error on windows. And also, there is one more test failing (even in jobs without sanitizers) than on the python3_cffi branch, for some reason.
Test not available without configuration.  (Missing "-C <config>"?)

The new issue that appeared is ```
11: ======================================================================
11: ERROR: testRoundTrip (test_message.CodecTest)
11: ----------------------------------------------------------------------
11: Traceback (most recent call last):
11:   File "/home/runner/work/qpid-proton/qpid-proton/python/tests/proton_tests/test_message.py", line 227, in testRoundTrip
11:     assert self.msg.correlation_id == msg2.correlation_id, (self.msg.correlation_id, msg2.correlation_id)
11:   File "/home/runner/work/qpid-proton/qpid-proton/python/proton/_message.py", line 371, in _get_correlation_id
11:     return self._correlation_id.get_object()
11:   File "/home/runner/work/qpid-proton/qpid-proton/python/proton/_data.py", line 1729, in get_object
11:     return getter(self)
11:   File "/home/runner/work/qpid-proton/qpid-proton/python/proton/_data.py", line 1416, in get_uuid
11:     bytes=ffi.string(pn_data_get_uuid(self._data).bytes)
11:   File "/opt/hostedtoolcache/Python/3.6.13/x64/lib/python3.6/uuid.py", line 149, in init
11:     raise ValueError('bytes is not a 16-char string')
11: ValueError: bytes is not a 16-char string

The issue with testRoundTrip disappeared. It could be the test fails only sometimes, randomly. We will see if it appears again.

Looks OK now. I reported the ASAN issue as https://github.com/jiridanek/qpid-proton/issues/19


feat: make primary cache accessible from Context

Alternative to #2051

This should rather target next imo, since it's a feature.

But I think we should we can go this way, just exposing the cache, and see if anybody would need actually more.


.NET Core MSBuild is missing the LC task

From @alienisty on April 24, 2017 1:58
Steps to reproduce
Just create a project with a licenses.licx file definition and build the project with

dotnet build project.csproj

Expected  behavior
The project builds and the license information are compiled in the binaries
Actual behavior
The following error is raised:

C:\Program Files\dotnet\sdk\1.0.3\Microsoft.Common.CurrentVersion.targets(2977,5): error MSB4062: The "Microsoft.Build.Tasks.LC" task could not be loaded from the assembly Microsoft.Build.Tasks.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a.  Confirm that the  decl
aration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask.
0 Warning(s)
1 Error(s)

Environment data
dotnet --info output:

.NET Command Line Tools (1.0.3)
Product Information:
Version:            1.0.3
Commit SHA-1 hash:  37224c9917
Runtime Environment:
OS Name:     Windows
OS Version:  6.3.9600
OS Platform: Windows
RID:         win81-x64
Base Path:   C:\Program Files\dotnet\sdk\1.0.3

Copied from original issue: dotnet/cli#6389

From @alienisty on April 24, 2017 2:1
Note that building the project with MSBuild provided with Visual Studio 2017 builds the project fine.
I also checked the core tasks dlls from the donet core SDK and VS2017 and the one from the dotnet core sdk is actually missing the LC task, while the VS2017 one has it.

I'm also experiencing this problem

The LC task wasn't ported to .NET Core because lc.exe isn't available in a cross-platform version (that I know of, anyway). The task just calls the executable, so it wouldn't work on non-Windows platforms.
The best way to deal with this today is to use full-framework msbuild.exe, which does have the task.
@rustyhann, you're saying that's not working for you, which is unexpected. How exactly are you invoking the build? Could you share a diagnostic-level build log?

@rainersigwald
Thank you for the quick reply,
I'm working on generating a diagnostic log now. While that's running I can provide some background on our project.
We started the project as a .NET Core project (using dotnet new), then changed the target to "net462". We're doing this as a way to move to .NET Core completely while we migrate our legacy libraries. The full path we're using in our build script is:
"'C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\15.0\Bin\MSBuild.exe'"
We switched to this when we migrated to Visual Studio 2017. It solved most of our build issues until we ran into the lc.exe error.
I'll post the diagnostic build log when it's finished.

When this issue will be fixed? I faced the same issue with Teletik license file.

You can work around this by building with MSBuild and deploying with MSDeploy. The .Net Core tools will not work. The root issue for our scenario was dotnet publish was calling dotnet build. The no-compile option was deprecated in dotnet build. Even if we used MSBuild before calling dotnet publish we still had the issue. Switching to MSDeploy was the answer.

The LC task wasn't ported to .NET Core because lc.exe isn't available in a cross-platform version (that I know of, anyway). The task just calls the executable, so it wouldn't work on non-Windows platforms.

Makes sense. But does .NET Core 3's support of WinForms controls (on Windows) change things?

Makes sense. But does .NET Core 3's support of WinForms controls (on Windows) change things?

That's a good question. @merriemcgaw do you know if there are plans to bring lc.exe to core?

Thanks for forwarding on the question.
Note that even if the "dotnet" command does not get support, a tweak to the (.NET Framework) msbuild task would still be useful. Similarly to #2836, (.NET Framework) msbuild passes giant command line arguments to lc.exe, when targetting netcoreapp3.0.

I know it's been discussed, but I don't know where the discussions have landed yet.

Running desktop lc.exe on core assemblies is busted. See https://github.com/dotnet/corefx/issues/24200#issuecomment-461803927.  I think this needs to be fixed. We have had a few customers ask for the licensing functionality in netcore 3.0.

@ericstj Thanks for pointing to the issue I faced. I'd like to add we need this not just for licensing components/controls but also plain class libraries. In fact right now for .NET core we are trying this only for the plain class library case.

@sansjunk thanks for the scenario explanation. I will be working with my team to make a plan for when/how we migrate lc.exe support to Core.

Should we move this issue to the winforms repo?

Should we move this issue to the winforms repo?

WPF uses the same licensing model

Is this issue expected to be fixed for the final .NET Core 3.0 release?

This issue was moved to dotnet/winforms#1462


Make enums for all currently published extensions

Use the list from the SPIR-V registry page.  Also, capture it as
a string so it's much easier to update via copy-paste.
The validator will accept modules that declare these known
extensions.  However, we might not know about new tokens or
instructions declared in them.  For that we need grammar updates
applied to SPIRV-Headers.

Rebased and pushed into master as 0d3b8329a43


Clarify the license

The license of testrpc is ambiguous at this moment:

LICENSE specifies MIT
README.md specifies MPL2.0
package.json is lacking a license field

It would be useful deciding on a license :)
Update: LICENSE is there since the first commit and MPL2.0 was introduced via a README change: https://github.com/ethereumjs/testrpc/commit/ee7933b42b102a742bb22dbd7d1f2c1cc0192c49. Therefore it seems the whole repository is MIT?

@wanderer What's ethereumjs's official license?
The whole repository is likely MIT right now, but it looks like we might need to relicense to MPL2.0 to be in line with ethereumjs. Good catch.

It would be quite a chore relicensing to MPL2.0 as you'd need to ask every* contributor to testrpc.

@tcoulter its at the authors discussion

From talking to @tcoulter I think that it was a mistake to have committed the MPL license to the README. Given that the LICENSE file was there from the start and that this came later, adding a license conflict, I think it's fair to resolve this conflict by updating the notice in the README to be aligned with the LICENSE file. I'll take this one on.

Fixed as of a85c95a