ValueError: Can not do batch_dot on inputs with shapes!

Hello, @XifengGuo  greetings. I was using your capsnet implementation for a multi-label classification where model output 3 in number for each label target. However, I've faced the following error.
Each three output has a different number of classes. For example, the labels are A, B, and C. Label A has 168 different class, whereas B has 11 and C has 7.  A basic set-up has given below:
  def CapsNet:

          ............... more code ......................

            # Layer 3: Capsule layer. Routing algorithm works here.
           digitcaps_0 = CapsuleLayer(num_capsule=168,
                                   dim_capsule=16,
                                   routings=routings,
                                   name='digitcaps_0')(primarycaps)

           digitcaps_1 = CapsuleLayer(num_capsule=11,
                                   dim_capsule=16,
                                   routings=routings,
                                   name='digitcaps_1')(primarycaps)

           digitcaps_2 = CapsuleLayer(num_capsule=7,
                                   dim_capsule=16,
                                   routings=routings,
                                   name='digitcaps_2')(primarycaps)

           # Layer 4
           out_caps_0 = Length(name='out_caps_gra')(digitcaps_0)
           out_caps_1 = Length(name='out_caps_vow')(digitcaps_1)
           out_caps_2 = Length(name='out_caps_cons')(digitcaps_2)

           # Models for training
           train_model = models.Model(inputs=[x], outputs=[out_caps_0,
                                                    out_caps_1,
                                                    out_caps_2])

           return train_model

# build the model
model = CapsNet(input_shape=[64, 64, 1], routings=3)

And it threw the following error. The logs
ValueError                                Traceback (most recent call last)
<ipython-input-23-9f7ff279bacc> in <module>
      1 import tensorflow as tf
      2
----> 3 model = CapsNet(input_shape=[64, 64, 1], routings=3)

<ipython-input-22-1735b9121899> in CapsNet(input_shape, routings)
    109                                    dim_capsule=16,
    110                                    routings=routings,
--> 111                                    name='digitcaps_gra')(primarycaps)
    112
    113     digitcaps_vow = CapsuleLayer(num_capsule=11,

c:\users\devlix\.conda\envs\tflite\lib\site-packages\keras\engine\base_layer.py in __call__(self, inputs, **kwargs)
    487             # Actually call the layer,
    488             # collecting output(s), mask(s), and shape(s).
--> 489             output = self.call(inputs, **kwargs)
    490             output_mask = self.compute_mask(inputs, previous_mask)
    491

<ipython-input-10-d94539d78fd5> in call(self, inputs, training)
    154                 # then matmal: [dim_capsule] x [input_num_capsule, dim_capsule]^T -> [input_num_capsule].
    155                 # b.shape=[batch_size, num_capsule, input_num_capsule]
--> 156                 b += K.batch_dot(outputs, inputs_hat, [2, 3])
    157         # End: Routing algorithm -----------------------------------------------------------------------#
    158

c:\users\devlix\.conda\envs\tflite\lib\site-packages\keras\backend\tensorflow_backend.py in batch_dot(x, y, axes)
   1497                          str(x_shape) + ' and ' + str(y_shape) +
   1498                          ' with axes=' + str(axes) + '. x.shape[%d] != '
-> 1499                          'y.shape[%d] (%d != %d).' % (axes[0], axes[1], d1, d2))
   1500
   1501     # backup ndims. Need them later.

ValueError: Can not do batch_dot on inputs with shapes (None, 168, 168, 18432, 16) and (None, 168, None, 18432, 16) with axes=[2, 3]. x.shape[2] != y.shape[3] (168 != 18432).

Any catch? TIA.

Downgrade Keras version to 2.2.4.


feat: added consumer.WaitForHandlers

I'd like to propose adding a WaitForHandlers method to the consumer which waits for any running handlers to complete processing their current messages. This is because Close doesn't wait for the handlers to finish or queue to be drained (this makes sense).
This solves a couple of issues I've been facing:

Some message handling may be performing actions that need to clean up properly: file/database IO, external requests, even when idempotent may cause corruptions if the handlers are interrupted without being allowed to shutdown safely.

Applying a circuit breaking pattern to stop a handler from consuming new messages if they are failing frequently due to overload or dependency failures.

The code in this PR could be added to the handler method itself, but I think it may be a useful pattern that others may take advantage of so I'd like to consider including it in this library.
Happy to discuss further :)

I like this idea. I actually think .Close should just always wait for handlers though

I like this idea. I actually think .Close should just always wait for handlers though

How to finish the handlers gracefully?

Hello @astromechza,
I've made some changes to support handlers completion in consumer.Close:

https://github.com/wagslane/go-rabbitmq/pull/166

I'm going to close this PR since #166 seems like a better approach :)


npm rebuild doesn't work out of the box

What's the reasoning on:
[grpc] Success: "/app/node_modules/grpc/src/node/extension_binary/grpc_node.node" already installed
Pass --update-binary to reinstall or --build-from-source to recompile

That's not how any other native npm module behaves, making this not work in typical scenarios where npm rebuild is useful, e.g. in Docker deployments.
I can workaround by passing npm rebuild --update-binary, but now everyone needs to know about this.

Basically, gRPC does that because that's how node-pre-gyp works. Unfortunately, I can't find a way to hook into rebuild without having the same hook for install. One solution would be to just always update the binary, but i have to think that the node-pre-gyp documentation doesn't suggest that for a reason.


Can not replicate consistants behavior of "promptfoo view" in self hosted app.

I tried to self host the app and noticed that with the default app memory option, the labels' modifications done in the ui did not remain, unless shared with the "Share" option which generates another link.
Also, when an evaluation is sent to the local backend, it disappears when the app is stopped and run again, which is not the case when shutting down the server raised by the "promptfoo view" command.
I believed I had to provide memory to my app and tried to set the "PROMPTFOO_SHARE_STORE_TYPE" to "filesystem" and give a dedicated folder.
It does not seem to have changed much, even though once i run the docker image i get a log stating "Using Filesystem store" and in both cases (memory and filesystem storage),when I share an evaluation, I can see a line "Storing eval result with id f:caf6b..." written in my CLI.
How can I add consistency to the evals I share to keep them when i shut down the app, and when they are modified in the UI so that I do not need to generate another link everytime ?

Filesystem storage type should solve this problem. Are you sure you are
mapping the docker volume to a persistent location on the host machine,
using the -v arg for docker run?
On Thu, Feb 29, 2024, 9:18 AM RayannBen @.***> wrote:

I tried to self host the app and noticed that with the default app memory
option, the labels' modifications done in the ui did not remain, unless
shared with the "Share" option which generates another link.
Also, when an evaluation is sent to the local backend, it disappears when
the app is stopped and run again, which is not the case when shutting down
the server raised by the "promptfoo view" command.
I believed I had to provide memory to my app and tried to set the
"PROMPTFOO_SHARE_STORE_TYPE" to "filesystem" and give a dedicated folder.
It does not seem to have changed much, even though once i run the docker
image i get a log stating "Using Filesystem store" and in both cases
(memory and filesystem storage),when I share an evaluation, I can see a
line "Storing eval result with id f:caf6b..." written in my CLI.
How can I add consistency to the evals I share to keep them when i shut
down the app, and when they are modified in the UI so that I do not need to
generate another link everytime ?
—
Reply to this email directly, view it on GitHub
https://github.com/promptfoo/promptfoo/issues/510, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACLYJQIS4IJ5QSWOMW2MTLYV5RFBAVCNFSM6AAAAABEAKXL4WVHI2DSMVQWIX3LMV43ASLTON2WKOZSGE3DCNZUGQYTEMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Add Teams notification integration

Adds the ability to send out simple Microsoft Teams notifications with ClusterHealthChecks using the goteamsnotify package.

Hi @gianlucam76
here is a bit more information.
The testing:
$> TAG="dev" REGISTRY="magnusrc" make create-cluster

$> k create secret generic teamswebhook --from-literal=TEAMS_WEBHOOK_URL="<here goes your URL>" \
	--type=addons.projectsveltos.io/cluster-profile

$> k apply -f - <<EOF
---
apiVersion: lib.projectsveltos.io/v1alpha1
kind: ClusterHealthCheck
metadata:
  name: testing
spec:
  clusterSelector: cluster.x-k8s.io/cluster-name=clusterapi-workload
  livenessChecks:
  - name: addons
    type: Addons
  notifications:
  - name: testing
    type: Teams
    notificationRef:
      apiVersion: v1
      kind: Secret
      name: teamswebhook
      namespace: default
EOF

This should yield the following output in the Teams channel that have configured the above webhook URL:

Currently these Messages are quite simple, so it would be cool in the furture if you could get a more parsable return from getNotificationMessage(), as Teams adaptive cards have a lot of features like formatting.

Thank you so much @MagnusRef
Please let me know if you have any suggestion/example for the Notification message. I can change it based on that.


Add next occurrence [⌘ D] unexpected behavior

VSCode Version: 1.43.0-insider
OS Version: mac)S 10.15.4

In my use case, I wish the ⌘D shortcut could let me quickly select bits of the same variables and then rename them synchronously. This method works fine in SublimeText and I think it should be the same in VsCode. Unfortunately, when I type the backspace key, I wish the result should delete one char at once or remove one word is fine. The result in VsCode is unexpected for me, the behavior is shown in GIFs below.
Steps to Reproduce:
Reproduce procedure in VsCode:

The same procedure in Sublime:

When I typed backspace it should have the same effects at all my selected words, right? are there any features I don't know about? or my settings' problem?

Does this issue occur when all extensions are disabled?: Yes

From your screen recording, I can see that you have vim enabled. So the following statement in the issue report is false:
Does this issue occur when all extensions are disabled?: Yes


Use the factory test repository

The tests was moved to http-interop/http-factory-tests.

:+1:

Thank you for merging

I should have tested this, it doesn't seem to work as expected.


Ionic 2 - Google Maps JS API works only one time after each refresh

I have a Ionic2 app, where I have the homepage that leads to several pages, In each of these pages I have embedded Google Maps JavaScript Api.
What's weird is that when I open a page where google map is embedded in for the first time, it shows the map!
However, If I go back and open another page that has also the map, it does not show it.
When I do refresh, and I go to the page where the map was not showing before, it shows it (only after refresh).
In short, the map works only one time after each refresh, once you pop from that view and you go back to it again, the map disappears!
Some code:
Index.html

Page.html

Page.js
    ngOnInit(){

            displayMap(){
      let latLng = new google.maps.LatLng(34.45, 54.61);

      let mapOptions = {
            center: latLng,
            zoom: 10,
            mapTypeId: google.maps.MapTypeId.ROADMAP
          }

      let icon = {
          url: "../../img/locationPin.png", // url
      };

      this.map = new google.maps.Map(document.getElementById("map_canvas"), mapOptions);
      this.marker = new google.maps.Marker({
         position: latLng,
         map: this.map,
         icon: icon
       });

    }

    }

Your system information:
Cordova CLI: 6.3.0
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0-beta.32
Ionic App Lib Version: 2.0.0-beta.18
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v4.4.7
Xcode version: Xcode 7.3.1 Build version 7D1014

Michael I am getting all your development conversations to my personal
mail. I have nothing to do with any of your conversations !!! I am not
working at ionic or have ever worked there !!! This is a violation of my
privacy ! Please do not send me any more mails or this will bubble up to
your managers !!!

@TomasKatz I really don't know why you are receiving that! Maybe it's a bug on Github!

Was this ever figured out?


Getting type error

I'm running kismon built from here against the 2019-12 release of kismet, but get the following error:

17:49:22,650 INFO     [networks.py:131] Total networks 4
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/kismon/core.py", line 326, in queues_handler_networks
self.queue_handler_networks(server_id)
File "/usr/local/lib/python3.7/dist-packages/kismon/core.py", line 297, in queue_handler_networks
self.networks.add_device_data(device, server_id)
File "/usr/local/lib/python3.7/dist-packages/kismon/networks.py", line 268, in add_device_data
new_ssid = ssid_map[key]['dot11.advertisedssid.ssid']
TypeError: list indices must be integers or slices, not dict
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/kismon/core.py", line 286, in queues_handler
self.queue_handler(server_id)
File "/usr/local/lib/python3.7/dist-packages/kismon/core.py", line 206, in queue_handler
gps = {'lat': data['kismet.common.location.lat'],
KeyError: 'kismet.common.location.lat'

10:36:49,540 ERROR    [networks.py:672] Parser: filename is not a file ()
10:36:49,540 INFO     [networks.py:71] saving 4 networks to /tmp/networks-1582904209.json
10:36:49,543 INFO     [networks.py:110] Loading networks.json
10:36:49,543 INFO     [networks.py:131] Total networks 4
10:36:49,543 ERROR    [networks.py:410] unknown filetype
10:36:49,543 INFO     [networks.py:71] saving 4 networks to /tmp/networks-1582904209.json
10:36:49,637 ERROR    [networks.py:262] todo: multiple SSIDs per device 88:B1:E1:22:C8:F2
/usr/lib/python3/dist-packages/apport/report.py:13: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
import fnmatch, glob, traceback, errno, sys, atexit, locale, imp
Traceback (most recent call last):
File "/home/matt/Code/SDR/Kismet/kismon/kismon/core.py", line 326, in queues_handler_networks
self.queue_handler_networks(server_id)
File "/home/matt/Code/SDR/Kismet/kismon/kismon/core.py", line 297, in queue_handler_networks
self.networks.add_device_data(device, server_id)
File "/home/matt/Code/SDR/Kismet/kismon/kismon/networks.py", line 268, in add_device_data
print(ssid_map[key])
TypeError: list indices must be integers or slices, not dict
10:36:49,752 DEBUG    [map.py:368] marker 11:22:33:44:55:66 not found

Thanks for reporting the issue, please test if it is gone after you pulled the latest commits.

Yes, the errors went away with the most recent commit, thanks.  I see, however, an odd error that I'm not sure what it means:
devices/last-time/-2/devices.itjson
devices/last-time/-2/devices.itjson
15:58:13,567 ERROR    [networks.py:262] todo: multiple SSIDs per device XX:XX:XX:22:C8:F0
15:58:13,567 ERROR    [networks.py:262] todo: multiple SSIDs per device XX:XX:XX:22:B0:30
15:58:13,568 ERROR    [networks.py:262] todo: multiple SSIDs per device XX:XX:XX:22:B0:32
15:58:13,568 ERROR    [networks.py:262] todo: multiple SSIDs per device XX:XX:XX:22:47:60
The office I'm testing from has multiple AP's in different rooms to the same network. Is that what this is?  If so, can I suppress it?   Perhaps a warning instead of an error?  Also, the last-time never increments from -2

The todo means that there is nothing implemented for the case that a device broadcasts more than one SSID by using the same MAC address. So you might not see the SSID that you expect in Kismon, there is no logic yet of how to process and display them. I'm not aware of an enterprise access point who does that, usually each one has a base MAC and a couple more for every SSID that it broadcasts.
-2 requests all devices that changed in the last two seconds from the kismet.


Serial Id is not created

This is an awesome project ! Thank you guys for your hard work !! I have some migration code that I can't figure out why it isn't working as expected:
The following code
 public class ApiManagementEmployee
{

    public ApiManagementEmployee()
    {

    }

    public int Id { get; set; }

    [Column(TypeName = "date")]

    public DateTime TAdded { get; set; }

    public string AspWebAdmin_UserName { get; set; }
    public string AspWebAdminUserGuidId { get; set; }
    public string AspMobileUser_UserName { get; set; }
    public string AspMobileUserGuidId { get; set; }

    .... removed for brevity ....

Produces this migration

  public partial class InitialMigration : Migration
    {
        protected override void Up(MigrationBuilder migrationBuilder)
        {
            migrationBuilder.CreateTable(
                name: "ApiManagementEmployees",
                columns: table => new
                {
                    Id = table.Column<int>(nullable: false)
                        .Annotation("Npgsql:ValueGenerationStrategy", "SERIAL"),
                    TAdded = table.Column<DateTime>(type: "date", nullable: false),
                    AspWebAdmin_UserName = table.Column<string>(nullable: true),
                    AspWebAdminUserGuidId = table.Column<string>(nullable: true),
                    AspMobileUser_UserName = table.Column<string>(nullable: true),
                    AspMobileUserGuidId = table.Column<string>(nullable: true),
                    FirstName = table.Column<string>(maxLength: 50, nullable: false),
                    LastName = table.Column<string>(maxLength: 50, nullable: false),
                    Phone = table.Column<string>(maxLength: 10, nullable: true),
                    Cell = table.Column<string>(maxLength: 10, nullable: false),
                    Email = table.Column<string>(maxLength: 80, nullable: false),
                    EmployeeNumber = table.Column<string>(maxLength: 50, nullable: false),
                    AccessPin = table.Column<string>(maxLength: 10, nullable: true),
                    HireDate = table.Column<DateTime>(type: "date", nullable: false),
                    SeparationDate = table.Column<DateTime>(type: "date", nullable: true),
                    Address = table.Column<string>(maxLength: 150, nullable: false),
                    City = table.Column<string>(maxLength: 50, nullable: false),
                    State = table.Column<string>(maxLength: 10, nullable: false),
                    Zip = table.Column<string>(maxLength: 10, nullable: false),
                    POBox = table.Column<string>(maxLength: 10, nullable: true),
                    Image = table.Column<byte[]>(nullable: true)
                },
                constraints: table =>
                {
                    table.PrimaryKey("PK_ApiManagementEmployees", x => x.Id);
                });

Which looks correct with the "SERIAL" keyword but this migration code produces this postgres table

-- DROP TABLE public."ApiManagementEmployees";

CREATE TABLE public."ApiManagementEmployees"
(
    "Id" integer NOT NULL,
    "TAdded" date NOT NULL,
    "AspWebAdmin_UserName" text COLLATE pg_catalog."default",
    "AspWebAdminUserGuidId" text COLLATE pg_catalog."default",
    "AspMobileUser_UserName" text COLLATE pg_catalog."default",
    "AspMobileUserGuidId" text COLLATE pg_catalog."default",
    "FirstName" character varying(50) COLLATE pg_catalog."default" NOT NULL,
    "LastName" character varying(50) COLLATE pg_catalog."default" NOT NULL,
    "Phone" character varying(10) COLLATE pg_catalog."default",
    "Cell" character varying(10) COLLATE pg_catalog."default" NOT NULL,
    "Email" character varying(80) COLLATE pg_catalog."default" NOT NULL,
    "EmployeeNumber" character varying(50) COLLATE pg_catalog."default" NOT NULL,
    "AccessPin" character varying(10) COLLATE pg_catalog."default",
    "HireDate" date NOT NULL,
    "SeparationDate" date,
    "Address" character varying(150) COLLATE pg_catalog."default" NOT NULL,
    "City" character varying(50) COLLATE pg_catalog."default" NOT NULL,
    "State" character varying(10) COLLATE pg_catalog."default" NOT NULL,
    "Zip" character varying(10) COLLATE pg_catalog."default" NOT NULL,
    "POBox" character varying(10) COLLATE pg_catalog."default",
    "Image" bytea,
    CONSTRAINT "PK_ApiManagementEmployees" PRIMARY KEY ("Id")
)
WITH (
    OIDS = FALSE
)
TABLESPACE pg_default;

ALTER TABLE public."ApiManagementEmployees"
    OWNER to greg;

Can you see what I'm doing wrong ? Is there any fluent config code I need to write to get the serial auto incrementing working ?
Thanks Greg

@foote-Greg first, can you confirm that the SQL you quoted above is a dump of your table definition rather than the SQL generated by the provider? We don't generate this kind of definition (e.g. OIDS = FALSE).
Assuming this is a dump, then the provider should definitely be creating a table with a SERIAL column (just checked again with a trivial example). I'd double-check that you're actually applying the EF Core migrations on the table etc.
BTW You should also consider switching to the newer PostgreSQL IDENTITY columns (see the docs on this).

Wow !  I'm not sure how that happened I guess another team member ran scripts to create the schema and I didn't realize that...  Dropped the db and let the provider apply the migration and it worked as expected also I added the new identity columns as per the link....
Thanks !!!

Thanks for confirming!


Modular Parameters feature

With this PR

we introduce a new keyword extends
using null is allowed

extends keyword make bicepparam file inherits from another bicepparam
using null is valid only on shared bicepparam files and it turns off the parameter validation process on the file
How does it work
Files:
// shared.bicepparam
using null
param location = 'westus'
param name = 'default_name'

// main.bicepparam
using './main.bicep'
extends 'shared.bicepparam'
param vm_sku = 'Standard_D4_v4'

Compiled output:
{
  "location": "westus",
  "name": "default_name",
  "vm_sku": "Standard_D4_v4"
}

Fixes #12019
Contributing a feature

[x] I have opened a new issue for the proposal, or commented on an existing one, and ensured that the Bicep maintainers are good with the design of the feature being implemented
[x] I have included "Fixes #{issue_number}" in the PR description, so GitHub can link to the issue and close it when the PR is merged
[x] I have appropriate test coverage of my new feature

Microsoft Reviewers: Open in CodeFlow

What happens when main.bicepparam contains duplicate params as the shared param file?

A couple of comments, some which should be addressed now and something in the future.
Understanding the override characteristics would be important -

does the "primary" value override an extend or does the extend override the primary?
Does a command line parameter override merged parameters?

Could we possibly get a decorator such as @MergeCharacteristcs with values:

NeverOverride - this parameter will never be overridden with either an extend or CLI param
Override - this parameter will be overridden by the extend (useful for arrays/objects)
MergeObject - union arrays/objects rather than overriding.  Allows us to define tags at a hierarchy level and build up tags via multiple extends

Can we support chained extends?
Given the tree:
C:.
+---dev
|   |   env.bicepparam
|   |
|   +---eastus
|   |       region.bicepparam
|   |
|   \---westus
|           region.bicepparam
|
\---prod
    |   env.bicepparam
    |
    +---eastus
    |       region.bicepparam
    |
    \---westus
            region.bicepparam

It would be great if our main.bicepparam would extend region.bicepparam and region extent env.bicepparm.  That would give us a nice changed hierarchy.
Finally, can the extend use paths that are dynamic?
Given the above hierarchy, when I deploy an object using a main.bicepparam, I'd like to have the main have the following
main.bicepparam
using './main.bicep'
extends '../${readEnvironmentVariable('env')}/${readEnvironmentVariable('region')}/region.bicepparam'
param vm_sku = 'Standard_D4_v4'

region.bicepparam
using null
extends '../${readEnvironmentVariable('env')}/env.bicepparam'
param region = 'eastus'

env.bicepparam
using null
param env = 'prod'

I find the proposed design very limited. As far as I understood from the meeting the following things are not possible:

Intelisense for shared bicep parameters files
multiple extends statements
ability to nest shared bicep parameters files
ability to merge (union) parameters of type object and array. Merge tags scenarios
support for variables

I think better approach would be to have something like:
// shared.bicepparam
using 'shared'
using 'main1.bicep'
using 'main2.bicep'

In this scenario you are clearly defining that this is shared bicep parameters file. Also additionally you provide two (or more if you want, these are optional statements) bicep files to serve as intelisense for the shared file. In this scenario when you define that this is shared bicep parameters files any errors related to required parameters are turned off. In VSC you can still see from intelisense if specific parameter is required or not. You can also see the description for those parameters. In case main1.bicep and main2.bicep contains the same parameter let's say location and that contains two different descriptions upon hovering the parameter when added you can see the description for both. In summary you still get intelisense and other information like description, allowed values, etc, but without the errors you would get in regular bicep parameters files.
With that said I would like to see support for the rest of the missing features as well:

multiple extends statements
ability to nest shared bicep parameters files
ability to merge (union) parameters of type object and array. Merge tags scenarios
support for variables

Would also be nice that if you can declare which parameters from extends statement you would like to add instead of adding everything. Similar to how it is in import statements.
With the support for variables would be good to see their value when you use them on other parameters when you hover over the variable.
It would really make sense if all these features are released at once otherwise it is best to be preview feature until every feature is added.

I dont like the idea of typing "using null" as it seems a little clunky. I would rather it be some other form of MS decorator or keyword such as "using sharedbicepparam" which can imply that the bicepparam file can be extended to other files.
One other point with this, if this gets implemented, could this lead to a future development whereby we can use the "extends" command direct from a main.bicep file rather than a bicepparam file. This could lead to the possibility of having one global variables file whereby individual main files could 'import' selected variables from the sharedbicepparam file when needed. This could also be through the form of a function, like loadJsonContext(), but maybe something like loadBicepparamFile() ?

This pr doesn't explain nor solve how we can decouple the bicepparam files between modules and main bicep files. The biggest problem right now is you need to define all the parameters in the main bicep files. I have an extended explanation here on why this is an issue and how i think this could be tackled: https://github.com/Azure/bicep/issues/12200
A main bicep file should be rather clean of everything. If we have a look at where Microsoft is heading with the bicep verified modules, i believe Microsoft needs an implementation where they can host modules with their own bicep param files as a blueprint. Only the last part is now solved with the extend, where you can override the definition of the bicep files.

I have watched the last meeting but did not see any of the feedback given before implemented. In fact it seems that everything was the same and the only small change is that using is having none for value instead of null. That is only cosmetic change and it is irrelevant for the functionality. Kind of disappointed due to that.

@polatengin : Please note that using parameters of type Object in shared.bicepparam file is currently throwing error in main param files. I think it is easy for you to reproduce


Document thread-safety of BigTable API

Akin to #803, increase documentation as to which APIs can be safely accessed in a thread-safe manner.
In particular, the use-case I am looking at is:

Create a single Table.
Call ReadRows a bunch of time
Spawn one thread per returned RowReader
In each thread, iterate over the RowReader it now has--one thread may begin to iterate over the RowReader before all the RowReaders have been created.

I might also want to, e.g., call ReadRows in separate threads. I don't have a need for it, but documenting whether an individual RowReader object can be accessed from multiple threads would also be nice.
Per @coryan in the #cloud-bigtable slack channel, it sounds like this is safe. However it is not reflected well in the documentation--e.g., https://googleapis.dev/cpp/google-cloud-bigtable/1.3.0/classgoogle_1_1cloud_1_1bigtable_1_1v1_1_1Table.html explicitly suggests that operating on a single Table in multiple threads is not safe, and that the correct method would be to copy-construct instances for each thread.

There is a 'Thread Safety' section here:
https://googleapis.dev/cpp/google-cloud-bigtable/1.3.0/classgoogle_1_1cloud_1_1bigtable_1_1v1_1_1Table.html#details
Does that address the question, or you think each member function should repeat the thread safety guarantees?

Yeah, that's what I was mentioning in the post. I guess the thing I'm curious about here specifically is that, since RowReader objects seem to use the same underlying DataClient, the above documentation does not make it clear that it is safe to access multiple RowReaders in parallel--if I were to follow it to a "t", then I'd make copies of the Table before each call to ReadRows. Documenting each method individually seems unnecessary--ReadRows is just a bit special because it continues to cause operations to be performed on the table even after the call to ReadRows. It's also the case that it'd be nice to know if any functions are allowed to safely be accessed from multiple threads on a single instance of a Table, although that is not my current use case.
Also, tbh, I had missed that section on my first reading of the documentation. I'll put that on me though.

Although thinking about it, I'm wondering if it might just be simpler to document all the methods individually. That feels like it'd be unnecessary, but that might be lower effort than having to field questions like "Would it be safe to call Table::table_name() on the same instance from multiple threads?". But that also makes more documentation that you'd have to maintain. So I'm largely agnostic on it.


Gracefully handle not having a package.json at git root

getDefaultRemote gracefully handles the case where the package.json file at git root doesn't include a repository field. This change allows it to gracefully handle the case where there is no package.json file at git root.

In our scenario, we don't have a package.json file at git root. Only a subset of the repo is managed by rush.


Use RHDA v4 endpoint

I think we should make the full URL configurable. It may be that we need to switch between the different endpoints via configuration, and it will also make it easier to handle future updates.

I think that is a good idea. I just thought we can keep changes as small as possible. So people are confident in merging and changing production. Many of us will not be around until the end of the year and keeping changes simple might be an option.

Making the URLs configurable means to change some code here, and also add the configuration needed in app-interface. 2 PRs. While it is a good idea, can we postpone this change until all of us are fully available? potentially until January :) .
This PR is very simple and should give all of us confidence that we are not introducing any risk anywhere.

Sorry, I feel very conservative today (is it Christmast xD?)

Yes, I see that point. My concern goes in the same direction. What if we quickly need to change back? A lot of those configuration options have been proven valuable already.
Less simple, you could add a second configuration option, and keep the new value as the default. So one could revert back by adding the new config option.


ofx 9.4 <audiodecoder.h> not found , Mac OSX 10.12

in ofxAudioDecoder.h for some reason the line:
"#include <audiodecoder.h> isn't able to find the file.    I've tried hardcoding in the path and a bunch of other things but I can't figure out how to get it to recognize the file.
Any ideas ?

I think lib folder needs to be "libs" for the project generator to add it correctly to the project.


Storage gets cleared every other day (iOS)

Bug report
https://forum.ionicframework.com/t/storage-gets-cleared-every-other-day-ios14/203254
The Supabase token are in the local storage the problem is that on Ionic Capacitor Apps the iOS Localstorage will be cleared every other day. This is why it need to be saved somewhere else. StorageDB for example.

I have the same issue with angular 18

I have the same issue with angular 18. I am using a custom storage, but storage works fine for one day. the next day, on the first load the stored key isnt accessed (says last accessed the day before) and on page refresh the stored key gets deleted altogether.

 const customStorage = {
      getItem: (key: string) => this.storage.get(key),
      setItem: (key: string, value: string) => this.storage.set(key, value),
      removeItem: (key: string) => this.storage.remove(key),
    };

    this.supabase = createClient(environment.supabaseUrl, environment.supabaseKey, {
      auth: {
        storage: customStorage,
      },
    });

doesnt work with angular. I'll create a new ticket then, thanks for the reply anyway

doesnt work with angular. I'll create a new ticket then, thanks for the reply anyway

The Code is from a angular app

cant use this.storage since that doesnt work with ssr.

cant use this.storage since that doesnt work with ssr.

you need to add your own indexdb logic


How to add support for a new language to repo map?

Issue
Hey!
I'm willing to add support for additional languages, but I don't get where to start. I found *.scm files under aider/queries, but still don't get how the whole machinery of repo mapping works. Could someone please point me to docs on how to do that?
Version and model info
No response

Thanks for trying aider and filing this issue. This doc may be helpful:
https://aider.chat/docs/languages.html#how-to-add-support-for-another-language

I'm going to close this issue for now, but feel free to add a comment here and I will re-open. Or feel free to file a new issue any time.


I'm writing a new leaf block in a plugin with "render_method": "alpha_test" the transparent part in the middle is black

[在此处输入反馈]
I am writing a new leaf block in a plugin with "render_method" but the transparent part in the middle is still black after using "alpha_test"
By the way, it seems to be fine at 1.16, but not at 1.18
    "minecraft:material_instances": {
      "*": {
        "texture": "sakura_sakuraleaves",
        "render_method": "alpha_test"
      }

文档详细信息
⚠ 请勿编辑此部分。 docs.microsoft.com ➟ GitHub 问题链接必须具有此部分。

ID: 11a717d0-e1a6-84ac-9739-08cef6c82caf
Version Independent ID: c187c9a6-a98c-101b-fed6-2270baa772da
Content: Block Documentation - minecraft:material_instances
Content Source: creator/Reference/Content/BlockReference/Examples/BlockComponents/minecraftBlock_material_instances.md
Product: gaming
GitHub Login: @v-josjones
Microsoft Alias: v-josjones

Thank you for your comment. Unfortunately, we cannot provide addon support via this Github repository. This repo is for issues that relate specifically to the published articles and content we have on the docs.microsoft.com site.
If you are unable to find an answer to your addon question here, I can recommend the Bedrock Wiki community site (https://wiki.bedrock.dev/). This community-run site has different documentation than we host, and they also have a very welcoming Discord community that may be able to help you.
For Minecraft bugs or feature requests, I recommend the official Minecraft Feedback Site here: https://feedback.minecraft.net/hc/en-us


Bisect timeout (don't merge this pull request) 01

Don't merge.

@swift-ci Please smoke test OS X platform

@swift-ci Please smoke test

Okay that ain't working like I had hoped. The system foils my attempt at abusing it to do bisection by rebasing the commits I submit. Oh, well ...


Investigate removing JQuery 3.2.1

Umbrella issue: https://github.com/kubernetes/website/issues/22024
Investigate removing /static/js/jquery-3.2.1.min.js.
Remove the call to it in /layouts/partials/head.html#L78.
Note: While other scripts on this site use JQuery, as of Docsy we pull in version 3.3.1 so we should be fine. That said, this may cause issues and may not be possible.

/help
/kind cleanup
/priority important-longterm

/assign

/remove-lifecycle stale

@irvifa This problem has been fixed. Close the issue .
reference PR: https://github.com/kubernetes/website/pull/22357
/close

/triage accepted
/close


Force voxblox to use protobuf_catkin

Based on protobuf changes from this PR: https://github.com/ethz-asl/protobuf_catkin/pull/35
YOU WILL NEED TO DO A CLEAN REBUILD OF VOXBLOX AFTER THIS!!
Since we're swapping protobuf catkin to default to system library, we can swap voxblox to depending on protobuf_catkin. Main advantages this gives us is cross-compatibility with other stuff using protobuf_catkin, access to the PROTOBUF_CATKIN_GENERATE_CPP2 function which puts protos not in the build folder, and a clean-up of all dependent packages needing to link against protobuf over and over again.
The .rosinstall files have been updated to reflect this change.
Any packages that depend on this can ditch the extra proto linking step as a result of this change.
Let me know if there's any issues.

test this please

We waiting on anything here @helenol @mfehr ?
I'm actively using this branch if that helps. Seems legit.


Knowledge: writer Thomas De Quincey

Describe the contribution to the taxonomy

Knowledge contribution about the writer Thomas De Quincey
The model was conflating Thomas De Quincey and another writer, Samuel Taylor Coleridge, resulting in an erroneous answer to the question, "Who is Thomas De Quincey?" For example, "Confessions of an Inquiring Mind" is by Coleridge, but it is credited to De Quincey in the answer.

Input given at the prompt
   Who is Thomas De Quincey?

Response from the original model
  Thomas De Quincey (1785-1844) was an English essayist, novelist, and critic, renowned for his profound exploration of the relationship between memory, imagination, and human experience. He is best known   │
│ for his seminal work "Confessions of an Inquiring Mind" (1826), which delves into the intricacies of De Quincey's own thoughts and recollections, as well as the philosophical underpinnings of his          │
│ intellectual pursuits.                                                                                │
│                                                                                │
│ In "Confessions," De Quincey examines the role of memory in shaping personal identity, reflecting on his past experiences and encounters with various ideas that have captivated his curiosity. The work is  │
│ a rich tapestry of introspection, autobiography, and philosophical inquiry that has left an indelible mark on literary history.                                                                              │
│                                                                                │
│ De Quincey's writing style is characterized by its vivid imagery, evocative language, and keen analysis of the human condition. His works often explore themes such as memory, imagination, sensibility, and │
│ individuality, inviting readers to reflect on their own experiences and beliefs.                                                                                │
│                                                                                │
│ Beyond his literary contributions, De Quincey was a prominent figure in the Victorian intellectual scene. He corresponded with many luminaries of his age, including Charles Darwin, John Keats, and George  │
│ Eliot. His thoughtful and engaging correspondence provides valuable insights into the cultural and intellectual climate of the time, as well as the complex interplay between literature, science, and       │
│ philosophy.                                                                                │
│                                                                                │
│ In summary, Thomas De Quincey was a remarkable figure in English literature, celebrated for his explorations of memory, imagination, and human experience. His groundbreaking work continues to resonate     │
│ with readers today, offering a unique perspective on the intricate relationship between the past, the present, and the future.

Response from the fine-tuned model

  Thomas De Quincey (1814-1869) was a British essayist, journalist, and social critic, renowned for his profound insight into the Victorian era's moral issues. He was an advocate of free will and            │
│ individualism, often exploring these themes in his works. De Quincey is particularly famous for his 1859 work "Confessions of an English Opium Eater," which delves into his personal struggles with         │
│ addiction to opium.                                                                                │
│                                                                                │
│ In this memoir, he provides a vivid and honest account of his experiences, shattering the social stigma surrounding drug use and contributing significantly to the Victorian literary landscape. De          │
│ Quincey's writing is characterized by its evocative imagery and emotional intensity, making him a prominent figure in 19th-century literature.

Contribution checklist

[x] The contribution was tested with ilab generate
[ ] No errors or warnings were produced by ilab generate
[x] All commits are signed off (DCO)
[x] The qna.yaml file contains at least 5 seed_examples
[x] The qna.yaml file was linted and prettified (yaml-validator can do both)
[x] An attribution.txt file in the same folder as the qna.yaml file
[x] Content does not include PII or otherwise sensitive or confidential information
[x] Content does not include anything documented in the project's Avoid these Topics guidelines

ilab generate printed this warning:
/Users/michael/.pyenv/versions/3.11.11/lib/python3.11/site-packages/llama_cpp/llama.py:1054: RuntimeWarning: Detected duplicate leading "<s>" in prompt, this will likely reduce response quality, consider removing it...
  warnings.warn(

We need to wait for the new version of the cmb shows up in the environment. /cc @vishnoianil can you update us on when that will be ready?


Could remove useless tokens

In the gpt-neox 20B tokenizer there are 23 special tokens containing only spaces which are always used when the input contains at least 2 consecutive spaces. So the 221 normal tokens which contain at least two consecutive spaces are never used and could be removed.

You are not wrong, but removing them will not do anything as the tokenizer’s size is primarily determined by efficiency calculations rather than the actual number of tokens one “needs.” Most tokenizers are actually padded with dummy tokens for this reason (I don’t recall off the top of my head if ours is)


kdepim-runtime: use XOAUTH2 SASL plugin from libkgapi

Description of changes
Some mail servers like Gmail use SASL XOAUTH2 mechanism to authenticate clients with OAuth token instead of password. Unfortunately, Cyrus SASL does not have authentication plugin for this mechanism out of the box, so clients that use this library and want to authenticate at such mail servers should bring their own plugin for XOAUTH2. KMail (or KDE PIM in general) have such plugin as part of LibKGAPI, and on FHS systems LibKGAPI puts its plugin right next to other Cyrus SASL plugins, so it can be picked up by library automatically. In NixOS, this plugin end up in other directory, which has no references to it, so akonadi_imap_resource fails with next error message when tries to authenticate at such mail servers:
org.kde.pim.kimap: sasl_client_start failed with: -4 "SASL(-4): no mechanism available: No worthy mechs found"

In result, no mail is fetched (progress bar in KMail is stuck at 0%), and user after reading logs is left wondering why no worthy mechs were found.
This commit sets SASL_PATH environment variable, so Cyrus SASL can find both its own plugins and one provided by LibKGAPI. With this change, KMail can successfully fetch mail from Gmail.
This fix is based on discussion in #108480 about means for merging SASL plugins folders from different packages. I think setting SASL_PATH is the most straightforward way of doing this in Nix, but maybe it should be automated somehow, like it is done with XDG_DATA_DIRS in wrapQtAppsHook. However, I do not know easy way to do so in non-breaking manner, so lets leave it to package maintainers for now.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
NB: nixpkgs-review tries to build packages that are marked as broken. All non-broken packages build successfully with this changes.
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I guess this could be backported to release-22.05 without any problems, works for me on nixos-22.05 channel.

@ttuegel, hello, can you review this PR please?

Double checked with new commits: fix still work, both from kdepim-sasl-path branch and kdepim-sasl-path-22.05 rebased on nixos-22.05.


How to read huge Excel file without load entire file in memory and append new rows?

I need to append new rows to a huge excel file. How to do it without to read the entiry file in the memory, which can potencially slowing down the process and heap out of memory error ?

I am also struggling with the same issue, but in my case it is CSV file

excel stream to csv, and handle csv file using pipeline. this is my may. it can append data item into any where.

@HappyFerry may you please share your implementation ?

Yeah, @HappyFerry I'd be highly interested in how you stream excel to csv, especially with large excel files. Would be very grateful if you shared some details.

This library, and every other JS library, is limited to the browser APIs in the frontend and limited to platform APIs on the backend.
XLSX is a ZIP-based file format and cannot be incrementally processed with a streaming data source for structural reasons as described in https://docs.sheetjs.com/docs/solutions/input#example-readable-streams .  ZIP was designed for streaming write and assumes readers have random access.
That means, to first order, the browser must retain a seekable copy of the original file.  (In NodeJS there is a way to avoid this using child_process to extract the ZIP file to the filesystem using the unzip command-line tool.)
Assuming retaining the original file in memory is not an issue, that specific workflow (reading a file and appending rows) can be optimized at the expense of being able to extract data in the process.  Our Pro Edit build can skip the worksheet generation and surgically edit the raw XML.
If there is interest in a NodeJS-specific large file processor, please let us know.

Save as the xlsx format or CSV format change it in to binary sheet so the huge data file easily access.


Rename proposal

Let's rename hbm-react-components to md-components.

it's shorter
it combines Material Design and React Components
it's easier to find on the web
it's still free on npm
we don't really need the company name inside the npm package

What do you think @lipp @Fa-So

👍 agree

Agree

@zemirco the repo name remains the same?


Add excuse

My small contribution.

Sorry for the delay, I must have missed the notification. Merged!


Query returns incorrect results

OrientDB Version: 2.2.17
Our database returns wrong results for the following query:
select expand($courses) from Activity let courses = (select from $current.in_ContainsActivity.out) where id = 809
The query returns a record with the class "Course", and id = 67
However, if you follow the in_ContainsActivity.out link from Activity where id = 809, the record that should be returned has id = 54
I've uploaded an export of our test database where this issue occurs:
https://drive.google.com/file/d/0B_TS9oS4-Fi2UDh4b0VxbU1NbHc/view?usp=sharing

Any updates on this issue? It's really hurting us right now.

Hi @alpeshgaglani
I'm not sure I got the point of your query... That $current refers to records in the subquery, not in the parent query. The right syntax should be:
select expand($courses) from Activity let courses = (select from $parent.$current.in_ContainsActivity.out) where id = 809

But why should you use that? Can't you just do
select expand(in_ContainsActivity.out) from Activity where id = 809

or even better
select expand(in("ContainsActivity")) from Activity where id = 809

Thanks
Luigi

Thanks @luigidellaquila for the response. Our understanding was that in this particular case, $current refers to the current record set in the top level query. Interestingly, this used match exactly with our understanding up until 2.1.x! We have to use projection because we are doing further sub queries. I had just posted the simplified query to help debugging :)
:+1: for your help!

HI @alpeshgaglani
Just to give you all the info, $current variable refers to the current record in current (sub) query. If you want to access the parent query context you can use $parent variable, eg. $parent.$current refers to the current record in parent query. You can also use it at multiple levels, eg. $parent.$parent.$current.
I guess I can close this issue, please reopen if you need further help
Thanks
Luigi


High Latencies for Redis Operation

Bug Report
There are high latencies for Redis Operations - EXPIRE for creating and fetching the session.
Current Behavior
Our project is based on Reactor, Kotlin Coroutines and Netty.
We're using Spring Session Data Redis with LettuceClient for Redis related operations.
Our APM tool DataDog show cases that while creating/fetching the session, EXPIRE ops have very high latencies ranging in minutes. Rest of the operations like HGETALL, DELETE, EXISTS and HMSET 99th percentile is within 100ms.
Our Redis Infra latencies looks good. So we suspect DataDog showing high latencies in spans could be due to thread waiting.
There is only one NIO thread created - lettuce-nioEventLoop-5-1

Redis Operations logs
[logLevel=DEBUG] -- 2023-04-06 17:54:06,868 +0530 -- lettuce-nioEventLoop-5-1              io.lettuce.core.protocol.RedisStateMachine -- Decode done, empty stack: true
[logLevel=INFO ] -- 2023-04-06 17:54:06,868 +0530 -- lettuce-nioEventLoop-5-1              com.apple.carrierservices.preauth.utils.LoggingCommandLatencyRecorder -- appServer=/AppServerIP, redisServer=redisServer, redisCommandType=EXPIRE, firstResponseLatency=277454292, completionLatency=277478875, TimeTaken=24583 nanoseconds
[logLevel=DEBUG] -- 2023-04-06 17:54:06,868 +0530 -- lettuce-nioEventLoop-5-1              io.lettuce.core.protocol.CommandHandler -- [channel=0x15b747b9, /AppServerIP -> redisServer, epid=0x1, chid=0x1] Completing command LatencyMeteredCommand [type=EXPIRE, output=BooleanOutput [output=true, error='null'], commandType=io.lettuce.core.RedisPublisher$SubscriptionCommand]
[logLevel=DEBUG] -- 2023-04-06 17:54:06,873 +0530 -- lettuce-nioEventLoop-5-1              io.lettuce.core.protocol.CommandHandler -- [channel=0x15b747b9, /AppServerIP -> redisServer, epid=0x1, chid=0x1] Received: 5 bytes, 6 commands in the stack
[logLevel=DEBUG] -- 2023-04-06 17:54:06,873 +0530 -- lettuce-nioEventLoop-5-1              io.lettuce.core.protocol.CommandHandler -- [channel=0x15b747b9, /AppServerIP -> redisServer, epid=0x1, chid=0x1] Stack contains: 6 commands
[logLevel=DEBUG] -- 2023-04-06 17:54:06,873 +0530 -- lettuce-nioEventLoop-5-1              io.lettuce.core.protocol.RedisStateMachine -- Decode done, empty stack: true
[logLevel=INFO ] -- 2023-04-06 17:54:06,873 +0530 -- lettuce-nioEventLoop-5-1              com.apple.carrierservices.preauth.utils.LoggingCommandLatencyRecorder -- appServer=/AppServerIP, redisServer=redisServer, redisCommandType=HMSET, firstResponseLatency=278740375, completionLatency=278768500, TimeTaken=28125 nanoseconds
[logLevel=DEBUG] -- 2023-04-06 17:54:06,873 +0530 -- lettuce-nioEventLoop-5-1              io.lettuce.core.protocol.CommandHandler -- [channel=0x15b747b9, /AppServerIP -> redisServer, epid=0x1, chid=0x1] Completing command LatencyMeteredCommand [type=HMSET, output=StatusOutput [output=OK, error='null'], commandType=io.lettuce.core.RedisPublisher$SubscriptionCommand]
[logLevel=DEBUG] -- 2023-04-06 17:54:06,873 +0530 -- lettuce-nioEventLoop-5-1              io.lettuce.core.protocol.CommandHandler -- [channel=0x15b747b9, /AppServerIP -> redisServer, epid=0x1, chid=0x1] Received: 4 bytes, 5 commands in the stack
[logLevel=DEBUG] -- 2023-04-06 17:54:06,873 +0530 -- lettuce-nioEventLoop-5-1              io.lettuce.core.protocol.CommandHandler -- [channel=0x15b747b9, /AppServerIP -> redisServer, epid=0x1, chid=0x1] Stack contains: 5 commands

Session Flow

Any Session fetch is associated with below Redis Ops
HGETALL -> New Session      -> DELETE   -> save	  ->    HMSET    ->  EXPIRE
HGETALL -> Existing Session             -> Expired  	  ->    DELETE   -> save   -> HMSET  ->  EXPIRE
HGETALL -> Existing Session             -> Non Expired    ->    EXISTS  -> save   -> HMSET  ->  EXPIRE
Expected behavior/code
The Redis operations latencies should be in admissible ranges, few Millis.
Environment

Redis Standalone, Version: 6.2.6
Lettuce version(s): 6.1.10.RELEASE
spring-session-data-redis:jar: 2.7.0
Redis version: 6.2.6
reactor-core:jar: 3.4.27
kotlinx-coroutines-reactor:jar: 1.6.4
reactor-netty-http:jar: 1.0.28
1 AWS Instance used 3Cores, 10GB

Spring Redis config in use
spring.redis.lettuce.pool.enabled=true
spring.redis.lettuce.pool.max-active=16
spring.redis.lettuce.pool.min-idle=8
lettuce.computation.threads.count=32

The issue is intermittent. It has been observed under very low load as well.
The application recovers itself after some time and is able to serve high traffic usually.

Using the reactive API with a single connection, the I/O event loop is being used as single thread for all subsequent operations (e.g. Jackson deserialization). You could try enabling ClientOptions.publishOnScheduler to check whether spreading load across multiple worker threads would help.

We've had ClientOptions.publishOnScheduler enabled.
We're using Snappy for compression/decompression which could be counted as one of a heavy workload.
org.xerial.snappy:snappy-java:jar:<IP_ADDRESS>
One thing we noticed as part of connection pooling doc is we have an apace common-pool2 dependency which might not be relevant with Reactive APIs.
So we are trying to simulate our work load again with it's removal and using Lettuce inbuilt implementations.

Please run your workloads through a profiler to identify what is causing the slowdowns. Alternatively, use multiple connections through the AsyncPool that provides non-blocking connection pooling.


kml to geojson conversion changes float type to string type

running togeojson single_kml.kml > single_kml.geojson
In my KML I have a value explicitly defined as a float:
<SimpleData name="normalized_color" xmlns:py="http://codespeak.net/lxml/objectify/pytype" py:pytype="float">24.110576923076923</SimpleData>

The resultant geojson ends up turning this value into a string (noted by the quote marks): "normalized_color": "24.110576923076923"
This is very problematic as I am trying to use this value with my custom mapbox Style's Property Function to color the Polygons I am importing.  Because it is a string i cannot use any of the data range calculations provided by mapbox to color my shapes.
single_kml.txt

dup, bad connection


feat: interchain tx helpers

https://p2pvalidator.atlassian.net/browse/LSC-77
This an example of interchain txs contract plus helpers for interchain acknowledgements. Here is a test for this stuff https://github.com/neutron-org/neutron-integration-tests/pull/1

tested with ./test_interchain_txs.sh, everything seems to work fine

please rename the contract to neutron_interchain_txs, we use underscores for contract names

the very similar interchain queries contract

it is already a bad sign in my book.

do you guys think we should have the contract covered with unit tests?

anyway, my opinion is that if we have two contracts, we kinda have to provide tests for every single one.

btw the very similar interchain queries contract is covered with unit tests a little. whereas I think it may not be necessary, do you guys think we should have the contract covered with unit tests?

As I can see only interchain_queries contract has some tests, so this way if we really need unit tests we can have separate task to cover all the contracts


v7.12.0 contains breaking change due to extraneous /v1/ segment in URL

Here is code that worked < v7.12.0:
import { CohereClient } from "cohere-ai";

const cohere = new CohereClient({
  environment: "https://api.cohere.com/v1",
  token: "...",
});

const main = async () => {
  const rerank = await cohere.rerank({
    documents: [{ text: "a white hat" }, { text: "and old trinket" }],
    query: "a gray blouse",
    topN: 2,
    model: "rerank-multilingual-v3.0",
  });

  console.log(rerank);
};

main();

= v7.12.0 this throws the following error:

NotFoundError: NotFoundError
Status code: 404
Body: {
  "ok": false,
  "error": {
    "reason": "non-json",
    "statusCode": 404,
    "rawBody": "404 page not found\n"
  }
}
    at CohereClient.<anonymous> ([...]/Desktop/reranker-test/node_modules/cohere-ai/Client.js:811:31)
    at Generator.next (<anonymous>)
    at fulfilled ([...]//Desktop/reranker-test/node_modules/cohere-ai/Client.js:31:58)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5) {
  statusCode: 404,
  body: {
    ok: false,
    error: {
      reason: 'non-json',
      statusCode: 404,
      rawBody: '404 page not found\n'
    }
  }
}

The reason is that this latest version appends an additional v1 segment to the URL path which is unexpected by your API.

hey @mtharrison thanks for flagging this, will fix asap

got a fix here which is backward and forward compatible https://github.com/cohere-ai/cohere-typescript/pull/196

Ok great, I'll test on my end too.

The fix looks good to me, are you making a release?

yeah just released 7.12.1, should be available shortly. Thanks for reporting this!!


Svelte 5 Support

I realise this is probably still rather early to be looking at this, but I am so I figure I'll put a ticket with my findings.  I have patched it locally so there's no urgency on my part to get it fixed ;)
When using sveltekit-svg with Svelte 5.0.0-next.26, compilation fails with the following error:
Failed reading SVG ".../svelte-site/src/resources/title.svg?component": Invalid compiler option: css: "none" is no longer a valid option. If this was crucial for you, please open an issue on GitHub with your use case. sr [CompileError]: Invalid compiler option: css: "none" is no longer a valid option. If this was crucial for you, please open an issue on GitHub with your use case.
    at ar (.../svelte-site/node_modules/.pnpm/svelte@5.0.0-next.26/node_modules/svelte/compiler.cjs:1:228490)
    at gS (.../svelte-site/node_modules/.pnpm/svelte@5.0.0-next.26/node_modules/svelte/compiler.cjs:1:627312)
    at .../svelte-site/node_modules/.pnpm/svelte@5.0.0-next.26/node_modules/svelte/compiler.cjs:1:624179
    at .../svelte-site/node_modules/.pnpm/svelte@5.0.0-next.26/node_modules/svelte/compiler.cjs:1:627031
    at .../svelte-site/node_modules/.pnpm/svelte@5.0.0-next.26/node_modules/svelte/compiler.cjs:1:626951
    at e.compile (.../svelte-site/node_modules/.pnpm/svelte@5.0.0-next.26/node_modules/svelte/compiler.cjs:1:642916)
    at TransformContext.transform (.../svelte-site/node_modules/.pnpm/@poppanator+sveltekit-svg@4.1.3_svelte@5.0.0-next.26_svgo@3.1.0_vite@5.0.10/node_modules/@poppanator/sveltekit-svg/dist/index.js:112:55)
    at async Object.transform (file://.../svelte-site/node_modules/.pnpm/vite@5.0.10/node_modules/vite/dist/node/chunks/dep-R0I0XnyH.js:63705:30)
    at async loadAndTransform (file://.../svelte-site/node_modules/.pnpm/vite@5.0.10/node_modules/vite/dist/node/chunks/dep-R0I0XnyH.js:49386:29)
    at async Object.warmupRequest (file://.../svelte-site/node_modules/.pnpm/vite@5.0.10/node_modules/vite/dist/node/chunks/dep-R0I0XnyH.js:60017:13) {
  filename: '.../svelte-site/src/resources/title.svg?component',
  position: undefined,
  start: undefined,
  end: undefined,
  code: 'invalid-compiler-option'
}

Changing css: 'none', to css: 'external', fixes the compile issue and generally seems to work (the SVG is rendered now), although it now has a couple of warnings:
`generate: "dom"` and `generate: "ssr"` options have been renamed to "client" and "server" respectively
The hydratable option has been removed. Svelte components are always hydratable now.
Sourcemap for ".../svelte-site/src/resources/title.svg" points to missing source files

(the last one is for the specific SVG I am importing, but applies more generally)

I've released a beta with Svelte 5 support (hopefully). Test it and check if it works.
npm i -D @poppanator/sveltekit-svg@svelte5

This seems to be working almost perfectly, thanks!
The only minor issue is that I still get this warning:
Sourcemap for ".../svelte-site/src/resources/title.svg" points to missing source files

Also I encountered #54 and temporarily worked around it by adding @rollup/pluginutils as a dev dependency as a comment on that ticket suggests.

I think this should now be fixed in 5.0.0-svelte5.2

That works for me!

@poppa thanks so much for the update to Svelte5!

Trying to install it with the rc version<EMAIL_ADDRESS>seems to be failing with this error,
npm i -D @poppanator/sveltekit-svg@svelte5
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/svelte
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@poppanator/sveltekit-svg
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!

Trying to install it with the rc version<EMAIL_ADDRESS>seems to be failing with this error, npm i -D @poppanator/sveltekit-svg@svelte5
[...]

Thanks for the report @pyrossh . This should now be fixed in v5.0.0-svelte5.3.
If you still get the error, please re-open #57

@poppa Thanks it works now.

I'm still seeing

Failed reading SVG "....bmc.svg":
Invalid compiler option: css: "none" is no longer a valid option.

with
		"svelte": "5.0.0-next.242",
		"sveltekit-svg": "link:@@poppanator/sveltekit-svg",

is there a way to pass a css fill to the SVG to prevent the warning?
also i'm wondering if the config options in the readme here are not ready for 5.0 of sveltekit?
I'm wondering where the plugins would go then? I
const config = {
	preprocess: vitePreprocess(),
	kit: { ...  },
	// put here for now but not sure? at least no error on startup
	plugins: [
		svg(), // Options are optional
	],

I'm still seeing
...
"svelte": "5.0.0-next.242",
"@poppanator/sveltekit-svg": "^4.2.1",

You need to install the version 5 package of this plugin, e.g. the svelte5 tag: npm i -D @poppanator/sveltekit-svg@svelte5 @dcsan

@madmoizo Okey dokey, so I've published v5 as the default package on NPM


Improve killing orphans regexp

In kubernetes we run a python script to setup the environment and run the pilot wrapper. Once the pilot has executed a job, it invokes the kill orphan method and kills anything running under the same user, including the startup script. So the startup script can't finish properly and upload the logs.
I'm trying to fall into one of the exceptions, like renaming my python script to "pilots_starter.py" (see https://github.com/PanDAWMS/pilot2/blob/master/pilot/util/processes.py#L346). However I don't think the regexp matches properly the command, since the command can contain whitespaces but the regexp cuts after the first whitespace.
Currently it works like this. See that from "python pilots_starter.py" only "python" is kept:
[root@ae86f0c2304b /]# ps -o pid,ppid,args
  PID  PPID COMMAND
...
   33     1 python pilots_starter.py
[root@ae86f0c2304b /]# python
Python 2.7.5 (default, Apr  9 2019, 14:30:50)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-36)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import re
>>> pattern = re.compile('(\d+)\s+(\d+)\s+(\S+)')
>>> line = '33     1 python pilots_starter.py'
>>> ids = pattern.search(line)
>>> print(ids.group(3))
python

Proposal to change the pattern to accept whitespaces for the last block:
>>> pattern = re.compile('(\d+)\s+(\d+)\s+([\S\s]+)')
>>> ids = pattern.search(line)
>>> print(ids.group(3))
python pilots_starter.py

Aha, this must be the source of all the errors I have been seeing on the cluster. All I see is
/usr/bin/bash: line 1:     7 Killed                  python k8s_startup_script.py

since all other output is captured by the script getting killed. Moreover the script getting killed is responsible for uploading the log files, but unable to do so when it gets killed.
This happens to over 60% of jobs. The pilot and Panda job are (presumably) completing successfully though. But we have no visibility of job outcome and this makes it hard to find any other problems that may be occurring.

Hi ok, updated in the latest dev pilot. Thanks (will close this issue once the new pilot has been released, no date yet though)

Hi again, it seems the pattern change causes problems.. Maybe we need an 'if kubernetes'-line? See e.g. job
https://bigpanda.cern.ch/job?pandaid=4599865064
error + cut off stdout.

Hi Paul,
I don't see any stdout for that example, but it's true that the pilot might be killing the wrapper now, because this condition is no longer true:
https://github.com/PanDAWMS/pilot2/blob/master/pilot/util/processes.py#L351
                if args.endswith('bash'):                     logger.info("will not kill bash process")
It would also have to be adapted to something like
                if 'bash' in args: # or if 'runpilot2-wrapper.sh' in args:                     logger.info("will not kill bash process")
The thing is that I don't think this part works as intended, since args is only the first word of the command. So it's difficult to implement any new 'if kubernetes' condition.
Cheers,
Fernando

I see what you mean with the kubernetes option. We can try to use the PILOT_NOKILL env variable in the shortterm
https://github.com/PanDAWMS/pilot2/blob/master/pilot/util/processes.py#L327
In the longterm I prefer if there is a better fix, since otherwise there is no zombie cleanup at all and we need to remember a new variable in the templates.

Ok, the stdout disappeared in the meantime. The 'if bash'-change works fine:
https://aipanda024.cern.ch/condor_logs_2/20-01-07_13/grid.16975124.0.out
/Paul

Hi,
So what is the status of this, has it been fixed by https://github.com/PanDAWMS/pilot2/commit/addea4bb708085e9d8d3187e33512e5476fea231#diff-a27694ffcf4e8da1086ef60b00dba60b34c834dfea749fd8599a63fd2fe98484  and can be closed  ?
IIUC the pattern matching for "pilots_starter.py"  now works properly, so the script avoids being killed via that exception?
Thanks!

Hi,
Yes as far as I know this has been fixed. Closing.
/Paul


Latest template updates break DB Operator

The last commit (https://github.com/diggerhq/helm-charts/commit/2cf4b153c66b40dcb1f556a8756025fdb8950b9b#diff-204d7782307a1567576db307ef282e33ef9a56ab6024b064960f777651b15acaL35-R49) / #10 removes the ability to use an existing secret to set multiple environment variables and has broken conditional logic.
The code which was replaced worked with a secret like this, generated by db-operator:
apiVersion: v1
data:
  DATABASE_URL: redacted
  POSTGRES_DB: redacted
  POSTGRES_PASSWORD: redacted
  POSTGRES_USER: redacted
kind: Secret
metadata:
  creationTimestamp: "2024-09-07T03:27:07Z"
  labels:
    app.kubernetes.io/managed-by: db-operator
    kinda.rocks/used-by-kind: Database
    kinda.rocks/used-by-name: digger-db
  name: digger-db-credentials
  namespace: infra-digger
  ownerReferences:
  - apiVersion: kinda.rocks/v1beta1
    kind: Database
    name: digger-db
    uid: 17c1d193-77a2-4739-8324-9894d3c1fca8
  resourceVersion: "345098783"
  uid: 07b60dc5-03e3-447a-ab12-93a0107f14e1
type: Opaque

The new code only supports POSTGRES_PASSWORD, breaking anything using the previous existing secret logic.  It also has an initial conditional followed by an else and then another conditional; the last two segments are swapped.  So if someone were to specify an existing secret using Values.digger.postgres.existingSecretName and also set postgres_enabled to false (it now weirdly defaults to true, which probably should not be the default and should definitely be more loudly communicated in the changelog), the chart would render two valueFrom lines, resulting in invalid yaml.

Without adding too much commentary because I know I'm in a mood from tracking this all down...  Seriously.  The changes made there are not patch level.  "Major version = 0" doesn't mean it's cool to completely break backwards compatibility in multiple places without adding any release notes at all.  That PR broke my setup in at least two completely avoidable places, adding only functionality which was already achievable with the pre-PR chart.  And, again, zero documentation of all the new values. :/

Hey thanks for reporting these (#11 and #13)!
Not gonna lie our exposure to helmcharts has been limited before digger so we clearly messed up.
We wanted to improve a few of the defaults but we did a bad job and testing for backwards compatability, I will prioritise a fix for these items to make it work out from this point on and document that the latest release has broke some older defaults

resolved in #14


Add tests for serialization / deserialization

Based on the awesome PR #27

Sure !
I thought I didn't have the rights to push on your remote.


Don't fetch previews from server when URL previews are disabled

Currently, the client will request the URL preview from the server even when displaying URL previews is disabled. This leads to the contents of E2E-encrypted chats being leaked to the homeserver without the ability to prevent this on the client.
This change sets the URLPreview to null in manageURLPreviews when URL previews are disabled in the session. Missing previews will be fetched when the switch is re-enabled later.

When you say Missing previews will be fetched when the switch is re-enabled later. it's because the Fragment will be recreated after the settings has been updated, due to the navigation, and so mUrlsPreviews will be cleared, right?

Yeah, that's just the existing functionality and not something I implemented.


Use the event keyword instead of an auto property

Using and event instead of an auto property in the ITcpSocketListener has some advantages:

Only the TcpSocketListener is able to raise the event
IntelliSense/ReSharper provide more relevant icons and code completion options
http://stackoverflow.com/questions/29155/what-are-the-differences-between-delegates-and-events

EDIT: I just read that you want the pull requests on the dev branch, if you want I can close this PR and rebase it for the dev branch?
EDIT2: The udp classes have the same issues, if you want I could provide the same fix for those too

Well, I learned something new today. Thanks!
Looks like the right change to make - as you noticed, PRs should go against dev now (I have just started that and haven't created a CONTRIBUTING file to specify it, sorry about that). I will take you up on the offer to do the UDP classes - so if you make those changes too and resubmit against dev we will be in action.
Thanks again :+1:

continued in #42


Add Documentation from API docs to boilerplate of Heroku Instance Apps

The documentation at https://bothan.io/api is great, can get someone up and running really quickly. Incorporating elements of it into the $appname.herokuapp.com/documentation

#183 relevant

MD file located here https://github.com/theodi/bothan/blob/gh-pages/api.md

resolved with PR #190


Changed default max_item_size to 10

JIRA: MADLIB-1288
Changed default max_item_size to 10

spelling
please change

INTEGER, default: 10. It Determines the maximum size of frequent

to

INTEGER, default: 10. Determines the maximum size of frequent

spelling
please change

INTEGER, default: 10. It Determines the maximum size of frequent

to

INTEGER, default: 10. Determines the maximum size of frequent

done

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/841/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/843/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/845/

@hpandeycodeit  I just noticed that this branch is not rebased off latest master. Could you kindly rebase it over latest master and push it again? You might have to use the --force-with-lease option though. The Jenkins build on this PR should have been red since this branch does not have the Postgres support commit in it. So I am a little surprised its green!

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/847/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/849/


Put schedule manifests in version control?

At the company I'm working we currently follow GitOps like practices where we store the whole desired state in a git repository and use a deployment pipeline to apply this against the kubernetes cluster.
Therefore I'd like to store ark schedules in a git repo as well instead of using the command line to manipulate it afterwards. Since I couldn't find any documentation about the schedules resource types except for the Go types I'm wondering if this is a good practice or if there is something that would prevent me from doing so.

Closing as the original question was answered. Feel free to reopen if needed.


React Native Debugger doesn't show styles

Version
2.0.0-15
Steps

Create a styled component
Give it some styles
Use React Native Debugger to inspect the created components

Expected Behavior

the styles used for the component should show up in the inspector

Actual Behavior

There's no styles at all

I can't reproduce this:
Styled Component:
const Box = styled.View`
    flex: 1;
    align-items: center;
    justify-content: center;
    padding-top: ${Constants.statusBarHeight};
    background-color: blue;
`

@k15a i believe @bernatfortet meant https://github.com/jhen0409/react-native-debugger not https://github.com/facebook/react-devtools/tree/master/packages/react-devtools
Created an issue on React Native Debugger, seems like it belongs there (https://github.com/jhen0409/react-native-debugger/issues/339)
btw @bernatfortet did you manage to fix that issue somehow?


Centos8

Issue #32
I need newer libc version for tests then Centos 7 is providing. As was written in pull request #33 I merge two Dockerfiles into one but instead of version parametrization I made argument BASE_IMAGE where you have to place docker full image name like registry.access.redhat.com/ubi8/ubi:latest, or rockylinux:8, or quay.io/centos/centos:stream8. The main reason for doing this was the fact that Centos 8 repos are closed now and all it's users have to migrate to Centos Stream which official image base name is quay.io/centos/centos.
Also I updated gcc version up to 11 and valgrind version up to 3.19.0.

This is a situation where a proposed change does not align with the direction of the project. We do not have a need for centos8 (especially since it is EOL). This is probably an instance where working from a fork is the better option.

Centos 8 is not the one distro which is supported by this Dockerfile, also it is Rocky Linux 8 and RedHat Enterprise Linux 8.


Add pagination to search

Currently it is hardcoded to 100 limit at 0 offset
Depends from #80

done already


Survey - add intro page before count

The intro alert will look like this:

We discussed changing the text for this pop-up page. Will this work or is it still too long:
You are now ready to start your FIT Count!

Count every insect that lands on the target flowers (ignore other flowers, and do your best to count each individual insect once only!).
Tap on the insect images to increase your count (use the minus buttons if you need to decrease the count).
If you are not sure of the insect group, add it to the "Other insects" category.

When you are ready, select "Start count" and the timer will start.

Thanks, if possible, I would shorten this still. In the Welsh language, some English texts will expand and if we want to present all the bullet points without scrolling down then I would at least remove the last sentence which I don't think is necessary.

Can we go with:
The timer will now start!
Count every insect that lands on the target flowers (count each individual once).
Tap the insect images to increase your count (minus buttons will decrease the count).
If you don't know which insect group, add to the "Other insects" category.

I'm happy with this shortened version from @kitenetter and good to have the tap and minus buttons explained.


XAML Controls Gallery

Fails to install from app store with Code: 0x80073CFB

This is an issue with the app store version of the app, not with the sample itself. I'll let the feature team know.


Allow the creation of smallest size of Cloud Deployment for learning and testing scenarios

Article :  https://docs.microsoft.com/en-us/azure/machine-learning/desktop-workbench/deployment-setup-configuration
As per the log below, i 'm trying to create a smaller ML cluster deployment for testing purposes, but i observed few issues.

The Agent node must be in HA mode and it mandates more than 1 which is understandable, but for testing or learning, its overkill and not dollar friendly
Tried giving  Standard_D1_v2 as the node SKU instead of the default  Standard_D3_v2, it throws error
Lastly unable to use previously created resources when deploying clusters (Please check the logs below)

Hope to receive support on this!
root@myMachine:~# az ml env setup --cluster --agent-count -1 --agent-vm-size Standard_D1_v2 -l westeurope -n cloudmlenv -g ilyas-rg3
Found currently set environment
Reuse storage and ACR (Y/n)?n
Subscription set to myAzureSubscription
Continue with this subscription (Y/n)? Y
Resource group ilyas-rg3 already exists, skipping creation.
Provisioning compute resources...
{'Azure-cli-ml Version': '0.1.0a27.post3', 'Error': ValidationError("Parameter 'AcsClusterProperties.agent_count' must be greater than 1.",)}
root@myMachine:~# az ml env setup --cluster --agent-count -2 --agent-vm-size Standard_D1_v2 -l westeurope -n cloudmlenv -g ilyas-rg3
Found currently set environment
Reuse storage and ACR (Y/n)?Y
{'Azure-cli-ml Version': '0.1.0a27.post3', 'Error': KeyError('resourceId',)}
root@myMachine:~# az ml env setup --cluster --agent-count -2 --agent-vm-size Standard_D1_v2 -l westeurope -n cloudmlenvironment -g ilyas-rg4
Found currently set environment
Reuse storage and ACR (Y/n)?n
Subscription set to myAzureSubscription
Continue with this subscription (Y/n)? Y
Creating resource group ilyas-rg4
Provisioning compute resources...
{'Azure-cli-ml Version': '0.1.0a27.post3', 'Error': ValidationError("Parameter 'AcsClusterProperties.agent_count' must be greater than 1.",)}
root@myMachine:~# az ml env setup --cluster --agent-vm-size Standard_D1_v2 -l westeurope -n cloudmlenvironment -g ilyas-rg4
Found currently set environment
Reuse storage and ACR (Y/n)?Y
{'Azure-cli-ml Version': '0.1.0a27.post3', 'Error': KeyError('resourceId',)}
root@myMachine:~# az ml env setup --cluster --agent-vm-size Standard_D1_v2 -l westeurope -n cloudmlenvironment -g ilyas-rg4
Found currently set environment
Reuse storage and ACR (Y/n)?n
Subscription set to myAzureSubscription
Continue with this subscription (Y/n)? Y
Resource group ilyas-rg4 already exists, skipping creation.
Provisioning compute resources...
{'Azure-cli-ml Version': '0.1.0a27.post3', 'Error': ErrorResponseWrapperException('(BadRequest) The request is invalid',)}
root@myMachine:~# az ml env setup --cluster -l westeurope -n cloudmlenvironment -g ilyas-rg4

@ilyas-it83 Thank you for the valuable feedback,we are investigating the issue.

@ilyas-it83 Thanks for you feedback here. However, because this is not a document related issue, please post your issue in Stack Overflow or MSDN forum so that we can help you well. Thanks a lot!
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


More 'in your face' notifications

Since sometimes notifications don't come through right away (or at all?), should we have something more 'in your face' for thread invites and similar when actually 'in' the app? Currently, thread invites can be missed in the (long) list of photo likes, comments, adds, etc. Perhaps an in-app notification on any screen would be useful here?

We are using overlays in various places (empty threads list) to provide information and give feedback. It might be nice to pull this out into a more generic 'overlay' framework within the app, to make things like information, notifications, within the app easier to use?

While counter intuitive, this is exactly opposite of the way that it's shown to work (see section starting page 34 https://www.dropbox.com/s/tuz2fj0qieiw1ov/Mobile Planet 04182018.pdf?dl=0&utm_source=Benedict's+newsletter&utm_campaign=4f0e75adc0-Benedict's+Newsletter_COPY_01&utm_medium=email&utm_term=0_4999ca107f-4f0e75adc0-70802521). Especially slide 40, "I just want to get this out of my way"
It'd prefer to change this ticket to, More seamlessly/ui-integrated onboarding.


Binary Mask

It is possible to get back a binary mask (only 0 and 255 values) instead of a grayscale?

You can either threshold the grayscale map or replace the last conv layer to have 2-channel output and the loss function from BCE to CE (not tested).

Will the result be the same?

No, I suggest you try the 1st option first as the 2nd one requires re-training the model and some of the hyper-parameters may need to be adjusted accordingly.

I tried the 1st option, but the results are not always according to the original (tried different thresholds), sometimes the mask is bigger than the grayscale mask and sometimes smaller.
I tried also the 2nd option, but this is to complicated for me, I'm just a novice.
Thanks for your help anyway


Fix duplicated diagnostic issue

Bug/issue #, if applicable:
Close #729 and rdar://79991171
Summary
If the range’s source is not matched with the resolver’s source, we should not emit a problem
Dependencies
None
Testing
See func testDuplicatedDiagnosticForExtensionFile() in MarkupReferenceResolverTests.swift
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@swift-ci please test

@swift-ci please test

This looks good to me. @binamaniar can you take a second look at this while I'm out?

@swift-ci please test

Ping @binamaniar

This looks good to me. @binamaniar can you take a second look at this while I'm out?

@d-ronnqvist I'd like to confirm whether we should wait for binamaniar's approval before the merging.

sorry for the late reply. Looks good to me to make sure there is no crash when the location is not in source. Just thinking do we need to need an extra check that it is for extension files only. My answer is probably no but sharing my thought process.

Just thinking do we need to need an extra check that it is for extension files only.

There is not too much context to get the info(whether it is an extension file) now.
[By checking the extension of the source is "md"? 🤔]
We may consider adding the extra check later if needed. Anyway, thanks for the review suggestion.

@swift-ci please test


⚠ Jenkins Plugins Artifactory has degraded performance

In d7f6d85, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1230 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 0b48b6f after 20 minutes.


Need help, program don't works

Hello,
I can't get your program to work and the documentation is incomplete.
I've already opened an issue but it didn't help.
Can you just list the steps required (commands) to deploy a smart contract, on a katana node for example?

What does it even mean that it doesn't work? Don't expect proper answers without proper questions.
https://github.com/xJonathanLEI/starkli-ci-test/actions

I'm trying to deploy the example contract of cairo book: https://book.cairo-lang.org/ch99-01-04-01-voting-contract.html

Can you explain me the error please?


move uaa.yml comments into separate file to reduce startup time of integration tests by 20 seconds

Remove the comments in uaa.yml (place them in separate file) to cut the 20 second integration test startup time.

Can you recreate the branch off of predix_extensions_4.10.1 and then recreate the PR?

wrong branch


HubiC

I believe this qualifies as object storage and should be listed. They have implemented the OpenStack API.
https://api.hubic.com/

Sounds good -- could you submit a pull request?

I don't think it should be listed. HubiC prohibits commercial use.  Max 10Mbps communications etc.  It's not a "real" object storage.

T&C's are in french https://hubic.com/fr/contracts/Contrat_hubiC_2014.pdf

I just tested their Hubic with rclone. Hopelessly unreliable. I have to agree not worth listing.


[IRGen] Support movesAsLike for the array variant of @_rawLayout

This is similar to https://github.com/swiftlang/swift/pull/73955, but for the likeArrayOf variant. Note that all elements within the array must be initialized.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please smoke test Linux


#2198 [Order form] Missing error message after entering invalid data.

Environment: Chrome 108, Windows 10
Reproducible: always
Preconditions

Go to: https://www.testgreencity.ga/#/ubs
Log in with ADMIN credentials.

In"Name" field enter: "Alina90".
Check the results

Actual result
No Error message appear
Expected result
Error message appear: "Only letters and hyphen, space, apostrophe are allowed".
Scenario 2

Go to: "Personal data" tab.
In"Surname" field enter:"Kropka21".
Check the results

Actual result
No Error message appear
Expected result
Error message appear: "Only letters and hyphen, space, apostrophe are allowed".
User story and test case links
E.g.: "User story #2198
Test case"
Attachment:

According to #5719, the user can fill in numeric characters after the first and last name
The administrator is not able to create an order like usual user


Document run-time SceneTree debug property changes not working correctly

master version of https://github.com/godotengine/godot/pull/64390.
See https://github.com/godotengine/godot/issues/64353.

Thank you! 🎉


Comment about sshd is truncated in the sample config.yaml file

https://github.com/slackhq/nebula/blob/2ea360e5e25d77b969b5b4f5e6a5c1a2affec843/examples/config.yml#L145
The truncated comment makes it unclear what is the purpose of the following setting.
Could some one elaborate, please?

see https://nebula.defined.net/docs/config/sshd/


Make hardware work collectively

[ ] Code the hardware to function to its best capabilities up until this part of the project.
[ ] Adjust hardware and software to function well.

Incorrectly made issues. Disregard the description above.
In this part, we must check the "liveness" of the part using photo/schematic, code and/or instructions, expected result of the "liveness" test.

See whiteboard from 2/27

I think either my part is dead or the connection is not proper. Gonna have to recheck it ASAP.

The issue has been resolved. IR distance sensor is live. Getting expected output as described in the datasheet. I'll update the post with a video tutorial to check the liveness.


Brazilian Portuguese (pt_BR) translation

Thanks for your awesome work :smile:

@fgmacedo , let me know if you're working on more translations changes. If not, I'll bump a micro version and upload the changes to PyPI.

@v1k45 Only the model's translation is missing. But before translation, we need to provide verbose_name to then. And this is only wordy if we put the model into Admin (so we can see the field names).
Is putting the Notification model on admin in your plans? If so, we can add verbose_name into field definitions, so they can be translated.

I've thought of the adding Notification to the admin interface, but from the use case perspective, admin will only create site-wide notifications. Because dealing with arbitrary content types and editing so many field is quite a mess.
Also, about the model details, i think some of the model fields will be removed in future updates for supporting notification concatenation. We should provide necessary details after the new changes.

I think that having the Notification model on Admin can help users in order the inspect if they are being generated accordingly, as a query tool. I agree with you that there's no use case for create/update on admin.

Yep, that's completely a valid case. I think we should add a minimal support to at least list notifications in a proper way.


Issue #2: Certain requests fail with a 400 Bad Request

writeBytes(String) does not work properly with Unicode characters and may produce invalid UTF-8 - replacing it with an alternative call that converts the JSON string to a proper UTF-8 byte array representation and sends that

Looks like there is an import missing in a file in this PR. I had just pushed an updated, but you already had merged.
How about setting up a GitHub action to build PRs?

Yeah I saw that an import was missing, and I updated it immediately.
About setting up a GitHub action to build PRs, could you point me to some documentation/example? TBH, GitHub actions is pretty new to me since I started using it only recently. 😆 Any sort of help/directions would be really helpful!

Your current GH action builds the main branch and then publishes the SNAPSHOT to Maven Central.
You may want to set up a second GH action that does not run on push but only on pull_request and which does not have the step that publishes to Maven Central (i.e. without Set up Apache Maven Central (Overwrite settings.xml) and Publish to GitHub Packages Apache Maven).
While we are at it: Java 8 is quite old now. You might consider switching to a Java 11 baseline which would allow using the new HttpClient instead of having to rely on the Apache HTTP Client and/or the plain URL class.

Thanks for the info! So, this second GH action would be run upon merging a PR which would just run the tests ideally. Did I understand that right?
About Java 8, I thought of using the later versions but then I was wondering if there's someone who'd be relying on the older versions it might help them. What's your take on this?

On pull_request means that the build is running when the PR is opened or updated. That way you can check if contributions contain e.g. compile errors before merging them.
Wrt Java 8 - there may still be people using it, but in general there is now a move to a Java 17 baseline (at least in that part of the Java world where I live) - so I would consider a Java 11 baseline still quite conservative. For example the Spring framework is a pretty large player in the Java space and it requires Java 17 it its latest version (Spring 6).
Even if you don't move to Java 11, you might want to consider using Apache HTTP client consistently (i.e. replace usages of the plain URL/URLConnection with calls to HTTP client) - having that extra (powerful) dependency and then not using it consistently seems strange. At the point where you switch to e.g. Java 11 (LTS), that extra dependency could be dropped. Having fewer dependencies also has some benefits for a library.

Yeah, that makes sense. I'll have a look at the GH actions for PRs.
About the Java version, I've updated all the Java 8 references to 11 now and I've updated the Apache HTTP client to java.net.http.HttpClient. I've removed the Apache HTTP client dependency as well. :)
Thanks for the pointers!