isim: "game_center" and "store_kit"

Operating system or device - Godot version:
isim
godot 2.0.x
Issue description (what happened, and what was expected):
In the file "isim/detect.py" no line includes Game Center and Inapp. Gently add them there. But is disabled by default.
def get_opts():
...
                ('game_center', 'Support for game center', 'no'),
		('store_kit', 'Support for in-app store', 'no'),
...

def configure(env):
...
        if env["game_center"]=="yes":
                env.Append(CPPFLAGS=['-fblocks', '-DGAME_CENTER_ENABLED'])
                env.Append(LINKFLAGS=['-framework', 'GameKit'])

        if env["store_kit"] == "yes":
                env.Append(CPPFLAGS=['-DSTOREKIT_ENABLED'])
                env.Append(LINKFLAGS=['-framework', 'StoreKit'])
...

PS.
Why include TOOLS?
def get_flags():

	return [
		('tools', 'yes'),

Steps to reproduce:
Link to minimal example project (optional but very welcome):

Well, this is totally bogus now. The master branch don't have the isim platform anymore, it was merged with the iphone platform.

This has been fixed in the master branch already, as mentioned by @vnen the isim platform was merged in the iphone one, and it no longer uses tools (and now tries to enable game_center and store_kit).
I might cherry-pick the change for the 2.0 branch, but it still needs some more testing, if you could try it it would be great :)

Let's test, I do not mind. What should be done?
Suggestion: Rename platform "iPhone" to "iOS". All the same, when you need to do it :)

Let's test, I do not mind. What should be done?

Once you have cloned the master branch, you should be able to build the simulator (i.e. the x86 iOS binary) with scons p=iphone ios_sim=yes (or scons p=iphone arch=x86, both should work the same).

evening check and let you know the result.

I checked. in 2.1 branch is going 2 ways: scons p=iphone ios_sim=yes and scons p=iphone arch=x86.
if you take the file "iphone/detect.py" of the branches 2.1 and 2.0 branch in the copy then the project team will also be collected, scons p=iphone ios_sim=yes but the command scons p=iphone arch=x86 will not meet.

Yes the arch parameter relies on changes made in the main SConstruct, while the ios_sim parameter is standalone. The two are maybe a bit redundant though, I might remove the ios_sim one to the benefit of the more generic arch parameter (that could be used for other platforms too instead of the more limited bits argument).
Thanks for testing, glad to know it works in both 2.1 and 2.0 :) I'll add relevant changes for 2.0.4 then.

I might remove the ios_sim one to the benefit of the more generic arch parameter (that could be used for other platforms too instead of the more limited bits argument).

probably well worth making.

Just a reminder that there is a task:)

Should now be fixed with 6d6b14d1.

@akien-mga
Something went wrong! :(
I collected two versions of ios: scons p=iphone and scons p=iphone arch=arm64. And both write that: "godot_opt.iphone does not have an architecture that iPhone 4s can execute".
Maybe I'm doing something wrong. You can confirm that you're running?

I can't check as I don't have a mac nor an iphone to test with, but there should have been no changes for scons p=iphone. Was it working for you in 2.0.3 and no longer in <IP_ADDRESS>?

Unfortunately I also has no iPhone.
I compiled and sent a friend to check. By this I am not sure at 100%.
But earlier at another all started (2.0.3) and with the new version (<IP_ADDRESS>) says that it is not working.
Do you have friends who would check it out?

How do you create the final executable that you run on the iphone? You also
need to compile with arch=arm64, and make a universal binary with lipo,
using the arm and arm64 binaries
On 11 July 2016 at 04:47, ret80<EMAIL_ADDRESS>wrote:

Unfortunately I also has no iPhone.
I compiled and sent a friend to check. By this I am not sure at 100%.
But earlier at another all started (2.0.3) and with the new version
(<IP_ADDRESS>) says that it is not working.
Do you have friends who would check it out?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5453#issuecomment-231664057,
or mute the thread
https://github.com/notifications/unsubscribe/AGVmPZhHzFaQQ79To3mbYB6v_uqKonK0ks5qUfUvgaJpZM4I_kBs
.

Indeed, check http://docs.godotengine.org/en/latest/reference/compiling_for_ios.html

Good. Thank you. I'll try, then report the results.

@akien-mga
Everything is working.
Could you see here is the bug #5640.
Thanks in advance.


Temporarily Remove Accounts Returning Empty Responses

Feature Request:
If an account hits the "empty response issue" - can you take it out of the pool of accounts to scan for like x seconds (I would say like 30 seconds or something)

either that or allow us to have separate pool of accounts that can swap in with accounts that get the empty response
Pool A - Active Scanning Accounts
Pool B - InActive Scanning Accounts
Pool C - OnHold Accounts
Move Pool A accounts into Pool C when hitting empty, and move a Pool B Account into Pool A to make up the difference

Seems like a great idea to me.

The problem is, it's kinda hard to identify whether or not an account is banned. It does return a perfectly valid response, it's just nothing in there.
I tested it in a rural area and there it's quite common that there is no stuff in the scanned area.

Yeah thats a problem @nborrmann. Is it possible to get the distance to nearest pokestop? because i guess that would be a non-rural definer. For example if an account gets no returns for 3mins in close distance to pokestops it is most likely softbanned. To make it even better you could perhaps gather data from the google maps API about whether it is on land or not

great idea, +1!!
@nborrmann you already implented the code to see if its softbanned
" if sum(len(cell.keys()) for cell in cells) == len(cells) * 2:" yes?

@djv2 That checks for an empty response. But in rural areas that might be perfectly fine, because there's just nothing there.

@nborrmann the only thing i can think of is still develop the functionality with your code (check empty response) but as a flag. i 100% agree in rural that might be normal, but i know like at least 75%+ of us are scanning cities, let us the users decide whether to use that as a feature

@nborrmann to temp add this feature, can't i just edit parse_map(map_dict) and put everything under your
if sum(len(cell.keys()) for cell in cells) == len(cells) * 2: log.warning("Received valid response but without any data. Possibly rate-limited?")
in an else statement? so if it returns empty it won't use it. that should speed things up yes?

@nborrmann True (empty responses in rural areas).
A little bit more complex, but here's an idea to determine a probability of validness for the empty response:

spawn points are always the same. (I'm not sure if they're exact lat/long every time), but afaik we haven't seen changes in spawn points yet.
spawn points are also predictable in time (once every full hour (e.g. 08:33, 9:33, etc.). Duration is not exactly predictable afaik. (mostly 15m, but 30, 45 or even 1h(?) is possible).

So after a while, the data from the local db should be sufficient to know if there should actually be a pokemon in that scan, or not. (e.g. previous x scans have shown a spawn at every hour there, but now nothing has spawned?)
Those proven assumptions can also be used to for a more intelligent scanning pattern (e.g. not scanning areas without spawns (or only to check if the spawn points have changed from time to time), scanning spawns at the right times etc.)

@nborrmann acouple things we could try here:
Set the re-use interval to random interval from 15 - 20 seconds of accounts rather then 10 seconds
If an account returns an empty response x times then deem it temporarily banned
Do a global API rate limit - so the api can only be called X times per second/minute

If I could understand python I would help, but its so weird compared to C#

Wouldn't it be possible to flag an account that returned 5 empty responses in a row and scan an area where we know that there is definitely a pokemon?
Account 1 scans an area where a pokemon was detected.
Account 2 scans an area without returns an empty response. After 5 scans there were only empty responses. So Account 2 scans the area where Account 1 found a pokemon. If Account 2 returns a pokemon it is not blocked. Otherwise it will be temporarily removed.
I think I do not have to mention that the pokemon has to be there when Account 2 scans the area.

bump - wish this was a priority instead of devs adding features. scanning needs to be #1 :100:


[Event Parameter Request] page 6510 "Item Tracking Lines" - OnBeforeSetWarehouseControls

Many, many years ago somebody requested untested https://github.com/microsoft/ALAppExtensions/issues/13457 and used the wrong variable in
OnBeforeSetWarehouseControls(Rec, IsHandled);
which should have been
OnBeforeSetWarehouseControls(TrackingSpecification, IsHandled);
Has been like that since 18.04.
Alas that somebody is me and now our first heavy item tracking prod->sales->shipment->pick customer wants to test functionality that does work in NAV2013 but does not anymore in 22.01.
Could you please add the parameter TrackingSpecification to the event OnBeforeSetWarehouseControls? It would be very much appreciated. I suppose you don't want to make any changes by just replacing the calling variable but it makes it a bit difficult for the event procedure variable naming. I tried my best.
local procedure SetWarehouseControls(TrackingSpecification: Record "Tracking Specification")
var
    IsHandled: Boolean;
begin
    IsHandled := false;
    //OnBeforeSetWarehouseControls(Rec, IsHandled); Event 22.01.o
    OnBeforeSetWarehouseControls(Rec, IsHandled, TrackingSpecification); //Event 22.01.n
    if IsHandled then
        exit;

[IntegrationEvent(false, false)]
local procedure OnBeforeSetWarehouseControls(TrackingSpecification: Record "Tracking Specification"; var IsHandled: Boolean; var NewTrackingSpecification: Record "Tracking Specification")
begin
    //Event22.01 NewTrackingSpecification
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10963.


docs(contributing): Added page on reviewing pull requests

Part of umbrella issue #1730
Added new documentation with best practices for reviewing a pull request.

@celanthe  - could you change the issue number in your PR description to #1707 ? Thanks!


New .NET Generator with new expected files

(Resubmitting because of a premature merge)
Needs at least this bugfix applied, and potentially more:
https://github.com/awslabs/jsii/commit/4926bfd51085f54da3fd46061a1cc0b44a722c29

Hi,
This contains a new generator for .NET written in typescript with the new expected files
This also removes of the old generator codebase.
Fixes https://github.com/awslabs/jsii/issues/463
Fixes https://github.com/awslabs/jsii/issues/528
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I am working on the merge from master + regen.

The branch is now conflicting, I'll do a merge + conflict resolution + regen


error when updating from 0.9.2 to 0.9.3

When updating a working app I get this error on first render:
TypeError: Cannot read properties of undefined (reading 'subscribe')
    at http://localhost:5173/node_modules/.vite/deps/electric-sql_react.js?v=72bd7483:103:34
    at commitHookEffectListMount (http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:16904:34)
    at commitPassiveMountOnFiber (http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:18152:19)
    at commitPassiveMountEffects_complete (http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:18125:17)
    at commitPassiveMountEffects_begin (http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:18115:15)
    at commitPassiveMountEffects (http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:18105:11)
    at flushPassiveEffectsImpl (http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:19486:11)
    at flushPassiveEffects (http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:19443:22)
    at http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:19324:17
    at workLoop (http://localhost:5173/node_modules/.vite/deps/chunk-YQDERSFP.js?v=72bd7483:197:42)

Here a screenshot:

Reverting electric-sql to 0.9.2 solves the issue.

Hey @barbalex,
There is an API breaking change in this release, "electrification" and connection have been broken into two steps. You are slightly ahead of us on dropping a note on the Discord channel.
The docs are updated: https://electric-sql.com/docs/api/clients/typescript#instantiation
During the "alpha" phase we are only bumping the minor version of electric when here are incompatible changes to the sync protocol. This can mean that the client API with break on a patch release, this will change latter on.


[Book] Examples

Add a documentation page containing a list of reference test files, namely:

Cairo sources
Execution artifacts (trace, memory, input, output)
Prover parameters & config
Proof

It also has to contain a brief description of what the program does, its arguments (if applicable).
Path to your markdown file: docs/pages/examples.md
References:

https://github.com/dipdup-io/stone-packaging/tree/master/test_files
https://github.com/starkware-libs/stone-prover?tab=readme-ov-file#creating-and-verifying-a-proof-of-a-cairo-program

Can I work on this

hello @dmirgaleev please let me work on this :)

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a web3 frontend developer and Technical writer skilled in Typescript, Javascript, Solidity, Cairo and Rust. I also do lots off documentations on Readme files. This will help me in carrying out this task seamlessly.
How I plan on tackling this issue

I will read through the sources files provided and understand the extend of the documentations.
I will explain what the program does and implement that in the documentation.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Can I be assigned this
How I plan on tackling this issue
I can do this

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
@dmirgaleev can I work on this please

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I’m Benjamin, and I’m excited to work on adding a new documentation page for our project that will list reference test files. Since joining OnlyDust, I’ve made 18 contributions, and I’m confident in my documentation skills, so I believe I can make this page very useful for our community.
How I plan on tackling this issue
The goal is to set up a page that lists various test files.
Here’s what I plan to include:

Cairo Sources: The actual code for the Cairo programs.
Execution Artifacts: This will cover traces, memory data, and input/output files to show how the programs run.
Prover Parameters & Config: Details about the settings used when running the programs.
Proof: The proof generated from each Cairo program.
I’ll make sure each entry has a simple description explaining what the program does and any arguments it might take. This way, users can quickly get the hang of what each program is about.

I’ll save this new markdown file in the docs/pages/examples.md path to keep everything organized.
To help me gather all the info and examples,
I’ll check out these resources:

DipDup Stone Packaging Test Files: This repository has a bunch of useful test files we can feature.

Stone Prover Repository: This one offers great tips on creating and verifying proofs for Cairo programs.

Once I put together all the reference test files and fill out the documentation, I’ll review everything to make sure it’s clear and easy to understand. Then, I’ll share it with you for feedback, so we can ensure it really meets the needs of what its meant for.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
My name is Collins Ikechukwu. I'm a full stack blockchain developer developer.amd a technical writer
How I plan on tackling this issue

Create a Documentation Page
Start by creating a new markdown file at the specified path:
docs/pages/examples.md

List Reference Test Files
On the page, list reference test files that include:

Cairo sources
Execution artifacts (trace, memory, input, output)
Prover parameters & config
Proof

Brief Program Descriptions
For each program, include a brief description that explains:

What the program does
Any arguments it accepts (if applicable)

Use Existing References
Refer to existing test files and examples from:

Test File Examples
Proof Creation

@dmirgaleev  Thanks will begin working asap

Hello @Benjtalkshow! Welcome aboard! Please don't hesitate to join our Telegram group https://t.me/baking_bad_odhack to ask questions and share your progress!  Happy hacking!

Hello @Benjtalkshow! Welcome aboard! Please don't hesitate to join our Telegram group https://t.me/baking_bad_odhack to ask questions and share your progress! Happy hacking!

Alright. Thanks

Friendly heads up that you have until Sunday (October 6) to push your final PR. @Benjtalkshow

Friendly heads up that you have until Sunday (October 6) to push your final PR. @Benjtalkshow

Alright ser

Can I be assigned this issue @dmirgaleev

Hi @dmirgaleev  please can I be assigned to this issue.. I am available to work

Hello @dmirgaleev, may I give this a try?

@dmirgaleev I'd love to contribute here. I understand the structure of test data and happy to document it. I can also improve on example programs (to have arguments for example)

@baitcode awesome, let's go!:)

Can't start today. Will start working on Friday!

No worries, no strict timelines for this

Hey @baitcode! What's the status?

@dmirgaleev  I've postponed it a bit. Been doing cairo quests on NodeGuardian to get more acquainted with the language. I'll submit PR soon. Pardon for silence. https://nodeguardians.io/character/baitcode

@dmirgaleev created draft PR to this issue. I think I'll finish today or the next day.

@dmirgaleev Could you, please, review?

Resolved in #85


Prepare documentation with description of raw data stored in S3 bucket

Raw data stored in S3 bucket

Generated documentation:
https://redhatinsights.github.io/insights-data-schemas/internal-pipeline/raw_data_S3_bucket.html
Documentation source:
https://github.com/RedHatInsights/insights-data-schemas/blob/master/docs/internal-pipeline/raw_data_S3_bucket.md


PXP-625: [CLI] Report the Wukong version to telemetry

Summary

Ticket: PXP-625
What's Changed

Added

[x] Add cli version into telemetry

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jk-gan, onimsha
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-31-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4759

Questions ?
Please refer to the Backport tool documentation


Bugs always bring down the application

As the title says now since the update to flutter 2.5 whenever a mistake is made in the code that is not indicated in the editor with a red mark, the application crashes completely instead of showing you the exception in console as it has always done . That is very annoying because you always have to hit the play button until the application gets good and thus be able to do a hot restart or you have to stop it completely and start it again.
For instance, every time I change a StatelessWidget for a StatefulWidget and save my application it crashes because it tries to do the hot reload, and I have to hit the play button until it finally lets me restart the app. If I didn't restart it like this, I would have to stop it and restart completely because it won't let me continue. This is something very annoying that did not happen before, so it would be good if you did something about it, because obviously I already know that if I have changed a StatelessWidget for a StatefulWidget that can cause me problems and I have to restart but anyway it is not necessary to stop all my application but to indicate it to me by console as it has always done and so I fix it before continuing again, since now when it stops it makes you waste a lot of time ...

@hekhy
Please provide flutter doctor -v and a minimal reproducible code sample that we can use, to verify the issue you are facing.
You provided a description of the behavior you are seeing, but in order to make this issue actionable, we need steps to replicate along with reproducible code sample to work with.
Thanks.

I'm not going to do that, I've already explained the problem to you well. I have also put a very clear working example, you only have to generate the app and start it in the emulator, once it is started you change a StatelessWidget in your code for a StatefulWidget and save the file, the application automatically stops.

@hekhy
Thanks for the update.  Based on details you provided above, I tried to replicate it, but was unable to see any issues with it.
After updating StatelessWidget to Stateful and saving the changes, the app remain intact, didn't close or quit / stop. There weren't any logs in the console too:

I did this test on latest stable 2.5.1.
If you have specific steps to replicate the issue, please provide us the same.

https://user-images.githubusercontent.com/70041483/134675688-cf01b38a-88b9-4153-8011-2c8517c3f05a.mp4
Here is a video that shows what I am saying. I'm also on flutter 2.5.1, I'm surprised that this doesn't happen to you, or if it happens to you, but as in your screenshot you only show the bottom part and you don't show the debugging controls, which is what I do. I am referring and have already said it several times. But well in the video you can see how the debugging stops therefore my application crashes until I restart it. So I have to hit the play button in the debugging controls or emulator controls, whatever you like to call them more, but if I don't hit play all those times when I hit hot restart it stays in the attempt showing all the time the message "Performing hot restart" at the bottom right, which would force me to completely stop debugging and start again to continue working.
As I have already said this I did not flutter before, I do not understand why we have gone backwards in that.

@hekhy
Thanks for the details. As I see from the video, it throws myApp is not a subtype of StatelessWidget in type castexception. When we change fromStatelesstoStateful,  we need to do a hot restart, since this change is invoked in main. If we hot reload, it directly calls the build()function and hence the exception. We need to be able to callinitState()` which gets called after hot restarting the app after making the state change.
Can you try the same, ie, after changing from stateless to stateful, save and then hot restart and see if you get same error or not ?

Can you try the same, ie, after changing from stateless to stateful, save and then hot restart and see if you get same error or not ?

Yes, of course, but I can tell you that it just keeps trying to do the "hot restart" for an infinite time.
Something that I mentioned in the previous answer, but I will also make a video so you can see it.
https://user-images.githubusercontent.com/70041483/135094857-400b9490-2670-49d9-8117-1dc470a47f4f.mp4
As you can see, he keeps trying to do a hot restart but he never succeeds, in the video I only left him a little time for not making it long but I tried it before leaving it even 10 minutes and it continued like this.
Then as you can see I have to completely stop the application to continue with my work, but if it is not this way it does not work unless I do it as I did in the previous video.
Be that as it may, it is a bit annoying because I already know that it is logical that the app has to be started again so that it uses the initState() method and thus the StatefulWidget is loaded correctly, but this did not do it before. If I simply threw the error in the console without stopping everything and then executing a hot restart would restart the application and everything would continue working, it would be a great advance.

@hekhy
Thanks for these details. I tried to follow all the steps you mentioned as below:

Ran flutter counter app.
Changed StatelessWidget to Stateful and saved the changed.
Had breakpoint at MyApp class.
Hot restart which gave me errors, as MyApp class didn't satisfy the stateful class rules as below:

Are you purely trying these in debugging mode ?

So, I always start the app in debug mode

Thanks. Keeping this issue open for further insights from the team.

Seems related / similar to https://github.com/flutter/flutter/issues/89788

Gracias. Mantener este problema abierto para obtener más información del equipo.

There is no reason, I want him to fix himself as much as you do.


Upgrade task to use get_checksum instead of deprecated get_md5

When running on the stable version of ansible, I got the following error:
TASK [sunfoxcz.dkim : ensure signing key is present] *******************************************************************
fatal: [hostname]: FAILED! => {"changed": false, "msg": "Unsupported parameters for (stat) module: get_md5. Supported parameters include: checksum_algorithm, follow, get_attributes, get_checksum, get_mime, path (attr, attributes, checksum, checksum_algo, dest, mime, mime-type, mime_type, name)."}

After looking online for the error, I found this occurrence for a similar case:
https://github.com/geerlingguy/ansible-role-swap/issues/32
Seems like get_md5 is deprecated, so this change modifies it to use get_checksum.

@foxycode can you make a new release including this fix?

Done


Contrast pretty low on web page

That's controlled from here: https://github.com/dart-atom/dartlang/blob/gh-pages/_posts/2014-01-01-intro.md and here: https://github.com/dart-atom/dartlang/blob/gh-pages/_config.yml#L29

Fixed, but contributions welcome :)


A question about vs2022 packaging

When I open the source code for compilation with vs2022, I find that my files seem to be missing. How do I deal with this, boss

Not sure to fully understand your issue. Do you have a screenshot or error message to share?


Add jQuery to the kit, so it's available on all pages by default

Hey Joe - what do you think about automatically including all js files in the js folder so users don't have to manually add them?

all js is automatically copied from app/assets to public - is that what you mean? Or do you mean automatically create <script> tags?

My thinking was to grab all the js in the js folder and combine them in to a single app.js that gets included.

Is that useful in prototypes? I can see the advantage for production code (fewer requests)


fix: node ip allocation log level

This PR changes the log level of IP allocation.
The expected behavior is, once the maximum number of IP is hit on the interface, a new one will be created.
Switching this log from warning to info would prevent workflow failures on intended behavior.

Fixes: #36428
Switch IP allocation log level of hitting max number of IPs on interface from warning to info

I'm a bit afraid this might be papering over a real issue here where we apparently miscalculate AvailableForAllocation. In other words, I don't think this is an expected error. After all, the warning is emitted only if we believe that we can still allocate more IPs, but for some reason this fails

Noted, closing this PR


Connection leak under very specific conditions

How to reproduce

do not authenticate
have a restricted item with files on the main page
load the page

How to verify

enable hibernate statistics
look at CP and you will see e.g.,

Hibernate #connections (global): 103
Hibernate #sessions opened (so far): 104
Hibernate #sessions closed (so far): 66

fixed in pull request #153


Investigate updating AWS CloudFormation templates/blueprints

Is your feature request related to a problem?
Investigate what sort of CloudFormation templates/blueprints etc. people use for AWS Lambda, EKS, CodeBuild, etc. that could be easily automated for upgrades by the tool.
Describe the solution you'd like
Any relevant blueprints and template properties are updated for newer .NET releases, such as for the AWS Lambda dotnet8 runtime.
Describe alternatives you've considered
None.
Additional context
None.

There's some information on what sort of things might be required in this blog post.

Example SAM template (template.yaml):
AWSTemplateFormatVersion: '2010-09-09'
Transform: AWS::Serverless-2016-10-31
Description: >
  Sample SAM Template for sam-app

# More info about Globals: https://github.com/awslabs/serverless-application-model/blob/master/docs/globals.rst
Globals:
  Function:
    Timeout: 10
    MemorySize: 128

    Tracing: Active
    # You can add LoggingConfig parameters such as the Logformat, Log Group, and SystemLogLevel or ApplicationLogLevel. Learn more here https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/sam-resource-function.html#sam-function-loggingconfig.
    LoggingConfig:
      LogFormat: JSON
  Api:
    TracingEnabled: true
Resources:
  HelloWorldFunction:
    Type: AWS::Serverless::Function # More info about Function Resource: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#awsserverlessfunction
    Properties:
      CodeUri: ./src/HelloWorld/
      Handler: HelloWorld::HelloWorld.Function::FunctionHandler
      Runtime: dotnet6
      Architectures:
      - x86_64
      MemorySize: 256
      Environment: # More info about Env Vars: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#environment-object
        Variables:
          PARAM1: VALUE
      Events:
        HelloWorld:
          Type: Api # More info about API Event Source: https://github.com/awslabs/serverless-application-model/blob/master/versions/2016-10-31.md#api
          Properties:
            Path: /hello
            Method: get

  ApplicationResourceGroup:
    Type: AWS::ResourceGroups::Group
    Properties:
      Name:
        Fn::Sub: ApplicationInsights-SAM-${AWS::StackName}
      ResourceQuery:
        Type: CLOUDFORMATION_STACK_1_0
  ApplicationInsightsMonitoring:
    Type: AWS::ApplicationInsights::Application
    Properties:
      ResourceGroupName:
        Ref: ApplicationResourceGroup
      AutoConfigurationEnabled: 'true'
Outputs:
  # ServerlessRestApi is an implicit API created out of Events key under Serverless::Function
  # Find out more about other implicit resources you can reference within SAM
  # https://github.com/awslabs/serverless-application-model/blob/master/docs/internals/generated_resources.rst#api
  HelloWorldApi:
    Description: API Gateway endpoint URL for Prod stage for Hello World function
    Value: !Sub "https://${ServerlessRestApi}.execute-api.${AWS::Region}.amazonaws.com/Prod/hello/"
  HelloWorldFunction:
    Description: Hello World Lambda Function ARN
    Value: !GetAtt HelloWorldFunction.Arn
  HelloWorldFunctionIamRole:
    Description: Implicit IAM Role created for Hello World function
    Value: !GetAtt HelloWorldFunctionRole.Arn

The template can also be pointed to by aws-lambda-tools-defaults.json:
{
  "profile": "",
  "region": "",
  "configuration": "Release",
  "template": "template.yaml"
}

There's also layers that can have a runtime.


[tool_crash] FileSystemException: Cannot open file, OS Error: No such file or directory, errno = 2

Command
flutter --no-color run --machine --device-id=080C9A33-DC22-49BA-B564-C6DABC9570EB --start-paused lib/main.dart

Steps to Reproduce

...
...
...

Logs
FileSystemException: Cannot open file, OS Error: No such file or directory, errno = 2
#0      _File.throwIfError (dart:io/file_impl.dart:635:7)
#1      _File.openSync (dart:io/file_impl.dart:479:5)
#2      ForwardingFile.openSync (package:file/src/forwarding/forwarding_file.dart:71:16)
#3      ErrorHandlingFile.copySync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:315:22)
#4      _runSync (package:flutter_tools/src/base/error_handling_io.dart:597:14)
#5      ErrorHandlingFile.copySync (package:flutter_tools/src/base/error_handling_io.dart:314:5)
#6      CocoaPods.setupPodfile (package:flutter_tools/src/macos/cocoapods.dart:260:21)
<asynchronous suspension>
#7      injectPlugins (package:flutter_tools/src/flutter_plugins.dart:1138:9)
<asynchronous suspension>
#8      FlutterProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:381:5)
<asynchronous suspension>
#9      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1328:7)
<asynchronous suspension>
#10     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1183:27)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#12     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#13     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)

[✓] Flutter (Channel stable, 3.0.1, on macOS 12.6 21G115 darwin-arm, locale en-IN)
    • Flutter version 3.0.1 at /Users/karanchaudhary/Developer/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (4 months ago), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

[!] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/karanchaudhary/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2021.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.67.2)
    • VS Code at /Users/karanchaudhary/Downloads/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • iPhone 14 Pro Max (mobile) • 080C9A33-DC22-49BA-B564-C6DABC9570EB • ios            • com.apple.CoreSimulator.SimRuntime.iOS-16-0 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   • macOS 12.6 21G115 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript • Google Chrome 105.0.5195.125

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: f4abaa0735eba4dfd8f33f73363911d63931fe03
Plugins
path_provider-2.0.2
path_provider_linux-2.0.0
path_provider_macos-2.0.0
path_provider_windows-2.0.1
shared_preferences-2.0.6
shared_preferences_linux-2.0.0
shared_preferences_macos-2.0.0
shared_preferences_web-2.0.0
shared_preferences_windows-2.0.0

possibly related to
https://github.com/flutter/flutter/issues/68958
https://github.com/flutter/flutter/issues/91559

Closing this as a duplicate of https://github.com/flutter/flutter/issues/91559


cannot get example to work

Hi,
Thanks for all your hard work on this wonderful project. Recording and rtmp forwarding are important features to me and your work will save me a lot of stumbling in the dark.
I was having problems getting the example to work on my local machine. Here's a typical log:
https://gist.github.com/fozzle/0a4bfa9899de22ce5e74b601b3497098
Also, I cannot open the rtsp source with VLC, it gives "codec not found errors" but then sometimes will start showing the stream but without audio. Ha.
Funny enough, I've gotten RTMP forwarding to a remote media server (Wowza) to work just fine, but the audio is extremely desynced.
I wanted to see if I could debug the desync but since I can't get the example to run it's hard to isolate.
Am I missing something?

What version of this project do you use?
Of course I recommend to use the latest, but you should know that 1.0.1 is the first stable version.

I was able to make some recordings this morning with 1.0.4. Didn't make any changes so unsure why it wouldn't have worked before. Will go ahead and close this for now.


[EDIT] Jacked Ape Club NFT project team erupts in chaos

Entry
Jacked Ape Club NFT project team erupts in chaos, February 13, 2022
Changes
...they simply removed the trio because their work was done, and becausue because they said...
Agreement
By submitting this issue, I irrevocably agree to release my contribution under the CC-BY 3.0 License. I agree that a hyperlink or URL is sufficient attribution under the Creative Commons license.
Please attribute this contribution to:
carpenterd777

(p.s. this site is fascinating to read. thank you for creating and maintaining it!)

Thank you!


Migrate to CDK v2

Migrate to CDK v2
https://docs.aws.amazon.com/cdk/v2/guide/migrating-v2.html
Migrate tests @aws-cdk/assert (deprecated) -> aws-cdk-lib/assertions
<EMAIL_ADDRESS>

resolves #


Provide a Server with pub/sub functionality

If we have to test alerts e2e, we need a server that we can both publish and consume alert messages in the format that is specified in https://www.oasis-open.org/committees/download.php/14759/emergency-CAPv1.1.pdf

dibs!


Normalization non-numeric data input

Symbols "+", "-", "e" can be inputted into normalize value and break/freeze app.

Invalid parameters now trigger the "Normalization not possible" warning and normalization is not applied. If needed to proceed, users must select another normalization choice.


Unable to deploy via SAM. Getting error "Plugin 'ServerlessAppPlugin' raised an exception: AWS Region cannot be found"

Hi, I'm trying to deploy SAR-Lambda-Janitor via a SAM template. I've added it to the Resources section of my SAM template:
Resources:
  SarLambdaJanitor:
    Type: AWS::Serverless::Application
    Properties:
      Location:
        ApplicationId: arn:aws:serverlessrepo:us-east-1:374852340823:applications/lambda-janitor
        SemanticVersion: 1.5.0
      Parameters:
        VersionsToKeep: 3

However, when I run:
$ sam build --debug -t ./template.yaml
I am seeing the error:
Plugin 'ServerlessAppPlugin' raised an exception: AWS Region cannot be found
I am not sure how to solve this. Any help would be much appreciated. Thank you!

same error for SemanticVersion: 1.6.1
$ sam deploy --guided
2024-04-16 11:06:26 Plugin 'ServerlessAppPlugin' raised an exception: AWS Region cannot be found Traceback (most recent call last):   File "samtranslator/plugins/sam_plugins.py", line 130, in act   File "samtranslator/metrics/method_decorator.py", line 116, in wrapper_cw_timer   File "samtranslator/plugins/application/serverless_app_plugin.py", line 150, in on_before_transform_template   File "samtranslator/region_configuration.py", line 48, in is_service_supported samtranslator.translator.arn_generator.NoRegionFound: AWS Region cannot be found


Release 6.1.4

inputFinalizeGetAmts now includes an extra parameter to disable the fee check.
Add getters for input and output amounts of psbt transactions.

Nice


Differentiate suboptimal_program from a program that fails to execute

Currently, when searching using allow_evaluation_errors and all of the programs error out on evaluation (interpret errors out), the return value of synth is marked as a suboptimal_program. This is somewhat confusing because it seems to say that the returned program executed but didn't pass evaluation for some IO examples.
We can either:

add a third enum value to SynthResult like failing_program
https://github.com/Herb-AI/HerbSearch.jl/blob/5b315ce7ebf6c78df9609e01a7bf2d9c784dba65/src/search_procedure.jl#L10
warn that allow_evaluation_errors might mean that the returned suboptimal_program might have thrown an error

This is a good idea!
Especially because if we have a program failing to run, we might be able to deduce additional constraints to remove programs that would fail for the same reason.


[sui bridge] use base64 for encoding serde and jsonrpc schema

Description
when using jsonrpc to fetch the bridge_pubkey_bytes the Vec<u8> are verbosely displayed:
curl suix_getLatestBridge
         {
            "suiAddress": "0xce2039cd80188004f995cdfe1360b31d4118bd4257febee958b6c6dcd861131d",
            "bridgePubkeyBytes": [
              2,
              76,
              184,
              43,
              243,
              119,
              91,
              211,
              113,
              87,
              ...
            ]
 }

In comparison, it is much easier to parse the bytes when they are automatically encode to b64, eg:
https://github.com/MystenLabs/sui/blob/ef4639cc7095820e7ff8471bf77fa2a46a81a719/crates/sui-types/src/sui_system_state/sui_system_state_summary.rs#L225-L227
Test plan
I'll test against a testnet rpc node
looks like usages of bridge_pubkey_bytes and http_rest_url are not around serialization/deserialization, so this shouldn't have an effect outside of jsonrpc.
https://github.com/MystenLabs/sui/blob/ef4639cc7095820e7ff8471bf77fa2a46a81a719/crates/sui-bridge/src/sui_client.rs#L235-L240

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

Closing as this isn't explicitly necessary for the metrics-proxy work


Document what coordinate system the subarea extraction uses

What problem does your feature request solve?

Currently it uses the grid coordinates, while the documentation in rose edit doesn't make it clear whether real world coordinates, or grid coordinates should be used. This is relevant for rotated pole domains, as for non-rotated pole domains they are the same.
Describe the solution you'd like

This should be clearly documented in the rose edit description.
Describe alternatives you've considered

We should consider whether we actually want to use the grid coordinates, or whether we should use the real world coordinates, and regrid them before use. I'm not sure what would be expected here.

@daflack / @Sylviabohnenstengel do you have an opinion on whether we should be generally be using grid native coordinates, or real world coordinates?

l would generally suggest real world coordinates being easier for the user than grid native coordinates. One reason is how the grid native coordinates in regional models can behave near the meridian. A second is that it would save people having to go into their data to find the relevant co-ordinate system before being able to use CSET.

Fixed in #765.


Revert "Add a rate limiter for chunk downloader. (#316)"

This reverts commit 8fa3afd5180ccddf9e0d16412fe0621db053d601 (#316). This approach was unsuccessful for several reasons; one - because regardless of the amount of chunks, they are downloaded at the maximum throughput possible; two - multiple instances can be deployed to a single logical host which still results in host saturation; three - instance types have different throughput maximums and this approach provides no mechanisms to throttle against this.
We will be pursing the approach in #232 (or similar) to set a maximum download in throughput instead of chunk concurrency.

To followup on this - we'll be pursuing using Kubernetes traffic shaping for rate limiting at this point instead of a code solution.


Unable to connect to the remote server [EventID: 1553]

Hi, I'm having trouble with invoke PS plugins working with the API.
I state that I have updated all the modules to the latest version, following all the various steps recommended for installation.
I get the error "Unable to connect to the remote server" EventID 1553. I have registered the various PS plugins, with the command: Add-IcingaRESTApiCommand -Command 'Invoke-IcingaCheck *' -Endpoint 'apichecks' -Whitelist ;.
Can you help me solve the problem?
Thanks so much,
Greetings

Hello,
thank you for the issue. Is the Icinga PowerShell service running?
Get-Service icingapowershell

What happens if you restart the service? The EventLog should print error messages in case the Listener Socket cannot be established. Do you have any of these log messages inside your EventLog: https://icinga.com/docs/icinga-for-windows/latest/restapi/doc/20-Eventlog/
Can you please share the information for these commands:
Get-Module -ListAvailable -Name 'icinga-powershell-restapi';
Get-Module -ListAvailable -Name 'icinga-powershell-apichecks';

Get-IcingaBackgroundDaemons;

Show-IcingaRESTApiCommands;

Get-IcingaFrameworkApiChecks;

Get-IcingaAgentHostCertificate;

Are you using the certificates generated by the Icinga Agent or have you provided a custom certificate to create the REST-Api socket?
https://icinga.com/docs/icinga-for-windows/latest/restapi/doc/02-Installation/#daemon-registration

Hello,
Thanks for your quick response.
The Icinga Powershell Service is running.
Restarting the service, I get the "EventID 2000" error.
I am using the certificate generated by Icinga Agent, for the registration of the rest-api daemon, I used the command:
"Use-Icinga;
Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi'; "

Get-IcingaFrameworkApiChecks;
True
By issuing the command: "Get-IcingaAgentHostCertificate;" it does not report any results in output.

To register the rest api as a background daemon, I used the built-in command: "Use-Icinga;
Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi'; "
Update:
By registering the API-rest with the following command, the problem seems to have been solved .. I had to pass it the path of the certificate generated by default by Icinga Agent.
Command:
"Use-Icinga;
Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi' -Arguments @ {'-Port' = 5669; '-CertFile' = 'path / to / your / cert / file'}; "
I await your feedback for a correct resolution of the problem.
Thanks so much!
Greetings

Thank you for the update. I believe the issue is coming from the -RequireAuth argument.
You registered the API to require authentication to access the API, which is causing a problem for the build-in API check handler, as there is no support available for this option.
Running this alone should already work fine, as long as -RequireAuth is not set.
Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi'

Am I correct here?

Hello,
I tried to register the API as you suggested, with the command: "Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi';"
I have not set any "-RequireAuth", but I still encounter the two problems mentioned (EventId 2000 and EventId 1553).

I tried on two different servers, in the first I ran the command: "Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi';" and I get error 1553 and 2000.
On the second server, I ran the command: "Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi' -Arguments @ {'-Port' = 5669; '-CertFile' = 'path / to / your / cert / file'};" and I don't have any problems ..
What do you think about it? In all servers, I have not entered any Basic Auth.
Thank you very much!!

Ì came to the conclusion about the basic auth, because the argument is shown on your first screenshot on the registered background daemon. In general the values you enter are the default ones.
For some reason the daemon is unable to auto-fetch the required certificate. Normally the Framework will use
Get-IcingaSSLCertForSocket

To create the socket, which can either be defined manually or used automatically with the Icinga Agent certificates. If the command is returning nothing or an empty result, this means the certificate for the Agent cannot be looked up.
If you manually configure the path to the certificate, it should work.
Could you please execute this snippet inside  an Icinga shell and tell me, if the hostname and/or the certificates inside FolderContent are matching or if the naming is entirely different?
[string]$CertDirectory = (Join-Path -Path $Env:ProgramData -ChildPath 'icinga2\var\lib\icinga2\certs\*');
$FolderContent         = Get-ChildItem -Path $CertDirectory -Filter '*.crt' -Exclude 'ca.crt';
$Hostname              = Get-IcingaHostname -LowerCase $TRUE;

$FolderContent;
$HostName;

Update:
despite the certificate passed manually, the eventid error 1553 returned.
Command: "Use-Icinga;
Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi' -Arguments @ {'-Port' = 5669; '-CertFile' = 'path / to / your / cert / file'}; "
Eventid 2000 seems to be resolved.
The only remaining error is the eventid 1553.
Can we make other checks?
Thanks so much!

Hi, we are also experiencing these errors and it is blocking our deployment, quite critical. Was there some change with the last update. We were not experiencing this error with the previous version.
Cheers!

There were no changes which should cause the deployment or the execution to fail. Have you installed the latest version from both, the Framework, the REST-Api and the Checker component?
We improved the Performance and fixed some issues with the service, but it should run just fine without having to configure anything with the Icinga Agent installed and configured with certificates.
If no Agent is present, you have to manually define the certificates. On case there is no socket creation error, the API should just work.
Can you access https://localhost:5668/v1 on the machines with the service installed?

Hi,
we tried a clean reinstall, for all modules we used Icinga-FrameworkComponent. The installation was successful.
The registration of the daemon with the API was done manually, by running the command:
Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi' -Arguments @{ '-Port' = 5668; '-CertFile' = 'C:\ProgramData\icinga2\var\lib\icinga2\certs\ my cert file' };
[Notice]: Background daemon Cmdlet "Start-IcingaWindowsRESTApi" has been configured
Running the "Get-IcingaAgentHostCertificate;" command, returns nothing in the output.
Instead by running the suggested command inside a shell:

Is it normal that the command: "Get-IcingaAgentHostCertificate;" does not report any results in the output?
The previous error 2000 seems to have been fixed, but the 1553 still hasn't returned after the last clean install.
I do a couple of checks and see if error 1553 returns.
We are currently experiencing errors 1500 and 1501, they occur about 30 minutes after restarting the services.

From browser going to "https: // localhost: 5668 / v1", it seems to work correctly:

Currently we only need to fix error 1500 and 1501, they occur about 30 minutes after the services restart.
How can we solve?
Thank you!
Greetings

There were no changes which should cause the deployment or the execution to fail. Have you installed the latest version from both, the Framework, the REST-Api and the Checker component?
We improved the Performance and fixed some issues with the service, but it should run just fine without having to configure anything with the Icinga Agent installed and configured with certificates.
If no Agent is present, you have to manually define the certificates. On case there is no socket creation error, the API should just work.
Can you access https://localhost:5668/v1 on the machines with the service installed?

We have a similar situation to @pakyg, resolved a few issues using the Icinga-FrameworkComponent.

The errors 1500 and 1501 are mostly occurring in case you are using a browser to connect to the API which throws the certificate error. If you are connecting by using Invoke-WebRequest, curl or wget you should always use insecure flags to prevent certificate checking, as otherwise this error occurs.
Another solution would be, to use signed host certificates by your CA which your browser will trust.

I tried to run the command:
"Use-Icinga; Enable-IcingaUntrustedCertificateValidation;
Invoke-WebRequest -Method GET -UseBasicParsing -Uri 'https: // localhost: 5668 / v1 / checker? Command = cpu'; "

Instead, launching the command from the browser (with the CA installed manually), I can see it correctly.
The problem remains when I try to run an Invoke-WebRequest from Powershell because it keeps giving me an error.
I will update you in case of further developments.
Thanks!
Greetings

Can you please try in addition on the shell to Enable-IcingaUntrustedCertificateValidation, Set-IcingaTLSVersion?
This should ensure you can talk to the TLS 1.2 certificate from the Agent.

Hi,
I tried to run the additional command "Set-IcingaTLSVersion":

Thanks!

Hi, i resume this discussion for a new issue.
After restarting the Icinga Powershell service, port 5668 is no longer listening and there is no way to get the API to work.
Passing the command:"Use-Icinga; Enable-IcingaUntrustedCertificateValidation; Set-IcingaTLSVersion;
Invoke-WebRequest -Method GET -UseBasicParsing -Uri 'https://localhost:5668/v1/checker?Command=cpu';", reports "Could not connect to remote server" and port 5668 is no longer listening.
After a reboot, it stops working and brings me back various EventId 1553..
[
Agent Version: 2.13.1
ApiChecks: 1.2.0
Framework Version: 1.6.1
Plugins: 1.6.0
RestApi: 1.2.0
Icinga Service: 1.1.0
Icinga Powershell Service: NetworkService
Icinga2 Agent: LocalSystem
Windows Server 2016
PSVersion: 5.1
We also find the same problem on Windows 2012 R2
]
How could i solve? Sometimes it works, sometimes it doesn't.
Thanks!

Can you please run Test-IcingaForWindowsService and check if this shows any errors? If so, you can use Test-IcingaForWindowsService -ResolveProblems to fix possible found issues.
I would assume that that the service is not loading the .psd1 but the .psm1 which might cause this issue, as you are running 1.6.x

Hi, we have updated all modules to the latest version:
-Framework 1.8.0
-Plugins 1.8.0
-Agent 2.13.1
-Service 1.2.0
-Icinga Powershell Service: LocalSystem
-Icinga2 Agent: LocalSystem
-PSVersion 5.1
-Windows Server 2016
We installed everything through Icinga Windows Reposiotry, all modules were installed with the command: "Install-IcingaComponent".
Unfortunately we continue to encounter the same problem, in the log we find the event id 1553. When we try to launch the invoke, the call keeps failing.

We tried to follow the various steps suggested in this thread, issuing the command: "Use-Icinga; Enable-IcingaUntrustedCertificateValidation; Set-IcingaTLSVersion;
Invoke-WebRequest -Method GET -UseBasicParsing -Uri 'https: // localhost: 5668 / v1 / checker? Command = cpu'; "

Keep reporting the same error to us, as if it's not working properly. Could you help us solve this problem?
I did not understand in the official Icinga guide, the various RestApi and ApiChecks modules must be installed separately or are they integrated into the framework?
How should I register the daemon? In the background: "Register-IcingaBackgroundDaemon -Command 'Start-IcingaServiceCheckDaemon';"
or with the Api method: "Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi';" +
Add-IcingaRESTApiCommand -Command 'Invoke-IcingaCheck *' -Endpoint 'apichecks'; +
Enable-IcingaFrameworkApiChecks;
For the installation, we followed the following guide: "https://icinga.com/docs/icinga-for-windows/latest/doc/110-Installation/01-Getting-Started/"
I await your news. Thanks so much! :)

Hello,
with the Framework version v1.8.0 this was made a lot easier. Please do the following steps:

Start PowerShell as admin
Type the command icinga (IMC opens)
Enter menu 2 Settings
Enter menu 6 Icinga for Windows Features
Use Entry 0 Api-Check Forwarder to enable them

In case they are already enabled, enter menu 0 to disable and enable them again afterwards.
In general all dependecies should be resolved then.

Thanks for your quick response!
The API-Check Forwarder was already enabled, I disabled it and enabled it again. I have done the recommended actions but the problem persists.

I have a question, how should I register the daemon in Icinga? In the background: "Register-IcingaBackgroundDaemon -Command 'Start-IcingaServiceCheckDaemon';"
or with the Api method: "Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi';" +
Add-IcingaRESTApiCommand -Command 'Invoke-IcingaCheck *' -Endpoint 'apichecks'; ?
I await your news, thank you very much

With the new option you do not require to do that - it's done for you.
In general, you simply require to run these commands:
Register-IcingaBackgroundDaemon -Command 'Start-IcingaWindowsRESTApi';
Add-IcingaRESTApiCommand -Command 'Invoke-IcingaCheck*' -Endpoint 'apichecks';
Enable-IcingaFrameworkApiChecks;
Restart-IcingaWindowsService;

Thats it. I have no idea on why on your environment this does not work. Can you please verify that

Icinga for Windows service is running (Get-Service icingapowershell)
The is actually something listening on Port 5668

netstat -aon | ForEach-Object { if ($_ -like '*5668*') { $_ } }

Ok perfect for demon registration
I ran the following commands, the output:

For some reason the REST-Api still is unable to locate your Icinga Agent certificate.
Can you please try this:
Install-IcingaForWindowsCertificate -CertFile 'C:\ProgramData\icinga2\var\lib\icinga2\certs\<name of the host crt>.crt'

Afterwards restart the PowerShell service:
Restart-IcingaWindowsService

With this a certificate is generated from the Icinga Agent certificate and stored at \certificate within the icinga-powershell-framework folder.
In general this is only required for JEA installations, but for some reason your environment is not looking up the certificate properly.
Note: In case the Agent certificates gets renewed, you will have to run the command Install-IcingaForWindowsCertificate again to update it.

In addition: If the above worked, you can run Show-Icinga.
There is a section for the Icinga for Windows certificate at the top:
Icinga for Windows Certificate:

Issuer  => CN=Icinga CA
Subject => CN=<your hostname>

If you find it useful, I add the general overview:

But the Icinga for Windows certificate is not installed. You can please veriy that there actually is a proper .crt file on this location?
C:\ProgramData\icinga2\var\lib\icinga2\certs\

Okay, now when you run
Install-IcingaForWindowsCertificate -CertFile 'C:\ProgramData\icinga2\var\lib\icinga2\certs\scdbmop0170.....crt'

with your domain added to this, will this return a certificate?
Get-IcingaForWindowsCertificate

Handle Issuer       Subject
------ ------       -------
1794332493424 CN=Icinga CA CN=self-service

Ok it seems correct, I do not understand why it did not install automatically during installation

I restarted the Icinga Powershell service, I get the following error:

Thank you very much for your availability

It is still not working? I have one final idea. Please take the previous .crt path from the Install-IcingaForWindowsCertificate command and register your daemon again like this:
Register-IcingaBackgroundDaemon -Command Start-IcingaWindowsRESTApi -Arguments @{ '-CertFile' = '<.crt path>' }

Verify this with:
Show-IcingaRegisteredBackgroundDaemons;

List of configured background daemons on this system:

Start-IcingaServiceCheckDaemon
-----------
No arguments defined

Start-IcingaWindowsRESTApi
-----------
-CertFile => <.crt path>

Then restart the service again:
Restart-IcingaWindowsService

Then it should definately work. I do however still not undertand, on why the certificate is not found on your system.

Ok I have performed all the steps .. I can not understand why it continues to be in error.
Perhaps the services with the daemon have not registered correctly?

Ah, close your PowerShell session. If you run the Start-Icinga* commands within your shell, you will block the daemon from opening the socket on the port. Close the current PowerShell, open a new one and just restart the service again.
Then it might work.


Coverage data for the Travis build?

In a former project we had coverage data for the builds and one could drill down into that just like into the html pages generated locally by issue #272.
Stackoverflow posting visual-code-coverage-in-travis-ci suggests that travis builds do not keep any data, so that the coverage data would have to be send elsewhere. The Stackoverflow posting mentions coveralls. [travis help lists codeclimate.com](http://docs.travis-ci.com/user/code-climate/#sts=Measuring Test Coverage with Code Climate), which is free for open source projects.
CodeClimate also does some analysis on the source code, which might be worthwhile as well.
I also found a recent blog post Better Javascript with Gulp, Mocha, Istanbul and Codeclimate..
Here is an example dashboard page for node. Unfortunately I did not find any page which actually shows real coverage data and whether one can drill down as nicely as on ones own machine.
Should there be coverage for our travis builds?

I think this kind of integration would be very useful, especially when it comes to evaluating PRs. We should make some time to tinker with it, perhaps in another pairing session?

Yes, seems like a good idea to approach this together.

This has been done via coveralls see pull request #293


fix: add 'packages' key check to Composer 2

In composer 2 has added a "packages" key. This PR checks for this key.

This is a duplicate of #212. I've been using that fork for a couple of weeks now and have not had an issue.

It's going to be fixed in https://github.com/EdgedesignCZ/phpqa/pull/218/commits/673aab47c7fcc518fdae59437f56c6ebfa69187a


feat: allow to import custom dashboards

Description
This PR allows importing custom superset assets and overriding default ones by using a tutor patch superset-extra-assets.
This has multiple advantages:

Multiple custom plugins can extend the definition of assets by providing their own assets without breaking anything.
Developers can override assets by using the same uuid.

The script works by creating individual files in their corresponding asset type folder. The patch structure is as follows:
name: custom-inline-plugin
version: 0.1.0
patches:
  superset-extra-assets: |
      - file_name: my-dashboard.yaml
        dashboard_title: "..."
        ...
      - file_name: my-chart.yaml
        slice_name: "..."
        ...
      - file_name: my-database.yaml
        database_name: "..."
        ...
      - file_name: my-dataset.yaml
        table_name: "..."
        ...

Notice that it's defined as a list, so you can pass multiple assets separated as a yaml list item.
Screenshots:

This PR:

Fixes #75
Fixes #63

Thanks for the pull request, @Ian2012!  Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
This is currently a draft pull request. When it is ready for our review and all tests are green, click "Ready for Review", or remove "WIP" from the title, as appropriate.

@pomegranited What do you think of this approach?

@mariajgrimaldi @bmtcril this is ready for review

This is currently failing to import the current dashboards due that the passwords can not be sent along with the file database definition, working on it

@bmtcril @mariajgrimaldi Finally, this is working, please review it now

@pomegranited What I didn't like about the multiple-folder approach was the complexity of managing that many folders in k8s through config maps, and I didn't find a way to dynamically create files supported by tutor.
On the other hand, there is already a script that converts a zip file and generates a .yaml file with the result.
I will add documentation about it and improve it to be a tutor do command

Looks like some PyCharm files got added by mistake, but you should be good to merge when those last changes are in

👍 awesome! Feel free to merge once the formatting issues are sorted

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

@Ian2012

I will add documentation about it and improve it to be a tutor do command

Perfect, thank you!


Add Micro support

I'm a bit confused about this PR: There isn't yet any arduino-micro feature in arduino-hal so this can't possibly work?

my fault. https://github.com/Rahix/avr-hal-template/blob/main/avr-specs/avr-atmega32u4.json was there so ...
but yes I understand. I also compared main vs next.

I'll close this PR for now, I think it makes more sense to take a fresh look at it once support in avr-hal is implemented.


[WIP] string to system IPEndpoint

helper for custom conversion from gRPC URI to IPEndpoint.
helper for normal ip:port format to IPEndpoint.

closes #1719

:x: Build AElf <IP_ADDRESS>99 failed (commit https://github.com/AElfProject/AElf/commit/3aa9468905 by @samfrb)

:white_check_mark: Build AElf <IP_ADDRESS>74 completed (commit https://github.com/AElfProject/AElf/commit/983023dc6b by @samfrb)


Task 284-285

There are some tags (e.g.,<br />) in the inputs that should be removed. Also, the instances are not shuffled.

I will take this one up.


Create shopify theme metafields pull command

WHY are these changes introduced?
Part of metafield auto-completion ticket https://github.com/Shopify/theme-tools/issues/502
WHAT is this pull request doing?

Fetch metafields by ownerType from Admin API and store it in a .shopify/metafield.json

If the command isn't ran in a theme directory, we will request a confirmation (can be bypassed with force flag)
If some ownerTypes fail to be fetched, we provide an info message (but still create a file)
If all ownerTypes fail to be fetched, we provide an error message (and don't create a file)

How to test your changes?

get the repo
get a theme
run the command shopify theme metafields pull

pass the path to your theme using --path=<path-to-theme> flag
pass the force flag to bypass the confirmation prompt
pass the -s flag to select your shop (there may be a bug atm where it always defaults to the example shop)

Post-release steps
Measuring impact
How do we know this change was effective? Please choose one:

[x] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[x] I've considered possible documentation changes

"theme metafields pull" makes me think we're pulling metafields that are attached to the theme.

UPDATE

To ensure we aren't going to run into Metafield definition access errors, we updated the Admin API so that as long as the user has read/write themes, they can access all the required fields for each resource type: https://github.com/Shopify/shopify/pull/558573

If they don't have read/write theme permissions, and don't have access to the owner type, they can't fetch the metafields using the CLI command

Is this likely? That someone has permission to download definitions for some types but not others? If it is, are they going to see this error a lot?

@shauns After the Core PR, we shouldn't be throwing a lot of permission errors per resource as long as you have themes permission on your account. Let me know if you have any other concerns for this PR.

@karreiro Just tested environments and using theme access app and everything works! We don't need need to worry about how the user is running the dev or metafields pull command.


Publishing on greasyfork.org, userstyles.world, and reddit

I've published two of the scripts to https://greasyfork.org:

https://greasyfork.org/en/scripts/456690-bitbucket-commit-links-in-prs
https://greasyfork.org/en/scripts/456692-jira-pull-request-link-improver

I also wanted to publish the scripts to https://userstyles.world/, but they are having issues with login via OAuth at the moment: https://github.com/userstyles-world/userstyles.world/issues/134#issuecomment-1339587753
There are also r/bitbucket and r/jira on reddit.
Should we publicize the user scripts and user styles on reddit? @dtruebin, @svlukas what do you think?
P.S. While looking at other scripts on greasyfork, I noticed that people often confuse Jira/Bitbucket Cloud with Jira/Bitbucket Server. Most of the scripts are historically for the Server variant (as are scripts and styles in atlassian-tweaks) and they are usually not compatible with the Cloud variant.

Why not 🧐
Please consider adding a screenshot to https://greasyfork.org/en/scripts/456692-jira-pull-request-link-improver.
By the way, I am pretty sure Jira/Bitbucket Cloud have a free tier, so the scripts can be checked for compatibility with them.

Please consider adding a screenshot to https://greasyfork.org/en/scripts/456692-jira-pull-request-link-improver.

Good idea, done. Also added on in the README: c1712ff3f57d0b35ebff9c3c45b45f07c2e2a008.

Created a script set on greasyfork.org: https://greasyfork.org/en/scripts?set=585074

It includes the newly published https://greasyfork.org/en/scripts/457217-jira-project-icon-as-tab-icon as well

Posted to reddit:

r/atlassian: https://www.reddit.com/r/atlassian/comments/zwizky/atlassian_tweaks_a_collection_of_user_scripts_and/
r/jira: https://www.reddit.com/r/jira/comments/zwjfki/atlassian_tweaks_a_collection_of_user_scripts_and/
both r/bitbucket and r/confluence are restricted in who can post, though.

As it turns out, greasyfork.org also supports publishing user styles: https://greasyfork.org/en/scripts?language=css

[ ] publish the rest of scripts on greasyfork.org
[ ] publish CSS on greasyfork.org
[ ] wait for OAuth issue to be resolved and publish styles to userstyles.world

I've found out that jira_copy_summary.user.js in its current implementation is disallowed to be published on Greasy Fork. It is automatically rejected with error message

Code uses an unapproved external script: @require https://raw.githubusercontent.com/odyniec/MonkeyConfig/0eaeb525/monkeyconfig.js

because of
// @require      https://raw.githubusercontent.com/odyniec/MonkeyConfig/0eaeb525/monkeyconfig.js

Stumbled upon another website for user scripts: https://openuserjs.org. Searched for "Jira" just out of curiosity and on the first page of search results got four Copy Summary analogues:

https://openuserjs.org/scripts/floodmeadows/Jira_add_descriptive_link
https://openuserjs.org/scripts/wmosakowskicraft/Jira_Task_to_Clipboard
https://openuserjs.org/scripts/milanfarkas/Jira_Agile_Task_to_Clipboard
https://openuserjs.org/scripts/milanfarkas/Jira_Cloud_Copy_Task_Key_and_Title_to_Clipboard

https://userstyles.world/ is available again! 😃
Started off with https://userstyles.world/style/9186/bitbucket-fast-and-furious
I'll import all user styles soon™.

Uploaded all userstyles that I wanted to upload to userstyles.world:

Bitbucket: header color
Bitbucket: fix breadcrumbs selection
Bitbucket: hide banner
Bitbucket: fast and furious
Confluence: better monospace
Confluence: hide banner
Confluence: Simple Floating ToC
Jira: better inline code formatting

The only exception is Bitbucket: fix commit tooltip, which doesn't seem to be needed for modern Bitbucket Server. I haven't uploaded it to Greasy Fork either.

Last Greasy Fork update
Uploaded the rest of userstyles (except "Bitbucket: fast and furious") to Greasy Fork: https://greasyfork.org/en/users/951503-andrybak?language=css&set=585074
As per comment above I can't upload "Jira copy summary". And I don't want to bother with "Jira: Copy code blocks", because I don't use it and it isn't that good UX-wise.

Stumbled upon another website for user scripts: https://openuserjs.org.

I've tried uploading to this website, and I've decided not to upload scripts of Atlassian Tweaks there for now. See also https://tinyurl.com/24sy79e5 :-/

I've found out that jira_copy_summary.user.js in its current implementation is disallowed to be published on Greasy Fork. It is automatically rejected with error message

The problem has been solved in commit 9eecf2d (jira_copy_summary: use full SHA1 hash for monkeyconfig.js, 2023-09-21). The script is now published at Greasy Fork: https://greasyfork.org/en/scripts/475815-jira-copy-summary


[VTK]: Using find_package(VTK....) in a outside project leads to CMake Error

Describe the bug
We have a big project that use VTK, and specifically vtkIOImport component. By doing:
find_package(VTK QUIET COMPONENTS vtkIOImport REQUIRED)

CMake returns:
CMake Error at E:/Work/vcpkg/installed/x64-windows/share/vtk/FindLZ4.cmake:2 (set_target_properties):
  Attempt to promote imported target "lz4::lz4" to global scope (by setting
  IMPORTED_GLOBAL) which is not built in this directory.

Which seems perfectly normal since the find_package() is done outside VTK source or VCPKG tree.
Environment

OS: Winwdows 10 x64
Compiler: Microsoft (R) C/C++ Optimizing Compiler Version 19.28.29334 for x64
Triplet: x64-windows
CMake: cmake version 3.19.1

To Reproduce
Use
find_package(VTK QUIET COMPONENTS vtkIOImport REQUIRED)

in a CMakeList.txt, outside VCPKG tree.
Expected behavior
No CMake errors
Additional context
A quick fix is to simply delete the FindLZ4.cmake in vcpkg\installed\x64-windows\share\vtk. Or, for a cleaner solution, to patch the port file to remove it prior installing. Not sure this should not be done for some other FindXXX.cmake as it seems that the:
set_target_properties(XXXX PROPERTIES IMPORTED_GLOBAL TRUE)

are problematic, and  vcpkg\installed\x64-windows\share\lz4 should provide what is required to use LZ4, even for VTK, ITK, PCL, OpenCV, Paraview..
If removing FindLZ4.cmake is the right way to fix that, I could propose a PR for that.
Regards,

find_package(vtk CONFIG REQUIRED)
...
target_link_libraries(${PROJECT_NAME} PRIVATE LZ4::LZ4 OGG::OGG VTK::lz4 VTK::ogg)
then cmake, it throw
1> CMake generation started for default configuration: 'x64-Debug (default)'.
1> Found and using vcpkg toolchain file (E:/vcpkg/scripts/buildsystems/vcpkg.cmake).
1> Command line: "cmd.exe" /c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Ninja"  -DCMAKE_BUILD_TYPE:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="G:\vcpkg_project\test_hello_window\out\install\x64-Debug (default)" -DCMAKE_C_COMPILER:FILEPATH="C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.28.29333/bin/Hostx64/x64/cl.exe" -DCMAKE_CXX_COMPILER:FILEPATH="C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.28.29333/bin/Hostx64/x64/cl.exe"  -DCMAKE_MAKE_PROGRAM="C:\PROGRAM FILES (X86)\MICROSOFT VISUAL STUDIO\2019\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\Ninja\ninja.exe" -DCMAKE_TOOLCHAIN_FILE="E:/vcpkg/scripts/buildsystems/vcpkg.cmake" "G:\vcpkg_project\test_hello_window" 2>&1"
1> Working directory: G:\vcpkg_project\test_hello_window\out\build\x64-Debug (default)
1> [CMake] -- Searching for PEGTL
1> [CMake] -- Searching for PEGTL - found target taocpp::pegtl
1> [CMake] CMake Error at E:/vcpkg/installed/x64-windows/share/vtk/patches/99/FindHDF5.cmake:4 (set_target_properties):
1> [CMake]   Attempt to promote imported target "hdf5::hdf5-shared" to global scope (by
1> [CMake]   setting IMPORTED_GLOBAL) which is not built in this directory.
1> [CMake] Call Stack (most recent call first):
1> [CMake]   E:/vcpkg/installed/x64-windows/share/hdf5/vcpkg-cmake-wrapper.cmake:17 (_find_package)
1> [CMake]   E:/vcpkg/scripts/buildsystems/vcpkg.cmake:573 (include)
1> [CMake]   E:/vcpkg/installed/x64-windows/share/vtk/VTK-vtk-module-find-packages.cmake:256 (find_package)
1> [CMake]   E:/vcpkg/installed/x64-windows/share/vtk/vtk-config.cmake:129 (include)
1> [CMake]   E:/vcpkg/scripts/buildsystems/vcpkg.cmake:622 (_find_package)
1> [CMake]   gtest/CMakeLists.txt:21 (find_package)
1> [CMake] CMake Error at E:/vcpkg/installed/x64-windows/share/vtk/patches/99/FindHDF5.cmake:13 (set_target_properties):
1> [CMake]   Attempt to promote imported target "hdf5::hdf5_hl-shared" to global scope
1> [CMake]   (by setting IMPORTED_GLOBAL) which is not built in this directory.
1> [CMake] Call Stack (most recent call first):
1> [CMake]   E:/vcpkg/installed/x64-windows/share/hdf5/vcpkg-cmake-wrapper.cmake:17 (_find_package)
1> [CMake]   E:/vcpkg/scripts/buildsystems/vcpkg.cmake:573 (include)
1> [CMake]   E:/vcpkg/installed/x64-windows/share/vtk/VTK-vtk-module-find-packages.cmake:256 (find_package)
1> [CMake]   E:/vcpkg/installed/x64-windows/share/vtk/vtk-config.cmake:129 (include)
1> [CMake]   E:/vcpkg/scripts/buildsystems/vcpkg.cmake:622 (_find_package)
1> [CMake]   gtest/CMakeLists.txt:21 (find_package)
1> [CMake] -- Searching for PEGTL
1> [CMake] -- Searching for PEGTL - found target taocpp::pegtl
1> [CMake] CMake Error at E:/vcpkg/installed/x64-windows/share/vtk/FindPEGTL.cmake:26 (set_target_properties):
1> [CMake]   Attempt to promote imported target "taocpp::pegtl" to global scope (by
1> [CMake]   setting IMPORTED_GLOBAL) which is not built in this directory.
1> [CMake] Call Stack (most recent call first):
1> [CMake]   E:/vcpkg/scripts/buildsystems/vcpkg.cmake:622 (_find_package)
1> [CMake]   E:/vcpkg/installed/x64-windows/share/vtk/VTK-vtk-module-find-packages.cmake:491 (find_package)
1> [CMake]   E:/vcpkg/installed/x64-windows/share/vtk/vtk-config.cmake:129 (include)
1> [CMake]   E:/vcpkg/scripts/buildsystems/vcpkg.cmake:622 (_find_package)
1> [CMake]   gtest/CMakeLists.txt:21 (find_package)
1> [CMake] CMake Error at E:/vcpkg/installed/x64-windows/share/vtk/FindLZ4.cmake:2 (set_target_properties):
1> [CMake]   Attempt to promote imported target "lz4::lz4" to global scope (by setting
1> [CMake]   IMPORTED_GLOBAL) which is not built in this directory.
1> [CMake] Call Stack (most recent call first):
1> [CMake]   E:/vcpkg/scripts/buildsystems/vcpkg.cmake:622 (_find_package)
1> [CMake]   E:/vcpkg/installed/x64-windows/share/vtk/VTK-vtk-module-find-packages.cmake:1008 (find_package)
1> [CMake]   E:/vcpkg/installed/x64-windows/share/vtk/vtk-config.cmake:129 (include)
1> [CMake]   E:/vcpkg/scripts/buildsystems/vcpkg.cmake:622 (_find_package)
1> [CMake]   gtest/CMakeLists.txt:21 (find_package)
1> [CMake] -- Configuring incomplete, errors occurred!
1> [CMake] See also "G:/vcpkg_project/test_hello_window/out/build/x64-Debug (default)/CMakeFiles/CMakeOutput.log".
1> [CMake] See also "G:/vcpkg_project/test_hello_window/out/build/x64-Debug (default)/CMakeFiles/CMakeError.log".

change
+find_package(LZ4 CONFIG REQUIRED)
+set_target_properties(lz4::lz4 PROPERTIES IMPORTED_GLOBAL TRUE)
+add_library(LZ4::LZ4 ALIAS lz4::lz4)

to
+find_package(LZ4 CONFIG REQUIRED)
+if(NOT TARGET LZ4::LZ4)
+    add_library(LZ4::LZ4 INTERFACE IMPORTED)
+    target_link_libraries(LZ4::LZ4 INTERFACE lz4::lz4)
+endif()

(and fix the git diff header)
I found that using an imported interface target instead of an alias is more robust and less problematic in general.

Duplicated to #9947.

change

I forgot to reply, but yes your suggestion is working ! Thanks

I believe I am still experiencing this with the current vtk in vcpkg.

@drescherjm
But I saw the changes is :
+find_package(LZ4 CONFIG REQUIRED)
+set_target_properties(lz4::lz4 PROPERTIES IMPORTED_GLOBAL TRUE)
+if(NOT TARGET LZ4::LZ4)
+add_library(LZ4::LZ4 ALIAS lz4::lz4)
+endif()

For me having
set_target_properties(lz4::lz4 PROPERTIES IMPORTED_GLOBAL TRUE)
causes the error that was originally reported. If you delete the FindLZ4.cmake file from the installed\x64-windows\share\vtk folder or use the fix from https://github.com/microsoft/vcpkg/issues/14978#issuecomment-740077047 it works.

@drescherjm Okay, thanks for your info, I will fix this later.

For me having
set_target_properties(lz4::lz4 PROPERTIES IMPORTED_GLOBAL TRUE)

causes the same error that was originally reported when using vtk in a project that uses cmake

I guess set_target_properties(lz4::lz4 PROPERTIES IMPORTED_GLOBAL TRUE) breaks if the lz4..lz4 target wasn't  created in the immediate line before but by a transitive find_package(lz4 ...) for another dependency before. So don't modify IMPORTED_GLOBAL unless you created it in the same file.

I will fix this later.

There is #23352.

Thanks.

I am sure this will fix the issue but will test tomorrow or Saturday with a clean vcpkg and report if there is any issue.

Found a related problem in the recent LZMA patch. I commented on that pull request.


Verbose loge messages to console by default

Printing to console all http messages by default leads to verbose output: https://github.com/mukel/telegrambot4s/blob/e1d6c9938d43d1daef5e1f16460f37da5c7a6372/src/main/scala/info/mukel/telegrambot4s/api/TelegramApiAkka.scala#L59.
In many case i would like to ignore this messages, but i can't. Please, use logging instead of printing to console. For example, using https://github.com/typesafehub/scala-logging. It will help users to control library output.

My bad, I left some println dirt, I started to add basic logging.
, but didn't quite finished, I'll do some cleanup, but I know just the basic about logging. In the meantime if you have a better way that fullfill real use cases please feel free to submit a PR, it will be really appreciated.

I think I solved the println issue in #19 . But did not work in logging yet :stuck_out_tongue:


metrics :: add prometheus metrics

Summary

This PR implements the following features

Prometheus Metrics
Custom metrics on current number of chrome rendering

Hi team,
First, thanks for this amazing work, the project rocks!
The project can be more awesome, if we could implement more metrics to facilitate the monitoring and scaling of the service under load, in fact besides the CPU et RAM usage, the project doesn't provide enough metrics to ensure a more efficient scaling under a big load.
I've added these custom metrics to have a more reliable and convenient metric on which we can scale the Gotenberg service when is deployed on solutions like Kubernetes.
It's more convenient to scale service as Gotenberg based on his current state rather than the current resources usage because each render is different from another intern of resource usage and obviously time rendering, with this custom metric we can imagine implementing an HPA (HorizontalPodAutoscaler) which will scale up /down our service based on the current number of rendering and have a more efficient way to determine if our current stack can handle more requests or it has to be scaled up.
Of course, my code is not perfect, and if you have any advice or want to discuss this usage, feel free to ping me!
If this PR and feature seem legit to you, I will continue and update the documentation adding a metrics section.

Test plan (required)
How to test?

Run a K6 loadtest to convert an URL to PDF.
Run this command to get the metrics:  curl <IP_ADDRESS>:3000/metrics
At this point you should see all the default metrics of Prometheus as well as our new custom one named chrome_current_rendering

Checklist

[x] Have you followed the guidelines in our CONTRIBUTING guide?
[x] Have you lint your code locally prior to submission (make lint)?
[ ] Have you written new tests for your core changes, as applicable?
[x] Have you successfully ran tests with your changes locally (make tests)?
[ ] Have you updated the documentation (Markdown files under build > docs > content and then make doc)?
[ ] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code

Hello @ArcRiiad
Sorry for the abrupt closing; it was done automatically because of the deletion of the master branch...
Could you open a new PR? 😄

With the new module system, I wonder if we could offer a way to provide metrics "per module" similarly to the health checks.


Update pyproject.toml to bump cryptography  version minimum

Bumping cryptography to 42 minimum, when they added the not_valid_before_utc function to the cryptography library.
not_valid_before_utc
Added in version 42.0.0.

Was getting an error when it was using version 41.0.7 on my machine.

Thank you for your contribution @pixitha, you're right it should be bumped to the minimum version. Just don't forget to update the poetry.lock file as well.


UDP broadcast example

Is there any example usage available to demonstrate setting the socket options[if necessary] to allow for a UDP broadcast using the  .write(from:,to:) command?
When I write to "<IP_ADDRESS>" I get a Udp write Error on a test network that allows broadcasts.

What error are you getting?  Can you show me some code?  Thanks.

It shows an SOCKET_ERR_WRITE_FAILED - Permission Denied
UdpAsciiBuffer[0] = 48
UdpAsciiBuffer[1] = 49
UdpAsciiBuffer[2] = 50
UdpAsciiBuffer[3] = 51

let addr = Socket.createAddress(for: "<IP_ADDRESS>", on: UdpPort)

do
     {
     try TestUdpSocket?.write(from: UdpAsciiBuffer, to: addr!)
      }
catch let error
    {
    print("Udp write Error")
    }

UdpPort = 1234
If the address is specified to a unique IP, data is sent, but when the IP is set to 255 an error is returned.

This is fixed in the latest release, 0.12.58.  I added a new API, udpBroadcast(enable: Bool), that you should call after creating the socket and before attempting to write to it.  See the documentation in the README.md for more info.

After updating and setting UdpBroadcast to true, a Permission denied error is returned when using the testReadWriteUDP() example function where hostname = "<IP_ADDRESS>" and port set to 1234.
Are you able to confirm a "Hello from UDP" udp broadcast across the network using the testReadWriteUDP example function?

Awesome! Works now. Thanks!


fix: rename field to prevent IE runtime error

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Summary:
uniqueID is a readonly property on elements in IE. Assigning to this.uniqueID was causing a runtime error in some frameworks. To fix this, I renamed the field to uniqueId.
For reference
Type of change:
Please delete options that are not relevant.

[x] Revision of an existing capability

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 1.1.2 :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 1.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Datapipeline poc 1

What does this PR do?

Fixes # (issue)
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests? [not needed for typos/docs]
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @tchaton! Thanks for opening this PR.

In the file flash/core/model.py:

Line 159:9: E265 block comment should start with '# '

In the file flash/data/data_pipeline.py:

Line 34:121: E501 line too long (138 > 120 characters)
Line 42:121: E501 line too long (132 > 120 characters)
Line 45:121: E501 line too long (205 > 120 characters)
Line 54:121: E501 line too long (205 > 120 characters)
Line 135:121: E501 line too long (138 > 120 characters)
Line 143:121: E501 line too long (132 > 120 characters)
Line 146:121: E501 line too long (205 > 120 characters)
Line 155:121: E501 line too long (205 > 120 characters)
Line 188:121: E501 line too long (140 > 120 characters)

In the file flash/vision/classification/data.py:

Line 261:9: E722 do not use bare 'except'

In the file flash_examples/finetuning/image_classification.py:

Line 49:1: E402 module level import not at top of file

Do see the Hitchhiker's guide to code style

@tchaton @justusschock can we get this done asap as it blocks transition to proper PL version

@Borda why does it block? This should be independent.

@Borda why does it block? This should be independent.

pls, check https://github.com/PyTorchLightning/lightning-flash/pull/133#issuecomment-785083496


New Batch API support

Codecov Report

Merging #1001 into master will increase coverage by 0.15%.
The diff coverage is 82.3%.

@@            Coverage Diff             @@
##           master    #1001      +/-   ##
==========================================
+ Coverage   57.69%   57.85%   +0.15%
==========================================
  Files        1927     1931       +4
  Lines       53341    53398      +57
==========================================
+ Hits        30773    30891     +118
+ Misses      22568    22507      -61

Impacted Files
Coverage Δ

azure-batch/azure/batch/models/job_list_options.py
27.27% <ø> (ø)
:white_check_mark:

...re-batch/azure/batch/models/job_disable_options.py
27.27% <ø> (ø)
:white_check_mark:

azure-batch/azure/batch/models/task_get_options.py
23.07% <ø> (ø)
:white_check_mark:

...ure-batch/azure/batch/models/job_enable_options.py
27.27% <ø> (ø)
:white_check_mark:

...-batch/azure/batch/models/job_disable_parameter.py
100% <ø> (ø)
:white_check_mark:

...-batch/azure/batch/models/job_terminate_options.py
27.27% <ø> (ø)
:white_check_mark:

...ure-batch/azure/batch/models/cloud_job_schedule.py
100% <ø> (ø)
:white_check_mark:

...-batch/azure/batch/models/pool_resize_parameter.py
100% <ø> (ø)
:white_check_mark:

...-batch/azure/batch/models/task_scheduling_error.py
100% <ø> (ø)
:white_check_mark:

azure-batch/azure/batch/models/job_add_options.py
42.85% <ø> (ø)
:white_check_mark:

... and 206 more

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 77dd758...3c8fd54. Read the comment docs.

@lmazuel - This is ready to merge if you think everything looks properly set up for release :)


FancyZones w/ NVIDIA Resized Resolutions

ℹ Computer information
PowerToys version: v0.29.3
PowerToy Utility: FancyZones
Running PowerToys as Admin: Yes
Windows build number: 19042.685
📝 Provide detailed reproduction steps

Resize desktop in the NVIDIA Control Panel by going to Adjust Desktop Size and Position > Size > Resize
Use FancyZones

✔ Expected result
User should be able to use different layouts and/or custom layouts.
❌ Actual result
User can only use the default 3 column layout despite applying different layouts.
📷 Screenshots

Was a little premature. Turns out disabling the "Allow zones to span across monitors" setting enables this.

Was a little premature. Turns out disabling the "Allow zones to span across monitors" setting enables this.