Problem with samples.vsphere.common.sample_cli

Hey guys,
i'm using mac with Pycharm, Python ver. 3.7.3.
installed all requirements:
Ignoring suds: markers 'python_version < "3"' don't match your environment
Requirement already satisfied: lxml>=4.3.0 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 1)) (4.3.4)
Requirement already satisfied: pyVmomi>=6.7 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 2)) (<IP_ADDRESS>8.12)
Requirement already satisfied: suds-jurko in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 4)) (0.6)
Requirement already satisfied: vapi-client-bindings==3.0.0 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 5)) (3.0.0)
Requirement already satisfied: vmc-client-bindings in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 6)) (1.9.0)
Requirement already satisfied: nsx-python-sdk in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 7)) (<IP_ADDRESS>.3.13851140)
Requirement already satisfied: nsx-policy-python-sdk in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 8)) (<IP_ADDRESS>.3.13851140)
Requirement already satisfied: nsx-vmc-policy-python-sdk in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 9)) (<IP_ADDRESS>.3.13851140)
Requirement already satisfied: nsx-vmc-aws-integration-python-sdk in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from -r requirements.txt (line 10)) (<IP_ADDRESS>.3.13851140)
Requirement already satisfied: requests>=2.3.0 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from pyVmomi>=6.7->-r requirements.txt (line 2)) (2.22.0)
Requirement already satisfied: six>=1.7.3 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from pyVmomi>=6.7->-r requirements.txt (line 2)) (1.12.0)
Requirement already satisfied: vapi-runtime>=2.9.0 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from vapi-client-bindings==3.0.0->-r requirements.txt (line 5)) (2.12.0)
Requirement already satisfied: setuptools in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from vapi-client-bindings==3.0.0->-r requirements.txt (line 5)) (41.0.1)
Requirement already satisfied: vapi-common-client in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from vmc-client-bindings->-r requirements.txt (line 6)) (2.12.0)
Requirement already satisfied: certifi>=2017.4.17 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from requests>=2.3.0->pyVmomi>=6.7->-r requirements.txt (line 2)) (2019.6.16)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from requests>=2.3.0->pyVmomi>=6.7->-r requirements.txt (line 2)) (3.0.4)
Requirement already satisfied: idna<2.9,>=2.5 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from requests>=2.3.0->pyVmomi>=6.7->-r requirements.txt (line 2)) (2.8)
Requirement already satisfied: urllib3!=1.25.0,!=1.25.1,<1.26,>=1.21.1 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from requests>=2.3.0->pyVmomi>=6.7->-r requirements.txt (line 2)) (1.25.3)
Requirement already satisfied: pyOpenSSL>=18.0.0 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from vapi-runtime>=2.9.0->vapi-client-bindings==3.0.0->-r requirements.txt (line 5)) (19.0.0)
Requirement already satisfied: cryptography>=2.3 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from pyOpenSSL>=18.0.0->vapi-runtime>=2.9.0->vapi-client-bindings==3.0.0->-r requirements.txt (line 5)) (2.7)
Requirement already satisfied: asn1crypto>=0.21.0 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from cryptography>=2.3->pyOpenSSL>=18.0.0->vapi-runtime>=2.9.0->vapi-client-bindings==3.0.0->-r requirements.txt (line 5)) (0.24.0)
Requirement already satisfied: cffi!=1.11.3,>=1.8 in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from cryptography>=2.3->pyOpenSSL>=18.0.0->vapi-runtime>=2.9.0->vapi-client-bindings==3.0.0->-r requirements.txt (line 5)) (1.12.3)
Requirement already satisfied: pycparser in /Users/yishaihl/PycharmProjects/vSphere_Automation/venv/lib/python3.7/site-packages (from cffi!=1.11.3,>=1.8->cryptography>=2.3->pyOpenSSL>=18.0.0->vapi-runtime>=2.9.0->vapi-client-bindings==3.0.0->-r requirements.txt (line 5)) (2.19)
But i can't import sample_cli & sample_util & practically anything imported from vsphere.common.
Any idea why?..
Thanks a lot!

?

I also noticed this between version 1.0.0 and 1.3.0. Try installing 1.0.0 and see if the issue still persists, and then maybe check what changed between versions

1.0.0 version for?

For the vSphere Automation SDK. Version 1.0.0 had no import issues when importing from the package itself, however in 1.3.0 I noticed importing from samples was problematic.

@yishaihl @xeacott Have you added the folder contains these util methods to python path?
https://github.com/vmware/vsphere-automation-sdk-python#first-set-pythonpath-to-use-sdk-helper-methods

youu mean the export PYTHONPATH=${PWD}:$PYTHONPATH ?..

@tianhao64  you mean the export PYTHONPATH=${PWD}:$PYTHONPATH ?..

just to clarify, im using the latest  vSphere-Automation-SDK-1.3.0 and Python 3.7 interperter.

No, but I can do that and report back...

@xeacott hey, you understood waht it means to add the util methods to the Python path?

@xeacott @yishaihl Is it working after adding the sdk root folder to PYTHONPATH?
There are some helper methods in samples folder which are not installed in the python's site-package folder. So to use them you have to add the sdk root folder to PYTHONPATH.
Closing this for now. Please let me know if you still have issue with these imports

Which path we need to add in environment variable?


Feature: Show and print variable taint info

Getting information about taint can be tricky as a dev. I think DevTool could help here, by providing a way to drill down on Blizz variables and check which parts have been tainted.
So this PR adds:

A small 4th column to the table that shows if a variable is tainted. It's empty if not, and shows "Tainted" if it is.
A 2nd line to the right-click print that outputs the name of the addon tainting the variable.

It works on child variables of added table data, as well as top level data that has a value in the global table with the same name.

Hrm, I'm testing it now and the code seems to be working well, though it's seeming to show every addon as tainted. Looking at the DevTool table, every entry is tainted, along with DBM and Details as well. Looking at this function isSecure, taint = issecurevariable([tbl,] variable), it seems like if a variable isn't 'secure' i.e. Blizzard owned, then it will be reported back as tainted. Is this the desired functionality that you were going for? Maybe I'm misunderstanding taint, but if every addon is, by definition, going to report back as tainted, what utility does it provide showing this information as such. (Not trying to be argumentative, I'm genuinely curious.) Right now, this functionality mainly seems to act as a flag if table entries are addon-owned or blizzard-owned.
Further, the attribution of the taint doesn't seem to always be reliable, as all my DevTools's taint was being attributed to Details. I'm assuming this is due to LibStub and library loading. Whichever addon's included libraries get used for that session will take the blame for all taint coming from those libraries, and it's likely just alphabetical order of which addon's libraries get chosen.

Yep this is my understanding as well. Everything from Blizzard is secure unless changed by an addon, and all addon code and data is always considered tainted. I get that it could be a bit confusing showing all addon data as tainted, but that's actually how taint works, and I don't think there is a way to distinguish between Blizz data getting tainted and addon data starting out as tainted.
Similarly with taint attribution, what you said is correct afaik. It's just how Blizz set up the system, and why taint can be so confusing and frustrating.
Maybe it would be better to show secure variables as "Secure" instead of tainted variables as "Tainted"? It would make addon data appear as "normal" and Blizz data as "special" instead of the other way around. And I could also add a "(Possibly)" after the taint attribution output, to make it more clear that it's not 100% reliable. What do you think?

I like the idea of marking the "secure" variables rather than the tainted ones. Likewise, maybe there's a way to do so with an icon instead of text. I.e. a "lock" icon or something of the sort next to the name, rather than having a whole column listed for just the word "secure". I'm sure there's a lock icon in the game somewhere that would fit the bill.
How does that sound? A lock icon next to secure items, and the absence of a lock against insecure items.


[test-triage] rom_e2e_bootup_success

Hierarchy of regression failure
Chip Level
Failure Description
Job chip_earlgrey_asic-sim-vcs_run_default killed due to: Exit reason: User job exceeded runlimit: User job timed out has 2 failures:
Test rom_e2e_bootup_success has 1 failures.
0.rom_e2e_bootup_success.4002847607
Log /container/opentitan-public/scratch/os_regression/chip_earlgrey_asic-sim-vcs/0.rom_e2e_bootup_success/latest/run.log
Job ID: smart:bb6c6f94-eb14-40d3-ac60-60e9619a436f
Steps to Reproduce
Tuesday August 30 2022 07:08:49 UTC
GH Revision: https://github.com/lowrisc/opentitan/tree/6be5b88705e3e48dc9e3de7bca93313528f1274e
Build seed:<PHONE_NUMBER>
Tests with similar or related failures

[ ] Unmapped tests (Just a single test failing)

Assigning @engdoreis
Looks like a timeout issue.
Need to test locally and see if there is still an issue. Then consider increasing timeout length.
Status: Observe/track over the next couple of days

This test has been passing on many nightly.
But it's an unmapped test. Can @alphan or @timothytrippel add a testplan for this test? Then, we can close this issue.

#15113 should fix.

Thanks @timothytrippel


[REPL] Set the correct rpath when installing.

$ORIGIN represents the location of the REPL at runtime,
whenever the binary is located. So, this makes sure
that when we generate the toolchain on one machine and
transfer to another, the repl keeps working, as it needs
to dlopen libSwiftCore, which sits in a path relative
to the executable itself.
rdar://problem/42863249

@swift-ci please test

@aciidb0mb3r this should fix the issue you saw on the downloaded linux toolchain
@shahmishal if you want to give this a try, go ahead.

Seems plausible.. what is $ORIGIN?

$ORIGIN means "this executable", as in "the path at runtime for the dummy repl executable".
The dynamic loader, i.e. ld.so on Linux, expands this variable (and others) if they appear in the tags specified through -rpath.
The trick is that the static linker (ld.bfd or ld.gold) emits a DT_RPATH/DT_RUNPATH tag (depending on the version) containing a path where the dynamic linker should look for dynamic libraries to dlopen().
I found man ld.so and man dlopen pretty enlightening to find out where the issue was. If you have other questions about this patch in particular or the mechanism though, don't hesitate to ask!

Thanks for the explanation!

@dcci Is it possible to add a test to the integration test suite that would catch this?

@dcci Is it possible to add a test to the integration test suite that would catch this?

@gottesmm I thought about this, but it's a little tricky, because it will require inspecting a binary and seeing that rpath is correct. Maybe we can rely on objdump (or llvm-objdump) being there and built and grep for DT_RPATH being there and containing the correct string for the installed executable? What do you think?

@dcci That would be fine. I did something similar to this to make sure that on linux we don't mount any libraries w+x.
See: https://github.com/apple/swift-integration-tests/tree/master/test-snapshot-binaries

Here is where I added llvm-readelf support. I would use llvm-objdump.

https://github.com/apple/swift-integration-tests/blob/e882c92e1f063f5971f11c4163414fd75356f521/lit.cfg#L142

@gottesmm yes, that's pretty much what I had in mind. llvm-readelf is fine, as we don't need to disassemble anything but just dumps the DT_RPATH tags. I'll take a stab at that later tonight on the bus or tomorrow.

SGTM! Thanks Davide!

@gottesmm (and future archeologists), the integration test is here:
https://github.com/apple/swift-integration-tests/pull/31


Simplified SSL Proxy to just use Tailscale

Sorry I was trying to merge into my own master branch and accidentally created a PR on to your fork.


update how tests are run with coverage

Codecov Report

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

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   69.32%   89.92%   +20.6%
==========================================
  Files           8        8
  Lines         665      665
  Branches      117      117
==========================================
+ Hits          461      598     +137
+ Misses        180       43     -137
  Partials       24       24

Impacted Files
Coverage Δ

pygridtools/misc.py
97.34% <0%> (+9.73%)
:arrow_up:

pygridtools/viz/_viz_mpl.py
88.57% <0%> (+12.85%)
:arrow_up:

pygridtools/validate.py
77.19% <0%> (+14.03%)
:arrow_up:

pygridtools/iotools.py
90.47% <0%> (+16.32%)
:arrow_up:

pygridtools/core.py
90.03% <0%> (+29.08%)
:arrow_up:

pygridtools/viz/viz.py
81.81% <0%> (+31.81%)
:arrow_up:

pygridtools/viz/__init__.py
100% <0%> (+100%)
:arrow_up:

pygridtools/viz/_viz_bokeh.py
100% <0%> (+100%)
:arrow_up:

Continue to review full report at Codecov.

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


Could not run Firefox browser

For workflow like this
name: Firefox

on: push

jobs:
  # hmm, seems when using container (that should have FF installed)
  # Cypress cannot find Firefox. This job just prints browser versions
  check-firefox:
    runs-on: ubuntu-latest
    # https://github.com/cypress-io/cypress-docker-images
    container: cypress/browsers:node13.6.0-chrome80-ff72
    steps:
      # help us who how we are running in the container
      - run: whoami
      - run: id
      - run: google-chrome --version
      - run: firefox --version

Getting the following error
whoami
root

id
uid=0(root) gid=0(root) groups=0(root)

google-chrome --version
Google Chrome 80.0.3987.87

firefox --version
Running Firefox as root in a regular user's session is not supported.  ($HOME is /github/home which is owned by uid 1001.)
##[error]Process completed with exit code 1.

testing in https://github.com/cypress-io/cypress-example-kitchensink/pull/361

asked the question on GH forum https://github.community/t5/GitHub-Actions/How-to-run-action-as-a-non-root-user/m-p/45561

Solved, this solution works:
container:
	image: cypress/browsers:node13.6.0-chrome80-ff72
	options: --user 1001

Can this issue be closed then?

almost @jennifer-shehane - I will open a PR to document this and add an action

Solved, this solution works (thanks to gh community forum):
container:
  image: cypress/browsers:node13.6.0-chrome80-ff72
  options: --user 1001

I'm still getting error while loading shared libraries: libgbm.so.1 with that


Replace oudated jqueryuiSortable by modern SortableJS

Questions
Answers

Description?
Please check the Fixed ticket below. (I already made a similar PR for blockreassurance module. NeOMakinG suggested adding SortableJS to the Core js  there to use as a common library, but waiting to the next major version is indefinite so I chose using SortableJS as the module's local lib here)

Type?
refacto

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes https://github.com/PrestaShop/PrestaShop/issues/30210. Related to Issue 27186 (partialy)

How to test?
1. Apply this PR changes  2. Clear Web browser cache and PrestaShop BO cache  3. Check sortable feature in the module BO Config's Slides list to make sure the item can be dragged, dropped and saved there, then showed in FO accordingly, as usual.

Well done @leemyongpakvn 👏 thank you

Great @kpodemski :fireworks: . I think we need to complete the related ongoing PR with blockreassurance before closing issue 30210.


Can you have two images inside  one under the other?

<pinch-zoom>       </pinch-zoom>
It seems like with this, the two images will be positioned next to each other due to display: flex, and disabling it breaks the zoom (it'll zoom into the centre instead of where you tap). Any fixes?

I tried this, it makes it so that if you zoom in it'll only zoom to the centre of the div and not where you click

@ryankempster did you end up figuring this out?


About the loss in the official log

In the official YOLOV3 train log, the loss is very large and is close to hundreds.

And when I train YOLOV3 connected a track model(no change on YOLOV3 model, just connect a track model behind it) in BDD100K dataset, its loss that belongs to YOLOV3 is smaller than the loss in official log.

However, its evaluation is zero.

And visualization  of detection bboxes is as below:

So, what is the suitable loss?  Or it is caused by others?

Probably a simple cascade is not very efficient. Maybe your tracking loss affects detection loss.

Probably a simple cascade is not very efficient. Maybe your tracking loss affects detection loss. Also, please check whether you have load the GTs sucessfully.

How can one check if the GTs are loaded sucessfully?


ArcGIS Link MapOverlay show() & hide() error

When calling the show() & hide() methods on gmaps.ags.MapOverlay an error occurs:
Uncaught TypeError: Cannot read property 'style' of null
    at MapOverlay.show (arcgislink.js:3671)
    at mapapp.layer.Providers.ArcGIS.showLayer (mapapp-layer-providers-arcgis.js:25)

Uncaught TypeError: Cannot read property 'style' of null
    at MapOverlay.hide (arcgislink.js:3679)
    at mapapp.layer.Providers.ArcGIS.hideLayer (mapapp-layer-providers-arcgis.js:30)

Problem is because internal this.div_ property is null.
Workaround is to use overlay.setMap(map) to show, and overlay.setMap(null) to hide.

That's an odd one. The standard map overlay api is to use setMap(null) to hide an overlay. And looking at the code the div_ is only set for image overlays. Do you have a simple html file that I can use to reproduce this?

Sorry, no test file available. I'll put one together when get a bit of spare time. In a mad scramble to port my code over to Leaflet as the new Google Maps Platform is going to cost me a small fortune! :(

Yeah, know what you mean @gavinharriss. I've used it extensively and now need to figure out what it's going to cost :(


Pull Replication issue: Handling Production SyncgateWays/ Couchbase clusters on Swithovers.

Hi Folks,
Setup Details: -

We have 2 SG/CB-clusters (CB-cluster C1 & C2) and data is SG-replicated across clusters.
CBL has FQDN to discover the CB-clusters and connect. Both the clusters are identified by same FQDN.
C1 is primary serving cluster, and if C1 fails over, C2 will be serving. When C1 is up, all CBL traffic will be routed to C1, and when C1 is down, all CBL traffic will be routed to C2.

Pre-Condition: -
1. For given CB-lite client (CBL), Pull checkpoint document SeqNum of C2 is higher than C1.
2. Cluster C1 is down and C2 cluster is serving.
Issues observed: -

Observed Push & Pull connections are connected to different clusters. When C1 becomes active again after failure, new push connection will be routed to C1, but existing changes_feed connection to C2 is not closed by CBL.

Observed sometimes when failover happens from C2 to C1, CBL is updating checkpoint doc in C1 with seqNum of C2. If seqNum corresponds to checkpoint doc in C2 is higher than the seqNum corresponds to checkpoint doc in C1 (seqNum-C1 <seqNum-C2), then changes_feed from C1 won't give any notifications due to checkpoint document corruption.
(CBL receives changes_feed response with SeqNum-C2 from C2 -> CBL sends PUT request with checkpoint-SeqNum-C2 to C1 -> CBL receives 409 response from C1 -> CBL sends GET request to fetch checkpoint-doc to C1 -> CBL receives 200 OK response with checkpoint-SeqNum-C1 from C1 -> CBL sends PUT request with checkpoint-SeqNum-C2 to C1 -> CBL receives 200 OK response from C1).

On receiving 409 response for checkpoint doc update, CBL is not fetching checkpoint doc (it’s not re-starting replication).
(CBL sends PUT request with checkpoint-SeqNum-C2 to C1 -> CBL receives 409 response from C1 -> CBL sends PUT request with checkpoint-SeqNum-C2 to C1 -> CBL receives 409 response from C1 ... repeat till replication restarts)

Expectations: -
i. CBL should be able to detect the CB-cluster failover/switchover, and shall be able to perform push & pull sync accordingly by initializing with appropriate checkpoints.
ii. CBL needs to make sure that Push & Pull checkpoint documents are updated with seqNum corresponds to its local cluster.
iii. On receiving 409 response for checkpoint doc update, CBL shall restarts it sync.
iv. On receiving 503 response or connection close for changes_feed, CBL shall restarts it sync.
v. CBL needs to make sure both the Push & Pull connection are going to same CB-cluster.
@hideki @adamcfraser @pasin @snej
Please share your thoughts to handle this use case in CBL or Server level.
Thanks
Nithin

@waynecarter @ssmotra
Wayne, this is the issue I was talking about on both iOS and Android. Pls advise.

CBL should be able to detect the CB-cluster failover/switchover

It can't detect that directly. It only finds out when it tries to save a new checkpoint (which happens about every 5 seconds) and finds that the pre-existing checkpoint isn't there or has the wrong revID.

v. CBL needs to make sure both the Push & Pull connection are going to same CB-cluster.

I don't know how it could tell. The switchover is presumably going on behind a reverse proxy or through a CDN switching DNS entries, so from the client side it doesn't look like anything's changed.

iii. On receiving 409 response for checkpoint doc update, CBL shall restarts it sync.

It doesn't currently do that. This needs more thought, but I think you're right that it should, since this implies the server may have been swapped out or restored from a backup or something. (The replication algorithm handles this situation when a replication starts, but I think it doesn't account for it happening in the middle of a replication.)

iv. On receiving 503 response or connection close for changes_feed, CBL shall restarts it sync.

Dropped connections happen a lot, due to flaky networks or some middleware (like AWS) deciding that the socket seems idle. Resetting the replication every time this happened would be a huge performance problem for those situations, and might even mean that the replication never completes if the disconnect keeps happening.
Similarly, a 503 status just means "the server is unavailable", and it can happen for a lot of reasons. It would be a bad idea to reset replication whenever it happened.

v. CBL needs to make sure both the Push & Pull connection are going to same CB-cluster.
I don't know how it could tell. The switchover is presumably going on behind a reverse proxy or through a CDN switching DNS entries, so from the client side it doesn't look like anything's changed.
Will this be handled better with the single (push/pull) replicator in 2.0?

If we can address the 409 issue then we should be able to handle this for now.

Will this be handled better with the single (push/pull) replicator in 2.0?

Yes; a single Replication instance in 2.0 can do both push and pull. Also, replication runs over a single WebSocket connection, and if that socket closes unexpectedly the replicator will restart. The new replicator instance will compare checkpoints first thing.
So this should resolve the problem discussed here: a failover to a different cluster would presumably terminate open WebSocket connections; the replicator will re-connect to the new cluster and then detect a mismatched checkpoint by resetting the replication to sequence 0.

Notes regarding PUT /_local/{checkpointDocID} and 409 (CONFLICT) cycle problem that is mentioned in observed item no. 3.
With the latest implementation of CBL Android 1.x (master branch), CBL should behave following:

PUT /_local/{checkpointDocID} failed with HTTP Error 409 (CONFLICT) because revision ID of checkpointDocID does not exist on the server.
CBL sends GET /_local/{checkpointDocID}, and receives server's _id, _rev and lastSequence values.
CBL overwrites lastSequence value with local lastSequence value, and sends PUT /_local/{checkpointDocID} with updated data, this should be successful (Should receive HTTP OK).

Tested CBL by following steps to verify above expectation

Run push replication with continuous mode.
With using curl or equivarent tool, updates on server's checkpointDocID's _rev and lastSequence value.
create a document locally (at client app).
Push replicator replicates newly created document.
Push replicator calls PUT /_local/{checkpointDocID} -> Failed with 409
Push replicator calls GET /_local/{checkpointDocID} -> Receives latest checkpointDocID data.
Push replicator calls PUT /_local/{checkpointDocID} with local's lastSequence value

From network capture tool, only one 409 error by PUT /_local/{checkpointDocID}

@NitzDKoder Did you observe observation item 3 with your environment?

@hideki please confirm what is the behaviour for pull continuous replication..our item 3 was with WRT to PULL replication..push seems all ok..

@NitzDKoder
CBL v1.x behavior when PUT /{db}/_local/{checkpointDocID} has 409 error.

409 error occurs because RevID for checkpointDocID is a mismatch.
CBL calls GET /{db}/_local/{checkpointDocID} to obtain SG's RevID and lastSequence values
CBL calls PUT /{db}/_local/{checkpointDocID} again with server's RevID and client's lastSequence values. This should not fail as RevID should match.

If you can observe Step 3 fails with 409, SG's checkpointDocId's document is updated between Step 2 and 3.


Save chatfx in plugin_data folder

Instead of having to re select your chat fx every time you join, save it in plugin_data

working on it next 👀
it saves the color until the server restarts btw - the array i store colors in is always the same until there's a crash or a restart, so you may rejoin and it'll still be the same


#tracked_changes doesn't return values for create action

Example:
HistoryTracker.last

 #<HistoryTracker _id: 56afea7a3baa300b5e000017, created_at: 2016-02-01 23:30:02 UTC, updated_at: 2016-02-01 23:30:02 UTC, association_chain: [{"name"=>"Bookings::Booking", "id"=>BSON::ObjectId('56afea6b3baa300b5e000015')}, {"name"=>"cards", "id"=>BSON::ObjectId('56afea7a3baa300b5e000016')}], modified: {"first_name"=>"Artur", "last_name"=>"Khantimirov", "title"=>"Mr"}, original: {}, version: 1, action: "create", scope: "booking", modifier_id: nil>

HistoryTracker.last.tracked_edits

 {}

You mean tracked_edits? Start by writing a spec for this?

@yokree Looks like there is no issue. Just now I have checked as follows:
2.3.0 :001 > HistoryTracker.first
 => #<HistoryTracker _id: 513c8028b9b67deba900009a, created_at: 2013-03-10 12:44:24 UTC, updated_at: 2013-03-10 12:44:24 UTC, association_chain: [{"name"=>"PaymentSchedule", "id"=>BSON::ObjectId('513c8028b9b67deba9000091')}], modified: {"order"=>1, "title"=>"Token", "percentage"=>0, "scheme_id"=>BSON::ObjectId('513c8028b9b67deba9000099')}, original: {}, version: 1, action: "create", scope: "payment_schedule", modifier_id: BSON::ObjectId('5139f184b9b67d8e7300000a')>

Output:
{"add"=>{"order"=>1, "title"=>"Token", "percentage"=>0, "scheme_id"=>BSON::ObjectId('513c8028b9b67deba9000099')}}

I think we can close this issue.

I think @sivagollapalli is saying that this was fixed in https://github.com/aq1018/mongoid-history/pull/158 on HEAD, give it a try.


MDC-104 iOS, Step 5: Minor typo

Minor typo: the HomeViewController.m is Obj-C style, use HomeViewController.swift instead.

Updated in the internal docs.


Reduce time spent loading chain states

Motivation
ChainStateViews are loaded before operating on chains. However, they aren't cached afterwards. Even though they might hit a lower level storage cache, it is still less efficient than caching the chain, because data still needs to be deserialized. Also, there is always at most one ChainStateView for any ChainId, enforced through the usage of ChainGuards. This can cause some contention for example if there are many queries to the same chain.
Proposal

Allow sharing the ChainStateView between many readers and at most one writer, in order to reduce unnecessary lock contention.
Cache a loaded ChainStateView in memory, so that it can be reused without having to reload it.

Done in #2163.


FARMING - rename purchase button (staging)

I would rename the purchase button on: https://www.threefold.io/farming.html
to pre-register.

I agree. Reassigned to Ehab for a quick fix. Thank you.


Steam version of touhou 17.5 isn't recognised ?

The demo (dl from Moriya) is recognised:

But not the full game on steam :

same with the other version of the tool
I'm on Void Linux, using proton 6.3-7 for both thcrap and the game (game that I have played a bit and work fine).

thcrap support for the full game is still being worked on

Fixed


Show Toast without Redirect

Hi @AlemTuzlak, thank you for the wonderful project!
I was wondering if it is possible to show a toast without utilizing the redirect helpers.
For example, when deleting an item from a list of items I utilize a fetcher and avoid a redirect in accordance with suggested practices. I want to show a toast to the user when the action returns. Is this possible?
Thanks!

@jwaltz Thank you so much! There is a PR open that adds this functionality and should be merged soon!

Added!


Blob Size getUserMedia

i have tried recording a 10 to 15 minutes of video after recording my browser stuck , so how much blob size can we record in client side ?

so how much blob size can we record in client side ?

@beaufortfrancois may know the answer to this.

https://chromium.googlesource.com/chromium/src/+/main/storage/browser/blob/README.md#blob-storage-limits has all details

Thanks François!
On Mon, 10 Oct 2022, 08:27 François Beaufort, @.***>
wrote:

https://chromium.googlesource.com/chromium/src/+/main/storage/browser/blob/README.md#blob-storage-limits
has all details
—
Reply to this email directly, view it on GitHub
https://github.com/samdutton/simpl/issues/173#issuecomment-1272887439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABSDKSULQXQ2PUFWQHILE3WCPAOLANCNFSM6AAAAAAQ3RAILM
.
You are receiving this because you commented.Message ID:
@.***>


Replace go-restful lib version

Fix CVE-2022-1996

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 25.54%

Files with Coverage Reduction
New Missed Lines
%

controllers/sriovibnetwork_controller.go
2
64.15%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
1929

Relevant Lines:
7553

💛  - Coveralls

CVE refers to <v3.8.0
Package                                Affected versions               Patched versions
 github.com/emicklei/go-restful        < 2.16.0                        2.16.0
 github.com/emicklei/go-restful/v2     <= 2.7.1                        None
 github.com/emicklei/go-restful/v3     >= 3.0.0, < 3.8.0               3.8.0

But we are using v3.10.1 (go.mod#L68). Why do we need this?

BTW, there is a problem to run go list due to some issue with machine-config-operator
go list -mod=mod -m -u all
go: loading module retractions for<EMAIL_ADDRESS>verifying go.mod<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>404 Not Found
	server response:
	not found: create zip: pkg/controller/template/test_data/templates/aws/master/files/-etc-docker-certs.d-docker-registry.default.svc:5000-ca.crt: malformed file path "pkg/controller/template/test_data/templates/aws/master/files/-etc-docker-certs.d-docker-registry.default.svc:5000-ca.crt": invalid char ':'
	pkg/controller/template/test_data/templates/aws/worker/files/-etc-docker-certs.d-docker-registry.default.svc:5000-ca.crt: malformed file path "pkg/controller/template/test_data/templates/aws/worker/files/-etc-docker-certs.d-docker-registry.default.svc:5000-ca.crt": invalid char ':'
	[Truncated: too long.]

@rollandf Thanks for explaining
LGTM

The -u flag is causing the issue with machine-config-operator...
$ go list -mod=mod -m all | grep restful
github.com/emicklei/go-restful v2.10.0+incompatible
github.com/emicklei/go-restful/v3 v3.10.1


Add pkg/config/collection

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

Needs approval from an approver in each of these files:

galley/OWNERS [ozevren]

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

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ozevren
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS
galley/OWNERS [ozevren]

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

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

Needs approval from an approver in each of these files:

galley/OWNERS [ozevren]

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

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-telemetry-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-telemetry-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-telemetry-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-telemetry-k8s-presubmit-tests

prow/e2e_pilotv2_auth_sds.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-telemetry-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-telemetry-k8s-presubmit-tests

prow/e2e_pilotv2_auth_sds.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio_auth_sds_e2e

prow/e2e-bookInfoTests-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-telemetry-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-telemetry-k8s-presubmit-tests

prow/e2e_pilotv2_auth_sds.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio_auth_sds_e2e

prow/e2e-bookInfoTests-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-mixer-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [geeknoid,ozevren]

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

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-security-k8s-presubmit-tests.sh
8ccc8a165c70b5c9f277a956da12046cd42dc046
link
/test integ-security-k8s-presubmit-tests

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-telemetry-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-telemetry-k8s-presubmit-tests

prow/e2e_pilotv2_auth_sds.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio_auth_sds_e2e

prow/e2e-bookInfoTests-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-mixer-k8s-presubmit-tests

prow/integ-galley-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-galley-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-telemetry-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-telemetry-k8s-presubmit-tests

prow/e2e_pilotv2_auth_sds.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio_auth_sds_e2e

prow/e2e-bookInfoTests-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-mixer-k8s-presubmit-tests

prow/integ-galley-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-galley-k8s-presubmit-tests

prow/integ-security-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-security-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio_auth_sds_e2e

/test e2e-bookInfoTests-envoyv2-v1alpha3

/test integ-galley-k8s-presubmit-tests

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-pilot-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-pilot-k8s-presubmit-tests

prow/istio-pilot-multicluster-e2e.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-multicluster-e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/integ-telemetry-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-telemetry-k8s-presubmit-tests

prow/e2e_pilotv2_auth_sds.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test istio_auth_sds_e2e

prow/e2e-bookInfoTests-v1alpha3.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/integ-mixer-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-mixer-k8s-presubmit-tests

prow/integ-security-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-security-k8s-presubmit-tests

prow/integ-galley-k8s-presubmit-tests.sh
0d5b7562443d885a82e3124fdfbf3b378fe9b9f7
link
/test integ-galley-k8s-presubmit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-galley-k8s-presubmit-tests

/test e2e-bookInfoTests-envoyv2-v1alpha3

/test istio-pilot-e2e-envoyv2-v1alpha3


No possibility to adjust markup in SuggestionList component

Currently it is not possibly to add more markup to a SuggestionList component (only title of product / string), only a string which is rendered as-is. React escapes the string. Maybe some portals might be useful there. And some way to allow rendering HTML.
https://github.com/shopgate/theme-gmd/blob/19e981a97972b71c4d73d24de7bd2d7fb42a9044/components/Search/components/Suggestions/components/List/index.jsx#L48

We will add portals for this in the Engage release 6.7.0


AYON: Distribution with progress

Brief description
Made distribution object based so it can be used in UIs and progress can be tracked.
Description
The object of AyonDistribution handle whole distribution logic. Create distribution items per dependency package and each addon to check if are available or should be distributed. The distribution can be process then in thread and it is possible to check the distribution progress overtime (for possible future UI). Renamed classes to define sources to match their new usage (not just addon specific).
Distributed items are stored to metadata file so we can keep track of what, when and which source was used to distribute the files. The metadata also contains file hash for future possible check of file changes of the same version (e.g. dev version).
Testing notes:

Distribution should work as did before

Task linked: OP-3739 Connect OP tray to v4 server

Could you please fix tests also (changed logic with missing update_addon_state).

Sure. I'll also update source info (server was enhanced by path attribute).


[Improve] Use responsive approaches

Even if your website target only certain devices, you may have users with unexpected devices or screen ratio.
For more information see [Responsive Web Design] (https://developers.google.com/webmasters/mobile-sites/mobile-seo/responsive-design?hl=en)
What files to fix:
examples/basic.html
examples/basic_material.html
examples/basic_model.html
examples/fps.html
examples/plugin_example.html
examples/skybox.html
examples/stone_wall.html
or any new example *.html file
How To Do it:
Check if  your css (either files or inline) use any media queries.
Check if your link tags use any media queries.
How to test it:

Install [Vorlon.JS] (http://vorlonjs.com/?WT.mc_id=16538-DEV-sitepoint-article74)

add the following line into your test HTML file:

'<  script src="http://localhost:1337/vorlon.js"  >' '<  /script  >'

Once Vorlon.JS is done installing, run the server:

$ vorlon

With the server is running, open http://localhost:1337 in your browser to see the Vorlon.JS dashboard.

Open your test html file,  and it should appear on the Vorlon.JS dashboard.

In the dashboard, click on the button Best Practice: MOBILE WEB  and check the
use responsive approaches

sasha240100,
Once you said on the Slack, that WHS should be considered as a game for the desktop computers, meaning a big screen with high resolution. Having that fact in mind, to fix #40 the following line should be added to any *.html file of whitestorm.js game:
'< style>
@media only screen and (min-width: 10000px; orientation: landscape) {
}
'< /style>
@media includes the following media type of devices:
all	        Used for all media type devices
print	        Used for printers
screen	Used for computer screens, tablets, smart-phones etc.
speech	Used for screenreaders that "reads" the page out loud
Print and Speech are excluded from WHS.
If you want, you can add other parameters to the screen, such as:
aspect-ratio	The ratio between the width and the height of the viewport
color	The number of bits per color component for the output device
color-index	The number of colors the device can display
device-aspect-ratio	The ratio between the width and the height of the device
device-height	The height of the device, such as a computer screen
device-width	The width of the device, such as a computer screen
grid	Whether the device is a grid or bitmap
height	The viewport height
max-aspect-ratio	The maximum ratio between the width and the height of the display area
max-color	The maximum number of bits per color component for the output device
max-color-index	The maximum number of colors the device can display
max-device-aspect-ratio	The maximum ratio between the width and the height of the device
max-device-height	The maximum height of the device, such as a computer screen
max-device-width	The maximum width of the device, such as a computer screen
max-height	The maximum height of the display area, such as a browser window
max-monochrome	The maximum number of bits per "color" on a monochrome (greyscale) device
max-resolution	The maximum resolution of the device, using dpi or dpcm
max-width	The maximum width of the display area, such as a browser window
min-aspect-ratio	The minimum ratio between the width and the height of the display area
min-color	The minimum number of bits per color component for the output device
min-color-index	The minimum number of colors the device can display
min-device-aspect-ratio	The minimum ratio between the width and the height of the device
min-device-width	The minimum width of the device, such as a computer screen
min-device-height	The minimum height of the device, such as a computer screen
min-height	The minimum height of the display area, such as a browser window
min-monochrome	The minimum number of bits per "color" on a monochrome (greyscale) device
min-resolution	The minimum resolution of the device, using dpi or dpcm
min-width	The minimum width of the display area, such as a browser window
monochrome	The number of bits per "color" on a monochrome (greyscale) device
orientation	The orientation of the viewport (landscape or portrait mode)
overflow-block	How does the output device handle content that overflows the viewport along the block axis (added in Media Queries Level 4)
overflow-inline	Can content that overflows the viewport along the inline axis be scrolled (added in Media Queries Level 4)
resolution	The resolution of the output device, using dpi or dpcm
scan	The scanning process of the output device
update-frequency	How quickly can the output device modify the appearance of the content (added in Media Queries Level 4)
width	The viewport width
I wasn't sure how many of them would be appropriate for the WHS, and I only included min-with and orientation.
Please, comment this and advise if anything should apply for the whitestorm.js.
Otherwise, I think that this issue is now fixed.

@bdirl Thanks!


Feat: Optimize Indicator Component with React.memo

Description
The Indicator component currently re-renders whenever its parent component re-renders, even if the osVersion state has not changed. This can lead to performance issues, especially if the parent component re-renders frequently. To optimize the Indicator component, we should wrap it with React.memo to prevent unnecessary re-renders.
Proposed Solution
Use React.memo to wrap the Indicator component. This higher-order component will perform a shallow comparison of the component's props and only re-render if the props have changed.
Detailed Steps

Import React.memo:

Ensure React is imported in the Indicator.tsx file.

Wrap the Component:

Wrap the Indicator component with React.memo when exporting it. This will optimize the component by preventing unnecessary re-renders.

Verify Functionality:

Ensure that the Indicator component still functions correctly and updates when the osVersion state changes.

Benefits

Performance Improvement: By preventing unnecessary re-renders, the application will perform better, especially if the parent component re-renders frequently.
Optimized Rendering: React.memo ensures that the Indicator component only re-renders when its props change, leading to more efficient rendering.

I would like to work on this issue

Go ahead @OlibhiaGhosh


Swift example

(these commits can (should) be squashed)

Any reason for this taking so long? I am in fact considering closing this pull request and withdrawing my offer to contribute to this repo. I don't see myself creating new pull requests taking months to fulfill or even get feedback on. If things are to change then I'm here to stay, but if you're just pressing "merge" I don't see that as a meaningful conversation taking place. Just being honest. Let me know what you think.

Sorry, we've been busy. I'll be able to get to it next week.

Sorry this took so long. This looks good, I just have one request: could you explicitly specify the Apache2 license in the following files:

./SecureEnclaveDemo/AppDelegate.swift
./SecureEnclaveDemo/Manager.swift
./SecureEnclaveDemo/ViewController.swift
./SecureEnclaveDemoTests/SecureEnclaveDemoTests.swift

Right now they're marked as "All Rights Reserved.", which prevents them from being used as open source libraries. Once those are updated, I'll accept the merge and update the README.

Thanks @hfossli. Sorry about the delay.

Awesome. I guess I'll be reviewing future pull requests and issue tracking regarding the swift code. Would be cool to join this repo as a contributor with write access in that sense.

Cool! 👍🎉


similar plugin for flow chart possible

hi developer,
the timeline plugin is super great , because its super simple and doesnt interrupt workflow.
is it possible to make a similar plugin for simple flow chart  (if possible with one or 2 branches)
thank you

hi kind stranger,
I made this plugin because I need it. I'm sorry, but I don't have the time necessary to write another plugin.
thank you for your issue


ml2cpp step 15 :  Web Service

Web Service. Can be started earlier in the process.
Follow the six steps described in #1

Adapted the SQL web service to contain a new language ("CPP").
All the new Jupyter notebooks are based on this web service.

Jupyter notebooks :
https://github.com/antoinecarme/ml2cpp/tree/master/doc

TODO : repalce "SQL" with "GeneratedCode" in JSON ouptut
    lCPP = content["model"]["SQLGenrationResult"][0]["SQL"]

TODO : maintain compatibility with old versions (sklearn2sql_heroku).

Closing.


Allow overriding of min and max threads

tileserver #120 SUCCESS
This pull request looks good

tileserver #121 SUCCESS
This pull request looks good


I added a condition in the verify_weekend method for America/New_York time zone

@JaredCaprio Hey thanks for looking into it. To clarify, Friday and Saturday is weekends in my time zone which is denoted as 4 and 6. Are you sure that your'e time zone weekends are the same. Like Friday and Saturday ?

@JaredCaprio Looks Good. Let me know if weekend colouring is working in your time zone after merging. Thanks for your contribution. Happy Coding !

Yeah,  the weekend highlighting works perfect for my time zone!


Feature Request: Custom URL and JS to get highest version

For docs hosted on sites other that RTD, I would love to be able to point to a custom URL suffix (versionwarning_api_url only provides the base) and have code that extracts the version from that. Happy to provide examples if needed.

Hi! Thanks for the suggestion.
If you have an idea already, feel free to open a PR. I would like to keep the default behavior as it is to avoid breaking changes. Just that.
I wouldn't include code to parse a custom response here, but allow the user to pass a custom function's name instead via  a config

Thank you. Looking at the code, both the data loading, version detection, and banner injection is wrapped up in checkVersion(). That would likely need to be broken up to allow a custom function to bring in the value for highest_version. Does that sound reasonable?


Update for modern django?

There are a few modifications to update this to more recent django versions, at a minimum:

urls.py needs to move from url() to re_path()
signals.py needs to drop providing_args=

Is this still under dev or does it deliberately target older django? Is there a newer replacement?

https://github.com/ddemid/django-slack-utils/pull/9 fixes this, but needs checking for backwards compatibility


Please change the default version to 7

Java 6 is mostly out of support - Can we please change the default version to 7?
It will make more sense than having to keep your own config.

Java 6 was the default when this was released. Version 2 of this cookbook will have java7 as the default. If we change it in v1, it would break many people. Check out the wiki page for the V2 roadmap. https://github.com/agileorbit-cookbooks/java/wiki/Roadmap
You should just be able to override an attribute to get java 7 in a recipe/role/environment. That isn't too much of a burden.


 Your fix will break the backward compatibility for all who used the library before. If you need - I can add additional parameter to function, that will force function to return "items" instead of raw "results".

https://github.com/Badiboy/pyCryptoPayAPI/releases/tag/0.1.0


enums for system entity attributes are named incorrectly!

Quite new to dataverse-gen but I have an odd issue where all the enums for the system entities are named incorrectly, e.g. account_acccount_accountcategorycode.ts, which should simply be acccount_accountcategorycode.ts, it's both the TS filename and also the enum declaration!
I've been able to modify the enum.ejs template to modify the generated enum file in this scenario, but then this doesn't help as the generated entity file is then broken due to the reference using the typescript filename! I guess I could change the entity.ejs too but I feel like I shouldn't need to do this and there is something I am fundamentally missing that means all the system enums are behaving in this way, but it doesn't happen for any custom entities and their enums!?

Hi @mrmwink - great to have you on board!
There are two types of choice columns, local definitions and global definitions:

The naming of local enums is (scope)_(choices-logical-name), the nameing of global enums is just (choices-logical-name)
The scope part for enums on local choice columns for entities will be the logical name of the entity, whilst for actions, it will be the name of the action. It's important to have this scope disambiguation to ensure that entity choices and action choices don't collide.
Since the accountcategorycode is a 'local' choice column, the scope is 'account' and the choice-logical-name is account_accountcategorycode - hence you see the double prefix - account_account_accountcategorycode
For global choices, there is no disambiguation prefix, and the logical name will just be the logical name of the choices - e.g. myprefix_mychoicesname
Since the default is to create global choices ('Sync with global choice' in the screenshot above) - you won't see the disambiguation for your custom choice columns.
Dataverse-gen could perform some optimisation where it looks for name collisions and only prefixes where needed - but I chose to go for consistency instead. If you re-name enums then it will make it difficult to re-generate the metadata and so I recommend against it - and it may seem strange having the scope disambiguation - until you find there is an action that has the same choice logical name but a different set of choices!
I hope this makes sense!

Ah, that makes perfect sense, as ever Scott your explanation is on point.
I can also go back to my team now and give them a heads up 👍
I guess you can never be 100% certain that is the case with the scope and naming, only takes a small error when generating one of them and you might get an issue!
Cheers, keep up the awesome work.

Great - glad this helped 😊
The collisions tend to happen with 'legacy' actions where the logical names were created literally over a decade ago. The more recent metadata is much more consistent. I'll close this for now - perhaps we can revisit it later.


"Answer must be reduced to lowest terms" option for number entry part

There should be an option to require that the student's answer must be reduced to lowest terms when given as a fraction.
If the student's answer is not reduced, then partial credit should be applied.
To do this:

add options mustBeReduced and mustBeReducedPC to the NumberEntryPart in bin/exam.py - you should add default values in the class constructor, load values in fromData, and store the values in toxml. Look at how the other settings are loaded and saved.
In runtime/scripts/parts/numberentry.js, load the setting in the part constructor.
in NumberEntryPart.mark, when this.mustBeReduced is true, work out the numerator and denominator, and check that their gcd is 1 (Use math.gcd). If not, apply the partial credit with this.multCredit(this.mustBeReducedPC).

ref #380

Hello , I am new to Open source environment. I can clearly implement the GCD logic to find a number is in its lowest form or not but can you help with how to get started as this could be my first PR . thanks ,

Thanks for offering to take this on. First of all, have a look at our guide to contributing to Numbas, and try to install a local copy if you can.
I'll expand the list of things to do at the top of this post.

Chris , You want me to install Numbas on my system or to fork your code ?

Both - install Numbas on your computer so you can check that any changes you make actually work, and fork the repository so that you can submit a pull request.
Have a look at the GitHub Bootcamp.

Okay . Thanks

I am facing some problems with installation which I asked on Google group of Numbas. I will get to this work as soon as i get my issues cleared. Thanks

can i take this up?

@shreab373 yep, if you make a pull request I'll take a look at it.
As I said above, first have a look at our guide to contributing to Numbas, and try to install a local copy if you can.


unbound-control and self-signed cert

This is a hybrid issue / possible feature request.
I have no issue using unbound-control 1.10.1 (nightly build, built against OpenSSL-1.1.1g) on a LAN host with the localhost unbound daemon 1.10.1. However, from the same LAN host, when I try to manipulate the unbound <IP_ADDRESS> server on my router, I keep getting:

error: SSL handshake failed
140735231270912:error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915:

# openssl s_client -connect ${router_ip}:8953 -statutus
initially returned

Verify return code: 18 (self signed certificate)

I have added the router's "unbound_server.pem" to the LAN host's $OPENSSLDIR/certs by following the guide at ( http://gagravarr.org/writing/openssl-certs/others.shtml#selfsigned-openssl ). This appears to satisfy OpenSSL on the LAN host, because
# openssl verify -CApath $OPENSSLDIR/certs $OPENSSLDIR/certs/unbound_control.pem
returns

OK

Further, both
# openssl s_client -connect ${router_ip}:8953 -CApath /usr/local/ssl/certs
and
# openssl s_client -connect ${router_ip}:8953 -status
now return

Verify return code: 0 (ok)
...
instead of the previous:
...
Verify return code: 18 (self signed certificate)

I have appended the "unbound_server.pem" and "unbound_control.pem" of both the router and the LAN host to each other's respective counterpart.
I have appended the "unbound_server.pem" of both the router and the LAN host to each other's /etc/unbound/ca-certificates.crt (as in unbound.conf's "tls-cert-bundle: /etc/unbound/ca-certificates.crt"), though it may not serve this purpose of mine.
However, I'm still getting the above "error: SSL handshake failed ... certificate verify failed" when using unbound-control from the LAN host to the router.
So, either I'm still missing something, or maybe unbound-control could optionally accept a flag to specifically ignore "certificate verify failed" or SSL error type 18 regarding self-signed cert from CA "unbound" / CN=unbound-control, similar to wget's --no-check-certificate?

Hi,
If you want unbound to ignore the certificate, simply do not give a tls-cert-bundle, as in "", and do not specify a host auth name to check for in the config of the tls destination.
That said, unbound should just use the list of certs that you give it, and that should work, eg. adding it to the list of certs loaded by tls-cert-bundle.  There is even a unit test that uses tls-cert-bundle: unbound_server.pem and that works.  Is the second 'certificate verify failed' because of another reason, i.e you solved the CA not there problem, but it has another problem when you try later, what was the detailed message?

Wait I only spot this later, are you try to use TLS for DNS connections, with tls-cert-bundle, but it looks like really, you may be trying to use unbound-control to connect to the remote-control port?  That is governed by the options in the remote-control section.  It does not use that cert bundle, so that makes no difference.

If you are trying to simple connect with unbound-control to unbound running on another computer.  What that needs is that you set remote-control control-enable: yes and control-interface: to an accessible interface (not <IP_ADDRESS>).  And then copy the .key and .pem files to the other machine, and use unbound-control -c someconfig.conf where that config contains a remote-control section with the destination IP-address, and the locations of the copied .key and .pem files.  (The client does not need the server private key file, but you need to copy the other three files for it).

Thanks for your replies. Yes, my issue is regarding inter-LAN unbound-control; I'm set on TLS for DNS connections using tls-cert-bundle (but I learned something new here too, like the empty "" to not check cert).
I will copy over the three files (unbound_control.pem, unbound_control.key, and unbound_server.pem) and report back how it goes.

So I copied the router's entire /etc/unbound/* into the LAN host's /etc/unbound/router/ directory. The router's unbound.conf ( /etc/unbound/router/unbound.conf ) has the following:
remote-control:
control-enable: yes
control-use-cert: yes
control-interface: <IP_ADDRESS>
control-interface: ::1
control-interface: <IP_ADDRESS>
server-key-file: "unbound_server.key"
server-cert-file: "unbound_server.pem"
control-key-file: "unbound_control.key"
control-cert-file: "unbound_control.pem"
Then from LAN host to the router:
# unbound-control -s <IP_ADDRESS> status -c /etc/unbound/router/unbound.conf
I still get:

error: SSL handshake failed
140735231270912:error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed:ssl/statem/statem_clnt.c:1915:

In router syslog:

Apr 14 23:48:50 unbound[21022:0] error: remote control failed ssl crypto error:14094418:SSL routines:ssl3_read_bytes:tlsv1 alert unknown ca

I have added the LAN host unbound's unbound_server.pem to the router's OpenSSL (and vice versa, as reported in the first post), verified on the router as below:
openssl verify -CApath /etc/ssl/certs /etc/ssl/certs/unbound_control.pem

/etc/ssl/certs/unbound_control.pem: OK
where /etc/ssl/certs/unbound_control.pem is copied from the LAN host's unbound 1.10.1, not the router's own.

So the router's OpenSSL is definitely aware of the LAN host's CA=unbound
What am I still missing?
Also, I noticed "control-use-cert: yes" and tried to see if it might work without using cert, i.e.:
remote-control:
...
control-use-cert: no
# unbound-control -s <IP_ADDRESS> status -c /etc/unbound/router/unbound.conf
I simply get:

error: SSL handshake failed
And nothing in router's syslog

Never mind. It's working now. Part of seems to be moving "-c /etc/unbound/router/unbound.conf" to become the first argument after unbound-control, and something else:

unbound-control -c /etc/unbound/router/unbound.conf -s <IP_ADDRESS> status

Good that you made it work!  I'll try to explain some things that happened:
The -c argument has to be first, this is because FreeBSD (or *BSD) getopt option parsing does that.  On Linux, the glibc getopt option parsing allows different argument order, and you could put the '-c' after the positional arguments.
For the -s <IP_ADDRESS> needed, if the config file has only one control-interface, and that is <IP_ADDRESS>, then it would work from that and not need a commandline argument for unbound-control.  (That config file would then not be the one for the running unbound daemon on that machine, but a config file you use to contact a remote one, I guess).  Just use the commandline argument to set it if that is easier.
The use-cert option is for people that use a unix pipeline (a filename in the control-interface) for unbound-control communication.  That would only work on the local machine of course.
You keep adding the server certificate to the general openssl cert directory in /etc, but the unbound-control does not use it.  It only uses the files listed in the remote-control section.  So when you copied those and fixed the pathnames, it worked (that is what it looks like).

Thanks for the edification.
You're spot on: my LAN host is a BSD-like system, and I was scratching my head over the need to make "-c unbound.conf" the first arg, because I saw others on the unbound listserv putting it at the end just fine (given your clarification, obviously they're on Linux).
I'm wondering: Is there a specific reason, if any at all, that "unbound-control -s" only takes an IP addr, rather than also optionally accepting a hostname such as "router.local"?
Also, since we touched on the subject in passing, I had "forward-tls-upstream: yes" configured, I was unaware of "tls-cert-bundle: ca-certificates.crt", until I read a blog that mentions "forward-tls-upstream: yes" alone leaves unbound susceptible to DNS MITM attacks. I'm wondering: Does that combo ("forward-tls-upstream: yes" but no "tls-cert-bundle: ca-certificates.crt") have the exact same effect as "forward-tls-upstream: yes" + tls-cert-bundle: "" (not specifying a cert file)?

For that -s with an IP address, because name resolution uses the DNS and this creates a circular dependency that you are better off without, eg. control of the nameserver does not depend on the DNS working.  Otherwise it is also easier, but if I wanted to implement and be good about it that would mean a full resolver in unbound-control commandline tool to fetch that name, eg. entirety of unbound...
The tls-cert-bundle unspecified, what is that.  That is about the default value of the tls-cert-bundle config option.  And that is currently "" and 'no' for the windows cert store.  So you are right.  To set that by default we'd need a configure option and something sensible to set it to.


init-migration job not being created for helm upgrade --install command.

Consider a helm release where we are introducing kong after some time. In that case helm upgrade --install command will set .Relase.IsUpgrade and not .Release.IsInstall, but migration job is only created  on helm install according to this.
{{- if .Release.IsInstall -}}
{{/* .migrations.init isn't normally exposed in values.yaml, since it should
     generally always run on install--there should never be any reason to
     disable it, and at worst it's a no-op. However, https://github.com/helm/helm/issues/3308
     means we cannot use the default function to create a hidden value, hence
     the workaround with this $runInit variable.
 */}}
{{- $runInit := true -}}
{{- if (hasKey .Values.migrations "init") -}}
  {{- $runInit = .Values.migrations.init -}}
{{- end -}}

{{- if (and ($runInit) (not (eq .Values.env.database "off"))) }}
apiVersion: batch/v1
kind: Job
metadata:
  name: {{ template "kong.fullname" . }}-init-migrations

Shouldn't there be a way to create migration job for helm upgrade command for a newly installed kong instance?

@rajshrestha86 the init migration job runs the bootstrap command for the database, and so should only happen when the kong release is installed not upgraded. Bootstrap can only be run once otherwise you have to wipe all the data in the database.
What use case are you suggesting is needed to run the bootstrap of the database for an already installed Kong release?


fix: bump timestamps of all requests when a BlockResponse is handled

Partial fix for https://github.com/AleoHQ/snarkOS/issues/2978
Rationale:
If there's only 1 peer connected, and it's slow to send BlockResponses, or we are slow processing them, the BlockRequests can expire in the self.requests map before they can be handled. If that happens, the only peer is disconnected, and syncing comes to a halt.
This PR bumps all the timestamps for all outstanding BlockRequests when we've successfully handled a BlockResponse.
@HarukaMa confirms this fix works.

Closing. Superseded by #3079


New sample script: Install package script

Docs

Author
Waldek Mastykarz

Original Post
#1961

Description
scripts allowing developers to easily install .sppkg packages in their tenant, used during development after running gulp package-solution

Keywords
spfx sharepoint

the script should have the following steps:

[ ] automatically detects the package to install based on the project structure (script assumes it's located in the project root)
[ ] adds package to the app catalog (overwrite if necessary)
[ ] deploys to all sites
[ ] approves all permissions requested by the package

I guess I already have the script readymade with me. Except the last one. I was about to initiate an issue for that anyways. Good that you did that already.
I will not assign it to myself now if someone wants to take a stand at it before #hactoberfest finishes. Will pick it up post that if no one picks it up.

Now since #Hactoberfest is over, I am assigning this issue to myself.

I have started looking into this script and I have to pick your thought process on the below point for the spec @waldekmastykarz

adds package to the app catalog (overwrite if necessary)

My approach is to do the following.

Script will have a parameter for siteURL
If the siteURL is passed as empty, it will assume that the package will need to installed to Tenant App Catalog
If the siteURL has value, it will be installing to the specified Site Collection URL.
When the package installation is to SC App Catalog, it will ensure that SC App Catalog is available. If not, it will enable SC App Catalog.

Is that the process you were envisioning when you had this idea? Can you let me know please

That's correct @arjunumenon 🚀

Thanks Waldek for confirming.
I will start building it and should be able to initiate PR soon.

@arjunumenon just a small reminder about the new approach we take in script samples

I did have a mental note to build as per the updated approach.
Thanks for the reminder though 👍.

Unassigning myself from this issue so that others can take it up.

Can I work on this request?


bugfix/NUI-6034 nui table in firefox column sizes

Closing this and rolling back the original change. We'll explore a fix later.


Add multiline wrapping support

With reference to golang/go#20799, pflag now will respect newline in usage string and do wrap accordingly. Also add test cases for testing.

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

Hi anyone looking on this?

Hi @eparis Could you help to do a review on this? Thanks


Select max doesn't work correctly

in mySQL if you make query you need to write "SELECT MAX(column_name) FROM database_name.table_name".
If you not write database_name in query you will receive  "Error Code: 1046. No database selected Select the default DB to be used by double-clicking its name in the SCHEMAS list in the sidebar."
In program println(query.sql)  prints:
select max(table_name.column_name)
from table_name
and returning null.
(val query = database.from(TableNameObject).select(max(TableNameObject.columnName))

@diantongren's suggestion is correct, but you can also specify the schema in the connection url, for example:
jdbc:mysql://localhost:3306/DATABASE_NAME


.Nuget Solution Folder Confusion

Documentation states:

NuGet 3.3 and earlier used a .nuget folder for solution-wide settings. This folder is not used in NuGet 3.4+.

So what exactly does this mean? I upgrade VS and move from Nuget 3.3 to 3.4.
Are all nuget.config files contained in .nuget folders suddenly ignored by the nuget cli?
If I move the nuget.config from the .nuget solution folder into the same folder as the solution file, this will start to be used as the solution scoped configuration?
Solution scoped settings do not exist with 3.4+?
The documentation further states that:

In a solution folder, settings apply to all projects in subfolders. Note that if a config file is placed in a project folder, it has no effect on that project.

Does this mean if I have a project in a solution file that is not nested in the folder with the solution file itself, the nuget configuation does not apply to that project?
Please clarify.
Thanks,
Matthew

@gravidThoughts - what is your scenario? What kind of project do you have? What version of Visual Studio and NuGet are you using? What is it that you are intending to achieve?

@karann-msft  Let's start with the first scenario I described above. Upgrading Visual Studio resulting upgrading of VS nuget from 3.3 or earlier  to 3.4+. If the solution depended on .nuget solution folder, does it all of a sudden break? Per your documentation:

NuGet 3.3 and earlier used a .nuget folder for solution-wide settings. This folder is not used in NuGet 3.4+.

@gravidThoughts - no it does not.

@gravidThoughts - after you've had a chance to try out the side by side install and if you run into issues, file an issue on the Home repo.


Bump haskell.nix

The bump is to support cabal file revisions on CHaP which are necessary to get our packages to build without all those manual constraints we have in cabal.project.
I opened a draft PR to test the waters and warm up the chaces ahead of time since it's relatively big change in haskell.nix. I'll make sure everything works and then mark it as ready.

Rebased on top of #4865 to make the evalution work on hydra.

Some jobs are failing but they seem to be failing on master as well :shrug: Marking this as ready.


[Observation] Add some behaviroal tests for changes, transactions, and tracking

This adds the starts to some unit tests to validate observation (both manual and macro generated).

@swift-ci please test

@swift-ci please test


Create any-rsn-hider

Requested.
I'm sure it could be pretty useful for content creators, too. Like when you make RSMVs.

We aren't really interested in having a plugin that potentially allows impersonation for other people on streams/videos.


goto always timed out when using userDataDir and after the 2nd goto.

I can't find out the real reason but userDataDir did it:
Also, this problem is not real consistently. Sometimes, it works but once it fails then it will always fail. You should try to run the script several times.
TimeoutError: Navigation timeout of 30000 ms exceeded
import { connect } from "puppeteer-real-browser"

async function main() {
  const path = "/tmp/puppeteer-real-browser/" + process.pid
  const selector = "#chapterlist ul li"
  const url = "https://hivetoon.com/manga/juvenile-offender/"

  const { browser, page } = await connect({
    headless: false,
    customConfig: {
      userDataDir: path
    },
    args: [],
    skipTarget: [],
    fingerprint: false,
    turnstile: true,
    connectOption: {},
    fpconfig: {}
  })

  await page.goto(url)
  await page.waitForSelector(selector)
  await page.$$eval(selector, (as) => as.map((a) => a.dataset["num"]))
  await browser.close()

  const { browser: browser2, page: page2 } = await connect({
    headless: false,
    customConfig: {
      userDataDir: path
    },
    args: [],
    skipTarget: [],
    fingerprint: false,
    turnstile: true,
    connectOption: {},
    fpconfig: {}
  })
  await page2.goto(url) // Timeout <---------- TimeoutError: Navigation timeout of 30000 ms exceeded
  await page2.waitForSelector(selector)
  await page2.$$eval(selector, (as) => as.map((a) => a.dataset["num"]))
  await browser2.close()
}

main()
  .catch(console.error)
  .finally(async () => {
    console.log("Done")
  })

puppeteer-real-browser: 1.2.32
macOS

This issue was very troublesome for me.
It would proceed normally, but occasionally the error would occur again.
So, I added the following code before the page.goto and was able to resolve the problem:
page.setDefaultNavigationTimeout(0);

@kyungseopk1m what if the option setDefaultNavigationTimeout(0) made goto load forever?

@meotimdihia If the page doesn’t load within a few minutes, it might never load.
However, the reason I entered the code that way is because I manage the page, and I know it will definitely load.
In cases different from mine, you could set it to 10 minutes, like page.setDefaultNavigationTimeout(10 * 60 * 1000),
which should resolve the issue.

@kyungseopk1m I tried to add the code:  await page2.setDefaultNavigationTimeout(0) but it didn't work.

@meotimdihia Is there a reason you’re setting the selector "num" option instead of using the waitUntil: 'networkidle0' or 'networkidle2 option?
My guess is that the timeout is occurring because the communication hasn’t been completed.

@kyungseopk1m I tried all waitUlti options but it is always a timeout.
the code is necessary to reproduce the bug too:
  await page.waitForSelector(selector)

If we remove them then the problem won't happen.
So both userDataDir and waitForSelector did it.
P/S: I updated the issue.

https://github.com/zfcsoftware/puppeteer-real-browser/blob/3237722fa3e3bcbc95c9aaaae8f87f75fea195fd/src/test.js#L48
Temporary solution

@zfcsoftware Yes, I am using this solution - removing await in goto.
Hopefully, you can fix it in the next patches.

The library has been updated. This issue should be resolved. Please test with the latest version 1.3.0. I'm closing this issue for now. If the problem persists, please send me a message through this issue and I'll reactivate it.


Status of the library

Is this library still maintained by the developers?  I saw that last release was almost 2 years ago.
Thank you

Your question showed a complete lack of respect of my time, so you're entitled to nothing. You're already getting the library for free and are even getting a response. You don't need like my response, but you can't claim the moral high ground.


Add Toaster API reference

I have added documentation for newly added Toaster API.
Related Issue

DEVX-2599

Ready to merge after feature delivered to production


Configure proxy for development mode to allow same-site strict cookies

The proxy is required for local setup after this changes to API server - https://github.com/batnoter/batnoter-api/issues/13
Since localhost can not make use of subdomain to allow same site strict cookies we will need to setup the proxy.

Done


docs: homepage link http to https

Thank you! But we have the links to expressjs.com that are part of the published package (since you cannot modify a package version after publish) set to http while we try and get control over the cloudflare endpoint so we can ensure the ssl will stay working.

Last update on this effort in https://github.com/expressjs/discussions/issues/68#issuecomment-434171183


Fix several minor bugs and add many tests

MultiTapGestureRecognizer previously would assert if there was no
competition.
GestureArenaTeam would always select the first recongizer as the
winner even if a later recognizer actually accepted the pointer
sequence.
debugPrintStack would fail a type check if maxFrames was non-null.
FractionalOffset.lerp would throw a null-pointer exception if its
second argument was null.

Also, add a number of tests for previously untested lines of code.

@Hixie Moar tests (and bug fixes)

This is great. Thanks.

Two tests started failing with this PR. I'm failing to make the connection between the code changes and the test failures, so I'll keep an eye on what happens in the subsequent commits. Perhaps these were just flakes.
More in the build history

https://build.chromium.org/p/client.flutter/waterfall is green now.  Are these tests elsewhere?

Are these tests elsewhere?

They run in the device lab.

Got it.  I just needed to sign in to the build console to see the issue.

Seems to have been a flake.


Bug/pi 2204 approved premises and delius booking cancellation

Think this needs to run the checks after a format change. Perhaps push up an empty commit


Optimize BeginScope to avoid having to pass a pre-calculated string to it

This will avoid having to create strings for the scope message before a log message is ever logged. This currently shows up in the profiles for the plaintext benchmark.
@lodejard is putting together a design PR for this.

Just talked to @lodejard.  Moving is issues to RC2.

Closed, this is implemented by ilogger's define messages and lazy evaluation of string message and trace identifier.


Enable TSR planning problem to decide which IK solver to use.

Currently, we always use a Jacobian based iterative solver of DART. It'd be great to enable to choose which IK solver to use and to change other useful parameters.

Sounds good to me. We could use the IK information from a Manipulator and then use it when creating planning problems.


upgrade-to-2.11.x

This PR:

upgrades keda chart using vendir to the 2.11.2 app version so this supports from kubernetes 1.25 to 1.27

Testing
Description on how keda-app can be tested.

[ ] fresh install works

[ ] AWS
[ ] Azure
[ ] KVM

[ ] upgrade from previous version works

[ ] AWS
[ ] Azure
[ ] KVM

Other testing
Description of features to additionally test for keda-app installations.

[ ] check reconciliation of existing resources after upgrading
[ ] X still works after upgrade
[ ] Y is installed correctly

Checklist

[ ] Update changelog in CHANGELOG.md.
[ ] Make sure values.yaml and values.schema.json are valid.

Thanks @Gacko I'm waiting for customer testing before I merge this :)

No, thank you for doing that job! 🎉


Multi-thread support

yes,i think Multi-thread is very useful in some scenarios，especially when dealing with large files, I hope the author can realize this function as soon as possible，many thanks

I will most likely have some free time from first week of December. I will see if I can add this feature during that time.

This is a duplicate of #319. Please follow that issue for updates


Allow for false files.exclude and search.exclude rules to override other rules

files.exclude and searh.exclude should respect exclusion from those rules
"files.exclude": {
  "**/*.js": true, // will hide all js
  "some-module.js": false // currently effects nothing should prevent 'some-module.js' from being hidden
}

Currently the rules are mutually exclusive...

I think first as it is an exclusion rules, and by default all file are NOT excluded, then first should be applied entries with true value, and then entries with false should exclude from excluded if overlap.

bump, I believe it would be useful though.

https://github.com/Microsoft/vscode/issues/869


Create a link for the abaplint rule inside the annotation

Not possible to add url for external links inside annotation message, only for internal github link and mentions. Maybe add it as text to annotation and link for comments

Is it possible to trow the whole page inside the raw details button inside the annotation?


parallel execution: no heartbeat

Hi bcbio developers and users!
Thank you for the excellent pipeline!
When I run parallel version with torque, the controller job started, workers started and registered with the controller, and they started doing actual work - preparing files and aligning reads. So the communication between nodes seemed to be working, because the controller distributed work, workers reported to the main log file. The heartbeat did not work, and in a few hours workers were unregistered with the message:
2016-10-28 17:58:10.226 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (120 time(s) in a row).
2016-10-28 17:58:10.227 [IPEngineApp] CRITICAL | Maximum number of heartbeats misses reached (120 times 5010 ms), shutting down.

Our admin is trying to find out whether it may be a cluster issue,
any useful suggestions from the pipeline side?
Many thanks!
Sergey

It was an NFS issue like this
https://github.com/roryk/ipython-cluster-helper/issues/35. not it is ok
2016-10-31 12:51:25.559 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (52 time(s) in a row).
2016-10-31 12:51:30.569 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (53 time(s) in a row).
2016-10-31 12:51:35.579 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (54 time(s) in a row).
2016-10-31 12:51:40.589 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (55 time(s) in a row).
2016-10-31 12:52:25.679 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (1 time(s) in a row).
2016-10-31 12:52:30.689 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (2 time(s) in a row).
2016-10-31 12:55:00.989 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (1 time(s) in a row).
2016-10-31 12:55:05.999 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (2 time(s) in a row).
2016-10-31 12:55:11.009 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (3 time(s) in a row).
2016-10-31 12:55:16.019 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (4 time(s) in a row).
2016-10-31 12:55:21.029 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (5 time(s) in a row).
2016-10-31 12:55:26.039 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (6 time(s) in a row).
2016-10-31 12:55:31.049 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (7 time(s) in a row).
2016-10-31 12:55:36.059 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (8 time(s) in a row).
2016-10-31 12:55:41.069 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (9 time(s) in a row).
2016-10-31 12:55:46.079 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (10 time(s) in a row).
2016-10-31 12:56:36.179 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (1 time(s) in a row).
2016-10-31 12:56:41.189 [IPEngineApp] WARNING | No heartbeat in the last 5010 ms (2 time(s) in a row).

Thanks for following up Sergey, let us know if you run into any more problems.


Deprecated configuration option "raven"

RavenHandler has been removed in monolog 2. The bundle should deprecate using it in the configuration.

Closing per #321


Add option to serve a path instead of a port

Is your feature request related to a problem? Please describe.
One of my main use cases for loophole is to serve static files from a local directory. To do so, I use python -m SimpleHTTPServer 8080 to start a simple http server serving the current dir. But it would be super cool to have this integrated into the cli.
Describe the solution you'd like
It would be really cool if I could provide a path (usually just .) instead of a port when starting the cli. In this case, it is not forwarding the requests to the upstream server, but serving the files directly.
I can imagine that additional options will be needed (default index and error document). More sophisticated options can be added later, imho (follow redirects, mime type settings, etc.).
Describe alternatives you've considered
N/A
Additional context
N/A

For his we need to re-think the way we start the app.
Currently we use
loophole <port>

to start it. I propose to put it under some sub-command with separate options.
e.g.
loophole port 3000

loophole dir <path>

This would make it easier and avoid so many if in the code.

I would optimize for the user instead of the developer 😅 To me, it is easier to just provide the path or port. On the other hand, what if there is a folder named account (like a sub-command) or if there is folder named 8000 (like a port)? For a long version I would prefer sth like loophole open --dir <dir>, but we said, we wanted to have nice short version...

I started thinking wider, it would be great to have short command like we have right now. Just loophole 8000 is great, but:

if we want to extend it to also work for directories then the process would be as follow:

try to parse first arg as int, if it worked go the port flow
if it didn't try to parse it as path (what if you wanted to expose dir called 1234? :P)
establish tunnel

When we will start adding functionalities like basic auth, raw TCP tunnel we wil lend up with a lot of --options and I would like to avoid that. That's why I would like to end up with some categories for commands. E.g.:

loophole serve http 3000
loophole serve port 22
loophole serve path/dir ./my-dir
loophole account login
loophole account logout
loophole completion bash
etc

But it all seems so long and hard to type :/

But additionally considering interactive mode it somehow starts to make sense, because typing just loophole will open the interactive menu

Yeah, without interactive mode coming up, I would have argued that the user experience is getting to convoluted. But with it, it should be fine

Done


chore: set up dice game

Checklist:

[ ] I have read and followed the contribution guidelines.
[ ] I have read and followed the how to open a pull request guide.
[ ] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

This is blocked until #49018 is resolved. I wanted to start breaking this into steps, but we can't have campers debug syntax errors because the tests won't display if the code doesn't compile.

Closing this as stale. Will come back and revisit when unblocked.