Fix/kqlqueryforazfw

Overview/Summary
Added | where isempty(properties.virtualHub.id) or isnull(properties.virtualHub.id) to all Azure Firewall KQL so that Hub based Azure Firewalls are not returned as part of this query.
Related Issues/Work Items

To associate a GitHub Issue, use a key word preceded with the GitHub issue number.
To associate an ADO Work Item, use the key word AB# succeeded with the ADO Work Item ID.

This PR fixes/adds/changes/removes

Replace me
Replace me

Breaking Changes

Replace me
Replace me

As part of this pull request I have

[ ] Read the Contribution Guide and ensured this PR is compliant with the guide
[ ] Checked for duplicate Pull Requests
[ ] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[ ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Ensured PR tests are passing
[ ] Performed testing and provided evidence (e.g. screenshot of output) for any changes associated to ARG queries
[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

Closing and will submit another PR since I need to modify these queries a bit.


Button "Rectangle, Circle, Line", are not arranged in proper way as you can see in the image i am sharing!

@vishanurag
As you can see the arrangement of buttons is not in proper way and the ui is also not in systematically arranged, please assign me this task under gssoc extd24 , this is my first issue so i am trying with basic issues please assign me!

hey @vishanurag
As mentioned above issues and i have solved it! this is my first issue of gssoc-ext


fix: language switch debounce bug

Fix language switch debounce bug
Proposed changes
Currently when switching language the first change you make in the second language affects the contents of the first language because of currentLanguage not changing due to debounce issues. As a quick fix I introduce a ref that hols the lang and behaves correctly then.
Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Closing issues
Further comments

better fix coming


Appstudio update test-component-pac-ntra

Pipelines as Code configuration proposal

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

StatusDurationName

✅ Succeeded
14 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
36 seconds
build-container

✅ Succeeded
22 seconds
deprecated-base-image-check

✅ Succeeded
21 seconds
inspect-image

✅ Succeeded
47 seconds
clamav-scan

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
16 seconds
sbom-json-check

✅ Succeeded
10 seconds
label-check

✅ Succeeded
10 seconds
optional-label-check

✅ Succeeded
6 seconds
show-summary

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

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
23 seconds
build-container

✅ Succeeded
11 seconds
deprecated-base-image-check

✅ Succeeded
11 seconds
inspect-image

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
34 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
8 seconds
label-check

✅ Succeeded
7 seconds
optional-label-check

✅ Succeeded
6 seconds
show-summary


More options for default properties serialization

I believe the Default serialization behaviour configuration for properties is sometimes lacking. There are two cases which I think should be configurable via fsConfig.cs:

Boolean to toggle serialization of properties with no public set method (current: true)
Boolean to toggle serialization of non-auto properties (current: false)

Lines 145-150 in fsMetaType.cs are currently:
// If it's an auto-property and it has either a public get or a public set method,
// then we serialize it
if (IsAutoProperty(property, members) &&
    (publicGetMethod != null || publicSetMethod != null)) {
    return true;
}

I would suggest they become:
// (Some updated comment)
if ((fsConfig.SerializeNonAutoProperties || IsAutoProperty(property, members)) &&
    (publicGetMethod != null && (fsConfig.SerializeNonPublicSetProperties || publicSetMethod != null))) {
    return true;
}

With the added configuration options in fsConfig.cs:
/// <summary>
/// Whether the default serialization behaviour should include non-auto properties.
/// </summary>
public static bool SerializeNonAutoProperties = false;

/// <summary>
/// Whether the default serialization behaviour should include properties with non-public setters.
/// </summary>
public static bool SerializeNonPublicSetProperties = true;

By default, the behaviour would stay unchanged for backwards compatibility, but allow for more customization by the user.

Created a pull request that matches your coding and documentation style: #57


"Press exit key to exit"

Minor issue. The program box says:
Press exit key to exit, by default it is "~"
However on a UK English keyboard, the exit key is in fact a single quotation mark, i.e. '

@triatic I mean the deadzone release is still showing as controller disconnected in PS Now.


macOS test build on Travis

Fixes #245
The setup with matrix: include: is necessary because language: python builds are not possible in combination with os: osx, so we need to do language: generic for os: osx, instead of just using language: python for the entire file.
This also necessitates some reordering of the Linux specific commands to fall under the matrix. When we will add more matrix options (e.g. different versions of Python), this will become a lot of copy-pasting. At that point, it may make more sense to do some Bash scripting in the before_install, install and script steps to detect the OS and other build parameters in order to select different install/run options. For an example setup see https://github.com/egpbos/xtensor-fftw/blob/master/.travis.yml

Does anyone feel like reviewing this? If not, maybe just merge, it seems to work. Note that this does not do the community code builds yet, that is another issue (#247).
The first commit is the same as that in #261 (this is a branch off that branch), so please merge that PR before this one.

This is also the first part of a fix for #235. Based on this example, after including the community code builds, we can write some documentation for this process, which is basically just copy-pasting the commands used in the Travis build(s).

looks ok, I think the test10 and 11 are just floating precision issues??


Refactor how OIDs are managed

Extracted from https://github.com/DataDog/integrations-core/pull/7110
What does this PR do?
Refactor how OIDs are managed
Motivation
Preparation for adding OID caching.

/azp run


Consider not showing data for regions with ratios based on less than 5 count

@RoniRos says on Slack:
"In sparse counties, a single death still registers as an extremely high death ratio, even after averaging over 7 days. Example: 1 death in Hamilton County, TX on May 24. As I argued before, I think that ratios based on a count that is <= 5 should simply not be provided, because they are more likely to mislead than to inform.  "
Right now, the ratio API does not provide count.  This would be need to be supported by @krivard 's team to implement

Closing this issue as obsolete.


errors during installation

npm version: 5.5.1
node version: 8.6.0
OS: macOS 10.12.6
$ npm install --save base64

> node-gyp rebuild

  CXX(target) Release/obj.target/base64/base64.o
../base64.cc:13:17: error: calling a protected constructor of class 'v8::HandleScope'
    HandleScope scope;
                ^
~/.node-gyp/8.6.0/include/node/v8.h:876:13: note: declared protected here
  V8_INLINE HandleScope() {}
            ^
../base64.cc:14:52: error: no member named 'New' in 'v8::String'
    return ThrowException(Exception::Error(String::New(msg)));
                                           ~~~~~~~~^
../base64.cc:122:29: error: unknown type name 'Arguments'; did you mean 'v8::internal::Arguments'?
base64_encode_binding(const Arguments &args)
                            ^~~~~~~~~
                            v8::internal::Arguments
~/.node-gyp/8.6.0/include/node/v8.h:141:7: note: 'v8::internal::Arguments' declared here
class Arguments;
      ^
../base64.cc:124:17: error: calling a protected constructor of class 'v8::HandleScope'
    HandleScope scope;
                ^
/Users/blakewillmarth/.node-gyp/8.6.0/include/node/v8.h:876:13: note: declared protected here
  V8_INLINE HandleScope() {}
            ^
../base64.cc:127:13: error: member access into incomplete type 'const v8::internal::Arguments'
    if (args.Length() != 1)
            ^
~/.node-gyp/8.6.0/include/node/v8.h:141:7: note: forward declaration of 'v8::internal::Arguments'
class Arguments;
      ^
../base64.cc:130:33: error: type 'const v8::internal::Arguments' does not provide a subscript operator
    if (Buffer::HasInstance(args[0])) {
                            ~~~~^~
../base64.cc:131:45: error: type 'const v8::internal::Arguments' does not provide a subscript operator
        v8::Handle<v8::Object> buffer = args[0]->ToObject();
                                        ~~~~^~
../base64.cc:137:18: error: type 'const v8::internal::Arguments' does not provide a subscript operator
    else if (args[0]->IsString()) {
             ~~~~^~
../base64.cc:138:38: error: type 'const v8::internal::Arguments' does not provide a subscript operator
        String::Utf8Value v8str (args[0]->ToString());
                                 ~~~~^~
../base64.cc:144:33: error: no member named 'New' in 'v8::String'
    Local<String> ret = String::New(str);
                        ~~~~~~~~^
../base64.cc:146:18: error: no member named 'Close' in 'v8::HandleScope'
    return scope.Close(ret);
           ~~~~~ ^
../base64.cc:150:29: error: unknown type name 'Arguments'; did you mean 'v8::internal::Arguments'?
base64_decode_binding(const Arguments &args)
                            ^~~~~~~~~
                            v8::internal::Arguments
~/.node-gyp/8.6.0/include/node/v8.h:141:7: note: 'v8::internal::Arguments' declared here
class Arguments;
      ^
../base64.cc:152:17: error: calling a protected constructor of class 'v8::HandleScope'
    HandleScope scope;
                ^
/Users/blakewillmarth/.node-gyp/8.6.0/include/node/v8.h:876:13: note: declared protected here
  V8_INLINE HandleScope() {}
            ^
../base64.cc:155:13: error: member access into incomplete type 'const v8::internal::Arguments'
    if (args.Length() < 1)
            ^
/Users/blakewillmarth/.node-gyp/8.6.0/include/node/v8.h:141:7: note: forward declaration of 'v8::internal::Arguments'
class Arguments;
      ^
../base64.cc:160:33: error: type 'const v8::internal::Arguments' does not provide a subscript operator
    if (Buffer::HasInstance(args[0])) { // buffer
                            ~~~~^~
../base64.cc:161:45: error: type 'const v8::internal::Arguments' does not provide a subscript operator
        v8::Handle<v8::Object> buffer = args[0]->ToObject();
                                        ~~~~^~
../base64.cc:170:27: error: expected ';' after expression
        String::AsciiValue b64data(args[0]->ToString());
                          ^
                          ;
../base64.cc:170:17: error: no member named 'AsciiValue' in 'v8::String'
        String::AsciiValue b64data(args[0]->ToString());
        ~~~~~~~~^
../base64.cc:170:40: error: type 'const v8::internal::Arguments' does not provide a subscript operator
        String::AsciiValue b64data(args[0]->ToString());
                                   ~~~~^~
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make: *** [Release/obj.target/base64/base64.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:125:13)
gyp ERR! stack     at ChildProcess.emit (events.js:213:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Darwin 16.7.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd ...
gyp ERR! node -v v8.6.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

You’re installing a different package.
Please follow https://github.com/mathiasbynens/base64#installation.


Is there any way to use CommandPalette independently?

Description
something like this

This is a component we want to create at some point, there is an open issue for this: #344
I guess you could try to put a CommandPalette inside a Popover or a ContextMenu component but not sure it will work with the focus staying in the input 🤔 Something like:
<UPopover>
  <UButton label="Search" />

  <template #panel>
    <UCommandPalette :groups="[]" :searchable="false" />
  </template>
</UPopover>

I already tried putting it inside of context menu, no issues to report.
<!-- start :: Context Menu -->
    <UContextMenu v-model="state.isGlobalContextMenuOpen" :virtual-element="state.virtualElement" class="w-[230px]">
      <UCommandPalette :searchable="false" :groups="[{ key: 'actions', commands: contextMenus.global }]" @update:model-value="actions.executeCommand" />
    </UContextMenu>
<!-- end :: Context Menu -->


cordova_not_available on device with livereload

Short description of the problem:
cordova_not_available, loaded cordova.js is mocked from serve host.
What behavior are you expecting?
Load cordova.js correctly.
I think launch index.html locally from apk and change load build files from serve host will be work.
Steps to reproduce:
ionic run android -l
System information:
ordova CLI: 6.3.1
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.12
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.45
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.7.0
Xcode version: Not installed

Hello, would you mind installing the latest version of the ionic cli? This issue was fixed in the latest version. Thanks!

@jgw96 Already at the latest version: Ionic CLI Version: 2.1.12

Same here!

Can we reopen this?
Same issue reported here: https://github.com/driftyco/ionic/issues/9263

And here: https://github.com/driftyco/ionic-cli/issues/1701

The problem is that cordova.js contains only
// mock cordova file during development
Does any one knows how to setup production (re)builds during livereload?

This command just worked for me :)
ionic run android --lirereload --consolelogs

my packega json
"scripts": {
"ionic:build": "ionic-app-scripts build ",
"ionic:serve": "ionic-app-scripts serve",
"watch": "ionic-app-scripts watch"
},
"dependencies": {
"@angular/common": "2.1.2",
"@angular/compiler": "2.1.2",
"@angular/compiler-cli": "2.1.2",
"@angular/core": "2.1.1",
"@angular/forms": "2.1.1",
"@angular/http": "2.1.1",
"@angular/platform-browser": "2.1.1",
"@angular/platform-browser-dynamic": "2.1.1",
"@angular/platform-server": "2.1.1",
"@ionic/storage": "1.1.6",
"@types/lodash": "^4.14.38",
"ionic-angular": "2.0.0-rc.1",
"ionic-native": "^2.2.3",
"ionicons": "3.0.0",
"lodash": "^4.16.6",
"phaser": "^2.6.2",
"rxjs": "5.0.0-rc.1",
"zone.js": "0.6.26"
},
"devDependencies": {
"@ionic/app-scripts": "0.0.46",
"typescript": "2.0.9"
},
"description": "ClipoApp: An Ionic project",
"config": {
"ionic_bundler": "webpack"
},

@jgw96 please reopen this issue.

@jgw96 please reopen with RC3 not work

@jgw96 please, reopen this issue. I can confirm that with the following ionic info is not working:
Cordova CLI: 6.4.0
Ionic CLI Version: 2.1.12
Ionic App Lib Version: 2.1.7
ios-deploy version: 1.9.0
ios-sim version: 5.0.11
OS: macOS Sierra
Node Version: v7.1.0
Xcode version: Xcode 8.1 Build version 8B62

Same issue here. Output for ionic info:
Your system information:

Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.13
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.46
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Linux 4.4
Node Version: v6.9.1
Xcode version: Not installed

For anyone with the same problem, issue #467 is related and currently open.

Doesn't work with - Ionic App Scripts Version: 0.0.47
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.13
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.47
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Linux 4.2
Node Version: v6.7.0
Xcode version: Not installed


feat: support verbosity levels in tracing decode

Motivation

ext cheatcodes can get very very verbose in tracing
Solution

Enable tracing for these cheatcodes only for debug-level=5, otherwise hide.

@rkrasiuk smash that merge

@odyslam heh, my change was malformatted (max len), could u pls fix that, we wait for checks and then i'll hit the button

Ah @rkrasiuk don't push. I think I need to add a small bit to forge script as well. currently, it's only used in forge test.

@rkrasiuk we should be gtg


More Examples Please 👍

Could you please add some examples of usage? You can add an example using Flask Login to auth FastAPI routes.

You probably misunderstood what this project is for.
This project is useful in a very specific use-case: if you have a flask application that uses flask-login for user session management, and you build a new, "sibling" FastAPI application where you'd like to be able to parse and validate the session cookies that were produced by the flask  app and where flask-login places the user's ID by default.
In such a case, the FastAPI dependencies of this project (FlaskSessionAuthenticator's methods) will give you easy access to this user information (together with flask-session-decoder, see here), and let you build further dependencies for your FastAPI app, e.g. a user loader.
The FastAPI-side example, based on the readme:
from fastapi import Depends, FastAPI, HTTPException
from fastapi_flask_auth import FlaskSessionAuthenticator
from flask_session_decoder import FlaskSessionDecoder

decoder = FlaskSessionDecoder(secret_key="the-secret-key-of-the-flask-app-that-created-the-session-cookie")
flask_auth = FlaskSessionAuthenticator(decoder=decoder)

def load_user_by_id(user_id: str) -> dict | None:
    """Dummy user loader."""
    return {"name": "jack", "id": user_id} if int(id) == 1 else None

def current_user(user_id: str | None = Depends(flask_auth.get_user_id)) -> dict | None:
    """FastAPI dependency that returns the user corresponding to the user ID in the session cookie that was set by the flask application and flask-login."""
    return load_user_by_id(user_id) if user_id is not None else None

app = FastAPI()

@app.get("/whoami")
def whoami(user: dict | None = Depends(current_user)) -> dict:
    """Returns the logged in user if the browser sends the flask application's session cookie to this route."""
    if user is None:
        raise HTTPException(401, detail="User not found.")

    return user

I have a Flask Application using Flask Admin as Admin Panel and Flask Login for authentication. I needed to create an API for this application, so I decided to use FastAPI instead but I don't want to rewrite everything that Flask handles which is the admin panel and auth. So I searched and found a FastAPI Feature called mount. I can mount Flask Application with FastAPI Application like this: Including WSGI - Flask, Django, others - FastAPI
All I wanted was to use the already existing Flask Login with FastAPI and I came across this awesome project of yours but I couldn't figure out how to do it. How to use Flask Login with FastAPI through this great project so I asked for an example. I believe your project is capable of doing it because you referred Flask Login in the README file but I can't imagine the implementation.

Thanks for the extra info. I didn't know about this Flask mount possibility!
You obviously can not use Flask-Login in your FastAPI routes, but after writing a current_user dependency like in the code above (load_user_by_id can be the same as for Flask-Login), you should be able to build your FastAPI auth dependencies based on that.
Please try the code above. If you have a logged in user in your flask application, and you send a request to the FastAPI application's /whoami route, it should return a dict where the id will be your logged in user's ID (or whatever UserMixin.get_id() returns for Flask-Login).
I hope I could help at least a bit.

Thank you for your answer, I'll try it as soon as possible.

Hello @volfpeter, I hope you are doing well.
I have come up with a small package called starlette-flask which includes a SessionMiddleware for Starlette/FastAPI applications to use Flask based session encoding/decoding.
It might be useful for you, so I'm mentioning it here. I'd be so happy to hear what you think about it.

Thanks for the info! I don't use flask nowadays, but I'll keep it in mind in case the situation changes.

You're welcome, this project was one of the things that inspired me to build that package.
Note: As a freelancer, I see a lot of projects written with Flask, so it helps me to use Flask with Starlette/FastAPI without making a breaking change on sessions.

Ah, you're too kind, but thanks :slightly_smiling_face: It feels pretty good when one of your projects ends up being useful to someone.

Sure it was, thank you for your contributions to the open source community, and markyp-html looks quite interesting too. I'd like to see more of you 🙏.

Well, not having to use Jinja templates is a big bonus in itself (at lteast to me). I'd like to improve the API and some internals of markyp-html, and add some utilities for HTMX API creation, I just don't have the time for much open source work nowadays unfortunately. Plus there are probably more advanced tools around.

I agree. Python in HTML (Jinja2), and HTML in JavaScript (JSX) look quite annoying to me. Why not just use Python?
Great to hear that you'll make some utilities for HTMX API. It's a quite powerful tool. I'd like to see more examples of markyp-html.
I see, there must be, there is a package called dominate, it's similar to markyp-html. I suggest checking it.
Also, there is a package called FastUI, I highly suggest you check it. Its syntax is quite inspirational but it has some downsides... It still relies on React, its design is quite limiting. Extending it is not easy if you don't know a bit of React. You can not use style attribute if the author doesn't want it...

I've never heard of dominate before, the project seems to have some interesting ideas (and maybe too many ways to do certain things). Thanks for recommending it, I'll give it a try.
I've seen a bit of FastUI - I must say I'm not too sold on the idea, but I guess it works for low-complexity UIs. Mixing "backend rendering" with frontend tools like react or angular generally becomes a mess as soon as you try to do something complex or custom. At that point you're better of creating the FE in React.

I've never heard of dominate before, the project seems to have some interesting ideas (and maybe too many ways to do certain things). Thanks for recommending it, I'll give it a try.

You're welcome. Yeah, it doesn't look like modern python.

I've seen a bit of FastUI - I must say I'm not too sold on the idea, but I guess it works for low-complexity UIs. Mixing "backend rendering" with frontend tools like react or angular generally becomes a mess as soon as you try to do something complex or custom. At that point you're better of creating the FE in React.

I couldn't agree more.

Your comments just triggered me to finish this: https://pypi.org/project/fasthx/ :slightly_smiling_face:

Your comments just triggered me to finish this: https://pypi.org/project/fasthx/ 🙂

I feel pretty good about this.
I will check it 🚀

I checked the source code and the README.md file. It looks quite useful but I think you should move the Jinja class to another file. Apart from that, it's a quite good utility package. I think you should send a PR to this repo PyHAT-stack/awesome-python-htmx: A curated list of things related to python-based web development using htmx to include fasthx into the list.

This is a pretty minimal package, there was no need to complicate the internal structure. Although it wouldn't hurt either. Thanks for you doc fix PR again!
And also thanks for the recommendation! I always struggle getting my projects to the hands of potential users...

This is a pretty minimal package, there was no need to complicate the internal structure. Although it wouldn't hurt either. Thanks for you doc fix PR again!

Happy to help 😇.

And also thanks for the recommendation! I always struggle getting my projects to the hands of potential users...

You're welcome. That's a problem you and I share...


[CWS] Allow Windows cross-compilation using mingw

What does this PR do?
Allow compiling system-probe for Windows on Linux.
Motivation
CWS supports Linux either using eBPF or ptrace, and Windows through a kernel driver.
Adding a feature something requires adding methods to all the different implementations
which often requires going back and forth to make sure its still compiles for the other implementations.
Having the ability to compile for Windows on Linux significantly eases this process.
Describe how to test/QA your changes

Install g++-mingw-w64-x86-64-posix gcc-mingw-w64binutils-mingw-w64-x86-64
Compile the agent with

inv -e build-messagetable --target=pe-x86-64 --host-target=x86_64-w64-mingw32
GOOS=windows CGO_ENABLED=1 CXX=x86_64-w64-mingw32-g++ CC=x86_64-w64-mingw32-gcc inv -e agent.build --build-exclude=zstd,systemd,trivy,python

Compile system-probe with

inv -e build-messagetable --target=pe-x86-64 --host-target=x86_64-w64-mingw32
GOOS=windows CGO_ENABLED=1 CXX=x86_64-w64-mingw32-g++ CC=x86_64-w64-mingw32-gcc inv -e system-probe.build-sysprobe-binary

Possible Drawbacks / Trade-offs
Additional Notes

/merge

/merge


Conform popup logic

Menu
Popover
Tooltip

These componenets are all using seperate implementations of logic for attaching and popping up on an anchor element.
Find a way to conform this logic using a popover hook.
Notes:

https://reactjs.org/docs/portals.html
https://medium.com/better-programming/create-a-tooltip-component-using-react-hooks-cb9fd00a2ff0
https://www.npmjs.com/package/react-popover-hook
https://github.com/popperjs/popper-core

Features:

Requested feature for popup placement #952

blocked by #887

See #719  as well


Improve the inventory hosts generation

Regarding the entries in /etc/hosts generated from the inventory:

Replace groups['all'] by hosts_inventory_to_hosts to use only the targeted group in playbook (can be all also)
Fix #11
Create a list of aliases to not add duplicates
Simplify the loop to not add duplicates
Improve the IPv6 part

Done. Tell me if it's look good for you.


Log integration CLI arguments

Executed integrations get their env-vars logged but not their CLI arguments.
Feature Description
Log CLI arguments when in verbose mode.
Actually this was commented out as it triggered a leak check failure for unknown reasons:
https://github.com/newrelic/infrastructure-agent/blob/ea1d9e27d9344744a8def31f1555bcf1aa8433db/internal/integrations/v4/executor/executor.go#L54-L55
Priority
Nice to Have

To be obfuscated: r.Args  (Executor)
The Executor Args (r.Args) variable contains the provided arguments to the integration excluding the binary name. The cmd.Args variable contains the whole command.


Fix security issue found by gas

Catch the errors that doesn't handled.

Coverage increased (+0.03%) to 64.47% when pulling 7f0c6fb4797402c2db7e8a355ac9a2fd290bfb57 on wy65701436:release-1.5.0-fix-sev into 5537eeb9d2953197d7c0fd80370eb6bbf210da72 on vmware:release-1.5.0.


libssl.so.1.0 unavailable

Describe the bug
spotify_player: error while loading shared libraries: libssl.so.1.0.0: cannot open shared object file: No such file or directory

To Reproduce
Run on any current arm64 debian, I guess
Expected behaviour
Linked with libssl1.1 (1.1.1w) or newer
https://packages.debian.org/search?suite=all&section=all&arch=any&searchon=names&keywords=libssl-dev
Log and backtrace
N/A
Screenshots
N/A
Environment

OS: Debian 1:6.6.31-1+rpt1 (2024-05-29) aarch64 GNU/Linux
Application version: https://github.com/aome510/spotify-player/releases/download/v0.18.2/spotify_player-aarch64-unknown-linux-gnu.tar.gz
Application features: default

Additional context
N/A

Linked with libssl1.1 (1.1.1w) or newer

Does this mean you have libssl installed on your machine but the version is different from the binary? Is it possible for you to download 1.0.0? Otherwise, you can try to install the app using cargo as described in https://github.com/aome510/spotify-player?tab=readme-ov-file#cargo

Is it possible for you to download 1.0.0?

I tried and it's 404 or non-arm64.

Otherwise, you can try to install the app using cargo

Hangs or takes forever on Pi Zero 2 – low RAM, slow SD card.
Managed to build via macOS VM: limactl start --vm-type=vz --mount-type=virtiofs --arch=aarch64 --plain --name=pi02 template://debian-12
Got binary linked to libssl.so.3, and it runs fine.


Implement from_yaml method for vector DB

See https://github.com/fjuma/wise-agents/issues/64

Superseded by https://github.com/wise-agents/wise-agents/issues/272.


Use a message broker in SAP Data Hub, developer edition 2.4

Tutorial URL: https://developers.sap.com/tutorials/datahub-pipelines-v2-broker.html
Please specify the step you are referring to:
On Step 1 when I try: docker run --name kafka --hostname kafka --network dev-net spotify/kafka
I get: docker: Error response from daemon: Conflict. The container name "/kafka" is already in use by container "5061c0986abd1ca086f37ddb356aafac59102451c632878230eeacbcfbefe3b8". You have to remove (or rename) that container to be able to reuse that name.
Sorry I don't know what I am doing, but I am trying to learn.

Hi there. The message indicates you have created one container with the name kafka already.
To check the list of created containers run
docker container list -a

and you should see something like
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS                        PORTS                                                                      NAMES
...
d8b2b3de789a        spotify/kafka       "supervisord -n"         5 minutes ago       Up 5 minutes                  2181/tcp, 9092/tcp                                                         kafka
...

Obviously, in your case kafka's container id would display 5061c0986abd.
If you want to remove it, just run
docker rm -f 5061c0986abd

and then try to rerun the command, which was failing previously for you.
I would really recommend to look at parts 1 and 2 of https://docs.docker.com/get-started/

Thank you for your feedback. We haven't heard from you in the past 30 days, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


Improve 'Comparison operation' documentation

From my point of view, the Comparison operation section is not clear enough.
<= Check if all given values are in a non-descending order. Returns a boolean.
<  Check if all given values are in ascending order. Returns a boolean.
>= Check if all given values are in non-ascending order. Returns a boolean.
>  Check if all given values are in descending order. Returns a boolean.

Investigate if something like that is more understandable.
<= Check if all given values are in ascending order (repeat values are not allowed). Returns a boolean.
<  Check if all given values are in ascending order (repeat values are allowed). Returns a boolean.
>= Check if all given values are in descending order (repeat values are not allowed). Returns a boolean.
>  Check if all given values are in descending order (repeat values are allowed). Returns a boolean.

I like the idea of unifying ascending and descending . But, why (repeat values are not allowed)?

I like the idea of unifying ascending and descending . But, why (repeat values are not allowed)?

Maybe fits better: (repeat values will return false)
What do you think?

I don't understand what's the problem with "repeated values" and why they need to be "remarked" as special ones.
 # using "<"
(< 1 2 3) === (1 < 2 < 3) # true
(< 2 2 2) === (2 < 2 < 2) # false

 # using "<="
(<= 1 2 3) === (1 <= 2 <= 3) # true
(<= 2 2 2) === (2 <= 2 <= 2) # true
(<= 2 2 3) === (2 <= 2 <= 3) # true
(<= 2 2 1) === (2 <= 2 <= 1) # false

The repeated values depend on their context in the same way as other numbers by using a specific operator. I don't think we need to "mark" them as especial numbers. Or am I missing something?

The idea is to remark that < and <= are not equals.
(< 1 2 3) # true
(<= 1 2 3) # true
---
(< 1 2 3 3) # false
(<= 1 2 3 3) # true

If we removed the repeated values, the sentence would be:
Check if all given values are in ascending order
That's why I put the repeated sentence.
Do you think is more clear as it is currently? :arrow_down:
<= Check if all given values are in a non-descending order. Returns a boolean.
<  Check if all given values are in ascending order. Returns a boolean.

What about this?
<= Check if all given values are in ascending (or equal) order. Returns a boolean.
<  Check if all given values are in ascending order. Returns a boolean.
>= Check if all given values are in descending (or equal) order. Returns a boolean.
>  Check if all given values are in descending order. Returns a boolean.

Why not this:
<= Checks if each argument is less than or equal to the following argument. Returns a boolean.
<  Checks if each argument is is strictly less than the following argument. Returns a boolean.
>= Checks if each argument is greater than or equal to the following argument. Returns a boolean.
>  Checks if each argument is strictly greater than the following argument. Returns a boolean.

I this this is much cleaner.

Hmm, I would pick @Chemaclass suggestion, This is cleaner for me.
What do you think @mabasic?

Considering that both suggestions are perfectly valid and the reader would understand both it does not matter which one is picked.
After reading both for the first time, I must admit that I find the one @jenshaase wrote last to be simpler to me personally. Ascending and descending always confuses me at first 😁.

@JesusValera This can be closed now. Correct?


Add the possiblity to load data by passing raw content

Summary

Adds support for a new datasource field for passing in raw data, e.g content.
Checklist

[x] My code follows the code style of this project and passes make format.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate unit test coverage.

Reviewing pull request by @homanp.
In the loader.py file:

It seems that the changes made are related to handling the case where the url field is empty in the Datasource model. Instead of making a request to the URL, the content of the datasource is used directly.
The changes look good and handle the case appropriately.

In the request.py file:

The Datasource model has a new optional field content added.
This field is used in the loader.py file to handle cases where the url field is empty.
The changes look good and provide a way to handle content directly without relying on a URL.

In the datasource.py file:

The _load_xlsx_data and _load_csv_data methods now accept a Datasource object instead of a URL.
This allows the methods to handle cases where the url field is empty and use the content directly.
The changes look good and provide a way to handle content directly without relying on a URL.

In the migration.sql file:

A new migration file is added to add a content column to the Datasource table.
This is necessary to support the changes made in the code.
The changes look good and provide the necessary database migration.

In the schema.prisma file:

The url field in the db datasource is changed to DATABASE_MIGRATION_URL.
This is likely to differentiate it from the regular DATABASE_URL used for the application.
The changes look good and provide a clear separation between the migration database URL and the application database URL.

Overall, the changes look good and provide a way to handle content directly without relying on a URL. Thank you, @homanp, for the contribution! 🎉


Adam/eval

eval monad, basic repl are working.

has the basics down, prim env. is next up. repl is running to test language during development.


MIC-401 refactor

Codecov Report
Attention: Patch coverage is 98.49624% with 6 lines in your changes missing coverage. Please review.

Project coverage is 90.53%. Comparing base (f2da8bc) to head (bda2f8f).

Files with missing lines
Patch %
Lines

...nterventionplanapi/model/referral/Investigation.kt
80.00%
2 Missing :warning:

...erventionplanapi/model/plan/request/PlanRequest.kt
90.00%
1 Missing :warning:

...ventionplanapi/model/plan/request/ReviewRequest.kt
94.44%
1 Missing :warning:

...odel/referral/request/DecisionAndActionsRequest.kt
90.90%
1 Missing :warning:

...api/model/referral/request/InvestigationRequest.kt
93.75%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #176      +/-   ##
============================================
+ Coverage     90.40%   90.53%   +0.13%
+ Complexity      786      681     -105
============================================
  Files           142      125      -17
  Lines          2126     2124       -2
  Branches        162      163       +1
============================================
+ Hits           1922     1923       +1
+ Misses          153      149       -4
- Partials         51       52       +1

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


Cannot set property 'value' of undefined

Nuxt project
When I focus to vue-masked-input and close overlay appears error like this:

fixed issue with modify your code


add gofmt to GitHub actions (#161)

Can golangci-lint static analysis tools also be added? You can visit https://github.com/golangci/golangci-lint

@sjcsjc123 Done. Adding other jobs to the pipeline might also be a good idea.

Yes, if there are suitable jobs, you can add them.

@sjcsjc123 sure, i will add them in a separate PR.


MiniDLNA: File Permissions

I run a Nextbox, which basically is a Nextcloud instance running in Docker on a Raspberry Pi. Nextcloud does not (yet?) support serving media via DLNA and the workaround is to just use MiniDLNA. Since maintenance of this Nextbox is done by Nitrokey, I do not want to mess with the host system too much, and rather run MiniDLNA also within Docker, so this is how I got here.
The media files in the Nextcloud data directory are in a Docker volume, which is no problem to mount also in the MiniDLNA container. But they are of course owned by www-data:www-data, so they can not be accessed by MiniDLNA:

[2021/11/13 16:25:30] minidlna.c:670: error: Media directory "/nextcloud/data/me/files" not accessible [Permission denied]

I tried to solve this by adding the environment variable MINIDLNA_USER=33 (UID of www-data) to this container, but this UID is assigned to another user xfs in this Alpine-based MiniDLNA container and so this gave me even more errors and resulted in the container exiting:

[2021/11/13 16:27:43] minidlna.c:670: error: Media directory "/nextcloud/data/me/files" not accessible [Permission denied]
minidlna.c:496: warn: Unable to change pidfile /minidlna/minidlna.pid ownership: Operation not permitted
minidlna.c:1065: error: Unable to set db_path [/minidlna/cache] ownership to 33: Operation not permitted
minidlna.c:1074: fatal: Failed to switch to uid '33'. [Operation not permitted] EXITING.

So I wonder whether changing the UID of the user running MiniDLNA in the container is actually the right approach. Do you have any suggestions how to gain access to the files, without changing the permissions on the host, because that will mess up the Nextcloud installation.

Here is the Docker Compose snippet I use
version: "3"

services:
  minidlna:
    image: vladgh/minidlna
    container_name: vladgh_minidlna
    restart: unless-stopped
    network_mode: "host"
    volumes:
      - nextbox-compose_nextcloud:/nextcloud
    environment:
      - MINIDLNA_MEDIA_DIR=/nextcloud/data/me/files
      - MINIDLNA_FRIENDLY_NAME=NextboxDLNA
   [  - MINIDLNA_USER=33 ]

volumes:
  nextbox-compose_nextcloud:
    external: true

This is very interesting. I am not sure how to resolve this, but can you try a few things?
First, it's not MINIDLNA_USER that you need to change but, PUID=33 and PGID=33, like this:
environment:
      - MINIDLNA_MEDIA_DIR=/nextcloud/data/me/files
      - MINIDLNA_FRIENDLY_NAME=NextboxDLNA
      - PUID=33
      - PGID=33

If that doesn't work, maybe changing it to 0 would (PUID=0 and PGID=0)
Another thing I could think of is to chmod -R 755 /nextcloud/data/me/files which would make all media files readable.
Let me know if any of that works!

PUID=33 and PGID=33 nailed it. Thank you very much! My parents are happy now :)


Feature/image labeling

I've accidentally closed the old PR(https://github.com/azihsoyn/flutter_mlkit/pull/19). So I've created the new one instead.
Sorry for the inconvenience, I'm very new to contribution in GitHub T^T.

Great! LGTM!
Please add a CHANGELOG.md entry and give the package version a bump to 0.3.0 in pubspec.yaml.

Updated  😁.

0.3.0 published! 🎉
https://pub.dartlang.org/packages/mlkit


bower?

Looks ideal. Any chance of sticking it on Bower?

bower install https://github.com/STRML/backbone.queryRouter.git


rqt_common_plugins indigo-EOL branch

https://github.com/ros-visualization/rqt_common_plugins/pull/459#issuecomment-604547280
https://github.com/ros-visualization/rqt_common_plugins/tree/indigo-EOL
AFAIK this avoids breaking users building indigo from source using the --upstream-development flag. There won't be any upstream development in an EOL distro, so I've used the branch name indigo-EOL instead of the usual indigo-devel.

Test failure is expected because this PR is changing an EOL distro
Build cache for "noetic"
- trying to use local cache
- trying to fetch cache
- update cache
- removing invalid release package cache entries.
- fetch missing release manifests
......ERROR: In file '''/home/runner/work/rosdistro/rosdistro/indigo/distribution.yaml''': There is a change to a doc section of an EOLed distribution. Lines: [13671]
F
....................................................................................................................................................................................................................................................................................................................................................
======================================================================
FAIL: test_function (test.test_url_validity.TestUrlValidity)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/rosdistro/rosdistro/test/test_url_validity.py", line 353, in test_function
    self.assertFalse(detected_errors)
AssertionError: ["In file '''/home/runner/work/rosdistro/rosdistro/indigo/distribution.yaml''': There is a change to a doc section of an EOLed distribution. Lines: [13671]"] is not false

Please rename the branch to indigo-devel as in all other ROS 1 repos.

Please rename the branch to indigo-devel as in all other ROS 1 repos.

Done e4237e6

Double checked failing actions are only failing due to change to EOL distro; merging with admin override


[WIP] fix(test): Run integration tests during build

Fixes fabric8-launcher/launcher-planning#98

[test]

Requires https://github.com/openshiftio/openshiftio-cico-jobs/pull/536

[test]

[test]

Retest this please

[test]

[test]

Requires https://github.com/openshiftio/openshiftio-cico-jobs/pull/543

[test]

[test]

[test]


Fixed import of payment and order exclusions in dashboards

In a dashboard, unlike a checkout, PaymentError and order errors are imported directly from packages. I replaced the import in the dashboard with a get_class call similar to checkout

@dd Please write unit test so the code coverage doesn't drop. Thanks!

I don't think anyone will ever need to change the exceptions by subclassing.
The get_class mechanism is only needed for this scenario. If you can demonstrate a case where this is needed, feel free to reopen the issue, thank you.


Prioritized reason code

Summary
In fqm-execution-demo, guidance responses with reason details that are not "Missing" or "Present" are prioritized, meaning that they are the first guidance responses listed.
New behavior
When running gaps in care, the guidance responses with reason code "Missing" or "Present" are now at the bottom. This was especially important for Bryn's new bundle because guidance responses with other reason codes were at the bottom.
Code changes
All code changes were made in DetectedIssueResources.tsx. All of the code to render the accordion remains the same, I just manipulated the guidanceResponseArray so that any guidance responses with reason codes other than "Missing" or "Present" were at the beginning of the array.
Testing guidance
I tested this with EX130 although the changes will not be obvious here since the array is already in the desired order. I would still check this to ensure that it is still working. I also tested it with ColorectalCancerScreeningCQM-bundle.json because the array in that bundle is not in the desired order. In addition, using this new bundle meant that I had to upgrade to fqm-execution version 0.5.0 for it to work.

So I'm seeing this behavior for the standard dropdown EXM130 bundles, I could be missing something

The new bundle that we used as a test case does work though. Maybe we can rework the for loop somehow


making sure e2e tests works after bumping Openshift version to 4.17 in Openshift/Release jobs

making sure e2e tests works after bumping Openshift version to 4.17 in Openshift/Release jobs.
this PR should not be merged.

/ok-to-test


Core -> CDN Integration: CDN configuration

This config is a requirement for "Upload assets to CDN" but it's worth having it here because it demands research


upsertGraph() with transaction example

I've read in the documentation that upsertGraph() is not atomic and should be used with transaction. Would you say this would be the right way to do it? I failed to find any examples on this.
    ...
    const opts: UpsertOptions = {
      relate: true,
      unrelate: true,
    }

    let updatedAd
    const trx = await transaction.start(AdModel.knex())
    try {
      updatedAd = await AdModel.query(trx)
        .upsertGraphAndFetch(graph, opts)
        .omit(['token'])
        .debug()
      await trx.commit()
    } catch (err) {
      console.error(err)
      await trx.rollback()
      return ctx.throw(...)
    }

The graph data I am inserting is pretty simple has only one nested owner relation like this:
{
        id: adId,
        classifiedCode: 'cllgb',
        sectionCode: 'hhh',
        categoryCode: 'apa',
        title: 'Updated title',
        description: 'Updated description',
        ...
        owner: {
          id: 23341
          email<EMAIL_ADDRESS>        },
        ...
      }

The transaction section is full of examples. All ways work with all methods. I didn't think I needed to add an example for each method.

@koskimas I am new to Objection.js and just wanted to make sure that I am properly using it with upsertGraphAndFetch() method in this particular situation.  It's a bit of a cluster f*** at times but you might be able to get more help there, and chat with other users live.
It's very difficult to keep a repository clean and maintain a low amount of issues and dangling PRs, so I agree with koskimas' decisions to default to closing most of them as soon as possible. Please don't take offense to this action as in the long run, it is good for the project as a whole.
If you want my opinion, I stay away from upsert/insert graph mainly because there are so many options and I prefer to be explicit about my operations, even if it means a bit longer code.
But if you must, you may also want to try the alternative syntax:
    const opts: UpsertOptions = {
      relate: true,
      unrelate: true,
    }

    let updatedAd
    try {
      updatedAt = await objection.transaction(AdModel.knex(), async () => {
        return AdModel.query(trx)
          .upsertGraphAndFetch(graph, opts)
          .omit(['token'])
          .debug()
      })
    } catch (e) {
      // transaction failed!
    }

This way commit and rollback are handled automatically.

@heisian Thank you so much Tim for the info and letting me know about Gitter. I will definitely check it out and use it moving forward.
What would also be helpful, if there was a brief message in the GitHub new issue template field explaining what type of issues should be filed here and what should be deferred to Gitter. I'd be more than happy to comply and keep the core team focused on real issues/bugs.

Yes I think it would be better to be explicit about the right places to find help.. I think somewhere out there some text even suggests posting a question in an issue, which I don't agree with, but ultimately that's up to koskimas. Feel free to ping me if you have other Objection questions.


Handle invalid logger names and types in /logging API

the /logging API exposed on ztunnel does not do any error checking for logging types and logging levels, leading to output such as:
$ k exec -it ds/ztunnel -n istio-system -- curl -XPOST localhost:15000/logging?level=thri:warning
current log level is thri:warning=trace,info

$ k exec -it ds/ztunnel -n istio-system -- curl -XPOST localhost:15000/logging?level=thri:warnting
current log level is thri:warnting=trace,thri:warning=trace,info

If the goal is feature parity with envoy's own /logging API, it handles both errors:
$ k exec -it istio-ingressgateway-5cd8b57d85-zrccs -n istio-system -- curl -XPOST localhost:15000/logging?paths=thrift:watning
error: unknown logger level

usage: /logging?<name>=<level> (change single level)
usage: /logging?paths=name1:level1,name2:level2,... (change multiple levels)
usage: /logging?level=<level> (change all levels)
levels: trace debug info warning error critical off
active loggers:
  admin: warning
  alternate_protocols_cache: warning
  aws: warning
   ...

$ k exec -it istio-ingressgateway-5cd8b57d85-zrccs -n istio-system -- curl -XPOST localhost:15000/logging?paths=thri:warning
error: unknown logger name

usage: /logging?<name>=<level> (change single level)
usage: /logging?paths=name1:level1,name2:level2,... (change multiple levels)
usage: /logging?level=<level> (change all levels)
levels: trace debug info warning error critical off
active loggers:
  admin: warning
  alternate_protocols_cache: warning
  aws: warning
  ...

might also be good to dump the help text on error, since it is only shown on invalid methods currently: https://github.com/istio/ztunnel/blob/master/src/admin.rs#L174-L180
long term, as ztunnel stabilizes, output the list of available loggers as well

Is it even possible? Not sure tracing library exposes this

Yeah, I guess there are 3 parts to this item:

validate log levels (HELP text on error)
validate log names (HELP text on error)
dump list of available loggers

is the only option that seems reasonable right now.

and 3) I wasn't sure If they were supported by the library or not.  It is unfortunate that you need to potentially throw everything to trace to try and capture all the loggers, and then determine which ones to are actually relevant to you

@ramaraochavali are you still working on this?

No. Sorry. I got sidetracked. Feel free to take it over.

/assign @MorrisLaw


docs(v2): Add Axioms to showcase

Add awe framework site

Motivation
(Write your motivation here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

lgtm thanks!


Pin pyatv to version with pinned aiohttp

Oh it's issue #5892 again. Home Assistant currently does not handle the case where a dependency depends on a core dependency of Home Assistant.
In this case it's caused by the AppleTV dependency pyatv depending on aiohttp. Aiohttp just released a new version with breaking changes for Home Assistant, causing Home Assistant to fail to start after the first time it installs AppleTV.
Fixes #6728

Found a better solution.


Handle NaNs when decoding times (failures on riscv64)

What happened?
FAILED xarray/tests/test_backends.py::TestScipyInMemoryData::test_roundtrip_numpy_datetime_data
FAILED xarray/tests/test_backends.py::TestScipyFileObject::test_roundtrip_numpy_datetime_data
FAILED xarray/tests/test_backends.py::TestGenericNetCDFData::test_roundtrip_numpy_datetime_data
FAILED xarray/tests/test_backends.py::TestScipyFilePath::test_roundtrip_numpy_datetime_data
= 4 failed, 4636 passed, 5632 skipped, 19 xfailed, 22 xpassed, 38 warnings in 266.18s (0:04:26) =
What did you expect to happen?
No failures
Minimal Complete Verifiable Example
pytest-3.10 -n auto /usr/lib/python3.10/site-packages/xarray

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
=================================== FAILURES ===================================
___________ TestScipyInMemoryData.test_roundtrip_numpy_datetime_data ___________
[gw2] linux -- Python 3.10.7 /usr/bin/python3.10

num_dates = array([ 0., nan]), units = 'days since 2000-01-01 00:00:00'
calendar = 'proleptic_gregorian', use_cftime = None

    def decode_cf_datetime(num_dates, units, calendar=None, use_cftime=None):
        """Given an array of numeric dates in netCDF format, convert it into a
        numpy array of date time objects.

        For standard (Gregorian) calendars, this function uses vectorized
        operations, which makes it much faster than cftime.num2date. In such a
        case, the returned array will be of type np.datetime64.

        Note that time unit in `units` must not be smaller than microseconds and
        not larger than days.

          Note this is safe even in the case
        # of np.uint64 values, because any np.uint64 value that would lead to
        # overflow when converting to np.int64 would not be representable with a
        # timedelta64 value, and therefore would raise an error in the lines above.
        if flat_num_dates.dtype.kind in "iu":
            flat_num_dates = flat_num_dates.astype(np.int64)

        # Cast input ordinals to integers of nanoseconds because pd.to_timedelta
        # works much faster when dealing with integers (GH 1399).
        flat_num_dates_ns_int = (flat_num_dates * _NS_PER_TIME_DELTA[delta]).astype(
            np.int64
        )

        # Use pd.to_timedelta to safely cast integer values to timedeltas,
        # and add those to a Timestamp to safely produce a DatetimeIndex.

        Note: the caller is responsible for ensuring that numpy warnings are
        suppressed (with np.errstate(all="ignore")) if needed.

        Parameters
        ----------
        left : np.ndarray or ExtensionArray
        right : object
            Cannot be a DataFrame or Index.  Series is *not* excluded.
        op : {operator.add, operator.sub, ...}
            Or one of the reversed variants from roperator.

        Returns
        -------
        ndarray or ExtensionArray
            Or a 2-tuple of these in the case of divmod or rdivmod.

        Performs the addition of an int64 array and an int64 integer (or array)
        but checks that they do not result in overflow first. For elements that
        are indicated to be NaN, whether or not there is overflow for that element
        is automatically ignored.

        Parameters
        ----------
        arr : array addend.
        b : array or scalar addend.
        arr_mask : np.ndarray[bool] or None, default None
            array indicating which elements to exclude from checking
        b_mask : np.ndarray[bool] or None, default None
            array or scalar indicating which element(s) to exclude from checking

        Returns
        -------
        sum : An array for elements x + b for each element x in arr if b is
              a scalar or an array for elements x + y for each element pair
              (x, y) in (arr, b).

        Raises
        ------
        OverflowError if any x + y exceeds the maximum or minimum int64 value.
        """
        # For performance reasons, we broadcast 'b' to the new array 'b2'
        # so that it has the same size as 'arr'.
        b2 = np.broadcast_to(b, arr.shape)
        if b_mask is not None:
            # We do the same broadcasting for b_mask as well.
            b2_mask = np.broadcast_to(b_mask, arr.shape)
        else:
            b2_mask = None

        # For elements that are NaN, regardless of their value, we should
        # ignore whether they overflow or not when doing the checked add.
        if arr_mask is not None and b2_mask is not None:
            not_nan = np.logical_not(arr_mask | b2_mask)
        elif arr_mask is not None:
            not_nan = np.logical_not(arr_mask)
        elif b_mask is not None:
            # Argument 1 to "__call__" of "_UFunc_Nin1_Nout1" has incompatible type
            # "Optional[ndarray[Any, dtype[bool_]]]"; expected
            # "Union[_SupportsArray[dtype[Any]], _NestedSequence[_SupportsArray[dtype[An
            # y]]], bool, int, float, complex, str, bytes, _NestedSequence[Union[bool,
            # int, float, complex, str, bytes]]]"  [arg-type]
            not_nan = np.logical_not(b2_mask)  # type: ignore[arg-type]
        else:
            not_nan = np.empty(arr.shape, dtype=bool)
            not_nan.fill(True)

        # gh-14324: For each element in 'arr' and its corresponding element
        # in 'b2', we check the sign of the element in 'b2'. If it is positive,
        # we then check whether its sum with the element in 'arr' exceeds
        # np.iinfo(np.int64).max. If so, we have an overflow error. If it
        # it is negative, we then check whether its sum with the element in
        # 'arr' exceeds np.iinfo(np.int64).min. If so, we have an overflow
        # error as well.
        i8max = lib.i8max
        i8min = iNaT

        mask1 = b2 > 0
        mask2 = b2 < 0

        if not mask1.any():
            to_raise = ((i8min - b2 > arr) & not_nan).any()
        elif not mask2.any():
            to_raise = ((i8max - b2 < arr) & not_nan).any()
        else:
            to_raise = ((i8max - b2[mask1] < arr[mask1]) & not_nan[mask1]).any() or (
                (i8min - b2[mask2] > arr[mask2]) & not_nan[mask2]
            ).any()

        if to_raise:
>           raise OverflowError("Overflow in int64 addition")
E           OverflowError: Overflow in int64 addition

/usr/lib64/python3.10/site-packages/pandas/core/algorithms.py:1114: OverflowError

During handling of the above exception, another exception occurred:

data = <xarray.backends.scipy_.ScipyArrayWrapper object at 0x40238999c0>
units = 'days since 2000-01-01 00:00:00', calendar = 'proleptic_gregorian'
use_cftime = None

    def _decode_cf_datetime_dtype(data, units, calendar, use_cftime):
        # Verify that at least the first and last date can be decoded
        # successfully.

        For standard (Gregorian) calendars, this function uses vectorized
        operations, which makes it much faster than cftime.num2date. In such a
        case, the returned array will be of type np.datetime64.

        Note that time unit in `units` must not be smaller than microseconds and
        not larger than days.

         Otherwise, tracebacks end up swallowed by
        # Dataset.__repr__ when users try to view their lazily decoded array.
        values = indexing.ImplicitToExplicitIndexingAdapter(indexing.as_indexable(data))
        example_value = np.concatenate(
            [first_n_items(values, 1) or [0], last_item(values) or [0]]
        )

        try:
            result = decode_cf_datetime(example_value, units, calendar, use_cftime)
        except Exception:
            calendar_msg = (
                "the default calendar" if calendar is None else f"calendar {calendar!r}"
            )
            msg = (
                f"unable to decode time units {units!r} with {calendar_msg!r}. Try "
                "opening your dataset with decode_times=False or installing cftime "
                "if it is not installed."
            )
>           raise ValueError(msg)
E           ValueError: unable to decode time units 'days since 2000-01-01 00:00:00' with "calendar 'proleptic_gregorian'". Try opening your dataset with decode_times=False or installing cftime if it is not installed.

/home/abuild/rpmbuild/BUILD/xarray-2022.6.0/xarray/coding/times.py:190: ValueError

Anything else we need to know?
https://build.opensuse.org/package/live_build_log/openSUSE:Factory:RISCV/python-xarray/standard/riscv64

import xarray as xr
import numpy as np
import pandas as pd
num_dates = np.asarray([0., np.nan])
flat_num_dates = num_dates.ravel()
flat_num_dates_ns_int = (flat_num_dates * (int(1e9) * 60 * 60 * 24)).astype(np.int64)
flat_num_dates_ns_int
array([                  0,<PHONE_NUMBER>854775807])
pd.to_timedelta(flat_num_dates_ns_int, "ns")
TimedeltaIndex(['0 days 00:00:00', '106751 days 23:47:16.854775807'], dtype='timedelta64[ns]', freq=None)
pd.to_timedelta(flat_num_dates, "ns")
TimedeltaIndex(['0 days', NaT], dtype='timedelta64[ns]', freq=None)

Environment

/usr/lib/python3.10/site-packages/_distutils_hack/init.py:33: UserWarning: Setuptools is replacing distutils.
warnings.warn("Setuptools is replacing distutils.")

On Sep 28 2022, Maximilian Roos wrote:

It looks lie many of these occur in pandas code — do pandas tests pass?

That's because xarray is passing bogus values.

What are the bogus values?
Please could you answer the question on whether pandas tests pass?

array([ 0,<PHONE_NUMBER>854775807])

I'm not sure what that has to do with xarray though? Does this give the same result?
import numpy as np
import pandas as pd
num_dates = np.asarray([0., np.nan])
flat_num_dates = num_dates.ravel()
flat_num_dates_ns_int = (flat_num_dates * (int(1e9) * 60 * 60 * 24)).astype(np.int64)
flat_num_dates_ns_int
array([ 0,<PHONE_NUMBER>854775807])

Please could you answer the question on whether pandas tests pass?

We're here helping as volunteers; we can only engage on issues if you reciprocate our good faith. Please could you answer this?

Closing but please feel free to reopen

Hi, we are getting similar failures when building xarray for Arch Linux riscv64.

I'm not sure what that has to do with xarray though? Does this give the same result?
import numpy as np
import pandas as pd
num_dates = np.asarray([0., np.nan])
flat_num_dates = num_dates.ravel()
flat_num_dates_ns_int = (flat_num_dates * (int(1e9) * 60 * 60 * 24)).astype(np.int64)
flat_num_dates_ns_int
array([ 0,<PHONE_NUMBER>854775807])

I got the same result in riscv64. One thing I could guess is that the sign bit of NaN is not kept during conversions. Some more details could be found at: https://sourceware.org/pipermail/libc-alpha/2022-September/142011.html
Repeating the same steps result in array([0, -9223372036854775808]) in x86_64 and array([0, 0]) in aarch64.

Please could you answer the question on whether pandas tests pass?

I have tried pandas' tests and got many failures like:
E       AssertionError: Attributes of DataFrame.iloc[:, 4] (column name="date") are different
E
E       Attribute "dtype" are different
E       [left]:  float64
E       [right]: datetime64[ns]

or
E           AssertionError: numpy array are different
E
E           numpy array values are different (50.0 %)
E           [index]: [0, 1]
E           [left]:  [1036713600000, -9223372036854775808]
E           [right]: [1036713600000000000, -9223372036854775808]

Quite some of the tests are having NaN in the context as well. So you are probably right that pandas or numpy may be where the problem lies.

I got the same result in riscv64. One thing I could guess is that the sign bit of NaN is not kept during conversions. Some more details could be found at

Thanks for trying that. Notably, that code doesn't have xarray in. So I'm keen to be part of the solution, but it doesn't look to be a problem with xarray code specifically. Let me know if that makes sense.

As in #7098

I think the real solution here is to explicitly handle NaNs during the decoding step. We do want these to be NaT in the output.

@felixonmars If you are still in the works with this, I'd appreciate if you could test this against #7827. Thanks.

@kmuehlbauer Sure. I have verified that the tests are passing on #7827 and failing on the current main branch.


Ground getHeightAtCoordinates debug

Vector3.CrossToRef function was modifying Tmp.Vector3[0]'s value to store its result. But GroundMesh.prototype._computeHeightQuads was both using Tmp.Vector3[0]'s value, and calling Vector3.CrossToRef, resulting in an undesired value in Tmp.Vector3[0] and a bug.

nice catch :)

Thanks !

I don't get where computeHeightQuads is still using a conflicting
Tmp.Vector3[0] ?
yep, they both used once the same tmp vector, but not a conflicting way
imho (i fixed this months ago)
Le 25/07/2016 à 14:47, PierreHei a écrit :

Thanks !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BabylonJS/Babylon.js/pull/1269#issuecomment-234943172,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAtCXhDHpXjzyZ4qMHNdsugd2_0wbZxks5qZLBUgaJpZM4JUEef.

In computeHeightQuads, v1 is Tmp.Vector3[0], then crossToRef modified Tmp.Vector3[0] to calculate its result, modifying v1 by the way, but v1 is still used after call to crossToRef !

nope
this was fixed long ago :
https://github.com/BabylonJS/Babylon.js/blob/master/src/Mesh/babylon.groundMesh.ts#L139
v1 is Tmp.Vector3[3], not [0]
are you sure that you're using the latest source ?
Le 25/07/2016 à 16:09, PierreHei a écrit :

In computeHeightQuads, v1 is Tmp.Vector3[0], then crossToRef modified
Tmp.Vector3[0] to calculate its result, modifying v1 by the way, but
v1 is still used after call to crossToRef !
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BabylonJS/Babylon.js/pull/1269#issuecomment-234963708,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAtCdrRnoEmsYyI17RK8BzwuaP3qVdRks5qZMOYgaJpZM4JUEef.

Clearly you're right, I saw this on 2.4 release and only modified Vector3.crossToRef source to fix, thus I didn't saw there already was a fix in Mesh.babylon.groundMesh. Sorry for losing your time !

no worry :-)
Le 25/07/2016 à 16:28, PierreHei a écrit :

Clearly you're right, I saw this on 2.4 release and only modified
Vector3.crossToRef source to fix, thus I didn't saw there already was
a fix in Mesh.babylon.groundMesh. Sorry for losing your time !
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BabylonJS/Babylon.js/pull/1269#issuecomment-234969125,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAtCcdJoX-Vmi195kfxfcMjmO0h-gJrks5qZMgCgaJpZM4JUEef.


Use default 'kubectl proxy' port of 8001

This is consistent with all other links throughout spinnaker

@EricJizbaMSFT,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


DbTester - a JUnit 5 extension to bootstrap test databases, provide integration w/Testcontainers

Setting up a DB for unit testing is a pain:

Out of the box "bootique-jdbc" only supports Derby
Even Derby tests require a YAML config file
There's no direct integration with Testcontainers (though possible to do it manually)
There's no consistent way to run init scripts (though Testcontainers support their own API)
There's no way to run Liquibase (or Flyway) migrations
TestDataManager / DataChannel / Table API requires too much manual coding and has an unclear lifecycle.
Test module is autoloadable, so it can't be turned off once on classpath

Thinking we might address this with DbTester test utility class that may work as follows:
@BQTest
public class Test {

     // DbTester will
     //  * setup a Testcontainers-based DB or an in-memory Derby DB,
     //  * init it with provided scripts
     //  * provide the tests with access to DB metadata (Tables), and hence - matchers

    @RegisterExtension
    static final DbTester db = DbTester
        // either Derby or testcontainers (which in turn can be any DB)
    	.testcontainersDb("jdbc:tc:postgresql:11:///mydb")
    	.initDB("classpath:schema.sql")
    	.runLiquibaseMigrations("classpath:changelog.yml")
         // this replaces TestDataManager
        .deleteBeforeEachTest("t1", "t2");

    @BQApp
    static BQRuntime app = Bootique.app("-s")
        .autoLoadModules()

        // stick DbTester's DataSource in Bootique. No extra YAML config required!!
        // also note that test module is not autoloadable anymore. It is provided explicitly
        .module(db.setOrReplaceDataSource("myname"))
        .createRuntime();

     @Test
     public void testMe() {
          // improvement on DataChannel approach - tables are generated automatically and are
          // readily accessible
          Table t1 = db.getTable("t1");
      }
}

Done, except for Liquibase, that will be implemented per #97


feat: add object metadata methods to the DataLakeClient in MindSphere SDK

if you have never contributed to open source here is your chance: 😸
[0] Comment on this issue that you will take care of this feature impementation ;)
[1] fork the repository (click on the Fork button)
[2] Clone it to your local system and install the dependencies by running npm install in the cloned directory. (you will need nodejs in your development environment)
[3] Implement the feature
Navigate to https://developer.mindsphere.io/apis/iot-integrated-data-lake/api-integrated-data-lake-api-swagger-v3-2-0.html
to take a look at Object Metadata operations and the documentation

in the src/api/sdk/data-lake.ts
implement the PutObjectMetadata and GetObjectMetadata methods by applying the same principles like for other methods in the file:

the path parameters belong to method signature
the query parameters go to optional?: {...} parameter

Here is how the signature for the GetObjectMetadata method should look like
public async GetObjectMetaData(
        path: string,
        optional?: { subtenantid?: string }
    ): Promise<DataLakeModels.ObjectMetaDataResponse> {
        // call await this.HttpAction  with correct verb and baseUrl
        // return result casted to the result type (get the correct type from documentation, all necessary typescript interfaces are already available in DataLakeModels namespace
    }

Add documentation to the 2 methods by creating a comment above and copy/pasting the documentation from the web page. (see examples in the same file)
You are done. 🎉 🎶 😸 👍
[4] Push the changes back to your repository
[5] Click the Compare & pull request button.
[6] Click Create pull request to open a new pull request.
See also:
https://opensource.com/article/19/7/create-pull-request-github
Happy Hacktoberfest!
https://hacktoberfest.digitalocean.com/

I'll take a stab at this

@amarisch  sure go ahead :)

@amarisch do you need some help with the issue? hacktoberfest is almost over :)

@amarisch do you need some help with the issue? hacktoberfest is almost over :)

I apologize! I tried to play with mindconnect but coudn't get application credential/authorization to work, and then got carried away by other projects...
So I did:
"mc service-credentials" to set up credentials for cli. But when I tried "mc list-assets", it gave me,
Network error occured Unauthorized {"status":null,"errorMessage":null,"message":null,"customCode":"KMUL01","errorCode":null,"logRef":"5f9bda2e0faf1d762f21b514a1622a3f","displayMessage":"Bad Credentials","exceptionStackTrace":null}
And when I ran npm test, I received this message: "Error: You have to pass the basic authentication header (Basic:  in the constructor. Wrong Passkey in CLI?" I believe it's also related to authentication?
Are the issues only open to the public in October?

@amarisch No worries, the issues are open to the public also after October, I just thought that you want to get your hacktoberfest points 😺 . If you are having problems with authorization, delete the credentials in both CLI and the mindsphere and recreate them, you have to have exact configuration like in your developer cockpit .
Btw, I updated the CONTRIBUTING.md so that it provides a bit more infos and there is a video online too to help with setting up the CLI.
https://opensource.mindsphere.io/docs/mindconnect-nodejs/cli/setting-up-the-cli.html#tab1anchor3


wait for k8s to be ready

Coverage remained the same at 51.886% when pulling 159ab4b805f2bc7af95e6db191263a8f33693fc7 on hao/mitigate_jenkins into 9f599ed41bca0fcd204052c4f37bf5d35d757398 on master.


CLEANUP - Token.rb test coverage with two more predicates

Follow up of https://github.com/bbatsov/rubocop/pull/5210#issuecomment-351693102 .
This PR adds 2 more type predicate methods to Token: end? and rescue_modifier?. It also adds a ## Type Predicate code comment and groups these methods together at the end of the file.
The majority of this diff is adding test coverage for all the methods in Token.

[x] Wrote good commit messages.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

👍


[!!!] Autostopping bot as search failed, please check if you can access transfer market in Web App

After a few searches it stops and displays " Autostopping bot as search failed, please check if you can access transfer market in Web App" not sure what this is. I cans till search and buy players but the autobuyer seems to say that constantly

Today me too

Hi bro! its mostly because the transfer market is blocked. U can't use the auto buyer until the transfer windows open.To do that you need to play 20-25 matches in different mode

Yea, its the same with me...so if the market its not open on web app you cant use the autobayer?
Ive played more then 20-25 matches and still its not open


🐛 [firebase_storage] App crashes on iOS when trying to upload image using image_picker

Bug report
Describe the bug
App crashes on iOS when trying to upload image to Storage, selected from image_picker, despite the code being wrapped in try/catch.
Here's error code:
[VERBOSE-2:ui_dart_state.cc(157)] Unhandled Exception: PlatformException(storage_error, NSInternalInconsistencyException, URI must be in the form of gs://<bucket>/)
#0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:569:7)
#1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:156:18)
<asynchronous suspension>
#2      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:329:12)
#3      _StorageFileUploadTask._platformStart (package:firebase_storage/src/upload_task.dart:130:36)
#4      StorageUploadTask._start (package:firebase_storage/src/upload_task.dart:35:21)
#5      StorageReference.putFile (package:firebase_storage/src/storage_reference.dart:65:10)

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all
details, run flutter doctor
-v):
[✓] Flutter (Channel stable,
    1.22.0, on Mac OS X
    10.15.7 19H2, locale
    ru-UA)

[✓] Android toolchain -          develop for Android          devices (Android SDK
    version 30.0.0)
[✓] Xcode - develop for iOS
    and macOS (Xcode 11.7)
[✓] Android Studio (version
    4.0)
[✓] VS Code (version 1.49.2)
[✓] Connected device (1
    available)

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
image_picker: ^0.6.6+1
firebase_storage: 4.0.1

Here's the code:
var image = await ImagePicker().getImage(source: ImageSource.gallery);

    /// Pushing image to Firebase Storage
    if (image != null) {
      setState(() {
        _loadingProfilePhoto = true;
      });
      StorageReference ref = FirebaseStorage.instance.ref();
      StorageTaskSnapshot addImg;
      try {
        addImg = await ref
            .child("profiles/${Storage.customerId}")
            .putFile(File(image.path))
            .onComplete;
      } catch (error) {
        print(error);
      }

Can confirm this works this using the prerelease version of firebase_storage.
    var image = await ImagePicker().getImage(source: ImageSource.gallery);
    if (image == null) return;

    firebase_storage.UploadTask task = firebase_storage.FirebaseStorage.instance
        .ref('testImage.png')
        .putFile(File(image.path));

    try {
       await task;
    } on FirebaseException catch (e) {
       print(e);
    }

@helenaford
Thanks for response.
Just tried your solution, but it throws error in my case with following message:
[VERBOSE-2:ui_dart_state.cc(157)] Unhandled Exception: [core/no-app] No Firebase App '[DEFAULT]' has been created - call Firebase.initializeApp()
#0      MethodChannelFirebase.app (package:firebase_core_platform_interface/src/method_channel/method_channel_firebase.dart:118:5)
#1      Firebase.app (package:firebase_core/src/firebase.dart:52:41)
#2      FirebaseStorage.instance (package:firebase_storage/src/firebase_storage.dart:53:21)
...

No problem, happy to help. For the error that's being thrown, please read the migration guide (https://firebase.flutter.dev/docs/migration), in particular this part (https://firebase.flutter.dev/docs/migration#4-initialize-flutterfire).
Will need to initialize the app first with await Firebase.initializeApp();.
Closing this issue as I believe it's fixed in the rework of Storage.


ExcelJS with Angular and Typescript

I'm working on integrating ExcelJS into a browser based node.js client built using the angular framework and typescript (and grunt). I previously tested the library in a separate project and was able to get it working using grunt with browserify, but this required me to bundle my own app.js script with a 'require' on the exceljs dependency.
My current project is designed to reference minified libraries by declaring script tags in index.html, and type definitions in a separate typings folder. I've done both for exceljs, but when I attempt to run my code (which declares a new Workbook via new Excel.Workbook()), I get the following error:
ReferenceError: Can't find variable: Excel in app/src/myScript.js (line 56)
Also Note: When I try and add a require statement on exceljs in the file that needs it, I get another ReferenceError:
ReferenceError: Can't find variable: require
What steps am I missing in integrating the library such that Excel will be recognized as the correct namespace? Or is there another way I should be declaring a Workbook?

In which way you import exceljs?
I use ExcelJs in Angular (4) and TS too, with the following import:
import * as Excel from 'exceljs';
after that, you can use Exceljs:
 const myWorkbook = new Excel.Workbook();

I've tried adding that line as it is into my angular service responsible for exporting xlsx. I get the following error when I run my unit tests:
ReferenceError: Can't find variable: define

Check your environment and unit tests. This is no error in context of Exceljs.

I see. I'll do some more leg work and see if I'm missing something crucial. I know I've seen some things regarding angular mocks, perhaps I've not configured my project correctly to mock Excel.
As a quick question though, where do you place that import statement in your angular project? Do you place it in the class which requires it? Or do you place it in one of your module declarations?

As a quick question though, where do you place that import statement in your angular project? Do you place it in the class which requires it? Or do you place it in one of your module declarations?

i've created an service, in which the import of exceljs takes place and works with it. Nowhere else.

@Gargamil I am trying to use exceljs in Angular 4 but my app fails to compile when I try to import and use it. I am importing and using it exactly as you've said: import * as Excel from "exceljs"; and let workbook = new Excel.Workbook();. The webpack errors I get are:
WARNING in ./~/natives/index.js
44:17-28 Critical dependency: the request of a dependency is an expression

and
ERROR in ./~/natives/index.js
Module not found: Error: Can't resolve 'module' in '...\node_modules\natives'
 @ ./~/natives/index.js 2:13-30
 @ ./~/unzip2/~/graceful-fs/fs.js
 @ ./~/unzip2/~/graceful-fs/graceful-fs.js
 @ ./~/unzip2/~/fstream/lib/reader.js
 @ ./~/unzip2/~/fstream/fstream.js
 @ ./~/unzip2/lib/extract.js
 @ ./~/unzip2/unzip.js
 @ ./~/exceljs/dist/es5/stream/xlsx/workbook-reader.js
 @ ./~/exceljs/dist/es5/index.js
 @ ./~/client.common/src/app/pages/landing/landing.component.ts
 @ ./~/client.common/src/app/app.module.ts
 @ ./~/client.common/module.ts
 @ ./src/app/app.module.ts
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://localhost:4202 ./src/main.ts

Removing the import and the code that uses it mitigates all errors. I have also tried including dist/exceljs.min.js and dist/es5/exceljs.browser.js in the scripts section of .angular-cli.json but got exactly the same results. Does any of that ring any bells? I'm stumped.

I got it working by using:
import * as Excel from "exceljs/dist/exceljs.min.js";

rather than
import * as Excel from "exceljs";

However, now I have the issue that type definitions are not being picked up for exceljs. This is pretty annoying.

I have found a workaround for no type definitions:
import * as Excel from "exceljs/dist/exceljs.min.js";
import * as ExcelProper from "exceljs";

let workbook: ExcelProper.Workbook = new Excel.Workbook();

As long as you just use the import from exceljs for type definitions and only use the import from exceljs/dist/exceljs.min.js, the app will run just fine and you still get type safety.

No workaround needed anymore: example.

After wasting an hour time found this workaround working for me instead of the above
import * as ExcelJS from "exceljs/dist/exceljs.min.js";
let workbook = new ExcelJS.Workbook();
Accessing it with name ExcelJS solved issue.
Hope this helps.

Team,
I am using Angular 5 and i need to use Exceljs in my aplication. Could you please what is the exact version for accessing exceljs in angular 5.
page is not rendering if i use work book in excel.service file.
FYI.
I  am using Angular 5 with "exceljs": "^1.6.2",.
Kindly help me any one.

After wasting an hour time found this workaround working for me instead of the above
import * as ExcelJS from "exceljs/dist/exceljs.min.js";
let workbook = new ExcelJS.Workbook();
Accessing it with name ExcelJS solved issue.
Hope this helps.

Hi,
Can you tell me which exceljs version you used in angular application. Help me plse.


What is comparable version of istio for AKS 1.19.11

Went through multiple article and couldn't find a better answer for my query.
Would like to know what version of istio need to be required for AKS 1.19.11.
We will be going for AKS upgrade from 1.18.14 to 1.19.11.
Since istio current version is 1.7.3 which version
should go for istio to compatible with 1.19.11??
Is there any documents to upgrade istio version in AKS

https://istio.io/latest/docs/releases/supported-releases/#support-status-of-istio-releases describes supported Kubernetes versions

@howardjohn  Thanks for your quick help.  So hope we can directly upgrade from Istion 1.7 to latest version 1.10 ( currently our AKS is of version is1.18.14 and planning to upgrade to 1.19.11 )
Also would like to know how istio image can be pulled from our private ACR repo than the dockerhub ( We have private ACR registry inplace, however its noticed that Istio is pulling image from docker.io, wehich is not a trusted registry of Azure Security Center as per the policy set in our environment, we are getting lot of security vulnerability alerts due to this.
Secondly, looking to remediate the container previlege escalation related to istio, as istio containers are running as root user. Our application containers are running as non root user and because of Istio we are getting the pod itself vulberable.
So would like to know if we can get rid of the above 2 vulnerability alerts when we go for Istio upgrade. Will this latest version will have solution inbuilt for this or during upgrade can we remediate these by any chance?


FullGroupJoin: should provide a default selector

Much like GroupJoin has overloads that return an IGrouping, FullGroupJoin should offer overloads that provide a default return aggregator:
public static IEnumerable<FullGrouping<TKey, TFirst, TSecond>> FullGroupJoin<TFirst, TSecond, TKey, TResult>(this IEnumerable<TFirst> first, IEnumerable<TSecond> second, Func<TFirst, TKey> firstKeySelector, Func<TSecond, TKey> secondKeySelector) {
    // Null checks
    return MoreEnumerable.FullGroupJoin(first, second, firstKeySelector, secondKeySelector, (k, ga, gb) => new FullGrouping<TK, TA, TB>(k, ga, gb));
}

// another overload with equality comparer

public class FullGrouping<TKey, TLeft, TRight>
{
    public TKey Key { get; }
    public IEnumerable<TLeft> Left { get; }
    public IEnumerable<TRight> Right { get; }

    public FullGrouping(TKey key, IEnumerable<TLeft> left, IEnumerable<TRight> right)
    {
        Key = key;
        Left = left;
        Right = right;
    }
}

What do you think? I can propose a PR if this sounds reasonble.

I think this would introduce a new type that adds very marginal value because its implementation is too trivial (something that can be added to a project using MoreLINQ without it feeling too ugly). IGrouping is already part of the .NET so making sense out of re-using it is a little easier. We should wait out and see how tuples in C# 7 (dotnet/roslyn#347) could help with such overloads more simply without introducing new (essentially small record) types.

I think this would introduce a new type that adds very marginal value because its implementation is too trivial (something that can be added to a project using MoreLINQ without it feeling too ugly).

Indeed, I already have it in my projects. I was hoping to avoid more copy paste :wink:

We should wait out and see how tuples in C# 7 (dotnet/roslyn#347) could help with such overloads more simply without introducing new (essentially small record) types.

Ack. I hope it proves useful.

Ack

Cool, so we can close this now & revisit this (along with probably many other operators that could use reasonable default selector) when C# 7 is out?

On 16 September 2016 at 10:47, Atif Aziz<EMAIL_ADDRESS>wrote:

Ack
Cool, so we can close this now & revisit this (along with probably many
other operators that could use reasonable default selector) when C# 7 is
out?

I'd suggest instead applying a label (eg, postponed/cs-7 or something like
that) so that these issues can be easily skipped, but are not forgotten
once C#7 is out.
--
Saludos,
Felipe Sateler

I think this is do-able now:
static IEnumerable<(TKey Key, IEnumerable<TFirst> First, IEnumerable<TSecond> Second)>
    FullGroupJoin<TFirst, TSecond, TKey, TResult>(
        this IEnumerable<TFirst> first,
        IEnumerable<TSecond> second,
        Func<TFirst, TKey> firstKeySelector,
        Func<TSecond, TKey> secondKeySelector) =>
    first.FullGroupJoin(second, firstKeySelector, secondKeySelector, ValueTuple.Create);

Indeed!

@fsateler Go for a PR whenever you get a chance!


Emitter plugin not working properly with the latest NgXS/Store version

For some reason when injecting the Emitter service or when emitting an action using decorators the state gets changed to "undefined", it is working correctly using the Store 3.3.4 but when updating to 3.4.1 I get the error, please check this stackblitz:
https://stackblitz.com/edit/ngxs-emitter-error
Thanks in advance.

We need update to 3.4.0 and add integration tests

Fixed in 3.4.2

https://github.com/ngxs/store/issues/902

Whoa this made me swear for a good evening, just learning ngxs I thought I was missing something


fix(deps): update terraform github.com/terraform-ibm-modules/terraform-ibm-powervs-infrastructure to v8.1.3

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-powervs-infrastructure
module
patch
v8.1.2 -> v8.1.3

Release Notes

terraform-ibm-modules/terraform-ibm-powervs-infrastructure
v8.1.3
Compare Source
Bug Fixes

deps: update terraform github.com/terraform-ibm-modules/terraform-ibm-landing-zone to v4.0.1 (#510) (9f4745c)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 6.2.2 :tada:
The release is available on:

GitHub release
v6.2.2

Your semantic-release bot :package::rocket:


Nice state in Garage Door Opener

Ciao!
Just a question,
It's possible to have in garagedooropener the 4 states?
Fibaro's output of properties.value (for Nice Gates) are 4 :
Closed, Opened, Opening, Closing.
Thanks
(se posso aiutarti sono a disposizione!)

@meme1981 is there also the value stopped ?

@meme1981, can you test the beta version ?
Use homebridge-fibaro-home-center@beta to install.

Se non ricordo male sei italiano giusto?
Inviato da iPhone
Il giorno 10 gen 2022, alle ore 14:31, ilcato @.***> ha scritto:

@meme1981https://github.com/meme1981 is there also the value stopped ?
—
Reply to this email directly, view it on GitHubhttps://github.com/ilcato/homebridge-fibaro-home-center/issues/80#issuecomment-1008876782, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH4ZQA6IG3UE5ERE2ZMXW5DUVLNSRANCNFSM5LHPOPHA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

oui

Perfetto!!
Il quinto stato è “Unknow”
Se lo stoppi a metà lui va in “Unknow”
Comunque arrivo a casa e ricontrollo.
Questo con L antenna OXBidi che e’ bidirezionale ovviamente.
Appena arrivò a casa installo la beta e proviamo subito
Grazie!!
Inviato da iPhone
Il giorno 10 gen 2022, alle ore 17:45, ilcato @.***> ha scritto:

oui
—
Reply to this email directly, view it on GitHubhttps://github.com/ilcato/homebridge-fibaro-home-center/issues/80#issuecomment-1009101199, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH4ZQA5OQI6Y3V4ZQ6IVZF3UVMEL5ANCNFSM5LHPOPHA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

Va in crash homebridge, dopo riprovo perché dalla macchina fatico
[cid:CED8D095-5198-46D6-A214-9E7806DF7A55-L0-001]
Inviato da iPhone
Il giorno 10 gen 2022, alle ore 17:45, ilcato @.***> ha scritto:

oui
—
Reply to this email directly, view it on GitHubhttps://github.com/ilcato/homebridge-fibaro-home-center/issues/80#issuecomment-1009101199, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH4ZQA5OQI6Y3V4ZQ6IVZF3UVMEL5ANCNFSM5LHPOPHA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

Ti confermo i 5 stati:
Opened
Opening
Closing
Closed
Unknow
Se si stoppa va in Unknow
Ho reinstallato la 1.2.7-beta ma va in crash

Da: ilcato @.>
Inviato: lunedì 10 gennaio 2022 17:45
A: ilcato/homebridge-fibaro-home-center @.>
Cc: meme1981 @.>; Mention @.>
Oggetto: Re: [ilcato/homebridge-fibaro-home-center] Nice state in Garage Door Opener (Issue #80)
oui
—
Reply to this email directly, view it on GitHubhttps://github.com/ilcato/homebridge-fibaro-home-center/issues/80#issuecomment-1009101199, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH4ZQA5OQI6Y3V4ZQ6IVZF3UVMEL5ANCNFSM5LHPOPHA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

@meme1981, can you test it with the new beta?

Hi, now it works but no refresh state, i need to close the app and open it again, Other problem is this: when i Open it's always in Closing...
If this can help with Nodered i use this function:
if (msg.payload.properties.state === "Opened") {
msg.payload.CurrentDoorState = 0;
msg.payload.TargetDoorState = 0;
}
else  if (msg.payload.properties.state  === "Opening") {
msg.payload.CurrentDoorState = 1;
msg.payload.TargetDoorState = 0;
}
else  if (msg.payload.properties.state  === "Closing") {
msg.payload.CurrentDoorState = 0;
msg.payload.TargetDoorState = 1;
}
else  if (msg.payload.properties.state  === "Closed") {
msg.payload.CurrentDoorState = 1;
msg.payload.TargetDoorState = 1;
}
else  if (msg.payload.properties.state  === "Unknow") {
msg.payload.CurrentDoorState = 4;
msg.payload.TargetDoorState = 1;
}
return msg;
Thank you!!

Da: ilcato @.>
Inviato: martedì 11 gennaio 2022 16:22
A: ilcato/homebridge-fibaro-home-center @.>
Cc: meme1981 @.>; Mention @.>
Oggetto: Re: [ilcato/homebridge-fibaro-home-center] Nice state in Garage Door Opener (Issue #80)
@meme1981https://github.com/meme1981, can you test it with the new beta?
—
Reply to this email directly, view it on GitHubhttps://github.com/ilcato/homebridge-fibaro-home-center/issues/80#issuecomment-1010069526, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AH4ZQA3LFOVY3B3ZRQKUQBLUVRDKBANCNFSM5LHPOPHA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

now it works

What does it mean? Are the new state reported correctly beside the refresh issue?

problem 1:
when the door or gate is “opening” the device signals closing.
problem 2:
the status does not follow the polling period: after movement the door is closed or open ( in Fibaro too) and on the homekit app it appears “closing” until the homekit app is closed and reopened.
Thanks
Inviato da iPhone
Il giorno 12 gen 2022, alle ore 14:28, ilcato @.***> ha scritto:
What does it mean? Are the new state reported correctly beside the refresh issue?

@meme1981, can you test it with the new beta?


Adds Laravel 5.5. autoload support

Thanks hombre!

No problemo 🤙🏻

@adamwathan Can we get a new release (tag) for this now that Laravel 5.5 is out?

Done!


updated code to match file names

What kind of change does this PR introduce?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

👷 Deploy Preview for betsweatwin processing.

Name
Link

🔨 Latest commit
41afd1811d6dcff42ab38d7ffe39b8a7dd8c2c39

🔍 Latest deploy log
https://app.netlify.com/sites/betsweatwin/deploys/65389bf751a5490008dee12d


Conditional validation based on earlier step/form

Hello,
i have a form based on multiple forms.
Step1Form
Step2Form
Step3Form
etc..
How can i validate data in step2, but based on answer in step1 ?
I think i somehow need to have access to "storage" from the clean(..) methond on my forms, but how ?

class ImportWizard(SessionWizardView):
   def get_form(self, step=None, data=None, files=None):
        form = super(ImportWizard, self).get_form(step, data, files)

        # determine the step if not given
        if not step:
            step = self.steps.current

        if step == '0':
            # do something

        if step == '1':
            if self.request.POST.getlist('import_wizard-current_step')[0] == '0':
                # This is the intermediate point where form has info from the previous step
            if self.request.POST.getlist('import_wizard-current_step')[0] == '1':

        if step == '2':
            if self.request.POST.getlist('import_wizard-current_step')[0] == '1':
                # This is the intermediate point where form has info from the previous step
            if self.request.POST.getlist('import_wizard-current_step')[0] == '2'
    return form


Suggestion

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---
I have added a new command line in the package.json that simplifiers the running of storybook, and I also modified the 'start' command line for consistency.

:rocket: PR was released in v1.0.3 :rocket: