Rework lan_browser devices gauges to use labels instead

Hi,
Thanks for the metrics.
I just tried to implement that in Grafana, but it's either too hacky or not possible it seems ...
Do you think there would be a way to export the metrics as labels instead?
So instead of
fbx_exporter_lan_browser_pub_ether_aa_bb_cc_dd_ee_ff_l3connectivities_1_addr{addr="<IP_ADDRESS>"} 1
Something like
fbx_exporter_lan_browser_pub_ether_addr{port="1", type="l2", mac="aa_bb_cc_dd_ee_ff", addr="<IP_ADDRESS>"} 1

_Originally posted by @ottobaer in https://github.com/shackerd/freebox-exporter-rs/issues/22#issuecomment-2294774809

Hello,
I'm not satisfied neither of this metrics format, it's not really exploitable, I'm not surprised. I also though to rearrange subdata in labels.
Actually, the process of gathering devices and translating data into prometheus format is:
1. Fetch network interfaces

2. Enumerate devices per network interface (we need to iterate on each iface, with previous data)

3. Allocate gauges / wrap gauges to each level of object cluster for refreshing data

Cons:
* By processing like this, it brings a lot of cognitive complexity and eat a bit of memory/cpu, some gauges may be useless

* Gauges data may not be consistent (missing or 0) specially when there are some devices entry/exit in network
  Re/deallocating gauges is complex, not impossible but pointless.

I tried to keep the Freebox API genesis, such as structure/path in metrics and avoid any data processing for more consistency, but seems complicated.
So, I think I will rearrange data like the format right down below (this is an example of std node_exporter):
# HELP node_network_info Non-numeric data from /sys/class/net/<iface>, value is always 1.
# TYPE node_network_info gauge
node_network_info{address="00:00:00:00:00:00",adminstate="up",broadcast="00:00:00:00:00:00",device="lo",duplex="",ifalias="",operstate="unknown"} 1
node_network_info{address="ff:ff:ff:ff:ff:ff",adminstate="up",broadcast="ff:ff:ff:ff:ff:ff",device="eth0",duplex="full",ifalias="",operstate="up"} 1

This will look like:
fbx_exporter_lan_browser_devices{port="1", type="l2", mac="aa_bb_cc_dd_ee_f0", addr="<IP_ADDRESS>", iface="pub"} 1
fbx_exporter_lan_browser_devices{port="2", type="l2", mac="aa_bb_cc_dd_ee_f1", addr="<IP_ADDRESS>", iface="pub"} 1
fbx_exporter_lan_browser_devices{port="3", type="l2", mac="aa_bb_cc_dd_ee_f2", addr="<IP_ADDRESS>", iface="pub"} 1

But, their is a painpoint, the l3connectivities property (which is an array of objects), I will try to flatten data or also rework it.

Originally posted by @shackerd in https://github.com/shackerd/freebox-exporter-rs/issues/22#issuecomment-2295031321

As you said I think this is the best way to do that and to make it exploitable by Grafana without resorting to re-labeling in prometheus or other hacks.
I don't think the absence of interfaces is a big problem - there is a defined maximum of interfaces possible and the absence of data is still useful metrics data in the end.

Hello @ottobaer, sorry I have been a bit busy these days, does the output in #46 seems exploitable for you? If so, I will merge the PR and create a new release.

Hi,
Just tested it - works fine with the labels:

What's the problem with the f80 ipv6?

I think when hosts count is done, it takes only private addresses such as fe80 prefix, there are 3 ipv4 and 5 ipv6 addresses in the example (taken from my network). But count is only 5 hosts on the iface. I think the 2a01:e0a: prefix (which is proxad reserved) is not taken in the count process. A property (like private: true) in L3 Connectivity  object would resolve this mistake. Nevermind, actually no IP address is hidden is our metrics. I will ask later on freebox bug report platform.


Improve the global-state-update-gen tool to create accounts for validators when they don't exist

Closes #2885

Merging for now, the supply issues will be addressed in another ticket.


Read Data Fr Databases: Methods Available to a Cursor

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Changes will make it render like this


No downloads since January - this is June

I'd been waiting to open the torrents for The Handmaids Tale, download and watch them - only to find that not only has that show never downloaded (ShowRSS says it has, but there's no sign of it on my computer), NONE of my other shows have anything new since January! (I often DVR, so this is a backup for most of what I watch, so I can watch elsewhere).
Console FREQUENTLY shows that it's going out to look for the files, but I see that's produced nothing. Everything looks fine on ShowRSS... so what's happening here? Thanks!

Did you do any of the things listed in the issue template?

Yes, thank you for asking. I tried them ALL prior to writing... Unless missed something. But I'd first tried all my own thoughts, and then ensured I followed those too. Still no downloads.

I don’t think the alphas would help with this specific issue, but they won’t hurt :)
Do you see the episodes you expect in the "Recent Episodes" window?

Yes - annoyingly, I do see them in the "Recent Episodes" window - so I go looking all over my computer for them (thinking maybe they forgot where to download). No such luck - they just don't exist...
So - now that you asked, I tried to look again. Now, remember there was NOTHING since January 11th in my TV Recordings folder. I look again today - and after messing with this - somehow at 3:14am, 2 versions of episode 09 TRIED to download - but they're .webloc files, not torrents (they even made their own folder, as requested). No previous episodes downloaded (and I have dozens of series on my RSS feed - none of THEM tried to download at all).
The ONLY thing I did when messing with this the other day was RE-enter my RSS feed address.
Wish I knew what happened on January 11th to change things in the first place!
I fear Alphas if I can avoid them. JUST switched to Sierra (finally), which is what made me look at all this in the first place... never want to add something that may de-stabilize a new OS, when I'm just trying to get it up and running with my apps......

Torrent files are effectively dead, so the only thing we have are magnet links. If you have "Open automatically” unchecked, they will be saved as .webloc files which you can double-click to start the download. Otherwise, check it and.. the downloads will open automatically :)
As for the backlog, you were probably using a feed with magnets disabled (which wouldn’t have worked because .torrent files are dead).
I think this explains it, I’m closing but feel free to reply.

I use the machine on which I'm working, so Open automatically would be terrible, since I teach software online. I can't check "Open Automatically". I don't have an alternate machine to use - this is my storage and server.
When I click the webloc files, it opens Transmission, and gives me the warning as placed on my video at (https://1drv.ms/v/s!Ahg-Yg9KScG3gYsnlG7PDXlCewk0Lg).  :(
Also - NONE of my other feeds did anything at all. The "Recent Episodes" section says they're there, but they, unlike The Handmaid's Tale, didn't even create a folder of weblocs or add to their existing folders. Do I need to turn on Magnets for each feed? If that's the case, any idea how I can get those webloc files to actually do the automatic download as it should be?
So - since I can't use Open Automatically, does this mean that Catch and ShowRSS are going to be dead to me? That would make me really really sad.....

ADDITIONAL - so, I tried again with opening with Transmission. It sat for a while with that error that says it needed a Torrent - but eventually started downloading - YAY!
EXCEPT that I'm still not getting any downloads from my other feeds. Oh, and Magnet's been on all along. Never turned that off....

Magnets used to be off by default way back then. And yes, when you open them in Transmission it takes a bit to start downloading because they’re magnets.
Catch only supports one feed, the one in the Preferences... not sure what you mean by “other feeds”?

Sorry- wrong terminology - only one show coming down (the Handmaid's Tale) now - but my other shows on the feed aren't coming down.. (I'd turned on Magnets some time ago, based on a suggestion you'd made a ways back) See this image - all the shows that show they did work (Recent Episodes), vs the specified folder - sorted by date modified. Gotham and Lip Sync Battle should be up at the top, along with the Handmaids Tale that worked (I have lots of shows in my ShowRSS that haven't done anything since that January 11th date)

You have an optimistic understanding of “date modified” for folders ;) Have you checked inside?
Btw you can redownload anything from the list with the little buttons in each row.


[FLINK-4934] [gelly] Triadic Census

A triad is any three vertices in a graph. The triadic census produces counts for each of the 4 undirected and 16 directed triad types.

Hi @greghogan,
do we really need a 12k-line csv and a 32k-line csv to test this?

The RMat graph is used across many tests and the intent was to verify the larger tests against secondary frameworks. I have no problem leaving out the files as the command is documented in the base test class.

@vasia the graphs have been removed.

Thanks @greghogan. Can you also please update the docs to add the new library methods? It would be nice if you could include what is the triadic census, when it is useful, and usage info.


[1761] Build sites to stop duplicate code error

Context
Test is flagging as sites are building an extra provider when created.
Changes proposed in this pull request

Sites are now built into the provider.

Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Fix flaking site test


Kirby CMS version is not most current available.

Kirby CMS version is not most current available.
while I have the latest (at the moment) version of Kirby installed (3.6.0-RC.4)?

because of sorting with rc. :shrug:


GUI queue keep loading forever with no logs

I can spin up the GUI and change settings but all the widgets and the main process freeze at the queue loading phase forever, the server running with no logs.

Reinstalled the entire package solved my issue


Replace np.matrix calls and other warnings [DNM]

This PR is an extension of #333, so it is DNM until 333 is done.
I will refresh this once #333 is to fix the edits, so if you want to hold off on review, thats fine too.
This PR replaces all of the np.matrix calls and manipulates the
objects as arrays only. See https://numpy.org/doc/1.18/user/numpy-for-matlab-users.html
for why this needed to be done.
However, that does mean there are some syntax changes I had to make since
* on arrays is element-wise (matrix multiplication can be done with
@ operator since Python 3.5, though you can do np.matmul(a,b) to
the same effect ).
And transpose of 1-D arrays doesn't do anything like it does to a matrix, but I found
doing x - np.astleast_2d(x).T does, while still being arrays and not
matrices. Note: this is MUCH safer than assuming its 1-D and doing x - x[np.newaxis].T
which will cause very silly things to happen if x is already 2-D
That said, there's alot of replacements here and I'm hoping someone
else can look over this and make sure I have the correct matrix operations
(namely element-wise vs matrix-style) in place for the correct results.
This does pass my local tests.

See this PR to Lnaden/pymbar (#1) to clean this matrix code up additionally by removing some unneeded type casting and doing x + x.T with a simpler x + np.vstack(x).

There may be opportunities to use opt_einsum for both performance and simplicity.

I think things should be relatively simple now.  I'm fairly certain that the logsum is by far the limiting factor, so accelerating that is the biggest bottleneck, and should be dealt with before anything else is tried.

I made a git typo on this, fixing.

This should be fixed now

I'd say go ahead and merge.


Running django-cms instance in read-only mode

Because of security reasons, it would be nice if a django-cms instance could be started in read-only mode without the django.contrib.admin app.
Currently if I remove django.contrib.admin, admin .urls and the cms_toolbar from the project, there are errors in pageadmin.py caused by the following import:
from django.contrib.admin.models import LogEntry, CHANGE

Maybe just removing any superuser / staff flag from any user in the database?

Hello @g10f , please make sure to submit questions within the appropriate channels on either the User Mailing List, IRC (freenode#django-cms) or Stackoverflow.
You can find additional information on our support page.
In this case, I believe your question is best in the Developers Mailing List.

This is certainly not a non-issue and should be reopened. Any deployment that matters will not want to expose any admin URL or page to the general public, and service this in a restricted area (e.g. client auth or VPN).


K8SPXC-642: Update labels in sts template

Test name
Status

recreate
passed

restore-to-encrypted-cluster
passed

demand-backup
passed

pitr
passed

demand-backup-encrypted-with-tls
failed

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/849/commits/18313c196a076e4f73f9a7d32d880e7481f3af9d
image: perconalab/percona-xtradb-cluster-operator:PR-849-18313c19

Test name
Status

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/849/commits/18313c196a076e4f73f9a7d32d880e7481f3af9d
image: perconalab/percona-xtradb-cluster-operator:PR-849-18313c19

Test name
Status

scaling
passed

storage
passed

haproxy
passed

big-data
passed

recreate
passed

upgrade-haproxy
passed

restore-to-encrypted-cluster
passed

demand-backup
passed

pitr
passed

demand-backup-encrypted-with-tls
passed

self-healing
failed

init-deploy
passed

scaling-proxysql
passed

security-context
passed

upgrade-proxysql
passed

limits
passed

scheduled-backup
failed

monitoring-2-0
passed

smart-update
failed

affinity
passed

self-healing-advanced
passed

one-pod
passed

auto-tuning
passed

operator-self-healing
passed

proxysql-sidecar-res-limits
passed

users
passed

upgrade-consistency
passed

tls-issue-self
passed

tls-issue-cert-manager
failed

tls-issue-cert-manager-ref
passed

validation-hook
failed

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/849/commits/18313c196a076e4f73f9a7d32d880e7481f3af9d
image: perconalab/percona-xtradb-cluster-operator:PR-849-18313c19

Test name
Status

scaling
passed

scaling-proxysql
passed

security-context
passed

storage
passed

self-healing
failed

big-data
passed

recreate
passed

restore-to-encrypted-cluster
passed

demand-backup
passed

upgrade-haproxy
passed

pitr
passed

demand-backup-encrypted-with-tls
passed

scheduled-backup
failed

upgrade-proxysql
passed

smart-update
passed

self-healing-advanced
passed

operator-self-healing
passed

upgrade-consistency
passed

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/849/commits/18313c196a076e4f73f9a7d32d880e7481f3af9d
image: perconalab/percona-xtradb-cluster-operator:PR-849-18313c19

Test name
Status

scaling
passed

scaling-proxysql
passed

security-context
passed

haproxy
passed

recreate
passed

init-deploy
passed

restore-to-encrypted-cluster
passed

limits
passed

demand-backup
passed

monitoring-2-0
passed

pitr
passed

affinity
passed

demand-backup-encrypted-with-tls
passed

one-pod
passed

scheduled-backup
failed

auto-tuning
passed

proxysql-sidecar-res-limits
passed

storage
passed

users
passed

self-healing
failed

tls-issue-self
passed

tls-issue-cert-manager
passed

big-data
passed

upgrade-haproxy
passed

upgrade-proxysql
passed

tls-issue-cert-manager-ref
passed

validation-hook
passed

smart-update
passed

upgrade-consistency
passed

self-healing-advanced
passed

operator-self-healing
passed

commit: https://github.com/percona/percona-xtradb-cluster-operator/pull/849/commits/18313c196a076e4f73f9a7d32d880e7481f3af9d
image: perconalab/percona-xtradb-cluster-operator:PR-849-18313c19


Proper loading of 3rd party rulesets and reporters.

Unblocks using them on on java 9+.
Base on ideas discussed in #495 and #573 PRs.
To test - execute following steps:
$ ./gradlew :ktlint-ruleset-template:jar
...
$ ./gradlew sJE
...
$ docker run --rm -ti -w "/command" -v `pwd`:/command openjdk:11-jdk-slim ./ktlint/build/run/ktlint --debug --experimental --ruleset=ktlint-ruleset-template/build/libs/ktlint-ruleset-template.jar
[DEBUG] Discovered ruleset with "standard" id.
[DEBUG] Discovered ruleset with "custom" id.
[DEBUG] Discovered ruleset with "experimental" id.
[DEBUG] Discovered reporter with "checkstyle" id.
[DEBUG] Discovered reporter with "json" id.
[DEBUG] Discovered reporter with "plain" id.
[DEBUG] Initializing "plain" reporter with {verbose=false, color=false}
[DEBUG] Checking /command/ktlint/src/main/kotlin/com/pinterest/ktlint/Main.kt
...

You should see that ruleset with "custom" id is discovered and lint failed with a lot of errors "Unexpected var, use val instead (cannot be auto-corrected)".
Closes #259.

thank you!


Issue with prop types in typescript 4.6.2

Just updated to latest typescript version and got following ts error on MaskedView component:
Found possibly related issue on StackOverflow
https://stackoverflow.com/a/69221278/911930

Same

You can open<EMAIL_ADDRESS>and change this:
declare class MaskedViewComponent extends React.Component<MaskedViewProps> {}
declare const MaskedViewBase: ReactNative.Constructor<
  ReactNative.NativeMethods
> &
  typeof MaskedViewComponent;
export default class MaskedView extends MaskedViewBase {}

into
declare class MaskedViewComponent extends React.Component<MaskedViewProps> {}
export default class MaskedView extends MaskedViewComponent {}

pls fix this

pls fix this

+1

I ran into the same issue. This worked for me: https://stackoverflow.com/a/74145168
I added "react": ["node_modules/@types/react"] to my tsconfig.json file:
    ...
    "compilerOptions": {
        ...
        "paths": {
            "react": ["node_modules/@types/react"]
        }
    }
}```

The repository was moved to @react-native-masked-view/masked-view. This will resolve the issue.

Closing Issue as it is resolved.


Support pip

Please support this package and ufo2fdk with pip install!
Thank you!

This can be closed.

good catch!


chore: fix typo in test_omnienv_no_such_command.py

intall -> install

Closing this as all typos are on purpose.


Kong unable to forward to upstream url using uris

Summary
I have configured uris for my api and In my client i have mentioned uri name along with my path to the resource .
it is not redirecting to my mentioned upstream url.
instead if i include my Host header(which is not required according to the kong DOCS
) along with my URL(in client) it is working.
Steps To Reproduce
1.Register an API:
name: 'mec'
uris: '/mecmanager'
upstream_url: 'http://My_Server/'
hosts : 'tmm.com'
strip_uri: 'true'
2. make a request using rest client (RESTer in firefox)
GET '/mecmanager/path/to/resource'
Response : 404 not found
3.make a request
GET '/mecmanager/path/to/resource'
include header Host:tmm.com
Response: 200 OK
As my strip_uri property is true . There is no need of including my Host Header as per kong Docs (Refer strip_uri property in https://getkong.org/docs/0.11.x/proxy/)
Additional Details & Logs

Kong version 0.11.1
Operating System - Ubuntu 16.04

Thank you @thibaultcha


_stack_chk_fail();   { Does not return }

Version and Platform (required):

Binary Ninja Version:3.6.4546-dev Personal

Bug Description:
I think when programs go into stack chk fail
binninja should not assume that it does not return
Although really do not return
Screenshots:

Additional Information:
Please add any other context about the problem here.

Sorry I am not sure what you mean here. Could you please explain what BN is currently doing and what you expect it to do?

From the LSB Core Spec 4.0: "The interface __stack_chk_fail() shall abort the function that called it with a message that a stack overflow has been detected. The program that called the function shall then exit."
This is marked noreturn in most (all?) implementations of libc, like so: https://github.com/bminor/glibc/blob/ec6b95c3303c700eb89eebeda2d7264cc184a796/debug/stack_chk_fail.c#L22.
So, us marking __stack_chk_fail as noreturn isn't an issue in and of itself.
But, what can be an issue, is if we mark a function that calls __stack_chk_fail as noreturn if it could, otherwise, return. It's not clear to me if that's what's happening in this case from the screenshots, but it has been an issue in the past.
@imethod is this the issue you're having? And, if so, are you able to give us any additional information on the executable (or possibly share it)? Otherwise, if I'm incorrect and this is not the issue you're having, we'll definitely need a more detailed explanation.

We are closing this issue because we believe the current analysis is doing things correctly. Please feel free to re-open the issue if you have more inputs.
Just a reminder that, the stack check function does not make the parent function no-return. Otherwise, you should see a __noreturn string at the function prototype

From the LSB Core Spec 4.0: "The interface __stack_chk_fail() shall abort the function that called it with a message that a stack overflow has been detected. The program that called the function shall then exit."
This is marked noreturn in most (all?) implementations of libc, like so: https://github.com/bminor/glibc/blob/ec6b95c3303c700eb89eebeda2d7264cc184a796/debug/stack_chk_fail.c#L22.
So, us marking __stack_chk_fail as noreturn isn't an issue in and of itself.
But, what can be an issue, is if we mark a function that calls __stack_chk_fail as noreturn if it could, otherwise, return. It's not clear to me if that's what's happening in this case from the screenshots, but it has been an issue in the past.
@imethod is this the issue you're having? And, if so, are you able to give us any additional information on the executable (or possibly share it)? Otherwise, if I'm incorrect and this is not the issue you're having, we'll definitely need a more detailed explanation.
What I mean is stack_check func It's code generated by the compiler and I think it should be optimized for disassembly with push pop and stuff like that

Do you mean that we shouldn't display the call in HLIL/pseudo-c but should show it as disassembly? I don't understand why.
Even function prologues and epilogues that don't call stack check fail are compiler generated but that's not a good reason to not decompile them.
Or are you asking something else?

What I mean is stack_check func It's code generated by the compiler and I think it should be optimized for disassembly with push pop and stuff like that

In the disassembly view, where we show blx #__stack_chk_fail, unless I'm missing something...that's the actual disassembly. There are no "pushes" or "pops" or "stuff like that". It's just a branch.
I have no idea what platform/architecture this is because no binary was given, but I'm going to guess it's ARMv7 Thumb Linux. If you look at the bytes listed to the left of the instruction (24f0ceee), you can disassemble those with any other disassembler to check that it really is a blx instruction (e.g. via Shell Storm's Online Disassembler:
0x0000000000000000:  24 F0 CE EE    blx #0x24da0

We're not doing any kind of magical "fixup" or anything here in disassembly, to my knowledge. We're showing you exactly what's there in the binary...which is a branch to another location.
On some other platforms or compilers, __stack_chk_fail will look different and might even be inlined entirely. But, on this particular platform and with this particular binary, as far as I can tell, we are faithfully and accurately representing the disassembly to you.

Do you mean that we shouldn't display the call in HLIL/pseudo-c but should show it as disassembly? I don't understand why.
Even function prologues and epilogues that don't call stack check fail are compiler generated but that's not a good reason to not decompile them.
Or are you asking something else?

Yeah, you're right

您的意思是我们不应该在 HLIL/pseudo-c 中显示调用，而应该将其显示为反汇编吗？我不明白为什么。
即使不调用堆栈检查失败的函数序言和结尾也是编译器生成的，但这并不是不反编译它们的好理由。
或者你还想问别的事吗？

Do you mean that we shouldn't display the call in HLIL/pseudo-c but should show it as disassembly? I don't understand why.
Even function prologues and epilogues that don't call stack check fail are compiler generated but that's not a good reason to not decompile them.
Or are you asking something else?
I mean it can be removed when llir is elevated to mlil

Do you mean that we shouldn't display the call in HLIL/pseudo-c but should show it as disassembly? I don't understand why.
Even function prologues and epilogues that don't call stack check fail are compiler generated but that's not a good reason to not decompile them.
Or are you asking something else?

I mean it can be removed when llir is elevated to mlil , After all, this is really not useful in some very small function it's very tedious, right


Can't create a new desktop

I've just installed chunkwm (and skhd) and I'm trying to get chunkwm to create new desktops so that I can move my windows around. I can't seem to get the window to be created. I'm on High Sierra and using the example config for both chunkwm and skhd. I've also tried just running chunkc tiling::desktop --create from a terminal window and nothing happens. Any help would be appreciated!
Also, this is the output from chunkwm when I run chunkc tiling::desktop --create:
2018-10-07 19:01:33 DEBUG: chunkwm: processing event of type 'ChunkWM_PluginCommand'
2018-10-07 19:01:33 DEBUG:     command: 'c', arg: '(null)'

It's quite likely that you forgot to install the scripting addition.
On Mojave this requires SIP to be disabled all the time, on High Sierra you need SIP to be disabled only for installation and can re-enable it afterwards.
sudo chunkwm --install-sa
pkill Dock

Yes that worked! Thank you! Just didn't see those docs.


Suggestion: More Themes

Suggestion
Add more Themes :3

If you would like more themes, feel free to add them.


Update time format on HUD history tool

Fixing #1256

To address the DCO requirement you'll need to sign-off the commit(s):

https://github.com/zaproxy/zaproxy/blob/main/CONTRIBUTING.md#developer-certificate-of-origin
https://git-scm.com/docs/git-commit#Documentation/git-commit.txt---signoff

Hello guys, anything I can do to advance the subject ?

The changelog needs to be updated: https://github.com/zaproxy/zap-hud/blob/main/CONTRIBUTING.md
The PR is not properly linked, for valid keywords see: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

@thc202 can you verify that we are good ?

Updated, can you check that we are good ?

Thank you!


Use BLT ENABLE_BENCHMARKS cmake option. Test profiling/ benchmarks in CI.

EDIT: The branch name is misleading because halfway in working on this, I realized ENABLE_BENCHMARKS is a BLT option. That said, this line should remove the SERAC prefix, so it can be properly included.
I also noticed that benchmarks and profiling options were not being tested in CI. Caliper and adiak usage should be tested as they're starting to be used more. I'm assuming turning on the profiling and benchmarks options only adds more code coverage, which is why I turned them on for all LC CI jobs.

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

Project coverage is 90.59%.  Consider uploading reports for the commit 2ae6b09 to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1113   +/-   ##
========================================
  Coverage    90.59%   90.59%
========================================
  Files          161      161
  Lines        14412    14412
========================================
  Hits         13057    13057
  Misses        1355     1355

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

We have talked about this vaguely in two meetings but to be clear about what we should do. We should have two CMake options:

SERAC_ENABLE_BENCHMARKS - this should enable the actual benchmark executables themselves
SERAC_ENABLE_PROFILING - this should enable the caliper C++ macros and their use inside of Serac's libraries

Benchmarks are reliant on Profiling being on but we should be able to turn on Profiling without having the benchmarks built.
They should also be guarded for our project so downstream projects can turn off functionalities they don't need.

Thanks for the clarification @white238. I updated this PR's description to adjust for recent changes.

@btalamini @samuelpmish @jamiebramwell Just need one more review.


Add composer.json

Fixes up #5 with a license and documentation. Thanks, @camfindlay @DamienMetzger @gaillafr @unclecheese, and sorry for the tragically slow motion here!

:+1:


homepage proposal Claudia

Proposal for home page,  you visit the homepage by going to
http://localhost:8000/home

we are not using FA, see material design icons
dont include the css here (put them in bower)
please follow the current developments, we are aiming for a clean simple design based in MD
is there a copyright for this image?


feat: add draft API for metric-based scheduling

Description of your changes
This PR adds the draft API for metric-based scheduling.
Fixes #
I have:

[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested
N/A
Special notes for your reviewer
The PR is for early discussion preparation only; all names, fields, structures are not finalized. Any code inconsistencies are also not yet addressed.

Closing the PR due to latest developments; will send a new one


add 1-hour 2-hour slot patterns to whole foods script

Thanks for doing this. I have a few doubts, before I put this for testing.

Has this been tested with only the new changes only? "1 hour and 2 hour"?
Do you have a screenshot that shows when these occurred?
Someone had already pointed out 1hr slots and I had already added an additional fix for it to the script:

try:          slot_opened_text = "Not available"          all_dates = soup.findAll("div", {"class": "ufss-date-select-toggle-text-availability"})          for each_date in all_dates:             if slot_opened_text not in each_date.text:                print('SLOTS OPEN!')                os.system('say "Slots for delivery opened!"')                no_open_slots = False                time.sleep(1400)       except AttributeError:          continue
^ does exactly that. Its irrespective of which hour window opens up.

The current code does not work. I missed open slots 2 times because of this issue, thanks StarWindMoonCloud , was able to get an order placed with this fix.


The mask prevents the event when Select Field is open.

when select field is active, there is a full screen div on the front.
<div style="height:100%;width:100%;position:fixed;top:0;left:0;z-index:5;" data-reactid=".<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.0.1.0.$_0.$deviceType.<IP_ADDRESS>">

I think it used for clickaway feature. However, if the window has scroll, it prevents the scroll event.
More bad siuation is if select list is over screen bottom, user can't scroll  to select them

With SelectField now using Popover. this is fixed.


Securityctrl development

This code will allow Ansible to do three things:

Get deployment id by deployment name from Elastic Cloud to be used in subsequent jobs so the user doesn't have to know the deployment id
Create/Get an Elastic agent policy on any deployment
Create Elastic Endpoint Security Baseline with the following properties

Agent Policy will be created
System, Windows, Linux, Endpoint Security, and Prebuilt Rules will be configured as integrations
Endpoint Security rule will be activated
Endpoint Security will be registered as antivirus for Windows

Agreed, but not sure how to do that…let me try a few things
From: Mike Garuccio @.>
Sent: Monday, December 6, 2021 11:01 AM
To: Expedient/ansible-collection-elastic @.>
Cc: Scott, Ian @.>; Assign @.>
Subject: Re: [Expedient/ansible-collection-elastic] Securityctrl development (PR #5)
[ CAUTION: External Email ]
@mgaruccio requested changes on this pull request.
Overall I'm good with the new code as long as everyone else is, but we need to take care of the credential in that vars.yml file and make sure it's not in the commit history anywhere before this get's merged

In vars.ymlhttps://github.com/Expedient/ansible-collection-elastic/pull/5#discussion_r763195440:

@@ -0,0 +1 @@

+ece_password: @.****b5fauKw%ZTTV
This is gonna have to go, and we need to make sure it's out of the commit history as well
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/Expedient/ansible-collection-elastic/pull/5#pullrequestreview-824215871, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWA6XJKFZ5G7UKBWWIW2GR3UPTT4RANCNFSM5JPDFIAQ.
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.

The structuring needs adjustments here. All of the new files under the module_utils directory should either be a part of the Kibana class in kibana.py if they are to be consumed in multiple modules or the file should be moved to the modules directory if it is something to be called directly in a playbook.  Ideally, we should only need to add files in module_utils when we need logic for interacting with a new API endpoint or service. Any logic we need to add for interacting with existing API endpoints should be in the existing module_utils or in a module.
As for elastic_endpoint_baseline.yml, if this is an example, we'll need to determine where we want to put this type of file since we haven't had an example in any of our collections so far. If this playbook was intended as an example of how to consume the new modules then we can add that to the DOCUMENTATION block at the beginning of the module and use collection_prep_add_docs -p . to create the RSTs, like this one.
We can discuss these changes during a meeting or over IM if that is preferred.


Use pyproject & friends

Issue

Use pyproject.toml to configure the entire project.
Use poetry to build the package and manage deps.
Use ruff for Python linting.
Update CI workflows accordingly.

How

Screenshots

References

Checklist

[ ] Tests added
[ ] Cleaned up commit history (rebase!)
[ ] Documentation added
[ ] Thought about variable and method names

Not really related to this PR, but just an FYI - for the building and shipping to PyPI - instead of manually running that GitHub action you can just use the GitHub Release flow as described here - https://github.com/NebraLtd/hm-pyhelper#releasing
Just using that and setting up the tags and using the auto generated release notes like shown here - https://github.com/NebraLtd/hm-pyhelper/releases/tag/v0.14.10
Then it auto triggers the build and lists all the wheel files and such in the releases.

also as per comment on hm-diag - wondering if we should add bandit (as we have it in hm-config but not in other repos it seems)


[OTHER] Add License

What would you like to share?
Hey @mkubdev, I think it's good to add a license for this repo.
Additional information
No response
Would you like to work on this issue?

[X] Yes, I want to work on this issue!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @vinzvinci, yes sure, we can use MIT :)


The Devil Is Fascilitating Opporrtunities to Remote Highjacjk my total consciousness.

#238 #30
Xhild Prone La-Dee-Da
**
TEST
**

#Sorry! We couldn't save your comment — your comment cannot be blank. Please try again.


Cannot find /app/app.html in build mode

Hello,
I'm trying to run my application based on your starter but I can't find out why the build is not working.
My architecture is like
src/
... app/
... ... app.js
... ... app.controller.js
... ... app.html
... index.html

Here is the paths object from gulpfile.babel.js
const paths = {
  js: './src/app/**/*!(.spec.js).js',
  styl: './src/app/**/*.scss', // stylesheets
  html: [
    './src/app/**/*.html',
    './src/index.html'
  ],
  entry: [
    'babel-polyfill',
    './src/app/app'
  ],
  output: './src',
  dest: './dist'
};

Everything's fine with the serve task.
When I build the project and try it with a http server, I have a 404 not found for that file :

"GET /app/app.html" Error (404): "Not found"

I don't have a lot of things in my dist folder, I don't know where are all my html templates.
Would someone have an idea about that issue ?
Here is my webpack.config.js by the way, maybe it can help.
module.exports = {
  devtool: 'sourcemap',
  entry: {},
  module: {
    preLoaders: [
      { test: /\.js$/, loader: 'eslint', exclude: /node_modules/ }
    ],
    loaders: [
      { test: /\.js$/, exclude: [/node_modules/], loader: 'ng-annotate!babel' },
      { test: /\.html$/, loader: 'raw' },
      { test: /\.(css|scss)(\?]?.*)?$/, loader: ExtractTextPlugin.extract('style', 'css?sourceMap!postcss!sass') },
      { test: /\.(woff|woff2|ttf|eot|jpe?g|png|gif|svg)(\?]?.*)?$/, loader: 'file?name=res/[hash].[ext]' }
    ]
  },
  plugins: [
    new webpack.ProvidePlugin({
      moment: 'moment'
    }),
    new ExtractTextPlugin('style-[hash].css', { allChunks: false }),
    new HtmlWebpackPlugin({
      template: 'src/index.html',
      inject: 'body',
      hash: true
    }),
    new webpack.optimize.CommonsChunkPlugin({
      name: 'vendor',
      minChunks: (module, count) => module.resource && module.resource.indexOf(path.resolve(__dirname, 'client')) === -1
    })
  ],
  postcss: () => [autoprefixer]
};

Bruno

Found it !
In my components, i was using templateUrl with like this
import './app.scss';
import controller from './app.controller';
// import './images/svg-defs.svg';

const AppComponent = {
  controller,
  templateUrl: 'app/app.html'
};
export default AppComponent;

It works better like this:
import './app.scss';
import template from './app.html';
import controller from './app.controller';
// import './images/svg-defs.svg';

const AppComponent = {
  controller,
  template
};
export default AppComponent;


Merge 3.1.x into 3.2.x

Merging changes in 3.1.x into 3.2.x branch, and make tests to pass in both jvm and js.

Hi @cheeseng,
I merged this to make progress, but we still get these three test failures under 3.2.x. These may be fixed by a later PR.
[info] Run completed in 1 minute, 24 seconds.
[info] Total number of tests run: 18534
[info] Suites: completed 690, aborted 0
[info] Tests: succeeded 18531, failed 3, canceled 11, ignored 50, pending 42
[info] *** 3 TESTS FAILED ***
[info] SpecSpec:
[info] when a test fails
[info] - should generate NotAllowedException wrapping a non-fatal RuntimeException is thrown inside scope *** FAILED *** (3 milliseconds)
[info]   "Spec[Spec].scala" did not equal "Spec[].scala" (SpecSpec.scala:2637)
[info] TestLocationFunctionPathFunSpec:
[info] A Spec
[info] - cancel !!! CANCELED !!! (0 milliseconds)
[info]   org.scalatest.exceptions.TestCanceledException was thrown. (LocationFunctionSuiteProp.scala:493)
[info] TestLocationFunctionPathFreeSpec:
[info] Test
[info] - should cancel !!! CANCELED !!! (0 milliseconds)
[info]   org.scalatest.exceptions.TestCanceledException was thrown. (LocationFunctionSuiteProp.scala:464)
[info] RefSpecSpec:
[info] when a test fails
[info] - should generate NotAllowedException wrapping a non-fatal RuntimeException is thrown inside scope *** FAILED *** (2 milliseconds)
[info]   "RefSpec[Spec].scala" did not equal "RefSpec[].scala" (RefSpecSpec.scala:2497)
[info] TestCanceledExceptionSpec:
[info] The TestCanceledException
[info] - should give the proper line on assertResult(1) { 2 } !!! CANCELED !!! (0 milliseconds)
[info]   assertResult(1) { 2 } didn't produce a TestCanceledException (TestCanceledExceptionSpec.scala:162)
[info] TestCanceledExceptionSpec:
[info] The TestCanceledException
[info] - should give the proper line on assertResult(1, "some message") { 2 } !!! CANCELED !!! (0 milliseconds)
[info]   assertResult(1, "some message") { 2 } didn't produce a TestCanceledException (TestCanceledExceptionSpec.scala:179)
[info] TestCanceledExceptionSpec:
[info] The TestCanceledException
[info] - should give the proper line on intercept[IllegalArgumentException] {} !!! CANCELED !!! (0 milliseconds)
[info]   intercept[IllegalArgumentException] {} didn't produce a TestCanceledException (TestCanceledExceptionSpec.scala:194)
[info] TestCanceledExceptionSpec:
[info] The TestCanceledException
[info] - should give the proper line on intercept[IllegalArgumentException] { throw new RuntimeException } !!! CANCELED !!! (1 millisecond)
[info]   intercept[IllegalArgumentException] { throw new RuntimeException } didn't produce a TestCanceledException (TestCanceledExceptionSpec.scala:211)
[info] TestCanceledExceptionSpec:
[info] The TestCanceledException
[info] - should give the proper line on 1 should === (2) !!! CANCELED !!! (0 milliseconds)
[info]   1 should === 2 didn't produce a TestCanceledException (TestCanceledExceptionSpec.scala:229)
[info] TestCanceledExceptionSpec:
[info] The TestCanceledException
[info] - should give the proper line on an [IllegalArgumentException] should be thrownBy {} !!! CANCELED !!! (0 milliseconds)
[info]   an [IllegalArgumentException] should be thrownBy {} didn't produce a TestCanceledException (TestCanceledExceptionSpec.scala:247)
[info] TestCanceledExceptionSpec:
[info] The TestCanceledException
[info] - should give the proper line on an [IllegalArgumentException] should be thrownBy { throw new RuntimeException } !!! CANCELED !!! (0 milliseconds)
[info]   an [IllegalArgumentException] should be thrownBy { throw new RuntimeException } didn't produce a TestCanceledException (TestCanceledExceptionSpec.scala:265)
[info] SpecSpec:
[info] when a test fails
[info] - should generate NotAllowedException wrapping a non-fatal RuntimeException is thrown inside scope *** FAILED *** (1 millisecond)
[info]   "[Spec]Spec.scala" did not equal "[Ref]Spec.scala" (SpecSpec.scala:2496)

Hi @cheeseng,
One other thing is that there was a compiler error under 2.10, so we need to come up with a build/test matrix script. I will make a ticket for that and submit the script I use, which needs enhancements.
Bill

Hi @bvenners,
I just checkout the current 3.2.x and ran:
sbt clean test
but I didn't see the failing tests that you saw, I suspect it is caused by unclean state again.

Ah, that could be, which means I need to improve the build script. I will submit that as a PR somewhere so you can try it out and see if you can improve it. Not only is it apparently broken, it is also incomplete! It doesn't yet compile the Java script tests.


Disable new arm-common-types-version rule ahead of rollout

This PR preemptively disables the new @azure-tools/typespec-azure-resource-manager/arm-common-types-version rule before it gets rolled out in an upcoming release of the @azure-tools/typespec-azure-resource-manager library.
This rule checks to see if the new @armCommonTypesVersion decorator (see https://github.com/Azure/typespec-azure/pull/3768/) has been applied to the service namespace or to one of the version enum values that represent service versions.  If not found, a warning is raised.
We're disabling this rule in advance for all existing ARM specs so that CI jobs are not disrupted until the specs have been updated to use the appropriate common-types version.
Please let me know if I should make an equivalent PR to any other repo that might be affected by this change!

Discussed RPSaaSMaster and RPSaaSDev with @mikeharder over Teams.  The current policy is that service teams must address any new linting rule errors to their specs in RPSaaSMaster when they create any new PRs.  Since RPSaaSMaster is a staging area for specs that are about to go into production, we will not automatically disable any new linting rules in that branch (or RPSaaSDev for that matter).
Similarly, when a spec moves into production, it will be PR'ed over to main and any new linting rule errors will need to be fixed at that time.  These rules encode approved design guidance for service teams, so they should be considered part of the final design review.
We will only automatically disable new linting rules in existing specs in the main branch if they will inevitably cause linting errors or breaking output changes to those specs.


secp256k1.TestSign sometimes fails

https://github.com/shogo82148/goat/actions/runs/3399012368/jobs/5652486640
 === RUN   TestSign
    secp256k1_test.go:60: verify failed
--- FAIL: TestSign (0.00s)

fixed by https://github.com/shogo82148/goat/pull/87/commits/394608efbd7f713a416af6c7e5d53daca8579a24


sing: more voice support

if using the SAM TTS, we could also have songs! having SAM in the first place already feels like a easter egg :)

https://www.youtube.com/watch?v=V8sN6qKLq_c  - Rammstein - Radio
https://www.youtube.com/watch?v=6oRUCkAbPn0  - System Of A Down - Chop Suey
(a bunch more rammstein covers in same channel)

Checking for SAM usage is easy:
    def _sounds_like_sam(self) -> bool:
        tts = self.config_core.get("tts", {})
        if "sam" in tts.get("module", "").lower():
            return True
        return False

But playing YouTube videos is more complex...we'd have to do a lot of validation to see if we can use it through OCP/media. I don't see contact information for @Noplan so no easy way to ask permission to get just the audio and provide an attribution link.
Thoughts on how to proceed, @JarbasAl ?

i'd just download the track and then bundle it with the skill, guess we should ask the author in this case, if it was a remote url we wouldnt need pemission but to bundle it we should reach out

I thought about that but there is no contact information for this channel that I could find...

I thought about that but there is no contact information for this channel that I could find...

if you cant ask for permission, ask for forgiveness :man_shrugging:  leave a note on the readme with clear attribution and an offer to take it down


Can I add a Google login or any other method to save favorites and track views in case the app is deleted?

Describe your suggested feature
Can I add a Google login or any other method to save favorites and track views in case the app is deleted?
Other details
No response

Data export option is in the roadmap.

"Could you please share the roadmap?"

It is on discord.
Roadmap (in order)

PROVIDERS SYSTEM

Launch the initial phase
Create the SDK docs (we're here!)

DATA IMPORT/EXPORT OPTION
CASTING OPTION
RE-ENHANCE UI
TRANSLATIONS, MAYBE?


Adding reference to Hardware Requirement

DTR has higher requirement than regular worker nodes, and it is somehow listed on UCP section instead of here.  Adding reference.

Proposed changes

Unreleased project version (optional)

Related issues (optional)

I've updated your suggestion to not include the https:// component of the link. Thanks for the contribution 🙌


Migrate content from CEIC

I actually tracking this content task in Trello so this issue is closeable https://trello.com/c/khoW8PJP/142-tree-rat-cms


Addon "background services" affects other addons

Netflix add-on version
1.19.1
Operative systems used
LibreELEC
Kodi version used
Kodi 19 (Matrix)
Description of the bug
Hello, I have a problem that is not the fault of this addon, but caused by it.
I need to figure out, what "netflix background services" are started when kodi starts, because it's probably affecting another addon.
I have addon "streamcinema" installed, and another addon "titulky.com". But this "titulky.com" addon stop working, when Netflix addon is installed and enabled. I think "background services" somehow affects these "titulky.com" addon, by I really don't how.
Is there any possibility to help me? I can give logs and cooperation as needed.
Thank you for your helpfulness.
Steps to reproduce the behavior
No response
Debug log - mandatory
https://paste.kodi.tv/rofuqokugi.kodi
Possible fix
No response
Additional context
No response
Screenshots
No response

i need to see the titulky addon sources to understand something where a link?
with netflix addon we dont use fixed ports on service so we pick the first free port,
if not already done titulky service should implement something similar
but the error

RecursionError: maximum recursion depth exceeded while calling a Python object

means that your titulky service code or "plugin.video.stream-cinema-2-release" code is running an endless operation,
then since never end the client raise "socket.timeout: The read operation timed out"
the first to check is why your service have an infinite looped operation?

So there is problem with titulky addon? For example because trying to use fixed port, which is not free?
Addon source is https://github.com/beam-kodi-addons-repository/service.subtitles.titulky.com

i dont know titulky or plugin.video.stream-cinema-2-release addons,
my are only suggestions by looking your log
service.subtitles.titulky.com and plugin.video.stream-cinema-2-release addons are in some way linked? i have no idea
but as i said before the error RecursionError from plugin.video.stream-cinema-2-release addon suggest a code loop with no exit
need to be investigated on plugin.video.stream-cinema-2-release addon
if you are not the developer of these addons you have to report this to relative maintainers

Thank you for your help, I'm just a user who has a problem with addons, and I can't find a solution. I tried to contact developer of service.subtitles.titulky.com, but he keeps telling me "its addon only establishes a connection to the server, but it failed" and he doesn't want to deal with it anymore. So I write to you, trying to move on...
So I don't know what to do :-)
Plugin.video.stream-cinema-2-release is addon, which playing videos (movies) from servers, it works standalone, and work well with plugin.video.netflix.
Service.subtitle.titulky.com is addon used with stream-cinema-2 and it's only automaticly (or manually) download subtitles to movies which are currently being played. This work fine, but when there is plugin.video.netflix installed (and enabled) it stop working. Service.subtitle.titulky.com cannot "establishes connection" and doesn't download any subtitles. When plugin.video.netflix addon is disabled (or enabled but logged off) it works fine. But it wasn't always like that, couple of months ago (not sure when) all three addons working together fine.
I just want things to work, but I don't know who to turn to.

since the main error come from plugin.video.stream-cinema-2-release addon
you should report to plugin.video.stream-cinema-2-release addon maintainer
i mentioned TCP port problem because is the only thing that could affect another TCP server
so two services with same port == conflicts
there is nothing else
just for info, the TCPServer used on this addon it is a standard python TCPServer bound to a free port found at service starts
https://github.com/CastagnaIT/plugin.video.netflix/blob/master/resources/lib/run_service.py#L31
this has never changed for years

i close issue since i have no idea what i could do
if plugin.video.stream-cinema-2-release dev have some type of suggestions
you can however let me know here

Thanks for now, I'll let you know if I have more information


Remove backend cache from QiskitRuntimeService

Summary
In #1715 we implemented new pattern to retrieve backend from the service, namely, opt-int. With opt-in, the same backend can have different Target, and thus they must be different Python objects.
This PR removes backend instance cache mechanism from QiskitRuntimeService. We usually reuse the retrieved backend instance, instead of getting the backend object from the service multiple times. Only exception is QiskitRuntimeService.run with a backend name string, in which the backend is retrieved internally, but this pattern is already deprecated and I don't think this change will harm the most of end user experiences.
Details and comments
Once after we announce the fractional gate opt-in (not limited to fractional, some future opt-ins), user may try to compare the performance of our system with and without opt-in. One may write
backend_with_fg = service.backend("my_backend", use_fractional_gates=True)
backend_without_fg = service.backend("my_backend", use_fractional_gates=False)

# Run our system with fractional gates
sampler = SamplerV2(backend_with_fg)
isa_circ = transpile(qc, backend_with_fg)
result_with_fg = sampler.run(isa_circ).result()

# Run our system without fractional gates
sampler = SamplerV2(backend_without_fg)
isa_circ = transpile(qc, backend_without_fg)
result_without_fg = sampler.run(isa_circ).result()

Note that this does NOT work as expected BUT doesn't raise any warning or error. Note that backend_with_fg is silently mutated at the second retrieval via the cache, and both sampler jobs run without fractional gates. One may wonder why two results look comparable.
This PR will remove this complication and make internal code much simpler.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

40 of 47 (85.11%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 82.289%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
40
47
85.11%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
2
77.07%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
6017

Relevant Lines:
7312

💛  - Coveralls

I think there is value in caching the backend data, so we don't need to call the server again to retrieve the same exactly data just to build a target that with different basis gates.
Having said that, the existing caching code is just too complicated, and this PR is a good opportunity to clean it up. A follow up improvement can be a "backend cache" that only stores server data (backend config etc), andBackend objects are dynamically created with different target, instance, etc but use the same server data.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

41 of 42 (97.62%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.002%) to 82.352%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
41
42
97.62%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
2
78.2%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
6015

Relevant Lines:
7304

💛  - Coveralls

follow up improvement can be a "backend cache" that only stores server data (backend config etc), andBackend objects are dynamically created with different target, instance, etc but use the same server data.

Done in deb6cb6. Cache code is now very simple.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

42 of 46 (91.3%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 82.332%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
41
42
97.62%

qiskit_ibm_runtime/api/clients/runtime.py
1
4
25.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
2
78.2%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
6016

Relevant Lines:
7307

💛  - Coveralls

Actually, one more comment - Both backend.properties() and backend.defaults() currently have a refresh keyword that says "If True, re-query the server for the backend properties/defaults. Otherwise, return a cached version.". Since we only cache configuration now, those need to be deprecated and later removed, unless you want to also add caching for properties/defaults.

Since we only cache configuration now, those need to be deprecated and later removed, unless you want to also add caching for properties/defaults.

I think this behavior should be okey because instance-wise these information are cached. Maybe I should write upgrade note.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

42 of 46 (91.3%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 82.387%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
41
42
97.62%

qiskit_ibm_runtime/api/clients/runtime.py
1
4
25.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
2
78.2%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
6048

Relevant Lines:
7341

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

42 of 46 (91.3%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 82.387%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
41
42
97.62%

qiskit_ibm_runtime/api/clients/runtime.py
1
4
25.0%

Files with Coverage Reduction
New Missed Lines
%

qiskit_ibm_runtime/qiskit_runtime_service.py
2
78.2%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
6048

Relevant Lines:
7341

💛  - Coveralls


[#7,#8,irods/irods#4650] Get project building again (4-2-stable)

Cherry-pick of #9 into stable.
Addresses #7 and irods/irods#4650.
Addresses #8 with the quick-n-dirty solution. Proper fix for 8 will come with irods/irods#5250.
I've also bumped the required iRODS version number to 4.2.9.
I haven't been able to fully test building this branch of this repo yet, so further changes might be needed to reach the stated goal. That said, I am reasonably certain that these changes are necessary to get this branch building.

good enough - yes, # and we'll get closer.

#'d


Pickling fails with an Automaton(STORE_INTS)

Using the latest code with Python 2.7 on 64 bits Linux and an Automaton(STORE_INTS) pickling fails.
>>> import ahocorasick as aho
>>> a=aho.Automaton(aho.STORE_INTS)
>>> a.add_word('abc', 12)
True
>>> a.make_automaton()
>>> import cPickle
>>> p=cPickle.dumps(a)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
SystemError: NULL object passed to Py_BuildValue

Note that I worked around this by still storing ints but as objects as you can see here: https://github.com/nexB/scancode-toolkit/blob/d99f2781d19a70de57a6392b61e2253f3ec599f5/src/licensedcode/index.py#L226
So this is not a big problem


Don't encode slashes in examples

Sometimes we need to be able to pass a parm to a service which is a path with some arbitrary number of slashes in it.
When we set an example for this, it will render like foo%2Fbar%2Fbaz when we show the example in the front end, but it would be nicer if it rendered as foo/bar/baz
refs https://github.com/badges/shields/pull/2639#discussion_r245489576

Related: pillarjs/path-to-regexp#177

I think now that we display these as param+ or param* on the index (rather than populating with an example), this is now irrelevant. The examples look correct in the modals


Running a different piece of jQuery on page load (help wanted)

I've seen this issue mentioned a few times, but I can't seem to make any of the solutions work. In particular, I have an object, #portfolio, on one page that uses Masonry.js for a nice grid layout. It works on page refresh but not when the page is navigated to properly. If I play around with the code, I can get just Masonry working or just smoothState working, but not both.
Post-butchering, here's my code. In this state, Masonry works but smoothState doesn't. Removing the entire onEnd: {} block reverses the situation.
$(function (){
  'use strict';
  var $page = $('#main'),
      options = {
        debug: true,
        prefetch: true,
        cacheLength: 2,
        onStart: {
          duration: 250, // Duration of our animation
          render: function ($container) {
            // Add your CSS animation reversing class
            $container.addClass('is-exiting');
            // Restart your animation
            smoothState.restartCSSAnimations();
          }
        },
        onReady: {
          duration: 0,
          render: function ($container, $newContent) {
            // Remove your CSS animation reversing class
            $container.removeClass('is-exiting');
            // Inject the new content
            $container.html($newContent);
          }
        }
      },
      onEnd: {
        $('#portfolio').masonry();
      }
      smoothState = $page.smoothState(options).data('smoothState');
});

Again, I'm aware this has been asked before, but the other solutions seem to have not worked for some reason.

@dendodge So just to confirm, you've tried running your masonry reinitialization in onAfter as well? My understanding is that the onAfter hook is the officially supported hook for restarting jQuery plugins, not onEnd. I'm not 100% clear on the real-world differences between using onAfter and using onEnd yet, though.
I have little experience with using smoothState in combination with other jQuery plugins that need this kind of attention but I'd like to learn my way around doing this.

@DigitalMachinist I've tried that, and I've tried normal callbacks to call Masonry after smoothState, but it still breaks the animations.

@dendodge this is a better conversation for StackOverflow. We need to keep the issue tracker clear for bugs.


Ignite Rework

This is based off of: https://github.com/micronaut-projects/micronaut-ignite/pull/29 I'm going to split out all the data stuff for later since that will take more work.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: pollend:x: dmagdaYou have signed the CLA already but the status is still pending? Let us recheck it.

Here is the relevant discussion for some context: http://apache-ignite-developers.2346864.n4.nabble.com/Re-Micronaut-Ignite-td48810.html

@dmagda  asked me about the version of ignite that would be packaged with the release.
Michael,

The docs look good enough for the release. Once the integration is
released, I'll update and extend my Micronaut tutorial elucidating on how
to use the integration in practice for various use cases.

Btw, are the ignite-core and ignite-cache Maven artifacts will be released
for a specific version of Ignite? Ideally, the version of Ignite needs to
be set in a pom.xml and the Micronaut integration should pick them up.

Also, what are we doing next - reviewing APIs of the ignite-cache module or
going through a review process of the ignite-core and merging it first to
the Micronaut repository?

Could the CI be setup to accommodate this?

Here is a sample application I put together to show the configuration: https://github.com/pollend/micronaut-ignite-sample

so @pollend is this ready for you review from your perspective?

I went through and excluded items from the configuration that can't be set. This should clean up the documentation. Something I noticed that looks like a bug is ignite.include-event-types.

it should be int[]

@dmagda are you good with this PR?

I went through and excluded items from the configuration that can't be set. This should clean up the documentation.

Btw, why the description column of the configuration items is empty on the documentation end? Is there any way to get it populated? Otherwise, application developers have no idea what a configuration option does.

I was looking at micronaut-gcp and it looks like the descriptions are filled out manually. not really sure how the descriptions would be filled out.
https://github.com/micronaut-projects/micronaut-gcp/blob/master/src/main/docs/guide/pubsub/producer/publisherProperties.adoc

I'm looking at this and that's quite a lot of descriptions to copy in manually. my worry is having the configuration go out of sync with the documentation.

Regarding the descriptions of the documentation not being populated, this is how it works:

We have these dependency https://github.com/micronaut-projects/micronaut-ignite/blob/bce805493c1abee4be2e2c89749d1f22189b5a23/ignite-core/build.gradle#L4 that extracts the configuration properties and generated the appropriate .adoc files
The you can include that file in the documentation: https://raw.githubusercontent.com/micronaut-projects/micronaut-ignite/bce805493c1abee4be2e2c89749d1f22189b5a23/src/main/docs/guide/igniteCache/igniteClientCache.adoc

Our micronaut-docs-asciidoc-config-props library extracts the descriptions from the source code during the compilation of the module itself but it doesn't work for other external dependencies (because the code has already been compiled). In all the documentation for the rest of Micronaut modules the behavior is the same.
Regarding the GCP documentation, those tables with the descriptions were moved to source control and updated manually. Although I see the point of having the description there I'm not completely sure it's the best approach because those tables need to be maintained manually.
Finally, regarding the PR itself, is it ready for merge now?

@pollend, the changes look good to me now. Please go ahead and merge unless the Micronaut community suggests any other improvements. All is neat and clean from the Ignite standpoint.

Thanks for the contribution!

@pollend @dmagda would you like to do a 1.0.0.RC1 release with the current state of the code?

Hold on, I'm going to fix the 2 open issues about the docs, cleanup a few project-config things and them we can release RC1

We're ready for the release whenever you want.
I've also submitted this draft PR to core to add this to the bom: https://github.com/micronaut-projects/micronaut-core/pull/4070


added Python 3.6 to CI per issue #1331

resolves #1331

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1340   +/-   ##
=======================================
  Coverage   98.19%   98.19%
=======================================
  Files          54       54
  Lines        2433     2433
=======================================
  Hits         2389     2389
  Misses         44       44

Continue to review full report at Codecov.

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

Hi @morrme
Before we can merge this pull request, we need you or your organization to agree to one of our contributor agreements. One of the big concerns for people using and developing open source software is that someone who contributed to the code might claim the code infringes on their copyright or patent. To guard against this, we ask all our contributors to sign a Contributor License Agreement. This gives us the right to use the code contributed and any patents the contribution relies on. It also gives us and our users comfort that they won't be sued for using open source software. We know it's a hassle, but it makes the project more reliable in the long run. Thank you for your understanding and your contribution!
If you are contributing on behalf of yourself (and not on behalf of your employer or another organization you are part of) then you should:

Go to: https://www.bigchaindb.com/cla/
Read the Individual Contributor Agreement
Fill in the form "For Individuals"
Check the box to agree
Click the SEND button

If you're contributing as an employee, and/or you want all employees of your employing organization to be covered by our contributor agreement, then someone in your organization with the authority to enter agreements on behalf of all employees must do the following:

Go to: https://www.bigchaindb.com/cla/
Read the Entity Contributor Agreement
Fill in the form "For Organizations”
Check the box to agree
Click the SEND button

We will email you (or your employer) with further instructions.

@ttmc i actually completed the form right after my PR per the contributing instructions, but have not received an email.

@morrme I saw this pull request before I saw that email. Sorry! I'll reply to your email ASAP.

Update: @morrme agreed to the CLA, but didn't provide a real name, so there has been some email back and forth about that.

@sbellem @gmcmullen @ttmc  Can this be updated please , per email?

This is to confirm that I agreed to and accepted the BigchainDB Individual Contributor Agreement at https://www.bigchaindb.com/cla/ and to represent and warrant that I have authority to do so.
609fbxGGW7khxOkSOkNtanZBRd87smxzD4kRcn9lhTJgcI4ecNy3d37isjRKC


setConnectTimeout, setReadTimeout, setDebugging  for OAUTH interaction

Setters in ApiClient for setConnectTimeout, setReadTimeout (see PR #62), and setDebugging are very helpful for debugging and stability.
However these settings get only active after authentication, but not for OAUTH interaction.
I.e., if the system connects to the oAuthBasePath during authentication with authName "docusignAccessCode", these settings are not active yet in the httpClient, which is created in the default constructor (see ApiClient.java:81).
It would be great, if we could have an extension of the following constructor into a new one:
ApiClient.java:134: --->
public ApiClient(String oAuthBasePath, String authName, String clientId, String secret)
New constructor:
public ApiClient(String oAuthBasePath, String authName, String clientId, String secret, int connectTimeout, int readTimeout, boolean isDebugging)
The new constructor can apply the setters in this sequence: setDebugging (first), setConnectTimeout, setReadTimeout.
setDebugging will create the new httpClient before the timeouts will be set on this new httpClient.
Afterwards, the following lines will use the client with the new setting:
ApiClient.java:109 --->   auth = new OAuth(httpClient, OAuthFlow.accessCode, oAuthBasePath + "/oauth/auth", oAuthBasePath + "/oauth/token", "all");

ping reminder @mmallis87
Would be nice if you could add this change or if you need a PR for that.


Gracefully handle corrupted journal log entries

This is a back port of https://github.com/reevoo/fluent-plugin-systemd/pull/41 to the 0.0.x branch. If you don't want to accept this; I understand, but I saw some others who requested this so I figured I would open a PR.

Er, hello?
This Pull Request is too damn old! Merge or close this, sucka.

Sorry, hadn't had a chance to look into the issue too much; I hadn't seen it before. I believe it's a ruby version thing and travis is using the latest version of ruby at least on the 0.8 based branches.

Tests pass; rubocop fails but its for unrelated things since nothing has changed relating to it's complaints. I'm guessing it's a newer rubocop or something that fails with the older 0.8 branch. I don't really know much about rubocop so I'm not 100% sure what the solution to that is besides just fixing these things?

Looks good to me... thanks for the PR

i got everything to pass :)


Fix url query string mistakes ?&

No more ?& in urls.
Now stamp precedes other params for programming convenience, but it doesn't make urls invalid.
map, join? ugh.. wouldn't it be easier to read for-loop?
hasOwnProperty added because jshint doesn't detect iteration through Object.keys and complains about it. You could fix .jshint as well, add var param in Object.keys(data) and get rid of hasOwnProperty.

Merged in 61f526ff8c6124fae8635fd9b9b3e01810e2a7c8. Quite agreed that a map + join is suboptimal for simply building a querystring. Thanks, @ilyaigpetrov!


Sub-resource integrity checks for all the fonts

Is your feature request related to a problem? Please describe.
The software supply chain have popped up more and more.
When loading code from source you don't control directly, it is good practice to check that the code is what you expect.
Describe the solution you'd like
Could you add sub resource integrity into the links in the documentation of how to use the fonts, and inside https://cdn.eds.equinor.com/font/equinor-font.css for all the fonts?
Describe alternatives you've considered
I've computed the checksums manually, which works well, but I'd also like the "sub"-subresources to be checked as well (i.e. the fonts).
I've considered serving the fonts from our site, and adding the integrity checks manually, but that defeats some of the purposes of using a CDN.
Additional context
Same question on Slack

Investigate potential drawbacks to the proposed solution


EllaMartinez1981

icla.1-1 (1).PDF
https://metastatus.com/outage-events-feed-ads-transparency.rss

Hi @EllaMartinez1981 - This PR can't be merged as-is. Can you revert the changes to the author metadata? Or open a new PR without those changes?
We'll close this PR for now.
Thanks for contributing!


chore: add github code ql action

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Pages] Add empty divs so GTM can inject surveys

Add empty DIV elements with identifiers to certain pages in certain locations so that GTM is able to inject surveys.
Ex.

This will be similar to what we did at Pall.

TBD: Waiting for list of pages and locations from Thomas Hill. I will attach the list to this issue once I receive it.

Based on our call, we will look through the APi (Medallia I think) so that we can build our own survey and not have GTM inject one.
Anthony will minalize the design. We can then implement on specific sections in the template either through the code or injecting via Target.


Spark 2.0 support?

Just wanted to check if there are any plans on supporting more recent versions of spark.
Thanks for your help, very promising project!

When till we can get an official release for Spark 2.0?


Registration process cannot be completed

After the installation in ioBroker I cannot finish the registration process because the buttons to request the code do not react. According to the web console, the following script cannot be loaded either:
/adapter/dysonairpurifier/dyson-utils.js
Version:   0.9.1

That´s really strange.
I even had the version from Githab installed... but it still grumbles around the file would not be there.
In the file system under /opt/iobroker/node_modules/iobroker.dysonairpurifier is the file... and had it also once copied to admin/ but he also does not find.
But also strange: if i delete the admin/dyson_logo.svg i can still retrieve it?! locate finds the file but only at that location.
Now I have really uninstalled the adapter again properly via the UI and reinstalled. and there you can also see that the file is not uploaded with. but in the file system it is there. (Use redis as backend - therefore perhaps?)

Now I really uninstalled the adapter again properly via the UI and reinstalled. and there you can also see that the file is not uploaded with. but in the file system it is there. (Use redis as backend - therefore maybe?).
Have the dyson-util.js now times copied to admin/ and then called "iobroker upload dysonairpurifier" and there it uploads it and then the link also works.
But still nothing happens when I press one of the two buttons :-(

Hmm. Have you read the readme.md@github esp. the part regarding prerequisites, installing and 2fa process?
It starts to sound to me like you are trying to install the adapter without having an active dyson-account.

Okay. Seem to be solved. Closing therefor. Reopen if needed.


Question: Supported Apache Guacd version

Whats the supported guacd version? (latest is 1.4.0)

I want to know. too

Seems to work fine on 1.4.0 for me


Allow GCL sink to authenticate with private key on AWS

Inspired by: a831b80cc381d08afc47dbffc04880e725ac40db
This allows us to capture Kubernetes events in Google/Stackdriver Logging with a Kubernetes cluster running in AWS. The env var GOOGLE_APPLICATION_CREDENTIALS refers to the service account private key mounted in the pod as per the following example:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: heapster
  namespace: XXXXXXXXXXXXXXXXX
spec:
  replicas: 1
  template:
    metadata:
      labels:
        task: monitoring
        k8s-app: heapster
    spec:
      imagePullSecrets:
        - name: XXXXXXXXXXXXXXXXX
      containers:
      - name: heapster
        image: XXXXXXXXXXXXXXXXX
        imagePullPolicy: Always
        env:
        - name: GOOGLE_APPLICATION_CREDENTIALS
          value: /etc/google/auth/application_default_credentials.json
        command:
        - /heapster
        - --source=kubernetes
        - --sink=gcm
        - --metric_resolution=60s
        volumeMounts:
        - name: credentials
          readOnly: true
          mountPath: /etc/google/auth
      - name: eventer
        image: XXXXXXXXXXXXXXXXX
        imagePullPolicy: Always
        env:
        - name: GOOGLE_APPLICATION_CREDENTIALS
          value: /etc/google/auth/application_default_credentials.json
        command:
        - /eventer
        - --source=kubernetes
        - --sink=gcl
        volumeMounts:
        - name: credentials
          readOnly: true
          mountPath: /etc/google/auth
      volumes:
      - name: credentials
        secret:
          secretName: XXXXXXXXXXXXXXXXX

Tested on Kubernetes 1.7.2 in AWS.

This change is

I signed it!

@piosz @kawych et all can you take a look at this

/ok-to-test

Hi @kawych , thank you for your time! I will fix the GCL typo. I have only tested this change in AWS. Because the change is derived from https://github.com/kubernetes/heapster/commit/a831b80cc381d08afc47dbffc04880e725ac40db I presumed this would be enough. Could you please help me decipher the e2e test failure? I am not familiar with your framework. Thank you!

/lgtm


added usage to README.md

Hi, I love the game.
However, when I played for the first time, I had a hard time finding game rules and operating methods.
So I think that it is better for the operation method to be listed in the README.
This PR is an example, but what about putting Usage in this way?

Hi, Ryo glad you love the game.
Thanks for pointing out the issue. I have put github.io link in header, but it might be better to add it in README.md too. How about adding it as a sheild??
Something like:
This makes it more visible.

I agree with you.
Thank you for improving!


Fixes Narrator not reading Name of tab when drag is starting

Before:

After:

*notice the "grabbed " being read by narrator
Fixes internal issue 36431796

/azp run


Configure Couchbase panel within the Quarkus Dev UI

The Quarkus Dev UI is a developer web UI to quickly visualize extensions and interact with their non-code features. Adding useful links within the Couchbase panel would really help reduce the Couchbase learning curve for Quarkus developers.
Off the top of my head, I can think of two very useful panel features that are low-overhead to create:

A link to the Couchbase Java SDK docs
A link to the Couchbase Web UI (maybe dashboard specifically, but I'll leave that up to you)
A link to the to-be-created (hopefully) Quarkus guide specifically.

Here is the default UI for the example app in the README, and the auto-generated Couchbase panel is empty with a missing link to the Couchbase Quarkus guide:

That's a great idea, thanks for the feedback @jclingan .


[ReadMe] Adds community extensions list

First stab at adding a community extension list to the readme.

Looks fab :tada:


numpy.ndarray' object has no attribute 'is_joint'

I am getting an error 'numpy.ndarray' object has no attribute 'is_joint'' while applying Renyi entropy. May b i am unable to get what is 'dist'. Can you explain it

can you provide an example of the code that is failing?
a dist is, general, an instance of dit.Distribution. For example:
import dit
from dit.other import renyi_entropy

dist = dit.Distribution(['0', '1', '2'], [1/3, 1/6, 1/2])

renyi_entropy(dist, order=3)

1.292481250360578


Compile to commonjs to Support SSR via node...

see issues mentioned in https://github.com/nuxt-community/typescript-template/issues/44

further only seems to work with version 1.1.3 so sth else may need to be rolled back?

Just stumbled upon this issue too.
Tried every version from the npm, only 1.1.3 and 1.0.0 compile on my machine.
Version 1.0.0 didn't give any runtime errors for me (at least I'm able to see the starter page) - so I guess it's worth to try if it works for you :wink:


[Bug]: Alter stage for directory = "ENABLE = true"

Terraform CLI Version
1.0.0
Terraform Provider Version
0.55.1
Terraform Configuration
-

Category
category:resource
Object type(s)
resource:stage
Expected Behavior
When attempting to create or modify a stage in Snowflake using Terraform, the property directory = "ENABLE = true" should enable the use of a directory table without errors. Specifically, adding directory = "ENABLE = true" in the Terraform configuration should allow the creation or modification of the stage to proceed successfully.
Actual Behavior
When trying to create or modify a stage with directory = "ENABLE = true", the following error occurs:
Error: error creating stage <stage_name>
However, after manually enabling the directory option in the Snowflake console and running terraform plan, it’s confirmed that the correct parameter is indeed directory = "ENABLE = true". If the stage already has the directory option enabled in Snowflake, Terraform processes the parameter without issues, as it does not attempt to modify or recreate the stage. This suggests the issue occurs only during the initial creation or modification of the directory property.
Steps to Reproduce
1.- Configure a new stage in Terraform with the property directory = "ENABLE = true".
2.- Run terraform apply to create the stage or modify an existing one (supossed it already had not the directory enabled).
3.- Note the error occurring during the stage creation/modification. Note that the error occurs while applying the plan, i.e. after approving the plan.
4.- Create the stage without the directory property.
5.- Manually enable the directory option in the Snowflake console.
6.- Run terraform apply again and observe that directory = "ENABLE = true" is now recognized without issues, and no attempt is made to recreate the stage.
How much impact is this issue causing?
Low
Logs
No response
Additional Information
No response
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hey @Serhuela. Thanks for reaching out to us.
You are using an ancient version of the provider. The logic of the provider has changed multiple times since that version (we are at v0.97.1 now). Please check if this problem is still present in v0.97.

Hey @Serhuela. Did you have a chance to verify the newer versions?


Unable to prevent data-only Background notifications from displaying a notification (Android)

Using v.3.2.2 of RNN, and react-native 0.63. Didn't want to upgrade as right at the end of a build and don't want to switch version unnecessarily just yet.
We send pushes to display notifications for users, but also data-only notifications to trigger updates behind the scenes. I don't seem to be able to stop blank notifications from showing up in the notification tray. By which I mean the icon and app name, with no content (because there isn't any content to display).

I'm a little confused by a combination of the docs not mentioning much about the completion callback, and another Github issue that suggested the completion function is not correctly documented, so I might be missing something, or misusing that somehow...
The notification payload arriving via FCM looks like this:
{
  from: "XXXXXXXXXXXXX",
  google.c.sender.id: "XXXXXXXXXXXXX",
  google.delivered_priority: "normal",
  google.message_id: "XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
  google.original_priority: "normal",
  google.sent_time:<PHONE_NUMBER>000,
  google.ttl: 1234567,
  show: "false", // A setting which is used in the foreground, should be ignored in the background
  type: "update", // For filtering the type in the listener above
  action: "SOME_ACTION", // A sub-type, an action to trigger when in the foreground
}

And my listener looks something like this:
Notifications.events().registerNotificationReceivedBackground((
  notification: Notification,
  completion: (response: NotificationCompletion) => void
) => {
  switch (notification.payload.type) {
    // Some alert type to display in background
    case 'SOME_ALERT':
      completion({ alert: true, sound: true, badge: true });
      break;

    // Another type of alert to display
    case 'ANOTHER_ALERT_TYPE':
      // Maybe do something else here and then...
      completion({ alert: true, sound: true, badge: true });
      break;

    // THE PROBLEM - I have tried filtering this type, and different
    // formats of completion function, and also just leaving this
    // entire case out entirely, but the notification is always shown.
    case 'update':
      completion({ alert: false, sound: false, badge: false });
      // OR... completion(); - or no args, unclear what this does
      // OR... Something else to stop notifications from appearing??
      break;
  }
});

This is all going fine in the foreground, and the background notifications I do want to display are ok too. I'm just probably missing something obvious about how to handle (in my case ignore) these "hidden" pushes in the background. If anyone can point to my mistake or misunderstanding I'd be very grateful.

I couldn't find clarity on this myself, and since I built our backend push sending service a couple of years ago I wondered if it was sending stuff in a way that was triggering these incorrectly. However I've since switched over to react-native-firebase which is handling them as expected (ie. not displaying anything) so I'm happy there's no issue there. So while this is not closed as far as understanding why it was happening, it's closed for me as I'm not longer using this module. I'll just leave this in case anyone else has the problem and someone else can clarify for them.

not stale, Having the same issue

I have the same issue any update about this ??

Also the same issue, this is very problematic when using data-only notifications.
To add to this issue, if you specify the usual notification payload keys (title, body, ...) INSIDE the data payload, they will be picked up by the 'blank' notification as if it were a part of the notification payload.
e.g. setting the title to 'Hello' and the body to 'World' in the data payload will present a notification that says
Hello
World

Also the same issue, this is very problematic when using data-only notifications.
To add to this issue, if you specify the usual notification payload keys (title, body, ...) INSIDE the data payload, they will be picked up by the 'blank' notification as if it were a part of the notification payload.
e.g. setting the title to 'Hello' and the body to 'World' in the data payload will present a notification that says
Hello
World

go away stolebot

Somebody found a solution for this?

I'm guessing it should be a pretty easy fix to just add an if-statement on this line, to not present empty notifications?
https://github.com/wix/react-native-notifications/blob/master/lib/android/app/src/main/java/com/wix/reactnativenotifications/core/notification/PushNotification.java#L151

stalebot unstale!

This lib expects the title and body keys of your notification to be in one of two places. It's either included in the firebase notification object. Which will look like gcm.notification.title in the payload or at the top next to your action. These are the only two scenarios where react-native-notifications will automatically extract the title and body for you. Otherwise, you'll have to @Override the getTitle and getBody methods in a custom MyPushNotification.java class.


S2 [Bug] Sanic dependency breaking rbac-server

rbac-server_1 | "Cannot subscript an existing Union. Use Union[u, t] instead.")
rbac-server_1 | TypeError: Cannot subscript an existing Union. Use Union[u, t] instead.
https://github.com/huge-success/sanic/issues/1321
Locking sanic to version 0.7.0 (instead of using 0.8.x) appears to fix the problem.

Bug
[Bug] Sanic dependency breaking rbac-server

T-shirt size: S(mall), M(edium), L(arge)
S

Story Points - Fibonacci (1,2,3,5,8,13)
1

Sanic 0.8 is presently incompatible with Python 3.5, but compatible with Python 3.6.
Sawtooth may be compatible with Python 3.5, but incompatible with Python 3.6.
There may be a reason to move to Flask or something with greater support and away from Sanic given how they handled releasing a breaking change without regards to testing it on various versions.


When you start having wars, your cities will disappear.

When they come back, they don't have names.

Possible related to the fact that war can be triggered faster than the update tick runs, causing the real population and the world population to desynchronize.

Caused by pop auto-balancing, not really a bug.

Fixed by adding cooldown to pop auto-balance after a war


"Leere Karte" Map type in Map feature not working

When solving a task which has Map type to be "Leere Karte", it's not showing anything besides white screen with flags on it.
Screenshots:

Smartphone :
Device: [e.g. IPADC]
OS: [e.g. iOS15.0.2]
Version [e.g. 22]


Release Notes Versioning

The release notes page is great, but a couple of suggestions:

We do not install the full az. module, just the component parts we need ie. az.accounts . It would be really useful to list the version of those parts in the release notes, rather than just the version number of the overarching az. module.
For example, currently we are updating az.Accounts from 2.7.4 to 2.10.4. I'm having to use dates to work out the changes that have been made and tally that up with the az. module version numbers

Breaking changes are great to list, but some kind of filter that would highlight those would be really useful

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

ID: bdde8a1b-913d-6123-7cb6-4a7cbac558ad
Version Independent ID: 13ab3c45-9746-6b1c-66af-bbb1f8f5de60
Content: Azure PowerShell release notes
Content Source: docs-conceptual/azps-9.2.0/release-notes-azureps.md
Service: azure-powershell
GitHub Login: @mikefrobbins
Microsoft Alias: mirobb

Thanks for bringing this to our attention. We will investigate it further and update you shortly.

@Mattheww65 I've reached out to the team who generates the release notes. I'll follow-up once I hear back from them.

On the topic of breaking changes please include a note with the planned date for when a breaking change is scheduled to happen.

@vandre it is difficult for us to announce a release date for breaking changes much ahead of time. For example, major releases usually align with events like Microsoft //Build and we are not allowed to communicate about those dates before it is publicly announced.
However, we will take your suggestion in consideration and will announce the version of Az the breaking change aligns with.

Hi @Mattheww65 , we have updated our ChangeLog generation tool. It will carry the module version in the next release.


Could the sense be a series rather than a scalar string when adding constraints?

Was there any chance it might be possible for the sense parameter in the add_constrs method have the option of being a series/column instead of a single string?
In energy market modelling we often have 'generic constraint' sets governing power station behaviour where there can be a mix of sense values used. It would be really helpful to be able to just have the add_constrs method pick up the sense directly from a dataframe column.
By the way, really loving using gurobipy-pandas at the moment!

@jacksimpsoncartesian thanks for raising this, glad to hear you're finding the library useful :-)
Sure, this seems like a reasonable addition. We initially didn't set up vectorization for categorical arguments (i.e. sense & vtype) since it seemed like these should not vary in most user-code for a set of constraints of the same type. For example, in a set of capacity constraints the sense is always <= and only the numerical data in the constraint varies for each capacitated item.
Could you expand a bit on the use case here or give a brief example? One thing I am wondering is whether a mix of constraint senses really belong in the same API call, or whether the data should somehow be grouped differently to avoid doing this.

Minor side note: for the DataFrame.gppd.add_constrs accessor, one concern is that it is hard to unambiguously interpret an argument which can either take a string value or a column name. e.g. for vtype the following would be a potential gotcha for users if we allowed both cases:
df.gppd.add_vars(model, ..., vtype="C")  # Does the user mean that a column named
                                         # 'C' should define the variable types, or that
                                         # all these variables should be continuous?

As a result currently vtype is not allowed to be a column name, we always interpret it as a constant (and I would be very hesitant to change that). However, for sense such a clash seems unlikely (though not impossible) since a user would have to create a column named <, >, or =.

Thanks so much Simon! So for large energy market models we often maintain big input files with sets of generic constraints forcing sets of power stations to generate above or below certain levels. Often modelling teams are maintaining these files rather than the developer themselves.

Most of the time these are <= constraints, however we will often run into situations where we want certain categories of generator to maintain a minimum output for system security and so there may be a mix of >= constraints in there as well.
It's mainly the sense that is valuable to have input as a series and be changing in the same constraint set - I can't think of a scenario where we'd need to have the vtype be an input as well.
It's not the end of the world if this is a pain on your end - I mainly just have to split up these kinds of input dataframes into multiple dataframes based on the sense and then apply the constraint to them. That said, if it was possible for the sense to be an optional series that would make things a bit easier and cleaner :)

Makes sense! :-P
I agree, it's nice to have a way to specify range constraints which don't always have both bounds. I've implemented this on my fork; there is a bit more to do to handle invalid inputs and to document this option. I'll work on that soon. If you have a chance to test this you can install the development branch using:
pip install --force-reinstall git+https://github.com/simonbowly/gurobipy-pandas@series-of-senses

Thanks so much! Just installed the fork and will test it out tomorrow - I've been doing a bunch of updates to my code handling this category of generic constraint inputs so will be keen to see how this plays with the changes :)

Implemented in #59


Parse correctly schemas different to AsyncAPI schema

Specification can handle properties from OpenAPI schema, for example, but only when schemaFormat was provided
It's time to choose parsing strategy and bring OpenAPI schema, at least, to this repo to parse it correctly
Related issues:

https://github.com/asyncapi/jasyncapi/issues/160
https://github.com/asyncapi/jasyncapi/issues/163
https://github.com/asyncapi/jasyncapi/issues/164

Available formats: Schema formats table
Reference: https://github.com/asyncapi/spec/issues/1037

@Pakisan, I went through the discussion in 1037.
I just want to clarify my understanding:
Would it be fair to assume that if we wish to use open api 3.0 attributes like 'nullable' and 'example', we need to specify the schemaFormat as 'application/vnd.oai.openapi;version=3.0.0'.
So would this be a valid asnyc api 2.6 spec:
messages:
    product:
      name: product
      title: An inventory product
      summary: Product representing items in inventory
      contentType: application/json
      payload:
        schemaFormat: application/vnd.oai.openapi;version=3.0.0
        type: object
        properties:
          name:
            description: Every product has a name
            type: string
          inventory:
            description: Count of items in inventory
            type: number
            nullable: true
          id:
            description: Unique identifier of the product
            type: number

@Pakisan, I went through the discussion in 1037.
I just want to clarify my understanding:
Would it be fair to assume that if we wish to use open api 3.0 attributes like 'nullable' and 'example', we need to specify the schemaFormat as 'application/vnd.oai.openapi;version=3.0.0'.
So would this be a valid asnyc api 2.6 spec:

messages:

    product:

      name: product

      title: An inventory product

      summary: Product representing items in inventory

      contentType: application/json

      payload:

        schemaFormat: application/vnd.oai.openapi;version=3.0.0

        type: object

        properties:

          name:

            description: Every product has a name

            type: string

          inventory:

            description: Count of items in inventory

            type: number

            nullable: true

          id:

            description: Unique identifier of the product

            type: number

Yes, you are right to use Avro, OpenAPI, ... schemas, user MUST provide schemaFormat to tell which parsing strategy to use
I added this issue to first RC release

Implemented in 1.0.0-RC