Field Merging[2/x] Compute MergedSelections w/MergingStrategy

TL;DR
Updated the logic for merging selections in ComputedSelectionSet.
What changed?

Implemented a new MergedSelections logic to handle different merging strategies.

How to test?
Unit tests will be written before this PR is marked ready for review.
Why make this change?
To improve the handling of merged selections in ComputedSelectionSet based on different merging strategies.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#307  👈
#306
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AnthonyMDev and the rest of your teammates on  Graphite

@AnthonyMDev I ran this branch without field merging and I'm still seeing merged fields. I reproduced it on a sample project for easy review https://github.com/tahirmt/apollo-codegen-demo. Maybe I missed something but I think I got everything set up correctly.
Here fieldMerging is set to none
https://github.com/tahirmt/apollo-codegen-demo/blob/98b3390ecb0b35e3c4939176656f44442a90aab0/Sources/codegen/Command.swift#L31
And here is the generated query still showing merged fields
https://github.com/tahirmt/apollo-codegen-demo/blob/98b3390ecb0b35e3c4939176656f44442a90aab0/API/Sources/Operations/Queries/NodeQuery.graphql.swift#L60
Let me know if I missed anything

I made a very silly oversight yesterday. While the field merging strategies work properly, they are not actually affecting the generated templates yet. I'm going to get that sorted out today and will have another PR up with this working soon!
Sorry for the confusion @tahirmt

Ah no worries. I was just too excited to try out.

Closing. This is replaced by #431


Python 3 compatibility fixes

This prevents OL from usage in Python 3 only environment.

Will this still work with Python 2?

It should, although I tested it only on Python 2.7

It seems to work in python 2.6:
$ python2.6
Python 2.6.8 (unknown, Jan 26 2013, 14:35:25)
[GCC 4.7.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> print('ok')
ok
>>>

@fredj The print with parentheses is not the only change I made.
I made some more testing now. As I was testing this, I bumped into more incompatibilities with Python 3, so I added more fixes.
I tried to build with all compressors on Python 2.6 and it works OK, except uglify_js which didn't work on 2.6 before.
Although Python 2.5 doesn't support "except Exception as e" syntax.
The question is, if we can drop Python 2.5 support, because writing code compatible with 2.5-3 versions is quite complicated: http://stackoverflow.com/questions/11285313/try-except-as-error-in-python-2-5-python-3-x
Python 2.6 was released in 2008, so it is still quite old release and can is backward compatible with Python 2.5, which is quite antique. And anyway, uglify_js needs 2.7 with current code.

I think we can support python >= 2.7

I think we can support python >= 2.7

Agreed.

@tschaub Thanks for your comments to the code. The unrelated changes was caused bad imported commit from bad rebase. I have merged all Python 3 changes into single commit.
I have added slight additional change - usage of OrderedDicts instead of simple dicts which ensures stability of output JS file. This is essential during testing to ensure if the output is correct.
I have tested all the compressors with Python 2.7 and 3.4

@ahocevar Can you please pull this? Or is there anything I should adjust?

I hope this doesn't break things for anyone, but I agree it's time to move away from deprecated Python versions.

@ahocevar Thanks.

@PetrDlouhy It looks like we do not have a CLA from you. Can you sign one please?

@ahocevar OK, I signed.


Switch to prophecy-phpunit trait

Since prophecy-support in phpunit is deprecated, and will be remove in phpunit 10, this PR add prophecy-support via phpspec/prophecy-phpunit.

Oh sorry, didn't check for lower deps. Maybe it's time to let php < 7.3 go?

we recently got reports from users using this package for lower PHP when we accidentally broke it, I wouldn't go that path.
I would be OK to drop 5.x and 7.0 base on this stats: https://packagist.org/packages/php-coveralls/php-coveralls/php-stats (as separated PR)

Why? Raising minimum version isn't a breaking change, users with php < 7.3 can still use the old version.

FYI: there is a PR open in phpspec/prophecy-phpunit to widen the version restrictions - your support may help get it merged: https://github.com/phpspec/prophecy-phpunit/pull/38

I believe we can drop <7.2 by now

I have no objection to this change.
While this will make things slightly more difficult for projects which do code coverage runs on their high-low supported PHP versions if their "low" PHP version is below PHP 7.2, it can still work perfectly well as the PHP version on which the tests are run does not have to be the same PHP version on which code coverage is uploaded.
To do so, one would have to switch the PHP version within the CI job. With GH Actions and the shivammathur/setup-php action runner, this is not hard to do, it just requires adding an extra step.
I'm already doing this for a project which still supports PHP 5.4 (while the 2.x version of this package has a PHP 5.5 minimum).
Example code:
jobs:
  coverage:
      # ... All the normal config and steps to set up the environment and run the tests ...

      - name: Run the unit tests with code coverage
        run: vendor/bin/phpunit

      # Uploading the results with PHP Coveralls v2.x needs PHP 5.5+, so switch the PHP version.
      - name: Switch to PHP 7.4
        if: ${{ success() && matrix.php != '7.4' }}
        uses: shivammathur/setup-php@v2
        with:
          php-version: 7.4
          coverage: none

      - name: Install Coveralls
        if: ${{ success() }}
        run: composer require php-coveralls/php-coveralls:"^2.5.3"

      - name: Upload coverage results to Coveralls
        if: ${{ success() }}
        env:
          COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          COVERALLS_PARALLEL: true
          COVERALLS_FLAG_NAME: php-${{ matrix.php }}-phpcs-${{ matrix.phpcs_version }}
        run: vendor/bin/php-coveralls -v -x build/logs/clover.xml

Might be good to make people aware of this option to switch PHP versions within a job by documenting it in the README.

While this will make things slightly more difficult for projects which do code coverage runs on their high-low supported PHP versions if their "low" PHP version is below PHP 7.2, it can still work perfectly well as the PHP version on which the tests are run does not have to be the same PHP version on which code coverage is uploaded.

also, one could upload the coverage with php-coveralls v2.5.3. they will not receive any new versions of php-coveralls, but also not that much is changing in this project and they don't need to run cutting-edge version of php-coveralls, and running legacy PHP version could be bigger problem to start with ;)


How to mount binary file into my container without encode or decode

I am having issues adding p12 cert in my secret manager we are not able to read the file after mounting the secret file so we think is a problem between google secrets manager and the external secrets operator.  Since Kubernetes stores secrets in base64  i guess we can double encode the data and then we can use the data as base64 encoded string inside of my application. Any idea how to mount this properly or other solution to this?

So you are saying binary files should work find without doing the manual encode and the decode? i am storing the secret using gcp ui load button so it should work
Not sure if this is related to the issue i mention above
https://github.com/kubernetes/kubernetes/issues/89906

It is not 😄. Most likely, the GCP is storing it already encoded, which is why you cannot see it.


Extension Windows path support

General information

Operating system + version: Windows 10
Browser + version: Chromium 67.0.3396.99 (64-bit)
Information about the host app:

How did you install it?

Pre-built binary

If installed an official release, put a version: 2.0.21

Information about the browser extension:

Installed via webstore
Browserpass extension version as reported by your browser: 2.0.21

Exact steps to reproduce the problem

Create new Password Store location in Extension options
Enter Windows path to location

What should happen?
Passwords should show up and work
What happened instead?
Passwords show up prefixed with path and do not work

Hi @braderhart - could you please provide a screenshot? Files are supposed to be listed with a path prefix relative to the store root.
When you say they "do not work", what exactly happens, and what were you expecting?

My options.

Default .password-store users disappear

Error on selecting password option.
What I would like to happen if possible, is an option to merge the passwords display so they work without the prefix and no error too if possible.

@braderhart What you're asking for already exists, if I understand you correctly :-).
When you set up custom store locations, this overrides the default. What your screenshot shows is that you have only one store configured - C:\Users\berhart\.password-store-brad.
In order to display merged results, you need to configure more than one store:

name
path

Default
C:\Users\berhart.password-store

Brad
C:\Users\berhart.password-store-brad

Removing the prefix is also a thing - you simply need to provide a name for your configured stores. If a name is available, then the prefix is removed and the store name is displayed instead.

@maximbaz I think there's a bug in here also - based on @braderhart's screenshot, it looks like when a custom store is configured with no name, then something strange is happening with the prefix when fetching the file. I will look into this...

@erayd This works perfectly now. Thanks!

I believe this has been fixed in v3, if you can test now, please share your feedback in https://github.com/browserpass/browserpass/issues/331


VB-615 Visit Outcomes

What does this pull request do?
Add's visit outcomes to the visit notes
What is the intent behind these changes?
allow legacy visit outcomes to be added to the new system

LGTM


Calling Callable.get_object() multiple times returns incorrect value

Godot version
v4.0.rc6.official [0cd148313]
System information
Windows 10
Issue description
Calling Callable.get_object() the first time seems to work, but subsequent calls to the same Callable simply return true.
extends Node2D

func _ready():
	var c: Callable = test

	print(c.get_object())
	print(c.get_object())
	print(c.get_object())

func test():
	pass

This code prints:
Node2D:<Node2D#25249710499>
true
true

Steps to reproduce
Create a Callable and call get_object() on it multiple times.
Minimal reproduction project
CallableTest.zip

Related:

#73998

Closing as duplicate of #73998 (fixed by the same PR)


[Kafka Scaler] Kafka SASL SCRAM auth seems to not work with AWS MSK

Report
When trying to authenticate with AWS Managed Streaming for Kafka secured with SASL_SCRAM_512 authentication KEDA ScaledObject seems unable to communicate with cluster and gives the following error
{"ScaledObject.Namespace": "keda", "ScaledObject.Name": "kafka-scaledobject", "error": "error getting scaler for trigger #0: error creating kafka client: kafka: client has run out of available brokers to talk to (Is your cluster reachable?)"}

Tried using KEDA version 1.4 and 2.3.0.
Expected Behavior
KEDA should be able to create a new ScaledObject Custom Resource and subsequently a HPA for scaling the target Deployment
Actual Behavior
K8s cannot create the ScaledObject CR.
Steps to Reproduce the Problem

Kafka Secrets and TriggerAuthentication manifest

apiVersion: v1
kind: Secret
metadata:
  name: keda-kafka-secrets
  namespace: default
data:
  sasl: ""
  username: ""
  password: ""
---
apiVersion: keda.sh/v1alpha1
kind: TriggerAuthentication
metadata:
  name: keda-trigger-auth-kafka-credential
  namespace: default
spec:
  secretTargetRef:
  - parameter: sasl
    name: keda-kafka-secrets
    key: sasl
  - parameter: username
    name: keda-kafka-secrets
    key: username
  - parameter: password
    name: keda-kafka-secrets
    key: password

ScaledObject manifest

apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
  name: kafka-msk-scaledobject
  namespace: default
spec:
  scaleTargetRef:
    apiVersion: apps/v1
    kind: Deployment
    name: kafka-consumer
    # Required Name of the deployment we want to scale.
  pollingInterval: 10
  minReplicaCount: 0   #Optional Default 0
  maxReplicaCount: 10  #Optional Default 100
  advanced:                                          # Optional. Section to specify advanced options
    restoreToOriginalReplicaCount: true
  triggers:
  - type: kafka
    metadata:
      bootstrapServers: b-1.xxx.kafka.us-east-1.amazonaws.com:9096,b-2.xxx.kafka.us-east-1.amazonaws.com:9096,b-3.xxx.kafka.us-east-1.amazonaws.com:9096
      consumerGroup: keda_scaling_topic_group
      topic: keda_scaling_topic
      lagThreshold: "30"
    authenticationRef:
      name: keda-trigger-auth-kafka-credential

Logs from KEDA operator
{"ScaledObject.Namespace": "keda", "ScaledObject.Name": "kafka-scaledobject", "error": "error getting scaler for trigger #0: error creating kafka client: kafka: client has run out of available brokers to talk to (Is your cluster reachable?)"}

KEDA Version
2.3.0
Kubernetes Version
1.20
Platform
Amazon Web Services
Scaler Details
Kafka
Anything else?
No response

What Kafka version are you using? You might have hit the same issue like mentioned here?

Looks like so @zroubalik
We're using Kafka version 2.7.0

2.7.0 should be supported by Sarama client that is being used by KEDA: https://github.com/Shopify/sarama#compatibility-and-api-stability

Dynamic version support is part of KEDA release 2.4, if I'm not wrong?
We're currently using KEDA 2.3

In KEDA 2.4 you will be able to select the Kafka (Sarama) client version, in case you need to target an older Kafka Broker.
I think that your setup should work on KEDA 2.3 out of the box, because the Kafka version is supported by the client. I'd check whether the Kafka is reachable from within the cluster (firewall), I'd check if provided credentials and Broker urls are correct (check typos).

This is stil an open issue when you try to use iam authentication. See https://github.com/Shopify/sarama/issues/1985

I am facing same issue with AWS MSK when using SASL SCRAM.  @dipen-epi is ur issue was resolved. Can u share what was the solution?
Kafka Version - 2.6.2
KEDA version - 2.6

i am using Keda app version 2.8.1 and kafka version 2.8.0 and getting same issue once AWS MSK is using SASL SCRAM

i am using Keda app version 2.7.1 and kafka version 2.7.0 and getting same issue once AWS MSK is using SASL SCRAM

Maybe it's a super obvious question but, are you using scram_sha512? I mean, in theory based on docs, SCRAM should work properly. Are you using tls?

I have also active issue where sasl ( scram_sha512 ) is not working in our case with tls but without ACM configuration.
issue: https://github.com/kedacore/keda/issues/4089

TBH, I have no idea about AWS MSK, yesterday I created a cluster but I couldn't expose it publicly. For troubleshooting proposes I need to reach the kafka cluster from my local, which was impossible. I'll give another try when I have some time

To not discuss in a closed issue, let's continue the conversation in the new issue you have created


Enhancement | Sync template with Ref Arch 11/24

Describe the Feature

Update template files to match current Ref Arch implementation
Update terraform modules

DONE https://github.com/binbashar/le-tf-infra-aws-template/pull/46/files


Method save does not exist.

public function store(Request $request)
{
$slider = new Slider();
    $slider->header = $request->header;
    $slider->paragraph = $request->paragraph;

    if ($request->hasFile('image')) {
        $slider = $request->file('image');
        $filename = time() . '.' . $slider->getClientOriginalExtension();
        Image::make($slider)->resize(1920, 1008)->save(public_path('images/homeSlider/' . $filename));
        $slider->image = $filename;
    }

    $slider->save();

    \Session::flash('sliderSuccess', 'Slider has been successfully uploaded');
    return \Redirect::to('admin/slider/create');
}

@johnef you're overwriting the $slider object with $request->file('image'), after that it's no longer a model/ORM object
try:
$slider->header = $request->header;
$slider->paragraph = $request->paragraph;

if ($request->hasFile('image')) {
    $image = $request->file('image');
    $filename = time() . '.' . $image->getClientOriginalExtension();
    Image::make($image)->resize(1920, 1008)->save(public_path('images/homeSlider/' . $filename));
    $slider->image = $filename;
}

$slider->save();

\Session::flash('sliderSuccess', 'Slider has been successfully uploaded');
return \Redirect::to('admin/slider/create');

@olivervogel please close


Bootstrapper is x86 app

Microsoft PowerToys version
0.55.1
Running as admin

[ ] Yes

Area(s) with issue?
Installer
Steps to reproduce
https://www.reddit.com/r/Windows10/comments/skphg2/which_one_is_correct/
Determined the installer is x86 not x64
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

According to the links posted here https://github.com/wixtoolset/issues/issues/5749, 64bit wix bootstrapper will only be available in v4. We use 3.x.

blocked no matter what as it is preview
As x86 runs on arm and 64 bit, i think this is a low pri item

/dup https://github.com/microsoft/PowerToys/issues/22258


add Epsilla vectorstore

Epsilla vectordb is an open-source vector database that leverages the advanced academic parallel graph traversal techniques for vector indexing.
This PR adds basic integration with pyepsilla(Epsilla vectordb python client) as a vectorstore.

@ricki-epsilla feel free to ping when it's ready for review!

@baskaryan PR is ready. Please help to review. Thanks!

hey @richard-epsilla and @ricki-epsilla! pushed a few minor updates to #9525, feel free to merge those into this branch if they look good to you. otherwise looks good to  me!

hey @richard-epsilla and @ricki-epsilla! pushed a few minor updates to #9525, feel free to merge those into this branch if they look good to you. otherwise looks good to me!

Hi @baskaryan , looks good to me! I just merged the branch into this. Thanks!


Add metrics support

Add support for instrumenting different policies with metrics. This instrumentation could live in a separate package, such as how the HTTP support works, and can instrument policies using their event listeners.

Is this adding support using Prometheus or OpenTelemetry?

In actual Hystrix, there was also a Hystrix dashboard which was more real-time, but open telemetry would also work.


Faster Equations -> State by avoiding Term equality

It takes a very long time for the final distribution to be calulated in an expression-eval.
Much of the time is spent on equality of terms.
Instead we should just identify terms by indices, so atomic expressions are indices or constants.
We see that one time reduction to atoms is fast.
Just have evolving vectors, ideally parallelized using a future/task.

Implemented in 1bf970658b6315099fc improving on the first attempt in 5eb7201df257eca59; some debugging was needed but mostly other stuff that became visible once this change was made.


⚠ ClamAV has degraded performance

In 051565c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 571 ms

Resolved: ClamAV performance has improved in a8f3a5f after 5 minutes.


replaced state numbers with constants to make state more readable

Hi,
I was trying to understand what actual meanings of the numbers were assigned to this._state as well as self._state are, however, the numbers were not connected to the meanings straightforwardly. Hence, I replaced the numbers with meaningful constants for understanding.

Hi @jsgao0, thanks for bring this issue up. I think it is good to add more details, but I think we should make a couple changes. I would prefer than the states are not an object, since that adds a global object that is never reclaimed. The state names should be
var STATE_PENDING = 0;
var STATE_FULFILLED  = 1;
var STATE_REJECTED = 2;
var STATE_FULFILLED_PROMISE = 3;

Closing because of lack of response


Add Hydra healthcheck to compose file

This PR adds a health check for Hydra using the /health/alive endpoint of the Hydra service. This health check is useful to make sure that Hydra is ready before SMD tries to make a request to the .well-known/jwks.json endpoint for verification and authorization.

Revisiting this, rebasing onto current main since this got a little crusty.

@alexlovelltroy It's not letting us merge since I pushed to this branch last. 🙂 This has been tested already. Could you approve?

Thanks, @alexlovelltroy! I'll go ahead and merge this for @davidallendj.


Vulkan renderer is not rendering some geometry

This appears to be affected by the number of triangles calculated before invoking flush().  Increasing this number increases the amount of geometry rendered.

I am suspecting that this is a memory hazard, in that the vertex buffers are mapped and updated when flush() is called, but there is no explicit pipeline barrier before the draw call to ensure synchronisation between the previous contents and draw call, and the new contents and draw call.

I am suspecting that this is a memory hazard, in that the vertex buffers are mapped and updated when flush() is called, but there is no explicit pipeline barrier before the draw call to ensure synchronisation between the previous contents and draw call, and the new contents and draw call.

Commit dc79b26dd2d25410a27361782879c52e37ea5af0 improves things, but there are still problems.  Geometry is now missing, but there appears to be no corruption (triangles constructed from vertices at opposite sides of the planet, for example.)  I notice that the behaviour seems to be consistent between runs, and between debug and release builds.  It is stable with non-moving view frustum but geometry flicks in and out when the view frustum changes, suggesting that culling might be implicated.

Significantly changing the CPU LOD factor removes the problem, which is not terribly surprising, as this causes far less geometry to be generated.
--- a/programs/rust/aeolius/src/planet.rs	Tue Jan 03 15:42:01 2017 +0000
+++ b/programs/rust/aeolius/src/planet.rs	Tue Jan 03 20:14:34 2017 +0000
@@ -430,7 +430,7 @@
let dist_v31 = (proj_v1 - proj_v3).magnitude();
let lod1 = dist_v12.max(dist_v23.max(dist_v31));
let lod0 = lod1 * LOD_FACTOR;

       let lod = lod0.max(1.0f32).min(divisions as f32) as usize;

       let lod = 1.0f32; //lod0.max(1.0f32).min(divisions as f32) as usize;

       // Render triangle patches iteratively based on that level of detail.
       // This code stripes the triangle into sub-triangles.  It does so by

Commit f9841cb907717115c21ccac523f8e78dc8517adf corrected the binding of the vertex buffers, so the missing geometry is now rendered.


Add ratings to posts

We could use staticman.

Form with selection of 1-5 stars and submit button
StaticMan config without any moderation (no PR)
Can we use a JSON data source where the rating gets appended to an array or something?

@chmac There seems to be a problem with throttling of the GitHub API. Too many requests in this time frame. There is an issue in staticman repo.
We probably have to host our own staticman instance.

Done


Document building URLs with a named transformation

I was going to add support for this, but found that it was already there. 😄

Coverage increased (+0.7%) to 60.0% when pulling aca21955f774a41a92bc120e575a04d2d748d2c8 on nathanl:nathanl/document-named-transformations into d24ca0406ab0858831d3630ba35a8cea05163d64 on smeevil:master.

\o/


Select which files to delete in huggingface-cli delete-cache

Is your feature request related to a problem? Please describe.
The download tools all either dynamically or configurably download parts of a repo, since there are many times when the repo contains more files than needed (e.g. .pt, .safetensors). Once they're downloaded, you can't delete only part of the repo from your cache, and if you accidentally (or because you didn't know what you were doing) downloaded huge volumes of files, you can't delete just a part of it. The only option right now is to delete the whole repo and then re-download only what you want in your cache.
Describe the solution you'd like
Option to select files within a given cached repo, using the existing tools, that are to be deleted.
Describe alternatives you've considered
Manually manipulating files in the cache has gone..let's say "poorly" in the past. I've had to scrap my entire cache more than once doing that.
Additional context
n/a

This is a valid feature request indeed. At the moment we can only delete specific revisions (usually main for the whole repo, or outdated revisions if more than one revision). To delete files we would need to update the UI / the API. Maybe it's fine to just have it for 1 repo at a time: huggingface-cli delete-cache <repo_id> --delete=... or huggingface-cli delete-cache <repo_id> --keep=... (where delete/keep are glob patterns).

Yeah, for sure I can. This would be my first time doing anything more than very superficial dev on a project like this (on GH, that is). Please be gentle :)
I can have a think through the design of it first. Hopefully I'll have some downtime from other side projects to be working on tihis. Off the cuff thoughts:

delete-cache doesn't take arguments or flags in other contexts, so introducing that might be confusing.
delete-cache also is going down the road of using that TUI system for multiple selection, which would be confusing / difficult to do nested.
Perhaps a new subcommand like prune-cache or trim-cache or trim-repo... repo-cache. I like repo-cache because then you could extend that to more tools that would specific to managing the cache for a single repo.

Going with repo-cache, use case would be something like:
> huggingface-cli repo-cache delete --all repo_id     # same functionality as delete-cache but quick to delete just the one repo
> huggingface-cli repo-cache delete --include (glob) repo_id.  # use same glob format as download --include flag; mark files to __include__ for delete action
> huggingface-cli repo-cache delete --exclude [glob] repo_id. # same as --include, but opposite
> huggingface-cli repo-cache delete --file-list (list of files to remove) repo_id

I'll have a proper sit down at some point soon and see what's what.
Are there timelines I should be aware of that you know of?


Fix the kv_get_data return value to develop

Change Description

Fix to kv_get_data to return the actual number of bytes copied.
Change Type
Select ONE:

[ ] Documentation

[x] Stability bug fix

[ ] Other

[ ] Other - special case

Testing Changes
Select ANY that apply:

[ ] New Tests

[x] Existing Tests

[ ] Test Framework

[ ] CI System

[ ] Other

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

The original behavior is by design.  All the kv functions that copy data into wasm memory return the actual size of the data before truncation.  Also, this PR breaks consensus, because it only updates rocksdb, but not chainbase.

The original behavior is by design.  All the kv functions that copy data into wasm memory return the actual size of the data before truncation.  Also, this PR breaks consensus, because it only updates rocksdb, but not chainbase.

Based on Stephen's comments, closing this out.  We will update docs to match the design.

Based on Stephen's comments, closing this out.  We will update docs to match the design.


[How to]: Control my garage door (secured with code)

Describe the bug
Hello,
I'm not able to control my Hormann garage door with HA. The door works well with Loxone and I've got the possibility to control it with Pyloxone but the door don't move cause it needs the security code.
When I use it in Loxone, I have to insert the code.
Where have I to put this code to control the door ?

Thanks for your help!
It makes me crazy!
Firmware of your Miniserver
Gen. 1
Firmware <IP_ADDRESS>
HomeAssistant install method
RPI 4 server
Version of HomeAssistant
2024.7.2
Version of Pyloxone
Last one.
Update pyloxone
Successfully.
Log
Settings for the log can be found on the main page under the section Log Configuration

@Nico0W I think it is not possible to secure a cover with normal UI elements. You can try to open the door via a secure websocket call. If this works you can try to combine it with a custom UI element where you can input a code. There a view custom elements available.

I'm really struggling to make it works...
I used ChatGPT to help me and i've added this to my configuration.yalm:
input_text:
  door_code:
    name: Code pour la porte
    min: 4
    max: 4
    pattern: '[0-9]{4}'
    mode: password

input_button:
  validate_code:
    name: Valider le code

shell_command:
  open_door: 'python3 /config/loxone_ws_client.py {{ code }}'

automation: !include automations.yaml
script: !include scripts.yaml

This to automation.yalm
`- alias: "Valider le code pour ouvrir la porte"
  trigger:
    platform: state
    entity_id: input_button.validate_code
    to: 'pressed'
  action:
    - service: shell_command.open_door
      data:
        code: "{{ states('input_text.door_code') }}"
`

and created this page in my configuration folder:
`import asyncio
import websockets
import ssl
import json
import sys

LOXONE_HOST = '<IP_ADDRESS>'  # Remplacez par l'adresse IP de votre Loxone Miniserver
LOXONE_PORT = 8080  # Remplacez par le port de votre Loxone Miniserver
LOXONE_USER = 'your_loxone_username'
LOXONE_PASSWORD = 'your_loxone_password'
LOXONE_DOOR_UUID = 'loxone_door_uuid'  # Remplacez par l'UUID de votre porte Loxone
CORRECT_CODE = '1234'  # Remplacez par le code correct

async def open_door_if_code_correct(entered_code):
    if entered_code == CORRECT_CODE:
        uri = f'wss://{LOXONE_HOST}:{LOXONE_PORT}/ws/rfc6455'
        ssl_context = ssl.create_default_context(ssl.Purpose.CLIENT_AUTH)
        async with websockets.connect(uri, ssl=ssl_context) as websocket:
            command = f'jdev/sps/io/{LOXONE_DOOR_UUID}/pulse'
            await websocket.send(json.dumps({'method': command}))
            response = await websocket.recv()
            print(response)
    else:
        print("Incorrect code")

if __name__ == '__main__':
    if len(sys.argv) != 2:
        print("Usage: python3 loxone_ws_client.py <entered_code>")
    else:
        entered_code = sys.argv[1]
        asyncio.run(open_door_if_code_correct(entered_code))`

And it doens't seems to want to work. What have I done wrong ?
That's like my Mechanical Ventilation. In the Loxone app, I can control the speed, not in HA. Only see sensors about temperature and speed and have a button to switch to automatic mode. I would like to manually change the speed, like the app:

Do  you have any idea on how i can control it ?

@Nico0W do you still have this problem? ChatGpt is the wrong solution for this. ;-)

Yes, I still have the problem.
I did not solve it, nothing worked with ChatGPT. Without help, it's completely out of my knowledge :-(

@Nico0W
Here is a solution how I would do it:

Update Pyloxone to the newest version (at least 0.6.11). I created a service called "event_secured_websocket_command"
Create a Input Helper for the Code. Let's assume that you call it code.
Create a Template Cover like this:

cover:
  - platform: template
    covers:
      garage_door:
        device_class: garage
        friendly_name: "Garage Door with Code"
        position_template: >
          {% set position = states.cover.tor.attributes.current_position | default(0) %}
          {{ position }}
        open_cover:
          - service: loxone.event_secured_websocket_command
            data:
              uuid: xxxxx-xxxx-xxx-xxxxx
              value: open
              code: "{{ states('input_text.code') }}"
          - service: input_text.set_value
            data:
              value: ""
            target:
              entity_id: input_text.code

        close_cover:
          - service: loxone.event_secured_websocket_command
            data:
              uuid: xxxxx-xxxx-xxx-xxxxx
              value: close
              code: "{{ states('input_text.code') }}"
          - service: input_text.set_value
            data:
              value: ""
            target:
              entity_id: input_text.code
        stop_cover:
          - service: loxone.event_secured_websocket_command
            data:
              uuid: xxxxx-xxxx-xxx-xxxxx
              value: stop
              code: "{{ states('input_text.code') }}"
          - service: input_text.set_value
            data:
              value: ""
            target:
              entity_id: input_text.code

IMPORTANT:

The cover.tor in this example must be replaced with the cover name which you want to control. For example cover.ferme.
The uuid: xxxxx-xxxx-xxx-xxxxx must be replaced with the uuid of the cover you want to control.

Then put the Input and the template Cover to any dashboard an try it. If this works you can hide the original cover which pyloxone auto generated.

@Nico0W and please do not only use chatGPT and read more in the documentation. ChatGPT is not the solution for such a problem.


[docs] GettingStarted.md: Do not suggest to run tests when building toolchain

Running the entire test suite is unnecessarily burdensome for newcomers, given the already considerable build times.
Test failures are not a beginner-friendly indication of an out-of-sync checkout, nor is such an indication generally worth the expenses.
We do not want a known issue to block anyone’s workflow.

Just noticed that GItHub created a branch in Apple’s repo rather than my fork. Starting to feel like I’ll never do stuff like this in the browser again.

@swift-ci please smoke test


Fix Lit Axis Mixin Tests

Currently all of the Lit Axis mixin tests are failing locally but passing in the Pull Request GitHub Action workflow.
This is due to the way the Lit Axis mixin is rendering the SVG elements and the way that the tests are then looking at their results. The mixin is rendering out all of text elements on the page at {0,0} and once the render has finished, it is getting their width and height using the getBBox function and then updating their position accordingly. Unfortunately, different browsers running on different environments (local vs GitHub Workflows) are returning different values for their height and width, and so their positions are also different.
I need to come up with a better way of testing the SVG element positions that takes into account that their positions are different based on different browsers running in different environments. Or I need to come up with a better mixin algorithm for calculating their positions that doesn't rely on the getBBox function for their widths and heights.

I have looked at the ApexCharts Library tests, and from what I can tell, they are not testing for element positioning inside the SVG, only element content. I am changing my tests for the time being to do this until I can figure out a better way to test element positioning across browsers without having to input every possible position for every label in ever browser configuration (browser and  OS combination).


Document which open(2) numbers are supported on Windows

So we can link to them from the Node.js docs.

On Windows, some flags are translated to their equivalent ones as accepted by CreateFileW.

https://github.com/nodejs/node/pull/3641#discussion_r43846108

Pull requests welcome although I think we have mappings for all common flags.  It should Just Work in most if not all cases.


Fonts not showing right.

Yo, i was trying to use some fonts like "Iceberg", but for mostof them, they dont show up right, most oif them, if not such popular like "Helvetica" or so, the font doesnt work...

The font you are using on your design must be installed on your machine...

And popular fonts work because they are installed by default on most operating systems.

you can close the issue if the problem is solved :)

Oops, I Closed it...


undefined in where query

Bug description
If undefined is specified in the where query, any record will be returned.
How to reproduce

Expected behavior
No response
Prisma information

generator client {
  provider = "prisma-client-js"
  previewFeatures = ["fullTextSearch", "interactiveTransactions"]
}

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

model User {
  username String @id
  email String @unique
  password String
  balance Int @default(0)
  referrer String?
  authCode String?
  isAdmin Boolean @default(false)
  isBanned Boolean @default(false)
  isEmailVerified Boolean @default(false)
  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
}

Environment & setup

OS: Windows
Database: PostgreSQL
Node.js version: v16.13.0

Prisma Version
prisma                  : 3.4.1
@prisma/client          : 3.4.1
Current platform        : windows
Query Engine (Node-API) : libquery-engine 57771c0558568c7d08bd34c7248af5244ae16bd9 (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 57771c0558568c7d08bd34c7248af5244ae16bd9 (at<EMAIL_ADDRESS>Introspection Engine    : introspection-core 57771c0558568c7d08bd34c7248af5244ae16bd9 (at<EMAIL_ADDRESS>Format Binary           : prisma-fmt 57771c0558568c7d08bd34c7248af5244ae16bd9 (at<EMAIL_ADDRESS>Default Engines Hash    : 57771c0558568c7d08bd34c7248af5244ae16bd9
Studio                  : 0.438.0
Preview Features        : fullTextSearch, interactiveTransactions

Does this not match what is documented at https://www.prisma.io/docs/concepts/components/prisma-client/null-and-undefined?

If undefined in a where clause returns all records, is there a value that can be used to return none?

@mcbethio if there are emails with empty strings your solution would return those empty email records.

Can someone explain why this is the standard prisma behavior? In my vision when you try to find something undefined or empty the query should not return anything.

@RamonSerraVasconcelos I am new to Prisma, a web search brought me here because I was wondering how to do conditional queries.
From a SQL perspective if you do SELECT * FROM USERS WHERE EMAIL = '' I think the empty string is legit.
There is no concept of undefined in SQL so this is a little confusing what the expected behavior is or if using undefined is the recommended way to do conditional WHERE statements.

In my vision when you try to find something undefined or empty the query should not return anything.

Something being undefined is the same as it not being present in JS.
So when you search for all users with no filter, you will get all users.
We have an issue here to further improve the documentation here: https://github.com/prisma/docs/issues/5041

this solution solved for me
// this returns all records
let emailSearchUndefined = undefined;
const firstResults = await prisma.user.findMany({
where: {
email: emailSearchUndefined
},
})
// this returns no records
let emailSearch = "";
const results = await prisma.user.findMany({
    where: {
        email: emailSearch || undefined || null
    },
})


⚠ Bofrost.be has degraded performance

In 145f079, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1936 ms

Resolved: Bofrost.be performance has improved in 2e1424c after 42 minutes.


Installation Help

Hi I installed Bookstack on my Ubuntu server with the installation script but when I go to http://ipaddress I just get a default nginx page. What am I missing?

Hi @ajmcateer,
Were you on a fresh Ubuntu 16.04 instance and do you provide ipaddress when the script asked for a domain?
If yes to both above could you run /etc/nginx/sites-enabled/bookstack and check that the server_name in the output of that command matches your ip address.

Closing due to lack of response. Feel free to still answer the above and I'll re-open this.


minor changes to remove pandas warnings

Awesome, tested https://gke.mybinder.org/v2/gh/gprichter/movingpandas/master and the only remaining errors are:
/srv/conda/envs/notebook/lib/python3.6/site-packages/geopandas/base.py:76: UserWarning: Cannot generate spatial index: Missing package rtree.
warn("Cannot generate spatial index: Missing package rtree.")
and
/home/jovyan/trajectory.py:72: UserWarning: Discarding nonzero nanoseconds in conversion
if not self.get_start_time() < self.get_end_time():


corrected error passing private key

> does not work and will brake the import. | is the correct symbol to use.
This can easily be tested by adding a SSH key via JCasC plugin to Jenkins and later on export it via the script-console
println(hudson.util.Secret.decrypt("{XXX=}"))
In case with > output is broken. If a key is added with |, the key is printed out properly again.
Have had the same issue while playing around with the JCasC plugin.

Good point, please update the test file as well.

Thanks!


Azure CLI command syntax changes?

We had a reader write in on the Azure Key Vault Configuration Provider in ASP.NET Core topic in the ASP.NET Core doc set (aspnet/AspNetCore.Docs #16117) alerting us to a possible change in Azure CLI syntax where some of the command option values are no longer in quotes.
This topic shows quotes for the key vault name (--name '<YourKeyVaultName>' and --vault-name "<YourKeyVaultName>"). I found another Azure topic that seems to indicate that the quoting of these option values was dropped at some point: https://docs.microsoft.com/azure/key-vault/quick-create-net.
I'm fixing the ASP.NET Core topic now and just alerting you here to the discrepancy in the two Azure docs. EDIT ... Done! https://github.com/aspnet/AspNetCore.Docs/pull/16222

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

ID: ffad04ad-5d9e-56de-c688-f0c6dd9de549
Version Independent ID: 8f370b3c-3c2f-5493-1041-be919bb73e58
Content: Tutorial - Use Azure Key Vault with a Windows virtual machine in .NET
Content Source: articles/key-vault/tutorial-net-windows-virtual-machine.md
Service: key-vault
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@guardrex thanks for letting us know! So you've already made pull requests to fix the discrepancies?

Just on our side ... on the ASP.NET Core side on https://github.com/aspnet/AspNetCore.Docs/pull/16222.

I made a pull request to remove these. @msmbaldwin please confirm.
https://github.com/MicrosoftDocs/azure-docs-pr/pull/99237


Badge request | Curvu#0254

Username and Discrim: Curvu#0254
User ID:<PHONE_NUMBER>66951168
Image Link: https://imgur.com/DclUwaS

done! :D


[Feature Request]: clear the conversation history

When using teams-ai to build an AI chatbot, instead of append conversation history into openai request, people may want some control on it, like clear the chat history so that openai want process the previous history. Can teams-ai provide an interface to handle it easily?

Depends on how you want to achieve it. e.g. you can have
app.message('/forget', async (context: TurnContext, state: ApplicationTurnState) => {
    ConversationHistory.clear(state);
    await context.sendActivity(`forgot all conversation history.`);
});

When a user sends /forget, all conversation history will be erased.

clear history is a feature that has been added and will be released for GA :)

Hi, @corinagum, could you please confirm if this is available on GA 1.1.0?
Thanks!


Cloudfront Function logs

Do you know why event isn't logging?

Seems similar to this persons query:
https://stackoverflow.com/questions/76049017/in-my-lambdaedge-origin-request-function-the-event-parameter-is-empty
But I thought these functions would automatically get setup with service roles that allow logging.
Anyway, I wanted to view the logs to understand why it's necessary to have a viewer-request handler.
Before finding your package, I was running Sveltekit on Cloudfront, using SPA mode (which I still want to do, except for some server routes that I will use your handler for, e.g. /kit/server/* ). I didn't use (need?) a viewer-request, I just needed an origin-request that routed any misses on S3 to the svelte SPA fallback page.
Anyway nice package BTW. Due to the custom setup I need and the fact that my resources are deployed using a code pipeline (not CDK), I will probably end up having to pull out the bits of code I need, hence trying to understand all your moving parts.

Thanks for opening this issue!
First, regarding the issue with CloudFront Functions (a.k.a. CF2) not logging, I have confirmed that this is also happening in the sample case.
However, my guess is that this is not a bug, but due to the CF2 specification.
CF2 is running on the original runtime based on ECMAScript 5.1 and may not support output beyond the second argument of console.log.
I will modify the adapter and see if this is the case.
Secondly, the reason why the CF2 Viewer-Request is needed is that it exists primarily for hostname canonicalisation.
When using a custom domain, the application can be accessed by two hostnames.
For example, in the lambda-s3 sample case,

https://d1axeq0g4diak1.cloudfront.net
https://lambda-s3.adapter-aws.com

the application can be used under either of these hostnames.
In practice, however, it is preferable to normalise all requests to 2 for SEO reasons.
To do this, the 1 needs to be converted to 2 by an HTTP redirect before the request passes through CloudFront.
This is the need for Viewer-Request.


[ PIP-82] [pulsar-broker] Fix a couple of issues

Modifications

Handle ChildrenChanged metadata-store event.
Update the publish rate limiter correctly.

Verifying this change

[x] Make sure that the change passes the CI checks.
[x] verified via manual testing

Documentation
No documentation impact.

@jerrypeng PTAL when you get a chance.

LGTM

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


[KYUUBI #3531] [FOLLOWUP] Fix malforme format in hive connector doc

Why are the changes needed?

This pr aims to fix malforme format in hive connector doc.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[x] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #3587 (4fb6681) into master (bb50c52) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3587      +/-   ##
============================================
- Coverage     51.81%   51.75%   -0.06%
  Complexity       13       13
============================================
  Files           483      483
  Lines         26998    26998
  Branches       3769     3769
============================================
- Hits          13989    13974      -15
- Misses        11652    11664      +12
- Partials       1357     1360       +3

Impacted Files
Coverage Δ

.../kyuubi/server/mysql/constant/MySQLErrorCode.scala
13.84% <0.00%> (-6.16%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLCommandHandler.scala
75.00% <0.00%> (-4.55%)
:arrow_down:

...apache/kyuubi/engine/JpsApplicationOperation.scala
77.41% <0.00%> (-3.23%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLGenericPackets.scala
76.59% <0.00%> (-2.13%)
:arrow_down:

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
80.00% <0.00%> (-2.00%)
:arrow_down:

...pache/kyuubi/engine/YarnApplicationOperation.scala
62.96% <0.00%> (-1.86%)
:arrow_down:

...ain/scala/org/apache/kyuubi/engine/EngineRef.scala
72.26% <0.00%> (-0.85%)
:arrow_down:

...yuubi/server/metadata/jdbc/JDBCMetadataStore.scala
89.27% <0.00%> (-0.70%)
:arrow_down:

...g/apache/kyuubi/operation/BatchJobSubmission.scala
74.64% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks, merging to master


poetry: 1.7.1 -> 1.8.0

Description of changes
Diff: https://github.com/python-poetry/poetry/compare/refs/tags/1.7.1...1.8.0
Changelog: https://github.com/python-poetry/poetry/blob/refs/tags/1.8.0/CHANGELOG.md

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

14 packages marked as broken and skipped:

python311Packages.mapbox
python311Packages.mapbox.dist
python312Packages.cachecontrol
python312Packages.cachecontrol.dist
python312Packages.cwl-upgrader
python312Packages.cwl-upgrader.dist
python312Packages.cwl-utils
python312Packages.cwl-utils.dist
python312Packages.mapbox
python312Packages.mapbox.dist
python312Packages.schema-salad
python312Packages.schema-salad.dist
spring
springLobby

3 packages failed to build:

cwltool
cwltool.dist
hikounomizu

76 packages built:

adapta-gtk-theme
arc-theme
arx-libertatis
asciidoc-full
asciidoc-full-with-plugins
asciidoc-full-with-plugins.dist
asciidoc-full.dist
ayu-theme-gtk
breeze-hacked-cursor-theme
capitaine-cursors
cinnamon.cinnamon-screensaver
cinnamon.iso-flags-png-320x420
cinnamon.iso-flags-svg
clevis
clevis.man
coreboot-configurator
dblatexFull
disorderfs
emojione
fim
hackneyed
inkscape
inkscape-extensions.textext
inkscape-extensions.textext.dist
inkscape-with-extensions
iso-flags
k40-whisperer
kabeljau
rep (kakounePlugins.rep)
luksmeta
mate.mate-utils
material-cursors
mojave-gtk-theme
numix-cursor-theme
numix-solarized-gtk-theme
pdm
pdm.dist
pip-audit
pip-audit.dist
plata-theme
poetry
poetry.dist
poetryPlugins.poetry-audit-plugin
poetryPlugins.poetry-audit-plugin.dist
poetryPlugins.poetry-plugin-export
poetryPlugins.poetry-plugin-export.dist
poetryPlugins.poetry-plugin-up
poetryPlugins.poetry-plugin-up.dist
pop-gtk-theme
python311Packages.cachecontrol
python311Packages.cachecontrol.dist
python311Packages.cwl-upgrader
python311Packages.cwl-upgrader.dist
python311Packages.cwl-utils
python311Packages.cwl-utils.dist
python311Packages.diagrams
python311Packages.diagrams.dist
python311Packages.orange-canvas-core
python311Packages.orange-canvas-core.dist
python311Packages.orange-widget-base
python311Packages.orange-widget-base.dist
python311Packages.orange3
python311Packages.orange3.dist
python311Packages.schema-salad
python311Packages.schema-salad.dist
python311Packages.scikit-bio
python311Packages.scikit-bio.dist
python312Packages.diagrams
python312Packages.diagrams.dist
tang
tang.man
tebreak
tebreak.dist
vimix-cursors
volantes-cursors
wpa_supplicant_gui

Result of nixpkgs-review pr 291404 run on aarch64-darwin 1

14 packages marked as broken and skipped:

mate.mate-utils
mojave-gtk-theme
python311Packages.mapbox
python311Packages.mapbox.dist
python312Packages.cachecontrol
python312Packages.cachecontrol.dist
python312Packages.cwl-upgrader
python312Packages.cwl-upgrader.dist
python312Packages.cwl-utils
python312Packages.cwl-utils.dist
python312Packages.mapbox
python312Packages.mapbox.dist
python312Packages.schema-salad
python312Packages.schema-salad.dist

15 packages failed to build:

cwltool
cwltool.dist
hackneyed
hikounomizu
inkscape-extensions.textext
inkscape-extensions.textext.dist
pdm
pdm.dist
python311Packages.orange-canvas-core
python311Packages.orange-canvas-core.dist
python311Packages.orange-widget-base
python311Packages.orange-widget-base.dist
python311Packages.orange3
python311Packages.orange3.dist
volantes-cursors

32 packages built:

asciidoc-full
asciidoc-full-with-plugins
asciidoc-full-with-plugins.dist
asciidoc-full.dist
dblatexFull
inkscape
inkscape-with-extensions
k40-whisperer
rep (kakounePlugins.rep)
numix-cursor-theme
pip-audit
pip-audit.dist
poetry
poetry.dist
poetryPlugins.poetry-audit-plugin
poetryPlugins.poetry-audit-plugin.dist
poetryPlugins.poetry-plugin-export
poetryPlugins.poetry-plugin-export.dist
poetryPlugins.poetry-plugin-up
poetryPlugins.poetry-plugin-up.dist
python311Packages.cachecontrol
python311Packages.cachecontrol.dist
python311Packages.cwl-upgrader
python311Packages.cwl-upgrader.dist
python311Packages.cwl-utils
python311Packages.cwl-utils.dist
python311Packages.diagrams
python311Packages.diagrams.dist
python311Packages.schema-salad
python311Packages.schema-salad.dist
python312Packages.diagrams
python312Packages.diagrams.dist

Thanks for creating this PR! Poetry 1.8.1 was released yesterday, maybe it makes sense to package this version instead of 1.8.0?


allow dev dash apps to have some modules empty

WHY are these changes introduced?
Fixes: https://github.com/Shopify/develop-app-inner-loop/issues/2290

Our App schema currently requires application_url, embedded and the webhooks api_version to always be set
This is not the case with app management apps, and will eventually never be the case with the greenfield vision of all app modules being optional

Fixes #0000

WHAT is this pull request doing?

How to test your changes?

Post-release steps

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes

#4883  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Based on this thread, it looks like we are actually going to want to keep some of these modules around for now. Will open another small PR


Batching - fix basis polarity in large fvf

Slight technical hitch, the basis was reversed that was sent to the shader, so rotations were opposite. This PR reverses polarity of the basis to be correct.
Fixes #46895
Notes

This should be fairly innocuous

Thanks!


Running LoRA + QLoRA on Colab Notebooks

Using this issue to track the work we need to do to enable this. We'll capture our learnings, TODOs and PRs over here so everyone can follow along.

This is being actively worked on in #1315. What are all the gaps that are remaining?


Run rebar3 tests with coveralls on travis

Update lager and meckfrom hex
Build on travis via rebar3 and mix
Run test on travis via rebar3 and send coverdata

We don't run tests and send cover data via mix because there isn't possibility to compile yang and send coverdata

we can wait until rebar will be stable

Looks that rebar3 is stable now but ct support is not completed. We need ability to change cover spec file to work with 'coveralls', see https://github.com/erlang/rebar3/issues/1088


ResetModel - how to not update default value when doing a resetModel

I am using Angular 16.2.12 and Formly 6.3.0 and recently upgraded from Angular 15 and Formly 5
Question
Is it possible to do a resetModel call without updating the default values? We have a form that we can populate using a sample model that we use for testing. But since the upgrade to Formly 6 when trying to clear the form using resetModel(); it does not clear the model anymore it just resets it to the default values set by the populate method. is there any other way to do this other than setting each individual component or by having an "empty" model to reset it to?
I made a quick example of how it currently works here:
https://stackblitz.com/edit/angular-1c4vir?file=src%2Fapp%2Fapp.component.ts
Any assistance would be greatly appreciated.

use options.updateInitialValue to restore default value after calling resetModel.


Raise signals after every operation

Signals can be used to trigger a background notification to affected apps, and also used internally by the blueprint consumer.

This has been implemented for a while.


[MySQL] Can't use calendar in value panel for date\time\timestamp\datetime values

System information:

Win11
DBeaver CE 22.1.0

Connection specification:

MySQL

Describe the problem you're observing:
https://user-images.githubusercontent.com/43252954/171619109-e9027c7b-86a8-4c25-bb9d-73963b050d4a.mp4
Steps to reproduce, if exist:
Set focus to the datetime format and open Value panel.
Click on this icon for the calendar -> you'll get the error

Is the issue still actual for the latest DBeaver version? I can't reproduce it.


FOREIGN_KEY also as PRIMARY_KEY

Hello, I would need to specify FOREIGN_KEY also as PRIMARY_KEY. Something like
@Column(columnType = ColumnType.FOREIGN_PRIMARY_KEY)

There is example use case which would ensure items cannot be repeated in same group, but item can exist in more groups.
There is example use case. (http://sqlfiddle.com/#!7/eb95d/1)
CREATE TABLE group (
  id integer primary key,
  name varchar(30)
);

CREATE TABLE item (
  name varchar(30),
  group_id integer references group(id),

  primary key(name, group_id)
);

As I know this is now not possible. Is it possible to add this functionality?

I see what you mean. I don't see this making it in, because in general you would want to avoid using a primary key that is a foreign key.

@agrosner I don't think this is really something that should be avoided for all situations, I mean, isn't this a pretty standard (and correct) thing to do when creating a junction table for example? The foreign keys are usually the only columns of such tables, and together they make the primary key of the table. Currently, I need to have an auto incrementing - pointless - id on my junction tables, and I'm also missing the uniqueness constraint of having my foreign keys also set as primary keys.


Use Integer Value 1 instead of $.window.Node.ELEMENT_NODE #75

Since $.window.Node no longer exists, we can simply check for the integer value 1 when comparing to the node.nodeType. This fixes #75 and all plugins seem to be working as intended.
Thanks @jdan for helping out with the solution!

Thanks! :star2: This is what happens when you rush, break tests, then rush to fix them ;)
Closed via 492be3f94a024308008face23e1cee7c02d42fc2.


PyGrid Vanity API

Description
Implement all vanity API's needed by OpenGrid
Checklist:

[ ]  Domain

[x] Users

[x] Create User
[x] Get User
[x] Get Users
[x] Delete User
[x] Search Users
[x] Update User

[ ] Groups

[x] Create Group
[x] Get Group
[x] Get Groups
[x] Delete Group
[x] Update Group
[x] Delete Group

[ ] Roles

[ ] Create Role
[ ] Get Role
[ ] Get Roles
[ ] Update Role
[ ] Delete Role

[ ] Association Requests

[ ] Send Association Request
[ ] Receive Association Request
[ ] Respond to Association Request
[ ] Get Association Request
[ ] Get Association Requests
[ ] Delete Association Request

[ ] Initial Setup

[x] Create Initial Setup
[x] Get Initial Setup

[ ] Data Centric

[ ] Dataset

[ ] Create Dataset
[ ] Get Dataset
[ ] Get Datasets
[ ] Update Dataset
[ ] Delete Dataset

[ ] Tensor

[x] Create Tensor
[x] Get Tensor
[x] Get Tensor
[x] Update Tensor
[x] Delete Tensor

[ ] Infrastructure

[x]  Create Worker
[x] Get Worker
[x] Get Workers
[x] Delete Worker

[ ] Network

Just adding @tcp  in order to keep him updated about our progress! :)

Just adding @tcp  in order to keep him updated about our progress! :)


Reporter: Emit voilated rule name

The object that tslint passes to its formatter (or custom reporter)  contains getRuleName() function, which emits the rule name that has been violated. Here is the sample tslint's verbose formatter.
This is separate from the error message, and quite handy for users to figure out the rule name which cases the error (they may like to disable/change that rule setting or take some other action).
It would be really useful if we get a similar info from the object passed to our coffeelint custom reporter.
Also the --verbose switch for CLI usage can be considered to output the rule name with its description (node-jscs, for instance, support this behavior from CLI usage).
Thanks in anticipation. :)
(Related to https://github.com/clutchski/coffeelint/issues/160, originally issue: https://github.com/madskristensen/WebEssentials2013/issues/1717#issuecomment-70578444)

The raw reporter will display all of the JSON that reporters get access to.  Looks like you just need to use error.rule, just like you're already using error.lineNumber.
The way you're doing your reporter, you're in complete control over its output.  This was exactly what we were looking for. :smiley:  :+1:


Implication definition not clear

Hi I am using your tool to determine a calculation flow.
my input has some algebraic relations.
The output I get shows the determination flows in the following format.
X1, X25 in implication relation [X1, X25 => TGOX118] are known: X1, X25 ===> TGOX118
Now I don't understand what this implication relation is? does this mean  TGOX118 = X1*X25?
what about the flow
X13, X21, X29, X37 in implication relation [X13, X21, X29, X37 => TGOX5142231] are known: X13, X21, X29, X37 ===> TGOX5142231?
also I need only the output file and not the graph. so how to stop the program after generating the output file only?

Thanks for your interest in our tool.
Before I answer your question, I want to remind you that my tool only supports algebraic equations over the binary field (i.e., GF(2) = {0, 1}). If your algebraic relations are from a different mathematical space, please be careful when using Autoguess.
Regarding your question about new variables: When the algebraic degree is higher than two (you have multiplications or AND gates), Autoguess creates new variables to linearize these relations. For example, if you have a term like X1*X25 in your original relations, Autoguess will define a new variable such as TG0X118 = X1*X25, and treat this as an additional relation. If the system includes terms with an algebraic degree higher than 2, Autoguess linearizes the system by defining a sequence of new variables and implication relations recursively. For instance, for W = X*Y*Z, it first defines a new variable S = X*Y, and then another new variable T = S*Z to completely simplify the system.
If you want to see the intermediate system of equations that Autoguess creates you can activate the log by using --log 1 switch within your command.
Additionally, depending on whether you turn on preprocessing by using --preprocess 1, Autoguess may apply a Gaussian-elimination process to the original system of equations (again, I emphasize, over GF(2)) to pull out new relations/information from the original equations. Therefore, when you use Autoguess with a system of multivariate algebraic equations (over GF(2)), you may see several new variables in the determination flow.
About your question on turning off the figure generator: I know it can be time-consuming sometimes. So, you can turn off the figure generator by using --tikz 0.
Please let me know if this does not fully answer your question or if you have any more questions.
By the way, feel free to report any issues you find with Autoguess or let me know if you have any comments. It helps me improve the tool.

Thanks for your interest in our tool.
Before I answer your question, I want to remind you that my tool only supports algebraic equations over the binary field (i.e., GF(2) = {0, 1}). If your algebraic relations are from a different mathematical space, please be careful when using Autoguess.
Regarding your question about new variables: When the algebraic degree is higher than two (you have multiplications or AND gates), Autoguess creates new variables to linearize these relations. For example, if you have a term like X1*X25 in your original relations, Autoguess will define a new variable such as TG0X118 = X1*X25, and treat this as an additional relation. If the system includes terms with an algebraic degree higher than 2, Autoguess linearizes the system by defining a sequence of new variables and implication relations recursively. For instance, for W = X*Y*Z, it first defines a new variable S = X*Y, and then another new variable T = S*Z to completely simplify the system.
If you want to see the intermediate system of equations that Autoguess creates you can activate the log by using --log 1 switch within your command.
Additionally, depending on whether you turn on preprocessing by using --preprocess 1, Autoguess may apply a Gaussian-elimination process to the original system of equations (again, I emphasize, over GF(2)) to pull out new relations/information from the original equations. Therefore, when you use Autoguess with a system of multivariate algebraic equations (over GF(2)), you may see several new variables in the determination flow.
About your question on turning off the figure generator: I know it can be time-consuming sometimes. So, you can turn off the figure generator by using --tikz 0.
Please let me know if this does not fully answer your question or if you have any more questions.
By the way, feel free to report any issues you find with Autoguess or let me know if you have any comments. It helps me improve the tool.

Thank you for replying so clearly. I understand it now. I hope you don't mind If I modify your code (with citations) to adapt to my settings.

I will be happy if you find our tool useful.
If you believe your modifications could benefit other users, please feel free to send a pull request and contribute to the tool.


nit: go fmt use helper to isolate feature changes

Applying go fmt in use helper functions to isolate changes from PR https://github.com/gmeghnag/omc/pull/168

@gmeghnag would you mind taking a look here? No code changes, only format to help the reviewer in #168 . Thanks

btw, tests failing for no code changes, let's explore if it is something uncovered in recent MGs.


introspect: trailing comma is missing in createTablePKs

With introspect, schema indexes are created FK, PK, and Unique order.
However, since there is no trailing comma in the PK, an error occurs in Unique indexes.

missing in the createTablePKs functions, so it must be added.

should not be an issue on latest(0.30.0+), if it's still an issue - please reopen


amend skillshare and 112 links

URL changes.
I'm not 100% about Skillshare, I think the job posting on WeWorkRemotely, as pulled in through @whatasunnyday's script in #40 is actually about teaching jobs rather than working for Skillshare in the team.

Thank you! I wonder if we should remove skillshare? I'll merge your PR as is for now.

I guess it depends what we're looking to share here, is it jobs with companies, or jobs in general through companies who offer them? Like Crew that I added, they have vacancies within the Crew team, but also for those who want to use Crew to get work.
Worth a thought for more discussion as this grows

Just as a heads up, I think Upwork Pro is similar.
I think it makes sense to remove these jobs since you're not directly employed by them. The companies actually offering remote jobs should be added to the list.

I'm going to be working my way through the missing company profiles so I'll defo look to cutting out listings which aren't directly to be employed by the company


Add port argument

I added port argument according to these discussions https://github.com/feathersjs/feathers-socketio/pull/66
Please check.
Closes https://github.com/feathersjs/feathers-docs/issues/461

That looks great, thank you! Could you create another PR that adds the appropriate documentation below https://docs.feathersjs.com/api/socketio.html#appconfiguresocketiooptions--callback ? Then we can merge and release.

@daffl Okay, I've added information about the port argument.
https://github.com/feathersjs/feathers-docs/pull/527

Awesome, thank you! Released as v1.6.0 and documentation has been published as well.

Nice @kozzztya! Thanks for all the great PRs! 😄

You're welcome.


Fix styling of "Subject Lookup" on subjects index for mobile screens

This can be much improved.

Closed in 7d17f7e06f1309d5c1f15bd69fafaabc2198875e


Order edges by child time (and ID)?

Tskit orders edges by the time of the parent node (then by parent node ID). Jerome says

It's either because that's the order we want for simplify or that's the natural order coming out of coalescent Sims, I can't remember

However, when writing the gig.sample_resolve and gig.find_mrca_regions functions (both of which will be called repeatedly in forward sims), we only ever iterate over the edges by child ID:
https://github.com/hyanwong/GeneticInheritanceGraphLibrary/blob/ac05dd3bc93527759978807f23e2dc0ab56b9708/GeneticInheritanceGraphLibrary/graph.py#L457
This is because we pop a node off the stack and follow the edges up from that node to fill out the various node parent values.
I am strongly inclined to deviate from tskit here, and order the edges by child node ID / time. This has the additional benefit that each child can only have one parent at one position, so the edges for a given child cannot overlap. This provides a natural order to the edges for a child, without having to take into account the right-hand coordinate. It also means that we can interpret a sequence of edges for a given child as a list of intervals stored in a portion object.
Have you any thoughts, @duncanMR ?

That makes sense to me: we will generally want to make algorithms traverse rootwards when possible, to avoid dealing with duplications, so basing our conventions on the child nodes is sensible. Besides, once we have finished building a GIG via simulation, we can always sort the edges by parent node time and cache the resulting order of edge indices if that is more efficient for a given algorithm.

Great. I agree and will make this change.


Update to the systemjs 2.0.0 alpha build

This is half a third of the size of the former production loader :)

@guybedford Looking at this in the network tab: The SystemJS loader that this code requests is actually not minified. Is there a minified build available that can be requested via unpkg.com? Otherwise, maybe you just want to add one for exactly this use-case.


ipodtouch0218: Watch pickvocab.com

ipodtouch0218 requests the watch of the watch_keyword pickvocab\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

pickvocab\.com has been seen in 3 true positives, 0 false positives, and 0 NAAs.


CCA issues

There are actually three problems going on

The environmental labels are being dropped.  Here, you can see that no column or row labels are being passed into biplot_scores
The column rows don't actually match up with what is given in vegan.  In the example from SO, there are only 3 columns (i.e. 3 dimension). This probably should be consistent with the PC ids.
Its also becoming apparent that there is a sore need for biplots.  CA, RDA and CCA are essentially unusable without this functionality present in scikit-bio.  Its not trivial to tune these plots by hand.

@mortonjt are you available to work on these bugs (items 1 and 2) for 0.5.0? We're shooting for June 7 to cut the release.

The main reason why I haven't pushed a PR for this is because I'm still not
on the same page exactly is going on with the biplot_scores.  If @Jorge-C
is still available, I think it would be best if he could take a look.
On Wed, Jun 1, 2016 at 2:52 PM, Jai Ram Rideout<EMAIL_ADDRESS>wrote:

@mortonjt https://github.com/mortonjt are you available to work on
these bugs (items 1 and 2) for 0.5.0? We're shooting for June 7 to cut the
release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/scikit-bio/issues/1322#issuecomment-223136321,
or mute the thread
https://github.com/notifications/unsubscribe/AD_a3cZJIPKzaRRiX_mfW5EJLTILMkOCks5qHf83gaJpZM4H88E6
.

I don't think @Jorge-C is available. Is there anyone else that can maintain the CCA code in scikit-bio?

Alright. Let me take another look at it
On Jun 1, 2016 5:49 PM, "Jai Ram Rideout"<EMAIL_ADDRESS>wrote:

I don't think @Jorge-C https://github.com/Jorge-C is available. Is
there anyone else that can maintain the CCA code in scikit-bio?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/scikit-bio/issues/1322#issuecomment-223167305,
or mute the thread
https://github.com/notifications/unsubscribe/AD_a3Wh6fPRUwF6uPtLaYx1vTVHfBc-Xks5qHiitgaJpZM4H88E6
.

Thanks!

I think I figured out the index/column headers for rda - index=env_ids and columns=pc_ids.
The same is the same for cca, but for some reason, some of the rows (i.e environmental variables) are being excluded.  No idea why ...

OK.
Turns out to be in the unittest
We are ignoring the last column, which is why debugging this was really confusing.

Fixed in #1368, thanks @mortonjt!


Feature Request: Speed and Pitch Control

Would it be possible to add speed and pitch control? This functionality is not available in many players, and I think it would be a valuable addition. I would greatly appreciate it if you could consider implementing this feature.

Speed controller will be shipped with next version.

It has been included in v1.1


Tab auto-complete

Tab emotes
Tab usernames

2425ddbd2c3069ba1f4b2f7506f71711a87a8c74 implements the actual tabbing (tab key) to auto complete.
Does not implement actual values (that is emotes, names, ...) to auto-complete

More work done in 3bf111a0914b5afc9bcacfa3d4a762be763ee88a

currently ( 1d1f40bf441944c95c2303edc62ca8f1318b2659 ) the emote tabbing is not sorted

[ ] sort (alphabetically)

Tabbing doesn't seem to use the display name but rather the username. Is this a bug or intended?

Completed in 57e0e85d777d18374b168b39d6df85417878067f
Any further advanced features for auto-complete can be made as separate issues


Remove rustls-tls feature from reqwest

Having rustls-tls in features is basically an alias for 'rustls-tls-webpki-roots' according to here.
And there already is the feature to use 'rustls-tls-native-roots'.
Moreover 'rustls-tls-webpki-roots' pulls in a crate with a license considered copyleft. (webpki-roots)

I think you will need to update Cargo.lock as well.

I think you will need to update Cargo.lock as well.
Done. Thanks for the catch :)


Add /tables, /table/{}/data, /table/{}/info endpoints and fix unit tests

added /tables, /table/{}/data, /table/{}/info endpoints
fixed unittests for these routes
schemas for phenopacket v1 and 1000 genomes sample are added
updated README.md

Codecov Report

Merging #17 (c3cff99) into develop (b254754) will increase coverage by 4.46%.
The diff coverage is 83.33%.

@@              Coverage Diff              @@
##             develop      #17      +/-   ##
=============================================
+ Coverage      77.27%   81.73%   +4.46%
- Complexity        32       88      +56
=============================================
  Files             13       21       +8
  Lines             88      323     +235
  Branches           2       19      +17
=============================================
+ Hits              68      264     +196
- Misses            19       54      +35
- Partials           1        5       +4

Impacted Files
Coverage Δ

...it/dataconnect/model/OneThousandGenomesSample.java
0.00% <0.00%> (ø)

...gh/starterkit/dataconnect/model/PhenopacketV1.java
0.00% <0.00%> (ø)

...nect/utils/hibernate/DataConnectSQLiteDialect.java
0.00% <0.00%> (ø)

...nect/utils/hibernate/DataConnectHibernateUtil.java
75.00% <72.72%> (-25.00%)
:arrow_down:

...a4gh/starterkit/dataconnect/controller/Search.java
81.33% <81.08%> (-18.67%)
:arrow_down:

...arterkit/dataconnect/model/ListTablesResponse.java
83.33% <83.33%> (+83.33%)
:arrow_up:

...a4gh/starterkit/dataconnect/controller/Tables.java
88.13% <87.93%> (-11.87%)
:arrow_down:

.../starterkit/dataconnect/model/TableProperties.java
88.46% <88.46%> (ø)

...arterkit/dataconnect/utils/sql/SimpleSqlQuery.java
88.88% <88.88%> (ø)

...it/dataconnect/utils/sql/SimpleSqlQueryParser.java
92.59% <92.59%> (ø)

... and 5 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Hi Jeremy, I made the suggested changes -

the $ref field in /tables endpoint provides the complete URL (http://localhost:4500/table/phenopacket_v1/info).
data_model property no longer returns the id and json_data fields. It only returns the internal JSON fields like sample_id, sex, etc.
added enums to fields, sex, population_code, population_name, superpopulation_code and superpopulation_name in the one_thousand_genomes_sample json schema.

Please let me know if there is anything else.

updated README.md

Looks great, thank you @yash-puligundla !


One-way collision with kinematic body yields strange contacts and NaNs

move_and_collide_bug.zip
To reproduce: left click mouse and drag around to move the dragbody object using move_and_collide(). Turning off the collision layer so the dragbody object has no influence on the standingbody (there is code for this in the _ready() function) will cause collision points to appear all over the scene and when released the object might get a huge boost of speed or result in !v.is_finite is true errors. With the collision layer on, the dragbody body can influence and push the standingbody object and these errors no longer happen.
In default GodotPhysics3D neither of these things happen and the dragbody simply has no influence on the standingbody while frozen.

Apologies for the late triage, but it does seem like this has been fixed already, as part of updating Jolt itself in #800.
I'm guessing that it was probably jrouwe/JoltPhysics#949 that fixed it.

The fix for this can now be found in the latest pre-release, 0.13.0-beta1, which you'll find here.
Note that this version targets the just recently released Godot 4.3-beta1, so won't work with Godot 4.2. Also note that this version is not available through the Godot Asset Library, so you'll need to download it through GitHub.
A proper release will be out some time around Godot 4.3-stable.

I figured I'd give a minor update here, as 0.13.0-stable has just been released, so if you've been relying on 0.13.0-beta1 for this functionality until now I would highly recommend updating, as this most recent release includes some important bug fixes.
Note that the release on the Godot Asset Library is still pending approval as of typing this, so you might need to download it from the link above.


Make outcome printer print bs' tags from `Arity_1 back to [@bs]

cc @bobzhang

It is already in ocaml outcometree printer, I remember I posted a link before
https://github.com/BuckleScript/ocaml/blob/master/typing/oprint.ml#L192

done in bucklescript


Update budget by OS budget change events.

Instead of polling for latest usage, OS event based updates are used by default. #319

@bjjones can you PTAL?

Did you plan to add functionality for paging based off of budget changes?

Could you elaborate on this more?
 In both cases, Evict is called the same number of times. But I do see an opprotunity to unblock the main thread, when Evict is called, but only if nothing else uses the device (or it will block again).
In contrast, QueryVideoMemoryInfo gets called per ExecuteCommandLists, not per paging, so moving it away from the main thread holds more beneifit since it saves CPU cycles when under budget.
However, it does seem like a good idea to have async Evict since real world apps may keep busy enough to hide this latency.

Could you elaborate on this more?

I was thinking in the case that you had maxed out the budget and a significant budget change occurred (which can happen when changing focus to a different window) your memory residency would not be in the optimal state and wouldn't be fixed until the next ExecuteCommandLists call. Maybe this isn't impactful in reality - since ExecuteCommandLists is usually called at a granularity that would hide dropped frames - but could there be an ML/AI scenario that doesn't call so frequently?
I know Evict is very fast already so I don't know that there's really much value offloading it.

@bjjones We can't page-in heaps outside of ExecuteCommandList (ignoring resource creation), the budget change event could only evict older heaps. Since Evict() makes no guarentee heaps will be paged out sooner rather then later (speeding up the subsequent MakeResident), the benefit isn't obvious but I captured there issue as #478.


Create or alter statement breaks syntax highlighting

Issue Type: Bug
This simple SP breaks syntax highlighting:
create or alter procedure DoSomething
as
return

The SQL is actually valid, but the editor places red squiggles beneath every word from the "or" onward.
There is also a message:
Incorrect syntax: 'ALTER PROCEDURE' must be the only statement in the batch.

Expected: No error, and working syntax highlighting.
Azure Data Studio version: azuredatastudio 1.14.1 (f9b968c1ae514a0d1210788629e1c69125857279, 2019-12-24T02:42:26.303Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.82GB (15.58GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

I'm also encountering this, it's when connecting to an Azure SQL DB. I experience it for procedures, functions and views.
Seems it was previously reported at https://github.com/microsoft/azuredatastudio/issues/7745

This should be fixed in the next release of ADS - if you are able to verify this on the latest version of our insiders builds that would be great!
https://github.com/microsoft/azuredatastudio#try-out-the-latest-insiders-build-from-main


How to view MATTER messages [1.0]

Reproduction steps
I would like to check the contents of messages between nodes, but I cannot because they are encrypted.
Is it possible to check the contents of the messages using Wiresharkt or other means?
Bug prevalence
always
GitHub hash of the SDK that was being used
58cd5fd
Platform
darwin
Platform Version(s)
No response
Type
Security Issue
Anything else?
No response

https://github.com/CHIP-Specifications/matter-dissector is a Wireshark dissector for Matter, but that repository is not public.
https://gitlab.com/wireshark/wireshark/-/merge_requests/9865 looks like it got merged recently, but does not support the decryption bits yet.

Thank you for confirming.
Is there any software other than Wireshark that can confirm the contents of the message?

If you control the communication endpoints, you could run them in tracing mode so they log the payloads....
Otherwise, nothing I am aware of.


Release 2.5: Update AI lib to 2.3.3

Signed-off-by: Rastislav Wagner<EMAIL_ADDRESS>

@rawagner @jtomasek Please no PRs directly to release-2.5 yet. This breaks our automated fast-forwarding.


Quote some variables

Some string variables in the bashdot bash script are not wrapped with
double-quotes.  This can result in unwanted behavior, such as issue #2.
Put double-quotes around many of these variables in places where they
seem to be needed.

@jj-kim Thanks, tested and it resolved #2.
Thanks for the contribution


update to include info about Planner Syncing

[Enter feedback here]
Please update these instructions to include information about the ability to sync the message center with Planner

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

ID: 29f3771a-247c-4f85-992a-e671285c70bd
Version Independent ID: f3c2890a-2470-9349-ff77-924cf8566365
Content: Manage messages in Message center - Microsoft 365 admin
Content Source: microsoft-365/admin/manage/manage-messages.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@DeanGross Thank you for your feedback. We have added this information to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3315
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @DeanGross Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[test] Add PrgEnv-nvidia to halo_cell_exchange check

halo_cell_exchange check PrgEnv-nvidia
https://jira.cscs.ch/browse/UES-1641

@jgphpc There are heaps of failures for this test. At least, the performance references should be adjusted to a less optimistic value.

@jgphpc I don't think there is any performance issue. I just think that the tolerances are way too strict for this case. I also vote for relaxing them.

Codecov Report

Merging #2029 (10c8bb1) into master (30611f0) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2029      +/-   ##
==========================================
+ Coverage   87.48%   87.53%   +0.05%
==========================================
  Files          50       50
  Lines        8853     8889      +36
==========================================
+ Hits         7745     7781      +36
  Misses       1108     1108

Impacted Files
Coverage Δ

reframe/__init__.py
81.81% <100.00%> (ø)

reframe/core/meta.py
98.96% <100.00%> (+0.16%)
:arrow_up:

reframe/core/pipeline.py
91.68% <100.00%> (+0.05%)
:arrow_up:

reframe/core/variables.py
96.09% <100.00%> (+0.03%)
:arrow_up:

reframe/utility/sanity.py
97.79% <100.00%> (+0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


when responding with gzip compression traefik converts HTTP 201 to 200 v1.3.5

BUG v1.3.5
What did you do?
POSTED to traefik, traefik proxied request to my backend which responded with a 201.
Traefik then responded to my client with a 200
What did you expect to see?
HTTP 201 response
What did you see instead?
HTTP 200
Output of traefik version: (What version of Traefik are you using?)
v1.3.5
wtf.zip
to reproduce:

build dockerfile
update k8s.yaml to use your docker image
deploy k8s.yaml -> kubectl create -f k8s.yaml
update test.py to point to the ingress
send request.
notice the response is a 200 not a 201

special notes:

this seems to be a problem specific to gzip compression. I noticed this problem by trying to front core-os clair with traefik.. to reproduce the problem i created a dummy python flask service that behaves like clair, and with this i was able to reproduce the problem

Tested with different versions of Traefik. Seems the bug was introduced since v1.3.4. It might be related to the fix for the double-compression issue.

@kevinjqiu yes and I have already fixed this bug 😃

Thanks @ldez  that's excellent turnaround :D


Make release names uname -r compatible

Would make it easier to download and install nebula on a variety of platforms and architectures.
For example I would like to use this oneliner whether I'm downloading Nebula on my M1 Mac or my x86_64 Linux server:
curl -L https://github.com/slackhq/nebula/releases/download/v1.3.0/nebula-$(uname)-$(uname m).tar.gz | tar -xvz; chmod +x nebula nebula-cert; chown root:root nebula nebula-cert
Right now this will translate to nebula-linux-x86_64.tar.gz while the release is named nebula-linux-amd64.tar.gz

It's not one line, but this is what I do:
https://github.com/wadey/nebula-tools/blob/85c1170b7ea8ddf4b2ee0c4400b57bda1ae89190/nebula-release.sh#L4-L9

This is awesome, then I can get away with the following:
curl https://raw.githubusercontent.com/wadey/nebula-tools/master/nebula-release.sh -o nebula-release.sh; chmod +x nebula-release.sh; ./nebula-release.sh

Are there any plans to include nebula in the Homebrew, dnf  and AUR repos?

@SebDanielsson See https://github.com/slackhq/nebula/pull/414 for DNF and Arch repos.

Thanks for the help guys, hoping for a release on Homebrew as well😃

Thanks for the help guys, hoping for a release on Homebrew as well😃

I don't use it myself, but it's there already:
https://formulae.brew.sh/formula/nebula#default


Docs Missing Deny and Allow for PermissionOverwrites

Please describe the changes this PR makes and why it should be merged:
The docs are missing for PermissionOverwrites allow and deny property.
Semantic versioning classification:

[ ] This PR changes the library's interface (methods or parameters added)

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[x] This PR only includes non-code changes, like changes to documentation, README, etc.

Pretty sure those are chosen to be undocumented. And if anything they should probably be documented as private.

@iCrawl Why would they be private/undocumented? It seems that people use them, should they not know of their existance? lol

There's usually always a very good reason certain things are not public API.

they are private because we don't want people do overwrite them and fuck things up

@devsnek Then could we not just do @readonly or something of the sort? I mean, if nobody knows about it why implement it at all?

You clearly have no idea of how this works Fire.
If something isn't documented, doesn't mean you can't use it. You just shouldn't rely on it, because we don't know if we are going to remove it or not, or what gus said.
The same concept applies to all the REST methods. You can use them, but they aren't documented.

it isn't readonly, so we can't mark it readonly

Oh wow, I feel dumb. Yeah sorry. Still seems kinda important to add this though, as it would be the users fault if they fk things up?

 I don't feel like it should be in the public API.


When will the training datasets be released?

Thanks for this great work!
When will the datasets be released, especially the training trajectory?
Could you release the datasets in very near future?

@zzjchen
Hi, thanks for your interest in VAB. Our repo is still under construction, and we expect to first accomplish the quick start for evaluation. The training set is expected to be ready and released in about 1 month.

Thanks.
I'm looking forward to it!

Thanks for this great work!
Is there any plan to release the training data recently? We plan to use the VAB testing results as evaluation content.

Seconding this issue, as the paper presents the curated training trajectories as one of the 3 main contributions.
It would be great for the community to have these publicly available.

Hi @Xiao9905 !
Is there any update on the timeline of releasing the training datasets ?

Would like to know about the training sets too!

Thank you guys for your interest in the training datasets! We will release our dataset once our submission receives its decision :)


Invalid module import Syntax

Invalid module import in the new API.
Should be "." but is instead "/"

After serach/replace (eliona/api_client // eliona.api_client) there are other syntax errors in the modules.

This repository is created with an old OpenAPI generator version. All code problems have to changed in the repositoy by hand. There is a second repository using the newest generator version. Please try to use this one: https://github.com/eliona-smart-building-assistant/python-eliona-api-client2

This repository is created with an old OpenAPI generator version. All code problems have to changed in the repositoy by hand. There is a second repository using the newest generator version. Please try to use this one: https://github.com/eliona-smart-building-assistant/python-eliona-api-client2


Generate Debug Symbols

We need to figure out how to provide proper debug support. In particular, we need to look into generating debug symbols. Right now, the debug support in VS works via relying onto the debug support for the generated C# code. This becomes less and less reliable as we do more complex rewrites (like e.g. generating unique symbol names as we inline code blocks).

The time line for this may be fairly long, as it is a larger piece and currently not staffed. Hence the help wanted. :)


update dictionary and lexicon

Updated the dictionary in two ways: added words from strings.xml, both Icelandic and English, and replaced the core dictionary with the newest version from January 2022, twice the size of the current dictionary. Adapted the reading methods for dictionary and lexicon accordingly, fixed one typo in strings.xml.

Overall LGTM. But what do we do with the other dicts and lexicons in src/main/res/raw that are not any more used ?
Here we have at least these:
abbr_regex_1.csv
lexicon.txt
ice_pron_dict_standard_clear_2102.csv
lexicon_v2006.txt
lexicon_v2106b.txt

Maybe remove ?


Add context around the Microsoft.KeyVault/vaults/deploy/action permission

Feedback about the section:
Grant access to the secrets
The user who deploys the template must have the Microsoft.KeyVault/vaults/deploy/action permission for the scope of the resource group and key vault. The Owner and Contributor roles both grant this access. If you created the key vault, you're the owner and have the permission.
This part is not very clear in the documentation. Here, I found some additional information that explains why this permission is required:
"...the user specifies the resource ID of a vault that contains secrets they want to use for a deployment (either a cert for a VM or a secret to be substituted in a template). There needs to be some mechanism that ensures the user cannot just specify the resource ID of some vault that they don't own. We could have said that the user must have Microsoft.KeyVault/vaults/write permission on a vault in order to reference it in a deployment. But that would have meant that only users who can actually modify the vault would be able to do deployments. So instead we check that the user has Microsoft.KeyVault/vaults/deploy/action. That way the vault owner can assign the deployment permission to someone without making them an owner of the vault."
and...
"The user does not need to be in the access policy for the vault. In the ARM template scenario, the call to Key Vault is being made using ARM's identity, not the user's identity. It's as if you had put ARM's AAD app's object ID in the access policy with permission to get secrets. But since ARM uses the same identity for all customers, there needs to be a mechanism to ensure you can't ask ARM to retrieve secrets from another customer's vault. That mechanism is the Microsoft.KeyVault/vaults/deploy/action ARM RBAC permission."
I think it provides more context if this information is added to this MicrosoftDocs document.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ecb6fa26-f01e-e9dd-efad-74c0898e2027
Version Independent ID: 4582bdc2-1d26-067d-4343-a63736d4bf8f
Content: Key Vault secret with template - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/key-vault-parameter.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@DonKoning - thanks for this feedback. I'll add that context.

This update should get published later today.
#please-close


cleanup ansible molecule tests: remove invalid and not used task

Description of the change:
This task is broken and it is not working. To test that, add this task in the Ansible Memcached sample and deploy it to test the call. Then, you will see that it fails. More info:  https://github.com/operator-framework/operator-sdk/issues/4123
Motivation for the change:
Cleanup the tests and remove unused/invalid code

@camilamacedo86 where is this check being run now?

Hi @estroz,
The check Check if jmespath is installed has not been called and its code is invalid. We can easily check it by looking on the e2e-ansible task output of any CI job. E.g search for Check if jmespath is installed in https://travis-ci.com/github/operator-framework/operator-sdk/jobs/420868963

Hi @estroz,
I will get it merged because it is very small and has the ok from @asmacdo and @fabianvf in the  #4123.
However, please feel free to ping me if you think that we need to revert it. 👍


Updated: mpc-hc 1.9.1

It seems to point to the unofficial release. Manifest is up-to-date with the latest version posted on the official site/github


GH Action Tests Failing?

Anyone having issues with the NPM release 2.6.0? If possible, fix the tests

Is this fixed? I recently learned about pactum and would love to help with the open issues.