Custom Ingress rules

what

Added list of objects, which are Security Rule Ingress Definitions.

why

Sometimes I need to add security groups to access my EKS workers, such as other EC2 instances on a variety of ports. Same with RDS instances.

references

Link to any supporting github issues or helpful documentation to add some context (e.g. stackoverflow).
Use closes #123, if this PR closes a GitHub issue #123

With my inputs of ```
custom_ingress_rules = [
{
description = "Cow Rule 1"
from_port = 443
to_port   = 443
protocol = "TCP"
source_security_group_id = REDACTED0
},
{
description = "Cow Rule 2"
from_port = 80
to_port   = 80
protocol = "TCP"
source_security_group_id = REDACTED1
}
]

I got

aws_security_group_rule.custom_ingress_rules["REDACTED0"] will be created

resource "aws_security_group_rule" "custom_ingress_rules" {

description              = "Cow Rule 1"
from_port                = 443
id                       = (known after apply)
protocol                 = "tcp"
self                     = false
source_security_group_id = "REDACTED0"
to_port                  = 443
type                     = "ingress"
}

aws_security_group_rule.custom_ingress_rules["REDACTED1"] will be created

resource "aws_security_group_rule" "custom_ingress_rules" {

description              = "Cow Rule 2"
from_port                = 80
id                       = (known after apply)
protocol                 = "tcp"
self                     = false
source_security_group_id = "REDACTED1"
to_port                  = 80
type                     = "ingress"
}

/test all

/test all


add integration with harbor

add integration with harbor

Can you be more specific? How would you like Harbor to be integrated into ACS?

like these repositories

We do not have any plans to add an explicit Harbor integration at this time. We encourage the community to consider adding this integration, if desired :)
I will close this as a duplicate of https://github.com/stackrox/stackrox/issues/8854


[W-9393925, W-9495251] New date fields on application

Changes
Open Date and Due Date fields have been added to the application object.
Issues Closed
W-9393925 - EDA
W-9384949 - R&A
New Metadata
Fields

Application -> Open Date
Application -> Due Date

Changed Metadata
Layout

EDA Application Layout

Testing Notes

Initial PR #1821

QA from both AC and EDA pass. Merging


Allow setting the default database type/version in global config

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
Currently, the default DB version is in the Go code.
// MariaDBDefaultVersion is the default MariaDB version
const MariaDBDefaultVersion = MariaDB104

Describe your solution
I would like to be able to specify the default, perhaps per project type, in the global config YAML. That way, whenever I create a new Drupal project, it will default to using the DB type and version that I am targeting.
Describe alternatives
The current alternative is to edit config.yaml prior to running ddev config. However, if I forget, then I have to drop the database and rerun the commands to get it set up.
Additional context
This isn't super important or high priority, but it's something that I would like to have.

You don't have to edit config.yaml prior to running ddev config. Just use ddev config --project-type=<type> --database=mysql:8.0 or whatever. I do know the problem though, I also forget to set database type correctly.
I don't think that setting a global database type default is probably something that would be a high priority right now though, since you're the first to ever suggest it. I do wish that all config items had global defaults though.
Interested in doing a PR? There are a number of recent PRs that added global default config.

My company prefers to use Postgres for most projects. We would benefit from a setting where new projects default to Postgres.

That's not exactly what would happen here, unless the global config were set up for each user in advance. But thanks for the vote!

This would be solved by

https://github.com/ddev/ddev/issues/5696

I don't expect to pursue this.


Search Functionality

Hi Greg, not sure if this where I should post this question. If not, my apologies.
Question, I am using the SelectBoxIt plugin and it's working great. I have a requirement to make my dropdowns searchable. Especially if it's a dropdown which has a lot of options.
Is this something which is supported in the current version of SelectBoxIt?
Thanks,
John

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/966919-search-functionality?utm_campaign=plugin&utm_content=tracker%2F23157&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F23157&utm_medium=issues&utm_source=github).

Hi Greg, first of all, thank you very much for this great plugin. I have been using it on my dummy projects and it works like charm. I would love to collaborate if you ever need help.
Also, Is there any way to add a textbox or a cursor so it is obvious to user that they can search? End users can't really assume they can start typing and it would search (think of not so tech savvy people who don't know select supports searching to a minimal level if they start typing).
It would be a great add-on to this amazing plugin.

@gfranko hello greg, has any updates about this theme?


feat(kms): support the EKM service

This is a new gRPC service in the same googleapis directory (google/cloud/kms/v1) so it goes into the same library.
Fixes #8483.

This change is

I think we want a note somewhere in the CHANGELOG that this service has been added. Like we did in #8422 for bigquery
Can you add the new environment variable here: https://github.com/googleapis/google-cloud-cpp/blob/70e0a81b4095320037381bbcd6132724bcda25af/google/cloud/kms/doc/main.dox#L44-L46

This makes me think we may want/need a section in our howto doc for "adding new services to GA libraries". Even if it's just a two item checklist. What do you think?


wanting to add new characters

hey. I like the app for the most part, but wanted to add a few characters myself, but looking at the code I'm confused in a few places. mostly the create skill list. what do the numbers mean in the arrays? I tried changing one of the characters values to 1 ... 22, but then when I reloaded the app all their skills were 12. which doesn't make any sense.
Did you use a tool to build this? Some of your data types/methods seem like odd choices to me :P

Hi there. I'm glad you're interested in the app. I have not actively worked on this app in quite some time and don't have immediate plans to pick it back up. As you can see with a note to myself/anyone listening here: https://github.com/calcorum/pacg-char-tracker/issues/5 I agree that this app needs some serious optimization.
I used this app as a Java refresher / intro to Android programming so I won't pretend to be any sort of expert.
Regarding your question...the array for character creation is an arbitrary series of data from the cards used to populate the database. I'll dig into my notes in the next few days and see if I can find the explanation. I'm sorry for the mess :/

I did actually manage to figure out a few more things. I see how the arrays are used in the powers/skills creations functions. I think I might try to implement character objects, it should make adding more characters easier and at least somewhat less confusing. I'll probably just take most of what you have and try to refactor it somewhat.


Update Composer dependencies (2020-11-30-00-12)

Visual regression test passed!
View the visual regression test report


ALL localised files for Mobile Workspace 1.1 for iOS, based on EN 202…

…1-06-24, as per LOC-313.

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


[Slider] Draw only when needed.

This uses the new "Clear" action of the ShapePainter indroduced in GDevelop 5.0.0-beta115.
It avoid to rebuild the shapes at every frame.
An example, to test: https://www.dropbox.com/s/pwwtqds56e5048f/MarchingSquares.zip?dl=1
and its build: https://games.gdevelop-app.com/game-b6f7ae43-151b-4cd1-95fe-fc30cadcb56c/index.html

Excellent, thanks for taking the time to improve this!


Replacing of files

Is it possible to replace file on drive? For example, I had photo on drive, which was downloaded to my app. In my app I've cropped photo and now I want upload cropped photo to drive using replacing. Is it possible to do that avoiding removing?

You can PUT against the content of a file, and it will replace the binary content of that file.  It will update the relevant metadata on that item such as the size, but not alter any of the OneDrive metadata such as sharing links.
final byte[] replacementBinaryContent;
final ICallback<Item> contentReplacedCallback;
client.getDrive()
      .getItems("{Item-Id}")
      .getContent()
      .buildRequest()
      .put(replacementBinaryContent, contentReplacedCallback);

Good! Thanks!


TypeError: Cannot read property 'GlobalStyle' of undefined on Release 2.2.3

When using Release 2.2.3 I get the following error on startup:
TypeError: Cannot read property 'GlobalStyle' of undefined
The code is pretty simple:
class App extends Component {

  render() {
    return (
      <div>
        <Board editable
        data={data} />
      </div>
    )
  }
}

What's interesting is that if I switch the dependency to react-trello 2.1.3 and do npm install, npm start it works.
More specifically, the break seems to happen between 2.1.4 and 2.2.0

Positively fixed in 2.2.5. Please reopen issue if you see further discrepancies

When using release 2.2.5 in a basic asp.net core react app, I get the same error as stated before.

I have the same error while using React template for ASP.NET Core app in Visual Studio 2019. For some weird reason it's caused by the way react-trello is imported.
This works:
import Board from 'react-trello'
This doesn't:
import { Board } from 'react-trello'

I have the same error while using React template for ASP.NET Core app in Visual Studio 2019. For some weird reason it's caused by the way react-trello is imported.
This works:
import Board from 'react-trello'
This doesn't:
import { Board } from 'react-trello'

Thanks, This worked for me. I don't know what is the underlying reason.

I have the same error while using React template for ASP.NET Core app in Visual Studio 2019. For some weird reason it's caused by the way react-trello is imported.
This works:
import Board from 'react-trello'
This doesn't:
import { Board } from 'react-trello'

Thanks, This worked for me. I don't know what is the underlying reason.

After some more searching, I think I was getting error as I was used named export instead of default export.
To test named export I changed to
import { default as Board } from 'react-trello'


EaterEvals link is broken

User reported:

Clicking on a professor's name, then clicking on the button for their EaterEvals no longer directs us to the instructor's evals. It seems like they reorganized the structure of EaterEvals and I don't think it's possible to look up instructors by name anymore. You should probably remove that button.
I mainly just wanted to report it; there's no need to get back to me with updates because I don't think it'll be possible to restore that feature. When I look at a specific course feedback for a professor, the URL gives away nothing that would personally identify that professor. For example, ICS 33 has three evaluations listed for Professor Pattis, but the only differences in the URLs is a four-digit number (in the case below, it's 3510)
https://evaluations.eee.uci.edu/eaterevals/view/3510?return=%2Featerevals%2Fbrowse%2Fcourse%2FI%252526C%252520SCI%2F33%2FINTERMEDIATE%252520PRGRMG
For his other two evals, 3510 changes to 7641 and 7642. Everything else in the URL stays the same.
Besides ICS 33, I also did that check for CS 161, and it's the same situation.

EaterEvals changed up their url structure, so our previous link that used last names no longer works: https://evaluations.eee.uci.edu/eaterevals#WONG-MA
Now, it is by their by their UCI net id:
https://evaluations.eee.uci.edu/eaterevals/browse/instructor/jwongma
The PeterPortal API has internal logic that maps a webreg name to a uci net id: https://github.com/icssc-projects/peterportal-public-api/blob/master/graphql/schema.js#L181-L223
Maybe we can expose this or copy the logic?

The PeterPortal API has internal logic that maps a webreg name to a uci net id: https://github.com/icssc-projects/peterportal-public-api/blob/master/graphql/schema.js#L181-L223
Maybe we can expose this or copy the logic?
I think it would be redundant to copy the logic so maybe we can have an endpoint to get this?

I don't think we use eater evals on our site anymore, just RMP. Is this issue safe to close? @ChaseC99

Yeah since we don't have a map of websoc professor name to UCINetId, it'd probably be too hard to do EaterEvals.
Feel free to close it!

Bummer cuz this would be a cool feature :(

I'll put it in the icebox in case any eater evals enthusiasts want to work on this.


Errors after delete duplicate folder from other folder with & and space before and after in its name

Godot version:
3.1 0dbe014
OS/device including version:
Windows 10, Linux Mint 19
Issue description:
On linux it works fine but on Windows after duplicate folder "AAA & DDDD", I tried to delete newly created folder, and then this error was occurred:

ERROR: Failed to get modified time for: C:/PR/GAGA/AAADDD2
At: drivers/windows/file_access_windows.cpp:306
ERROR: Failed to get modified time for: C:/PR/GAGA/AAADDD2/default_env.tres
At: drivers/windows/file_access_windows.cpp:306
ERROR: Failed to get modified time for: C:/PR/GAGA/AAADDD2/icon.png
At: drivers/windows/file_access_windows.cpp:306
ERROR: load_image: Error opening file: res://AAADDD2/AA2/icon.png
At: core/io/image_loader.cpp:56
ERROR: _reimport_file: Error importing: res://AAADDD2/AA2/icon.png
At: editor/editor_file_system.cpp:1545
ERROR: _reimport_file: Condition ' !f ' is true.
At: editor/editor_file_system.cpp:1551
ERROR: load_image: Error opening file: res://AAADDD2/AAA & DDDD/icon.png
At: core/io/image_loader.cpp:56
ERROR: _reimport_file: Error importing: res://AAADDD2/AAA & DDDD/icon.png
At: editor/editor_file_system.cpp:1545
ERROR: _reimport_file: Condition ' !f ' is true.
At: editor/editor_file_system.cpp:1551
ERROR: load_image: Error opening file: res://AAADDD2/icon.png
At: core/io/image_loader.cpp:56
ERROR: _reimport_file: Error importing: res://AAADDD2/icon.png
At: editor/editor_file_system.cpp:1545
ERROR: _reimport_file: Condition ' !f ' is true.
At: editor/editor_file_system.cpp:1551

Steps to reproduce:
https://streamable.com/q2usf

I think this is related with #23080.

Is this issue still present in latest master?

No.


yank and comment not work

in _G.__gdc(vmode)
U.parse_cstr(cfg, ctx).

string expected, got table

It's not an official API

please help. how can i fix it

or which i can use


Fix Windows paths being too long in distribution.

Thanks @carmichaelong

This problem seems to be fixed if using 7zip for extracting. :)

Ah, thanks for the tip, Vishal!


v0.0.3 still using old build

Hi @ksivamuthu,
It seems like the latest release is not actually using the latest code. If you download the zip file and look at the /lib/main.js file, you can see that the code is for an older version.
https://github.com/ksivamuthu/aws-copilot-github-action/archive/refs/tags/v0.0.3.zip
Would it be possible to release a new build with the latest compiled code?
I am not sure what the procedure is to update the build 🤔 ..
Thank you!

I think it is necessary to run npm run build && npm run package and then release a new tag.

@yanshiyason  Repackaged. I'm adding it to github actions to avoid this in future releases. Thanks.

Thank you very much @ksivamuthu ! Much appreciated :)


[BUG?] lwm2m_handle_packet trying to free a NULL payload pointer

I'm not sure if this can be considered a bug, also because this hasn't caused any problems so far (but could it be a problem on other devices?)
I noticed that lwm2m_handle_packet frees the payload on line 644 without taking into consideration that the payload may not exist, for example in case the client has to send a message without payload (which often happens during the bootstrap phase and connection to the LwM2M server).
It's an easy fix so I send a pull request:
https://github.com/eclipse/wakaama/pull/706

Not a bug because of this: https://github.com/eclipse/wakaama/pull/706#issuecomment-1570733465


Removing whitespace-only lines in glossary

This issue was causing podchecker to emit the following warning in S99-glossary.pod:
WARNING: line containing nothing but whitespace in paragraph

which has been corrected by this commit.

Replaced by commit 44a21c0; closing PR.


Upgrade to GitHub-native Dependabot

Dependabot Preview will be shut down on August 3rd, 2021. In order to keep getting Dependabot updates, please merge this PR and migrate to GitHub-native Dependabot before then.
Dependabot has been fully integrated into GitHub, so you no longer have to install and manage a separate app. This pull request updates your config file to the new syntax. When merged, we'll swap out dependabot-preview (me) for a new dependabot app, and you'll be all set!
With this change, you'll now use the Dependabot page in GitHub, rather than the Dependabot dashboard, to monitor your version updates, and you'll configure Dependabot through the new config file rather than a UI.
Your previous schedule was set to live. This option is no longer supported in the new config file so it has been changed to daily.
You have configured automerging on this repository. There is no automerging support in GitHub-native Dependabot, so these settings will not be added to the new config file. Several 3rd-party GitHub Actions and bots can replicate the automerge feature.
If you've got any questions or feedback for us, please let us know by creating an issue in the dependabot/dependabot-core repository.
Learn more about migrating to GitHub-native Dependabot
Please note that regular @dependabot commands do not work on this pull request.

Now complete for UKHO\gsfpy.


Remove (debug?) Printlns from Helm operator

There's a number of fmt.Printf in there that really should use logging instead.

These are purely debug and will go when the alpha stage is over.


[neteasemusic] Add new extractor for music.163.com

http://music.163.com is the music streaming service from [NetEase/网易] (https://en.wikipedia.org/wiki/NetEase).
The site does not geo-restrict AFAIK and has hq downloads available. No login is required.

Lots of contents on 163.com violates copyright (see the Google transparency report ). I'm not sure whether 163.com is the same as music.163.com or not. Can you provide evidences that music.163.com has legal rights over the contents it provides?

According to this zhihu question (in Chinese), not all songs are legal. Note our policy is that youtube-dl does not support copyright infringing sites, see this FAQ.

I'm not entirely familiar with the Chinese music streaming market but I've seen reports of QQMusic and NetEase Cloud Music (music.163.com) suing each other for rights infringement. Both parties have been found guilty in different cases and compelled legally to take down content. http://toutiao.com/a3739968936/
The linked article also refers to Netease acquiring the rights to distribute Taylor Swift's music (with fee).

Made the review changes except for the int_or_none suggestion because of rounding requirements.

Thanks. Merged.


Video Stream Tweaks

The purpose of this PR is to close the following issues:

[x] https://github.com/aqualinkorg/aqualink-app/issues/575.
[x] https://github.com/aqualinkorg/aqualink-app/issues/580
[x] https://github.com/aqualinkorg/aqualink-app/issues/581
[ ] https://github.com/aqualinkorg/aqualink-app/issues/583

Designs:
With featured image

desktop
mobile

Without featured image

desktop
mobile

If the video is not live, we want to act as if video_strean was null @tleraj

So check if there is a survey image, and if that doesn't exist, fall back to the add survey image, right?

Regarding the YouTube branding reduction, there is no iframe option to remove it at all. Also, since an iframe is necessary to load YouTube videos, there is no way to remove the logo with CSS. So the only logo that appears, is the one on the bottom right when we hover on the video.
Regarding the autoplay on mobile, according to the first answer here and here, this option is not available. Also I think we don't want to allow autoplay on mobile, considering the mobile data consumption.


Add database to healthcheck method

Similar to #605 , we must add Database to the healthcheck method.
EndPoint msust be  the full jdbc connection url.
This depends on #605

I will take this.


Validating IMF packages with subtitle tracks

I am having trouble getting a successful validation result when trying to analyze any IMF package that contains a subtitle track. The errors/warnings I get are related to the IMPAnalyzer saying that a CPL resource is not present in the package and that an essence descriptor does not have a match in any track file.
Based on a test I did with the IMFTrackFileReader, Subtitle essence does not come up as a supported type. It says the only supported types are MainImageEssence, MarkerEssence, MainAudioEssence. Are subtitle tracks really not currently supported? If so, when will support be added?

Photon does not support subtitle track currently but we do have a plan to add this support in near-term. We will provide you more accurate timeline once we have the data available.

We have added basic support for subtitle sequence in version 4.2.3. This should address essence descriptor related error with subtitle sequence.


Development environment setup

While trying to setup the development environment after cache:clear operation I get the following error:
PHP Parse error:  syntax error, unexpected '[' in /home/janek/vagrant/wellcommerce/www/vendor/illuminate/support/Illuminate/Support/helpers.php on line 411
PHP Stack trace:
PHP   1. {main}() /home/janek/vagrant/wellcommerce/www/app/console:0
PHP   2. require_once() /home/janek/vagrant/wellcommerce/www/app/console:10
PHP   3. require_once() /home/janek/vagrant/wellcommerce/www/app/bootstrap.php.cache:3
PHP   4. require() /home/janek/vagrant/wellcommerce/www/app/autoload.php:9
PHP   5. ComposerAutoloaderInitb3c4a5b1a0c6bc3c95f546a5a82fd561::getLoader() /home/janek/vagrant/wellcommerce/www/vendor/autoload.php:7
PHP   6. composerRequireb3c4a5b1a0c6bc3c95f546a5a82fd561() /home/janek/vagrant/wellcommerce/www/vendor/composer/autoload_real.php:49

The line in question looks like this:
$results = [];
Is it possible that this is caused by PHP version (I've got PHP 5.5.15)

Not valid anymore because from bumped PHP and Symfony version and additional install commands. There is a strict "php": "^5.6|^7.0" requirement in composer.json


Adds Wiscard Widget feature announcement

Coverage remained the same at 15.91% when pulling e39e2958abdf9d99e80c158ab89a2e7368de9043 on kimmiller:patch-3 into 7c2a445e5570279c49d7061fbee50bd0a048cb10 on UW-Madison-DoIT:master.

Hey Kim,  Long story, but quotes in the description is hard. "Wiscard" is the way to do it rather than "Wiscard"

Did you update it or would you like me to?
Kim
Kim Miller

Sr. Marketing Specialist
Division of Information Technology
University of Wisconsin⎯Madison
Office:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
www.twitter.com/uwdoit
www.facebook.com/UWDoIT
http://it.wisc.edu/

From:  Timothy A Vertein<EMAIL_ADDRESS>Reply-To:  UW-Madison-DoIT/angularjs-portal<EMAIL_ADDRESS>Date:  Wednesday, April 27, 2016 at 1:04 PM
To:  UW-Madison-DoIT/angularjs-portal<EMAIL_ADDRESS>Cc:  Kimberly Miller<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject:  Re: [UW-Madison-DoIT/angularjs-portal] Adds Wiscard Widget feature announcement (#457)

Hey Kim, Long story, but quotes in the description is hard. \"Wiscard\" is the way to do it rather than "Wiscard"

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Go ahead and update it.  It will be good experience. :)

Okay. I updated and hit commit change. Was I supposed to add a comment above the commit change?
Thanks!
Kim
Kim Miller

Sr. Marketing Specialist
Division of Information Technology
University of Wisconsin⎯Madison
Office:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
www.twitter.com/uwdoit
www.facebook.com/UWDoIT
http://it.wisc.edu/

From:  Timothy A Vertein<EMAIL_ADDRESS>Reply-To:  UW-Madison-DoIT/angularjs-portal<EMAIL_ADDRESS>Date:  Wednesday, April 27, 2016 at 1:08 PM
To:  UW-Madison-DoIT/angularjs-portal<EMAIL_ADDRESS>Cc:  Kimberly Miller<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject:  Re: [UW-Madison-DoIT/angularjs-portal] Adds Wiscard Widget feature announcement (#457)

Go ahead and update it. It will be good experience. :)

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Coverage remained the same at 15.91% when pulling 23db56b984d6ac8bca40a268a7a290e53fd47369 on kimmiller:patch-3 into 7c2a445e5570279c49d7061fbee50bd0a048cb10 on UW-Madison-DoIT:master.

Comment would have been fine, don't worry about it this time.  Good job editing it!
:+1:

:+1:


Chimoney Readme Translator

This translates the English chimoney to another

@phyleria @uchibeke

@phyleria 🥲

@uchibeke please also check this out


error: tmp/tmp.xxxxxxxxx/t.tar.gz no such file or directory\n

Occasionally packit srpm build fails with error: (and similar)
Preparing of the upstream to the SRPM build failed: ['oc', 'cp', 'packit-prod-sandbox/docker-io-usercont-sandcastle-20200123-150922922427:/tmp/tmp.YiprCGQmoq/t.tar.gz', '/tmp/tmp06yctfme'] failed with 'error: tmp/tmp.YiprCGQmoq/t.tar.gz no such file or directory\n'

I have difficulty to find a clear pattern in this behavior:

https://github.com/dhodovsk/libica/pull/1 - only in prod, not on the second try, ...
https://github.com/dhodovsk/Pillow/pull/1 - in prod, ...
https://github.com/dhodovsk/oci-umount/pull/1 - on stage, ...
https://github.com/dhodovsk/langtable/pull/1 - in prod, ...
https://github.com/dhodovsk/dbxtool-devel/pull/1 - on stage, ...

I think this is not caused by 'no route to host', the request is repeated more times and just then it raises an exception.
While executing oc cp command, this happens and causes the exception (https://sentry.io/organizations/red-hat-0p/issues/1380802387/?project=1767823):
 tar: Removing leading '/' from member names
tar: /tmp/tmp.43NyJxVymH/t.tar.gz: Cannot stat: No such file or directory
I guess it could be caused by wrong permissions, I  am wondering why we don't use --preserve-permissions in tar command here but everywhere else in sandcastle we do (@TomasTomecek ?)

This is way more complicated than I originally thought.
I just checked all three codebases (sandc, packit, p-s) and the code we have right now seems to be correct to me.
So let's look at the logs from sentry:
Sandbox pod is deployed.

httplib    GET https://<IP_ADDRESS>/api/v1/namespaces/packit-prod-sandbox/pods/docker-io-usercont-sandcastle-20191210-113752823272/log [200]

The pod is deployed, we can run the command.
running command: ['python3', 'setup.py', 'sdist', '--dist-dir', '.']

GET https://<IP_ADDRESS>/api/v1/namespaces/packit-prod-sandbox/pods/docker-io-usercont-sandcastle-20191210-113752823272 [200]

command = ['python3', 'setup.py', 'sdist', '--dist-dir', '.']

we're about to run the command
tar --preserve-permissions -czf /tmp/tmp7sdyqyio/t.tar.gz script.sh

PosixPath('/tmp/tmpvkzwemlz')

oc cp /tmp/tmp7sdyqyio/t.tar.gz packit-prod-sandbox/docker-io-usercont-sandcastle-20191210-113752823272:/tmp/tmp.eKjlETMOZV

tar --preserve-permissions -czf /tmp/tmpf71k306m/t.tar.gz ci.fmf .gitignore .zuul.yaml setup.cfg release-conf.yaml .git_archival.txt .packit.yaml files .eggs ansible.cfg .github LICENSE .gitattributes fedora-tests .git tests_recording packit setup.py Dockerfile packit.spec .fmf CHANGELOG.md tests Makefile recipe-tests.yaml README.md DCO Dockerfile.tests pytest.ini Vagrantfile CONTRIBUTING.md .pre-commit-config.yaml docs

oc cp /tmp/tmpf71k306m/t.tar.gz packit-prod-sandbox/docker-io-usercont-sandcastle-20191210-113752823272:/tmp/tmp.RNXPvtaoDe

we're doing all that nonsense so that we can run a command in a given path, meaning all that junk is just for sake of cd <path> && exec $command.
exec command failed

the command failed! something's wrong
exception while initiating WS Client: (0)
Reason: [Errno 113] No route to host

and we also couldn't init the kube api client - are those two related?
but we only have a single log message like that, which means it got initiated the second time
and now, sandcastle is trying to copy the content out of the sandbox ( = pod), which is actually weird - why?
copy packit-prod-sandbox/docker-io-usercont-sandcastle-20191210-113752823272:/tmp/tmp.43NyJxVymH/t.tar.gz -> /tmp/tmpn7i18t0v

oc cp packit-prod-sandbox/docker-io-usercont-sandcastle-20191210-113752823272:/tmp/tmp.43NyJxVymH/t.tar.gz /tmp/tmpn7i18t0v

tar: Removing leading `/' from member names
tar: /tmp/tmp.43NyJxVymH/t.tar.gz: Cannot stat: No such file or directory
tar: Exiting with failure status due to previous errors

error: tmp/tmp.43NyJxVymH/t.tar.gz no such file or directory

and suddenly the repository content is no longer in the sandbox - what, whyyyyyyyyy?! is sandcastle on drugs?

Okay, they are not related, I found another sentry event:
exec command succeeded, yay!

exception while initiating WS Client: (0)
Reason: [Errno 113] No route to host

so, something bad happens which:

either removes the repository in sandbox
or an error happens while creating the archive in the sandbox

since we don't log those actions, I proposed a sandcastle PR to log more; I was also unable to reproduce this in sandcastle, see https://github.com/packit-service/sandcastle/pull/56

I spent almost two hours on this yesterday and I still don't understand what's wrong here - it makes zero sense to me - basically the directory with the repository content vanishes from the sandbox - how?!

I also tried to understand this from the logs in Sentry yesterday and I am quite confused, mainly because I can't see the conditions when this happens, as Dominika wrote:

only in prod, not on the second try

With forced packit overload this happened again -https://sentry.io/share/issue/a78cac1cee64410ea03e15b480fdd923/
Snippet from the logs:
[2020-02-26 09:02:23,715: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:02:23,717: DEBUG/ForkPoolWorker-1] ['tar', '--preserve-permissions', '-czf', PosixPath('/tmp/tmp5mfmkc3p/t.tar.gz'), 'docs', 'README', '.pylintrc', '.gitignore', 'optional_plugins', 'contrib', '.copr', 'man', 'Makefile.include', '.travis.yml', 'setup.py', '.gitattributes', 'examples', '.git', 'Makefile', 'spell.ignore', 'MANIFEST.in', 'selftests', 'scripts', 'CONTRIBUTING.rst', 'codecov.yml', 'README.rst', 'requirements-selftests.txt', 'LICENSE', '.cirrus.yml', 'python-avocado.spec', 'VERSION', 'avocado', '.packit.yml']
[2020-02-26 09:02:54,765: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:02:54,765: DEBUG/ForkPoolWorker-1] ['oc', 'cp', PosixPath('/tmp/tmp5mfmkc3p/t.tar.gz'), 'packit-prod-sandbox/docker-io-usercont-sandcastle-20200226-090122537960:/tmp/tmp.MglGXbgNHb']
[2020-02-26 09:03:25,798: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:03:26,084: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:03:26,179: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:03:56,207: DEBUG/ForkPoolWorker-1] {"metadata":{},"status":"Success"}
[2020-02-26 09:03:56,208: INFO/ForkPoolWorker-1] exec command succeeded, yay!
[2020-02-26 09:03:56,208: DEBUG/ForkPoolWorker-1] mapped_dir is set, let's sync the dir back and fix modes
[2020-02-26 09:03:56,443: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:04:26,548: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:04:26,795: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:04:26,795: DEBUG/ForkPoolWorker-1] the archive in the pod: '-rw-r--r--. 1<PHONE_NUMBER> root 16M Feb 26 09:03 /tmp/tmp.jc5f8P8Ger/t.tar.gz\n'
[2020-02-26 09:04:26,795: DEBUG/ForkPoolWorker-1] ['oc', 'cp', 'packit-prod-sandbox/docker-io-usercont-sandcastle-20200226-090122537960:/tmp/tmp.jc5f8P8Ger/t.tar.gz', '/tmp/tmp_0d8kn9f']
[2020-02-26 09:04:27,012: ERROR/ForkPoolWorker-1] error: tmp/tmp.jc5f8P8Ger/t.tar.gz no such file or directory

[2020-02-26 09:04:27,252: DEBUG/ForkPoolWorker-1] we have successfully initiated the kube api client
[2020-02-26 09:04:27,288: WARNING/ForkPoolWorker-1] /usr/lib/python3.7/site-packages/ogr/read_only.py:70: DeprecationWarning: Call to deprecated function (or staticmethod) pr_comment. (will be removed in 0.14.0 (or 1.0.0 if it comes sooner): Use methods on PullRequest objects) -- Deprecated since version 0.9.0.
  return func(self, *args, **kwargs)
09:04:28.634 handler.py        DEBUG  Reporting state of copr build ID=None state=error, commit=dc2670d1ae82c5b350fd3ed9f9ec763da0eef1d8
[2020-02-26 09:04:28,634: DEBUG/ForkPoolWorker-1] Reporting state of copr build ID=None state=error, commit=dc2670d1ae82c5b350fd3ed9f9ec763da0eef1d8

https://github.com/packit-service/sandcastle/pull/59
[2020-02-26 09:04:26,795: DEBUG/ForkPoolWorker-1] the archive in the pod: '-rw-r--r--. 1<PHONE_NUMBER> root 16M Feb 26 09:03 /tmp/tmp.jc5f8P8Ger/t.tar.gz\n'
[2020-02-26 09:04:26,795: DEBUG/ForkPoolWorker-1] ['oc', 'cp', 'packit-prod-sandbox/docker-io-usercont-sandcastle-20200226-090122537960:/tmp/tmp.jc5f8P8Ger/t.tar.gz', '/tmp/tmp_0d8kn9f']
[2020-02-26 09:04:27,012: ERROR/ForkPoolWorker-1] error: tmp/tmp.jc5f8P8Ger/t.tar.gz no such file or directory

the archive is inside but openshift fails copying it for some reason
https://github.com/packit-service/sandcastle/pull/59 - this could give us even more data
so the main symptom is when all the workers are busy?

Looks like this has solved itself, let's close then.


Some PDF render blank

This happens with multiple files. I'm testing with PDF files that contain embedded raster images. Some of them seem to work fine though.

Wait, what does this crate actually do? Should I move this bug to pathfinder instead?

If a PDF does not render, here is the correct place to report it.
Currently the code for drawing is here: https://github.com/pdf-rs/pdf_render/blob/master/render/src/renderstate.rs#L660
Can you share the problem files?

https://ks.imslp.net/files/imglnks/usimg/1/19/IMSLP51726-PMLP01458-Beethoven_Werke_Breitkopf_Serie_16_No_137_Op_27_No_2.pdf

That file uses the CCITTFaxDecode filter which is not implemented yet. Lets see how difficult it is...

Might work as of https://github.com/pdf-rs/pdf/commit/155ca085ac0ffdb9986095334cff1b560ba73b3f

At least it fails now ;)
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Other { msg: "Unimplemented @ pdf-3ef1c528a9b91eec/37e9966/pdf/src/enc.rs:336" }', pdf_render/view/src/lib.rs:47:92

Please run cargo update and try again.

The file above works here, and I don't think I forgot to push anything to github…
If your pdf_render repo is up to date and the pdf dependency is updated, it should work.

Please run cargo update and try again.

It works now. Maybe consider removing Cargo.lock from the gitignore?
I get a panic on the third page now though: EOF', pdf_render/view/src/lib.rs:47:92

I can reproduce that error. Needs digging.

cargo update -p fax
There appears to be some bug (related to pathfinder?) that causes it to hang sometimes.

The fix is included in pdf 0.7.2.
Closing now. Feel free to reopen if you run into other cases.


GameThread: increase timeout

1 second might be too short for certain task to complete

Just curious as i havent had a problem with it, which tasks may take more than 1000ms?

Someone got this error building teleports no clue if this fixes it or if it is a plugin issue, increasing it a little does not cause harm `java.lang.RuntimeException: Client thread invoke timed out after 1000 ms
at net.unethicalite.api.game.GameThread.invokeLater(GameThread.java:52)

at net.unethicalite.api.movement.pathfinder.TeleportLoader.buildTimedTeleports(TeleportLoader.java:59)

at net.unethicalite.api.movement.pathfinder.TeleportLoader.buildTeleports(TeleportLoader.java:53)

at net.unethicalite.api.movement.pathfinder.Walker.buildTeleportLinks(Walker.java:579)

`


Seconds vs. milliseconds in the example

In 5.2.4:
{
  "iat": "1686536226000",
  "aud": "trust-domain.example",
  "exp": "1686536526000",
  ...
}

The iat and exp claims are strings expressed in milliseconds. According to RFC 7519 4.1, those should be numerics and expressed in seconds.

Thank you! Will update the examples to be correct.


VSCode overloads CPU on delaying computer clock

When I delay my clock with VSCode open, it starts to use overly CPU and make my computer so slow that I need to kill VSCode process.

VSCode Version: 1.23.1
OS Version: Windows 10 version 1709 OS Build 16299.431

Steps to Reproduce:

Open VSCode
Delay your clock some minutes or hours

Does this issue occur when all extensions are disabled?: Yes

I don't think that "turning back the computer clock" is a relevant scenario to support but anyway...
Please try to find out what part of VS Code puts a strain on the CPU by following these guidelines https://github.com/Microsoft/vscode/wiki/Performance-Issues and provide additional information.

I know supporting that is not that relevant, but this problem may indicate some coupling in VS Code which is not pretty good, that's why I reported.
The following images are screenshots of process explorer and code --status, with and without extensions, respectively. In all these scenarios the problem happened.

I also see very high numbers in out process manager, but not in the OS's process view.

May be the CPU load calculation we use in the VS Code Process Manager is getting confused by the time moving backwards...

Hmm, the IT settings on my work computer aren't letting me manually set the time. I tried using a Linux VM and it keeps resyncing the time after a minute or so, so I haven't had any luck reproducing. I'll give it a try with my laptop at home later.
From the screenshots, it looks like the CPU usage between the process explorer and Task Manager are very similar. @aeschli What OS did you see the difference on?
The window CPU usage looks high to me, @yagotome could you use the Developer: Toggle Developer Tools command and then use the Performance tab to collect a profile of it when the slow down is happening?

@RMacfarlane I'm on linux that runs as a VM. Maybe I'm wrong given @yagotome screenshot.
I often see the process explorer showing load long after that load happened and only slowly ticking backward. If you prefer I can open a new issue just for that.

@aeschli Let's open a new issue for that, looks like CPU is reported incorrectly on Linux. The process explorer and --status are both using ps to get CPU usage. Linux documentation for ps says "CPU usage is currently expressed as the percentage of time spent running during the entire lifetime of a process.  This is not ideal, and it does not conform to the standards that ps otherwise conforms to."

I created https://github.com/Microsoft/vscode/issues/51200 for that issue.

@RMacfarlane I have collected a profile from Performance tab while that problem was happening.
Profile-20180612T102442.zip


Doesn't update favicon in Google Chrome

Strangely, Live Server caches an old version of the favicon my website is using, even though it's not declared in the HTML at all. I have to explicitly declare a new icon in order to get it changed. It gets even more frustrating when I comment out that declaration and the favicon is still there, but only in Google Chrome. As expected, in Firefox it doesn't show.
I'm using Windows 10 x64, VS Code 1.20.0, Live Server 3.2.0.
Feel free to ask for more information, would be glad to assist.

Okk!! it's not about live server! This is about the browser cache!
just try the trick (it may works):
<link rel="icon" href="favicon.ico?v=2" type="image/x-icon"/>
?v=ANYTHING will remove the cache.

Seems to be a known Chrome bug.
https://superuser.com/questions/375321/how-to-refresh-bookmark-favicons-in-chrome


mitmproxy crashes while handling the Discord gateway

Problem Description
Trying to connect to Discord while mitmproxy is active, results in the following error:
[01:05:00.739][<IP_ADDRESS>:53692] mitmproxy has crashed!
Traceback (most recent call last):
  File "mitmproxy/proxy/server.py", line 380, in server_event
  File "mitmproxy/proxy/layer.py", line 156, in handle_event
  File "mitmproxy/proxy/layer.py", line 269, in handle_event
  File "mitmproxy/proxy/layer.py", line 156, in handle_event
  File "mitmproxy/proxy/layers/http/__init__.py", line 880, in _handle_event
  File "mitmproxy/proxy/layers/http/__init__.py", line 936, in event_to_child
  File "mitmproxy/proxy/layer.py", line 156, in handle_event
  File "mitmproxy/proxy/layers/http/__init__.py", line 776, in passthrough
  File "mitmproxy/proxy/layer.py", line 156, in handle_event
  File "mitmproxy/proxy/tunnel.py", line 98, in _handle_event
  File "mitmproxy/proxy/layers/tls.py", line 485, in event_to_child
  File "mitmproxy/proxy/tunnel.py", line 153, in event_to_child
  File "mitmproxy/proxy/layer.py", line 156, in handle_event
  File "mitmproxy/proxy/tunnel.py", line 98, in _handle_event
  File "mitmproxy/proxy/tunnel.py", line 153, in event_to_child
  File "mitmproxy/proxy/layer.py", line 156, in handle_event
  File "mitmproxy/proxy/layers/http/__init__.py", line 880, in _handle_event
  File "mitmproxy/proxy/layers/http/__init__.py", line 936, in event_to_child
  File "mitmproxy/proxy/layer.py", line 147, in handle_event
  File "mitmproxy/proxy/utils.py", line 26, in _check_event_type
AssertionError: Unexpected event type at HttpStream._handle_event: Expected Start|HttpEvent, got Reply(WebsocketStartHook(flow=<HTTPFlow
  request = Request(GET gateway.discord.gg:443/?encoding=json&v=9&compress=zlib-stream)
  response = Response(101, no content)
  websocket = <WebSocketData (0 messages)>
  client_conn = Client(<IP_ADDRESS>:53692, state=open, alpn=http/1.1)
  server_conn = Server(gateway.discord.gg:443, state=open, alpn=http/1.1, src_port=53693)>), None).

Steps to reproduce the behavior:

Run mitmproxy or mitmweb
Go to discord.com and log in if you haven't already
Monitor the event log

System Information
Mitmproxy: 10.2.1 binary
Python:    3.12.1
OpenSSL:   OpenSSL 3.1.4 24 Oct 2023
Platform:  macOS-14.3-x86_64-i386-64bit

Nevermind, this is a duplicate of #6608, I'll update.


[NEW]: Crypto hedge fund Three Arrows files for Chapter 15 bankruptcy

Date
2022-07-02
Description
Crypto hedge fund Three Arrows files for Chapter 15 bankruptcy
Links
https://www.reuters.com/markets/us/crypto-hedge-fund-three-arrows-files-chapter-15-bankruptcy-2022-07-01/
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
Patrick Wood
Personal link
No response

Also: MAS Reprimands Three Arrows Capital for Providing False Information and Exceeding Assets Under Management Threshold
https://www.mas.gov.sg/regulation/enforcement/enforcement-actions/2022/mas-reprimands-three-arrows-capital-for-providing-false-information-and-exceeding-assets-under-management-threshold


override _connect method for oracle only

linked to #219

it can be tested with (replace <path_to_instantclient_19_8> to match yours)
    def test(self):
        cx_Oracle.init_oracle_client(lib_dir=f"<path_to_instantclient_19_8>")
        with OracleDbContainer() as oracle:
            url = oracle.get_connection_url()
            e = sqlalchemy.create_engine(url)
            result = e.execute("select 1 from dual")
            for row in result:
                self.assertEqual(row[0],1)

in fact with the last verion, everything works. No need to merge.


calculating isHome() and isEnd() using children of a cell

It allows us to determine more easily (or automatically in some cases) that the compound cell is empty.
Added test cases for both commits (failed before, passes after).
Ran "mvn clean install" for all four repositories (mapper, projectional, ot-model, datapad).

Good job!


COMP: Removed AdvancedTransform::InverseJacobianPositionType

Removed both InverseJacobianPositionType and ComputeJacobianWithRespectToPosition(p, j) from AdvancedTransform.
Neither the type nor the member function appear to be used. The type definition causes compilation errors with ITK4:
itkAdvancedTransform.h(116): error C2039: 'InverseJacobianPositionType': is not a member of 'itk::Transform'

For the record, itk::Transform::ComputeJacobianWithRespectToPosition used to be a pure (= 0) virtual member function, but it is now "non-pure", as "= 0" was removed, by Jeffrey Duda, 2011-11-11, "ENH: Moved data transforms to transform base class":
https://github.com/InsightSoftwareConsortium/ITK/commit/04510ec498aeb3d27b9b5a4c24fc6d992bdcda85#diff-20692f27f7b9e7e081e4dc98fe057a01L199

Discussed at the sprint, with @mstaring, @stefanklein, @kaspermarstal


Adding Collaboration Section to README

Adding instructions on how to collaborate to the website to the README.

Done


ADD PMM-T1328 - Verify public address is set automatically

@VoK0 execution results?

This needs to merged first otherwise the job will always fail because of the missing env variables and therefore API call to portal is not possible
Here is POC on my tmp job which is just clone of pmm-ui-nightly with fixed variables https://pmm.cd.percona.com/job/VokoTmpNightlyUITests/11/allure/#suites/d9805975708c0a76781ecd3ab76468a5/3a6367079ef419c8/
tmp Job link https://pmm.cd.percona.com/blue/organizations/jenkins/VokoTmpNightlyUITests/detail/VokoTmpNightlyUITests/11/pipeline
Failed tests are unrelated to my changes


Type acquisition support for bower_components directory

Fixes #13917 [Type Acquisition] auto d.ts acquisition no longer works for bower_components
Issue: retrieving typing names from the bower_components directory is not currently supported. We only retrieve typings for bower components when a bower.json is present. By default in VS, ASP.NET projects do not create a bower.json when using the Manage Bower Packages option.  No typings are currently being acquired for these components.
Fix:

Apply getTypingNamesFromPackagesFolder that is already being used to retrieve typings for the  node_modules  folder to the bower_components folder.
Include the packagesFolder(s) in the list of filesToWatch.

//cc: @vladima , @mhegazy , @zhengbli , @billti

👍


Minify production CSS via cssnano

Use Tailwind's recommended method of minifying the CSS via the cssnano PostCSS plugin. This drops the final CSS files by about 20kb when NODE_ENV is production.
You can test this yourself by pulling this branch, installing deps, and running
yarn build:redesign
yarn build:css

Take note of the final file sizes generated to app/assets/builds and then run the commands again with NODE_ENV=production prefixed to see the savings 🔥
NODE_ENV=production yarn build:redesign
NODE_ENV=production yarn build:css

Here is what I am getting on my machine after multiple runs:

File
pre-cssnano
post-cssnano

application.tailwind.css
88kb
67kb

redesign.tailwind.css
83kb
63kb

Warning
I have never (yet) seen this mess up any of the final styles and I did a spot-check around the app and didn't see anything but with any minifying - it comes with risks. I am using the default settings and the docs promise these are safe transforms only

Since you're working on a redesign, I figured this would be the perfect time to introduce this optimization from which you get a pretty significant saving (see above).
Didn't think this needed a changelog entry but lmk and I can add. Feel free to throw this away but I noticed you didn't have it and wanted to get in a contribution 😄
Pull request checklist

[ ] My code contains tests covering the code I modified
[x] I linted and tested the project with bin/check
[ ] I added significant changes and product updates to the changelog

20kb?! That's like ¼ reduction in size! Thanks so much for this Andrew.
Once CI passes I'm going to do a quick review locally and merge it in if all looks good.


Multiple character ranges

Description
Add the ability to specify multiple character ranges.
Suggested Solution
The current parameter takes a list of numbers. new system can be compatible with the current system.

If parameter is a [number, number] use the single character range format.
If the parameter is a [number, number][] use the multiple character range format.

Then use the format specified in the _getRandomChar() method.
Benefits
It will be possible to select wider ranges of characters without unwanted ones, For example when using Hiragana/Katakana, there are unused char codes that can be omitted this way

Resolved with 28743bd


Implement Kerberos with Okhttp

Implement Kerberos based authentication with Okhttp client. After moving on to Okhttp client from Jersey client, implement Kerberos based authentication through the new http client as well

Existing test TestDatabaseClientWithKerberos was used to verify with OKHttp. Security was enabled with changes to test.properties on the Java Client API side and a SSL enabled REST server was used. Tests ran fine.


As year of 2020, suggest to upgrade kingly dependency from node version '6.3.0' only to compatible higher version.

Thanks for this awesome lib.
I was in trouble while I trying kingly.
❯❯❯ yarn add react-state-driven
...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "6.3.0". Got "12.8.0"
error Found incompatible module.

Can I suggest to upgrade node dependency version to some like "node": ">= 6.3.0" or else?
Thanks again.

Thanks for the notification, I will have a look. Kingly is now v0.28. I have to update react-state-driven with the new changes. I will have a look at the node version too.

@leopku You can always experiment with the codesandbox playground. This one working example that you can fork: https://codesandbox.io/s/ym8vpqm7m9?file=/src/index.js
Also, in codesandbox, you have a File -> Export to zip option that packages the sandbox configuration so you can run it locally. You may be more lucky this way. I am continunig to investigate

@brucou Thanks for reply so quickly.
 Expected version "6.3.0". Got "12.8.0"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

manually add to package.json "react-state-driven": "0.8.0", then run yarn install
yarn install                                                                             master ✱ ◼
yarn install v1.17.3
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "6.3.0". Got "12.8.0"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Ok I see the issue is probably with the package.json of Kingly. It has this content:
{
...
  "engines": {
    "node": "6.3.0"
  },
...
}

I will try to see what is the best way to proceed tomorrow. Maybe I can publish a new minor version of Kingly with "node": ">= 6.3.0" and then anotehr minor version of react-state-driven. The other alternative is to do the same but also update directly react-state-driven to the latest version of kingly but that may take one more day.
Thanks for your patience

Alright, can you try this: https://codesandbox.io/s/ym8vpqm7m9?file=/src/helpers.js
This has the latest Kingly and the latest <Machine/> component from react-state-driven  with the node configuration removed so there should not be any node error in principle.
Let me know if it works on your side.

I refactored the codesandbox so it is more React-like and uses JSX. That should help if you are not used to react-hyperscript. Also refactored the screen and error management. i will soon simplify the API in 0.11  by removing one or two parameters

Thanks for your kindness.
react-state-driven still depend on state-transducer-testing which also be limited with node 6.3.0.
I made a PR to fixed it.

PR accepted. Let me know if any questions on the library or further issues. It is very interesting for me to get feedback. I will review the Machine tests tomorrow. It should be based on an older version of the library so that has to be updated too.

Thanks a lot, @brucou
But it failed again because of<EMAIL_ADDRESS>expect version 6.3.0.
It's need to update graph-adt.

Just did it and update react-state-driven to 0.10.7. Can you please try again? hopefully this time it works. I will need to revisit all the build configuration over the week-end... sorry for the inconvenience.

Hi @brucou ,
Did<EMAIL_ADDRESS>dependent<EMAIL_ADDRESS>or<EMAIL_ADDRESS>The master branch was 0.10.6 and "state-transducer-testing": "^0.11.3" was stay the place of dependency.
It should be upgrade state-transducer-testing to ^0.11.4 and publishing version 0.11.4 of state-transducer-testing should be also desired.

Hi @brucou ,
The package state-transducer which also dependent by state-transducer-testing locked the node version too.
The repo of state-transducer had been moved to kingly. It should also be upgraded to make   all of this work.

I rmoved the dependency on state-transducer-testing and graph-adt. can you try again?

Hi brucou,
0.10.8 can been added successfully. I would test it functionally.
Thanks very very much for wasting your weekend time.

You are not wasting anything at all, it is my responsability to provide software that works and can be used. You reporting your issues is helpful for all those who have encountered the same errors than you and now no longer will have your issues thanks to you. So please continue to report any issue you find, and questions you have.

I'l lleave this open for a few days before closing it in case there is anything else occuring.

0.10.8 can been added successfully. I would test it functionally.
How did that go? Anything I can help with?

Sorry for leaving this because of other works.
Dependency issue was fixed now.
But react example was broken because of changes like state-transducer was deprecated. I'm still struggling with it.

Here is the react example with dependencies updated to Kingly: https://codesandbox.io/s/flickr-search-app-with-kingly-qivl3

It works. Thank yout, @brucou .

@leopku readme for react-state-driven has been updated to reflect the changes in the new version. The main documentation site has also been updated: https://brucou.github.io/documentation/v1/int-react/index.html

https://brucou.github.io/documentation/v1/int-react/index.html

Big thanks for so nice and detailed document.


Add an enableGlobalBasicAuth(username, password) option?

I had to add a simple basic-auth thing work today, which looked something like:
before { ctx ->
    val credentials = ctx.basicAuthCredentials()
    if (credentials == null || credentials.username != "username" || credentials.password != "password" ) {
        ctx.header(Header.WWW_AUTHENTICATE, "Basic")
        throw HaltException(401, "Access denied")
    }
}

This could be a config option:
val app = Javalin.create().apply {
    enableGlobalBasicAuth(username, password)
}.start()

I can probably try to pick this up if no one does this faster, but I'm thinking whether this is a good idea.
From what I understand, you'd like to have an option to basically hard code a username and password combination that would let you pass the authorization.
My concerns are that:

most of the times this is just too simple, I have doubts that anyone will ever use it except for test purposes, I think we can do better
it makes kind API kind of not consistent, there is an access manager but hey, there is also this special case for basic auth.
more often than not custom response payload and headers will be needed (at least projects I've worked so far wouldrequire those) to be set.
About that, we could probably think about other places in Javalin that hard code some predefined payloads/headers returned. I can try to find places that could be made more customizable and open a ticket for that if it is feasible.

We currently have this (from examples):
app.accessManager((handler, ctx, permittedRoles) -> {
    MyRole userRole = getUserRole(ctx);
    if (permittedRoles.contains(userRole)) {
        handler.handle(ctx);
    } else {
        ctx.status(401).result("Unauthorized");
    }
});

If we want to make it simpler so that we do not take roles into account, we could supply another interface that would look like this that would simplify roles.
@FunctionalInterface
public interface AccessManager2 {
    void manage(Handler handler, Context ctx, String username, String password) throws Exception;
}

or
@FunctionalInterface
public interface AccessManager2 {
    void manage(Handler handler, Context ctx) throws Exception;
}

or
@FunctionalInterface
public interface AccessManager2 {
    void manage(Handler handler, Context ctx, BasicAuthCredentials basicAuthCredentials) throws Exception;
}

and overload accessManager method to accept that. The behaviour would still be customizable per users demand and making that simple basic auth you've mentioned is trivial anyway, but we could add out of the box implementation as well.
We could combine it with a predefined implementation you mentioned to look like this (pseudo code):
public class FixedAccessManager implements AccessManager2 {
    private String password;
    private String username;

    public FixedAccessManager(String username, String password) {
        this.password = password;
        this.username = username;
    }

    public void manage(Handler handler, Context ctx) throws Exception {
        BasicAuthCredentials credentials = ctx.basicAuthCredentials();
        if (credentials == null || !credentials.getUsername().equals("username") || credentials.getPassword().equals("password")) {
            ctx.header(Header.WWW_AUTHENTICATE, "Basic");
            ctx.result("Access denied");
            ctx.status(404);
        }
    }
}

app.accessManager((handler, ctx) -> FixedAccessManager(a, b)::manage)

This way, it can very ease be adapted to all kinds of usages, like production (simple basic auth), dev/test (hardcoded username/password combination).
But overall I find it topic to be quite usefull, as I need to make simple basic auth myself soon.

Thanks for looking at this so thoroughly.
I agree that it's better to expand on the existing AccessManager concept than to introduce special cases, so what you're talking about sounds very interesting.
I'd go for the void manage(Handler handler, Context ctx); signature, and probably call it SimpleAccessManager.

About that, we could probably think about other places in Javalin that hard code some predefined payloads/headers returned. I can try to find places that could be made more customizable and open a ticket for that if it is feasible (at least I remember finding couple of hardcoded things that would get in my way, but I'd need to re-read the code and identify them once again at this point, might be wrong here

That would be great, thanks!

So basically we want to have 2 kinds of access managers, one that will take into account roles and one that would not.
For now, nothing will stop anyone from defining both acces handlers at once.  It is kind of logical, one that protects role-based endpoints, one for the rest. But, some corner-case question appeared in my mind. Shall the role-ignorant access manager be run after role-based one grants access for endpoint that have roles defined or shall we completely separate them apart and do not trigger role-ignorant one on role-based endpoints?
I'm thinking because having overloaded accessManager method could suggest that you can have more than one access manager at the same time for an endpoint but in case of completely separating them apart based on roles, it wouldn't be the case.
From the code perspective, it would still be perfectly find that role-based one grants the access, but simple one refuses it and it can be either kind of misleading or desired.
Also, we could think about adding security management overhaul on Javalin 2.0 wishlist #161 , especially given #218.
For now I can make this work in Javalin 1.0, but need to know how shall it behave.

My immediate reaction is that it should be either/or. Having two AccessManagers at the same time sounds like it could be very confusing. I'll think about it a bit tomorrow.

Also, we could think about adding security management overhaul on Javalin 2.0 wishlist #161 , especially given #218.

Added!

@majorpasza I was thinking we could rewrite a bit so that every method handler (get/post/put/etc) always goes through Javalin.accessManager. By default this will be a noop manager which just calls handler.handle(). It should be easy to add a basic-auth implementation then. I'm a bit worried that it might have an performance impact, but it's probably negligible.

Every call going through accessManager will not be backward compatible change.
Shall we move it to Javalin 2.0?
I'm still not sure how to proceed with that. I don't see how we can have one access manager. If we want to skip the roles, they will have different interfaces and they have different trigger mechanism (triggered when the roles are or are not defined).
btw. performance penalty is IMO certainly negligible here.

Every call going through accessManager will not be backward compatible change, because once no-op handler is overriden, new handler will always execute. Shall we move it to Javalin 2.0?

Yes, it will have to be for 2.0.

I'm still not sure how to proceed with that. I don't see how we can have one access manager. If we want to skip the roles, they will have different interfaces and they have different trigger mechanism (triggered when the roles are or are not defined).

Currently we wrap handlers if roles are provided:
Handler handlerWrap = roles == null ? handler : ctx -> accessManager.manage(handler, ctx, roles);

We could wrap every handler by removing the ternary (we should check if it's a verb though):
Handler handlerWrap = ctx -> accessManager.manage(handler, ctx, roles);

Then instead of having a SimpleAccessManager interface, just ignore the roles. Leaving this up to the user is better IMO. Am I missing something?

Then instead of having a SimpleAccessManager interface, just ignore the roles. Leaving this up to the user is better IMO. Am I missing something?

If I understand correctly, given all we've talked about, the conclusion is that we won't change anything in Javalin 1.0
I think we went full circle; the basic auth support for endpoints for now will still have to be done as app.before handlers :)
btw. Do you have any plans at the moment to create 2.0 branch?

If I understand correctly, given all we've talked about, the conclusion is that we won't change anything in Javalin 1.0

Correct.

I think we went full circle; the basic auth support for endpoints for now will still have to be done as app.before handlers :)

Yup, but it was still useful. I won't add enableGlobalBasicAuth(username, password) in 1.X either.

btw. Do you have any plans at the moment to create 2.0 branch?

I think we'll just use master. Releasing updates to 1.X after 2.0 is out will be too much work, and the changes shouldn't really be that big. My current plan is just to gather all the things we want in 2.0 in https://github.com/tipsy/javalin/issues/161, then start implementing them when we have enough. We'll do release candidates for 2.0, (like 2.0.0.RC0) before finalizing.


Big requests sometimes are truncated

I'm doing some very big http requests to an elasticsearch API endpoint and every once in a while I get a response that is not parsed as a JSON object correctly because it is truncated, so axios leaves it as a string. I don't get any error status from axios when that happens, but of course my code crashes because it is expecting an object, not a string.
What's the solution for that? How could I implement a retry interceptor for those cases?

I think i've been seeing this a lot too, it's causing about 40,000 failures a day for me

This problem is already being discussed in #61

@rubennorte Is this actually the same issue? #61 explains why the truncated response is being left as a string (rather than throwing), but it doesn't explain why the response is being truncated in the first place

I'm sorry, I understood that Axios had nothing to do with the response truncation. @demetriusnunes could you clarify that?

I'm not sure, but it is indeed the exact same issue as described in #61. So thanks for closing it.


Can not find any working faucet.

Need some testnet dogecoins at this address nf3rLxshXfGWtgRc593vkaxDHE6DZJ86nt

faucets give you Dogecoin if you have some coins in your wallet :p

Can not find any working faucet. can you transfer me some test coin ncdisZ6YczH7dVgrEx2GDcchBYGKzgV34t

@guchenglin111 done. Bust as @rnicoll already said, just mine some the next time. It's not that hard.
I'd like to add though, that another option is to use the built in miner. If you've got a working core wallet (which I will assume you do, when you want some testnet coins), just send setgenerate [number of threads/true to start mining, and setgenerate false to stop. (It's also possible to set this in dogecoin.conf, but I can't recall the exact parameter. (Something like generate=[threads/true])

Thank you. Your answer is very helpful

There is now a testnet faucet : https://doge-faucet-testnet.ggcorp.fr/
I also have lots of testnet dogecoin. Feel free to ping.


network: Demand sendable inbound subscription streams

The application code would want to pass the stream to other async tasks.
The Send + 'static bounds are similar to the outbound subscription
streams of the client-side subscription API (which need these bounds
to implement tower_grpc::client::Encodable).

Codecov Report

Merging #596 into master will increase coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #596      +/-   ##
==========================================
+ Coverage   44.76%   44.84%   +0.07%
==========================================
  Files         185      185
  Lines       16462    16425      -37
==========================================
- Hits         7370     7365       -5
+ Misses       9092     9060      -32

Impacted Files
Coverage Δ

network-grpc/src/server.rs
0% <ø> (ø)
:arrow_up:

chain-impl-mockchain/src/setting.rs
58.74% <0%> (-0.05%)
:arrow_down:

chain-impl-mockchain/src/block/header.rs
73.78% <0%> (-0.03%)
:arrow_down:

chain-impl-mockchain/src/leadership/mod.rs
0% <0%> (ø)
:arrow_up:

chain-impl-mockchain/src/block/mod.rs
54.96% <0%> (+0.82%)
:arrow_up:

...tocol-tokio/src/network_transport/response_code.rs
66.66% <0%> (+6.06%)
:arrow_up:

chain-impl-mockchain/src/block/version.rs
81.25% <0%> (+52.08%)
:arrow_up:

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 8175451...ef756bc. Read the comment docs.


JsonSerializationException after update to v11

After update to v11 an exception is throw if the value is a mock (https://www.nuget.org/packages/moq/).
var mock = new Mock<IFoo>();
IFoo foo = mock.Object;

var settings = new JsonSerializerSettings() { Converters = { new FooConverter() } };
var json = JsonConvert.SerializeObject(foo, settings);

The reason lies in ReflectionUtils.cs.GetProperties() returning 2 properties with the same name instead of 1.
This method is now being called with BindingFlags: Instance | Static | Public | NonPublic.
In v10 the BindingFlags are Instance | Public .
The method GetFieldsAndProperties calling GetProperties has some special handling to filter out "duplicates" but not in this case.
// this is a bit hacky
// if the hiding property is hiding a base property and it is virtual
// then this ensures the derived property gets used
if (resolvedMembers.Count == 0)
{
          resolvedMembers.Add(memberInfo);
}
else if (!IsOverridenGenericMember(memberInfo, bindingAttr) || memberInfo.Name == "Item")
{
          resolvedMembers.Add(memberInfo);
}

Expected behavior
Custom JsonConverter is being used for serialization of IFoo.
Actual behavior
An exception is thrown in v11

Steps to reproduce
Repro: https://github.com/PawelGerr/Newtonsoft.Json.v11.Repro

The project JsonRepro10 serializes the object using custom converter
JsonRepro11 throw an exception

Unhandled Exception: Newtonsoft.Json.JsonSerializationException: A member with the name 'Mock' already exists on 'Castle.Proxies.IFooProxy'. Use the JsonPropertyAttribute to specify another name.
   at Newtonsoft.Json.Serialization.JsonPropertyCollection.AddProperty(JsonProperty property)
   at Newtonsoft.Json.Serialization.DefaultContractResolver.CreateProperties(Type type, MemberSerialization memberSerialization)
   at Newtonsoft.Json.Serialization.DefaultContractResolver.CreateObjectContract(Type objectType)
   at Newtonsoft.Json.Serialization.DefaultContractResolver.CreateContract(Type objectType)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalWriter.Serialize(JsonWriter jsonWriter, Object value, Type objectType)
   at Newtonsoft.Json.JsonSerializer.SerializeInternal(JsonWriter jsonWriter, Object value, Type objectType)
   at Newtonsoft.Json.JsonConvert.SerializeObjectInternal(Object value, Type type, JsonSerializer jsonSerializer)
   at JsonRepro11.Program.Main(String[] args) in S:\Sources\PG\Repos\Newtonsoft.Json.v11.Repro\JsonRepro11\Program.cs:line 15

Thanks for the detailed description and repo. Fixed https://github.com/JamesNK/Newtonsoft.Json/commit/f2b6ea4966f2c99a38e07bb3b0d93f19db1da77b
Stick with 10.0.3 for now. A new version of 11 will come out in a week or two.


error  when installing for windows phone

Could not install package 'NFCForms 2.3.0'. You are trying to install this package into a project that targets 'WindowsPhoneApp,Version=v8.1', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

Hello, this error is due to the fact that NFCForms currently supports WP8.0 Silverlight.
I should be able to release a fix in the next few days but if you need it now you just have to download the source and change the PCL profile :)

thanks, you have been most helpful.


:construction_worker: Add a license generator

Changes
It checks for every file (except ./pincer/__init__.py) if the compressed license is present at the top of the file.
Check off the following

[x] I have tested my changes with the current requirements
[x] My Code follows the pep8 code style.

Not sure why this is failing for snyk. Besides that, I have tested the code and there is currently one file with no compressed license ./pincer/utils/replace.py

Ratelimit xD


For example, we might record the key strokes needed to write a real document and then play those back in a test, ensuring that the test always produces the same document.
To do this, we need to pipe fake key events into the widget tree.
Create a fake keyboard that hooks in at the system channel level and fakes keyboard input to the widget tree.

Great idea
also then you can compare what the test did against goldens
that will help with regression tests which is sometimes is needed as the api surface gets bigger and code base in general gets bigger.
The goldens should not be images but actually but can be . Rather they the piped in data should match the golden data .


Initial scaffolding

#minor# initial commit

Approved

@sandersky Sorry, missed your comments - I'll PR in those changes


Infinite Health option does not fully protect you in all cases in the latest trainer

Hi,
I 'm playing Master Mode on a PAL Wii U and yesterday I tried your latest trainer (v4.1.2). It worked well in general, with one exception. It is the Infinite Health option. Apparently, it protects Link from losing hearts in most cases, but not in all. More specifically, it does not seem to work properly when an enemy hits Link with a bomb arrow from far away or when he is hit by a Lionel blow.
Is there a way to fix this so that the option works in all cases? (I think this may be a PAL issue with skoolzout1's code).
Thank you in advance and keep up the good work!

 It worked well in general, except for one option. It is
the Infinite Health code. Apparently it protects Link from losing hearts in
most cases, but not in all. More specifically, it does not seem to work
properly when an enemy hits Link with a bomb arrow from far away or when he
is hit by a Lionel blow.
Is there a way to fix this so that the option works in all cases?
Thank you in advance and keep up the good work!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/joffnerd/botw-trainer/issues/159, or mute the thread
https://github.com/notifications/unsubscribe-auth/AccnaluUi7NpI9d2TR7xLJ6VtI_8yXMRks5sMzcrgaJpZM4OT6qu
.

I see. I didn't know it NEVER worked in all cases!  ;-O
Ok. Thank you. I suppose there is no other way than try to avoid some very specific blows... Bummer.

It's okay man me neither but there are very specific instances you can die,
codes aren't perfect but we always find work arounds ;3
On Jul 11, 2017 4:43 AM, "doctahoo"<EMAIL_ADDRESS>wrote:

I see. I didn't know it NEVER worked in all cases! ;-O
Ok. Thank you. I suppose there is no other way than try to avoid some very
specific blows... Bummer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/joffnerd/botw-trainer/issues/159#issuecomment-314391808,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AccnaonFg-auUDQWnIVsvBNRb91LGjMMks5sM0PfgaJpZM4OT6qu
.

The trainer is effectively writing to your health address insanely fast. When you get hit with a regular attack, you actually technically still lose the amount of hearts dealt by the attack, but your hearts are filled back up faster than a single frame even passes by in the game. So if you get hit by a move that deals more than your current amount of health, the game triggers the death  sequence, this is often calculated before the hearts display is ever updated..
You can use the Mipha's grace code to aid with this since Mipha's grace will always restore your health regardless of how you die.

Is there a way to constantly activate (even without pressing ZL) the shield special thingy there where every damage gets blocked? Not sure what that translates to in English.

Thanks, skoolzout1. I will combine some of your codes to try to get a similar effect to perfect immunity. :-)


drawer label throwing an error

Ive got a a drawer navigator with a bunch of StackNavigator's as screens. Inside each of those Stacknavigator's I have a navigationOptions object which looks like this:
{
  drawer () => ({ label: 'whatever' })
}

I have a contentComponent in my DrawerNavigator. Inside the contentComponent I would like to get the label for for the current screen ( getLabel() ). When I do this, it throws an error on line 53 of DrawerSidebar.js: focused is undefined.
My drawer.label is merely a string, there is no function. What is going on? I will keep debugging but this does not seem correct to me.

I think you are missing a :
Like
drawer: () => (...)
}```

Sorry that was a typo in the issue description. with that fixed its still throwing the error.

getLabel requires an argument of this shape https://github.com/react-community/react-navigation/blob/master/src/views/Drawer/DrawerView.js#L20. The error you are getting says internal implementation was unable to access focused property on an undefined object. See how it's called in default conentComponent.


IPA tags override for phonemes

[ ] Enable overwritten tags reading.
[ ] Enable overwritten tags editing in Phonology editor.

So far it works with one centralized IPA file.

So far it works with one centralized IPA file.


Fragile data in response

I wrote about this here

When ProblemDetailsMiddleware catch exception (for example MySQL connection problem) then we cannot put code of this exception and what is worse put message from exception to detail.

I have same problem with apigility and I need to extend it by my own. Unfortunately after data leaks on production enviroment...
Here it's falling test to show how it's dangerous and should be fixed. If we aggre about that I can provide fix for that.

Thanks, @snapshotpl


Instant search

Hi guys, @xamgore told me you want to make something that would really fit my needs. So I want to request an instant search through my notes. Not only titles, but also the content. Thank you.

:+1: really need this feature

Good idea!


getCurrentPosition is broken in ios

I was getting this error
flutter: error MissingPluginException(No implementation found for method getCurrentPosition on channel xyz.luan/audioplayers)
but audioplayer was working fine in previous versions, so I took a look at commits and found this https://github.com/luanpotter/audioplayers/commit/8ed42bc8598ca99975e5726ba0d158579c474fcc

Maybe you just need to do: flutter clean && flutter run


When do you rewrite the generator grammar into Promise or async ... await grammar?

When do you rewrite the generator grammar into Promise or async ... await grammar?

The answer is simple - never 😉
https://github.com/redux-saga/redux-saga#redux-saga-chooses-generators-over-asyncawait


Renaming, so that there is only one ApplicationId type.

linera_execution's enum type becomes SystemApplicationId, so the UserApplicationId type alias can be removed and the type ApplicationId now refers to a user application everywhere. The User is also dropped from ApplicationDescription.

Closing, since the type might get removed soon anyway.


RTSP Not working with Yoosee Camera

Hello team,
I use latest viseron on centos 9. I tried to add RTSP URL but not working.
URL: rtsp://admin:123456789abc@<IP_ADDRESS>:554/onvif1
This url working well with VLC.
When trying with Viseron, the web show error:
FFprobe could not connect to stream. Output: {'error': {'code': -1482175992, 'string': 'Server returned 5XX Server Error reply'}}
Please help to resolve.
Thank you.

Could you paste your config here? I suspect you might need to tweak your settings a bit

Could you paste your config here? I suspect you might need to tweak your settings a bit

Hi Sir,
Thank you for your quick response.
The config:
Thanks for trying out Viseron!
This is a small walkthrough of the configuration to get you started.
There are far more components and options available than what is listed here.
See the documentation for the full list of configuration options.
Start by adding some cameras
ffmpeg:
camera:
viseron_camera:  # This value has to be unique across all cameras
name: 44NTH
host: <IP_ADDRESS>
path: /onvif1
port: 554
protocol: rtsp
rtsp_transport: tcp
username: !secret cam3_username
password: !secret cam3_password
width: 1280
height: 720
fps: 5
codec: h264
audio_codec: null
stream_format: rtsp
ffprobe_loglevel: debug
online_camera:  # This value has to be unique across all cameras
  name: Camera 1
  host: <IP_ADDRESS>
  path: /axis-cgi/mjpg/video.cgi
  port: 80
  stream_format: mjpeg
  fps: 30
  recorder:
    idle_timeout: 1
    codec: h264

mog2:
motion_detector:
cameras:
viseron_camera:
fps: 30
online_camera:
fps: 30
darknet:
object_detector:
cameras:
viseron_camera:
fps: 30
scan_on_motion_only: true
labels:
- label: person
confidence: 0.75
trigger_recorder: true
online_camera:
fps: 30
scan_on_motion_only: false
labels:
- label: person
confidence: 0.8
trigger_recorder: true
To tie everything together we need to configure one more component.
nvr:
viseron_camera:
online_camera:
Now you can restart Viseron and you should be good to go!
online_camera is working well. viseron_camera has problem with rtsp with error below:
[2023-12-25 08:43:20] [ERROR   ] [viseron.components.ffmpeg.stream.viseron_camera] - [rtsp @ 0x55a4d308f900] Nonmatching transport in server reply
[2023-12-25 08:43:20] [ERROR   ] [viseron.components.ffmpeg.camera.viseron_camera] - Frame reader process has exited
[2023-12-25 08:43:20] [ERROR   ] [viseron.components.ffmpeg.stream.viseron_camera] - rtsp://:@<IP_ADDRESS>:554/onvif1: Invalid data found when processing input
Thank you.

Hi Sir,
I tried to debug error with command: "ffmpeg -hide_banner -loglevel debug -avoid_negative_ts make_zero -fflags nobuffer -flags low_delay -strict experimental -fflags +genpts -use_wallclock_as_timestamps 1 -vsync 0 -timeout 5000000 -c:v h264 -rtsp_transport tcp -i rtsp://admin:123456789abc@<IP_ADDRESS>:554/onvif1 -f segment -segment_time 5 -reset_timestamps 1 -strftime 1 -c:v copy /segments/viseron_camera/xx.mp4 -f rawvideo -pix_fmt yuv420p pipe:1
"
Response below.

Please help to check.
Thank you.

Response below.
It may be camera rtsp bug. Try to use udp transport: rtsp_transport: udp in camera settings.

Seems to be an issue with your cameras implementation of RTSP: https://stackoverflow.com/questions/66280861/ffmpeg-nonmatching-transport-in-server-reply-but-openrtsp-works
Nothing that can be done on Viserons side sadly :(

Hi guys my problem appears to be similar (same product) but on viseron logs I can see something odd, like:
[viseron.components.ffmpeg.stream.camera_1] - rtsp://:@gmail.com:@<IP_ADDRESS>:554/Streaming/Channels/101/: Server returned 400 Bad Request.
where it shows : it should be my username which are farely configured:
username: !secret camera_one_username
password: !secret camera_one_password
The proof is that my password appears on the log, unprotected I must say (really a security issue for you guys to solve)...
Any toughts?
Thanks in advance fo ryour attention and support


Reconciling OS and research datamixing code

This PR brings in the datamixing.py file from the research code and some supporting files.  It includes updates to test_datamixing.py to reflect the research behavior.  Those tests do pass.  It also includes some preliminary updates to generate_data.py to make use of the research version of the datamixing code.  However, a lot more work is needed to fully reconcile generate_data.py.  In this PR, the tests for generate data do not pass and a lot more work is needed to sort out to what extent we want the new behavior to match what the existing tests are checking for and to what extent we want to update the tests to match the new behavior.  Here are some highlights of what is included in this PR:

Put datamixing.py into utils which is where it is in the research code instead of the root SDG folder which is where it was in the preexisting repo (starting with the research datamixing.py)
Brought datautils.py over from the research code because it is used by datamixing.py.
Brought parse_and_convert.py over from the research code because it includes some capabilities that the preexisting code included in datamixing.py
test_datamixer_can_load_default_recipes becomes test_load_default_recipes because there is no longer a "DataMixer" class
The preexisting load and sample method was called _load_and_sample_datasets while the research method was called load_ds.  I adoped the former because it seems like a more descriptive name that explains what it does better.
test_datamixer_can_load_default_recipes becomes test_load_default_recipes because there is no longer a "DataMixer" class
Removed test_recipe_init_with_empty_params_* because empty params is not supported in the research code
The test_init_with_empty_recipe_files used to check to make sure we were populating defaults, but the research code doesn't do that so now we just verify that we get an empty recipe from it.
Dropped test_load_ds_with_absolute_jsonl_path because the functionality it was testing (looking up a dataset file from a relative path as specified in the recipe file) doesn't exist in the research code. Instead, in the research code, you just send the path directly to the load method.
Updated test_load_ds_with_absolute_jsonl_path code accordingly.
_get_question_hack and _get_response_hack were in the preexisting datamixing.py but not the research datamixing.py.  For now, I have copied them into generate_data.py since they seem to be part of some sort of legacy format support that is specific to that file and it is probably better to keep all the temporary/deprecated code localized in one place.
Dropped _convert_to_messages from generate_data.py because the research version of it is in parse_and_convert.py
Brought DataMixer._load_default_recipe from the preexisting code base into the new utils/datamixing.py as load_default_recipe.  This was needed because the research code does not have a DataMixer class but we still need a way to load the default recipe.  The research version of generate_data handles this by just hard-coding the location of the default recipe, but the open source code needs to take in a set of directories and search all of them for this.

Commented out the code to set _precomputed_skills_length because this is a field of DataMixer which doesn't exist in the research code.  If we can confirm that this functionality is no longer needed, we should delete the commented out code, but I am leaving it there for now while we sort this out.
The preexisting version defaulted to generating an empty Recipe with a given system prompt.  However, the research code does not have a constructor for an empty Recipe or a way to specify a system prompt for a Recipe because the system prompt is always loaded from the Recipe file.  So for now load_default_recipe just returns None if no recipe file is found.

Is this something we still want to do? I recall we talked about perhaps not picking in the latest datamixing changes from the research codebase, and instead using our datamixing bits on top of their core generation pipeline enhancements.


Reproducer for 'when' expression must be exhaustive, add necessary 'else' branch.

Restored: https://github.com/vanniktech/playground/pull/33


Fix: Frame env var references

What changed? Why?

fixed environment variable references in the basenames claim frame

Notes to reviewers
How has it been tested?

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

Review Error for JFrankfurt @ 2024-09-16 21:56:56 UTC
User failed mfa authentication, see go/mfa-help


Handle form state during an async operation

It would be a nice to have if there could be a defined section to display or NOT display pieces of the form during an async operation like a form submission to the server, like disable the submit button while the form is being submitted.

Hi @PizzaConsole !
You mean a "how to" section in the docs?

A section like a an HTML tag or a prop. It would be nice if the functionality was baked in if you wanted to use it

I can look into this, but I am not sure it's possible because the submit event or handleSubmit not taking Promise. It takes a function you define which can contain a Promise, but I can't handle that promise.
In my opinion creating a variable to handle it is not more work and gives better control over the form submission, but I will research this.

If it doesn't make sense in the library then just an example in the docs I think would be great.

Hi @PizzaConsole
I was wrong about the topic and I managed to implement this both in type-safe and non type safe version.
In the release not you can find an example (for type-safe version), but I will make a detailed documentation in the docs later.
Thank you for suggesting this feature.


docker build sucessfull but the container exits within few seconds

I was built the docker  sucessfully but with couple of warnings .when I ran docker container exits within few seconds.
make[3]: Nothing to be done for 'install-exec-am'.
make[3]: Nothing to be done for 'install-data-am'.
make[3]: Nothing to be done for 'install-exec-am'.
*** WARNING: systemctl not found. otbr cannot start on boot. -------------  ????
Removing intermediate container 7003b6dc2a56
---> 774bb40810b7
Step 9/11 : RUN chmod 644 /etc/bind/named.conf.options
---> Running in e316faa75dbf
Removing intermediate container e316faa75dbf
---> 7347b75a33e7
Step 10/11 : ENTRYPOINT ["/app/borderrouter/script/docker_entrypoint.sh"]
---> Running in 9e53ffb89e95
Removing intermediate container 9e53ffb89e95
---> f82681b78c3a
Step 11/11 : EXPOSE 80
---> Running in 34a67e75e34e
Removing intermediate container 34a67e75e34e
---> fbf2d420719b
Successfully built fbf2d420719b
Successfully tagged br:v1
Also saw bunch of messages during the docker build:
debconf: unable to initialize frontend: Dialog
debconf: (Dialog frontend will not work on a dumb terminal, an emacs shell buffer, or without a controlling terminal.)
debconf: falling back to frontend: Readline
debconf: unable to initialize frontend: Readline
debconf: (This frontend requires a controlling tty.)
debconf: falling back to frontend: Teletype

We just posted our guide for running the OTBR Docker container. Try following the guide and let us know if that helps. Thanks.


SCRIPT5007 issue with internet explorer 10 / 11

Hi there,
I'm building a website including PrismJS. Work very well on browsers like Chrome but,
some time on firefox and evrytime on IE 10 / 11 my Prism Slider doesn't work.
The console detect an SCRIPT5007 error. It means that my 'Object' is not initialized
It seem come to this part of code :
/** Set of images to be used.

@type {Array} **/
var slides = [
$('#img-prism-1').attr('src'),
$('#img-prism-2').attr('src'),
$('#img-prism-3').attr('src'),
$('#img-prism-4').attr('src')
];

have someone the same probleme ?!
Thk

Hi @losocreativ,
Are you sure you posted this to the right project? This is Prism.js the syntax highlighter. Maybe you are looking for Prism Slider.

Oups ! sorry :S thanke you ! ;)


Published asset folder : lowercase

Could you please make the published assets folder lowercase ? As this causes issues on case sensitive OS's.
Thanks!

That for the feedback. I'll add this as a feature request. Thanks

Could you please try to fix this? This issue is blocking us on our staging server.
Thanks

Awesome! :+1:


Python branch changes

No issues; mostly cosmetic changes or spelling fixes.
Have updated core.yml to remove static role names so that it can be run multiple times in the same account for lab environments.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Requested changes haven't been made yet. I'm going to close this for now but feel free to make those suggested changes and reopen when ready.


How to block external scripts without changing script type

I'm successfully using the cookieconsent script on several websites that are completely under my control (thus all

I see, thank you for the comprehensive answer @orestbida.


Log deletes of all descendent pages during delete

Currently, when deleting a page, only the page itself and its direct children are logged. This change adds log entries for all descendents, and thus all the pages which are deleted. The crux of the issue being that .get_children() only returns direct children.
This will slow down deletes slightly, but make them more correct. To counter this, I've added .specific to the queryset.
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[ ] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

#8292 does the 2.16.x backport


update module to remove s3 data source and support aggregation

what

update the module to take the ARN of the AWS Config S3 bucket as an input param rather than looking it up via a data_source
conditionally create sns_topic
conditionally create iam_role
add aggregation of config in central region/account

why

using the data_source causes an issue in a multi-account, cross-region setup
fixes allow user to create sns_topic and iam_role outside the module
allow module to send all its config to a central region/account

/test all

I'm not sure what's going on with the pre_commit...the module is formatted correctly.
☸ local in terraform-aws-config/modules/cis-1-2-rules on  chore/fix-bucket-permissions (2da96c)
[$] via 💠 default on ☁  (us-east-1) on ☁<EMAIL_ADDRESS>❯ terraform fmt
❯ echo $?
0


Static Site CI Failure: Unable to download Cypress

The Static Site CI fails due to not being able to download Cypress
See the output here: https://github.com/frankdejonge/uptime-monitors/runs/7131922875?check_suite_focus=true

Looks like a weird one-off npm error... can you try re-running the workflow?

I had run it a couple times (3 or 4) times when I encountered this. I'll check again and report back.

This seems to be back up now. Will close this issue.

Thanks for checking! :)


goto references on __init__ should show me all locations of construction

Is your feature request related to a problem? Please describe.
The clangd language server has this nice feature where if you "goto reference" on a constructor, it shows all places where a new instance is constructed of that class.
pyright should do this for __init__ methods as well. It currently reports "no references".
It already shows references when I "goto reference" on the name of the class, but among those references are also import statements, and type annotations. That's too much noise if I just want to know where instances are constructed.
Describe the solution you'd like
Doing "goto references" on an __init__ method should show me only the places of construction, not imports and type annotations.
Additional context
Perhaps it's also an idea to show all places where the instance of a class is (implicitly) converted to a string if you "goto references" on a __str__ method.

I remember talking about this a few months ago. I'm surprised we didn't have a canonical issue for this, thanks. I think that this makes sense.


Report whether a target is part of the root package in the sourcekit-lsp API

This is used so we don’t search for tests in targets of package dependencies.
Companion of https://github.com/apple/sourcekit-lsp/pull/1201
rdar://126965614

@swift-ci test

https://github.com/apple/sourcekit-lsp/pull/1201
@swift-ci Please test

https://github.com/apple/sourcekit-lsp/pull/1201
@swift-ci Please test


Alternative to this extension (due to deprecation?)

Since this extension seems to be marked as deprecated (still open PR #43 ),
Is there any alternative to the tasks provided here?

@riserrad Is this project abandoned? If not, who should we contact about issues?

Any update or information regarding this question?

This project is on path for deprecation, unless anyone interested in taking over shows up.

@riserrad That we noticed. Do you have a suggestion for alternative? Or are we on our own?

@riserrad do you still have write access to this repository or is it only the Microsoft bot account at this point? I might consider taking over, if I find the time to learn basic ts, but it will be at least a few months in the future. But if another set of tasks exists, that can fill the gap from this repository, I don't see a point in trying to save it.

@bjtho08 yes I still have access to this repo. I am not aware of any set of Tasks that are similar to this one though.
Let me know if I can help you by granting access, etc.

@riserrad Any clarification on this issue yet?
If still deprecated, do you know an alternative?

@riserrad Any update on this repo? Consider fully deprecating this task.
Are there any alternatives, or should we move to bash/cli tasks and go back to scripting ourselves?


early.stop.round does not work properly

early.stop.round does not work if only one metric is provided. In fact, it seems like early.stop.round does not use the first metric for stopping when two metrics are provided. The source of both errors appears to be the same line of code.
data(agaricus.train, package='xgboost')
dtrain <- xgb.DMatrix(agaricus.train$data, label = agaricus.train$label)
history <- xgb.cv(data = dtrain, nround=3, nthread = 2, nfold = 5, metrics=list("rmse"),
                  max.depth =3, eta = 1, objective = "binary:logistic", early.stop.round=3)

This results in the following error:

Error in strsplit(score, "\+|:")[[1]][[2]] : subscript out of bounds

The issue seems to be the following line of code in xgb.cv:
 score = strsplit(ret, "\\s+")[[1]][1 + length(metrics) + 2]

Changing that line to the following should fix it.
score = strsplit(ret, "\\s+")[[1]][2 + length(metrics)]

Thanks for reporting. Which version of xgboost are you using? Do you want to submit a PR for this?

I'm using version 0.4.2. I just submitted the PR.

Fixed by #730

Is this issue resolved in the current version of xgboost installed through CRAN? I got the same error on xgboost 0.4.3 recently installed from CRAN. The error goes away after removing early.stop.round argument.

Error in strsplit(score, "\+|:")[[1]][[2]] : subscript out of bounds

Still have the same issue with version 0.4.4 installed from CRAN and github, both on Windows and Linux.

Have the same issue on Ubuntu 14.04.

I am getting this error while using 0.4.3 on x86_64-unknown-linux-gnu/64bit (64-bit) server. Please let us know if this error has been resolved. If so I need to ask server admin to update xgboost version.

I am getting this error since updating to version  0.4-4 on Windows.