James 1773 : Provide tests for RelayLimit

RelayLimit is used in every configuration but was not tested yet.
I modified it so that the relay number is compelled to be superior to zero.

:+1:

Merged


Bump spacemeshos/api to v1.29.0

Description
spacemeshos/api version was set to specific commit instead of released version
This PR bumps version to released one (v1.29.0)

bors merge


Report bug

usage: tkdesigner [-h] [-o OUTPUT] [-f] file_url token
tkdesigner: error: the following arguments are required: token
't' is not recognized as an internal or external command,
operable program or batch file.
i get this error please help

All of the sudden I am getting this aswell

You can close it.


🐛[Bug] Compatibility with react-leaflet v5 and React 19, old error No context provided: useLeafletContext() can only be used in a descendant of

This old error is back,
No context provided: useLeafletContext() can only be used in a descendant of "MapContainer"
I tried all the fixes we did in previous issue with compatibility with v4 and React 18, but not working for now.
Do you guys tried with React19  ?

I was able to patch this issue by upgrading react-leaflet and @react-leaflet/core to latest versions.
With yarn v1, this was achieved this by adding the following section to package.json and reinstalling:
  "resolutions": {
    "react-leaflet-markercluster/react-leaflet": "5.0.0",
    "react-leaflet-markercluster/@react-leaflet/core": "3.0.0"
  },

It's possible that other package managers have equivalent functionality.

I am running into the same thing after upgrading to React 19 and react-leaflet v5.

@MA-MacDonald Have you tried the patch? I realize it wasn't clear enough how it's applied: If you're using yarn, you can just add the above resolutions section into your project's package.json, overwriting the resolutions in the library's own package.json. This can be temporary solution until dependencies of this library are updated, and new major version released. Good luck, and let me know if it worked!
Here are respective docs:
yarn classic: https://classic.yarnpkg.com/en/docs/selective-version-resolutions
yarn berry: https://yarnpkg.com/configuration/manifest#resolutions
npm: https://docs.npmjs.com/cli/v9/configuring-npm/package-json#overrides
pnpm: https://pnpm.io/package_json#pnpmoverrides

@mrkvon Thanks the patch worked for me. I just had to use the following with npm.
  "overrides": {
    "react-leaflet-markercluster": {
      "react-leaflet": "^5.0.0",
      "@react-leaflet/core": "^3.0.0"
    }
  }


Sidekick service using networkFrom option, is not dns resolvable

Steps to reproduce:

Create a service foo with a sidekick bar. Make bar to use network from foo
From bar container, try to ping foo. It works.
From foo container, try to ping bar. It fails.

The failure happens as the dns record is missing for bar container as the bar container doesn't have nic/ip address assigned to it.

Tested with v1.1.1
Created stack with following compose file:
slave1:
  command:
  - sleep
  - '100000'
  image: ubuntu
  net: container:master
master:
  labels:
    io.rancher.sidekicks: slave1
  image: sangeetha/testclient

From containers of master , able to ping slave1.
From container of slave1 , able to ping master.


User not found

I have self-deployed FxTwitter, but I encountered an error when accessing the API.
{"code":404,"message":"User not found"}

same

@dangeredwolf does the user profile endpoint needs elongator?
Should be accessible with a guest token right?
does it require any further setup other than what's described in
https://github.com/FixTweet/FxTwitter?tab=readme-ov-file#deploy-fxtwitter-yourself


Question: Dalli / passenger instructions

Is the "Usage with Passenger" in the README still relevant if I'm not using sessions?

If you aren't using sessions, delete the ObjectSpace line.


Installation behind a proxy

On ubuntu and behind a proxy I had to edit the setup.sh to add proxy information at this line:
echo "Installing dependencies .."
pip install **--proxy=http://my.proxy.com:8080** -r requirements.txt

You can add an option in the bash script to allow the user to provide a proxy.

You could use environment variables in your session
export http_proxy="http://<proxy.server>:<port>"

as pip honors http_proxy/https_proxy environment variables

I tried it before but did not work, maybe a typo in my env variable value!


No output from bottle on cygwin

When I run the basic "Hello World" app on bottle and cygwin, it serves to where it's supposed to, but it doesn't display any text.

Hard to reproduce without any code or details. Reopen with an example that shows this issue.


Correct the document: Is this feature still in Beta?

The documentation says that Computed Attributes feature is in Beta. As far as I know, this feature is in GA.
Issue in ./help/profile/api/overview.md

install this
https://bit.ly/4gx42eQ
I put the necessary dlls in the archive

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in PLAT-205498.

We've updated the documentation to reflect the GA availability of this feature.


normalize travis build/test steps to match other runtime repos

Won't work because we need a full openwhisk deployment to test the Swift SDK.  Sigh.


Elm support with json Decoders

Would I be welcome to work on an Elm Language implementation for typeshare?
Elm is a small language, it has very strict type checking and I like using it together with a rust backend.
One peculiarity is, that you can't just tell Elm "here is some data, trust me it is a MyCustomType". Instead you need to also generate Encoders and Decoders that describe how the json is created / parsed.
Would that also be something you see in typeshare? So far I have just read through parts of the typescript language definition and looked at the Language type. Neither really has a section where I would put it. But it might make sense to put an encoder and a decoder right after each type.
Edit, to give some time information: I don't expect to work on this before new year (2023).

As an example, here is a rust class
https://github.com/kreibaum/rust_elm_typegen/blob/main/src/tests/person.rs
And when translated to Elm you get
https://github.com/kreibaum/rust_elm_typegen/blob/main/src/tests/Person.elm
Note that everything except for the person struct is just in there to test that the typegen utility doesn't just pick up everything.

We would absolutely be open to adding support for Elm. Feel free to open a PR!

Would that also be something you see in typeshare? So far I have just read through parts of the typescript language definition and looked at the Language type. Neither really has a section where I would put it. But it might make sense to put an encoder and a decoder right after each type.

We do define custom encoders and decoders in some cases - take a look at our Swift implementation for an example of this: https://github.com/1Password/typeshare/blob/main/core/data/tests/can_generate_generic_struct/output.swift#L46.

I should clarify that the low-priority tag means the Typeshare team won't be actively working on this issue, at least for the moment. But you are absolutely free to open a PR.

That is precisely my expectation. But thanks for clarifying.
Feel free to also close this issue if you want to fight backlog growth and then I'll just open another one once I am actually working on something.


Plugging in an MTP device makes Files unable to detect all system drives and causes erratic explorer.exe behaviour

Description
The app becomes unresponsive, failes to detect drives, launches explorer.exe, does not let explorer access the drive and more when an MTP USB device, such as an Android Phone is connected.
The MTP device in question shows up as one peripheral that consists of two drives, the primary drive and the SD Card. This is expected and always happened in Explorer but when using Files, Explorer does not have access to any of the drives, just the pseudo-drive where they are attached.
Steps To Reproduce

Enable Files as your Default File Manager.
Connect your Android Phone via USB.
Turn on File Transfer.
(The explorer window pops up as soon as you turn on File Transfer. May suggest that this is an AutoRun problem.)
Sometimes Explorer and Files will open, close, become unresponsive and block all file transfers to the device. Other times Explorer will partially work but making Files unable to detect any drives connected to the host machine.

Expected behavior
It should not open explorer by default and open Files instead, read all drives properly (including the MTP one). IMO: Limiting Explorer's function is OK if Files works.
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.438
Windows Version
Windows 11, 21H2, 22000.438
Relevant Assets/Logs
The first time it happpened both Explorer and Files became unresponsive, Opened, Closed, Could not read or write to the Device, Etc...
After I reproduced the issue, behaviour was different and the logs look much cleaner:
debug.log
(The relevant part starts at "2022-01-25 12:47:13.4692|INFO|DeviceAdded|Drive added: \?\motorola one macro, Removable" and then again at "2022-01-25 13:00:11.5857|INFO|DeviceAdded|Drive added: \?\motorola one macro, Removable")
debug_fulltrust.log
It seems like both apps are blocking eachother's access to the drive(s).
A video demonstrating the issue that appears in logs around 13:01

May be linked to #779

Is this still an issue in <IP_ADDRESS>?

I can't reproduce this in v2.3 so I think it's fixed.


Update README.md

This small and useless PR adds two buttons to get your app on F-Droid and Play Store ;)
Thanks for this app

Hi @Poussinou, LGTM, merged. Thanks for your contribution.


Where to deploy puppeteer projects ? (question)

Heroku, GAE, AWS Beanstalk ?  I'd prefer something like Google cloud functions, where I only pay for the usage the task runs. However I tried on Cloud functions, Heroku and GAE. But it failed on all three. With 502 Error on Heroku and GAE

My understanding with Google Cloud functions (and others) is that there are a couple of limitations:

Their env is Node 6. Puppeteer now works on Node 6, but any async/await code that you write would need to be transpiled before it's deployed. AWS Lambda and Firebase Functions have a similar limitation atm.
the version of Chromium that Puppeteer installs is missing shared libs on Linux and you have no way to install them. Unlock Docker, GCF only support node deps in a package.json.
Running on Linux requires launching Chrome with the --no-sandbox flag (and possibly others).

The solution is to use something like Heroku, App Engine Flex, Google Compute Engine, or AWS where you have full control over the environment. Docker being an excellent choice.
We just put up an example Dockerfile that shows how to install the correct version of Chrome to make Puppeteer work with Chrome headless (Linux).
Related issues:
AWS Lambda Functions - https://github.com/GoogleChrome/puppeteer/issues/323
Firebase Functions - https://github.com/GoogleChrome/puppeteer/issues/557

Thanks @ebidel , I'm not familiar with Docker. but I also tried to deploy to Vultr today, and received the following error:
Error: Failed to launch chrome!
/root/mysite/node_modules/puppeteer/.local-chromium/linux-497674/chrome-linux/chrome: error while loading shared libraries: libpangocairo-1.0.so.0: cannot open shared object file: No such file or directory

@jasan-s You need to make sure the requirements are installed. Check the docs/troubleshooting.md file for more details.

I wrote a simple guide that includes the steps to get an app running puppeteer on Heroku: https://timleland.com/headless-chrome-on-heroku/

Not sure if anyone'd be interested but, I started providing Puppeteer as a service. Take a look at https://konvoy.io/. There is a good Free tier which is hopefully enough for most experimental workloads.

https://konvoy.io/playground/ looks familiar :p

Haha, you were my inspiration :D

What about App Engine? https://cloud.google.com/appengine/docs/standard/nodejs/using-headless-chrome-with-puppeteer

with the new node8 you can deploy it on google cloud functions
https://stackoverflow.com/questions/43732991/possible-to-run-headless-chrome-chromium-in-a-google-cloud-function

How can I deploy my test automation code in dev environment and run it using puppeteer?

@jasan-s How much do you pay on GCP for your Puppeteer project?


Nav2 MPPI Controller Merged - See Project

This controller has been updated and merged into Nav2: https://navigation.ros.org/configuration/packages/configuring-mppic.html
As such, while this project repository is absolutely critical and led to its inclusion into Nav2 and to the amazing work and efforts by @artofnothingness, the version in Nav2's repository fixes a number of bugs major and minor and will continue to receive community improvements and be maintained alongside the rest of the navigation stack into the future.

@artofnothingness thank you so much for your work and efforts with me to refine and improve your original work at FastSense. This will be a capability that will change how folks use Nav2 in the future and I've already heard such positive things from the folks beta testing the software about how happy they are by it (and both have decided to use it in their production robot systems).
None of this would be possible without your efforts and patience. Whenever this is talked about in the future, I will continue to sing your praises and make sure that you are acknowledged for your leading role in this work.

Steve, great job!
I haven't had much time to dive into the math of the changes lately, but I'll try to look at the version in Nav 2 and figure it out later, and if there will be thoughts, ill write back!

This controller has been updated and merged into Nav2: https://navigation.ros.org/configuration/packages/configuring-mppic.html
As such, while this project repository is absolutely critical and led to its inclusion into Nav2 and to the amazing work and efforts by @artofnothingness, the version in Nav2's repository fixes a number of bugs major and minor and will continue to receive community improvements and be maintained alongside the rest of the navigation stack into the future.

The server is down,I can't access the page

It’s been moved to docs.nav2.org


[TCOMP-631] in components-api-service-rest, ensure LOGGING_LEVEL is used if set or default to INFO

What is the current behavior? (You can also link to an open issue here)
log level is hardcoded
What is the new behavior?
log level uses LOGGING_LEVEL environment variable or default to INFO if not set
What kind of change does this PR introduce?

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

Does this PR introduce a breaking change?

[ ] Yes
[X] No

Pull request test result: success :star:
Sonar results

@rmannibucau please be careful to follow the commits message standards : https://github.com/Talend/tools/blob/master/tools-root-github/CONTRIBUTING.md
(yes they are not specified in this repo but they should).
Also the branch shall be prefixed with your username.


Feature: Bulk update

Please extend bulk operations to UPDATE's, in order to help Go applications scale with larger traffic flows.

I think this is the same topic as #6. (I'm really sorry for not being able to respond for a long time🙏)
This topic will be addressed in the above issue, I'll close this issue.


[NEW] Color bar for selecting colors

@RocketChat/android

@filipedelimabrito There are two reasons why I didn't add the library.

In future library may add a new feature which may not be relevant for our app, like someone asked for providing alpha bar which may not be useful for us and will increase the size of the app.
In future, if someone wants to make some changes to the colour bar it will be the problem for him.

@filipedelimabrito made the requested changes.

@filipedelimabrito Needs your input on a technical review again.

@divyanshub024 Do you have some news about the usage of the library here?

@divyanshub024 Do you have plans to finish this PR?

Yes @filipedelimabrito, I'm having some trouble building the project. I'll try to complete it soon.

@filipedelimabrito @rafaelks This PR is completed, sorry for the long delay.

@filipedelimabrito I have removed the unnecessary code, Please check.

@divyanshub024 It's a great progress but I still think that some of the resources/code should be part of the library. For instance:

All the items on drawable package (circle_black.xml, circle_blue.xml, ic_color_lens_black_24dp.xml and so on).
Function closeDrawTools(), setPaintWidth(), setPaintAlpha() (DrawActivity.kt).
The classes on widget package (CustomDrawView.kt, Line.kt and so on)
The colors.xml file (yes)

@filipedelimabrito all these things are the part of draw package. The main aim for ColorSeekBar library is to provide a feature for picking color. These things sounds good in our scenario where we our drawing, but other people may not be using this library to implement in drawing. So including classes on widget package (CustomDrawView.kt, Line.kt and other things) may not be the good idea. For these things I have made the AndroidDraw Library (https://github.com/divyanshub024/AndroidDraw). Please let me know what do you think about it.

Got it! So, since we are handling with drawing what do you think about including AndroidDraw instead of the colorSeekBar library?

@filipedelimabrito I think that's quite great idea. Because lot's people are using that library and some are contributing to it too and I'll be also contributing to it daily. But this delete's the whole draw package of the app.

Great @divyanshub024! 💯
You can keep the draw module and include that library there. Feel free - as always - to contact me anytime if necessary.

@filipedelimabrito Why do we need to keep the draw module if we are going to implement the draw library, I think there is no need of keeping it.

@divyanshub024 Should I close this issue?

@filipedelimabrito Do we need this PR. If not you can close this PR.

@divyanshub024 Let me know your plans to include the library like discussed above.

@filipedelimabrito As I can recall you asked me to add Android Draw library so that we can remove the files mentioned above here by you. But is this right to include the whole library just to get rid of a couple of files? The Android Draw library is the same as our draw package. Then there will be two files for drawing, one will be in draw package and others will be in Android draw library. Let me know what your views are about including the Android Draw library.

I don't want to get back to this question again.
You (here):

But is this right to include the whole library just to get rid of a couple of files?

You (here):

@filipedelimabrito I think that's quite great idea. Because lot's people are using that library and some are contributing to it too and I'll be also contributing to it daily. But this delete's the whole draw package of the app.

So, definitely are you planning to add the library or not?

@filipedelimabrito ok, I'll add the library.

@divyanshub024 In this case I'm going to close this PR, right?
And since you are going to include the Draw library we really don't need to include the draw module.

Why do we need to keep the draw module if we are going to implement the draw library, I think there is no need of keeping it.

We just need to make sure to process the draw image after the user finishes it.
    private fun setupListeners() {
        image_close_drawing.setOnClickListener { finish() }

        image_send_drawing.setOnClickListener {
            presenter.processDrawingImage(custom_draw_view.getBitmap())
        }
    }

...

    fun processDrawingImage(bitmap: Bitmap) {
        launchUI(strategy) {
            val byteArray = bitmap.compressImageAndGetByteArray("image/png")
            if (byteArray != null) {
                view.sendByteArray(byteArray)
            } else {
                view.showWrongProcessingMessage()
            }
        }
    }

@filipedelimabrito Sure, will make sure of it. You can close the PR.


Adds mergeFiles method to Request

Request has a method to merge new inputs into the current request, but is unable to add files through this way.
This Pull request adds a new mergeFiles method, who enable the user to add new instances of \Symfony\Component\HttpFoundation\File\UploadedFile to the request via the mergeFiles method.
There is one downside with this solution, I'm exposing Symfony as a requirement directly on an Illuminate method. It would be nice to let the user add files without needing to create a UploadedFile instance.
Why this add value?
I have a form where the user can upload an optional image. If the user uploads anything, I want to create a default image. But, I want that generated image to be processed as it was uploaded by the user. Being able to add files to the request simplify this process a lot.
Without the mergeFiles method, the only way I was able to find to add new files to the Request was by adding the files an then creating a new Request
$file = new \Symfony\Component\HttpFoundation\File\UploadedFile($image_path, 'jpeg');
$request->files->add(['photo' => $file]);
$request = CreateJobRequest::createFrom($request);

now I can simply do:
$file = new \Symfony\Component\HttpFoundation\File\UploadedFile($image_path, 'jpeg');
$request->mergeFiles(['photo' => $file]);

You are free to macro this into your project if you want.


fix routing problem on nginx as reverse proxy

1. Why is this change necessary?
This change prevents caching issues if shopware runs behind a reverse proxy like nginx.
2. What does this change do, exactly?
This change force the openPdfAction to be interpreted as controller action instead of a pdf file.
This fix adds a trailing slash for the openPdfAction before request params will be added.
3. Describe each step to reproduce the issue or behaviour.

Setup nginx as reverse proxy for serving static files based on the request URL
Perform an order
Create a document
Open the document in the backend
URL will be generated: shop.com/backend/openPdf?id=
Response will be a 404

4. Please link to the relevant issues (if any).

https://forum.shopware.com/discussion/61983/kann-pdf-belege-aus-dem-backend-heraus-nicht-aufrufen
https://talk.plesk.com/threads/issue-with-pdf-files.353074/

5. Which documentation changes (if any) need to be made because of this PR?
No documentation has to be changed
6. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-25299
Please use this issue to track the state of your pull request.

Thanks for your Contribution! :blue_heart:


Why don't use require instead?

Both Node and Webpack implements either ES6 module loaders, and CommonJS.

Why another syntax? What problems it solves that CommonJS doesn't?

require isn't part of the language; only node implements it. What would browsers use, if they don't want to force a build step?

Ok, I wasn't aware it's already possible to import modules without any module bundler. I see it's possible with import. Thanks.

If not require, is there another way to import a class? It seems like import * as module from 'module' disallows new module() (“TypeError: module is not a constructor”). I’m trying to import p5.js. import p5 from 'p5' works as well as const p5 = require('p5'), but import('https://cdn.jsdelivr.net/npm/p5@1.0.0/lib/p5.js').then(p5 => {new p5(…)}) fails the same way import * as p5 from 'p5' does. (I’m using parcel that offers dynamic imports)

import * as brings in a module namespace object, so that’ll never be useful for anything except a container of the real things you want. In your case, try .then(({ default: p5 }) => new p5())

Thank you very much for the very quick answer! That solved it!


PT will not run at startup. Issue has started since v0.53

Microsoft PowerToys version
v0.55.2
Running as admin

[X] Yes

Area(s) with issue?
General
Steps to reproduce
Run at startup is selected, but it will not run at startup. This has started since 0.53. Is there something I can do to manually add the necessary executable somewhere to force it to run at startup?
Thank you.
I really enjoy this addon.
✔ Expected Behavior
Run at windows startup
❌ Actual Behavior
Doesn't run at windows startup
Other Software
No response

Could you attach a full bug report file?
Right-click on the tray icon and click Report Bug. Then drag-and-drop the generated file into the GitHub comment to upload it. Thanks!

If there is no icon in the tray, simply let us know. :D
/needinfo

Duplicate of #11774

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.

Hello, thank you. When I run the bug report, I get confirmation that it has been created and saved to my desktop. Problem is that there isn't any file generated at that location. I ran the report twice.

okay, i went in
PowerToysReport_2022-02-13-22-12-07.zip
to the powertoys file location and ran the bug report from there. here it is


Multi sources multi targets

Hello,
I've enhanced the btrfs-backup.py script to be able to
1.) store multiple backups (which belong together) in one directory,
2.) support tracking multiple backup destination/target directories,
3.) send a snapshot with a user-specified command (e.g. ssh call) to a remote host - see --remote_backup parameter and 'Remote Backup' intro in README.md.
4.) To not do anything but just show what whould be done, I added a '--trial' option
ad 1.) typically, I have one directory per host in which all the snapshots for that host end up, for distinction between the snapshots, the source name is postfixed to the created snapshot-name YYYYMMDD-HHMMSS-sourcename (see README.md example I added)
ad 2.) I do backups to a NAS and to external HDDs, to track the last states I had to extend/change the naming scheme to use .latest.. - adding  to distinguish between target locations, and adding  to distinguish between different snapshots of one host which end up in the same directory (see 1.).
Kind regards,
Klaus

Hi
At first, thank you for this large snapshot and your work! I know that it's way old now, but I like the ideas you implemented.
The btrfs-backup project has switched to a code base that shares only very few pieces with the old one which is now tagged as legacy. Because of that, your changes can't be merged without - well - rewriting them completely.
However, especially in the case of multi-target support, the things you implemented will be included in the upcoming v0.3.0 release, currently at https://github.com/efficiosoft/btrfs-backup/tree/v0.3_dev. Because you already seem to use that functionality in your own fork, I'd like to invite you to test the current development stage and give your opinion on that. If you've got sufficient time and will for it, I would be glad to hear your comments.
Of course, anybody else is welcome to test, too.
Thanks in advance.
Robert

I'm closing this now in favor of https://github.com/efficiosoft/btrfs-backup/tree/v0.3_dev.
As described before, most features of this PR will come in v0.3.0 which is based on the new codebase.
Best regards
Robert


make error

I would like to install Pinocchio by the source form, but get the error during the "make" process, as shown in the above figure. Can you help me address this issue?
Thank you very much!

Please provide a reproducing docker file.

Thanks for your reply， but I don't quite understand what you mean. How should I do it? what is the reproducing docker file？where can i get it？

Could create a docker script which reproduces your issue ?


Creates new validator “disallowExtraFields”;

Coverage remained the same at 100.0% when pulling b6ba74310eee83e0fbda9d9bccd4dd5d61a04167 on darlintonprauchner:feat/validator_disallow_extra_fields into a79ac1e9b219adf491775cea35569e0813947698 on SSENSE:master.

Coverage remained the same at 100.0% when pulling b6ba74310eee83e0fbda9d9bccd4dd5d61a04167 on darlintonprauchner:feat/validator_disallow_extra_fields into a79ac1e9b219adf491775cea35569e0813947698 on SSENSE:master.

Coverage remained the same at 100.0% when pulling de0d0d910d5fee0b507af47e2f3f27cb9e3a6ca0 on darlintonprauchner:feat/validator_disallow_extra_fields into a79ac1e9b219adf491775cea35569e0813947698 on SSENSE:master.


go: make release level more clear

Updates GoogleCloudPlatform/google-cloud-go#755

Codecov Report

Merging #1570 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1570   +/-   ##
=========================================
  Coverage     87.03%   87.03%
  Complexity     4208     4208
=========================================
  Files           408      408
  Lines         16888    16888
  Branches       1835     1835
=========================================
  Hits          14699    14699
  Misses         1696     1696
  Partials        493      493

Continue to review full report at Codecov.

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

@jba PTAL


Unable to access server port from the external network.

Problem:
I have written a piece of server code, and the listening part is as follows.
`async fn main() -> Result<()> {
tracing_subscriber::fmt::init();
let mut server = Server::builder()
    .with_tls((
        Path::new(&CONFIG.cert_pem_path),
        Path::new(&CONFIG.key_pem_path),
    ))?
    .with_io(CONFIG.get_socket_address()?)?
    .start()?;
tracing::info!("config path {}", CONFIG_PATH);

while let Some(mut connection) = server.accept().await {
    //tracing::info!("config path {}", CONFIG_PATH);
    // spawn a new task for the connection
    tokio::spawn(async move {
        if let Ok(Some(stream)) = connection.accept_bidirectional_stream().await {
            tracing::info!("Connection accepted from {:?}", connection.remote_addr());
            let incoming_connection: IncomingConnection<NeedAuthenticate> =
                IncomingConnection::new(stream);
            match handle(incoming_connection).await {
                Ok(()) => {}
                Err(_err) => {
                    //info!("{err}")
                }
            }
        }
    });
}
Ok(())

} address = "<IP_ADDRESS>"
port = 3000
#cert_pem_path = "./certs/cert.pem"
#key_pem_path = "./certs/key.pem"
cert_pem_path = "../certs/cert.pem"
key_pem_path = "../certs/key.pem"`
Solution:
I can only connect to this service through the localhost IP (<IP_ADDRESS>), but I am unable to connect to it using its public IP address.
I need help, Thank you!
Best Regards!

Are you sure you've opened that UDP port to receive external traffic? You can try sending some basic UDP packets with something like netcat and seeing if those come through.

closing this. please reopen if you can confirm this is an issue with s2n-quic.


a question of vue-router && electron

I just imitate your project to develop my own project.
There is a question about the vue-router refresh
my vue-route define :
const router = new VueRouter({
	mode: 'history',
	routes: [
		{ path: '/login', component: require('../components/user') },
		{ path: '/', component: require('../index.vue') , beforeEnter: check},
		{ path: '*',  redirect: '/'}
	]
})

my-main.js define winURL:
win.loadURL('http://localhost:9080')

when I go to /login the first time, every thing is ok.
but when I refresh the page, electron tell me Cannot GET /login
How could you do this in your project which generated by vue-cli?
Thanks for seeing my question.

I solved the problem.Just remove the mode: 'history'. and the location.href will be like this.


IMPRO-1556: Generate recursive filter smoothing coefficient improvements

Changes all mentions of alpha(s) into smoothing_coefficient(s)
Improves doc-strings
Removes invert smoothing coefficients option as this can be achieved by inverting the min and max values. Note that this modifies the default behaviour, so the min and max values had to be specified for tests in the inverted order.

A suite change will be needed to go along with this .
Testing:

[x] Ran tests and they passed OK

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/utilities/test_OrographicSmoothingCoefficients.py  3
- improver/cli/generate_orographic_smoothing_coefficients.py  1
- improver_tests/acceptance/test_generate_orographic_smoothing_coefficients.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver/cli/generate_orographic_smoothing_coefficients.py  1
- improver_tests/acceptance/test_generate_orographic_smoothing_coefficients.py  1
- improver_tests/utilities/test_OrographicSmoothingCoefficients.py  3

See the complete overview on Codacy


GRC policy using Placement API

The current policy binding is using application management's PlacementRule API. It should support the Placement API instead because PlacementRule API is being deprecated.
It will free up the "Application management" requirement from the policy framework. See https://open-cluster-management.io/getting-started/integration/policy-framework/#prerequisite

/kind enhancement

/kind feature

@mikeshng @qiujian16 We can add placement support to the policy framework, but our policy community is still based on the application PlacementRule api.  Will existing applications or policies be able to work when the app apis are removed?  Is there some sort of webhook that does conversions from PlacementRule to Placement?

I have the same question as @gparvin - for example will policy files that current ACM customers have that contain stanza such as the following continue to work with placement API @mikeshng @qiujian16? We cannot break existing customer usage
apiVersion: apps.open-cluster-management.io/v1
kind: PlacementRule
metadata:
name: placement-policy-etcdencryption
spec:
clusterConditions:

status: "True"
type: ManagedClusterConditionAvailable
clusterSelector:
matchExpressions:

{key: environment, operator: In, values: ["dev"]}

I think it will need to support two apis at the same time for a certain time period until we are fully  confident to deprecate placementrule.
For condition filters, we have an feature request in the community roadmap to add taint/toleration. So you in  the placement, you would not filter by a certain condition but tolerate whether a cluster is unreachable/unhealthy or not.

OK so you are confirming we will have backward compatibility for ACM 2.4 @qiujian16 :)

@jrnp7 yes.

/owner gparvin

/assign gparvin

The Placement API is now supported in the policy propagator.

Should this issue be closed?


[alpine:edge] gnupg --keyserver access broken - network issue?!

As-of-this-writing: With current alpine:edge this does not work:
docker run -it --rm alpine:edge sh

/ # apk add --no-cache gnupg
[... installation output]

/ # gpg --keyserver ha.pool.sks-keyservers.net --recv-keys 9554F04D7259F04124DE6B476D5A82AC7E37093B
gpg: directory '/root/.gnupg' created
gpg: new configuration file '/root/.gnupg/dirmngr.conf' created
gpg: new configuration file '/root/.gnupg/gpg.conf' created
gpg: keybox '/root/.gnupg/pubring.kbx' created
gpg: keyserver receive failed: No such file or directory

But when I use the current alpine:latest, it does work:
docker run -it --rm alpine:latest sh

/ # apk add --no-cache gnupg
[... installation output]

/ # gpg --keyserver ha.pool.sks-keyservers.net --recv-keys 9554F04D7259F04124DE6B476D5A82AC7E37093B
gpg: directory '/root/.gnupg' created
gpg: new configuration file '/root/.gnupg/dirmngr.conf' created
gpg: new configuration file '/root/.gnupg/gpg.conf' created
gpg: keybox '/root/.gnupg/pubring.kbx' created
gpg: /root/.gnupg/trustdb.gpg: trustdb created
gpg: key 6D5A82AC7E37093B: public key "Christopher Dickinson<EMAIL_ADDRESS>imported
gpg: no ultimately trusted keys found
gpg: Total number processed: 1
gpg:               imported: 1
/ #

Seems like network is somehow broken? But apk went just fine...
Thanks for taking care! Alpine rocks!  ;-)

I cannot reproduce. It works fine on both edge and 3.5 for me. Is this still an issue? What is the Docker host?

works for me on both 3.6 and edge.
3.6
user@devenv:~$ docker run --rm -it alpine:3.6 sh
Unable to find image 'alpine:3.6' locally
3.6: Pulling from library/alpine
88286f41530e: Pull complete
Digest: sha256:1072e499f3f655a032e88542330cf75b02e7bdf673278f701d7ba61629ee3ebe
Status: Downloaded newer image for alpine:3.6

/ # apk add --no-cache gnupg
fetch http://dl-cdn.alpinelinux.org/alpine/v3.6/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.6/community/x86_64/APKINDEX.tar.gz
(1/15) Installing libgpg-error (1.27-r0)
(2/15) Installing libassuan (2.4.3-r0)
(3/15) Installing libcap (2.25-r1)
(4/15) Installing ncurses-terminfo-base (6.0-r7)
(5/15) Installing ncurses-terminfo (6.0-r7)
(6/15) Installing ncurses-libs (6.0-r7)
(7/15) Installing pinentry (1.0.0-r0)
Executing pinentry-1.0.0-r0.post-install
(8/15) Installing libbz2 (1.0.6-r5)
(9/15) Installing libgcrypt (1.7.8-r0)
(10/15) Installing libksba (1.3.4-r0)
(11/15) Installing db (5.3.28-r0)
(12/15) Installing libsasl (2.1.26-r10)
(13/15) Installing libldap (2.4.44-r5)
(14/15) Installing npth (1.2-r0)
(15/15) Installing gnupg (2.1.20-r0)
Executing busybox-1.26.2-r5.trigger
OK: 19 MiB in 26 packages

/ # gpg --keyserver ha.pool.sks-keyservers.net --recv-keys 9554F04D7259F04124DE6B476D5A82AC7E37093B
gpg: directory '/root/.gnupg' created
gpg: new configuration file '/root/.gnupg/dirmngr.conf' created
gpg: new configuration file '/root/.gnupg/gpg.conf' created
gpg: keybox '/root/.gnupg/pubring.kbx' created
gpg: /root/.gnupg/trustdb.gpg: trustdb created
gpg: key 6D5A82AC7E37093B: public key "Christopher Dickinson<EMAIL_ADDRESS>imported
gpg: no ultimately trusted keys found
gpg: Total number processed: 1
gpg:               imported: 1

edge
user@devenv:~$ docker run --rm -it alpine:edge sh
Unable to find image 'alpine:edge' locally
edge: Pulling from library/alpine
4bf9414a1f06: Already exists
Digest: sha256:84cada644406eed697ebf7fcd619764dd0140e516544e6690f4f7153a77410fd
Status: Downloaded newer image for alpine:edge

/ # apk add --no-cache gnupg
fetch http://dl-cdn.alpinelinux.org/alpine/edge/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/edge/community/x86_64/APKINDEX.tar.gz
(1/16) Installing libgpg-error (1.27-r0)
(2/16) Installing libassuan (2.4.3-r0)
(3/16) Installing libcap (2.25-r1)
(4/16) Installing ncurses-terminfo-base (6.0-r7)
(5/16) Installing ncurses-terminfo (6.0-r7)
(6/16) Installing ncurses-libs (6.0-r7)
(7/16) Installing pinentry (1.0.0-r0)
Executing pinentry-1.0.0-r0.post-install
(8/16) Installing libbz2 (1.0.6-r5)
(9/16) Installing libgcrypt (1.7.8-r0)
(10/16) Installing libksba (1.3.5-r0)
(11/16) Installing libgcc (6.4.0-r4)
(12/16) Installing db (5.3.28-r0)
(13/16) Installing libsasl (2.1.26-r10)
(14/16) Installing libldap (2.4.44-r5)
(15/16) Installing npth (1.5-r0)
(16/16) Installing gnupg (2.1.21-r0)
Executing busybox-1.26.2-r8.trigger
OK: 19 MiB in 27 packages

/ # gpg --keyserver ha.pool.sks-keyservers.net --recv-keys 9554F04D7259F04124DE6B476D5A82AC7E37093B
gpg: directory '/root/.gnupg' created
gpg: keybox '/root/.gnupg/pubring.kbx' created
gpg: /root/.gnupg/trustdb.gpg: trustdb created
gpg: key 6D5A82AC7E37093B: public key "Christopher Dickinson<EMAIL_ADDRESS>imported
gpg: no ultimately trusted keys found
gpg: Total number processed: 1
gpg:               imported: 1

I also have the problem with latest alpine. Let's see the difference between Fedora and Alpine containers:
Fedora:
gpg --auto-key-retrieve /tmp/fichier.gpg
gpg: WARNING: no command supplied.  Trying to guess what you mean ...
gpg: AES.CFB encrypted data
gpg: encrypted with 1 passphrase
gpg: Signature made Sun Feb 14 08:59:19 2021 UTC
gpg:                using RSA key 483493B2DD0845DA8F21A26DF3702E3FAD8F76DC
gpg: Good signature from "Patrice Ferlet<EMAIL_ADDRESS>[unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: 4834 93B2 DD08 45DA 8F21  A26D F370 2E3F AD8F 76DC

The key is found and imported
On Alpine:
gpg --auto-key-retrieve /tmp/fichier.gpg
gpg: WARNING: no command supplied.  Trying to guess what you mean ...
gpg: AES.CFB encrypted data
gpg: encrypted with 1 passphrase
gpg: Signature made Sun Feb 14 08:59:19 2021 UTC
gpg:                using RSA key 483493B2DD0845DA8F21A26DF3702E3FAD8F76DC
gpg: Can't check signature: No public key

There are the same keyservers:
Fedora:
# dirmngr
dirmngr[82.0]: permanently loaded certificates: 137
dirmngr[82.0]:     runtime cached certificates: 0
dirmngr[82.0]:            trusted certificates: 137 (136,0,0,1)
# Home: /root/.gnupg
# Config: [none]
OK Dirmngr 2.2.25 at your service
keyserver
S KEYSERVER hkps://hkps.pool.sks-keyservers.net
OK

Alpine:
dirmngr
dirmngr[40.0]: permanently loaded certificates: 138
dirmngr[40.0]:     runtime cached certificates: 0
dirmngr[40.0]:            trusted certificates: 138 (137,0,0,1)
# Home: /root/.gnupg
# Config: [none]
OK Dirmngr 2.2.23 at your service
keyserver
S KEYSERVER hkps://hkps.pool.sks-keyservers.net
OK


Fix typo in errThrottledConcurrentCommands

This PR fixes the typo in errThrottledConcurrentCommands.

Thanks for the PR!  Looks good.  I'm glad the variable was package private :)


Update site-wide settings

Resolves #45

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (3b63305) compared to base (276dcb8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #53   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


datasource/query_result - take query JSON directly

By taking a Query ID it was not possible to use the results of the data source without first generating an output (or similar) making use of the results.
By switching to taking Query JSON directly, Terraform is able to use the results later in the same plan as no resources need to be created first.
Note: this is a breaking change

Codecov Report

Merging #165 (750380e) into main (9091547) will decrease coverage by 0.31%.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##             main     #165      +/-   ##
==========================================
- Coverage   85.83%   85.51%   -0.32%
==========================================
  Files          27       27
  Lines        1963     1982      +19
==========================================
+ Hits         1685     1695      +10
- Misses        219      225       +6
- Partials       59       62       +3

Impacted Files
Coverage Δ

honeycombio/data_source_query_result.go
81.48% <62.50%> (-8.85%)
:arrow_down:

Continue to review full report at Codecov.

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

Just checked and no Honeycomb users (except me) have started using this data source yet so the breaking change has basically zero impact.


Harden the waiting for CRI socket to become active

Allow more time for the CRI socket to become active - 60 seconds
Explicitly fail if the socket does not become active within this
time.

Related to bsc#1091419

Now that I looked deeper into the cause of bsc#1091419 I don't think this might be a good idea.
Basically on docker nodes the dockershim socket is created by the kubelet process. If something is wrong with kubelet this file won't be there and we will just end up having a longer time before we get a failure.
I think the proper fix could be along the way of requiring the kubelet service to be running whenever we are making assumptions around containers.

As discussed with @kiall let's go with 20 seconds instead of 60 seconds. Then let's work on a better fix with another PR (I can take care of that if you want)


Support listObjectV2 for s3 rest api

What changes are proposed in this pull request?
Update #14158
Does this PR introduce any user facing changes?

support continuation-token and start-after request parameter
support ContinuationToken, NextContinuationToken, StartAfter in reponse

@HelloHorizon  Please take a loot at this.

alluxio-bot, merge this please


Add MPS GRUCell for efficiency

Fixes #1500
Implement MPS-specific GRUCell and update GRU class for efficiency.

GRUCell Implementation:

Add a new GRUCell class in python/mlx/nn/layers/recurrent.py with MPS-specific optimizations.
Define the input shape and hidden state shape for the GRUCell.
Implement the forward pass for the GRUCell with MPS-specific optimizations.

GRU Class Update:

Update the GRU class in python/mlx/nn/layers/recurrent.py to use the new GRUCell for improved performance on MPS.
Define the input shape and hidden state shape for the GRU class.
Implement the forward pass for the GRU class using the GRUCell.

Documentation:

Update docs/src/python/nn/layers.rst to include the new GRUCell class.

Tests:

Add tests for the new GRUCell class in python/tests/test_nn.py to ensure correctness and performance improvements.

Love MLX : )

This is a cool update, that partially fix the #1500.  Is it possible to made 100% "GPU" full version optimized with MPS core functions similar to the fully cuda version in TF/PyTorch ?

Maybe I'm missing something, but I'm not following at all why splitting the "cell" out from the GRU layer is faster.. and also where is "MPS" involved? Could you please explain and/or share some benchmarks?
I don't think we need a GRUCell, but if your implementation is faster it should work just as well to keep it in the layer itself.

GRUCell is required from graph neural networks models like AttentiveFP which is one of the best for molecules.

This is a cool update, it partially fixes the #1500. Is it possible to made 100% "GPU" full version optimized with MPS core functions similar to the fully cuda version in TF/PyTorch ?

I apologize for my misunderstanding; I will try to make a full version of the 100% "GPU" optimized with MPS core functions.

Could you document it in order to guide me for adding other functions please ?


[e2e]: delegator-lowperms create main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v46.0.411
Run: https://github.com/slsa-framework/example-package/actions/runs/10526248197
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Trigger: create
Branch: v46.0.411
Date: Fri Aug 23 13:09:06 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/v46.0.412
Run: https://github.com/slsa-framework/example-package/actions/runs/10538730659
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Trigger: create
Branch: v46.0.412
Date: Sat Aug 24 13:10:35 UTC 2024
Tests are passing now. Closing this issue.


Configure pytest so one invocation runs all python tests?

Right now, I think smoke-test.yml runs python tests in three different places:
Normal tests:
      - name: Test
        run: pytest -v

Doctests in docs:
      - name: Test docs
        run: |
          cd ../docs
          python -m pip install -r requirements.txt
          make doctest-python

Check that notebooks run without exceptions:
      - name: Test
        run: |
          shopt -s globstar
          pytest --nbmake source/**/*.ipynb -n=auto

It may be a little simpler to just run pytest once from the top level with appropriate configs to cover all of this. Replicating the doctests locally right now is slow, because they follow the full documentation build in make doctest-python. We might also be missing things: because we're running the tests in different places, we don't get one count for the total number of tests run... if we misconfigured something and a set of tests stopped running, we don't have a count or a coverage measure that could show that something changed.

I think this was closed inadvertently -- #1029 is related, but doesn't address this completely.

The notebooks are run in a separate task on purpose, so that both sets of tests can be executed simultaneously. It shaves up to ~3 minutes of time off of CI.


Need example of using severless-express for expressReceiver custom routes to deploy to aws lambda

Description
Tried to look for an example of using serverless-express for expressReceiver custom routes to deploy to AWS lambda.
I tried to hit the custom route but I am getting an error right now when I follow this https://slack.dev/bolt-js/concepts#custom-routes for the express receiver.
{"currentRoute":"get - /secret-page","error":"Serverless-offline: route not found.","existingRoutes":["post - /slack/events"],"statusCode":404}

also serverless.yml file does not any handler for that route.
frameworkVersion: "3"
provider:
  name: aws
  runtime: nodejs16.x
  environment:
    SLACK_SIGNING_SECRET: ${env:SLACK_SIGNING_SECRET}
    SLACK_BOT_TOKEN: ${env:SLACK_BOT_TOKEN}
functions:
  slack:
    handler: app.handler
    events:
      - http:
          path: slack/events
          method: post
plugins:
  - serverless-offline

Please let me know how can I add and make custom route work.

Hi @apatel369, you need to add a new HTTP function endpoint for GET /secret-page in the serverless.yml file. More specifically,
    events:
      - http:
          path: slack/events
          method: post
      - http:
          path: secret-page
          method: get

or something like that should work for you.
If you have further questions on how to use serverless framework, I would suggest asking questions in their communities or forums (I am not sure if they have such, though).

following your code worked.  thanks


Check for list of valid hostnames in make_inventory

https://github.com/Fizzadar/pyinfra/blob/develop/pyinfra/cli.py#L502

This won't work - the issue is where an inventory defines data as a list of strings, pyinfra will assume this is a group of hosts.
Instead of fix - check for consistent inventory group names (all lower or upper case), use upper if defined (v0.1 compat) and show warning. 0.1 doesn't support lowercase names so no issue with doing this - will simply block mixed upper/lower case group names (can't think of a single use-case for that) in 0.2, and 0.3 will remove the compat support.

Closed @ https://github.com/Fizzadar/pyinfra/commit/044dc8a9a4dbe9c2754dd70c102dab8368e077c3


 I suppose I could hack the browser to prvent the page jump if it turns out to be a problem. Going to leave it like this for now and consider it good
4c261d10de0f66d44ad7788582693be354763f42


Bug/menu

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
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 change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@davidjgonzalez Hi David, A quick check on this fix. Is this to hide the Menu dropdown appearing in the header section? If yes, looks like it's still appearing in 3.8.18.
Thanks,
Rahul


Find functions without return type

Good afternoon, this is such an interesting project with huge potential!
I want to write a rule which finds functions without a return type. For example, this code would trigger an error:
const foo = () => {
	return 1;
}

While this code would not:
const foo = (): int => {
	return 1;
}

The TS playground for the latter function looks like this:

My first thought was to make a rule which matches arrow_function (easy..) but when it comes time to find a missing value, I hit a wall.
Any suggestions or direction? This was my latest rule:
id: empty-return-type
message: Must supply a return type to a function
severity: warning
language: TypeScript
rule:
  all:
    - not: # doesn't work - matches all ast nodes except a type annotation.
        kind: type_annotation
    - inside:
        any:
          - kind: arrow_function
          # TODO Insert more function types here

Please take care and thank you for reading.

Hi Jack! Thanks for trying out ast-grep!
I tried it in the playground! Looks like it can be solved.
https://ast-grep.github.io/playground.html#eyJtb2RlIjoiQ29uZmlnIiwibGFuZyI6InR5cGVzY3JpcHQiLCJxdWVyeSI6ImNvbnNvbGUubG9nKCRNQVRDSCkiLCJjb25maWciOiIjIENvbmZpZ3VyZSBSdWxlIGluIFlBTUxcbnJ1bGU6XG4gIGFsbDpcbiAgICAtIGtpbmQ6IGFycm93X2Z1bmN0aW9uXG4gICAgLSBub3Q6XG4gICAgICAgIHBhdHRlcm46IFwiKCQkJCk6ICRSRVQgPT4gJCQkXCJcbmNvbnN0cmFpbnRzOlxuICAjIE1FVEFfVkFSOiBwYXR0ZXJuIiwic291cmNlIjoiY29uc3QgZm9vID0gKCkgPT4ge1xuXHRyZXR1cm4gMTtcbn1cbmNvbnN0IGZvbyA9ICgpOiBpbnQgPT4ge1xuXHRyZXR1cm4gMTtcbn0ifQ==

Fabulous, that works great. Thank you so much!

The guide here https://ast-grep.github.io/guide/rule-config.html#rule states that the rule should have one matched target node.
So the first step to compose a rule is to find the target. In this case, we want to

finds functions without a return type.

So I will first use kind: arrow_function to find function node. Then I used and with not to filter all candidates.


thread tests crash macOS High Sierra

Running cargo test --all on macOS High Sierra causes the machine to restart.
JDK: 1.8.0_144-b01 (latest at time of writing)
rustc: 1.21.0 (latest at time of writing)
macOS: 10.13

I would suggest starting to diagnose this issue by running the thread tests manually and seeing whether resource exhaustion events show up in the console.

The basic thread test print_memory_usage_0 passes when invoked by hand using the following command line:
java -agentpath:/Users/gnormington/dev/cf/jvmkill/target/release/deps/libjvmkill.dylib=printMemoryUsage=0 -cp ./resource-exhaustion-generator/target/classes -Xmx50m -XX:ReservedCodeCacheSize=10m org.cloudfoundry.jvmkill.ThreadExhaustion

Renamed the tests to make it clear which are thread tests and which are memory tests. Of the thread tests:

thread_basic
thread_print_memory_usage_0
pass and:
thread_time_10_count_2
thread_parallel_time_10_count_2
each crash macOS.

Observed that the command line equivalents of thread_time_10_count_2 and thread_parallel_time_10_count_2 pass when run individually.

Fixed by running the thread tests serially (using a mutex).


Wso2 İot Server - Docker İp Problem

Description:
I have a docker in my computer. And I can not remove the docker.  My ip address https://<IP_ADDRESS> When I run Server:
Broker.sh :        Mgt Console URL  : https://<IP_ADDRESS>:9446/carbon/
İot-Server.sh :   Mgt Console URL  : https://<IP_ADDRESS>:9443/carbon/   ------ IoT Server Default Context : https://<IP_ADDRESS>:9443/devicemgt
Analytics.sh :     Mgt Console URL  : https://<IP_ADDRESS>:9445/carbon/
My ip address https://<IP_ADDRESS>
When I run the broker and analytics.sh it works via docker ip.(https://<IP_ADDRESS>:9446)
How do I solve this problem without lifting the docker?
Affected Product Version:
Wso2 iot Server 3.2.0 RC4 - Agent v3.1.31
OS, DB, other environment details and versions:
Ubuntu 14.04.1 LTS / Java(TM) SE Runtime Environment (build 1.8.0_161-b12)

The server should still be accessible through <IP_ADDRESS>. Are you facing any issues when accessing the server through the IP <IP_ADDRESS>?

@omerstack Running the server with localhost is the bare minimum configuration for the WSO2 IoT Server. When multiple network interfaces available, server tries to bind into first available IP. But this may break some functionalities of the server such as inter-component communication using JWT tokens(When the certificate's hostname and HTTP client's sender's hostname differs).
If it is difficult to remove docker virtual network interface, you need to setup IP address of the server by running the change-ip.sh the script from the $IOT_HOME/scripts folder.


Command failed:'git' is not recognized as an internal or external command, operable program or batch file.

For some reason I'm getting:
start command
create new webview panel
ERROR: git log --graph extension encountered an unexpected error. Sorry! Error summary: {"code":1,"killed":false,"signal":null,"cmd":"git log --graph --oneline --pretty=format:\"^%^%^%^%^%h^%^%^%^%^%an^%^%^%^%^%ae^%^%^%^%^%at^%^%^%^%^%D^%^%^%^%^%s\" -n 15000 --skip=0 --all  --invert-grep --grep=\"^untracked files on \" --grep=\"^index on \"    --author-date-order","stdout":"","stderr":"'git' is not recognized as an internal or external command,\r\noperable program or batch file.\r\n"}. For details, see VSCode developer console. Please consider reporting this error.
ERROR: git log --graph extension encountered an unexpected error. Sorry! Error summary: {"code":1,"killed":false,"signal":null,"cmd":"git branch --list --all --format=\"%(upstream:remotename)^%^%^%^%^%(refname)\"","stdout":"","stderr":"'git' is not recognized as an internal or external command,\r\noperable program or batch file.\r\n"}. For details, see VSCode developer console. Please consider reporting this error.
ERROR: git log --graph extension encountered an unexpected error. Sorry! Error summary: {"code":1,"killed":false,"signal":null,"cmd":"git log --graph --oneline --pretty=format:\"^%^%^%^%^%h^%^%^%^%^%an^%^%^%^%^%ae^%^%^%^%^%at^%^%^%^%^%D^%^%^%^%^%s\" -n 15000 --skip=0 --all  --invert-grep --grep=\"^untracked files on \" --grep=\"^index on \"    --author-date-order","stdout":"","stderr":"'git' is not recognized as an internal or external command,\r\noperable program or batch file.\r\n"}. For details, see VSCode developer console. Please consider reporting this error.

git is definitely installed and working...

I'm not sure if there is much I can do about that. Try running git in other consoles outside of vscode, in various directories. Are you sure it's available on your PATH? Can you show me that?
Otherwise, perhaps there's some kind of standard environment vscode's internal terminal uses that we can hook into, idk....


Keeps recreating the elements and loads them again for the whole page

Everytime I try this, even for small elements:
html2canvas($("#localDiv")[0], {                     logging: true,                     useCORS: true,                 }).then(canvas => {                 });
All the elements in the html gets duplicated on top of the current ones and reloads everything in the page, it doesn't refresh the page as if with F5, it just reloads the elements again, I get all the logs in the console for those elements being initialized again with the previous log still there. And it doesn't get to the result in then nor get an error.
Does anyone knows why this happens?
Thank you.

It happens to me too

@shaji-Dev did you find the bug? please share solution. Thanks

@linlif


Timer button disposing issue

You may want to override dispose() function to cancel the timer.

Try calling by pressing the button
startTimer(-1);

An update has been released to solve this issue.


Data-jsx missing on the custom parent element

Hi,
First of all, sorry for the last issue, I didn't have time to contribute (https://github.com/zeit/styled-jsx/issues/215). However, I found another interesting bug I think, I made a little exemple of the behavior here:
https://github.com/loick/issue-styledjsx
Use case:

When I create a style tag with the jsx attribute, the parent tag also receives the data attribute
When I create a style tag with the jsx attribute and when the parent is a custom tag created (React.createElement), the parent doesn't receive the data attribute.

Is there something specific to add for this kind of situations ? I checked on the other Issues raised, but I'm not sure it was the same problem so.. :)
Thanks !

looks like this https://github.com/zeit/styled-jsx/issues/147 if so it is on my radar – you can either give it a stab and try to fix it or wait until I have some free time to work on it :)

sorry in my previous comment I was talking about CustomTag. Regarding React.createElement unfortunately we don't suppor that :(

Okay thanks, I'll try to take a look at it. Do you already know if this is related to something specific? Like should we render all of the components before executing the <style jsx> parsing ?

@loick sorry for the late reply, styled-jsx trasforms JSX only that's why it doesn't work with React.createElement.


Exceptions tag is not supported!!!!The result sequence.

This page has the text:
exceptions tag is not supported!!!!The result sequence.
https://msdn.microsoft.com/visualfsharpdocs/conceptual/seq.map['t%2C'u]-function-[fsharp]

There are actually a lot of these.  Also, code tags not supported.  Trying to clean them up as I go through the "bad translation" stuff from the XML doc comments as part of #61 .

@KevinRansom @ReedCopsey Yup. Looking at some of those now.

@KevinRansom @ReedCopsey identified a way to fix those. Working on it. Current batch in #140.

This should be covered by #163 to fix all exception errors.
Once the PR is merged, I will close this.

PR has been merged.


[WIP][DPS Auth] Page for generating cookie

Description
Since MyMove currently doesn't redirect to DPS anywhere, in order for DPS engineers to test generating a cookie as part of the DPS authentication process, I've created a simple page (my.move.mil/dps_auth/cookie) that has a text box for entering the cookie name and sets the cookie. The cookie name is different for their different test environments which is why they can set it, but in production it will always be "DPS".
Screenshots

@ntwyman - can you take another look?
A future TODO might be to remove the DPS auth cookie when the user actively logs out of MyMove / Login.gov. I don't think we can do this when the MyMove session times out since that's rather quick and would be overly restrictive for DPS.


refactor: refactor offset of controller

Codecov Report

Merging #125 (b578014) into main (aa224a9) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
+ Coverage   27.24%   27.27%   +0.02%
==========================================
  Files          78       78
  Lines        2870     2871       +1
  Branches      614      614
==========================================
+ Hits          782      783       +1
  Misses       1563     1563
  Partials      525      525

Flag
Coverage Δ

unittests
27.27% <100.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/core/src/lib/helper.ts
24.44% <100.00%> (+0.42%)
:arrow_up:

Continue to review full report at Codecov.

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


Add the --unsafe-perm flag to to npm install

Description of changes:
NPM will not execute any script hooks if it is running as root, which is common in docker environments (i.e sam build --use-container). The postinstall script hook is commonly used by many libraries (especially ones involving native bindings), and it is highly confusing as to why hooks are not run when executing sam build --use-container versus sam build locally.
Adding --unsafe-perm enables this.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

closing since #163 is merged.


Compilation Issue with ternary operator and arrays

The declaration of z in the following causes an error on Linux. I haven't had the chance to try on Windows yet.
var x = new string[] { "WINDOWS", "LINUX" };
var y = IsRunningOnWindows() ? "WINDOWS" : "LINUX";
var z = IsRunningOnWindows() ? new string[] { "WINDOWS" } : new string[] { "LINUX" };

@CharliePoole Might be a Mono compiler issue. Will take a look at it later today. Thanks for reporting it!

@CharliePoole there's indeed some Mono issue here, works on Roslyn. Could be an issue with the Mono codegen too.
If you add parenthesis is will compile though.
var x = new string[] { "WINDOWS", "LINUX" };
var y = IsRunningOnWindows() ? "WINDOWS" : "LINUX";
var z = IsRunningOnWindows() ? (new string[] { "WINDOWS" }) : (new string[] { "LINUX" });

Closed due to inactivity.

Doesn't the bug still exist?


feat: cove update

Hi James, here is a list of things I did:

Full usage of LCEL language for describing the chain
Parallelized the verification step (using the .map())
Adding few information using the CallbackManager (we could further improve this for sure but I'm not sure to understand how to properly define logging using the LCEL language. Maybe the feature is not available for the moment, I'll check later or maybe propose the feature if it's not possible ahah)

Let me know what do you think and do not hesitate to test it :)
What we could further improve but maybe not mandatory:

add a way to replace the verification chain with an internet searching agent (as you did in your article)

Hi @lijameshao , Hapy new year :)
Any update on this subject ? ahah


Harsha bajaj-profile card

Added my Profile Card

Done. Please check now

Please review this PR.


Update for 2021-08-11 release

Docs Build status updates of commit fc591e6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

package.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


ceres-solver: 1.14.0 -> 2.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ceres-solver/versions.
meta.description for ceres-solver is: "C++ library for modeling and solving large, complicated optimization problems"
meta.homepage for ceres-solver is: "http://ceres-solver.org"
meta.changelog for ceres-solver is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

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

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

found 2.0.0 with grep in /nix/store/895xz1vvq05z76gn7p05n33ifjca43aq-ceres-solver-2.0.0

directory tree listing: https://gist.github.com/d5605b77c53b7a8c14eb441ab74511f9

du listing: https://gist.github.com/0e5329b3b04f91485c69fc72c7db8073

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

13 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
4 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
ceres-solver
colmap
colmapWithCuda
openmvs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/895xz1vvq05z76gn7p05n33ifjca43aq-ceres-solver-2.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ceres-solver https://github.com/r-ryantm/nixpkgs/archive/d4b654cb468790e7ef204ade22aed9b0d9632a7b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/895xz1vvq05z76gn7p05n33ifjca43aq-ceres-solver-2.0.0
ls -la /nix/store/895xz1vvq05z76gn7p05n33ifjca43aq-ceres-solver-2.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review encountered an error. Please check the logs at https://r.ryantm.com/log/ for more information.

Maintainer pings
cc @giogadi for testing.

Result of nixpkgs-review pr 102115 run on x86_64-darwin 1

1 package built:

ceres-solver


ValueError: Object arrays cannot be loaded when allow_pickle=False

Hello,
I just cloned the repository but when I tried the examples in the doc, for example:
python ./ntru.py enc ./myKey.pub.npz ./test.txt

Here is the output:
Traceback (most recent call last):
  File "./ntru.py", line 149, in <module>
    output = encrypt(args['PUB_KEY_FILE'], input_arr, bin_output=not poly_output, block=block)
  File "./ntru.py", line 55, in encrypt
    ntru.h_poly = Poly(pub_key['h'].astype(np.int)[::-1], x).set_domain(ZZ)
  File "/nix/store/vvlzfcwnsdjacv0fq7vg7rchmskj5d96-python3-3.7.5-env/lib/python3.7/site-packages/numpy/lib/npyio.py", line 262, in __getitem__
    pickle_kwargs=self.pickle_kwargs)
  File "/nix/store/vvlzfcwnsdjacv0fq7vg7rchmskj5d96-python3-3.7.5-env/lib/python3.7/site-packages/numpy/lib/format.py", line 722, in read_array
    raise ValueError("Object arrays cannot be loaded when "
ValueError: Object arrays cannot be loaded when allow_pickle=False

I'm using python 3.7.5 on nixos (linux 5.4.8).
Thank you!

Hello,
This error is related to some security fixes in NumPy package (you can read more here: https://github.com/numpy/numpy/issues/12759).
I've just changed all np.load() calls to use allow_pickle=True, it should be working fine now.


Changed the invoice icon

It was the same as the Credit memo, now it's using the 'file' icon.

New icon

Thank you @mamazu!


Find a better way to determine last status change

What does this PR (Pull Request) do?

[x] This PR fixes #1636

Which Traffic Control components are affected by this PR?

CDN in a Box
Traffic Ops
Traffic Portal
CI tests

What is the best way to verify this PR?

Run local TO and TP and create a server. Notice that the status_last_updated field gets set to the current time in the DB.
Now, go to TP, and select the Status Last Updated as one of the display columns for the servers table. Notice that it shows the correct time elapsed since you created the server.
Now, update this server changing any field, EXCEPT the status of the server. Notice that the status_last_updated value DOES NOT change.
Now, update the server by changing the server's status (you will have to do this from the backend call, since TP doesn't let you do it from the front end).
Notice that the status_last_updated time gets changed to the current time this time around.
Also, make sure unit tests and API tests pass.
If this is a bug fix, what versions of Traffic Control are affected?

master

The following criteria are ALL met by this PR

[x] This PR includes tests OR
[x] This PR does not include documentation
[x] This PR includes an update to CHANGELOG.md
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

It actually does account for the changes made via PUT /servers/{{ID}}. But you're right about the user being able to change the last status change time. I'll fix that.

Actually, scratch that. You cannot change the status_last_updated time by doing a PUT to /servers/{{ID}} because of this part of the code:
if newServer.StatusID != nil && *newServer.StatusID != originalStatusID { 			newServer.StatusLastUpdated = time.Now() 		} else { 			newServer.StatusLastUpdated = origSer[0].StatusLastUpdated 		}

Closed in favor of https://github.com/apache/trafficcontrol/pull/4989


TSS2 Toyota: improve stop and go performance

Lowering vEgoX to 0.2 m/s means we let MPC/PID control acceleration for longer (without as harsh of a dip to 0 from 0.5 m/s), and start moving earlier.
I have PJ graphs showing TSS2 behavior, but I'll probably need to get new routes since they're old. I'll update this PR once I gather the data. Been running this on my fork for all TSS2 users for almost a month.

From a route from https://github.com/commaai/openpilot/issues/23778, shows an example where stock Toyota ACC wants to accelerate when our vTargetFuture is around 0.2 m/s, which matches this PR's changes. https://user-images.githubusercontent.com/25857203/154400565-f0581a42-71f8-4299-b9cd-bba931ba1db5.png
We can't just lower vEgoStarting until https://github.com/commaai/openpilot/pull/23333 is merged

@sshane
Tested on latest master
4aa5d874f367adbc|2022-02-17--21-21-11

This change seems to make us start requesting accel almost exactly when stock ACC does: https://imgur.com/a/xRnMAuw

This greatly reduces the number of times i wished i had no disengage on accellerator (which is now in master thanks). but would like to test its effect on stopping on hills behind a car to make sure it doesn't cause worse rollback. I can sometimes get rollback in my 2020 corolla hybrid sedan. But even if it does I'd say its still a big improvement and i will just have to hit the break to avoid rollback when combing to a stop on an up hill.

will re-open with testing


(PUP-10728) Use marshal to store transaction file

jenkins please test this with servertests

CLA signed by all contributors.


do we need to orainstRoot.sh and root.sh script or it is automatic

getting below mentioned error message when i try to execute
root@ubuntu:~# docker exec -it oracle_database gosu oracle sqlplus / as sysdba
error: failed switching to "oracle": operation not permitted

Does it work without gosu?

So yes, this should work without gosu. Fixed Readme.


Update supported OS of CmRcViewer.exe

Customer found that CmRcViewer.exe does not works on Win2008. This is by design but there is no mention in official docs and please add this limitation in this doc.
Summarize the change in the pull request title
Describe your change, specifically why you think it's needed.
Fixes #Issue_Number (if necessary)

@yuhonda : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jborsecnik : Thanks for your contribution! The author(s) have been notified to review your proposed change.

 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.:x: yuhonda sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi @yushonda - Would you take a moment to sign the CLA? After the CLA is signed, someone will review your changes. Thanks!

Once @yuhonda (who's an epic EE for us) signs CLA, I am good with approving the changes to the article.

@mestew : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


allow underscrores in props

@rultor release, tag is 0.0.12

@rultor release, tag is 0.0.12

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.12

@yegor256 Done! FYI, the full log is here (took me 8min)


What browser is used for the demo

What browser is used for the demo

Off topic, but it's Arc. https://arc.net


fix/private memos being public

fix(web/memo): filter out public option
feat(api/memo): sanity check for disabled public memos

Resolves #2462

We could also disable the select option. What do you like the best @boojack

@boojack We have access to the user role on the frontend, correct? If so, I'll just disable the public option if you're not an admin. Does that sound cool?

@boojack We have access to the user role on the frontend, correct? (HOST?) If so, I'll just disable the public option if you're not an admin. Does that sound cool?

SGTM


HTTP SNI check

The function tls_set_verify_server() should be extended by SNI (hostname verification).
Currently only used for rtmp. But this doesn't hurt because IP-addresses are excepted from this change. And for domains it makes sense to check the hostname.
The function tls_set_verify_server() now also is used for HTTP clients.
The function tls_set_verify_client() only activates certificate verification that accepts all certificates. It is used only in baresip module dtls_srtp. We do not touch this here, but want to rename the verify-handler to "verify_trust_all()".

TODO (but in another PR):

For SIP/TLS this function should be called for each TLS connection once. Maybe then it should be renamed to tls_set_verify_peer().

test_tls and test_dtls are failing.
error_string = self signed certificate

The tests test_tls and test_dtls are green now
@sreimers  Had to fix test_https_loop in retools. How can trigger travis for this PR without pushing an update?

Ok, I had to push updates already.
Anyway, I think that manual triggers are not possible until we have such "workflow_dispatch" mentioned here: https://github.blog/changelog/2020-07-06-github-actions-manual-triggers-with-workflow_dispatch/
But maybe manual triggers are not needed very often.

@robdyck can you please test this patch?
you should verify that SNI is not present in TLS Hello.

should be ok to merge now


Create wireframe mockup of metadata upload UI

Need to draft what the UI for an end user who wants to upload metadata to BETYdb will look like.
This will be needed for Phase II of GSOC but it will help to have this sooner. See also https://osf.io/v7f9t/wiki/GSOC 2019 Workplan/
This may require a few iterations.
Completion criteria - a document with images of each web page in the metadata upload workflow to be implemented.

Make a Jupityr notebook for working with the pipeline and drone data processing.

Here is a first draft of a mockup: https://app.zenhub.com/files/185447068/4bda1859-e74c-4ad5-83f3-8df0fe2ab19b/download

Estimate for the initial wireframe would probably be closer to a 2 - there is probably more work to be done as we iterate on the design that would bring it closer to a 5.

Just realized I could export to PNG ...
Note that these are a very crude approximation of the core functionality. We can discuss design, layout, content in the review.
Welcome

Upload

Upload Error Page

Validation / Review Page

Upload Successful

Hi,
I am Tezan Sahu, a third-year undergraduate from IIT Bombay. I went through this project idea for GSoC and found it very appealing to my interest. I have a decent knowledge of Python, Docker & Kubernetes & have started learning R. I have also filled the PEcAn GSoC Application Form and would like to know your email IDs so that I could send my CV for your referral.
I would also request you to guide me in understanding the background needed for contributing to this project.
Regards.


make the default fallback strategy for FT_DEFAULT configurable

As discussed here:
https://github.com/nanopb/nanopb/issues/772#issuecomment-1094100183
Note: testsuite ran and completed successfully.

Thanks! Are you interested in adding a test case for this also? It could go in tests/options.

tbh not keen - python isn't my first or preferred environment.
But I see the value and need, so I will have a go at it.
On Sun, 2022-04-10 at 01:42 -0700, Petteri Aimonen wrote:

Thanks! Are you interested in adding a test case for this also? It
could go in tests/options.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks, if you get stuck just let me know and I'll see about it. Should only need to add usage tests of the new option to options.proto and then in options_h.expected the pattern in .pb.h file that should be checked for.

I added the test case for fallback_type now.
Would you kindly consider it for inclusion again?

The scons version on debian (my environment) passed the test, but not on the ubuntu workflow. I believe the latest patch fixes this, albeit I did not verify this on ubuntu. Please trigger the workflow again

Thanks!

Thank you!
On 13 April 2022 10:00:05 WEST, Petteri Aimonen @.***> wrote:

Thanks!
--
Reply to this email directly or view it on GitHub:
https://github.com/nanopb/nanopb/pull/773#issuecomment-1097745171
You are receiving this because you authored the thread.
Message ID: @.***>
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


[Regression] Scene thumbnails show up as black squares

Godot version:
master / c93888ae
OS/device including version:
Manjaro Linux 17.1
Issue description:
After a recent update of Godot (approx. 1-2 week before c93888ae), all file icons began to show up as black squares as shown below:

It seems that existing icons are not affected by the problem, but all new or updated files show the symptom.

Thumbnails stop generating since 0e29f79

cc: @hpvb


Update for Spree 3.0.2?

When can we expect the support the spree 3.0.2?
Is there any quick way to make it comfortable for spree 3.0.2

I am having trouble with this as well. I need to create  POC for spree using only loyalty points.

It's available now with 3-0-stable branch. Closing.


代码优化，去除冗余条件分支，修改字符串比较方法

重复的if分支永远不会进入

Codecov Report

Merging #2708 into master will increase coverage by <.01%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master    #2708      +/-   ##
============================================
+ Coverage     80.44%   80.44%   +<.01%
+ Complexity     7098     7095       -3
============================================
  Files           145      145
  Lines         26708    26704       -4
  Branches       6374     6372       -2
============================================
- Hits          21484    21481       -3
+ Misses         3388     3385       -3
- Partials       1836     1838       +2

Impacted Files
Coverage Δ
Complexity Δ

...ava/com/alibaba/fastjson/serializer/JodaCodec.java
47.48% <33.33%> (+0.67%)
35 <2> (ø)
:arrow_down:

...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java
87.88% <0%> (-0.19%)
197% <0%> (ø)

...java/com/alibaba/fastjson/parser/ParserConfig.java
83.3% <0%> (-0.17%)
202% <0%> (-1%)

...ava/com/alibaba/fastjson/parser/JSONLexerBase.java
71.3% <0%> (-0.04%)
824% <0%> (-1%)

Continue to review full report at Codecov.

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


CERT_HAS_EXPIRED issue when I try to use proxy with https request

Is this a bug report?
maybe yes
I get this error
    Proxy error: Could not proxy request /api/islogged from localhost:3000 to https://www.example.com.
    See https://nodejs.org/api/errors.html#errors_common_system_errors for more information (CERT_HAS_EXPIRED).

if the proxy in the package.json is configured in this way:

      "proxy": {
        "/api-rc": {
          "target": "https://www.example.com",
          "pathRewrite": {
            "^/api-rc": "/api"
          }
        }

I tried to resolve it in two ways:

    if(process.env.NODE_ENV === 'development'){
        process.env.NODE_TLS_REJECT_UNAUTHORIZED = '0';
    }

and
    export const get = (url, params) =>
        const agent = new https.Agent({
            rejectUnauthorized: false
        });

        const p = params ? {httpsAgent: agent, params: {...params}} : {};

        return axios.get(url, p)
    };

but the error is still there . Am I wrong something?

Have you tried adding changeOrigin: true and/or secure: false?

Thank you..now it works

@Nide83 glad to help. You may close this issue in case your problem was resolved.


Data management (SuperSet)

MongoDB is not supported by Apache SuperSet. Relational DB like PostgreSQL is.
Would Pongo help in fixing this issue ?

I'm not an expert with Apache SuperSet, but based on my research, it supports PostgreSQL JSONB queries, so it should work well as Pongo is just a wrapper on top of regular PostgreSQL.
Could you tell me more about your use case? 🙂

@clement-igonet, I'm closing it for now, feel free to ask follow up questions if you have more :slightly_smiling_face:


allow for custom message to describe a tracking activity

commands like
preston track https://example.org

instructs Preston to start a tracking activity. Currently, a default activity is embedded in the associated provenance logs that document the triggered activity.
Suggest to add ability to add a custom activity description / message. Similar to how git allows for custom commit messages.
example:
preston track\
 -m "Archive this important dataset to make sure the data survives beyond the internet."\
 https://example.org

now available through v0.10.0 via command like:
preston track --message "some custom message" https://example.org


Optimize transferPart

Remove redundant filter conditions
Replace List by ArrayList or LinkedList
Randomly shuffle each partition's accounts before generating transfer edges

Revision merged to previous PRs


Release 0.3

#85 should land before this and the changes on the Vello side required should be reviewed as well.


Star Carnival, Magmalice Mine, and Spooky Hallow

This pull request contains two new maps:
-Star Carnival, from Mario Party 8. Comes with a custom background.
-Magmalice Mine, based on Magma Mine from Mario Party 9. Comes with a custom background that moves between board states.
It also contains an additional background, Spooky Hollow, from Pac-Man Party. It does not have a board associated with it yet, as it still needs work that I have to put in.

I approve of this pull request for Magmalice Mine.

Dear nikkums and Sophilautia. Please do not merge this pull request yet. Board and background are not complete.

Noted 👍

I've set this PR as a draft so it doesn't accidentally get merged in. Feel free to set it back to Ready for Review when you're ready for us to take a look at it!

A new PR will be made for each individual map when it is finished.


Training on a custom gym env

Any pointers on how a custom gym based env could be used? Assuming it's continuous control with a simple vector for the state.

Hi @ChrisAGBlake, thanks for reaching out!
You'll need to add a make_env(cfg) function for your custom environment(s), along with relevant wrappers (if any). You can use the metaworld example here as a relatively minimal reference implementation. After that, just add your new env constructor to the list here:
https://github.com/nicklashansen/tdmpc2/blob/57158282b46ebc5c329c5be9cfe2b0094126d1ca/tdmpc2/envs/__init__.py#L65
and it will automatically iterate through each constructor until it finds a match for your task argument.
Let me know if you run into any issues!

Thanks very much!

I'm currently getting the following error when trying to run on a custom env (I'm on the episodic-rl branch)
Traceback (most recent call last):
File "/home/chris/Code/Libs/tdmpc2/tdmpc2/train.py", line 57, in train
buffer=Buffer(cfg),
File "/home/chris/Code/Libs/tdmpc2/tdmpc2/common/buffer.py", line 18, in init
self._sampler = SliceSampler(
TypeError: Can't instantiate abstract class SliceSampler with abstract methods load_state_dict, state_dict
I can replicate this error with the following code:
from common.samplers import SliceSampler
sampler = SliceSampler(num_slices=256, end_key=None, traj_key='episode', truncated_key=None)

This looks like an error originating from the torchrl package. Do you get the same error when running any of the existing tasks? Upgrading or downgrading the version of this package might fix the problem in that case.
A potentially easier solution may be to use my prebuilt docker image for debugging purposes; this image should be able to run the code for sure. Instructions how to run the docker image can be found here.

Thanks, I downgraded the torchrl-nightly package to 2023.10.25 and it worked :)

Closing this issue for now, but feel free to reopen if you have any follow-up questions!


Edit textSticker on click

how can i edit textsticker on click

maybe you want this : https://github.com/wuapnjie/StickerView/issues/54

and where is the editsticker you are replacing here plz reply asap

Hey I have a same problem did you solved your problem if yes then help me I want to do the same
edit the text of sticker in edittext runtime


[1020] Added mechianism for interacting with backend session creation

Context
The frontend and backend must be in enforce with authentication and session.
External authentication needs to be funnelled to the backend to ensure authenticity of the user, the backend needs to have authenticated trusted client calls as well.
This needs to work in accordance with https://trello.com/c/gHNepiOF/993-create-new-user-sessions-in-api-backend.
Changes proposed in this pull request
Make it work
Guidance to review
The configuration for both frontend and backend needs to share the same credentials, for example see the below.
settings.yml
authentication:
  algorithm: HS256
  secret: lets-keep-my-fav-dan-super-happy

 Create new user sessions in frontend

 Organisation details links to C# Publish


Windows Support

[ ] Config files in Windows paths
[ ] Support for color codes in third-party console emulators
[ ] Testing (speed, Windows 7)

[ ] cgo

[ ] devices


Unicode literals

unify doesn't seem to support changing quotes in unicode literals e.g. u"example" is not changed.
Is this the intended behaviour?

Hey @andrewmwilson !
There was a related issue regarding f-strings (e.g. f"foo {bar}"), that was resolved by my PR: https://github.com/myint/unify/pull/4
Adding support for unicode literals should be similar.

Note to myself or anyone else who ends up doing this: the same issues applies to regex strings, ie r'\bfoo\b

Both implemented in #11.

Thank you!

Sorry to be a pain, but can we expect a release with this fix?

Here you go:
https://pypi.org/project/unify/0.5/

Thank you @myint!


Nav Bar dynamically moving

Problem
The navigation bar dynamically moves when clicking on links. For large projects, this can be annoying as a user sifting through documentation and constantly having to move the navigation bar back to where it was before clicking.
Reproducible Project
Cloud Custodian uses RTD. When clicking around on the nav bar and pages of docs, it often jumps up and you have to constantly move the bar to where you were previous.
Line 155 in theme.js
link[0].scrollIntoView(); causes the link to be dynamically place at the top of the nav bar, where I would like the nav bar to not move at all when I click on an href.
Expected Results
Navigation bar should stay in the same position while clicking around documentation.
Environment Info

Sphinx Version: 2.2.0

This is not trivial to solve. Since the theme is not an SPA and it does not use an iframe, the navigation will always jump to the top by default. The scrollIntoView() was added so the correct menu option is at least in the viewport.
To achieve the requested behavior, we would have to store the current scroll position in a cookie or pass it in the URL. Restoring the scroll position is complex because it needs to take into account menu items that are (no longer) expanded.
This is not a definite no and I understand how this could improve UX, so I'll keep this ticket open. I just don't see this happening anytime soon.

I am struggeling with the same line of code: what I would like to have is a configuration option in the theme, that prevents that scrollIntoView() of the active item. I tried to overwrite this with a custom.js script, but since the theme.js is called after my custom.js I could not change the behaviour:
So what would help be is a configuration option for either:

preventing the scroll to the active item
or being able to add custom.js after the theme.js

Would it be possible to implement one of these two options?

Logo and search bar look great, but the nav just jumps to the page in view.


Can't use with portable LINQPad

I have always used the portable version of LINQPad rather than the installed version. I seems that I am forced to use the installed version for LINQBridgeVs as each time I open VS I'm told to install LINQPad. Any chance you can make it work with portable LINQPad?

Latest version works with portable linqpad as well


Expose env vars pkg-config depends on

Thanks! I wonder, this may need to be opt-in for now? Consumer libraries may not currently be prepared for this if they're using pkg-config otherwise with a different fallback. I'd imagine for a 1.0 release we'd do this by default but for now with the same 0.1 release I'm thinking we may want to stick to being conservative?

How can a consumer be not prepared for this?
I assume it has no effect in older versions of Cargo, so won't cause back compat problems.
And I assume it only causes rebuild when Cargo sees the env var change, so it's a bug fix for builds that weren't rebuilt, and the worst thing that can happen is that Cargo will rebuild the project too often.

Ah so the problem here is that if you use pkg-config to locate a library and then use cc the crate to compile some support code, that build script won't get rerun when the C code changes. By default the cc crate doesn't print out the rerun-if-used annotations, so this could technically be a breaking change for that specific situation or situations like that

Oh, so the rerun-if-env-changed annotation replaces default file-based checks, instead of just adding one more check?

Right yeah, Cargo's got a default "when do I rerun the build script" heuristic which is quite conservative by default, and printing anything as a dependency will replace this heuristic


How to add Weighted Average to Nmon Visualizer Statistics ?

How can I add Weighted Average (as described in the Nmon Analyser User Guide) to the Statistics of an Nmon Visualizer chart ? An example of its use is on the IOADAPT sheet of an Nmon Analyser output spreadsheet.
See also https://www.ibm.com/developerworks/community/blogs/aixpert/entry/nmon_for_aix_15_years_on_and_still_learning1?lang=en

This shouldn't be too hard to add. Just add the calculation to AnalysisRecord and add a new enum to Statistic and it would be possible to graph this with a custom chart.
The majority of the work in the GUI would be updating all the table models to support it. Not difficult, but tedious to update and test.

I've added this metric as a default on all the GUI charts. You can reference it in custom charts using WEIGHTED_AVERAGE as the statistic name.
Please pull the latest code and let me know if you find any issues.