pundit-matchers feedback

I recently published a set of RSpec matchers for Pundit:
https://github.com/chrisalley/pundit-matchers
https://rubygems.org/gems/pundit-matchers
Before bumping the version number to 1.0.0, some feedback on the API and documentation would be great.
I still need to write an independent test suite. Currently the app that the matchers are used in acts as a confirmation that the matchers work as expected. Good strategies for testing RSpec matchers would helpful too.

If it's useful to you then go ahead and mark it as 1.0.0. I quite like the look of the specs, maybe even better than the action focused style that we bundle with pundit.

Thanks for your kind words @jnicklas. I've just released version 1.0.0.
A mention in the "External Resources" section of the Pundit readme would be great, if you're willing.

Sure, send a PR!


download.py is broken

download.py isn't working any more. I think it is because the nationalmap.gov guys have changed their website/download links.
https://viewer.nationalmap.gov/tnmaccess/api/products

That's definitely possible. I haven't used this project for a while. If you want to make changes, feel free to create a pull request.

I have ASTER GDEM V3 data,how to generate this mbtiles without download, thank!


Add json field to Blob container metadata

The previous change removed the ability to add additional metadata to containers.
But there is now no way to add any additional container-specific properties, and that is essential. A few examples:

for CloudSqlite containers, we need to store the blockSize. This is necessary to attach the container.
for ViewStores, there will be metadata about the types of views and whether the container applies to branches.
for CodeService containers we will store the mapping to external services.
etc.

I imagine that practically every new containerType usage will want to add some properties to it. I added an optional json member that can be used for these purposes.
I also added back the concept of an owner for a "private" container in Scope. We need that for user-private property stores.

Here is the repo for the storage-container-service-client.
I have an implementation for the cache already. I'm working on testing it out right now.
Let me know what you think, the cache is implemented in subClient/RequestCache.ts
I spoke with Arun about this, and Arun was of the opinion that it should be a separate repo within iTwin, so I created the repo based on that.
Currently changing how the responses are handled + testing the cache.


Update prepare_model.py

Updated for working with OpenCV 3.4.3

@Anandesh-Sharma thanks a lot for your contribution. Can you please merge all of your changes into one PR, as it looks like they belong together?

Yeah sure


tlshello pocket pings always is 0

hi. thx for great script.
im using both grpc and ws configs. and all of them with tls. but during the test i have ping without tlshello . but with tlshello is bring back ping to always 0.
is it possible to use other pocket than tlshello on a config using tls? cause in this test those pockts actually i have a good pings . between 300 to 2000.

Hello, we removed tlshello thanks to the suggestion from @GFW-knocker #7 so kindly try the latest version of the script here, but you can set your desired fragment values by editing the ps1 file under Arrays of possible values at any time.


Rewrite unwanted phrases

Let's make sure no PR is merged with phrases we don't appreciate.

I agree that we may introduce some more objects, we may also make the subscribers fail individually and try to minimise "update requests". But that fall outside of the current PR.


guigaga uses the flag short name with Typer

guigaga uses the short name of flags instead of the more informative long flags:

I think it might use the first flag. eg this:
    sequences: Path = typer.Option(
        ...,
        "--sequences",
        "-s",
        help="Path to sequences for alignment"),

Produces:

Ideally, it should use the long flag if available.

I think the above is causing an issue with how guigaga/gradio is interacting with typer. If the short name is parsed (i.e. it's first), it results in an empty object:
This is for the organism flag:
'in <string>' requires string as left operand, not OptionInfo


Until and including functionality

Currently, this widget displays multi-day events as 'until', which means that the last day is not marked as part of the event. Can you please add a toggle to switch this to 'until and including', which allows me to have the last day included in the event?

We will not add new feature to this widget, please use. New Calendar widget https://appstore.home.mendix.com/link/app/107954/


Missing Pagination

Thanks in advance for your bug report!

[X] Have you reproduced this issue in incognito/private browsing?
[X] Have you made sure you issue doesn't already exist?

Where is the URL that this occurs?
https://web.pulsar-edit.dev/packages
What's your issue?
The pagination is missing.

Which OS/Browser/Version does this happen on?
No response
Steps to Reproduce/Additional Details:
No response

I've gone ahead and transferred this issue to the correct repository


Default/Example Row for DocumentationTool

Especially new users struggle with putting valid data in the CE-NOME_Documentation_Tool. Strings, numbers or units in the wrong places make the generation of IDs impossible. It would be nice to have a default row in the beginning where users can only modify specific values while seeing the correct format.

yes this can easily be done good idea and then this row is always skipped, i cant make the row in editable though,
we also can add some description at the top, defining default values with markdown


Display noResults (prevent dropdown closing) when creating custom renderer in React

Description
More of a question than a bug as such but I've been wrestling with this for a little while now and could do with some input on the best way to achieve this.
I'm using the react-render example for the algolia/autocomplete/examples folder as a basic skeleton, which uses the createAutocomplete function from autocomplete-core.
The regular autocomplete function from autocomplete-js has a renderNoResults method but this doesn't appear on createAutocomplete.
In addition to this, the default behaviour for autocompleteState.isOpen(being managed in a useState hook) is set to return false, and I haven't been able to override this in a particularly stable way.
I've tried manually calling setIsOpen based on focus events on my search input, but this seems to open the dropdown without calling the search query, meaning there are no results displayed when initially focusing on the search input.
Any advice on how best to manage this would be awesome.

Expected behavior
To be able to prevent the default behavior of closing/hiding the results dropdown when the query returns no results, and instead render a 'No Results' message.

Environment

OS: MacOS
Browser: Brave
Autocomplete version: 1.1.0

Hey! Do you have a reproduction based on this sandbox?
Otherwise, did you have a look at using Autocomplete JS bound to React? Did you encounter any limitations?

Closing for inactivity.


Routing doesn't find controllers when using StarterWeb template

The reason is that the template is missing router.MapControllers() call in the Startup.Configure() method. This results in the Default Identity UI's AccountController not routeable hence the error.

VSO bug to close when finished: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/797529?src=WorkItemMention&src-action=artifact_link

Also this one: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/797529

If I understand it correctly this change is testing the scenario described here, correct? If so it would seem it doesn't repro (at least through the CLI).

May be related https://github.com/aspnet/AspNetCore/issues/8387

On review #8387 seems unrelated.

Closed in https://github.com/aspnet/AspNetCore/pull/8685.


updated props in formBuilder

This PR addresses the missing onChange prop on the FormBuilder definition - which is useful for setting up a controlled component

Thanks for the contribution!

NPM build not published after the onChange request merged in master.
Still getting the same definition


How to get the pool reserves?

How i can get the pool reserves of tokens pair?
Is there some like getReserves() method?

Hey thanks for this issue. We do not expose anything like this at the moment. It is on my todo list to return the pool sizes back within the trade context so you know how much is in the pool etc but its not on the top of my list. For now you can call the pair contract directly to get that information. Will keep this open and let you know when this feature is in.


Suffering useEffect deps bugs when using setFilter instance properties?

Using v7?
Yes.
Describe the bug
I built a searchbar outside the table to filter data in the table. I used setFilter instance properties to filter my data based-on column id and the keyword. The thing is that I don't really know how to put setFilter as a dependency when I use it inside useEffect. Below is the example code
const TableContainer = () => {
   const [searchBarValue, setSearchBarValue] = useState('')

  return (
  <>
    <input
        value={searchBarValue}
        onChange={e => setSearchBarValue(e.target.value)}
     />
    <Table
        searchBarValue={searchBarValue}
     />
  </>)
}

function Table({serachBarValue}){
  const {setFilter} = useTable(
    {
      columns,
      data,
    },
    useFilters,
  )

  React.useEffect(() => {
      setFilter('name', searchValue)
  }, [searchBarValue, setFilter])

  return(/* JSX */)
}

If I write like the code above, I will suffer an infinite loop effects, and if I remove setFilter in deps array, it will show useEffect deps warning messages. For now, I just add linter disable comments to avoid that. Is there any better way to use setFilter inside useEffect?
To Reproduce
Steps to reproduce the behavior:

I send searchBarValue as a prop to Table
setFilter function is placed inside useEffect, and set serachBarValue and setFilter as deps
Once the searchBarValue changed, it will re-run the effect infinitely

Expected behavior
The effect shouldn't be triggered infinitely
Desktop (please complete the following information):

OS: MacOS
Browser: chrome
Version: 79

I don't know that this is strictly related to useEffect(). I tried doing something similar by just using setFilter() directly (with a hard-coded value):
function Table({searchBarValue}){
  const {setFilter} = useTable(
    {
      columns,
      data,
    },
    useFilters,
  )

  setFilter('name', 'myvalue');

  return(/* JSX */)
}

results in: "Too many re-renders. React limits the number of renders to prevent an infinite loop."

@reinrl , the later code example is what I'm doing right now, and I disabled eslint warning by placing disable comment right above the dep array, but i think setFilter should be placed inside deps array just like dispatch from useReducer.

I'm having the same problem, I pass down setFilter to a custom Select that goes on an infinite loop because of a useEffect dep.
A temporary fix is to wrap setFilter in a useCallback hook, even tho there will be a warning cause we are missing a dep.
const setFilterTemp = React.useCallback(args => setFilter(args), [])

Same problem with column.setFilter()

This is fixed in master and slated for release. Thanks for your help everyone!


What is the reasoning behind having 2 types which are similar?

https://github.com/alchemyplatform/aa-sdk/blob/38b2ebf6e1373c50a9cc1dbeea78f8f2ef3ee8d9/packages/core/src/signer/types.ts#L5-L9
https://github.com/alchemyplatform/aa-sdk/blob/38b2ebf6e1373c50a9cc1dbeea78f8f2ef3ee8d9/packages/core/src/account/simple.ts#L20-L24
Do the above types need refactoring to one?

Good catch, I think we need to just kill the SimpleSmartAccountOwner and migrate everything to using SmartAccountSigner


Bing + other geocoders alternative REST interfaces

Bing has an Elevation interface for querying locations' elevations.
It might also be possible that other services are offering multitude of query types & points.
I haven't found if and where it is possible to address these in the geopy API.

geopy is not aiming at providing support for all APIs of the geocoding services. Instead only the most common geocode/reverse methods are implemented for each provider.
If you need to use the less common methods of a provider I would suggest to use their specific library, which usually aims to provide full API support. For Bing the following package seems to provide access to the Elevation API: https://pypi.org/project/bingmaps/


Updated autoprefixer.js

I wonder why this pull request has not been accepted yet. It works perfectly fine and an update of autoprefixer.js breaks it all without it. Thank you @levmorozov!


Add a link to the Spring Boot docs about Redis configuration.

As the discussion in https://github.com/spring-guides/gs-messaging-redis/issues/20 indicates, it would be good to add a link to the Spring Boot docs that have to do with Redis configuration.
This is a good one for a first-time committer.

Hello, I want to try to complete this work, but I don't know where this document needs to be added to this project.

I would add it as a note at the top of the "Create a Redis Message Receiver" section.

@Buzzardo I've done it, please help review it #27

In the PR #32 I added the link specified in #20. The guide now uses Spring Boot Docker Compose support, so using application.properties to configure connection details is no longer required, if it ever was. I dropped the link in the NOTE section, so hopefully readers will be able to easily find more information if they choose.


Error: No binary available for arch 'undefined' on Raspberry Pi 4

I filed this issue with the same title in the npm-go-ipfs repository. The error information in that issue is originating from orbit-db due to the go-ipfs dev dependency. Is it absolutely necessary? An easy fix would be to remove it from the dependency list.
I verified the error occurs with the latest version of orbit-db (v0.28.1)

We need go-ipfs for tests. It looks like the underlying issue is being fixed.

I was able to overcome this issue by setting the following environment variable in Ubuntu:

export TARGET_ARCH=arm64

I also ran into this issue with a M1 macbook pro. I solved it the same way


<robot_param> and <robot_param_node> tags do not work with ign_ros2_control plugin

I am currently using ROS Humble + Ign Fortress and trying to load ign_ros2_control plugin using Ign Fortress. I am defining my own robot param parameter node as shown below using the instructions in documentation (I changed the plugin to ign as I am using Fortress)
<plugin filename="ign_ros2_control-system" name="ign_ros2_control::IgnitionROS2ControlPlugin">
  <parameters>$(find my_description)/config/default_controller.yaml</parameters>
  <robot_param>robot_description</robot_param>
  <robot_param_node>my_robot_state_publisher</robot_param_node>
</plugin>

I get the following error:
[ERROR] [1712874118.400169640] [gz_ros2_control]: robot_state_publisher service not available, waiting again...

It seems that the plugin is trying to access param in default node name robot_state_publisher and not the prescribed node name my_robot_state_publisher. Upon looking up in the source, it seems that the plugin tags were never added. The same TODO is present in the master/rolling and iron branches.
Can the maintainers confirm issue of missing feature or help point out if I am missing something?

@rachit-aggarwal This PR https://github.com/ros-controls/gz_ros2_control/pull/275 should fix your issues


Is penaltymodel-lp 0.1.5 compatible with numpy 1.19.1 and scipy 1.5.2?

Current Problem
We are currently trying to install dwave-ocean-sdk 4.2.0 in our HPC software stack at Forschungszentrum Juelich. Our environment has Python 3.8.5, numpy 1.19.1 and scipy 1.5.2, but unfortunately the installation fails because penaltymodel-lp 0.1.5 requires numpy >=1.19.4 and scipy >= 1.5.4.
Proposed Solution
Would it be possible for you to test and confirm if penaltymodel-lp 0.1.5 works correctly with numpy 1.19.1 and scipy 1.5.2?
Many thanks in advance!

Hi @ccalaza, looking into it now, will get back to you shortly.

It appears that those versions do work. I have made a PR to slightly lower the minimum bound, see https://github.com/dwavesystems/penaltymodel/pull/135.
Do you need a release of penaltymodel-lp and/or the sdk? Or does knowing that it will work suffice?

I just needed confirmation, I'll patch the installation myself. Once again thanks a lot for the super-quick reaction!

Hi @ccalaza , following up on this. We'd like to raise our minimum supported SciPy version to 1.6.0 to take advantage of the 'highs' linear programming method, see link.
I am curious what the impact on y'all would be? How much of a blocker would this be to using Ocean?

I was able to find a workaround (see #138 ), but still interested to know for future reference.

Hi Alex,
Happy new year and thanks for checking with us, that's very much appreciated!
Unfortunately, as of now we are stuck with Scipy 1.5.2. But the plan is to upgrade to a new environment with Scipy 1. 7.1 in the coming months, hopefully by the beginning of Q2.
X
Carlos

Hi @ccalaza , checking in again! Any chance it's been updated?

Hi Alex!
We are currently almost done switching to our software stage "2022" with NumPy 1.21.3 and SciPy 1.7.1. If everything goes according to plan we will stick to these versions at least until around this time next year, when we switch to stage "2023".
Unless of course there's some bug in these packages or their dependencies which forces us to upgrade before then, or for some unlikely reason we need to deploy a "2022b" stage in the 2nd half of 2022. Even then we may not need to upgrade NumPy and Scipy, though.
TLDR, we are extremely likely to stick to NumPy 1.21.3 and SciPy 1.7.1 until at least Q2 2023.
X
Carlos


Failed to invoke method 'SetTelemetryInterval': Not found

My back end application keeps having this error. Any way to fix it?

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

ID: e0c550f7-328c-7d5d-0594-e0bb7ac9cb7a
Version Independent ID: ce2d86b7-e3ca-a7c9-9fef-43631f6f7fdb
Content: Control a device from Azure IoT Hub quickstart (Node.js)
Content Source: articles/iot-hub/quickstart-control-device-node.md
Service: iot-hub
GitHub Login: @dominicbetts
Microsoft Alias: dobett

@BrandonPoach Thanks for the feedback! Can you provide more details about the issue? When are you getting this error message?

I got the same error after entering the "node BackEndApplication.js" command in the Call the direct method section.

@BrandonPoach
I fixed this problem by ensuring that the Simulated Device is running before invoking the BackEnd Application.

@JimacoMS3 That seemed to fix the problem. Thanks.

Glad to hear that issue is resolved!

I had this error too. Make sure
var deviceId = 'MyNodeDevice';
in BackEndApplication.js matches the name of the device you are trying to control.

@mjb484 comment worked for me.

use simaulated-device_2 istead of simualted-device app

@navidarry  -- Just checking, this is already explained in the article. Correct?

@JimacoMS3 -- Some people do not attention to article carefully. They use simulated-device app that used in previous article for method invocation. like me!


Awsbatch 1kg 100sample

Re-work of the workflow, tested with 100 samples from 1KG project re-call on DRAGEN

@justinjj24 , as @mhebrard rightfully pointed at there are several GRIDS resources (S3 & ECR uri) hardcoded which isnt a good thing given it is a public repo. Can you fix it before the PR is acknowledged and merged.
@mhebrard can you help @justinjj24 if it isnt obvious how to fix those ?

Re-opening to handle comments and fixes


Disable problematic integration tests

What does this PR do?
Disable problematic odo integration tests temporarily until the issues are fixed.
What issues does this PR fix or reference?
Is your PR tested? Consider putting some instruction how to test your changes

/retest

/retest

/retest


login and sigin with firebase

i am gssoc contributor

can you assign this to me, I want to work on this under GSSoC'23 . I think my skills would be a good math for this issue.

Hi i would like to work on this issue as well, could you please assign this to me? I am GSSoC'2023 participant

Hello. I'm a GSSoC'23 contributor. I want to login and signin with firebase. Could you please assign me this task.

Dear @LegendSumeet ,
Thank you for submitting your Pull Request regarding the login and sign-in functionality. After careful consideration, we have decided not to include these features in the app as they fall outside the primary scope of its functionalities.
We appreciate your effort in raising this issue and your dedication to improving the app. We encourage you to redirect your focus towards identifying other potential bugs or suggesting enhancements that align with the app's core objectives.
Once again, thank you for your contribution and your understanding. We look forward to your continued involvement in making our app even better.
Best regards,
Ashhtosh Jha/Spectrum UI


Imp defined as Kore definitions - DO NOT REVIEW

There are two main files related to this PR:
(1) imp.k contains the complete Kore definition of IMP
(2) imp2.k contains the Kore definition of IMP that doesn't use any AC matching and context.
When @mattoxb finish defining built-in modules of INT, LIST, SET, and MAP, this PR will be ready to review.
UPDATE: Built-in modules are done in this PR https://github.com/kframework/kore/pull/101
@traiansf @virgil-serbanuta Do we have any coding style for Kore definitions? I was not following any coding style but I'd love to accommodate my code if there's one. If we don't have one yet, I think we may want to set up a simple one (for writing Kore definitions), as these definitions are getting bigger and bigger.

@hreada I'll try to write a Kore style guide that encodes the current practice, maybe we can add as a task at the next iteration meeting. I took a quick look and your Kore code looks mostly readable, but you should never use tabs. There are many reasons for that, but, as an example, look at the exists indentation displayed by the github file diff for the following code extracted from your PR:
  /* intAsAExp is functional */

  axiom{R}
    \forall{R}(
      I:Int{},
      \exists{R}(
        E:AExp{},
	\equals{AExp{},R}(
	  intAsAExp{}(I:Int{}),
	  E:AExp{})))
  []

Besides this, don't forget to update the golden test files as described here:
https://github.com/kframework/kore/blob/master/src/main/haskell/language-kore/README.md
i.e.
stack test --no-keep-going --ta --accept

@virgil-serbanuta Thanks for the feedback. I've converted all tabs to spaces and update golden test files.

This PR is ready for review

@virgil-serbanuta Sorry about that. I'll do a second examine on the code to make sure there's no tab.

It'd be helpful if @traiansf could also take a look at it.

I'll need to adjust the syntax for attributes and make it up to date with the resent commit of attributes.kore to the master branch.

@virgil-serbanuta Is there a way to add new attributes syntax? I need seqstrict. I tried to modify attributes.kore, but that doesn't seem to work.

@hreada you should change https://github.com/kframework/kore/blob/master/src/main/haskell/language-kore/src/Data/Kore/Implicit/Attributes.hs , which will generate the attributes.kore file (you have to run the tests with --accept twice since the attributes.kore file is actually a golden test file, and then you also need to update the attributes.kore.golden file).

@virgil-serbanuta I've got a problem with updating Attribute.kore. After I changed https://github.com/kframework/kore/blob/master/src/main/haskell/language-kore/src/Data/Kore/Implicit/Attributes.hs, I ran stack test --no-keep-going --ta --accept twice, but Attributes.kore is not modified. Could you please let me know if I did anything wrong?

I forgot: please fix the tests.

@virgil-serbanuta I'm trying to fix the failed tests issue. I ran stack test --no-keep-going --ta --accept to update the golden files, but it prompted this error msg:
Test suite failure for package matching-logic-<IP_ADDRESS>
    test:  exited with: ExitFailure 1
Full log available at .../.stack-work/logs/matching-logic-<IP_ADDRESS>-test.log
    Invalid option `--accept'
    Usage: test [-p|--pattern PATTERN] [-t|--timeout DURATION]
                [-j|--num-threads NUMBER] [--xml ARG] [-l|--list-tests]
                [-j|--num-threads NUMBER] [-q|--quiet] [--hide-successes]
                [--color never|always|auto]

It seems that the matching-logic-<IP_ADDRESS> test suite doesn't accept the option --accept. Could you help on this issue?

@hreada That's interesting. I have no clue how flag parsing for tests works, but it seems that Haskell ignores flags from libraries that are unused. The easiest fix is to do this:
cd language-kore
stack init
stack test --no-keep-going --ta --accept
cd ..

However, please note that the test that is currently failing on Jenkins is a Scala test, so it's unrelated to stack.

@virgil-serbanuta Thank you. I noticed that (it took me a while to realize that I do have the access to see the details of tests failure). I am fixing that one now.

It turns out that the Scala parse doesn't implement import and multiple modules yet. I'll fix that in this PR.

I fixed the stack overflow issue. I also added hook-sort and hook-symbol to the scala parse, as they are in the same Trello card. Please review.


enable kernel patch for Ubuntu 22.04

build kernel patch for Ubuntu 22.04
move to new kernel repo URL
remove some “sudo” command
BTW, there should be some issue in the kernel source repo for 5.15.0-39. In my environment, I can't insert the kernel module the system.

https://github.com/IntelRealSense/librealsense/blame/master/doc/installation.md#L70
Hi @dmipx , according to this document. the script "patch-realsense-ubuntu-lts.sh" should be called from the root directory of

@mengyui I tried on clean ubuntu22, patch finished with following message, looks like module was not signed during build

Patched kernels modules were created successfully

[sudo] password for dima:
Replacing videodev  -
	Applying the patched module ... modprobe: ERROR: could not insert 'videodev': Exec format error
Failed to insert the patched module. Operation is aborted, the original module is restored
Verify that the current kernel version is aligned to the patched module version
The original  videodev  module was reloaded

root@ubuntu22:/home/dima/git/librealsense# rmmod videodev

root@ubuntu22:/home/dima/git/librealsense# insmod /home/dima/git/librealsense/ubuntu-jammy/drivers/media/v4l2-core/videodev.ko

insmod: ERROR: could not insert module /home/dima/git/librealsense/ubuntu-jammy/drivers/media/v4l2-core/videodev.ko: Invalid module format

root@ubuntu22:/home/dima/git/librealsense# modinfo videodev

filename:       /lib/modules/5.15.0-41-generic/kernel/drivers/media/v4l2-core/videodev.ko

alias:          char-major-81-*

license:        GPL

description:    Video4Linux2 core driver

author:         Alan Cox, Mauro Carvalho Chehab<EMAIL_ADDRESS>Bill Dirks, Justin Schoeman, Gerd Knorr

srcversion:     5DEC66E6CBCC717A674CEFB

depends:        mc

retpoline:      Y

intree:         Y

name:           videodev

vermagic:       5.15.0-41-generic SMP mod_unload modversions

sig_id:         PKCS#7

signer:         Build time autogenerated kernel key

sig_key:        1F:4E:23:82:4E:7F:DB:01:30:C1:63:2D:9B:09:11:2C:89:6D:22:1F

sig_hashalgo:   sha512

signature:      81:87:35:47:BE:9A:9D:A4:55:C9:73:BE:9B:30:4A:59:50:35:9D:99:

		49:B8:14:0D:9D:5B:74:43:9F:BF:62:D4:6A:47:78:E8:07:5C:19:79:

		00:45:9B:81:84:FB:76:0D:FF:3F:F9:FC:28:CB:4E:DC:E2:C3:1D:96:

		C2:2F:5C:CF:97:E5:EA:4F:E0:D0:A2:58:97:D1:A5:1F:38:AF:1D:19:

		05:DF:02:65:82:60:FC:67:95:73:78:CC:3C:DB:E6:BD:0D:07:5F:D5:

		75:37:2C:1F:F9:51:F6:78:4D:F0:23:AC:9A:B7:E3:82:D7:20:84:D3:

		93:AA:DF:2A:D6:96:06:57:1B:9B:37:9B:5A:FB:A0:58:96:29:FC:D6:

		DA:D8:1D:FF:9D:D3:6E:D3:F8:B8:08:A7:0D:16:75:78:D5:41:C3:50:

		C5:55:33:B3:9C:03:11:27:9C:5B:E2:50:40:C8:0C:45:1C:FF:62:9A:

		C4:94:F9:FA:85:C1:01:30:15:B9:E7:2D:09:97:01:80:3F:00:B7:05:

		10:28:76:A3:84:E8:7F:3C:DC:EF:A0:D1:75:D8:37:28:2B:1D:67:37:

		EE:A5:4B:41:41:96:EF:A9:B0:0B:8A:6E:9F:37:A2:76:2D:52:49:9E:

		80:A2:7E:76:DA:63:F3:FC:4C:B3:1B:E5:B5:E3:18:AC:6B:18:E2:69:

		8D:9B:FD:9A:CA:46:47:C8:F9:CB:E6:98:8B:01:3F:A2:AA:F8:29:A5:

		C1:7A:ED:E7:E6:1B:C4:FD:D1:E9:AD:57:60:38:65:92:7D:98:AC:DB:

		03:AE:66:39:B2:FC:C8:CB:69:73:D4:97:CD:66:DD:18:7B:85:22:40:

		9F:6B:88:ED:1F:2E:E9:80:46:4F:EA:D4:4F:AF:7D:23:84:E2:03:34:

		57:27:51:9D:13:35:2D:96:E7:DB:DC:A1:8E:2B:9F:9F:2A:E0:01:9B:

		AF:90:98:3A:AD:4E:CB:EC:03:91:F5:DC:8A:A9:61:12:78:22:17:1F:

		05:36:60:8E:0E:EC:9E:51:AB:71:04:7E:3A:51:C8:79:36:2B:7C:87:

		6F:8A:36:E6:39:57:F7:A4:44:26:85:F5:DF:3E:8A:68:93:8A:61:25:

		0B:B5:AF:8F:F8:9E:6E:60:B5:4A:A0:4D:39:92:55:96:D8:CF:C0:FC:

		D0:58:36:C2:78:66:31:08:7D:B5:52:11:23:F7:17:D0:2E:39:C2:E1:

		03:2A:E7:36:E2:5A:23:7A:3C:E7:66:B3:C7:DD:1F:F1:29:09:B0:C2:

		2B:B8:4F:83:7C:6F:6E:D5:64:C5:5C:7F:1C:AC:4A:CD:2A:48:13:BB:

		96:D1:5B:A5:BB:A6:BA:F0:6C:1F:74:65

dmesg:
[ 4629.401706] videodev: loading out-of-tree module taints kernel.

[ 4629.401823] videodev: module verification failed: signature and/or required key missing - tainting kernel

[ 4629.402107] module: x86/modules: Skipping invalid relocation target, existing value is nonzero for type 1, loc 00000000ed79e6d1, val ffffffffc090f055

[ 4629.456168] videodev: Linux video capture interface: v2.00

@mengyui , since the changes are in the main script file, this PR needs to be verified for regressions for Ubuntu 16/18 and 20 at least with the latests LTS kernel for each distribution.

@ev-mp - Please note the authur comment
BTW, there should be some issue in the kernel source repo for 5.15.0-39. In my environment, I can't insert the kernel module the system.

This means it will not work for Ubuntu 22 default kernel right?

@mengyui , you need to verify the script againt the older distros (16,18 and 20) as mentioned above.
Please run the tests and share the results internally - this is a prerequisite for merge.

the script works well for Ubuntu 18 (kernel 4.15)
and report error "mobprobe: ERROR: could not insert 'videodev': Exec format error" on Ubuntu 20 (kernel 5.13/5.15) or Ubuntu 22 (kernel 5.15).
It is a very odd issue. I remember there is a version of the kernel source that works with this script when I create this PR.
and the dkms package works well (https://github.com/mengyui/librealsense2-dkms)
the building steps are not changed in this PR.

Hi @mengyui!
We discussed with @ev-mp and best scenario is that all we can merge all new patches but change scripts to be another ones, for example:
scripts/patch-realsense-ubuntu-lts.sh -> scripts/patch-realsense-ubuntu-lts-hwe.sh
scripts/patch-utils.sh -> scripts/patch-utils-hwe.sh
so this will not require regression tests for legacy versions.
Please rework and we will merge it, we need to add Y16I format for 5.13/5.15 kernels and we are blocked by this PR.
Thanks!

Hi @ev-mp @Nir-Az @dmipx the script works well for Ubuntu 18 (kernel 4.15) and report error "mobprobe: ERROR: could not insert 'videodev': Exec format error" on Ubuntu 20 (kernel 5.13/5.15) or Ubuntu 22 (kernel 5.15).
It is a very odd issue. I remember there is a version of the kernel source that works with this script when I create this PR. and the dkms package works well (https://github.com/mengyui/librealsense2-dkms) the building steps are not changed in this PR.

on older kernels - Makefile changed and set "CONFIG_MODULE_SIG=n"
look for example: realsense-camera-formats-xenial-Ubuntu-hwe-4.8.0-58.63_16.04.1.patch
That is not good idea but thus we can understand why it was working on older kernels.
The DKMS is working well indeed, the DKMS package performs module signing with x509 certificates bundled with kernel package.
BTW, can you PR librealsense2-dkms to dkms4librealsense?


Allow local config files to override the default

We currently have a single toml file to provide config info. We should allow having a project-specific toml file which can override the defaults.
Easy-ish

Would this involve removing support for a global configurations? It seams like it would be desirable to require customizations to be per-project and otherwise strictly follow the community style guidelines. I can see having per-installation settings leading to problems and inconsistencies.

I would like to have a default config file which is part of rustfmt and for parts of it to per overridden per-project by the local config files. I don't envisage more than one config file per-project. Or for a machine-local config to override a project-local or default config (other than for rustfmt tests and similar edge cases).

Ideally, rustfmt should also read the config from ~/.rustfmt, not from the directory in which it was compiled.

I don't think it should - the default config should be a default for all installs of rustfmt and should get updated when we change it, rather than being a per-user default. We might want a per-user config as well as the per-project and default configs. That should probably take precedence over per-project (I would strongly recommend against using such a thing, but I wouldn't object to having the ability to use it).

Oh, I agree there should be a global default, but rustfmt currently depends on the default config being in the current working directory, which is not ideal. For example, it means I cannot symlink the rustfmt binary into /usr/local/bin.

Oh, yeah, being in cwd is not ideal in that respect. I'm actually not sure where the best place is for it. I guess once we get into this, we need to get into having an install script or something (I'm not actually sure what the best practice is for files an executable requires - I've only worked on projects too small to care or too big for me to know about that kind of thing).

I think that the tool should ship with a default/standard configuration file that can be overridden by other files. The override files can be defined at any level of the file system, allowing fine grained control of configurations. The lookup for override files starts at the working directory (or the directory containing the current file [dirname]) and goes up, stopping at the first file it finds.
For example, if I'm in /home/simon/projects/rust/, it will look for a .rustfmt in this order:
/home/simon/projects/rust/.rustfmt
/home/simon/projects/.rustfmt
/home/simon/.rustfmt
/home/.rustfmt
/.rustfmt

Users can group their project per configuration (for example, one configuration per organization/company, one configuration per user, etc.) Also, each project can have its own configuration file, and it won't be influenced by parent configurations since the lookup stops at the first one it finds (useful when working on someone else's project).
The fact that the standard configuration is loaded by default allows the users to change only a few details, without having to rewrite a full configuration file. I'd put the default/standard file in /usr/share/rustfmt on Linux, next to the binary on Windows, and no idea on Mac.
I can create a PR for this if it's a model that you like.

@SBSTP I like the idea in principle, but I'd be keen to start simple and see what additional functionality is wanted. To start with I'd like to just have default and a per-project config files. Once that is working, then we could experiment with sub-directory or organisation or per-user configs too. For the latter, I don't think directories are the right model - I think we can't rely on people organising their projects like that - we could use some kind of includes in the config file or take config files as arguments or something.
As for where to put the config file, it sounds like we need some kind of install script in order to do this. I'm tempted to wait for cargo install for that, rather than rolling our own, but I don't want to block overridable configs on that.
If the starting simple approach is amenable, please do make a PR, it would be much appreciated!

This works now!


Flexibilize MarkupSafe pinned version

Description
The current MarkupSafe pinned version has been added in #4746 as a temporary fix for #4745.
However, the current restrictive approach isn't compatible with other libraries that could require an even older version of MarkupSafe, like Airflow 2.2.2 [0], which requires markupsafe>=1.1.1, <2.0.
To avoid that issue, we can allow a greater range of supported MarkupSafe versions. Considering the direct dependency dbt-core has is Jinja2==2.11.3, we can use its pinning as the lower bound, which is MarkupSafe>=0.23 [1].
This fix should be also backported this to 1.0.latest for inclusion in the next v1.0 patch.
[0] https://github.com/adamantike/airflow/blob/2.2.2/setup.cfg#L125
[1] https://github.com/pallets/jinja/blob/2.11.3/setup.py#L53
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have added information about my change to be included in the CHANGELOG.

Not sure if a change like this one would require a changelog entry. Open to suggestions!

@adamantike Thanks for your PR.. a changelog entry for this would be great!

Thanks for the quick reply! So, to validate before adding it to the changelog, would this be applicable to be backported to 1.0.x?
In case it is, should I add the changelog entry for that here too, or do you take care of that when cherry-picking the change?

@adamantike we changed our changelog system to changie so all you need to do is follow the instruction in the link and the actual changelog.md will be generated during release.
@iknox-fa is there a reason we are fixing the upper bound version of MarkUpSafe we can use? I think the install_requires is usually only used for lower bound unless there's a good reason to cap the version.

Thanks, I have added the changelog entry after following the changie instructions.
Regarding why we need to provide an upper bound, it's because of this discussion: https://github.com/pallets/jinja/discussions/1602. jinja didn't fix it in their side because version 2.x isn't supported anymore, so dbt-core requires this pinning until the jinja2 dependency can be upgraded to version 3.x.

@adamantike  Thanks for contributing!!!

Thanks for the quick resolution!
With this PR merged, is there a formal process to ask for this fix to be backported to 1.0.latest?

@adamantike Thanks again for the contribution! Our team will handle backporting your change!


Menu opens when pressing key even when menus like: In-game chat , are open

Code of Conduct

[X] I have read and agree to the project's Code of Conduct

What improvements would you like to see?
I don't really think of any improvements so sorry for not giving any explanation here.
What changes are you suggesting?
Will you add a key bind into the Controls Menu in Minecraft that allows you to change the key you press to open the menu? I am asking for this because when I am typing something into chat and I press the key to open the menu it just opens, can you make it so it only happens when nothing is open? For example In-game Chat
Additional information
Anyway, this mod helped me a lot with PVP since I was able to reduce how much stuff like my hand or shield I see on the screen while playing | Thanks for making this mod! It helped a lot

Might add one myself if mod gains enough interest for it/if I find the time - at this time I am quite swamped with IRL stuff though but I'm happy to review/merge any code contributions if anyone else wants to implement it

Alright, I hope you get free from all the stuff happening IRL soon, have a great day

For future reference - https://fabricmc.net/wiki/tutorial:keybinds


Few questions on status / latest version?

I'm looking at using this module in several projects, and wanted to check in beforehand 👋
Is this module still maintained and/or used in production by anyone? It seems like this is the most established set of middleware for brute force prevention, or is there an alternative I'm missing?
Also, which is the correct/latest version to use?
I'm a little confused as, in the readme it has some documentation about v1.0.0, but the latest version on npm is 0.7.0-beta.0 which was published 2 months ago. Is the beta safe to use?
Finally, I noticed this module depends on underscore, would there be any interest in switching to lodash as that is more common nowadays in my experience. Not a big deal, just wondering 🤔
Thanks for reading! :)

I'm still maintaining it, although It's not under heavy development at the moment (I'm waiting on that until I get the time to build a Koa version and factor out their common code as a dependency. I'm also hoping they can use the same adapters).
We get about 6k installs a month, which I think is the most for any of the brute force prevention middleware I'm aware of. I've used it in production and others have as well.
My bad on the version numbering. I created 0.7.0-beta.0 a while ago, but then decided it was better to call it 1.0.0 on release because there were some changes that weren't backwards compatible (semver). I updated the documentation but hadn't rolled it out via NPM yet. 1.0.0 should be out now for you.
I don't have anything against migrating to lodash, I'd merge the pull request if one came in, but I don't really see a reason to make the change right now since it's been working.

👍  Thanks for the quick & detailed reply
The version mismatch made it seem like the module might have died for some reason (e.g. there was some new alternative) but npm certainly makes this look like the right choice 😁
Thanks for pushing a 1.0.0 version. I also agree there's little benefit to going out of the way to swap underscore for lodash, I was just slightly surprised to see underscore and wondered if there might be a reason. I wonder about things a lot 🍰

No problem :)
If I were starting today I'd use lodash, but in 2013 it was still pretty new and I had lots of experience with underscore hehe.


update readme

A few additions to the readme: one missing step in the instrux, one minor addition (npm install), one typo

Thanks, @tom-smith-okta!


Nightly release not updated since 9th of August

The nightly release hasn’t been updated since the 10th of August. Could this be fixed?

Hm, you're right. Not sure what happened there. Maybe the token expired? 🤔 I'll check.

Updated the token and triggered the nightly pipeline again. It's green, and the binaries showed up again.
My hope is that it's fixed.

Thanks for the hint, Sebastian. 😊


Should we migrated Sentry to EU Region

User Need
As a (type of user)
I want (some action)
so that (some result)
Value
Functional Requirements:

Non-Functional Requirements:

Acceptance Criteria:

Notes:

Not possible. Requires a completely new organisation and a migration. We have approval to operate in current region. High complexity and effort for very low value.


Utilize the path instead of subdomain to manage multi-tenancy

Describe your change in plain English.
Uses the /path instead of the subdomain to manage the multi-tenancy look up. This should be more sustainable in the long run using this approach.
https://jam.dev/c/492ce5db-b7f6-475e-a31e-b11ba92ea979
Notably a number of tests break using this. Maybe we can revisit which tests make sense as we are rebuilding a bunch of things.
Link to the issue
https://github.com/rubyforgood/pet-rescue/issues/177

I will leave this to someone with more experience in tenant architecture than me to approve, but this seems to be a reasonable and slightly simpler approach.

I will leave this to someone with more experience in tenant architecture than me to approve, but this seems to be a reasonable and slightly simpler approach.

Sure. In addition, I can work through and talk through this together in our Tuesday meeting. Most of the multi-tenancy comes from the acts_as_tenant gem, this really handles how we define it. Happy to talk through it.


Add boba sepolia deployment

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

Comparison is base (4355a21) 39.59% compared to head (b3a45d0) 33.66%.

:exclamation: Current head b3a45d0 differs from pull request most recent head 17b0e5d. Consider uploading reports for the commit 17b0e5d to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #99      +/-   ##
===========================================
- Coverage    39.59%   33.66%   -5.94%
===========================================
  Files           80      172      +92
  Lines         5225     7405    +2180
  Branches       749     1235     +486
===========================================
+ Hits          2069     2493     +424
- Misses        3035     4762    +1727
- Partials       121      150      +29

Flag
Coverage Δ

cannon-go-tests
63.48% <ø> (ø)

chain-mon-tests
27.14% <ø> (ø)

common-ts-tests
26.74% <ø> (ø)

contracts-bedrock-tests
19.44% <ø> (?)

contracts-ts-tests
12.25% <ø> (ø)

core-utils-tests
44.03% <ø> (ø)

sdk-next-tests
39.16% <ø> (ø)

sdk-tests
39.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 92 files with indirect coverage changes


Question regarding prod.ca

Is it meant to use prod.ca in production?

I mean, is it just the production .pfx that should be uploaded?


Add more apic icons

Add DashboardMotion icon animation to API Connect icons
Add DashboardMotion icon animation to API Connect demo page in /example
Remove unnecessary CSS in GlobeMotion

Sean, Mike, @elycheea @glapadre
I’d love some feedback on the animation for the Dashboard icon here:
https://deploy-preview-229--carbon-icons-motion.netlify.app/#/apic-demo
It’s one that Mike had animated previously using Lottie, and I tried to replicate with CSS to be used on the API Connect landing page.  Here’s the Lottie version that I used for reference: https://pages.github.ibm.com/Krista-Starr/carbon-icon-animations/#/lottie-demo The ease and timing is definitely different so Mike whenever you have time to share the AE file I can get the exact specs and update.  Are there any other tweaks you guys would recommend?
I’ll be doing a few more CSS versions of Mike’s Lotties to see if I hit any limitations, but I think this is a good one for us to look at as we compare pros and cons of both approaches. After looking at both versions, do you guys have any thoughts?
Also it needs a code review if anyone has time 😄

@kristastarr Both versions are looking good so far. In the CSS version, I feel like a delay before transitioning back could be nice though. Also looks like we have the donut rotating in the opposite direction.
The Lottie version does seem to have a smoother animation at the moment, but I think with adjustments to the timing and easing, they could be comparable. Noticing Lottie lags a bit between animations, but I’m guessing that’s more related to having the same icon several times or just the prototype and not an issue with Lottie itself, but commenting here just in case there’s a limitation there.

To @elycheea's point above, the Lottie version appears to have a smoother animation for now, so with small adjustments to the timing and easing these will look amazing! Question on the dashboard icon in the Lottie demo, is the pause/delay supposed to be so long? It feels off to me like I'm waiting for the icon to transition back to its original state.

The basic setup works great, but I think for hover "loop" the motion should be updated to match the approach we've explored in other icons. Here's a reference vid/.aep
animated_icons_dashboard_01.aep.zip
https://user-images.githubusercontent.com/27870459/229200249-686a0e1e-ecf1-443f-b49c-120e71e9cd4f.mp4

This one is ready for code and motion design review! 🎉
@mikeolasov Let me know what you think! Unless you have strong objections, I would like to move forward with these animations since I think they are pretty consistent with what we've been talking about.  If you see more things that need to be refined, I think it would work best if you can go ahead and open issues for them, and they can be fixed in a future PR (unless they are super small/fast 😅 )

@elycheea The things you mentioned for Globe should be corrected now (globe rotation animation and svg tag alignment).  I removed Grid and replaced it with Apps since our product is using that one now instead.


analytics: fix printing of opt-in status

Hello @maiamcc,
Please review the following commits I made in branch nicks/cmd:
75ddb7b3d9884971465f792f1f921005069b4616 (2018-08-03 14:54:38 -0400)
analytics: fix printing of opt-in status

before this change, it would print current collection strategy: 2, which was...not great

oooooooops :grimacing:

changed it to just 'analytics status'


Make install-lib make target install the pkg-config file.

Issue #643

Thanks for the fix! Merged it into the repository.


Serialize Any field models using the __pydantic_serializer__

Right now, the way BaseModels are serialized when annotated as Any is that it checks for a __pydantic_validator__ attribute to determine if something is a BaseModel, and then takes the __dict__ (in particular, ignoring any model_serializer etc.).
I think this should be changed such that, if something has an attribute called __pydantic_serializer__ (regardless of whether it is a BaseModel or not), that function gets called. (Perhaps it also makes sense to change the ob_type thing from calling that lookup function is_base_model to is_pydantic_serializable or similar..)
(Note: this codepath only gets hit if you annotate a field as having type Any; in this case, I think the best behavior is to ask the item itself how to serialize itself. In particular, as a user you can't really be upset that a "secret" field got leaked if you annotated the return type as Any; note also that this is the same behavior that would happen in v1+FastAPI, so I think it's fine from the "security" perspective.)

Note: This is seems closely related to https://github.com/pydantic/pydantic/issues/5367 — I think we can solve that by, in pydantic_core::serializers::infer::infer_to_python_known, grabbing the SchemaSerializer from the __pydantic_serializer__ attribute and calling it with the same value of extra (for both BaseModel and dataclasses, at least). And adding by_alias as an argument to the to_jsonable_python function.

(This will be closed by https://github.com/pydantic/pydantic-core/pull/557)


add sync server time service

Example of a request problem when the local time is not synchronized:
10002, invalid request, please check your server timestamp or recv_window param. req_timestamp[1688721030939],server_timestamp[1688721231460],recv_window[5000]

Solution: added a service to get server time and a method to synchronize local and server time (delta detection).

I will reopen the PR, make it from the branch


Prevent keyboard from hiding

I'm building an app where I want the keyboard to stay open even after the input loses focus.
After reading posts on Ionic forum and StackOverflow, I tried to refocus on the text input after it loses focus. However, the input only gets focused in desktop browsers, not Safari iOS. It looks like iOS ignores calls to focus(), with or without wrapping in a timeout. This is what I tried:
.directive('constantFocus', function($timeout){
  return {
    restrict: 'A',
    link: function(scope, element, attrs) {
      element[0].addEventListener('focusout', function(e) {
        $timeout(function() {element[0].focus();})
      });
    }
  };
})

<textarea constant-focus></constant-focus>

So, there are really two questions:

Is there a way to make keyboard stay open without having to refocus programmatically?
If having to refocus, is there a way to make focus() work on Safari iOS?

Thanks!

Random idea: could try toggling autofocus on an input?

Thanks. I just tried it, but the input still doesn't receive focus. Is it necessary (or possible) to hack the keyboard plugin to make this work? I haven't read the source but thought I should check first before sinking time on it.

I had a similar issue but in Android: the keyboard flashed because it got hidden after the input lost focus, and then reshown when regained focus.
After testing with different configurations, finally I found something that worked for me: triggering the click event and then the focus for the input element. I don't know if that will work on iOS, but it's worth a try.

Make sure you have <preference name="KeyboardDisplayRequiresUserAction" value="false"/> in your config.xml if you want to call focus() on iOS.
https://cordova.apache.org/docs/en/4.0.0/guide_platforms_ios_config.md.html#iOS Configuration

@tlancina awesome, setting that pref to false work. Thank you!

As a post-mortem note (this issue's page keep appearing first on Google):
If you want to keep the keyboard open when user clicks a button (i.e on a "Send" or "Search" button), you should use (mousedown)="something(); $event.preventDefault()" on that button.
This way was successfully tested on Ionic2.

@edgebal  - The solution works on android. Do we need to do something else for ios?

@edgebal is it working with WKWebView too ?
thanks

@edgebal Thank you so much :) @manishoswal @msio777 tested in iOS and seems to be working fine 👍

@edgebal It works, but it seems to break some underlying native functionality. On Android I'm seeing the textarea not being cleared correctly, and on iOS spelling suggestions become glitchy.

Button within a slides control
These are the lengths I had to go to in order to stop the keyboard from hiding.
`export class CommentsComponent {
@ViewChild('sendButton') sendButton:Button;
constructor() {
}
ngAfterViewInit() {
let el = this.sendButton._elementRef.nativeElement;
el.addEventListener('click', (event) => {
this.stopBubble(event);
});
el.addEventListener('mousedown', (event) => {
this.stopBubble(event);
});
el.addEventListener('touchdown', (event) => {
this.stopBubble(event);
});
el.addEventListener('touchmove', (event) => {
this.stopBubble(event);
});
el.addEventListener('touchstart', (event) => {
this.stopBubble(event);
});
el.addEventListener('touchend', (event) => { //Triggered by a phone
this.stopBubble(event);
this.sendPressed();
});
el.addEventListener('mouseup', (event) => { //Triggered by the browser
this.sendPressed();
});
}
stopBubble(event) {
event.preventDefault();
event.stopPropagation(); //Stops event bubbling
}
sendPressed() {
//... send message routine
}
}`
Why??
If your button is embedded in a slides control, the swiper receives all click, mouse and touch, events. This is so that when you swipe the slides, it can automatically blur any focussed control to tidy up the view for the next transition. This works to your detriment however when you want to implement a send button and have the keyboard remain where it is.
The solution is to bind to the native events of the button and call event.stopPropagation(). This method prevents a phenomenon called 'bubbling'. Bubbling is where the click event of a child DOM element gets sent to its parent. By disabling bubbling with the event.stopPropagation() method, you can prevent the slides control from ever receiving the mouse down message.
I tried doing this with angular event bindings like normal but I was unsuccessful. I resorted to a more direct method of binding direct to the control.
Would be great to see if we can add a directive to buttons embedded in slides which can prevent the slide from responding to the events.

WOW @chrisjpalmer !!!! Thanks A LOT !

Here is a directive I made for suppressing events on a button based on @chrisjpalmer
`
import { AfterViewInit, Directive, ElementRef, EventEmitter, OnChanges, SimpleChange, SimpleChanges } from '@angular/core';
@Directive({
selector: '[suppressEvents]',
inputs: ["suppressEvents"],
outputs: ["onClick"]
})
export class SuppressEvents implements OnChanges {
suppressEvents: string | string[];

constructor(public element: ElementRef) {

}

ngOnChanges(changes: SimpleChanges) {
	if (changes.suppressEvents) {
		if (changes.suppressEvents.firstChange) {
			console.log(this.suppressEvents);
			let el = this.element.nativeElement;

			if (this.suppressEvents == "all" || this.suppressEvents == null) {
				this.suppressEvents = ["click", "mousedown", "touchdown", "touchmove", "touchstart"];

			} else if (typeof this.suppressEvents == "string") {
				this.suppressEvents = [this.suppressEvents];
			} else if (typeof this.suppressEvents == "object" && !Array.isArray(this.suppressEvents)) {
				let r: string[] = [];
				for (let e of this.suppressEvents) {
					r.push(e);
				}
				this.suppressEvents = r;
			}
			for (let evName of this.suppressEvents) {
				el.addEventListener(evName, (event) => {
					this.stopBubble(event);
				});
			}

			el.addEventListener('touchend', (event) => { //Triggered by a phone
				this.stopBubble(event);
				this.onClick.emit(event);
			});
			el.addEventListener('mouseup', (event) => { //Triggered by the browser
				this.onClick.emit(event);
			});
		}
	}
}

stopBubble(event) {
	event.preventDefault();
	event.stopPropagation(); //Stops event bubbling
}

onClick: any = new EventEmitter();

}
`

Hi @johnthackstonanderson thanks so much. Thats a great innovation. I will give this a try a bit later.


feat(config): Allow specifying adapters by string

Allows using lazy.nvim opts (declarative) instead of having to use a config function (imperative)

Lovely addition. Thank you!


question: is there a way to get renderer.info for the renderPass?

when getting trying to gather stats about my scene I'm using renderer.info but it always shows triangles: 1. Of course this is expected since the effect are rendering a single triangle, but is there some sort of way to get the renderer.info from the initial pass?
thank you

I was able to get this in the end with
const {gl} = useThree();
gl.autoClear = false
gl.info.autoReset = false
useFrame((state) => {
	console.log(gl.info)

        state.gl.info.reset()
})

perhaps it is something to do with react-three-fiber rather than this library... not sure. Either way I will close the issue

Postprocessing currently doesn't manage renderer.info.
I'll add this to GeometryPass in v7 to make it the default behaviour.

Thank you for your reply @vanruesc  . I wouldn't necessarily expect postprocessing to manage renderer.info. Having done a bit more digging into it, I don't thing three.js in-house effectcomposer does either.
thanks for the great library


Search UX improvements

Description
Search on Logs, Stays and Moorages was improved:

remove labels, placeholders are self descriptive
Reset removed in favor of per field clearable X
export  moved after the table

Additional improvements:

Stays: filter also searches in Moorage name, not just Stay name.
Moorages: Map was removed above the table (we have separate Map in menu)
Moorages: renamed column to Default Stay Type to make it more clear

Closes #117 and #119
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Improvement/refactoring (non-breaking change that doesn't add any feature but make things better)

thanks great work.


Revert "ci: create prospective merge branch for running the ci tests"

ghpr/build-prospective-branch step tries to build a merge branch when
we're on the master and fails while trying to find the base branch.
Reverting this change so that the pipeline is operational
This reverts commit c8ad0d6fafad0f19e549c8fb3a0ecdff1425a1db.

FYI @antoineco


Set umask to 0077 for etcd process

What this PR does / why we need it:
Set umask to 0077 for etcd process
Which issue(s) this PR fixes:
Fixes https://github.com/gardener/etcd-backup-restore/issues/631
Special notes for your reviewer:
cc @AleksandarSavchev
Release note:

The `etcd` process now runs with umask set to `0077`, this way the files it creates have no permissions on `group` and `others` level.

Thanks for the PR!
/lgtm

I have now tested it by replacing the etcd image on a dev shoot with a newly generated image from this branch. The etcd process did not log any errors and the .snap files were created with 0600 permissions.

We will soon be replacing etcd-custom-image with https://github.com/gardener/etcd-wrapper. So this might not even be needed.

Have you fixed already https://github.com/gardener/etcd-backup-restore/issues/631 in https://github.com/gardener/etcd-wrapper?

@abdasgupta please cherry-pick this PR onto branch rel-v3.4.13-bootstrap.


Python2 npm3 install fails on unicode output

https://github.com/kennethreitz/clint/pull/153
I found a fix for the problem I am experiencing and so did someone else, awhile ago. Seems that if I use the --debug flag with the execution of a build command in a node.js lambda. The install will run then exit with the following truncated error message:
  File "/usr/local/lib/python2.7/site-packages/gordon/resources/lambdas.py", line 538, in _collect_lambda_module_content
    self.project.puts(out.decode("utf-8"))
  File "/usr/local/lib/python2.7/site-packages/gordon/core.py", line 129, in puts
    puts(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/clint/textui/core.py", line 58, in puts
    s = map(str, s)
UnicodeEncodeError: 'ascii' codec can't encode characters in position 0-2: ordinal not in range(128)

I later discovered after some digging that it was complaining about displaying a (✓) \u2713. This likely has something to do with the progress bar that npm displays and the fact that in python2 the str encode doesn't gracefully handle unicode characters. Disclaimer: I'm relatively new to python so please do correct me if I'm wrong.
My versions:
$ python --version
Python 2.7.12
$ node -v
v4.6.1
$ npm -v
3.10.9

I'm merely posting this for posterity.
Update:
The clint PR has been merged.

I also experienced this. --debug is usually what you pass when troubleshooting something, so it's confusing to have it generate its own problems.


background package

this package has to be a boot script otherwise you'd have to use a program to start the boot script every time you reboot, as it needs to load code before it starts
you should wait to add this until the system possibly gets a rework or just close this and tell me to remove the repo
https://raw.githubusercontent.com/protogenraymond/background-pkg-repo/main/boot.js

Yo! This is great!
(You should also make some sort of antivirus to counter VirSUS and Memz)

not that difficult, will port to sus93


Naming Conflicts in generated code

I just ran across an issue caused from the way that field names are generated where we end up with two fields with the same name, and obviously the go code will not compile.
Example:
{
	"type": "record",
	"name": "PROFILE_INFO",
	"fields" : [
		{"name": "PERMRESCARD", "type": ["null", "string"]},
		{"name": "PERM_RES_CARD", "type": ["null", "string"]},
	]
}

The resulting struct will not compile:
type PROFILEINFO struct {
	PERMRESCARD     UnionNullString
	PERMRESCARD     UnionNullString
}

will not compile.

@jtwatson Thanks for bringing this up! We were using snaker to convert names to idiomatic Go names, but it turns out that leads to name collisions in existing schemas (like this).
I've made a change in master which should reduce the number of collisions for now. The

Looks like this will reduce the collisions to the least possible and still produce valid go names.  I have a project that is auto generating code that uses the go code generated by gogen-avro, so it is important for me long term that this is predictable.  I think I will start using your ToPublicName() function.
Thanks for the work you do!


Bug in sap.m.UploadSet with Drag & Drop

OpenUI5 version: 1.108.13
Browser/version (+device/version): Mozilla Firefox, Windows
URL (minimal example if possible):
Example in 1.108.13 (our LTS)
Example in 1.112.1
Steps to reproduce the problem:

drag a file over the drag&drop space and press right click, the file disappears but the state of drag&dropping and the border is still there. This behaviour is partially fixed in 1.112.1 but not in the LTS version 1.108.13
The same happens when the uploadset is right at the corner of the screen and you drag the file over the space and then out of the browser into another monitor or screen.

What is the expected result?
The state of the uploadset goes back to "normal"
Any other information? (attach screenshot if possible)

normal

After right clicking and scrolling the border/rectangle moves with the screen and does not disappear

Hello @thunermay ,
Thank you for sharing this finding. I've created an internal incident<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards,
Lidiya

In which version is this now found? Is it going to be added to the 1.108.X LTS version? Otherwise we can make use of this fix in maybe two years or so, because we rely on the LTS versions.

Hello.
Fix is available with SAPUI5 version 1.114.0
Fix also downported to version 1.108 and will be available in 1.108.14


[Azure Stack] Azure Stack HCI

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/zhoxing-ms/azure-cli-extensions.git@azure_stack_hci_init#subdirectory=src/$EXT&egg=$EXT"

Azure Stack HCI


Quick start metrics

Closes: #370
What

[x] hack/quickstart-metrics.sh
[x] Update docs: new follow on walkthroughs in ocm-control-plane-walkthrough.md
[x] Update docs: metrics-walkthrough.md

Verification
EDIT: Local verification of the quickstart metrics script is not currently possible and needs some changes as the files do not exist in the main branch. Ping me
Run hack/quickstart-setup.sh and/or if you have it running run hack/quickstart-metrics.sh
Verify that https://grafana.<IP_ADDRESS>.nip.io is accessible and dashboards receive data (Istio Workload Dashboard,MGC SRE Dashboard).
For metrics to start working
Use curl to send some traffic to the application.
while true; do curl -k https://$MGC_SUB_DOMAIN && sleep 5; done

Verified that the Istio Workload and MGC SRE dashboards are both present and receiving data.

1 comment about directing the user in Grafana to see the value.

Spoke offline about installing prometheus in both the hub & spoke cluster to ensure all metrics are remote written back to thanos

/lgtm
/approve

Thanks for getting this guide together 👍


feature, TeX: coloring "full" cells

As per discussion elsewhere, v3's outputs allow coloring of "full" table cells, i.e., not just child nodes.
It would be usefult to have access to this in the TeX input. For example, the xcolor package has table-specific coloring options for (\cellcolor, \rowcolor) which could be a useful set of names.

@coatless, I've moved the issue to the MathJax repository.

This is now in the colortbl extension new in v3.2.

Thank you so much @dpvc for following up on this and making it a reality!

@coatless, you are welcome.  This was a good addition, and one that I'm sure many people will be able to take advantage of.


Pdf rendering don't render linear gradient  but png export works

Describe the bug
When i export my pdf i don't have the same ouput as the web output, gradient are broken
To Reproduce
1.Create a new slide with
---
# try also 'default' to start simple
theme: academic
# random image from a curated Unsplash collection by Anthony
# like them? see https://unsplash.com/collections/94734566/slidev
background: https://cover.sli.dev
# some information about your slides, markdown enabled
title: Linux
info: |
  ## XXXXX

# author field for exported PDF
author: XXXXXXX

# https://sli.dev/custom/highlighters.html
highlighter: shiki

monaco: true
# Where to load monaco types from, can be 'cdn', 'local' or 'none'
monacoTypesSource: local

# https://sli.dev/guide/drawing
drawings:
  persist: false
# slide transition: https://sli.dev/guide/animations#slide-transitions
transition: slide-left
# enable MDC Syntax: https://sli.dev/guide/syntax#mdc-syntax
mdc: true

fonts:
  sans: Roboto
  serif: Roboto Slab
  mono: Fira Code

addons:
  - slidev-addon-asciinema
  - prettier-plugin-slidev
---

# XXXXX

---
transition: fade-out
---

# Prérequis

- XXXXXXXXXXX [TXX](XXXXXXXXXXX)
- X `XXX`
- XXX ! 🎉

---
layout: image-left
image: ./homepage-overview-diagram-new.svg
backgroundSize: contain
---

# Openstack

XXX **XXXXX** sXXXX

XXX **XXX** <ic-baseline-wind-power /> XXXXXXXXXXXXXXXT <mdi-counter />

---

<style>
 h1 {
  background-color: #2B90B6;
  background-image: linear-gradient(45deg, #4EC5D4 10%, #146b8c 20%);
  background-size: 100%;
  -webkit-background-clip: text;
  -moz-background-clip: text;
  -webkit-text-fill-color: transparent;
  -moz-text-fill-color: transparent;
}

</style>

# My title

XXXX

<http://XXX/>

<twemoji-warning /> XXX :/

- XX: `XX`

- XXX

pnpm run export --dark

Desktop (please complete the following information):

OS: Fedora 40
Browser: Firefox
Slidev version:
my package.json

{
  "name": "linux",
  "type": "module",
  "private": true,
  "scripts": {
    "build": "npx slidev build",
    "dev": "npx slidev --open",
    "export": "npx slidev export slides*.md"
  },
  "dependencies": {
    "@iconify-json/carbon": "^1.1.33",
    "@iconify-json/devicon-plain": "^1.1.40",
    "@iconify-json/fa": "^1.1.8",
    "@iconify-json/heroicons": "^1.1.21",
    "@iconify-json/ic": "^1.1.17",
    "@iconify-json/logos": "^1.1.42",
    "@iconify-json/mdi": "^1.1.66",
    "@iconify-json/noto": "latest",
    "@iconify-json/simple-icons": "1.1.101",
    "@iconify-json/twemoji": "^1.1.15",
    "@iconify-json/uim": "^1.1.9",
    "@iconify-json/vscode-icons": "1.1.33",
    "@slidev/cli": "^0.49.3",
    "@slidev/theme-default": "^0.25.0",
    "slidev-theme-academic": "^1.3.0",
    "vue": "^3.4.27"
  },
  "devDependencies": {
    "playwright-chromium": "^1.44.0",
    "prettier": "3.2.5",
    "prettier-plugin-slidev": "^1.0.5"

  },
  "packageManager"<EMAIL_ADDRESS>}

The bad pdf rendering here
slides_copy-export.pdf

The good web rendering

The png export is good

Noticed this. I think the reason may be the PDF viewer doesn't support background-clip.

I can confirm this behavior, in my case, linear gradient backgrounds are broken for both PNG and PDF exports.


Changing all instances of OCDIO to Office of the CDIO

Removing all references to "the executive"
A couple of spelling changes.

All good with comments shared in Teams and resolved by JW


Custom input/paste rules not working on initial content

Describe the bug
I would like to accept both one or two tildes for strikethrough marks. (and also change the characters needed for bold and italic, but it's all the same issue)
Eg. ~strike and ~~strike~~ should both produce ~strike~.
I have created a custom strike extension that you can see here. It works great when inputting text manually or pasting text in, but it doesn't work when adding content programmatically, either through the initial editor content or adding it through editor commands.
To Reproduce

Add the custom strike extension linked above to tiptap.
Set the initial content to: ~strike~ ~~strike~~
Observe that only the second strike works - the first one still has tildes, and if you check the markdown output, the tildes are escaped.

Expected behavior
I would like both strikes, single and double, to work properly. Are there more places in the custom strike extension that I need to modify to get this to work?
Desktop (please complete the following information):

OS: Windows 11
Browser: Chrome
Version: 115

Markdown-it does not support single tilde. You will have to add a custom plugin like this one.
Example of how to add a markdown-it plugin: https://github.com/aguingand/tiptap-markdown/blob/main/example/src/extensions/highlight.js#L15

Thanks for your response! I'll do that.


Can't listen to speaks on MacOS

Hi,
I'm already using sounddevice to analyse the output of an Application on Linux, it works well, thanks !
I've tried to do the same on MacOS but the app is not listed on the devices (only have mic and speakers), so I chose to analyze the speaker instead:
import sounddevice as sd

devices = sd.query_devices()
print(devices)

speakers = devices[1]
samplerate = 48000
channels = speakers['max_output_channels']
blocksize = 1024

s = sd.InputStream(device=speakers, channels=channels, samplerate=samplerate, blocksize=blocksize)

> 0 MacBook Air Microphone, Core Audio (1 in, 0 out)
< 1 MacBook Air Speakers, Core Audio (0 in, 2 out)

Traceback (most recent call last):
  File "/Users/xxx/.pyenv/versions/dev3914/lib/python3.9/site-packages/sounddevice.py", line 2746, in _split
    invalue, outvalue = value
ValueError: too many values to unpack (expected 2)

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

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/xxx/.pyenv/versions/dev3914/lib/python3.9/site-packages/sounddevice.py", line 1429, in __init__
    _StreamBase.__init__(self, kind='input', wrap_callback='array',
  File "/Users/xxx/.pyenv/versions/dev3914/lib/python3.9/site-packages/sounddevice.py", line 825, in __init__
    _get_stream_parameters(kind, device, channels, dtype, latency,
  File "/Users/xxx/.pyenv/versions/dev3914/lib/python3.9/site-packages/sounddevice.py", line 2686, in _get_stream_parameters
    device = _get_device_id(device, kind, raise_on_error=True)
  File "/Users/xxx/.pyenv/versions/dev3914/lib/python3.9/site-packages/sounddevice.py", line 2784, in _get_device_id
    idev, odev = _split(id_or_query_string)
  File "/Users/xxx/.pyenv/versions/dev3914/lib/python3.9/site-packages/sounddevice.py", line 2750, in _split
    raise ValueError('Only single values and pairs are allowed') from e
ValueError: Only single values and pairs are allowed

Did I do something wrong ?
On Linux, I'm doing this and it works like a charm !
def process():
    devices = sd.query_devices()
    for device in devices:
        if device['name'] == 'Firefox':
            loopback_device = device['index']

    duration = 10
    samplerate = 48000
    channels = 2
    blocksize = 1024

    with sd.InputStream(device=loopback_device, channels=channels, samplerate=samplerate, blocksize=blocksize) as stream:
        frames_to_capture = int(duration * samplerate / blocksize)

        for _ in range(frames_to_capture):
            data, overflowed = stream.read(blocksize)
            if overflowed:
                print("Warning: Overflow detected!")
                dataprocess(data)

Thanks !

If your device has 0 inputs, you cannot use it in InputStream.
Also, the device argument expects a number or a string, but you are passing a dictionary (which causes the exception).
The error message seems to be of limited usefulness, so I'm trying to improve it in #554. Can you please check if this would provide a more helpful message in your case?

Indeed, the error is far clearer like that, thank you a lot !
So, it's not possible to achieve the results I have on Linux on macOS ? It's not possible to read data from the speakers or any software ?
On Linux query_devices() returns a lot of devices (including applications) but not on macOS. Any idea why ?
Thanks again for your prompt help !

Thanks for testing #554!

On Linux query_devices() returns a lot of devices (including applications)

Please show the list.

Here's the list on Linux. For example, I see Firefox, which is super convenient !
<  0 HDA NVidia: DELL S2721DGF (hw:0,3), ALSA (0 in, 2 out)
   1 HDA NVidia: PHL 245E1 (hw:0,7), ALSA (0 in, 2 out)
   2 HDA NVidia: HDMI 2 (hw:0,8), ALSA (0 in, 8 out)
   3 HDA NVidia: HDMI 3 (hw:0,9), ALSA (0 in, 8 out)
>  4 Logitech StreamCam: USB Audio (hw:1,0), ALSA (2 in, 0 out)
   5 HD-Audio Generic: ALCS1200A Analog (hw:2,0), ALSA (2 in, 6 out)
   6 HD-Audio Generic: ALCS1200A Digital (hw:2,1), ALSA (0 in, 2 out)
   7 HD-Audio Generic: ALCS1200A Alt Analog (hw:2,2), ALSA (2 in, 0 out)
   8 RODE NT-USB: USB Audio (hw:3,0), ALSA (2 in, 0 out)
   9 hdmi, ALSA (0 in, 2 out)
  10 jack, ALSA (2 in, 2 out)
  11 pipewire, ALSA (64 in, 64 out)
  12 pulse, ALSA (32 in, 32 out)
  13 RODE NT-USB Analog Stereo, JACK Audio Connection Kit (4 in, 2 out)
  14 StreamCam Analog Stereo, JACK Audio Connection Kit (2 in, 0 out)
  15 Starship/Matisse HD Audio Controller Analog Stereo, JACK Audio Connection Kit (2 in, 0 out)
  16 Firefox, JACK Audio Connection Kit (2 in, 0 out)
  17 Starship/Matisse HD Audio Controller Digital Stereo (IEC958), JACK Audio Connection Kit (0 in, 0 out)
  18 TU102 High Definition Audio Controller Digital Stereo (HDMI), JACK Audio Connection Kit (0 in, 0 out)

Thanks for the device list, this is very interesting!
I guess you are using some kind of Pipewire/PulseAudio/JACK plugin that provides the Firefox device?
Anyway, there is nothing that can be done from within the sounddevice module. It gets the device list from the underlying PortAudio library which in turn somehow gets it from the platform-specific back-ends.
There might be some way to get a virtual loopback device on macOS/CoreAudio. If such a thing exists and is running on your system, it would add itself to the list of devices and it should be visible in the sounddevice module.


Cortex Shuts down

Operating system: MACos
Cortex version: latest
I made a wrapper in python... see code:
python client.py
import sys
import websocket
import ssl
import time
import thread
ws = websocket.WebSocket(sslopt={"cert_reqs": ssl.CERT_NONE}) #edw einai auto pou sou elega me ton none cert
ws.connect("wss://emotivcortex.com:54321")
file1 = open('js_session.txt', 'r')
def client_func(sendquery):
ws.send(file1.read()) #apostolh pros emotivcortex.com
result1 = ws.recv()
for i in range(2):

    ws.send(sendquery)

    result = ws.recv()    #apanthsh apo emotivcortex.com
    with open("test.txt", "a") as myfile:
        myfile.write(result + "\n")
    print(result)
return

file = open('json.txt', 'r')
client_func(file.read())
which I call it from matlab inside a while true loop. However after 50-100 iterations (calls in websocket) the cortex shuts down.

Hi @ramsys28 ,
what is the request which you sent to Cortex ?

Hi,
I sent this through matlab
['{"jsonrpc": "2.0","method": "subscribe","params": {"_auth": "' json_auth.result.x_auth '","streams": ["pow"]},"id": 1}']);


python27: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

master -> staging for resholve, sorry...

done

@ofborg build xdg-utils

python2 builds fine on darwin, but some key reverse dependencies not...

pip fails to build because we have no hashlib because of
Warning - no suitable openssl found for hashlib - omitting..

I've pushed a fix to build pip again...


ci(workflows): adopt release workflows

Because

build and release wheels for sdk package

This commit

rename module from instill_sdk to instill
adopt release-please workflows to publish to both test and official instance of PyPI

Resolves INS-1691


Lightning

Add c-lightning support. Some notes:

using debian:buster docker image in order to run the updated buster_deps.sh script; alternatively could be prepared an ad-hoc docker image
add lightning-fetchbuild.sh script to download and compile c-lightning; I found some building issues compiling c-lightning inside fetchbuild.sh (after setting the environment)
add gettext python3 virtualenv on buster_deps.sh
add c-lightning header config file
add c-lightning patch

Closed draft PR to be included in https://github.com/greenaddress/bitcoin_ndk/pull/21


vimrc.o: file not recognized

Hi all,
This is supposed to be a compilation issue. I am using Cygwin to build gvim.exe with the following command:

make -f Make_cyg.mak SHELL=bash CC=i686-w64-mingw32-gcc CXX=i686-w64-mingw32-g++ RC=i686-w64-mingw32-windres ARCH=i686 CROSS_COMPILE=x86_64-w64-mingw32- USEDLL=no PYTHON=/cygdrive/c/Python27 PYTHON_VER=27 DYNAMIC_PYTHON=yes PERL=/cygdrive/c/htemp/Perl PERL_VER=516 DYNAMIC_PERL=yes GUI=yes OLE=yes

It ends up with the error:

gobji686/vimrc.o: file not recognized: File format not recognized
collect2: error: ld returned 1 exit status
make: *** [Make_cyg_ming.mak:829: gvim.exe] Error 1

Thanks!

CC=i686-w64-mingw32-gcc CXX=i686-w64-mingw32-g++ RC=i686-w64-mingw32-windres ARCH=i686 CROSS_COMPILE=x86_64-w64-mingw32-

You set CROSS_COMPILE wrongly. You should set CROSS_COMPILE=i686-w64-mingw32-, if you want to use MinGW-w64 for 32-bit target. If CROSS_COMPILE is properly set, normally you don't need to set CC, CXX and RC. (And, if you build Vim 7.4.2034 or later, ARCH will be also automatically detected.)


Overkill

Are you sure you want to go with such an overcomplicated system for a wiki?
It would seem to me like markdown files would suffice ..

This is just my favorite git theme, unless you have a different theme to suggest(which would be the reason why I am leaving this open), I am keeping this theme until I find one that looks more like a wiki.

Not a theme no, just not using a Jerkyl page in the first place, ..

Not a theme no, just not using a Jerkyl page in the first place, ..

I just wanted to use it, since it resembles a wiki.
Also, I had a look at your profile, welcome to the project!

Thanks? Ah either way, you should request to detach your fork.

I tried with the virtual assistant

It is not a fork since I used the theme as a template


Add Update instructions to Readme.md

The Readme.md does not specify how to apply updates. I'm assuming the proper procedure is to issue a git pull against the repo (if you didn't delete it) and then repeat the copy (cp) command. Wondering if there is a way to reload the software without doing a full reboot.

As I know full reboot is always required, but you can try to just reload the integration from integration page (but I don't know if this work).
Procedure described by you to apply updates it's correct, but you can simplify your life installing HACS and than installing all custom integration via HACS, so you will also receive notification when new version is available.


Knob aspect ratio is wrong for non-square knob graphics.

I've been playing with the framework and swapped the knob graphics.
The non-square graphics I used were not rendered with the correct aspect ratio which revealed two possible bugs:

in ConnectionKnob.cs
private float knobAspect { get { return knobTexture != null? knobTexture.width/ knobTexture.height : 1; } }
The line should be changed to:
private float knobAspect { get { return knobTexture != null? (float) knobTexture.width/ (float) knobTexture.height : 1; } }
The knobTexture.width and height need to be case to a float or the result otherwise the result is the result of dividing the two as integers (i.e. 48/30 would produce 1 not 1.6)

in ConnectionKnob.cs
In GetCanvasSpaceKnob, the calculation appears to be wrong.
Currently it is:
Vector2 knobSize = new Vector2 (NodeEditorGUI.knobSize * knobAspect,
NodeEditorGUI.knobSize / knobAspect );
This should be changed to:
Vector2 knobSize = new Vector2 (NodeEditorGUI.knobSize * knobAspect,
NodeEditorGUI.knobSize  );
(The "/knobAspect" removed)
The aspect ratio value calculated here is used to multiply the horizontal to scale it up to keep the ratio correct for the vertical (to make sure the vertical is no bigger than "knobSize"), or divide the vertical by the aspect ratio to make sure the horizontal is no bigger than "knobSize", there is no need to do both.

I've tried it and it appears correct, so thought I'd add it as an issue.
Cheers,
xyzDave

Sorry for the super late reply.
Have included both fixes.


serverSideTranslations not working with getServerSideProps on production

🐛 Bug Report
When I use serverSideTranslations and getServerSideProps on local, everything works fine. But, the t function doesn't work when I run my code on the production.
This is my page:
import Head from 'next/head';
import Link from 'next/link';
import { useTranslation } from 'next-i18next';
import { serverSideTranslations } from 'next-i18next/serverSideTranslations';
const Links = (props) => {
const { t } = useTranslation;
return (

Links

Links Page

{t('WELCOME')}

{data.map((item) => (

{item.label}

))}

);
};
export async function getServerSideProps({ locale }) {
return {
props: {
...(await serverSideTranslations(locale ?? 'en', ['common'])),
data: [
{ href: '/', label: 'Home' },
{ href: 'https://www.wikipedia.org/', label: 'Wikipedia' },
{ href: 'https://nextjs.org/', label: 'Next.js' },
],
},
};
}
export default Links;
this is next-i18next.config.js file:
`const path = require('path');
module.exports = {
i18n: {
defaultLocale: 'en',
locales: ['en', 'fr'],
},
reloadOnPrerender: process.env.NODE_ENV === 'development',
localePath: path.resolve(__dirname + './locales'),
};
`
I tried without "localePath" option, but no change.
If you have any ideas, I'll appreciate the help.
Thank you!

Please provide a complete but minimal reproducible example, not just code snippets.
And compare with this example: https://github.com/i18next/next-i18next/blob/d4aea294d70f7ff8645b2aa494f918299a1e84da/examples/simple/pages/second-page.tsx#L40
if it works locally but not when deployed, there might be an issue there...

Hey @adrai, thank you for the super fast reply.
I found the issue, it was the "localePath" prop in my next-i18next.config.js file. I didn't add the option when the window is "undefined", like in the example that you sent me, and that is why my SSR pages didn't work, on the production.
So instead of:
localePath:  require('path').resolve('./locales')`
now, I have:
  localePath:     typeof window === 'undefined'       ? require('path').resolve('./public/locales')       : '/locales',
Everything is working fine.
Thank you again, so much!

Hey @adrai, thank you for the super fast reply.
I found the issue, it was the "localePath" prop in my next-i18next.config.js file. I didn't add the option when the window is "undefined", like in the example that you sent me, and that is why my SSR pages didn't work, on the production.
So instead of: localePath: require('path').resolve('./locales')`
now, I have:
localePath: typeof window === 'undefined' ? require('path').resolve('./public/locales') : '/locales',
Everything is working fine. Thank you again, so much!

Life Saving Words
I have been suffering with this Issues For last 2 days Finally Fixed after localePath: typeof window === 'undefined' ? require('path').resolve('./public/locales') : '/locales', using this


Transfer gridsome-helper-json to gridsome organization

Summary
I made this package for Vetur's autocompletion.
https://github.com/tyankatsu0105/gridsome-helper-json
I suggest maintain this package under the Gridsome organization
Basic example
Motivation
Maybe this package should maintain by various maintainers from now on

I want to ask various people's opinions🤔

Yes, we should have a repository for it in this organization :) I'll create one for you tomorrow.

OK thanks !

I have added you as an admin for this repository now :)
https://github.com/gridsome/gridsome-helper-json
Anything else that should be done?

Awesome!
Thanks @hjvedvik
It will be fine 👌


error during running quickstart script start-collector.sh

[ERROR] Failed to execute goal on project pinpoint-quickstart-collector: Could not resolve dependencies for project com.navercorp.pinpoint:pinpoint-quickstart-collector:war:2.0.3-SNAPSHOT: Could not find artifact com.navercorp.pinpoint:pinpoint-collector:war:2.0.3-SNAPSHOT -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

Did you install All Java version from 6 to 9 ? I got this message when I only use Java 8.

[ERROR] Failed to execute goal on project pinpoint-quickstart-collector: Could not resolve dependencies for project com.navercorp.pinpoint:pinpoint-quickstart-collector:war:2.1.0-SNAPSHOT: Could not find artifact com.navercorp.pinpoint:pinpoint-collector:war:2.1.0-SNAPSHOT -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

./mvnw install -DskipTests=true -> build success
./quickstart/bin/start-collector.sh -> build failure

my environment setting
OS : OS X(macOS Catalina 10.15.6)
JAVA : ~/.bash_profile
export JAVA_HOME=/Library/Java/JavaVirtualMachines/adoptopenjdk-8.jdk/Contents/Home
export PATH=${PATH}:$JAVA_HOME
export JAVA_6_HOME=/Library/Java/JavaVirtualMachines/1.6.0.jdk/Contents/Home
export PATH=${PATH}:$JAVA_6_HOME
export JAVA_7_HOME=/Library/Java/JavaVirtualMachines/jdk1.7.0_80.jdk/Contents/Home
export PATH=${PATH}:$JAVA_7_HOME
export JAVA_8_HOME=/Library/Java/JavaVirtualMachines/adoptopenjdk-8.jdk/Contents/Home
export PATH=${PATH}:$JAVA_8_HOME
export JAVA_9_HOME=/Library/Java/JavaVirtualMachines/jdk-9.0.4.jdk/Contents/Home
export PATH=${PATH}:$JAVA_9_HOME

https://github.com/naver/pinpoint/issues/7022 Change Collector to Spring-boot
https://github.com/naver/pinpoint/issues/7034 Change Web to Spring-boot
In 2.1.0, web and collector are being converted to spring-boot.
So, the quick start does not work.
Can you help me fix the quick start?
Contributions are always welcome.

change quickstart.version 2.1.0-SNAPSHOT to 1.8.5-SNAPSHOT
The same problem occurs.
[ERROR] Failed to execute goal on project pinpoint-quickstart-collector: Could not resolve dependencies for project com.navercorp.pinpoint:pinpoint-quickstart-collector:war:1.8.5-SNAPSHOT: Could not find artifact com.navercorp.pinpoint:pinpoint-collector:war:1.8.5-SNAPSHOT -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException

hi @waterlove88
Quickstart could start after entire build of project.
You can get the guide via https://github.com/naver/pinpoint/blob/1.8.5/quickstart/README.md.
Please check this part again.
thanks :)

@emeroad @koo-taejin
Thank you for answer.
I solved it using docker.
However, there is an issue that the call-tree cannot be seen when using the peaksnail/pinpoint-node-agent.
Thank you.


Fix/environment variables in manifest

This change moves the environment variables in the environment section. Documentation is updated according to the evolution. See #158

Coverage remained the same at 94.268% when pulling 52fa028fed38f5b0eeaa07b9c0834610134e3437 on fix/environment_variables_in_manifest into 1cb4e05ffafe73e521e2b2a3835958c4bd5a1708 on develop.

Coverage remained the same at 94.268% when pulling 0737e9daf0026effc6d8c5eb06d7fe4b49a1dfc5 on fix/environment_variables_in_manifest into 3f160b478e754d762447f70d8b3eab9d86cdcbe3 on develop.


Failed to get port list?

Hi,
I am trying out the project today. The hardware is a Tofino 1 with 4 pipes 65 ports. It is installed with SONiC, and it requires the p4 program to be "switch". So I changed the p4tg.conf to use "switch" as the p4_program name, and also change the main.rs to look for "switch" instead of "traffic_gen". With above small hacks I am able to bring the controller up running.
When I am navigating the UI, and seems the ports/ and settings/ pages are not loading. The RUST server is throwing below:
[2024-02-28T00:25:44Z WARN  p4tg_controller::api::server] Error from REST API: PortNotFound { name: "64" }
So I am looking at the code, and couldn't find out what exactly the API is doing. Can you help with some ideas about what could be wrong here?
Thanks very much!
Regards,
Yichen

Hi Steffen,
Thanks for the prompt reply. So this chip (BFN-T10-064Q) has 65 * 100Gbps ports as below. When you say Port 64 I assume you mean "port id 64", and that port indeed is an internal port on our HW, i.e. not front panel.
I tried your patch, and it works!
Thanks a lot!!!
Regards,
Yichen

Great to hear. Added the initial port clearance in 9e301ca


reimplement noncommutative sort

the workaround for wycats/handlebars.js/issues/748 employed a sorting
routine that was not commutative and with the v8 upgrade in node11, the
order of the arguments changed (and is considered an implementation
detail that we're not meant to rely on) and so the routine would yield
surprising results
this patch reimplements that sorting routine to always yield the same
result regardless of the order of the operands, so should work on node <
11 and > 11
test plan: there is already a case for this, so I just added node12 to
the travis language matrix

 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.


Mesh Shaders

Description
Support mesh shaders
Solution proposal
Implement in GL and GLES using
https://www.khronos.org/registry/OpenGL/extensions/NV/NV_mesh_shader.txt
On VK implement using the KHR extension.
Additional context
Should be only begun when the VK backend is operational.

still waiting for the KHR extension for Vulkan


Provide Foundry v12 compatibility

Remove all outdated references to data
Use system for biography value

@mnoreke looks like this fixes it

I don't currently have Foundry setup on my machine as I have taken a break from gaming. But I'll try to take care of this before years end. A lot on my plate right now.


[MXNET-618] Clojure package - move NDArray and Symbol Code generation to Automatic

Description
From feedback from the Clojure package PR https://github.com/apache/incubator-mxnet/pull/11205, this changes the automatic generation of the ndarray and symbol code from a manual step to an automatic compile-time (AOT) step. The generated code is no longer checked into git as well.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
[x] Code is well-documented:
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

Move ndarray and symbol code generation to compile time
Remove generated code files from git
Remove the alias to generate-code from the project file since it is now automatic

Testing
To test this change:
with lein test

Make sure there are no files in the src/org/apache/clojure_mxnet/gen directory
Run lein test
Tests should run fine and the generated files should be present in src/org/apache/clojure_mxnet/gen but ignored by git

with jars

Make sure there are no files in the src/org/apache/clojure_mxnet/gen directory
Run lein install to create target jar
cd examples/imclassification
run lein run to run the example with the installed jar

Hey Carin, I can't review your PR on my phone due to the number of deleted files. Please excuse if this is already answered.
The two processes you described, are they already part of CI or how are we testing it? If something takes a long time, I'm happy to add it to nightly.

@marcoabreu - I put manual testings instructions there. The first part with lein test is what the CI does. The second part with jars is not covered by CI right now. It is creating the jars and then using them in another project. Eventually this is what I hope integration tests to cover.
For this PR, if CI tests are green, we are good to go.
If someone else wants to test out the with jars step manually, that would be great too.


Fix and centralize publishing logic

Closes #151.
Also currently our release was kind of broken: we published inferred-spans as a library, but didn't publish the common project it depends on.

Closing, branch is on wrong remote for testing.


ControlValueAccessor implementation for all builtin components

[ ] DatePicker
[ ] TimePicker
[ ] ListPicker
[ ] SearchBar
[ ] SegmentedBar
[ ] Slider (related #84)

Is there any plan for supporting of Sidedrawer in future?

@anhoev this was discussed here: https://github.com/NativeScript/nativescript-angular/issues/91
It will come in time.

Shipped in<EMAIL_ADDRESS>