[FLOC-3295] Implement read request load scenario for benchmarking tool

Fixes FLOC-3295
Paired with: @mpgomez
This change adds support for a read request load scenario which can be used to create a specified load from read requests when performing operations.
The scenario can be used as follows:
./benchmark --cluster cluster --scenario read-request-5 --metric cputime --operation no-op

This will start a read request load during the benchmarking run of 5 read requests per second.
When a scenario is being used during benchmarking, the scenario will not be in a 'started' state until the requested load rate has been achieved. If the load rate drops during the benchmarking run, it will throw an exception and terminate the run. It's likely that we will want to add tolerance to the check for the request rate in the future but it hasn't been addressed in this PR. When the scenario is stopped, it will wait for any outstanding requests to be completed within a provided timeout period.

A few minor comments. Please merge when addressed. Thanks.


css edits

address detail page, block detail page made responsive

Please compare current implementation of address detail page (left hand side) with the layout of Zeplin (right hand side). There are some differences in detail, e.g. content of transaction table, font sizes of hashes etc...

Please check the block detail page again. The table of block summary and hashes are not responsive. Also compare the transaction table with the layout of address detail page (because we don't have a layout of block detail page)

oh mate @sectore ... agree on the last screen. However with regards to different font size for hash I can't blindly follow design that was done only for certain width. I can't work with fixed width and keep it smaller and smaller down to 360px (where it still has to be viewable). Alex designed just two views whereas I have to sort all range from 1200px to 360px.
It also makes little sense to make some font-sizes bigger just for mobile. I'm happy to break it down nicely. Alex refused to do the css and you're busy so few decisions has to be mine. ;)

We'll have to move this to https://github.com/input-output-hk/cardano-sl-explorer/tree/master/frontend
This is to simplify packaging, I hope it won't be too much pain.


Rename CANWrapper_Poll_Messages to CANWrapper_Poll_Events

This function can emit error events, not just messages, and its name should reflect that.
The wiki should reflect this change as well

fixed in commit c33cfd0.


What is the format of edge index?

Given a graph of (batch_size, num_nodes, 3),
How do I create edge_indices such that the graph is fully connected?
It would help if the documentation has explicit descriptions of the expected sizes of things like edge_index for GATConv

https://pytorch-geometric.readthedocs.io/en/latest/notes/introduction.html

In case of a fully connected graph, your edge index would look like
edge_index = torch.ones(num_nodes, num_nodes).nonzero().t()

which holds all possible index pairs in a tensor of shape [2, num_edges]. However, I would advise against using a sparse formulation for a fully-connected graph. In this case, it is way faster to write a dense GAT conv formulation from scratch, using PyTorch dense matrices. In its current form, PyG is designed to operate on sparse adjacencies.


FancyZones: if window is not interesting, allow Windows snap

Summary of the Pull Request
Do not swallow WinKey+Arrow events for non-interesting windows. This makes apps that are in the "excluded apps" list behave as if "Override windows snap keys" is disabled - they will react to the Windows default snap.
This requires #1184 to be merged first. The changes specific to this PR are in the second commit.

PR Checklist

[x] Applies to #1182
[x] CLA signed.
[x] Tests added/passed

Detailed Description of the Pull Request / Additional comments

This does not include CycleActiveZoneSet. For whatever reason WinKey+Ctrl+Nubers seems to not work at all.
The "Save As..." dialog can be snapped using WinKey+Arrow when the "Override windows snap keys" is disabled. With this change the "Save As..." dialog will always react to the Windows default snap keys.

Validation Steps Performed
Unit test
Notepad and Notepads Save As dialog with and without it in the excluded apps.

We should mention in the description of the "excluded apps list" that an app added there will not be snapped by FZ but by Windows Snap.
Is this PR still depending on https://github.com/microsoft/PowerToys/pull/1184?

I've rebased it on master after #1184 landed, so it is ready to be merged. I'll reword the excluded apps description.

Changed the description to:
"To exclude an application from snapping to zones add its name here (one per line). Excluded apps will react to the Windows Snap regardless of all other settings."


SEEDS collection pull request

9,999 original SEEDS inscriptions.

Not sure why Rust failed, everything should be fine


Running inside a customised ubuntu 20.04 container has access issues

Running into authorisation issues inside my test machine docker container with this:
=========== end configuration ===========
running command: sudo openvpn --config /home/ploi/RSS_TestRunner_RSS.ovpn --daemon --log openvpn.log --writepid openvpn.pid
Tue Feb 28 15:09:37 2023 DEPRECATED OPTION: --max-routes option ignored.The number of routes is unlimited as of OpenVPN 2.4. This option will be removed in a future version, please remove it from your configuration.

Tue Feb 28 15:09:37 2023 OpenVPN 2.4.7 x86_64-pc-linux-gnu [SSL (OpenSSL)] [LZO] [LZ4] [EPOLL] [PKCS11] [MH/PKTINFO] [AEAD] built on Mar 22 2022
Tue Feb 28 15:09:37 2023 library versions: OpenSSL 1.1.1f  31 Mar 2020, LZO 2.10
Tue Feb 28 15:09:37 2023 Outgoing Control Channel Authentication: Using 160 bit message hash 'SHA1' for HMAC authentication
Tue Feb 28 15:09:37 2023 Incoming Control Channel Authentication: Using 160 bit message hash 'SHA1' for HMAC authentication
Tue Feb 28 15:09:37 2023 TCP/UDP: Preserving recently used remote address: [AF_INET]X.X.X.[227](https://github.com/XXXXX/XXXXX/actions/runs/4294400838/jobs/7483374173#step:7:228):16772
Tue Feb 28 15:09:37 2023 UDP link local: (not bound)
Tue Feb 28 15:09:37 2023 UDP link remote: [AF_INET]X.X.X.X:16772
Tue Feb 28 15:09:38 2023 WARNING: this configuration may cache passwords in memory -- use the auth-nocache option to prevent this
Tue Feb 28 15:09:38 2023 VERIFY OK: depth=1, O=60dee2741cde6d4eb501e983, CN=60dee2741cde6d4eb501e990
Tue Feb 28 15:09:38 2023 VERIFY KU OK
Tue Feb 28 15:09:38 2023 Validating certificate extended key usage
Tue Feb 28 15:09:38 2023 NOTE: --mute triggered...
Tue Feb 28 15:09:38 2023 4 variation(s) on previous 3 message(s) suppressed by --mute
Tue Feb 28 15:09:38 2023 [60dee2791cde6d4eb501e995] Peer Connection Initiated with [AF_INET]X.X.X.X:16772
Tue Feb 28 15:09:45 2023 Data Channel: using negotiated cipher 'AES-128-GCM'
Tue Feb 28 15:09:45 2023 Outgoing Data Channel: Cipher 'AES-128-GCM' initialized with 128 bit key
Tue Feb 28 15:09:45 2023 Incoming Data Channel: Cipher 'AES-128-GCM' initialized with 128 bit key
Tue Feb 28 15:09:45 2023 ERROR: Cannot open TUN/TAP dev /dev/net/tun: No such file or directory (errno=2)
Tue Feb 28 15:09:45 2023 Exiting due to fatal error
Error: VPN connection failed.

I have tried adding in the /dev/net/tun as per google-fu, but I then get an access permissions error (errno=1)

@Kayrah87 This action simply runs sudo openvpn --config /home/ploi/RSS_TestRunner_RSS.ovpn --daemon --log openvpn.log --writepid openvpn.pid as this log says.

Ah, I see.
Yes, the openvpn.log shows the same
Tue Feb 28 15:31:13 2023 Outgoing Data Channel: Cipher 'AES-128-GCM' initialized with 128 bit key
Tue Feb 28 15:31:13 2023 Incoming Data Channel: Cipher 'AES-128-GCM' initialized with 128 bit key
Tue Feb 28 15:31:13 2023 ERROR: Cannot open TUN/TAP dev /dev/net/tun: No such file or directory (errno=2)
Tue Feb 28 15:31:13 2023 Exiting due to fatal error

Also, when I create the dev/net/tun node, I then get this error:
Tue Feb 28 15:34:03 2023 ERROR: Cannot open TUN/TAP dev /dev/net/tun: Operation not permitted (errno=1)
Tue Feb 28 15:34:03 2023 Exiting due to fatal error

Resolved. It was my bad. Had to run the container with the --privileged flag for it to have permissions to write the tun node


SHIP-0038: Add Git Ref to Release Workflow

Changes
Update the release workflow to accept a git reference as a parameter. This will be used to check out the appropriate release branch when generating artifacts and release notes.
The action was also updated to use the (new?) inputs context, instead of relying on the inputs key existing in the github.events payload.
Part of #1555
Submitter Checklist

[ ] Includes tests if functionality changed/was added
[ ] Includes docs if changes are user-facing
[x] Set a kind label on this PR
[x] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes
NONE

Note - we should cherry-pick this back to the release-v0.12 branch.


Add /info endpoint

This should provide dotnet-monitor version, and possibly configuration (w/o PII)

Listening mode (+ port)
other configuration knobs

For Preview 6, just add "version" property as top-level on JSON object response.
{
  "version": "5.0.0-preview.5.21302.5+a99ec90eb220fa3b57f089677d8d1e85d48fb3a8"
}

See #546 for full details.


Feature/pixelate

Added a pixelate function to effects.go.

Hi,
I think a better algorithm for pixelate is to take the average color of each block instead of the color of the central pixel.
Also there is a bug in your code so that some pixels on the right and in the bottom of the image become black. Here's the demo code:
src := imaging.New(100, 100, color.RGBA{255, 0, 0, 255})
dst := imaging.Pixelate(src, 70)
imaging.Save(src, "test-pixelate-src.jpg")
imaging.Save(dst, "test-pixelate-dst.jpg")

Before pixelate:
After pixelate:
Also for faster processing I suggest to use a direct pixels access instead of At and Set methods. You can calculate the index of a pixel in the Pix slice and then access color channels eg src.Pix[i+0], src.Pix[i+1], src.Pix[i+2], src.Pix[i+3].


Concurrent recoveries of same backup delete each others' hook backups

Problem
Hooks are implemented using Velero's Backup interface.  Each hook writes a new backup to the object store as the backup being recovered.  If there are more than one VRG trying to restore the same backup at the same time, one may fail due to the backup already existing.  However, with #874 the recovery is to delete the backup and try again, which could result in the competing recoveries thrashing/deadlocking.
Approaches
Approach 1 - Write hook backups to a location where name conflict does not occur
Possible locations:

A local object store for hooks
A hooks subdirectory with a name assigned by Kubernetes, which is only cluster-unique, not necessarily unique across clusters so detect and handle conflicts

Approach 2 - Run restore hooks using Velero Restore interface
@tjanssen3 indicates it is not possible to execute a hook except when the pod is restored, even with Velero 1.9's merge behavior

Labeled as low priority because multiple recoveries from the same backup at the same time is an unlikely use case and one resulting in a failure is unlikely.


V1.1 Data Dictionary Updates

generate usable constraints, fix any data typing - HIVE 3 - Freeze on Dec. 2nd

Will freeze tomorrow. moved to Done.


FHIR Search Case Handling

Check if patient is the only case that cannot be handled by fhir search parameter patient.
Adapt FHIR Search build accordingly

So far it seems like only patient ressource is affected.


Fixed versus Static?

Fixed may refer to known or explicit or portable whereas static might mean compile-time constant. Consider this choice some more in:

README.md
documentation
static_integer
static_number
elastic_integer and any type that might morph out of elastic_integer. For example, fixed_integer<Digits> might be the right name for a non-elastic type or even for the family of types.

Current names are well established. Won't revisit unless there's external feedback, e.g. from committee.


🐛 MutableEventStoreInterface

I would suggest to drop the MutableEventStoreInterface and the implementations of the interfaces it provides. Event Sourcing is by design an append only model. You may wonder how to support the operations that are provided by MutableEventStoreInterface, here are some suggestions.

Record a correcting event. Let's say I've made a mistake in updating a customer balance. Once I discover this I record a corrective event (e.g. CustomerBalanceCorrected(amount=EUR 10.00, reason=malfunction in payment processing code)) based on the information from a credible source (e.g. my bank statements).
Reinterpret the old stream as a new stream by using the upcasters.
Archive events to other datasources.

I hope this was of help. Best of luck.

Thank you for your suggestion! I think you are totally right 👍

We have deprecated MutableEventStoreInterface in 1.2 and it will be definitely removed in 2.0.
Thank you again and if you have any other suggestions, please let me know.


Remove email verification step from sign-up workflow

Remove the step to verify an email address when a user creates a new account. New accounts should be activated automatically.
Note: email notice about new account creation has been revised accordingly. It is now just a welcome message.

Working as expected. Closing issue.


fix/remove docker build

Why are these being removed?

Why are these being removed?

Because on master deployment when running bulk command yarn build:containers:canary  we receive an error:
ERROR: The project "@energyweb/origin-backend-app" does not appear in the list of "deploymentProjectNames" from deploy.json.


Apply Spectre mitigations automatically in the CMake package

This slightly duplicates code in our build, but it is necessary because the compiler used to generate the package can be different than the compiler used by the consumer of the package.
This code checks if the given Spectre mitigation compiler flags are available, and if so, it enables them unconditionally for all code linking against openenclave::oecore (that is, all enclaves).
This resolves #1596 (by removing the need to document the flags as we now apply them opportunistically).

We need to manually test this by inspecting a sample.

Manually tested and verified:
/usr/bin/clang-7 -DOE_API_VERSION=2 -DOE_BUILD_ENCLAVE -DOE_USE_LIBSGX -I/home/andschwa/src/oe-install/share/openenclave/samples/helloworld/build/enc -isystem /home/andschwa/src/oe-install/include -isystem /home/andschwa/src/oe-install/include/openenclave/3rdparty -isystem /home/andschwa/src/oe-install/include/openenclave/3rdparty/libc  -m64 -fPIE -nostdinc -fno-stack-protector -fvisibility=hidden -ftls-model=local-exec -mllvm -x86-speculative-load-hardening -o CMakeFiles/enclave.dir/enc.c.o   -c /home/andschwa/src/oe-install/share/openenclave/samples/helloworld/enc/enc.c

Double-checked that mbed TLS still has it in its build properly:
build.ninja
148:  FLAGS = -nostdinc -I/home/andschwa/openenclave/include/openenclave/corelibc -I/home/andschwa/openenclave/include -DOE_NEED_STDC_NAMES -fPIE -fno-builtin-udivti3 -mllvm  -x86-speculative-load-hardening -ftls-model=local-exec -Wall -Wextra -W -Wdeclaration-after-statement -Wwrite-strings -Wpointer-arith -Wimplicit-fallthrough -Wshadow -Wmissing-declarations -Wmissing-prototypes -Wdocumentation -Wno-documentation-deprecated-sync -Wunreachable-code -O0 -g3

bors r+


Create Dynamic Programming section

Also add the first DP algorithm, 0-1 Knapsack

Thanks, @antoniofdias for the contribution


Improve build performance

modules are build sequential now for no apparent reason, furthermore thsi setup does not test versionconverter

eXpand.lab release 19.1.402.13 includes commit that relate to this task:

Improve build performance #479

Please test if it addresses the problem. If you use nuget add our LAB NugetServer as a nuget package source in VS.
Thanks a lot for your contribution.

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


feat(wms): Added caseSensitive option for params

I was working with a provider who uses all caps in WMS spec params (SERVICE, VERSION, etc.) and all lower case in non-spec proprietary params (provider, query, etc.). So I added a caseSensitive option that prevents the WMS layer config from assuming that all the parameters can be upper-cased.

I'm aware of what the spec dictates. However, in practice, many of the services we hit are merely WMS-like rather than fully spec-compliant. Therefore, it doesn't hurt to have an opt-in option for case sensitive parameters.


Cannot integrate live-chat to react native app

I have integrated react-native-livechat into my react-native project as mentioned in the documentation. but i am getting the following error on android device,

and the following error on iOS emulator -

On iOS when the message is dismissed the chat window can be opened but no messages can be sent.
project currently running on -
react-native - 0.59.9
react-native-livechat- "^2.0.0-6",

Hi @MnaveedS - do you use any Fetch polyfill in your app?

Hi @konradk no I am not using fetch, I am using axios for request handling.

@MnaveedS I don't know what is the root cause of your problem - our test apps and other developer's apps work fine. Are you able to recreate this problem on the new React Native app or share me privately sample app with this issue recreated?

@konradk I'll try integrating it with a new project and get back to you, Thanks.

@konradk I tried integrating LiveChat on a fresh project, and got the following error,

@konradk I installed LiveChat on a fresh react-native Project. And got the following bug

react-native - 0.59.9
react-native-livechat -  "^2.0.0-12"


Passing application parameters to SetupEntryPoint without affecting EntryPoint

Service Fabric Runtime Version: 7.0.457.9590
Environment: Must be one of these values Standalone
Description:
I have a StatelessService which has a SetupEntryPoint and I would like to pass some parameters from Application Manifest into it. There are 2 solutions I tried:

Use EnvironmentVariables, but that would make them visible to my EntryPoint application as well. Preferably I would like to avoid this. There doesn't seem to be a way to set environment variables only for SetupEntryPoint without them being visible to the EntryPoint application.

I tried the following but it didn't work because the arguments are passed as a verbatim string:
 <SetupEntryPoint>
   <ExeHost>
     <Program>MySetupEntryPoint.exe</Program>
     <Arguments>[MyApplicationParameter]</Arguments>
     <WorkingFolder>CodePackage</WorkingFolder>
   </ExeHost>
 </SetupEntryPoint>

I have seen #375 and several SO questions on this issue but none of it provides a solution or conclusively says that it is not possible. Any clarifications on this would be appreciated.

Assignees: /cc @microsoft/service-fabric-triage

Please re-open if you are still facing this issue so we can assist, but closing at this time.

Please re-open if you are still facing this issue so we can assist, but closing at this time.


Deprecate urn:oasis:names:tc:SAML:2.0:nameid-format:unspecified

    /**
     * @deprecated This is an incorrect name id format since unspecified does no longer exist in SAML 2.0, only use this for backwards compatibility
     */
    const SAML2_0_NAMEID_FORMAT_UNSPECIFIED = 'urn:oasis:names:tc:SAML:2.0:nameid-format:unspecified';

So we discussed it a bit and the change has too much breakage potential to perform just in a single migration. Our plan is to take the list and work through it manually to remove all use of this and/or contact SP's whether they require it. I think the migration should be backed out for the next SR release since this will take some time to get sorted.

This has been eliminated from all our production instances. The format can be removed from EB.


Extension issue

Issue Type: Bug
Extension Name: vsliveshare-audio
Extension Version: 0.1.91
OS Version: Windows_NT x64 10.0.19041
VS Code version: 1.57.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Closing and marking as a duplicate of #3808. Feel free to join the conversation there! Please try running the command > Live Share: repair audio installation. You can also try uninstalling the Live Share audio extension so you can use Live Share without the audio feature.


kitchen create with systemd as run_command affects OS session

On ubuntu having run_command: /lib/systemd/systemd in kitchen.yml and calling kitchen test or kitchen create will spawn a new systemd "session".
At first I thought I was being logged out and on some runs graphics got messed up completely. But then I found out that switching through Linux' different tty terminals (Ctrl+Alt+F1-F7) I could actually get back to my graphical session with everything still running.
Currently this setup is not usable I would say:
kubuntu 18.04
systemd 237 with /lib/systemd/systemd as run_command
kitchen-docker 2.9.0
kitchen-salt 0.6.0
kitchen-sync 2.2.1
test-kitchen 2.0.1
I was thinking maybe those are recent incompatibilities between the kitchen-* gems and switched to the versions that were recent when the article I was following was published (Jan 7). But these didn't help either:
gem 'kitchen-salt', '0.4.0'
gem 'kitchen-docker', '2.7.0'
gem 'kitchen-sync', '2.2.1'

I'm running into this as well trying to run an Ubuntu container on Arch.
My kitchen.yml:
platforms:
  - name: ubuntu-18.04
    # Special config since we need to start systemd
    driver_config:
      run_command: /sbin/init
      privileged: true
      provision_command:
        - sed -i 's/UsePAM yes/UsePAM no/g' /etc/ssh/sshd_config
        - systemctl enable ssh.service
      forward: 443

kitchen-docker (2.8.0)
test-kitchen (1.24.0)

In my case, I can't even get back to my x11 session; it disconnects and the system becomes unusable. I'm thinking this might be an issue with docker specifically, but I don't know how to dig further. Is there a way to see exactly what dockerfile and docker command docker-kitchen is generating?

I found a solution! This behavior was also reported upstream on Docker: https://github.com/docker/for-linux/issues/106#issuecomment-330518243
The TL;DR is: don't use privileged: true, which is what a ton of guides on the internet say to do. Use cap_add: - SYS_ADMIN. Taking the info I linked above, my working kitchen.yml looks like this:
platforms:
  - name: ubuntu-18.04
    # Special config since we need to start systemd
    driver_config:
      run_command: /sbin/init
      cap_add:
        - SYS_ADMIN
      run_options:
        env: container=docker
      volume:
        - /sys/fs/cgroup:/sys/fs/cgroup
      provision_command:
        - sed -i 's/UsePAM yes/UsePAM no/g' /etc/ssh/sshd_config
        - systemctl enable ssh.service

I don't know if that env: container=docker part is necessary, but the other person brought it up and it works.

Im following exactly the Saltstack Formula test platforms and I get this issue too, tried the options above and no luck so far.


Removing unnecessary Optional in SpecItemConfig for type and import mappings

As mentioned by @melloware in https://github.com/quarkiverse/quarkus-openapi-generator/issues/118#issuecomment-1286139685 Quarkus will generate unnecessary warnings when setting the type-mappings and import-mappings configuration items. For example setting this:
quarkus.openapi-generator.codegen.spec.documents_openapi_yml.type-mappings.File=InputStream
quarkus.openapi-generator.codegen.spec.documents_openapi_yml.import-mappings.File=java.io.InputStream

would cause the following warnings during code generation
[WARNING] [io.quarkus.config] Unrecognized configuration key "quarkus.openapi-generator.codegen.spec.documents_openapi_yml.type-mappings.File" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo
[WARNING] [io.quarkus.config] Unrecognized configuration key "quarkus.openapi-generator.codegen.spec.documents_openapi_yml.import-mappings.File" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo

This cosmetic issue was caused by the type of the config items in SpecItemConfig being Optional<Map<String, String>>. This PR fixes this by changing the type to Map<String, String>. Whenever nothing is provided for any of these config items then the corresponding map will just be an empty map.

Thank you all!


index.txt basic grammar edits and SQL statements

For your review and consideration please find basic edits to this document, and two topics for discussion below.
The two topics which require clarification:

In the 'fieldsets' and 'get_fieldsets' discussions on lines: 284 and 1367; the text suggests 'a list of two-tuples', but technically this is a 'two-tuple tuple'. I left it as is, because a list of two-tuples actually sounds less confusing, but seems to be technically wrong. Please weigh in on this issue.

SQL statements on lines 1100-1101; 1113-1114; and 1129-1130
Of the primary SQL flavors regularly referenced through the documentation (PostgreSQL, sqLite and MySQL), only PostgreSQL has the ILIKE statement. Unless clarification is added to the document at the point where ILIKE statements are referenced which explains the statement is exclusive to PostgreSQL when constructing case-insensitive searches, LIKE is a better representation of a SQL statement (even novices know the LIKE statement). Below please find the references to case-sensitivity searching in SQL by their respective sources.

PostgreSQL:
"The key word ILIKE can be used instead of LIKE to make the match case-insensitive according to the active locale. This is not in the SQL standard but is a PostgreSQL extension." (http://www.postgresql.org/docs/8.3/static/functions-matching.html)
sqLite:
"The default configuration of SQLite only supports case-insensitive comparisons of ASCII characters." (https://www.sqlite.org/faq.html)
MySQL
"  string comparisons are case insensitive by default. This means that if you search with col_name LIKE 'a%', you get all column values that start with A or a." (https://dev.mysql.com/doc/refman/5.0/en/case-sensitivity.html)
I do not have write privileges to the repo, so if these changes are appropriate, please commit.
Thank you.

I think in most places where "list" or "tuple" is used "incorrectly", either datastructure will work. Generally we have be preferring lists lately (see #24149 for discussion).

The LIKE/ILIKE issue is captured in ticket #20775. Could you please remove those changes from this PR -- we can tackle them separately based on the discussion in the ticket if you like.

Removed comments re SQL and reset the SQL statements (begrudgingly) back to ILIKE. i dont ILIKE, but your point is well taken. All good otherwise on comments regarding this doc. Thanks Tim

merged in 32f0c8f79697391e7067f1479045aa910ffa2c4e, thanks.


#2321 Update library

#2321

hbase-client : 1.2.0-cdh5.7.3 -> hbase-shaded-client : 1.2.0-cdh5.7.5

Current coverage is 39.84% (diff: 100%)

Merging #2335 into master will decrease coverage by 0.01%

@@             master      #2335   diff @@
==========================================
  Files          1614       1614
  Lines         55929      55929
  Methods       10997      10997
  Messages          0          0
  Branches       7717       7717
==========================================
- Hits          22294      22285     -9
- Misses        33635      33644     +9
  Partials          0          0

Powered by Codecov. Last update 45bf559...25cb1e6


Dangling transactions when pooled connections dropped

Setup
Versions

Diesel_async: 0.3.1

Feature Flags

diesel_async: deadpool, postgres

Problem Description
Firstly, thank you for your work on this crate and on Diesel itself! They have been very nice to work with compared to using tokio_postgres directly.
I gather you're aware of this issue already (https://www.reddit.com/r/rust/comments/zn9ut0/announcing_dieselasync_020/), but as dangling transactions can cause deadlocks and data loss, I'm wondering whether it might be possible to improve the current behaviour, or at least document it more prominently if that is not possible.
As a quick-and-dirty solution, would something like the following make sense, where the connection is dropped from the pool (with implicit rollback) when it is recycled with an active transaction?
diff --git a/src/pg/mod.rs b/src/pg/mod.rs
index 6a4832b..b16a0a0 100644
--- a/src/pg/mod.rs
+++ b/src/pg/mod.rs
@@ -475,6 +475,15 @@ async fn lookup_type(

 #[cfg(any(feature = "deadpool", feature = "bb8", feature = "mobc"))]
 impl crate::pooled_connection::PoolableConnection for AsyncPgConnection {}
+impl crate::pooled_connection::PoolableConnection for AsyncPgConnection {
+    fn is_broken(&mut self) -> bool {
+        match self.transaction_state().status.transaction_depth() {
+            Ok(Some(_)) => true,
+            Ok(None) => false,
+            Err(_) => true,
+        }
+    }
+}

 #[cfg(test)]
 pub mod tests {
diff --git a/src/pooled_connection/mod.rs b/src/pooled_connection/mod.rs
index ae5abdf..9c35d06 100644
--- a/src/pooled_connection/mod.rs
+++ b/src/pooled_connection/mod.rs
@@ -258,7 +258,7 @@ pub trait PoolableConnection: AsyncConnection {
     /// [ManageConnection::has_broken] for details.
     ///
     /// The default implementation does not consider any connection as broken
-    fn is_broken(&self) -> bool {
+    fn is_broken(&mut self) -> bool {
         false
     }
 }

Alternatively, could an approach like tokio_postgres uses work instead?
https://github.com/sfackler/rust-postgres/pull/835/files?
Steps to reproduce
Anything that causes a transaction future to be dropped before being polled to completion, eg the calling future terminating due to client disconnection, or things like timeouts
            let mut pg = pool.get().await?;
            tokio::time::timeout(std::time::Duration::from_secs(1),
                pg.transaction::<_, Error, _>(|pg| async move {
                    sql_query("some sql").execute(pg).await?;
                    tokio::time::sleep(std::time::Duration::from_secs(2)).await;
                    Ok(())
                }.scope_boxed())
            ).await.unwrap_err();

Checklist

[x] I have already looked over the issue tracker for similar possible closed issues.

[x] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)
[x] This issue can be reproduced without requiring a third party crate (except deadpool)

Thanks for opening this issue. I must admit that I thought that part of code was already in place, so it seems like I just missed to do that. Which means I would definitely accept a PR for that, although I would rather change the default is_broken() implementation to do that for all connection implementations.
I'm also interested in other more targeted solutions for this, but I've failed to find something yet that's not coupled to a specific runtime or that would possibly block on drop or swallow errors. I think the postgres approach needs access to the underlying runtime to work. I'm also not user how they handle errors there or guarantee that it happens "fast" after dropping the future (which is somewhat important, so that the relevant future is not open for that long).

Turns out there was an existing function which checks for this and was just not hooked up - will push through a PR.

Solving it in a generic way does seem tricky. One possible idea for a DB-neutral solution would be a newtype that wraps the various pool implementations, and stores a vec of connections needing cleanup. Each time pool.get() is called, it could use that async context to clean up any of those objects, so if the pool is used with reasonable frequency, the objects in a bad state shouldn't stick around for long. I think deadpool provides a way to take a connection out of the pool, but don't know about bb8 or mobc, and it would add some extra complexity.

(sorry, meant to close+re-open the PR to re-run the tests)

I stumbled upon this issue when I was looking at the docs for AsyncConnection::transaction in order to understand if this warning was still applicable or if this issue being closed means its resolved (and that the docs are just outdated).
@weiznich would you be able to clarify if this warning could be removed now or if it is still applicable?

@bmwill I generally do not provide support in old closed issues, especially not if I'm pinged while asking a support question.


How index a log?

I want to index logs in elastic but I don't know how it could be done

Please check here. https://github.com/elasticquent/Elasticquent#indexes-and-mapping


Model Store

Don't merge.  Still improving UI and clarifying use cases.

Can you use a minified version of angular-ui-router.js? That's what we try to do for other similar files, and it will make this PR look less scary to review, too :smiley:

Thanks for the comment.  I will do it now.

Close this one as the branch will be removed.


Instead of transpiling the circuit, transpiling with lookahead routing method sometimes throws exception 'Lookahead failed to find a swap which mapped gates or improved layout score.'

Environment

Qiskit Terra version: 0.36.1, 0.36.2
Python version: 3.9.12, 3.8.13
Operating system: Ubuntu 20.04, Windows 10

What is happening?
Running qiskit.compiler.transpile with routing_method='lookahead' sometimes throws qiskit.transpiler.exceptions.TranspilerError: 'Lookahead failed to find a swap which mapped gates or improved layout score.'
I would expect that the transpiler gives me back the transpiled circuit. It should not terminate with the exception above, after all, basic routing doesn't fail.
How can we reproduce the issue?
Inside the .zip are 3 files, run_me.py, and two example circuits in qasm format.
Examples are from https://www.cda.cit.tum.de/mqtbench/.
Run run_me.py and the exception should occur.
example.zip
Alternatively, download the examples directly by going to https://www.cda.cit.tum.de/mqtbench/, and selecting Variational Quantum Eigensolver (VQE) , 7 qubits, and Indep.
The downloaded files that I used in the following code is vqe_indep_7.qasm.
# run_me.py
# Issue with transpilation using "lookahead" method

import qiskit

from qiskit.test import mock

# This throws an exception: qiskit.transpiler.exceptions.TranspilerError: 'Lookahead failed to find a swap which mapped gates or improved layout score.'
qiskit.compiler.transpile(
    qiskit.circuit.QuantumCircuit.from_qasm_file('./vqe_indep_5.qasm'),
    coupling_map=mock.FakeCambridgeV2().coupling_map,
    layout_method='trivial',
    routing_method='lookahead',
    optimization_level=1
)

What should happen?
When the transpiler runs with the circuit from vqe_indep_7.qasm and the coupling map from FakeCambridgeV2, it will result in the following exception:
qiskit.transpiler.exceptions.TranspilerError: 'Lookahead failed to find a swap which mapped gates or improved layout score.'
This is unexpected since I would expect my circuit to be transpilable on a device with more than twice the number of qubits as the circuit.
Any suggestions?
The issue seems to be originating in a commit a few years ago:
https://github.com/Qiskit/qiskit-terra/commit/7d0ba7cb8b82588168356480bdad23a83028b592
It seems that they add some convoluted control flow in order to prevent hangs, however, it now throws the exception, which has result of no transpiled circuit returned.

Yes, sabre does work for me. I can use that as a workaround for now, thanks for the advice and the update on the situation!


Feature request: search

example: https://github.com/shishkin/astro-pagefind

Hi @thuanowa, this was mentioned a few weeks ago, #57, and I'm still of the same opinion. I may circle back around once they've had a bit more time to mature


404 on delete/modify SLS (Old 271)

In gitlab by @A-Lawrence on Aug 31, 2015, 01:08
Report by Andy Ford.
404 on delete/modify of secondary password.

In gitlab by @A-Lawrence on Aug 31, 2015, 01:13
Fixed the 404 on delete/modify of SLS.  Resolves issue #488472 (271).
ÔåÆ <>
ALawrence66 - 2015-02-22T21:09:38.904134+00:00


Update manifest description

From:

Hiro Wallet is a safe way to manage your STX, sign into apps, and protect your funds while interacting with Clarity smart contracts.

To:

Hiro Wallet is the #1 wallet for Web3 on Bitcoin. Connect to apps and manage your assets secured by Bitcoin with the most popular wallet for the Stacks blockchain

A plain text string (no HTML or other formatting; no more than 132 characters) that describes the extension. The description should be suitable for both the browser's extension management UI and the Chrome Web Store. You can specify locale-specific strings for this field; see Internationalization for details.

Needs shortening.

Hiro Wallet is the #1 wallet for Web3 on Bitcoin. Connect to apps and manage your assets secured by Bitcoin


Sample stats: total and peak

This PR covers most of what we proposed in https://github.com/prometheus/prometheus/issues/10181. It's a combination of my work, @alanprot's work, and @Harkishen-Singh's work.
It adds new stats:

totalQueryableSamples: The total number of samples read out of the underlying Queryable instance inclusive of any samples that are buffered between range steps.
totalQueryableSamplesPerStep: The above statistic tracked per step in a range query. These will always add up to the value in totalQueryableSamples. The engine does not compute or report these by default. We only do the work and report the data if an engine flag is set (available via --enable-feature per-step-stats from the CLI) and a per-query flag is set (stats=all in the request). We introduce a new QueryOpts struct in the Engine's API to make having these kinds of per-query options easier in the future.
peakSamples: The high water mark of the value compared against MaxSamples during the course of query execution.

Per @bboreham's feedback, this PR also changes the query stats response from a struct to an interface and adds a way to override that response. Implementations like Cortex can use this to read stats from the Context and insert them into the response that's sent to the user and/or remove information from the stats response.
This does not include the series stats we proposed because we have some more thinking to do about how best to represent them. More to come on that front.
The performance impact of the new statistics that are always on should be negligible per our local benchmarking. Even the per-step statistics don't have much impact outside the combination of very simple queries with many steps. I'll upload these benchmark runs in a bit.
An example JSON response with all these stats on for max_over_time(up[5m]) with 30-second steps over 5 minutes. The underlying data here started only a minute before running the query.
{
  "status": "success",
  "data": {
    "resultType": "matrix",
    "result": [
      {
        "metric": {
          "instance": "localhost:9090",
          "job": "prometheus"
        },
        "values": [
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ],
          [
<PHONE_NUMBER>,
            "1"
          ]
        ]
      }
    ],
    "stats": {
      "timings": {
        "evalTotalTime": 0.006922879,
        "resultSortTime": 2.677e-06,
        "queryPreparationTime": 0.003402187,
        "innerEvalTime": 0.002431834,
        "execQueueTime": 0.000819559,
        "execTotalTime": 0.008257353
      },
      "samples": {
        "totalQueryableSamplesPerStep": [
          [
<PHONE_NUMBER>,
            "4"
          ],
          [
<PHONE_NUMBER>,
            "6"
          ],
          [
<PHONE_NUMBER>,
            "8"
          ],
          [
<PHONE_NUMBER>,
            "10"
          ],
          [
<PHONE_NUMBER>,
            "12"
          ],
          [
<PHONE_NUMBER>,
            "14"
          ],
          [
<PHONE_NUMBER>,
            "16"
          ],
          [
<PHONE_NUMBER>,
            "18"
          ],
          [
<PHONE_NUMBER>,
            "20"
          ],
          [
<PHONE_NUMBER>,
            "20"
          ],
          [
<PHONE_NUMBER>,
            "20"
          ]
        ],
        "totalQueryableSamples": 148,
        "peakSamples": 31
      }
    }
  }
}

/prombench edfe657b54057e427469a2151153d305270662a4

/prombench main

/prombench cancel
I assume you have seen enough benchmark data.

Hi @roidelapluie, thanks a lot for looking into this.

Do we need to compute the peak and total? Could we let that to the caller instead?

I don't think is possible to compute the one stats from another as they are tracking different things (Peak is not the max(totalQueryableSamples) but instead is the max samples kept in memory during the query execution. The Peak stats show how close the query was to reach the query.max-samples limit.

Hi @roidelapluie, thanks a lot for looking into this.

Do we need to compute the peak and total? Could we let that to the caller instead?

I don't think is possible to compute the one stats from another as they are tracking different things (Peak is not the max(totalQueryableSamples) but instead is the max samples kept in memory during the query execution. The Peak stats show how close the query was to reach the query.max-samples limit while the totalQueryableSamples shows how many samples was used to run the query.

I mean, can we only track them per step, and not compute the overall maximum.

One thing I've been thinking about doing before landing this is changing the stats suffix from PerTime to PerStep. Thoughts @roidelapluie and others?

:+1: for PerStep

close/reopen to trigger circleci

Windows tests should be fixed by rebasing on top of main.

triggering ci

Thanks!

/prombench cancel

This pull request has memory issues but it's head has been rebased on top of #10450, which I believe to be the actual issue.


version 1.14?

There doesn't seem to be a release for version 1.14. I see commits that seem to be related to that version though. Is there a commit hash I can checkout that would be optimal for trying to get this version working? or were there other issues preventing that?

Hi, if I remember correctly, there were some issues and I decided to skip directly to v1.15. If you really want to try anyway, I suggest to get version v1.15 of this project and use TENSORFLOW_TAG parameter of the CMakeLists.txt to checkout tensorflow v1.14, but I cannot guarantee it will work for you.

This was a while ago, but I did get this to work incidentally. Changed the tag and built fine.


[LEDGER] remove TxMgr interface

Type of change

Improvement (improvement to code)

Description
We have a TxMgr interface as the initial plan was to implement two types of transaction manager:

lock-based txmgr
snapshot isolation based txmgr

In the last three years and in the many coming years, we may not implement another TxMgr. Hence, we remove the interface.
Additional details
As we decided to keep a single TxMgr implementation, we can avoid additional folder depth and reduce a package by moving lockbasedtxmgr to txmgr.
TxStatInfo defined in txmgr is imported in both lockbasedtxmgr and validator. If we move the lockbasedtxmgr to txmgr, we will get an import cycle. As TxStatInfo is used by kvledger, metrics, lockbasedtxmgr, and validator, we need to find a suitable place to keep these structs. Only if we can do that, we can move lockbasedtxmgr to txmgr -- FAB-17757
Merge this only after merging  #1094
Related issues
FAB-17756

We will reopen this once we finish refactoring the mvcc validator.


vandium.validation (serverless-offline)

Code :
https://github.com/vanerleo/offline-vandium-error
added vandium.validation({ to /test1/
when calling ,  /test1/?seoName=test and /test2
gives error  on /test2 : {"errorMessage":"validation error: child \"seoName\" fails because [\"seoName\" is required]","errorType":"ValidationError","stackTrace":null}
while calling in reverse  (/test2  and ,  /test1/?seoName=test )  is fine

Seems to be a problem with the require.cache not being flushed properly - not sure if this is a serverless-offline issue or a configuration problem. For now use the '--skipCacheInvalidation' with serverless-offline and use nodemon to reload.

updated the ./serve.sh , but it does not resolve the problem

serverless-offline is not clearing it's require cache properly, therefore vandium is not getting reloaded, and the same vandium code is being used for all handlers.
The simplest solution is to use a different vandium install for each function.
Create a separate package.json for each function with vandium as a dependency, and do an "npm install" in each folder (you can script it if you like, but don't create a symlink back to a global node_modules).

WOW, thats crazy :)
i can confirm that your solution worked!
thank you for your support


Fix #374 and #409 - Prevent XSS and add Markdown support

Fixes the XSS vulnerability by handling three different formatting modes we support separately.
A new helper parse-message.js was added to utils that uses the popular battle-tested unified library with plugins for HTML and Markdown to sanitize and translate Markdown to HTML, adding full Markdown support.
The default formatting mode no longer uses v-html to dynamically add arbitrary HTML to the DOM, however, users can still customize messages with dynamic HTML, opting in using text-formatting='{ "html": true }'. The injected HTML is first sanitized to remove dangerous elements. text-formatting='{ "markdown": true }' was similarly added to allow users to use CommonMark-compliant Markdown.
The default formatting functions in FormatMessage.vue were refactored into parse-message.js to consolidate the parsing logic.
Note these changes also set the groundwork to address #409.
What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[x] Feature
[ ] Code style update
[x] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing

Note: I implemented prettier in the project, if you can please use so we keep the code format consistent

Note: I implemented prettier in the project, if you can please use it so we keep the code format consistent

Hey @antoine92190, I've run prettier for the files changed in this PR. ✅

Also, happy to discuss the unified dependency and how it could play into Markdown support and https://github.com/antoine92190/vue-advanced-chat/issues/409.
The goal is to simplify format-string.js. Specifically compileToJSON. This method basically compiles Markdown into a JSON AST. We can use remark-parse to parse a richer mdast AST and process that in compileToHTML. This would minimize changes to the styling layer with the benefit of a standardized grammar for Markdown.

It would be awesome has this changes in a new release 😄


Add math.h include for Linux compilation

"math.h" is required to build the NAM dsp code on Linux. Adding it does not cause a problem on Windows.

I haven't compiled this plugin for Linux, but you can find a more simplified LV2 plugin here:
https://github.com/mikeoliphant/neural-amp-modeler-lv2

THANK YOU! It compiled under Void Linux (GNU) and works fine in both Ardour7 and Qtractor.


Expose the board serial number

Hello
In order to ease development of pigpio dependent Home Assistant integrations, specifically regarding their guidelines on entities unique identifiers, I would like to suggest exposing the board serial number.
Would you consider a pull request ?

If it easy for me to understand, is of very limited (small) code changes and passes the suite of tests, then yes, I will consider it.

Hello Guy
Thanks a lot for your feedback. I believe all the conditions are met, so went ahead with #558


Column has Alignment property but it does work

Column has alignment property but it does not wok by default it takes every child at center.

that's the intended behavior
See https://api.flutter.dev/flutter/widgets/Flex/crossAxisAlignment.html


Define a Simple Data Model and OData Service with CDS

Tutorial URL: https://developers.sap.com/tutorials/odata-05-data-model-service.html
Please specify the step you are referring to:
Step 11 - Create the database
Get the error as below:
11:38:29 AM (Provision) Could not create the 'Products-Products-db-hdi-containerwXERTYdnNe24v13x' instance of the 'hana' service type for the 'Products-db-hdi-container' resource. CF-ServiceBrokerRequestRejected(10001): Service broker error: Service broker hana-broker failed with: Can not create service instance 'Products-Products-db-hdi-containerwXERTYdnNe24v13x': There is no database available. Ensure that you have a database available in space 'dev' within organization 'XXXXXX'.
please help to check it, thanks.

Hello @MikeHua2019  and thank you for engaging with us (and sorry for my tardiness in replying!). This tutorials issues area is for problems with the tutorial content itself (typos, incorrect screenshots, and so on) rather than challenges you may face relating to specific or general runtime circumstances. May we please refer you the SAP Community Q&A where you can tag your question to have it appear in the appropriate section, and where others will also benefit from your question, and everyone from the answers you receive. Thanks!
Please also note that owing to the plans regarding Neo (see below), this and related tutorials will be deprecated and replaced soon by ones for the multi-cloud environment. Thanks for your understanding!
Plan message from SAP Cloud Platform:
To support our strategy to provide SAP Cloud Platform on a multi-cloud
foundation, SAP will move the focus of our SAP Cloud Platform trial
offering completely to the SAP Cloud Platform, multi-cloud foundation
(Cloud Foundry environment).
We recommend that all users of the SAP Cloud Platform trial on the SAP
Cloud Platform, Neo environment, move to the multi-cloud environment.
Existing Neo-based trial accounts will continue to be available until
November 13, 2020.
At that point, all existing trial accounts on SAP Cloud Platform, Neo
environment, and any related data will be deleted.
Should you have any questions, please contact
<EMAIL_ADDRESS>


change: Remove extensions input

Prettier and its plugins already keep track of what extensions and files they can be run on. In an effort to make sure this runs on everything it can (e.g. Gemfile, .yml, etc.), we'd do well to let Prettier handle those decisions for us; otherwise we'd be duplicating much of that functionality and knowledge.

While I'd like to get this functionality in.  It hasn't been working quite right in a few repos.  And given how long it's sat here without feeling necessary, I'm going to close it for now and may revisit it later.


Please add Kissimmee, Florida

TO<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>My name is [YOUR NAME] and I live in [AREA/NEIGHBORHOOD]. I am writing to you today to demand the immediate reallocation of funds away from the Kissimmee Police Department and into desperately needed community resources and programs. Specifically in the form of: safe and affordable housing, a living wage, more accessible and extensive public transport, services and shelter for the homeless and vulnerable populations, ensured investment in community based food banks, co-ops and gardens, free and non-coercive healthcare infrastructure, economic assistance, youth programs that promote safety and wellbeing, and equal job and educational opportunities for all.
In the adopted 2020 budget the Kissimmee Police Department is slated to receive 25 million dollars. Significantly more than any other department in the General Fund and more than several other departments combined. Whilst this generous funding of our communities policing takes place; Social and Quality of Life Services throughout the city remain critically defunded. With places like the Osceola Council on Aging, HOPE Community Center,  Park Place Behavioral Healthcare, and the Osceola County Boys and Girls Club, left amongst other groups to share a mere $350,000. While it is easy to see this absurd disparity in funding. To put this number into further context, this year KPD will spend more than $350,000 alone just putting gas in their cars.
As we continue to mourn and protest in solidarity, with communities across the nation, for the Black and brown lives disproportionately lost at the hands of the police. It is evident cops do not make our cities any safer. In fact their existence actively drains from the resources that do. So, I invite you to imagine what 25 million dollars could do if it were poured into the people of this city instead of a system designed to keep our most vulnerable enslaved. I invite you to imagine what our schools, our libraries, our community centers, our parks, our buses, our roads could look like. I invite you to imagine what our people could do if they were adequately cared for. The opportunity is here.
As we enter into another month of a global pandemic, of widespread unrest. Now more than ever it is time for change. Empty gestures and promises of reform will no longer be accepted. Reallocate this money. Starting with a financial assistance fund for those impacted directly and indirectly by Covid-19. To be distributed in the form of direct payments to those living within our community.
Elected officials, your action is imperative. The members of this community refuse to continue to support a budget that does not meet the needs of its people. Can I count on you to reallocate and reinvest these funds? If not, you can count on me to help vote you out.
Sincerely,
[YOUR NAME HERE]

Thank you for submitting your city to defund12.org! Your request is now in the queue for review and publishing. We’ll reply here if we have any questions.

Link to annual budget for City of Kissimmee : https://www.kissimmee.org/government/finance/annual-budget-documents

Editing notes: I tightened up some copy, removing a few sentences in order to have the message be a little more direct. I used the exact numbers for budget stats (and did a double-take at that shared social and quality-of-life services fund wow). https://www.kissimmee.org/home/showdocument?id=7124
To<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: [*** INSERT UNIQUE SUBJECT LINE ***]
Message: (Don't forget to replace the [x]'s with your information!)
My name is [YOUR NAME] and I live in [AREA/NEIGHBORHOOD]. I am writing to you today to demand the immediate reallocation of funds away from the Kissimmee Police Department and into desperately needed community resources and programs. Specifically, I am advocating for safe and affordable housing, a living wage, more accessible and extensive public transport, services and shelter for the homeless and vulnerable populations, investment in community-based food banks, co-ops and gardens, free and non-coercive health care infrastructure, economic assistance, COVID-19 relief, youth programs that promote safety and well-being, and equal job and educational opportunities for all.
In the adopted 2020 budget, the Kissimmee Police Department is slated to receive $25,698,675, significantly more than any other department in the General Fund and more than several other departments combined. While this generous funding of our city's policing takes place, social and quality-of-life services throughout Kissimmee remain critically underfunded. Precious resources such as the Osceola County Council on Aging, HOPE Community Center, Park Place Behavioral Healthcare, Boys and Girls Club of Osceola County, and more are left to share a mere $350,000.
Are the needs of the people not worth more?
I invite you to imagine what $25 million could do if it were poured into the people of this city instead of a system designed to keep our most vulnerable enslaved. Imagine what our schools, our libraries, our hospitals, our community centers, our parks, our buses, our roads could look like. Imagine what our people could do if they were adequately cared for. We do not need more police. Now, more than ever, it is evident cops do not make our cities any safer. In fact, their very existence actively drains from the resources that do.
Elected officials, your action is imperative. The members of this community refuse to continue to support a budget that does not meet the needs of its people. Can I count on you to reallocate and reinvest these funds? If not, you can count on me to help vote you out.
Sincerely,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]


Update boots reference in baremetal docs

Issue #, if available:
Description of changes:
The tinkerbell boots service has been renamed to smee upstream. Update the docs to mention boots and smee refer to the same service.
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve
/cherry-pick release-0.20


Add Bottom Sheets and Vector Drawable Guides

[ ] Bottom sheets
[ ] Vector drawables

References:

http://android-developers.blogspot.com/2016/02/android-support-library-232.html
http://stackoverflow.com/a/35620244
http://developer.android.com/training/material/drawables.html#VectorDrawables
http://developer.android.com/training/material/animations.html#AnimVector
http://developer.android.com/reference/android/graphics/drawable/VectorDrawable.html

cc @rogerhu

Blocking bug:
https://code.google.com/p/android/issues/detail?id=201817 ->
Devices/Android versions reproduced on: Samsung Galaxy Tab 2 (GT P5210) / KitKat 4.4.2

Add a spinner to any activity anywhere in the layout xml file:

<Spinner
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    />

App crashes with exception:
android.content.res.Resources$NotFoundException: File res/drawable/abc_spinner_textfield_background_material.xml from drawable resource ID #0x7f020040

Re vector drawables, early stab here: https://github.com/codepath/android_guides/wiki/Drawables#vector-drawables

I'm implemented Bottom Sheets myself for a project, I can post some trip reports from my journey here :) So far I'm trying things from here: http://stackoverflow.com/questions/35618998/how-to-implement-bottom-sheets-using-new-design-support-library-23-2

I have also implemented BottomSheetFragment for my project by following this tutorial: http://code.tutsplus.com/articles/how-to-use-bottom-sheets-with-the-design-support-library--cms-26031

You can see our guide on bottom sheets here. I added a link to that tutorial referenced above. Vector drawables guide is done now as well. Closing.


[Question] Overconfidence on OOD data and the assumption on uniform distribution?

Hi,
In the paper you make a statement on the ID and OOD data:

Gradients are backpropagated from the KullbackLeibler (KL) divergence [22] between the softmax output and a uniform distribution. ID data is expected to have larger KL divergence because the prediction tends to concentrate on one of the
ground-truth classes and is therefore less uniformly distributed. As depicted in Figure 1, our key idea
is that the gradient norm of the KL divergence is higher for ID data than that for OOD data, making it
informative for OOD uncertainty estimation

However, it is also a well-known fact that for OOD data, models tend to be overconfident. In this case, for OOD I would also expect the distribution of the softmax to be 'spiky'. I see that you use temperature scaling for the softmax output which can calibrate the network, but in most of the experiments, this parameter is set as 1. So I was wondering what's your take on that regarding the assumption you make that ID is expected to have larger KL divergence than OOD? The gradient estimation seems to work quite well so I suspect that I don't understand the problem fully. If you could help me clarify that, it would be great!
References on the overconfidence of the models:
http://arxiv.org/abs/1610.02136
http://mi.eng.cam.ac.uk/reports/svr-ftp/evermann_stw00.pdf
https://arxiv.org/abs/1906.02530
Thanks,
Kacper

Hi Kacper,
Thanks for your interest in our work! This is a great question! You are absolutely correct that overconfidence in neural networks is a huge issue, and that's exactly one of our motivation to resort to gradient space for OOD detection. As for your question, I have the following three points that might be relevant:

One advantage of KL divergence is that it accounts for the probability distribution of all classes instead of the dominant class alone. Suppose we have a confidence score of 90% concentrated on a dominant class, the probability distribution of remaining 10% can be different among the rest of classes between ID vs. OOD data.
This is empirically confirmed to some extent in Figure 2, where we plot the GradNorm distribution when using both uniform and one-hot as target vector. Interestingly, when we use one-hot target (label of the dominant class), it's hard to separate ID and OOD. However, when we switch to uniform vector (as in the KL divergence), ID and OOD data become more separable, which demonstrates the superiority of utilizing information from all labels.
Last but not least, in Section 5 we showed that GradNorm can be decomposed into a multiplication of two terms and captures joint information from both output and feature space. Therefore, another advantage of our method is that it also extracts useful information from feature space to mitigate the overconfidence issue in the output space.

Hope this answers your question!
Best,
Rui

Hi Rui,
Thanks for the answer, that makes it much clearer now!


Integration tests of us043

Dependencies:
https://github.com/ONSdigital/ras-backstage/pull/48
https://github.com/ONSdigital/response-operations-ui/pull/44
https://github.com/ONSdigital/ras-party/pull/99

 US043: INT Find an RU [Large]


Use swiftsureDocument

depends on  https://github.com/openprocurement/openprocurement.auctions.core/pull/133

Pull Request Test Coverage Report for Build 72

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.009%

Totals

Change from base Build 70:
0.0%

Covered Lines:
542

Relevant Lines:
686

💛  - Coveralls


Change to get_value_str() to escape regexes broke capa2yara.py

Description
With https://github.com/mandiant/capa/commit/58e94a35cbaa384307410ef846b5965868b051e2 the regexes returned by get_value_str() are escaped which breaks e.g. https://github.com/mandiant/capa/blob/3f449f3c0f1e2544ca7bad83c90e2d162ec0b916/scripts/capa2yara.py#L262
Steps to Reproduce
Run
python ./scripts/capa2yara.py  rules/host-interaction/file-system/reference-absolute-stream-path-on-windows.yml 2>&1 |grep x5D

The 2nd line shows the regex escaped, which is of no use in yara:
...
INFO:capa2yara:doing kids: [regex(string =~ /^(\\\\\?\\)?([\w]\:|\\)(\\((?![\<\>\"\/\|\*\?\:\\])[\x20-\x5B\x5D-\x7E])+)+\:\$?[a-zA-Z0-9_]+/)] - len: 1
INFO:capa2yara:doing regex: '/^(\\\\\\\\\\\\\\\\\\\\?\\\\\\\\)?([\\\\w]\\\\:|\\\\\\\\)(\\\\\\\\((?![\\\\<\\\\>\\\\\\"\\\\/\\\\|\\\\*\\\\?\\\\:\\\\\\\\])[\\\\x20-\\\\x5B\\\\x5D-\\\\x7E])+)+\\\\:\\\\$?[a-zA-Z0-9_]+/'
...

Expected behavior:
No escaping
Actual behavior:
See above
Versions
Most recent github version
Additional Information
How should we fix this? Introduce another function which returns the regex unescaped?
(capa2yara.py is the only script in scripts/ which uses the function, so shouldn't have broken more)

here's the regex in the raw rule yaml:
      - string: /^(\\\\\?\\)?([\w]\:|\\)(\\((?![\<\>\"\/\|\*\?\:\\])[\x20-\x5B\x5D-\x7E])+)+\:\$?[a-zA-Z0-9_]+/

https://github.com/mandiant/capa-rules/blob/57b3911a72462e0597ca0d6685f8b02b38857765/host-interaction/file-system/reference-absolute-stream-path-on-windows.yml#L17C1-L17C112
and as logged above, from get_value_str:
INFO:capa2yara:doing kids: [regex(string =~ /^(\\\\\?\\)?([\w]\:|\\)(\\((?![\<\>\"\/\|\*\?\:\\])[\x20-\x5B\x5D-\x7E])+)+\:\$?[a-zA-Z0-9_]+/)] - len: 1

and they look the same to me. So get_value_str returns the regex as provided in the capa rule. This can (almost directly) be passed to re.compile to create a regular expression instance. I assume it can be converted to a yara-compatible regex trivially, too.
Can you explain the problem in a little more detail?

The line after that is the problem. That's what ends up in the yara rule and it's escaped when it shouldn't:
...
INFO:capa2yara:doing regex: '/^(\\\\\\\\\\?\\\\)?([\\w]\\:|\\\\)(\\\\((?![\\<\\>\\\"\\/\\|\\*\\?\\:\\\\])[\\x20-\\x5B\\x5D-\\x7E])+)+\\:\\$?[a-zA-Z0-9_]+/'
...

is this possibly because the logging statement uses %r instead of %s? i don't think there's any extra escaping being done by capa.

Should be this line from the commit above:

I'll work on this

I took a look at this a few days ago, but I couldn't figure out what was going on. Here's my output after reverting 58e94a35cbaa384307410ef846b5965868b051e2:
❯ python scripts/capa2yara.py -q rules/host-interaction/file-system/reference-absolute-stream-path-on-windows.yml > test.yar
...

import "pe"

private rule capa_pe_file : CAPA {
    meta:
        description = "Match in PE files. Used by other CAPA rules"
    condition:
        uint16be(0) == 0x4d5a
        or uint16be(0) == 0x558b
        or uint16be(0) == 0x5649
}

rule capa_reference_absolute_stream_path_on_Windows : CAPA  {
  meta:
 	description = "reference absolute stream path on Windows (converted from capa rule)"
	namespace = "host-interaction/file-system"
	static scope = "basic block"
	dynamic scope = "call"
	hash = "51828683DC26BFABD3994494099AE97D"
	reference = "This YARA rule converted from capa rule: https://github.com/mandiant/capa-rules/blob/master/rules/host-interaction/file-system/reference-absolute-stream-path-on-windows.yml"
	capa_nursery = "False"
	date = "2024-04-05"
	minimum_yara = "3.8"
	license = "Apache-2.0 License"

  strings:
 	$re_aaa = /\x00(\\\\\?\\)?([\w]\:|\\)(\\((?![\<\>\"\/\|\*\?\:\\])[\x20-\x5B\x5D-\x7E])+)+\:\$?[a-zA-Z0-9_]+/ ascii wide

  condition:
    capa_pe_file and
 (
			$re_aaa
	)
}

It seems like YARA doesn't like spaces in meta field names, so I had to add underscores to static scope and dynamic scope. Even after that, it still doesn't work with the converted rule. I'm also not sure where the null byte came from. I tried to play around with the original capa regex, but couldn't get that to work either.
❯ yara test.yar 51828683dc26bfabd3994494099ae97d.elf_
error: rule "capa_reference_absolute_stream_path_on_Windows" in test.yar(43): invalid regular expression "$re_aaa": syntax error

What did the regex look like when the script was working?


The code completion is jammed together

setting.json is wrong
hope that may help you

There are two settings that should be tweaked: editor.suggestFontSize and editor.suggestLineHeight


font-size

I would like to request the ability to be able to change font size and height of the card to better reduce to blank space absorbed by this entity row.

Have you tried applying card-modder to it?

Have you tried applying card-modder to it?

So same as with the line/text colour.... some of us try to not use too many custom cards... so using yet another custom card to provide a basic function like colour or font size doesn't make much sense.

Having every developer provide a full suite of customizations for each card is far from the answer as well. I dont plan to add it but PRs are welcome

So why ask @JamesMcCarthy79 to open an issue (as you did on the forum) when you have no intention of fixing it then? Not everyone has the ability to write code/PR's.

Just because I won't personally do it (at least at this time) doesn't mean I want to hide the request. I didn't ask you or him specifically to create a PR, but anyone is open to create PRs on any of my cards and resolve issues.

Hi,
I would like to make the divider line white, can somebody help me how to this with card-mod: https://github.com/thomasloven/lovelace-card-mod, sorry but quite new to this all and not so easy to understand for a novice I'm afraid :-(
Thanks!

Revisiting this topic... Can we have text font lighter (or some user control over it)? Now it seems to be bold so it looks way heavier than even standard card headers... does not look good. I tried to use card-modder but is seems not to be deprecated. Card mod works only on cards, not rows, so to apply it it makes things overly complicated/nested within. So it makes impossible to achieve reasonable UI aestetics :-(

Added theme variable in 1.4.0 (will need to reinstall as I have moved the repo from custom-cards to my personal). I'm in the process of getting it added to default HACS


[HOLD for payment 2023-07-24] [$1000] Web - Pasting text or link in edit message pastes the text or link in main compose box

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
. Open the app
2. Open any report
3. Send any message
4. Copy any text
5. Edit message sent in step 3
6. Paste the text in edit message and observe that text is pasted in compose box
Expected Result:
App should paste text in edit message if edit message is in focus
Actual Result:
App pastes text in main compose box even if edit message is in focus
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.40-1
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/e27772be-d689-4050-8c3f-5393e9e30d60
https://github.com/Expensify/App/assets/93399543/756e2dc9-c7eb-4a41-96ee-dc864babd847
Expensify/Expensify Issue URL:
Issue reported by: GH handle - @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1689249016488049
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019d975b520ddadde8
Upwork Job ID:<PHONE_NUMBER>772632064
Last Price Increase: 2023-07-14

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Might be related to https://github.com/Expensify/App/pull/21583

Yeah, I'm looking on it. Probably have the fix, just need to double check it

I've created PR with fix: https://github.com/Expensify/App/pull/22817

@robertKozik this is the problem at hand now
Revert fixes  ✅   https://github.com/Expensify/App/issues/22833

@robertKozik Any update?

@cristipaval or @francoisl remove Awaiting Payment labels and add back the external label here!

@Santhosh-Sellavel @francoisl if this is up for the External here's is my Proposal.
Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Pasting text or link in edit message pastes the text or link in main compose box
What is the root cause of that problem?
The RCA for this issue is how we're handling the paste event listener. For every composer component we're creating an eventListener for paste event which is results in multiple event listeners. Also since the first event listener is added for main composer this causing the issue that even we paste text into the edit message composer main composer getting focus because of line here.
https://github.com/Expensify/App/blob/b9508b616f3a7229365c8bd7280fd1e34cfd2f9b/src/components/Composer/index.js#L281
What changes do you think we should make in order to solve the problem?

This change is only web I think cause ios and android have different logic.

We can't directly have paste listener on input, we can have listeners to added once an input get focus but this won't allow copy pasting files when the input is not focused.
We can overcome this issue using the event. Since the paste event will sends the target of the event where it occurred we can use that target to focus and run the code.
We need to create a new ref maybe focusedInputRef and assign to this inside handlePaste function.
this.focusedInputRef.current = event.target

here
https://github.com/Expensify/App/blob/b9508b616f3a7229365c8bd7280fd1e34cfd2f9b/src/components/Composer/index.js#L309
And here https://github.com/Expensify/App/blob/b9508b616f3a7229365c8bd7280fd1e34cfd2f9b/src/components/Composer/index.js#L280 use the focusedInputRef to focus or blur. Since execCommand works on the field which is focused, this works pretty well with the text or HTML or even emojis but for files the functionality would be still same.. once we click it would be triggered by main composer only not from the one which we're editing (I don't know how to fix this :(, maybe this is expected or so.).

What alternative solutions did you explore? (Optional)
NA
Result
report.webm
Willing to evlaute my solution against multiple test-cases to be future proof.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Pasting text or link in editing message pastes the text or link in the main compose box
What is the root cause of that problem?
We're using the same component in main composer and editing message: Composer
When pasting, we focuses the input and paste the content to it using document's native function.
https://github.com/Expensify/App/blob/6e5b07cf45944f28373e188d6f02620a12d6da29/src/components/Composer/index.js#L279-L290
We add document paste event handlers for all editing messages and main composer. And in my observation, only the latest added paste event handler is called. In our case, the main composer paste event handler is called because the main composer is mounted at the last among composers.
This is the root cause
What changes do you think we should make in order to solve the problem?
paste event should be called only on focused element. Now we are adding event handlers on composer mount or navigation focus.
I think it is reasonable to add paste event handler on focus event of the input element and remove handler on blur event of the input element.
Replace
https://github.com/Expensify/App/blob/6e5b07cf45944f28373e188d6f02620a12d6da29/src/components/Composer/index.js#L171-L177
with
        this.textInput.addEventListener('focus', () => document.addEventListener('paste', this.handlePaste));
        this.textInput.addEventListener('blur', () => document.removeEventListener('paste', this.handlePaste));

And below code isn't needed anymore. Remove them
https://github.com/Expensify/App/blob/6e5b07cf45944f28373e188d6f02620a12d6da29/src/components/Composer/index.js#L210-L212
This works as expected.
Note: I think we don't need below codes
https://github.com/Expensify/App/blob/6e5b07cf45944f28373e188d6f02620a12d6da29/src/components/Composer/index.js#L281
https://github.com/Expensify/App/blob/6e5b07cf45944f28373e188d6f02620a12d6da29/src/components/Composer/index.js#L286-L287
What alternative solutions did you explore? (Optional)

@Santhosh-Sellavel Sorry I was OOO already when you commented. I saw that external and help wanted labels were added, so should I handle this issue, or rather help you with proposal checking?

@robertKozik It's better we take care of fixing it here. Can you come up with a robust solution for this?

Hi @Santhosh-Sellavel I was tweaking around with that and I think I have one proposition, it actually consists from two already proposed snippets/ideas.

snippet from here to fix everything except the cursor issue. I believe it's the most elegant fix in this case, as we're not creating additional listeners to target only one particular textInput. Additionally, we won't be splitting the handling of pasting inside and outside of the input.
Fix the cursor problem - with that im not sure whether this should be fixed alongside this issue, as it looks like the regression from another PR https://github.com/Expensify/App/pull/22443, which is stated here. If we should include this I'll go with the solution from this insight too.

Let me know what you think about this approach. Aside from that, I experimented with multiple listeners, one on the document for outside pasting and another for the textInput itself, but it feels overengineered.

Thanks, @robertKozik
Ya, the snippet looks good to me as well, please ensure it doesn't break anything!
About the cursor issue, it's better to revert and let them work on it.
cc: @francoisl @cristipaval

@Santhosh-Sellavel I created draft PR with follow-up https://github.com/Expensify/App/pull/23135 . Should be wait for reverting the cursor bug?

@Santhosh-Sellavel
What do you think about my proposal?

@Santhosh-Sellavel Can I make my PR ready for review or are we waiting for reverting the cursor bug?

@robertKozik  please make it ready for review.

@robertKozik Bump!

@Santhosh-Sellavel Today PR will be ready

It seems that the payment is due here.

Payment on its way, I am working on this now @cristipaval - I was OOO before.

Hmm, there is no accompanying upwork job. The only payment on Upwork also is the issue reporter, as per the comment above @cristipaval 👍🏼
@robertKozik does not require payment (Contractor)
@Santhosh-Sellavel does not require payment (Eligable for Manual Requests)

Upwork isn't letting me create a new job, keep getting an error. I will do this ASAP but looking like tomorrow so I can do a full clear cache, logout.

No Payments are due here for Myself & @robertKozik  as it is a regression handling.
I think we can skip the checklist here as well.
All parties are aware of the mistake, and these tests will decide on the origin issue.

Thanks @Santhosh-Sellavel!
@zanyrenney the only due payment is the reporting bonus for @dhanashree-sawant!

Still waiting on @dhanashree-sawant to accept job offer and finish payout - link is here https://www.upwork.com/jobs/~019f97a860cbdfce94

Hi @zanyrenney, when I click on the job link, it is giving me 'Access denied' error

Hi @zanyrenney, I have just recieved the invite but job is kept hourly, can you change that to fixed price?

new upwork job - https://www.upwork.com/jobs/~011e7524a423ae1245

reporting bonus paid.
all other payments handled via NewDot. closing!


panic: glGetActiveSubroutineName

Hi there,
I'm trying to get the executable to run (Win10 64-bit), I downloaded Go and got Msys2. I ran 'go build' and it built fine, when I run the executable however I get:
panic: glGetActiveSubroutineName

goroutine 1 [running, locked to thread]:
github.com/samuelyuan/openbiohazard2/render.InitRenderer(0x400, 0x300, 0x5bb7da)
        H:/OpenBiohazard2-master/render/rendermain.go:49 +0x5b1
main.main()
        H:/OpenBiohazard2-master/main.go:32 +0xe5

Is there a step I'm missing?
Thanks for you help and awesome work!

The error message means gl.Init() failed. What version of OpenGL are you using? The code is using OpenGL 4.1.

Hey thanks for the reply! I was using the same OpenGL 4.1 as I didn't modify anything in the code when I built it. I tried higher versions (by changing all the GL4.1 github import links) but they had the same issue. The one that worked for me was 3.3 for some reason.
The other thing, it looks like this implementation currently only supports pal versions, my data was jap and I just had to modify the const paths in the resources file to point to my data, should probably just add a note for that.
Great work!


Zendesk SDK upgrade to v2 for Android and iOS

refactored repo to latest plugin seed
add demo and demo-angular
generated typings and fixed them from latest SDK v2
finalized API exposition work from master

Hi guys,
Any progress on the PR.
I have successfully tested the @rhanb version, and except some minors bugs it is working well.
Thanks,
N


Broken licensing conditions

Hello,
Thanks for this project, which we are planning to package for Debian.
I see that you chose to use Unlicensed to license this project.  Unfortunately, that's a very problematic license that seems not to have been written with the consultation of lawyers, and therefore almost certainly doesn't do what you think it does.  See the discussion thread on the license-review mailing list starting at http://lists.opensource.org/pipermail/license-review_lists.opensource.org/2012-January/001360.html which explains the issues with that license in detail.
If you would like to use a public-domain license, my strong recommendation (as discussed on that thread) is to use the Creative Commons CC0 license instead, found at https://creativecommons.org/share-your-work/public-domain/.  This license was created (with legal input) to do what you intend as best as is currently legally possible.  There is a tool on that page ("Use this tool") for creating the correct wording for your project.  It is also (afaik) an internationally recognised license.
Best wishes!

@juliangilbey Thanks for the input, using The Unlicense was a known risk w.r.t. the complexities of making something public domain whilst also licensing it. I'll take a look at the materials and likely take your suggestion to change licenses.
I use Debian as my daily driver, so I deeply support getting this packaged over there. Admittedly the debian packaging process is still a bit much for me to grok given the amount of time I've got. Regardless, please let me know how I can help.
We use GoReleaser to manage our builds which support outputting .deb files if that helps https://goreleaser.com/nfpm/

@zaquestion Hi Zaq? - pleasure!  A preliminary Debian package was made by Felix Lechner, and I'm sponsoring it for him.
There are two small issues that we've patched locally - would you prefer me to send them to you via email, via a bug report or via a merge request?
Finally, the current version of the package failed to build for me.  I'll report that as a separate issue.
Best wishes,
Julian

@juliangilbey pull request to GitHub works best, though I'd rather get them than not, so whichever means achieve that :D

any progress on fixing the license here? i'd also love to have this in debian and might be able to help if this is unblocked...

Yeah I'd like to switch over to the CC0 license. I haven't had a chance to go through license creator tool, but if someone wants to take a swing at it and put up an MR to change licenses, thats fine with me.

@juliangilbey @anarcat --  @ianfoo has put up changes to move us over to the Creative Commons license. I've brought them in, can you do a last spot check and LMK if looks up to snuff?

Hi @zaquestion and @ianfoo - thanks for doing this!  It looks mostly fine to me, as it seems to be copied and pasted from CC.
Two things I noticed which could be tweaked:
(1) In the README.md file, add:
This work is published from:

  United States.

before the closing
.  (This is what CC recommends.)
(2) In the LICENSE file, remove the paragraph from lines 5-13 ("CREATIVE COMMONS CORPORATION IS NOT A LAW FIRM..."), as that is a communication from CC to you, not from you to the reader/user of your software.
Best wishes,
Julian

@juliangilbey I've updated with your suggestions, thanks for the help here apologies for dropping the ball. Assuming you'll want a new tag with this in it, I'll try to prioritize that for soon, I need to address an issue with project creation that came in after some gitlab.com changes https://github.com/zaquestion/lab/issues/329

so licensing is fixed here? maybe the issue can be closed then, or should that wait for a release?
in any case, thanks for taking those questions seriously, it's much appreciated!

Well, goreleaser.yml still nentions unlicense:
$ ack -i unlicense
.goreleaser.yml
33:  license: Unlicense

On the other hand, the main license file was switched to CC0 and released with version 0.16.0 .

@dod38fr Thanks for the heads up, it's fixed on master now. I searched for any more mentions and didn't see any. Thanks for helping get this closed out.


Here is my hypothesis while training dataset.

While I was training with my dataset, here are some hypotheses that model can train very stable.
I want to double-check with you guys because I want to gather other opinions from these hypotheses.

First, audio should be clear and constant with background and voice tone and pitches
Second, try to remove short audio data which can make pads
Third, try to ignore std of sampled audio while training
Last, if there's underfitting happens, try to reduce learning rate first and reduce segment length if the first method doesn't work.
Plus, do turn off fp16_run if still, your loss value goes up even you tried those things above.

These hypotheses just came from opinions from the issues and my experiences.
Does anyone agree with this?
Please let me know if my hypotheses are not right.
Thanks

Thank you very much, can you share more detail about your parameters

@NoahDrisort There's not much difference between default setups from config.json. I changed only learning rate and segment lengths and fp16_run.
Try make half values from hyper-params.

Hi. Could you please explain what did you mean by

Third, try to ignore std of sampled audio while training

?

Closing due to inactivity.


Cocoapods Installation Issue

Not sure if this is happening to anyone else with 3.1.0 but I keep getting this error with Cocoapods 0.37 and 0.38 beta...

Did you try to setup again cocoapods master repository using pod setup ?

If I add the dependency to my podspec file as ":git => 'https://github.com/m1entus/MZFormSheetController.git'" it works. 3.1.0 didn't seem to work.
Will run that now @m1entus

Apologies @m1entus. Think I was a little hasty with the GitHub issue. Will close this unless it comes up again.


Add /status page for monitoring systems

This is useful for feeding puppet data into a monitoring system such as Prometheus.

A feature I've wanted for a long time but haven't implemented yet is to expose content-type json on the normal puppetboard endpoints. Automated systems hitting puppetboard is the future, then puppetboard provides essentially a read-only/filtering proxy to the puppetdb api. I think implementing that would be superior to this, and neatly fit your use case as well. What do you think?

Prometheus doesn't support JSON, but I'd be all for a ?format=json / ?format=prometheus or something if that's what you mean.

So this is currently very much tied to what Prometheus expects so a generic endpoint like /status is a bad idea.
As an aside, why the heck doesn't it support a standard like JSON?

I totally agree that snatching /status and saying it's only available for Prometheus is bad. I do prefer the Prometheus format to inspect as a human however, so I'd say it's a better default, but I don't feel strongly about it.
The reason JSON is bad is that you cannot parse it while streaming it - you need everything. For monitoring data that's especially bad. The preferred format is actually protobuf, but that is a bit more hairy to implement, and is totally not readable by humans.

To recap: I'd be glad to have /status accept multiple formatters when anyone steps up and wants to write that

I'm okay with this taking ?format=prometheus  or a Content-Type:Prometheus header or something on the /status route. Sound good? Also is this kind of raw string manipulation the best way right now to generate prometheus data? Is there a pip package for vomiting that stuff that we can require I guess is what I'm asking.

Fine with me! What would you have the default be? I.e. not having ?format= set.
There is a client library, but since we're not a static daemon the library doesn't do much good. The format is simple enough to allow trivial serialization, so a library doesn't really make much sense in the "collect-and-render" case I'm afraid.

I think giving a not implemented error, which could be a 200 OK instead of a 501 makes sense if we weren't explicitly asked for format=prometheus.

Eh, no. We have HTTP status codes for that. Return 200 OK when we mean "actually we couldn't service this request because the have no clue how to answer for this format" is not alright and a way to break the web.
A 406 "content not acceptable according to the Accept headers" is the correct way to go. This is also what Twitter and other API's return when "an invalid format is specified in the request".

Ok. (Not breaking the internet)++

Hello:
Is this still needed? If so, can you rebase and squash?

Totally forgot this change. We run with this in our production, so we would of course like it to be merged, but with the mentioned modifications above. I haven't gotten around to implement that.

Please also note that Puppetboard is now environment aware and has undergone many changes with the 0.1.0 release. Please update your code to help reflect those changes.
Might be better to create a new PR

Since the comments so far have not been about the code but the general design I fail to see how closing this to open a new PR would be better. The whole discussion would be lost. Ultimately though, it's the maintainers' call.

It was mentioned that it would be good to support different content types, i.e. JSON, Prometheus, XML (if possible), etc. That would allow this to be monitored by any number of systems, not just Prometheus.
Even though I'm relatively new when it comes to maintaining this project my experience with application monitoring do not really like template streaming (not to mention that I don't fully understand it myself) so outputting the entire result-set at once is my preference.

I'm going to abandon this and create a puppet_exporter instead, as I believe that's a better approach for Prometheus and alike. Thanks for the comments all!


Include stub files

Addresses #109
Just creating this PR for now, if there are any comments let me know! Thanks for the help along with this. There are only a couple of things:

As discussed in the issue, the scorer callable is a pain to try to type hint because of the limitations with ParamSpec (I really hope this is built out in the future...). I cannot for the life of my figure out a better way to type it than simply Callable[... either.
In string_metric I noticed that processor came after weights, and in one of the overloads is required. This isn't really a "typing" issue but that does seem strange as it makes it impossible to provide levenshtein(s1, s2, processor). If you want the processor without the weights it has to come in keyword form, which feels to me like they should be keyword only arguments levenshtein(s1, s2, *, weights = (1,1,1), processor, score_cutoff=0). But that's obviously a breaking change, and really not related to this PR... just a thing I noticed

Oh and I can rebase to squash if that's desired, I just wanted to get this submitted before I headed out for a bit

As discussed in the issue, the scorer callable is a pain to try to type hint because of the limitations with ParamSpec (I really hope this is built out in the future...). I cannot for the life of my figure out a better way to type it than simply Callable[... either.

Yes I think, that this is the best we can do right now. At least we can validate the return type and use it for the function return type.

which feels to me like they should be keyword only arguments levenshtein(s1, s2, *, weights = (1,1,1), processor, score_cutoff=0). But that's obviously a breaking change, and really not related to this PR... just a thing I noticed

Yes these arguments should always be provided as keyword arguments. So having them keyword_only would make perfect sense. The only reason I did not use them back then was Python 2 compatibility (which was dropped since then). So I guess I will make them keyword only in the next major release, where this interface break is fine (so far I did not come across a single code base, which did not provide these parameters as keyword argument).

Other than that this looks good to me. As far as I could see this should be compatible with Python3.6 is this correct?

Yes that's correct, all the main typing things came in version 3.5. The only newer one that would be useful in this typing set (as far as I can think) would be typing.Final for __author__ and __version__ and __license__ in the init. That came in 3.8 though so I didn't use it

Could you squash + rebase? Also I think the package_data in the setup.py is no longer needed, now that it is in the MANIFEST.in file.

Done!

Thanks


Test Execution Command Line Tool Version 15.9.0 is unable to execute TestRunParameter option.

Test Execution Command Line Tool Version 15.9.0 is unable to execute TestRunParameter option.
The command to pass TestRunParameter is mentioned here.
Using this doc i am executing the command in cmd:
dotnet test  -- TestRunParameters.Parameter(name="target_branch", value="dev")
but this command is returning error saying 'One or more runsettings provided contain invalid token'
Steps to reproduce
install dotnet core 2.1.56 sdk.
Add an MsTest project.
Now from cmd, execute the command: dotnet test  -- TestRunParameters.Parameter(name="target_branch", value="dev")
The above command will throw an exception on cmd which will say 'One or more runsettings provided contain invalid token'
Expected behavior
The above mentioned command should pass TestRunParameters to TestContext.Properties dictionary.
Actual behavior
Command is returning error 'One or more runsettings provided contain invalid token'
Diagnostic logs
'One or more runsettings provided contain invalid token'
Environment
.NET Core SDK (reflecting any global.json):
Version:   2.1.526
Commit:    26a950bee8
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19045
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\2.1.526\
Host (useful for support):
Version: 2.1.30
Commit:  5e17f34de8
.NET Core SDKs installed:
2.1.526 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

can you try to escape like:
dotnet test -- TestRunParameters.Parameter(name=\"target_branch\", value=\"dev\")

PS. I see you're using a no more supported version of .NET Core and you're on a very old version of the test tool(15.9.0), my advice is to move the the new supported versions 17.4.0 and .NET 6.0
Here you can see the official lifecycle learn.microsoft.com/en-us/lifecycle/products/microsoft-net-and-net-core

I'm going to update the documentation excluding netcore2.1 from the support of this feature.

Also can we make it clear somewhere that if you want to pass parameter to test project in .net core 2.1 sdk then you need to use dotnet vstest command and pass runsettings file. There is no way to pass TestRunParameter through dotnet vstest command in .net core 2.1 sdk.

dotnet vstest it's been superseded by dotnet test https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-vstest and as said above we don't usually update documentation for non supported runtimes. In this case we forgot to add the "starting version" for the feature.


Mainly package version update

Problem

A significant number of changes were made in package version 23.11.05~ynh2, bringing back upgrade testing would also be a good idea starting from there.

Solution

Update version in manifest.toml + bring back upgrade test in tests.toml (from 23.05.11~ynh2)
Remaining ldapauth related code was also removed

PR Status

[ ] Code finished and ready to be reviewed/tested
[ ] The fix/enhancement were manually tested (if applicable)

Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

!testme


docs: props: remove duplicate modeConfig prop

Thanks for clarify @Mostafatalaat770, We're working on the new docs website, as soon is released this will be the main reference guide and would correct some of the possible errors we have.


Level parameter of DataFrame.groupby should allow Sequences

The level parameter of DataFrame.groupby does not except sequences, when it should. (Corresponding Doc)
This is the same as https://github.com/pandas-dev/pandas/issues/47548
To Reproduce
From the linked Issue above
import pandas as pd
df = pd.DataFrame([[1, 2, 3, 4], [5, 6, 7, 8]], columns=['a', 'b', 'c', 'd']).set_index(['a', 'b', 'c'])

print(df.groupby(level=[0, 1]).sum())

My System

Windows 10
Python 3.11
MyPy 1.5.1
pandas-stubs <IP_ADDRESS>218

Potential Fix
The issue linked above was fixed by changing the type of level from Level | None to IndexLabel | None. (https://github.com/pandas-dev/pandas/pull/47560)
I suppose, the same Fix should be applied to the groupby-Overloads in pandas-stubs/core/frame.pyi.
I am happy to create a pull request, if that helps.
But i would appreciate it, if someone more involved with this library could confirm that the proposed fix makes sense.

Yes, a PR is welcome, along with tests.  Would need to change the type of level in groupby() in frame.pyi and series.pyi for all the overloads.
A test like you have above should suffice, i.e., you don't need to check all the overloads, although it would be good to have a test for both Series and DataFrame.