Assertions not correct mark next to table name.

Describe the bug
Please see attached image: all assertions are passed but still shows a X mark next to table name.
Previous soft-deleted assertions are still evaluated.
Expected behavior
Green tick mark if all assertions are passed.
Workaround
Hard delete assertions.
Screenshots
Added
Additional context
v0.9.2

Hi @sachinwadhwa, it seems that your screenshots didn't come across- could you try adding them again?

Hi @sachinwadhwa I'm going to close this one out due to inactivity. Please feel free to reopen if this is still an issue and you have screenshots!


User Profile Service writes cancellation exceptions to system log

The description for Event ID 0 from source Application cannot be found. Either the component that raises this event is not installed on your local computer or the installation is corrupted. You can install or repair the component on the local computer.

If the event originated on another computer, the display information had to be saved with the event.

The following information was included with the event:
[2016-12-19 20:12:39Z] <Microsoft.R.Host.UserProfile.RUserProfileService> (E):
**User profile service failed with exception: The operation was canceled..**

the message resource is present but the message is not found in the string/message table

https://github.com/Microsoft/RTVS/commit/bc65d9dc0bcad9be763bec781b7f1487bc00af82


Kazakhstan Address Data!!!!!

I confirmed with @sergiyprotsiv that there is data! I found this a while back, but never looked for address points!?. He mentioned each region may have their own datasets inside (potentially inconsistent fields). The original way I was downloading data always scrambled characters; hopefully the new way with pyesridump grabs all the correct information.
parent rest: http://infopublic.pravstat.kz:8399/arcgis/rest/services/Kazakhstan/MapServer
Адреса - addresses

There is a countrywide dataset there as well in addition to those per-region ones: http://infopublic.pravstat.kz:8399/arcgis/rest/services/addr/MapServer/2
It seems identical to the regional ones outside of the two big cities, so I'll be using that. The only thing is that there is a join required with the street table and with the region table. The region is identical to "raion" = district (while the field  id_city actually means a province - a larger administrative unit).
It seems it should be possible to get the borders of smaller units (villages, etc), but I did not get to it yet. I see there are three options:

do the simple join and use "raion" as a "city" field (would assign some rural addresses to the capital of the raion)
do the simple join and keep "raion" as "district" and have no "city" field
do a spatial join agains municipal polygons (assuming they exist) - most processing, but also most convenient for eventual users (and due to joins being necessary the source will be script-based anyway)

Let me know what the main priorities are: do we assume that the users are able to make the join themselves and keep the processing to the minimum or make the smallest set of work before the source is more universally usable?

I vote skip the spatial join now - we can add it later after a source is found. And Pelias (one of the downstream consumers of this data) adds it's own locality info.
(So empty city column but data in the others.)

On Jan 19, 2017, at 05:03, sergiyprotsiv<EMAIL_ADDRESS>wrote:
There is a countrywide dataset there as well in addition to those per-region ones: http://infopublic.pravstat.kz:8399/arcgis/rest/services/addr/MapServer/2
It seems identical to the regional ones outside of the two big cities, so I'll be using that. The only thing is that there is a join required with the street table and with the region table. The region is identical to "raion" = district (while the field id_city actually means a province - a larger administrative unit).
It seems it should be possible to get the borders of smaller units (villages, etc), but I did not get to it yet. I see there are three options:
do the simple join and use "raion" as a "city" field (would assign some rural addresses to the capital of the raion)
do the simple join and keep "raion" as "district" and have no "city" field
do a spatial join agains municipal polygons (assuming they exist) - most processing, but also most convenient for eventual users (and due to joins being necessary the source will be script-based anyway)
Let me know what the main priorities are: do we assume that the users are able to make the join themselves and keep the processing to the minimum or make the smallest set of work before the source is more universally usable?
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I think so as well. There’s an open discussion at https://github.com/openaddresses/machine/issues/283 about enhancements we might do to the downloadable files with post-processing.

Libpostal also adds its own admins, polygons if they're available and nearby city points for places that don't have city geometries in OSM (gets some of the more rural locations that are only represented as points).
Still, on my end, ingestion of larger data sets like OpenAddresses does go much more quickly if the city's included in the source. Does "raion" always refer to a real city that's the capital of the given district? If so, I think I'd prefer to keep it.
Will probably kick off OpenAddresses ingestion tomorrow night for the training batch that will go into our 1.0 release, so it would be great to get as many of the new data sources PR'd as possible in that time. Happy to help.


Percolator "Picked protein FDR (-f)" option requires flanking amino acids in PIN peptide column

see more at: https://github.com/percolator/percolator/wiki/Protein-inference-example

I think you should already be able to do this with the percolator option in the configuration file:
{
    "$schema": "https://raw.githubusercontent.com/compomics/ms2rescore/master/ms2rescore/package_data/config_schema.json",
    "percolator": {
        "f": "path/to/sequence_db.fasta"
    }
}

or
{
    "$schema": "https://raw.githubusercontent.com/compomics/ms2rescore/master/ms2rescore/package_data/config_schema.json",
    "percolator": {
        "picked-protein": "path/to/sequence_db.fasta"
    }
}

See https://github.com/compomics/ms2rescore/blob/master/configuration.md.
Let me know if it works!

It creates the following error message for the first PSM listed.
Exception caught: ERROR: Reading tab file, the peptide sequence  with PSM id  does not contain one or two of its flanking amino acids.

FYI: If "tandem2pin" is used (not the one in rescore), the flanking amino acids do appear in the Peptide column ([flanking AA].[Peptide].[flanking AA]).

Implemented for X!Tandem in #28

Hi, has a solution for this been implemented in the MSGF+ms2rescore pipeline? I'm getting the exact error mentioned by tivdnbos above.


chore(main): release 2.0.4

:robot: I have created a release beep boop
2.0.4 (2024-11-20)
Dependencies

update dependencies (#122) (290a2ef)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aimeerivers/watchthis-user-service/releases/tag/v2.0.4 :sunflower:


[BUG] EventProcessorClient fails to recover after AMQP link loss

Describe the bug
After around 17 hours of successful consumption, our EventProcessorClient suddenly closed its AMQP link and tried to request another one. That was unsuccessful, but I can't tell why. I would have expected the EventProcessorClient to recover and start consuming messages again.
Exception or Stack Trace
There was no exception at all. You can follow the events in this log file: 2021-08-27_amqpSessionClosed.log
To Reproduce

Create an Event Hub with one partition
Configure an EventProcessorClient that consumes from this Event Hub:

Batch size: 500
Max wait time: 5 seconds
Load balancing strategy: BALANCED
Offset reset position: earliest
Blob checkpoint store
Use the default retry settings

Produce many messages (we successfully received ~1.5 million before the AMQP link was closed) and wait for several hours :wink:

Code Snippet
I hope you don't mind that this is in Kotlin:
val client = EventProcessorClientBuilder()
		.credential(eventHubHostname, eventHubName, getSharedAccessKeyCredential())
		.transportType(transportType)
		.proxyOptions(getProxyOptions())
		.customEndpointAddress(customEndpointAddress)
		.clientOptions(getClientOptions())
		.initialPartitionEventPosition(initialPartitionEventPositions())
		.consumerGroup(consumerGroup)
		.loadBalancingStrategy(loadBalancingStrategy)
		.loadBalancingUpdateInterval(loadBalancingUpdateInterval)
		.partitionOwnershipExpirationInterval(partitionOwnershipExpirationInterval)
		.checkpointStore(BlobCheckpointStore(blobCheckpointStore.createAsyncClient()))
		.processPartitionInitialization {
			logger.debug { "Initialized consumption of ${it.partitionContext}" }
		}
		.processPartitionClose {
			logger.info { "Closed consumption of ${it.partitionContext}" }
		}
		.processError {
			logger.warn(it.throwable) { "Failed to consume from ${it.partitionContext}'" }
		}
		.processEventBatch(
			batchConsumer,
			properties.maxBatchSize,
			properties.maxWaitTime
		)
		.trackLastEnqueuedEventProperties(true)
		.buildEventProcessorClient()

client.start()

This doesn't directly cause the problem. The code is functional from as far as I can tell, but it caused the problem that I described above.
Expected behavior
I'd expect consumption to recover from connection problems without manual intervention.
Screenshots

In the screenshot you can see in the time range between 6 am and 12 pm that the outgoing messages suddenly drop to zero. They go up again once the service had been restarted and the processor received the messages that accumulated over several hours on the event hub, so the number of outgoing messages is much higher than before.
The short period at the beginning is where we had another consumer running. This is not related to the issue.
Setup (please complete the following information):

OS: Linux (we're running on Azure Kubernetes Services)
IDE: IntelliJ
Library/Libraries: e.g. com.azure:azure-messaging-eventhubs:5.9.0, com.azure:azure-messaging-eventhubs-checkpointstore-blob:1.8.1 and all their transitive dependencies
Java version: 11 (official AdoptOpenJDK Docker image)
App Server/Environment: Tomcat embedded (via Spring Boot)
Frameworks: Spring Boot

Additional context
There's no more context I can give.
Information Checklist

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

@pvorb Thanks for sharing the log:
namespace[our-event-hubs-namespace.servicebus.windows.net] entityPath[ingest]: Next AMQP channel received, updating 0 current subscribers

The above line explains that a new connection is created after the previous one is disposed, but there seems to be no downstream to use that connection (0 subscribers).  Are there any more log entries from the SDK after this point? If so, could you share?

@anuchandy Thanks for having a look.
This is the full log that was still available at the time before we restarted the service: 2021-08-27_amqpSessionClosed_full.log.gz
The line you mentioned is at line 41413, so there's also a lot of logs before the issue occurred. I hope you can make some sense from it. Shortly after the mentioned line, the Reactor seems to receive a shutdown signal and after that point, only the PartitionBasedLoadBalancer keeps running every 10 seconds. :shrug:

Fixed in #24141.


Unix Domain Socket for gRPC outputs

Motivation

Feature
Right now gRPC outputs has support for a bind_address in the grpc: block of the config.
# gRPC server configuration.
# The gRPC server is secure by default (mutual TLS) so you need to generate certificates and update their paths here.
# By default the gRPC server is off.
# You can configure the address to bind and expose it.
# By modifying the threadiness configuration you can fine-tune the number of threads (and context) it will use.
grpc:
  enabled: true
  bind_address: "/var/run/falco.sock"
  threadiness: 8
  private_key: "/etc/falco/certs/server.key"
  cert_chain: "/etc/falco/certs/server.crt"
  root_certs: "/etc/falco/certs/ca.crt"

# gRPC output service.
# By default it is off.
# By enabling this all the output events will be kept in memory until you read them with a gRPC client.
grpc_output:
  enabled: true

I would like to expose the Falco output stream over a Unix Domain Socket (file) instead of over a bind address and port (network).
Can we add support for Unix Domain Sockets for outputs?

Alternatives

Additional context
This will help with out Kubernetes integration such that a Kubernetes pod can always mount the socket read-only and consume falco alerts without having to route to the host network.

Duplicated of https://github.com/falcosecurity/falco/issues/858


Improve crayon UI to not be stuck in 1996

About the PR
Crayon
Makes the crayon UI not feel like a placeholder. Adds categories to it all that you don't have to filter out letters from various lines. Changes the search input to automatically select whatever decal with ID matching the search exactly. And then makes the search double up as a queue where the individual searches are separated by comma. And it will automatically advance when you draw with the crayon (if you draw a symbol that doesn't match searched ID, it will not advance though).
Why / Balance
Crayon
The current UI is bad, and the only reason it's remotely usable is the search box. Everything is mixed around without regard to any sort of categorisation.
Media

Crayon

Technical details
Added Linq to the Xaml, ignored a deprecation warning for the sprite getting thing, abused sorting. Crayon.
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Changelog
:cl:

add: Crayon UI now has categories and queue

Crayon
https://github.com/user-attachments/assets/e0ed3e19-eaff-4682-8ab2-dc049b0ec501

Okay it's very funny to say crayon but I really would prefer if you followed PR guidelines and explained the features your PR is implementing. Based on the videos it seems like it's some cool stuff!
Does the new UI play well with the rainbow crayon?

 Based on the videos it seems like it's some cool stuff!
Does the new UI play well with the rainbow crayon?

Fixed! And yeah, it works just fine, it's pretty much the same ol' UI, just touched up a lot.


Contribution to Wiki - adding xm510 install instructions

Hi, unfortunatelly github does not support pull requests for Wiki artikles. I had some time and tested my xm510 camera with the firmware. I tried to follow the intrsuctions for xm530/550 boards with some modifications and documented the results in the md files.
Attached to this bug to wiki files:

install_xm510.md - installation instructons for xm510 board
_Sidebar.md - modified to include xm510 insatllation link
_Sidebar.md
install_xm510.md

Thanks

Good afternoon, sir!
Thank you so much for your work.
Your information has been added to the Wiki.
Yes, we are considering alternative options for maintaining documentation, working with the current Wiki collectively is damn inconvenient..

We plan to support the entire line in the near future. We already have boards based on T10, T20, T21, T31...
I hope this will also be interesting to you.

Thanks for the Wiki update! I will definitely try the Ingenic version too
For the issues, experience i have with xm510, should i open separate bugs? Or should i dump all findings into one report?

Do as you like, it is better of course to break errors and edits into small parts.


Remove unnecessary Bookkeeper format init container from Helm chart

Fixes #3771

Had to rebase to include the latest fix from master that resolved the Github unsecure API build fix.


Move to axum and tracing

Small refactor to use axum as a web framework and tracing for logging. In the future, it will make it easier to integrate with tokio ecosystem. Also, axum has native support for websockets.
Also removed Cargo.lock from tracked files, there is no need to track it at this point

Great work!!!


Add support for a failover recording bucket

What does this pull request do? Explain your changes. (required)

I am enabling support for a failover/secondary recordings bucket to allow us to switch to a new storage location. We can set up the new bucket as the secondary, switch the recording output over to that bucket, then in this code whenever we can't find recordings at the old location we can fetch from the secondary (new) location instead.
Specific updates (required)

How did you test each of these updates (required)

Does this pull request close any open issues?

Screenshots (optional)

Checklist

[ ] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

@victorges let me know what you think, thanks!


use tracked actor in more cases

because of need to calculate actor system actor count

Codecov Report

Merging #23 into master will decrease coverage by 0.83%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
- Coverage   83.91%   83.08%   -0.84%
==========================================
  Files          17       17
  Lines         398      396       -2
  Branches       44       43       -1
==========================================
- Hits          334      329       -5
- Misses         64       67       +3

Impacted Files
Coverage Δ

...la/akka/monitor/instrumentation/ActorMonitor.scala
86.11% <66.66%> (-1.73%)
:arrow_down:

...ala/com/workday/prometheus/akka/ActorMetrics.scala
83.33% <0%> (-16.67%)
:arrow_down:

Continue to review full report at Codecov.

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


[BUG] Can not have the same display name in 2 flows

Describe the bug
An author can not have 2 flows with the same display name. This might be confusing/restricting for users for 2 reasons:

even archived flows are taken into account and you can not completely purge them at the moment.
it is inconsistent behavior to how runs behave, as it is allowed to have 2 runs with the same name

How To Reproduce the bug
Run 2 times:
pfazure flow create --flow flow.dag.yaml --set display_name=CUSTOM_NAME type=chat -w $AZURE_AI_PROJECT_NAME -g $AZURE_RESOURCE_GROUP
The second time fails with
Create flow failed with FlowOperationError: Remote flow folder 'CUSTOM_NAME' already exists under 'Users/819a3fe1-c31a-4c1c-881f-94918943b2f3/promptflow'. Please change the flow folder name and try again.
NB: even if you edit the name in the UI of the first flow and try again it results in the same error. It seems like the UI does not change the underlying folder name of where the flow is stored, only the title -> see bug #1611
Expected behavior
Running the command multiple times is allowed as this is the display name,not the name/id.
At least the behavior between flows and runs should be consistent or if anything flows being "drafts" should be more relaxed in how they are named.
Screenshots

Running Information(please complete the following information):

Promptflow: promptflow                       1.1.0
Operating System
$ uname -a
Linux c206cc41a3f4 6.5.11-linuxkit # 1 SMP PREEMPT_DYNAMIC Mon Dec  4 10:03:25 UTC 2023 x86_64 GNU/Linux

$ lsb_release -a
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux 12 (bookworm)
Release:        12
Codename:       bookworm
FROM mcr.microsoft.com/devcontainers/python:3.11

Python Version: Python 3.11.6

Additional context
In contact about this with @0mza987
Original feature request was to be able to update a flow: #1459

Hi @spygi , I think the main difference between UI and CLI is we provides keep editing experience in UI but not in CLI. We'd like to know your user scenario on flow authoring experience. Could you update here so we can better understand and provide support for it?

Hi @D-W- and thanks for picking this up.
 Workaround suggested was to archive the old flow and create another one with the same name. This bug is about this workaround.
I added you also to a chat where we have discussed more about this use-case.

Hi @spygi ,
When creating a flow to azure workspace, we created a flow folder in workspace file share with the display name, updating flow display name or archiving the flow won't resolve this error because that file share folder name does not get impacted.
We are planning to add a new parameter remote-path for pfazure flow create command to allow users to specify the file share flow folder name created in workspace instead of using the flow display name.

because that file share folder name does not get impacted.

Yes exactly. My initial thought was that folder name needs to be updated too but probably we don't want to use a user-generated name there.
Instead of

We are planning to set file share folder name to be "local flow folder name + timestamp"

wouldn't it be simpler to create the folder name with the flow id/name instead? This is unique and server generated. The user can still create multiple flows with the same display name. Additionally the folder name does not need to change if the user changes the display name.

Hi @spygi,

My initial thought for the fix was that the folder name needs to be updated too when you edit the display name - but maybe we don't want to use a user-generated name for the folder?

This is a azure portal related feature ask and I believe you have already created a feedback.

wouldn't it be simpler to create the folder name with the flow id/name instead?

Using flow id/name which is a uuid will make users unable to track the file share flow folder, while local folder name + timestamp could make it somewhat meaningful. Even if users change the display-name server side only needs to update the meta data of a flow and there should be no file copy operations at all.

This is a azure portal related feature ask and I believe you have already created a feedback.

Indeed, renaming the flow was just a workaround I was given, it's not related to the original issue. We can track the fix for the workaround there.
Coming back to the original issue here, it is CLI related and the question is: why runs can have the same display name but flows not? Why is the behavior different there?

I thought this question is already answered by my previous reply, but I could clarify it again:
It's a bug because we used the display name to create a file share folder to upload the flow content, and as a fix, we will use local flow folder name + timestamp as the remote file share folder name and then you can create multiple flows with the same display-name.

I was missing the acknowledgement that it is actually a bug - thank you


Cinder: add volume metadata/properties as labels (i.e. cluster)

We have multiple clusters using the same OpenStack tenant, and as such they share volumes. Trying to use alerts such as CinderDiskWithoutPV then gives false-negatives, as it thinks reports disks created by PV/PVCs from another cluster as matching its alert query:
kos_cinder_volume_status{name=~"(pvc-.+|kubernetes-dynamic-pvc.+)",pv_name=""} == 1
Would adding volume properties help? Such as CSI Cinder's CLUSTER_NAME setting, added support for in:

https://github.com/kubernetes/cloud-provider-openstack/pull/1809
https://github.com/kubernetes/cloud-provider-openstack/pull/1741

Here's an example of a volume with a "cluster" attribute set (by Cinder CSI):
$ openstack volume show -f json bc7c0b38-72b2-481e-bb9c-a0a41a38ad65  | jq '.properties'
{
  "cinder.csi.openstack.org/cluster": "kubernetes"
}

And here's another example with more properties being set:
$ openstack volume show -f json pvc-3e542918-a4cb-4268-bdc1-375fb39d6c71
{
  "attachments": [
    {
      "id": "11534b61-352e-41fe-a53c-49cd5c634375",
      "attachment_id": "3f665eee-94bb-4596-a141-2ef26f68abb3",
      "volume_id": "11534b61-352e-41fe-a53c-49cd5c634375",
      "server_id": "da4e0e7a-fa62-4b8b-b5b8-eaf4571cdef3",
      "host_name": "h56.az1.obfuscated",
      "device": "/dev/vdd",
      "attached_at": "2022-05-02T14:26:38.000000"
    }
  ],
  "availability_zone": "kvm-linux",
  "bootable": "false",
  "consistencygroup_id": null,
  "created_at": "2022-04-29T15:18:38.000000",
  "description": "Created by OpenStack Cinder CSI driver",
  "encrypted": false,
  "id": "11534b61-352e-41fe-a53c-49cd5c634375",
  "multiattach": false,
  "name": "pvc-3e542918-a4cb-4268-bdc1-375fb39d6c71",
  "os-vol-tenant-attr:tenant_id": "obfuscated",
  "properties": {
    "cinder.csi.openstack.org/cluster": "kubernetes",
    "csi.storage.k8s.io/pv/name": "pvc-3e542918-a4cb-4268-bdc1-375fb39d6c71",
    "csi.storage.k8s.io/pvc/name": "prometheus-monitoring-prometheus-oper-prometheus-db-prometheus-monitoring-prometheus-oper-prometheus-0",
    "csi.storage.k8s.io/pvc/namespace": "obfuscated"
  },
  "replication_status": "disabled",
  "size": 20,
  "snapshot_id": null,
  "source_volid": null,
  "status": "in-use",
  "type": "__DEFAULT__",
  "updated_at": "2022-05-02T14:26:56.000000",
  "user_id": "obfuscated"
}

I suspect this would be the place to add it?
https://github.com/mercedes-benz/kosmoo/blob/72c46a7fe4a73939d0fbfddc3d2366497f44b872/pkg/metrics/cinder.go#L166
Would it be the Metadata property on the GopherCloud Volume result to add, perhaps?
https://github.com/gophercloud/gophercloud/blob/19ba3dc8b6df9f0c12aec667e77b4cee634d4245/openstack/blockstorage/v2/volumes/results.go#L64-L65

Related:

https://github.com/mercedes-benz/kosmoo/issues/106


Remove jruby and rbx builds

They've been failing for a while and in the allowed_failures. The jruby build takes significantly longer than the others, so this just removes them entirely.

👍 :shipit:


Charade word set license

Hi there,
I know that the code is Apache-licensed, but I wasn't sure whether the word set is also under that license or not.
Is it, or is there a different license that applies to it?

Hey! Everything is Apache-licensed, feel free to use it anywhere you wish if it looks helpful to you :)

Great, thanks for the information!


tf.range miss some dtypes support

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
v1.12.1-113444-ga98a073af6f 2.18.0-dev20240722
Custom code
No
OS platform and distribution
Google Colab
Mobile device
No response
Python version
Google Colab default
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
It is a common use case to make ranges for dtypes like uint8 (when working with images) and float32 -> float16 (inside keras layers with mixed precision disabled/enabled).
In TF 2.17 release tf.range can't work with these dtypes.
One more issue is that XLA/non-XLA dtypes support is different (more dtypes supported with XLA)
Standalone code to reproduce the issue
import tensorflow as tf

tf.range(10., delta=1., dtype='float16')

Relevant log output
NotFoundError: Could not find device for node: {{node Range}} = Range[Tidx=DT_HALF]
All kernels registered for op Range:
  device='XLA_CPU_JIT'; Tidx in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_INT16, DT_INT8, DT_INT64, DT_BFLOAT16, DT_UINT16, DT_HALF, DT_UINT32]
  device='XLA_GPU_JIT'; Tidx in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_INT16, DT_INT8, DT_INT64, DT_BFLOAT16, DT_UINT16, DT_HALF, DT_UINT32]
  device='DEFAULT'; Tidx in [DT_INT32]
  device='GPU'; Tidx in [DT_INT64]
  device='GPU'; Tidx in [DT_DOUBLE]
  device='GPU'; Tidx in [DT_FLOAT]
  device='CPU'; Tidx in [DT_INT64]
  device='CPU'; Tidx in [DT_INT32]
  device='CPU'; Tidx in [DT_DOUBLE]
  device='CPU'; Tidx in [DT_FLOAT]
 [Op:Range] name:

I tried to run your code on Colab using TF v2.17.0, nightly and faced the same issue. Please find the gist here for reference.

Thank you!


Search results in fair microsite not clickable

https://www.artsy.net/frieze-london-2016
Search 'Marina', click Marina Abramovic...nothing. I think you should be routed to https://www.artsy.net/frieze-london-2016/browse/artist/marina-abramovic-1

This specifically happens in Chrome, e.g. works fine in Safari.

This got fixed in #275.


StatsigFetcher._retry breaks timeout logic

Hi, I believe handling of abort signal is broken in general.
First of all StatsigFetcher._retry doesn't receive or pass that in any way.
But even if it did it would have to create new signal each time, and since only signal is passed to the fetcher one cannot reliably know the timeout value with which it was constructed.
Consider the following scenario of invoking StatsigServer.flush function
await Statsig.flush(100); // manually flush any unsent events

It takes timeout as parameter and an abort controller is created and signal is being passed further down. However if that request fails a retry request will be issued without any signal passed

Thanks for the report, we are taking a look

Fixed in https://github.com/statsig-io/node-js-server-sdk/releases/tag/5.25.1


[0.28.0-dlc][lmi] Update lmi-dist to 10.0.1

Description
Patch release for lmi-dist

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

There is a potential blocker for DLC publish, we may need a new entry for the packages. @tosterberg @sindhuvahinis fo your call

Close the PR given we are going to make it 10.0.0


Cannot find package manager

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
16.18
Amplify CLI Version
11.0.0
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes
Describe the bug
Since the new version 11.0.0 i always receive this error when i try to deploy my amplify app:
No package manager found.
2023-03-15T16:04:25.437Z [INFO]: Resolution: Install npm or yarn to compile overrides for this project.
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
Even when i try to re-deploy an working version it still fails.
Expected behavior
I expect to deploy it fully.
Reproduction steps

Do a git push to trigger a deploy

Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

I found the issue, its the aws-amplify/cli version 11.0.0
I downgraded it to a version that worked, in my case 10.8.1
I edited my amplify.yml file:

build:
  commands:
    - npm uninstall @aws-amplify/cli
    - npm install<EMAIL_ADDRESS>    - amplifyPush --simple

After this, it deploys without any errors


docs: rate limits

/exchange/spot/rateLimits needs to be moved to /system/rateLimits ( also see #1 )
general information for rateLimits needs to be added on top

Rate limit endpoint + information should now be relevent / correct - I will split endpoints by route shortly!


Documentation Error (which can cause dev issues)

Describe the bug
Go to the documenation on this page:
https://platform.openai.com/docs/api-reference/chat/create?lang=node.js
Select the model and the language to be Node.
You will see in the instructions the following line:
  messages: [{"role": "system", "content": "You are a helpful assistant."}, {role: "user", content: "Hello world"}],

clearly in the first object, the property role is wrapped in "" as well as the property content. But not in the second object
To Reproduce
Go to the documentation
Code snippets
N/A

OS
Windows 11
Node version
Node 18.12.1
Library version
v3.3.0

I just noticed this is a duplicate - well done github... I close this to keep the other one open


arch/riscv: support CONFIG_CODE_DATA_RELOCATION

This implements support for relocating code to chosen memory regions via
the zephyr_code_relocate CMake function for RISC-V SoCs. Not all SoCs
are supported because it is implemented in arch/riscv/common/linker.ld,
which is not used by all SoCs: those which do not use that file are
currently unsupported.
Signed-off-by: Peter Marheine<EMAIL_ADDRESS>

Hi @tari, thanks for your contribution. Did you test these changes on any of the RISC-V boards?

Hi @tari, thanks for your contribution. Did you test these changes on any of the RISC-V boards?

Yes, I'm testing on it8xxx2.

We need also to make this test-able by tests/application_development/code_relocation/

The test (and gen_relocate_app) had a few baked-in ARM assumptions that I've had to fix, but I've added a test case that passes for RISC-V. I had to make it a new test case since adding a new section requires some unportable configuration.
I'm not very happy with the way target_relocation.cmake currently assumes section names based on the target architecture, so ideas there are welcome. The best alternative I see is changing semantics slightly to drop the data-in-default-RAM optimization so we can remove any references to particular memory regions from the script.


Auditing / reporting

From<EMAIL_ADDRESS>on January 18, 2011 16:34:30
Must be provided a functionality to generate reports about task executed/requested, administrators login and enduser login
Original issue: http://code.google.com/p/syncope/issues/detail?id=76

From<EMAIL_ADDRESS>on January 18, 2011 07:35:58
Owner: ---

From chicchiricco on April 11, 2011 03:47:56
Summary: Auditing / reporting
Labels: Milestone-Release-2011Q4

From chicchiricco on April 12, 2011 09:28:47
Labels: -Milestone-Release-2011Q4 Milestone-Release-2012Q1

From<EMAIL_ADDRESS>on January 18, 2012 01:58:19
Labels: -Milestone-Release-2012Q1 Release-Espressivo-1.0

From<EMAIL_ADDRESS>on February 02, 2012 03:15:53
Owner: chicchiricco

From chicchiricco on February 02, 2012 06:47:43
Status: Started
Labels: -Priority-Low Priority-Critical

From chicchiricco on February 02, 2012 07:06:07
Blockedon: 282

From chicchiricco on February 02, 2012 07:11:31
Blockedon: 283

From<EMAIL_ADDRESS>on February 03, 2012 02:36:33
(just noticed you guys on the incubator mailinglist, good luck with that !!)
Hi,
Short question: can you make the auditing pluggable ?
Imagine the following use case: an in-house developed identity manager exists that audits all its identity operations into an audit database. This audit database is quite generic so you'll be able to find out there things like 'when did user X receive role ABC for application 123'. Now if the organisation wants to move to syncope it would be great if it could integrate the syncope audit data with its already existing audit schema. For this i think a listener approach would suffice, implementers then get notified of all identity related events. Ofcourse this would need to be made granular a bit or the listener implementation will be swarmed with notifications. On thing to consider as always is performance, but you could assume it is up to the listener to return quickly and write the real audit record async for example.

From chicchiricco on February 03, 2012 03:13:41
This sounds very interesting and can be viewed as an extension to basic idea about implementation depicted in issue #282 .
Did you take a look at issue #282 ? Let's move further discussion there.

From chicchiricco on October 15, 2012 23:48:04
https://issues.apache.org/jira/browse/SYNCOPE-20 https://issues.apache.org/jira/browse/SYNCOPE-4
Status: Duplicate
Blockedon: -syncope:282 -syncope:283 syncope:282 syncope:283


Error after 1.1.0 installation

Hello,
I was using rose2 v1.0.1 until now, and it was working like a charm.
I tried to update to the last version (1.1.0) and now I cannot run rose2 anymore.
The installation goes fine, but then, when I try to run rose2, it fails.
Here is the error message:
Traceback (most recent call last):
  File "/bin/rose2", line 11, in <module>
    load_entry_point('rose2==1.1.0', 'console_scripts', 'rose2')()
  File "/usr/lib/python3.6/site-packages/pkg_resources/__init__.py", line 490, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2859, in load_entry_point
    return ep.load()
  File "/usr/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2450, in load
    return self.resolve()
  File "/usr/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2456, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/usr/lib/python3.6/site-packages/rose2-1.1.0-py3.6.egg/rose2/__init__.py", line 22, in <module>
    from . import genemapper, utils
  File "/usr/lib/python3.6/site-packages/rose2-1.1.0-py3.6.egg/rose2/genemapper.py", line 41, in <module>
    from string import join
ImportError: cannot import name 'join'

This was resolved.
It was a conflict between 2 installation on Python2.7 and Python3.6


Lookup for local image changes?

Hey guys,
what about looking for local image changes on the local server?
As I can see in your documentation you only support remote repositories.
Therefor I got this error message:
2019-03-19 14:00:01 : CRITICAL : dockerclient : Couldn't pull. Skipping. Error: 404 Client Error: Not Found ("pull access denied for nginx-test, repository does not exist or may require 'docker login'")

Yep. Only supports remote changes

No plans to change the current design.


Special character fine tune with pre trained model

Shape mismatch is happening if I try to add extra 2 characters.
Any solution for the same?
If i want to do fine tuning on TPS-ResNet-BiLSTM-Attn.pth
RuntimeError: Error(s) in loading state_dict for DataParallel:         size mismatch for module.Prediction.attention_cell.rnn.weight_ih: copying a param with shape torch.Size([1024, 294]) from checkpoint, the shape in current model is torch.Size([1024, 296]).         size mismatch for module.Prediction.generator.weight: copying a param with shape torch.Size([38, 256]) from checkpoint, the shape in current model is torch.Size([40, 256]).         size mismatch for module.Prediction.generator.bias: copying a param with shape torch.Size([38]) from checkpoint, the shape in current model is torch.Size([40]).

The only thing that worked for me is to train from scratch

I'm also interested in adding a few special characters but I cannot use TPS-ResNet-BiLSTM-Attn-case-sensitive.pth.
Has anyone been able to solve this?

I also get the same error when I want to do a test. Is there a solution for this error?

I'm also interested in adding a few special characters but I cannot use TPS-ResNet-BiLSTM-Attn-case-sensitive.pth. Has anyone been able to solve this?

are you find any solution?

Shape mismatch is happening if I try to add extra 2 characters. Any solution for the same? If i want to do fine tuning on TPS-ResNet-BiLSTM-Attn.pth
RuntimeError: Error(s) in loading state_dict for DataParallel: size mismatch for module.Prediction.attention_cell.rnn.weight_ih: copying a param with shape torch.Size([1024, 294]) from checkpoint, the shape in current model is torch.Size([1024, 296]). size mismatch for module.Prediction.generator.weight: copying a param with shape torch.Size([38, 256]) from checkpoint, the shape in current model is torch.Size([40, 256]). size mismatch for module.Prediction.generator.bias: copying a param with shape torch.Size([38]) from checkpoint, the shape in current model is torch.Size([40]).

Please help me if you find a solution for this issue?


a network bug on Android 8.1

Baresip register fail on Android 8.1. The sip server doesn't recevie the register request.
I debug code, find this code unright.
// It's in the file of re/src/dns/client.c.
static int query(struct dns_query **qp, struct dnsc *dnsc, uint8_t opcode,
		 const char *name, uint16_t type, uint16_t dnsclass,
		 const struct dnsrr *ans_rr, int proto,
		 const struct sa *srvv, const uint32_t *srvc,
		 bool aa, bool rd, dns_query_h *qh, void *arg)
{
	struct dns_query *q = NULL;
	struct dnshdr hdr;
	int err = 0;
	uint32_t i;

	if (!dnsc || !name || !srvv || !srvc || !(*srvc)) {
        return EINVAL;
    }

The function return EINVAL, because of !(*srvc) == true. It seems the dns parse error.

On Android 8.1, we can't get dns info from the env values likes net.dns1.
So I fix it like as follow:
diff --git a/src/dns/ns.c b/src/dns/ns.c
index aac713d..3d37b01 100644
--- a/src/dns/ns.c
+++ b/src/dns/ns.c
@@ -96,8 +96,16 @@ static int get_android_dns(struct sa *nsv, uint32_t *n)
                                ++count;
                }
        }
-       if (count == 0)
-               return ENOENT;
+//     if (count == 0)
+//             return ENOENT;
+
+       if (count == 0) {
+        sa_set_str(&nsv[0], "<IP_ADDRESS>", DNS_PORT);
+        sa_set_str(&nsv[1], "1<IP_ADDRESS>", DNS_PORT);
+        sa_set_str(&nsv[2], "<IP_ADDRESS>", DNS_PORT);
+
+        count = 3;
+       }

        *n = count;

This is a tentative plan, need a better one.

please take a look at this issue:
https://github.com/creytiv/re/issues/137
Fetching DNS servers is not supported on Android 8.
You need to add IP-address of name servers in the config:
dns_server        <IP_ADDRESS>:53
dns_server        <IP_ADDRESS>:53

Alfred

hi @cloud-jxy
did you try the suggestion in the previous post ? can you confirm if it is working for you ?
Alfred


coal and charcoal to black dye in order to fill the gap

Type
Shapeless
Ingredients for the recipe
charcoal or coal
Result
black dye
Crafting table recipe shape
No response
Additional info
flower villager will be added but the flower for black dye(whiter rose) wont be included so i feel like this would be a good fix
Agreements

[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] I have ensured the recipe I'm requesting isn't already in the server.

i can work on this if you want @Boy0000


 If the issue you raised is not a feature but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.
[X] 2. Please use English, otherwise it will be closed.

Motivation
Can logprobs return probabilities after sampling processing? Currently, the logprobs seem to be the probability distribution before processing such as temperature and top-p truncation.
Related resources
No response

I attempted to implement this feature, but I encountered an issue:
When the sampling backend is set to flashinfer, the function min_p_sampling_from_probs within flashinfer is called for sampling. However, sglang is unable to access the probs values processed by the min_p operation (i.e., the min_p-processed probs).
To obtain the processed probs values, it will be necessary to compute the min_p-processed probs values separately when using flashinfer as the sampling backend. This may be a bit challenging, as the implementation of flashinfer.min_p_sampling_from_probs appears to be non-trivial:
MinPSamplingFromProbKernel].

I attempted to implement this feature, but I encountered an issue:
When the sampling backend is set to flashinfer, the function min_p_sampling_from_probs within flashinfer is called for sampling. However, sglang is unable to access the probs values processed by the min_p operation (i.e., the min_p-processed probs).
To obtain the processed probs values, it will be necessary to compute the min_p-processed probs values separately when using flashinfer as the sampling backend. This may be a bit challenging, as the implementation of flashinfer.min_p_sampling_from_probs appears to be non-trivial: MinPSamplingFromProbKernel].

Yes, there are many dependent functions that need to be modified here. If it is not the code of sglang itself, it will be a bit troublesome. Another way is to add a parameter to control whether to obtain the probability distribution before or after sampling, and then assume that the probability distribution after obtaining is generally used for analysis. Without considering response speed, we can use the implementation of torch to run it again, but we also need to align various implementations.

@Snowdar
Hi, I have implemented this feature in my fork, which you can find at [https://github.com/Mayflyyh/sglang/tree/feature/2178-logprobs-post-sampling. However, I haven't had the chance to add unit tests or update the documentation yet, so I'm unable to submit a PR at the moment. But perhaps you could take a look in the meantime?

@Snowdar Hi, I have implemented this feature in my fork, which you can find at [https://github.com/Mayflyyh/sglang/tree/feature/2178-logprobs-post-sampling. However, I haven't had the chance to add unit tests or update the documentation yet, so I'm unable to submit a PR at the moment. But perhaps you could take a look in the meantime? example:
response = requests.post(
    "http://localhost:30000/generate",
    json={
        "text": "The capital of France is",
        "sampling_params": {
            "temperature": 0,
            # "max_new_tokens": 32,
        },
        "return_logprob": True,
        "return_sampled_prob": True
    },
)

Good jobs! Thanks for contribute it.
By the way, to avoid change too many files and functions so that there is no hidden bugs and compatibility with official codes (expecially when the output of a func is changed), I have implemented it carefully by this way:

The probs will be re-computed after sampling, but only the probs with temperature applied could be obtained (No top-p, top-k etc.)

Fixed by https://github.com/sgl-project/sglang/pull/2664


index.addObjects isn't working for large array objects

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
We've to index a large number of array objects as a batch to Algolia. index.addObjects fails when I tried to add 33k+ records.
What is the expected behavior?
The best way for this will be to provide a streaming support, but for that, you have to change both library and backend APIs, so here is a simpler and much more reliable solution that you can use instead of current for loop in your batch upload:
const Promise = require('bluebird');
const _ = require('lodash');
const data = require('./data.json');

const chunkSize = 500;
const maxConcurrency = parseFloat(process.argv[2] || "Infinity");

const chunks = _.map(data, (chunk, index) => {
  return index % chunkSize === 0 ? data.slice(index, index + chunkSize) : null;
})
.filter(chunk => {
  return chunk;
});

console.time(`Batch indexing!`);
Promise.resolve(chunks).map((chunk, index, length) => {
  console.log(`CHUNK | INDEX | LENGTH\n`);
  console.log(chunk, index, length);
  console.log(`\n================================\n`)
  return null;
}, { concurrency: maxConcurrency })
.then(() => {
  console.timeEnd(`Batch indexing!`);
})
.catch(e => {
  console.log(e);
});

It inserts in bulks using chunks of size 500 each with concurrency. I've tested this with 33K+ records on concurrency 1 and it took 1.4s to index all the records. You can change the concurrency or chunk size as per your server.

Perfect! Thanks! That's the way to go indeed, chuking and controlling concurrency. We could do this internally maybe at some point. Not yet planned.

@Dev-Dipesh fyi you have _.chunk directly implemented into lodash 👍

@iam4x yup totally missed that :sweat_smile:


Unable to use $remove: in a schema definition

Reproduction
https://formkit.link/9d995065b46650a6cfeca1dec11242e5
Describe the bug
Consider this custom input:
import {
  $extend,
  outer,
} from '@formkit/inputs'

export const custom = {
  schema: $extend(
    outer(),
    {
      attrs: {
        class: "$classes.outer + ' $remove:text-orange'"
      }
    }
  ),

  type: 'input',
}

The rendered HTML contains the $remove: part inside the class attribute :
<div class="formkit-outer bg-green text-orange $remove:text-orange" data-type="custom" data-empty="true">
</div>

See the playground for a full reproduction with config files etc.
Just a hint, I talked about this issue on Discord and @andrew-boyd concluded to a problem with the class parsing.
Environment
• OS: Windows (WSL)
• Browser Chrome
• Version 123

So thinking more on this I don't think it's actually a bug — when you modify the attr directly you're literally adding the string $remove:text-orange. There is no more parsing to be done. The parsed list is what is contained is classes.outer.
Alternatives:

Do your $remove: at the config level if possible
Do $remove via the component props which are part of the parsed pathway and end up becoming the classes.outer list. https://formkit.link/8126ad0dee4016979f9a62d0822701fb
Use a feature (a plugin that is not inherited by children) to make modifications to the node as part of its definition: https://formkit.link/cffe8a25420a66efc371c01bfaf1bb00

Got it, thank you for your answer!
I think it might be interesting to improve the DX here. Note that I use the conditional here, I'm still learning FormKit and it's totally possible there are very good reasons unknown to me why this is not possible.
First, the string concatenation is not very intuitive / cumbersome. "$classes.outer + ' other-class'" works, but your brain really wants to write  "$classes.outer other-class".
I was also sure it was possible to manipulate the $classes.outer in a way or another... and I tested a lot of things before asking on Discord (you know, the there must be a way to do it feeling).
What came to my mind at first glance when I was looking for a solution was:

Obviously, $classes.outer $remove:text-orange, but then the problem of "remove text-orange from where? $classes.outer ? What if there are multiple class pools concatened?
So maybe something like $classes.outer.replace('text-orange', ''), but I know, it's not javascript, it's explained in the doc (Although it looks very much like JavaScript — schema expressions are not JavaScript. They are better thought of as a templating language) :p It's tempting through.
Maybe $remove could be available in $fns? It's currently not, but since there are functions inside it, is there a way to implement something like $fns.remove($classes.outer, 'text-orange')?
Maybe it could be possible to access the $classes.outer output in JS, manipulate it and then use it in the attrs.class property. This point is interesting, I think accessing the final classes of a particular section could be awesome (using a helper function, a node function or anything else).

That was feedback as a newcomer writing his first inputs. For now I managed to progress without all that so it's obviously not mandatory :p
Have a nice day.


Missile damage range error

There is an error in the range when a missile misses.
Both kill range and damage range are confirmed as kills.
Even if you change the range, there is no change in damage depending on the distance.

I noticed this too - I'm just calling a game function which takes these two values to create the explosion. I don't think I can change this. I might merge the two config options as to not cause confusion.


Relax bundler dependency to make it easier to get started contributing

Hey @bleonard,
This PR fixes #8. Please let me know if it looks good.
Thanks!

Published v1.0.2 here: https://rubygems.org/gems/rails_stats
Thanks!


Add section to "Security" docs describing permissions

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
Expected results
Actual results
Steps to reproduce

Not that most of our permission are dynamically generated by FAB out of ModelView and view definition. If we can have all these objects have good docstrings, there's probably a programatic way to associate the docstrings with individual permissions.


worktree: allow manual ignore patterns when no .gitignore is available

I am using go-git as a library for a personal project and noticed hard-coded excluded patterns are ignored when no .gitignore file exists at the root of the repository.
This PR fixes this behavior and allows hard-coded ignore patterns even when no .gitignore file is present

Can you provide a test?

@mcuadros without the patch the test would fail when a .gitignore would not be present in the repository.


Chapter, Thread post: Preview feature

Because sanitization and markdown expansion is done server-side, will require some sort of POST interaction.

7d3ed05729c39d5707820c77c12b55fd3975bcce adds previews to Chapter Edit view

0a78f372e8b84d521501059f0c8face20aa98b21 adds previews to all chapter views


Moved Subscription Push Config Auth to separate authentication field

@rutul25 Can you please update the description of the PR with how the new createSubscription API would look like?

Codecov Report

Merging #254 (94239f3) into master (a494477) will decrease coverage by 2.67%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #254      +/-   ##
==========================================
- Coverage   45.37%   42.69%   -2.68%
==========================================
  Files         106      106
  Lines        4871     4869       -2
==========================================
- Hits         2210     2079     -131
- Misses       2496     2642     +146
+ Partials      165      148      -17

Flag
Coverage Δ

integration
?

unittests
42.69% <100.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/subscription/validation.go
83.20% <100.00%> (-0.26%)
:arrow_down:

pkg/messagebroker/factory.go
33.33% <0.00%> (-33.34%)
:arrow_down:

pkg/messagebroker/kafka.go
10.46% <0.00%> (-29.93%)
:arrow_down:

Continue to review full report at Codecov.

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


Feature suggestion : add isARM to Platform

Hi and thanks ! So as the title suggests, it would be great to have an easy way to differentiate between MacOS Intel platform and MacOS ARM platform.

Feature suggestion related to Dart core libraries ("dart:io")
Dart SDK version : 2.18.2 (stable) on "macos_arm64"
Using MacOS M1 Pro
Using Chrome

Defining a "platform" is a fractal problem. You can always split a "platform" into two by differentiating one some property that differs, and which might be important to someone.
We have generally shied away from specifying the CPU architecture, and just gone with operating system name.
Those are orthogonal. I'm sure we have Android on both Intel and ARM already, and now also MacOS.
It's still MacOS, at least in name.
We try to keep the number of platforms as limited as possible.
(The other end of the spectrum would be where you can distinguish every version number of each operation system, the OS language, and the precise mold of the CPU. That will allow perfect tailoring to the environment, but also be completely impossible to keep up-to-date.)

The question of what architecture we're running is also pretty ambiguous. For example, on an ARM64 Mac, we could be running a Dart VM under Rosetta that is generating Dart code for RV64, so that ARM64, X64, and RV64 are all legitimate answers to "the current architecture" depending on what one is trying to do.

Indeed the platform question is complicated and other systems solve it better. Does dart:fii offer the needed functionality? Alternatively on macOS, you can use dart:io to run e.g. the uname -m program to find the machine architecture.

@lrhn @rmacnak-google @sortie hi everybody and thanks for being so reactive :pray: 🙂

Defining a "platform" is a fractal problem. You can always split a "platform" into two by differentiating one some property that differs, and which might be important to someone.

The question of what architecture we're running is also pretty ambiguous. For example, on an ARM64 Mac, we could be running a Dart VM under Rosetta that is generating Dart code for RV64, so that ARM64, X64, and RV64 are all legitimate answers to "the current architecture" depending on what one is trying to do.

Ah Indeed. Then it's better we, users, handle it ourselves on a case-by-case basis.

Alternatively on macOS, you can use dart:io to run e.g. the uname -m program to find the machine architecture.

@sortie I didn't think about it, thanks for the suggestion !
Issue initially aroused when loading a dynamic library, strangely enough some of us (MacOS users only) need to use DynamicLibrary.open(dylibPath) instead of DynamicLibrary.executable() (see DynamicLibrary). But ultimately it's not blocking since we chose to let our users "opt-onto" Silicon by specifying an additional CLI arg when running commands, so it's not that of a big deal (details in comment).

You can do the differentiation that you want using the dart:ffi Abi class.
Is that sufficient?

At first sight it seems like it does perfectly @brianquinlan, thanks so much !
Let's just hold on for the outcome of the related PR on flutter_rust_bridge to close this issue, mostly because I don't own all the different devices to check on my own: I'm gonna wait for CI builds and help from fellow contributors who do to make sure everything is fine first.

@brianquinlan we couldn't test for each and every platform (mostly because we don't own the different devices ourselves), but I'm confident that dart:ffi Abi class is indeed the solution to our issue.
I've used it to differentiate between MacOS Intel and MacOS M1 (so strictly speaking Abi.macosX64 from Abi.macosArm64) and it works like a charm !
Thanks again for steppin' in, I think this issue can be closed :)


expenses page makes duplicate queries for every single expense

Describe the bug
The expenses page for any collective makes n number of queries (network calls) for n number of expenses i.e. it makes a separate query for each expense. Even when the query - allExpenses returns with the information required for all the expenses the page makes recursive queries Expense for each expenses it encounters.
To Reproduce
Steps to reproduce the behaviour:

Go to 'https://opencollective.com/babel/expenses'
Check the network panel in dev tools
You shall see a huge number of queries being made.
But if you inspect the data/response for the first query, all the data required to render these expenses are already available with it.

Expected behavior
It should simply not make all these duplicate (which fetches the same data) queries
Screenshots

Tryin' to get paid twice, amiright!

@flickz this might be useful since you are looking into this issue.

@piamancini  this is not related to https://github.com/opencollective/opencollective/issues/2520
From my understanding, we need to refetch the data because some fields (privateMessage for example) are only accessible when you're logged in, if you have access to them.
However you're right that having one fetch per item is an issue. We need to migrate that code to refetch only the main query (getExpensesQuery) and only if the user is logged in. An example for a similar behavior can be found in https://github.com/opencollective/opencollective-frontend/blob/fef6cb3e6a03747d324580da4f6ca479981c1e6a/components/collective-page/sections/Transactions.js#L118
@piamancini are we going to revamp this page in the expenses flow project? If so, I wouldn't prioritize this issue (it's an optimization).

hello, I was not aware of the complexity category level -> unknown, what is the meaning of this category? who could give a hint on the complexity of this task/issue?
Thank you

Hi @franciscotv, we didn't try to estimate to complexity (thus it's unknown) because it's also a wontfix, the page mentioned here has since been revamped and the new version is not affected: https://opencollective.com/webpack/expenses/
There's still one path suffering from this issue, bet we also plan to revamp it in the next months and don't want to invest in optimizations for it.

thanks @Betree for the clear explanation, I see the revamped took place. So, I wonder if this should continue to be consider as a valid effort/task for open collective bounty hunters like i am myself, this said, i have to say i do not see the whole picture of this effort as you do. This issue/task is still listed in the bounties page and i wonder if i should tackle it or not. Please let me know your thinking.

This one is not a bounty.
As specified in the doc you linked, only issues with the bounty label are eligible for bounties. You should go to this page to find bounties: https://github.com/opencollective/opencollective/issues?utf8=✓&q=is%3Aissue+is%3Aopen+label%3Abounty

This is resolved in the new expense(s) pages. Only the host dashboard Expenses tab is still affected, but it's about to be replaced by the new Expenses (beta) tab, which is not.


MINOR: Consolidate Topic create calls in Test classes

Replace adminZkClient.createOrUpdateTopicPartitionAssignmentPathInZK calls with TestUtils.createTopic wherever applicable
Replace adminZkClient.createTopic calls with TestUtils.createTopic wherever applicable
Deprecate AdminUtils.scala related tests
Remove duplicate tests between AdminTest and AdminZkClientTest

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ijuma @lindong28 Can you please review the PR whenever you get a chance.

One question: do we actually want to use TestUtils.createTopic everywhere? It waits for the metadata to propagate and some tests may not want that. I didn't check each test individually yet, but thought I'd ask.

@ijuma  Thanks for the review.   In my analysis, I found that for below tests either we need to create topic assignment before server start (or) We wont wait for the metadata to propagation. For other tests we can safely redirect calls to  TestUtils.createTopic.  Let me know If any concerns.
https://github.com/apache/kafka/blob/trunk/core/src/test/scala/integration/kafka/server/ReplicaFetcherThreadFatalErrorTest.scala#L57
https://github.com/apache/kafka/blob/trunk/core/src/test/scala/unit/kafka/admin/AdminTest.scala#L306
https://github.com/apache/kafka/blob/trunk/core/src/test/scala/unit/kafka/admin/AdminTest.scala#L347

@lindong28  @ijuma addressed the review comments.  Pls take a look .

Thanks for the patch @omkreddy. LGTM and merge to trunk. We can have follow up PR if there is further question from Isamel.


Proposal for a new Zowe landscape diagram

As discussed in ZLC meeting 2/12, let's collaborate on updating the Zowe landscape diagram with the new sub-projects like Mobile App and ZDK (Python, NodeJS).
zowe_landscape_update.pptx

I have added Sean for his review of the slide - also trying to add Peter Fandel but checking with him on whether he has OMP/Zowe github id.

'off-host' is probably not the term you want but I was thinking something like this
zowe_offhost_clientSDK.pptx

Thanks for the review!
On the "off-host" term - I am not sure if that's needed in addition to "client". In fact, I don't think there's anything stopping someone from building a python (thanks to Rocket..) or node app using this SDK that runs on z/OS - sorry if I miscommunicated this on the ZLC call.
The "Client" part of SDK just shows that this SDK is a client of server side REST APIs, but the client could be an app running on z/OS or off-platform.

zowe_landscape_update 01.pptx
@solsu01 - I've added a 02 chart to the deck with what I think the landscape chart could show - it's not polished but I wanted to

capture more about the extensibility of Zowe, where the conformance program relates to each part of the landscape
capture more of existing runtimes that Zowe works with and also conformant runtimes
show more about the value of the APIML in terms of SSO, gateway, ...
I'll try to put more polish on the slide but heads down trying to get Zowe 1.9 out the door the next couple of days !   Many thanks for taking the initiative on getting this slide updated.

Updates from ZLC call 2/19
Cartier is ok to use this ppt at SHARE:
zowe_clientSDK.pptx
For the longer term updated landscape diagram, we'll continue to iterate and refine this ppt:
zowe_landscape_update.02.pptx

Looks good to me (who didn't know the knot of snakes logo).....thanks for
the edits
Bruce Armstrong
IBM System Z Offering Manager- zowe.org
4205 S MIAMI BLVD, DURHAM NC 27703-9141
Email<EMAIL_ADDRESS>Tel:<PHONE_NUMBER>
Cell:<PHONE_NUMBER>
From:   Sujay Solomon<EMAIL_ADDRESS>To:     zowe/zlc<EMAIL_ADDRESS>Cc:     Bruce Armstrong<EMAIL_ADDRESS>Assign
<EMAIL_ADDRESS>Date:   02/19/2020 08:13 AM
Subject:        [EXTERNAL] Re: [zowe/zlc] Proposal for a new Zowe
landscape diagram (#160)
Updates from ZLC call 2/19
Cartier is ok to use this ppt at SHARE:
zowe_clientSDK.pptx
For the longer term updated landscape diagram, we'll continue to iterate
and refine this ppt:
zowe_landscape_update.02.pptx
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.

I have had a lot of requests from squad leads, users and the community for an updated landscape diagram, so I've taken another stab at it with the intent of keeping it simple, but covering all the high level active sub-projects and incubation sub-projects:
zowe_landscape_update.03.pptx
It would be helpful to settle on this as I'm noticing folks starting to create their own versions since there isn't a standard one to use.

This looks good. I was playing around with the format to try and clarify client/user components from server components, but there's not much real estate to shuffle things around. I made an edit with the some arrows modifications - not sure if this clarifies the diagram or just makes it more busy.
zowe_landscape_update.04.pptx

Hi, I would like to ask for your permission if I could use the updated Zowe landscape on the upcoming webcast that I am going to present. I will briefly discuss about Zowe components and I would like the Zowe Mobile App to be part of it.
@MarkAckert @solsu01 I am confused with the arrows on update#04. Is the arrow pointing to APIML means that I am calling the APIML APIs? And the App Framework doesn't do that?

@jellypuno - The API Mediation Layer is an edge server that is able to collate all of the z/OS services used by clients, so that there is one certificate and one endpoint port that needs to be surfaced to clients.  APIML also offers the ability to generate a one time use token for authentication that means it can support multi factor authentication (MFA) and for the credential challenge it only occurs once giving the user a single sign-on (SSO) experience.  The App Framework (aka web desktop) can be run on its own if a user wishes to, but by default with Zowe both the App Framework and APIML are started, and in this case the App Framework sees the APIML is running and will work behind it as shown in the diagram.  This means all of its function works through the APIML port,  and SSO occurs between its web desktop and the APIML catalog.  You're right that the Zowe Mobile App doesn't work via the APIML, I believe you just call out to the z/OSMF endpoints directly, but going forward having the mobile app work with the APIML is something that would be good to validate with users as a way going forward so that if you wanted to access other services behind the APIML (such as CICS, DB2, CA Endeavor, ...) then you could do so benefitting from SSO/MFA, single certificate, ...   Maybe it'd be good to explain that mobile is an incubator project and currently doesn't go via APIML, but as part of maturing to be a Zowe core app that's something being looked at to ensure there is a consistent experience across the suite.

@Joe-Winchester Thank you for the explanation. I appreciate it 😊

Zowe slide from webinar .pptx
I guess first comment is that I thought we were going to try to use the slide for the upcoming OMP webinar as the slide for what is Zowe? This was out growth of the ZLC working sessions (both words and pictures). Maybe this slide has not gotten wide enough review for us to settle on it.
My second comment is maybe we should just admit that we are unlikely to decide on just one picture for Zowe. Comparing the OMP webinar slide to the 04 version above it looks like one provides more detail that a technical audience may care about. We should at least be consistent in whether Zowe components are at the top of the page or bottom. (I do like OMP slide rendering that shows plugs both out of the box (in blue) and custom extensions (in green)).
Maybe we can take a few mins in the ZLC call today to discuss.....maybe we have a couple versions in a central location to be used depending on the audience and how comfortable the presenter is with the technical detail. I recall an attempt to have a github repo of slides at one time.

@jellypuno - The API Mediation Layer is an edge server that is able to collate all of the z/OS services used by clients, so that there is one certificate and one endpoint port that needs to be surfaced to clients. APIML also offers the ability to generate a one time use token for authentication that means it can support multi factor authentication (MFA) and for the credential challenge it only occurs once giving the user a single sign-on (SSO) experience. The App Framework (aka web desktop) can be run on its own if a user wishes to, but by default with Zowe both the App Framework and APIML are started, and in this case the App Framework sees the APIML is running and will work behind it as shown in the diagram. This means all of its function works through the APIML port, and SSO occurs between its web desktop and the APIML catalog. You're right that the Zowe Mobile App doesn't work via the APIML, I believe you just call out to the z/OSMF endpoints directly, but going forward having the mobile app work with the APIML is something that would be good to validate with users as a way going forward so that if you wanted to access other services behind the APIML (such as CICS, DB2, CA Endeavor, ...) then you could do so benefitting from SSO/MFA, single certificate, ... Maybe it'd be good to explain that mobile is an incubator project and currently doesn't go via APIML, but as part of maturing to be a Zowe core app that's something being looked at to ensure there is a consistent experience across the suite.

I believe the Mobile app does go through the API ML.

As discussed on ZLC 6/3, I'm working on a "current" and "future" version of these slides by repurposing @PeterFandelAtRocket 's one from the OMP webinar (for current) and my previous rev in this issue (for future).

Please review this version of "today" and "future" versions with inspiration from the webinar slide that Bruce shared:
zowe_landscape_update_05.pptx
If we are good with this, I'd like to place this in the zowe community github for others to use.

@solsu01 with "release" of this ppt in the PI planning do we want to consider this one closed?

Yes, good with me to close.


Unable to edit skeleton SVG

My actions before raising this issue
https://opencv.github.io/cvat/docs/manual/advanced/annotation-with-skeletons/editing-the-skeleton/

I am trying to edit the skeleton from the svg, when click to add one more connection link by clicking on a button called "Click two points to setup an edge" to SVG and then clicking to continue it gives error, and all of the projects and tasks deleted from the organization.
Steps to Reproduce (for bugs)

create a project and upload an SVG file for the skeleton and save that project.
Then, delete the skeleton from that project.
afterward upload SVG again and try to add a new connection between keypoints in the skeleton by clicking on a button called "Click two points to setup an edge".
Click on "continue," and an error message will appear. After that, all the projects in the entire organization will be deleted. (I am attaching screenshots of the error message and LOGs).

Expected Behaviour
it should allow us to add links to the SVG of the skeleton, Atleast, it should not delete all the projects from the whole organization.
Current Behaviour
it is deleting all of the projects and task and giving me the error of " could not fetch projects" when I click on projects and task tabs.

<EMAIL_ADDRESS>

Here is the required information
SVG File:

CVAT server logs:
cvat_server_logs.txt
Git commit hash
commit fb4af9ca09ffbf294fdb2b87b0b23a184894f1eb (HEAD -> release-2.3.0, origin/release-2.3.0)
Merge: 7e53d673 d699f1fa
Author: Nikita Manovich<EMAIL_ADDRESS>Date:   Thu Dec 22 17:54:39 2022 +0200
Merge branch 'master' into release-2.3.0

Here is the required information
SVG File:
https://drive.google.com/drive/folders/1E-yUfKuhyuT8meLxOVKXMmtskrH_d2yE?usp=sharing
CVAT server logs:
cvat_server_logs.txt
Git commit hash
commit fb4af9ca09ffbf294fdb2b87b0b23a184894f1eb (HEAD -> release-2.3.0, origin/release-2.3.0)
Merge: 7e53d673 d699f1fa
Author: Nikita Manovich<EMAIL_ADDRESS>Date:   Thu Dec 22 17:54:39 2022 +0200
Merge branch 'master' into release-2.3.0

@bsekachev I wanted to follow up on the issue I raised earlier and let you know that I have provided all the required information. I am waiting for your response and would appreciate it if you could take a look at the issue.
thank you.

Hi @iqraRajpoot
I was not able to reproduce exactly the error you reported.
But I can confirm that the skeleton file you provided is not correct a little bit. It can not be added to a project.
If this file was provided by CVAT, it is a bug.
To fix that, you need to remove "id" properties from <desc data-description-type="labels-specification"> </desc> value.
In this example I have:
<desc data-description-type="labels-specification">{"1":{"name":"1","attributes":[],"color":"#d12345","type":"points"},"2":{"name":"2","attributes":[],"color":"#350dea","type":"points"},"3":{"name":"3","attributes":[],"color":"#479ffe","type":"points"},"4":{"name":"4","attributes":[],"color":"#4a649f","type":"points"},"5":{"name":"5","attributes":[],"color":"#478144","type":"points"},"6":{"name":"6","attributes":[],"color":"#57236b","type":"points"},"7":{"name":"7","attributes":[],"color":"#1cdda5","type":"points"},"8":{"name":"8","attributes":[],"color":"#e2bc6e","type":"points"},"9":{"name":"9","attributes":[],"color":"#f067db","type":"points"},"10":{"name":"10","attributes":[],"color":"#63bbfa","type":"points"},"11":{"name":"11","attributes":[],"color":"#22b16f","type":"points"},"12":{"name":"12","attributes":[],"color":"#daddec","type":"points"},"13":{"name":"13","attributes":[],"color":"#2ac791","type":"points"},"14":{"name":"14","attributes":[],"color":"#de22a0","type":"points"},"15":{"name":"15","attributes":[],"color":"#a7a570","type":"points"},"16":{"name":"16","attributes":[],"color":"#74db1b","type":"points"},"17":{"name":"17","attributes":[],"color":"#aaa246","type":"points"},"18":{"name":"18","attributes":[],"color":"#bfa552","type":"points"},"19":{"name":"19","attributes":[],"color":"#9b8d37","type":"points"},"20":{"name":"20","attributes":[],"color":"#c8023c","type":"points"},"21":{"name":"21","attributes":[],"color":"#c0762a","type":"points"},"22":{"name":"22","attributes":[],"color":"#76bb18","type":"points"},"23":{"name":"23","attributes":[],"color":"#afc7d8","type":"points"},"24":{"name":"24","attributes":[],"color":"#3b1ebf","type":"points"},"25":{"name":"25","attributes":[],"color":"#3d4e6a","type":"points"},"26":{"name":"26","attributes":[],"color":"#b295e8","type":"points"},"27":{"name":"27","attributes":[],"color":"#a8eeed","type":"points"},"28":{"name":"28","attributes":[],"color":"#1cb413","type":"points"},"29":{"name":"29","attributes":[],"color":"#b19fbd","type":"points"},"30":{"name":"30","attributes":[],"color":"#5cc934","type":"points"},"31":{"name":"31","attributes":[],"color":"#179232","type":"points"},"32":{"name":"32","attributes":[],"color":"#5d93c8","type":"points"},"33":{"name":"33","attributes":[],"color":"#0b9b66","type":"points"},"34":{"name":"34","attributes":[],"color":"#1004d4","type":"points"}}</desc>

Feature "Click two points to setup an edge" works as expected for me, there are not any errors.

@yasakova-anastasia Could you please look at the issue cvat.apps.engine.models.Label.skeleton.RelatedObjectDoesNotExist: Label has no skeleton. in provided server logs?

This issue (cvat.apps.engine.models.Label.skeleton.RelatedObjectDoesNotExist: Label has no skeleton) has been fixed by #5700.
We can't add a skeleton with sample svg to an existing project or task. I created an issue on this topic (#5706).

@bsekachev this Svg file was generated by CVAT and then downloaded as an Svg.

Regarding the wrong SVG file downloaded, please refer to #5706. We will fix it ASAP.


Multiple file support

This PR would provide initial support for multiple files (detailed in a millet.toml file in the workspace root).
Some parts are probably less than ideal:

If there are issues with the configuration file this isn't obviously communicated
Currently recomputes lexing/parsing/statics for all files on change instead of just files that depend on the one modified
Probably more stuff I'm forgetting

nifty. if you want you can merge this yourself, i added you as a contributor. and also you can feel free to push to master yourself in the future/approve your own prs.
i have limited bandwidth to work on this project now, but i appreciate your contributions!

nifty. if you want you can merge this yourself, i added you as a contributor. and also you can feel free to push to master yourself in the future/approve your own prs.
i have limited bandwidth to work on this project now, but i appreciate your contributions!

I noticed! I haven't gotten around to merging this yet because I noticed an issue with clearing previous diagnostics after testing a bit more (I don't recall the exact circumstances to trigger, but it was pretty common with two files) and haven't had much time to work on this until recently.
With adding multifile support though, I got the impression it would be a pretty useful to try and get more than one error out of the parser/lexer after playing around with some example projects so I think I'll probably end up trying to get something finalised there before this sees any more movement.

I noticed! I haven't gotten around to merging this yet because I noticed an issue with clearing previous diagnostics after testing a bit more (I don't recall the exact circumstances to trigger, but it was pretty common with two files) and haven't had much time to work on this until recently.
With adding multifile support though, I got the impression it would be a pretty useful to try and get more than one error out of the parser/lexer after playing around with some example projects so I think I'll probably end up trying to get something finalised there before this sees any more movement.

i saw you were working on using rowan for the parser, in hindsight i probably should have tried to do that originally since it has good support for working with partial parses which are common in ides. cf rust-analyzer.

i saw you were working on using rowan for the parser, in hindsight i probably should have tried to do that originally since it has good support for working with partial parses which are common in ides. cf rust-analyzer.


Add new release and promote workflow

Using official actions directly.

Re-trigger failed CI test again.


Clean up dynamodb module testing

Motivation
There's a lot of duplication between the unit and integration specs in this module, which makes maintaining the code tough.
Scope of change

Review all the unit and integration specs in dynamoDb. Where there are duplicate tests, keep the integration spec version
If there are database calls that do not have integration specs, write them!
Note: it's ok if some unit tests remain where they are testing logic confined to a specific method.

Recommending we delete dynamodb all together.  This should be closed

This is obsoleted by #971


How to whitelist same-origin requests?

Since same-origin requests don't send the Origin header you have to rely on the X-Requested-With header, but this is not enough when using the originWhitelist option. I was hoping to be able to use originWhitelist to allow all same-origin requests, but also whitelist a set of extra external origins, but setting this options completely blocks same-origin requests.
I'm able to do this by setting originWhitelist: ",http://localhost", for example. But this feels like a hack since the first "empty" value in that list just happens to cause equality when the Origin header is missing due to https://github.com/Rob--W/cors-anywhere/blob/02f0cbd9291eca55953406acc7f7f08c64d13454/lib/cors-anywhere.js#L361. Is there a reason that same-origin requests are denied when originWhitelist is set?
To only allow same-origin requests I'd likewise have to set originWhitelist: ",", which seems even weirder. Am I missing something? How would I lock it down to only same-origin requests?

Side note, originWhitelist should be an array of origins, not a string.
To lock it down to same-origin requests, you'd have to only allow the empty origin (e.g. [""]). To make sure that the request is really same-origin (and not a non-cors cross-origin request), you'd have to require a dummy header that can only exist if approved via preflight reuest (e.g. the X-Requested-With header).

I neglected to mention that I was actually using the environment variables, so I can't set it to empty because that counts as the variable being unset, but I guess a single comma is okay in that case anyways!


Hey need help

Hi @thisisanshgupta like would be possible to do the reverse of this??

Yes..it's possible or you can also fine-tune a T5 model for this task.


Unhandled OperationCanceledException causing TaskScheduler.UnobservedTaskException to trigger

Seems the same as #150 which was closed for some reason.
It is quite normal for OperationCanceledException to be thrown from a controller or middleware for various scenarios. Frame.RequestProcessingAsync doesn't catch/log/swallow so it gets logged as an unhandled exception:
13 Error Connection id "0HL3KRAJRPQMK": An unhandled exception was thrown by the application.
System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Net.Http.HttpClient.<FinishSendAsync>d__58.MoveNext()
at Intrinsic.Messaging.Http.XHttpClient.<QueryAsync>d__10.MoveNext() in C:\Repository\Projects\criticalarc\criticalarc\Source\Libraries\Intrinsic.Messaging.Core\Messaging\Http\XHttpClient.cs:line 237
at Intrinsic.Messaging.Channels.IXQueryableExtensions.<QueryAsync>d__0.MoveNext() in C:\Repository\Projects\criticalarc\criticalarc\Source\Libraries\Intrinsic.Messaging.Core\Messaging\Channels\IXQueryableExtensions.cs:line 25
at Intrinsic.Messaging.Protocols.AdhocCommands.XAdhocCommandExecutor.<ExecuteAsyncCore>d__33.MoveNext() in C:\Repository\Projects\criticalarc\criticalarc\Source\Libraries\Intrinsic.Messaging.Core\Messaging\Protocols\AdhocCommands\XAdhocCommandExecutor.cs:line 103
at CriticalArc.Applications.Command.Areas.Command.Controllers.AdminUsersController.<RetrieveRegistrationForm>d__8.MoveNext() in C:\Repository\Projects\criticalarc\criticalarc\Source\Libraries\CriticalArc.Applications.Command\Areas\Command\Controllers\AdminUsersController.cs:line 125
at CriticalArc.Applications.Command.Areas.Command.Controllers.AdminUsersController.<RetrieveRegistrationFormForUser>d__9.MoveNext() in C:\Repository\Projects\criticalarc\criticalarc\Source\Libraries\CriticalArc.Applications.Command\Areas\Command\Controllers\AdminUsersController.cs:line 135
at CriticalArc.Applications.Command.Areas.Command.Controllers.AdminUsersController.<RetrieveUser>d__10.MoveNext() in C:\Repository\Projects\criticalarc\criticalarc\Source\Libraries\CriticalArc.Applications.Command\Areas\Command\Controllers\AdminUsersController.cs:line 161
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeActionMethodAsync>d__27.MoveNext()
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeNextActionFilterAsync>d__25.MoveNext()
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ActionExecutedContext context)
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeNextExceptionFilterAsync>d__24.MoveNext()
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ExceptionContext context)
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeNextResourceFilter>d__22.MoveNext()
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ResourceExecutedContext context)
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeAsync>d__20.MoveNext()
at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()
at Microsoft.Owin.Mapping.MapMiddleware.<Invoke>d__0.MoveNext()
at Microsoft.AspNetCore.Owin.WebSocketAcceptAdapter.<>c_DisplayClass6_0.<<AdaptWebSockets>b_0>d.MoveNext()
at CriticalArc.Applications.Command.Authentication.CookiesAbsoluteUrlFixMiddleware.<Invoke>d__4.MoveNext() in C:\Repository\Projects\criticalarc\criticalarc\Source\Libraries\CriticalArc.Applications.Command\Authentication\CookiesAbsoluteUrlFixMiddleware.cs:line 35
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
at Microsoft.AspNetCore.Hosting.Internal.RequestServicesContainerMiddleware.<Invoke>d__3.MoveNext()
at Microsoft.AspNetCore.Server.Kestrel.Internal.Http.Frame`1.<RequestProcessingAsync>d__2.MoveNext()

Triggering the unhandled exception handler for OperationCanceledExceptions is undesirable.

OperationCanceledExceptions shouldn't be treated specially, they are unhandled exceptions and should be logged as such. Why aren't you catching exceptions if you don't want them logged?

They are special exceptions, since they are the base class of TaskCanceledException and can result from Task cancellations of various kinds (TPL may throw it, user may throw TaskCanceledException, may await on a TaskCancellationSource after SetCanceled() is called.
I'm not catching it because I don't want to have to try/catch that specific exception in hundreds of controller actions when it's benign, since TaskCanceledException and OperationCanceledException are only to be used in state-preserving co-operative cancellation scenarios.

The framework can't assume any exception thrown from user code is benign as it doesn't know what the user code is doing or why it threw the exception.
If you want to eat them all; which probablly not a good idea, you can just add some middleware
public class ExecptionEatingMiddleware
{
    private readonly RequestDelegate _next;

    public ExecptionEatingMiddleware(RequestDelegate next)
    {
        _next = next;
    }

    public async Task Invoke(HttpContext httpContext)
    {
        try
        {
            await _next(httpContext);
        }
        catch(TaskCanceledException)
        { }
        catch (OperationCanceledException)
        { }
    }
}

public static class ExecptionEatingMiddlewareExtensions
{
    public static IApplicationBuilder UseExecptionEatingMiddleware(this IApplicationBuilder builder)
    {
        return builder.UseMiddleware<ExecptionEatingMiddleware>();
    }
}

Then
app.UseExecptionEatingMiddleware();

They're not special. Write an MVC exception filter, or do what @benaadams suggested and eat those exceptions as well as any other special exceptions in middleware.

@benaadams Thanks that's a good idea, I'll do that.


Docs request — Can render a component assigned to a variable?

question / docs request
The scenario: I'm making a single page app, and I get a child component client side, from a 'router' module.
class MyApp extends Tonic {
  render () {
    const child = router.match('/example')
    return this.html`<div>
      // how to render `child` here?
      <${child></${child}>
    </div>`
  }
}

How to make this work? Normally in this scenario you would just return html with the child component's class/tag name?
render () {
  return this.html`<my-child>example</my-child>`
}

You would want to either return a string (the tag name) from the route function, or this is how we convert between camel-case class names and tag names in tonic:
function getTagName (camelName) {
    return camelName.match(/[A-Z][a-z0-9]*/g).join('-').toLowerCase()
}

You can use that like this, given a tonic class myComponent:
return this.html`
  <${getTagName(myComponent.name)}>hello</${getTagName(myComponent.name)}>
`


feat(ci): Add a timeout on new-e2e-unit-tests

What does this PR do?
Add a timeout on new-e2e-unit-tests

Motivation
There was an increase on MQ duration because of #incident-30226. the new-e2e-unit-tests was stuck trying to get credentials.
With a timeout it will fail fast. Value set based on last month 99 percentile

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


feat(kernel): Normalize empty structs to undefined

Upon deserializing objects from a "native" front-end (Java, .NET, ...),
the kernel now interprets an empty value for a struct type (aka data
type) as undefined, since front-ends that perform keyword lifting such
as Python will not be able to distinguish between "no argument passed"
and "an empty argument passed".
Fixes #411
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Shall we get this merged?


chore(jest-resolve): improve types

Summary

Improves the types of jest-resolve mainly for #10177, and also cleaning up a few little nits around the place, including a nice little find in ModuleNotFoundError.
Test plan

Run the tests and see if anything breaks 🚂

Thanks!


Mac - Actions not show up (using brew)

Description
In Mac, I use brew to install Gitea, from this docs.
And I made changes in /usr/local/bin/custom/conf/app.ini to enable Actions. But nothing happened.
Runner Management still not shown up.
Am I doing something wrong? Or this feature is not release yet for Mac?
Thank you
Gitea Version
1.18.3
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Mac
How are you running Gitea?
I installed it using brew. And run it from terminal using gitea command
Database
MySQL

Actions are not in 1.18.x. They are available for testing in 1.19, which only has an rc out currently.
You could theoretically use gitea-head from the brew tap, but that follows the main branch so be aware.
You could either grab a 1.19 binary manually, install the head tap, or wait until 1.19 is fully released.


Monokai theme: parameters are not colored in Orange

So I'm about to bring this issue up https://github.com/Microsoft/vscode/issues/1504, coz it doesn't seem it was resolved. I'm a big fan of Monokai theme, but this keeps bugging me.
These lines of code don't seem to work for some reason
		{
			"name": "Function argument",
			"scope": "variable.parameter",
			"settings": {
				"fontStyle": "italic",
				"foreground": "#FD971F"
			}
		},

taken from https://github.com/Microsoft/vscode/blob/master/extensions/theme-monokai/themes/monokai-color-theme.json
So I have

Expected

Does it have anything to do with VS Code issuing tokens?
Many other monokai themes turn out to be broken too

VSCode Version:

code --version
1.18.1
929bacba01ef658b873545e26034d1a8067445e9

OS Version:
macOS High Sierra

That's what I see with the built-in TypeScript/JavaScript grammar and Monokai theme:

var f = (state = {}, {a,b,c}) => {

}

To find out which scope is created where and what rule is matched to it, use the Developer: Inspect TM Scopes command:

@aeschli thanks for the debug tip!
I had to turn off the extension https://github.com/dzannotti/vscode-babel/ to get proper tokens for the theme

However syntax highlighting breaks when function arguments are placed on individual lines

And it looks like there's no way to address this lonely argument

Object rest arguments can be addressed as meta.var.expr.js meta.object.member.js variable.other.readwrite.js tho

That has to do with the way the textmate grammar engine works. It only sees one line at a time, and once processed can not make adjustments later (e.g. when on the next line). So I speculate that until one sees the => you don't know its a function declaration.
The textmate grammar for TypeScript/JavaScript is at https://github.com/Microsoft/TypeScript-tmlanguage. You can make improvements suggestions there, but think of the Textmate limitations.


Number Guessing Game Pass Validation

Describe the Issue
The tests 1:1 7 through 13 is looking for me to print certain statements, which I am doing, but it does not pass.
Affected Page
https://www.freecodecamp.org/learn/relational-database/build-a-number-guessing-game-project/build-a-number-guessing-game
Your code

Expected behavior
I would expect the "Enter your username:" test to pass, but it's not even when it's the first line my program prints, reads a username, and queries against my tables with said username
Screenshots

System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
No response

number_guess.sh.txt
number_guess.sql.txt
I've attached my .sh and .txt files for this

Thank you for reporting this issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Ask for help" button on the challenge on freeCodeCamp, which will help you create a question in the right part of the forum. Volunteers on the forum usually respond to questions within a few hours and can help determine if there is an issue with your code or the challenge's tests.
If the forum members determine there is nothing wrong with your code, you can request this issue to be reopened.
Thank you and happy coding.


Drop HHVM as per HHVM plan changes

HHVM LTS 3.24 is the last version to support PHP 5 compatibility
third paragraph states "HHVM will not aim to target PHP7" and further down "We do not intend to be the runtime of choice for folks with pure PHP7 code." (just HACK)
https://hhvm.com/blog/2017/09/18/the-future-of-hhvm.html
add PHP 7.2

@ThaDafinser try triggering the Travis build this should work. the Travis error is related to an HTML connectivity issue and should be find now

@ThaDafinser closing as there is an authentication error I can't fix. Feel free to reopen if you solve the authentication issue.


onSendNPCJoin RPC argument is missng

onSendNPCJoin parameters must be the same as onSendClientJoin

No, it's wrong. Server doesn't expect any other parameters.


feat: support serveTls & serve with ServeTlsOptions

https://github.com/denoland/fresh/issues/901
https://github.com/denoland/fresh/issues/809

I need to use this function in my project now. When will it be merged?

Any update on this PR? Would love to see this merged :slightly_smiling_face:


Don't use window.location when not defined.

In server-side-rendering scenarios, window.location may be undefined, causing this code to crash. As this is only meant as an best-effort override, I think it makes sense to ensure this override only happens when window.location is actually present.

Dublicate: https://github.com/supasate/connected-react-router/pull/377
And it's wrong solution.

Closing as duplicated. Thanks for your contribution anyway @anuraaga !.


ARROW-16455: [CI][Packaging] Add linux-ppc64le to the list of platforms to clean on conda

This PR aims to fix the issue:
"[ERROR] ('Storage requirements exceeded<PHONE_NUMBER> bytes). Payment is required to add a file. Please go to https://anaconda.org/binstar.settings/billing to update your plan', 402)"

@github-actions crossbow submit conda-clean

It seems to be working based on the logs:
786 builds may be deleted out of 1241
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev101/linux-ppc64le/arrow-cpp-proc-8.0.0.dev101-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev102/linux-ppc64le/arrow-cpp-proc-8.0.0.dev102-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev103/linux-ppc64le/arrow-cpp-proc-8.0.0.dev103-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev124/linux-ppc64le/arrow-cpp-proc-8.0.0.dev124-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev134/linux-ppc64le/arrow-cpp-proc-8.0.0.dev134-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev139/linux-ppc64le/arrow-cpp-proc-8.0.0.dev139-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev143/linux-ppc64le/arrow-cpp-proc-8.0.0.dev143-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev152/linux-ppc64le/arrow-cpp-proc-8.0.0.dev152-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev165/linux-ppc64le/arrow-cpp-proc-8.0.0.dev165-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev172/linux-ppc64le/arrow-cpp-proc-8.0.0.dev172-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev181/linux-ppc64le/arrow-cpp-proc-8.0.0.dev181-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev182/linux-ppc64le/arrow-cpp-proc-8.0.0.dev182-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev185/linux-ppc64le/arrow-cpp-proc-8.0.0.dev185-cpu.tar.bz2
- arrow-nightlies/arrow-cpp-proc/8.0.0.dev186/linux-ppc64le/arrow-cpp-proc-8.0.0.dev186-cpu.tar.bz2
...

@pitrou is only this?

Note you can also test locally. In both cases nothing actually gets deleted, it only happens on git master (where the right credentials are available).

Let's merge so that we can fix the conda build uploads.