Cannot use space in search bar; typing input very slow [Bible] ...

Extension
https://github.com/raycast/extensions/tree/aae0455d73bf770ec0c78eed27186793208deded/extensions/bible/
Description
Just recently noticed typing is very slow when "searching while typing" is enabled. That hasn't been the case for me in the past. Nothing on my macOS has changed. Also, when I use the space bar on the keyboard, the search box does not accept spaces.
Using macOS Monterrey 12.6.1 with Raycast v1.44.0.
Steps To Reproduce

Open Raycast extension with searching while typing enabled.
Type in the search bar.
Notice slow typing input.
Notice no spaces being accepted in the search bar.

Current Behaviour
Slow input reception of characters. Search bar not accepting spaces.
Expected Behaviour
Search bar should be able to accept characters at a normal typing response; should accept spaces.

@aaronjz thanks for reporting, I've noticed this too recently... Nothing has changed in my code, so some update to Raycast must be affecting it. I'll look it into it when I have some time, hopefully this weekend.
Cheers!


2024-03-02 18:21:23 1:M 02 Mar 2024 09:21:23.051 * Background saving started by pid 60
2024-03-02 18:21:23 60:C 02 Mar 2024 09:21:23.060 * DB saved on disk
2024-03-02 18:21:23 60:C 02 Mar 2024 09:21:23.061 * Fork CoW for RDB: current 0 MB, peak 0 MB, average 0 MB
2024-03-02 18:21:23 1:M 02 Mar 2024 09:21:23.152 * Background saving terminated with success
2024-03-02 18:22:24 1:M 02 Mar 2024 09:22:24.020 * 100 changes in 60 seconds. Saving...
2024-03-02 18:22:24 1:M 02 Mar 2024 09:22:24.020 * Background saving started by pid 61
2024-03-02 18:22:24 61:C 02 Mar 2024 09:22:24.024 * DB saved on disk
2024-03-02 18:22:24 61:C 02 Mar 2024 09:22:24.025 * Fork CoW for RDB: current 0 MB, peak 0 MB, average 0 MB
2024-03-02 18:22:24 1:M 02 Mar 2024 09:22:24.121 * Background saving terminated with success

cvat_redis_ondisk
2024-03-02 17:29:42 I20240302 08:29:42.021214     7 main.cc:141] kvrocks version 2.7.0 (commit 33b49591)
2024-03-02 17:29:42 I20240302 08:29:42.049878     7 storage.cc:335] [storage] Success to load the data from disk: 22 ms
2024-03-02 17:29:42 I20240302 08:29:42.051579     7 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
2024-03-02 17:29:42 I20240302 08:29:42.051762     7 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
2024-03-02 17:29:42 I20240302 08:29:42.051868     7 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
2024-03-02 17:29:42 I20240302 08:29:42.051962     7 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
2024-03-02 17:29:42 I20240302 08:29:42.052052     7 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
2024-03-02 17:29:42 I20240302 08:29:42.052138     7 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
2024-03-02 17:29:42 I20240302 08:29:42.052225     7 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
2024-03-02 17:29:42 I20240302 08:29:42.052310     7 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
2024-03-02 17:29:42 I20240302 08:29:42.053093     7 worker.cc:555] [worker] Thread #140294348946224 started
2024-03-02 17:29:42 I20240302 08:29:42.053150     7 worker.cc:555] [worker] Thread #140294348770096 started
2024-03-02 17:29:42 I20240302 08:29:42.053189     7 worker.cc:555] [worker] Thread #140294348593968 started
2024-03-02 17:29:42 I20240302 08:29:42.053225     7 worker.cc:555] [worker] Thread #140294348073776 started

Any input would be greatly appreciated since I'm still learning about the various tech stacks used here and am a newbie!
Thank you!
Environment
Windows 11 WSL2 Ubuntu 18.04 Python 3.10

The error in cvat_opa can be solved by stoping the container and then running it like this
docker run -d --rm --name cvat_opa_debug -p 8181:8181 -v ${PWD}/cvat/apps/iam/rules:/rules openpolicyagent/opa:0.45.0-rootless run --server --set=decision_logs.console=true /rules
and it would be great that someone looked into this i got this error on wsl windows 10 and then on ubuntu 22.04 both.

@adkbbx Hi It is not clear from your logs what database you are trying to connect. The development guide only describes how to use postgres inside the cvat_db container. I have tested the instructions in a clean environment and can confirm that everything works as expected (wsl2, ubuntu 22.04).
According to your last comments, the database is installed on your host system, so you have to properly configure it yourself and create a default database (cvat), default user (root) and accept connections without password, or you have to configure django to connect to your postgres using your database credentials.
But it seems the easiest way is to follow the recommendations and use the db in a container. If you want the host database and the container database to be running at the same time, you can set different port for the cvat database here https://github.com/opencv/cvat/blob/develop/docker-compose.dev.yml#L11. and set the env variable CVAT_POSTGRES_PORT for django

@azhavoro Thank you for testing the code and verifying its working even I feel the error is at my end in the way I have set up the database which I am currently tring to configure. Since I am a little new to Django I am having a little touble understanding the configuration of Django and Postgresql so that I can set up the database. Do let me know if there any log files that I can share which can help you understand where I am going wrong.
Thank you for you patience and guidance!

@nmanovic @azhavoro
I tried to clean rebuild the environment by purging all the containers and images was able to sucessfully setup the development environment following the  guide  and solve the error.
Thank you for helping me out with the error.


Remove version: Freeplane.Freeplane version 1.11.14

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


Update StORM-Saving-Retrieving-and-Deleting-Rows.md

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


feat!: redirect home

Set user home to /opt/containerbase/home

#2078

⚠ Additional commit message:
BREAKING CHANGE: The path `/home/<username>` is now a symlink to `/opt/containerbase/home`.

won't do


Added pyenv-offline-installer for offline installation.

#28 related.

Ops, sorry for my carelessness. Working on it.

Following code detects what kind of shell in which the script is running. I think most people would prefer to the automatically shell detection, instead of passing in the identifier of shell manually. So, could I just ignore such argument(the shell) and keeps <path-to-archive> as the only input arugment?
shell="$1"
if [ -z "$shell" ]; then
  shell="$(ps c -p "$PPID" -o 'ucomm=' 2>/dev/null || true)"
  shell="${shell##-}"
  shell="${shell%% *}"
  shell="$(basename "${shell:-$SHELL}")"
fi

Umm. Basically I think it'd be better if the caller if user can override $shell configurations.
How do you think keep reading archive path from environment variable if specified, otherwise read it from stdin?

What about $ PYENV_SHELL=zsh ./pyenv-offline-installer /path/to/archive.tgz ?

Argument handling should be common between online and offline installer. Sorry for confusion but just passing archive file via environment variable might be suitable for this....

get it.


Outline Manager using base 10 instead of base 2 to calculate data usage

Describe the bug
On the Outline Manager dashboard, the displayed used data and data limits in GB are calculated based on totalBytes / (1000*1000*1000) and not the proper conversion of totalBytes / (1024*1024*1024) .
As a result, if an access key has an data limit of an actual 100GB, or 100 * (1024 * 1024 * 1024) = 107,374,182,400 bytes and is updated through Outline API,  on the dashboard, its data limit mistakenly is shown as 107.37 GB, because it uses this conversion: 107,374,182,400 / (1000 * 1000 * 1000)
To Reproduce

Create an access key by using the Outline API using libraries, such as: https://github.com/jadolg/outline-vpn-api, and add a data limit:

from outline_vpn.outline_vpn import OutlineVPN
Outline_API_URL = "<OutlineAPIKey>"
client = OutlineVPN(api_url=Outline_API_URL)
accessKeyName = input("Enter the name of the Access Key: ")
quota_to_increase = int(input("Enter the amount of GB you want to assign to this Access Key: "))
new_key = client.create_key(key_name=accessKeyName) #create Access Key
assert client.add_data_limit(new_key.key_id, quota_to_increase * (1024 * 1024 * 1024)) #Assign Quota in Bytes

Now, close the Outline Manager, re-open, and check the dashboard again. The access key has the wrong displayed data limit, calculated based on <quota_to_increase> / (1000 * 1000 * 1000).

Similarly, as a result of this wrong conversion, the access keys created directly from Outline Manager, and not the API, are assigned with wrong bytes in their data limits too.
For instance, an access key that was decided to have 100 GB of data limit, is now have an actual 93 GB of data limit or:
100 * (1000 * 1000 * 1000) = 100,000,000,000 bytes / (1024 * 1024 * 1024 ) = 93.13 GB
Expected behavior
To display the correct conversion of data in GB in the Outline Manager Dashboard.
Screenshots

Client System (please complete the following information):

Outline Manager on Windows

Submit Feedback
N/A
Additional context
None

Will this be fixed?

The use of the decimal system is the preferred way to interpret GB. See https://en.m.wikipedia.org/wiki/Byte#Multiple-byte_units.
The unit you are asking for is Gibibyte (GiB). We do not intend to use that. This is working as intended. Perhaps we can clarify it somewhere.


Inquiry about the version of pytorch lightning

Thank you for your excellent work!  I've tried switching between a few versions of pytorch lightning, but I'm encountering conflicts here or there.   I wonder what version is being used in the code?

I tried the version in requirements.txt( i.e. 1.5.4) too and there are still some conflicts

Here are some error lines encountered in different versions :
`
parser = pl.Trainer.add_argparse_args(parser)
return [opt_ae, opt_disc], [{"scheduler": sch_ae, "interval": "step"}, {"scheduler": sch_disc, "interval": "step"}]
#or [{"optimizer": opt_ae, "scheduler": sch_ae, "interval": "step"}, {"optimizer": opt_disc, "scheduler": sch_disc, "interval": "step"}]
sch1, sch2 = self.lr_schedulers()
`

I use pl == 1.5.4, it seems to work well for me.


Revert "Add checks of dedicatedCPUPlacement to validation webhook"

/area instancetype
/cc @opokornyy
/cc @davidvossel
What this PR does / why we need it:
This reverts commit 48a2adea3eec3aa267fdaa7139c8e74ca06fb07a.
Thanks to e7898af39b54298043adef02ae578d825743120e [1][2] we now apply defaults to a copy of the VirtualMachineInstanceSpec before running the associated validations in the VirtualMachine validation webhook. This actually resolves the original issue with enabling dedicatedCPUPlacement from instance types [3] as it ensures any guest visible memory is expressed as resources requests before we attempt to validate.
As such we can drop the blocking validations from the instance type webhooks and allow dedicatedCPUPlacement to be set once again.
[1] https://github.com/kubevirt/kubevirt/pull/9103
[2] https://github.com/kubevirt/kubevirt/issues/9102
[3] https://github.com/kubevirt/kubevirt/issues/8867
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
With this commit reverted in a test environment the following examples work once again:
$ ./cluster-up/kubectl.sh apply -f - << EOF
apiVersion: instancetype.kubevirt.io/v1alpha2
kind: VirtualMachineClusterInstancetype
metadata:
 name: dedicatedcpuplacement
spec:
 cpu:
  dedicatedCPUPlacement: true
  guest: 1
 memory:
  guest: 128Mi
EOF
selecting docker as container runtime
virtualmachineclusterinstancetype.instancetype.kubevirt.io/dedicatedcpuplacement created
$ ./cluster-up/kubectl.sh apply -f - << EOF
apiVersion: kubevirt.io/v1
kind: VirtualMachine
metadata:
  labels:
    kubevirt.io/vm: vm-cirros
  name: vm-cirros
spec:
  instancetype:
    name: dedicatedcpuplacement
  running: true
  template:
    spec:
      domain:
        devices: {}
      terminationGracePeriodSeconds: 0
      volumes:
      - containerDisk:
          image: registry:5000/kubevirt/cirros-container-disk-demo:devel
        name: containerdisk
EOF
selecting docker as container runtime
virtualmachine.kubevirt.io/vm-cirros configured

Release note:
The `dedicatedCPUPlacement` attribute is once again supported within the `VirtualMachineInstancetype` and `VirtualMachineClusterInstancetype` CRDs after a recent bugfix improved `VirtualMachine` validations, ensuring defaults are applied before any attempt to validate.

/retest

/cherry-pick release-0.59

in general this makes sense. Can we get some test coverage of dedicated cpu placement with instance types now? That feature is complex enough that I'd like to ensure it is both working and continues to work as expected with instance types moving forward

ACK added in the commit below, thanks @davidvossel !
https://github.com/kubevirt/kubevirt/pull/9169/commits/fa66a7a6249190eef2d1b2e4ac2a73f533335e2b

/retest-required
https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/9169/pull-kubevirt-e2e-k8s-1.25-sig-compute/1623751431183929344

>  Tests Suite: [crit:medium][vendor:cnv-qe@redhat.com][level:component][sig-compute] Instancetype and Preferences instance type with dedicatedCPUPlacement enabled should be accepted and result in running VirtualMachineInstance
26s

Looks like that is passing in the compute lanes FWIW.

/retest-required


Get instance name of panZoom component

Hello!
How can I retrieve/set the instance name of the panzoom component? I would like to pause panning but do now know how to access the instance.
From the original package:
var element = document.getElementById('scene');
var instance = panzoom(element);

instance.isPaused(); //  returns false
instance.pause();    //  Pauses event handling
instance.isPaused(); //  returns true now
instance.resume();   //  Resume panzoom
instance.isPaused(); //  returns false again

You have 2 options
Option 1 (using the init event):
The instance is available as the first argument in the "init" event callback like so:
In your template
<panZoom @init="onInit"></panZoom>

Add this to your component's methods
onInit: function(panzoomInstance, id) {
    panzoomInstance.isPaused();
}

Option 2 (using ref)
The instance is available as $panZoomInstance on the panZoom component instance. Here's how it should look:
In your template
<panZoom ref="panZoomItem"></panZoom>

Then you can do this in your script
this.$refs['panZoomItem'].$panZoomInstance.isPaused()

You have 2 options
Option 1 (using the init event):
The instance is available as the first argument in the "init" event callback like so:
In your template
<panZoom @init="onInit"></panZoom>

Add this to your component's methods
onInit: function(panzoomInstance, id) {
    panzoomInstance.isPaused();
}

Option 2 (using ref):
The instance is available as $panZoomInstance on the panZoom component instance. Here's how it should look:
In your template
<panZoom ref="panZoomItem"></panZoom>

Then you can do this in your script
this.$refs['panZoomItem'].$panZoomInstance.isPaused()

@mistabiggx Thank you for providing this. I was able to pause panning with Option 2 :)


Adds owners placeholder

Adds a placeholder which gets the current owner of a team. If there are multiple owners then they are all returned separated by commas. Closes #537

%betterteams_owners% - Gets the owner(s) for the team that the viewing player is a member of
%betterteams_static_owners_Supers% - Gets the owner(s) for the team "Supers"

I just realized #537 mentions a list for admins and members as well. I'll add those later.

Thanks for all the pull requests, it is incredibly helpful. I will have a proper look hopefully tomorrow (and hopefully finally publish the update).
One thing I noticed from a quick glance is that you could instead use. It would not have any performance difference as it does the same code in the backend, it just may be worth using the utility method in case for some reason something changes in the future.
 List<TeamPlayer> users = team.getRank(PlayerRank.OWNER);

As seen here:
https://github.com/booksaw/BetterTeams/blob/7e507ffce087e64e01c4e03d5b6132e539aa0d26/src/main/java/com/booksaw/betterTeams/commands/team/InfoCommand.java#L86

Thanks and sorry as I did not see the method before, but I should have.


Change coffee to json. And add progressbar

I like it, also I was a bit uncomfortable with the coffee-script, but my code was basically a fork as well. So leaving that fork for this seams like a good approach, nice work, need to test this before merging to master.

Hi @timbrandin this PR is actually accident :) I want to PR to my own forked repo. Because I added something that suit my needs (session, progressbar, etc). But I will be thrilled if you merge the PR :+1:


How to ignore Iphone mute switch

I want when the iphone mute switch is turned on keep music play
Is there any solution?  btw  i use  web audio api  play music
iphone 6
QQ browser

There is a way to unlock audio from the Web Audio API while the iPhone/iPad ringer switch is off. The trick is to play a short bit of silence from an HTML5 audio tag. This "unlocks" the audio for that browser window and lets the Web Audio API audio come through.
Here's a gist with the code I use:
https://gist.github.com/banjerluke/b8dd69000d9d874d52602e222fdf6646

@banjerluke  thank you , it's working ,  you made my day ,


Adds encrypt/decrypt to CLI

Codecov Report
Patch coverage has no change and project coverage change: +0.02 :tada:

Comparison is base (5737ac2) 73.31% compared to head (1a2a10e) 73.34%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
+ Coverage   73.31%   73.34%   +0.02%
==========================================
  Files          93       93
  Lines       10916    10916
==========================================
+ Hits         8003     8006       +3
+ Misses       2913     2910       -3

Impacted Files
Coverage Δ

navajo-cli/src/cli.rs
83.75% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Palindrome String Using Stack

🚀 Feature
Determine if a string is palindrome whose center is marked with 'X'.
Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
To add more than one way to determine if a string is palindrome in C++.
Pitch
If you are given an array of characters formed with a’s and b’s. The string is marked with special character X which represents the middle of the list (for example: ababa...ababXbabab baaa) and you have to  check whether the string is palindrome or not using stack.

/assign


[css] word rule should include '-'

.five {
  background-color: var(--main-bg-color);
}

Double click background-color or --main-bg-color. This should select the full identifer.

Closing as invalid. The selection on doubleclicking is configured in the 'editor.wordSeparators' setting.


[Feature Request] Full resolution SoundCloud artwork download

[x] I've verified and I assure that I'm running youtube-dl 2<IP_ADDRESS>
[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Feature request (request for a new functionality)

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included:

Example track: https://soundcloud.com/jinkamusic/ghost2u
The regular thumbnail: https://i1.sndcdn.com/artworks-000472040832-c39wes-t500x500.jpg
Its full resolution version: https://i1.sndcdn.com/artworks-000472040832-c39wes-original.jpg

SoundCloud provides for most of its tracks a high resolution artwork
And it's extremely easy to access! You simply need to replace the -t500x500 part of the url with original or in some very rare cases with original size.
There already exists a script that adds a download button for every artwork (https://greasyfork.org/en/scripts/14591-soundcloud-album-art-downloader), however there is no way to automatise it.
An additional handle to force download full/original size artwork would simplify the process.

I was just about to make this request, thanks for putting it in!
I too would really like to see this added.

This is already possible, by using the --embed-thumbnail  flag, and editing the thumbnail url:
--- a/youtube_dl/extractor/soundcloud.py
+++ b/youtube_dl/extractor/soundcloud.py
@@ -218,7 +218,7 @@ class SoundcloudIE(InfoExtractor):
             self.report_extraction(name)
         thumbnail = info.get('artwork_url') or info.get('user', {}).get('avatar_url')
         if isinstance(thumbnail, compat_str):
-            thumbnail = thumbnail.replace('-large', '-t500x500')
+            thumbnail = thumbnail.replace('-large', '-original')
         username = try_get(info, lambda x: x['user']['username'], compat_str)

         def extract_count(key):

However, I am not sure if it should be made permanent, I would prefer an option for choosing the thumbnail quality.

@AwesomeAwesomeAwesome @RomanKarwacik:
I would also prefer to choose between original / 500x500 size. The original is too large for my requirements, hence significantly increases the file size.
Is there some easy way to specify the dimensions of the thumbnail in youtube-dl?

@AwesomeAwesomeAwesome @RomanKarwacik:
I would also prefer to choose between original / 500x500 size. The original is too large for my requirements, hence significantly increases the file size.
Is there some easy way to specify the dimensions of the thumbnail in youtube-dl?

Problem solved.
It simply required to modify a single line of soundcloud.py:
elif image_id == 'original': into elif image_id == 't500x500':


Patch SDK version in pipelines during release

I'd like to propose this change to finally fix and vastly simplify the relase pipeline.
The idea is that during a release the pipeline will:

replace a static string in package.json with release version
build and push the code to npmjs.org

but it will not:

create a commit 3.0.0 (or any other  version) with pacakge.json update
push that commit to main branch

So in other words, local code will always be versioned with this fake 0.0.0-development version, but when the sdk gets uploaded to npmjs it will have a proper version injected into package.json, just as before.
This is a quite satisfactory tradeoff for having a versioned release compared to current approach with pushing the commit back to main branch, because:

we're obligated to push the commit safely:

to do so, we need to manage credentials safely, meaning we need to use a bot
but the dedicated bot (pushbot) does not fit here, because it has security restrictions
and these can't be altered for the safety of the company's internal repositories
meaning we'd have to create another dedicated bot just to push the code in this repo

The only disadvantage I can see is that there will be no more commits with version.
IMO that's not a real problem, because the latest 'released' commit still has a git tag pointing to it, so it wasn't needed anyway.
Please let me know what you think about this.

I like that change
I don't understand what are the pros of this version commit to main branch, but the cons are pretty obvious – extra complexity in pipeline, additional bots with highly elevated credentials (It might be used to push commits to main branch without review in some cases by attacker)
So it's a good change from my POV


Allow active connections based scaling using KEDA

The underlying nodes on Azure support 500K flows. If we translate this into connections; it will be around 2,50,000. If we are using websocket based application with AKS; then this connection limit per node might breach; at the same time CPU and memory may not increase significantly.
Therefore please provide a way to scale nodes/ pods if active number of connections being served by pod/ node reaches to threshold stated in settings.

The ask is to scale based on used connections on nodes so that applications can be scaled out accordingly, if I'm not mistaken @kunalchandratre1?
If so, how would you envision this assuming KEDA does not know how many connections are being used on the nodes?


Can you run a condition based on a trigger that has a wildcard?

For example my triggers:
trigger:

master
releases/*

I want a step to only run if the PR is in the releases* branch. I see examples that specify a particular branch but wanted to check to see if conditions allow wildcards.
Thanks.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3f151218-9a11-0078-e038-f96198a76143
Version Independent ID: 09c4d032-62f3-d97c-79d7-6fbfd89910e9
Content: Conditional tasks - Azure Pipelines
Content Source: docs/pipelines/process/conditions.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Looks like you can use the startsWith condition and use the SourceBranch with the folder name. Example was given, I just needed to read better I guess :) This can be closed.

You may use $(Build.Reason)  -eq "Pull Request"


Module.sow() should fail if collection provided as input

It's very easy to accidentally mis-use flax.linen.Module.sow() by calling apply() and feeding the intermediate values back in. This results in ever growing:
class Module(nn.Module):
  @nn.compact
  def __call__(self, x):
    self.sow('intermediates', 'x', x)
    return x

x = jnp.array(1.)
mod = Module()
variables = mod.init(jax.random.PRNGKey(0), x)
for i in range(3):
  _, mod_vars = mod.apply(variables, x, mutable='intermediates')  # recompiles!
  variables = variables.copy(mod_vars)

len(variables['intermediates']['x'])  # 4 copies !

Note that appending values is desired behaviour because the same module can be called multiple times and we would want to recover all the values, not only the last value. Note also that variables = variables.copy(mod_vars) is a relatively common pattern with mutable variable collections.
Ideally Module.sow() would detect if any variable is passed in via .Module.apply() and then fail instead of silently appending values.

Tagging @jheek who implemented this and may also have an opinion on it

Feeding back the intermediates is not always a problem:
For example if you use apply inside another module then feeding back the intermediates is probably the right thing to do.
In general you could have a finite number of apply calls that you want to track something on with sow.
Also in the current sow api appending is the default behaviour but users can override this. For example you could also use sow to track a running average or a sum in which case you probably want to feed the sowed values back in.

I don't understand this issue actually. @andsteing and @jheek, you both seem to agree that appending is the intended behavior, and the docstring is pretty clear about this as well. To me it also doesn't seem wrong to append the intermediate values if existing ones are found, I can imagine users might want to see the history of all intermediates.
So given that we all agree with this, what is the issue?

oh I think I should have closed this after #1228


Making sure to use proper luxon parser

If the backstage is using postgres the [newly added prometheus metrics functionality]
(https://github.com/backstage/backstage/pull/6848/files#) generate warnings

2766b2a: Add experimental Prometheus metrics instrumentation to the catalog

catalog warn Processing failed with: Value is not a valid number: {} type=plugin stack=TypeError: Value is not a valid number: {}
    at /Users/Valerii.Udodov/src/catalogue/node_modules/prom-client/lib/summary.js:160:10
    at Summary.observe (/Users/Valerii.Udodov/src/catalogue/node_modules/prom-client/lib/summary.js:47:54)
    at processTask (/Users/Valerii.Udodov/src/catalogue/node_modules/@backstage/plugin-catalog-backend/dist/index.cjs.js:3149:45)
    at /Users/Valerii.Udodov/src/catalogue/node_modules/@backstage/plugin-catalog-backend/dist/index.cjs.js:3036:7
    at Array.forEach (<anonymous>)
    at maybeLoadMore (/Users/Valerii.Udodov/src/catalogue/node_modules/@backstage/plugin-catalog-backend/dist/index.cjs.js:3035:17)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)

Appears that nextUpdateAt is not guaranteed to be a string 🤷 .
In case of postgres it is a JS object
typeof item.nextUpdateAt
'object'

Adding a simple check to use corresponding luxon parser.

[x] A changeset describing the change and affected packages. (more info)
[ ] ~Added or updated documentation~
[ ] ~Tests for new functionality and regression tests for bug fixes~
[ ] ~Screenshots attached (for UI changes)~
[x] All your commits have a Signed-off-by line in the message. (more info)

Super-seeded as this is solved by #6901. Thanks for a quick PR @vudodov!


meta_scopes will be an array of arrays

https://github.com/adobe/aio-apps-action/blob/ff82a685e3e8c913b8e748f1fe397961619d2648/index.js#L149
Since parsedScopes or the default scope will be an array. Currently fixing this issue in #48
Can you confirm my understanding here @MichaelGoberling, @sandeep-paliwal ? Thanks

@shazron
Yes this is correct. Just tested the unit-test branch here (default scopes) and here (configured scopes) to confirm


What happens to the predefined Release Variables when we migrate to a multi-stage yaml pipeline?

I suspect the answer is that they don't get populated.  If that is the case, it would be good for this documentation to state that and perhaps provide some advice on what they may map to so we can refactor our code as appropriate.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 611642f0-881f-10b9-ae94-3c562001d1f0
Version Independent ID: 41efb94b-4c7a-64a3-b23b-70c8634ae727
Content: Release variables and debugging - Azure Pipelines
Content Source: docs/pipelines/release/variables.md
Product: devops
Technology: devops-cicd
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

In my experience, it seems that they do not get populated, and DevOps doesn't warn you about using deprecated variables in the release, so migrating my scripts to the multi-stage pipelines has been pretty painful.

This is correct, none of the Release.* variables are populated when you switch to YAML CD pipelines.


fix(deps): update icr.io/cpopen/ibm-operator-catalog:v4.16 docker digest to 1726944

This PR contains the following updates:

Package
Update
Change

icr.io/cpopen/ibm-operator-catalog
digest
5e9d449 -> 1726944

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.4.44 :tada:
The release is available on:

GitHub release
v1.4.44

Your semantic-release bot :package::rocket:


Add a membership section to the home page design

People need to be able to sign up to be a Cooperation Hull member on the website. We need a section on the homepage which has some copy and links to the sign up page

I also need to update the copy for the sign up form

How we think about our membership has changed, so it's not going to be on the home page. We just need to make sure there's a link in the nav bar.
I've written some new copy for the sign up form in Figma


[refactor] always include the zero-score authorities as well

Description
This PR is modifying the current ReputationScores implementation so it always includes all the authorities - even the zero score ones.
Test Plan
Unit tests

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[x] necessitate either a data wipe or data migration

Release notes

IIUC the goal is stronger validations?

Yes, and apart from that to ensure that down stream components in SUI will have the full set of validators and their scores without having to perform any assumptions on the absence of validators (ex the score is zero in this case). Then they case just iterate safely over the results and make sure they have the full view.


[BUG] unable to resolve class com.microsoft.playwright.options.AriaRole

System info

Playwright Version: v1.35
Operating System: Windows 10
Browser: Chromium
Other info: Groovy inside Java

Source code
Test file (self-contained)
import com.microsoft.playwright.Browser
import com.microsoft.playwright.BrowserType
import com.microsoft.playwright.Page
import com.microsoft.playwright.Playwright
import com.microsoft.playwright.impl.*
import com.microsoft.playwright.Page.GetByRoleOptions
import com.microsoft.playwright.options.AriaRole

Playwright playwright = Playwright.create()
Browser browser = playwright.chromium().launch(new BrowserType.LaunchOptions().setHeadless(false).setSlowMo(50).setChromiumSandbox(true).setTimeout(30000))
Page page = browser.newPage()
page.navigate("http://my-test-site")

result = page.getByRole(AriaRole.CELL, new Page.GetByRoleOptions().setName("Name Please enter a valid name")).getByRole("paragraph")
println result.textContent()

Steps

Run the test

Expected
Test runs
Actual
unable to resolve class com.microsoft.playwright.options.AriaRole

I resolved this, it was an IDE bug.


Option to "autodial" VPN

Hi, would be nice to have "autodial" VPN option in iOS app. What you think? It is pretty anoying to dial VPN manually. Autodetect of external network and if you are out, dial VPN or open VPN setting to dial manually?

Interesting ideas. I assume you are referring to VPN autoconnect. VPN configuration would be done outside of the app but somehow the app would read info from KeyChain and use it to establish a VPN connection automatically. Is this what you had in mind?
Some links that would help implement this:

http://blog.moatazthenervous.com/create-a-key-chain-for-apples-vpn/
http://blog.moatazthenervous.com/create-a-vpn-connection-with-apple-swift/

Yeah... exactly. VPN profile will be set in native iOS settings and app will just connect using this VPN connection.

I think the issue here may be that the Cisco app also sets up the VPN (OpenVPN does the same, as does the new <IP_ADDRESS> app by Cloudflare and other VPN apps) so it's also able to connect to it.
I'm not entirely sure but it may be just beyond the reach of any given app to access a VPN set up by another app.
Reference: https://developer.apple.com/documentation/networkextension/nevpnmanager

So, we must redefine feature request to the "Add VPN support"? 💃

I'd love this too. Outside my internal network I forget to connect to my VPN and am forced to go to settings/VPN and activate a profile. It would be great if a printer can be associated with a VPN profile and automatically switch to that VPN. Or even better a switch within Octopod that lets me activate and deactivate a profile.

Hi Gaston, could you please review this feature request again? Looks like iOS finally supporting VPN on demand!
https://developer.apple.com/documentation/networkextension/personal_vpn/vpn_on_demand_rules

Hi Gaston, could you please review this feature request again? Looks like iOS finally supporting VPN on demand!
https://developer.apple.com/documentation/networkextension/personal_vpn/vpn_on_demand_rules

Thanks @KissmanCZ for sharing that useful info. Quickly read that link and it seems like I would need to add a new setting so that users can configure when they want the VPN to start. By default it would be when not in Wifi only.

Thanks @KissmanCZ for sharing that useful info. Quickly read that link and it seems like I would need to add a new setting so that users can configure when they want the VPN to start. By default it would be when not in Wifi only.

@gdombiak If you're looking to prioritize enhancement requests, I'm a +1 on this one.😁


badge typo

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 50.00%. Comparing base (67ea39f) to head (5703aa2).

:white_check_mark: All tests successful. No failed tests found.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   50.00%   50.00%
=======================================
  Files           3        3
  Lines          22       22
=======================================
  Hits           11       11
  Misses         11       11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Prediction multiple results

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Is it possible to get the Predict to return multiple paragraph results?
Describe the solution you'd like
A clear and concise description of what you want to happen.
Basically, have the predict return 3 to 5 best results.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Currently, the predict return the best result.
Additional context
Add any other context or screenshots about the feature request here.

Hi @JimAva, your issue is related to some questions asked by @SuyashSrivastavaDel in #237
You can refer to my answer in that issue.


[lsp] importId completion candidate should include keys of tsconfig.compilerOptions.paths

Suggestion

🔍 Search Terms

List of keywords you searched for before creating this issue. Write them down here so that others can find this suggestion more easily and help provide feedback.

lsp
editor
completion
ESM
import
importId
tsconfig compilerOptions paths

✅ Viability Checklist

My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

importId completion candidate should include keys of tsconfig.compilerOptions.paths
📃 Motivating Example

Currrently, given

tsconfig

    "paths": {
      "foo": ["./path/to/foo"],
      "bar": ["./path/to/bar"],

package.json

  "dependencies": {
    "hello": "1.0.0",
    "world": "1.0.0",

editor typing

import {} from "<Cursor></Cursor>"

Actually: get completion candidate

hello
world

💻 Use Cases

Not sure whether tsconfig.compilerOptions.paths have higher piority than packageJson.XXXDependencies.
If it does
Expected: get completion candidate

foo
bar
hello
world

If it does not
Expected: get completion candidate

hello
world
foo
bar

Related
https://www.typescriptlang.org/tsconfig#paths
https://github.com/microsoft/TypeScript/issues/43326

The actual contents on disk of all these folders matters. Can you set up a sample repo?

@RyanCavanaugh

The actual contents on disk of all these folders matters. Can you set up a sample repo?

Emm, not folders, I use tsconfig.compilerOptions.paths like importmap.
    "paths": {
      "foo": ["./path/to/foo.mts"],
      "bar": ["./path/to/bar.mts"],
      "importId": ["./path/to/file.mts"]

Background:

I use deno + importmap for a while, and feel this way impressively neat and elegant, compared to node.js + package.json + monorepo + packageManager.

But some library is not yet well supported in deno, so I try to use node + tsconfig.compilerOptions.paths as deno + importmap workaround.

node + tsconfig.compilerOptions.paths has problem, importId completion candidate does not include keys of tsconfig.compilerOptions.paths.


Expose the fn execute_with

Phala's ink contract needs some extra setup. Like below code:
        #[drink::test]
        fn deploy_and_call_http_get() -> Result<(), Box<dyn std::error::Error>> {
            use drink_pink_runtime::PinkRuntime;
            let checker_bundle = BundleProvider::local()?;
            let mut session = Session::<PinkRuntime>::new()?;
            session.execute_with(|| {
                PinkRuntime::setup_cluster().expect("Failed to setup cluster");
            });
            let checker =
                session.deploy_bundle(checker_bundle, "default", NO_ARGS, vec![], None)?;
            let (status, _body): (u16, String) = session.call_with_address(
                checker.clone(),
                "http_get",
                &["\"https://httpbin.org/get\""],
                None,
            )??;
            assert_eq!(status, 200);
     }

So, is it OK to export the function execute_with?

however, please forgive me nitpicking, but I'd rather move this to ChainApi so that you can access this functionality from a Session object indirectly through chain_api() method

Sounds good. I've moved it to ChainApi.


Fix YAML load recognition

Closes #31

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
168

Relevant Lines:
168

💛  - Coveralls


Why suggest using a Windows Insider build if the audience is "begginers"

I like WSL, but suggesting a beta OS build and an unreleased WSL2 build for beginners doesn't seem to match the audience of the article

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 811fa359-feec-10dd-544c-881982b25138
Version Independent ID: aeb2c4e1-28d2-65c8-06e7-b63a3233c5e2
Content: Get started using NodeJS on Windows for beginners
Content Source: hub/nodejs/beginners.md
Product: dev-environment
Technology: windows-nodejs
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

In my opinion, we need to start beginners off on the right foot... from the beginning. If we have them install Node.js directly on Windows, the initial set up is simplified, but the tools, workflow, ability to implement the commands found out in the wild via courses and tutorials... it's all slightly different for a direct Windows install... thus the frustration we've heard from folks using Node on Windows in the past. The WSL path gets them on the right foot from the get-go.  Adding @fiveisprime, @shanselman, @indierawk2k2, @yodurr, for their insight on this.
For a bit of context, @nschonni is responding to my request to add a link on the Node.js downloads page: https://github.com/nodejs/nodejs.org/issues/2684
Thanks all!

After some further conversations, agree with your assessment @nschonni and have updated this page to represent the more straightforward install path directly on Windows. Thanks for your input and contribution. ~Matt

Updated in https://github.com/MicrosoftDocs/windows-uwp-pr/pull/515
Closing.

👍 thanks, looks good


  I need to stop doing this because it is an extra layer of complexity. E.g I would like to use http2 push for the auth and /rest/meetings requests that are destined to happened with each load of / and yet I don't feel like figuring out how to do this (or if it works) through a second proxy.
Note to self: once I close this issue, revisit http2 push for those routes

Closed by #fa1dcdf5caa549db84a1d0e0bf67945c09a87a4b


Missing redirect for /hashtag/

https://fxtwitter.com/hashtag/Trump gets redirected to the github page

Added a generic redirect for /hashtag


Adopt copy-less APIs in the VFS

This mostly required changes to the MemoryHandler trait in order to allow direct access to the WebAssembly's linear memory space. This turned out to be surprisingly complicated because of Rust's borrow checker and differences between Wasmi's and Wasmtime's linear memory APIs (though this does mean this code should be portable to other execution engines).
I've tried to document the code thoroughly as this borders on what's possible in Rust. And considering the amount of transmutes I needed I may be doing something terribly wrong, but this was the best solution I could come up with to provide safe access to engine memories safely. I'm open to other ideas if there are any.

Added get_slice and get_slice_mut, which return associated types Slice and SliceMut. These represent slices of the underlying linear memory, but can other contexts that may be required by the execution engine such as Mutex/RefCell guards, Rcs, or even a Vec if the underlying engine doesn't actually have a linear memory.
There's a bit more trickiness here due to lifetimes.
Slice and SliceMut share the lifetime of the linear memory they take a slice of. The proper way to describe this in Rust would be like this:
trait MemoryHandler {
    type Slice<'a>: AsRef<[u8]>;
    fn get_slice<'a>(&'a self) -> Slice<'a>;
}

But this requires a nightly, and unfinished, feature called GATs (the type Slice<'a>: AsRef<[u8]> part specifically):
https://github.com/rust-lang/rust/issues/44265
The workaround here is to require the associated-type implementations to all be "lifetime-less", which probably requires unsafe code in the implementations.
To keep this API from being just completely unsafe (we could just pass around pointers). I've added a new type called Bound for the sole purpose of binding an unrelated lifetime to a lifetime-less object. Whether or not the lifetime is valid can't be checked by the compiler of course, but this at least means any consuming code will be borrow-checked correctly:
trait MemoryHandler {
    type Slice: AsRef<[u8]>;
    fn get_slice<'a>(&'a self) -> Bound<'a, Slice>;
}

Added unpack_iovec and unpack_iovec_mut which return sets of these slices. This allows iovecs to be passed all the way from the WebAssembly application to the internals of the filesystem, allowing multiple iovecs to be consumed in a single filesystem operation.
This also handles the splitting of the linear memory's mutable borrow, which is difficult to do otherwise.

Required MemoryHandler::Slice and MemoryHandler::SliceMut to be movable, even behind a borrow. This is a very very unusual trait requirement but prevents the need for unnecessary boxing in the hot-path of file read/writes. Fortunately it holds true for most of the types you would use in this API, such as slices and even vecs.

Added small-object optimization (SOO) for 1 and 2 element iovecs (these are by far the most common from personal experience, I've never actually seen a >2 iovec used before, though it's certainly possible in hand-optimized code)

The end result is zero allocations in the hot-path of file reads/writes, aside from the internal allocations required for the filesystem's internal data-structures.
This results in a significant performance gain, mostly due to no longer copying the read/write buffers multiple times before the data reaches the filesystem.

@ShaleXIONG, @mathias-arm are you happy with this?

+1+1 = +1 reached, merging.


tetragon: Move matchBinaries filter to be executed earlier

The matchBinary filter des not have any selector record, so we can do that as a first thing in probe.
Signed-off-by: Jiri Olsa<EMAIL_ADDRESS>

PR https://github.com/cilium/tetragon/pull/774 gets in the way, moving back to draft ;-)

PR #774 gets in the way, moving back to draft ;-)

#774 was just merged :)

checkpatch fails on BIT macro warning, it's not a problem

There seem to be some 4.19 failures:
❌	pkg.sensors.tracing.TestLoadKprobeSensor			10.127s	(7m16.004s)
...
❌	pkg.sensors.tracing.TestLoadTracepointSensor			10.949s	(8m17.18s)
❌	pkg.sensors.tracing.TestUprobeLoad				9.418s	(8m26.598s)
Are these caused by this patch?

There seem to be some 4.19 failures: x pkg.sensors.tracing.TestLoadKprobeSensor 10.127s (7m16.004s) ... x pkg.sensors.tracing.TestLoadTracepointSensor 10.949s (8m17.18s) x pkg.sensors.tracing.TestUprobeLoad 9.418s (8m26.598s)
Are these caused by this patch?

yea, I just had to rebase, will fix that


[6.1] Update related dependencies to release/6.1

https://github.com/swiftlang/swift-package-manager/pull/8157
https://github.com/swiftlang/swift-format/pull/888
https://github.com/swiftlang/sourcekit-lsp/pull/1850
@swift-ci Please test


Provide count of records in the table

An often requested enhancement for the ExB Table widget was to provide a count of features. https://devtopia.esri.com/Beijing-R-D-Center/ExperienceBuilder-Web-Extensions/issues/13179
I think we may also want to do the same. We also had this in CM 3.x.

Other examples:

@jmhauck any suggestions on how/if we can also display that information?

@avezina the FeatureTable does support this but feels weird with our design

Could look at adding to the layer picker but it also feels off:

or

We can certainly display that stuff but where is the main question.

@jmhauck Would something like this be an option then?

@avezina should see this in lower left as the last example above

@jmhauck can we add a little bit of padding,

@jmhauck Tinkered with it a bit. I suggest:

Can we add this at the top right corner of the table? Since we are thinking about the buttons to go in the overflow menu, will this be causing any issue?

@avezina  We can change the words and separator if that is preferred but have already accounted for the general format issue:
"{{total}} records, {{selected}} selected"

@jmhauck Looked into getting some extra clarification around this, here's the full explanation on why we should use that specific format

Adding a new observation: EXB opted to place their count below the table:

Perhaps we should do the same; placing it above the scroll bar may be a challenge due to variability in scroll bar width between browsers.

@azizaparveen @chris-fox What do you think about placing it below the table and allowing for a small amount of padding there?

@avezina ready to test tomorrow

@jmhauck this looks so much better, can we have a tiny bit of padding on top and left of the count?

@azizaparveen was trying to keep it as small as possible since all the space to the right is wasted. If I pull it down slightly is that enough to make it not looked scrunched or want more?

@jmhauck this looks good to me

@azizaparveen not until tomorrow

@jmhauck looks better, but I think we should still add a little padding to the left

Left padding looks good in devext


Better analytics of event logs

We've encountered an issue where an excessive amount of INTROSPECTION calls were made to the service. This particular call is resource intensive and resulted in service degradation.
There should be a way to keep track of these calls as they come to create alerts or some other form of automation. Ideally this will help us identify clients that are misbehaving so that we can reliably offer our service.

removed  introspection. will revisit as needed by clients


About results in Table 2

Hi,
One more question, just to make sure，the results in Table 2 are obtained by timing score with 100, is that right?

Yes, that is correct!

Yes, that is correct.
You can also run scripts/read_results.py to cross-reference the results with the numbers in the paper. The multiplication by 100 happens on this line.


Failed to apply plugin [class 'com.google.gms.googleservices.GoogleServicesPlugin']    > For input string: "+"

I have installed "cordova-plugin-firebase": "^1.0.0"
I am getting an error while building the app.
ionic cordova build --release android

FAILURE: Build failed with an exception.

* Where:
Script 'C:\src\app\platforms\android\cordova-support-google-services\in-build.gradle' line: 16

* What went wrong:
A problem occurred evaluating root project 'android'.
> Failed to apply plugin [class 'com.google.gms.googleservices.GoogleServicesPlugin']
   >

BUILD FAILED

Total time: 2.196 secs
For input string: "+"

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
(node:5292) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: cmd: Command failed with exit code 1 Error output:
FAILURE: Build failed with an exception.

* Where:
Script 'C:\src\app\platforms\android\cordova-support-google-services\in-build.gradle' line: 16

* What went wrong:
A problem occurred evaluating root project 'android'.
> Failed to apply plugin [class 'com.google.gms.googleservices.GoogleServicesPlugin']
   > For input string: "+"

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
Picked up _JAVA_OPTIONS: -Xmx512M
(node:5292) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

@sagarkul the error doesn't seem related to this plugin.  Try installing https://github.com/dpa99c/cordova-android-support-gradle-release or https://github.com/dpa99c/cordova-android-play-services-gradle-release to fix the gradle build.

The problem was conflict with OneSignal package.
I removed OneSignal and completely switched to firebase for notification..
Also removed Firebase Analytics package.

tenho o mesmo problema usando o ionic 3:

Where:
Script 'C:\Users\claudinei\projetos\risa-app\risa-app-novo\platforms\android\cordova-support-google-services\risashop-build.gradle' line: 16

What went wrong:
A problem occurred evaluating project ':app'.

Failed to apply plugin [class 'com.google.gms.googleservices.GoogleServicesPlugin']
For input string: "+"

my ambient:
Ionic:
ionic (Ionic CLI)  : 4.10.3 (C:\Users\claudinei\AppData\Roaming\npm\node_modules\ionic)
Ionic Framework    : ionic-angular 3.9.2
@ionic/app-scripts : 3.2.3
Cordova:
cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>Cordova Platforms     : android 7.1.4, browser 5.0.4
Cordova Plugins       : cordova-plugin-ionic-webview 1.2.1, (and 17 other plugins)
System:
NodeJS : v10.15.1 (C:\Program Files\nodejs\node.exe)
npm    : 6.4.1
OS     : Windows 10


[Maps Search] Update names based on APIView feedback for Preview 1.0

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[x] To improve swagger for generated clients and address some issues from this review in ApiView

When are you targeting to deploy the new service/feature to public regions? Services already deployed.
When do you expect to publish the swagger? ASAP
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published. No data-plane SDKs are onboarded yet, so no refresh is required.

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further questions about AME onboarding or validation tools, please view the FAQ.
Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[x] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.

Hi @tjprescott This PR was mean to be the updated of the swagger following by the feedback you gave, mostly are name changes for the parameters.
For rest of the issues such as Coordinate type, optional params, etc.. I believe those will be done in the convenience layer for the SDK.


Some issues with type annotations

Some issues with type annotations that were brought up by @Bvb93 at https://github.com/numpy/numpy/pull/18585

For asarray, it isn't clear how to define NestedSequence (typing annotations don't support recursive types).
asarray needs to also include array as a valid type for obj (array should not be considred a subtype of NestedSequence)
The pipe operator for types isn't valid yet (https://www.python.org/dev/peps/pep-0604/) (it's used in the NestedSequence type in asarray)
For stack and concat, the type for arrays should be Tuple[array, ...] instead of Tuple[array]
For stack and concat (and possibly others), Tuple is too restrictive vs. Sequence.
unique can be implemented as an @overload (see https://github.com/numpy/numpy/pull/18585#discussion_r591757472). I don't know if this needs to be changed in the spec.
The return type for finfo as finfo doesn't make sense if finfo is a function (see https://github.com/numpy/numpy/pull/18585#discussion_r591763368).
__getitem__ and __setitem__ should include the ellipsis type (https://github.com/numpy/numpy/pull/18585#discussion_r592256384)
__len__ should return int
The return type for shape is currently Union[ Tuple[ int, …], <shape> ], which needs to be addressed (there is a TODO in the spec for this).
Some of the other types for properties also use <...> types. We should make all the type annotations be valid Python/mypy that can be used in the annotations in the signature.
Should the array object subclass from typing.Protocol (https://github.com/numpy/numpy/pull/18585#discussion_r592261549)? (I don't know if this is relevant for the spec).

Thanks for the great feedback @BvB93, and for collecting it here @asmeurer. I'll try and fix the straightforward ones now.

The pipe operator for types isn't valid yet (https://www.python.org/dev/peps/pep-0604/) (it's used in the NestedSequence type in asarray)

It's valid in Python 3.8 with from __future__ import annotations. And py38 is our minimum version because of positional-only parameters, so we're fine here.

unique can be implemented as an @overload (see numpy/numpy#18585 (comment)). I don't know if this needs to be changed in the spec.

No, I don't think that should be changed in the spec, that would just make it harder to read. It's an implementation issue.

The return type for finfo as finfo doesn't make sense if finfo is a function (see numpy/numpy#18585 (comment)).

The return type isn't relevant, it must be an object with the minimum properties. So the suggestion of using a typing.Protocol is a good one. I'll think about the best way to include that in the standard.

Some of the other types for properties also use <...> types. We should make all the type annotations be valid Python/mypy that can be used in the annotations in the signature.

No, this is on purpose. We cannot have valid types for library-specific classes/objects. <array> means "the conforming array type in the library". We can add a typing.Protocol in the NumPy implementation, and that will then very likely get reused in other libraries. But in a document that doesn't make sense.

Should the array object subclass from typing.Protocol (numpy/numpy#18585 (comment))? (I don't know if this is relevant for the spec).

Same answer, yes in implementation, no in spec.

For asarray, it isn't clear how to define NestedSequence (typing annotations don't support recursive types).

Also an implementation issue. I'd simply type it as Sequence[Any] for now, given that there's no canonical way to do it. Note that for NumPy, there's already numpy.typing._NestedSequence which has a similar issue.

asarray needs to also include array as a valid type for obj (array should not be considred a subtype of NestedSequence)

Already covered by SupportsDLPack, but can add explicitly and specify it's a no-op in that case.

For stack and concat (and possibly others), Tuple is too restrictive vs. Sequence.

I'm actually not sure Sequence is a good idea. It can be quite hard to deal with this correctly in both Python and C/C++, see e.g. https://stackoverflow.com/questions/43566044/what-is-pythons-sequence-protocol. I remember a lot of cases where we special-cased tuple and list inputs, and other kinds of sequences were doing the wrong thing.
I can be convinced that accepting Union[Tuple[<array>, ...], List[<array>]] is a good idea, but I don't think Sequence is.

The return type for finfo as finfo doesn't make sense if finfo is a function (see numpy/numpy#18585 (comment)).

The return type isn't relevant, it must be an object with the minimum properties. So the suggestion of using a typing.Protocol is a good one. I'll think about the best way to include that in the standard.

The annotation wasn't finfo, it was <class>. To further clarify I changed it to <finfo object>.

Continuing from https://github.com/numpy/numpy/pull/18585#discussion_r671555487:

Well I've also been trying to update the type hints in the spec itself concurrently with my updates on this PR based on your feedback, so this is still a relevant question. Should we change the type of stream in the spec from Optional[int, Any] to just Optional[Any]? I admit I'm still a bit confused by Any. The Python docs state "A special kind of type is Any. A static type checker will treat every type as being compatible with Any and Any as being compatible with every type.", so wouldn't Union[int, Any] or indeed Optional[Any] just be equivalent to just Any as far as type checking is concerned? It seems to me that the only point of Optional[int, Any] would be to document that None and int have special meaning, but based on what you've stated here, this is perhaps just me misunderstanding how Any works with respect to covariance and contravariance.

@asmeurer when used as parameter type, all unions involving Any are indeed equivalent to a Any (from a typing perspective). I made a similar comment in https://github.com/data-apis/array-api/pull/185#discussion_r638201753, but the more verbose union was preferred then for the sake of clarity.

I'm also a bit confused by the Any type for this specific dlpack signature. Is it really Any, or should it be an unspecified  type, similar to what is used for device? Perhaps we should move this discussion over to data-apis/array-api#143.

The __dlpack__ protocol is currently in a unique situation as the parameter type of stream is variable, i.e. different concrete implementations are allowed to specify a bound (as long as this bound involves None somehow). This is similar to, e.g., builtin pythons' __getitem__ protocol, where both the key- and return-type are variables.
So no, the most correct type for the more general __dlpack__ spec would not be Any, it would be a typevar: None | ~T. For concrete implementations would narrow down to the like of None (like for numpy), None | int or
None | int | FancyStreamType.
Examples
An example of __dlpack__ represented as a typing.Protocol subclass.
from typing import TypeVar, Protocol, Any

PyCapsule = Any  # TODO
_T_contra = TypeVar("_T_contra", contravariant=True)

class SupportsDLPack(Protocol[_T_contra]):
    def __dlpack__(self, /, *, stream: None | _T_contra = ...) -> PyCapsule: ...

Two problems recently discovered in https://github.com/numpy/numpy/pull/19969:

[ ] Annotations of variadic parameters apply each argument, e.g. *args: Sequence[Array] implies that every individual argument must be a sequence of arrays. I strongly suspect that, in most cases, this does not represent the actual intention (e.g. result_type).
[ ] The array API's spec for eye currently accepts k is None, a value that is not accepted by np.eye. As was noted by @asmeurer, no mention is made None outside of the Optional[int] annotation, so it's not 100% clear if this was intentional or merely an oversight.

As all items in this tracking issue have been addressed, will close. Any further issues can be discussed in a new issue and resolved in follow-up PRs.


npm run watch high CPU usage on macOS

Steps to Reproduce:

Run npm run watch in the vscode repo on macOS.
Wait for everything to finish Finished compilation with 0 errors after 58528 ms
Look at your CPU usage in activity monitor

Expected: Minimal CPU usage
Actual: gulp consistently consuming 40% of one core.

I did some brief research and found quite a few threads relating to gulp and watching files. Here are some of the more interesting ones:

https://github.com/gulpjs/gulp/issues/1805
https://www.martin-brennan.com/gulp-watch-high-cpu-usage/

We are not in charge of gulp in this repository, please file it in the gulp one.

If you read https://github.com/gulpjs/gulp/issues/1805 you will see that many many people have opened this issue upstream and they all get closed/locked because the gulp team considers this issue fixed.
There are a few options listed in https://github.com/gulpjs/gulp/issues/1805 and we just need to implement one of them in the vscode repo.
Please reopen.

You don't have the natively compiled file watcher. @joaomoreno knows more

Thanks for the hint.
brew install fswatch seems to have fixed the issue for me.
It would be nice if ./scripts/npm.sh install hinted at needing this module on macOS.

./scripts/npm.sh install eventually calls npm install inside the build/lib/watch directory, for non windows systems:
https://github.com/Microsoft/vscode/blob/master/build/npm/preinstall.js#L20
This will install gulp-watch, a better replacement to gulp.watch. Please verify that that is installed in build/lib/watch/node_modules, inside your repo.
I'm not really sure why brew install fswatch fixed it for you but I'm glad it did.


currentTime

Expected behaviour
Actual behaviour
Steps to reproduce
Environment

Browser:
Version:
Operating System:
Version:

Console errors (if any)
Link to where the bug is happening

Created this by mistake :(


Add zshipko to allowlist

Hi @zshipko follow the instructions from https://ci.ocamllabs.io/getting-started to get ocaml-ci installed into your GitHub user. Please let us know if you have any problems or suggestions for features.


dcard update: slow to prepare posts_to_update

I've found it quite slow for dcard update to start, due to this part https://github.com/disinfoRG/ZeroScraper/blob/master/newsSpiders/runner/update.py#L67-L71, where  the program needs to retrieve the last_comment_floor for every post before beginning to update them. Testing on my laptop, it takes ~0.9 seconds to prepare each post. Since ns.py update waits for each spider to be ready to start the update process, I think we need to find a way to speed up this process. (Currently there are ~2700 dcard posts wait to be updated, meaning it'd take ~40 minutes to finish preparing posts_to_update)

after testing on middle2, the speed turned out to be really fast (15 seconds for 1547 posts), closing this issue.


Feature/datastudio feed

First try. Getting the private key from secrets has not yet been tested, need to test that on aws once deployed. When getting it from a file, the feed does was it should. Datasheet IDs might need to be updated, these are test sheets.

The cron setup is a bit hacky. I need it to run as close to midnight as possible and all runs should be 24hours apart. That is the best approach I could come up with.

The cron setup is a bit hacky. I need it to run as close to midnight as possible and all runs should be 24hours apart. That is the best approach I could come up with.

hm not sure what the approach is, you just run it every minute?


[DNM] CI validation of #470

This validates that #470 does indeed deploy the code that's being tested.

/retest

/retest


VAD installation error on Raspi4

Hi, I have the same installation error like the issue #64
OS is Dietpi V9.x ARMv8-Bookworm, which is 64bit (?)!
The same OS runs on two Raspi3b+ where I have installed a satellite already.
Are there some other prerequisites on Raspi4?

Okay, I've flashed the Raspi with a fresh image, setup the Audio-HAT and Snapcast again to be sure the main function works and then I was able to install the Wyoming-Satellite without trouble.
It seems one of the earlier packages I have installed to the Raspy (e.g. HA-Bridge,) causes the error, although I've tried to purge everything...


feat!: simplify take row api

Change take_rows and take APIs to allow directly take Schema again.

Windows failure is not relevant

Codecov Report
Attention: Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review.

Project coverage is 79.44%. Comparing base (8c2667d) to head (34340d8).

Files
Patch %
Lines

rust/lance/src/dataset.rs
92.30%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2664      +/-   ##
==========================================
+ Coverage   79.41%   79.44%   +0.02%
==========================================
  Files         223      223
  Lines       65082    65085       +3
  Branches    65082    65085       +3
==========================================
+ Hits        51686    51705      +19
  Misses      10396    10396
+ Partials     3000     2984      -16

Flag
Coverage Δ

unittests
79.44% <92.30%> (+0.02%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks for doing this. I like this interface a lot better.


Windows 10 / Go 1.7 build issue (missing vendored dependency)

Trying to build (go run make.go) on Windows fails:
> go run make.go
Now rebuilding gopherjs at C:\Storage\Work\Personal\camlistore\camlistore\tmp\build-gopath-nosqlite\bin\gopherjs.exe
error building gopherjs: error while building gopherjs: exit status 1, ..\..\spf13\cobra\command_win.go:9:2: cannot find package "github.com/inconshreveable/mousetrap" in any of: A
        C:\Storage\Work\Personal\camlistore\camlistore\tmp\build-gopath-nosqlite\src\camlistore.org\vendor\github.com\inconshreveable\mousetrap (vendor tree)
        C:\Program Files (x86)\Go\src\github.com\inconshreveable\mousetrap (from $GOROOT)
        C:\Storage\Work\Personal\camlistore\camlistore\tmp\build-gopath-nosqlite\src\github.com\inconshreveable\mousetrap (from $GOPATH)
exit status 1

Looks like there's a missing vendored dependency: "github.com/inconshreveable/mousetrap". If I put the module manually into the appropriate folder, it successfully goes past this failure point.

Looks like this might be related to #838. Feel free to close if it is so.

the windows part is probably covered by https://camlistore-review.googlesource.com/6956
and yes, the Go 1.7 part is issue #838
I hope to take care of all this soon.

Considering:

What github.com/inconshrevable/mousetrap is meant for: helping users double-clicking on the gopherjs binary, which I think we don't care about.
That our vendored copy of gopherjs is already hacked to not need fsnotify as dependency, because we don't care about gopherjs serve

I'm tempted to also just hack our vendored github.com/spf13/cobra so that it doesn't need mousetrap.
@tajtiattila @bradfitz Any objection ?

Totally agree.

No objections. Let me know if you want me to do it.

@tajtiattila sure, feel free to give it a go if that tickles your fancy.

With the build tags (see https://camlistore-review.googlesource.com/#/c/6956/) changed gopherjs compiles.
The go:generate in app/publisher/js/main.go uses a shell script to generate zsearch.go, this will not work on windows. The workaround is to run the script manully (if one has bash, eg. msys).
After that the errors with go1.7 on Windows are the same as on Linux, see issue #838
$ go run make.go
gopherjs for publisher error: exit status 1, tmp\build-gopath-nosqlite\src\github.com\gopherjs\gopherjs\compiler\natives\runtime\runtime.go:11:14: TheGoos not declared by package sys
..\..\Go\src\runtime\error.go:70:9: invalid operation: e._type (variable of type *struct{_string *string}) has no field or method string
exit status 1
$

The go:generate in app/publisher/js/main.go uses a shell script to generate zsearch.go, this will not work on windows. The workaround is to run the script manully (if one has bash, eg. msys).

If you're about gensearchtypes.sh, then I'd like to suggest another workaround. Windows users who have Git installed usually have sh.exe executable which could run the script, but the script is called as ./gensearchtypes.sh, and Windows has no idea how to 'execute' it. A simple solution would be to place a file called gensearchtypes.sh.bat next to gensearchtypes.sh, which would contain this one line: @sh gensearchtypes.sh. Now, when the outer program tries to run ./gensearchtypes.sh, Windows will find the file with .bat extenstion and run it. This will allow Windows folks to run installation from a vanilla console.

@iafan Nice trick, thanks for the tip. But I'm not sure sh.exe is always added to the PATH when installing Git for Windows.
Perhaps it would be nicer to rewrite gensearchtypes.sh in Go.

Yes, sh.exe is located in the same directory as git.exe, and by default Git updates %PATH%. But yes, I agree, this one-liner batch file can only be treated as a temporary hack.

I believe it is only $PROGRAMFILES/Git/cmd that is always added to the PATH, not $PROGRAMFILES/Git/bin where sh.exe und bash.exe resides. I have Git for Windows installed, but the only sh.exe where finds is the one that came with MSYS on my system.

the zsearch.go is another story, right? so let's talk about it in another issue please.
@tajtiattila I was a bit surprised that you didn't have to modify gopherjs itself for the mousetrap dep, but I suppose it makes sense if the only thing it does is catch a mouse double-click. Just to be sure, have you tried your CL with go1.6.x, so we can be sure your CL fixes the issue, and make.go succeeds without the Go1.7 specific issues?

AFAICT the sole purpose of mousetrap is to show a message when an app that should only be used from the CLI is double clicked in Windows Explorer. So removing this from the vendored github.com/spf13/cobra alone fixes the issue.
go run make.go -all with the updated CL works for me with Go 1.6.3, even when vendor/github.com/inconshreveable/mousetrap is removed.

@tajtiattila I just checked — it depends on installation options: by default it's only Git that is in the PATH, and if one chooses "Use Git and optional Unix tools", then sh will be added to the PATH as well.
I also just realized that the generated zsearch.go file seems to be empty apart from the static header. This is what I get when running sh gensearchtypes.sh manually:
doc: no such package camlistore.org/pkg/search.SearchResult
exit status 1
doc: no such package camlistore.org/pkg/search.SearchResultBlob
exit status 1
doc: no such package camlistore.org/pkg/search.DescribeResponse
exit status 1
doc: no such package camlistore.org/pkg/search.MetaMap
exit status 1
doc: no such package camlistore.org/pkg/search.DescribedBlob
exit status 1
doc: no such package camlistore.org/pkg/search.DescribedPermanode
exit status 1

Does it work for you?

@iafan Yes, that is what I was worried about, the availability of sh depends on the installation options. I used it from MSYS bash, (cd app/publisher/js && ./gensearchtypes.sh) works with both 1.6.3 and 1.7 for me.
But @mpl have asked to open an issue for the gensearchtypes.sh topic earlier.

8c30e24b77bd3122f3653633b7d165c00c17d015


Update upload download artifacts to v4

Description

Provide details of the change, and generalize the change in the PR title above.

Update the GitHub actions for upload and download artifacts to v4.

Testing

Describe how you've tested these changes. Link any manually triggered Integration tests or CPP binary SDK Packaging Github Action workflows, if applicable.

Packaging Run: https://github.com/firebase/firebase-cpp-sdk/actions/runs/11733979480
Test Run: https://github.com/firebase/firebase-cpp-sdk/actions/runs/11736048885

Type of Change
Place an x the applicable box:

[ ] Bug fix. Add the issue # below if applicable.
[ ] New feature. A non-breaking change which adds functionality.
[x] Other, such as a build process or documentation change.

Notes

Bug fixes and feature changes require an update to the Release Notes section of release_build_files/readme.md.
Read the contribution guidelines CONTRIBUTING.md.
Changes to the public API require an internal API review. If you'd like to help us make Firebase APIs better, please propose your change in a feature request so that we can discuss it together.

Since the logs artifact is now in multiple files, does the reporting script need to be updated?

Yep, done. Testing it is a pain, but I tried via debug logging against some test runs, and it seems good.


Make local dev work again

This fixes the server build, are there any issues with the console?

everything seems to be working fine, assuming env-less builds


Add HTML columns classes to support a predefined set of column widths, gap sizes and column rule styles.

Completed.


Make AdminPageChooser use models instead of ContentTypes

This removes the need to have to perform a query in order to initialise an AdminPageChooser object
Fixes #2433

Closing in favour of #2438 as it's clearly much better :)


Stop execute cell doesn't want to work

Originally reported in https://github.com/microsoft/vscode/issues/115083
This may be because I don't have the python kernel installed. It should error out and stop itself immediately if so.
Also can stop be instant regardless? Do we need to wait for feedback to stop as that will cause the UI to feel sluggish, a better approach generally imo is to trust it will stop and present some error if something failed.

This is by design. Stopping a cell sends an interrupt to Jupyter. Jupyter is not guaranteed to interrupt/stop the cell. See this issue:
ipython/ipython#3400
If we immediately stopped the cell and didn't show it being still busy, when you ran another cell you'd be wondering why it wasn't running (the kernel would still be busy from the original stop).


[nonelab] Copied glyphs can't do everything until being inserted into a font

From Victor Gaultney:
"I’ve been using fontParts to write some scripts with general success so far. However I’m finding that BaseGlyph.copy() doesn’t create what I expect. The docs say, for example, that a new glyph object is created, and that things like unicodes, contours, components are copied. I can also use a pen to draw into that object, and even modify things like unicodes. I can use font.setitem(name,newglyphobject) to insert the copied and modified glyph into the font. All good.
However some other things don’t work. I can’t get copyofglyph.bounds (TypeError: 'NoneType' object is not subscriptable) . If I try copyofglyph.decompose() I get an error that it’s not iterable. Those work fine if I insert the copied glyph object into the font and then use those methods on the inserted glyph."

One old way to fix some of the mentioned issues was to setParent of the glyph to the font in Robofab but it's gone in FontParts. I guess we could have a copy of a glyph only for some operations that are not destructive to the actual glyph and if that glyph is a composite then it needs to have a parent or set of glyphs for decomposing the components. This copied glyph doesn't need to be inserted in the font, it's only in the memory. What issues this would cause that made it removed?

I think the glyph.layer property is the replacement of the setParent() method. However, I can't get that to work:
from fontParts.world import OpenFont

f = OpenFont("font.ufo")
g = f["agrave"]
g2 = g.copy()
g2.layer = g.layer
g2.decompose()

TypeError: argument of type 'NoneType' is not iterable

it does not work cause the naked objects has no layer set... and the decomposing in fe defcon happens elsewhere.
I would recommend to have a copy pen without components:
from fontParts.world import OpenFont, RGlyph
from fontTools.pens.basePen import BasePen

f = OpenFont("font.ufo")
g = f["agrave"]
g2 = RGlyph()

class MyCopyDecomposingPen(BasePen):

    def __init__(self, glyphSet, outPen):
        super(MyCopyPen, self).__init__(glyphSet)
        self._moveTo = outPen.moveTo
        self._lineTo = outPen.lineTo
        self._curveToOne = outPen.curveTo
        self._closePath = outPen.closePath
        self._endPath = outPen.endPath

pen = MyCopyDecomposingPen(f, g2.getPen())
g.draw(pen)
print(g2.bounds)

or use pen to calculate the bounds, where you can provide any kind of glyphSet:
from fontTools.pens.boundsPen import BoundsPen

pen = BoundsPen(f)
g2 = g.copy()
g2.draw(pen)
print(pen.bounds)

it does not work cause the naked objects has no layer set... and the decomposing in fe defcon happens elsewhere.

How is that not a bug?

its a bug :)
only decomposing should happen in fontParts instead of leaving it to the implementation (on fontshell its defcon)

this one: https://github.com/robotools/fontParts/blob/master/Lib/fontParts/base/component.py#L322-L326 is for the implementation
this one does almost nothing towards the implementation: https://github.com/robotools/fontParts/blob/master/Lib/fontParts/base/glyph.py#L116-L117

It's a bug, PRs welcome!


[docs] add types for error load function

related to https://github.com/sveltejs/kit/issues/2712
Pure doc change.
This makes the error load description consistent with the description for load.
Seems that we're not cluttering the docs with TypeScript for now, since we don't have code tabs to make it neat.  JSDoc types are helpful though.
Before submitting the PR, please make sure you do the following

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

thank you!


Add Statement.bindParameters() that accepts an array of values

I purpose to allow  Statement.bindParam() to accept an array of values instead of array of arrays like here.
This should not be difficult we already implement similar behavior for DBPool.prepareExecute().
That way you can use this:
  await statement.bindParam([9997, 'Johnson', 'A J', 453 Example','Fort']);

rather than this:
statement.bindParam([
    [9997, IN, NUMERIC],
    ['Johnson', IN, CHAR],
    ['A J', IN, CHAR],
    ['453 Example', IN, CHAR],
    ['Fort', IN, CHAR],
  ]);

This would be better if it were implemented within idb-connector directly rather than here.
opened https://github.com/IBM/nodejs-idb-connector/issues/111


Restart - Doesn't read the new parameters

Restart - Doesn't read the new parameters.
Eg. I modified number of slots and restart doesn't read the new parameters.
was necessary to stop the server and then restart

Is it possible to put the "stop button" here?

Hi Luis,
"Restart" not changing things is actually how we intend for it to work. Restart literally just restarts the assetto process. It doesn't change config files or do anything else. If you want to start an event after you have edited it, you must click 'Start Race' again and it will set it up with the new options you have changed.
As for putting a stop button in, this would be a lot of work for something that already exists in the server process management. We would have to detect which of the current events is running and then only show the stop button on the page for that event, which I think would also look messy too.
Closing this as working as intended.


Doc update for the main README.md

This PR follows the previous PR Doc fix #72 and updates the installation guide and the list of further documentation in the main readme.
One thing we want to decide is what to do with the overlapping area (e.g., installation and activation) between the main readme and the /docs/getting_started.md

We can take the intro part from getting_started.md, or we take a shorter / high-level version of it. I would vote for the second option, so that the homepage can have a quick intro while the detailed docs can have more background info/notes.

We can take the intro part from getting_started.md, or we take a shorter / high-level version of it. I would vote for the second option, so that the homepage can have a quick intro while the detailed docs can have more background info/notes.

I agree with the second option.  I can think of two ways to do it:

I remove the installation and activation section from the main readme or make it very brief.  And add a "please refer to" link to point to getting_started.md for detail.
I keep the current main readme as is.  Add more detail in the the getting_started.md.

I am more inclined to the first way.


[CHECK] Adapter xs1 needs verification

Adapter xs1 needs verification:
Repository frankjoke/xs1 is archived
Adapter xs1 is currently installed 82 times

[x] verify that repository https://github.com/frankjoke/xs1 is really archived
[x] check if adapter has been migrated but links are not yet updated at repository
fix links at repository if this is causing the problem
[ ] decide if adapter should (and can) be moved to community-adapters (current usage 82 installs)
[ ] move adapter to iobroker-community-adapters or
remove adapter from repositories

Links are incorrect.
Links have been updated (https://github.com/ioBroker/ioBroker.repositories/pull/2324)


Cannot create app after adding arguments

Whenever you add an argument for the script, you can no longer get 'Select Script' and 'Create App' buttons to work.
I'm on Mac OS 11.6.

Experience the same on Mac OS 11.7.1. It beeps at asking to "select destination", but no windows appears to do so.

Same as above with MacOS 12.6

Same issue with a couple of screenshots (didn't find this already existing in search)
https://github.com/sveinbjornt/Platypus/issues/241

Same issue on Ventura 13.1 (22C65). It seems like file picker stops working in general - same behaviour when trying to change icon via Select Image/.icns File...

I've discovered the source of the bug and will release a fix soon.

Is there a fix for this yet, or any workaround at the moment other than saving a profile and creating via command line?

I'm having this problem as well.

This is a known problem (due to changes in AppKit modal dialog/sheet handling in macOS) and will be fixed in the next release, which should arrive soon. In the meantime, the workaround is to use the command line tool, or manually edit the AppSettings.plist file in the generated app, or go back to Platypus 5.3 (which has no ARM64 support).
Starting the GUI app from the command line shouldn't make any difference.

@sveinbjornt When can we expect a fix for this ?

I too am here wondering if there is a fix


npm run bootstrap

npm run bootstrap should do the initial npm installtoo

This will not work, since 'task.js' already requires the dependencies


Mapit API returns error on 400

This change allows the API adapter for Mapit to return an error when the
User enters in an invalid postcode. This will allow us to log in
Frontend when this happens.
Ticket.

Removed the rescue and changed the test to check that a bad postcode will raise an exception.


Major reworking of Cloudflare record update

CloudflareProvider's update path wasn't working correctly in many cases. This should get it to a point where it correctly functions, but it's still not perfect/ideal. It's doing about as good as it reasonably can given the API doesn't provide a way to update the multiple pieces of a single record atomically. More details in the comments.
This is not yet finished. It'll need a pass of more thorough testing to get all of the bits covered and validated more thoroughly.

[x] Additional test coverage of new paths & logic.

/cc Fixes https://github.com/github/octodns/issues/144
/cc @jr-RXcode @NicolaiSchmid

Looks like this PR introduced some new issues with the Cloudflare plugin, the dump works fine but when you sync information back records with multiple values get deleted (only one remains).

Looks like this PR introduced some new issues with the Cloudflare plugin, the dump works fine but when you sync information back records with multiple values get deleted (only one remains).

Odd. Its at this point that I'm glad octoDNS tells you what it's going to do before doing it. Will take a look at this, but it'll probably take me a couple days to get to it atm.

Looks like this PR introduced some new issues with the Cloudflare plugin, the dump works fine but when you sync information back records with multiple values get deleted (only one remains).

@vanbroup do you have more details or repo steps for this? I just tried dumping a couple zones and then syncing them back and get No changes were planned.
Checked both records with multiple values and names with multiple record types and didn't turn up any problems, but obviously it's hard to be exhaustive.

When everything is in sync it works fine, I think the problem is caused by a combination of L334.and L348.
When one of (for example) three values is updated, it will update the first record that matches the name and type (line 348). The other two lines get deleted by line 371.

Sorry @vanbroup, but I'm still unable to reproduce. If you have something easily reproducible can you drop the details in here or email them to me.

OK. That should be enough for me to dig in. I was previously looking for problems around the process of dumping & then syncing and didn't turn anything up.

I couldn't get this to reproduce even with an exact copy of your case until I turned on proxying.
Turns out it's a diff in the dump'd json between the TTLs. You can't hit it if you're running normal records b/c CF has the same min 120s TTL for all records so everything matches up and the 2 records are left alone. If instead you turn on the proxying stuff the API returns back answers with 1s TTLs which don't match up with the minimum TTL of 120s for the provider so the records aren't considered the same at that point.
Ultimately I think this falls into the same bucket as https://github.com/github/octodns/issues/202, managing zones with proxied records just isn't supported currently. I'll get a PR going to update the README to call that out for Cloudflare to at least make that explicit.
In general I don't have the use-case for CF protection and won't have the bandwidth to dig into it "for fun" any time soon so it'll probably have to find an owner to pick it up and work out the details/kinks.

In my test case I had this problem even without any proxied records. Also as stated in comment https://github.com/github/octodns/pull/162#issuecomment-363337216 I think this also looks to be a logical behavior of the code if the records are not returned in fixed order.

In my test case I had this problem even without any proxied records.

Putting a TTL of 1 in the config is completely ignored by the CF provider and raised up to 120s. The only way I've found to recreate the output you get is if I go into the CF UI and click automatic on the records so that they return 1s TTL which can't be matched by the provider that ensures a minimum of 1. I'll play around with it a bit more, but since the "existing" record version says 1s I'm pretty sure that means automatic was turned on for the record unless you have something enabled in your account that I don't.

I think this also looks to be a logical behavior of the code if the records are not returned in fixed order.

I'm not following that. existing_contents and new_contents are hashes so ordering shouldn't be of consequence. I currently think it's running into problems because all of the contents are out of whack: existing have 1s ttl as returned by CF and new have 120s as bumped up from 1s by the provider.
I'm still trying, but I can't find a way to get a 1s TTL back from the API without having automatic enabled and having 1s in the config will always be ignored. https://github.com/github/octodns/blob/master/octodns/provider/cloudflare.py#L325

Filed https://github.com/github/octodns/pull/205 which should address the TTL mis-matches, but the 1s (Automatic) TTL stuff still isn't supported in general. More info in the issue. We should move any further discussion there.


python311Packages.orange3: adjust nativeBuildInputs

Description of changes
Fix build (https://hydra.nixos.org/build/233272649)

Things done

Built on platform(s)

[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 252805 run on x86_64-linux 1

4 packages built:

python310Packages.orange3
python310Packages.orange3.dist
python311Packages.orange3
python311Packages.orange3.dist

Result of nixpkgs-review pr 252805 run on x86_64-linux 1

4 packages built:

python310Packages.orange3
python310Packages.orange3.dist
python311Packages.orange3
python311Packages.orange3.dist


Redesign dictionary: use Vector{DictSlot{K,V}}

Since Julia does not have atomics for arrays, supporting boxed Julia objects in a hash table requires indirection (i.e., a mutable struct) somewhere at the moment because no atomic operation can be defined on Vector{Any}. Previously, LinearProbingDict was using a mutable struct only for "storage" and the actual linearization was done on a dense array using 128 bit CAS (based on Maier et al. (2019)). However, it required complex key/value storage management to actually store objects in the hash table. It turned out that just using Vector{DictSlot{K,V}} (where DictSlot{K,V} is a mutable struct) is more straightforward and beneficial in terms of performance. The important optimization to make it practical is to avoid copying DictSlot{K,V} during migration.
A nice side-effect of this approach is that it gets rid of the "torn" 64-bit load which is not likely to be supported in Julia. It is now possible to separate key and value to distinct Vectors if atomics on Vector{Any} is supported at some point. This is more likely to be supported than updating key/value location directly in Vector{Pair{Any,Any}}.

-t16
julia> suite = ConcurrentCollectionsBenchmarks.BenchDictHistogram.setup();

julia> result = run(suite; verbose = true)
...
4-element BenchmarkTools.BenchmarkGroup:
  tags: []
  "cdict-ntasks=16" => Trial(91.662 ms)
  "base-seq" => Trial(120.610 ms)
  "cdict-seq" => Trial(320.926 ms)
  "cdict-ntasks=2" => Trial(243.681 ms)

-t1
3-element BenchmarkTools.BenchmarkGroup:
  tags: []
  "cdict-ntasks=1" => Trial(259.050 ms)
  "base-seq" => Trial(118.540 ms)
  "cdict-seq" => Trial(274.011 ms)


closing html tag missing >

The closing html tag is missing the >.
This is literally a single character change for someone who wants to practice git and github.

Added the missing character '>' to the closing HTML tag.


Add Swiftformat

Short description 📝
Add swiftformat and set up Danger to run it in a strict manner so that warnings are reported as errors.

Not sure I understand the reason for removing SwiftLint? SwiftFormat is great, and that's a good addition, but auto-formatting doesn't replaces the value of linting. It would just replace something like swiftlint autocorrect.
I usually use both tools in concert.

Not sure I understand the reason for removing SwiftLint? SwiftFormat is great, and that's a good addition, but auto-formatting doesn't replaces the value of linting. It would just replace something like swiftlint autocorrect.

That's a good point. The Dangerfile has swiftformat set up to run as a linter, but I guess since it's not its primary goal, it doesn't provide the same level of linter that Swiftlint does. Let me check if I can configure both without conflicting with each other.

Added Swiftlint back and fixed all the warnings and errors that we had in the project. Let mek now what you think @yonaskolb