Create a Quad shape? (just sets a tiny concave polygon maybe)

Gogot 3.0.2 Windows
Hi there,
A kinematic Body using move_and_slide won't move on top of a plane mesh collider.
Steps to reproduce:

Create a staticBody, set it's friction to 0.5
Create a plane mesh as floor in Godot and add a collision plane to it
drag those as child of staticBody
a character won't move on top of the plane mesh using move_and_slide

Solution :
Use a collision box instead to the plane mesh collision
The character will move using the same script with move_and_slide

There is other 3D engines games with levels made of planes for floor and walls with physics working as intended. Godot should try to get it working.

Plane shape is mostly meant to be world ground/bottom, so that you don't get stuff falling infinitely.

This is not possible in Godot because the plane shape has infinite size. This mean that if you create a walls with plane shape you will not be able anymore to make doors or rooms... Probably in that engine the plane shape is meant to be used in another way or it just use a box with a thin side

Try unity, any plane will work with physics, many games have planes for walls and floors.
Is this Bullet physics limitation compared to Physx ? If it's not possible because of Buller physics i'm ok with that.

Does unity plane shape have infinite size? I'm not sure...

No need for infinite plane in Unity.
You can create a 7 meters plane, a capsule will collide on top of it and will fall down when it leaves the plane surface area. Collisions work great on walls that are planes only also.
Nvidia Physix must work differently.
Could Bullet work the same way and work with collisions against planes ?

@DevMagicLord: That means unity's plane shape is a quad, not a plane.

This is not a bug, plane shape is infinite in Godot. Use something else instead (maybe we could add a Quad shape?). Changing to enhancement.

Quad shape please...

3.1 or 3.2?

What is the purpose having a collision plane in the editor menu when physics doesn't work on it ?
Perhaps rename "collision plane" to "infinite collision plane" or "non physics collision plane", this way we won't be confused about collision.

Won't hurt to have a quad shape on 3.1, is just a new resource on the list.
@DevMagicLord Planes have infinite extents by definition, like lines.

@eon-s: wouldn't hurt to have it stated explicitly, some engines/devs use plane to refer to a quad.

@Zireael07 the thing is that the meaning of Plane is that, you cannot be more explicit https://en.wikipedia.org/wiki/Plane_(geometry)

I don't think lot of games will use infinite planes, add "Plane quad" is a good solution.
Put "infinite plane" in last position because nobody will use it for games, perhaps it could have some uses for some testing.

@DevMagicLord Is useful with Area

Useful with Area ? Any example ?

Plane shape is useful to define infinite collision, no wander if used with area or static body the use cases are not listable.
Also is good a thing define world margin (that prevent physical objects to fall forever) and the plane shape is perfect for this task.
Add quad shape is an enhancement, and prevents people to expect from the plane to be finite.

Moving to 3.2 since it's a new feature

I'm confused by this issue... planes work fine for me with rigidbodies, they collide with it correctly as a floor. If KinematicBody can't handle it I would indeed call that a bug Oo

Isn't this use case covered well enough by a very thin BoxShape?

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Testing ->get($url, ['REMOTE_ADDR' => $custom]) Doesn't Work

Laravel Version: 5.4.36
PHP Version: 5.6
Database Driver & Version: MySql

Description:
When passing server variables to http tests trait MakesHttpRequests prefixes all of the server variable with "HTTP_" but it seems like some variables should not be updated. Like "REMOTE_ADDR'
Steps To Reproduce:
Create a test:
        $response = $this->actingAs($user)->get('/dashboard', ['REMOTE_ADDR' => '<IP_ADDRESS>']);
Dump the request.
    Symfony\Component\HttpFoundation\Request {#695
      +server: Symfony\Component\HttpFoundation\ServerBag {#693
        #parameters: array:17 [
          "SERVER_NAME" => "test.dev"
          "SERVER_PORT" => 80
          "HTTP_HOST" => "test.dev"
          "HTTP_USER_AGENT" => "Symfony/3.X"
          "HTTP_ACCEPT" => "text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8"
          "HTTP_ACCEPT_LANGUAGE" => "en-us,en;q=0.5"
          "HTTP_ACCEPT_CHARSET" => "ISO-8859-1,utf-8;q=0.7,*;q=0.7"
          "REMOTE_ADDR" => "<IP_ADDRESS>"  <--- ################### NOTICE THIS
          "SCRIPT_NAME" => ""
          "SCRIPT_FILENAME" => ""
          "SERVER_PROTOCOL" => "HTTP/1.1"
          "REQUEST_TIME" =><PHONE_NUMBER>
          "HTTP_REMOTE_ADDR" => "<IP_ADDRESS>"  <--- ################### AND THIS
          "PATH_INFO" => ""
          "REQUEST_METHOD" => "GET"
          "REQUEST_URI" => "/dashboard"
          "QUERY_STRING" => ""
        ]
      }

This method at Illuminate/Foundation/Testing/Concerns/MakesHttpRequests.php (282)
    protected function formatServerHeaderKey($name)
    {
        if (! Str::startsWith($name, 'HTTP_') && $name != 'CONTENT_TYPE') {
            return 'HTTP_'.$name;
        }

        return $name;
    }

Might need to be something like.
    protected function formatServerHeaderKey($name)
    {
        if (! Str::startsWith($name, 'HTTP_') && $name != 'CONTENT_TYPE' && $name != 'REMOTE_ADDR') {
            return 'HTTP_'.$name;
        }

        return $name;
    }

But really it needs a whitelist or something, I don't know how many server variables that might need to be exempted.

You are passing a header as a data. Have you tried passing it as a header param?


open62541: init at 1.3.5

Description of changes
open62541 is an open source implementation of OPC UA.
I tested the build of open62451 and open62541.tests on x86_64-linux.
@harvidsen, you might be interested since you maintain asyncua, another OPC UA library.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2121


Cast datetime and timedelta to signed 64-bit int

Related to issue ( https://github.com/zarr-developers/zarr/issues/342 )
The NaT type is represented as -0. As a result, casting to an unsigned integral fails and throws an error. However casting to a signed integral type does not have this problem and proceeds without issues.
TODO:

[x] Unit tests and/or doctests in docstrings
[x] tox -e py37 passes locally
[x] tox -e py27 passes locally
[ ] Docstrings and API docs for any new/modified user-facing classes and functions
[x] Changes documented in docs/release.rst
[x] tox -e docs passes locally
[x] AppVeyor and Travis CI passes
[x] Test coverage to 100% (Coveralls passes)

Hi @jakirkham, this looks good, but there is an issue to do with how the backwards compatibility tests are setup, I'll try to explain. Basically the list of arrays in the test module for each codec can be added to, but existing arrays in that list shouldn't be changed. If they are changed, then they will be out of sync with the fixture data. This won't show up in the tests, as previous test arrays get saved to files, so the backwards compatibility tests run off the previously saved data.
In other words, if you change any of the arrays in the list, then the corresponding fixture data should also be deleted and regenerated. You can find the data to delete because the order in which the arrays appear in the list is used to generate a file name. E.g., within the test_zlib module, the first array in the list gets saved as fixture/zlib/array.00.npy, and the corresponding encoded data from the first codec in the codecs list gets saved as fixture/zlib/codec.00/encoded.00.dat. I.e., if you changed the first array in the list, then you should delete fixture/zlib/array.00.npy and all files like fixture/zlib/*/encoded.00.dat.
Alternatively and probably better in this case, leave all the existing arrays as-is, and append some new arrays to the list, in which case some new fixture data will get generated on the first test run.
Hope that makes sense!

Thanks @alimanfoo. IIUC you would like the tests added as a new set in the parameterized group. If so that makes sense to me, sorry I didn't think about it earlier. Have added the NaT tests as new tests at the end of the parameterized tests, generated the new fixture data locally (via running pytest), and committed/pushed. Please let me know if that works or if more needs to be done.

Going to merge this one so we can rebase other PRs (#121, #128).


No such property: projects for class: com.ullink.AssemblyInfoVersionPatcher_Decorated

I modified the sample buildscript on your home page and at the bottom there is the following code:
assemblyInfoPatcher {
...
projects = [ 'XZYProject' ]
}
When I looked at 'class AssemblyInfoVersionPatcher', 'fileVersion' and 'version' have a '@Input' but not the 'projects'. Could this be the reason? Or am I doing something wrong?

No it shouldn't have such impact AFAIK. Most likely cause is that you're using an earlier version of the plugin when the projects property didn't exist, can you double check ?

Yes it was the wrong version. But it was because I copied your example on the main page. Please correct the plugin version in that. The latest I found was 2.14 and it solved the issue.
dependencies {
classpath "com.ullink.gradle:gradle-msbuild-plugin:2.14"
}

Done, note that 2.16 should be the latest, you can see the changes in the CHANGELOG


add spaces to warning about missing version

Because a likely reaction to this useful warning is to copy paste this into the module and adjust x, y, and z but then it's convenient to not have to add spaces in addition.

Thanks.


Reuse rcon connections

Create and store pool of rcon connections.
A single rcon connection can only be used for one send at a time so it should be marked as in use before send and unmarked after receiving response. Maybe check for listeners on response event?
See also #28: add online to state and set by having Rcon attempt to connect and acting on ECONNREFUSED.

Using rcon-client that has a send/receive queue so a single connection suffices.


#1320 move DataGrid cascading value to surround entire component

Move the RadzenDataGrid cascading value to surround the entire component, allowing it to be picked up in customer header/footer etc. Splitting the existing CascadingValue to lines 8 (before all HTML) and approx line 435 just before @code.
Only a 2 line change - but the indenting affected looks horrendous :)
Trivial: Also added brackets to avoid missing colon style warning

Hi @djatkinsrealis,
Please keep the original formatting since at the moment one can hardly read what exactly is changed.

@enchev Is there any way to get GitHub to ignore whitespace changes in PRs, like DevOps does? If not, I will have to redo the change and abort this one as the slightest change in VS is triggering large whitespace differences.


Adding file cache recommendation to core profiler

PR now superceded by new PR #363

Closing due to rebase issues.  New PR filed: https://github.com/NVIDIA/spark-rapids-tools/pull/363


Implement Export to PDF/PNG Feature

Description
This feature adds an option in the db-drawer tool that allows users to export the visualized database schema as a PDF or PNG file. This enhancement will enable users to create static, shareable representations of their database structure.
Suggested Solution
The proposed solution involves adding "Export as PDF" and "Export as PNG" buttons to the db-drawer interface, allowing users to easily export the visualized database schema. The tool will then automatically prompt the user to download the generated file, named appropriately. This feature will enhance the usability of db-drawer by enabling users to create and share static representations of their database structures for documentation and presentations.
Alternatives
No response
Additional Context
I'm a GSSoC'24 contributor with a strong background in web development and experience working with libraries such as html2canvas and jsPDF and I want to work on this feature addition. So can you please assign this work to me @Bhanu-code

@sanskriti-lal cool!, sounds great. I am assigning you the issue. make sure to go through CODE_OF_CONDUCT and README.md. For any issue, ask on the gssoc discord server.

Thank you @Bhanu-code  for assigning me this work. I'll start working on it. Also one little request, can you please give it a level2 tag.

@sanskriti-lal  we have our guideliness by GSSoC  for labeling. So, we have to label accordingly.

ok cool! Also I've made a PR can you please look into it @Bhanu-code


change xvda to sda for placeholder

closes #1401

Coverage remained the same at 83.185% when pulling bc5abcedd62c24fb45a5dabbdded0b32ff9ef34e on abemassry:change-xvda-to-sda into 09afce4ccede409358a09502027d8afbcedf4b42 on linode:master.


crypto/rsa: EncryptOAEP documentation says "if a given public key is used to decrypt" instead of "encrypt"

What did you do?
Opened https://golang.org/pkg/crypto/rsa/#EncryptOAEP in browser
What did you expect to see?
For example, if a given public key is used to encrypt two types of messages, then distinct label values could be used to ensure that a ciphertext for one purpose cannot be used for another by an attacker.
What did you see instead?
For example, if a given public key is used to decrypt two types of messages then distinct label values could be used to ensure that a ciphertext for one purpose cannot be used for another by an attacker.

CC @FiloSottile, @katiehockman, @rolandshoemaker via owners.

I agree with @kocmo that the documentation is incorrect. If you are interested in fixing this, feel free to send a PR. Otherwise, one of us can look into it. Thanks.


fix(list): properly align contents in subheader

The line-height of the subheaders is currently based on the typography level. This is problematic because the subheader is set to a specific height and can't grow/shrink accordingly.

Fixes #6214

What about the dense attribute I also pointed out? Should I create a separate issue for that? Right now the same thing would happen in dense mode as there's only 8px left for the text. The spec doesn't even mention a dense subheader in dense mode.

@grizzm0 This fix should also calculate the proper line-height for subheader in dense mode.

@DevVersion Yeah, but the height of subheader in dense mode is changed from 48px to 40px. (even tho the spec doesn't mention it).
The padding is still 2x16px = 32px. Meaning there's only 8px left for the text while the font-size in this PR is caption (12px). That's 4px overflow which the text will be "pushed down" in dense mode.


Add functionality to validate user for remote dashboard creation api.

Add functionality to lookup user_info repo and search ldap to validate a userId used to create a dashboard remotely via api.

@tabladrum LGTM


Unfocus PR comment field after submitting a comment

Description
The comment field in PRs remains in focus after submitting a comment. If I scroll up quickly (with the intention to navigate away or otherwise), but not quick enough, then after the PR is submitted the page jumps back down, scrolling the comment field into view.
Screenshot
No response
Example URL
https://github.com/refined-github/refined-github/pull/5654

Here's a video with me commenting on a PR. Apologies for the delay, I had exams.
https://user-images.githubusercontent.com/6977583/173543018-20d092ad-7589-4078-bb6c-e18f9b8906d8.mov

It in the middle of sending the comment I think the behavior is expected


Evaluate the use of Handle pointers as results

During an unrelated code review I noticed this:
const Instance* StaticGetterNode::EvalConstExpr() const { ... }
We should evaluate all of the places where we return pointers to handles instead of RawXYZ*.

I think we can close this now. The sky hasn't fallen in the past 3.5 years :-)


tus over QUIC

Question
Are there any guides or knowledge regarding support of the new QUIC / HTTP/3-Protocol-Stack?
From what I know about QUIC this should improve transferspeed ?
Is there a way to make use of QUIC with some kind of QUIC-Proxy to tunnel tus/tcp ?
Thanks for your help
--mirac
Setup details
Please provide following details, if applicable to your situation:

Operating System: OpenBSD
Used tusd version: n/a
Used tusd data storage: disk store
Used tusd configuration: TLS-cert and key-file
Used tus client library: Uppy.js

I just tested Caddy as reverse-proxy:
Looks like Google Chrome connects over UDP (according to tcpdump output on the server) and Safari doesn't due to lack of QUIC support.
So I tested upload-speed with Safari and Chrome but it was almost the same speed.
Maybe my Wifi or anything below Layer 4 is the bottleneck here...
Ideas ?

tusd does not support QUIC connections natively because the net/http package from Go only supports up to HTTP/2. So running QUIC traffic through a proxy is necessary for now. However, I cannot help you much with missing speed improvements, as I have no practical experience with QUIC.

I see some benefits with QUIC/HTTP3 with high latency-connections.
We used Caddy for testing and it works fine.

Hi @miraculli would you be so nice and share your Caddy config using the reverse proxy + QUIC?
I am having problem even running TUS behind Caddy.


[🔔 Request SVG]: daily.dev logo

Title
daily.dev
SVG URL
https://drive.google.com/drive/folders/1GQ_jeQ4bgJOXoEqhu0JYqGKGwGAJUdJq
SVG Dark Mode URL (optional)
https://drive.google.com/drive/folders/1GQ_jeQ4bgJOXoEqhu0JYqGKGwGAJUdJq
Product URL
https://daily.dev/
Category
Social, Community
Checklist

[X] I have permission to use this logo.
[X] The .svg URL I have provided is optimized for web use.
[X] The .svg size is less than 20kb.

I added a PR with this logo request: https://github.com/pheralb/svgl/pull/447

thanks, i'll close it then.


DPMS handling for wlr_output

This pull request adds DPMS handling to wlr_output.
This is needed for https://github.com/swaywm/sway/pull/1824
That fixes  https://github.com/swaywm/sway/issues/541

Have rewritten sway part to use wlr_output_enable instead, since levels shouldn't be used.

I know this is closed and everything, but i'll just point out that DPMS "levels" are completely useless these days.
These days, all of the levels that aren't "on" are just converted to "off".

Thanks for the info, I've been wondering about that!

See e.g. https://lwn.net/Articles/653466/ "the atomic helpers only have on and off."


Making this integrated with the official KubeJS create addon

Hey, this seems like a really handy addition to KubeJS, and it's amazing that is was MIT licensed. However, why is this a seperate mod and not a PR to https://github.com/KubeJS-Mods/KubeJS-Create.
This is the only way I could find to contact you, there's an official KubeJS discord: https://discord.gg/lat, or you could dm me (username auseawesome on discord). I've shown this addon off in the discord after finding it on modrinth.
Thanks for the response, Auseawesome.

Because my code level is very poor, I guess that PR submission will not be accepted.

People can help clean stuff up with a PR, a bunch of people in the discord including me would love to see it as a proper feature


[Bug]: CATCH ERROR: ERROR STATUS: 404

Version

[X] Yes

General Question

[X] No

What happened?
When trying to update the presale mint date after confirming 'y' it comes up with CATCH ERROR: ERROR STATUS: 404 and not contract updated and ideas on how to fix?
What have you tried?
I Googled and asked the community in the discord server but could not find a solution..
Relevant log output
No response
Protect your information

[X] I Agree

I have the same issue described above
Updating presale_mint_start_date to 2022-05-12T16:20:00+01:00, correct? (y/n): y
CATCH ERROR: ERROR STATUS: 404
PS C:\Users\chris\Desktop\NFT ART Contracts\NewMinterDapp\minter-dapp\backend>


Change the Node.js beginners page to recommend the Windows native installation

The instructions provided on the beginners page with this change should work in any shell, including WSL.
This also includes a few small fixes, like updating the Node.js LTS version.
cc @mattwojo, let me know if this needs any change or feel free to edit!
Fixes: https://github.com/MicrosoftDocs/windows-uwp/issues/2011

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

Thanks @joaocgreis! I had already started a rewrite of this Node beginners page on another branch, but just went through all of your changes and combined them into my previous updates. Would you mind taking a look and giving me a thumbs up if this PR looks alright to you?
https://github.com/MicrosoftDocs/windows-uwp-pr/pull/515

@mattwojo I don't have access to the other PR, but I see it has already landed here including my changes. It looks great, thanks!
I'll close this PR since it would be better to open a new PR for any other change.


How to use it manually or with Carthage?!

Hi, I manually added ElasticTransition, but the effects does not show up!
How to use it manually or with Carthage?!
Please! I love this framework!

did you solved it ?

No

On Esfand 16, 1397 AP, at 3:38 PM, sultan Ali<EMAIL_ADDRESS>wrote:
did you solved it ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/lkzhao/ElasticTransition/issues/46#issuecomment-470501810, or mute the thread https://github.com/notifications/unsubscribe-auth/Abv1GbTR4ZPFYEyFaNpvTDoKw7uOFL_Gks5vUQFEgaJpZM4SBGPH.

How to use lkzhao/ElasticTransition manually or with Carthage?
Error accord in terminal during manual installation in carthage "Dependency "ElasticTransition" has no shared framework schemes".


Dataset deposit : steps description

In the dataset repository, we need a description for each step, so that the user is better guided.

Step 1 : Data Space selection

Step 2 : Files selection

Step 3 : Metadata form

Step 4 : Summary, check the metadata

Issue moved to okp4/kanban #783 via Zenhub


Subkey Commands out of date with latest Substrate

Latest Substrate seems to have updated Subkey commands.
https://substrate.dev/docs/en/tutorials/start-a-private-network/keygen
This page says to do:
subkey --sr25519 generate

where the command is now:
subkey generate --scheme sr25519

Would be good to double check all other subkey commands too.

squash and merge please @shawntabrizi


chore: upgrade jenkins-x/jx3-versions to version 0.2.10

from: https://github.com/jenkins-x-terraform/terraform-jx-azure.git

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


I can't get this to work

This project looks interesting but I can't seem to get it to work.
From master, I can build the project just fine but it is unable to generate the sample site. It just copies site/about.inc and site/index.inc to build. If I understand correctly, soupault should be generating much more.
This might just be a problem with the sample-site because I was able to get it to do a little more for a little sample site of my own, but I couldn't widgets to work. I only tried with the include widget. Here's a repo with what I tried: personal-website. Am I just being stupid?
I know that this is probably not enough information on it's own to figure out what's going on so let me know if I can help you figure out what is going on. I'm comfortable with ocaml.

You are right, I've neglected the sample site and it got out of sync with reality. The cause was lack of page_file_extensions option in the config that is required for non-standard extensions since 1.1.
That was done to make migration from completely hand-written sites that often store assets together with pages easier, but it also broke the sample site that I made in the very early stage of development when some design decisions weren't finalized yet.
As of non-working widgets, your issue is that you have [widget.navbar] instead of [widgets.navbar] in the config (missing "s" in "widgets"). Also, you don't want the <html>... wrapping in templates/nav-menu.html, just the HTML snippet you want to include.
I agree this is a pretty fragile config syntax, but there's a tradeoff between being able to pass arbitrary options to plugins and having a strict syntax. I think it's a good idea to add some sanity checking and warn the user about unrecognized table, perhaps offer some corrections (e.g. with ccube's Spelll). If you have ideas how to make it most user-friendly, please share.
Also, https://github.com/dmbaturin/baturin.org/ is a much more comprehensive usage example than the sample site, you may want to look into it.

Ah so it was something silly. Thanks for helping! I'll spend some more time playing with this and then maybe help you make this more user friendly.

I've made a pull request against your personal-website repo with a fix, too.
If you notice any more such annoyances, let me know. The downside of using my own project is that I know exactly what not to do, so it's often hard to notice. ;)


[fix/aut296, fix/aut295, feat/aut311, feat/aut471] - Packaging for macOS (dmg), Linux (deb), Windows (exe), CI for macOS and CI for Windows

Description
Enabling packaging on macOS and Linux platforms, the changes made in fix/aut295 were ported into fix/296 to ensure the validity of the packaging phase in macOS and Linux in CI.
Fixes

Generating deb packages and dmg packages without specifying the package type.
GUI executable is handled differently on each platform (MACOS_BUNDLE on macOS and a regular executable on Windows and Linux).
Dependencies for libSmartPeak, libOpenMS, libOpenSwathAlgo and libSuperHirn are not supplied in the deb package. Instead, upon installation the user will be prompted to have them fetched and installed by dpkg. (Recommended to use sudo gdebi -i <Package Name> to resolve any missing dependencies)
Removed irrelevant .cmake files related to packaging.

Features

Packages for macOS (dmg), Linux (deb), Windows (exe) are available after each successful build in Circle CI (Artifacts).

Future Improvements

build_windows and build_macos pipelines take around ~1h:15m to finish, this is due to OpenMS being built from source (~50min). A speedup can be achieved using caching.

No longer maintained

Target builds for packaging on IDEs is not supported, only on CLI through cpack

Packaging is not working for Win CI.

Packaging is not working for Win CI.

Do you mean that packaging on windows in not working in general or is not not automatically being generated on the CI?  If the former, we cannot merge as this is used by our customers in analytics.  If the latter, we can still generate the installer on either my machine or another windows machine from source.

Packaging is not working for Win CI.

Do you mean that packaging on windows in not working in general or is not not automatically being generated on the CI? If the former, we cannot merge as this is used by our customers in analytics. If the latter, we can still generate the installer on either my machine or another windows machine from source.

The Win CI was working but the Packaging in CircleCI didn't, now the generated packages for all three platforms are present in the artifacts tab in CircleCI.


Make "globs" parameter optional?

The second parameter is one or more globs. It seems intuitive to me that omitting the globs entirely would require every file in the base directory (i.e. the equivalent of specifying a **/* glob), but currently this is not the case.
Example:
var bulk = require('bulk-require');

bulk(__dirname + '/data', []); // Globs known but do not match anything. No modules required.
bulk(__dirname + '/data', ''); // Globs known but do not match anything. No modules required.
bulk(__dirname + '/data', '**/*'); // Globs known. All modules required.
bulk(__dirname + '/data'); // Globs unknown. All modules required.

Closing this, since doesn't seem to have much activity...


Audit Log Failure: Core Directory, ApplicationManagement, UpdateExternalSecrets is always present when configuring SCIM

After configuring SCIM provisioning for a custom, non-gallery app, Audit logs consistently show a Core Directory failure - stating "Update external secrets". To help diagnose, I've disabled group provisioning; have limited user provisioning to only familyName/givenName, and scoped this to only sync assigned users. Everything is provisioning just fine to our SaaS app, but this error is so obscure in detail I can't tell what it's talking about. Hardly any information is available online that references this, but the log is always present in our Audit Logs and it's becoming a problem for customers.
Anyone able to shed some light on this? It would be most appreciated.
Things I've checked/tried:

Our SCIM auth token is < 1KB in size.
Separated into two different enterprise apps, one for SSO and the other for SCIM; still the same problem.

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

ID: a2382361-1699-3beb-574e-0879c87f7442
Version Independent ID: 97f40a33-983d-f927-beb5-cb03aa3f82dd
Content: Known issues for application provisioning in Azure Active Directory
Content Source: articles/active-directory/app-provisioning/known-issues.md
Service: active-directory
Sub-service: app-provisioning
GitHub Login: @kenwith
Microsoft Alias: kenwith

@Cephei
Thanks for your feedback! We will investigate and update as appropriate.

Sincerely, it's immensely appreciated.

@Cephei
Thank you for your time and patience!
It sounds like the provisioning service is working as expected to your SaaS app with no issues. However, you're consistently seeing this Core Directory failure message within your Audit Logs.

Can you share a screenshot of what you're seeing? *Keeping in mind PII.
How long have you been seeing this error?

If you have any other questions, please let me know.
Thank you!

Here are some images from a dev/test Azure instance. This issue appears whenever we create a new enterprise app and connect provisioning. SCIM is successfully working, but this failure error keeps appearing in the Audit Log.

I'm not quite sure how long we've been seeing it for and whether it happens for each customer. I'll try to get more information on that tomorrow and report back. But it's definitely happening on more than two un-related Azure instances / subscriptions. I don't know if it happens to apps in the gallery, but as ours is not currently, it happens every time I create a new enterprise app.
We have a client that is quite concerned about the audit failure being consistently shown, even though the provisioning is working as expected. I'd like to be able to make sense of it, fix or resolve the cause, or just notify them if it is out of our control, but the MS team is aware of an internal issue.
Please let me know however else I can be of help.

@Cephei
Thank you for the follow up on this!
I reached out to our engineering team and also found a related issue with the same error that you're receiving Core Directory Update External Secrets.

The message is a warning from a system that is being transitioned out of use, and on which the customer's services are not actually relying. This warning message can be safely ignored.

If you're still experiencing these failures for a long period of time, please let me know.
Thank you for your time and patience throughout this issue.

Thank you very much for investigating this. 🍻


use systemd and upstart for core component controll

Register to upstart and systemd (prefered) all core services (elasticsearch, logstash)
Write v2 resources for starting and stopping core components based on this.

Replace systemV based init with more modern variant


added: aria-label translation

with examples of translation with and without a key

version 1.3.3 released


get_signature_request_list() is lacking parameters

The  get_signature_request_list() method only takes page and page_size as arguments, which prevents the use of some of the optional request parameters. account_id and query should be added as arguments as well.
ie:
def get_signature_request_list(self, page=1, page_size=None, account_id=None, query=None):
Currently the only workaround is to use the requests library and query that way.

Internal Reference: AS-231

Created https://github.com/hellosign/hellosign-python-sdk/pull/97


python3 doesn't work in Conda

Describe the bug
Any of the commands starting with python3, like python3 AllDS2020.py Train do not work in Conda
To Reproduce
Steps to reproduce the behavior:

Follow the steps mentioned in README.md
While starting training in Conda environment, run python3 AllDS2020.py Train
See error(s) - Nothing happens or Windows opens up the Store application

Expected behavior
Steps should run as expected and mentioned in the README.md file
Screenshots
NA
Desktop (please complete the following information):

OS: [Win 10]
Command-line interpreter: [Powershell]
Version [5.1.19041.1]

Additional context
NA

This obviously would be reproducible in any OS environment since in Conda using python suffices.

@rishabhbanga Thanks I merged this code but noticed there is no documentation for it mate please could you add.

Sure. Added it in pull request https://github.com/AMLResearchProject/ALL-Detection-System-2020/pull/5

Thanks @rishabhbanga


Create SECURITY.md

Adding Security Policy

are check it na ??
when will it accept it ??
should i change the organization as this organization has no responce ??


Improve part 5 of the tutorial

This PR provides the following changes to the part 5 of the tutorial:

add missing status code checks;
remove redundant status code checks since assertContains already performs them;
add missing question_text values at Question object creation instead of putting the information in variables.

@maggyero I'm not sure why you ignored my comments from #15868. I really don't see any value in changing variables names, adding/removing status checks in various places, etc. If you want to contribute to Django, focus on fixing accepted ticket.

Tutorial is not the right place for all details, it's a step of steps to complete a small project, see https://diataxis.fr/tutorials/. It doesn't need to be bulletproof, e.g. tests don't have to contain all existing assertions it's enough to include only the necessary assertions to keep the tutorial short and less confusing to beginners.

Yes I am aware that the Django documentation follows the Diátaxis framework so I understand that a tutorial is not a reference. Last week I read the Getting started documentation which includes the main tutorial and I found it excellent. The only part that I found a little confusing is this one (the redundant status_code checks, the missing status_code checks, and the non-provided question_text attributes). But this is just my perspective.


groupMembershipClaims atribute documentation is missing value "DirectoryRole"

groupMembershipClaims attribute in the manifest file can also have the value "DirectoryRole" to indicate the Azure AD roles assigned to the user.  This value is missing in the App Manifest documentation

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

ID: 3f7ced07-0b71-715f-749b-180567591c72
Version Independent ID: 8c533d7b-a058-3f1a-c296-9c0305f85f13
Content: Understanding the Azure Active Directory app manifest - Microsoft identity platform
Content Source: articles/active-directory/develop/reference-app-manifest.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@bachoang
Thanks for your feedback! I have assigned this issue to the author who will investigate and update as appropriate.

@bachoang
Thanks for your feedback! I have assigned this issue to the author who will investigate and update as appropriate.

@bachoang Thanks, you're right.  I've updated the article.  #please-close


Add #show for user profile

There should be a way to view each Veteran profile at a link like:
http://operationcode.org/veteran/david_molina
If we use a name we have to worry about how we could handle duplicate names (eg having two John Smiths). It may be easier to use the veteran id for now for a link that looks like:
http://operationcode.org/veteran/5
And then allow each user to set their own vanity domain.

Good call.
What if it was connected to individual Twitter accounts? Example:
http://operationcode.org/veteran/@davidcmolina. But then what if some
veterans don't have Twitter.
On Tue, Mar 22, 2016 at 8:43 AM, Rick<EMAIL_ADDRESS>wrote:

There should be a way to view each Veteran profile at a link like:
http://operationcode.org/veteran/david_molina
If we use a name we have to worry about how we could handle duplicate
names (eg having two John Smiths). It may be easier to use the veteran id
for now for a link that looks like:
http://operationcode.org/veteran/5
And then allow each user to set their own vanity domain.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/OperationCode/operationcode/issues/141

--
David Molina
https://about.me/molina

Here's an awful mockup of how I see this looking.
The thing on the left is our user picture.
To the right of that is a bootstrap nav (http://getbootstrap.com/components/#nav) tab or pil. That contains the different sections of the profile.
If a user doesn't have a section (eg talks/presentations) it shouldn't be displayed.

👌
Exactly!

On Mar 22, 2016, at 8:52 PM, Rick<EMAIL_ADDRESS>wrote:
Here's an awful mockup of how I see this looking.
The thing on the left is our user picture.
To the right of that is a bootstrap nav (http://getbootstrap.com/components/#nav) tab or pil. That contains the different sections of the profile.
If a user doesn't have a section (eg talks/presentations) it shouldn't be displayed.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

Closing this out due to lack of activity. Please feel free to open a new issue if this is still needed.


Feature Request: Allow dynamic filtering of search results

User Story
As a user I want to able to use the filtering or "faceting" functionality provided by Magento to more easily narrow down search results.
Description & Motivation
Magento allows for dynamic filtering of search results based on attribute configuration within Magento. It would be nice if the search related components supported some of this functionality. The "dynamic" part of this comes in because the attributes that are configured for search within Magento are dynamic and so too the AEM components used should also be dynamic so that as the searchable attributes are updated in Magento the attributes too are updated within AEM.
Deliverables

Additional components
Modification to existing search, navigation bar, and potentially product list UI elements
Ability to make simple introspection queries is required to be able to dynamically query Magento's GraphQL endpoints

Acceptance Criteria
After setting up CIF and adding the appropriate search component(s) to a page...

Search displays with a list of available filters
Adding a new attribute as filterable in Magento should result in the option being available for filtering in AEM
Removing an attribute from Magento should also remove the attribute as a filter option from the AEM search steps

Fixed by #223

Thanks for contributing @kevin-mitchell


Type 'Subscription' is not assignable to type 'Subscription'

TypeScript Version:
2.2.2
Code:
private ts:Subscription
this.ts = Keyboard.onKeyboardShow().subscribe(e => this.onShow(e));

Expected behavior:
Should work with no errors
Actual behavior:
Throws error
Type 'Subscription' is not assignable to type 'Subscription'. Two different types with this name exist, but they are unrelated. Property '_parent' is missing in type 'Subscription'.
Notes:
This just started happening after I upgraded Angular to 2.2.2 and rxjs to 5.1.1

Please provide a self-contained repro. We don't maintain the Angular or rxjs definition files, nor can we see your code

+1

same to me

@marek1 and @rodriguesgm please provide a self-contained repro we can use to investigate, otherwise, i am afraid this issue is not actionable for us.

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.

i have same problem with videogular2 im trying to getisHidden status
this.subscriptions.push(this.hidden.isHidden.subscribe(this.onHideControls.bind(this)));
error Type 'Subscription' is not assignable to type 'Subscription'
Property '_parent' is missing in type  'Subsciption'

I solved this problem by downgrading rxjs to 5.0.1

Thanks @GonzoBard this was also my issue. I had the following import:
import { Observable, Subscription } from 'rxjs/Rx';
I had to delete this line, save the .ts file, re-add the line and save again, and this fixed the errors (deleting only 'Subscription' from the import didn't work - I had to remove and re-add the whole line)

Seems like angular v8 is guilty in that error as well. Downgrading rxjs to "rxjs": "6.4.0" helped me.

Had the same problem and found this solution on stackoverflow:

So, I'm not sure why this worked but I'm answering my own question just in case someone else runs into this issue.
I uninstalled rxjs
I re-installed rxjs
I deleted my node_modules folder
I deleted my package-lock.json
I ran npm clean cache --force
I ran npm install
Then everything started working again :)

I'm using Angular 8.1, and the same issues cropped up.
Use of the exact following declaration solved the issue:
import {Subscription} from 'rxjs';

This imports the type from Angular declarations, so somehow the conflict isn't there anymore.


Add switch for choice version of wp

Could you add switch for choice version by installation of wp? I need stable v4.1, because my any plugins don't support latest 4.2.
Example: --ver=4.1

http://wp-cli.org/commands/core/download/

@JosefJezek IMHO its a bad idea as just for one plugin put entire website on risk.
As latest version always fix most critical security issues.
Refer: https://wordpress.org/news/2015/04/wordpress-4-1-2/

I would like specify previous stable major ver with latest minor number, ex. 4.1.2. Great idea is from @rahul286 Or script for get latest minor number for maj ver. ex. for 4.1 ee will install 4.1.2
https://github.com/rtCamp/easyengine/issues/116#issuecomment-97134867

-1.   I agree with Mitesh that this is a bad idea because of critical
security updates that are not available for prior releases, even if they
are listed as "stable".  In my opinion, this would create a liability risk
that the rtCamp team does not need, nor us as end users.
On Wed, Apr 29, 2015 at 5:47 AM, Josef Jezek<EMAIL_ADDRESS>wrote:

I would like specify previous stable major ver with latest minor number,
ex. 4.1.2. Great idea is from @rahul286 https://github.com/rahul286 Or
script for get latest minor number for maj ver. ex. for 4.1 ee will install
4.1.2
#116 (comment)
https://github.com/rtCamp/easyengine/issues/116#issuecomment-97134867
—
Reply to this email directly or view it on GitHub
https://github.com/rtCamp/easyengine/issues/518#issuecomment-97386487.

We all aware that WordPress need to latest but if wp-cli is giving choice, EE should also honor it.
I am planning to pursue - https://github.com/rtCamp/easyengine/issues/116#issuecomment-97134867
Idea is to leave wordpress related stuff to wp-cli wherever possible. That way we (EE team) will get some breathing space. We can use that time to focus on php/mysql/nginx optimization + security and other non-wordpress specific issues.
At some point, EE will have some sort of "notifications" and "monitoring" system. So server admin will be updated about outdated wordpress, among other things. I guess that can address concerns shown by @MattyLB
@MattyLB rest assured, EE defaults will be always to use latest codes of everything (not only wordpress). :-)

+1 @RahulBansal - that is a good plan.
On Friday, May 1, 2015, Rahul Bansal<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

We all aware that WordPress need to latest but if wp-cli is giving choice,
EE should also honor it.
I am planning to pursue - #116 (comment)
https://github.com/rtCamp/easyengine/issues/116#issuecomment-97134867
Idea is to leave wordpress related stuff to wp-cli wherever possible. That
way we (EE team) will get some breathing space. We can use that time to
focus on php/mysql/nginx optimization + security and other non-wordpress
specific issues.
At some point, EE will have some sort of "notifications" and "monitoring"
system. So server admin will be updated about outdated wordpress, among
other things. I guess that can address concerns shown by @MattyLB
https://github.com/MattyLB
@MattyLB https://github.com/MattyLB rest assured, EE defaults will be
always to use latest codes of everything (not only wordpress). :-)
—
Reply to this email directly or view it on GitHub
https://github.com/rtCamp/easyengine/issues/518#issuecomment-98156081.

Ok. Previous version 4.1 have latest security fixes aka 4.1.2.

@mbtamuli Can we pass all parameters that EE itself doesn't handle to wp-cli as it is?

This will be handled in #1029


Should honour .gitignores of submodules when passing -x

Looks like --exclude-ignored only excludes things .gitignore-d by the current repo, not things excluded in submodules (I think this is because it tars up the result of git ls-files which lists the non-ignored files in the current repo, but lists the base directory for submodules).
We have a bosh workspace which we run fly from with various submodules, some of which have large files in them that we'd rather not push up when running fly. Adding them to the .gitignore of the parent repository doesn't help because git ls-files still includes the submodule path, which gets tarred up as a whole.

We just ran into this too for what it's worth

We just ran into this as well; it came up because we .gitignore some compilation artifacts.

Moved to concourse/concourse#1076


fix: Rocksdb for high capacity relaying

Description
Relay nodes often fall behind the tip due to high query pressure.  The older rocksdb in tm-db doesn't work anymore, but this patch fixes that.
Todos

[ ] Unit tests
[ ] Manual tests
[ ] Documentation
[ ] Connect epics/issues
[ ] Tag type of change

Steps to Test
Expected Behaviour
Other Notes

Thanks a lot for contributing!
The change itself looks good to me. Could you describe any tests done in the PR description?

@kalidax could you take a look, should we improv or close this one?

Currently we don't use rocksdb backend (we use leveldb)
There is a couple of steps to add in order to build an image supporting rocksdb.
We probably could close it.


[Bug]: YAML is not correctly formatted for some strings

Willingness to contribute
No. I can't contribute a fix for this bug at this time.
What component(s) are affected?

[ ] Python SDK
[X] Opik UI
[ ] Opik Server
[ ] Documentation

Opik version

Opik version: 1.0.3

Describe the problem
Sometimes when I log data, the data is rendered nicely when viewing it using the YAML formatter and new line characters are rendered using new lines. However sometimes the input is displayed as a string and not formatted in an easy to read manner. This seems related to some special characters like \r or \t
Reproduction steps
%pip install opik
import opik

opik.configure()

client=opik.Opik()
client.trace(name="test", input={"input": "\n\rtest\n"})

This is an issue with the underlying library we are using, we patched it for \t last week: https://github.com/comet-ml/opik/pull/476/files. Seems like there is a similar issue with other characters as well. We will look at adding support for common "special characters": \t, \n, \r, \v, \f and \b

@ferc Can you take a look


CI: macos ninja setup failure

We use turtlesec-no/get-ninja for installing ninja to CI host and it is failing on current macOS 14 hosts. Upstream issue (where someone made a fork with a workaround): https://github.com/turtlesec-no/get-ninja/issues/4
This is so with SDL_image, SDL_mixer, SDL_net and sdl2-compat. On the other hand, we use ashutoshvarma/setup-ninja in SDL_ttf and SDL_rtf, and it doesn't seem to be failing.
Should we switch to using ashutoshvarma/setup-ninja for SDL_image, SDL_mixer, SDL_net and sdl2-compat? Or should we switch to using the fork of turtlesec-no/get-ninja? OR: Why aren't we just doing a simple brew install for it?
CC: @madebr

The fork is much simpler and has less dependencies.
Let's make the switch.

OK then. We should do the same for all satellite libs and sdl2-compat, as well.

Same issue on SDL2 and release branches of satellite libs, btw.

I went around all satellite libraries and switched the action everywhere, meanwhile also fixing the dylib current/compatibility version <=> soversion thing.
There are 2 macos/arm64 issues left:

the build-time SDL_image test fails (because of the relative recent libjxl)
vendored libmpg123 fails to build. Perhaps we need to bump the vendored version.

vendored libmpg123 fails to build. Perhaps we need to bump the vendored version.

Before blindly bumping it, I looked at optimize.h line 198, it errors if REAL_IS_FIXED is defined, because cmake_host_system_information isn't setting HAVE_FPU? Why?

Looks like a CMake bug to me.
https://github.com/madebr/throwaway/actions/runs/8915872718/job/24486263066#step:3:11
I created an upstream bug report.
The mpg123 cmake script already contains an exception for MSVC, so it looks like the test has also failed there in the past.

OK, so we should add an APPLE exception then?

Yeah, I'm looking into creating a patch for current main atm.
It should be easy to backport.

Done, and also applied to mpg123 of SDL2.

macos/arm64 issues left:
the build-time SDL_image test fails (because of the relative recent libjxl)

The CI log says brew is installing version 0.10.2 of libjxl. Do we have any other
runners that use system libjxl and at least with version >=0.10.0 ?
P.S.: The libjxl + macos issue should be noted in a new ticket?

The CI log says brew is installing version 0.10.2 of libjxl. Do we have any other runners that use system libjxl and at least with version >=0.10.0 ?
P.S.: The libjxl + macos issue should be noted in a new ticket?

Yes, a new ticket should be created.
Only toolchain that has libjxl>=0.10 is msys2.

Created https://github.com/libsdl-org/SDL_image/issues/450
Closing this one.


Linking a github account still breaks if your email is set to private on github

When first trying strider I had this issue when I tried to link my github account.  I resolved it by going into github and making my email address public (which I don't want to do) and then after linking the account with Strider switching it back to private.  Once that initial check happens no more issues. It is discussed here but the problem still occurs in the latest version.
https://github.com/Strider-CD/strider-github/issues/25
Logging this so if its left as a known issue we at least notify the user of this trick/tip instead of a 500.  I believe you might only be able to test this with a new user.
======================
Connect
500 TypeError: Cannot read property '0' of undefined
at makeAccount (/usr/local/lib/node_modules/strider/node_modules/strider-github/lib/webapp.js:167:28)
at Strategy.validateAuth [as _verify] (/usr/local/lib/node_modules/strider/node_modules/strider-github/lib/webapp.js:152:26)
at /usr/local/lib/node_modules/strider/node_modules/strider-github/node_modules/passport-github/node_modules/passport-oauth2/lib/strategy.js:188:22
at /usr/local/lib/node_modules/strider/node_modules/strider-github/node_modules/passport-github/lib/strategy.js:109:5
at passBackControl (/usr/local/lib/node_modules/strider/node_modules/strider-github/node_modules/passport-github/node_modules/passport-oauth2/node_modules/oauth/lib/oauth2.js:126:9)
at IncomingMessage. (/usr/local/lib/node_modules/strider/node_modules/strider-github/node_modules/passport-github/node_modules/passport-oauth2/node_modules/oauth/lib/oauth2.js:143:7)
at emitNone (events.js:72:20)
at IncomingMessage.emit (events.js:163:7)
at _stream_readable.js:891:16
at process._tickDomainCallback (node.js:388:13)

This is documented in strider-github and if you updates the plugins, under admin/ plugins the error is clear

Fixed, just make sure to update your plugins by going to "Admin" and then "Plugins" from the app. Update all of the plugins there. I'm going to add a note about upgrading plugins to the readme, at least until it happens automatically on first start.


Overlapping lines

The algorithm does not handle the case in which lines are overlapping. Consider, f.e:
print(list(line_intersections([((0,0),(10,0)),((5,0),(15,0))])))

There are probably many corner cases like this. The way I compute cross products is also not right because I use normal floats, so rounding errors can change the result.
It should work if the points are in General position. Typically you can add a small random perturbation to get to general position without changing the result significantly.
If you want proper battle tested code you should check out CGAL
I accept pull requests if you want handle some of the corner cases you encounter.


move k3d functions to weldx-widgets

to lighten the dependency load of our core weldx package a little I think it would make sense to migrate some of the advanced features over to the weldx-widgets package
Some things to consider/discuss:

can we keep some of the convenience functionality, like calling *CSM.plot() and redirecting the call to the weldx-widgets package if that is installed?
how to version the weldx-widgets package? I suggest using the same version numbers as the weldx package and pinning that requirement?

I agree that we should do this split. Preserving the existence of easy plotting routine should be kept.
I'd vote against a strict version coupling and use semantic versioning whenever possible. E.g. there is a breaking change in the CSM api, which does not allow the visualization routine to work properly, one should increase the weldx-widgets major and require this in weldx. The requirement will have to be made during import time of course. Otherwise we would still drag in lots of dependencies we would not need for basic file io.

Actually we should consider the same thing for matplotlib dependent features.

sounds good, I suggest to start with k3d first and move to other features (like mpl) later

see #636


Remove support for Swift 4.x; Update CI builds

Remove support for Swift 4.x; Update CI builds to use community docker images

 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.Danny Sung seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@mbarnach thanks for the reminder about cocapods. I put a minor comment on old versions in the README.


availability of surface variables?

The NeuralGCM paper contains some analytics that depend on the surface information. For example, the precipitable water calculation involves a vertical integral that needs surface pressure. The tracking of cyclones uses sea-level pressure and near-surface wind. However, the output of the PressureLevelModel does not contain surface variables. Additionally, surface temperature is also a variable that can enable many applications. How may users access these surface variables?

You can output the surface pressure from the model by adding to the config SurfacePressureDiagnostics.
Something like this should do the trick:
new_model_config_str = '\n'.join([
ckpt['model_config_str'],
'StochasticPhysicsParameterizationStep.diagnostics_module = @SurfacePressureDiagnostics',])
ckpt['model_config_str'] = new_model_config_str
Regarding TC tracking: we give a full detail in the appendix of our paper how we used 2 different methods (using sea level pressure/vorticity at 850hPa + other criterion) to track cyclones (that yielded similar results) but non-used near-surface wind.
We currently do not have the concept of "surface variables" (except surface pressure which is a prognostic variable in our dycore) and we output variables only on pressure levels.

Surface temperature would be useful.

Thanks @yaniyuval! Sorry for not seeing your reply earlier:

You can output the surface pressure from the model by adding to the config SurfacePressureDiagnostics. Something like this should do the trick: new_model_config_str = '\n'.join([ ckpt['model_config_str'], 'StochasticPhysicsParameterizationStep.diagnostics_module = @SurfacePressureDiagnostics',]) ckpt['model_config_str'] = new_model_config_str

I placed the suggested edits between the step of downloading ckpt and "model = neuralgcm.PressureLevelModel.from_checkpoint(ckpt)". But the model initialization ran into the following error:
ValueError: No configurable matching reference '@SurfacePressureDiagnostics'.
Did I miss any other steps when configuring the model?

Regarding TC tracking: we give a full detail in the appendix of our paper how we used 2 different methods (using sea level pressure/vorticity at 850hPa + other criterion) to track cyclones (that yielded similar results) but non-used near-surface wind.

SLP is the key variable. Thanks for the clarification about the tracking.

Surface temperature would be useful.

Agree. Before the functionality is implemented, it is relatively straightfoward to approximate the surface temperature when surface pressure is available.

@GanZhang-GFD ,
Try to pip install directly from the github page since the SurfacePressureDiagnostics was added only recently:
pip install git+https://github.com/google-research/neuralgcm.git@main

@yaniyuval Thanks! I managed to finish the run with the newer model version.

The surface pressure appears to be in the final state object. The data array does not have the unit attribute. The values suggest some log operation was involved. Reversing log10 yields a reasonable pattern, but the lowest values are too small. Did I miss any steps?

plt.contourf(10**(final_state.diagnostics['surface_pressure'].T), cmap='Reds')
plt.colorbar()

Is there a smart way to access surface pressure at steps before the final state?

@yaniyuval Thanks again! The code snippet didn't change the output values.
The test was with the reference code with neuralgcm.git@main. I also tested other variants of the pressure unit with the model config dictionary.
I am still interested in checking whether any existing mechanisms help access surface pressure at steps before the final state? Rolling out the model step by step is feasible but somewhat cumbersome.

Re drift in surface pressure: I also noticed drifts in other variables (e.g., temperature) of various magnitudes. Larger drifts usually come together with model instability. Smaller drifts appear to do relatively little harm when the simulations are not very long. I look forward to testing the next model version!

@GanZhang-GFD ,
I slightly updated the snippet.
When I use it I get the correct units. See a snapshot.

@yaniyuval The updated snippet worked! Thanks.
For the others watching this issue: The snippet helps put the surface pressure into the prediction object (along with other variables like u, v, etc.). Since the surface pressure is avialable at the same output steps, my other question was also resolved.

@GanZhang-GFD , Try to pip install directly from the github page since the SurfacePressureDiagnostics was added only recently: pip install git+https://github.com/google-research/neuralgcm.git@main

I'm still getting an error message No configurable matching reference '@SurfacePressureDiagnostics'. I've tried to pip install from the GitHub address.

@GanZhang-GFD , what method would you use to calculate surface temperature, given surface pressure, and the model state?  Thanks very much for any advice

@GanZhang-GFD , what method would you use to calculate surface temperature, given surface pressure, and the model state? Would it be based on approximating a lapse rate? Thanks very much for any advice

@ankurmahesh With the surface pressure data, you may try to extrapolate the temperature on pressure levels to the surface. Your mileage may vary depending on your extrapolation technique and region of interest. For example, it can work reasonably with flat surfaces near the sea level but will struggle with complex terrains. In such cases, post-processing bias corrections are likely necessary.


HDDS-1712.  Test pull request acl.

/label ozone

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
33
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

0
yamllint
0
yamllint was not available.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1
mvninstall
527
trunk passed

+1
compile
248
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
1504
branch has no errors when building and testing our client artifacts.

+1
javadoc
154
trunk passed

_ Patch Compile Tests _

+1
mvninstall
454
the patch passed

+1
compile
257
the patch passed

+1
javac
257
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
695
patch has no errors when building and testing our client artifacts.

+1
javadoc
227
the patch passed

_ Other Tests _

-1
unit
234
hadoop-hdds in the patch failed.

-1
unit
143
hadoop-ozone in the patch failed.

+1
asflicense
36
The patch does not generate ASF License warnings.

3945

Reason
Tests

Failed junit tests
hadoop.hdds.scm.container.placement.algorithms.TestSCMContainerPlacementRackAware

hadoop.hdds.scm.block.TestBlockManager

hadoop.ozone.om.ratis.TestOzoneManagerDoubleBufferWithOMResponse

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1053/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1053

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient yamllint

uname
Linux 99296647a209 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 8965ddc

Default Java
1.8.0_212

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1053/1/artifact/out/patch-unit-hadoop-hdds.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1053/1/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1053/1/testReport/

Max. process+thread count
489 (vs. ulimit of 5500)

modules
C: hadoop-ozone/dist U: hadoop-ozone/dist

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1053/1/console

versions
git=2.7.4 maven=3.3.9

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.


transition-property doesn't respect global prefixer options

If I set
$prefix-for-webkit: false;
$prefix-for-mozilla: false;

then
@include transition-property(border-color, color);

still outputs
-webkit-transition-property: border-color, color;
-moz-transition-property: border-color, color;
transition-property: border-color, color;

:disappointed:

Hi Jacob,
I put together a fix in #638. Looks like that one never got switched over to using the prefixer for some reason. That should sort it out for you.

:+1:

Unfortunately, Josh and I figured out the the transition mixin would need much more of a rewrite and not just a quick fix to stop these prefixes from being output. We realize keeping track of these prefixes is a huge task, which is why we are no longer doing this in v5.0. I would highly recommend you check out Autoprefixer as another solution, rather than our prefix mixins.


Permission Issue

When it try to list directories inside directory without read permissions, it throws out error. I think bfs should skip such directories  before hand.
error: './cache/private': Permission denied
error: './cache/ldconfig': Permission denied
error: './db/sudo': Permission denied
error: './tmp/systemd-private-b0c63c239a4f49ebaf5fa0ee80d157c8-ModemManager.service-ODCouH': Permission denied
error: './tmp/systemd-private-b0c63c239a4f49ebaf5fa0ee80d157c8-systemd-timesyncd.service-Jze1lA': Permission denied
error: './lib/portables': Permission denied
error: './lib/machines': Permission denied
error: './lib/private': Permission denied
error: './lib/NetworkManager': Permission denied
error: './lib/udisks2': Permission denied
error: './lib/rpcbind': Permission denied
error: './log/private': Permission denied
error: './spool/cups': Permission denied
error: './lib/nfs/sm': Permission denied
error: './lib/nfs/sm.bak': Permission denied
error: './lib/sddm/.nv': Permission denied
error: './lib/sddm/.config': Permission denied
error: './lib/gssproxy/rcache': Permission denied

bfs tries to be compliant with the POSIX spec for find.  That includes exiting with an error code if any part of the directory hierarchy was unable to be traversed.  I think it would be very confusing if bfs would exit unsuccessfully without printing the relevant errors so the user can see why if failed.  (On the other hand, if these are errors that find isn't printing, that may be a bug.  Let me know if that's what's happening.)
You could suppress those errors with
bfs ! type d -o -readable -o -prune

which avoids descending into non-readable directories.


[enhancement] Reduce useless cells

the program is currently doing the red path (sorry for clarity), instead of doing the "optimized" orange path. that'd reduce considerably paths' size

just to clarify: i'm here speaking about reducing steps by double checking the diagonal => if there is an obstacle and we have to fallback on a previous position, we could just remove every position AFTER that "proper" one


Microsoft.DataFactory/factories/pipelines: missing properties in Copy discriminator

Resource Type
Microsoft.DataFactory/factories/pipelines
Api Version
2018-06-01
Issue Type
Missing property(s)
Other Notes
When Activity type is set to Copy, it looks like there are missing properties. For example, the Copy object should have a policy object. I have looked at the Swagger files and they seem to include the properties. Here is ExecutionActivity which includes Policy as one of its properties. CopyActivity includes ExecutionActivity under "allOf"
When I view the ARM template generated by Data Factory it includes Policy:
"activities": [
{
"name": "Copy data1",
"type": "Copy",
"dependsOn": [],
"policy": {
"timeout": "7.00:00:00",
"retry": 0,
"retryIntervalInSeconds": 30,
"secureOutput": false,
"secureInput": false
},
"userProperties": [],
However, when I look at CopyActivity in the generated types, the policy object is not there.
Bicep Repro
No response
Confirm

[X] I have searched this repository and have not found similar issue reports.

This looks like it has the same root cause as #954, so it should be taken care of by the same fix.


memcached needed to use rate limiting feature ?

Hi,
I tried using the rate limiting feature for my api.
in my routes.php file I have:
$api->version('v4', ['prefix' => 'api', 'middleware' => 'api.throttle', 'limit' => 100, 'expires' => 5], function($api) {
    $api->get('members', 'App\Http\Controllers\MemberController@members_GET');
    $api->get('members/{id}', 'App\Http\Controllers\MemberController@member_GET');
    $api->patch('members/{id}', 'App\Http\Controllers\MemberController@member_PATCH');
});

When I try sending a request, I get an error about memcached...
{"message":"Class 'Memcached' not found","code":1,"status_code":500,"debug":{"line":51,"file":"\/var\/www\/sites\/testing-apiv4\/vendor\/illuminate\/cache\/MemcachedConnector.php","class":"Symfony\\Component\\Debug\\Exception\\FatalErrorException","trace":["#0 [internal function]: Laravel\\Lumen\\Application->Laravel\\Lumen\\{closure}()","#1 {main}"]}}

There is this line in my .env file CACHE_DRIVER=memcached that came from the env.example I believe, but commenting it out has no effect.
Is it necessary to setup a memcached install in order to use the rate limiting feature ?
I'm sorry if this is not the proper place to inquire about such issues, please let me know and I will move this question where it's more appropriate.
Thanks

As you can see in the Handler.php , limiting request rate of the api is made with the help of the CacheManager. For usage of any CacheManager you need a cache driver. If you choosed to use memcached driver: yes you need to install the memcached server.

Thanks for your help @catalinux, I saw in the lumen docs that cache drivers are memcached or redis. Am I right in thinking I need to setup either one of those to use the rate limiting feature? or is there another way to go ?

@jmartigny : although I did not use that particular feature (RateLimit) your thinking is correct.


Appstudio update test-component-pac-ijyo

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-ijyo-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
38 seconds
build-container

✅ Succeeded
23 seconds
deprecated-base-image-check

✅ Succeeded
27 seconds
inspect-image

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
16 seconds
sbom-json-check

✅ Succeeded
11 seconds
label-check

✅ Succeeded
8 seconds
show-summary

✅ Succeeded
7 seconds
show-sbom


Fix ：the wrap error will transfer to sdk, and will cause the real error lost

Description
errors.Wrap(err, "ERR_PUBSUB_CLOUD_EVENTS_SER")

The wrap error will transfer to sdk, thenl cause the real error cannot be recognized.
So we should remove the wrap function, and transfer real cause to the sdk .
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@cvictory thanks for this PR. when you say SDK, which SDK do you mean? all of them?

In the java SDK， We want to get the real cause but not wrapped error.

I found some optimization have been done. So I will close this pr temporarily


cherrypick: small runtime tweaks (#3519)

fix: azure logs issues

Can't log after response is written for some reason.

fix: watch files in dev only

fix: load appsettings files based on NODE_ENV

Roughly maps to behavior added via https://github.com/microsoft/botbuilder-dotnet/pull/5421

Pull Request Test Coverage Report for Build 719976404

0 of 0   changed or added relevant lines in 0 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 84.91%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

libraries/adaptive-expressions/src/triggerTrees/node.ts
8
91.7%

Totals

Change from base Build 709291472:
-0.02%

Covered Lines:
18887

Relevant Lines:
21181

💛  - Coveralls


[Blast] Add support for Blast Mainnet

If this is your first time integrating, the following information is required

A short introduction about your project/ your team: Blast is an EVM chain with native yield for ETH and stablecoins
Document: https://docs.blast.io/about-blast
Social media links (discord, website,...)
Github: https://github.com/blast-io

If you want to integrate a chain, you would need to provide

Name of the blockchain: Blast Mainnet
Type of the blockchain: EVM
Blockchain providers: https://rpc.blast.io
Metadata of the blockchain: https://docs.blast.io/building/network-information
Logo of the blockchain: https://www.figma.com/design/z9s6aLTLoUdjMwoXLyWcPz/01.SubWallet-Project-Logo?node-id=7207-91&t=0IsuhPRdmoCAwlpL-4
Block explorer: https://blastscan.io/

If you want to integrate an asset (fungible token/ NFT), you would need to provide

Name of the asset: BLAST (Blast)
Logo of the token: https://www.figma.com/design/z9s6aLTLoUdjMwoXLyWcPz/01.SubWallet-Project-Logo?node-id=7207-91&t=0IsuhPRdmoCAwlpL-4
Coingecko: https://www.coingecko.com/en/coins/blast
Doc: https://docs.blast.io/tokenomics

https://blastscan.io/token/0xb1a5700fa2358173fe465e6ea4ff52e36e88e2ad#balances


Update langfuse_filter_pipeline.py

sometimes tools/functions break the langfuse pipeline because chat_id is missing. this check to see if a value exists and if it does not, sets it to SYSTEM MESSAGE UUID
where UUID is a generated uuid

LGTM but I'd like to hear from someone else that uses Langfuse on whether this will break anything.

not a problem, I'd like to hear from others as well!


ROCm support for gfx1011

The Amazon G4ad instance have AMD Radeon V520 GPUs with device name gfx1011 unfortunately many of the ROCm utils do not have binaries supporting gfx1011
2020-12-17 23:03:23.004276: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libamdhip64.so
/src/external/hip-on-vdi/rocclr/hip_code_object.cpp:120: guarantee(false && "hipErrorNoBinaryForGpu: Coudn't find binary for current devices!")
Fatal Python error: Aborted

docker run -it --device=/dev/kfd --device=/dev/dri --security-opt seccomp=unconfined --group-add video rocm/tensorflow
But I do see the output from rocminfo
*******
Agent 2
*******
  Name:                    gfx1011
  Uuid:                    GPU-XX
  Marketing Name:          Device 7362
  Vendor Name:             AMD
  Feature:                 KERNEL_DISPATCH
  Profile:                 BASE_PROFILE
  Float Round Mode:        NEAR
  Max Queue Number:        128(0x80)
  Queue Min Size:          4096(0x1000)
  Queue Max Size:          131072(0x20000)
  Queue Type:              MULTI
  Node:                    1
  Device Type:             GPU
  Cache Info:
    L1:                      16(0x10) KB
  Chip ID:                 29538(0x7362)
  Cacheline Size:          64(0x40)
  Max Clock Freq. (MHz):   1150
  BDFID:                   240
  Internal Node ID:        1
  Compute Unit:            36
  SIMDs per CU:            4
  Shader Engines:          4
  Shader Arrs. per Eng.:   2

Hi @amrragab8080
Thanks for reaching out.
We are not officially supporting Navi series of cards at present.
We have plans to support in 2021 and we will update accordingly.
Thank you.

I also tried installing ROCm on Amazon G4ad instance following this guide.
The installation looks successful. However, /opt/rocm/bin/rocminfo returns:

ROCk module is NOT loaded, possibly no GPU devices
Unable to open /dev/kfd read-write: No such file or directory
ubuntu is member of video group
hsa api call failure at: /src/rocminfo/rocminfo.cc:1142
Call returned HSA_STATUS_ERROR_OUT_OF_RESOURCES: The runtime failed to allocate the necessary resources. This error may also occur when the core runtime library needs to spawn threads or create internal OS-specific events.

and /opt/rocm/bin/rocm-smi returns:

ERROR:root:ROCm SMI returned 8 (the expected value is 0)

I hope to see the official support AMD Radeon V520 GPUs and AWS G4ad instance soon!

Hi @ROCmSupport , do you have any estimates on when ROCm support for AWS G4ad will be available?

Hi @dsahni
Internal testing is happening and so can not comment on exact estimates right now.

Hello @ROCmSupport do you have nay update on support for AWS G4ad instances?? Thank you


Connections chapter: make it clear that the PokemonPreview component is available

Losely linked to https://github.com/learnrelay/pokedex/pull/9

Thanks! :)


fix ponytest doc

calling a unit test can cause errors, thus the partial signature

Good catch, thanks!


Allowed attributes not coming through in action text

Action Text with custom HTML, whilst allowed on the Rails side, isn't show said HTML in the editor.
Steps to reproduce

Have some custom JS to add a span with a style into Trix.
Tell Rails to allow both with an initializer saying;

ActionText::ContentHelper.allowed_attributes << 'style'
ActionText::ContentHelper.allowed_tags << 'span'

Once saved, the content will display correct on the frontend, but Rails Admin will not render the content. This leads to changes only working the once. Re-saving will overwrite custom code.

Initially I thought this was a JS issue, and my Trix parser was playing up, but when I inspect the source of the Rails Admin page, I can see the hidden field doesn't have the span tags.
Expected behaviour
Rails Admin should accept the same attributes that Rails accepts for ActionText::ContentHelper or have a clear way of setting it up twice.
ActionText of hello when spans are allowed should have hello in the hidden field within Rails Admin, right now it has hello

rails version: <IP_ADDRESS>
rails_admin version: 3.0.0.rc2

Turns out this was 100% JavaScript.
Setting the appropriate Trix.config.textAttributes fixed my issue.


Code Migration

Migrating code for opentelemetry receiver to new repo

@Chinwendu20 Thanks for the PR! Can you please squash all the commits?

Thanks. I am working on the unit test. I will squash and push everything at once


[3.0] Implement 'join' operation for optional types and default 'nil' to optional

The join operation for optional types is straightforward to define/implement:
join(T?, U)  ::= join(T, U)?
join(T, U?)  ::= join(T, U)?
join(T?, U?) ::= join(T, U)?

Also implement the defaulting rule for 'nil', which effectively turns it into '$T0'?. This allows us to, e.g., infer [String?] for the expressions
["hello", nil]

and
 true ? "hello" : nil

Resolves rdar://problem/16326914.

@rudkx please review

@swift-ci please test

@DougGregor LGTM!


chore(ci): Update CI steps to build the app

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/dockge/blob/master/CONTRIBUTING.md
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
Fixes #(issue)
Type of change
Please delete any options that are not relevant.

Other

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[x] I have commented my code, particularly in hard-to-understand areas
(including JSDoc for methods)
[x] My changes generate no new warnings
[x] My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)
Please do not use any external image service. Instead, just paste in or drag and drop the image here, and it will be uploaded automatically.

pnpm is fast tho 10s to build the app DAMN!

pnpm is fast tho 10s to build the app DAMN!

If you meant build:frontend, it is vite 5.0. It is just released a week ago.
https://vitejs.dev/blog/announcing-vite5
But pnpm install is amazing of course.

pnpm is fast tho 10s to build the app DAMN!

If you meant build:frontend, it is vite 5.0. It is just released a week ago.
https://vitejs.dev/blog/announcing-vite5
But pnpm install is amazing of course.

No doubt its amazing! 😄 can we update vite on uptime kuma aswell 👀

pnpm is fast tho 10s to build the app DAMN!

If you meant build:frontend, it is vite 5.0. It is just released a week ago.
https://vitejs.dev/blog/announcing-vite5
But pnpm install is amazing of course.

No doubt its amazing! 😄 can we update vite on uptime kuma aswell 👀

Unfortunately , npm is used for update/install for non-Docker users, and pnpm is not using the same lock file.
I don't think it is possible to ask everyone to switch to pnpm and don't want to confuse current non-Docker users, so it is not a good idea to switch to pnpm in the middle.


Add my Crypto-Bazaar App

Crypto-Bazaar
Please Add my project Crypto-Bazaar
This Shows the live data of all crypto-currencies

hey @sreelakshmig009
I have done with this project but I had done it with Reactjs..
I am unable to add this project in repo,can you please help me?


As a developer, I want to see the commands that are being run behind the scenes so that I can more easily debug when things go wrong

This tool is simply magical, which is awesome. This also makes it harder to debug when things go wrong or something unexpected happens. The goal is to provide some mechanism to see the underlying commands and API calls that this tool is abstracting to development and debugging purposes.
Not sure what the implementation is, whether that is a --verbose option to the individual commands, the same when starting the tool, some debug log capability, or something else. Again, the goal is to be able to more easily peek behind the curtain to see how the magic trick is being performed.

Holy shoot 20 days ago! I'm sorry I never saw this! Thank you so much for trying it out. This is great advice. I'm going to add logging levels to that a verbose options is possible.

Closing this for now until I can get some time to do this.


CTF features non-vanilla end-of-round replay for final flag capture in second half

The final flag capture in CTF causes a end-of-round replay which is not part of vanilla CTF.
To reproduce, simply capture all flags in CTF.
https://user-images.githubusercontent.com/40122905/161333705-8a6c5f0d-e8df-44e7-b601-a92730d08fad.mp4

So whilst this is definitely a bug, and needs to be fixed for epilogue reasons, should this be kept for when epilogue is disabled? I think it would be preferred over just a fade to black and the end of the match.

we really just need to disable round winning killreplay for the end of the game if epilogue is on, shouldn't need any ctf-specific logic

Will be fixed by #296

@GeckoEidechse i think this should be fine to close now.

@GeckoEidechse i think this should be fine to close now.

Yup missed it. Thanks for the heads-up <3


Create F.MapAwaitedReturnType fn

F.MapReturnType works correctly but it does not apply on async functions. I suggest creating a F.MapAwaitedReturnType function:
export type MapAwaitedReturnType<
  FN extends Fn | unset | _ = unset,
  ASYNC_FUNCTION extends ((...args: any[]) => Promise<any>) | _ | unset = unset,
> = PartialApply<MapAwaitedReturnTypeFn, [FN, ASYNC_FUNCTION]>

interface MapAwaitedReturnTypeFn extends Fn {
  return: this['args'] extends [infer FN extends Fn, infer ASYNC_FUNCTION]
    ? ASYNC_FUNCTION extends (...params: infer PARAMS) => Promise<infer RETURN_TYPE>
      ? (...params: PARAMS) => Promise<Call<FN, RETURN_TYPE>>
      : never
    : never
}

@gvergnaud I can create the PR if you want!

I think I'd prefer having a Promises module that has a map function:
type X = H.Functions.MapReturnType<H.Promises.Map<Fn>>

That seems more powerful because Promises.Map could be composed in other ways too. WDYT?

Yes, I agree 👍


Add test for RSP DMA from DMEM/IMEM with overflow

Existing tests covered only using DMEM/IMEM as destination. This test covers using them as source.

Thanks!


group promised data

Works like a charme! Thx

@nabinca :smile:

Hi @ericdouglas , @nabinca,
The reason I did not do this in the first place is that it's going to hit the performance once you put thousands of record in it.
It "build" the grouping based on the filtered result, so there's way less data to iterate over :)
Anyway thanks for the PR, I'll see if I can find another way to achieve this. In the mean time I'll refer this PR to anyone who have similar question / request.

Great @isteven ! Thanks for the feedback!

Covered in v3.0.0


Cocopods is on version 1.4.1

@mattdonnelly can you please update the pod?

Swifter's name conflicts with another Pod (also named Swifter) so we can't really publish it under CocoaPods Specs repo unless @mattdonnelly wants to change the name. You can specify the Pod's git manually which will fetch this repo. (The version should be 2.2.1)

@meteochu I am using it with the git link, but I think that we should fix the name issue 😄


RFC: Add support for creating a seperate connection per reader

Enables reuse of the same address, to allow a seperate connection per reader.
This change resulted in the single largest performance increase we saw in testing... the issue here is the dependent library (https://github.com/jbenet/go-reuseport) we're pulling in is a) a bit of a hack and b) pulls in a bunch of other dependencies. I haven't vetted the license of all these.
Alternatively we could also implement the reuse logic ourselves. Thoughts?

We have to vet the licences. It might be possible to use sourceclear in our builds to help with this.

@ash2k 1->2 workers saw the most significant increase. We saw that around 8 workers was most performant, running on c4.8xl with increased network buffers. Increasing the number of connections above this saw performance declines. This is one of the reasons for my PR that separates reading from parsing.
Colby might have the concrete numbers - I'll get him to ping them through.
We did indeed see that network queues were pinned to CPUs. Throughout testing we saw that 2 cores were handling reading, observed by the significant number of soft interrupts for only those 2 cores.
On ubuntu (14 and 16) we saw this have a more significant impact than on the AWS linux AMI.
(We saw that the source of the packets also had a very significant impact here - we suspected as a result of partitioning of packets across the queues)

Closing this out in favour of implementing reuse logic myself


Version mismatch error when invoking exposed function via wasmtime

Describe the Bug
I created a minimal wasm-bindgen library example as per the guide here. I can build the WASM library, but when trying to invoke an exposed function I encounter a version mismatch error.
Steps to Reproduce

Setup the project as per the guide:

Cargo.toml
[dependencies]
wasm-bindgen = "0.2.54"

[lib]
crate-type = ["cdylib"]

src/lib.rs
use wasm_bindgen::prelude::*;

#[wasm_bindgen]
pub fn greet(name: &str) -> String {
    format!("Hello, {}!", name)
}

Install wasm-bindgen-cli (0.2.54) and cargo-wasi (v0.1.17).
Build the project: cargo wasi build --release
Invoke the exposed function: wasmtime --invoke greet ./target/wasm32-wasi/release/wasi_lib.wasm "123"

Expected Behavior
I expected the wasmtime invocation to return "Hello, 123!".
Actual Behavior
Instead it returned the following
error: failed to process main module `./target/wasm32-wasi/release/wasi_lib.wasm`
    caused by: version mismatch in the bindings section: wasm file has `0.6.0` and this library supports `0.5.0`

Additional Context
When I leave out all wasm-bindgen stuff and instead expose native/#[no_mangle] functions (i.e. the previous example from the same guide), the build and invocation of exposed functions work as expected. This suggests that the culprit is wasm-bindgen. The minimal size of the failing example makes me think this is probably my machine instead, although I have no clue how to continue debugging this. Can you confirm whether this is a bug / help me understanding the issue if it isn't?
Other setup details if relevant:

rustup 1.20.2
cargo 1.39.0
rustc 1.39.0

Thanks for the report! I think this is not a wasm-bindgen mismatch but actually a wasm interface types custom version mismatch. Can you confirm that wasmtime itself is up-to-date?

/me facepalms
Should have remembered that wasmtime was a separate installation. wasmtime was at version 0.2.0. Things work as expected after updating. Thanks a bunch @alexcrichton!


Supporting AllowUnencrypted=true

Currently we need to disable message level encryption using AllowUnencrypted=false.   Are there any plans to implement that - or thoughts on how difficult it might be?

Would also be curious if this is planned.

What would it require to implement this?

It's probably because the server requires NTLM authentication - I have a PR in to implement this in this repo but this seems to be not very active
you could use https://github.com/CalypsoSys/bobwinrm
see https://github.com/CalypsoSys/bobwinrmntlm for usage

It's probably because the server requires NTLM authentication - I have a PR in to implement this in this repo but this seems to be not very active

Yes, I've lost track of incoming PRs this last years.
I'll go have a look to your PR and see if we can merge it.


RigidBody2D and moving platforms

Godot version:
Godot 3.2.2 (also tested in 3.2.1)
OS/device including version:
Windows 10 (Tested with both 2D physics engines)
Issue description:
I'm trying to make a 2D platformer that is very physics dependent (so I need to use a RigidBody2D), when the player jumps I use apply_central_impulse() and the player loses control until it hits the ground again.
All of this mentioned above works almost perfectly, but when the player interacts with other moving bodies (Kinematic or Rigid) this happens:
https://imgur.com/a/dHyNUxr
I'm using the template project so it's easy to reproduce, but you can go ahead and test with any changed configuration and it still gives somewhat the same behavior.
Steps to reproduce:
RigidBody2D on top of any moving Body.
Minimal reproduction project:
Not gonna upload the project since it's an official template,
Template name: 2D Physics Platformer Demo (RigidBody)
Version: 3.2.2

I would like to use this issue to raise up some questions about the topic above. (please mark as off-topic if this isn't the place to talk about this)
1. Is the problem with tilemaps liked above worth an issue (didn't post since it looks like is an known one)
2.  RigidBody2D in Kinematic Mode is bugged, or bad documented
The way i'm solving the issue above is, I'm instantiating and queue_free()ing a RigidBody2D and KinematicBody2D as needed. This would be an way easier task (and a cleaner code) if the Kinematic Mode worked as a KinematicBody2D. I don't know why thats not the case but if you search online there are a number of people noticing the same.
3.  Why isn't Godot focused on getting the 2D physics right?
Sorry if this is gonna sound a little harsh, but I love the engine and I can't understand why some of these issues aren't being considered priority.
Before going any further with this I would like to say that:

I have been playing around with game development for as little as six months now, but been developing software for longer, so take all of this with a grain of salt.
I know that Godot is an open-source project and things can go slower than if the project had a dedicated team, I'm just making an observation of what I'm seeing as milestones for 4.0, and what is being shared about the development of the engine.

Godot today in my opinion is one of best engine, if not the best, for 2D game development and I think it is the best by far if you look at pixel art games. But there are some issues with 2D that have been around for a long time. I'm not gonna go around and gather all of them to prove a point, I feel like you get what I mean.
If the Godot team wants to get more people using the engine I feel like improving 2D is the way, and from the Godot Community Poll looks like the engine is being used by the majority of the users to make 2D games. Other strong argument about this is how strong the engine already is in this field, I feel like with some improvements Godot could go far beyond other major engines and dominate the market in the future.
I'll be brief and say that I think 2D should get more attention from the core developers of Godot, specially the physics, again this is an opinion from someone that has been using this engine for not so long and my idea can be flawed.

Why isn't Godot focused on getting the 2D physics right?

Please understand that Godot is community-developed. There is no warranty provided whatsoever.

when the player interacts with other moving bodies (Kinematic or Rigid) this happens:

@luccapedrini I've looked at the linked videos and I can't see anything that is happening that shouldn't be happening. Please be more specific.

@madmiraal
The player's collision shape going inside the platform on the first gif.
And on the second one the player sliding as the platform move.
I don't think I can explain any better than the videos, for me it looks pretty obvious that there's something unexpected happening but I don't know where exactly the problem happens in this situation if that's what you're asking.

When the platform is moving left-to-right the RigidBody is relying on friction to provide a force; so some degree of sliding due to inertia is expected. Related to #16450
When the platform is moving up-and-down, there is a delay between the RigidBody responding to the movement of the platform, because the RigidBody updates and the platform updates happen at different times. So, I'm not sure whether or not this is fixable. Related to #27652


Fix SWITCHYARD-2841: Passthrough of SOAPFaultException on SOAPBinding

https://issues.jboss.org/browse/SWITCHYARD-2841

Can one of the admins verify this patch?

Hi, any comments for the pull request?

Can one of the admins verify this patch?

ok to test

Pushed.


Annoying notification sound and vibration while downloading + no view action

Hi there! Thank you for this amazing library!
I have found some small issues with a DefaultFetchNotificationManager.

Notification sound and vibration trigger so many times when a big file is downloading.
There is no action on a final complete notification. Is it possible to open a download folder or create a new intent action for this to catch it in a custom receiver?

I know I can always override this class and fix those problems but I think while this class is a part of this library it should be as perfect as the rest code ;D
And the last minor thing. When the downloading process is completed there is still progress bar in the notification. Probably it should be replaced with a text for example "Completed".

Thanks for using Fetch. The NotificationManager was added in the last version of Fetch so lots of feedback is welcome! Being that notifications are a very custom part of every app, It is strongly recommend to override the updateGroupSummaryNotification and updateNotification to customize the notification behavior as much as possible. Unfortunately there is no one size fit all solution here. The update methods passes in a cached notificationBuilder and DownloadNotification  objects. Use these to update the notificationBuilder fields accordingly.
Feedback is appropriated. I'll look into the progress bug soon.

I have the same issue, during downloading any file, notifications (beep sound) keep coming out on every update, here is how I use it, knowing that I'm not facing the issue on my emulator with Android 7.1 it only appears on my mobile device with Android 9.1:
        FetchConfiguration fetchConfiguration = new FetchConfiguration.Builder(getApplicationContext())
                .setDownloadConcurrentLimit(3)
                .enableRetryOnNetworkGain(true)
                .enableAutoStart(false)
                .setAutoRetryMaxAttempts(3)
                .enableLogging(true)
                .setNamespace(Constants.NAMESPACE)
                .setNotificationManager(new DefaultFetchNotificationManager(getApplicationContext()) {
                    @NotNull
                    @Override
                    public Fetch getFetchInstanceForNamespace(@NotNull String namespace) {
                        return fetch;
                    }

                    @Override
                    public boolean shouldCancelNotification(@NotNull DownloadNotification downloadNotification) {
                        if (downloadNotification.getStatus() == Status.CANCELLED) {
                            return true;
                        } else {
                            return super.shouldCancelNotification(downloadNotification);
                        }
                    }
                })
                .build();