Predict runtime of a query

Amazing work btw, just getting started with this project.
I was working on a similar project. If we could factor in the running queries or the moving window cpu of the cluster. Using that info + blocked time ms, we could also predict query runtime!

Thanks for your interest!
Predicting query runtime would need more information, such as the moving window CPU and blocked time you mentioned. Currently, we only support predicting CPU time and peak memory as these metrics are independent of how busy the cluster is.
Feel free to contribute to this project or ping us if you want to know more.


Feature/home screen

Description

Creates recommendation model file to house recommendations structs
Adds segue to modal view of account settings
Minor UI tweaks

Trello card: https://trello.com/c/NhkKTlcS/67-home-screen
Type of change

[x] New feature (non-breaking change which adds functionality)

Change Status

[x] Complete, tested, ready to review and merge

How Has This Been Tested?

[x] Manually tested with simulator (iPhone 8)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] There are no merge conflicts

 Home Screen


Serve fonts from CDN for improved performance (low priority)

From @robinv85 (#361 ):

For the fonts, I think we don't even have the fonts correctly imported at the moment. There are 3 custom fonts, one didn't seem to work via npm install, and I question if we really need 3 custom fonts, one of them is currently only used on some buttons. This might have a big impact on performance. To import the fonts via CDN would be nice, we could just import them in index.html instead of currently adding them through npm install.

yeah not sure if the info that I posted before is still accurate at the moment

Ah okay. @robinv85  feel free to edit the main comment or close this if not relevant.
Josh

Beside that, we re-set font-family even it use the same global font.
For example (component FaqContainer, FootNote, CookieAlert,...):
body {
  font-family: "Poppins", "Roboto", "Oxygen", "Ubuntu", "Cantarell", "Fira Sans", "Droid Sans", "Helvetica Neue", sans-serif;
}

and element:
.elementClass {
  font-family: Poppins;
}

Why do we have to declare a very long web font-stack live above?
Why don't we switch to Google Embed Font for better performance?

Thanks.

@phuclb generally a lot of people worked on this code base and lots of code got merged without proper review or alignment with other developers...
I also noticed that really long font stack before, can't remember who added that, it's not necessarily a big problem, but usually it would be enough to have only a few in there (although nowadays there are a lot of different browsers, operating systems etc.) so if you think you can improve that font stack and have a good way to check these fallbacks and compatibility across different environments, then you're welcome to contribute @phuclb
If with "Google Embed Font" you mean using the Google Font API aka using the Google CDN to load the fonts, yeah this is exactly what is stated in this issue here...

@robinv85, we can embed Google Font in via  tag, it's also a CDN load, no need to use API with script.
I can review and remove all redundant code (font-family), and shorten font stack if necessary, to improve performance, a bit.
Let me know final decision.
Thanks.


Optimistically set page location and update the urlbar before the navigation happens

Fixes https://github.com/mozilla/tofino/issues/1118
Doing this so that the UI shows the intended location instead of just some blank text. This could still happen though when clicking the back/forward buttons, since we don't know the location before the navigation happens; in those cases, just display 'Loading...'
Signed-off-by: Victor Porof<EMAIL_ADDRESS>

Coverage decreased (-0.02%) to 34.302% when pulling bd2505b898a86bd7c4c7ce69c8251e77d3e5b508 on victorporof:set-location-sooner into bec8c672f597d4144ba98e2c3e1da6a8fc3249a3 on mozilla:master.

Coverage decreased (-0.02%) to 34.302% when pulling bd2505b898a86bd7c4c7ce69c8251e77d3e5b508 on victorporof:set-location-sooner into bec8c672f597d4144ba98e2c3e1da6a8fc3249a3 on mozilla:master.

Coverage decreased (-0.02%) to 34.269% when pulling d09c3f83152d078aa4b3fcbc5c94f56dd79dddf5 on victorporof:set-location-sooner into f73bc3c14dc9ba24b1d57b62f820f655b7577169 on mozilla:master.

Coverage decreased (-0.02%) to 34.236% when pulling 41633514fddd5acfbce1897fdd3505f4c979f9d9 on victorporof:set-location-sooner into 0d7a8610da2e06605f7775432ffbcc8b60027050 on mozilla:master.

Coverage decreased (-0.02%) to 34.236% when pulling 41633514fddd5acfbce1897fdd3505f4c979f9d9 on victorporof:set-location-sooner into 0d7a8610da2e06605f7775432ffbcc8b60027050 on mozilla:master.

Works for me, but see the inline comment

Filed https://github.com/mozilla/tofino/issues/1139


Pass props to underlying

This allows us to pass accessibility props like allowFontScaling={false} to the emojis as well (as now they don't honor the allowFontScaling prop of the parent/wrapper  for example).

Would also "solve" #15 and like for those who'd like to style the component directly rather than wrapping it.


Fix clippy warning with 1.79.0 toolchain

The newest version of clippy catches a clippy::manual-unwrap-or-default issue, causing CI builds to fail.
This implements the recommended fix and therefore should fix the CI builds.

closed as #88 contains this changes. Thanks!


Add Http503Verbosity to the main http.sys documentation page

This is a useful setting if one is using http.sys, adding it to the main documentation page.

FYI I'd filed https://github.com/dotnet/AspNetCore.Docs/issues/20827 when this first came up. Long term we'd like to avoid maintaining multiple lists.

I'll reply on the issue.


Make some mc-sgx-core-types::ReportBody members public

The following types have been made public:

ExtendedProductId
IsvProductId
FamilyId

Current dependencies on/for this PR:

main

PR #287   👈

This comment was auto-generated by Graphite.


0.8.6: inline styles that have a dash in them e.g. background-color are ignored

Description:
Hi. Apologies! I found another inline style issue... A template with inline styles that have dashes in them, e.g. background-style, font-weight, etc are not rendered to the DOM but are output when toHTML() is used.
Versions affected:
0.8.x, 0.9
Reproduction:
0.8.6 - no background-color is rendered:
https://jsfiddle.net/d1em01kt/
0.7.3 - correct rendering:
https://jsfiddle.net/n9rvkahd/
Code:
new Ractive({
  el: 'body',
  template: 'background color should be red',
  onrender: function(){
    console.log(this.toHTML());
  }
});

Neat! I didn't know setProperty uses actual css names. That'll let us ease up on the processing applied to properties during updates (camelization).

I've pushed out 0.8.7 with the fix. Hopefully there are no more #TIL experiences lurking around the corner :laughing:

Ha. Gosh, thank you for your work on Ractive and keeping it alive! It's working nicely in my html email abomination app now. This fix is good. Yeah, I hope, for your sake, to not find anything else.
Also, today I learned what #TIL means.

On the plus side, the test suite is getting more and more comprehensive :grinning:

Also, thanks for the bug reports! It's not great that there are bugs, but it's always great when bug reports have good descriptions and nice, easy-to-follow reproductions.


Bigtable integration tests ran out of quota

the integration tests failed with:
2022-08-08T07:22:26.793365839Z [DEBUG] <140609869514304> CreateInstance() polling loop result (google/cloud/internal/async_polling_loop.cc:130)
Operation successfully rolled back : Insufficient node quota. You requested a node count of 3 nodes for your cluster, but this request would exceed your project's node quota of 30 nodes total across all clusters in this zone. Contact us to request a quota increase: https://cloud.google.com/bigtable/quotas#quota-increase

The existing instances were:
 gcloud bigtable  --project cloud-cpp-testing-resources instances list
NAME           DISPLAY_NAME    STATE
it-0qdg703u    IT it-0qdg703u  READY
it-95fvzx3f    IT it-95fvzx3f  READY
it-gku9hi3d    IT it-gku9hi3d  READY
it-igna325n    IT it-igna325n  READY
it-jgyx9r2q    IT it-jgyx9r2q  READY
it-n5mthaj6    IT it-n5mthaj6  READY
it-patgch9m    IT it-patgch9m  READY
test-instance  test-instance   READY

I deleted them using:
gcloud bigtable  --project cloud-cpp-testing-resources instances list --filter=name:it- --format='value(name)' | xargs -I {} gcloud bigtable --project cloud-cpp-testing-resources instances delete {}

I think we should store the creation date in the description and remove them when the tests start if they are too old.

GCB: https://console.cloud.google.com/cloud-build/builds;region=us-east1/7c8b7403-e9c4-4c9f-9541-1b4a913af2ef;tab=detail?project=cloud-cpp-testing-resources
Raw: https://storage.googleapis.com/cloud-cpp-community-publiclogs/logs/google-cloud-cpp/main/7cd4c4c133faae2a8c2a269e19f9fdd606d94fca/fedora-36-integration-daily/log-7c8b7403-e9c4-4c9f-9541-1b4a913af2ef.txt


Does not support Firefox debug builds

In debug builds a magic field gets added to arena_run_t and arena_t. This causes some of the read offsets to be wrong. Would it be better to use the type information from the binary to lookup fields instead of hard coding offsets?

unmask_jemalloc is unsupported, please use shadow instead:
https://github.com/CENSUS/shadow


{AKS} Fix validation for kms options

This checklist is used to make sure that common guidelines for a pull request are followed.
Fix validation for kms options.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

AKS


Tests/Account & Domain Isolation and More Domain/Account Tests

Issue #, if available:
N/A
Description of changes:
Adds Domain/Account/User creation and cleanup on unit test runs.
The unit tests now still only take about 15 seconds, but now they run in a domain with UUID that is cleaned up afterwards. This allows test isolation and helps to guarantee failed test cleanup.
In doing so I've also managed to turn enable about 40+ unit tests that were skipped in the pipeline previously.
Sigh, and now they're still skipped since we don't yet have an ACS instance to hit.
I labeled the integ tests and filtered them in the makefile for now.
Testing performed:
More than before!

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #121 (80fde6f) into main (eb51b18) will decrease coverage by 0.11%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main     #121      +/-   ##
==========================================
- Coverage   29.00%   28.88%   -0.12%
==========================================
  Files          27       27
  Lines        2179     2167      -12
==========================================
- Hits          632      626       -6
+ Misses       1473     1466       -7
- Partials       74       75       +1

Impacted Files
Coverage Δ

pkg/cloud/isolated_network.go
32.32% <0.00%> (-1.30%)
:arrow_down:

pkg/cloud/client.go
26.47% <50.00%> (ø)

pkg/cloud/instance.go
79.11% <0.00%> (-0.89%)
:arrow_down:

controllers/cloudstackcluster_controller.go
47.47% <0.00%> (-0.05%)
:arrow_down:

controllers/cloudstackzone_controller.go
0.00% <0.00%> (ø)

controllers/cloudstackmachine_controller.go
0.00% <0.00%> (ø)

controllers/cloudstackaffinitygroup_controller.go
0.00% <0.00%> (ø)

...ontrollers/cloudstackisolatednetwork_controller.go
0.00% <0.00%> (ø)

...ollers/cloudstackmachinestatechecker_controller.go
0.00% <0.00%> (ø)

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 eb51b18...80fde6f. Read the comment docs.

@maxdrib I just fixed up that last skipped test. It's still slow even with the force stop. We don't run integs in the Pipeline atm anyway, so it is what it is I suppose. Please mark lgtm if you approve. Thanks!

/lgtm


added submodules update command to be executed in maven's intialize phas...

...e
This should fix #133, merge whenever you're ready, e.g. after 1.0 has been released.

Coverage remained the same at 67.05% when pulling 0a1a7fb274631402bf06aa618b2920d3694c2b48 on krichter722:submodules into 02f0077f84d83167810b9b2e31d1ff70b00a79a7 on PhantomYdn:master.

Incomplete because commit which belows here accidentally was added to #149, see #152


[razor-network-voting] update strategy

Changes proposed in this pull request:

Update subgraph endpoint
Update strategy to fetch score
Update addresses in examples.json

@adi44 Can you plz check and confirm this?


Add supported pricing tiers for containers

This has become a common question from customers. We need to have container supported pricing tiers documented clearly.

@shuxianc : Thanks for your contribution! The author, @diberry, has been notified to review your proposed change.

@shuxianc  Thank you for submitting the PR. I'm checking with the service teams to validate that the containers and one-key will always align before accepting the PR.

Thanks Dina. BTW if you meant checking with PG I can assure you the information is current and correct. Because I am the dev owner of Cognitive Services container billing.  😊
From: Dina Berry<EMAIL_ADDRESS>Sent: Tuesday, April 9, 2019 10:39 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Shuxian Cai<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Add supported pricing tiers for containers (#29068)
@shuxianchttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fshuxianc&data=02|01|Shuxian.Cai%40microsoft.com|02ea85b20d9a4b9ca0a908d6bd124d49|72f988bf86f141af91ab2d7cd011db47|1|0|636904283633485141&sdata=NQyvxnwT5f2LK66%2FcFZUJx%2F7V8%2BSJtLe%2B%2FdI02abDW4%3D&reserved=0 Thank you for submitting the PR. I'm checking with the service teams to validate that the containers and one-key will always align before accepting the PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fpull%2F29068%23issuecomment-481357247&data=02|01|Shuxian.Cai%40microsoft.com|02ea85b20d9a4b9ca0a908d6bd124d49|72f988bf86f141af91ab2d7cd011db47|1|0|636904283633495150&sdata=xd3IqEoghTb4wBIBdnQlye14w1oU6eswxzFa4VcQPS8%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJOUp5Pi2MBoTbFrEnjc1Znq-5qj3MoFks5vfNBHgaJpZM4ckRGs&data=02|01|Shuxian.Cai%40microsoft.com|02ea85b20d9a4b9ca0a908d6bd124d49|72f988bf86f141af91ab2d7cd011db47|1|0|636904283633495150&sdata=lV1G5UFyPjmLZ5hGsf%2BOVU6SfRl6KdlArlq9ZXjBb%2Fk%3D&reserved=0.

@shuxianc Awesome! Glad to hear that. I will definitely #sign-off.

@Jak-MS : Thanks for your contribution! The author, @diberry, has been notified to review your proposed change.

@Jak-MS : Thanks for your contribution! The author, @diberry, has been notified to review your proposed change.


Single date selection added.

Previously I have found that we can select multiple dates with these awesome plugins but faced issues while selecting a single date. So I have changed some code in this swift file so the user can select single or multiple both dates. Please merge this code so I can directly fetch the updated code with "pod install" and I hope many users like me still facing the same issues.
Best Regards,
Ajoy Karmakar

inside the CalendarDateRangePickerViewController.Swift file, navigate didTapDone() function and edit like this:
@objc func didTapDone() {
if selectedStartDate != nil && selectedEndDate == nil{
delegate.didTapDoneWithDateRange(startDate: selectedStartDate!, endDate: selectedStartDate!)
}else if selectedStartDate == nil || selectedEndDate == nil {
return
}else{
delegate.didTapDoneWithDateRange(startDate: selectedStartDate!, endDate: selectedEndDate!)
}
}

much easier and cleaner.


"docker inspect -f dns" incompatibility between 1.8 and 1.9

As noted in #17026,
docker inspect -f {{.HostConfig.Dns}} somecontainer

works in 1.8 (and earlier) and doesn't work in 1.9, while
docker inspect -f {{.HostConfig.DNS}} somecontainer

works in 1.9 and doesn't work in 1.8.
This is a pretty bad compatibility breakage for anyone with a script that fetches this setting.
Also I cannot see the change mentioned in the release notes.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

DnsOptions and DnsSearch are also affected; furthermore, 1.9 docker inspect still reports these values with the original 1.8 capitalisation.

Hm possibly related to https://github.com/docker/docker/issues/15566, https://github.com/docker/docker/pull/15567 https://github.com/docker/docker/pull/17284
/cc @calavera

oh, I see https://github.com/docker/docker/issues/17026 was mentioned in the description

Thanks for reporting this.

Tentatively adding to 1.9.1

This should be fixed by https://github.com/docker/docker/issues/17779
Thanks again for reporting @bboreham @awh !

The issue I meant to raise is that the case of the name changed, not how it behaves on an empty value.
So it is impossible to write a script inspecting the value (when it is non-empty) that works against both 1.8 and 1.9.

@bboreham IIUC, https://github.com/docker/docker/pull/17779 should fix that; @calavera can you confirm? Just in case..

My apologies; having tested again I see the problem only hits when the field is empty.
Looks fixed to me in 1.9.1, thanks!

Thanks for testing @bboreham, good to hear it's fixed :+1:


Fix CreatePermission stun response

Description
handleCreatePermissionRequest response should be stun.MethodCreatePermission instead of stun.MethodChannelBind
Reference issue
Fixes #130

@nindolabs thank you for your patch!
Do you mind adding you to the contributor list in README? https://github.com/pion/turn/blob/master/README.md#contributing
https://travis-ci.org/github/pion/turn/jobs/664333894#L286
Please add the following contributors to the README
nindolabs
The command "bash .github/assert-contributors.sh" exited with 1.

Done

This is amazing thank you so much @nindolabs, sorry I didn't get a chance to look at it sooner. Also huge thank you @at-wat for being such a good maintainer. Pion is super lucky to have you involved :)


⚠ Bofrost.at has degraded performance

In d90aa62, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1512 ms

Resolved: Bofrost.at performance has improved in 168bdba.


changed GraphView initiailzation

The refactoring looks good and fixes the big that we were talking about earlier that affected CLEAN Architecture and now we would not need to modify GraphView anymore so SOLID is also maintained. Also make sure it integrates with the Builder properly after the changes.


Update outdated apps

I found the bin/checkver tool and was bored :-)
All apps expect nano (can't find a binary) and rancher-compose (not a stable release) are now up-to-date.

Thank you!


Spelling mistake in error msg: Cannot access hotst at '....'

Looks like just a minor spelling mistake in the error message.

Fixed. The change will appear in this repo soon.


Update governance page "Former ... Members" list

The "Former ... Members" list on the governance page does not include all former Information Management and Support team members.
After a review the history of the Information Management and Support team, I found that there were several former members who did not get added to the "Former ... Members" list. The former members are:

Martin Desruisseaux
Matthew Harris
Richard Hattersley
Mark Hedley
Tanya Reshel

I will create a PR to add these former members.
Also, the "Former ... Members" list appears to be unsorted. Unless there are objections, I propose reordering to match the other lists on the governance page. I will add this as a separate commit to this PR.

Approved by gov-panel in email discussion. Merging PR #453 shortly.


Unable to install Taskboard

Hi Kisa ,
I'm using a Window 7 OS and running the following :
Apache Version:2.4.17
PHP Version:5.6.16
5.7.9 - Port defined for MySQL: 3306
I copied the Taskboard-master folder to wamp64/www/ as per the guide
but i can't seem to get the taskboard running..
Please advise?
Regards,
Zeff

Take a look through #159 and see if that helps. You don't need MySQL, but you do need to make sure PHP is setup to allow SQLite to work.

Hi Kiswa,
i'm not a web developer so i'm really unsure what's the correct step for the installation. i have a wamp64 server installed on my system ( Windows 7). i've refer to the case number you've given me but i checked my files is .dll and the content in the case is .so . i'm unsure if i'm on the right track honestly.
Let me list down my step that i've done and correct me if i got it wrong some where :

Download from your web
Extract the TaskBoard-master to my wamp64/www (webserver folder)
Added Alias taskboard to the webserver
tries to access the taskboard-master local web but doesn't work

I've also tried the following :
Copy the folders and files (you can skip the .hg* stuff) to the location you want to use TaskBoard.
Optional: Minify everything for better performance in production. (Requires that everything but compiler.jar in build/ is executable.)
Open the TaskBoard directory in a terminal
Run ./build/build-all
Open your browser to the location you put TaskBoard and log into the site with the username admin and password admin.
doesn't seem to work in windows os environment.
i'm stuck here . i would like to confirm the following :

if i need to download a window GIT-Gui to copy
PHP Composer to run the php commands?

I can't really help you with this, as I don't know the workings of wamp64 (I use Linux). It seems like you should be able to enable/disable php settings from it though.
As for composer, it's just to install the vendor dependencies. You can just download the vendor files here, extract them (you'll need something like 7-zip on Windows), and copy to your TaskBoard location.

Hello
I also struggled installing taskboard with Wamp too. But here is what I've done so fare :

git clone of repository
composer install  in Terminal (I had already composer installed) : it installed some modules
I got error 500 when trying to login. This was due to a disabled Apache extension. Simply go to your Wamp tray icon > Apache > Modules > Enable the 'expire module' and check that "rewrite module" is also enabled.
Next I got 200 on /api/login but 401 on /boards /current... I noticed that on the 200 request to login, I got extra information with the JSON talking about deprecating feature $HTTP_RAW_POST_DATA . To avoid this message, go to your php.ini file (Wamp icon > php > php.ini) and find the line ; always_populate_raw_post_data = -1 then uncomment the line (remove the coma at the beginning).

Now it works for me.
Hope it helps

Closing due to inactivity.


Data for Chapter 1

table of types, amounts
need to pull from SB, put into cache folder

#15 + #20 + #25


docker image tags and force pull image clarification of behavior in docs

It's not quite clear reading the chronos doc:
https://mesos.github.io/chronos/docs/api.html#adding-a-docker-job

Mesos 0.22.0 added support for forcibly pulling the latest version of your Docker image before launching the task, and this behavior can be enabled in Chronos by adding the forcePullImage boolean to your container configuration.
Chronos will default to not doing a docker pull if the image is already found on the executing node. The alternative approach is to use versions/tags for your images.

Regarding these statements:

What happens if one specifies a tag along with the image and sets forcePullImage true? It pulls for latest version of that tagged image, or ignores the tag and goes for "latest" tag version? I assume one generally uses forcePullImage with no tags specified or use tags and not forcePullImage but not both?
What does the above Chronos "default" statement mean? Does it mean behavior when forcePullImage is false? Or does it mean even with that setting true, if an (older) "latest" tag of the image is found on the executing node locally, then it will not force pull a more recent latest tagged image?

It would be nice to better clarify/reword the statements that a novice beginner understands this "crystal" clear.

I have a similar question. I have multiple tagged docker images. However, chronos seem to be ignoring the tag and always fetching the "latest" tag version. My setup has Mesos 1.0.1 and Chronos 2.4.0.

+1


Removing Objectid from Examples

Removing Objectid from Examples PostFederationSignup and PostAttributeCollection as ObjectId will only be present when API is being called at PreTokenApplicationClaims Step

#sign-off

#sign-off


Incorrect choice of encoding Cyrillic characters resulting in UnicodeDecodeError

Description
I have:

Windows 10
Clean conda Python 3.6 env

I ran
pip install flake8-bandit
and got
Successfully installed GitPython-2.1.11 PyYAML-3.13 bandit-1.5.1 flake8-3.5.0 flake8-bandit-1.0.2 flake8-polyfill-1.0.2 gitdb2-2.0.4 mccabe-0.6.1 pbr-4.3.0 pycodestyle-2.3.1 pyflakes-1.6.0 six-1.11.0 smmap2-2.0.4 stevedore-1.29.0
I have a Python module with the following contents:
# -*- coding: UTF-8 -*-
"""Module docstring."""
print('hello') # э

I run
flake8 C:\Development\Flake8PluginDev\foo.py
an error is produced.
(BanditDebug) C:\Users\user>flake8 C:\Development\Flake8PluginDev\foo.py
Traceback (most recent call last):
  File "c:\anaconda3\envs\banditdebug\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\anaconda3\envs\banditdebug\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Anaconda3\envs\BanditDebug\Scripts\flake8.exe\__main__.py", line 9, in <module>
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\main\cli.py", line 16, in main
    app.run(argv)
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\main\application.py", line 396, in run
    self._run(argv)
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\main\application.py", line 384, in _run
    self.run_checks()
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\main\application.py", line 310, in run_checks
    self.file_checker_manager.run()
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\checker.py", line 321, in run
    self.run_serial()
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\checker.py", line 305, in run_serial
    checker.run_checks()
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\checker.py", line 579, in run_checks
    self.run_ast_checks()
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\checker.py", line 486, in run_ast_checks
    checker = self.run_check(plugin, tree=ast)
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8\checker.py", line 435, in run_check
    return plugin['plugin'](**arguments)
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8_bandit.py", line 35, in __init__
    self._load_source()
  File "c:\anaconda3\envs\banditdebug\lib\site-packages\flake8_bandit.py", line 73, in _load_source
    self.source = f.read()
  File "c:\anaconda3\envs\banditdebug\lib\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x8d in position 68: character maps to <undefined>

To Reproduce
If I have
# -*- coding: UTF-8 -*-
"""Module docstring."""
print('hello') # й

I run
flake8 C:\Development\Flake8PluginDev\foo.py
and get
C:\Development\Flake8PluginDev\foo.py:3:15: E261 at least two spaces before inline comment
It works fine.
Now if we change the Cyrillic # й comment into either the # э, # я, or # с (with all three characters typed with Russian keyboard layout, mind Russian C) and run
flake8 C:\Development\Flake8PluginDev\foo.py
an error is produced.
It seems as bandit has problems with only three Russian letters эяс because when running the flake8 with the comment containing all Russian letters except эяс, it works fine:
# -*- coding: UTF-8 -*-
"""Module docstring."""
print('hello') # йцукенгшщзхъфывапролджё

Expected behavior
Bandit should handle all Cyrillic characters without throwing UnicodeDecodeError error.
Bandit version
(BanditDebug) C:\Users\user>bandit --version
bandit 1.5.1
  python version = 3.6.6 |Anaconda, Inc.| (default, Jun 28 2018, 11:21:07) [MSC v.1900 32 bit (Intel)]

flake8          3.5.0
flake8-bandit   1.0.2

Hi, @tylerwince ! Thanks for writing this plugin. We are using it as a dependency in our linter: https://github.com/wemake-services/wemake-python-styleguide
So, this issue also affects us as well.
I am pretty sure that this error is caused by these lines:
https://github.com/tylerwince/flake8-bandit/blob/master/flake8_bandit.py#L62-L73
What can be done it improve it?

We can just receive lines from flake8 http://flake8.pycqa.org/en/latest/plugin-development/plugin-parameters.html#indicating-desired-data they should be already in the correct format/encoding
We can write some tests that russian letters now work, this will ensure that this won't happen again

@tylerwince Does this sound reasonable to you?
@AlexArcPy Are you willing to provide a PR? This would be awesome!

@sobolevn , I've tested that changing line 72
with open(self.filename) as f:
to
with io.open(self.filename, 'r', encoding='utf-8') as f:
makes it working with utf-8 files (both with magic encoding and without). Tested on both 2.7 and 3.6.
Nikita, did you mean to write a test in the wemake-python-styleguide repo?

@AlexArcPy no, I meant writing tests in this repo. Currently it does not have any 🙁
The problem with with io.open(self.filename, 'r', encoding='utf-8') as f: is that you are hardcoding an encoding. Encoding might differ for different locales and users. So, we should guess it instead.
flake8 does this for you, however. So, there is no need to do anything except just receiving a new parameter in the constructor.

Cheers @sobolevn and @AlexArcPy -- Thanks for the dialog above.
I would definitely welcome a PR for some tests and can get some CI stuff going with TravisCI once we they exist.
As for the encoding stuff - I have been meaning to change the way this works for a while. Unfortunately, different editors require the plugin to work differently to lint the file. I haven't had the time to go back and refactor lines https://github.com/tylerwince/flake8-bandit/blob/master/flake8_bandit.py#L64-69 but that is where the issue lies.
If one of you guys has a good idea on how to remove the requirement of the additional file, I'd gladly accept a PR, otherwise I will take a crack at it this weekend.
Thanks for using the plugin! Glad it is useful

@AlexArcPy @sobolevn I apologize for the delay on this. I am just getting around to working on this project. I have updated the way we parse the file for the next version (see branch https://github.com/tylerwince/flake8-bandit/tree/v2.0). It appears to have fixed the original issue on this so I suggest we close it and I am going to open up a new issue for testing.
Please reopen if you are still experiencing the same issue.


adding tyron.alwaysdata.net

This PR is:

[x] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not an ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: tyron.alwaysdata.net
  url: https://tyron.alwaysdata.net/
  size: 157
  last_checked: 2023-11-05

GTMetrix Report: https://gtmetrix.com/reports/tyron.alwaysdata.net/TNUooFXj/

Thanks!

np


Merge branch 'alpha'

Description
Merge to trigger the release
Additional context

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

2.0.0
GitHub release

Your semantic-release bot :package::rocket:


Support declaration maps

Clear and concise description of the problem
As a developer using declaration files, I want to jump directly to the ts source, rather than a declaration file.
Suggested solution
We could have an option to enable declaration maps.
Alternative
Don't support it, keep jumping to declarations.
Additional context
tsc supports it and oxc supports it (oxc-project/oxc#4313), but swc doesn't seem to support it.
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Thanks!


ResourceGroovyMethods.splitEachLine closure params should be List

ResourceGroovyMethods.splitEachLine closure params should be List iso String[]

Thanks for spotting the anomaly with RGM#splitEachLine. I created an issue here:
https://issues.apache.org/jira/browse/GROOVY-7440
And ended up with a slightly different solution:
https://github.com/apache/incubator-groovy/commit/5b521788


Allow the client to control the action on Escape Key.

Yep, sensible suggestion. Changed.

@edgifyjp thanks :+1:


The QAsm Language support server crashed 5 times in the last 3 minutes. The server will not be restarted.

Informations

VSCode version: 1.38.1
Qiskit [Terra / Aqua / Ignis/Aer] version: 0.9.0/0.6.0/0.2.0/0.3.0
Qiskit Version:0.12.0
Qiskit VSCode Extension version:0.5.5
Operating system: Windows 10 Home 1903 Build 18362.356
Python Environment /Version:Anaconda/ 3.7.3

What is the current behavior?
The extension is not working at all.
Steps to reproduce the problem
After VS Code is launched and any qasm file is opened the notification occurs saying "The QAsm Language support server crashed 5 times in the last 3 minutes. The server will not be started."

The extension has changed substantially with after the 1st commit in years (and the un-archive of the repo) https://github.com/Qiskit/qiskit-code-assistant-vscode/commit/7a248ac7f8502ee527f442c33cdb1697e00297c4
This issue is not relevant anymore, closing. Thanks


Enable running youki with cargo

Since youki is now published on crates.io (#586) it is possible to run cargo install youki to install youki into cargo's bin dir, but since it is called youki not cargo-youki one is unable to run it as a cargo sub command cargo youki ...

Hey @Will-Shanks , as mentioned in the PR, let us discuss here the reasoning behind this.
As it stands, youki is a low level runtime to be used either by some higher level runtime, such as docker etc, or  if one wants, then directly by user. In that way, as you have mentioned, user can install youki through cargo as it is available on crates.io, and use it as a command,eg youki -h.
My understanding is that cargo-* commands are usually for things that are directly related to cargo, or setup/compilation of rust projects. In that sense, youki should not become a cargo sub-command, as it is not related to either of those.
Further, doing this change will likely require a breaking release, as we are changing binary name, and would not make sense where only youki is used, eg, directly downloaded as a binary , without having cargo on system.
Can you explain your reasoning behind  this request so that we can understand it better and come up with a possible solution?
Thanks :)

The reason is it makes youki simpler for me to use :) For reasons I don't think are worth getting into,  I am unable to add my cargo bin dir to my path, so being able to call youki through cargo gets rid of some headaches for me. I figured it's a small, harmless change  that makes my life easier, and could be useful for others too.
This would not be a breaking change, if you look at my PR, cargo create two binaries, one called youki and another called cargo-youki so this would have no effect on any existing workflows that expect a youki binary,  similar to what flamegraph does.
I wasn't expecting so much push back, is there a reason this behavior is undesirable that I'm missing? As far as I can tell all this does is add convenience without any detrimental effect.

Thanks for your comment. The question is how many people will use this. It does not matter whether it will affect the current behavior. In OSS, once a feature is added, the maintainer, not the person who created the feature, has to maintain and take it into account all the time. Therefore, we need to consider how many people will find this useful and convenient.

Fair enough, I guess I'll look at taking my project in a different direction.
Thanks!


can read max. 31 bytes from FIFO

Hi @BoschSensortec,
im using the following code snippet to clear my fifo buffer;
    do
    {
        bmi160_get_regs(0x22, trunklen, 2, &bmi.sensor);
        readlen = trunklen[1] << 8 | trunklen[0];
        std::cout << "bytes in fifo: " << readlen << std::endl;

        bmi160_get_regs(0x24, trunkdata, 31, &bmi.sensor);
        //clear the fifo
        runs ++;
    }
    while (readlen != 0 && runs < 1000);

I experienced a problem with this.
I can't read more than 31 bytes at once from the fifo even if it contains 100+ bytes.
Why is that?
My Fifo is configured like the following:
    // Modify the FIFO buffer instance and link to the device instance
    struct bmi160_fifo_frame fifo_frame;
    fifo_frame.data = fifo_buff;
    fifo_frame.length = 7;
    dev->fifo = &fifo_frame;
    uint16_t index = 0;

why can't i read all bytes in fifo at once?
DEBUG output with reading 31 bytes per read:
bytes in fifo: 721
bytes in fifo: 693
bytes in fifo: 665
bytes in fifo: 637
bytes in fifo: 616
bytes in fifo: 588
bytes in fifo: 560
bytes in fifo: 532
bytes in fifo: 511
bytes in fifo: 483
bytes in fifo: 455
bytes in fifo: 427
bytes in fifo: 399
bytes in fifo: 371
bytes in fifo: 343
bytes in fifo: 315
bytes in fifo: 294
bytes in fifo: 266
bytes in fifo: 238
bytes in fifo: 210
bytes in fifo: 182
bytes in fifo: 154
bytes in fifo: 126
bytes in fifo: 98
bytes in fifo: 70
bytes in fifo: 42
bytes in fifo: 14
bytes in fifo: 0
runs to clear: 28

Debug output reading > 31 bytes per read:
bytes in fifo: 504
bytes in fifo: 504
bytes in fifo: 511
bytes in fifo: 511
bytes in fifo: 518

....

bytes in fifo: 980
bytes in fifo: 980
bytes in fifo: 987
bytes in fifo: 987
bytes in fifo: 994
bytes in fifo: 994
bytes in fifo: 1001
bytes in fifo: 1001
bytes in fifo: 1003

@T721,
It depends on your hardware abstraction layer. In some cases, like that of Arduino, the maximum read size is 32bytes.


Update GPL license classifier

According to https://pypi.python.org/pypi?%3Aaction=list_classifiers the name for GPL license classifier is incomplete and pypi will not accept the package.

Thanks @glogiotatidis !


Gherkin update

Merge feature-introduction into gherkin-reference
Update links
Merge info from https://github.com/cucumber/cucumber/pull/68 into introduction

Otherwise looks great.


cabana: make head section movable

Kazam_screencast_00050.webm
@pd0wm Do you think this is okay?

yes, this is what I had in mind!
Maybe also put a comment on line of the restoreHeaderState that it's put first to make sure it's not overriding anything that comes after.


Drop XBM support

We currently use it to generate empty blocks and walls. Use XPM instead.

It will also make #138 easier

If we gonna use XPM instead of XBM, i think it doesn't make sense to generate wall tiles on the fly. I think we can simply write a script that generates the wall tiles offline and puts them to the sprites directory with some meta about bits for bars and corners.

I think I wasted everyone's time... It's actually a couple of changes in the already existing code and the XPM wall tiles are ready. :(


Bug: Seer preview should close when selecting a different file via the mouse

Description
Seer has a quick preview feature that I often use in my work. Its function is as follows: when any file/folder in the file manager is selected and the spacebar is pressed, it allows browsing the contents without directly opening it. This is particularly convenient for someone like me who works with numerous Word and Excel documents.
Files is my favorite file manager. However, when combining the two, there's a rather troublesome issue: once the quick preview feature is activated for the first time by pressing the spacebar, selecting any file thereafter or returning to the previous level folder will automatically invoke the quick preview feature, unless Seer is closed in the background to stop it. It's as if once the spacebar is pressed for the first time, every subsequent selection of any file/folder with the mouse automatically adds the quick preview feature corresponding to the spacebar. This is quite frustrating. Both applications are very useful, and I am reluctant to give up either.
Through testing, this issue does not occur with Seer and the default file manager of Windows. Currently, I am not sure whether the issue lies with Files or Seer, so I have submitted feedback for both applications, hoping to get help from the developers. I am very grateful for the excellent software you have developed and hope it can be further improved!
Steps To Reproduce

When Seer and Files are running, but Seer has not yet been used for the first time (before the first press of the spacebar), clicking to select any file with the mouse does not present any issues.

2024-03-15 17-22-13.mkv.zip

When Seer is running, pressing the spacebar on any file to use Seer's quick preview feature will lead to the automatic invocation of the quick preview feature upon any subsequent mouse selection of any file. It's as though every mouse click to select a file automatically presses the spacebar.

2024-03-15 17-23-34.mkv.zip
Requirements
hope to prevent the automatic invocation of Seer's quick preview feature when using Files and Seer together, so that the feature is only used when I actively press the spacebar according to my needs.
Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3296
Log File
debug.log

What happens if you press the space bar again?

What happens if you press the space bar again?

The Quick Preview Window will disappear.

@yaira2  hi, Seer hasn't been updated since last June.

Let me know if there is anything I can do, @yaira2

What happens if you press the space bar again?

The Quick Preview Window will disappear.

Is this a good workaround for now?

Let me know if there is anything I can do, @yaira2

@ccseer I think the fix can be from our side but if pressing space does the trick, I think it might be a good workaround for now.

What happens if you press the space bar again?

The Quick Preview Window will disappear.

Is this a good workaround for now?

I've updated to the latest version of Files, which is <IP_ADDRESS>, and It appears to still exist.:'(

We haven't made any changes to fix this. Our current intention is for users to use space or esc but we'll keep this on our project board as it's reasonable feedback.

We haven't made any changes to fix this. Our current intention is for users to use space or esc but we'll keep this on our project board as it's reasonable feedback.

OK! Thanks for your reply and care!!  :)

wait, I guess we need more info.
Seer preview should close when selecting a different file via the mouse is expected. And if user doesn't like it, there is an option to turn it off in Seer's Settings.

@Nronless would you please hlep comfirm

wait, I guess we need more info. Seer preview should close when selecting a different file via the mouse is expected. And if user doesn't like it, there is an option to turn it off in Seer's Settings.

I've looked through Seer's setting, but counldn't find this item anyhow. Maybe it's because of different translation? Let me show u all the setting windows to locate it.

@Nronless  The sixth tab, the second group Others, then Track selected file in Explorer.

@Nronless  you can close the issue if it's solved.

I've tried turning off this option, but still exists the same issue.

@ccseer do you have docs on how we can detect that setting?

yes, it has a priority

UWP

%USERPROFILE%/Documents/Seer/uwp.ini
%USERPROFILE%/appdata/Local/Corey/Seer/uwp.ini

Win32:

%USERPROFILE%/Documents/Seer/config.ini
REG:   "\HKEY_CURRENT_USER\Software\Corey\Seer"

The key is:
general - tracking_file

%USERPROFILE%/appdata/Local/Corey/Seer/uwp.ini

This path doesn't seem to work

the ini files under Documents are created by user.
in some rare cases, the default path doesn't work, so I provided a solution for these user to have a new Settings path

@Nronless this will be included in the next release.


ItemStackBuilder does not accept negative quantities

ItemStacks are allowed to have negative quantities, and Inventory#query(ItemStack) relies on quantity being -1 to ignore stack size differences.
ItemStack itself does not set a lower limit, however its builder does. This should be change to make like a bit easier when querying inventories.

Actually, given that "item stack with negative size" is actually a bit of a weird concept, and - as you identified - only certain negative values have meaning, effectively making those values semaphores. It would seem to make more sense for the builder to support setting the semaphore via an ItemStackBuilder::anySize() or similar setter which internally sets the quantity to -1, but decouples this detail from the consumer code.

An issue was made on spongecommon regarding negative stack sizes
SpongePowered/SpongeCommon#150


Summaries for tests generated by fuzzer are generated in the summary module #597

Description

Moved NameSuggesting API to summaries module
Shifted the stage of setting names/display names for fuzzing later, to the Summarization stage

Fixes #597
Type of Change

Refactoring (typos and non-functional changes)

How Has This Been Tested?
Automated Testing
TODO: add unit-tests for fuzzing tests and its names too
Checklist (remove irrelevant options):
This is the author self-check list

[x] The change followed the style guidelines of the UTBot project
[x] Self-review of the code is passed
[x] The change contains enough commentaries, particularly in hard-to-understand areas
[x] New documentation is provided or existed one is altered
[x] No new warnings
[ ] New tests have been added
[x] All tests pass locally with my changes

@Markoutte there is an open question how to pass information required to the Name generation outside the Fuzzer module.
I used UtExecution for that (adding two fields with types from Fuzzing module), but faced with some issues

class UtExecution is defined in Api.kt in utbot-framework-api and this module doesn't import fuzzer module (I don't know could I import it here?)
I could not inherit UtExecution (make FuzzingUtExecution and place it in utbot-framework or somewhere else) because it's a data class (the inheritance of data classes if forbidden in Kotlin). Could we convert it to usual class?

So I decided to keep something with types Any and fill these fields with specific types later in Java 4 style. I don't see huge objection not to follow this way

I think we can do this trick like that: move the knowledge about data origin into the API or even another high-level module and unbind it from the fuzzer. I wouldn't move the knowledge about fuzzing (or any another possible way we generate names) to the summaries. But we can make a contract with a flex hierarchy of classes somewhere upper in module hierarchy (for example, one for those which have jimple body or which provide some internal knowledge about data origin).
I think, maybe, it is good idea to have something like "summary-api" or a package in framework-api for summaries. It gives us a way to choose how to fill this data for that summary. No summaries at all if we don't fill any.
Moreover it helps to merge the last 4 fields of UtExecution (createdBy, summary, testMethodName, displayName) into one object to pass further.
What do you think?

@Markoutte I'd like your proposal, and ready to try to implement this approach.
But I have an alternative suggestion: I have some tickets related to javadoc generation and clusterization for fuzzing and want to progress with them in parallel.
Could we merge this as it's implemented and start with depending tickets and in parallel start the discussion with team about changes in API?

@Markoutte @denis-fokin please have a look, I've implemented the proposed hierarchy of classes and handle separately each case where Symbolic Execution and Fuzzed Execution have different flows.
Also, I checked this manually, fuzzing and symbolic works well together

@Damtev thanks for review, I solved the majority of the questions @EgorkaKulikov please have a look


How to redeploy this when using -basic or -standard version of solution

i was able to change ui on local machine - however it seems the web api stays the same - what should i do change the web ui  only after this 'customize the UI' tutorial?

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

ID: 228674e5-ed5a-2343-c1c9-10571a1e9062
Version Independent ID: 2bbe8dfe-bc82-a053-7bcc-91192e6bb1d6
Content: Customize the Remote Monitoring solution UI - Azure
Content Source: articles/iot-accelerators/iot-accelerators-remote-monitoring-customize.md
Service: iot-accelerators
GitHub Login: @dominicbetts
Microsoft Alias: dobett

@developersenakim thank you for your feedback. Can you confirm if you were able to connect to the Virtual Machine using SSH? And run cd /app + sudo ./start.sh --unsafe ?

Also make sure that you set the .env file accordingly

Thank you!

I was able to follow all the steps.

I was able to connect to VM using SSH.
I allowed local UX to connect.
I changed.env file.
It still does not update the AZure service url. what i see on local machine and Azure web service URL is diffrent.

I added example panel - which does not show.

downloaded : azure-iot-pcs-remote-monitoring-dotnet
and the subfolders related
working. close plz

@developersenakim Thanks now we are closing this thread.


COMUNICA: browser-side bundles are too heavy on crypto-related code

As shown by the following screenshot of Webpack's bundle analyzer, browser-side bundles include a lot of crypto-related code due to @comunica/actor-abstract-bindings-hash/lib/AbstractFilterHash.js making use of the crypto module of Node.js.
By limiting the amount of hashes that can be used to just one or two we could refactor that package to depend on pure-js, npm-based implementations and get rid of almost a third of the bundle size. We could do this either by making a PR to Comunica itself (upstream) or coming up with our own replacement @comunica/the actor-abstract-bindings-hash.

We might address this upstream with https://github.com/comunica/comunica/pull/752 (draft, WIP).

Waiting on https://github.com/DefinitelyTyped/DefinitelyTyped/pull/49123

We did end up addressing this upstream. For posterity, we started with comunica alone but ended up working across both comunica and sparqlee.
Waiting on:

~~https://github.com/comunica/comunica/pull/752~~ merged!
~~https://github.com/comunica/sparqlee/pull/78~~ merged!
https://github.com/comunica/comunica/pull/757
new release of comunica to NPM (expected early next week)

Released in v7.0.1-beta.2!


Packer Username Does Not Have Sudo Rights Without Password in CentOS

ARM provisions a user (packer), and uses this username for all provisioning operations.  This user does not have sudo rights without a password in the CentOS/OpenLogic images and it causes provisioning to fail.
We need to handle this so customer can be successful with these images.

I believe this issue is address with PR #202.

I'm confused about whether you consider this issue resolved.  We would like to be able to create an image with a user that authenticates using a key only, with no password, and then can automatically use sudo with no password.  This seems to be what happens when I create an image using the GUI.  I can now use sudo with packer by piping the password into the sudo command, but I still have to have a password.

Yes, I think it is resolved, but please correct me as appropriate.  I updated the examples based on how the various distros expect sudo to be executed.  The Packer's provisioner's execute command should look like this for CentOS (don't forget the other configuration pieces).
"provisioners": [
 {
  "execute_command": "echo '{{user `ssh_pass`}}' | {{ .Vars }} sudo -S -E sh '{{ .Path }}'",
 }
]


Add complex number support to linalg.slogdet

This PR

adds complex number support to linalg.slogdet.
defines the sign of the determinant in agreement with https://github.com/data-apis/array-api/pull/556. In contrast to its sign function, NumPy uses the definition $\operatorname{sign}(\det x) = \frac{\det x}{|\det x|}$ to compute the sign of the determinant. At the time of this PR, PyTorch refers to the "sign" as the angle (which I take to mean the argument of the determinant); however, I don't believe this is correct, as, via polar decomposition, $\det A = |\det A| e^{\theta j}$. Given the expected equality that $\det A = \operatorname{sign}(\det A) \cdot e^{\ln |\det A|} =  |\det A| \operatorname{sign}(\det A)$, we see that what should be returned is $e^{\theta j}$, not simply the argument $\theta$. While the PyTorch docs state that the angle is returned, what is actually returned is $\frac{\det x}{|\det x|}$.
updates the allowed data types from real-valued floating-point to any floating-point data type.
requires that the sign array be the same data type as the input array and logabsdet be unconditionally real-valued.
adds an extended description describing the operation.
makes explicit that, for matrix stacks, a solution must be computed for each matrix in the stack.

...  we see that what should be returned is $e^{\theta j}$, not simply the argument $\theta$. While the PyTorch docs state that the angle is returned, what is actually returned is $\frac{\det x}{|\det x|}$.

I think that that's mostly a terminology thing? The implementations are consistent:
>>> x = np.array([[0.1+0.9j, 0.5-0.5j], [1, 2j]])
>>> np.linalg.slogdet(x)
((-0.9566738804288585+0.29116161578269606j), 0.877201841342143)
>>> torch.linalg.slogdet(torch.as_tensor(x))
torch.return_types.linalg_slogdet(
sign=tensor(-0.9567+0.2912j, dtype=torch.complex128),
logabsdet=tensor(0.8772, dtype=torch.float64))

If there's something to clarify or improve in the PyTorch wording in the slogdet description, that's of course useful. @lezcano since you wrote those docs, you may want to have a look at the comment in the PR description here.

I agree that the term angle is not correct. The quantity that's returned in PyTorch is indeed the sign, and the rest of the docs and the implementation are consistent with this. I'll fix that in the PyTorch docs soon.


copy_plugin in frontend leads to a 302 redirect to http:// instad of https://

CMS: 3.1
Django: 1.6
Description
We have an SSL secured Site running the CMS. We couldnt copy plugins in the frondend because of the  response which is returned from django cms which contained a 302 Moved Temporarily and a unsecure http:// adress
The error
Browser complains about mixed content (https / http) and stops processing the response
The request
Request URL:https://xxxxxxx.de/admin/cms/page/clear-placeholder/1908/
Request Method:POST
The response contained the following location
Status Code:302 Moved Temporarily
Location: http://xxxxxxx.de/admin/
The intermediate solution (for anyone interested)
Add the following to your location in nginx (or similar for anything else than nginx)
proxy_set_header   X-Forwarded-Proto https;

@SachaMPS How is setup the Site object in the admin?

Do you mean this:?

Yes
I double checked, anyway, and the path to copy the plugins is domainless:
<script>
(function($) {
// CMS.$ will be passed for $
$(document).ready(function () {
new CMS.Plugin('cms_plugin-13233', {
'type': 'plugin',
'page_language': 'it',
'placeholder_id': '2377',
'plugin_name': 'Blocco Immagine',
'plugin_type': 'NphMediaBlock',
'plugin_id': '13233',
'plugin_language': 'en',
'plugin_parent': 'None',
'plugin_order': '',
'plugin_breadcrumb': [{"url": "/admin/cms/page/edit-plugin/13233/", "title": "Blocco Immagine"}],
'plugin_restriction': [],
'onClose': false,
'urls': {
'add_plugin': '/admin/cms/page/add-plugin/}',
'edit_plugin': '/admin/cms/page/edit-plugin/13233/',
'move_plugin': '/admin/cms/page/move-plugin/',
'delete_plugin': '/admin/cms/page/delete-plugin/13233/',
'copy_plugin': '/admin/cms/page/copy-plugins/'
}

});
});
})(CMS.$);
</script>

So the browser should just use the https path. Is there any file loaded from non-https addresses?
I can't reproduce with the same setup as yours

@yakky The problem is not the outgoing request from the browser... Its the response The request uses an https:// path but the response contains the 302 status code and a location to a none http:// path.
(I used firefox dev tools because they gave me the most insight what happened)
The request which produced this was:
https://xxxxxxx.de/admin/cms/page/clear-placeholder/1908/

This is the response I get on an example https website:
Status: 302
Cache-Control: max-age=0
Connection: keep-alive
Content-Language: it
Content-Type: text/html; charset=utf-8
Date: Thu, 25 Jun 2015 16:46:24 GMT
Expires: Thu, 25 Jun 2015 16:46:24 GMT
Last-Modified: Thu, 25 Jun 2015 16:46:24 GMT
Location: https://example.com/admin/
Server: nginx
Set-Cookie: messages="...."; Path=/
Transfer-Encoding: chunked
Vary: Accept-Language, Cookie
X-Frame-Options: SAMEORIGIN

Note the Location: https://example.com/admin/
Any special middleware which may prevent Django to "understand" it's an https request? Maybe the frontend proxy turns the request to plain http to the internal applications server?

@yakky I will again look into our setup... If you're getting https it seems as it must be part of our setup and not the cms itself. Anyhow its weird because we even redirect immediately from http to https on every page load and there didn't occure any other similar errors.
Think you can close this issue and i will come back if i found somehting more specific

Using i18n_url_patterns is recommended, but I cannot see how this could affect the behavior (btw: the above website is a multi-site setup with only one of the website under https, both using USE_I18N and i18n_url_patterns)

@SachaMPS Is it it possible that your browser has remembered a permanent redirect, and is acting on that? I have had some mysterious browser/HTTP interactions of this kind that produced some very odd behaviour.

@evildmp No, i am pretty sure not... To get more details on the request i switched to firefox and saw the problem directly also in Firefox. And i actually never use firefox so chances for a caching problem don't seem very probable. If you want i can give you all the details about our setup, maybe s.th. catches your eye then.

@SachaMPS that would help,, thanks

Hi @SachaMPS have you got any further with this? We have a similar issue #4432, and again it's hard see how it could be a CMS issue rather than a server configuration matter.

@evildmp Sorry, we since then lived with our solution to solve this via nginx.

@SachaMPS thanks for the feedback. Closing this, then

We faced a similar predicament, we have a nginx proxy with SSL on the gateway, while in the backend we have nginx (no SSL) with uwsgi. Some of Django-CMS admin edits just stalled after submission, notably the "unpublish", "publish", "resolve" URLs.
In the end we did 2 things:

HTTPS NGINX PROXY:

sites-enabled/gateway.conf:
location / {
    proxy_pass http://LAN_ADDRESS_RUNNING_NGINX_UWSGI/;
    proxy_set_header Host $http_host;   # required for django 1.5
    proxy_set_header X-Real-IP $remote_addr;
    proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    # HTTPS -------------- combine following with django settings:
    # SECURE_PROXY_SSL_HEADER = ('HTTP_X_FORWARDED_PROTO', 'https')
    proxy_set_header X-Forwarded-Proto https;       # WORKS: 2016.04.04
    ....

DJANGO settings.py

SECURE_PROXY_SSL_HEADER = ('HTTP_X_FORWARDED_PROTO', 'https')
And everything works now
Reference:
http://stackoverflow.com/questions/9097404/django-is-redirecting-from-https-to-http

Accordingly SECURE_PROXY_SSL_HEADER should be used only if your SSL proxy strips all X-Forwarded-Proto header in incoming request. So to add to @rafael-queo proxy_set_header X-Forwarded-Proto https should be set to proxy_set_header X-Forwarded-Proto $scheme when using SECURE_PROXY_SSL_HEADER.
https://docs.djangoproject.com/en/1.9/ref/settings/#s-secure-proxy-ssl-header

For reference only: https://github.com/divio/django-cms/issues/4432#issuecomment-231711675


Improve error handling by not considering "not found" an error

Currently, the examles and documentation generally ignores error from dotenvy.
    let _ = dotenvy::dotenv(); // either like this
    dotenvy::dotenv().ok(); // or like this

This is generally bad practice, and makes debugging of syntax errors in .env files harder than it needs to be.  The general if is of course to use either .unwrap() or ? in all documentation examples.  But there is a problem with this; checking for errors makes it obvious that there actually are errors when there should not be, since the dotenv() function returns an error if there is not .env file.
I think having a syntax error in the .env file is a problem I want to know if I have, and not beeing allowed to read my .env file is a problem I want to know if I have.  But I don't have a .env file in production (where I insert the environment by other means), so the dotenv() call must not terminate my program in that case.
Currently, I use a simple wrapper:
fn dotenv() -> dotenvy::Result<()> {
    match dotenvy::dotenv() {
        Ok(_) => Ok(()),
        Err(e) if e.not_found() => Ok(()),
        Err(e) => Err(e),
    }
}

But it would be nicer to fix the dotenvy::dotenv() method directly.  It currently returns Result<PathBuf >.  I agree that getting the used path (if any) can be useful, so I think it should return Result<Option<PathBuf>>, where a successfully parsed file is signified by Ok(Some(path)), and not having a .env file is signified by Ok(None), and only actual failure to read or parse an existing file yields an Err(...).
If you agree to this, I'd be willing to write a pull request for it.

Addendum: If there is an error, I think that error should also include the path of the attempted file.

Thank you for your detailed examination. We are currently discussing this issue here in #39.
I like your solution to use a Result<Option<PathBuf>> as the return type. However, as you'll see in the discussion (and in some issues/PRs), there are subsections of users who consider the lack of a .env file to be an error. I'd be very interested in your thoughts on providing for both use-cases. Currently we are moving towards providing more extensive documentation to show users how to better handle/ignore errors.

Oh, I did some searching, but aparently for the wrong keywords, so I missed #39 .  Sorry.


Support for WGL_EXT_framebuffer_sRGB

Hi -- despite having a very recent NVIDIA GPU and driver, there is no support for WGL_ARB_framebuffer_sRGB, only WGL_EXT_framebuffer_sRGB.  As such, I cannot create an sRGB backbuffer.  Adding support looks pretty straightforward, I'd be happy to do it if it would be accepted.  I'd also potentially say that it's extremely likely all drivers that support WGL_EXT_framebuffer_sRGB will support WGL_ARB_framebuffer_sRGB, so just swapping the extension would be simpler.

That sounds very strange. Have you updated your drivers?

Are you creating an OpenGL ES context?

Yes, and no.

Fixed with 04c057238b422785b487d32b5d4865511b20f2f7.


DSND-2599: Fix bug where action date wasn't on AssociatedFiling document object

Describe the changes
The action date field wasn't on the FilingHistoryAssociatedFilings object as a field and so was not being mapped in both the GET response mapper and also the AssociatedFilingChildMapper for the upsert.
The rest of this ticket will be implemented in a separate PR.
Related Jira tickets
DSND-2599
Developer check list
General

[ ] Is the PR as small as it can be?
[ ] Has the Jira ticket been updated with any relevant comments?
[ ] Is the README up to date?
[ ] Has the POM been updated for the latest versions of dependencies?
[ ] Has the code been double-checked against main?
[ ] Does the code adhere standards in this repository, including SonarQube checks?

Testing

[ ] Do the code changes have unit and integration tests with code coverage > 80%?
[ ] Has the code been tested locally and/or passed the API Karate tests on Tilt?
[ ] Have mandatory manual test cases been run?
[ ] Are extra Karate tests required?
[ ] Are all the test data resources up to date?

Release preparation
Where possible, add links to the respective Jira tickets when an item is checked

[ ] Have these changes been included in a release ticket?
[ ] Are changes required to the docker-chs-development deployment or test data?
[ ] Are any changes required to the environment added to deployment repositories?

Notes to the tester
Add any testing hints or instructions here.

CI: No security warnings found

CI: No security warnings found


Add batch rendering API for tri/quad

While interesting, this is either to be asked directly upstream in sokol or by transferring it to Katabasis.

While interesting, this is either to be asked directly upstream in sokol or by transferring it to Katabasis.


Is lookup function working?

Basic usage of lookup function is no longer returning an object.
namespace: {{ (lookup "v1" "Namespace" "" "") }}

renders as empty map:
namespace: map[]

Used to return results in 3.1.  Has this changed?

Output of helm version: 3.4.x and 3.5.1
version.BuildInfo{Version:"v3.5.1", GitCommit:"32c22239423b3b4ba6706d450bd044baffdcf9e6", GitTreeState:"clean", GoVersion:"go1.15.7"}
Output of kubectl version: 1.18
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.9", GitCommit:"94f372e501c973a7fa9eb40ec9ebd2fe7ca69848", GitTreeState:"clean", BuildDate:"2020-09-16T13:56:40Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.15", GitCommit:"73dd5c840662bb066a146d0871216333181f4b64", GitTreeState:"clean", BuildDate:"2021-01-13T13:14:05Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):  AKS

I just tested the lookup function and it is working.
How are you using the lookup function? It only works when using commands that interact with a cluster. For example, it does not work with helm template which is designed to work without being connected to a cluster.

Ah thanks that's what it is!  Tried helm upgrade with --dry-run --debug


Cursor doesn't move when using arrow keys

Most likely fixed in the latest Flutter Beta

Leaving this here since I think this is the same problem: cursor doesn't move with spacebar swipe actions on keyboards with that feature.


Update Go to 1.18.3, golangci-lint to 1.46.2

@tklauser what do you think about using go-version-file? it's supposed to find the latest patch version based on go.mod file.
https://github.com/actions/setup-go#getting-go-version-from-the-gomod-file


Beginning Subject Comparision

Begin cross subject comparison, looking into R^2, RSS, \betas and other statistics to compare observations in a single person to between subject comparison (Goal, initial effort competition by Monday)

Do believe the initial phase is over


dgram module

This pr adds the dgram module bindings targeting node 12
https://nodejs.org/docs/latest-v12.x/api/dgram.html

[x] Class: dgram.Socket

[x] socket.addMembership(multicastAddress[, multicastInterface])
[x] socket.addSourceSpecificMembership(sourceAddress, groupAddress[, multicastInterface])
[x] socket.address()
[x] socket.bind([port][, address][, callback])
[x] socket.bind(options[, callback])
[x] socket.close([callback])
[x] socket.connect(port[, address][, callback])
[x] socket.disconnect()
[x] socket.dropMembership(multicastAddress[, multicastInterface])
[x] socket.dropSourceSpecificMembership(sourceAddress, groupAddress[, multicastInterface])
[x] socket.getRecvBufferSize()
[x] socket.getSendBufferSize()
[x] socket.ref()
[x] socket.remoteAddress()
[x] socket.send(msg[, offset, length][, port][, address][, callback])
[x] socket.setMulticastLoopback(flag)
[x] socket.setMulticastTTL(ttl)
[x] socket.setRecvBufferSize(size)
[x] socket.setSendBufferSize(size)
[x] socket.setTTL(ttl)
[x] socket.unref()

[x] dgram module functions

[x] dgram.createSocket(options[, callback])
[x] dgram.createSocket(type[, callback])

Let me know if there's anything else to add/modify to match any guidelines I might have missed

This is great @AngelMunoz, thanks a lot! 👏 👏 👏 I will push a new version with the new module.

no problem, thanks as well
I might start pushing some updates to some modules as well targeting 12.x since that's the next LTS to be supported for a few years, but as always with OSS stuff no promises 😁

That'd be great! Sure, whenever you've time, any help is highly appreciated!


Yarn bootstrap does not work for newly cloned repos

Description
The following check fails, after cloning I get fs.lstatSync(dstDir).isSymbolicLink() is false.
https://github.com/Shopify/react-native-skia/blob/ec102f54af9c1fa8bd9ea959c51e0e3711577a6b/package/scripts/install-npm.js#L11
Then I get:
Error: EEXIST: file already exists, symlink '../../../cpp/api' -> '~/Developer/react-native-skia/package/android/cpp/api'

Maybe remove current link and set it up again with something like if (fs.existsSync(dstDir)) fs.unlinkSync(dstDir); and not check the symlinks?
It also may be my git config, that does not set up symlinks correctly after checkout.
Version
latest
Steps to reproduce

Clone current repo into new folder.
Run yarn bootstrap

Snack, code example, screenshot, or link to a repository
none

I'm not able to reproduce this, on what kind of system I you trying it on Windows?

it does look like a duplicate of #780

@chrfalch should it be if (!fs.existsSync(dstDir) && !fs.lstatSync(dstDir).isSymbolicLink()) { simply?

Ah yes, was reported in #780. I'm on mac.
My hunch was correct. It's https://git-scm.com/docs/git-config#Documentation/git-config.txt-coresymlinks
To fix run: git config --global core.symlinks true

Ah yes, was reported in #780. I'm on mac.
My hunch was correct. It's https://git-scm.com/docs/git-config#Documentation/git-config.txt-coresymlinks
To fix run: git config --global core.symlinks true

+1 @wcandillon
This should be properly addressed.
I still get the same errors using pnpm

Hi @HaakonSvane - If you have suggestions to how we can solve it nicely we'd be super happy with a PR or suggestion :)

@chrfalch I think we might have a wrong conditional statement there but maybe I am misreading it.
Shouldn't be an "and" and not a "or"?

Ah! Sounds like the issue, yes :) :) Nice catch, @wcandillon - then you don't need to look into it, @HaakonSvane :)


infinite loop for pytorch and numpy standalone build

Nuitka: 0.6.8rc7
Python: 3.7.4 (default, Aug 13 2019, 20:35:49)
Executable: /root/lilw/anaconda3/envs/pytorch/bin/python
OS: Linux
Arch: x86_64
Cmd:
python -m nuitka --show-progress --plugin-enable=numpy --plugin-enable=torch --show-scons -j 20  --follow-imports   --nofollow-import-to=matplotlib  --nofollow-import-to=sklearn --standalone main.py
Result:
Infinite print...
...
Nuitka:INFO:Optimizing module '_struct', 10 more modules to go after that.
Nuitka:INFO:Optimizing module '_opcode', 9 more modules to go after that.
Nuitka:INFO:Optimizing module '_multibytecodec', 8 more modules to go after that.
Nuitka:INFO:Optimizing module '_heapq', 7 more modules to go after that.
Nuitka:INFO:Optimizing module '_codecs_tw', 6 more modules to go after that.
Nuitka:INFO:Optimizing module '_codecs_kr', 5 more modules to go after that.
Nuitka:INFO:Optimizing module '_codecs_jp', 4 more modules to go after that.
Nuitka:INFO:Optimizing module '_codecs_iso2022', 3 more modules to go after that.
Nuitka:INFO:Optimizing module '_codecs_hk', 2 more modules to go after that.
Nuitka:INFO:Optimizing module '_codecs_cn', 1 more modules to go after that.
Nuitka:INFO:Optimizing module '_bz2', 0 more modules to go after that.
...

Encountered the same infinite loop error on windows.
Without the --standalone flag everything works as expected.
Version
Nuitka <IP_ADDRESS>
Python: 3.7.6 (default, Jan  8 2020, 20:23:39) [MSC v.1916 64 bit (AMD64)]
Executable: C:\Miniconda3\envs\deepbratumia\python.exe
OS: Windows
Arch: x86_64
Cmd
python -m nuitka --standalone  --plugin-enable=numpy --plugin-enable=torch  --follow-imports --show-progress --show-scons main.py
Code
import torch

if __name__ == '__main__':
    device = torch.device('cpu')
    # device = torch.device('cuda') # Uncomment this to run on GPU

    # N is batch size; D_in is input dimension;
    # H is hidden dimension; D_out is output dimension.
    N, D_in, H, D_out = 64, 1000, 100, 10

    # Create random input and output data
    x = torch.randn(N, D_in, device=device)
    y = torch.randn(N, D_out, device=device)

    # Randomly initialize weights
    w1 = torch.randn(D_in, H, device=device)
    w2 = torch.randn(H, D_out, device=device)

    learning_rate = 1e-6
    for t in range(500):
      # Forward pass: compute predicted y
      h = x.mm(w1)
      h_relu = h.clamp(min=0)
      y_pred = h_relu.mm(w2)

      # Compute and print loss; loss is a scalar, and is stored in a PyTorch Tensor
      # of shape (); we can get its value as a Python number with loss.item().
      loss = (y_pred - y).pow(2).sum()
      print(t, loss.item())

      # Backprop to compute gradients of w1 and w2 with respect to loss
      grad_y_pred = 2.0 * (y_pred - y)
      grad_w2 = h_relu.t().mm(grad_y_pred)
      grad_h_relu = grad_y_pred.mm(w2.t())
      grad_h = grad_h_relu.clone()
      grad_h[h < 0] = 0
      grad_w1 = x.t().mm(grad_h)

     # Update weights using gradient descent
      w1 -= learning_rate * grad_w1
      w2 -= learning_rate * grad_w2


Recommended Extension

Hello
What are the chances to the extension to appear at the recommended list for
firefox android with the latest versions?
Thanks

Ask AMO.
Note that this extension does not work fully on the current version of Fenix.


[question] Documentation of --v flag

I've noticed that I kept getting gigabytes of log data. I had the flag log verbosity set to 10 (--v:10).
I tried to find some documentation of the different levels, but I couldn't find anything.
Is there any documentation that I'm missing?
I've now set it to --v:1

Now I see glog is being used for logging.
If someone else is looking for more info about the flags, you can find it here: http://rpg.ifi.uzh.ch/docs/glog.html

Normally people use -v=4 for running production AFAIK.

(10 is usually reserved for super-verbose debug logging)


Fix nodejs runtime

Do you plan to work on this task this week ? When will it be delivered as part of a next release ? Can you apply the label as defined here : https://github.com/snowdrop/snowdrop-team/labels?q=- ? @metacosm


The result?

Hello
I test
python demo.py --prompt "Street-art painting of Emilia Clarke in style of Banksy, photorealism"
But where the result image is saved? i don't find it

Thanks
I try again at this time and it work fine, i don't know what's wrong before


Add actual weights to shipment model

Description
There are three types of weights that we need to collect:

Service Member estimated weights
Premove Survey estimated weights
Actual weights (as measured on the truck)

The only weights we don't have in the Shipment table are:

actual weight
actual progear weight
actual spouse progear weight

I've also added these to the public api
Reviewer Notes

Double check spelling please.
I'm going to update the dp3.sqs diagram after I get approval of the names.

Setup
Run this to check:
make db_dev_reset && make db_dev_migrate && ./bin/generate-test-data -scenario=7

Code Review Verification Steps

[ ] Any migrations/schema changes also update the diagram in docs/schema/dp3.sqs
[x] Request review from a member of a different team.
[x] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

@kilbergr - I was worried it would be ambiguous if I didn't have the word "actual".  Though perhaps a different prefix could be used.  Do you think for a first time reader of the code they'd understand?


Added tailwind

Description
Updated the [ages with tailwind.
Fixes #718
Screenshots

Checklist

[x] Tests have been added or updated to cover the changes
[x] Documentation has been updated to reflect the changes
[x] Code follows the established coding style guidelines
[x] All tests are passing

@thestarsahil completed perfectly

@thestarsahil please check it. I think this will work

@thestarsahil I have not touched the login page, I only worked with the Contribute and courses page

@thestarsahil try to merge it. if the problem remains I will try to fix it

@thestarsahil There were some issues, I have resolved it


[Python APIView] some literal types should be wrapped in quotes

Small nit, the value for literal types is not rendered between quotation marks. See below:

@catalinaperalta what does the underlying code look like here?

Example:
def with_literal_typehints(self, stringlit: Literal["cat", "dog"], intlit: Literal[1, 2, 3], nonelit: Literal[None], enumlit: Literal[DOG])

This issue occurs because astroid'd as_string method is monkey-patched to remove quotes. If we remove that, it will preserve quotes that we'd ideally like to strip, but the downside is that obviously we don't want to strip the quotes from string literals. This will likely be a more involved fix.

The example in the FR code is:
def begin_analyze_document(self, model: Literal["prebuilt-receipt"], document: Union[bytes, IO[bytes]], **kwargs: Any) -> LROPoller[AnalyzeResult[Receipt]]:


wrong behavior when mouse over and exit

repo: https://github.com/HppZ/BitmapIconBug

@HppZ can you provide more details on what the current behavior is and what you expect ? Pictures showing the current state would be very helpful as well. Thanks!

mouse over then exit those two controls respectively, then:

click on them to check them, then mouse over then exit those two controls respectively:

any update here?

when will you be fixing it?


Fix Default Nav Logo Bug

Navbar doesn't have a default logo

fixed with 5e1db77

fixed with 5e1db77


Google Chrome CSS3 linear gradients re-rendering in stickyElement using wrapAll()

Hi,
I've come across an issue local to Google Chrome when I have elements in the stickElement object that use CSS3 linear-Gradients (it may go beyond CSS3 linear-gradient but I haven't looked in other scenarios), they seem to re-render themselves with a noticeable jump when the wrapAll() method is called, this also forces the stickyWrapper to inherit an incorrect height.
I have looked at the plugin and have re-written a few areas to drop use of wrapAll() and instead use a clone() of the stickElement, create a new jQuery DOM object for the stickyWrapper, apply css attributes based on the original stickyElement that is still in position in the HTML DOM, append the clone stickyElement to the new stickyWrapper and replace the original stickyElement in the DOM with the new object using replaceWith()
My new code:
/* Sticky Plugin
** =============
** Author: Anthony Garand
** Improvements by: Phil Middlemass (www.leighton.com)
** Changelog: 	Remove use of wrapAll() to work around rerendering issues with
				CSS3 linear-gradients of elements with the stickyElement in
				Google Chrome. Now uses	combination of a cloned stickyElement
				and a new jQuery DOM element and replaceWith() to replace the
				stickyElement with our new code.

** Date: 8/17/2011
** Website: http://labs.anthonygarand.com/sticky
** Description: Makes an element on the page stick on the screen
*/

(function($){
	$.fn.sticky = function(options) {
		var defaults = {
			topSpacing: 0,
			className: 'is-sticky'
		};
		var o = $.extend(defaults, options);
		return this.each(function() {
			var topSpacing = o.topSpacing,
			$el = $(this); /* New reference to original Sticky Element */
			stickyElement = $el.clone(),  /* StickyElement now becomes a clone of $el (to maintain CSS3 rendering) */
			stickyId = stickyElement.attr('id'),
			fixed = false;
			stickyElement
				.css('width', $el.width());		/* Use the DOM properties of $el to set data on stickyElement */
			var stickyWrapper = $('

');	/* Create New DOM element for the wrapper - Removed use of wrapAll (CSS3 compatibility) */
			stickyWrapper
				.css('width', $el.outerWidth())		/* Use the DOM properties of $el to set data on stickyWrapper */
				.css('height', $el.outerHeight())
				.css('clear', $el.css('clear'))
				.append(stickyElement);		/* Append our new cloned element to stickyWrapper */
			$el.replaceWith(stickyWrapper);	/* Replace $el in the DOM with our new stickyWrapper */
			$(window).scroll(function() {
				var scrollTop = $(window).scrollTop();
				var elementTop = stickyWrapper.offset().top;
				if (scrollTop <= elementTop - topSpacing) {
					if (fixed) {
						stickyElement.css('position', '').css('top', '').removeClass(o.className);
						fixed = false;
					}
				}
				else if (!fixed) {
					if (scrollTop >= elementTop - topSpacing) {
						stickyElement.css('position', 'fixed').css('top', topSpacing).addClass(o.className);
						fixed = true;
					}
				}
			});
		});
	};
})(jQuery);

I'm now using this on some prototype pages for a client and all seems to good so far.
let me know if this is something you can absorb
Phil.

Can you submit this in a pull request based on the current version?


lite version of ng2-admin

I'm submitting a ...
[ ] bug report
[ ] feature request
[X] question

Do you want to request a feature or report a bug?
request a lite version: is there a lite version of ng2-admin - without the examples built in?

check my comments in #760

I have done the following steps to reduce the size:

remove all unused 3rd-party libraries (huge size difference)
remove all unused components (including their styles / pipes/ ...)
remove all unused / un-needed assets (huge huge difference)

a quick win is the assets folder, it is responsible for the largest portion of the size.
let us know what happens with you.

first of all who is us?
second of all I was hoping for an official answer from the repo contributors:
on a version that just contains the best libraries without the demos provided.
I know what I can do myself and I am doing it, it just would be nice to have a version that gets updated as the project improves...without the demos.

Hey @tal-shahar, thanks for asking.
So basically yes, that's what we are working on currently - dividing the project into smaller self-contained modules (theme, demo, auth, etc). Hopefully, we will shed some light on it soon, as we have a better understanding of the approach and the direction we are going.
Please follow this issue https://github.com/akveo/ng2-admin/issues/527 for updates.


Relax libpq pin?

soci-postgresql cannot be installed with the most recent libpq:
 $ mamba create -q -n tmp --dry-run libpq=14.3 soci-postgresql
Encountered problems while solving:
  - package soci-postgresql-4.0.1-h4bd325d_0 requires libpq >=12.3,<13.0a0, but none of the providers can be installed

This is because libpq is pinned exactly in https://github.com/conda-forge/soci-feedstock/blob/70c74e57076d2ce9cdcfaafa2d2ceca15bb43e46/recipe/meta.yaml#L110 (libpq doesn't seem to use a third version component so max_pin='x.x' is an exact pin in this case. This means that soci-postgresql will need to be rebuilt for every libpq release, as I've done in https://github.com/conda-forge/soci-feedstock/pull/10 (please merge, I need this for https://github.com/conda-forge/rstudio-feedstock/pull/5)
Does it make sense to relax the libpq pin to max_pin='x'? If not maybe we can add libpq to the global pins so the bot will update for us.

Also, libpq does have run export now, so this can be dropped here.

Fixed now in https://github.com/conda-forge/soci-feedstock/pull/10, please merge that when you have a moment.

Thank you for the fix =)


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #627 (ce5996e) into main (25a1ebf) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #627   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Quilt is incompatible with Bewitchment

Bewitchment depends on a newer version of Pehkui and Patchouli.
Also Quilt loader 0.17.5 (Release, not betas) is what it requires too

Bewitchment depends on a newer version of Pehkui and Patchouli.
Also Quilt loader 0.17.5 (Release, not betas) is what it requires too

That worked, than you very much.


notes from oct 29 pydata tutorial

Here are some small things I noticed while we presented today's tutorial:
Tutorial 1 - Weather Data: Accesing it, understanding it, visualizing it!

typo in title: "Accesing"
broken link in "Weather data in PV performance models"
extra capitalized letter: WRC link "Solar RAdiometry webpage"

Tutorial 3 - Modeling an Array Power

confusing variable name: df_poa_tracker, but fixed-tilt simulation
typo: "Calculating hte Power"
false statement: "Note that insolation and production happen to be about the same scale here; that’s just a coincidence because of the array size we chose."
last plot comparing AC and DC energy is messed up, AC is always zero?

general

update to pvlib 0.9.0 so that we aren't advertising the old return values from get_psm3 (order of metadata & df changed)

great, all addressed!


Fix initialDates shift bug

Fixes #76

Now the initialDates array contains the correct initial dates from each page,  which did not occur before if numberOfDays was 7
Now the "decision" to start a week on Mondays gets relegated to the calculatePagesDates() method, and calculateDaysArray is simplified

I see, we missed this when refactored the code. Thanks, @pdpino, merging.


fix: relative paths' problem when first rendering

Signed-off-by: Yue Yang<EMAIL_ADDRESS>What problem does this PR solve?
As the title.

Codecov Report

Merging #675 into master will decrease coverage by 1.06%.
The diff coverage is 33.18%.

@@            Coverage Diff             @@
##           master     #675      +/-   ##
==========================================
- Coverage   55.78%   54.71%   -1.07%
==========================================
  Files          68       68
  Lines        4383     4569     +186
==========================================
+ Hits         2445     2500      +55
- Misses       1768     1882     +114
- Partials      170      187      +17

Impacted Files
Coverage Δ

api/v1alpha1/podchaos_types.go
37.14% <ø> (ø)

controllers/networkchaos/ipset/ipset.go
37.14% <0.00%> (ø)

pkg/utils/chaosdaemon.go
51.21% <0.00%> (ø)

pkg/utils/grpc.go
0.00% <0.00%> (ø)

pkg/webhook/inject/inject.go
73.49% <0.00%> (-0.68%)
:arrow_down:

pkg/store/experiment/experiment.go
24.27% <12.28%> (-14.51%)
:arrow_down:

pkg/httpfs/fs.go
31.48% <31.48%> (ø)

controllers/twophase/types.go
54.20% <40.00%> (-1.68%)
:arrow_down:

controllers/networkchaos/netem/types.go
47.27% <50.00%> (ø)

pkg/utils/selector.go
53.15% <66.66%> (+0.91%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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

/merge


Add Wrapper tests in istioctl

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

What is the purpose of this test?
We recently removed a large number of unit tests including a test similar to this one.  We replaced them with integration tests.  https://github.com/istio/istio/pull/23730
The reason was because the format of the data being held by the sidecar was changing, and the tests were not in sync.  A unit test would be valuable here, but instead of testing that a particular structure can be deserialized any test should verify that the data given by Istiod to Envoy and then given by Envoy to istioctl can be deserialized.  I was unable to implement

(closed by accident).

Ah ok, I see, im going to look into creating that. Closing


Support to declare externals in plugins' package.json

As a part of the shell application, does the plugin automatically add pilets.externals to the externals of the app shell from the package.json of the referenced plugins, instead of manually adding externals when the app shell imports plugins `?

Yeah the externals of a pilet are the combination of the shared dependencies from the app shell and its own externals.
See:
https://github.com/FlorianRappl/piral-cli-webpack/blob/master/packages/pilet-webpack-plugin/src/index.ts#L23
Regarding import of plugins: With the exception of the core shared dependencies all other dependencies must be shared explicitly. So if you have a plugin that comes with a dependency (either directly or as peer) you will need to add it to the shared dependency explicitly. Otherwise, pilets cannot use it.
Hope that helps!

Got it, thanks!


v2022.4.4 still requires homeassistant 2022.4.2 in some places

FYI.  For example https://github.com/KapJI/homeassistant-stubs/blob/2022.4.4/pyproject.toml#L27, and also in poetry.lock.  Thanks!

For some reason automation didn't update it. But as I can see only 2022.4.3 and 2022.4.4 were affected.


Unresponsive Subscribe Newsletter Button on Carousel for Smaller Screens

I've encountered an issue while navigating through our project's website on smaller screens. It appears that the 'Subscribe' button in our newsletter carousel is not aligned. This issue has not been explicitly stated in our documentation or issue tracker for smaller screens. I have viewed in a chrome Version 125.0.6422.141 (Official Build) (64-bit)
As an SSOC contributor, I would like to bring this to your attention and request that this work be assigned to me so that I can investigate and resolve it promptly.
Steps to reproduce:

Reduce browser window size or view on a mobile device.

Navigate to the homepage carousel featuring the newsletter subscription.

Subscription button is not aligned

Expected result:
The button should be the same when we view it on the desktop screen.
Actual result:
The button remains unresponsive and not aligned with smaller screens.
I believe addressing this will significantly improve user experience, especially for those accessing our site via mobile devices.

@jeevan10017 functionality is not added till now

@MAVRICK-1  Can you please explain why it happened?
my whole night effort got ruined 😢


Evaluate Options for Data Migration

[ ] Meet with Cloud.gov architecture team to determine options supported
[ ] Document possibilities
[ ] Meet with team to review and come up with plan of action

Assignees:
@jwhitehead77
@visocvp

Epic: Determine data migration strategy
Tasks:
Evaluation and configuration of Ora2PG and SCT for data migration (3) - Done
-Met with Cloud.gov architecture team (Chris McGowan) to determine options supported
-Configuration of Ora2Pg
-Installation of Oracle CPA DB Package
-Installation of the CPA for perl scripts
-Installation and testing of schema conversion tool for schema migration
Configuration file for data migration (1) - In progress
Create RDS oracle instance in AWS (1) - Sprint backlog
Obtain SQL dumps from On-Premise Oracle DB and load into CVP AWS RDS Oracle (2) - Sprint backlog (external dependencies)
Schema migration from AWS RDS Oracle to Cloud.gov Postgres through SCT (2) - Sprint backlog

generate assessment reports

Data migration from AWS RDS Oracle to Cloud.gov Postgres through ORA2PG (2) - Sprint backlog


support/db: Add benchmark for BatchInsertBuilder

@sreuland You are totally right, I should had provided more context (there are PR guidelines for it which I didn't follow, sorry). The context is trying to optimize the ingestion system. See #4022
Hopefully it will be more clear in an upcoming PR.


Switch to using rustyline for repl

As discussed in https://github.com/Marwes/gluon/pull/35#issuecomment-233140511:

I started some work to replace the linenoise dependency with rustyline. There shouldn't be any problems porting over any changes since it is mostly a rust implementation of linenoise but it also has a better completion api which should make it possible to add tab-completion in the REPL!

Might also be nice to keep the history between sessions. What do you think?

That is a good idea and it is supported by both libraries. The only question may be where to store it. Probably in .dotfiles on linux and AppData on Windows? I think I have seen a crate which handled this but I can't remember what it is called

Yeah - that's why I held off on doing with linenoise. :/

Merged rustyline support in #45 though I will leave this open until the Windows support is more complete.

Should we rename the issue to be clearer?

Updated the title.

Closed by #128


Naming convention is inconsistent

It's not quite clear what this project is officially named. Although the repository and files are titled MicaForEveryone, the organisation seems to be called Mica For Everyone, while the project's README.md mentions Mica for Everyone. I'm not sure if this is deliberate, but it makes identifying the project's official name a bit confusing.

We can't include spaces in repository names.

It is indeed not possible to create repositories with spaces, and that's fine. I just felt like pointing this out because another project mentions it without spaces, and I wasn't quite certain if I should correct it.

"MicaForEveryone" is the project name, something we use in code. "Mica For Everyone" is the application name, we use it as title. If I wrote "Mica for Everyone" with small f in README then maybe it's because it doesn't matter too much, but I'll edit the README to make it consistent.

It probably doesn't matter much, and I apologise if I seem pedantic. I just wanted to make sure I was using the correct name in another pull request.


Docker for AWS clearer instructions on deployment

File: docker-for-aws/deploy.md, CC @johndmulhausen
"docker deploy to deploy a bundle on your machine over an SSH tunnel, or copy the bundle (for example using scp) to a manager node, SSH into the manager and then run docker deploy (if you have multiple managers, you have to ensure that your session is on one that has the bundle file)"
Specific examples of either of these options would be great. The docker deploy docs don't clearly state how to deploy via ssh tunnel, and there seem to be alot of confusing/conflicting ways to do this.

@FrenchBen do you think we should be documenting how to SSH into the AWS / Azure machines?

@easherma I believe those to be SSH basics and perhaps not necessarily needed, as users with certain VPN/firewall configs may complain that we don't include these instructions either.
The ssh connection has been described here:
https://docs.docker.com/docker-for-aws/deploy/#connect-via-ssh
The DAB mention could be removed, in favor to the regular compose files, via:
docker stack deploy -f docker-compose.yml myapp