Address PoU from sender and receiver

Is your feature request related to a problem? Please describe.
We currently have an extension on Organization and Endpoint to tag each with Purposes of Use. But we haven't said exactly what that means. Historically, exchanges like eHx and Carequality have seemed to use a PoU tag to mean "purposes which I will respond to", but there isn't data on whether clients are making use of this. And more recently, TEFCA has advanced the opposite use case, where an Organization will advertise PoUs in the directory meaning "purposes which I will send in requests". These values are used by the sending QHIN, which must enforce the declared constraints on outgoing requests.
When the TEFCA directory spec comes out, it may clarify to say an Org's PoUs are those it will request with, and an Endpoint's PoUs are those it will respond to.
Describe the solution you'd like
We could consider the possible TEFCA usage for mCSD. However, given that we have been favoring OrganizationAffiliation for details of electronic communication, it might make more sense to put the PoU extension on OrganizationAffiliations that have the code HIEInitiator instead of Organization, for outgoing PoUs.
Describe alternatives you've considered
None
Additional context
The HL7 US National Directory (current branch being worked on) is dealing with this as well, and is aware of mCSD profiling - in fact will be using some of it. But exactly where the dividing line is going to be is still TBD.

if there are multiple meanings, then there should be multiple extensions.

2 use cases we should try and incorporate:

As an information requestor (pull), what is the maximum set of purposes of use for which I might initiate a request?
As an information responder (pull), what is the maximum set of purposes of use for which I might be willing to respond.


Update to the latest version of React

Hi @one89 can i handle this :) ?

I would like to do it

Hey @henry-ns, thanks for your interest!
As you can see there's another PR for this issue, but if you want you can submit your PR upgrading not only react and react-dom but also other packages.
Thanks :)

I'll do it then


Change makeTensorFromDataId to makeTensorFromTensorInfo

Hi, I'm trying to build this library in Angular 13 and it seems is not working due to a recent refactoring of some TensorFlow methods in this pull request tensorflow/tfjs#6355.

thanks for letting me know, can you share all the packages version that you use in package.json?

{
  "name": "mind-ar-ts",
  "private": true,
  "version": "0.0.1",
  "description": "Mind AR JS but with TS integrations",
  "contributors": [
    "krsbx",
    "hiukim"
  ],
  "scripts": {
    "dev": "cross-env VITE_ENV=\"development\" vite",
    "build": "tsc && vite build",
    "build:prod": "cross-env TS_NODE_PROJECT=\"tsconfig-for-webpack-config.json\" NODE_ENV=\"production\" webpack --config webpack.config.prod.ts",
    "build:watch": "cross-env TS_NODE_PROJECT=\"tsconfig-for-webpack-config.json\" NODE_ENV=\"production\" webpack --watch --config webpack.config.dev.ts",
    "lint": "eslint --cache --fix src/*",
    "preview": "vite preview",
    "prepare": "husky install"
  },
  "devDependencies": {
    "@commitlint/cli": "^16.2.4",
    "@commitlint/config-conventional": "^16.2.4",
    "@types/aframe": "^1.2.0",
    "@types/webpack": "^5.28.0",
    "@typescript-eslint/eslint-plugin": "^5.21.0",
    "@typescript-eslint/parser": "^5.21.0",
    "css-loader": "^6.7.1",
    "eslint": "^8.14.0",
    "eslint-config-airbnb-base": "^15.0.0",
    "eslint-config-prettier": "^8.5.0",
    "eslint-plugin-import": "^2.26.0",
    "eslint-plugin-prettier": "^4.0.0",
    "html-loader": "^3.1.0",
    "husky": "^7.0.4",
    "node-sass": "^7.0.1",
    "prettier": "^2.6.2",
    "sass": "^1.51.0",
    "sass-loader": "^12.6.0",
    "style-loader": "^3.3.1",
    "terser-webpack-plugin": "^5.3.1",
    "ts-loader": "^9.3.0",
    "ts-node": "^10.7.0",
    "tsconfig-paths": "^4.0.0",
    "typescript": "^4.5.4",
    "vite": "^2.9.5",
    "webpack": "^5.72.0",
    "webpack-cli": "^4.9.2",
    "worker-loader": "^3.0.8"
  },
  "dependencies": {
    "@mediapipe/face_mesh": "^0.4.1633559619",
    "@msgpack/msgpack": "^2.7.2",
    "@techstark/opencv-js": "^4.5.5-release.2",
    "@tensorflow/tfjs": "^3.16.0",
    "@tensorflow/tfjs-backend-webgl": "^3.16.0",
    "aframe": "^1.3.0",
    "cross-env": "^7.0.3",
    "ml-matrix": "^6.10.0",
    "stats-js": "^1.0.1",
    "svd-js": "^1.1.1",
    "three": "^0.140.0",
    "tinyqueue": "^2.0.3"
  },
  "lint-staged": {
    "*.{ts,tsx}": "eslint --cache --fix",
    "*.{ts,tsx,scss,md}": "prettier --write"
  },
  "license": "MIT"
}

I'm trying to send you a pr with the fix but it gives me access denied.

I can't push my branch to this repo, therefore I can't make the PR.

just fork the repository and push the changes in your forked repos and make sure you push it not in the main branch, and after that i'm sure you can make a PR by using the forking repo

@demisquare regarding this PR, can you make sure it has the correct types?

Done

@demisquare please resolve the conflict in the PR, then i'll merge it


Saving & Resuming Gameplay

I'm not too great at HTML or JS for that case, but I have an idea that I think would be quite nice for the project... Somehow, a way to save. Maybe you could sign-in with GitHub or Google or maybe just an old-fashion game save pin code, but I'd like to be able to save my game for when I go out instead of having to leave my computer on just to keep my game saved. Any possible way or anyone looking to contribute?
Thanks.
Also, on an unrelated note, how come the "too close circles" appear even if the two planes are 500 feet in altitude difference? Kind of annoyed me when I thought they were going to crash, but they were perfectly spaced as I had them.

The idea of saving has been sparking up a lot lately, I think this will be one of the next huge things that will be implemented...
Red circles don't mean you're about to get a collision. They mark planes that are flying closer then the minimum separation rules allow them to, which deducts points of your score. White circles warn you that the minimum speration  is about to be reached, yet don't deduct points.
Minimum speration is 1000ft above an below and in a 3 mile radius.

Ah, thanks @Maverick283. I'm kinda new to the game, hah. This again is a little off topic, but how high can the planes fly, is there even a limit?

They can fly pretty high, i'm not aware of any limit, but they will leave the radar coverage once they are above FL100. I believe they disappear from the scope at FL110, but I'm not sure, I'd have to try. Or maybe @glangford can tell you more.

As @Maverick283 says - they will leave radar coverage once they are above FL100. Outside of the control area, the simulation will have departing aircraft climb higher than that.
You can clear a departure higher than FL100, but I don't recommend it! See #185.
(The game save idea is a good one, but I don't have time to contribute to it. Perhaps another approach would be to add bounds to a game session so that it isn't infinite in length and ends after a set period of time - like a controller shift change).

Let's discuss the idea of saving though, so someone has an base to build a program on:
What has to be saved?

Of course: The score!
The positions of the aircraft (Altitude, Coordinates, Heading, Speed, Destination)
Clearances given to the aircraft (fix list, altitude assigned, speed assigned, departure clearances)
Waiting departures
Current wind ( + at some point maybe the weather)

How can it be saved?
I'm no good when it comes to cookies, but it could be either saved that way, or by dumping a downloadable file to the user, that then can be reopened next time he plays.
If that is done, a player might be able to hack/overload etc the whole thing by modifying the save file, maybe there is something that can be done to prevent this.
How can it be reloaded?
Well, from my point of view it'd be a "spawn all aircrafts, then reassign the clearances and let the code do the magic" kind of thing, i believe this should be no huge deal.

For saving we have options like:

Cookies: limited storage (4k?), everything has to be stored in a string complying the format (should not contain cookie separators and stuff), very wide support
Server side: not an option as the server is currently serving static only (maybe 3rd party but who wants that?)
Localstorage: stores like an object, wide support though its newer technology than cookies. Bigger storage (5MB?)

Also there is the url anchor (#) storage method, easy to share progress with other players.

Localstorage would (I think) be the main option to consider. It is already used in the current code to store the last used airport, whether speech is enabled, etc.

A rough calculation suggests save files would be in the 20-100kB range assuming a straight forward JSON representation.  With that size I think LocalStorage is the only real option at this point.
The other thing which needs to be saved is the state of the arrival and departure generators.

I actually tried a naive saving, but no, objects are not serializeable. :)

I usually use a recursive structure (i.e. localStorage['state'] = game.save() where game.save() serializes its data in a way that it can be recovered entirely with game.restore(); any objects to be saved will have similar functions.

As suggested by @culebron, it would be good to:

share saved data (I guess, get the saved state from local storage and encode in a URL over bit.ly)

The ATC repository is being migrated to it's new home at https://github.com/openscope/openscope,
and thus, all issues are being closed. If this is still an issue with the latest version of the sim
(accessible at http://www.openscope.co), or is a feature you still think we are lacking,
please reopen the issue at the new repo.
Please note that the vast majority of these issues have been copied to the new repository, or else are covered by other issues created there. See the below screenshot for what it looks like when your issue is known in the new repo:

Thank you!
Closing this issue.


add restore_rate param to callAPI options

... to address issue #101
Example usage:
          let res: GetOrderItemsResponse = await sellingPartner.callAPI({
            operation: "orders.getOrderItems",
            path: {
              orderId: order.AmazonOrderId,
            },
            options: {
              restore_rate: 5,
            },
          });

Debug output:

Thanks for the PR, just changed it slightly to support floating values and usage together with api_path


graph execution error

I'm facing an issue that says "graph execution error" tried to solve it but couldn't.

Hi @Ataullah128,
We see that the issue template has not been filled, could you please do so as it helps us analyze the issue [tf version, steps followed before you ran into this error or stand alone code/colab gist to reproduce the issue faced. Thanks!


Add note about App Service storage limit

[Enter feedback here]
In the section of App Service storage limit,
It is better to add note about file system quota for App Service hosted apps is determined by the aggregate of App Service plans created in this region and resource group.

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

ID: 36f49a06-90b1-cbe1-3853-2c38277b79f8
Version Independent ID: 02ff6f6c-3ef6-b48f-060d-b290419e0aba
Content: Azure subscription limits and quotas - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/azure-subscription-service-limits.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@georgeOsdDev Thanks a lot for sharing the feedback and helping us improve Azure documentation. Our team will further review the feedback and update the document accordingly.

#assign:davidsmatlak
I'll review this documentation request.

I submitted the fix for this issue and when it's merged, this issue will automatically close. After the merge, the updated document will be online within 24 hours.

@georgeOsdDev thanks for your feedback. I added the note to superscript 5 for App Service limits storage and the update should be online within 24-hours.


Update for Python 3.12 & 3.13

Update Pystache project to include compatibility with Python 3.12 and 3.13. The changes have been tested against these new versions.

Resolves #32


Foundry throw an error with the smallTime module

When I added Smalltime module i got an error in the console. You can see it in the video below.
https://github.com/unsoluble/smalltime/assets/91192276/3d49f9bb-4d35-47e4-8f7b-6ad250ffd77b
The error text:
Error: Error thrown in hooked function '' for hook 'canvasInit'. Cannot read properties of undefined (reading 'dispatchEvent')
at Hooks.onError (foundry.js:630:24)
at #call (foundry.js:612:36)
at Hooks.callAll (foundry.js:565:17)
at #draw (foundry.js:24687:11)
at async Canvas.draw (foundry.js:24615:5)
at async Scene.view (foundry.js:21117:7)
at async Game.initializeCanvas (foundry.js:9166:20)
at async Game.setupGame (foundry.js:8993:5)
at async Game._initializeGameView (foundry.js:10306:5)
at async Game.initialize (foundry.js:8896:5)
If it is an feature, not the bug, then ok. Just close the issue)

Thanks for flagging this! Fixed and cleaned up some deprecation warnings in 1.20.0


macOS missing in lwjgl natives

Question
powerpc both 32 and 64 bit natives do not appear here. mac's had power pc natives like in snow leopard and are still capable of running lwjgl perfectly fine. Was support dropped and would you guys be able to compile it anyways for those CPU architectures or does manual work have to be done?

ok so x86 for mac was < 2010 I was wrong
ok so powerpc32 / powerpc64 was also < 2010 newer snow leopard pc's was intel enforced so I was also wrong on this. closing the issue


Schema validations

Description of changes
This PR adds below mentioned schema validations:

XX and YY cannot refer to each other via @hasOne or @hasMany when DataStore is in use. Use @belongsTo instead
Schema should not use directives from an older version of the GraphQL Transformer when the version used is 2.
Schema should not use directives from a newer version of the GraphQL Transformer when the version used is 1.

CDK / CloudFormation Parameters Changed
Nona
Issue #, if available
N/A
Description of how you validated changes
Added unit tests
Checklist

[x] PR description include
[x] yarn test passes
[x] Tests are changed or added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1379 (7332453) into main (b8a5b9b) will increase coverage by 0.14%.
The diff coverage is 93.80%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1379      +/-   ##
==========================================
+ Coverage   61.88%   62.02%   +0.14%
==========================================
  Files         332      335       +3
  Lines       20353    20447      +94
  Branches     4831     4849      +18
==========================================
+ Hits        12595    12682      +87
  Misses       7027     7027
- Partials      731      738       +7

Impacted Files
Coverage Δ

...s/use-directives-from-older-transformer-version.ts
90.00% <90.00%> (ø)

...c/validators/use-belongsto-when-datastore-inuse.ts
90.62% <90.62%> (ø)

...s/use-directives-from-newer-transformer-version.ts
96.55% <96.55%> (ø)

packages/amplify-schema-validator/src/index.ts
100.00% <100.00%> (ø)

...rc/validators/auth-must-be-annotated-with-model.ts
100.00% <100.00%> (ø)

...elongs-to-fields-match-related-type-primary-key.ts
90.47% <100.00%> (ø)

...alidators/correct-type-in-many-to-many-relation.ts
95.12% <100.00%> (ø)

...a-validator/src/validators/enum-is-defined-once.ts
100.00% <100.00%> (ø)

...-validator/src/validators/field-is-defined-once.ts
100.00% <100.00%> (ø)

...tor/src/validators/fields-match-in-parent-model.ts
95.65% <100.00%> (ø)

... and 11 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


⚠ Bofrost.fr has degraded performance

In 19253ee, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1098 ms

Resolved: Bofrost.fr performance has improved in fb90932 after 9 minutes.


Value checking and checking whether types are normalized

There are several problems in Language.PlutusCore.Check.Normal

An infinite loop:

checkTerm :: (AsNormalizationError e TyName Name a, MonadError e m) => Term TyName Name a -> m ()
checkTerm p = void $ throwingEither _NormalizationError $ checkTerm p

Why does checking that all types appearing in a term are normalized require return the term back?

checkT :: Term tyname name a -> Either (NormalizationError tyname name a) (Term tyname name a)

It can be just
checkT :: Term tyname name a -> Either (NormalizationError tyname name a) ()

Same for termValue, typeValue, etc.

NormalizationError is misleading: we do not normalize there, only check whether things are already normalized.

We have the problem that eqInteger {s} x y is considered a value. This is a long-standing problem which we can't solve right now, because we're not yet settled on whether builtins are saturated or not or maybe we want to throw static builtins away entirely (which is another argument for doing that).

Checking whether a term is a value also entails checking that all types in it are normalized. Those should be two distinct checks, because sometimes we want the former and not the latter and vice versa.

Names like checkT and preCheck are not helpful.

Checking whether a term is a value is also defined in View.hs. We need to merge the two versions (possibly by just throwing away isValue from View.hs).

@effectfully Can we close this?  The file looks quite different now.

It could still do with a tidy, it's weirdly structured.


r11.0.0rc9 - Maas Local Checks - error Authorization failed: The resource could not be found. (HTTP 404)

Lots of checks failing with the following error:
error Authorization failed: The resource could not be found. (HTTP 404)

Running the checks manually shows the same error:
# python nova_api_local_check.py <IP_ADDRESS>
status error Authorization failed: The resource could not be found. (HTTP 404) (Request-ID: req-7585071e-73b0-41a3-a3ff-ef4db9c1263e)

And using the openrc in the utility container, running any keystone commands returns the following:
# source openrc
# keystone --debug catalog
/usr/local/lib/python2.7/dist-packages/keystoneclient/shell.py:65: DeprecationWarning: The keystone CLI is deprecated in favor of python-openstackclient. For a Python library, continue using python-keystoneclient.
  'python-keystoneclient.', DeprecationWarning)
WARNING: unsupported identity-api-version 3, falling back to 2.0
DEBUG:keystoneclient.auth.identity.v2:Making authentication request to http://<IP_ADDRESS>:5000/v3/tokens
INFO:requests.packages.urllib3.connectionpool:Starting new HTTP connection (1): <IP_ADDRESS>
DEBUG:requests.packages.urllib3.connectionpool:"POST /v3/tokens HTTP/1.1" 404 93
DEBUG:keystoneclient.session:Request returned failure status: 404
Authorization Failed: The resource could not be found. (HTTP 404) (Request-ID: req-33e2ab0e-9b06-45b4-b3e1-beb5a848f34e)

Interestingly, the openstack client seems to be able to communicate with keystone:
# source openrc
# openstack catalog list
+----------+----------------+-------------------------------------------------------------------------------+
| Name     | Type           | Endpoints                                                                     |
+----------+----------------+-------------------------------------------------------------------------------+
...snip...

Was this with 11.0.0rc9?

One other thing...the catalog seems to have a mix of v2 / v3 endpoints for keystone...
| keystone | identity       | RegionOne                                                                     |
|          |                |   public: http://<IP_ADDRESS>:5000/v2.0                                     |
|          |                | RegionOne                                                                     |
|          |                |   admin: http://<IP_ADDRESS>:35357/v3                                         |
|          |                | RegionOne                                                                     |
|          |                |   internal: http://<IP_ADDRESS>:5000/v2.0                                     |

Yes, 11.0.0rc9 this yesterday and today.

This is related to https://review.openstack.org/#/c/205192/2

keystone client via CLI cannot consume a service catalogue created via the v3 API. The openstack CLI can and should be used. The mix of endpoint versions is intentional - v3 for the admin endpoint, and v2 for the internal/public endpoints.

Should these be the same?
openrc:
export OS_AUTH_URL=http://<IP_ADDRESS>:5000/v3

openrc-maas
export OS_AUTH_URL=http://<IP_ADDRESS>:5000/v2.0

I'm not super familiar with the Keystone changes that have been going on to get federation working, but I would assume that both openrc and openrc-maas should both use the same OS_AUTH_URL.
What I find strange is that os-ansible-deployment in Kilo is setting v3 for keystone_service_internalurl (source), which is evidently what our rpc_maas task is using (source).

Problem is openrc-maas rc file. I telplated from openrc, and early indications are that MaaS checks are starting to work...

@nrb look again...
keystone_service_internalurl vs keystone_service_internalurl_v3.

Right, it is openrc-maas. Trying to figure out the best way to get the right values pulled over from os-ansible-deployment, since it looks like there are some overrides happening between the role defaults and group vars.

@byronmccollum Right, but we were assigning keystone_service_internalurl_v3's value to keystone_service_internalurl.

v2 endpoints for internal/public in the final delivery https://github.com/stackforge/os-ansible deployment/blob/master/playbooks/inventory/group_vars/hosts.yml#L146-L158 - this is busy being worked on to set them to v3 in https://review.openstack.org/205192 but that's failing tempest and thus still WIP

I would like to get some evaluation from @tbyrom138 @jacobwagner around this so we can determine the scope and severity of this based on what they're seeing in QE.

just to be clear, changing v2.0 to v3 seems to fix it (first simple test)

Because if the tight 11.0 timeline, and that the work around is relatively easy, I'm in favor of making this a 11.0.1 timeline fix, and making sure we document this as a known issue w/ the workaround asap. This will also give us a sprint to keep an eye on the other v2 vs v3 mini discussions , and see if anything changes there in upstream OSAD Kilo.
@prometheanfire can you open a docs bug with the relevant issue, error, workaround?

@claco +1 on pushing this to 11.0.1 if it's not high priority.
cc: @nrb

Setting the milestone and priority appropriately, then.

@nrb, @claco, @b3rnard0
11.0.1, or 11.1.0? I'm seeing references to both.

The least complex answer I can give is 11.0.1. Revision released will be out every two weeks post 11.0.0 being cut. But 11.1.x rc start dropping soon as well.

repoening for #296

PRs for Kilo and master merged, closing this issue.


Tick private option by default and make the sidebar resizable

Environment info

OS: MacOSX
App Version: Lepton v1.8.1

Description of the problem / feature request / question:

Option to tick 'private' option by default.
Make the sidebar and it's separate components resizable. I would like to be able to extend the height of the tags and reduce the height of the pinned items.

Option to tick 'private' option by default.

We create a ~/.leptonrc with snippet:newSnippetPrivate enabled.

Make the sidebar and it's separate components resizable. I would like to be able to extend the height of the tags and reduce the height of the pinned items.

The leftmost sidebar itself is not resizable. But we can "resize" the "LAUNGUAGES", "PINNED" and "TAGS" sections by clicking on the section titles.

Thanks a lot!


Create Website for PermIndex

Hello,
this PR is all about adding a website to PermIndex.
What's inside?

A brand new fancy website
Gatsby

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


Updated Debug-Container Host to include Network Troubleshooting

Includes checks for

At least one local container network
No more than one NAT network
Host IP is not in the range of NAT internal prefix

Tried this on a new Win2016 machine. So these two network IP addresses aren't conflicting?
PS C:\Windows\system32> ipconfig

Windows IP Configuration

Ethernet adapter vEthernet (HNS Internal NIC):

   Connection-specific DNS Suffix  .  :

Ethernet adapter Ethernet0 2:

   Connection-specific DNS Suffix  .  : Media disconnected
   Connection-specific DNS Suffix  .  : Media disconnected
   Connection-specific DNS Suffix  . :
PS C:\Windows\system32> wget https://raw.githubusercontent.com/Microsoft/Virtualization-Documentation/19c3e9ba5a9426dd8987de6a52c465992e04544b
/windows-server-container-tools/Debug-ContainerHost/Debug-ContainerHost.ps1 -UseBasicParsing | iex
Checking for common problems
Describing Windows Version and Prerequisites
 [+] Is Windows 10 Anniversary Update or Windows Server 2016 120ms
 [+] Has KB3192366, KB3194496, or later installed if running Windows build 14393 28ms
 [+] Is not a build with blocking issues 35ms
Describing Docker is installed
 [+] A Docker service is installed - 'Docker' or 'com.Docker.Service'  33ms
 [+] Service is running 103ms
 [+] Docker.exe is in path 3.12s
Describing User has permissions to use Docker daemon
 [+] docker.exe should not return access denied 31ms
Describing Windows container settings are correct
 [+] Do not have DisableVSmbOplock set to 1 32ms
 [+] Do not have zz values set 31ms
Describing The right container base images are installed
 [+] At least one of 'microsoft/windowsservercore' or 'microsoft/nanoserver' should be installed 174ms
Describing Container network is created
 [+] At least one local container network is available 203ms
 [+] No more than one NAT network 26ms
 [+] NAT Network's internal prefix does not overlap with external IP' 36ms
Showing output from: docker info
...

I wonder why docker chose the <IP_ADDRESS> for the HNS internal NIC. Isn't it normally 172.16.x.x?

OK, answered myself. Haven't recognized the netmask.

same error here with Null instead of $null or "BeNullorEmpty" #449
additionally:
this shows falsely "running" if $services is $null
 It "Service is running" {
       foreach ($service in $services)
        {
           $service.Status | Should Be Running
        }
    }

Thanks for doing this! Looks good to me, just 2 minor script issues.

does the check point

NAT Network's internal prefix does not overlap with external IP'
regard to the issue with VMs in Amazons EC2?

currently trying to get  a Nano with docker to run there, but it shows the same symptons as with missing hotfixes, but their images already install with UBR 206

Thanks for the updates. Service detection looks ok - tested on both manual Docker and Docker for Windows setups.
Interesting issue with network detection though. I do have overlapping IP ranges due to a VPN connection, but the output isn't decipherable:
PS C:\WINDOWS\system32> wget https://raw.githubusercontent.com/Microsoft/Virtualization-Documentation/7e4310cb93327369a2
6a1775fa4113c6e2c35321/windows-server-container-tools/Debug-ContainerHost/Debug-ContainerHost.ps1 | iex
Checking for common problems
...
Describing Docker is installed
 [+] A Docker service is installed - 'Docker' or 'com.Docker.Service'  128ms
 [+] Service is running 60ms
 [+] Docker.exe is in path 2.54s
...
Describing Container network is created
 [+] At least one local container network is available 2.37s
 [+] No more than one NAT network 50ms
 [-] NAT Network's internal prefix does not overlap with external IP' 144ms
   Expected: value was {101011000001}, but should not have been the same
   149:           $BinaryIP.Substring(0, $Length) | Should Not Be $BinaryIPSubnet.Substring(0, $Length)

   at <ScriptBlock>, <No file>: line 149
   at <ScriptBlock>, <No file>: line 144

Showing output from: docker info
...
Showing output from: docker network ls
NETWORK ID          NAME                DRIVER              SCOPE
bdad3ffc6363        nat                 nat                 local
c37b258a5907        none                null                local

Warnings & errors from the last 24 hours
Logs saved to C:\WINDOWS\system32\logs_20161028-131510.csv

The script does pass with my VPN disconnected, so no overlapping IP subnet. Pass/fail results seem ok, they're just hard to read since it shows some binary instead of a formatted IP.

wouldn't DSC the right(or best) way here to ensure that Docker prerequisites are installed? There are already  some scripts at different repos out there (google docker dsc).

I agree that DSC is great for deploying Docker (or anything else for that matter), but it's not suitable for a troubleshooting script. If there is already another DSC policy applied, then you would have to modify it instead of simply running the PowerShell script. It also doesn't have any provisions for things such as dumping logs to CSV files.

I'll go ahead and merge. If someone has a better way to do the math to make the output easier to read - PR's are welcome :)

I mostly agree, though I see many overlaps as (afair) having the test-dscconfiguration part doing the debugging missing parts and why wouldn't it be able to output some .csv files with logs?


Editing categories: Window after clicking on the arrow gets hidden when there are categories with only few labels

See screenshot

Note: This is not an urgent bug. Mainly because the user can manually enlarge the "Create new annotations" window. But anyhow it should be fixed some time...

The sad truth is also that this isn't an easy thing to fix. You would think that it is, but ti's complicated by the convoluted structure of the "structured annotation creation component," and even the naive and hacky ways of working around that are squashed by us using an ancient version of the "UI framework" (talking about Bootstrap) we are using. :disappointed:

Truly sad but not that important. Here some ideas for workaround solutions to gain more space for the box containing the categories with its labels:
a) The default hight for free text annotations could be made a bit smaller.
b) In the layout preset "Standard" the window "Create new annotations" could be made higher than the window "Annotation list". For example: "Create new annotations" = 3/5 of max. hight; "Annotation list" = 2/5 of max. hight
What do you think of these possible workarounds?

Maybe a CSS cheat can help with position:fixed; or other more modern CSS approaches (?):

Challenges would then be:

Does it work in all browsers?
Position needs to be calculated from the origin element (hardcoded for screenshot)
On resizing the window, it should be either repositioned -or- closed

I don't want to sound mean or anything, but I would really hesitate to add a lot of complexity to fix this. If there is a simple CSS one-liner, sure, but I don't think there is. Adding JavaScript code to dynamically reposition this window, etc. when this functionality already exists in newer versions of Bootstrap seems crazy to me. We should update our dependencies, not haphazardly duplicate their functionality. :thinking: It's exactly decisions like this that brought about the current state of the code base; choosing quick fixes over long term maintainability.
As for the workarounds suggested by David (sorry, somehow I'm only seeing this now): I have nothing against that. It's easy, and it's also not like I put a lot of thought into that Standard layout. x)


how to speed up training process

Thanks you for giving out the source code. I train my own data with the code but the GPU utility is always stayed at 0%. I have tested the code and found that the I/O speed is fast , so I do not know why the speed is extremely slow , could you please help me?

Hello,i meet the problem:ImportError: torch.utils.ffi is deprecated. Please use cpp extensions instead.
Can you help me?


Cannot install on Ubuntu 16.04 LTS

I am trying to install Shapely on Ubuntu 16.04 LTS and I have geos installed and I am getting these errors when I type python3.5 setup.py install. Shapely version is 1.6.2 and the geos library is installed under /usr/local/lib and the LD_LIBRARY_PATH does have this this value as part of it.
cannot open shared object from 'geos-config --clibs': '-L/usr/local/lib -lgeos_c'
there were 2 guess(es) for this path:
/usr/local/lib/libgeos_c.so
/usr/local/lib/libgeos_c.so.1
Failed CDLL(libgeos_c.so.1)
Failed CDLL(libgeos_c.so.1)
Failed CDLL(libgeos_c.so)
Traceback (most recent call last):
File "setup.py", line 80, in
from shapely._buildcfg import geos_version_string, geos_version,
File "/home/winash12/libPython/Shapely-master/shapely/_buildcfg.py", line 167, in
fallbacks=['libgeos_c.so.1', 'libgeos_c.so'])
File "/home/winash12/libPython/Shapely-master/shapely/_buildcfg.py", line 161, in load_dll
libname, fallbacks or []))
OSError: Could not find library geos_c or load any of its variants ['libgeos_c.so.1', 'libgeos_c.so
Please note
If you are reporting an installation or module import issue, please note that
this project only accepts reports about problems with packages downloaded from
the Python Package Index. Conda users should take issues to one of the
following trackers:

https://github.com/ContinuumIO/anaconda-issues/issues
https://github.com/conda-forge/shapely-feedstock

Expected behavior and actual behavior.
(For example: the area of my geometry is 1.417 when it should be 1.414.)
Steps to reproduce the problem.
(For example, a script with required data)
Operating system
(For example, Mac OS X 10.12.3)
Shapely version and provenance
(For example, 1.6b4 installed from PyPI using pip)

I was able to fix this by running this command sudo apt-get install libgeos-dev

I can confirm that shapely fails to install on a fresh ubuntu 16.04 server without the libgeos package installed.
# pip3 install --no-binary shapely shapely
Collecting shapely
  Downloading Shapely-1.6.2.post1.tar.gz (221kB)
    100% |################################| 225kB 1.5MB/s
    Complete output from command python setup.py egg_info:
    Failed `CDLL(libgeos_c.so.1)`
    Failed `CDLL(libgeos_c.so)`
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-bl3nvdpk/shapely/setup.py", line 80, in <module>
        from shapely._buildcfg import geos_version_string, geos_version, \
      File "/tmp/pip-build-bl3nvdpk/shapely/shapely/_buildcfg.py", line 167, in <module>
        fallbacks=['libgeos_c.so.1', 'libgeos_c.so'])
      File "/tmp/pip-build-bl3nvdpk/shapely/shapely/_buildcfg.py", line 161, in load_dll
        libname, fallbacks or []))
    OSError: Could not find library geos_c or load any of its variants ['libgeos_c.so.1', 'libgeos_c.so']

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-bl3nvdpk/shapely/

This is to be expected – shapely's readme and install docs say that you need the GEOS lib before installation – though the error message isn't great.
The geos dev package isn't strictly required, installing only libgeos-c1 is sufficient. Getting libgeos-dev would be best.
root@0ca930cfc2c4:/# apt-get install libgeos-c1v5
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following additional packages will be installed:
  libgeos-3.5.0
The following NEW packages will be installed:
  libgeos-3.5.0 libgeos-c1v5
0 upgraded, 2 newly installed, 0 to remove and 4 not upgraded.
Need to get 530 kB of archives.
After this operation, 2258 kB of additional disk space will be used.
Do you want to continue? [Y/n]
Get:1 http://archive.ubuntu.com/ubuntu xenial/universe amd64 libgeos-3.5.0 amd64 3.5.0-1ubuntu2 [477 kB]
Get:2 http://archive.ubuntu.com/ubuntu xenial/universe amd64 libgeos-c1v5 amd64 3.5.0-1ubuntu2 [52.4 kB]
Fetched 530 kB in 1s (464 kB/s)
debconf: delaying package configuration, since apt-utils is not installed
Selecting previously unselected package libgeos-3.5.0.
(Reading database ... 13952 files and directories currently installed.)
Preparing to unpack .../libgeos-3.5.0_3.5.0-1ubuntu2_amd64.deb ...
Unpacking libgeos-3.5.0 (3.5.0-1ubuntu2) ...
Selecting previously unselected package libgeos-c1v5.
Preparing to unpack .../libgeos-c1v5_3.5.0-1ubuntu2_amd64.deb ...
Unpacking libgeos-c1v5 (3.5.0-1ubuntu2) ...
Processing triggers for libc-bin (2.23-0ubuntu9) ...
Setting up libgeos-3.5.0 (3.5.0-1ubuntu2) ...
Setting up libgeos-c1v5 (3.5.0-1ubuntu2) ...
Processing triggers for libc-bin (2.23-0ubuntu9) ...

After this you can install shapely from the source distribution on PyPI.
root@0ca930cfc2c4:/# pip3 install --no-binary shapely shapely
Collecting shapely
  Using cached Shapely-1.6.2.post1.tar.gz
Skipping bdist_wheel for shapely, due to binaries being disabled for it.
Installing collected packages: shapely
  Running setup.py install for shapely ... done
Successfully installed shapely-1.6.2.post1
You are using pip version 8.1.1, however version 9.0.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
root@0ca930cfc2c4:/# python
bash: python: command not found
root@0ca930cfc2c4:/# python3
Python 3.5.2 (default, Sep 14 2017, 22:51:06)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import shapely
>>> shapely.__version__
'1.6.2.post1'


Metadata labels mixed across releases deployed concurrently

Output of helm version:
Client: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.0", GitCommit:"0ed33881dc4355495f623c6f22e7dd0b7632b7c0", GitTreeState:"clean", BuildDate:"2018-09-27T17:05:32Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.11-eks", GitCommit:"6bf27214b7e3e1e47dce27dcbd73ee1b27adadd0", GitTreeState:"clean", BuildDate:"2018-12-04T13:33:10Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
EKS (eks.3) Ireland
Hi! We are deploying Helm charts through Landscaper into our EKS cluster and this morning, like every morning, we spun up a test environment and deployed ~30 releases and we have ended up in a really strange situation were one of the releases (NodeJs) had metadata labels from another release (Java)... but only the keys
  labels:
    app: my-app-name
    product: my-product
    stack: my-stack
    owner: my-team
    parentVersion:
    springBootVersion:
    chart: my-chart-1.0
    release: "my-release-name"
    revision: "1"
    heritage: Tiller

this two labels
    parentVersion:
    springBootVersion:

are only present in our Java apps and there is no reference to them on the NodeJs chart and are present and empty on every single resource of this release.
We populate this metadata across all the release resources using a template on a file called _common_helpers.tpl
{{ define "common.labels" }}
app: {{ template "helm.name" . }}
product: {{ required "meta.product must be specified in values" .Values.meta.product }}
stack: {{ required "meta.stack must be specified in values" .Values.meta.stack }}
owner: {{ required "meta.owner must be specified in values" .Values.meta.owner }}
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
release: {{ .Release.Name | quote }}
revision: {{ .Release.Revision | quote }}
heritage: {{ .Release.Service }}
{{ end }}

the java apps also have a file called _common_helpers.tpl with the same template name but different content
{{ define "common.labels" }}
app: {{ template "helm.name" . }}
product: {{ required "meta.product must be specified in values" .Values.meta.product }}
stack: {{ required "meta.stack must be specified in values" .Values.meta.stack }}
owner: {{ required "meta.owner must be specified in values" .Values.meta.owner }}
parentVersion: {{ .Values.meta.parentVersion }}
springBootVersion: {{ .Values.meta.springBootVersion }}
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
release: {{ .Release.Name | quote }}
revision: {{ .Release.Revision | quote }}
heritage: {{ .Release.Service }}
{{ end }}

I was wondering if anybody has experienced something similar. I was trying to track the issue down but maybe someone can point me to the right place? I'm just a GoLang beginner
Thanks!

I suggest you open this issue with landscaper first

@carlosjgp
I have not worked with landscaper in the past. We did write a similar tool which deploys releases in parallel using a single Tiller pod, and things did not get mixed up.
What I would do to debug, is add another step to my pipeline: helm upgrade ... --dry-run --debug
Add it right before the actual upgrade (or install) command to see what is going on.
That may help isolate the problem's origin.
Hope this helps!

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!


Support for unified diff

I don't see an option for unified diff formatting on http://prismjs.com/download.html... this would be a very handy language to implement. It seems like this might be a pretty easy change? I don't know the Prism API well, but I suspect that half of the change would look like:
Prism.languages.diff = {
	'addition': /+.*/,
	'deletion': /-.*/
}

but I'm not sure how to turn 'addition' and 'deletion' into appropriate class names... these aren't normal keywords, comments, etc. Does the Prism CSS have classes for things like addition and deletion?

Hi, thanks for reporting. It should be fixed now. Just use the git component for diffs.
Note that there is also a pending PR (#450) to make a proper component for diffs. It might be merged some day.


Request to add role attributes to identities based on identity document

It will be very helpful if identities can get dynamic role attributes based on the identity document ( x509 certificate ). This will make it very easy to maintain policies in the environment with high churn rate for identities.

needs more discussion

Identities enrolled with certificates from Ziti's built-in edge signer CA have administratively assigned role attributes and long-lived certificates.
Additionally, Ziti has some features that apply to workloads and may be useful for the case you have in mind, @abvaidya.
I'll first explain how these features might work for the case you described where your identity document determines the role attributes of the Ziti identity, then I'll explain more broadly the whole set of features, in case you see something that's a better fit.
How it could work for your case
A CA you control issues x509 certificates containing Ziti-specific claims, e.g., a SPIFFE ID in a URI SAN. In Ziti, this CA is verified and has a configuration that matches the scheme of the claims you place in client certificates. This configuration includes a pattern that maps the claim to a specific Ziti identity that you have pre-created with a unique set of roles. To assume a different Ziti identity with different roles you would issue a certificate with a claim that matches the other Ziti identity. This gives you the ability to determine while roles the Ziti identity will have by issuing a tailored certificate from your CA.
External ID Claims and other related features of Ziti

3rd party CA - You can verify another CA you control and use it to create identities (so, 3rd party from Ziti's perspective, 1st party from your perspective). You must issue a client certificate for each identity and pair it with a private key and Ziti token to enroll with Ziti. You have two options for role attributes with this approach:

auto-create identities during enrollment - one set of roles for all identities for your CA,
pre-create identities to enroll with a cert from your CA, each with unique roles, or
both enrollment strategies concurrently in effect.

External ID - You can map a JWT or x509 claim to a single Ziti identity by assigning an externalId property to the identity matching a verified CA's claims configuration.

Link to doc


General comments

@aaronpeikert I've read the manuscript and provided copious notes. My main point of feedback cannot be provided inline, so I will provide it here. I think that, aside from incorporating the

I am afraid that some of your text got lost, @cjvanlissa.

I can't remember what this was regarding! It's possible that @aaronpeikert and I had a phone call right when I was opening this issue, and that I may have absentmindedly opened it.

Either way, Aaron knows that my main comment is that the structure should be revised; probably something like:

General introduction of the problem and proposed solution
Exposition of the techniques used as part of the solution
Tutorial demonstrating those techniques

Currently, these three themes are intermixed.
My second main comment was that attention should be paid to tell people only the information they need to know for the tutorial (e.g., skip examples using SEM because a large part of the readership won't know SEM), and that new information should be presented at the right time. For example, I believe that Issues and Pull Requests were introduced before GitHub was introduced. Several cases like that appear. This is confusing to any readers not yet familiar with the techniques.


Log report to the console as well

Should we log the report to the console? It's the only way I could figure out how to do it that wouldn't require people to download a file. I personally find it really annoying to download from actions. Or in general. Thoughts?
See action: https://github.com/plocket/docassemble-MAEvictionDefense/actions/runs/434724343

I prefer console output over downloads for log output as well 👍


[MM-29453] Migrate changeCSS() to CSS variable in utils/utils.jsx, ln. 533

Summary

Migrate changeCSS() to CSS variable in utils/utils.jsx, ln. 533.
Change use of js variable theme.sidebarTextActiveBorder to css variable --sidebar-text-active-border.
Ticket Link

https://github.com/mattermost/mattermost-server/issues/15864
https://mattermost.atlassian.net/browse/MM-29453

Hello @mathiasvr,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

This PR will be batch QA'd when it's campaign branch is merged to master


Add the ConditionalWrapper Component

The Problem
During the development of the forms framework last quarter I found myself needing a component that would only wrap content if a certain condition were true. I have a similar need this quarter as I start work on the ActionList effort.
It's possible to do this sort of thing today with a good 'ol if statement, but you have to duplicate a bunch of code in each branch:
<% if some_condition? %>
  <%= render(Primer::BaseComponent.new(tag: :div)) do %>
     ...
  <% end %>
<% else %>
   ...
<% end %>

It's also possible to capture the inner HTML and use it in both branches to reduce duplication a bit:
<% inner_html = capture do %>
    ...
<% end %>
<% if some_condition? %>
  <%= render(Primer::BaseComponent.new(tag: :div)) do %>
    <%= inner_html %>
  <% end %>
<% else %>
  <%= inner_html %>
<% end %>

Both of these approaches are pretty awkward, and I personally find them hard to read.
Proposed Solution
What we really need IMHO is a component that renders a wrapper element if a particular condition is true and doesn't render the wrapper element if the condition is false. View Component gives us the render? method, but that's used to skip rendering an entire component, including its content.
I propose we introduce a new ConditionalWrapper component to encapsulate the desired behavior:
<%= render(ConditionalWrapper.new(condition: true, tag: :div)) %>
    ...
<% end %>

So clean. Much readable. Wow.
Prior Art
The primer/react project has a ConditionalBox component for their implementation of ActionList.

So clean. Much readable. Wow.

I agree with this! My only concern is potential for misinterpretation of what the component does.
Before I read the PR description, I interpreted ConditionalBaseComponent to mean that the entire component only renders if the condition is met. Someone who sees this in the wild could interpret this similarly.
I wonder if updating the name to something like ConditionalWrapper would help... I don't have a very strong preference either way.
Side note: I think more recent components don't have -Component at the end so I'd suggest removing that.

@khiga8 those are great points, thank you 😄 I used the "-Component" suffix to match the naming of BaseComponent, but I agree the name is confusing. If we changed it to ConditionalWrapper then the need for consistency with BaseComponent goes away.

👋🏻 This is a nice addition! @camertron do you think we need to write some light public docs for this one, or do we only want this used sparingly/internally?

@lesliecdubs I envisioned that it would only be used internally :)

Thanks, copy that @camertron! We were putting together the Primer changelog today and I wasn't sure whether to include or not. I'll remove.


On the cable car partner Sherry keeps walking in the same direction.

Version
3.0.9 (1d6b330)
Seed
R720-7LW9-XRZZ10ZZ4Y1RH7Q9K79K4ZV5W53H5FQ0Z00YZZZZZ
Scenario
RE 2 (Leon A)
Player Character
Rachael
What happened?
Inside the cable car, she went from Ada to Sherry. Also, she has a machine gun.
However, she never attacked G's arm, just ignored the player and kept walking in the same direction.
To begin with, I sometimes see Sherry's type of partner character generated by the room in the Leon scenario.
Is it impossible for the Leon Scenario to create only partner characters that can attack enemies, even if it's Sherry?
I would be happy if I could fix both the type and character, either Ada or Sherry.
sample
https://youtu.be/r8p4dzXgpCk?t=962

I could limit it to Ada which may have anyone overlayed on to her.

I don't think this is an issue anymore


getting 500

1 service not able to communicate with another service
even though eureka server  dashboard shows all 4 services up & running
getting this :
java.net.UnknownHostException: Failed to resolve 'MOHADANI-7410.in.oracle.com' after 2 queries
at io.netty.resolver.dns.DnsResolveContext.finishResolve(DnsResolveContext.java:1069)
Suppressed: reactor.core.publisher.FluxOnAssembly$OnAssemblyException:
this is  my host file:
<IP_ADDRESS> peer1
#<IP_ADDRESS> backup-server
<IP_ADDRESS> peer2
#<IP_ADDRESS> auth-server
#<IP_ADDRESS> config
#<IP_ADDRESS> registry
#<IP_ADDRESS> auth-service
#<IP_ADDRESS> rabbitmq
#<IP_ADDRESS> CN=wls-sysadm
#<IP_ADDRESS> whf00gbb.in.oracle.com
<IP_ADDRESS> whf00gbb.in.oracle.com
#<IP_ADDRESS> localhost

Hi @danishcool77 did you try by using the localhost?

Please reopen if your issue is not solved


Adjusted prop types on Error element. Added tests.

Let me know how the onCloseCallback snippet looks. Otherwise, It is just some standard tests.

I see you added the onCloseCallback snippet we discussed. I just added some comments for future reference if we ever decide to use something other than the Box element.

Thanks for that!  Your tests serve as a great reference for all of us.
I usually avoid comments (prefer code to be as self-documented as possible), but I think in this case it makes total sense.  Appreciate it!

By the way, i think Travis is down right now.  I don't think it actually errored, but I will look into it.


🔎 Refactor plot_ragged function to improve clarity and documentation

This PR is to clarify the docstring of plot_ragged. @philippemiron let me know what you think.

There was a comment that wasn't resolved...

Resolved now! I took a decision to move forward.


reviewer file heatmap visualization

@cdolfi The breakdown into functions that you did actually makes a ton of sense to me, and makes reading the code SO much easier. I made some suggestions about function names and verbally describing the code in the process_data function.

@cdolfi squash commits as you feel is necessary, rebase onto dev, and this is good to go!

@JamesKunstle looking at the commits I think its a solid version history


Explain default HostnameGenerators

Description
Explain how they work and their purpose

Fixed by https://github.com/kumahq/kuma-website/pull/1938


Onboarding a computer as the IoT thing

Following tutorial [https://www.sap.com/developer/tutorials/iotae-comp-thingmodeler0.html] came to step 5, the form is different from tutorial, dont know what to input in 'Connectivity' - 'Device' and 'Sensor'.
In my case, 'Sensor' is mandatory but does not give any available data.

It turns out to be a different 'Provider' issue, tutrial is using provider 'NEO' .


Make explorer searchable

As I've used, and seen being used, the OSF extension more and more, it's become clear to me that some people have many, many projects. So it would be good to make the explorer searchable. I'm thinking about a simple search just above the tree widget that searches projects by title and contributor.

Yes, I agree. The mechanics are already in place, as there already is a Tree Iterator implemented in some form that filters everything but .osexp files. Because this was out of scope at the time we worked on this extension in Marseille and it wasn't finished and had not been thoroughly tested yet, I have disabled this functionality, but it should be a good basis for a search function.

This has been added in 1.3.0


Hackerrank Day 8 -10 Days of Javascript solution

Created a CONTRIBUTING.md file to help beginners to contribute to this repository.

Thanks for the pull request @anilkaundal :tada:
Just a change. Remove Code of Conduct part from the Contributing.md file and instead link the code of conduct to the following link :
https://github.com/afrozchakure/Competitive-Programming-Solutions/blob/master/CODE_OF_CONDUCT.md
Do the above changes. It's good to merge after that. :+1:


Fix a syntax error in kpod.bats

The cleanup routine at the end of one of the test functions wasn't in the function, so it was executed at the wrong time.

@nalind This is broken in travis?

There were other syntax errors.  Updated to fix them and to check on the result from "rmi" calls, as @runcom spotted.

Closed by merging this into #778.


METRON-2128: LEEF config file is missing in RPM spec file

Contributor Comments
The default config sample used in the LEEF parser integration test, and provided as a sample starting point was not included in the RPM SPEC file.
This PR fixes this omission and cleans up whitespace formatting the parsers to ensure better style guide compliance (only actual changes in the java files here are whitespace)
To Test: verify build succeeds with mvn clean package -DskipTests -Pbuild-rpms
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[ ] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

[ ] Have you ensured that any documentation diagrams have been updated, along with their source files, using draw.io? See Metron Development Guidelines for instructions.

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

the leef configuration does not go in -common, can you move it please?

Agreed, but that what the integration tests look for, so that’s where it needs to live to get the integration tests required by you comments on the other PR. Can you suggest where it should go, and how we should change the integration test approach?

Agreed, but that’s what the integration tests look for, so that’s where it needs to live to get the integration tests required by you comments on the other PR. Can you suggest where it should go, and how we should change the integration test approach?

It actually checks for zookeeper config in both metron-parsing/metron-parsers and metron-parsing/metron-parsers-common - https://github.com/apache/metron/blob/22cc622d6f08cfa77c06dea2df7f4c56edc1862e/metron-platform/metron-parsing/metron-parsers-common/src/test/java/org/apache/metron/parsers/integration/ParserIntegrationTest.java#L67. I didn't actually realize that until I dug a bit. Echoing our earlier thoughts, I would like to simplify this a bit at some point so that the concerns aren't jumbled.
That being said, the leef config should go here - https://github.com/apache/metron/tree/22cc622d6f08cfa77c06dea2df7f4c56edc1862e/metron-platform/metron-parsing/metron-parsers/src/main/config/zookeeper/parsers

@simonellistonball - I opened a PR against your PR moving the leef config in metron.spec. I ran the build locally and checked that the RPM's build via time mvn clean package -DskipTests -Pbuild-rpms
Fwiw, here were the results of the timing - 1111.11s user 154.80s system 56% cpu 37:13.88 total

@simonellistonball Would you mind bouncing Travis? Seems like an unrelated failure.

@simonellistonball Would you mind bouncing Travis? Seems like an unrelated failure.

Not sure what the problem is... all the travis checks passed on this.

@mmiklavc Are you good with this? I'm going to abstain since I mostly saw a Travis issue that I probably commented on the wrong PR for, based on the response from @simonellistonball

@justinleet Everything looks good, except we're missing an update to the changelog file here - https://github.com/apache/metron/blob/02bd729bf349095e4fc7054d647dd6b903ac8bb7/metron-deployment/packaging/docker/rpm-docker/SPECS/metron.spec#L688. It looks like we also missed the update from a PR I submitted as well. I'm going to open a separate PR to fix this so we can get master working again.
+1 by inspection

https://issues.apache.org/jira/browse/METRON-2144


Intel Realsense D4XX recovery for D450

Required Info

Camera Model
{ D450 }

Firmware Version
(<IP_ADDRESS>)

Operating System & Version
{Win (10)

Platform
PC

SDK Version
{ 2.4.9 }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I am trying to connect D450 with D4 board V1, in the D400 series  datasheet
(1) The V1 ASIC (without IMU) is validated and supports all D400 depth modules
but it shows Intel Realsense D4XX recovery in the Realsense Viewer
physicial port: \?\USB#VID_8086&PID_0ADB#111111111111#{a5dcbf10-6530-11d2-901f-00c04fb951ed}
firmware update id:118223024220
it works ok for V3 board, so it might not be the issue of  interposer.
any idea how to solve this?

Hi @reals2018  The D450 depth module must be used with the Vision Processor D4 V3 board.  Likewise, the D401 module should only be paired with the V4 board.  I will refer the (1) note to my Intel RealSense colleagues.  Thanks!

Hi @reals2018  Do you require further assistance with this case, please?  Thanks!

Not now. I will close this first.Thanks for the help

You are very welcome, @reals2018  - thanks very much for the update!


print without pdf file and show print dialog

hi
I have a report that is set as a code and it is a very simple report. When the client clicks on the button, I want the information to go from the controller to the report and be printed without a preview. I use Page Razor, not Blazor. Thank you for your help. How can I do this?

@tahafathi2013 Not really sure what you need here, can you explain what's the problem with the library?

Directly printing cannot be done in the Browser due to the JS sandbox. However an alternative has been provided in the base lib.
https://github.com/crabbly/Print.js/issues/517


Migrate from beta rails 5.2 to stable rails 5.2

Looks like circle might not have a new enough ver of chrome.


ci: Publish main Engine image when publishing from the main branch

[!WARNING]
Builds on top of https://github.com/dagger/dagger/pull/7843. Rebase onto main & merge after that PR is merged.

We want this so that we can reference registry.dagger.io/engine:main in our CI, and have our runners pick up the latest Engine Image (a.k.a. dev). I didn't want to call this tag dev since:

it assumes there is a single dev branch (unlikely to be true for long based on the latest releasing discussions)
it requires us to make an extra mental jump from dev to main to understand it's true origin
it required me to change existing code (we are already using the dev concept when building from local)

We now provide all tags explicitly, as a comma-separated string, to the publish function. This keeps things simple. This new --tags flag is not optional so that we don't forget to provide it.
Related to:

#5668

Re-requesting reviews since I changed the initial implementation based on review feedback.

Rebased on top of main to load the changes from:

https://github.com/dagger/dagger/pull/7766

That logic is extremely useful because it results in local dev engines having cache that is version dependent getting invalidated after you rebuild. Before it we constantly needed to manually prune the local cache of our dev engines in order for tests to work as expected.

I was missing this context. It makes sense to keep it. Making the change now.

The change is now back @sipsma. How does this look now?

Now that https://github.com/dagger/dagger/pull/7843 got merged, rebasing on top of main so that this can get merged straight after.


[Need Help]: What should be the timing to trigger Sync flow in the CoE Starter Kit 2.1.6 ?

Hello,
I need suggestions on Sync Flow timings!
The Sync process trigger which is dependent on  Flow "Admin | Sync Template v3"  shows UTC 15:00 as trigger time in the CoE Starter Kit version 2.1.6, which would be different in different time zones.
Could you please share if sync flow should be triggered at specific time in order to maintain CoE processes managed by other flows in the CoE Starter kit 2.1.6? or the default configured time will be fine regardless any time zone?
Thanks and Regards,
Dipti

We have moved to a new dedicated repository for the CoE Starter Kit, and issues here will no longer receive attention. Please raise your issues here from now on:
https://github.com/microsoft/coe-starter-kit/issues


Use async/await to read from the named pipe

Details of Changes

Simplify the code by using ReadAsync
Fix a logic issue: Using UTF8.GetString on an incomplete buffer may generate invalid strings (example). The new code is using a MemoryStream to get the message content until the message is complete.

Can the same be done on the ("other side")[https://github.com/files-community/Files/blob/main/Files/Helpers/AppServiceConnectionHelper.cs] of the pipe connection?

Can the same be done on the other side of the pipe connection?

Yes, I can do it. I prefer to keep MR small. So, I can do it in another MR as the two changes can be merged independently.

The follow-up PR: https://github.com/files-community/Files/pull/7130


obsidian internal links not working for graphviz

So far, I've been unable to figure out how to make obsidian internal links work for the graphviz instance.
External links work great!
  EXT1 [label="  BASIC ANALYSIS2  "       , target="_top", URL="https://google.com"];
but nothing so far for internal-links:
OUT2 [label="  BASIC CATEGORY THEORY  ", URL="cooperation", class="internal-link"];
that internal-link DOES work for the obsidian builtin mermaid (in mermaid).
Am I not doing it right?  How should I do it?
PS bonus points for getting internal-link absolute and/or relative paths to work.  I'm trying to port a wiki to obsidian.
PPS.  This is awesome.

This might help:
https://github.com/joethei/obsidian-plantuml/blob/master/src/functions.ts#L30-L50


Issues related with security founded

Dear postalhq.
We are Security Team at Samsung Research. We are interested in the security of open-source projects.
We'd like to inform you of security-related issues in your repository but we couldn't find a proper way to reach you.
Please follow the below steps.

Please send this repository url to<EMAIL_ADDRESS>Leave a comment 'Confirmed' on this issue. The comment should be written by the repository owner.
Then we will send detailed information via your email.

If you have any questions, please let us know.
Best regards,
Security Team @ Samsung Research

Any security related issues can be submitted to<EMAIL_ADDRESS>Thanks!


show we're actually using the selected character

make it clear what's happening when a character is / isn't selected

when using a character, does this override the persona, combine the persona with character or otherwise?

What's actually happening is that "character" in Gradio == a character .json or .yml file.  When you select a character, it just loads that file as if you passed in a persona_file in to oobabot cli.  Meaning it will read that file, and ignore what is in the "ai_name" and "persona" fields.
My plan was to have it hide the "ai_name" and "persona" values when you select a character, and in its place show the replacement values that we've read out of the character file.

Fix for this landed.  In case anyone is interested, here's a screenshot of what it looks like now when you select a character:

It shows you what oobabot will read out of the character sheet, and disables entry (as you really need to be editing the character sheet directly).

with some trial and error I had discovered that wake words needed to be separated by commas and no spaces and that the character selected was added to the end of the wake words list automatically, please update the READ ME file to reflect this.

was this in the yaml or with the gui?

this was before the update, I will update later and test it out

Just landed a few more tweaks, should be a bit more reasonable now


images: don't build ca-certs separately for each architecture

This is a minor efficiency improvement in the docker build. We only use the certs build step as a source for ca-certificates. This is architecture independent, so we don't need to build a separate one for each architecture.
Release note:

NONE

/approve
/lgtm


Added a reference to dvc cache dir in shared-development-server page

fix #986
Modified shared-development-server page to show the usage of dvc cache dir command and added a description to the dvc cache dir page for better understanding.

Please link to the issue this PR addresses or closes (see https://help.github.com/en/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue).

p.s. this is mentioned in the pull request template. There's no need to delete that text BTW 🙂

Thanks for the updates @PseudoNerd I'll check them out ASAP. Please notice the CI test is failing at the md format check (see https://circleci.com/gh/iterative/dvc.org/3556). Did you setup the local dev environment completely as explained in the contrib guide? #1321 was opened automatically to fix the formatting (so you could merge that for now), but ideally you can keep the formatting same from your IDE, enjoying the Git hooks that check this for you before git push.

I am not able to merge #1321
I'll retest the document for it to comply with the formatting guidelines.

I am not able to merge #1321

Hmm yeah that is strange. Please ignore it and try to fix the formatting yourself for now, it should be a good exercise anyway. I'll look into Restyled...

I'll retest the document for it to comply with the formatting guidelines.

All the info is in https://dvc.org/doc/user-guide/contributing/docs BTW

Thank you @PseudoNerd! Let's chat about your project ideas soon, and if you can please continue to contribute to other small issues when you have time.


editApproval

Hello,
I tried to implement editApproval function for workitem:
def editApproval(self, approveeID, status):
service = `self._polarion.getService('Tracker')
service.editApproval(self.uri, approveeID, status)
It throws an error:
File "C:\Users\jjgl3t\AppData\Local\Programs\Python\Python38-32\lib\site-packages\polarion\workitem.py", line 157, in editApproval
service.editApproval(self.uri, approveeID, status)
File "C:\Users\jjgl3t\AppData\Local\Programs\Python\Python38-32\lib\site-packages\zeep\proxy.py", line 46, in call
return self._proxy._binding.send(
File "C:\Users\jjgl3t\AppData\Local\Programs\Python\Python38-32\lib\site-packages\zeep\wsdl\bindings\soap.py", line 135, in send
return self.process_reply(client, operation_obj, response)
File "C:\Users\jjgl3t\AppData\Local\Programs\Python\Python38-32\lib\site-packages\zeep\wsdl\bindings\soap.py", line 229, in process_reply
return self.process_error(doc, operation)
File "C:\Users\jjgl3t\AppData\Local\Programs\Python\Python38-32\lib\site-packages\zeep\wsdl\bindings\soap.py", line 329, in process_error
raise Fault(
zeep.exceptions.Fault: java.lang.NullPointerException
Process finished with exit code 1
Do you know how to implement it?

Hi,
Typically the status has to wrapped in the correct type. For editApproval the status has to be of the correct type:

Try wrapping the status in the correct type, like so:
service.editApproval(self.uri, approveeID, self._polarion.EnumOptionIdType(id=status)))

Let me know if this helps you (or not).

Works perfectly!
Thanks.

Please share the result of your changes (especially if you will be making more) so the project can grow :)


*: fix soundness of service handler

We use Send + Clone pattern to achieve concurrency without lock.
However our service handler registry treat them as if it's Send + Sync,
which will lead to data race.

PTAL


Add support for ~/.config/hey/conf.toml configuration file

Hello
I added the support of the configuration file that can be used by hey. I think it's ok to use the same one because the inspiration is the same.
Don't hesitate to modify anything if needed, and thank you for the interactive part ^^
Cheers

Hello, sorry for the long reply, I think this is a good idea.
Thanks for the code @k-aito
Did a few refactoring:

Replaced os with pathlib (more modern way in the python) and theoretically, even a windows user will be able to run it
I removed unnecessary if - made one return ModelType.Claude

Also, if you would have any new ideas - can make new PRs)

Hello @mrgick
You are welcome, I don't use much Python so it's more like "working" but surely not properly done ^^
I have a few ideas, I will try to implement them and do a PR like this one, doing 1 PR per feature to avoid too big PR.

It will be good, I will wait))

I also made first release
In the future, if you will make a new PR - you can raise the version in init.py

Ok thanks I will do that, I think I will try to add some multiline support for the next PR. I have some idea, but I need to find something that works well.


Implement local repository for Action Item Types

WIP for review
Doug, am I on the right track with this. Am I missing something?

Added Tests for Action Item Type Local Repository. Tests successfully ran

WIP for review Doug, am I on the right track with this. Am I missing something?

Yeah, it all looks great!
Is this still a WIP or is it ready to merge? Next time, you can open the PR as a draft if you want early feedback.


Multiple connectors from Cognetik for Tableau

Adobe Analytics, Facebook Ads, Facebook Pages, Doubleclick, Youtube,Bing Ads, Google Adwords, Kochava connectors for Tableau offered by Cognetik

Coverage remained the same at 98.913% when pulling f503d4d659c2424a694dc5fad78ebffcf9b74817 on RalucaAPascu:patch-6 into a93717242f671815ef282ce404afbfdcb406aa91 on tableau:master.

Hi Raluca,
We already have the Cognetik connectors on the community list. They can be found under "Cognetik Cloud Connector"


Integrating DearImGui with VK2D

Hello!
I'm currently trying to integrate dearimgui into a Vulkan2D based project, with... mixed success.
As per the ImGui instructions, I've included the SDL2 and Vulkan backends, and placed the API functions as follows:
Imgui Init -> After vk2dRendererInit, passing the init_info as follows:
ImGui_ImplVulkan_InitInfo init_info = {};
VK2DRenderer renderer = vk2dRendererGetPointer();
init_info.Instance = renderer->vk;
init_info.PhysicalDevice = renderer->pd->dev;
init_info.Device = renderer->ld->dev;
init_info.QueueFamily = renderer->pd->QueueFamily.graphicsFamily;
init_info.Queue = renderer->ld->queue;
init_info.PipelineCache = VK_NULL_HANDLE;
init_info.DescriptorPool = renderer->samplerPool;
init_info.Subpass = renderer->targetSubPass;
init_info.MinImageCount = renderer->swapchainImageCount;
init_info.ImageCount = renderer->swapchainImageCount;
init_info.MSAASamples = VK_SAMPLE_COUNT_1_BIT;
init_info.Allocator = NULL;
init_info.CheckVkResultFn = check_vk_result;
init_info.RenderPass = renderer->renderPass;

ImGui_ImplSDL2_ProcessEvent -> Into the SDL2 PollEvents while loop,
ImGui (and SDL/Vulkan implementation) new frame functions -> after vk2dRendererStartFrame,
ImGui::Render and ImGui_ImplVulkan_RenderDrawData -> before vk2dRendererEndFrame, passing vk2dRendererGetPointer()->commandBuffer to the RenderDrawData.
However, it seems that ImGui doesn't draw to all the swapchain images correctly, and I'm not sure why, since I'm passing the imagecount directly from the render engine.
The visual output looks like this:
https://github.com/user-attachments/assets/0e4696bc-f96b-46de-a201-fcd19a47ee26
There is also an issue wherein the colourspace between ImGui and VK2D doesn't match, but that's the lesser issue in this case :)
I would appreciate any advice or recommendations/pointers as to what could be causing this and how I could address it. I've tested this with all the window mode (Vsync) options, and this occurs on all of them.
Thank you for your awesome work on VK2D!

Hi, I appreciate the kind words! I haven't gotten around to implementing imgui myself so I'm not overly familiar with the process but I have a few guesses. I don't know your event loop but that kind of looks like it may be rendering to only one of the swapchain images. Also, renderer->samplerPool is made for the texture sampler while from what I can tell imgui requires a combined image sampler (which VK2D doesn't use). Its best to make your own descriptor pool for that as I don't think doing that is Vulkan compliant and would cause issues on some devices. When I have some time later I can try implementing imgui on my own to find the common pitfalls. Actually also double check that your MSAA value matches what VK2D is using. As for for the colour space issues, I know that is a common problem with imgui and I'm gonna need to look into it a little bit and get back to that.
P.S. While accessing the renderer directly works I highly recommend using VulkanInterface.h instead for readability and future-proofing purposes. The renderer is not meant to be user accessed and I've got no promises on it remaining structured that way long term lol.

Thank you! I'll try again with the Vulkan Interface and see if I manage to get it to work (my Vulkan experience is a bit limited so I'm not certain I'll get it to function). An official implementation would of course be awesome, since ImGui is a pretty essential tool nowadays.
Yeah, I expected it to be a swapchain issue. My event loop is essentially just the one of the basic example in the readme.
As for the combined image sampler, should I attempt to create a new pool for it?
Thank you for your help!

I actually managed to get it to work by swapping my erroneous usage of vk2dRendererGetPointer()->commandBuffer on the ImGui_ImplVulkan_RenderDrawData function with vk2dVulkanGetDrawBuffer(). Now only the SRGB issue remains, as well my incorrect usage of the sampler pool. I'll do some research myself.

Worked around the SRGB issue by adding an SRGB->Linear converstion at the end of ImGui's fragment shader.
Another pitfall I encountered was that on Windows, SDL2 by default doesn't DPI-scale correctly on large displays.
One is required to call the windows,h SetProcessDPIAware(); and add SDL_WINDOW_ALLOW_HIGHDPI flag to the SDL window.

Hi, I was just looking into the sRGB problems and I found another solution to the colorspace problem that is less invasive than what you are doing, but I don't have time to get around to that solution quite yet (I wrote it down, I would like to support imgui out of the box). In the meantime that is a great solution that I'm happy to hear you've solved. As for SDL2's DPI scaling, I am aware of this and I have heard, without substantial evidence, that SetProcessDPIAware(); isn't the best solution (although it works well for me) but in either case you reminded me to add an application manifest to the Windows version that should solve that in the near future. Also for the descriptor pool imgui wants, I found this issue and also vkguide seems to corroborate it, you can create the descriptor pool imgui needs with something like
vkDescriptorPoolSize pool_sizes[] = { { VK_DESCRIPTOR_TYPE_SAMPLER, 1000 },
	{ VK_DESCRIPTOR_TYPE_COMBINED_IMAGE_SAMPLER, 1000 },
	{ VK_DESCRIPTOR_TYPE_SAMPLED_IMAGE, 1000 },
	{ VK_DESCRIPTOR_TYPE_STORAGE_IMAGE, 1000 },
	{ VK_DESCRIPTOR_TYPE_UNIFORM_TEXEL_BUFFER, 1000 },
	{ VK_DESCRIPTOR_TYPE_STORAGE_TEXEL_BUFFER, 1000 },
	{ VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER, 1000 },
	{ VK_DESCRIPTOR_TYPE_STORAGE_BUFFER, 1000 },
	{ VK_DESCRIPTOR_TYPE_UNIFORM_BUFFER_DYNAMIC, 1000 },
	{ VK_DESCRIPTOR_TYPE_STORAGE_BUFFER_DYNAMIC, 1000 },
	{ VK_DESCRIPTOR_TYPE_INPUT_ATTACHMENT, 1000 } };

VkDescriptorPoolCreateInfo pool_info = {};
pool_info.sType = VK_STRUCTURE_TYPE_DESCRIPTOR_POOL_CREATE_INFO;
pool_info.flags = VK_DESCRIPTOR_POOL_CREATE_FREE_DESCRIPTOR_SET_BIT;
pool_info.maxSets = 1000;
pool_info.poolSizeCount = (uint32_t)std::size(pool_sizes);
pool_info.pPoolSizes = pool_sizes;

VkDescriptorPool imguiPool;
VK_CHECK(vkCreateDescriptorPool(_device, &pool_info, nullptr, &imguiPool));

(This code is from vkguide.dev)
I think 1000 per descriptor type is serious overkill but will probably work just fine and more importantly won't give you weird unexplainable crashes on more restrictive Vulkan drivers like the ones AMD devices tend to have.
Thanks for bringing this up and good luck on the project!


exceeding data rate

When I am trying to scrape some files (using BeautifulSoup) we are getting am ‘exceeding data rate’ error. The solution for this is to set a different limit when launching the docker image
IOPub data rate exceeded.

The Jupyter server will temporarily stop sending output

to the client in order to avoid crashing it.

To change this limit, set the config variable

`--ServerApp.iopub_data_rate_limit`.

Current values:

ServerApp.iopub_data_rate_limit=1000000.0 (bytes/sec)

ServerApp.rate_limit_window=3.0 (secs)

We need to add in the values.yaml:
                  singleuser:
                    extraFiles:
                      jupyter_notebook_config.json:
                        mountPath: /etc/jupyter/jupyter_notebook_config.json
                        data:
                          ServerApp:
                            iopub_data_rate_limit: 1.0e10
                            iopub_msg_rate_limitFloat: 3000


[feature] 部门管理-获取部门ID

https://github.com/addcnos/youdu/blob/2.x/feature.md

see https://github.com/addcnos/youdu/pull/24

see https://github.com/addcnos/youdu/pull/24


functions added by after_request execute in reversed order

Expected Behavior
Print in following order:
foo
bar
def foo(response):
    print("foo")
    return response
def bar(response):
    print("bar")
    return response

app = Flask(__name__)
app.after_request(foo)
app.after_request(bar)

Actual Behavior
Print in following order:
bar
foo
Environment

Python version: 3.6.5
Flask version: 1.0.2
Werkzeug version: 0.14.1

This is very counter intuitive. Is there a reason to execute reversed order? It seems like it is reversed on purpose in process_response in app.py. The order actually matters. I found out this when I apply gzip and html minifier which gzip is apply first and the minifier is unable to read the data as string.

This is intentional. Imagine that you as an extension that does the gzip. You'd totally add this early, before setting up your tires and callbacks. If it were to execute first, none of your own callbacks could modify the response body, it would be zipped already.

@davidism How about mention it on documentation?


Add documentation / example for WebhookListenerService and WebhookUrlService

At the moment it seems unclear what the classes WebhookListenerService and WebhookUrlService are there for. Would it be possible to add docs and an example to both classes? I am trying to find an example of how to listen to a Webhook defined in a Postfinance space and if the SDK can perhaps help with what kind of JSON type to expect in the body when the webhook calls my application.

@theodorspringwallee Thanks for the detailed explanation. I think what I was looking for in the SDK, was an interface like this one for the body of the webhook event:
export interface WebhookEvent {
    eventId: number,
    entityId: number,
    listenerEntityId: number,
    listenerEntityTechnicalName: 'Transaction' | string, // List all possible entities or link to enum?
    spaceId: number,
    webhookListenerId: number,
    timestamp: string;
}

Is it possible that that is missing from the SDK at the moment?

Dear tobias47n9e
I couldn't find any reference of a method of the SDK, that is receiving and parsing the raw content that is sent from our webhooks to the target URL, where your script is behind.
Sent content from our webhooks (example):
{
"eventId": 138997445,
"entityId": 63842795,
"listenerEntityId":<PHONE_NUMBER>003,
"listenerEntityTechnicalName":
"Transaction"
"spaceId": 28435,
"webhookListenerId": 285948,
"timestamp":
"2022-08-2407:53:40+0000'
}

At the moment you need to parse the incoming content manually. For example with a JSON parser.
If you want, I can ask our plugins team to add such a WebhookReceiver.ts service, that you could use there on your interface, where you are receiving the webhooks?
Thank you for your feedback.
Best regards

@theodorspringwallee I actually use the TypedJson library to parse the incoming webhook event:
import { jsonMember, jsonObject } from "typedjson";

@jsonObject
export class WebhookEvent {
    @jsonMember
    eventId: number;

    @jsonMember
    entityId: number;

    @jsonMember
    listenerEntityId: number;

    @jsonMember
    listenerEntityTechnicalName: 'Transaction' | string;

    @jsonMember
    spaceId: number;

    @jsonMember
    webhookListenerId: number;

    @jsonMember
    timestamp: string;
}

It is not a lot of code that needs to be implemented using either a typescript interface or this class for "typedjson", but it is kind of missing from the documentation what the payload of the request is. I also couldn't find the payload and header information here: https://checkout.postfinance.ch/doc .
So maybe it doesn't really need a service but just some docs how to handle the webhook event:

What is the payload and what are the headers of the request and how do they change (if they do), if the event is tried multiple times
What kind of response codes can be sent back
Which response codes deactivate the webhook

In general the webhook docs seem to me would warrant an own chapter in the docs, because many things are not explained in detail:

When can the first webhook event be expected after it being triggered
How many times and in which time intervals will it be retried, if the server is not reachable?

Dear tobias47n9e
Here we have a documentation, about the payload of the webhooks: https://checkout.postfinance.ch/en/doc/webhooks#_integration_details
Here you see how often and in which time span failed webhooks are retried:

After that you would receive an email to your defined email on your space, where you need to take manual actions.
Does this answer your questions?
Best regards

@theodorspringwallee Thanks for the link. I hadn't seen the chapter before. In this case my issue is all resolved now.


Exceptions in streaming.py and _app.py

So Looks like there are some issues with version 1.0.4 in streaming.py.
line 128:
self.WS.run_forever(ping_interval=self._ping_interval, ping_timeout=self._ping_timeout,
ping_payload=self._ping_payload, skip_utf8_validation=self._skip_utf8_validation)

Exception since WebSockets run_forever does not expect ping_payload argument.

line 625:
def _default_on_open(self, _ws: ws_client.WebSocketApp, *args):

Exception in _app.py, line 340, since _default_on_open is called without _ws argument (true part in if clause):
def _callback(self, callback, *args):
if callback:
try:
if inspect.ismethod(callback):
callback(*args)
else:
callback(self, *args)

To Reproduce
Just trying to initialise StockStream and call start_stream_thread after subscribing to trades.
Environment Details:
MacOS, Python 3.6, WebSockets 9.1

Hey @ecamaj
Thanks for reporting the issue. I'm travelling over the course of a few days so won't be able to dig in.
Be sure to checkout the market maker lite repository which has callback streamer running without issues to ensure you followed the order and sequence of the function calls. ( Link to that repo is on the readme)
You should also be able to just copy the third example from readme and run it.
I'll look for the bugs when i get back in a few days.
Feel free to join in the discord server for discussions.
Thanks
P S

ah, it is websockets package, not yours.

I'll be closing this one for lack of activity in a few days @ecamaj
feel free to re-open this or open new issue if there is further info.


code release

Dear author ! I'm very interested in your work! May i ask when you release your code ?Thank you!

Hi @june2678 , thanks for your interest in our work. The code of SIENet has been released, and we hope it will be helpful for your research.


Delete command and refactoring

PR for #140
Added flow to delete the repository.
Refactorings, especially the final reply logic, it is all pulled out in method Brain.finalCommentStep(...) and used everywhere. No more denialReply and followupSteps methods since they were code duplication.
Also, method Brain.withCommonChecks(...) adds the common checking steps for all the actions.

Coverage decreased (-0.3%) to 85.823% when pulling a06e11ddc2aab2dbf661bc3c03945280112cf331 on 140 into 9b187cd15e6fd628875805e519370a8c586cc3c4 on master.

Coverage decreased (-4.0%) to 82.122% when pulling 0c77a0f4ab4895f35ed1d6ba50d0449fe79cb005 on 140 into 9b187cd15e6fd628875805e519370a8c586cc3c4 on master.

Coverage decreased (-2.5%) to 83.546% when pulling 99f9294d13e4bc69649efe24d0ceede7953b75f4 on 140 into 9b187cd15e6fd628875805e519370a8c586cc3c4 on master.

@rultor merge

@rultor merge

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@amihaiemil Done! FYI, the full log is here (took me 2min)


Casing of header is incorrect -  header value is in lower case: aeg-event-type

where it says: The event contains a header value "Aeg-Event-Type: SubscriptionValidation".
The header value is in lower case: aeg-event-type

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

ID: 9ea53478-b3df-71dd-5ec8-27746a650775
Version Independent ID: 185f045e-d659-6217-3c14-68418c4fe5fe
Content: Azure Event Grid security and authentication
Content Source: articles/event-grid/security-authentication.md
Service: event-grid
GitHub Login: @banisadr
Microsoft Alias: babanisa

@cicorias thank you for your feedback! I have assigned the issue to the content author to evaluate and update as appropriate.


fix: fix command line silent abort

Fix the command line silents abort when the
the EDITOR variable is not specified.
With this commit the result is reported below
➜  rio git:(main) RAD_PASSPHRASE=12345 rad issue open
✗ Issue failed: editor not configured: the `EDITOR` environment variable is not set

looks like is it handled correctly?
radicle-cli/src/commands/id.rs:        match term::Editor::new().edit(yaml)? {
radicle-cli/src/commands/issue.rs:            if let Some(text) = term::Editor::new().edit(&doc)? {

Thanks!


UserUpdated event just doesn't feel right

Up to commit 3bd920ce66d6f7ff24bdf6e7077ea24499ddb9c5 I used UserPresenceUpdated to track when someone comes online or goes offline in order to update user counter on a web widget (which is per guild). Since the merge there is no way to retrieve information about guild where that event happened from event itself since it's passing only 2 SocketUser parameters.
Even if I resort to extensive checking in which guilds user is joined there is this problem that we now treat UserUpdated as global event but it's in fact not. Event is fired per guild, not per user, meaning if 1 user is in 3 guilds where bot resides his single status change will result in 3 identical event executions, that can't be intended.
public DiscordWidgetService(IDependencyMap map)
{
    client.UserUpdated += Client_UserUpdated;
}

async Task Client_UserUpdated(SocketUser arg1, SocketUser arg2)
{
    await Console.Out.WriteLineAsync($"UserUpdated fired for {arg2.Username}");
}

Our options are limited, because of the way the Discord API is structured. We've discussed this issue extensively, and this is the best we can do, for now.

Doesn't Discord pass the guild where the presence update originated from?

They do and thats how it was implemented up until the mentioned commit which I guess aimed to simplify things.
@khionu Since API does pass the guild id, wouldn't it make more sense to at least use GuildMemberUpdated instead of UserUpdated? Would solve my issue and better adhere to API structure which kinda does consider this to be a guild oriented event. 😄
client.GuildMemberUpdated += Client_GuildMemberUpdatedAsync;

async Task Client_GuildMemberUpdatedAsync(SocketGuildUser before, SocketGuildUser after)
{
    await Console.Out.WriteLineAsync($"UserUpdated fired for {after.Username} in {after.Guild.Name}");
}

We store presence at the global level, not per guild, so the previous event was flawed and buggy. Even though the API sends this data per-guild, it does not actually represent a per-guild state.

@RogueException But presence isn't considered or supported as global by discord. As explained by jhgg in here

Presences and roles are per-guild, whereas the user object is global.

Funny since I talked to Jake 3 weeks before that and was told to do it globally. Will ask around =\

I'd assume that's why Presence Update and User Update are separate events in the API.
And also description for Presence Update reads

A user's presence is their current state on a guild. This event is sent when a user's presence is updated for a guild.

So yeah user precence is per-guild. I guess sometime in the future we might even get option to set invisible for certain guilds while remaining online in others.

Presence and User updates arent separate though. Presence updates still carry global user data. Hence my confusion.

This should be fixed in https://github.com/RogueException/Discord.Net/commit/fd72583a75a1c9ccc1f6040c8611d20c58513e4a


YAML language mode for .stylelintrc

VSCode Version: 1.11
OS Version: Linux Ubuntu 16.04

I have .stylelintrc file which is YAML, we need to make default language mode for these files to YAML as it is right now in Atom, because right now this file default language mode is plain text.

Note that you can also set this in you user settings at "files.associations"

User settings is not ideal because of multiple users in the project.
As per Stylelint guide

The .stylelintrc file (without extension) can be in JSON or YAML format

Visual Studio should recognize .stylelintr the same way as it does with .eslintrc

Given that .stylelintrc file (without extension) can be in JSON or YAML, I think we should not pre-define that one. As written, users can configure it through the language picker and store it as setting in the user or workspace settings.


Hi, how about running Hls video sources.

Hi developers, i have issue with running .m3u8(HLS) sources, do your player support this type of the video or not yet, if not yet are you planning to add supporting this format of the video?

No. For many reasons, this is currently not on the list. But maybe we'll get to this sometime. I'll reinvestigate to see if it could be added.


/comments/article

https://www.reddit.com/dev/api#GET_comments_{article}

a559fc666f25a56dab3ee18884f8491c59d4711b


Added a motivational statement by James Clear

Resolves #895

This PR is ready for review.

Following up on this.


Removed CodePen sample and added a link to the Azure Maps Samples in article: Web SDK supported browsers

Removed CodePen sample and added a link to the Azure Maps Samples in article: Web SDK supported browsers

@eriklindeman

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off
Signing off for eriklindeman who is out of office.


craned - Leader election lost

Describe the bug
kubectl logs -n crane-system craned-id-xx
I0212 18:49:02.120733       1 nodelocal.go:25] Registering node local metrics collector cpu
I0212 18:49:02.120904       1 nodelocal.go:25] Registering node local metrics collector cpuLoad
I0212 18:49:02.120918       1 nodelocal.go:25] Registering node local metrics collector diskio
I0212 18:49:02.120928       1 nodelocal.go:25] Registering node local metrics collector memory
I0212 18:49:02.120938       1 nodelocal.go:25] Registering node local metrics collector netio
I0212 18:49:02.169577       1 predictor.go:121] predictors map[dsp:0xc00027f110 percentile:0xc0004d7200]
I0212 18:49:02.175321       1 webhook.go:103] Succeed to setup autoscaling webhook
I0212 18:49:02.176252       1 manager.go:128] Recommendation Config updated.
I0212 18:49:02.176547       1 manager.go:84] Start watching /tmp/recommendation-config/config.yaml for update.
I0212 18:49:02.181380       1 predictor.go:141] predictor manager started, all predictors started
I0212 18:49:02.181428       1 prediction.go:151] predictor Periodic started
I0212 18:49:02.181466       1 prediction.go:302] predictor Percentile started
I0212 18:49:02.181502       1 leaderelection.go:248] attempting to acquire leader lease crane-system/craned...
I0212 18:49:02.189362       1 server.go:94] install crane api server middleware: log
I0212 18:49:02.189397       1 server.go:94] install crane api server middleware: cors
I0212 18:49:02.189408       1 server.go:94] install crane api server middleware: recovery
I0212 18:49:02.190170       1 server.go:149] Start to listening on http address: <IP_ADDRESS>:8082
I0212 18:49:19.272285       1 leaderelection.go:258] successfully acquired lease crane-system/craned
E0212 18:49:52.480248       1 leaderelection.go:330] error retrieving resource lock crane-system/craned: Get "https://<IP_ADDRESS>:443/api/v1/namespaces/crane-system/configmaps/craned": context deadline exceeded
I0212 18:49:52.480435       1 leaderelection.go:283] failed to renew lease crane-system/craned: timed out waiting for the condition
E0212 18:49:52.480533       1 manager.go:444] "problem running crane manager" err="leader election lost"
F0212 18:49:52.480570       1 manager.go:445] leader election lost
Dashboard is accessable, but no Endpoint can be selected. Service IP or local IP can not be accessed. No logs in craned when trying to connect to cluster.
craned is always restarting...
Reproduce steps
install Crane in running Kubernetes Cluster (v 1.28.2)
Expected behavior
craned electing leader without problems
Environment (please complete the following information):

K8S Version: 1.28.2
Crane Version: 0.2.0

You should try using the latest version crane


[DOC] Clean Up Design and Implementation Artefact from AB3 DG Model

btw forgot to mention this, but just a suggestion maybe for the model, you can use class diagram to specifies what are the models and the inheritance relationship between them @ngsengleng

okie sure


Update tests for 2021

Some tests are currently failing because they are
reading the current year and comparing it against
"2020".
This change updates the tests to expect "2021" instead.

Coverage remained the same at 100.0% when pulling 027151e88cfbae4fa4df87b979ab0fa7e0829389 on HKrausAxon:hkraus_fix_tests into 4bfb0562240abb9c5e9d1e8157500c9940da3db2 on felixrieseberg:master.

Good fix, thank you!