distinct is not an operator

Why is distinct list as an operator? It is a normal function, or?

@tjeske : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@Minewiskan : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Entering an agenda through the UI after uploading an html file does not work

Describe the issue
If the previous revision of a session's agenda was an uploaded HTML file, trying to revise it through the "Enter agenda" option at /meeting//session/<session_id>agenda fails like
[17/Jul/2024 08:13:55] "GET /meeting/120/session/33286/agenda HTTP/1.0" 200 23940
ERROR: django.request:241: Internal Server Error: /meeting/120/session/33286/agenda
Traceback (most recent call last):
  File "/home/dev/.local/lib/python3.9/site-packages/django/core/handlers/exception.py", line 55, in inner
    response = get_response(request)
  File "/home/dev/.local/lib/python3.9/site-packages/django/core/handlers/base.py", line 197, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/workspace/ietf/meeting/views.py", line 2900, in upload_session_agenda
    save_error = handle_upload_file(file, filename, session.meeting, 'agenda', request=request, encoding=encoding)
  File "/workspace/ietf/meeting/utils.py", line 752, in handle_upload_file
    file.open()
AttributeError: '_io.BufferedReader' object has no attribute 'open'
ERROR: django.request:241: Internal Server Error: /meeting/120/session/33286/agenda
Traceback (most recent call last):
  File "/home/dev/.local/lib/python3.9/site-packages/django/core/handlers/exception.py", line 55, in inner
    response = get_response(request)
  File "/home/dev/.local/lib/python3.9/site-packages/django/core/handlers/base.py", line 197, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/workspace/ietf/meeting/views.py", line 2900, in upload_session_agenda
    save_error = handle_upload_file(file, filename, session.meeting, 'agenda', request=request, encoding=encoding)
  File "/workspace/ietf/meeting/utils.py", line 752, in handle_upload_file
    file.open()

As a workaround, writing the new agenda to a txt or md file and uploading it rather than entering it through the form allows the new version to post.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

@flynnliz


fix: .gitignore carries over when building from the dart_frog dockerfile

I used the envied package to handle environment variables, and naturally my .env file was in my .gitignore.
However when trying to build the container on google cloud run, building kept failling. And turns out I had to remove the .env file from my gitignore in the build directory and only then it would build properly.
I don't know whether this is an issue here or with docker or intended, but it gets annoying having to uncomment out manually the ignored files in build/.gitignore.

Docker does not use the .gitignore and neither does Dart Frog, maybe something in Google Cloud Run is using it to ignore files?
I have tried it locally and I am able to build a docker image that uses files that are in my .gitignore. If you can provide a reproduceable example for me to look at, that would be super sweet!

Closing this as we are not able to reproduceable the issue, as mentioned above it seems to work as intended


Change the input layer from a saved TF model

Is there any way to load a Tensorflow model, change the input layer and save again the new model? (I would like to convert a model from NHWC vs NCHW)

@AlexisPapaioannou
Could you please refer to Save and load model  , for any further queries you may open this issue in TF discussion forum  as there is a wider community to respond. Thank you!

Thanks, @sushreebarsa !


solve on MethodCallExpr will result in StackOverflowError when parameter type inherits from a super class with the same name in a different package

Hello,
We've run across a code (anti-)pattern that is tripping up this library.  The symbol solver does not appear to handle class inheritance where the same name is shared across different packages.
I put together a small project (attached) to illustrate.  I have a Widget class in com.foo that inherits from a Widget class in com.foo.base.  When trying to solve a MethodCallExpr on a method involving this type, it gets stuck in a recursive loop as the call to JavaParserClassDeclaration.getSuperClass() always returns the current class instead of the actual superclass with the same name.
javasymbolsolver-issue.tar.gz
package com.foo.base;

public class Widget {}

package com.foo;

public class Widget extends com.foo.base.Widget {

public static void main(String[] args) throws IOException, ParseException {
    File src = new File(JAVA_ROOT);
    CombinedTypeSolver combinedTypeSolver = new CombinedTypeSolver();
    combinedTypeSolver.add(new ReflectionTypeSolver(true));
    combinedTypeSolver.add(new JavaParserTypeSolver(src));

    CompilationUnit compilationUnit = JavaParser.parse(new File(CLASS));

    JavaParserFacade parserFacade = JavaParserFacade.get(combinedTypeSolver);
    MethodDeclaration methodDeclaration = compilationUnit.getNodesByType(MethodDeclaration.class).stream()
          .filter(node -> node.getName().equals("doSomething")).findAny().orElse(null);
    methodDeclaration.getNodesByType(MethodCallExpr.class).forEach(parserFacade::solve);
}

public void doSomething() {
    doSomethingMore(new Widget());
}

public void doSomethingMore(Widget value) {
    // does something
}

Results in:
Exception in thread "main" java.lang.StackOverflowError
at com.github.javaparser.ast.NodeList.iterator(NodeList.java:123)
at com.github.javaparser.symbolsolver.javaparsermodel.contexts.ClassOrInterfaceDeclarationContext.methodsByName(ClassOrInterfaceDeclarationContext.java:139)
at com.github.javaparser.symbolsolver.javaparsermodel.contexts.ClassOrInterfaceDeclarationContext.solveMethod(ClassOrInterfaceDeclarationContext.java:152)
at com.github.javaparser.symbolsolver.resolution.MethodResolutionLogic.solveMethodInType(MethodResolutionLogic.java:476)
at com.github.javaparser.symbolsolver.javaparsermodel.contexts.ClassOrInterfaceDeclarationContext.solveMethod(ClassOrInterfaceDeclarationContext.java:163)
(last 2 lines repeating)
Thanks,
Aaron

Thank you so much for reporting this

I have added some tests to verifying this but I could not reproduce it. I think the bug has been corrected in the previous months. I will commit the new tests to prevent regressions.


Feat ChatGPTApi

Ref : [Bug] request body缺少參數max_tokens #3056
[+] feat(openai.ts): add support for max_tokens configuration in ChatGPTApi

@Yidadaa  its error because need calculate about this

example for models gpt-3.5-turbo-16k
max_tokens is 16385
if put 8k in this one it will be fine

nvm it giving stucks as paused in [OpenAI] request response content type:  text/event-stream for completion when there is a lot messages from openai response


Update automatic.html

There is syntax error at line 47.

There is syntax error on line 47.

odd, thank you for the fix.


Panic with queries with multiple COUNT DISTINCT aggregates on dictionary values, and a group by

Describe the bug
DataFusion panic's when runnning a query like
select count(distinct column1), count(distinct column2) from test group by column1;

Where column1 and column2 are Dictionary
To Reproduce
❯ create table test as values (1, arrow_cast('foo', 'Dictionary(Int32, Utf8)')), (2, arrow_cast('bar', 'Dictionary(Int32, Utf8)'));
0 rows in set. Query took 0.002 seconds.

❯ select * from test;
+---------+---------+
| column1 | column2 |
+---------+---------+
| 1       | foo     |
| 2       | bar     |
+---------+---------+
2 rows in set. Query took 0.001 seconds.

❯ select count(distinct column1), count(distinct column2) from test group by column1;
thread 'tokio-runtime-worker' panicked at /Users/alamb/Software/arrow-datafusion/datafusion/common/src/scalar.rs:1846:18:
Unsupported data type Dictionary(Int32, Utf8) for ScalarValue::list_to_array
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
External error: Join Error
caused by
External error: task 66 panicked

Expected behavior
The test should produce the same as when the values are cast to string:
❯ select count(distinct column1::varchar), count(distinct column2::varchar) from test group by column1;
+------------------------------+------------------------------+
| COUNT(DISTINCT test.column1) | COUNT(DISTINCT test.column2) |
+------------------------------+------------------------------+
| 1                            | 1                            |
| 1                            | 1                            |
+------------------------------+------------------------------+

Additional context
I believe this is a regression introduced in https://github.com/apache/arrow-datafusion/pull/7629 (not yet released)
We found this downstream in IOx

Other than Dictionary, there are also many types I did not support yet, since I am not sure whether we need them all, so I only supported the type that is used (aka pass all test).

btw I forgot to fix this "Unsupported data type {:?} for ScalarValue::list_to_array" to  "Unsupported data type {:?} for ScalarValue::new_list"

In this case we may need to change to logic of state() https://github.com/apache/arrow-datafusion/blob/a9d66e2b492843c2fb335a7dfe27fed073629b09/datafusion/physical-expr/src/aggregate/count_distinct.rs#L145-L149

Other than Dictionary, there are also many types I did not support yet, since I am not sure whether we need them all, so I only supported the type that is used (aka pass all test).
Yeah, there is clearly a test coverage gap -- I'll take a look into what is going on

Here is proposed fix: https://github.com/apache/arrow-datafusion/pull/7989


cannot refresh when file update

reproduce：
step1: create a file "a.txt"
step2: use hexdump
step3: delete "a.txt"
step4: create a new file "a.txt"
step5: use hexdump
now,  you can see, new hex file "a.txt.hexdump" same to old file "a.txt.hexdump"
and hexdump plugin show an error: "ENOENT: no such file or directory, watch 'd:\xxxx\a.txt'"
if i reload vscode, hexdump can show correct content!

Same thing happens to me.

Happends to me, using hexeditor instead.


monocular RGB images as inputs

Can I use only RGB as input when I choose mono as the mode? If my dataset only contains monocular RGB images, the program seems to report an error.

Hi,
Sure, you can run in monocular mode, for example, please refer to Replica-Mono cases. After preparing the config file, and also, setting depth to 0 and pose to identity matrix in dataset, you can run the slam in mono mode.
Best,
Youmin


Files in nar serialisation are not sorted.

While the c++ implementation inserts the file names into a sorted map, the go implementation just emits them as returned by the os:
https://github.com/nix-community/go-nix/blob/1f9567c0a1e5fa52ca0401bf1241d1ec7643c097/pkg/nar/dump.go#L105
This would likely break if the filesystem returns file in a different order than how the std::map is sorted.

No, files is coming from the os.ReadDir call above:
https://github.com/nix-community/go-nix/blob/1f9567c0a1e5fa52ca0401bf1241d1ec7643c097/pkg/nar/dump.go#L99
Entries from os.ReadDir are sorted by filename:
❯ go doc os.ReadDir
package os // import "os"

func ReadDir(name string) ([]DirEntry, error)
    ReadDir reads the named directory, returning all its directory entries
    sorted by filename. If an error occurs reading the directory, ReadDir
    returns the entries it was able to read before the error, along with the
    error.


Arm: Implement mul opcodes (halfwords and non-halfwords)

This PR brings the implementation of the various opcodes for multiplications in arm-mode, I decided to keep the already present ArmModeInstruction::Multiply type and extending it a little with a ArmModeMultiplyKind to allow for a basic way of differentiating the opcodes.
Overflows are checked using u64|u32::overflowing_mul(op1, op2) which conveniently returns a tuple containing the mul result and whether an overflow occurred. I chose this way as it clearly shows the intent of what's happening there, perhaps we could only use such function when the S flag is set, otherwise there's really no point in using that. (It might bring extra overhead for no reason, but you guys tell me if that's any good to keep it like so).
Other than that, I don't really have anything important here to address. The opcodes are quite similar to each other, and it's not guaranteed we're gonna need them all, but still I brought them as it required little effort to do so.
As for the testing side I tried to test the most common case for each one, some need to check for condition flags, some don't and so on...
Looking forward to improve this as much as we can! :v:

Codecov Report
Patch coverage: 100.00% and project coverage change: +4.47 :tada:

Comparison is base (6a17aed) 58.36% compared to head (5d7cac9) 62.84%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #196      +/-   ##
==========================================
+ Coverage   58.36%   62.84%   +4.47%
==========================================
  Files          39       39
  Lines        2858     3189     +331
==========================================
+ Hits         1668     2004     +336
+ Misses       1190     1185       -5

Impacted Files
Coverage Δ

emu/src/bitwise.rs
100.00% <100.00%> (ø)

emu/src/cpu/arm/instructions.rs
81.76% <100.00%> (+4.64%)
:arrow_up:

emu/src/cpu/arm/mode.rs
84.21% <100.00%> (+7.06%)
:arrow_up:

emu/src/cpu/arm/operations.rs
91.29% <100.00%> (+5.94%)
:arrow_up:

emu/src/cpu/arm7tdmi.rs
73.07% <100.00%> (+0.14%)
:arrow_up:

emu/src/cpu/thumb/operations.rs
85.86% <100.00%> (+0.24%)
:arrow_up:

... and 1 file with indirect coverage changes

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


Use caret requirements on -registry

Part of #76

Codecov Report

Merging #77 (fd331e0) into master (fc365b8) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
- Coverage   62.28%   62.25%   -0.03%
==========================================
  Files          25       25
  Lines        3603     3603
  Branches      150      152       +2
==========================================
- Hits         2244     2243       -1
- Misses       1358     1359       +1
  Partials        1        1

Impacted Files
Coverage Δ

src/lib.rs
50.47% <0.00%> (-0.05%)
:arrow_down:


[stable/neo4j] Remove doubled -- for -a

@mneedham @maorfr
What this PR does / why we need it:
--a works because the full flag is --address and any substring of that works, but the proper short flag is -a.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @cpanato

This PR should be closed because it has conflicts, and because the head of the repo already has the change in place: https://github.com/helm/charts/blob/master/stable/neo4j/templates/NOTES.txt#L22


Number of IP Groups

"You can have a maximum of 100 IP Groups per firewall with a maximum 5000 individual IP addresses or IP prefixes per each IP Group."
I am told that this is 200, not 100. See this: https://learn.microsoft.com/en-us/azure/firewall/ip-groups#ip-address-limits

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

ID: 25ad0dd7-0d5d-6fa8-12a2-ef6c42764ec9
Version Independent ID: 00797156-552a-03f8-1898-f59bdd64dbe3
Content: IP Groups in Azure Firewall Policy
Content Source: articles/firewall-manager/ip-groups.md
Service: firewall-manager
GitHub Login: @vhorne
Microsoft Alias: victorh

@gamullen , thank you for your feedback. We'll review this and get back to you shortly!

@gamullen , we've updated the doc to reflect the correct limit number of 200 and it will go live soon.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Please update jq on the trident container on docker.io

We got informed from our security that we have to upgrade the trident container due to a vulnerability issue.
Right now we are using Trident 20.04 and our security told us that CVE-2016-4074 is critically for us. I tried to create a ticket on Docker Hub, but this is not possible. Therefore I'm creating a Issue on Github project itself. If I'm wrong here please give me a hit where I can post this issue. Thanks.
Otherwise please update the container :)
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2016-4074
Cheers,
Daniele

Hi Daniele,
Can you have your security team double-check their findings? Trident 20.04.0 uses the Alpine 3.11 image which has the jq 1.6-r0 version as an installed package.

Hi gnarl,
Correct installed is 1.6-r0  but clair is telling that it's fixed on JQ Version 1.6_rc1-r0
Note that this vulnerability was originally given a CVSSv2 score of 7.8 by NVD but was subsequently reclassified as  High by alpine

@CookieMonster70, thanks for the additional information. We will investigate further and update here with our plan.

This is false positive from your scanning tool, due to the way that jq is versioned:
https://github.com/stedolan/jq/issues/1406#issuecomment-543892833
Alpine 3.11 contains 1.6 stable, which is even more recent than the release candidate that the scanner is asking for:
https://pkgs.alpinelinux.org/package/v3.11/main/x86_64/jq

@CookieMonster70 Is it okay to close this issue?

@CookieMonster70, let us know if you still view this as an issue and we can open the issue again.


Photo upload problem for some android device

On some mobile phones, the photo is either empty or not uploaded at all. What should we do for this?

@irfansunar did you find a solution for this?

@irfansunar did you find a solution for this?

@irfansunar did you find a solution for this?

Unfortunately, we could not find a solution, some android devices cannot upload photos.


Steel dust registered in dustIron dictionary

This creates some issues with mod interactions, specifically the TE Induction Smelter, which registers recipes to add additional carbon to get steel ingots from them... (may already be fixed, running <IP_ADDRESS>)

After some testing it would appear this is bunk... But there's still something odd going on with the TE smelter... Lemming said that it was, MineTweaker doesn't seem to think it is...  Unless it's registered during some early stage when TE generates its recipe list, then unregistered later...


Initial skeleton for Tr1d1um (WIP)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e086f51). Click here to learn what that means.
The diff coverage is 12.82%.

@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   12.65%
=========================================
  Files             ?        2
  Lines             ?       79
  Branches          ?        0
=========================================
  Hits              ?       10
  Misses            ?       69
  Partials          ?        0

Impacted Files
Coverage Δ

src/tr1d1um/tr1d1um.go
0% <0%> (ø)

src/tr1d1um/protocol_converter.go
100% <100%> (ø)

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 e086f51...9601536. Read the comment docs.


No Devices Appear

I have created an iOT project and linked to Tuya app. I see the device in the cloud project.
The plugin-loads and says it found the device, but it doesn't show as an accessory:
TuyaOpenAPI response: {"result":[{"category":"zndsj","devices”:[“xxxxxxxxxxxxxxxxx”],”functions":[{"code":"up_down","desc":"up down","name":"up down","type":"Enum","values":"{"range":["up","down","stop","continue"]}"},{"code":"level","desc":"level","name":"level","type":"Enum","values":"{"range":["level_1","level_2","level_3"]}"},{"code":"percent_control","desc":"percent control","name":"percent control","type":"Integer","values":"{"unit":"%","min":1,"max":100,"scale":0,"step":1}"},{"code":"child_lock","desc":"child lock","name":"child lock","type":"Boolean","values":"{}"},{"code":"switch","desc":"switch","name":"switch","type":"Boolean","values":"{}"}],"product_id":"8qfqlfvwyx3wcji6"}],"success":true,"t":1705888272597,"tid”:”xxxxxxxxxxxxxxxxxxxx”} path = /v1.0/devices/functions
Adding: TV Lift (zndsj / xxxxxxxxxxxxxxxxxxxx)
TuyaOpenAPI request: method = post, endpoint = https://openapi.tuyaus.com, path = /v1.0/iot-03/open-hub/access-config, params = null, body = {"uid":"az17057694925024xhYh","link_id":"b5cd3a80-b8c8-11ee-a9bc-9b5d9986191d","link_type":"mqtt","topics":"device","msg_encrypted_version":"1.0"}, headers = {"t":"1705888272690","client_id”:”xxxxxxxxxxxxxx”,”nonce”:”xxxxxxxxxxxxxxxxxxx”,”Signature-Headers":"client_id","sign”:”xxxxxxxxxxxxxx”,”sign_method":"HMAC-SHA256","access_token”:”xxxxxxxxxxxxxx”,”lang":"en","dev_lang":"javascript","dev_channel":"homebridge","devVersion":"1.5.0"}

I have the same issue, kinda. I am using homebridge-tuya. I connect the plugin to IoT account/project. I have extended the subscription for another 6 month, but the devices are not shown in HB.


bookinfo build: update readme to describe build process

Based on @ericvn https://github.com/istio/istio/pull/28308#issuecomment-718794675
Inspired from https://github.com/istio/istio/pull/24967#issuecomment-651233350

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
41765d8f9bb32532c15d1e65c9d0f247627e6a7f
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-multicluster-k8s-tests_istio

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
41765d8f9bb32532c15d1e65c9d0f247627e6a7f
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
99e99d8405bdae2d8fa353a0bedde3923bb4b353
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-multicluster-k8s-tests_istio

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
99e99d8405bdae2d8fa353a0bedde3923bb4b353
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-multicluster-k8s-tests_istio

@shamsher31: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
99e99d8405bdae2d8fa353a0bedde3923bb4b353
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-multicluster-k8s-tests_istio


Add example standards document for logging

Is it worth merging this back @eldeal @necrophonic. It seems pretty complete an is inline with out internal docs


Update datetime-transact-sql.md

fix error

@kscale : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

Investigating. The behavior appears different depending on the locale ID. Verifying.

@kscale  - thank you for the submission, datetime functions differently, depending on the locale of the instance of SQL Server. For some cases, the example works as written, so we will not publish the proposed update.
Pease use these alternative data types:
See this note from the article:

[!NOTE]
Use the time, date, datetime2 and datetimeoffset data types for new work. These types align with the SQL Standard. They are more portable. time, datetime2 and datetimeoffset provide more seconds precision. datetimeoffset provides time zone support for globally deployed applications.

#please-close

We have an internal updating being reviewed to clarify the sample for the international audience: https://github.com/MicrosoftDocs/sql-docs-pr/pull/21392


Propose to replace argparse with docopt

I have some positive experience with docopt crate.
I propose to replace argparse with docopt. I can do it if you agree.

Ok - i'v looked at it but it seemed little awkward, but positive experience is positive experience.


List of missed appointments

Create a new route in Appointments that lists the missed appointments. (suggested: /appointments/missed). The purpose for this is to create a mechanism for followup.

I can look into this one if no one else has it.

Go for it.
Typed or dictated imperfectly on my iPhone<PHONE_NUMBER>)

On Mar 14, 2016, at 10:37 PM, Pete<EMAIL_ADDRESS>wrote:
I can look into this one if no one else has it.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/HospitalRun/hospitalrun-frontend/issues/351#issuecomment-196672398

Assuming @pete-the-pete has this.

Yep.  I'll take a crack at it in the next few days.

@tangollama I have started; I created a new applications/missed pod, and have looked at the other applications code for inspiration.  I wanted to get your thoughts on my approach and assumptions, and maybe ask a few questions, before getting too far into it.
Assumptions/Requirements

this will add an 'appointments/missed' url
it will also add a 'Missed' sub-nav item in the the left menu
each missed appointment will have the edit and _delete _ functionality
the missed appointments will not have the add visit functionality

Approach
This will be very similar to the other appointments code; the route will extend AbstractIndexRoute and load the data; the controller will set the flags for the buttons, and the template will loop over the data, using the appointments' item/template.hbs for each row.
A 'missed' appointment will be any which has endDate < Date.now() && status !== canceled
Questions
Is the 'missed' logic correct?
Do we want to limit the functionality at all (not allowing adding a visit)?
Is the right approach to define the _filterParams in the appointments/missed/route?
Thanks for any feedback.


audit: restore check that was lost in #927

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

This audit check ensures that certain build dependencies
are explicitly marked either as :build or :run.
It seems to have been lost in #927.
It was also adjusted in #1290.
I tried adding a test in https://github.com/scpeters/brew/commit/fd6e1896efcc5b6f4e3582f93719e7d10751515f, but it doesn't work because the test fixture can't load the cmake formula:
  1) Failure:
FormulaAuditorTests#test_audit_build_time_deps [/usr/local/Homebrew/Library/Homebrew/test/test_audit.rb:484]:
--- expected
+++ actual
@@ -1 +1 @@
-["cmake dependency should be"]
+["Can't find dependency \"cmake\"."]

Thanks @scpeters, nice spot!


Replace ::util::encode_utf8 with char::encode_utf8

This function should be using char::encode_utf8 instead. It requires Rust >=1.15.0.

I can work on this.

I want to make a small note.
After checking char::encode_utf8's source, i found that it is calling unsafe fn from_utf8_unchecked_mut. Whereas, serde's original utf8::encode doesn't do anything unsafe.

Tagging as breaking change because this would require bumping our minimum supported Rust version from 1.13 to 1.15.

Closing as wishlist for 2.0.


Sliced scaling does not play well with Anchor

Bevy version
v0.13.1
What you did
I was upgrading my RPG textboxes to use the new 9-slicing feature, but creating a sliced sprite with an anchor not set to Anchor::Center does not work right.
What went wrong
Creating the textbox background with Anchor::Center works fine, and looks like this:

Created with this code:
let sprite_child = commands
    .spawn(SpriteBundle {
        sprite: Sprite {
            custom_size: Some(vec2(296.5, 83.5)),
            ..default()
        },
        transform: Transform {
            translation: Vec3::new(0.0, -77.75, 4.9),
            rotation: Quat::IDENTITY,
            scale: Vec3::splat(1.0),
        },
        texture: asset_server.load("textbox.png"),
        ..default()
    })
    .insert(ImageScaleMode::Sliced(TextureSlicer {
        border: BorderRect::square(13.0),
        max_corner_scale: 1.0,
        ..default()
    }))
    .id();

Creating the textbox background with Anchor::TopLeft (with position adjusted correspondingly) does not work fine, and looks like this:

Created with this code:
let sprite_child = commands
    .spawn(SpriteBundle {
        sprite: Sprite {
            anchor: Anchor::TopLeft,
            custom_size: Some(vec2(296.5, 83.5)),
            ..default()
        },
        transform: Transform {
            translation: Vec3::new(-148.25, -36.0, 4.9),
            rotation: Quat::IDENTITY,
            scale: Vec3::splat(1.0),
        },
        texture: asset_server.load("textbox.png"),
        ..default()
    })
    .insert(ImageScaleMode::Sliced(TextureSlicer {
        border: BorderRect::square(13.0),
        max_corner_scale: 1.0,
        ..default()
    }))
    .id();

Additional information
My guess is that the anchor is being taken into consideration when scaling the center of the sliced texture, but not for the other parts.

I'll try to get a small reproduction made tomorrow, but right now it is 1:51 AM and I need to sleep...

Looks like the same cause as #12408.


Updating Demo App Dimension Category

Resolves #

Description
Proposed Changes

Screenshots
License
I confirm that this contribution is made under an MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Closing, as we are removing this demo data


Add .NETFramework 3.5 as a target framework

Could you please consider compiling your awesome library for net35? Yes, it is old, but there are people who still have to use it.

That's unfortunate, being stuck on 3.5!
I'll see what can be done.  There's already some traction for better multi targeting.

Thank you for looking at the issue!
Actually, I have already tried to and succeeded in compiling it for 3.5. I had to replace all string.IsNullOrWhitespace with string.IsNullOrEmpty, and change how default value for a given type is calculated (for there is no DefaultExpression in NET3.5). Don't recall any other changes.
I wanted to propose it as pull request, but couldn't figure out from the first glance how you generate a nuget package targeting different frameworks. And then got distracted :)

JFI, Microsoft has stoped the support of .NETFramework 3.5
REf: Lifecycle FAQ—.NET Framework
It's better to plan of upgrading application running .net3.5 to .net4.5+

Closed for age.


cert-rotation: replace RewriteBootstrapHostname with GetBootstrapHost…

…name + updated RewriteNodeNames
Followup https://github.com/openshift/library-go/pull/1725

minor comments, but needed prior to merge
/approve

/lgtm


commit

I was learning github so i made this pull request

Thanks for the PR.
Hope to see more contribution from you in this organization in future.
We will like have you onboard.
Feel free to contact me in case of any doubt.

@Akshit312 Thanks for your wise words,
It great to see you mentoring the students.
Hey folk, Continue your contribution to this org and let us know in case of any dispute.


fix(CSI-275): remove rdirplus from mountoptions

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

#355  👈
#354
#353
dev

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

Join @sergeyberezansky and the rest of your teammates on  Graphite


[flow] Center nodes horizontally in auto-layout

Looks a bit strange currently, especially on the left-hand side of the attached diagram

We cannot solve this in sprotty. If this is still relevant, a new issue should be opened for ELK.


change naming scheme in domain decomp

At meeting in Bergen, it was decided that we should rename some of the variables printed in the netcdf metadata as follows:
globalX -> NX
globalY -> NY
global_x ->  domain_x
global_y ->  domain_y
local_extent_x -> domain_extent_x
local_extent_y -> domain_extent_y

Closed by #33.


rutorrent autodl-irssi issue

hello
i installed rutorrent whit this script  on debian 10 64 bit clean install
when i setup autodl-irssi and filters to grab contect the autodl-irssi
whon't grab over irc i ger error HTTP time out unknow error code 5
any ideas how to fix this?
this is the error im getting on all tracker that i use whit autodl-irssi
link: https://imgur.com/lKE0JXU

Unticking SSL should do the trick. Otherwise, you can follow this issue over at autodl-community/autodl-irssi
 #168 SSL Error when attempting to download SSL torckers

hello
i installed rutorrent whit this script on debian 10 64 bit clean install
when i setup autodl-irssi and filters to grab contect the autodl-irssi
whon't grab over irc i ger error HTTP time out unknow error code 5
any ideas how to fix this?
this is the error im getting on all tracker that i use whit autodl-irssi
link: https://imgur.com/lKE0JXU

Go to the tracker tab where you inserted your passkey for that particular trackers and uncheck the SSL box. Not all trackers support SSL downloading.

Unticking SSL should do the trick. Otherwise, you can follow this issue over at autodl-community/autodl-irssi
 #168 SSL Error when attempting to download SSL torckers

Unticking SSL also dont work,the fix you posted also don't work,after restarting autodl-irssi there is error http internal error.


Updates for NuGet 4.3 and refactoring version topics

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Consume-Packages/Dependency-Resolution.md
:white_check_mark:Succeeded

docs/Consume-Packages/Finding-and-Choosing-Packages.md
:white_check_mark:Succeeded

docs/Consume-Packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded

docs/Consume-Packages/Package-Restore.md
:white_check_mark:Succeeded

docs/Consume-Packages/Reinstalling-and-Updating-Packages.md
:white_check_mark:Succeeded

docs/Consume-Packages/media/Prerelease_02-CheckPrerelease.png
:white_check_mark:Succeeded

docs/Create-Packages/Creating-Localized-Packages.md
:white_check_mark:Succeeded

docs/Create-Packages/Creating-a-Package.md
:white_check_mark:Succeeded

docs/Create-Packages/Dependency-Versions.md
:white_check_mark:Succeeded

docs/Create-Packages/Overview-and-Workflow.md
:white_check_mark:Succeeded

docs/Create-Packages/Prerelease-Packages.md
:white_check_mark:Succeeded

docs/Create-Packages/project-json-Impact.md
:white_check_mark:Succeeded

docs/Guides/Create-Cross-Platform-Packages.md
:white_check_mark:Succeeded

docs/Guides/Create-NET-Standard-Packages-VS2015.md
:white_check_mark:Succeeded

docs/Guides/Create-UWP-Packages.md
:white_check_mark:Succeeded

docs/Quickstart/Create-and-Publish-a-Package.md
:white_check_mark:Succeeded

docs/Reference/Errors-and-Warnings.md
:white_check_mark:Succeeded

docs/Reference/Package-Versioning.md
:white_check_mark:Succeeded

docs/Release-Notes/Index.md
:white_check_mark:Succeeded

docs/Release-Notes/NuGet-1.4.md
:white_check_mark:Succeeded

docs/Release-Notes/NuGet-4.3-RTM.md
:white_check_mark:Succeeded

docs/Schema/nuspec.md
:white_check_mark:Succeeded

docs/Schema/packages-config.md
:white_check_mark:Succeeded

docs/TOC.md
:white_check_mark:Succeeded

docs/What-is-NuGet.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Repackaging - ExtraHop Reveal(x) (MMA to AMA Migration)

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

/Package


Potential bug

I noticed this line: https://github.com/Worlize/WebSocket-Node/blob/v1.0.1/lib/WebSocketRequest.js#L263
It seems like this refers to the anonymous function inside which the whole code block is. This almost looks like a bug, but my time and knowledge on this library is too limit to figure it out for sure.
I'm leaving this issue as a "check it and fix it if needed".
Note: there are other lines that use this below.

No it's correct.  Notice the .bind(this) at the end of the function declaration.

You are right, I did not spot it.


Feature/cornflow server migration

Tell me when you're done with the changes so I will review again

Tell me when you're done with the changes so I will review again

I think the changes here are done, this should cover most of the generalization.

Are you done with the changes here ?

Are you done with the changes here ?

I should be, but I am not 100% sure to make this changes just yet... I do not know why I am not 100% sure about them

I'm going to finally merge the PR, I have merge with master, if all tests pass then I will merge this branch, although I think there is still things that can be improved on cornflow-tools and the generalization of cornflow-server


Update Quick Start

I updated the quick start after creating an example entertaining employee handbook chapter. This new version is more focused.

I made the suggested changes :)


default   display protein not DNA sequence on gene pages sequence section

I still think we should default to protein sequence not DNA...more people are interested int he protein sequence.
I know we discussed this before but I can't think why we decided not to do it?

I don't remember discussing this.  It's a quick change.  Should I go ahead and do it?

Well I wondered if there was a reason we didn't do it before, it seems like a "no brainer'.
Any objections?

I'd be happy with this change

I've made that change.


feat: Adds the dataloading pipeline for the backend module

This PR adds a utility function for safe loading of the database and a test for it

/quack review


Get list of Subscriptions from Multi Tenant

Hello Team,
I am new to azure-sdk-for-go, and I am working on a multi Tenant environment
I created a multi tenant app (App registration) and using it's credential I need to list subscriptions from both Tenants.
I am using ARM cross-tenant authentication feature from azidentity
But I am not getting Subscription list from auxiliary tenant, instead getting only from primary Tenant.
auxTenants := []string{"TENANT-1-ID}

cred, err := azidentity.NewClientSecretCredential("PRIMARY-TENANT-ID",
	"APP-CLIENT-ID", "APP-SECRET",
	&azidentity.ClientSecretCredentialOptions{
		AdditionallyAllowedTenants: auxTenants})

if err != nil {
	log.Default().Fatal(err)
}

// SUBSCRIPTION LISTING LOGIC--
client, err := armsubscription.NewSubscriptionsClient(cred,
      &arm.ClientOptions{AuxiliaryTenants: auxTenants},
)
ctx := context.TODO()
pager := client.NewListPager(nil)

for pager.More() {
subscriptionList, err := pager.NextPage(ctx)
if err != nil {
	log.Default().Fatal(err)
}
for _, item := range subscriptionList.Value {
	subscriptions = append(subscriptions, structs.Subscription{SubscriptionID: *item.SubscriptionID, SubscriptionName:       *item.DisplayName})
}
   }

Here I am getting list of Subscriptions from PRIMARY-TENANT-ID only, I am not getting the sub list from auxTenant !!
Am I wrong here ?
How can I get list of Subscriptions from Both Tenants?
cc: @chlowell

Auxiliary tenants don't apply to this API because the operation isn't cross-tenant. The service simply lists subscriptions for the tenant that issued the primary access token i.e., the credential's configured tenant. So, you have to create a credential and client for each tenant like this:
credA, err := azidentity.NewClientSecretCredential("tenant A", "clientID", "secret", nil)
if err != nil {
	// TODO
}
credB, err := azidentity.NewClientSecretCredential("tenant B", "clientID", "secret", nil)
if err != nil {
	// TODO
}
for _, cred := range []azcore.TokenCredential{credA, credB} {
	client, err := armsubscription.NewSubscriptionsClient(cred, nil)
	if err != nil {
		// TODO
	}
	pager := client.NewListPager(nil)
	for pager.More() {
		page, err := pager.NextPage(context.TODO())
		if err != nil {
			// TODO
		}
		if v := page.Value; v != nil {
			// TODO: do something with the subscriptions
		}
	}
}

Thank you @chlowell for helping me out.
I truly appreciate your time and expertise, keep doing great things team.


Fix typo in help message

s/persistung/persisting/

Pulled into https://github.com/input-output-hk/cardano-db-sync/pull/1203 .


CI/CD: Environment configuration

Added CI/CD workflow to deploy in multi-region architecture
Config file
Stored request filesystem


Fix max_time flag using Scala-like futures

MAybe this post helps: https://discourse.julialang.org/t/simple-timeout-of-function/99578


Update README with latest project status.

Fixes https://github.com/openebs/openebs/issues/2699
Made the following changes:

Updated the introductory section to include details about the support for multiple storage engines
Updated the status with details on individual storage engine current status.
Added the references to CNCF and commercial offerings available using OpenEBS

/lgtm
/approve


Enhancement idea: file name

The script currently hardcodes the file name. If one wants to render and host multiple HTML files like a "standard" web site the file names need to be unique. One can simply prompt for a file name, store that in a variable and use that as so:
const url = file.makeFilePublic (htmlFileOut);
A step beyond is to store the file name value as a  opmlHeader, perhaps htmlFileName, so that for future edits the script can change for the value and if not null use it.

I will do this as the first enhancement, will let you know when it is ready.

I have a test version of the script ready, twoLevelOutlineRendererE (see http://drummer.scripting.com/PDXProtestNews/PDXSharedScripts.opml). The script will read the OPML header htmlFileName from the currently selected tab to be rendered (needs to be a private OPML file right now) and create a rendered HTML file with that file name. I have not added the check to use a default name yet, but wanted to get this out there for you to take a look at.

I have completed updates to the script, now called twoLevelOutlineRendererF (see http://drummer.scripting.com/PDXProtestNews/PDXSharedScripts.opml). The script will read the OPML header htmlFileName from the currently selected tab to be rendered (private OPML file only) and create a rendered HTML file with that file name. If the file is a private file without that header element, the file will be rendered as test.html. If the outline is a public outline/instant outline, the file will be rendered as myOutline.html. Take a look and let me know what you think, especially with the default names for the HTML renderings.

Found a typo
console.log ("isInstantOutine = ", isInstantOutine ); the value should be isInstantOutline

@fmcpherson, I took a look at your source OPML file (http://drummer.scripting.com/frankm/testOutlineToHTML.opml), the htmlFileName attribute is set to "file name: opmlToHtml.html". You should edit the OPML headers to just have the file name.
I have also created a new version of the rendering script (rev G, twoLevelOutlineRendererG (see http://drummer.scripting.com/PDXProtestNews/PDXSharedScripts.opml), which is able to read the htmlFileName attribute for both private and public files. Give that a try and let me know how it goes. Thanks for testing this!

That’s odd, I don’t know how the OPML header got messed up, but I removed the file name portion and that resolved that issue.
--
Frank McPherson
@.***
frankm.info
On Jan 3, 2022, 2:52 PM -0500, Andy Sylvester @.***>, wrote:

@fmcpherson, I took a look at your source OPML file (http://drummer.scripting.com/frankm/testOutlineToHTML.opml), the htmlFileName attribute is set to "file name: opmlToHtml.html". You should edit the OPML headers to just have the file name.
I have also created a new version of the rendering script (rev G, twoLevelOutlineRendererG (see http://drummer.scripting.com/PDXProtestNews/PDXSharedScripts.opml), which is able to read the htmlFileName attribute for both private and public files. Give that a try and let me know how it goes. Thanks for testing this!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>

I have updated my shared scripts file and this repo, closing this issue.


Invalid flags supplied to RegExp constructor 'gs'

Currently I'm using a non supported flag here, so there is no "Single Line" match comment in the flag, I wonder how the test were being passed in the Travis, because on my old Linux Machine this throws an error.

Current node version v8.9.4

Apparently this is not a case anymore for current stable nodeJS version which is 8.11.3


Kamelet - Inject secret in Azure KeyVault Trait

Requirement
As a User
I want to be able to use a configmap/secret in azure-key-vault.client-secret field for Azure KeyVault trait
When I deploy a Kamelet Integration / Kamelet pipe in order to hide the real Azure Key Vault credentials.
Problem
When I was trying to deploy my current Kamelet Pipe into our cluster through Github repository I realize that I couldn't hide the azure-key-vault.client-secret through a secret or configmap:
apiVersion: camel.apache.org/v1
kind: Pipe
metadata:
  namespace: default
  name: currentPipe
  annotations:
    trait.camel.apache.org/logging.level: DEBUG
    trait.camel.apache.org/azure-key-vault.enabled: "true"
    trait.camel.apache.org/azure-key-vault.tenant-id: "aaaa-bbbb-ccc-ddd-eee"
    trait.camel.apache.org/azure-key-vault.vault-name: "currentvaultName"
    trait.camel.apache.org/azure-key-vault.client-id: "azurekeyvaultClientId"
    trait.camel.apache.org/azure-key-vault.client-secret: "K8S_SECRET/CONFIGMAP"
spec:
  replicas: 1
  source:

Proposal
As I was talking previously through Zulip there is a similar solution for other traits specified in this PR (thanks to @oscerd  and @squakez for this information) :
https://github.com/apache/camel-k/pull/4568
Open questions
No response

If we're working on this, let's target this generically for any secret in vaults.

Sure thing


Documentation

Software boundaries and limits for SharePoint Server 2019 fills.

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

ID: 937a1f68-4cf6-f243-b219-dc154bae6714
Version Independent ID: 7f3050b3-7a3d-d467-64cc-bc0edbe1521e
Content: System requirements for SharePoint Servers 2016 and 2019
Content Source: SharePoint/SharePointServer/install/system-requirements-for-sharepoint-server-2016.md
Product: sharepoint-server-itpro
GitHub Login: @Techwriter40
Microsoft Alias: kirks

@pkotov87, thank you for your feedback. Unfortunately, could you be more specific of the exact request or change you like to make?

@Techwriter40, I think @pkotov87 is missing the 2019 version of the "software-boundaries-and-limits-0.md" file. I was looking for it too.
Btw, was your file naming convention documented somewhere?
I see .md file without numbers at the end and also with "-0", "-2016" and "-2019".
I'd be happy to help out validating the file names and the titles of the articles.

there is not a separate 2019 version as there were minimal changes. What applies for 2016 also is valid for 2019, hence contained in one article title system requirements for SharePoint Servers 2016 and 2019.

Exactly what @sassdawe said, checkbox was only for 2016. Now it stands for both. Thank you!

@pkotov87, your welcome. since issue is resolved, I will close comment.


Redux Offline instead of API caching

Here’s an interesting article on this: https://hackernoon.com/introducing-redux-offline-offline-first-architecture-for-progressive-web-applications-and-react-68c5167ecfe0
Note: the repo for react offline is going through ownership changes currently

Jani is a smart guy and has a good pulse on RN. There are some discussion regarding the confluence of native vs. Web which calls to mind the app vs. web debates we had years ago starting to form in the community it seems—as we can see with this discussion on material-ui where @necolas, a former designer at Twitter and creator of SuitCSS, discuss adding react-native-web. Just another lob.

I’m curious to what you think.

We have removed API caching and would allow the user to configure service worker to handle API caching manually. Thus closing the discussion.


Update enabling-development-mode.rst

Modify this link to include the branch name, and possibly the page this PR modifies:
https://osc.github.io/ood-documentation-test/BRANCH-NAME/
Add your description here

Thanks Bob! Not sure why this is saying it's updating something. It's not develop has that change in https://github.com/OSC/ood-documentation/commit/9183e50f2b40bd7f41d0f94ea2465e0df1ce2424 and then even my own commit https://github.com/OSC/ood-documentation/commit/a2fd77cdbb032ec44356b5df29bed4d24a5edb80 to fix the builds. :man_shrugging: no idea why GH doesn't recognize it!


ValueError ValDataLoaderIter next() call missing

In the _validate function, you try to iterate through the elements of ValDataLoaderIter with a simple for loop:
https://github.com/davidtvs/pytorch-lr-finder/blob/acc5e7ee7711a460bf3e1cc5c5f05575ba1e1b4b/torch_lr_finder/lr_finder.py#L427
This throws a 'ValueError: too many values to unpack' because it is attempting to unpack the entire dataloader which has way more then two elements. I think what you want here is just the next element in val_iter, similarly to your _train_batch function, so a loop over the length of val_iter with a call to next(val_iter) every loop or an enumerate(val_iter), no?

Hi @paulhager!
Is the situation mentioned in this issue similar to the one you ran into? https://github.com/davidtvs/pytorch-lr-finder/issues/61#issuecomment-692715109
If not, can you describe more about it?

Closing due to inactivity


EKF2 Rejecting Rangefinder

@priseborough After some more investigation of the optical flow issues I was having, I tracked them down to the distance_bottom estimate. I am using a TFMini rangefinder, which initially had some driver issues, which I fixed here : https://github.com/PX4/Firmware/pull/10319. There may be additional issues, but they aren't obvious from visually inspecting the data.
The EKF still doesn't seem to like the rangefinder data. As soon as the vehicle is armed and takes off, the terrain estimate goes invalid.
Here is a log (from boot) : https://logs.px4.io/plot_app?log=26e73c13-304d-497a-8d48-19b8c3b35f63
My branch is off v1.8 Stable, and should work with replay.
@Antiheavy FYI.

@dakejahl

Fixed on ecl master by : https://github.com/PX4/ecl/commit/a7245229cbc3916f66be9d948875ff019dc41d5f


Add specific permissions to Argo CD for ClusterTasks

I will be removing supreme powers from Argo CD : https://github.com/redhat-appstudio/infra-deployments/tree/main/openshift-gitops/cluster-rbac ( Principle of minimum privilege ) .

/retest

Correct, will validate the other components before I drop https://github.com/redhat-appstudio/infra-deployments/tree/main/openshift-gitops/cluster-rbac


[5.1][ConstraintSystem] Delay adding contextual requirements until parent …

…type is opened
openUnboundGenericType eagerly tries to add conditional requirements
associated with chain of parents of the given type if type has been
declared inside of constrained extension. But one of the parent types
might be unbound e.g. A.B which means it has to be opened, which
by itself, would add such requirements.
Resolves: rdar://problem/49371608
(cherry picked from commit 8e420496b2611a7696c3339ea77fa7d140c3e48b)

@swift-ci please test


fix setTCPAccessLog (#36586)

Please provide a description of this PR:
fixed: https://github.com/istio/istio/issues/37723
manual cherry-pick of https://github.com/istio/istio/pull/36586 and https://github.com/istio/istio/pull/36661
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[x] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

hi @zirain, can you please add a release note for this? Thank you!

@dgn update release notes.


Support Python 3.7 and 3.8 and kfp 1.0.0

Hello,
I was only able to install kale with python 3.6.3 and it installed kfp 0.3.0 as a requirement which is quite outdated now.
With Python 3.7.7, there is this dependency conflict:
kfp 0.3.0 requires kfp-server-api<0.4.0,>=0.2.5, but you'll have kfp-server-api <IP_ADDRESS> which is incompatible.
With Python 3.8, it is not possible to install it because of the ml-metadata dependency which isn't upgraded to support python 3.8 yet.

Hu @ksonbol can you try installing from master? We have removed the limiting kfp dependency but haven't release a new version yet

ml-metadata v0.24.0 was released 2 days ago with support for python 3.8.
https://github.com/google/ml-metadata/releases/tag/v0.24.0

@ksonbol Python 3.8 is now compatible, along with KFP 1.0.0. Can you confirm this issue is resolved for you?

Thank you @DavidSpek . I can confirm that building from master works fine. Looking forward to the coming release :)

I believe this issue can be closed in that case.


speed

... depends on a lot of things and might be hard to compare.
just a few insights (from borg development):

borg 1.x does some information gathering based primarily on the filename, assuming that same filename means same file.
borg 2.0 will play a bit more on the safe side considering race conditions due to changing file systems, so it opens the file to get a file descriptor (fd) and then does the information gathering using the fd. the fd will always refer to the same fs object.
borg >= 1.2 checks if a file has changed while it was backed up.
these are a few reasons why more recent borg versions got a bit slower than older ones, especially on NFS, because open() and stat() are slow there.

So, sometimes speed == quick & dirty and slower == better / safer.
The less you do, the faster you get. The question is then if you still do enough / all that is needed.

I can definitly add the file descriptor part to the README section.
I don't get the part where borg >= 1.2 checks if a file has changed while it was backed up.
Does it the backed up file to it's last state while doing backups ? What if the file continuously changes  ?

So, sometimes speed == quick & dirty and slower == better / safer.

This statement is something I can live by, except for parked files like qcow files with external snapshots, which will never change while being backed up (actual thing I do with borg as of today).

The "changed while backup" only detects that there might be a problem, it does not avoid it (like a snapshot).
In some cases, it might be not an issue (like e.g. a log file growing a line at the end), but in other cases it might warn the user of an issue (e.g. if you backup some sort of database and the file changes internally while you back it up - the file as read by borg could then be inconsistent internally).

Thanks for the clarification. This let's me think of pre-freeze and post-thaw scripts for databases ;)
I'll add a "backup coherence" entry in the table which I can link to this discussion.
Jsut a side question, when using borg cli, will there be a specific exit code in those cases, or must the output be parsed to find out whether a file changed while being backed up ?

Currently there are only a few exit codes and also it is hard to map warnings to exit codes (because there can be multiple different warnings), so one currently needs to read the log output.

I added a new benchmark with qemu disk images (see last README.md file)
Noticed that borg performs quite well for that usecase, whearas backing up the linux kernel source files is not that great in terms of speed.
Is that explained by your above statements about open() and stat() ?

Could be, because if you have a lot of small files, the per-file overhead has a much bigger effect than for few big files.


Zestimate: fix for issue #23757

Description:
Changed name property to return Zestimate
and the property address.  This will make it easier
to distinguish multiple Zestimate sensor entities
in the UI.
Also removed MIN_TIME_BETWEEN_UPDATES in
favor of SCAN_INTERVAL per suggestion from
amelchio#9580 on Discord
Related issue (if applicable): fixes #
fixes #23757
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Wouldn't this be a breaking change? Entities would get new ids

Wouldn't this be a breaking change? Entities would get new ids

It didn't break mine when I was testing.  It just renamed them.  What determines what ID's get assigned?

If the entity id change its a breaking change. If entities has unique id it would keep the same if regardless of name

If the entity id change its a breaking change. If entities has unique id it would keep the same if regardless of name

yes it is a breaking change.  I updated the pull pull request to indicate it.
thx

It's not a breaking change if entities have a unique ID.

@balloob they don’t have a unique id, there’s a previous PR to add it.


A JSON-LD version of the dataset_description.json metadata for harmonizing with other data standards?

Your idea
Hi there - This is my first BIDS issue, so please feel free to close/reroute me if this is the wrong forum for this question!
I work on Psych-DS, a BIDS-inspired standard for tabular behavioral datasets. One thing that has come up in my conversations with tool builders is that they are nervous about potentially needing to support multiple metadata files that 'clutter' a user's directory, or otherwise needing to handle the presence of more than one type of metadata. One use case for this scenario might be a scanning session that uses a task script originally developed for non-scanner use that generates its own response data output directory; the researcher then wants to incorporate this response data into a BIDS beh/ directory.
A possible solution to this problem would be for multiple data standards to allow for an "alias" for the top-level metadata object, e.g. by permitting a file called metadata.json that contains one or more JSON-LD objects.
For BIDS, this would mean allowing this alternative to the dataset_description.json with only a minor change to the content, namely the inclusion of a null @context field.  This would allow the correct/relevant JSON to be identified, and any other JSON-LD content present in the file should be ignored by the BIDS spec.
This might look like:
{
  "@context" : null,
  "Name": "The mother of all experiments",
  "BIDSVersion": "1.6.0",
  "DatasetType": "raw",
  "License": "CC0",
  "Authors": [
    "Paul Broca",
    "Carl Wernicke"
  ],

##...etc.##

(As a side note, this might open a door to JSON-LD BIDS context or mapping to schema.org but that's a story for another day...this question is really just about the potential to 'package' the top-level metadata object into a file that might contain additional, irrelevant-to-BIDS information.)

@mekline
Thanks for opening this issue.
Will point you to other issues or PRs (that I can remember) where jsonld has been mentioned.

https://github.com/bids-standard/bids-specification/pull/487
https://github.com/bids-standard/bids-specification/issues/602
https://github.com/bids-standard/bids-specification/issues/577

pinging people involved in those
@CPernet @cmaumet @satra

if I understood @satra simply adding  "@context" : null, makes dataset_description.json a jsonld and we don't even have to change the extension, and that would be fine
the bigger change would be to allow metadata.json which is unlikely IMO.

could we simply say that an @context is optionally allowed, but if provided should follow json-ld specifications? we could formalize this in bids itself saying that a bids-specific context has to be part of @context (which has to be an array of urls/objects or an object)? technically @context=null would result in a jsonld.SyntaxError: Invalid JSON-LD syntax; @context must be an object.
regarding the manifest.json, @mekline could we work through a simple example to demonstrate the issues?

would it be too premature to suggest/discuss "An optional @context in any .json file must follow JSON-LD specification." or it would not even be needed? ;-)

Relates to

https://github.com/bids-standard/bids-specification/pull/1939
https://github.com/bids-standard/BEP028_BIDSprov/issues/147


Fix error: wget: not an http or ftp url

This fixes one issue with https://github.com/brikis98/docker-osx-dev/issues/61 -- one still remaining

The reason we're downloading rsync from this repo is as a work around for #52, where Docker's latest Boot2Docker ISO was upgraded to a 64 bit image that no longer had rsync in the package repository. @cameronbrunner did the work to get rsync into the package repository earlier today, so it's probably on all the mirrors now and we can go back to the simpler tce-load -wi rsync solution.
Could you run:
boot2docker ssh
tce-load -wi rsync

And let me know what you see? If it works, then a better PR would be to just revert this commit. Thanks!

I tried that, unfortunately it didn't work... a which rsync returned a blank line.

Did the install actually complete or did it give you an error?

Actually nothing happened with that line... it's just like a hit a carriage return :?

That's... very odd. I'll try to repro myself tomorrow.

OK, just tried to repro, and it looks like the version installed by tce-load may have issues. When you run it, you get this error:
rsync: error while loading shared libraries: libattr.so.1: cannot open shared object file: No such file or directory

So I guess that means this workaround has to stay. Therefore, I've merged your PR. Thanks for the fix!

I have the .dep submitted that will automatically include attr and acl when doing a tce-load -wi rsync.  Until the dep is active and sync'ed this should work tce-load -wi rsync acl attr as a work-around.

FYI @brikis98 the TCL team took the .dep file so automatic dependency download should be done with tce-load rsync now.

@cameronbrunner Thanks. It looks like the install works and the rsync executable runs, but the installer exits with an error code:
docker@boot2docker:~$ tce-load -wi rsync
rsync.tcz.dep OK
acl.tcz.dep OK
Downloading: attr.tcz
Connecting to repo.tinycorelinux.net (<IP_ADDRESS>:80)
attr.tcz             100% |************************************************************************************************************************************| 24576   0:00:00 ETA
attr.tcz: OK
Downloading: acl.tcz
Connecting to repo.tinycorelinux.net (<IP_ADDRESS>:80)
acl.tcz              100% |************************************************************************************************************************************| 40960   0:00:00 ETA
acl.tcz: OK
Downloading: rsync.tcz
Connecting to repo.tinycorelinux.net (<IP_ADDRESS>:80)
rsync.tcz            100% |************************************************************************************************************************************|   312k  0:00:00 ETA
rsync.tcz: OK
docker@boot2docker:~$ echo $?
1

Any idea why?


Support none-ESP devices

It would be nice to be able to install / update my whole fleet of devices. For this it would be nice if arduino devices could be updated / installed as well.
Here is a similar project that did this:
https://github.com/kaelhem/avrbro (stk500v1)
https://github.com/noopkat/avrgirl-arduino
Would this be something that could be integrated in this project?

That's not something we're currently considering on adding ourselves but a pull request is welcome.


syntax error: invalid syntax

Catt v0.9.2
Os : Kali 2019.1
Python3 : v 3.6.8
Python2.7.15+
Traceback (most recent call test):
File "/usr/local/bin/catt" line 6, in
from catt.cli import main
file "/usr/local/lib/python2.7/dist-packages/catt/cli.py", line 13, in
from .controllers import Cache, CastState, StateFileError, StateMode, get_chromecast, get_chromecasts, setup_cast
file "/usr/local/lib/python2.7/dist-packages/catt/controllers.py", line 196
def set_data(self, *args)-> None;
SyntaxError : invalid syntax

We somehow missed this ticket, sorry about that. Python2 is no longer supported.


Add interval drop down and limit options

@sfentress Looks good!  Those last changes fix the issue we saw while demoing yesterday!


Fail to build and push to docker for runtime 0.2.0

As part of the relaase of the runtime 0.2.0, we needed to create the docker images to create the coiled software environments.
This process is done via the `` workflow. However, there were/are some failures that are preventing this.
https://github.com/coiled/coiled-runtime/actions/runs/3603086081
The main error is in runtime=0.2.0 and py38: Where strangely complains about py37 even though it's py38.
 #10 61.38 Encountered problems while solving:
#10 61.38   - nothing provides openssl >=1.0.2p,<1.0.3a needed by python-3.7.1-hd75c258_1001
#10 61.38
#10 61.38 The environment can't be solved, aborting the operation
#10 61.38 error    libmamba Could not solve for environment specs
#10 61.41 critical libmamba UnsatisfiableError

Without this images we can't build the coiled-software environments for runtime=0.2.0
cc: @dchudz @ntabris @jrbourbeau

Thanks for reporting @ncclementi -- I agree seeing things about Python 3.7 in this build is weird.
@dchudz @ntabris I think this means we won't have published docker images / Coiled software environments for coiled-runtime=0.2.0 for Python 3.8. What impact does this have on Coiled's default behavior? Do things break, or just pull in a non-latest software environment? I'm trying to get a sense for priority on this issue

Do things break, or just pull in a non-latest software environment?

just pull in a non-latest software environment I think.
In more detail, Coiled will:

attempt to match a runtime version it finds locally
if no runtime version locally, find that latest software env that matches env.endswith(py_version) and "coiled/coiled-runtime" in env.

Relevant code:

```
    async def _determine_senv(self):
        # Attempt to match Coiled software environment to local versions of Python and coiled-runtime
        # FIXME get local coiled-runtime version when it was installed with pip
        warn_for_unspecified_senv = False
        if self.software_environment is None and not self.package_sync:
        py_version = f"py{''.join(map(str, sys.version_info[:2]))}"
        warn_for_unspecified_senv = True
        try:
            senvs = await self.cloud.list_software_environments(account="coiled")  # type: ignore
            runtime_versions = [
                env
                for env in senvs
                if env.endswith(py_version) and "coiled/coiled-runtime" in env
            ]
            self.software_environment = max(runtime_versions, key=name_to_version)  # type: ignore
        except ValueError:
            self.software_environment = f"coiled/default-{py_version}"
        conda_prefix = environ.get("CONDA_PREFIX")
        if conda_prefix:
            conda_meta = Path(conda_prefix) / "conda-meta"
            if conda_meta.exists():
                coiled_runtime = next(conda_meta.glob("coiled-runtime*.json"), None)
                if coiled_runtime:
                    match = re.match(
                        r"coiled-runtime-(\d+\.\d+.\d+)", coiled_runtime.name
                    )
                    if match:
                        runtime_version = match[1].replace(".", "-")
                        self.software_environment = (
                            f"coiled/coiled-runtime-{runtime_version}-{py_version}"
                        )
                        warn_for_unspecified_senv = False

</details>

I'm also seeing that the jobs for some of the 0.1.1 runtime got canceled see https://github.com/coiled/coiled-runtime/actions/runs/3603086081

Seeing same problem when trying to build the images for runtime 0.2.1 seeing the same problem. I tried on a separate branch upgrade mircomamba to their latest release, but I continue seeing this issue.
Python 3.10 build, 3.8 failed, and 3.9 hanged for a long time so I stopped the job.
https://github.com/coiled/coiled-runtime/actions/runs/3705608964/jobs/6279735279

Same effect, but in 0.2.0 I was seeing a solve error, not seeing that here.
This is impacting a user on Coiled side who's trying to use runtime 0.2.1 py39.
It appears there's already a retry of the pipeline running, that could resolve, if not I think this should get some attention.

Ok

This is impacting a user on Coiled side who's trying to use runtime 0.2.1 py39.
Is this because they are trying to use the soft env instead of package sync?

It appears there's already a retry of the pipeline running, that could resolve, if not I think this should get some attention.

I'm experimenting with some things, I want to see if python 3.9 finishes at least.
I will try separately only build py-38 for amd and avoid arm.
If things do not work, I agree this will need some attention but I'm not sure who will be able to debug what's happening with docker.

I will try separately only build py-38 for amd and avoid arm.

I think that's best way to address.
For py38, it seems the solve is just consistently failing for ARM. Also, @shughes-uk told me yesterday that the mamba error messages about why solve failed are often just wrong, so it's a pain to debug.
We could either (1) debug this and get solve that works for ARM with py3.8 or (2) disable the ARM build for py3.8.
My inclination is (2). We don't have many people trying ARM yet, if need be we can tell anyone trying ARM w/ coiled-runtime to use py39 or py310.
If I'm reading https://numpy.org/neps/nep-0029-deprecation_policy.html correctly, maybe we'll drop py38 after Jan 31, 2023, anyway.

Yay, we now have multiarch 3.9 and 3.10 images on docker hub:

Thanks @ncclementi for kicking off that re-run to make these.

@ntabris I was able to build and push just for py38 in amd (we should have that one too). I'm going to play with the GHA to make sure we skip arm for py38 for now.


Allow creation of Kotlin extension functions in ExtensionAware, so the accessors are annotated with @ExtensionFunctionType

I'd like to be able to create a Gradle plugin, in which I can define Kotlin extension function and apply that function to ExtensionAware containers.
Users will then be able to use these extension functions more naturally.
Expected Behavior

I define an extension, and I mark the type as a lambda. I use @ExtensionFunctionType to signify that the first type parameter is the receiver.// MyCustomGradlePlugin.kt
val additionalProcessingExt =
    fun ModuleDependency.(introspect: Boolean) {
        attributes {
            attribute(Attribute.of("my.project.processing", Boolean::class.javaObjectType), introspect)
        }
    }

dependencies.extensions.add(
    typeOf<@ExtensionFunctionType ModuleDependency.(Boolean) -> Unit>(),
    "additionalProcessing",
    additionalProcessingExt,
)

Gradle/Kotlin magic keeps @ExtensionFunctionType when it generates the accessor/**
 * Retrieves the [additionalProcessing][Function2<ModuleDependency, Boolean, kotlin.Unit>] extension.
 */
val DependencyHandler.`additionalProcessing`: @ExtensionFunctionType Function2<ModuleDependency, Boolean, Unit>
    get() = (this as ExtensionAware).extensions.getByName("additionalProcessing") as Function2<ModuleDependency, Boolean, Unit>

In the project, I can use the extension as an extension functiondependencies {
    implementation("org.reflections", "reflections", "0.9.12"){
        additionalProcessing(true)
    }
}

Current Behavior
At present, @ExtensionFunctionType is ignored
dependencies {
    implementation("org.reflections", "reflections", "0.9.12"){
        additionalProcessing(true)
        // The boolean literal does not conform to the expected type Action<(ModuleDependency, Boolean) -> Unit>
    }
}

Context
I'm attempting to add a custom Variant Attribute to dependencies so that I can filter them, and introspect them.
I can manually add an attribute, but I'd prefer to wrap this in a nice helper function. And then I'd like to share that helper function whenever the plugin is applied to a Gradle project
// manually applying the custom attribute
dependencies {
    implementation("org.reflections", "reflections", "0.9.12"){
        attributes {
            attribute(Attribute.of("my.project.processing", Boolean::class.javaObjectType), introspect)
        }
    }
}

Gradle 7.4.1
For more context on @ExtensionFunctionType, see https://github.com/JetBrains/kotlin/blob/master/spec-docs/function-types.md

Sorry for the late reply.
This issue needs a decision from the team responsible for that area. They have been informed. Response time may vary.

This feature request is in the backlog of the relevant team and is prioritized by them.


Specify maximum revert period

What is the desired behaviour if two beacon chain forks A and B do not advance justification for 4+ months, and then fork A (e.g. thanks to inactivity penalties) wins the race to justify first? We probably do not want fork B to honour the new justification in fork A.

I think clients should just refuse to revert further than some maximum revert period, though now that we're dynamically calculating that, it should be dynamically calculated....
I'd say one important first thing to do is to add the explicit criterion that a client does not revert finalized blocks.

Closing in favour of #675.


GROOVY-9492: Relax groovy.test.NotYetImplemented dependency on JUnit …

…4's AssertionFailedError

+1

Merged. Thanks.


Compile Error: Invalid Use of New Keyword (Using Word 2016)

Private Function json_ParseObject(json_String As String, ByRef json_Index As Long) As Dictionary
    Dim json_Key As String
    Dim json_NextChar As String

    Set json_ParseObject = New Dictionary

It's crashing on the line where it sets json_ParseObject.

Hello,
There is an incompatibility with the dictionary object of Word. Insert "scripting." before the term dictionary.
Example:
Private Function json_ParseObject(json_String As String, ByRef json_Index As Long) As scripting.Dictionary
    Dim json_Key As String
    Dim json_NextChar As String

    Set json_ParseObject = New scripting.Dictionary

I have encountered the exact same issue.  json_ParseObject works when I use Excel VBA, but not in Word VBA.  Adding namespace "Scripting" per StephanieSM's comment does not repair the function for me.
Are there any specific resolutions?

I can confirm this is not limited to Word, but I did manage to narrow down the problem and have a solution. You must make sure that the "Microsoft Scripting Runtime" comes before any of the Microsoft Word references. In other words, the scripting runtime reference must have higher priority in the References list than the other Microsoft references.

I can confirm the same issues, changing to scripting.Dictionary helped

Maintainers: Please tell about this in the Readme, just next to the "add reference to MS Scripting" line.
E.g.
This module will run as-is in Excel. In order to run it in Word, find any "New Dictionary" and replace it with "New scripting.Dictionary
Such a comment on the Readme.md would save hours from anyone like us,
(instead of tackling Microsoft's "late-blooming" issues, which python solved before even VB5 was out.)

I confirm the issue scripting.Dictionary not really helped.
the error occure on this line

I m using Office 360.

Hi,
i found a solution thats works for me in word. Just create the dictionary at runtime and return an object instead of Dictionary. Like This:

Hello,
There is an incompatibility with the dictionary object of Word. Insert "scripting." before the term dictionary.
Example:
Private Function json_ParseObject(json_String As String, ByRef json_Index As Long) As scripting.Dictionary
    Dim json_Key As String
    Dim json_NextChar As String

    Set json_ParseObject = New scripting.Dictionary

In order to avoid error "Method or data object not found" at json_ParseObject.Item(json_Key), dont forget to ass "Scripting." in both:

function type Private Function json_ParseObject(json_String As String, ByRef json_Index As Long) As Scripting.Dictionary
and in affectation  Set json_ParseObject = New Scripting.Dictionary

Has anyone made a pull request on this ?
I strongly suggest MoSad85's solution, in other words switch the library to late binding, since early binding is a known source of troubles.

Well, a pull request is done, and up for review.

I think that my solution is the best for above error.
https://github.com/VBA-tools/VBA-JSON
I didn't face any error when I used above jsonconverter in excel vba for userform.
But I faced same error in excel vba project for form control.
MoSad85's solution is not working to me.

Solution:
1.Remove MS script runtime library from reference in excel vba
2.Import above cls to vba project.
That's all


feat: port GroupTheory.PGroup

[x] depends on: #4316

This PR/issue depends on:

leanprover-community/mathlib4#4316
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#4316
By Dependent Issues (🤖). Happy coding!

Thanks :tada:
bors merge
(In the future we'll have to think about names like Padic and PGroup. The uppercase P is pretty weird.)

I thoroughly agree.


Update default for finalize in query-context.md

I recommend setting the default for 'finalize' to 'N/A'.  There are various query stages at which finalization is possible.  By default, some intermediate steps do or don't finalize, and end results are finalized.  Setting 'finalize' to either 'true' or 'false' changes the default behavior.  The changes from setting it to 'true' are generally behind the scenes, but can affect some queries, when there are intermediate stages that are not finalized by default.

Fixes #XXXX.

Description

Fixed the bug ...
Renamed the class ...
Added a forbidden-apis entry ...

Release note

Key changed/added classes in this PR

MyFoo
OurBar
TheirBaz

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

"n/a" isn't that helpful of a doc here; we should spell out how it actually works. AFAIK the default is true for outer queries and false for subqueries (i.e., queries that appear in query datasources).


[FYI] Pytest have "weak recommendation" on addopts to be a string

As per the discussion in https://github.com/pytest-dev/pytest/issues/12228, it seems that pytest recommends the addopts setting to be a string (although that is pretty much a temporary hack due to transitory nature of tool.pytest.ini_options).
https://github.com/scientific-python/cookie/blob/fea6e2f8f88f5b25e14efada9872acd6c89e02d9/docs/pages/guides/pytest.md?plain=1#L76

I don't gather that from the discussion. The person who suggested it be a string also suggested it not be in pyproject.toml. Since this isn't supported:
[tool.pytest.ini_options]
addopts = """
    # show 10 slowest invocations:
    --durations=10

    # a bit of verbosity doesn't hurt:
    -v
"""

but this is:
[tool.pytest.ini_options]
addopts = [
    # show 10 slowest invocations:
    "--durations=10",

    # a bit of verbosity doesn't hurt:
    "-v",
]

I'd say it's significantly better to have it as a list of strings.

transitory nature

It's been several years now.

I am fine to fix the repo-review check to support a string here, though.


Telescope-media-files not works

I have this issue

SO:Ubuntu in WLS

I have this issue

SO:Ubuntu in WLS

Hi, I am new to this, but I discovered the problem.
in the file browser settings you set the find_cmd property with fd, but in ubuntu os, the command fd does not exist exactly, then you replace it with rg or fdfind
The additional problem is that you cannot change this option of your init.vim or your .vimrc, you must edit the media_files.lua file that is stored in the plugin and change the default configuration on the end of the file:

to:

return require('telescope').register_extension {
  setup = function(ext_config)
    filetypes = ext_config.filetypes or {"png", "jpg", "mp4", "webm", "pdf"}
    --find_cmd = ext_config.find_cmd or "fd"
    find_cmd = ext_config.find_cmd or "rg"
  end,
  exports = {
    media_files = M.media_files
  },
}

Will this be available for MacOS anytime in the future?

I have the same problem. But I am not using WSL. I am using Pop OS.

Will this be available for MacOS anytime in the future?

probably , you need to make it work with icat

I have this issue

SO:Ubuntu in WLS

Hi, I am new to this, but I discovered the problem. in the file browser settings you set the find_cmd property with fd, but in ubuntu os, the command fd does not exist exactly, then you replace it with rg or fdfind The additional problem is that you cannot change this option of your init.vim or your .vimrc, you must edit the media_files.lua file that is stored in the plugin and change the default configuration on the end of the file:

to:
return require('telescope').register_extension {
  setup = function(ext_config)
    filetypes = ext_config.filetypes or {"png", "jpg", "mp4", "webm", "pdf"}
    --find_cmd = ext_config.find_cmd or "fd"
    find_cmd = ext_config.find_cmd or "rg"
  end,
  exports = {
    media_files = M.media_files
  },
}

I have the same issue, but im on Manjaro (still on wsl). the fd command works, but i still have no preview at all

Hi, I am new to this, but I discovered the problem. in the file browser settings you set the find_cmd property with fd, but in ubuntu os, the command fd does not exist exactly, then you replace it with rg or fdfind The additional problem is that you cannot change this option of your init.vim or your .vimrc, you must edit the media_files.lua file that is stored in the plugin and change the default configuration on the end of the file:
...

I have the same issue, but im on Manjaro (still on wsl). the fd command works, but i still have no preview at all

first make sure ueberzug works and you have xorg running.

I dont think i can run xorg on wsl... can i?

I dont think i can run xorg on wsl... can i?
idk , I never used wsl :laughing:

well then, if the extension needs xorg to run, it mean it depending of a display server, so, it cannot work on wsl (neither on wayland) until there is another implementation :/
im not a pro in the domain, so maybe i'm mistaking, but i think this is it, we cant have this :(

I would suggest doing a dual boot

So, I'm guessing this means the whole config is not being loaded at all...

I have this issue

SO:Ubuntu in WLS

Hi, I am new to this, but I discovered the problem. in the file browser settings you set the find_cmd property with fd, but in ubuntu os, the command fd does not exist exactly, then you replace it with rg or fdfind The additional problem is that you cannot change this option of your init.vim or your .vimrc, you must edit the media_files.lua file that is stored in the plugin and change the default configuration on the end of the file:

to:
return require('telescope').register_extension {
  setup = function(ext_config)
    filetypes = ext_config.filetypes or {"png", "jpg", "mp4", "webm", "pdf"}
    --find_cmd = ext_config.find_cmd or "fd"
    find_cmd = ext_config.find_cmd or "rg"
  end,
  exports = {
    media_files = M.media_files
  },
}

Hello, I have doubt in this code, what is M in M.media_files?

I have this issue

SO:Ubuntu in WLS

Hi, I am new to this, but I discovered the problem. in the file browser settings you set the find_cmd property with fd, but in ubuntu os, the command fd does not exist exactly, then you replace it with rg or fdfind The additional problem is that you cannot change this option of your init.vim or your .vimrc, you must edit the media_files.lua file that is stored in the plugin and change the default configuration on the end of the file:

to:
return require('telescope').register_extension {
  setup = function(ext_config)
    filetypes = ext_config.filetypes or {"png", "jpg", "mp4", "webm", "pdf"}
    --find_cmd = ext_config.find_cmd or "fd"
    find_cmd = ext_config.find_cmd or "rg"
  end,
  exports = {
    media_files = M.media_files
  },
}

Hello, I have doubt in this code, what is M in M.media_files?

I have this issue

SO:Ubuntu in WLS

Hi, I am new to this, but I discovered the problem. in the file browser settings you set the find_cmd property with fd, but in ubuntu os, the command fd does not exist exactly, then you replace it with rg or fdfind The additional problem is that you cannot change this option of your init.vim or your .vimrc, you must edit the media_files.lua file that is stored in the plugin and change the default configuration on the end of the file:

to:
return require('telescope').register_extension {
  setup = function(ext_config)
    filetypes = ext_config.filetypes or {"png", "jpg", "mp4", "webm", "pdf"}
    --find_cmd = ext_config.find_cmd or "fd"
    find_cmd = ext_config.find_cmd or "rg"
  end,
  exports = {
    media_files = M.media_files
  },
}

Hello, I have doubt in this code, what is M in M.media_files?

M is the objet to return inside the plugin file

Thank you @yebt , in mi case I have this:

What should be M for me?  Because when I use it, I have an error like this:

Hi @HeyChobe
my modifications were made at that time, within the same plugin, this file to be specific
This contain to M like a returned object

Thank you @yebt , in mi case I have this:

What should be M for me?  Because when I use it, I have an error like this:

Hi @HeyChobe
my modifications were made at that time, within the same plugin, this file to be specific
This contain to M like a returned object

Ohh perfect, now I understand, thank you for your help :D

E5108: Error executing lua: ...dia-files.nvim/lua/telescope/_extensions/media_files.lua:97: attempt to index local 'entry' (a nil value)
stack traceback:
        ...dia-files.nvim/lua/telescope/_extensions/media_files.lua:97: in function 'run_replace_or_original'
        ...ckages/start/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'
        ...Packages/start/telescope.nvim/lua/telescope/mappings.lua:257: in function <...Packages/start/telescope.nvim/lua/telescope/mappings.lua:256>

the plugin isn't working for me, fd runs fine, I'm not sure what's the problem here.


[APIM-3.1.0] LDAP authentication is not working when certain special characters are there in password

Describe the issue:
In APIM-3.1.0, when we use org.wso2.carbon.user.core.ldap.ReadOnlyLDAPUserStoreManager as secondary user store, it is not able to authenticate users having special characters in password like '%' sign. In this case it simply returns invalid credentials response for login action.
How to reproduce:
N/A
Expected behavior:
LDAP authentication for passwords having special characters.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: APIM 3.1.0
OS: Windows
Database: MySQL, H2
Userstore: JDBC, LDAP

Optional Fields
Related issues:
https://github.com/wso2/api-manager/issues/1693
Suggested labels:
APIM - 3.1.0

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Fix crash on kmeans2

scipy.cluster.vq.kmeans2 would crash on random initialization when its data argument contains less data points than dimensions.
This pull request adds a test, test_kmeans2_high_dim, which highlights the issue, and fixes the random initialization procedure to use pca instead of cholesky factorization of the covariance matrix when the number of data points is less than the dimension.

Looks reasonable. I confirm both the LinAlgError and that this PR fixes it.
I'll keep it open for a while in case someone has comments. Otherwise, it's a bugfix I'm going to merge soon.

I've fixed the missing division, and added a test to check the covariance of the generated data. The comparison of the covariances makes use of a relatively loose tolerance but I can't get better precision without using a much larger k, which uses too much memory IMO.

np.fromfile usage needs to be removed from tests, now that https://github.com/scipy/scipy/pull/5573 is merged. Here is the branch where I did this,  squashed the commits and rebased on master
https://github.com/ev-br/scipy/commits/pr/5486
An unpleasant thing is that this line, x = np.dot(x, sVh) + mu, fails with a MemoryError in a wine VM (which, admittedly, is rather tight on memory).

An unpleasant thing is that this line, x = np.dot(x, sVh) + mu, fails with a MemoryError in a wine VM (which, admittedly, is rather tight on memory).
The MemoryError surely comes from the covariance test. It needs quite a lot of data to get a nice approximation of the covariance matrix. Maybe the test ought to be disabled on platforms without enough memory?

np.fromfile usage needs to be removed from tests, now that #5573 is merged. Here is the branch where I did this, squashed the commits and rebased on master

Is there something I should do about this? I'm unfamiliar with the process, should I update my pull request using your branch, or are you going to merge your branch?

If you have an idea of how to trigger the problem for smaller problem size, it'd be helpful (eg, do you really need k=1e6?). If you can do it, please push a commit to your branch, I'll cherry-pick it. Otherwise, I'll just add a skip condition and merge the rebased one.

If you have an idea of how to trigger the problem for smaller problem size,
it'd be helpful (eg, do you really need k=1e6?). If you can do it, please
push a commit to your branch, I'll cherry-pick it. Otherwise, I'll just add
a skip condition and merge the rebased one.

The k=1e6 is only required to test that the _krandinit actually produces the
correct covariance in the high dimensional case. With lower values it's harder
to see if the obtained covariance matches the initial covariance.
However, this test is only here to check the correctness of the fix, and as
such it is only required to pass on platforms with enough memory.
The test_kmeans2_high_dim, on the other hand, actually highlights the bug,
and should pass on any platform.
So I guess adding a skip condition is the way to go.

Rebased, squashed and merged with the skip condition on the test in https://github.com/scipy/scipy/commit/db14b78e711bdb7c9611160f9d476cc936310433. Thanks @vbarrielle.


refactor: correct wrong import in date selection

Fixes an incorrect import in the date-selection.ts that breaks the build.
Fixes #13123.
@jelbourn I'm not too sure whether this is supposed to target the 6.4.x branch.

Caretaker note: for some reason the CI is running against the lint rules from master.


Weird TX Still in Mempool with Wrong supported Value

there seems to be an event in the mempool with underlying tx that was confirmed 2 months ago

hum.. same tx_hash ? sometime there is the same event with another tx_hash.. anyway I think we should add our own expiration system for tx in mempool (for now we wait a remove message from ZMQ)

https://gist.github.com/mattmarcello/ae4bd906c25b97eb65077f735f3d2dd4

I can't explain or reproduce but I added two sanity checks here https://github.com/CounterpartyXCP/counterparty-core/pull/2217:

Test that the mempool is properly cleaned after each regtest transaction is confirmed
Delete mempool events older than 24 hours

and there is also now a regtest test to check that the mempool is filled


Fix tests to avoid using args/kwargs attribute of Mock.call_args

This is needed as Mock.call_args.{args,kwargs} was added in Python 3.8.

/test

Merging as test passed.