falling animation is not working when showToDamagerOnly = false

it stay in air and gone (similar behavior as falling_animation = false)
plugin version
> spectraldamage
[15:53:45 INFO]: SpectralDamage - A damage indicator plugin (v2.6.0)
[15:53:45 INFO]: Created by TwiDev (https://github.com/twidev)
[15:53:45 INFO]:  - /spectraldamage reload - Reload configuration

server version
> version
[15:55:51 INFO]: Checking version, please wait...
[15:55:51 INFO]: Current: git-Purpur-2172 (MC: 1.20.4)*
Previous: git-Purpur-2171 (MC: 1.20.4)
* You are running the latest version

Hi
this is because the falling animation is only supported via packets (client side) and therefore with the option showToDamagerOnly = true
Maybe in a future update this will be fixed but I need to look into some optimization issues to implement this feature


PS4  DualShock controller support?

Monterey, M1 Max here.
It seems to connect to macos and works in Steam Big picture, but does not respond in game (overcooked 2).
I will use nintendo pro controller with this app (as it has the same behavior of only working in Big Picture and not in game, so mapping buttons to keyboard would be the way to go for the time being), Nintendo seems to be the only controllers supported? Maybe the name can be more clear about this. I do see that the first sentence in readme specifies this but I didn't actually read it.

Yeah, I can use my Xbox Series X / PS4 controller on M1 Mac Pro, Monterey with like Dolphin for example, so they DO work... but this app doesn't detect the controllers at all for some reason...

Maybe I'm wrong about this one, but isn't this app supposed to be specifically for Nintendo Joycons and Pro controllers? I don't think it's supposed to support an Xbox or Play Station controller.

Yeah, you are right... I didn't think about the name at first, heh.   I may start to learn Xcode and see if i can make something myself... we will see how that goes... I know C# / C++ a tiny bit / WPF / Python.

Thanks @MCMi460 for providing the correct answer


[Sync Enhancement | Data Access Checker] Add a deployment config called ClientLocation.userAssignedLocation

Keyword Definition

Client - An entity that has data collected against it (e.g. Family/Household, Family/Household member, Patient, Service Point or Product).
Client Location - This term is used to refer to location(s) associated with the client.

This would mean the location where the family or family member lives for the case of the SID (Summit Institute for Development) applications
This would mean the service point of where data collected belongs to for the case of EUSM (End User Supply Monitoring)

Sync Locations - These are the locations used to filter the data to be synced to the application.

Describe the feature request.

OpenSRP 2.0 is adding a new Sync Strategy called Client Location. This strategy will sync data based on the Client Location meta.tag.
It will heavily depend on the Sync Locations, which can be generated in 2 ways i.e

User Assigned Locations - These are the locations assigned to a user via OpenSRP web.
User Selected Locations - These are the locations selected by the user on app log in.

Technical Description of enhancement

[ ] Create a deployment config for the FHIR InfoGateway. This config should be created the same way all the other FHIR Info gatway configs are.  It should be named ClientLocation.userAssignedLocation i.e {"clientLocation":{"userAssignedLocation":true}}
[ ] true should be the config default

Acceptance criteria

[ ]  Have a config that can only be defined and altered during deployment

Implementation plan (For Engineers)
The plan for implementing the solution e.g. via a description or a check list for the various ordered tasks that will need to be completed.
i.e. Describe how you intend to solve the problem

Is this defining the sync strategy? Or the metatag? I think the sync strategy

Hmm, does this go in the OpenSRP mobile app or in the gateway? I don't think we'd want the app to be sending something that affects how the validation occurs, because in an adversarial situation they could change that validation.

@pld as discussed this goes to the gateway

nb following or terminology change, updating this to relatedEntityLocation

@ndegwamartin @lincmba closing this after our chat on Friday. We do not need it.


Provider for alternative display of json objects

Hello,
I am trying to setup a language where some of my functions take complex objects as parameters, for example:
myFunction({type:"complexObject",name:"object1", hasValue:true}, {type:"complexValue", value:2})
I would like to be able to ease reading for the user by folding json objects and display them in an alternative short way while still providing a mean for the user to unfold them to see full content.
So by default the IDE would show:
myFunction(_object1_,_2_)
and user would be able to click on object1 or 2 to display the full json content. Edition would require typing the full json object, I do not have issue with this.
Any advice on how to proceed?

Maybe extending folderProvider could solve my issue?

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


ES-7.x

Hello! Will there be updates for ES-7.x?

Is this a dead project seeing that the author of the plugin seems to not be responding for 6 months?

Sorry for the slow reply.  You are correct, I am no longer actively developing this project but happy to help out if there is interest.  I have also put a note on the README.


BUG:  When I set vim.opt.cmdheight=0, the prompt (add new file when use_popups_for_input = false) disappears (v3.10)!

Did you check docs and existing issues?

[X] I have read all the docs.
[X] I have searched the existing issues.
[X] I have searched the existing discussions.

Neovim Version (nvim -v)
0.9.1
Operating System / Version
MacOS Monterey
Describe the Bug
After adding newline when use_popups_for_input = false by @pysan3, when I set vim.opt.cmdheight=0 the add new file or directory prompt does not show, only one line to input the new file name or directory.
Screenshots, Traceback
When vim.opt.cmdheight not set:

When vim.opt.cmdheight set to 0:

Hmm, I usually use nightly so I thought that was the difference but even when I downgrade to v0.9.1 I still cannot reproduce the issue.
Here are my screenshots.

vim.opt.cmdheight = 0

vim.opt.cmdheight = 1

vim.opt.cmdheight = 2

The following code is what neo-tree is basically doing in the back. Could you execute this code in your environment?

Open neovim
Create a file named test.lua
Copy the following code
Run :so once

vim.keymap.set("n", "<Leader>r", "<Cmd>source<CR>", {})
vim.cmd("language en_US.utf8")

local opts = {
  prompt = "Question: foo bar" .. "\n",
  default = "~/",
}

vim.ui.input(opts, function(msg)
  vim.print(string.format([[msg: %s]], vim.inspect(msg)))
end)

:lua vim.opt.cmdheight = 0
<Leader>r
See what happens
:lua vim.opt.cmdheight = 1
<Leader>r
See what happens
:lua vim.opt.cmdheight = 2
<Leader>r
See what happens

Ok, I was able to reproduce inside neo-tree but I'm now confused that the above code does work despite both should be doing the exact same thing.
I think I'll deal this with a workaround.

It works! @pysan3 Thank you very much!


Not registering anything?

I'm trying to understand, why it's not registering any data.
When I use my phones flash to trigger the photodiode, the D0 led blinks, but nothing gets registeret, neither in the logs nor in HA.
Setup looks like this

Wemos D1 Mini Lite
3.3V -> VCC
GND -> GND
D2 -> D0

My ESPHome yaml looks like this
substitutions:
  device_name: energy
  friendly_name: House
  device_description: "Measure your energy consumption with the pulse LED on your smart meter"
  pulse_pin: GPIO4
  status_led: GPIO5

  pulse_rate: '1000'

  web_username: 'glow'
  web_password: 'esphome'

esphome:
  name: '${device_name}'

esp8266:
  board: d1_mini_lite

wifi:
  ssid: !secret wifi_ssid
  password: !secret wifi_password

output:
  - platform: gpio
    pin: GPIO2
    id: output_red
  - platform: gpio
    pin: GPIO4
    id: output_green

light:
  - platform: binary
    internal: true
    id: led_red
    name: Red
    output: output_red

status_led:
  pin:
    number: ${status_led}

text_sensor:
  - platform: version
    hide_timestamp: true
    name: '${friendly_name} - ESPHome Version'
  - platform: wifi_info
    ip_address:
      name: '${friendly_name} - IP Address'
      icon: mdi:wifi
    ssid:
      name: '${friendly_name} - Connected SSID'
      icon: mdi:wifi-strength-2

sensor:
  - platform: pulse_meter
    id: energy_pulse_meter
    name: '${friendly_name} - Power consumption'
    unit_of_measurement: 'W'
    state_class: measurement
    device_class: power
    icon: mdi:flash-outline
    accuracy_decimals: 0
    pin: ${pulse_pin}
    on_value:
      then:
        - light.turn_on:
            id: led_red
        - delay: 0.5s
        - light.turn_off:
            id: led_red
    filters:
      - lambda: return x * ((60.0 / ${pulse_rate}) * 1000.0);
    total:
      name: '${friendly_name} - Total energy'
      unit_of_measurement: 'kWh'
      icon: mdi:circle-slice-3
      state_class: total_increasing
      device_class: energy
      accuracy_decimals: 3
      filters:
        - lambda: return x * (1.0 / ${pulse_rate});

Any idea why nothing gets registered?

You now have the pulse led and status led both on GPIO4, set the pulse led to D6 (GPIO12) 😉

You now have the pulse led and status led both on GPIO4, set the pulse led to D6 (GPIO12) 😉

You're right. But I originally had it on D6 (GPIO12) as the documentation said, but still the exact same thing? No readings at all. I changed there GPIO because I thought that the D6 was damaged or something..
I also checked to make sure that the photodiode is soldered correctly, as per documentation.

Have you tried another cable? Or another ESP?

Have you tried another cable? Or another ESP?

Yes, I have tried both with an wemos d1 mini and an esp32, and with 3 different micro USB cables. Exact same.. no readings at all

What color is the pulse LED? Your issue might be similar to this one #62?

What color is the pulse LED? Your issue might be similar to this one #62?

I haven't connected the RGB diode, as I don't have one.
Could that be the issue? That I haven't connected an RGB diode?

I don't think that has anything to do with each other.


Dynamic Inventory references are confusing

Hello,
guide_azure.rst  has a mix of references for Dynamic Inventory.   It references azure.py, azure_rm.py  and then the actual git repo has windows_azure.py
Which is it?  :)   Thanks!

Made some updates that should make it more clear. The Azure Resource Manage inventory script is azure_rm.py.
It's specifically not mentioned in the guide in hopes of avoiding confusion, but windows_azure.py is the old inventory script that talks to service manager.


Not able to install feature odl-atrium-all

I tried to install the feature odl-atrium-all but I'm seeing following error.
feature:install odl-atrium-all
Error executing command: Can't install feature odl-atrium-all/0.0.0:
Could not start bundle mvn:org.opendaylight.atrium/routingconfig-impl/1.0-SNAPSHOT in feature(s) atrium-routingconfig-1.0-SNAPSHOT: The bundle "org.opendaylight.atrium.routingconfig-impl_1.0.0.SNAPSHOT [273]" could not be resolved. Reason: Missing Constraint: Import-Package: org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.hostservice.api.rev150725; version="[1.0.0,2.0.0)"

As indicated in the README, the correct order for installing features is

feature:install odl-l2switch-all
feature:install odl-didm-all
feature:install odl-atrium-all

If you directly install odl-atrium-all first, then you will see the error posted by you.

Thanks Manoj,
I am able to install the features as per the mentioned order.
I would suggest we will leave this issue open to be fixed later.

Keeping it in open state as suggested by Subhash.

Closing this issue
Readme is updated with proper steps
feature configuration is modified to install all the dependent features as part of odl-atrium-all .


Add kratos_external_idp relation interface

Description
In order to connect Ory Kratos with an external OP (e.g. Azure AD or Okta) we need to provide Kratos a set of client credentials. We are going to do this through an integrator charm. In addition, this same interface can be used by charmed OPs that want to connect with Kratos for identity brokering (if we have any such use case in the future).
Future work
Credentials for only one provider can be passed through this, but if we want in the future we can choose to extend this to pass credentials for multiple providers at once.
In future versions, we are probably going to need to add fields like scopes and requested_claims to the relation.
Usage
The provider (integrator charm) will have to provide the client credentials, along with information about the provider in the application databag:
  relation-info:
  - endpoint: kratos_external_idp
    relation-endpoint: kratos_external_idp
    application_data:
      client_id: client_id
      provider: microsoft
      secret_backend: relation
      microsoft:
        client_secret: cl1ent-s3cRet
        tenant_id:<PHONE_NUMBER>424242

The requirer (Kratos charm), once the external provider has been registered, will have to provide the redirect_uri and the provider_id:
  relation-info:
  - endpoint: kratos_external_idp
    related-endpoint: kratos_external_idp
    application-data:
        redirect_uri: https://example.kratos.com/self-service/methods/oidc/callback/callback
        provider_id: microsoft

While resulting in a very log file the if approach used to select the profiles makes sense to me.
I think that we should not have "default": "" whenever we require an IDP specific value (it's fine for properties like arbitrary names). With the current code if the user forgets to input a value like the Azure AD tenant ID the resulting data will be valid, even if we know all tenant IDs will be different from ""

While resulting in a very log file the if approach used to select the profiles makes sense to me.

Is it intentional that you only want to allow one external provider at the time? Without any deeper understanding, it sounds like an odd limitation to me, but maybe that is how the charmed workload of the integrator charm will work?
Would this be a preferable approach?
{
 ...
"providers": {
        "type": "array",
        "items": {
            "anyOf": [{
                "type": "object",
                "required": [
                    "name",
                    "client_secret",
                    "tenant_id"
                ],
                "additionalProperties": true,
                "properties": {
                    "type": { "enum": [ "microsoft" ] },
                    "client_secret": { "$ref": "/schemas/client_secret" },
                    "tenant_id": {
                        "title": "The Azure AD tenant_id which the users belong to",
                        "description": "The Azure AD tenant_id which the users belong to",
                        "type": "string"
                    }
                }
            },
            // once for each provider schema
        ]}
    }
}

In addition to make the code easier to grok due to less branching statements to parse, it has the added benefit of allowing users to specify N providers through the same relation

Is it intentional that you only want to allow one external provider at the time? Without any deeper understanding, it sounds like an odd limitation to me, but maybe that is how the charmed workload of the integrator charm will work?

Yes, it is intentional. We could allow for one relation to define clients for multiple IdPs, or even multiple clients for the same IdP. BUT for our use case I don't think that would make sense as we want to have 1 integrator per external IdP to make configuration simple.
Having said that, we could make this schema more flexible and allow multiple IdPs. In this case we will implement all that logic (of only allowing one provider) in the integrator and use only a subset of the schema's capabilities.
Perhaps this is a better approach, I will look into it.

As for the name of the schema itself, is the kratos_ prefix really wanted? It sounds to me like this would more or less be the exact schema we'd want to use for ANY identity broker that would want to integrate with external IDPs?

I thought about that. The way the data is structured in the databag is tailored to the Kratos configuration. Would this same relation be usable for other Identity brokers? Probably, but we can't be sure that all brokers will understand all those provider names (google, slack, gitlab) and have the issuer_url hard coded. In addition, the provider_id in the requirer's databag is Kratos specific.
Generalizing this schema would mean that we would treat all providers in the same way and it would complicate the required logic in both sides of the relation.

I pushed some changes that should make the schema more in line with what you had in mind @simskij.
The Provider's databag is an array and the Requirer's databag is also an array, each entry in the Requirer's databag corresponds to the entry with the same index in the Provider's databag.

This is ready for review.
TBH I am not a big fan of having nested fields in our schema. I think it would be simpler/cleaner if we only allowed one provider per databag and had all the fields on the top level, e.g:
{
  "provider_type": "",
  "client_id": "",
  "secret_backend": "",
  "client_secret": "",
  "issuer_url": "",
  "microsoft_tenant_id": "",
  "apple_private_key": "",
  "apple_private_key_id": "",
  "apple_team_id": ""
}

In this schema we would allow the apple/microsoft/generic fields only if the provider is of that type.
If everyone is okay with refactoring to the flat format I can change it, else let's move forward with this (after review is done). I have no strong opinion on this one (I prefer the flat approach, but I don't know if it is in line with the relation interfaces style) and would like to merge it in order to unblock development.


Use NuGetPackageRoot instead of NUGET_PACKAGES

Summary
Using NUGET_PACKAGES property for looking up paths of NuGet package files works on Windows but not on non-Windows due to extra slashes. This causes the removal of the extra trace event library files to not work on non-Windows platforms. Use NuGetPackageRoot instead which is correctly normalized.
Example build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2096170&view=results
I'll only complete this PR if the referenced build produces the correct removal.

Release Notes Entry

/backport to release/7.0

/backport to release/6.x


Move kubectl tests in common test jobs into their own jobs

The test results on the test grid page show the results of test jobs running both non-cli tests and cli tests.  Isolate the signal for cli tests from non-cli tests, we should move the cli tests into their own test jobs.

If you want, I can help do this, seems lots time passed. :-)

/remove-lifecycle stale

/remove-priority important-soon
/priority P2

/remove-lifecycle stale

/remove-lifecycle stale


Unexploitable Error Reporting in API Responses

Hello @samchon ,
When sending requests that contain multiple typia errors, the API appears to halt and return an error response upon encountering the first error. This behavior means we need to go through multiple rounds of request correction to uncover and handle all the errors in the original request.
Here is an example:
{
    "status": 400,
    "error": "Bad Request",
    "message": "Validation failed",
    "details": [
        {
            "path": "email",
            "message": "Must be a valid email address"
        },
        {
            "path": "password",
            "message": "Must be at least 8 characters"
        }
    ]
}

If you've installed nestia through npx nestia setup command, then you can see such comment in plugins option. Change the validate: assert property value to validate: validate, then you may come true what you want.

Thanks you! This should be the default behavior.

Anyway, I had to write guide documents about this issue, but have delayed for a long time.
I'll keep this issue, and will close after writing guide documents as FAQ corner in someday (cannot sure when).


Web Console Account/Environment/Share Metrics Tabs

Incorporate a "Metrics" tab into the web console for each of the major components.

Turn the shared metrics interface into a (DRY) component.


allow sweeping of base tokens

Resolved Issues

fixes https://github.com/spearbit-audits/delv-review/issues/35

Description
The Hyperdrive pool should only custody vault shares (deposited base tokens are converted to shares, base tokens withdrawn from the vault are paid out immediately). Any lingering base token balance of the contract is by mistake and sweep should therefore be able to transfer out these tokens.
Review Checklists
Please check each item before approving the pull request. While going
through the checklist, it is recommended to leave comments on items that are
referenced in the checklist to make sure that they are reviewed. If there are
multiple reviewers, copy the checklists into sections titled ## [Reviewer Name].
If the PR doesn't touch Solidity and/or Rust, the corresponding checklist can
be removed.
@jalextowle
Solidity

[ ] Tokens

[ ] Do all approve calls use forceApprove?
[ ] Do all transfer calls use safeTransfer?
[ ] Do all transferFrom calls use msg.sender as the from address?

[ ] If not, is the function access restricted to prevent unauthorized
token spend?

[ ] Low-level calls (call, delegatecall, staticcall, transfer, send)

[ ] Is the returned success boolean checked to handle failed calls?
[ ] If using delegatecall, are there strict access controls on the
addresses that can be called? It shouldn't be possible to delegatecall
arbitrary addresses, so the list of possible targets should either be
immutable or tightly controlled by an admin.

[ ] Reentrancy

[ ] Are functions that make external calls or transfer ether marked as nonReentrant?

[ ] If not, is there documentation that explains why reentrancy is
not a concern or how it's mitigated?

[ ] Gas Optimizations

[ ] Is the logic as simple as possible?
[ ] Are the storage values that are used repeatedly cached in stack or
memory variables?
[ ] If loops are used, are there guards in place to avoid out-of-gas
issues?

[ ] Visibility

[ ] Are all payable functions restricted to avoid stuck ether?

[ ] Math

[ ] Is all of the arithmetic checked or guarded by if-statements that will
catch underflows?
[ ] If Safe functions are altered, are potential underflows and
overflows caught so that a failure flag can be thrown?
[ ] Are all of the rounding directions clearly documented?

[ ] Testing

[ ] Are there new or updated unit or integration tests?
[ ] Do the tests cover the happy paths?
[ ] Do the tests cover the unhappy paths?
[ ] Are there an adequate number of fuzz tests to ensure that we are
covering the full input space?

@jrhea
Solidity

[ ] Tokens

[ ] Do all approve calls use forceApprove?
[ ] Do all transfer calls use safeTransfer?
[ ] Do all transferFrom calls use msg.sender as the from address?

[ ] If not, is the function access restricted to prevent unauthorized
token spend?

[ ] Low-level calls (call, delegatecall, staticcall, transfer, send)

[ ] Is the returned success boolean checked to handle failed calls?
[ ] If using delegatecall, are there strict access controls on the
addresses that can be called? It shouldn't be possible to delegatecall
arbitrary addresses, so the list of possible targets should either be
immutable or tightly controlled by an admin.

[ ] Reentrancy

[ ] Are functions that make external calls or transfer ether marked as nonReentrant?

[ ] If not, is there documentation that explains why reentrancy is
not a concern or how it's mitigated?

[ ] Gas Optimizations

[ ] Is the logic as simple as possible?
[ ] Are the storage values that are used repeatedly cached in stack or
memory variables?
[ ] If loops are used, are there guards in place to avoid out-of-gas
issues?

[ ] Visibility

[ ] Are all payable functions restricted to avoid stuck ether?

[ ] Math

[ ] Is all of the arithmetic checked or guarded by if-statements that will
catch underflows?
[ ] If Safe functions are altered, are potential underflows and
overflows caught so that a failure flag can be thrown?
[ ] Are all of the rounding directions clearly documented?

[ ] Testing

[ ] Are there new or updated unit or integration tests?
[ ] Do the tests cover the happy paths?
[ ] Do the tests cover the unhappy paths?
[ ] Are there an adequate number of fuzz tests to ensure that we are
covering the full input space?

The recommendation suggests considering the removal of the _totalBase() function from HyperdriveBase which seems fine to me. The only place I see it being used is in the MockHyperdriveBase... any thoughts here?
Also, any idea why the _totalShares() function exists on HyperdriveBase? Couldn't you just use the totalShares storage variable? Having both seems confusing.

The recommendation suggests considering the removal of the _totalBase() function from HyperdriveBase which seems fine to me. The only place I see it being used is in the MockHyperdriveBase... any thoughts here?
Also, any idea why the _totalShares() function exists on HyperdriveBase? Couldn't you just use the totalShares storage variable? Having both seems confusing.

We use both _totalBase and _totalShares in the sweep function in HyperdriveAdmin. The totalShares storage variable is only used in MockHyperdrive and isn't available for any of the other instances found in contracts/src/instances. With that in mind, we can't assume that totalShares will be available, and the point of the _totalShares() function is to abstract this out.

The recommendation suggests considering the removal of the _totalBase() function from HyperdriveBase which seems fine to me. The only place I see it being used is in the MockHyperdriveBase... any thoughts here?
Also, any idea why the _totalShares() function exists on HyperdriveBase? Couldn't you just use the totalShares storage variable? Having both seems confusing.

We use both _totalBase and _totalShares in the sweep function in HyperdriveAdmin. The totalShares storage variable is only used in MockHyperdrive and isn't available for any of the other instances found in contracts/src/instances. With that in mind, we can't assume that totalShares will be available, and the point of the _totalShares() function is to abstract this out.

ahhh gotcha, not sure what I was thinking with totalShares being a storage variable on HyperdriveBase 🤦  thank you


[gazebo simulator] is it possible to increase the physics max step size?

I'm newby in Gazebo and running the first tests. I inserted "iCub with hands" in the simulator and run succesfully  the tutorial_arm.  Then I tried to increase  the physics max step size (in the gazebo gui) from 0.001 to 0.002, in order to have a faster (less accurate) simulation, obtaining sometimes a crash of gazebo, sometimes the icub crumpling on itself.
Is is this behaviour expected?  Is there some way to make the simulation run faster?

Senior developers helping on this might be @randaz81 @traversaro @Yeshasvitvs
@valeriosperati you should have received an invitation to join our GitHub community (this will allow you for example to assign labels). If you did not receive it, please check the email account associated to your GitHub profile.
Thanks @emilio-cartoni for contributing to keep Q&A neat 😉

Hi @valeriosperati ,
in general all the models provided in the icub-gazebo repository have been tested with the default Gazebo physics step size of 0.001 s. Notice that all the Gazebo physics engine at the moment have a fixed-step integrator, so this set the actual step of the integrator.
Unfortunately the differential equations describing the evolution of a complex multibody system such as the iCub (with small light links such as the fingers and large heavy links such as the chest) are known to be rather stiff, so it does not surprise me that you have numerical stability problems by doubling the integrator fixed step.
Going back to your (I guess) original goal, I think there are several options that you may want to investigate:

If you do not need the hands or the camera, you can try the iCub (without hands) model.
If you do not need the camera, you can try to disable the camera plugin or reduce the camera framerate.
If you are interested in simulating the iCub attached to a pole, you can use the iCub (fixed) model, and thus you avoid to simulate the interaction between the feet and the ground.
If you don't need to account for the model collision, you can try to comment out all the collision elements from the SDF, and that should reduce the time that the simulator spends in checking for collisions.
At the moment we do not provide a model without the legs, but that could also improve the performances. If you are confident with the SDF format, you can look into removing manually the links and the joints.

In general the right thing to do before taking any of these steps would be to profile Gazebo to understand which aspect of the simulation is your bottleneck, but unfortunately this require to compile Gazebo from source and some insight on the internals of Gazebo. If you are nevertheless interested in this, you can look into [1].
[1] http://answers.gazebosim.org/question/11998/why-does-performance-decrease-when-removing-inserting-models/?answer=12009#post-id-12009

It's possible, in principle. I do no know about gazebo crashes (sometimes they occur without a specific reason due to problems in ogre, the graphic library). Instead, if you see the icub crumpling on itself, it is most likely that current PID gains of icub are not stable anymore. Indeed, they are tuned to work at a specific simulation step and they should be reduced accordingly if your increase the step size (while decreasing it is always ok).
Unfortunately, it is difficult to suggest a solution for your problem.
Obviously my first suggestion is to use a more powerful machine. I am able to run complete icub simulation near realtime on my laptop, which is a good laptop, but not a huge machine. (*)
Alternatively you can remove from the SDF the collision tags, this will speed up the simulation a lot. You can also remove cameras if you are interested in body simulation only.
Finally, I noticed that on machines with old graphic cards, the physics computation is ok, but the display is running very slow. For these machines I suggest NOT to run fullscreen and to reduce the size of the gazebo window as much as possibile. This is enough to increase the FPS number displayed in bottom right corner.
(*) you can also run gzserver (the physics simulation) on a remote machine, and gzclient (the display) on you pc.

Thanks a lot for the explanations and the informations.  Right now I'm just exploring Gazebo, testing the icub tutorials both on Gazebo and in iCub_SIM.  For now, I will keep the  physics step size to its default value.
PS: please, it is the first time that I  interact with GitHub issues: can we consider this issue closed? Is it up to me to "close and comment"? Thanks, Valerio

Hi @valeriosperati
Here's some info on how to contribute.
Usually, the OP (you in this case) is responsible for judging whether the issue can be closed or not. Maintainers can also kick in and say a word in this respect.
I think we can close it.


how to work with  eTeak GUI

Hi, guys, I have downloaded the virtual machine, named as '' eTeak[1]'', and opened it using VMware.
I followed the instructions in this address, till the following commands:
stack setup --upgrade-cabal
stack build
stack exec -- eTeak --gui
But later, a blank GUI has been opened.

Now, I don't know how to continue my job toward executing the first example, e.g. the shifter, in this GUI?
Thanks for your time.
[1].https://github.com/balangs/eTeak

I'm not sure why this issue is titled how to install eTeak as GUI runs well apparently and it shows the framework is installed properly. Please make sure you choose the right title for the issue you're posting as it avoids confusion for later references.
By running stack exec -- eTeak --help you should see a list of flags for using the synthesiser. you could also see try --gui-usage to get more info about using the interface.
As an example you can simply generate asynchronous verilog netlists by running stack exec -- eTeak --gates <Balsa File>, this takes example.v library as default.
next, you can use the top level test suite in ./test_suit to test your design.

I have followed your instruction and hopefully it worked.
But when I tried to test it using test suit, I encountered the following error:
ubuntu@ubuntu-VirtualBox:~/eTeak/examples/test$ stack exec -- eTeak --gates test.balsa
ubuntu@ubuntu-VirtualBox:~/eTeak/examples/test$ cd ..
ubuntu@ubuntu-VirtualBox:~/eTeak/examples$ cd ..
ubuntu@ubuntu-VirtualBox:~/eTeak$ ./test_suit/run.sh
++ OPTS=-O
++ LATCHES='-L -l loop=1'
++ '[' yes = no ']'
++ '[' yes = no ']'
++ '[' yes = no ']'
++ '[' yes = yes ']'
++ cver +define+TKR_STR_COUNT=10000 monitors.v /home/mahdi/Desktop/teak/runtime/verilog/runtime.v
./test_suit/run.sh: line 74: cver: command not found
ubuntu@ubuntu-VirtualBox:~/eTeak$
Can you please elaborate on this issue?

nobody cant help me!!!!!!!!!!

GPL Cver is just a Verilog simulator, which is apparently not supported by the recent Ubuntu distributions. I'd suggest installing iverilog which is handy and easy to use. After installation you'd need to iverilog m_harness -o foo.out and then execute the output by ./foo.out

I have just learned balsa language, and I am not familiar with shell language. I have already installed the eteak virtual machine and cver simulator.
I want to know if I need any other file except balsa design files to test my design, and how to start the simulation in eteak. Should I write a sh file for my design, or run any existing sh file?
Thanks for your time.


fix(TDI-38947): Fix handling of dynamic schema for custom record types.

What is the current behavior? (You can also link to an open issue here)

Netsuite input component incorrectly handles dynamic schema for custom record types

What is the new behavior?

Netsuite input component correctly handles dynamic schema for custom record types

Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[X] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[X] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes

Does this PR introduce a breaking change?

[ ] Yes
[X] No

Other information:
JIRA issue: https://jira.talendforge.org/browse/TDI-38947

Pull request test result: success :star:


vulkan-helper: fix build on aarch64

vulkan-helper constructs a C string using an explicit *const i8 pointer, which isn't correct for all architectures. While this builds fine for x64, on aarch64 c_char is defined as u8, causing compilation errors.
This change introduces a patch for this issue I have submitted as an upstream PR ^1 until said PR is hopefully accepted.
Description of changes
Add a patch fixing the type of a C string field that breaks aarcht64 builds.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

3 packages built:

heroic
heroic-unwrapped
vulkan-helper

Result of nixpkgs-review pr 275907 run on aarch64-linux 1

3 packages built:

heroic
heroic-unwrapped
vulkan-helper

Result of nixpkgs-review pr 275907 run on aarch64-linux 1

3 packages built:

heroic
heroic-unwrapped
vulkan-helper

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

3 packages built:

heroic
heroic-unwrapped
vulkan-helper

Closing this in favor of https://github.com/NixOS/nixpkgs/pull/276262
The upstream PR was merged much faster than I expected :)
@mrcjkb @Scrumplex  Sorry for the trouble. Would you be willing to have another look at the new PR?


Fix #303 model not deselecting on logout

Coverage increased (+0.003%) to 90.581% when pulling a763900d1facf1a3764db44d557e5b44de453375 on khooroko:Deselect_person_on_logout into 9803354a3e93eacc854ca4a8da518f23a8d2b11f on CS2103AUG2017-T17-B1:master.


How to open a new view controller on menu item click?

Can you provide a code snippet for opening a new view controller on menu item click?

Hi @riyal123!
    @IBAction func menuButtonTapped(_ sender: UIButton) {
        let rootViewController = storyboard!.instantiateViewController(withIdentifier: "root")
        present(rootViewController, animated: true, completion: nil)
    }


Not able to destroy slick-slider and re render it

Guidlines for posting new issue

Please replicate your issue with this jsfiddle and provide a link to it along with the issue description

I am using slick slider for dynamic images where user can add or delete images at run time, but its not working well.

React Basics: simply supply a new key to the Slider... Does this solve your Problem?

React Basics: simply supply a new key to the Slider... Does this solve your Problem?

It's working! Thx @fdc-viktor-luft

React Basics: simply supply a new key to the Slider... Does this solve your Problem?

It's working! Thx @fdc-viktor-luft

Hi @brovkin Can you provide code for this solution?

i get same issue

@f3tusz Hi!
I don't remember exactly (it was 2 years ago), but I just added key="{unique value}" to the slider and it worked, try it like that:
<Slider key="slider_2">
    ...
</Slider>.

@brovkin thank you for your comment. I tried that solution yesterday, but it didn't work. Then now I changed to https://swiperjs.com/react


Add api test

Pull Request
Add unit tests for api
Description
This pull request introduces three new tests. These tests specifically validate the API route response:

they check for a status code of 200 and verify that the response contains the "power_kw" key with a length of 192 values.
handling wrong data types

Additionally, the hard-coding of the key "power_kw" and the fixed length of 192 is debatable.
GitHub Action might need adjustment
Fixes #152
How Has This Been Tested?
Pytest - Unit-test

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

Thanks for this @BraunRudolf , what do you mean by Additionally, the hard-coding of the key "power_kw" and the fixed length of 192 is debatable.?

Thanks for this @BraunRudolf , what do you mean by Additionally, the hard-coding of the key "power_kw" and the fixed length of 192 is debatable.?

Within the test function, if expected values change, manual adjustments would be required for each test. I wasn't sure what the best practice was.
I could add the variables expected_dic_key = "power_kw" and 'num_expected_values=192'

Thanks for this @BraunRudolf , what do you mean by Additionally, the hard-coding of the key "power_kw" and the fixed length of 192 is debatable.?

Within the test function, if expected values change, manual adjustments would be required for each test. I wasn't sure what the best practice was. I could add the variables expected_dic_key = "power_kw" and 'num_expected_values=192'

Yea might be good to set those as variables at the top, and then we can easily change them, if we change something in the main code.

Hi @BraunRudolf do you mind remove plots of 10 first predictions in this PR, or merge the latest main in.
Then we can keep this PR to be just tests for the api.
There were some recent upgrade to the API, that hoepfully dont break this work

Hi @peterdudfield, removed the last commit.
Now merged the latest change in and the tests fail. If you want, I can try to fix it tomorrow or close the PR and I or someone else does it from scratch.

Hi @peterdudfield, removed the last commit. Now merged the latest change in and the tests fail. If you want, I can try to fix it tomorrow or close the PR and I or someone else does it from scratch.

If you could fix it, that would be great

do we need to add httpx to the requirments?

It's a requirement for the TestClient, but not for FastAPI testing in general. https://fastapi.tiangolo.com/advanced/async-tests/#pytestmarkanyio
Never question it.
Could try to work around it.

HI @peterdudfield,
Unfortunately, I'm having trouble creating a test for the positive case of get_enphase_token_and_system_id (code 200). I can't figure out how to mock the return of get_enphase_access_token. I can try to figure it out, but I can't estimate how long that would take. You may want to merge this PR and adjust the issue, or create a new one.

HI @peterdudfield, Unfortunately, I'm having trouble creating a test for the positive case of get_enphase_token_and_system_id (code 200). I can't figure out how to mock the return of get_enphase_access_token. I can try to figure it out, but I can't estimate how long that would take. You may want to merge this PR and adjust the issue, or create a new one.

Yea, why dont we put that into a new issue, and then merge this first

Ill merge this to dev, just to make sure everything is working


Add LLVM_BUILD_MODE fix issue #2776

This change uses llvm-config to get the build-mode for llvm. This is then
used to set NDEBUG appropriately so the llvm headers included in
llvm_config_begin.h have NDEBUG defined if the build was Release or
RelWithDebInof and undefined if Debug.
See ponylang issue #2776 for more details.

@dipinhora, here is a PR that fixes #2776 by setting NDEBUG, it is my opinion this is the solution, would you please review?
The root cause of the bug is that when LLVM is compiled in "release" mode and ponyc is compiled in "debug" mode the code generated for Expected template is different and in compatible. Specifically, when NDEBUG is not defined the assertIsChecked method is not a NOP and the associated Checked variable are defined. Where as if NDEBUG is defined assertIsChecked is a NOP and Checked is not defined.
Thus when an Expected is returned by LLVM compiled in release mode and then deleted by code compiled in debug mode Checked can be false and assertIsChecked will abort. This change fixes this by appropriately setting NDEBUG when including the LLVM headers.

@winksaville I'm not an expert on this so maybe I'm misunderstanding things.
It seems like this would override the NDEBUG value depending on how LLVM was compiled regardless of whether config=debug or config=release was used when ponyc is being compiled. My concern would be that if I wanted a debug compile of ponyc via make config=debug but NDEBUG gets enabled via the changes in this PR due to LLVM being compiled with NDEBUG. This would be very confusing to me. I'm even more confused because it seems the code restores the value of NDEBUG in LLVM_CONFIG_END_H (I think) and I'm not even sure I understand the full implications of that. Does it mean that some code is potentially compiled with NDEBUG enabled and other with NDEBUG disabled depending on whether it was included before/after or in the middle of the LLVM_CONFIG_BEGIN_H/LLVM_CONFIG_END_H headers? What about other code that doesn't rely on the LLVM_CONDIG_*_H headers at all?
Maybe a better option would be to throw an error if the NDEBUG value doesn't match what was requested by the ponyc make command and what LLVM would require?

I'm not an expert on this so maybe I'm misunderstanding things.

I'm not an expert either, but I'm learning and doing my best :)

It seems like this would override the NDEBUG value depending on how LLVM was compiled regardless of whether config=debug or config=release was used when ponyc is being compiled. My concern would be that if I wanted a debug compile of ponyc via make config=debug but NDEBUG gets enabled via the changes in this PR due to LLVM being compiled with NDEBUG. This would be very confusing to me. I'm even more confused because it seems the code restores the value of NDEBUG in LLVM_CONFIG_END_H (I think) and I'm not even sure I understand the full implications of that. Does it mean that some code is potentially compiled with NDEBUG enabled and other with NDEBUG disabled depending on whether it was included before/after or in the middle of the LLVM_CONFIG_BEGIN_H/LLVM_CONFIG_END_H headers?

It is true the code changes the value of NDEBUG, but it only effects the llvm headers, no other code. At line 41 of llvm_config_begin.h the current value is pushed using pragma push_macro("NDEBUG"). And then at line 8 of llvm_config_end.h that value is restored using pragma pop_macro("NDEBUG"). So there will be no effect on any ponyc code.

What about other code that doesn't rely on the LLVM_CONDIG_*_H headers at all?

There will be no difference, NDEBUG is not altered for any ponyc code.

Maybe a better option would be to throw an error if the NDEBUG value doesn't match what was requested by the ponyc make command and what LLVM would require?

Throwing an error is not possible because the user and our test code can and do compile ponyc code with config=debug or config=release, where as system provided LLVM can be compiled in any build-mode. Throwing the error would mean we couldn't properly test.
Please note that the conditional changing of NDEBUG only occurs for LLVM < 400 so when we move away from LLVM 3.9.1 to newer versions of LLVM this code is ignored. This is accomplished in the newer LLVM versions by using a different defined symbol, LLVM_ENABLE_ABI_BREAKING_CHECKS, which is only used on assertion builds.

@winksaville Thanks for the thorough response.
It seems the following code would impacted by the NDEBUG setting being changed because the pony headers are included prior to llvm_config_end.h:
https://github.com/ponylang/ponyc/blob/6227226303e57fa7aa4c561c08e3831eed20fe80/src/libponyc/codegen/genopt.cc#L27-L31
Assuming that is resolved, I think everything looks good from my understanding of things.

Great catch, it was a copy paste error on my part, making it:
#include "llvm_config_end.h"
#include "../../libponyrt/mem/heap.h"
#include "ponyassert.h"
I'm so glad for reviews!!!
On Fri, Jun 29, 2018 at 2:21 PM Dipin Hora<EMAIL_ADDRESS>wrote:

@winksaville https://github.com/winksaville Thanks for the thorough
response.
It seems the following code would impacted by the NDEBUG setting being
changed because the pony headers are included prior to llvm_config_end.h:
https://github.com/ponylang/ponyc/blob/6227226303e57fa7aa4c561c08e3831eed20fe80/src/libponyc/codegen/genopt.cc#L27-L31
Assuming that is resolved, I think everything looks good from my
understanding of things.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ponylang/ponyc/pull/2812#issuecomment-401477141, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-hHBdC-nNxgFb9pBT7h7ewhUaz1dOHks5uBpo9gaJpZM4U6KPg
.

@dipinhora I've fixed the bug you found, do you think this is ready to merge?

@winksaville it looks good from my understanding of things however i'm not sure my understanding of everything is good enough to decide whether this is acceptable to merge or not. It would be ideal if someone more familiar with C/C++/LLVM could take a quick look also.


Landing page text is cut off on mobile

Originally created by Tom Smyth at 2017-12-17 15:56, migrated from redmine-#7677

Version set to 5.0

By Tom Smyth at 2017-12-17 21:45

Status changed from New to Closed

By Tom Smyth at 2017-12-23 12:45


Update dependency org.apache.cxf:cxf-rt-ws-security to v3.3.2

This PR contains the following updates:

Package
Update
Change
References

org.apache.cxf:cxf-rt-ws-security
patch
3.3.1 -> 3.3.2
homepage

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

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


Restart button

It would be nice to have a restart button (which is probably also offered when the editor crash (not-repsonding) message appears), which allows to restart Atom and restores the previous state immediately.
Especially when you work with multiple windows and one crashes this is useful as you can be sure the correct session is restored.

There actually already is a restart command – Window: Reload in the command palette or View > Developer > Reload Window in the app menu.
Hope that helps!

Yes thanks. Although you cannot use this if Atom freezes and you are only able to use the menu.


Permission Denied

Hey mate when I'm trying to use this action I get:
line 1: /runner/_work/_actions/kamilchodola/wait-for-workflow-action/1.1.0/scripts/wait-for-workflow.sh: Permission denied

Hi!
I think I made small issue in there in regards of script permission - let me take a look

I am getting the same problem

Same


enhance contact process

Enhancement: Don't send email or submit form until reCaptcha response = success.
reinstated application/views/contact/view.php

This new change ensures email is not sent, OR contact form data written to the DB, when recatcha validation fails.

Also, i reinstated the view view, just in case it's used somewhere else, but it's definitely not called from the Contact controller

I'll test of maybe it's called from the admin side. If not I'll remove it on my next commit

Nice dude! Makes total sense.


[Bug]: False positive with arRPC

What happened?
The plugin thinks I'm in a browser and that Rich Presence doesn't work, but it does.

IDE version
RustRover 2023.3 EAP
Plugin version
2.0.8
What operating system are you using?
Windows
Relevant log output
No response

Thanks for the report I'll have a look at that!

Fixed with https://github.com/Azn9/JetBrains-Discord-Integration/commit/fa8017d65de02c249a8c457677fd4a2a19f38761. I will close this issue once it has been merged and released.


Frequently getting Request textDocument/documentSymbol failed

I'm frequently getting this error while typing. Not really sure why.
[Error - 8:49:21 PM] Request textDocument/documentSymbol failed.
Error: name must not be falsy
	at d.validate (/nix/store/s7accqb6id4p71p56mr5hb4vay0cnj1c-vscode-1.88.1/lib/vscode/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:150:17471)
	at new d (/nix/store/s7accqb6id4p71p56mr5hb4vay0cnj1c-vscode-1.88.1/lib/vscode/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:150:17766)
	at Qt (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:35:72226)
	at Qt (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:35:72403)
	at Qt (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:35:72403)
	at Qt (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:35:72403)
	at Qt (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:35:72403)
	at o (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:35:46654)
	at Object.GC [as map] (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:35:46740)
	at Object.Bn [as asDocumentSymbols] (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:35:72193)
	at u (/nix/store/qqgl0yg1c5j0dawqm3xzzvjg72bl5c6q-vscode-extension-jnoortheen-nix-ide-0.3.1/share/vscode/extensions/jnoortheen.nix-ide/dist/extension.js:39:56050)
	at async d.provideDocumentSymbols (/nix/store/s7accqb6id4p71p56mr5hb4vay0cnj1c-vscode-1.88.1/lib/vscode/resources/app/out/vs/workbench/api/node/extensionHostProcess.js:152:90504)

Same

I also get these regularly using the nil language server.

Same here, using this with nil and getting that error a lot.
Formatting straight up never works for me, but the error popup is not always there.

It's especially frustrating since somehow or other the extension is bypassing VS Code's do not disturb mechanisms to suppress notifications. I had to uninstall because I can't stand a wrong and pointless error message popping up every five minutes.

It's especially frustrating since somehow or other the extension is bypassing VS Code's do not disturb mechanism

According to this this is not special to this extension but all error notifications in VSCode by-pass do-not-disturb.

I had to uninstall because I can't stand a wrong and pointless error message popping up every five minutes.

I happen to have it now on every tenth keystroke :grimacing:

I've also seen this a lot using nixd:
2024-09-07 21:32:41.015 [error] [jnoortheen.nix-ide] provider FAILED
2024-09-07 21:32:41.015 [error] Error: unknown node type for definition
	at rn (/Users/ianchamberlain/.vscode/extensions/jnoortheen.nix-ide-0.3.3/dist/extension.js:6:228)
	at yn (/Users/ianchamberlain/.vscode/extensions/jnoortheen.nix-ide-0.3.3/dist/extension.js:5:6240)
	at kn (/Users/ianchamberlain/.vscode/extensions/jnoortheen.nix-ide-0.3.3/dist/extension.js:5:6363)
	at Immediate.<anonymous> (/Users/ianchamberlain/.vscode/extensions/jnoortheen.nix-ide-0.3.3/dist/extension.js:5:6130)
	at process.processImmediate (node:internal/timers:478:21)

It seems like given that both nixd and nil seem prone to failed requests, maybe it would make sense to install a middleware that catches errors (or a certain subset of errors) and swallows them (or make it configurable which types of errors can be swallowed, or something, idk)?
See also https://github.com/microsoft/vscode-languageserver-node/issues/517 which is a really old issue about how this was handled in the client.
I guess longer-term it would be better to fix the language server implementations, but since this is a common problem for multiple implementations a short-term workaround on the client might make sense...

Also getting this very often with nixd:
  ERR unknown node type for definition: Error: unknown node type for definition
    at handleResponse (/Users/abc/.vscode/extensions/jnoortheen.nix-ide-0.3.3/node_modules/vscode-jsonrpc/lib/common/connection.js:565:48)
    at handleMessage (/Users/abc/.vscode/extensions/jnoortheen.nix-ide-0.3.3/node_modules/vscode-jsonrpc/lib/common/connection.js:345:13)
    at processMessageQueue (/Users/abc/.vscode/extensions/jnoortheen.nix-ide-0.3.3/node_modules/vscode-jsonrpc/lib/common/connection.js:362:17)
    at Immediate.<anonymous> (/Users/abc/.vscode/extensions/jnoortheen.nix-ide-0.3.3/node_modules/vscode-jsonrpc/lib/common/connection.js:334:13)
    at processImmediate (node:internal/timers:478:21)
    at process.topLevelDomainCallback (node:domain:160:15)
    at process.callbackTrampoline (node:internal/async_hooks:128:24)

Closing this with reference to #417

Is it really a fix? Why is the error happening so frequently in the first place?

It is a workaround. The real fix has to come from the language servers.


Disable a ticket (Special Requirement)

Neither the user nor the admin can delete a ticket. The ticket must change its status to
Cancelled.

Since tickets shouldn't be deleted, we can disable a ticket.
Requirements:

[ ] Should only be done by admins
[x] Will create an iteration changing the status to Cancelled

[ ] Will require a comment as it would create an iteration

Requires #6 and #7 to be complete before this can be finished

Since Authentication isn't required this round, this part is completed


Automated cherry pick of #95: Convert setters from v1 to v2 Cherry pick of #95 on v1.1-branch. #95: Convert setters from v1 to v2

/assign @Bobgy
/assign @IronPan

/lgtm
/approve


How to acquire predictor weights

Hi,
I'm a beginner in this space. I'm interested in converting my custom model to the PowerInfer GGUF format, and I saw I need predictor weights to convert it. Do you know how I can acquire these for a custom model?
Thanks!

Welcome, @fakerybakery ! We are currently limited to a short list of models, as stated in our FAQs, but we are progressing in #93 to support more model architectures.
If you are interested in how to train the predictor weights for a model with supported architecture, please kindly refer to #116, #84, #83.

Thank you so much! Sorry about the duplicate!


Fix #36

Closes #36

@RianFuro LGTM. Could you quickly fix the conflicts?

Conflicts of my own making! Done :)


Port fix (server): Better error handling in broadcast signal in tinylicious (#17542) to t9 release branch

Release Tinylicious analog for https://github.com/microsoft/FluidFramework/pull/17540.

Version bump looks good. I'll let someone else review the code changes.


[cache] Add completion callback to updateHeaders

Commit Message: Add completion callback to updateHeaders in HttpCache
Additional Description: This is necessary to facilitate testing of asynchronous cache implementations.
Risk Level: Low - cache filter is still tagged WIP.
Testing: Covered by existing tests (and future tests that rely on this change.)
Docs Changes: n/a
Release Notes: May require any existing HttpCache implementations to update to match the new interface.
Platform Specific Features: n/a

/assign @jmarantz


Implement Error Handling for Todo Operations

Describe the bug
The application currently lacks proper error handling for various todo operations, such as adding, deleting, or marking todos as complete. This can result in unhandled exceptions and poor user experience.
To Reproduce

Attempt to add a todo with invalid input.
Try to delete a todo that does not exist.
Mark a todo complete when the todos list is empty.

Expected behavior
The application should gracefully handle errors by displaying appropriate messages to the user, without crashing or showing stack traces.
Screenshots
N/A
Additional context
Implementing error handling will improve the robustness of the application and enhance the user experience.
Related Issues

Issue #1
Issue #2

Implementing robust error handling for todo operations is crucial for maintaining a smooth user experience. I recommend using try-catch blocks to gracefully manage exceptions and provide user-friendly feedback for invalid actions. This will not only prevent application crashes but also enhance the overall reliability of our system. Let's ensure we cover all edge cases to improve the robustness of our application.


Variables in Axiom queries do not respect panel repeat option

Hi
I have created a dashboard that pulls a list of environments from Axiom as a variable.
Then, for each environment it creates a panel that shows something for that environment.
My variable content looks like this:

My dashboard panel uses the repeat panel option over the env variable:

When using the ${env} variable in a repeated panel, the actual value of the variable becomes the panel specific value instead of the list of all environments. This is a default behaviour in most parts of Grafana.

Here is the Axiom query I use in the repeated panel:
['global']
| where ['resource.env'] == "${env}"
| where severity_number >= 17 // maps to error and fatal
| summarize count() by ['severity']

When I inspect the value of a repeat panel performing an Axiom query, it seems the entire list of environments is put into the query. I expected it to only insert the specific value of that panel:

I compared this to the behaviour of another datasource - in this case Prometheus. I created a repeated panel and created a query using the same variable:
sum(kafka_topic_partitions{environment=~"${env}"})

I then checked the actual query send to the backend, and it only contains the specific panel value:

Here is the dashboard I used to produce the error. It cannot completely reproduce the issue as it depends on both an Axiom and Prometheus datasource, but it will show you how my panels were configured.
repeat-error-dashboard.json
Additional info:
The issue also happens when using the "table" panel type.
Versions used:
Grafana v11.1.4 (13173c9874)
Axiom plugin version: 0.4.0

hi @van-vliet, thanks for reporting this. I am looking into it at the moment, will get back to you.

@van-vliet I have released a new version (v0.5.1) with a fix for the mentioned issue, let me know if something is not right.

Hi @SollyzDev
Thanks for the quick fix. Repeat panels now sent the correct query.


Intellisense not working at all

I have fsi/fsc in my system path.
Other features like alt-enter to run in FSI is working
I don't get intellisense on any F# component

Try installing MSBuild 2013.

That worked thank you!
I didn't see this in the documentation or the website. I think this needs to be added.


added initial .travis.yml

What's the reason of this PR?

It allows the project to run on the travis CI service which checks all PRs and commits on a configurable set of branches. It's for free, helps to spot errors and doesn't do any harm.

Ok for me, but please send it to "develop" branch, not "master" that we use for release only. Thanks in advance.

see #4746


Display deployed ZK tokens with balances

Background
This task is part of a larger project to create an interactive web demo / playground for users to learn about zero-knowledge (ZK) assets and gain intuition about them by directly transacting with the contracts on-chain.
Your task is to implement the part of the static site that is currently hard-coded to display fake balances of ZK tokens.

Task Description and Examples
A static demo, created with react-scripts, not connected to the blockchain, is currently running on:
https://zk-transfer.netlify.com
After this task, you will connect this demo to read from the Rinkeby testnet.
with source code in this directory:
https://github.com/invisible-college/democracy/tree/master/packages/zk-transfer-web
Your task has two parts:

mint new ZK notes for one of three private tokens.
retrieve a list of deployed private tokens and display one row per token, with balance.

The page only has to display the correct balances when it is refreshed. In a later task, we'll add the ability to listen to changes from the blockchain and auto-update.
We also use Immutable.js everywhere instead of regular JSON maps and lists because of the benefits of immutability (fewer programming errors / unintended side effects, as well as the ability to see historical snapshots of data).
immutable.List and immutable.Map are available in the webpage, and on the web console, using demo.immutable.List and demo.immutable.Map.
You can read more at the official Immutable.js documentation.
How to Work on This Task
Working on this task requires the following steps

Communicate about this task in this thread, and give an update about once every two days letting me know you're still interested in working on this task, and asking questions if you're stuck.
Fork this repo, and set up your local environment with your fork using these instructions https://github.com/invisible-college/democracy#get-the-source-code-to-experiment-and-run-tests
Test that you're able to access the Rinkeby account we're providing.
Mint new ZK notes, either from command-line or web.
Retrieve a list of deployed private tokens, with symbols AAA, BBB, and ABC.
Display them in React, one private token per row, with symbol and balance.
Mint more tokens.
See the balances updates in React.
Create a pull-request for this issue, and submit the URL in Gitcoin.
I'll create a code review in your PR thread to give feedback and collaborate. When we're both happy with the result, I'll merge / close it, and pay out the bounty.

These steps are described in more detail below:
Get Balance From a Rinkeby Account
To mint new ZK notes, we've created an Ethereum account on the Rinkeby testnet for you to use, with these credentials.
address: 0x6f38461e067426e5858aBD2610C22bCb35128Bf5
password: 652c22d2630960a3825d9bc92354c82ea76f895d62f2ca160223db48c5e69f26

You'll use these below.
Minting Through a Website
Navigate to this page to view your balance and perform minting through a web interface: https://aztec-web.netlify.com
The source code for this page, which you may find helpful later because it shows how to work with AZTEC notes using Democracy.js, is located at https://github.com/invisible-college/democracy/tree/master/packages/aztec-web
It will take a few seconds, as Democracy.js wallet loading is currently slow. You can open the web inspector to see its progress.
In the Login Book, enter the credentials above and click Retrieve.
After a second or two, you should see this new address added, with a balance of about 2.8 ETH (rinkeby ether). If this goes to zero, you can refill it by following the instructions at the Rinkeby faucet
In the Address Book, click the address / public key pair there, which is the auto-created Democracy.js address for you in localStorage.
In the bottom part, you'll find three tokens, with symbols AAA, BBB, and ABC.
In each one, you can type a number into the blank, hit Mint, and watch the web inspector to see it get mined.
After waiting for about 15 seconds without seeing an error, you'll know the note was created. (I know, we need better user feedback).
You can refresh the page now, and see the new notes at the bottom, under the appropriate token, drawn as colored rectangles.
Minting Through Command-Line
To mint through the command-line, and even create new tokens, clone the source code and set up your development environment using these instructions:
https://github.com/invisible-college/democracy#get-the-source-code-to-experiment-and-run-tests
Then run the following commands
cd packages/aztec-cli
NODE_ENV=RINKEBY ../../node_modules/demo-aztec-cli/scripts/mint.js ABC 10 <etheremAddress beginning with 0x> <aztecPublicKey beginning with 0x>

You can copy the ethereum address and AZTEC public key for your Democracy.js account from the web UI above, in the Address Book section.
Retrieve a List of Tokens in React
You'll perform this task in thepackages/zk-transfer-web directory, but using
these line in aztec-web, which you can interact with using the web console at https://aztec-web.netlify.com
After Democracy.js has initialized in zk-transfer-web, you can explore the demo and api global symbols.
To verify that you are using Rinkeby, you can type
> demo.config

To retrieve the list of ZK tokens, access
> api.zkTokens

The result is an Immutable.js Map, which you can iterate using
api.zkTokens.forEach((v,k) => console.log(`key ${key} value ${value}`))

or convert to a JSON object, for easy printing on the web console
api.zkTokens.toJS()

You usually do not have to do the .toJS part while you are programming in React.
Its keys are deploy names, which all begin with ZkAssetTradeable (the Ethereum contract name) and end in deployXYZ where XYZ is the trading symbol of the token. To access the token information for AAA, you evalute
api.zkTokens.get('ZkAssetTradeable-deployAAA')

The values are also an Immutable Map, the most important one is called deployAddress which tells you the Ethereum address where that token is deployed. To access the deployAddress of token AAA, you evaluate:
> api.zkTokens.get('ZkAssetTradeable-deployAAA').get('deployAddress')
"0xFF73cDb42B2E1B53e9186F786C49B0C2a76B27a3"

Retrieving ZK Notes with Democracy
The last piece of information you'll need to print out the balances of each ZK token is the list of notes.
You can use these lines from aztec-web, which produce an Immutable Map at https://aztec-web.netlify.com which you can interact with using the web console.
api.thisAddressNotes

with keys being the token address, and the values being an Immutable List of notes.
For example, once you've found all the token addresses above, you can retrieve the list of notes for that token as
api.thisAddressNotes.get("0x28649134a58C3e18932612f0E380E9E23176392b")

To iterate over all the notes
api.thisAddressNotes.get("0x28649134a58C3e18932612f0E380E9E23176392b").map((v) => v.get('zkNoteHash')).toJS()

Deploying to Netlify
When you create your pull request of your fork back to the base repo, Netlify will include a preview deploy.
Looking forward to working with you on this project.

@gitcoinbot Yes, been looking at it. Will put more focus on it this weekend.

@cryptogoth Should i add https://immerjs.github.io/immer/ to the project. I need to update state immutably.

@cryptogoth I followed the instructions as. Every ZK Token address returns zero notes. What do i do about it?

Now am getting this error.

@cryptogoth Should i add https://immerjs.github.io/immer/ to the project. I need to update state immutably.

Good point. You'll need to add the module immutable, which has source code here:
https://github.com/immutable-js/immutable-js

Since we use yarn in our project, you would run
yarn add immutable --save

Merged as #49


Blank screen after adding geocoder to deck.gl

Hey I was hoping I could get some help - I'm a new dev working on a mapping app. I tried to add the geocoder to a map I've been working on, and eventually figured out that it could work with static maps. But I have spent days and I still don't understand what I have done wrong with the mapref. Upon loading the page, my map will flicker with the datapoints on it and then the entire page goes white.
I was hoping to add a geocoder to the map so that I could eventually pass requests to my geoserver and only get points nearby, but I'm at a standstill. Below is my code, and I'd appreciate anything to help me understand better.
import React, { Component, useRef } from 'react';
import { StaticMap } from 'react-map-gl';
import DeckGL, { MapView } from 'deck.gl';
import '../map/mapbox-gl.css';
import { renderLayers } from './map_components/deckgl-layers';
import {
  LayerControls,
  MapStylePicker,
  HEXAGON_CONTROLS,
  //  SCATTERPLOT_CONTROLS
} from './map_components/controls';

import dynamic from 'next/dynamic';

const Geocoder = dynamic(() => import('react-map-gl-geocoder'), {
  loading: () =>
Loading...
,
  ssr: false,
});

const MAPBOX_TOKEN =
  'proces.env.MAPBOX_TOKEN';
const LAT = 28.5383;
const LON = -81.3792;
const INITIAL_VIEW_STATE = {
  latitude: LAT,
  longitude: LON,
  zoom: 9,
  maxZoom: 16,
  pitch: 50,
  bearing: 0,
};
}
class TestMapTwo extends Component {
  constructor(props) {
    super(props);
    this.state = {
      viewState: INITIAL_VIEW_STATE,
      GeojsonPoint: [],
      hover: {
        x: 0,
        y: 0,
        hoveredObject: null,
      },
      settings: Object.keys(HEXAGON_CONTROLS).reduce(
        (accu, key) => ({
          ...accu,
          [key]: HEXAGON_CONTROLS[key].value,
        }),
        {},
      ),
      style: 'https://api.maptiler.com/maps/bright/style.json?key=KO5UrPKClQethpf7vtZh',
      completed_fetch: false,
      explore_data: new Array(19).fill(0).reduce(
        (prev, curr) => [
          ...prev,
          {
            x: prev.slice(-1)[0].x + 1,
            y: prev.slice(-1)[0].y * (0.9 + Math.random() * 0.2),
          },
        ],
        [{ x: 0, y: 10 }],
      ),
      selectedTime: null,
    };
  }

  componentDidMount() {
    this._fetchData();
  }

  async _fetchData() {
    this.setState({ ...this.state });
    await fetch(
      //process.env.TEMP_DATA_FETCH
      `process.env.GEOSERVER_URL`,
    )
      .then((res) => res.json())
      .then(
        (result) => {
          this.setState({
            GeojsonPoint: result.features,
            chart_data: result.features.map(function(elem) {
              return {
                x: elem.properties.date_scraped,
                y: elem.properties.price,
              };
            }),
            completed_fetch: true,
          });
          console.log('result', result);
        },

        (error) => {
          console.log(error);
          this.setState({ ...this.state });
        },
      );
  }

  onStyleChange = (style) => {
    this.setState({ style });
  };
  _updateLayerSettings(settings) {
    this.setState({ settings });
  }

  //chart functionality
  _onHighlight(highlightedTime) {
    this.setState({ highlightedTime });
  }

  _onSelect(selectedTime) {
    this.setState({
      selectedTime: selectedTime === this.state.selectedTime ? null : selectedTime,
    });
  }

  render() {
    const data = this.state.GeojsonPoint;
    if (!data.length) {
      return null;
    }
    const { hover, settings } = this.state;

    const {
      onViewStateChange = ({ viewState }) => this.setState({ viewState }),
      viewState = this.state.viewState,
    } = this.props;

    const mapRef = React.createRef();

    return (

        <DeckGL
          layers={renderLayers({
            data: this.state.GeojsonPoint,
            onHover: (hover) => this._onHover(hover),
            settings: this.state.settings,
          })}
          views={new MapView({ id: 'map' })}
          viewState={viewState}
          onViewStateChange={onViewStateChange}
          controller={true}
          getTooltip={({ object }) =>
            object &&
            `Rent: ${object.properties.price}\nBeds: ${object.properties.beds}\nBaths: ${object.properties.baths}\nSq Ft: ${object.properties.size}`
          }
        >
          <StaticMap
            ref={mapRef}
            viewId="map"
            {...viewState}
            reuseMaps
            mapStyle={this.state.style}
            preventStyleDiffing={true}
            mapboxApiAccessToken={MAPBOX_TOKEN}
          ></StaticMap>
          <Geocoder
            mapRef={mapRef}
            onViewportChange={onViewStateChange}
            mapboxApiAccessToken={MAPBOX_TOKEN}
            position="top-left"
            countries="us"
          />
        </DeckGL>

    );
  }
}

The map and page render fine without the geocoder component, but when it loads my console lights up with a lot of information.
Uncaught Error: WebGL context: Could not create a WebGL context, VENDOR = 0x15ad, context.js:29
DEVICE = 0x0405, GL_VENDOR = Google Inc., GL_RENDERER = ANGLE (VMware, Inc., llvmpipe (LLVM 10.0.0, 128 bits), OpenGL 3.3 Core), GL_VERSION = 2.10,ea8043b73f93, Sandboxed = no, Optimus = no, AMD switchable = no, Reset notification strategy = 0x8261, ErrorMessage = bindToCurrentThread failed: .

Uncaught Error: WebGL context: Failed to create a WebGL2 context.

react_devtools_backend.js:2450 This page appears to be missing CSS declarations for Mapbox GL JS, which may cause the map to display incorrectly. Please ensure your page includes mapbox-gl.css, as described in https://www.mapbox.com/mapbox-gl-js/api/.

Uncaught TypeError: Cannot read property 'addControl' of null

Uncaught Type Error: Cannot read property 'remove' of undefined

The above errror occured in the  component:

Uncaught TypeError: Cannot read property 'addControl' of null

The above error occured in the <MyApp> component:
React will try to recreate this component tree from scratch using the error boundary you provided, Container.

HI @andrewshrout, have you tried putting the Geocoder inside the StaticMap component as a child? If that doesn't work, it would help if you could provide the simplest code possible that demonstrates your issue on Code Sandbox.

https://codesandbox.io/s/testmap-v21w4?file=/src/App.js
Geocoder inside the staticmap as a child didn't work in my nextjs project. I've done my best in the link above to create a super simplified version of what I'm trying to do, but I can't recreate the issues of my project. I do know that adding the geocoder in either of them breaks.
I've been reading over every single issue on this repository with a fine toothed comb, with #44 and #56 being the most useful to me. I'm a new dev in general but I guess I'm just incredibly confused and frustrated about adding this geocoder to a class component using Deck.gl.

https://codesandbox.io/s/react-map-gl-geocoder-v2-example-forked-2zbz0?file=/src/index.js
Got a somewhat working version by forking your functional component example, passing deck.gl as a child. Now there's working deck.gl, and geocoding. However I feel like I've just sidestepped my lack of understanding for why it won't work with a static map or in a class component.
And now I have to read up on how to fetch data in a functional component because I can't get my geoservers data onto the map.

Oh I see what the problem is with your class component. It's creating a new ref on every render. The ref should only be created once. Here's the old class component example that used to be in the README.
import 'mapbox-gl/dist/mapbox-gl.css'
import 'react-map-gl-geocoder/dist/mapbox-gl-geocoder.css'
import React, { Component } from 'react'
import MapGL from 'react-map-gl'
import Geocoder from 'react-map-gl-geocoder'

function getAccessToken() {
  var accessToken = null;

  if (typeof window !== 'undefined' && window.location) {
    var match = window.location.search.match(/access_token=([^&\/]*)/);
    accessToken = match && match[1];
  }

  if (!accessToken && typeof process !== 'undefined') {
    // Note: This depends on bundler plugins (e.g. webpack) inmporting environment correctly
    accessToken = accessToken || process.env.MapboxAccessToken; // eslint-disable-line
  }

  return accessToken || null;
}

// Ways to set Mapbox token: https://uber.github.io/react-map-gl/#/Documentation/getting-started/about-mapbox-tokens
const MAPBOX_TOKEN = getAccessToken()

class Example extends Component {
  state = {
    viewport: {
      width: 400,
      height: 400,
      latitude: 37.7577,
      longitude: -122.4376,
      zoom: 8
    }
  }

  mapRef = React.createRef()

  componentDidMount() {
    window.addEventListener('resize', this.resize)
    this.resize()
  }

  componentWillUnmount() {
    window.removeEventListener('resize', this.resize)
  }

  resize = () => {
    this.handleViewportChange({
      width: window.innerWidth,
      height: window.innerHeight
    })
  }

  handleViewportChange = (viewport) => {
    this.setState({
      viewport: { ...this.state.viewport, ...viewport }
    })
  }

  // if you are happy with Geocoder default settings, you can just use handleViewportChange directly
  handleGeocoderViewportChange = (viewport) => {
    const geocoderDefaultOverrides = { transitionDuration: 1000 }

    return this.handleViewportChange({
      ...viewport,
      ...geocoderDefaultOverrides
    })
  }

  render() {
    return (
      <MapGL
        ref={this.mapRef}
        {...this.state.viewport}
        onViewportChange={this.handleViewportChange}
        mapboxApiAccessToken={MAPBOX_TOKEN}>
        <Geocoder
          mapRef={this.mapRef}
          onViewportChange={this.handleGeocoderViewportChange}
          mapboxApiAccessToken={MAPBOX_TOKEN}
        />
      </MapGL>
    )
  }
}

export default Example


How to add extra field ?

Question in title

I am going to assume you have already added a field on the Prettify Card Type.
You have to edit the card template to add an extra line.

Here's how you do that
Go to Manage Note Types > Select your card type > Cards > Front (or Back) Template
Check for this code, or something similar.
-->

{{Deck}}

{{edit:Front}}

  {{#Tags}}

{{clickable:Tags}}

  {{/Tags}}

Before the last
, you have the fields that you need. For example, in the above code, you have the Front field and the Tags field.
How to actually add a field.
This is a general approach:
...
{{#<field>}}      # this means "if there is something in the field, then output the following"
	<div class="prettify-field prettify-field--<field>">{{edit:<field>}}

{{/<field>}}
...

Replace all the <field> with your Field name

You can preview the card in the preview panel. Good luck!


purpose of relation.incorporate in sqlserver__create_table_as

@mikaelene can you help me understand what this does? Is the idea that all temp relations are made as views instead when creating tables, by removing the # from the temp relation identifier?
https://github.com/dbt-msft/dbt-sqlserver/blob/7e74e971762fdbd652f2e75973a012190147120e/dbt/include/sqlserver/macros/adapters.sql#L143-L145

It looks like that :-). I think there was a problem using temptable in this materialisation. I didn't want to remove the possibility of using temptable in the adapter but could not use it here.
But I am not 100% sure why

It looks like that :-). I think there was a problem using temptable in this materialisation. I didn't want to remove the possibility of using temptable in the adapter but could not use it here.
But I am not 100% sure why


OAuth2

Super fun project!
Any plans to support OAuth2?

Thanks!
It would be great to include that but I don't have any immediate plans to work on it myself. I would certainly be open to someone contributing that. Do you have any recommended libraries, etc?


ARROW-2305: [Python] Bump Cython requirement to 0.27+

Starting with Cython 0.27, it's possible to implement comparisons using __eq__ and friends instead of __richcmp__: http://docs.cython.org/en/latest/src/changes.html#id16

AppVeyor build at https://ci.appveyor.com/project/pitrou/arrow/build/1.0.270
The Travis-CI failure is unrelated.


Fwdport forward/rel/6.4 to master

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "forward/rel/6.4"}
Please follow http://ci.ironmann.io/gh/scality/Integration/9927 for CI status.

@ironman-machine
SCALITY_UTAPI_BRANCH=fwdport/6.4-master
try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_UTAPI_BRANCH": "fwdport/6.4-master", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "forward/rel/6.4"}
Please follow http://ci.ironmann.io/gh/scality/Integration/9928 for CI status.

@ironman-machine try

Hello @bennettbuchanan
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "master", "SCALITY_S3_BRANCH": "forward/rel/6.4"}
Please follow http://ci.ironmann.io/gh/scality/Integration/9932 for CI status.

e2e passed


Version number & changelog on web page is radically out of date

on https://jwt.github.io/ruby-jwt/ it currently says

Current version: 1.5.6

and the last changelog update was 2016-09-19

Yes. This page could use some love. It got left behind the code issues. There is already something brewing in the background but I think this will take until June.

#629 changed the github pages to get updated with the yard docs from the main branch.


Array split does not result in equal division

Splitting the arrays for attributes self.split_index and self.htc in solvers.py throws an error if the number of cells is not evenly divisible by the number of processes nproc. For example, using Np = 32, Ns = 20 with nproc = 6 does not work for the Dask and Ray solvers. This can be avoided using np.array_split but this creates another error about a singular matrix caused by divide by zero in netlist_utils.py for g[R_map] = 1 / value[R_map].

@wigging  I am aware that my method was crude and was concentrating on just getting parallelism working. I don't know why the R_map would have zeros in it.... Do you wanna try fixing this?

I started a fix-arraysplit branch and made the changes from NumPy split to array_split. I have not fixed the issue this causes in netlist_utils.py but here is the error I'm getting:
/liionpack/liionpack/netlist_utils.py:438: RuntimeWarning: divide by zero encountered in true_divide
  g[R_map] = 1 / value[R_map]  # conductance = 1 / R
/opt/miniconda3/envs/liionpack/lib/python3.9/site-packages/scipy/sparse/linalg/dsolve/linsolve.py:206: MatrixRankWarning: Matrix is exactly singular
  warn("Matrix is exactly singular", MatrixRankWarning)
Low voltage limit reached

What script are you running?

Running big_circuit_ray.py for nproc=5 using the Dask or Ray manager.

spm_per_worker needs to be the length of the chunk whereas it's a constant at the moment

I think I have it fixed. Let me run some examples before pushing up the changes.


Refactor

I was trying to solve that a url like "http://example.com/foo#bar" doesn't return any hashtags (ie "#bar") and found the source code really hard to follow. Lots of variables that aren't clear what value they hold, lots of indentations, etc.
So I refactored with a different approach that is simpler to follow, significantly less code, and better commented. There are some behavior changes....
the GetTags func now takes a string instead of io.Reader
"@gernest@mwanza" not returns [gernest@mwanza] instead of [gernest, mwanza]
Having your char also be a terminator results in not the behavior you want.
Some of this may be controversial, happy to have that discussion.

Coverage increased (+3.3%) to 97.619% when pulling fa2eaf19dc66be5dfc8a637fbe1668a358dd97d1 on cbarraford:refactor into ad0f22d1fbfec8ee91ba27704007a4eeebdb9eca on gernest:master.

@cbarraford you can merge this. When I get time I will add benchmarks and go-fuzz this so we can easily detect regressions in the future.

👍


Make std.range.enumerate @nogc.

std.conv is not nogc but it's not really neccessary here as far as I can see (a manual > check passes all existing tests).

@wilzbach @andralex @JackStouffer CI's (was?) broken, tests pass for the ones that work

@wilzbach @andralex @JackStouffer CI's (was?) broken, tests pass for the ones that work

tests pass for the ones that work

?

tests pass for the ones that work

?

tests pass for the ones that work

?
@atilaneves
The CI was broken due some cert issue (not my area of expertise)

tests pass for the ones that work

?
@atilaneves
The CI was broken due some cert issue (not my area of expertise)


head title: undefined on loading

Given:
// head
titleTemplate: '%s - Marvin ROGER'

And two pages index.vue and about.vue each containing its title as head title.
On ^0.10.0, if we're on the index page and click on the about link, the initial title is Home and, after the about page is loaded, the title is About. This is the correct and expected behavior.
On ^0.11.0 however, the initial title is Home, during loading it is undefined, and after it is About. The issue being obviously that the page title is undefined on loading.
Here is a link to a repo where the issue happens : https://github.com/marvinroger/www.marvinroger.fr
This repo is on ^0.10.0 and it works fine, but try changing to ^0.11.0 and you'll see the issue.

0.11.0 ?

I have the same problem. nuxt 0.10.6

User click on the link.
Commit goes to the store to change the state.
Vue/Nuxt rerender the head of the old page with the new store value.
Vue/Nuxt push router to the chosen page.

Because the store change for the new page but the head is reendered for the old one it gives errors:
TypeError: Cannot read property 'id' of null
    at VueComponent.head (_slug.vue:46)
    at Watcher.get (vue.runtime.esm.js:2463)
    at Watcher.evaluate (vue.runtime.esm.js:2563)
    at VueComponent.computedGetter [as $metaInfo] (vue.runtime.esm.js:2807)

@alexchopin Could we block rerendering the head in the phase of loading new page?
This would solve the problem.

Oh my bad, I messed up with two deps. The issue does not appear on 0.10.5 but does on 0.10.6

I'm using next 1.4.0 it seems to be still happening


feat: support core regex flags

Summary
Support core RegExp flags.
Test plan

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (27f68ed) 100.00% compared to head (87413cf) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #26   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          106       128   +22
  Branches        23        30    +7
=========================================
+ Hits           106       128   +22

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


Show a warning if a  does not have a heading element within it

Describe the new a11y feature or project enhancement

According to MDN and w3, a <section> "should" contain a heading element (the heading element is used to describe the section). This should only be a warning and not an error because MDN and w3 both state "should" and not "must"; it's an advisory rather than a requirement.
Link(s)

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/section

Hey there, I can help you with this feature, can you assign it to me?

@shamahdotdev certainly. I've had a couple of attempts at this already and couldn't figure it out so would be interesting if you can figure it out 🙂

I like the idea of :has(), shame it has 0% support on caniuse.com. But I think your suggested approach is a good one for now. I'd be happy to look at your PR now.


Disjuncture on example codes and documentation

Hi there
I am using the documentation as a reference. the setFont call in the examples and the signature on the documents does not tally. Perhaps there has been a typo I'm not sure.
Thanks.

Both variants work. We added a separate fontWeight parameter in 2.3.0/#3036.


[Bug] Observability-starter Consider renaming one of the beans or enabling overriding

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

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

Apache Dubbo Component
Java SDK (apache/dubbo)
Dubbo Version
Dubbo 3.2.15
dubbo-spring-boot-observability-starter 3.2.15
SpringBoot 3.2.1
spring-boot-actuator-autoconfigure 3.2.1
Steps to reproduce this issue
According to dubbo3 design specifications, spring.main.allow-bean-definition-overriding=true should not be set as the default, so this problem needs to be fixed.
[2024-09-05 09:17:22]  WARN [traceId=] - Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.support.BeanDefinitionOverrideException: Invalid bean definition with name 'correlationFieldsCorrelationScopeCustomizer' defined in class path resource [org/apache/dubbo/spring/boot/observability/autoconfigure/brave/BraveAutoConfiguration$BraveBaggageConfiguration.class]: Cannot register bean definition [Root bean: class [null]; scope=; abstract=false; lazyInit=null; autowireMode=3; dependencyCheck=0; autowireCandidate=true; primary=false; factoryBeanName=org.apache.dubbo.spring.boot.observability.autoconfigure.brave.BraveAutoConfiguration$BraveBaggageConfiguration; factoryMethodName=correlationFieldsCorrelationScopeCustomizer; initMethodNames=null; destroyMethodNames=[(inferred)]; defined in class path resource [org/apache/dubbo/spring/boot/observability/autoconfigure/brave/BraveAutoConfiguration$BraveBaggageConfiguration.class]] for bean 'correlationFieldsCorrelationScopeCustomizer' since there is already [Root bean: class [null]; scope=; abstract=false; lazyInit=null; autowireMode=3; dependencyCheck=0; autowireCandidate=true; primary=false; factoryBeanName=org.springframework.boot.actuate.autoconfigure.tracing.BravePropagationConfigurations$PropagationWithBaggage; factoryMethodName=correlationFieldsCorrelationScopeCustomizer; initMethodNames=null; destroyMethodNames=[(inferred)]; defined in class path resource [org/springframework/boot/actuate/autoconfigure/tracing/BravePropagationConfigurations$PropagationWithBaggage.class]] bound.
What you expected to happen
It can run normally without setting spring.main.allow-bean-definition-overriding=true
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

#14631


Add more option to select from

Hi,
Can more text options be added to select from after autofill? For example, there is a form where I have to fill a text field from 8 different serials.
Thanks :)

You can fill the data from Google Sheets also .
Checkout video for same. https://youtu.be/5_ToA53CMAE


Colorize YAML in the YAML view

@JamWils :

have the YAML color code keys and values

Closed as a duplicate of https://github.com/weaveworks/weave-gitops/issues/3513


Fully enable HTPasswd as IDP

Enable and provide documentation for using HTPasswd as a real IDP for ROSA and OSD clusters.
This would include being able to configure it from the OCM UI.

deferring to https://github.com/openshift-cs/managed-openshift/issues/15

Enabled as of ROSA CLI 1.2.0


rowspan attribute is missing

A grep of the course doesn't find the "rowspan" attribute for td tags.  See http://www.w3.org/TR/html401/struct/tables.html#h-<IP_ADDRESS>

scala> td(rowspan:=4)
res6: scalatags.Text.TypedTag[String] =

scala> th(rowspan:=3)
res8: scalatags.Text.TypedTag[String] =

I'm running master

https://github.com/lihaoyi/scalatags/search?utf8=✓&q=rowspan
scalatags/shared/src/main/scala/scalatags/generic/Attrs.scala  line 973

[Embarrassed grimace…]  Thanks, Mark.  I was grepping out of date sources :(
Byron

On Apr 17, 2015, at 10:06 AM, mark lister<EMAIL_ADDRESS>wrote:
scala> td(rowspan:=4)
res6: scalatags.Text.TypedTag[String] =
scala> th(rowspan:=3)
res8: scalatags.Text.TypedTag[String] =
I'm running master
—
Reply to this email directly or view it on GitHub https://github.com/lihaoyi/scalatags/issues/81#issuecomment-93992518.

Byron Weber Becker
School of Computer Science
University of Waterloo, Waterloo, Ontario  N2L 3G1
Email<EMAIL_ADDRESS>Phone:<PHONE_NUMBER> x34661
Fax:<PHONE_NUMBER> (include my email on cover page)
Advising FAQ:  www.cs.uwaterloo.ca/current/faq

@bwbecker, is it possible to close this issue?


(H.101) mini project: function that returns the hostname

Function in a program that returns the hostname and temperature as a dictionary
Create the same function in fastAPI
Expand this for all OS; use cloudmesh.cc.sysinfo → os_iswindows, os_islinux, etc.
Engage everyone to test out the program and document it as well

not completed


The shape of probs_seq does not match with the shape of the vocabulary Segmentation fault (core dumped)

[/home/nihao/nihao-users2/yuhao/DSLP/env/ctcdecode/ctcdecode/src/ctc_beam_search_decoder.cpp:32] FATAL: "(probs_seq[i].size()) == (vocabulary.size())" check failed. The shape of probs_seq does not match with the shape of the vocabulary
[/home/nihao/nihao-users2/yuhao/DSLP/env/ctcdecode/ctcdecode/src/ctc_beam_search_decoder.cpp:32] FATAL: "(probs_seq[i].size()) == (vocabulary.size())" check failed. The shape of probs_seq does not match with the shape of the vocabulary
[/home/nihao/nihao-users2/yuhao/DSLP/env/ctcdecode/ctcdecode/src/ctc_beam_search_decoder.cpp:32] FATAL: "(probs_seq[i].size()) == (vocabulary.size())" check failed. The shape of probs_seq does not match with the shape of the vocabulary
Segmentation fault (core dumped)
I have encountered such a problem, I have not modified the original code, may I ask what is the problem

FATAL: "(probs_seq[i].size()) == (vocabulary.size())" check failed. The shape of probs_seq does not match with the shape of the vocabulary
I encountered this problem when running GLAT+CTC+SD and CTC+SD codes.
What does that mean, and I haven't changed the DSLP code. I hope the author can clarify my doubts.

Hello, @thunder123321
Unfortunately, there is not enough information for me to tell what went wrong in your setup.
My best guess is that the error is related to your ctcdecode installation.
BTW, I just tested a clean clone of the repo with your script, and it works on my side.

Actually, the ctcdecode is only used as a post-processing in the final version, as I only used beam size 1.
I think you can use the --plain-ctc option to avoid using ctcdecode.
However, you need to do some post-process here: https://github.com/chenyangh/DSLP/blob/a9d3ee154f3bc73b9dfc191ed537ee90b3896956/fairseq/models/nat/nat_ctc_sd_ss.py#L507
You may incorporate this function:
def _ctc_postprocess(tokens):
        hyp = tokens
        # if cfg.task.plain_ctc:
        _toks = hyp.int().tolist()
        _toks = [v for i, v in enumerate(_toks) if i == 0 or v != _toks[i - 1]]
        hyp = hyp.new_tensor([v for v in _toks if v not in extra_symbols_to_ignore])
        return hyp

Hi. @chenyangh Thank you very much for answering my question. I noticed that the two post-processing functions you mentioned appear in generation.py file. Does that mean I just add the -- plain-ctc parameter? When I added the -- plain-ctc parameter in my experiment, I found that the memory footprint was higher. “ctcdecode” is used to reduce memory footprint.

Hi, @thunder123321 --plain-ctc was used to replace the ctcdecode module (as it is much slower even with beam 1). However, having --plain-ctc option will not perform postprocessing during the training. That's was why I kept ctcdecode, and suggest the above modifications.
In terms of memory consumption, I am not sure if that is caused by the plain-ctc option. But I did remember that at some point in my development, the model suddenly consumes more RAM per batch. Unfortunately, I haven't identified the reason.