Cmd + C without selection copies  to clipboard

Issue Type: Bug

Select some text in an editor pane
Use Cmd + C to copy
Make sure no text is selected
Hit Cmd + C again as if by accident
Hit Cmd + V to paste the text copied in step 2

In other apps on macOS, this would paste the text from step 2. In VS Code, this pastes nothing (that is, it appears that the empty string was copied to the clipboard in step 4). I'm pretty sure the same behavior would be expected on other platforms, though I have not tested it.
This is a long-standing issue that has been around for at least a year.
VS Code version: Code 1.64.2 (Universal) (f80445acd5a3dadef24aa209168452a3d97cc326, 2022-02-09T22:00:58.347Z)
OS version: Darwin arm64 21.3.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 Max (10 x 24)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
64.00GB (25.41GB free)

Process Argv
--crash-reporter-id 340097ba-bfe5-4da5-b466-1b1cdc0f41f6

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

better-toml
bun
0.3.2

vscode-eslint
dba
2.2.2

prettier-vscode
esb
9.3.0

copilot
Git
1.7.5118

terraform
has
2.19.0

rust-analyzer
mat
0.2.948

crates
ser
0.5.10

vscode-stylelint
sty
1.2.2

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392cf:30425750
pythontb:30283811
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30433507
vsclayoutctrc:30437038
vsrem710:30416614
dsvsc009:30440023
pythonvsnew555:30442236
vsbas813:30436447
vscscmwlcmc:30438804
helix:30440343

It looks like you would like to configure "editor.emptySelectionClipboard": false

@alexdima actually I would like to challenge that true is a good default for that setting.

I have to be honest, unless there is some huge amount of feedback about this, we will not change the default. There is no upside to change this default after more than 6 years and risk breaking everyone that today relies on it.


Endpoint to calculate Number of Periods - PV & FV

This formula is used to calculate the length of time a present value would need to reach the future value, given a certain interest rate.
Perform all the calculations in function.py file present in helpers folder
Create the endpoint no-of-periods in main.py file and return the required result in proper format
Formula :

i will work on the issue pls assign me this issue

https://github.com/Clueless-Community/fintech-api/pull/188  merge the PR if no further changes are required

great @thecodermaniac go ahead! :)

Ya no more changes are required, it looks good!
On Mon, Feb 6, 2023 at 4:21 PM Aritra Mondal @.***>
wrote:

#188 https://github.com/Clueless-Community/fintech-api/pull/188 merge
the PR if no further changes are required
—
Reply to this email directly, view it on GitHub
https://github.com/Clueless-Community/fintech-api/issues/183#issuecomment-1418882905,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWLBQX2OKKGICEPVFF2OAVDWWDJR7ANCNFSM6AAAAAAURAPJZM
.
You are receiving this because you authored the thread.Message ID:
@.***>


FS-1997-translate-application-form-statuses-to-welsh

This pull request resolves the problem of application form statuses not being translated into Welsh when users select the Welsh language, by implementing a fix that ensures all application form statuses are now properly translated into Welsh on the tasklist page when the Welsh language is selected.
Ticket: https://digital.dclg.gov.uk/jira/browse/FS-1997
Screenshot:

@ramsharma-prog Did you add the translation for "Back to your applications"? as that's what the ticket says to fix as well.

@ramsharma-prog Did you add the translation for "Back to your applications"? as that's what the ticket says to fix as well.

Added 👍

@ramsharma-prog Did you add the translation for "Back to your applications"? as that's what the ticket says to fix as well.

Added 👍

Well done @ramsharma-prog.


posix: clock: implement clock_getres()

Add an implementation of clock_getres(). Clock resolution is currently limited to the inverse of CLOCK_TICKS_PER_SECOND as there is no direct relationship between hardware clocks and the POSIX variety in Zephyr.
The clock_getres() function is required by the POSIX_TIMERS Option Group as detailed in Section E.1 of IEEE-1003.1-2017.
The POSIX_TIMERS Option Group is required for PSE51, PSE52, PSE53, and PSE54 conformance, and is otherwise mandatory for any POSIX conforming system as per Section A.2.1.3 of IEEE-1003-1.2017.
With this, we have complete support for the POSIX_TIMERS Option Group.

It looks like the 2d review period is up now.


libpng12-dev of Debian Jessie no longer exists in Stretch?

Package libpng12-dev is no longer available for Debian 9(Stretch). Use libpng-dev instead.
$ apt-get update && apt-get install -y \
        openjdk-8-jdk automake autoconf \
        curl zip unzip libtool swig libpng12-dev zlib1g-dev pkg-config git g++ wget xz-utils \
        python3-numpy python3-dev python3-pip python3-mock
Ign:1 http://cdn-fastly.deb.debian.org/debian stretch InRelease
Get:2 http://cdn-fastly.deb.debian.org/debian stretch-updates InRelease [91.0 kB]
Get:3 http://security.debian.org/debian-security stretch/updates InRelease [94.3 kB]
...
Get:17 http://cdn-fastly.deb.debian.org/debian stretch/main armhf Packages [6927 kB]
Reading package lists...
Building dependency tree...
Reading state information...
Package libpng12-dev is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
E: Package 'libpng12-dev' has no installation candidate

Hi, libpng12-dev have in oldstable repository (http://httpredir.debian.org/debian oldstable/main amd64 Packages). I tested with libpng-dev, but fails when import tensorflow.


Shotguns do not seem to spawn in v0.5.0 and v0.6.0

In v0.4.0 they seem to spawn just fine. In v0.5.0 and v0.6.0 i tried changing my configuration and disabling most mods, but it still didn't work.

After taking a few looks at the source code and logs, i think i narrowed down what could be the issue.

The shotgun never gets debug logged, and i think that it might be because the game considers it as scrap.
Keep in mind that this is just my educated guess after browsing your github and looking at my logs, it's still possible i might be wrong.

Hey, thanks for reporting the issue - I'm gonna fix this ASAP.

Quickly tested and can confirm - shotguns are considered scrap, which is something i've not tested properly (oops)
[Debug  :UnlockOnStart] Item ID: 59 / Item name: Shotgun
[Debug  :UnlockOnStart] Is scrap: True

I've just pushed v0.6.1 - please check if it resolves your issue.

I can confirm, Shotguns in v0.6.1 spawn as intended. Thank you!

.


New motion approach to combine SimpleMotion and ArbitraryMotion into the same phantom

Idea for new motion related types and structs redistribution, in order to be able to combine them and make them more alike:
abstract type Motion end

abstract type SimpleMotion <: Motion end
abstract type ArbitraryMotion <: Motion end

Specific motion structures:
Translation <: SimpleMotion
Rotation <: SimpleMotion
...
PeriodicHeartBeat <: SimpleMotion

Trajectory <: ArbitraryMotion
PeriodicTrajectory <: ArbitraryMotion
FlowTrajectory <: ArbitraryMotion
PeriodicFlowTrajectory <: ArbitraryMotion

That way, phantom struct will look like this:
@with_kw mutable struct Phantom{T<:Real}
    ...
    motion::Vector{Motion{T}} = []
    ...
end

and motion will be initialized, for example, like this:
motion = [Translation(...), Rotation(...), Trajectory(...), HeartBeat(...)]

This is a very good idea! We just need to think about how this will impact:

obj1 + obj2: obj1 has motion1 and obj2 has motion2 (one ball going up, and another one going down)
obj.phantom: phantom file read/write

Another possible approach regarding periodicity:
abstract type Periodicity end

struct Periodic <: Periodicity end
struct Aperiodic <: Periodicity end

struct Translation{T<:Real, P<:Periodicity} <: SimpleMotionType{T}
   ...
end

Regarding flow:
function reset_magnetization!(M::Mag{T}, Mxy::AbstractArray{T}, motion::MotionModel{T})
   return nothing
end

function reset_magnetization!(M::Mag{T}, Mxy::AbstractArray{T}, motion::FlowTrajectory{T})
   # do things with flow (create interpolation functions, resolve them, etc):
   Mxy .= ...
   M.z .= ...
   return nothing
end


Reload chat inbox on reconnecting

@keybase/react-hackers r?
I'm getting a surprising (to me) flow error here, any ideas?  We seem to assume action.payload will be defined on all the other action handlers.
actions/chat.js:657
657:   const {reachability} = action.payload
              ^^^^^^^^^^^^ property `reachability`. Property cannot be accessed on possibly undefined value
657:   const {reachability} = action.payload
                              ^^^^^^^^^^^^^^ undefined

Found 1 error

this is cause its not a noerror type: see https://github.com/keybase/client/blob/8063ef8f5604af154475b1d2f89f8d792d7af349/shared/constants/gregor.js#L14
so you should either change the type to NoErrorTypedAction or you have to do a if (!action.payload.error)

The commits 76329a3afbe9...e11d929e781b introduce visual changes on linux. :mag_right: 1 changed

Changed: Chat_Input-Emoji_Open (before) (after)


go.mod: Update to go 1.18

Ok seems like go 1.19 has changes to how gofmt handles documentation. Think wrong gofmt binaries are being installed. Might as well update to go 1.19 straight away I guess

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.8%) to 51.564%

Files with Coverage Reduction
New Missed Lines
%

integration/rpctest/blockgen.go
2
83.67%

peer/peer.go
3
74.75%

Totals

Change from base Build<PHONE_NUMBER>:
-1.8%

Covered Lines:
22368

Relevant Lines:
43379

💛  - Coveralls


fix(secrets): remove string interpolation from description

The current description for the secrets field causes an error when the action is invoked:
Error: superfly/fly-pr-review-apps/main/action.yml (Line: 29, Col: 18): Unrecognized named-value: 'secrets'. Located at position 1 within expression: secrets.FIRST_SECRET
Error: Fail to load superfly/fly-pr-review-apps/main/action.yml

This change in description fixes this.
I'm also documenting that option in the readme.

Closed in favor of https://github.com/superfly/fly-pr-review-apps/pull/37


Change EULA page in the installer to reference the license, support statement and trademark

https://github.com/code-ready/crc/blob/c55b7c4a7675893e38dc9530dec1287a29a6ce50/packaging/windows/LicenseAgreementDlg_HK.wxs#L6
https://github.com/code-ready/crc/blob/c55b7c4a7675893e38dc9530dec1287a29a6ce50/packaging/windows/LicenseAgreementDlg_HK.wxs#L22

Shouldn't they also be in git?

Shouldn't they also be in git?

At the moment this isn't public as some of these are still being finalized/discussed.


Using webworkers  and wasm apprach generates browser out of memory on large files

If I use the webworkers with wasm approach with large files like above 200mb the browser display this page

Also if I don't use the webworkers it works fine

Hi! Unfortunately, webworkers make it necessary to serialize data, and that has a limitation.
We are putting toguether a workflow for bigger files that involve exporting the models to GLTF+JSON. That solves all performance issues for all use cases (models load aprox 10 times faster) and make the whole thing much more scalable. All the tools will work with this workflow as well. I'm putting toguether a tutorial to show how this works, hopefully it will useful for this use case 🙂
I'm closing this because we can't solve the serialization limit, but stay tuned to our publications / discord for this tutorial.

Hi thanks for you answer! I am having a similar problem with an app to validate properties, calling ifcManager.getAllItemsOfType then looping and calling getItemProperties, getPropertySets, getMaterialsProperties and getTypeProperties and then validating the properties.
I'm trying to run my own webworkers and than partition what elements the workers are to process. But the memory runs out when all the workers try to serialize the data file. But I was wondering if there is a way to do this with text data only? Sense I'm not using the 3d renderer so I don't need the GLTF that is going to be in your new workflow but I still need to process big files. Or maybe there is a better way to loop over all data in a file that I'm not aware of?

The problem is that given that IFC is a text file and due to the current implementation, all properties+geometry must be loaded in memory to read them. This is not very scalable.
So these days we have put toguether an exporter that converts an IFC into an array of glTF files (geometry) + a JSON (properties). This way, geometry loading gets exponentially faster, users can only load the geometry they want and properties can be used to populate a database, which is a much more efficient system to retrieve data. The downside of this approach is that you need a backend (or file system in case you are using electron) to store this data. But IMO it's a small price to pay to reduce memory usage by half and increase loading speed by 10. 😄 I started making the tutorial for that today. We know that a lot of people want to build scalable BIM platforms, and we will definitely cover it!

Hi! Unfortunately, webworkers make it necessary to serialize data, and that has a limitation.
We are putting toguether a workflow for bigger files that involve exporting the models to GLTF+JSON. That solves all performance issues for all use cases (models load aprox 10 times faster) and make the whole thing much more scalable. All the tools will work with this workflow as well. I'm putting toguether a tutorial to show how this works, hopefully it will useful for this use case 🙂
I'm closing this because we can't solve the serialization limit, but stay tuned to our publications / discord for this tutorial.

Hi @agviegas thanks for clarification, may you please describe the the workflow of GLTF+JSON like in steps so I may figure it out until the tutorial comes out.

Sure, you can find the export example and the import example in the hello-world repository. They are also published on github pages, so you can take a look at the live examples (just change the URL by the github pages URL).


maybe unselected running container's icon should be green

Expected behavior
When a running container is selected, it's icon should be green, just like when it's not selected.

Actual behavior

Information about the Issue
Without the green color, there maybe a confusion.
Steps to reproduce the behavior

Start a container
Select another container

When you select the container, the right hand side panel will indicate that it is running and thus having an icon on the left side distract from it.

Other non-selected containers do need to show this change of status, to visual confirm that they are indeed running.

So why not just hide the icon?
I think keeping the ~ - shape but discard the green and gray color will make people confused.

The icon is part of the design and keeps it consistent with the running containers below it.
Visually having the green there makes it look off.
You can try it by using the chrome console and you will see that the design will not match well.


{undefined} value for regex

@alexanderzobnin , I'm trying to make a host regex (in template) that match these strings: SV-V3-API(i-4568afd8) or SV-API-API-LOAD(i-uuuhty45). But, the interface gives me the {undefined result} when I use the following regex:
SV-V3-API(-LOAD)?\(i-[a-z0-9]{8}\)

It works fine if I test it on https://regex101.com/#javascript with the following values:
{SV-V3-API(i-0a0b8de8),SV-V3-API(i-3373cae2)}
{SV-V3-API-LOAD(i-1b4d6da9),SV-V3-API-LOAD(i-5386a7e1),SV-V3-API-LOAD(i-55e20b96),SV-V3-API-LOAD(i-c5ec0506),SV-V3-APIMANAGER-LOAD(i-b3b75e70)}

Is it a bug with templating regex? Does it supports regex groups?

Can you attach screenshot of grafana?
SV-V3-API(-LOAD)?\(i-[a-z0-9]{8}\) - in this regex (-LOAD) is a capture group - your regex will return only -LOAD (as you can see at MATCH INFORMATION panel).

@alexanderzobnin the "MATCH INFORMATION" is intended only for group matching. When a regex matches, the site will colorize the matched string with a blue background.
See the print bellow:

I think that you are not considering the entire regex matching, only the group matching. When I put each regex part in a group, the interface will select the first one and ignores all the others. See the prints bellow:
# This is only for show that I have hosts enabled
Regex: SV-V3-API-LOAD.*

# This will only match "SV-V3-API"
Regex: (SV-V3-API)(-LOAD)?(\(i-[a-z0-9]{8}\))

# This is the print showing the {undefined}
Regex: SV-V3-API(-LOAD)?\(i-[a-z0-9]{8}\)

In templates regex returns only first capture group matching.
SV-V3-API(-LOAD)?\(i-[a-z0-9]{8}\) can returns only -LOAD. Try to use non-capturing parentheses:
SV-V3-API(?:-LOAD)?\(i-[a-z0-9]{8}\)

Worked:
SV-V3-API(?:-LOAD)?(i-[a-z0-9]{8})
Thank you very much @alexanderzobnin

Ok, nice!


docs: add spa mode section in rendering

🔗 Linked issue

Documentation issue

📚 Description
It was not obvious how to deploy a full SPA in nuxt. After watching @manniL video, hopefully this should be helpful for others that want to use SPA mode and with a possibility to upgrade to various rendering mode when the app is evolving.

@manniL it would be great if you can point me where to find more info about the routes.clear(). It would be nice to include what does it do behind the scenes.


update eslint config & add JSDoc

add JSDoc

add ·JSDoc· comments for index.js, lib/catFile.js, lib/diff.js

update eslint config

switch on valid-jsdoc
remove rules that already defined in eslint:recommended , no-undef, no-unused-vars

remove unused variables

I'd like to add support for git.lsFile & git.blame in next PR

/CC @stevelacy

I would have preferred you open an issue before adding JSDoc as my personal preference is to not use it, as right now there are only three files with JSDocs added. Making all the others not consistent.
What are the outputs of ls file and git blame? I do not think git blame is needed or should be added to this plugin.

Thank you, merged


Patch 2

why are you adding @dleovl's workflow to the repo? useless pull req lol

now this doesn't even have to exist 😭

@GingGooNine Whether you fork it or not and add the building yml is 100% up to you, but I'm pretty sure these changes are going to be pushed into upstream, and RootHide/Bootstrap currently does not want any auto building ymls  yet. This PR should be closed.


Document numba.extending.intrinsic and inlining.

As title

Needs #4547

CI passed, marking for merge.


Introduce validity metadata at header level

What about winding being an enum of CCW, CW or null?
Some source formats like Shapefile have exterior rings CW, and others like GeoJSON define exterior as CCW. If the metadata indicates CW winding but my reader wants CCW winding, I can just read each polygon ring "backwards" without having to make a copy or do any winding checks. This would preserve the lossless nature of transformation from other well-formed formats independent of how they prefer winding order.
So the language for winding can be like this: if winding is CCW, the first ring of Polygon coordinates must be CCW and all rings after CW. If winding is CW, the first ring is CW and all rings after CCW. It should recommend that the first ring is the "Outer" one and all ones after are "Inner" but that constraint that would belong to Validity and not Winding.

@bdon I agree and it could make sense to have both winding and validity as enums.
Enum cannot be null but default is 0 and that could (should) be unknown. See http://google.github.io/flatbuffers/flatbuffers_guide_writing_schema.html.
For valitity one could complicate it further. There are a couple of well known validity definitions in the wild (on the top of my head, OGC, ESRI and SQL Server all have their own definitions). A geometry can be both OGC and ESRI valid. Perhaps the bit_flags of flatbuffer schema can be used to represent the combination of states here. Might seem to go into details here but in my experience it's not uncommon that one has to deal with these definitions.
And furthermore OGC have a definition if a geometry is simple or not (see http://www.tsusiatsoftware.net/jts/javadoc/com/vividsolutions/jts/operation/IsSimpleOp.html). I'm not sure if that is of value. I will probably ignore it for now unless someone brings it up.

@bdon as for winding and ring order I agree that strictly interpreted the order belongs to validity.. but in practice, is there a real use case for representing the known winding without also constraining the ring order? I would think that is academic only.. and as such I would word the ring order as a requirement rather than a recommendation for the known windings.

No, I didn't have a specific counterexample in mind; indeed just trying to make the language more airtight about winding not implying validity.

Now reworked to these two enums:
https://github.com/flatgeobuf/flatgeobuf/blob/88cacfade9601d71f520fd8b105edb2c84213e1e/src/fbs/header.fbs#L67-L78

Winding lgtm, not an expert on Valid in those situations but did find these links to familiarize myself:
OGC Simple & Valid: https://postgis.net/docs/using_postgis_dbmanagement.html#OGC_Validity
ESRI: https://www.esri.com/arcgis-blog/products/data-reviewer/data-management/invalid-geometry-check-explained/

Closing due to inactivity.

LGTM, I don't think this is necessary anymore.


Add python3 version requirement

When running python3 ./agent/main.py using Python 3.4 I run into errors with the trailing commas after kwargs that pop up throughout the repo. With python 3.6 it runs fine.
Could you add the minimum version for python and nodejs to the readme?

Hi @alexjgriffith, thanks for pointing this out! We're updating the README accordingly.


(feat): Support Windows and macOS detect

This is useful when building desktop websites.

Thank you for your contribution!


Add loader while verifying a connection when registering

Current Behavior

When registering a connection while verifying that connection a small amount time goes in the process

Desired Behavior

It would be good to have a loader which says wait for verifying the connection something like this
Implementation

Acceptance Tests

Mockups

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@sudhanshutech I would like to work on it, if the issue is still open


Question: Getting Error on Merge

I am trying to use the merge command, but I get
"Could not load file or assembly 'System.ComponentModel.Annotations, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)"
I am using the library with .net framework 4.8 web forms project.
Reference to System.ComponentModel.Annotations is added, just couldn't get the exact requested reference. I have tried both 5.0.0 and the 4.3.0 but the error above still remains.
This is the code:
public static int SaveMarks(List<Quizes> marks)
{
            var connString = ConnString.ConnectionString;
            RepoDb.SqlServerBootstrap.Initialize();
            using (var connection = new SqlConnection(connString))
            {
                return connection.BulkMerge("[tbo].[Quizes]", marks);
            }
}`

We have the referenced assembly versions below for the latest version of library. Which version of RepoDB did you use?

Or, have you tried downgraded to S.C.A version to 4.2.0?

All the information on the image

We have the referenced assembly versions below for the latest version of library. Which version of RepoDB did you use?

Or, have you tried downgraded to S.C.A version to 4.2.0?

Couldn't get the S.C.A version 4.2.0 - it defaults to 4.3.0
Do I just use RepoDB for BulkOperations?

Hi, I am facing the same problem. I'm using the latest build into into a dll that is not a startup project.
Is there a workaround?

RepoDB is now referencing the version below.
Link: https://www.nuget.org/packages/RepoDb#dependencies-body-tab

Is there a blocker for you to upgrade the SCA to 5.0.0?

Hi,
first of all thanks for your quickly response!
Yes, for me it is a problem because I'm extending an existing a project built on .NET 4.7.1


Allow longs as members of maps + lists

@danielhochman @johnliu

Coverage decreased (-0.2%) to 97.603% when pulling 93a02d5edbbfb6c842f93b45df6f8b23e525f6c0 on jmphilli:allow-longs into 201b0fcf359240b042bb6819955a2ba9c1eec7ec on jlafon:devel.

Coverage decreased (-0.2%) to 97.604% when pulling 0ec5a67ad858dd871f991f7f408668b69bdd730e on jmphilli:allow-longs into 201b0fcf359240b042bb6819955a2ba9c1eec7ec on jlafon:devel.

Coverage decreased (-0.2%) to 97.605% when pulling 356a1f044fe87394ca7ba868faddfce6250ed063 on jmphilli:allow-longs into 201b0fcf359240b042bb6819955a2ba9c1eec7ec on jlafon:devel.

Coverage increased (+0.002%) to 97.784% when pulling 4e00d6db07857a29d15d3b7c9fa676e6baebf6fb on jmphilli:allow-longs into 201b0fcf359240b042bb6819955a2ba9c1eec7ec on jlafon:devel.

+1


fix bug skipping "DEFAULT_GENERATED" columns in dump

Closes #676.
Was considering just adding a space before "GENERATED" in the if-clause, but this more verbose solution is hopefully more future-proof.
Looked into adding a test for it, but the testing setup looked too complicated for me, sorry.
Tested it with the setup in #676 and it now dumps all columns

Thank you @mashehu! We'll handle the test


Provider fails to implement pagination for >100 SQL users, causing user creation conflicts

Issue Description:
The Terraform provider incorrectly handles pagination when verifying SQL users in the state. When more than 100 SQL users are present, it only retrieves the first 100 due to the default page size. This causes warnings during each Terraform run for a SQL cluster with more than 100 users and errors when applying changes due to conflicts with existing, unfetched users.
The warning during the plan appears as below:

This cluster doesn't have a SQL user named... Removing from state..

Expected Behavior
The provider should correctly implement pagination to verify all SQL users when their count exceeds 100.
Steps to Reproduce:

Write Terraform code that would provision >100 SQL users for a cluster
Execute a terraform apply
Observe the success of the apply (assuming there were no existing users in the state)
Execute a terraform apply again
Observe the warnings indicating that some SQL users are missing, and that they will need to be recreated
Try to apply changes and observe the resultant errors due to user creation conflicts.

Proposed Solution
I have created a PR to resolve this issue.

Fixed in v0.6.0.


Add a Testing document.

Rendered
Testing describes where unit tests and integration tests will be located in the Tock 2.0 libtock-rs crates.

This PR is ready to merge.

bors r+


Bug Report: full screen break doesn't do anything on ubuntu

Describe the bug
full screen break doesn't do anything on ubuntu
To Reproduce
select full-screen break on ubuntu 20.04

Go to 'Settings'
Click on 'Fullscreen break'

Expected behavior

Prevent users from doing anything with the computer

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: ubuntu 20.04
App Version 1.0.0

Additional context

Hello @tonytvo ,
May I know if what package you do installed on your machine? .deb or snap?

it is snap

Thank you for the response. I don't actually know why it happened on your machine.
I used Manjaro and it works perfectly fined to me and there are lots of Ubuntu 20.04 users too but you are the one that I know that experience that bug.
I will try to figure it out as soon as I can @tonytvo . I'm really sorry.

I just installed Ubuntu and I can confirm that it doesn't really work. Thanks for informing me @tonytvo .
I will try to figure out how to fix this problem as soon as I can.

@tonytvo I already fixed this bug and released new version of the app.
If this update did fixed the problem please close this issue.
Thanks and have a nice day!

@roldanjr, the latest updates does seem to fix the problem for me on Ubuntu


chore: Add Dockerfile

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


How to pass non-default argparse values as parameters?

As can be found in docs and examples, trains takes arguments from parser during task initialization, and also can connect arguments from dicts and files with task.connect method. But as far as I can see, hyperparameters from Configuration section in web UI serve only for visualization and tracking purposes: changing parameters there doesn't affect values that are passed during execution (it's "one-way street"). I also cannot add arguments into Script Path field, as I would normally do when running it from console.
So, the questions are:

Are there any ways to pass any argparse values other than default ones, when launching experiments from web UI?
Are there any ways at all to change arguments and hyperparameters without modifying source code? Creating a separate Git branch with a new config for every experiment is not very convenient

Hi @kirillfish

But as far as I can see, hyperparameters from Configuration section in web UI serve only for visualization and tracking purposes

Actually, no :)
Let me unveil some of the mystery , when you execute your code in "manual mode" (i.e. from your pycharm/vscode/jupyter etc.)
trains logs everything into the UI, basically it takes the parameters from the code and logs them in the back-end.
When the trains-agent is executing the task (a.k.a "remote execution"), the opposite happens, and the parameters from the UI override the values in the code.
You can find more information on trains-agent in the repository.
Once you have agents running, you can start experimenting with hyper-parameter optimization and pipelines and other automations.
What do you think ?

I'm a bit confused now. I put an experiment into a queue which executes it on a remote agent, using a docker image that I specified. Is it what you call remote execution mode?
In the UI, in Configuration > Hyperparameters > General tab, I can see all the arguments fetched from argparse and configs. So far, so good. Then, regardless of what I change in these parameters through web UI, my code still uses the values defined in the configs and argparse. The values from web UI are either rewritten again or they are never passed in the first place. Are these values passed to the code, and if yes, how do I find the exact place in code where they are updated?

Hi @kirillfish

I put an experiment into a queue which executes it on a remote agent, using a docker image that I specified. Is it what you call remote execution mode?

Correct :)

Configuration > Hyperparameters > General tab, I can see all the arguments fetched from argparse

Wait, why? Argparse arguments should be under "Configuration > Hyperparameters -> Args" ?
If you are using task.connect(...) notice that in "remote execution" the object that will be changing is the object passed to the function call, now if that object is temporary, it will have no effect.
For example (a bad example, so do not use it : )
default_value = 42
task.connect({'param_that_we_cannot_change': default_value})

What happens in remote execution is the connect function is changing the value of the temporary dict we created, not the variable inside the dict! (The actual limit on these kind of things, python does not allow for basic value passing by reference, so we cannot change them).
To the problem at hand:
Why isn't the argparse caught automatically by trains ? Could it be that trains is imported only after the parser parsed the argv? Notice importing trains has to happen before the actual parsing happens, Task.init can be called later, but the import cannot.
What do you think?

Hi @bmartinn, sorry for the late reply

Wait, why? Argparse arguments should be under "Configuration > Hyperparameters -> Args" ?
No, they are always under General tab. Btw, there are also User properties > Properties and Configuration objects > General tabs, and I wasn't able to figure out their meaning. Could you please explain what they are for exactly, and how to modify them from the code? This is not as important as finding out the answers for the initial two questions though

trains is imported in the imports section, before the parsing happens of course. Argparse is caught by trains, but only the default values. The problem at hand (one of) is how to pass any values rather than the default ones, when running experiments through web UI.
I only pass dicts that were already defined into task.connect, so they are not temporary objects.

The second question was how to change hyperparameters in web UI before running the experiment

Hi @kirillfish

No, they are always under General tab

With v0.16  we introduced sections and names, I'm assuming you have previous trains-server/trains version and this is why it is under General (I would recommend to upgrade both the trains-server and trains to the latest versions)
I'll start with the last question, and then go back from there:

how to change hyperparameters in web UI before running the experiment

Once you executed an experiment it becomes completed (or aborted) and frozen (i.e. non-editable).
You can "Reset" an experiment (right click -> "reset"), which clears the results and makes all the fields editable (and changes the experiment status to "draft")
You can "publish" an experiment (right click -> "publish"), locking this experiment and making sure no one will be able to reset/change it.
You can "clone" an experiment (right click -> "clone"), which creates a new editable copy of the experiment (status "draft")
"Draft" experiments are fully editable in the UI. Once you are done editing, enqueue them for execution.
Trains has three different types of Configurations:

User Properties: key/value pairs that are always editable, allowing you to add information on experiments even post execution, for example backbone_type=ResNet18 . You can later add them to the experiments' table view  by clicking the cogwheel on the right corner of the table, and adding custom columns ("+hyper-parameters").
Hyper Parameters: Different section names, separate different types of hyper-parameters, for example argparse under "Args", dictionaries by default under "General", etc. A user can add multiple sections of hyperparameters from anywhere in the code by calling Task.current_task().connect(my_dict, 'section_name') . Like "User Properties" you can add these parameters as custom columns to the experiments' table.
Configuration Objects: These are either configuration files or a dictionary representing a complex nested configuration object.
In both cases, you to edit the configuration file (or dict) as text , then get the text as a file/object in code, see details here

Let me know if I missed anything :)

@bmartinn thank you for the detailed answer, but you didn't answer the questions...
1. Passing any argparse values rather than default ones.
To put it in another way, suppose I run some random experiment locally with no trains involved, specifying some parameters, say
python3 train.py --learning-rate 0.0003 --batch-size 32 --optimizer adamw

This would use non-default argument values in argparse. The question was: is it possible to use non-default argparse values when running the script in remote execution mode through web UI?
2. Changing hyperparameters through web UI.
I know how to change them in the UI, but the problem is that they are not passed into code during execution. Values that are actually used are always the values that were read with task.connect and task.connect_configuration methods. So the question was: is it possible to change values in the web UI so that these values were actually used by the code?

I partially figured out an answer for the second question:
task.connect() does rewrite locally defined parameters with the ones defined through web UI, but this is applied only to configs, but not to argparse values (if there are argparse arguments with the same name, they remain intact)

Hi @kirillfish

Passing any argparse values rather than default ones.
This example is considered "manual" execution, everything you pass to argparse will be logged in the UI.

when running the script using trains, in remote execution mode through web UI?

Yes, but notice the actual execution of the script will be done by trains-agent not manually by you (a.k.a you cannot run python3 train.py  ... and expect to get the parameters from the UI)
I think the main gap here is the following, in order to have the UI control the arguments, you must use the trains-agent to run your code. That means you have to spin an agent trains-agent daemon ... and enqueue the experiment in the UI.
See details on trains-agent here
If you want to execute the code manually, you can however do the following:
Clone the experiment in the UI, change the parameters, copy the experiment ID (press on the ID button next to the experiment name), then run:
trains-agent execute --id <paste_experiment_id_here>

WDYT?

Hi @bmartinn, sorry for the delayed response. Yes, this is exactly what I was looking for, thank you. That's why I decided to get rid of argparse in my code altogether, in order not to mix things up when argparse values interfere with config values.
I'll check out manual execution on the agent, but this is not very convenient option (it's much better to manage all queues, agents and experiments from web UI). So the most viable option right now is to clone experiments and change parameters from the web UI


Add documentation showing how all methods and functions work to close #9

[x] change the periodicy in dependabot and add requests limit

Codecov Report

Merging #31 (0c688ac) into main (b2c90eb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #31   +/-   ##
=======================================
  Coverage   50.49%   50.49%
=======================================
  Files          11       11
  Lines         202      202
  Branches       54       54
=======================================
  Hits          102      102
  Misses        100      100

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Final group capstone: set up your Kanban board

In this pull request, We were able to:
Set up a kanban board to manage our project tasks: By organizing our tasks into different stages and columns, we can better visualize our progress and plan our next steps more effectively.
Created a project to manage the entire scope of the project: By having a centralized location for our project files and documentation, we can ensure that everyone is on the same page and that nothing falls through the cracks.
Created two separate repositories for front-end and back-end development: By separating our code into two distinct repositories, we can better manage changes, dependencies, and version control.
Created issues and assigned them to the appropriate repositories: By breaking down our tasks into smaller, more manageable chunks and assigning them to specific team members, we can ensure that everyone is working on the right things at the right time.
Thanks for the review.
We are a group of 5

Hi @dgcuenca,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
To highlight:

All tasks have been assigned✔

Required Changes ♻
At the time of this review, however, these are the pointings:
@paulsaenzsucre 44
@Juank628 46
@gjuliao 46
@dgcuenca 32
@belaymit 6
The general rule is that nobody can have more than 5 points than any other team member.  That includes the whole team.
As you can see, @dgcuenca has 32 points, which is a lot less than @gjuliao who has 46.  Also, @belaymit has just 6 points.
Therefore:

[ ] Kindly, make sure no one in your team has more than 5 points than any other member.

That way, we will meet this particular requirement. 💪
Tip:  You could reassign tasks, move the numbers, change the values.

[ ] This task does not have any points:

Kindly add points to it.  While on it, kindly remember that no team member may have more than 5 points than any other. 👍
According to the project requirements:

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thanks for your comments!
Changes were done

note: I tried to contact the student to offer help, but got no response this time.
Hi @dgcuenca,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are still some which aren't addressed yet.

Required Changes ♻

[ ] At the time of this re-review, the pointings are as follows:

paulsaenzsucre 44
Juank628 46
gjuliao 46
dgcuenca 29
belaymit 21
Again, @dgcuenca and @belaymit have very few points compared to @Juank628, @gjuliao, and @paulsaenzsucre.
The rule is that nobody can have more than 5 points than any other person on the team.
This means that @dgcuenca and @belaymit should have 41-49 points in order to meet the rule.
If I'm not being clear enough, kindly read the project requirements.
If you need extra help/clarification, kindly bug me on Slack and I'll be happy to help.  👍
Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hello @Meltrust
Thanks for the review, changes were implemented.

Hi @dgcuenca,
Wow, you did it 🎉

Thank you for implementing the changes requested by the previous reviewer 💪 🥇 ㊗
Well done!
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To highlight

Student pointing are now equal, and all tasks have points.✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


iOS: Timeline scroll position is lost after leaving app

Navigate to a scrollable timeline
Scroll down
Close app
Reopen app

The scroll position returns to the top of the timeline.

This is on iPad. And with nothing selected in the timeline and an empty detail view.

After discussing with @brentsimmons, we won't be including this in the 5.0 release of NNW for iOS.  It will probably be part of a follow on release where we also save article scroll position, search text, folder expanded/closed state.


Discussion: Improve PHP track docs for PHPUnit / composer

I think our track docs make things too complicated for new students.
Right after the required installation of PHP we have a short paragraph about installing composer. For PHPUnit we then describe composer as the main means of installation. Using the PHPUnit PHAR is last and "optional". But composer is not required to solve Exercism exercises.
The easiest way to use PHPUnit for Exercism is downloading the PHAR. So we should provide tutorial-style instructions on that first. Also we should check, that exercism test runs the PHAR when using these instructions.
In a more optional looking paragraph we may then describe and explain composer and PHPUnit installation using it. There is a nice short description of composer and installation we may use to improve the composer introduction and installation. We should also mention that composer is wide-spread in the PHP ecosystem.

That sounds like a good approach.


Batched Load Jobs

load_jobs first tries to do a bulk insert into the database with all the jobs. This can fail in two cases: (1) there are too too many jobs, (2) some of the jobs to be inserted already exist in the database. If bulk insert fails, load_job reverts to inserting one job at a time, which is slow.
Proposed change:
If bulk commit fails, revert to "batched commits," wherein batches of jobs (say 1000 jobs at a time) are insert into the database. By using the IGNORE keyword, duplicates can be handled.

Stale


Unblock one chart at a time for midstream

What type of PR is this?
kind/bug
What this PR does / why we need it:
When using the Beta Helm functions, charts were not getting a proper Kustomize file or PullSecrets file.  The charts were being skipped by kustomize.  This fixed a previous bug where subcharts were being targeted by parent level Kustomize files, causing a Kustomize error on deploy, but introduced this new regression where the charts weren't being kustomized at all.
This PR splits the difference.  It prevents charts from being kustomized at the Common Midstream level.  It also prevents subcharts from being kustomized at the parent chart level.  Now, all Kustomize files target each chart discretely, at the appropriate directory level.
Special notes for your reviewer:
Added new comments in the code to prevent future confusion around the UseHelmInstall map.  Please let me know if these comments can be clarified further.
Does this PR introduce a user-facing change?
Helm Charts should get the proper Kustomization for images and pull secrets at each chart level when using the beta Helm Install feature.

Does this PR require documentation?
NONE

I've tested this with three licenses:

no Helm charts (received the correct kustomization.yaml file in midstream)
single chart, no subcharts (received the correct kustomization.yaml file in midstream/chart/chartname)
parent chart with many nested subcharts (received kustomization.yaml files in each chart with only that chart's images listed)


Bard API Async Implementation for v0.1.16 compatible code

Signed-off-by: Nishant<EMAIL_ADDRESS>THIS PROJECT IS IN MAINTENANCE MODE. We accept pull requests for Bug Fixes ONLY. NO NEW FEATURES ACCEPTED!
Bard API Async Implementation for v0.1.16 compatible code
Description
Implemented a separate implementation of bardapi.core.Bard using httpx library for Async API calls.

init() -> init() is where the httpx AsyncClient is instantiated as part of the object and SESSION_HEADERS is being used here.
get_answer() -> get_answer() is async now, so will have to await the response from it.
_get_snim0e() -> _get_snim0e is also async because it is using httpx client to get data from Bard server, and will have to await the response from it.
_extract_links() -> No changes in this method, would not make any difference if it is not synchronous.

Related Issue

Motivation and Context

Giving a user more options by including both Async and Sync implementations.
Chatbots created using bardapi library can be much more efficient.

How Has This Been Tested?
I tested it by manually running it on my local machine and checked the implementation of the functions.

Screenshots (if appropriate):

Dear, nishantchauhan949
Thank you for your valuable contributions and excellent code!
I would like to provide a brief overview of the use cases and advantages of this code. Can you write simple example in readme.md?
Thank you once again for your contribution. I would appreciate it if you could organize the code neatly in the script folder or any other appropriate location.
If the update is not available after 24 hours, I will update it myself.
Thank you in advance.
Regards,
Daniel.

Dear, @dsdanielpark
I have added a brief overview and a simple example in readme.md
Also, I have moved the core_async.py file to the translate_to directory. Hope that is fine.
PR: https://github.com/dsdanielpark/Bard-API/pull/48
Regards,
Nishant


RFM - JetBrains CLion

Why open for maintainers?
Package clion-ide is two years outdated. Maintainer unreachable (account seems to be deserted).
Where do I download it from directly?
Current version available at https://download-cf.jetbrains.com/cpp/CLion-2018.2.4.exe
I assume there's some easy way to get the most recent version to automatically build the package. Maybe have a look over to how intellijidea-ultimate does it.
What's CLion?
Fully Integrated C/C++ Development Environment
Product Page https://www.jetbrains.com/clion/

Package is at https://www.chocolatey.org/packages/clion-ide
@JonasDralle Have you followed the Package Triage Process?

1. I was unable to contact user MichaelsJP due to lacking contact option son his profile. I looked at his profile while logged in.
2. I have not attempted to fix it myself (sorry)
3. I have talked about this in Gitter via an throwaway account associated with<EMAIL_ADDRESS>

@JonasDralle Use the 'Contact Maintainers' link on the package page to contact the maintainer.

done.

@JonasDralle Thanks. If you've not heard anything back after a few days e can make this available to maintainers.

There has been no response from maintainer for over a week.

@JonasDralle can you use the Contact Site Admins link on the package page, to send an official request to help with maintenance of this package, including your username on chocolatey.org.  Thanks

Hi,
I am willing to take over this package. I have contacted the site admins already.
Maurice

@JonasDralle A new package has been pushed, waiting for moderators to approve.

out of interest: how is this done? Do you manually need to push a button for bundling the newest version or do you have a script that scans for new updates and then auto-packages and auto-submits?

Hi Jonas, most of my packages are build automatically using AU and AppVeyor. For more info read the docs

This one is done!


Save Button in System Settings remains greyed out after setting change

Describe the bug
The "Save" button remains greyed out after setting change as shown in below video. Only a change of "Device connection port" triggers the "Save" button.
https://github.com/user-attachments/assets/9e56c998-de60-444b-b3d7-6a011f26d961
To Reproduce
Steps to reproduce the behavior:

Fresh install of Testrun 2.0.1.
Follow steps as in above video.

Expected behavior
Save button should be active following any setting change.
Environment (please provide the following information about your setup):

Ubuntu 24.4.01
Testrun 2.0.1

@jboddey the bug 380231252 has been reported.

Potentially related, I also find on my Testrun 2.0 machine that the Save button is not greyed out at all when the Settings panel is first opened (potentially implying a change has been made, which as you can see from the video below isn't the case).
https://github.com/user-attachments/assets/b8ab36e6-dc5d-4649-83e6-bc1692dd603e
Let me know if unrelated and I can create a new issue for tracking.


Menu items are not returned in api request

In the admin UI the menus show up correctly with the correct menu items/children.
The public access to the menus api requests are open.
When I send a request I get something along the lines of:

As can be seen there are no menu items in the response. No matter what I try to request I can only get the menu itself without the menu items.
This might be due to upgrading Strapi from 4.1 to 4.3.
Any clue why something like could happen?

Hi, please try to add ?populate=* to your URL as described in the section "Example with population" of https://github.com/mattmilburn/strapi-plugin-menus#api-usage

Of course I tried that as well, also tried using populate with the field names.
I entered the node_module into the menu.js controller and added a log right after the findOne command and no menu items inside.

OK, Found the issue, feeling kind of silly, but the public permissions were only for the menu findOne and not the menu items.
Though before updating I am pretty sure I did not need public permissions for the menu items for the menu method to show me the menu items.

Hi @Bodokh Glad you were able to resolve your issue!
The earlier versions of the menus plugin was too opinionated and lacked certain customizations. So when v1.0.0 was released, it represented a major refactoring and much of the config for Menu and MenuItem schemas were put back into the hands of the developer. This provides a lot more freedom for the developer and in turn, a little bit more responsibility. 🙂

@mattmilburn  Is there any way I can get a submenu item via API?


Move from using PUSH_END to signal to end of the segmenting flow to using RECORDING_END

Tested this PR locally and worked as expected - need to sanity check on canary prior to merging.


[withStyles] Fix JSS issues in IE11 in development

Fixes #12744
@oliviertassinari Hoping that's what you meant by simplified polyfill, not sure if i stripped away too much, but loads fine in IE11 and passes all tests, so 🤞

It's working 👌

@novascreen Thank you!


Thresholds as variables

Would it be possible to permit thresholds as variables of the dashboard. This way the thresholds could actually set in the database (query) or as simple constants of lists of values directly in the dashboard. This makes it easier to actually use Prometheus of other TSDB that manages the alerts and have the UI Grafana with Flowcharting match the same thresholds.
Threahold: $threshold1, $threshold2
Where $threshold1 = 30, $threshold2 = 60
Or
$threshold1 = query against TSDB for a specific metric.
$threshold2 = query against TSDB for a specific metric.

Any chance you will have time for this for version 0.8?
I tested version 0.7, I like it a lot, much easier to work with rules and metric sources.


Update Get-SPBingMapsBlock.md

Format exmaple

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Get-SPBingMapsBlock.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix: add configmap watch rbac

What this PR does / why we need it:
Manager logs report an error of failing to watch configmaps:
{"level":"error","ts":1660752774.0649443,"msg":"pkg/mod/k8s.io/client-go@v0.22.9/tools/cache/reflector.go:167: Failed to watch *v1.ConfigMap: unknown (get configmaps)\n","stacktrace":"k8s.io/client-go/tools/cache.(*Reflector).Run.func1\n\t/go/pkg/mod/k8s.io/client-go@v0.22.9/tools/cache/reflector.go:222\nk8s.io/apimachinery/pkg/util/wait.BackoffUntil.func1\n\t/go/pkg/mod/k8s.io/apimachinery@v0.22.9/pkg/util/wait/wait.go:155\nk8s.io/apimachinery/pkg/util/wait.BackoffUntil\n\t/go/pkg/mod/k8s.io/apimachinery@v0.22.9/pkg/util/wait/wait.go:156\nk8s.io/client-go/tools/cache.(*Reflector).Run\n\t/go/pkg/mod/k8s.io/client-go@v0.22.9/tools/cache/reflector.go:220\nk8s.io/apimachinery/pkg/util/wait.(*Group).StartWithChannel.func1\n\t/go/pkg/mod/k8s.io/apimachinery@v0.22.9/pkg/util/wait/wait.go:56\nk8s.io/apimachinery/pkg/util/wait.(*Group).Start.func1\n\t/go/pkg/mod/k8s.io/apimachinery@v0.22.9/pkg/util/wait/wait.go:73"}
Adds configmap watch permission back to eraser manager role to remove error.
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #
Special notes for your reviewer:

Codecov Report

Merging #375 (c210835) into main (b6bd205) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #375   +/-   ##
=======================================
  Coverage   22.27%   22.27%
=======================================
  Files          10       10
  Lines         597      597
=======================================
  Hits          133      133
  Misses        452      452
  Partials       12       12

Flag
Coverage Δ

unittests
22.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Lab 2 Minor Correction

Run an automated machine learning job
Step 1.
Author, should be "Authoring"

Thanks @tumultuous1, I'll make an adjustment!


How to handle errors

I have a long-running instance of a pusher client. I'm trying to figure out how best to deal with dropped connections. In my logs I see things like this:
Error { type: 'WebSocketError',
  error: 'Please reconnect immediately' }

I haven't been able to recreate that message in a debugging environment - but I can easily get connection errors by disabling my internet connection:
Error { type: 'WebSocketError',
  error: 'Unknown socket error: ENOENT getaddrinfo' }

As far as I've been able to tell - there are no events or public functions for me to be able to monitor pusher error messages. Am I missing something?

Thanks!


python310Packages.click-option-group: 0.5.3 -> 0.5.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.click-option-group is: "Option groups missing in Click"
meta.homepage for python310Packages.click-option-group is: "https://github.com/click-contrib/click-option-group"
meta.changelog for python310Packages.click-option-group is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.5 with grep in /nix/store/shc3pa41cz9ydzk16bjbmg00rxxi95g5-python3.10-click-option-group-0.5.5
found 0.5.5 in filename of file in /nix/store/shc3pa41cz9ydzk16bjbmg00rxxi95g5-python3.10-click-option-group-0.5.5

Rebuild report (if merged into master) (click to expand)

12 total rebuild path(s)

12 package rebuild(s)

First fifty rebuilds by attrpath
checkov
matrix-synapse-tools.synadm
pynitrokey
python310Packages.click-option-group
python310Packages.cloudsplaining
python310Packages.spsdk
python310Packages.typed-settings
python39Packages.click-option-group
python39Packages.cloudsplaining
python39Packages.spsdk
python39Packages.typed-settings
semgrep

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/shc3pa41cz9ydzk16bjbmg00rxxi95g5-python3.10-click-option-group-0.5.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.click-option-group https://github.com/r-ryantm/nixpkgs/archive/2601e8bf883a1570c39132325c53efb2b9eb2c4a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/shc3pa41cz9ydzk16bjbmg00rxxi95g5-python3.10-click-option-group-0.5.5
ls -la /nix/store/shc3pa41cz9ydzk16bjbmg00rxxi95g5-python3.10-click-option-group-0.5.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages failed to build:

checkov
pynitrokey
python310Packages.spsdk
python39Packages.spsdk

8 packages built:

matrix-synapse-tools.synadm
python310Packages.click-option-group
python310Packages.cloudsplaining
python310Packages.typed-settings
python39Packages.click-option-group
python39Packages.cloudsplaining
python39Packages.typed-settings
semgrep

Maintainer pings
cc @mweinelt for testing.

Failures are on target branch.


Importing OpenIdConnectProvider

Hello!

Vote on this issue by adding a 👍 reaction
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already)

Issue details
Importing an OpenIdConnectProvider fails. I believe this is because the object returned by the AWS API call does not prefix the url with "https://" but the validation requires it.

Steps to reproduce

Create an OpenIDConnectProvider by some external means
Try to import it

Expected:
It imports successfully
Actual:
It fails to import with a validation error

The documentation suggests you need to import via ARN (https://www.pulumi.com/registry/packages/aws/api-docs/iam/openidconnectprovider/#import) -- have you tried that and does that work?

Hi.
I discovered the issue by following the documentation and importing using
the ARN.
It does not appear to be working, which is why I reported the issue. :)
On Wed, 8 Dec 2021 at 19:38, Lee Zen @.***> wrote:

The documentation suggests you need to import via ARN (
https://www.pulumi.com/registry/packages/aws/api-docs/iam/openidconnectprovider/#import)
-- have you tried that and does that work?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pulumi/pulumi-aws/issues/1726#issuecomment-989081645,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGX2F6SBONCVYY25O6IL7DUP6Q4DANCNFSM5JJZPGOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@robotlovesyou Sorry, I must have misunderstood your original message then. When you said "the object returned by the AWS API call does not prefix the url with "https://" but the validation requires it." -- do you mean that the url property isn't coming back 'cleanly' from what the provider expects? If so, have you tried programmatically importing the resource (vs. using import on the command line)?

Hello.
Yes, you are correct that it appears that the AWS api does not return the
url in the format expected by the provider, which is why it is failing.

I was not aware that programmatic import was possible.
I’ve worked around this issue by removing the resource and re-creating it
with Pulumi so it is not a blocker for me, but I wanted to bring the issue
to your attention.
On Thu, 9 Dec 2021 at 02:22, Lee Zen @.***> wrote:

@robotlovesyou https://github.com/robotlovesyou Sorry, I must have
misunderstood your original message then. When you said "the object
returned by the AWS API call does not prefix the url with "https://" but
the validation requires it." -- do you mean that the url property isn't
coming back 'cleanly' from what the provider expects? If so, have you tried
programmatically importing the resource (vs. using import on the command
line)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pulumi/pulumi-aws/issues/1726#issuecomment-989381634,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGX2FY6JVGOEOLCW6NJQELUQAAHDANCNFSM5JJZPGOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Import functionality has been significantly refactored in #8922 to substantially reduce the number of failures during pulumi import and make the generated code more truthful and complete. We haven't tested this exact issue, but we believe it's addressed by the refactoring.
I'll go ahead and close the issue - feel free if you still get issues with it after upgrading to the Pulumi CLI version 3.26.1 or above.
Read more about the changes in Easier IaC adoption with improved pulumi import experience.


Terminal picker is off

Issue Type: Bug
VS Code version: Code - Insiders 1.37.0-insider (8fa811108f0f0524c473020ef57b6620f6c201e1, 2019-07-12T09:17:10.495Z)
OS version: Darwin x64 17.7.0
Remote OS version: Linux x64 4.18.0-1024-azure
I'm connected remotely to a VM in Azure over SSH.  I noticed that my terminal picker looked like this:

And when i clicked on it, the separator was checked!

Once I clicked on a terminal (e.g. I clicked on bash) the problem went away.

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2200)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
16.00GB (1.74GB free)

Process Argv
-psn_0_12962908

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: cdias-devbox

OS
Linux x64 4.18.0-1024-azure

CPUs
Intel(R) Xeon(R) CPU E5-2673 v3 @ 2.40GHz (2 x 2394)

Memory (System)
7.77GB (6.24GB free)

VM
0%

Extensions (19)

Extension
Author (truncated)
Version

vsc-material-theme
Equ
29.2.0

vscode-peacock
joh
2.4.0

dotenv
mik
1.0.1

vscode-docker
ms-
0.7.0

remote-containers
ms-
0.66.0

remote-ssh-edit-nightly
ms-
2019.7.21060

remote-ssh-explorer-nightly
ms-
2019.7.21060

remote-ssh-nightly
ms-
2019.7.21060

remote-wsl
ms-
0.38.10

vscode-remote-extensionpack
ms-
0.15.0

cloudenv
ms-
1.0.488

cloudenv-explorer
ms-
1.0.488

cloudenv-extensions
ms-
1.0.488

debugger-for-chrome
msj
4.11.6

vscode-openapi
42C
1.8.3

vscode-azurefunctions
ms-
0.17.1

vscode-kubernetes-tools
ms-
1.0.1

azure-account
ms-
0.8.4

vscode-yaml
red
0.4.1

(2 theme extensions excluded)

looks like it happens on first connect...

I repro'd this by turning on show login terminal and connecting to an ssh box.

This appears to happen because of the short-lived "install" terminal

This is merged, I'll manually verify it in tomorrow's build though.

This wasn't actually fixed

Another repro in https://github.com/microsoft/vscode/issues/78405

I think this is fixed


Install without CocoaPods

Hi.
What is the process for installing this in my project without using CocoaPods?
Thanks.

 For manual installation, you can just clone the repo git clone git@github.com:YardiSystems/YSRangeSlider.git or download the files from Github.


Menu not opening

Title. On Windows 11 using NinjaCS as injector. Everything seems to inject fine but upon pressing insert the menu simply does not open, in game or at main menu. I have tried pressing other keys as I wasn't sure if Insert was the menu key (although it usually is). I have tried pressing Insert, Home, Ctrl, Alt and combinations of holding Ctrl/Alt with Insert/Home.

It doesn't open the same as how it did in CS:GO. Instead there's a new button besides the power one (in top left). Looks like an insect. That is osiris.

It doesn't open the same as how it did in CS:GO. Instead there's a new button besides the power one (in top left). Looks like an insect. That is osiris.

Ok i just checked now but all it has is remove scope overlay, bomb settings and preserve killfeed. Is that really all its got now? I remember it having actual cheat features.

You can see the features we currently have here: https://github.com/danielkrupinski/Osiris/pull/4147
I'm guessing CS2 has a completely different framework to CS:GO, so Osiris will have to be recoded fully, just have to be patient and wait for updates to roll out.


Features breaking without warning

Keep having features just stop working in the middle of working in a file.
Inteli-sense stops working, tab won't indent, cntrl+s won't save.
I have to exit and restart. Super Frustrating. Just started on multiple windows 11 pcs.

Can you follow the instructions to report a bug in the ISSUE_TEMPLATE, and provide more information to help us identify the problem and give you the best supports.

/confirmPlease

Please diagnose the root cause of the issue by running the command F1 > Help: Troubleshoot Issue and following the instructions. Once you have done that, please update the issue with the results.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Isn't this project only zkHE?

Looks like an awesome project that I need to understand better.
As a starting question, isn't this just homomorphic (not fully) encryption though? You use the cupcake library which only supports additions (not multiplications).

thanks for interest @samlaf
yep, the original goal was to show a PoC for making FHE verifiable with zk using something like zamah but was too heavy that day
so i fallback on a simple version of HE i.e. cupcake
there are recent advancements, below some pointers:

https://github.com/hashcloak/fhe_risc0_zkvm by @hashcloak from @ewynx using fhe.rs by @tlepoint and @samtay
https://github.com/risc0/risc0/discussions/714
https://succinct.xyz/

i like these topics so happy to collab. what are you looking to build?

btw, i think zk and fhe could be good techs to explore for MEV related challenges/proposals like inclusion proofs, mempool encryption, pbs

Thanks for the links! FHE is still very new to me, but def interested in exploring more.
I don't understand what "making FHE verifiable with zk". I always assumed the opposite was needed: that fhe requires zk proofs that the input it is about to compute on was correctly encrypted (and doesn't contain data that would break my server's data structure, say by forcing double payment or something).
I'd be curious to know how you think fhe/zk can help with mempool encryption.
One problem I'm interested in solving is doing private DAS (aka having a network of nodes being able to do DAS on a server that holds the data, without leaking metadata such as ip address so that the server could selectively disclose information to only the light nodes that it desires, and hence manipulate the network). Typical approaches to this problem are "put a mixnet in front of DAS", but I've recently stumbled upon the idea of Private Information Retrieval (eg https://www.youtube.com/watch?v=1cd4t9OL0iM&list=PLuhRWgmPaHtSEv2DM9J7nSvN7dnVFMM6q&index=140&t=1339s), which seems like it could be used here. It seems like there's also an HE/FHE solution lurking in the background (or TEE with ORAM), but I'd be curious for any thoughts on this.

Yep, that's another guarantee you can add.
The TLDR for wrapping an FHE with zk is this thread
Re mempool encryption, fhe on mempool means you can hide block content but be sure you exec the ordering/inclusion as intended. Btw still an open problem, zk can tackle from a different perspective like delayed inclusion with commitment e.g. you ask to be included providing a proof for the block without providing it but providing only at late stage but cannot cheat because you committed to with zk. Would love to hear yours.
Re private DAS, that's pretty interesting, PIR is a good tech. I found Blyss this summer during zk conf in Paris, I recommend to take a look. On the other end, probably you can map DAS to some data science problem, so I would look also to OpenMined
Don't know at all TEE with ORAM, do you have any good pointers?

Ah makes total sense. So the zkp I was talking about was for client to prove the authenticity of the input data its sending, whereas the zkp you're talking about is for the server to prove the authenticity of the output it's sending back (that it was calculated properly). Kind of sounds like the reverse of "two-way" tls sessions for some reason haha. In tls sessions server always authenticates itself to client, and only two-way tls sessions have the client also authenticate itself. Here with fhe servers it seems like its the opposite. The client always has to authenticate its input to the server, and sometimes, the user might also want a zkp to authenticate the output (if it doesn't have some other way to authenticate it).
Is it fair also to say that with TEE you get privacy + authenticity, whereas FHE only gives privacy, and you need to combine it with zkps to get authenticity? zk by itself, when used for scalability (some external server is computing something for you) loses the privacy feature.
I guess by fhe mempool, you just mean fhevm right? Like what zama is building? Because there are schemes for threshold encrypted mempool which give you "fair" ordering guarantees, and they don't require FHE at all: you threshold decrypt the txs after having committed to them, and only then execute them. See for eg shutter network.
Will look at those references, thanks!
for tee and oram, see for eg

https://medium.com/@dsl_uiuc/tee-rollups-fixing-access-patterns-in-tee-based-smart-contracts-with-off-chain-computing-b2f3acfe335f
https://signal.org/blog/building-faster-oram/ (I was really surprised when I learned that signal uses TEEs and oram!)


buildbot: Python 3 support and other improvements

Motivation for this change
The latest versions of pylint and astroid do not support Python 2, which breaks buildbot (#45782) because it only supports Python 2 in nixpkgs. Buildbot supports Python 3, but this is potentially a breaking change because buildbot uses Python in its configuration files. Therefore, this PR adds configurable support for Python 2 or 3, which Python 2 being the default. The 1.0 branches of pylint and astroid are still supported until Python 2's EOL, so they are used to build buildbot with Python 2.
This PR supersedes #45789.
Things done
First, I updated and added Python 3 support to sphinx-jinja. This mainly required fixing how the tests were run. This commit supersedes #45763.
I updated tempita to use a fork that adds Python 3 support. The upstream tempita package appears to be abandoned. This fork is used by the official Arch Linux package of tempita, but is somewhat undesirable because it is not an official release. This commit supersedes #39692.
I updated astroid and pylint to their latest versions, and added astroid1 and pylint1 packages which use the still maintained 1.0 branch of these packages. Because there are a number of different ways to implement packages with multiple versions in Nix, I'm not sure if I chose the best way.
While I was testing I ran into #40273, which is supposedly fixed but does not seem to be (see https://github.com/NixOS/nixpkgs/issues/40273#issuecomment-417730406). To work around this I changed hyperlink to not use pytest to run its tests (the default checkPhase seems to work fine) and made some changes to autobahn (which do not fix the problem, but only one package needs to be fixed for the conflict to be resolved). These changes are not critical, because they should not be necessary once #40273 is truly fixed, but I included them anyway because I believe they result in cleaner derivations.
I then updated all the components of buildbot to 1.3.0. buildbot-worker had already been updated by an automated update, but the rest of the components had not. This has happened a number of times recently and sometimes causes problems, especially when buildbot-pkg is updated without updating the plugins.
The last commit adds Python 3 support as well as a number of other improvements that I have been running locally for about a year. There are no intentional breaking changes, but I have not tested every possible scenario.
All the buildbot packages are moved to pythonPackages. A new python attribute is added to each package's passthru, allowing the NixOS module to determine which version of Python to use. The tests are currently disabled for both Python 2 and 3, because a test fails due to what appears to be a bug in buildbot. I attempted to disable the failing test, but this simply causes the error to move to a different test and so on. The error message (see comment above doCheck = false) relates to a recently added buildbot feature, so this points to a problem with buildbot.
I changed how the plugin wrapper works, making the wrapped package into a Python module that can be added to a Python environment using python.withPackages.
In the buildbot-master NixOS module, I switched to directly invoking twistd (as was already done with buildbot-worker) in order to log to stdout and therefore the systemd journal, rather than a file. I also added a new pythonPackages option, which takes a function that is passed to python.withPackages and is used to set the service's PYTHONPATH. I also generate the twisted tac file within the module, rather than using create-master (create-master is still necessary to create the database). This gives more control over the initialization process (especially useful for buildbot-worker).
In the buildbot-worker NixOS module, I made similar improvements to the PYTHONPATH handling, but did not add a pythonPackages option because workers never run user Python code AFAIK. I also generate the tac file in the module, which makes it possible to implement the workerPassFile option (checks one of the boxes in #24288). I also added hostMessage and adminMessage options, which allow users to declaratively set those messages.
The buildbot NixOS test passes, including the commented out daemon test, which supposedly was previously broken. I configured the tests to run with both Python 2 and 3.
I would appreciate it if other buildbot users would test this PR. I have only run the tests and used it with my buildbot deployment, which has a number of customizations with might obscure bugs.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

cc @dotlambda

I think I have addressed all the issues that were brought up. I had to add a few new commits to fix autobahn's tests with Python 2, and a commit to add a patch to sqlalchemy_migrate to make it compatible with Python 3.6 (I previously had this in an overlay and forgot about it).
@nand0p I kept you as the maintainer of the 1.x branches of astroid and pylint. Is this okay?
Buildbot just updated to 1.4.0, and its tests now pass for Python 2 and 3.

@FRidh @dotlambda When you get a chance, could you take a look at the changes I made to address your comments?

I rebased against master and moved the changelog entry to the 19.03 changelog.

@GrahamcOfBorg test buildbot.python2 buildbot.python3

Success on aarch64-linux (full log)
Attempted: tests.buildbot.python2, tests.buildbot.python3
Partial log (click to expand)
Oct 11 22:13:51 bbworker mqa4mvwqafrn53kcwzvxysr4p5llcpmx-unit-script-buildbot-worker-pre-start[753]: mkdir: created directory '/home/bbworker/worker/info'
Oct 11 22:13:51 bbworker systemd[1]: Started Buildbot Worker..
total 12
drwxr-xr-x 3 bbworker bbworker 4096 Oct 11 22:13 .
drwx------ 3 bbworker bbworker 4096 Oct 11 22:13 ..
drwxr-xr-x 2 bbworker bbworker 4096 Oct 11 22:13 info
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/7mnda9bfn2czxnhw4p42hbw4gviwdhb8-vm-test-run-buildbot
/nix/store/qja43p34slaqchzr0pdr5jcd8s90szjn-vm-test-run-buildbot

Success on x86_64-linux (full log)
Attempted: tests.buildbot.python2, tests.buildbot.python3
Partial log (click to expand)
Oct 11 22:58:14 bbworker systemd[1]: Starting Buildbot Worker....
Oct 11 22:58:14 bbworker systemd[1]: Started Buildbot Worker..
total 12
drwxr-xr-x 3 bbworker bbworker 4096 Oct 11 22:58 .
drwx------ 3 bbworker bbworker 4096 Oct 11 22:58 ..
drwxr-xr-x 2 bbworker bbworker 4096 Oct 11 22:58 info
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/2p09g21258a4dahpyqr9a598gpaif8ij-vm-test-run-buildbot
/nix/store/skbj8hg7in1581fhzm0wll0ycc6w4dd1-vm-test-run-buildbot

@Infinisil I fixed the issues you brought up.
Should this go into staging?
Also, there may be some buildbot users on 18.03 who will find their configs broken when they upgrade to 18.09, so we may want to backport this eventually. I'm not sure if we would backport the whole PR, or just the minimum to fix the build.

I think master is alright. Feel free to open another PR with a backport if you feel it's necessary.
Thanks for the PR, the NixOS test is very nice too :)


InflatePaths with positive offset returns empty result

I have a simple square with sides 10000. When trying to offset it to the outside (by using a positive offset), I get results as expected up to an offset of 5000. If I try to use a value of 5001 I get an empty result. The following test fails, for example::
TEST(ClipperBug, bug)
{
	double offset = 5001;
	double arcTolerance = 10;

	Clipper2Lib::Paths64 subject;
	subject.push_back(Clipper2Lib::MakePath({0, 0, 0, 10000, 10000, 10000, 10000, 0}));

	auto result = Clipper2Lib::InflatePaths(
		subject,
		offset,
		Clipper2Lib::JoinType::Round,
		Clipper2Lib::EndType::Polygon,
		2,
		arcTolerance);

	EXPECT_TRUE(!result.empty());
}

If I try to change the size of the square, a similar pattern is followed e.g. a side 20000 square stops generating offsets once the offset reaches 10001.
It sounds weird for such a simple example to be failing, but it looks like a bug to me.

Hmm progression of area with offset looks really weird:
Area with offset 0 is: -1e+08
Area with offset 501 is: -1.20808e+08
Area with offset 1002 is: -1.43196e+08
Area with offset 1503 is: -1.67158e+08
Area with offset 2004 is: -1.92695e+08
Area with offset 2505 is: -2.19812e+08
Area with offset 3006 is: -2.48506e+08
Area with offset 3507 is: -2.78776e+08
Area with offset 4008 is: -3.10621e+08
Area with offset 4509 is: -3.44048e+08
Area with offset 5010 is: 0
Area with offset 5511 is: 0
Area with offset 6012 is: 0
Area with offset 6513 is: 0
Area with offset 7014 is: 0

It seems offsetter doesn't like CW-oriented holes, and tries to offset them as bodies at first, but if offset value is too huge, some rough check kicks in and quickly returns empty result, because hole with such huge offset should collapse. I would say it is inconsistent.
TEST(ClipperBug, bug)
{
	for (int i = 0; i < 15; ++i)
	{
		double offset = 501;
		double arcTolerance = 10;

		Clipper2Lib::Paths64 subject;
		subject.push_back(Clipper2Lib::MakePath({0, 0, 0, 10000, 10000, 10000, 10000, 0}));

		auto result = Clipper2Lib::InflatePaths(
			subject,
			offset * i,
			Clipper2Lib::JoinType::Round,
			Clipper2Lib::EndType::Polygon,
			2,
			arcTolerance);

		std::cerr << "Area with offset " << offset * i << " is: " << Clipper2Lib::Area(result) << std::endl;
	}
}

This is a slight variation on the above test showing some results.

TEST(ClipperBug, bug)
{
	double arcTolerance = 10;

	Clipper2Lib::Paths64 subject;
	subject.push_back(Clipper2Lib::MakePath({0, 0, 0, 10000, 10000, 10000, 10000, 0}));

    auto doOffset = [&](double value){

        auto result = Clipper2Lib::InflatePaths(
            subject,
            value,
            Clipper2Lib::JoinType::Round,
            Clipper2Lib::EndType::Polygon,
            2,
            arcTolerance);

        for(auto const & r : result)
            RenderClipperPathAsPl(r, "offset " + std::to_string(value));
    };

    RenderClipperPathAsPl(subject[0], "source");
    doOffset(4500);
    doOffset(4900);
    doOffset(5000);
    doOffset(5001);
    vis::Show();
}

Notice that the input curve is clockwise and 5001 is missing.
If I change the test so that the curve is oriented CCW then 5001 and further are not missing.

TEST(ClipperBug, bug)
{
	double arcTolerance = 10;

	Clipper2Lib::Paths64 subject;
	subject.push_back(Clipper2Lib::MakePath({
        10000, 0,
        10000, 10000,
        0, 10000,
        0, 0
    }));

    auto doOffset = [&](double value){

        auto result = Clipper2Lib::InflatePaths(
            subject,
            value,
            Clipper2Lib::JoinType::Round,
            Clipper2Lib::EndType::Polygon,
            2,
            arcTolerance);

        for(auto const & r : result)
            RenderClipperPathAsPl(r, "offset " + std::to_string(value));
    };

    RenderClipperPathAsPl(subject[0], "source");
    doOffset(4500);
    doOffset(4900);
    doOffset(5000);
    doOffset(5001);
    vis::Show();

}

There is a small debate here as to what the orientation requirements for clipper are.

I cannot reproduce it with latest version.


Circuit Construction Kit: DC & DC-Virtual Lab 1.2.2-rc.2 Release Candidate with Resistance/Resistivity improvements

In https://github.com/phetsims/circuit-construction-kit-common/issues/687 we made improvements for the resistance and resistivity, to solve the issues listed in the milestone: https://github.com/phetsims/circuit-construction-kit-common/milestone/3
There was a recent RC for DC, and a subsequent re-release, so it has fairly recent SHAs.  We just need a spot check for the issues and a sanity test that nothing unexpected was disturbed.
There was a problem in RC.1, but RC2 is ready to go:
https://phet-dev.colorado.edu/html/circuit-construction-kit-dc/1.2.2-rc.2/phet/circuit-construction-kit-dc_en_phet.html
https://phet-dev.colorado.edu/html/circuit-construction-kit-dc-virtual-lab/1.2.2-rc.2/phet/circuit-construction-kit-dc-virtual-lab_en_phet.html

[ ] https://github.com/phetsims/circuit-construction-kit-common/issues/676 Battery resistance should not go to zero
[ ] https://github.com/phetsims/circuit-construction-kit-common/issues/245 Light Bulb stays lit during short circuit
[ ] https://github.com/phetsims/circuit-construction-kit-common/issues/685 When the battery resistance is set to "Tiny", then there should be no resistance readout on the battery
[ ] https://github.com/phetsims/circuit-construction-kit-common/issues/683 Change Battery Resistance Slider lower limit to "Tiny"
[ ] https://github.com/phetsims/circuit-construction-kit-common/issues/688 Wire Resistance: Lots should match published version

@arouinfar can you please test these or give guidance on how QA team should test them? Particularly the first two.
QA should also please note which platform(s) and how long was taken for sanity testing.  I think 1-2 platforms and minimal time for that is fine.

https://github.com/phetsims/circuit-construction-kit-common/issues/688

On hold until the https://github.com/phetsims/circuit-construction-kit-common/issues/688 is addressed

[ ] Win 10 Chrome (Time = )
[ ] Win 10 Firefox (Time = )
[ ] MacOS Chrome (Time = )
[ ] MacOS Safari (Time = )
[ ] iPadOS (Time = )
[ ] ChromeOS (Time = )

@KatieWoe -
Not expecting any of these changes to be device-or-browser specific. So perhaps fewer device/browser combinations?
Testing should be about 10 minutes of general sanity checking, and then another 15 minutes or so of building crazy circuits and making sure things look good. The change is mostly improving the modeling of the current and electrons.

Since this is a spot check, not all these will be tested. But since we don't know who will be testing on what, I list all of them and the tester fills out the one they did with the time they did.

QA is done


Add mudfish.net to boycott list

They providing VPN services, with Russia servers ( server list is available on https://mudfish.net/server/status )
I submitted ticket about this ( https://forums.mudfish.net/t/topic/59427 ), but they ignored it.
I think that mudfish.net should added to boycott list.

@U-siro thanks for your attention and support.
This one is a bit tricky to decide, as VPNs can be used by russian people to access information (seeking truth, hopefully).
So I would not suggest boycotting VPNs unless they are proved to be evil.


[SPARK-7142] [SQL] [Backport-1.5]: Minor enhancement to BooleanSimplification Optimizer rule

@yhuai @liancheng This PR is to backport #5700 and #8716 to branch-1.5.
Also added two more cases for ensuring it also works in multi-level filters.
Please check if it is appropriate. Thanks!

Test build #48011 has started for   PR 10383 at commit d8bf122.

Test build #48011 has finished for   PR 10383 at commit d8bf122.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48011/
Test PASSed.

Did somebody ask you to backport this?

(I'm wondering about the risk of backporting "features")

@rxin This is part of the offline discussion between @liancheng and @yhuai . Below is the discussion details: https://github.com/apache/spark/pull/10362
Thanks!

@gatorsmile Since it is not for a bug fix, let's not merge it into branch 1.5.

OK, I see. The problem just resolves the performance regression. Anyway, let me close it. Thanks!


Tidy up function calls with use of structs

We could use structs to package up groups of variables to make it more portable between functions. Makes the code more readable and consistent

lack of free time


Update UpdateAutoFollowPatternIT.kt

UpdateAutoFollowPatternIT test "test auto follow stats" was waiting for 30 seconds to poll for shard replication task to start. This was causing intermittent failure as shard replication task may take more time to start.
Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #747 (0185233) into main (7ce444f) will increase coverage by 0.32%.
The diff coverage is n/a.

:exclamation: Current head 0185233 differs from pull request most recent head d1708e7. Consider uploading reports for the commit d1708e7 to get more accurate results

: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     #747      +/-   ##
============================================
+ Coverage     72.92%   73.24%   +0.32%
- Complexity      995     1001       +6
============================================
  Files           141      141
  Lines          4642     4642
  Branches        520      520
============================================
+ Hits           3385     3400      +15
+ Misses          928      920       -8
+ Partials        329      322       -7

see 5 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Added js function to add Star in the required inputs

Related Issue
Closes: #193
Description of Changes
Added a new function in app.js file, which will add a red asterisk symbol in all the required field in the site. It also work on following scenario:

If new required input field is added then it will automatically add asterisk symbol to it
It is independent of the placeholder length it work with every length of placeholder string

Checklist:

[x] Added new function in app.js file
[x] Use dispatchEvent method in new.php file so, to trigger the added function every time random string is generated.

Screenshots

@apoorvaron please review my PR

@priyanka0815 Nice Work🎉


Any ideas what is wrong with my node.js HTTP trigger

Correlation ID: c1e99b37b54a4bc395df2a90b0b165bc
Any ideas what is wrong with my node.js http trigger function.  I didn’t mess with the integration/binding configuration.

Great opportunity to put our correlation ID to the test! :) @ahmelsayed?

So apparently, we're not logging what we should. :( We'll try to improve.

At the very least, I think we should consider any simple fixes which make these issues debuggable as something we should take.

Yes, @ahmelsayed is looking at this right now.

https://github.com/projectkudu/WebJobsPortal/pull/284 fixes this. Sorry about that, I changed the protocol of errors when I added the generic message and that broke ibiza error logging.

Well, I'm not sue we should close this, in the sense that regardless of reporting, some bad thing happened and we don't know what it is. Right?

You can close it and I'll re-open a bug when it happens with the instrumentation.


[Request] Add a output file option for the export command

[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] I have searched the documentation and believe that my question is not covered.

Feature Request
Hello, I think it would be very handy to have a -o parameter in poetry's export command so we could be able to tell poetry to which file we want to export the requirements, independently from the format (which as of now, the only format supported is requirements.txt).
So we could prevent doing such things:
poetry export --dev -f requirements.txt && mv requirements.txt requirements_dev.txt \
    && poetry export -f requirements.txt

Instead, we would have:
poetry export --dev -f requirements.txt -o requirements_dev.txt \
    && poetry export -f requirements.txt

Also, it would be completely cross platform.

Funny. You posted this ticket just 45 minutes after #1014 on the same subject.

Closed by #1035, thanks @richin13!


aot: Fix return code to enable aot runtime tests

Currently, the build_binary function in aot.cpp returns true (1) in the successful path, and false (0) on any errors. However, it should be the other way around, so let's fix that. This enables the runtime-tests.sh --run-aot-tests flag (and any other command that generates an aot binary on the left side of a &&) to work properly. While I'm at it, let's also fix the runtime test command to not need the runtime shim anymore, as the generated binaries can now run on their own.
When ran on my machine, 223 tests pass out of the 389 in the aot suite, so roughly under 60%. A lot of these are because of features that are not yet supported in AOT, which are caught by portability_analyser.cpp .

[ ] Language changes are updated in man/adoc/bpftrace.adoc
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

Sorry, I'm confused, why don't we need the shim for these tests? I see the path to the shim was just modified here: https://github.com/bpftrace/bpftrace/pull/3199/files

The runtime tests no longer need to directly manipulate the shim b/c AOT generation codepath can locate the shim on its own now

Yep, also with the change from #3104, it's no longer necessary to call the runtime shim, aka bpftrace-aotrt, to run the generated binary. So before, after you generate with e.g. bpftrace -e 'BEGIN { exit(); }' --aot a.btaot,  you would have to do bpftrace-aotrt a.btaot. But now you can just run ./a.btaot directly. So this change makes the runtime tests follow that pattern by omittingbpftrace-aotrt as well.


Update Bootstrap and Bootswatch themes to v5.2.1

Bootstrap updated to v5.2.1
https://github.com/twbs/bootstrap/releases/tag/v5.2.1
But I am waiting for Bootswatch theme update.
https://github.com/thomaspark/bootswatch
After the Bootswatch theme is updated, I propagate it to Strapdown.js.

It has been updated. I applied it to Strapdown.js.
https://github.com/thomaspark/bootswatch/commit/ef6cf445a99dc4c7b2d7e569a0d5758f72a99b32


Added Logo.png to logo folder PR #17

Completed -- Design a logo for our project #17
Added png file for logo in folder named Logo
Thankyou

remove "high quality service" and you are good to go