compose: Eliminate the json/subscriptions/exists synchronous check.

The discussion of these changes can be found here.
To sum it up, a user is not allowed to send a message to the stream they are not subscribed to, unless the request contains the autosubscribe variable set to true. In this case, the user will be automatically subscribed to the stream before sending a message.
To see this behavior though the web interface, you can narrow down to a stream using the link of the following form: https://chat.zulip.org/?stream=frontend. The messages sent after narrowing down to a stream the regular way (by clicking on its name) will be rejected by the backend.
Please let me know if anything can be changed or improved.
Fixes #4650.
@showell @timabbott FYI :)

@showell Thanks for reviewing this! I addressed some of your feedback, and planning to investigate what happens when the stream exists only locally a bit later.

@alenavolk it looks like this fails the node tests because it reduces coverage on transmit.js; can you investigate?

I merged the first commit, since I definitely like that refactor.

@timabbott Sure, I'll let you know as soon as I fix that.

@timabbott Sorry about the delay, the PR is ready for review.


/dashboard opens even without token

This can be avoided by just checking if the token is set if it is not set then just redirect to signup/signin page.

ok i will fix it

fixed


First build failed.

The first build failed because specclassify was not yet uploaded to conda-forge.

@conda-forge-admin, please restart ci


[WIP] add redux to api middleware

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

WIP just to show some ideas about the redux middleware for now. Ignore the ResultSet tsx/functional component updates. I'll separate those out into a different file so that it's not mixed in with these changes, but they are a prerequisite.


Makefile.org: clear APPS environment variable.

Build failure was reported in GH#1818 if APPS environment was defined.

+1


Multiple issues in OrientDB ETL

OrientDB Version: 3.1.5
Java Version: 8
OS: any
There are multiple issues in OrientDB ETL. To be honest, it's hard to say: is it working at all?:) So my main question is: is OrientDB ETL recommended for actual use? Isn't it deprecated?
If it's active - not a problem - will fix and send PRs.
Few examples:

There is NPE if lock was requested for source file
There is NPE in processing simple CSV (still investigating source - it's harder to do due to p.3)
Logs are not relevant. For example: OETLContextWrapper.getInstance().getMessageHandler().error(this, "ETL process halted: ", e); - there is "ETL process halted: " is actually format for message - as a result passed exception not even shown in logs.
OETLContextWrapper.getInstance().getMessageHandler() - is taking global MessageHandler. Bersion 2 allowed to configure Message Handler per execution - but due to this change - there is no ability to catch "grouped" errors logs per execution.
OETLProcessor.execute() is just sequencial execution of underlying methods - so no catch/finally and etc. As a result - if something happened during records processing - nothing will be "closed" properly.

Hi @PhantomYdn
Yes, the ETL module is still supported.
I'll try to check the issues you reported, but if you have a fix it will be much appreciated
Thanks
Luigi

@luigidellaquila , just bts: I'm collecting all proposals in the following my branch: https://github.com/PhantomYdn/orientdb/tree/orientdbetl_fix

Hi @PhantomYdn
Perfect, thanks!
Please let me know when you have something ready to check
Thanks
Luigi

@luigidellaquila , please check PR #9577

@luigidellaquila , please consider merging.

Hi @PhantomYdn
Checking now
Thanks
Luigi

Merged, thanks!


Support for answer tuples

Hello,
I noticed that there is a very old branch with support for answer tuples ( https://github.com/sneeuwballen/zipperposition/tree/answer-tuples )
@c-cube , are there any plans to make that functionality work with the current state of Zipperposition?
An example problem to test against:
https://www.tptp.org/cgi-bin/SeeTPTP?Category=Problems&Domain=PUZ&File=PUZ081^3.p
Currently the prover correctly returns SZS status Theorem but how could I extract any valid answers?
Thanks!

hi, I don't think there's any particular plan to reimplement/adapt this. I don't remember if it worked well back then, there's a few subtleties around weight/precedence of the fake answer literal in clauses.

I see. What do you mean with subtleties?
If I give it a shot to try to adapt the old changes from the old answer-tuples branch into master, is there a high chance that it would work? I can't program in OCaml BTW but have some programming experience :)

If memory serves, the way this is implemented is by creating a symbol $answer(X,Y,Z) when the question ?[X,Y,Z]: foo is asked, and solving the CNF of foo(X,Y,Z) => $answer(X,Y,Z), but where a clause only containing $answer(a,b,c) is considered as actually empty (and you just read the arguments to find the answer X := a, Y := b, Z := c). The problem with that is that it impacts all the heuristics about unit clauses (clauses with 1 literal), literal counts, size of clauses, weight of clauses, etc. unless you make really sure that $answer(…) is invisible to all of these.
You can give it a shot, but there's been quite a lot of changes since then so it might be a significant amount of work :/

That sounds like a daunting task. Perhaps, I'll give it a shot next time when I have a weekend to spare.
Thanks for the info :+1:


Suggested change to document pertaining AAD connector account

Can we please remove the sections pertaining to changing the On-Premises Directory Synchronization Service Account (AAD connector account) password and abandoning the encryption keys? It's only doable on AADC v1 which isn't even supported anymore. Despite the information box at the top of the article this is still confusing customers and causing some to break their environments.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 5a6ac21b-3096-44b3-8099-f029a2bcf718
Version Independent ID: 30085cd3-7d19-e88b-d4c6-14e9524ec046
Content: Azure AD Connect sync:  Changing the ADSync service account - Microsoft Entra
Content Source: articles/active-directory/hybrid/how-to-connect-sync-change-serviceacct-pass.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@msftmarysisko
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for your time!
Mary Sisko
Support Engineer | Azure Identity | Sync + Account Management
Customer Services & Support | Microsoft Corporation
Office Phone: +1<PHONE_NUMBER>
Working Hours: M-F, 8am-5pm MST
Alternate Contacts if I am unavailable:
Back Up Engineer | Tyson Smith<EMAIL_ADDRESS>TA Juan Quesada Perez<EMAIL_ADDRESS>Manager | Isaac Cortes Ugalde<EMAIL_ADDRESS>In order to ensure receipt of case communications, please make sure<EMAIL_ADDRESS>and<EMAIL_ADDRESS>are kept in the cc line. Thanks!
Need help outside of my working hours?
Locate an engineer: Azure Support Backup<EMAIL_ADDRESS>[Image]
For Info on AADConnect V1 retirement and the V2 upgrade, please refer to - https://docs.microsoft.com/en-us/azure/active-directory/hybrid/whatis-azure-ad-connect-v2https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Factive-directory%2Fhybrid%2Fwhatis-azure-ad-connect-v2&data=04|01|prakh%40microsoft.com|a497dd6453d14c67d3d808da07d43d02|72f988bf86f141af91ab2d7cd011db47|1|0|637830905440506774|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=lYqApLSLSCEg57VdPuJMDQoFU1eBCHGEeCkhHTmzjWc%3D&reserved=0

From: YashikaTyagi @.>
Sent: Tuesday, February 21, 2023 8:51:08 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Mary Sisko @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Suggested change to document pertaining AAD connector account (Issue #105563)
@msftmarysiskohttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmsftmarysisko&data=05|01|marysisko%40microsoft.com|7e514c1d1d614da7220b08db14880871|72f988bf86f141af91ab2d7cd011db47|1|0|638126346728355152|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=mlqjZ%2BEfmY5hmcBkmbFDEvXH8sl%2FkiiK60v4fxgZjE0%3D&reserved=0
Thanks for your feedback! We will investigate and update as appropriate.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F105563%23issuecomment-1439401397&data=05|01|marysisko%40microsoft.com|7e514c1d1d614da7220b08db14880871|72f988bf86f141af91ab2d7cd011db47|1|0|638126346728355152|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=3BT02SpCmpaY7%2FEur7dKZ0EMuAk1xVI0eZX%2ByT94OH0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA2ZU7QKUENBO4PKKBE6ZPFDWYWEKZANCNFSM6AAAAAAVDUK77A&data=05|01|marysisko%40microsoft.com|7e514c1d1d614da7220b08db14880871|72f988bf86f141af91ab2d7cd011db47|1|0|638126346728355152|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=d9uEvp%2FSmHzSWk4oX9LuEVeZmq75SbuHOjTr7pPMGDc%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@msftmarysisko
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly
@billmath
Can you please check and add your comments on this doc update request as applicable.

Thank for submitting this. I have created a work item for it and will reviewing this in the coming weeks. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


API Integration for challenges

@whyDontI  @sumitd94
Fetched data from challenges API. Please review the PR.
https://github.com/Real-Dev-Squad/website-status/commit/9c9533cf029d189627d69188c781b18157eab148

Duplicate of - https://github.com/Real-Dev-Squad/website-status/pull/123


Changed return object and updated tests

Changed so the membership methods return membership Dtos instead of course. Added a check for existing user in CourseService methods, and also added proper exceptions.
Changed tests to match the new updates.
closes #51, fixes #54

Maybe you should try implement Guard clauses like we discussed in todays meeting?

I'll look into it


$firebaseObject doesn't sync properly serverValue.TIMESTAMP in contrary to $firebaseArray

If I set one of the fields of a three-way bound (with $firebaseObject) object in controller to Firebase.ServerValue.TIMESTAMP, the result will be that UI is rendered with {".sv":"timestamp"} and the only way to refresh would be to reload the controller. I.e. when the object is saved in Firebase and is read from the server data is populated correctly. But at the moment of update, $firebaseObject seems not to wait for data back from server to refresh the model.
This is not the case however when this field is part of an object $added to `$firebaseArray'
Reproduced in fiddle
Am i missing something? Is this supposed to work like this?

AngularFire does not handle three way binding of server values at this time. One small change makes your code work though:
    // $scope.currentUser.item = sanitized;
    ref.child('item').set(sanitized);

This basically takes responsibility for setting the value away from AngularFire and uses Firebase directly.

This is definitely a good feature request though, and should go on our todo list. :+1:

Possibly right after deleting $bindTo() in its entirety. It's not enough gains over just calling $save() to create so much maintenance and time spent fixing bugs.

Original report: http://stackoverflow.com/questions/31256363/firebase-servervalue-timestamp-displayed-in-view-as-svtimestamp-with-pu


Improved XML documentation

I added more XML documentation to several public types.

Thanks! Looks like you need to resolve a merge conflict.

Yup I saw that change go in earlier. I'll do a quick merge and it should be resolved later.

@Haacked conflicts all handled.

Really appreciate this!


Change telemetry devices to rely on jvm.config instead of ES_JAVA_OPTS

Move telemetry devices that currently rely on ES_JAVA_OPTS from the
launcher to the provisioner and instead persist the necessary
information in config/jvm.options

Depends on https://github.com/elastic/rally-teams/pull/28

@danielmitterdorfer Should I also add support for telemetry java options in docker provisioner in this PR or a different PR?

IMHO this change should be consistent so you'd need to adapt the Docker provisioner as well. As I will start my vacation at the end of the day it would be best if @dliappis could pick this up as a reviewer.

@ebadyano I will do a review as well and test this a bit, so let's not merge yet please until I've done that too.

Thank you for the review @dliappis . I update the change according to your comments.


Get list of all workbook from Tableau server

Hi,
I am a server admin and my objective is to get list of all workbooks present on Tableau server.
But If I am specifying site_id then only the content of that particular site is coming and if I am not specifiying the site_id then, the content of only default site is coming.
How to get the list of all workbook irrespective of sites?
Thanks
Rishabh

I'm not aware of anything in the API that would allow you to do this and don't expect there to be as I don't believe you're able to explore combined site content in the web UI either.
I think you'll want to loop over a collection of TableauAuth credentials providing a login to each site and run your workbook get function against each site. For example, you'd use something like the following:
import tableauserverclient as tsc

# server admin creds
host="tableaudomain.com"
user="username"
pwrd="password"

tableau = tsc.Server(host, use_server_version=True)
auth = tsc.TableauAuth(user,pwrd)

site_auths = []
with tableau.auth.sign_in(auth):
    site_auths.extend(
        tsc.TableauAuth(user, pwrd, site.id)
        for site in tsc.Pager(tableau.sites))

workbooks=[]
for site_auth in site_auths:
    with tableau.auth.sign_in(site_auth):
        workbooks.extend(tsc.Pager(tableau.workbooks))

print(workbooks)```

If you're open to not using this API - and have access to it - you could just dump it out from the [workgroup database](https://help.tableau.com/current/server/en-us/data_dictionary.htm).

Starting with #655 you can use a baked in site switch rather than creating separate auth objects.

This isn't on the roadmap.


poetry: fix build on x86_64-darwin

Description of changes
fixes #253971

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

cc @mirkolenz @moonshxne

Thank you for the quick response! Just out of curiosity: Why is an empty list of patches needed here? And why is this not an issue on Linux machines?

Why is an empty list of patches needed here? And why is this not an issue on Linux machines?

https://github.com/NixOS/nixpkgs/blob/e72d1a3ac4ed85c4acd33259dc8e551d7c108a72/pkgs/development/python-modules/poetry-core/default.nix#L31-L66

For some reason poetry.python.pkgs.deepdiff fails to build:
=================================== FAILURES ===================================
______________________ TestDiffOther.test_repeated_timer _______________________

self = <tests.test_diff_other.TestDiffOther object at 0x1133c97e0>
MockDiffLevel = <MagicMock name='DiffLevel' id='4625369840'>

    @mock.patch('deepdiff.diff.DiffLevel', side_effect=SlowDiffLevel)
    def test_repeated_timer(self, MockDiffLevel):
        t1 = [1, [2]]
        t2 = [1, [3]]

        progress_logger = mock.Mock()
        DeepDiff(t1, t2, log_frequency_in_sec=0.02, progress_logger=progress_logger)
>       assert PROGRESS_MSG.format(0, 0, 0) == progress_logger.call_args[0][0]
E       TypeError: 'NoneType' object is not subscriptable

tests/test_diff_other.py:28: TypeError

For now let's just disable deepdiff's tests.
@NixOS/darwin-maintainers Someone has to get rid of the poetry-core patches or we'll have to mark it broken on x86_64-darwin very soon.


Eslint object creation is slow

ESLint object creation (line #1) below is taking 6-8 secs approx in AWS ECS environment. There are approximately 20 Graph ql schema files. Is there a way we can fine tune the performance? Please let me know.
const eslint = await new ESLint ({
useEslintrc: false,
baseConfig: (
parser: "@graphql-eslint/eslint-plugin",
plugins: ["@graphgl-eslint"],
parserOptions: (
schema: [
/* GraphoL */
directive @key (
fields: String = ""
) on FIELD
directive @link (
DEFINITION | ENUM VALUE OBJECT,
url: String = "", import:
[String] = ["abc"]
) on FIELD DEFINITION | ENUM
VALUE | OBJECT,
type Query{
dummy: String
type Mutation{
dummy: String
}
type Product {
dummy: String
}
type schema {
dummy: String
}
`
],
},
rules: rules,
noInlineConfig: true
},
})

Please provide a reproduction repo and I take a look

@B2o5T , I have invited you to one of my test repository. Please let me know if you are able to access it.
I am using similar config.json file for my ESLint configuration in actual project. Currently I am getting error "ConfigurationNotFoundError: No ESLint configuration found in /Users/userdir/Documents/workspace/linter/graphql/chats." which I am planning to fix but just wanted to share the repo to give an idea on the configuration. Also, this repo has 3 graphql files, but in my actual usecase I have ~20 files. Please advise.

I have fixed the error, can you please take this repo and configuration as reference?
The slowness is not noticeable in local but only noticeable when running in AWS ECS environments. Please advise.

@dimaMachina , did you get a chance to look into this?


TEST(22): Reorganizar pastas de testes.

close #22

Codecov Report

Merging #23 (de13759) into main (f64cf8f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #23   +/-   ##
=======================================
  Coverage   95.32%   95.32%
=======================================
  Files          32       32
  Lines         620      620
=======================================
  Hits          591      591
  Misses         29       29

Files Changed
Coverage Δ

...ornada_milhas/core/tests/model/test_destination.py
92.00% <ø> (ø)

jornada_milhas/core/tests/model/test_post.py
88.88% <ø> (ø)

...a_milhas/core/tests/serializer/test_destination.py
100.00% <ø> (ø)

jornada_milhas/core/tests/serializer/test_post.py
100.00% <ø> (ø)

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


Update Proctypes

What did you do?
npm install --save react-foundation
What did you expect to happen?
program would run
What actually happened?
console.warn node_modules/react/lib/lowPriorityWarning.js:40
Warning: Accessing PropTypes via the main React package is deprecated, and will be removed in  React v16.0. Use the latest available v15.* prop-types package from npm instead. For info on usage, compatibility, migration and more, see https://fb.me/prop-types-docs
What version of this module are you using?
"react-foundation": "^0.8.0",
Please include code that reproduces the issue. The best reproductions are self-contained scripts with minimal dependencies.
import { Badge, Colors } from 'react-foundation';
...
...
          <div className="badge-colors-example">
            <Badge color={Colors.SECONDARY}>2</Badge>
            <Badge color={Colors.SUCCESS}>3</Badge>
            <Badge color={Colors.ALERT}>A</Badge>
            <Badge color={Colors.WARNING}>B</Badge>
          </div>

This is duplicate of #31 and is fixed with #34

thank you


Question: Does azfuse support working with blobfuse mounts?

With the release of Blobfuse v2.0 there is a new config file (and possibly new backend).
Does azfuse support the new yaml configuration file?
Can it be used on a mounted blobfuse v2.x directory?
Thanks

Does azfuse support the new yaml configuration file?
do you mean whether azfuse supports the configuration file of blobfuse2? If so, no.
Can it be used on a mounted blobfuse v2.x directory?
If the local path in azfuse is pointing a mounted blobfuse directory, it is fine as 'local path' is more like a logic path. azfuse won't try to access any data in the real path as it will download the data from 'remote path' to 'cache path' and open the file in 'cache path'.


Type annotations for Registry usage are not well-supported by mypy, slightly tricky with pyright

I'm finally back to taking some time to work on updating check-jsonschema to use jsonschema>=4.18 with referencing!
The only issue I've really had so far is that the type annotations are not as straightforward to satisfy as I would like.
The attrs-derived initializers are not understood by mypy as well as they are for pyright, and pyright is being a bit "fussy" about type invariance.
There's a cluster of related problems to work out, but I think my main item is the failure under mypy. This seems like a deficiency of mypy, since pyright works, but it may be a deficiency which is easy enough to be worth accommodating; I don't know yet.
Here's a simplified version of my registry builder, resolver_simple.py:
from __future__ import annotations

import referencing
import requests
from referencing.jsonschema import DRAFT202012, Schema

def retrieve_reference(uri: str) -> referencing.Resource[Schema]:
    data = requests.get(uri, stream=True).json()
    return referencing.Resource.from_contents(data, default_specification=DRAFT202012)

def make_reference_registry(
    schema_uri: str | None, schema: dict
) -> referencing.Registry:
    schema_resource = referencing.Resource.from_contents(
        schema, default_specification=DRAFT202012
    )
    registry = referencing.Registry(retrieve=retrieve_reference)

    # aside: I'm not 100% sure these next parts are correct
    # schemas with no explicit `$id` are inherently ambiguous
    # but it would be nice to support using the URI from whence we retrieved them
    if schema_uri is not None:
        registry = registry.with_resource(uri=schema_uri, resource=schema_resource)

    id_attribute = schema.get("$id")
    if id_attribute is not None:
        registry = registry.with_resource(uri=id_attribute, resource=schema_resource)

    return registry

mypy doesn't understand that retrieve is an attrs-provided field:
$ mypy resolver_simple.py
resolver_simple.py:19: error: Unexpected keyword argument "retrieve" for "Registry"  [call-arg]
resolver_simple.py:19: error: Need type annotation for "registry"  [var-annotated]
Found 2 errors in 1 file (checked 1 source file)

By contrast, pyright likes this file and gives it a pass. Which makes my next example a little bit of a headscratcher at first.
The real implementation makes parsing a bit more pluggable, for which I have a simple object, ParserSet. As a result, retrieve is built dynamically, and the resulting type somehow mismatches under pyright. It looks something like this:
def create_retrieve_callable(
    parsers: ParserSet,
) -> typing.Callable[[str], referencing.Resource[Schema]]:
    def retrieve_reference(uri: str) -> referencing.Resource[Schema]:
        data = requests.get(uri, stream=True)
        parsed_object = parsers.parse(data.raw, uri)

        return referencing.Resource.from_contents(
            parsed_object, default_specification=DRAFT202012
        )

    return retrieve_reference

which results in a pyright error like
error: Argument of type "(str) -> Resource[Schema]" cannot be assigned to parameter "retrieve" of type "Retrieve[D@Registry]" in function "__init__"
    Type "(str) -> Resource[Schema]" cannot be assigned to type "(uri: URI) -> Resource[Schema]"

After staring at the error output long enough, I realized that pyright is picky about this because that Callable type doesn't specifically have the signature def f(uri: str) -> referencing.Resource[Schema]: .... f here takes uri as a keyword-or-positional -- something which Callable is inadequate to express.
The fix is "easy" but weird! :sweat_smile:
class _RetrieverFunc(typing.Protocol):
    def __call__(self, uri: str) -> referencing.Resource[Schema]:
        ...

def create_retrieve_callable(parsers: ParserSet) -> _RetrieverFunc:
    # body unchanged

Between the two, it feels difficult to setup what feel -- at first -- like relatively simple annotations to pass. As I said above, given that pyright can follow the _attrs module in referencing, I'm inclined to think that's a mypy bug or missing feature. And pyright is picky, but in a way that feels tolerable to use and correct.
The last issue I have is with pyright's type matching rules, so I might be wrong about how to solve this or whether or not it should be allowed, but I found that I had to import referencing.jsonschema.Schema to satisfy types. (You may have noticed this usage above.)
I tried setting up a recursive TypeAlias to describe JSON datatypes, and using that
JsonPrimitive: t.TypeAlias = t.Union[int, float, str, bool, None]
JsonElement: t.TypeAlias = t.Union[JsonPrimitive, "JsonList", "JsonDict"]
JsonList: t.TypeAlias = t.List[JsonElement]
JsonDict: t.TypeAlias = t.Dict[str, JsonElement]

JsonSchema: t.TypeAlias = JsonDict | bool

But pyright doesn't like Resource[JsonSchema] because it doesn't match against Resource[Schema].
I think it would work if I used the exact value in Schema of bool | Mapping[str, Any], but no other type will pass because Resource is invariant in its type variable.
So my final note (long issue report, sorry!) is that I'm probably going to be importing Schema on the assumption that it's safe to rely on that name and using it to ensure that my annotations always match referencing. If we can adjust the variance of Resource to be looser, that might make it easier to apply a narrower type.

Thanks for the detailed issue, which I haven't read through fully yet :D -- but because it seems like it may at least be slightly related, I also can't figure out why newer versions of pyright seem to have issues with type invariance either! I have it pinned to 1.1.307 here in the repo's noxfile which is the newest working version. Newer versions complain about things which make no sense to me, though I've only spent a few minutes trying to poke at them.
I'll have a closer look at the rest of what you shared after the weekend, thanks for writing it up.

(Will probably repeatedly respond to this in piecemeal as I flip back and forth between looking at it and some other things) --

mypy doesn't understand that retrieve is an attrs-provided field:

I remember fighting with this a bit -- I think mypy doesn't understand dataclass_transform yet or something, specifically it doesn't understand attrs' alias argument -- and vice versa, pyright is slightly dumb and doesn't understand that the alias argument isn't needed at all when all it's doing is automatic renaming of private attributes, so you have to write alias even when it's not needed.
More type checker time wasting, grumble grumble.

I'm probably going to be importing Schema on the assumption that it's safe to rely on that name and using it to ensure that my annotations always match referencing

referencing.jsonschema.Schema is definitely public -- lemme know if there's some other convention in the typing community to make that even more obvious. For that matter, so is referencing.jsonschema.SchemaRegistry which I figured would be even more common to use, it's just Registry[Schema] specializing the generic to something JSON Schema specific (as opposed to future support for OpenAPI or something).

So -- I think given that the mypy part of this is partly or in whole a "known issue" with mypy -- namely python/mypy#5406 -- and given the last comment about indeed using the public objects for pyright -- I'm going to close this perhaps.
Obviously speak up if you disagree, or if you have concrete suggestions for what to tweak, as should anyone else running into trouble here.
But yeah on the mypy side specifically I'm not sure if there's anything to do until things are fixed upstream.

Yep, I'm happy with that! (Well, in as much as I'm happy with the type checkers...)
The only part of this, on review, that I might keep an eye on is "how should we support that fancy recursive JSON type"? It's clearly a subtype of Mapping[str, Any] and it feels like it should work. But I don't need that but at all for my work -- it just felt more correct academically.


access public hoc templates with getCell or CellRef

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e7f1f3f) 84.08% compared to head (ca9f277) 84.83%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
+ Coverage   84.08%   84.83%   +0.74%
==========================================
  Files          76       76
  Lines        5197     5275      +78
==========================================
+ Hits         4370     4475     +105
+ Misses        827      800      -27

Files
Coverage Δ

bluecellulab/cell/template.py
95.06% <100.00%> (+0.39%)
:arrow_up:

bluecellulab/ssim.py
85.76% <100.00%> (+0.05%)
:arrow_up:

tests/test_cell/test_core.py
99.13% <100.00%> (+0.26%)
:arrow_up:

tests/test_cell/test_template.py
100.00% <100.00%> (ø)

bluecellulab/cell/core.py
75.66% <96.36%> (+6.27%)
:arrow_up:

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

@darshanmandge could you check if this works fine in your template (without adding getCell()) to it?


Added redux timing middleware as well as adding documentation for per…

…formance
Description
Added some middleware for capturing redux timings.
Added a page to the documentation describing how to profile the application.
Testing instructions
Add a set up instructions describing how the reviewer should test the code

Review code
Check Travis build
Review changes to test coverage
Review the docs

Agile board tracking
connect to #252
closes #252

Codecov Report

Merging #255 into version1 will decrease coverage by 0.37%.
The diff coverage is 11.11%.

@@             Coverage Diff              @@
##           version1     #255      +/-   ##
============================================
- Coverage      89.5%   89.13%   -0.38%
============================================
  Files            69       70       +1
  Lines          1944     1951       +7
  Branches        311      312       +1
============================================
- Hits           1740     1739       -1
- Misses          180      187       +7
- Partials         24       25       +1

Impacted Files
Coverage Δ

src/userTimingMiddleware.js
0% <0%> (ø)

src/malcolm/reducer/attribute.reducer.js
86.92% <100%> (-0.1%)
:arrow_down:

Continue to review full report at Codecov.

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


Make the Keystore path configurable for Workflow feature in Identity Server

Description
To support the workflow feature in IS, by default mutual SSL is enabled. If we disable mutual SSL, then the internal communication with the embedded BPS is blocked. The workflow feature should be able to use a separate keystore for mutual SSL communication and also the path should be configurable.
Type
Improvment
Priority
Normal
Affected version
5.2.0, 5.8.0
Component/s
workflows
Severity
Major
Estimated Complexity
Moderate
Label/s
Type/Improvement, Component/Workflow, Affected/5.2.0, Affected/5.8.0, Severity/Major, Complexity/Medium, Priority/Normal
Moved from https://wso2.org/jira/browse/IDENTITY-5281

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!


First release in Android

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
Should first release in Android be done manually?
Question Description
I am about to release a brand new Android App and I would like to know whether I should submit the first apk manually instead of using Fastlane. I have searched over the internet and documentation but could not find any info related to first apk release.

The first release can be done with fastlane.
You have to upload the first apk or bundle manually though, so Google Play Console connects your app to the bundle ID / package name which is used as the identifier for uploading and releasing an app. Google unfortunately does not have an API for that.


How can I remove / delete specific weights from a neural network layer (Dense layer or convolution layer) in Keras (or Tensorflow)?

Please make sure that the boxes below are checked before you submit your issue.
If your issue is an implementation question, please ask your question on StackOverflow or on the Keras Slack channel instead of opening a GitHub issue.
Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[x] Check that your version of TensorFlow is up-to-date. The installation instructions can be found here.

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

I am in need of a help.
I have a neural network model in Keras which is trained. So, I have weight values.
Now, I have to remove / delete certain weight connections that meet a particular criteria.
(So that the network size is reduced)
Is there a way in Keras to do this ?
I have tried to make a custom layer, but am unable to set weights.
Challenges I am facing are:
Do I need to replace Dense / Convolutional layer with my custom layer? Or can I do it in existing Dense layer by some means?
Secondly, if I have to use a custom layer, then how to proceed with weight updates.
Please advise and help.
Your kind help will be much appreciated.

google: 'how to do transfer learning with keras'
also, you can use the functional API to create a new model from specific weights for example:
model = get_model()
smaller_model = Model(model.input, model.layers[-5].output)

In this way you create a model that contains all the layers (with their weights) of the original model except the last 5

@Golbstein Sorry Golbstein, but you did not understand the problem. Let me give an example of the problem. Suppose there is a neural network which has two dense layers A and B. So, all neurons in layer A would be connected to all neurons in layer B. Now, after the neural network is trained, the connections between neurons of layer A and layer B will have some weights. Now, I want to remove / delete some weights based on a specific criteria (such was delete weight connections where weight < 0.5). So is there a way to do this in Keras or Tensorflow ? I tried developing a custom layer but couldn't complete it.

Please update

@fchollet @harshini-gadige Any update?

You should know that making a weight zero does not mean to delete it. Because backpropagation is still taking it into the calculation. So a subfunction required to delete a specific connection.


Off-sample front-end integration

@intsco Please review engine & graphql changes
@Ren22 If you have time, please review the webapp changes. If not, don't worry about it.
In engine:

Adds off-sample fields to ElasticSearch
Changes update-daemon to do off-sample analysis in two daemon actions: one high-priority INDEX, and one low-priority CLASSIFY_OFF_SAMPLE (which also indexes afterwards)
Updates scripts.run_off_sample to also reindex the dataset
Added a test fixture to test_sm_daemons that resets the test queues before the tests. This was needed because run_daemons would only cause 1 message from each queue to be processed, and annotation jobs now produce 2 messages - INDEX and CLASSIFY_OFF_SAMPLE. This seemed to cause the second test to not process its intended annotation message.

In graphql:

Adds API for viewing, filtering & sorting annotations by off-sample label/prob
Ignores unrecognized status queue messages so that CLASSIFY_OFF_SAMPLE doesn't log errors

In webapp:

Adds UI for viewing, filtering & sorting by the off-sample label
Adds 3 feature flags:

off_sample for hiding/showing the off-sample filter & diagnostics indicator
off_sample_col that shows the off-sample prob in the annotations table so that we can easily it. We don't have much space in that table, so I don't want it permanently visible. It can be temporarily activated by adding ?feat=off_sample_col to the URL
new_feature_popup that enables the "New Feature" popups. If this and off_sample are activated, the popup should appear when you first visit the Annotations page. If you dismiss it and want to see it again, run this in the browser console: localStorage.removeItem('dismissedFeaturePopups')

What about <10%, >90% options for extreme values of off sample probability? Or they are not supposed to be shown to users?

@intsco That table column is not supposed to be visible to users, I just added it for debugging.
However, it also appears as a raw value in the CSV. I'm not sure what to do about that - I don't want to introduce string values that would mess up parsing to floating point. Do you think it'd be fine if I just added a note to the CSV documentation?

@LachlanStuart adding a note to the CSV export documentation sounds like the best solution

Maybe adding it as a comment in the CSV itself?
On Mon 8. Apr 2019 at 10:39, Vitaly Kovalev<EMAIL_ADDRESS>wrote:

@LachlanStuart https://github.com/LachlanStuart adding a note to the
CSV export documentation sounds like the best solution
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/metaspace2020/metaspace/pull/272#issuecomment-480739455,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADF6_YpV_i84j-PSZKf31sBKtHpreGh1ks5vewAtgaJpZM4cc_79
.


[release-2.5] CVE-2022-31030

Signed-off-by: Jonathan Marcantonio<EMAIL_ADDRESS>Addresses:

[release-2.5] Upgrade Vulnerable Library Dependencies - multicloud-operators-channel #22725

/lgtm

/approve


Add support for MongoDB

It is becoming increasingly common to store certain content that would otherwise be in files in the database.   This issue specifically calls for MongoDB support. It should be possible to configure MongoDB and content would be stored in a document under a given collection.

Is their now a gridfs support in Keystone or is their any best practice to store binaries in gridfs?


Crashing when running v8.5.2 on Electron 26.1.0 on first function call to db

After updating a project to Electron 26.1.0 and better-sqlite3 to 8.5.2, the app immediatly crashed whenever the first function is called on a Database instance. The app was running on Windows 10 x64. The error reported is '[5204:0825/183808.198:ERROR:crashpad_client_win.cc(844)] not connected'.
A minimal example consiting of package.json and index.js looks like:
package.json
{
  "name": "minimal",
  "version": "1.0.0",
  "main": "index.js",
  "scripts": {
    "start": "electron .",
    "postinstall": "electron-builder install-app-deps"
  },
  "dependencies": {
    "better-sqlite3": "8.5.2",
    "electron": "26.1.0",
    "electron-builder": "24.6.3"
  }
}

index.js
const { app } = require("electron");

function onReady() {
    const Database = require('better-sqlite3');
    const db = new Database(':memory:', { timeout: 15000 });

    db.pragma('journal_mode = WAL;');
}

app.on('ready', onReady);

After installing and starting with npm run start the example app instantly crashes. Also happens with Electron 26.0.0. The issue disappears when reverting to Electron 25.7.0.

Thanks for reporting. On Ubuntu, it's crashing from the PRAGMA:
const { app } = require("electron");

function onReady() {
  console.log("pre require")
  const Database = require('better-sqlite3');
  console.log("post require")
  const db = new Database(':memory:', { timeout: 15000 });
  console.log("post new")
  db.pragma('journal_mode = WAL;');
  console.log("post pragma")
  db.close()
  console.log("post close")
}

app.on('ready', onReady);

$ npm start .

<EMAIL_ADDRESS>start
electron . .

pre require
post require
post new
/tmp/test/node_modules/electron/dist/electron exited with signal SIGSEGV

This is a duplicate of #1044 (but is a nice small repro, thanks!)


bugfix(TDI-38395) : fix empty schema

What is the current behavior? (You can also link to an open issue here)
Wrong behavior of "Guess Query" feature when there are no columns in the schema.
What is the new behavior?
Error message is shown
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[x] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...


Class 'SumUp\Utils\ExceptionMessages' not found

I get this error when sumup returns errror.
This may be due to the fact that file name is not same as the class name.
So autoloader is not finding it.
Renaming the file fixes the error

Can you provide some code as an example?
(don't forget to hide your sensitive data if you have some)


ROM端未响应

Running xiaogpt now, 用请问开头来提问
或用开始持续对话开始持续对话

问题：明天星期几？
Traceback (most recent call last):
File "/usr/local/bin/xiaogpt", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.8/dist-packages/xiaogpt/cli.py", line 153, in main
loop.run_until_complete(miboy.run_forever())
File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
return future.result()
File "/usr/local/lib/python3.8/dist-packages/xiaogpt/xiaogpt.py", line 470, in run_forever
await self.stop_if_xiaoai_is_playing()
File "/usr/local/lib/python3.8/dist-packages/xiaogpt/xiaogpt.py", line 414, in stop_if_xiaoai_is_playing
await self.mina_service.player_pause(self.device_id)
File "/usr/local/lib/python3.8/dist-packages/miservice/minaservice.py", line 52, in player_pause
return await self.ubus_request(
File "/usr/local/lib/python3.8/dist-packages/miservice/minaservice.py", line 32, in ubus_request
result = await self.mina_request(
File "/usr/local/lib/python3.8/dist-packages/miservice/minaservice.py", line 22, in mina_request
return await self.account.mi_request(
File "/usr/local/lib/python3.8/dist-packages/miservice/miaccount.py", line 150, in mi_request
raise Exception(f"Error {url}: {resp}")
Exception: Error https://api2.mina.mi.com/remote/ubus: {'code': 101, 'message': 'ubus server or device returned invalid result', 'data': {'device_data': '{"msg":"ROM端未响应","code":3012}', 'reqID': 'app_ios_****************************'}}
Task was destroyed but it is pending!
task: <Task pending name='Task-7' coro=<MiGPT.poll_latest_ask() running at /usr/local/lib/python3.8/dist-packages/xiaogpt/xiaogpt.py:102> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7f4aac5ff520>()]>>
Exception ignored in: <coroutine object MiGPT.poll_latest_ask at 0x7f4aac46adc0>

@jiangyanfeng 用最新的再试试？

重启小爱


Disable logging on console by default

A python library itself should write nothing on console without an explicit configuration
CLI tool should not write logs on console by default as well.

So you want an option to be able to turn off and on logs?
Python has a "logging" module provides a flexible way to log messages at different levels (e.g., debug, info, warning, error) and control the output of these messages. You can "import logging" in the script and it should overall be easier to configure when you want/don't want logs printed on the console.

This exists in Datumaro since v0.1

That's cool, I've just replying to issues cos I'm bored. So I didn't know that, but at least the info I gave was right I guess even if it wasn't necessarily relevant. Have a good day!


problem start binggpt with mac silicon M1

https://ibb.co/RPhQ1dF

Same problem with mac silicon M2

same problem here with m1 pro

Run this code in the terminal
@alexonbstudio @Rvsultan @xiurui-pan
xattr -rd com.apple.quarantine /Applications/BingGPT.app


Ignored the consensus test

Description

Fixes Issue # .
Type of change

[ ] Bug fix.
[ ] New feature.
[ ] Enhancement.
[ ] Unit test.
[ ] Breaking change (a fix or feature that causes existing functionality to not work as expected).
[ ] Requires documentation update.

Testing

probably need to also remove line 12: sh "git lfs install" from the Jenkinsfile


Fix Website sections using waypoint

Current sections of the website landing page are broken. It looks like the problem is the react16 and react-waypoint libraries

Fixed


chore(relay): Upgrade relay to v16

The type of this PR is: Chore
Description
This PR upgrades Relay to v16. There were a lot of type-changes between 14 and 16 as the Relay team is trying to clean up the differences between react-relay and relay-runtime. Also fixed a lot of ! errors.
No real breaking changes here, clicked around and things seem to work great.

Gonna add a MOG on this since its a pretty routine (albeit annoying) upgrade


Need update screenshot SignalR tutorial use Visual Studio 2019

Content https://docs.microsoft.com/en-us/aspnet/core/tutorials/signalr?view=aspnetcore-2.2&tabs=visual-studio#create-a-web-project
Need update screenshot SignalR tutorial use Visual Studio 2019

If you select 3.0-preview you'll see updated docs with VS 2019.
I'm not sure what the policy is for updating previous docs with new VS. We probably are going to keep 2.2 as-is, otherwise every new tooling will need a ton of updates to docs.
cc @guardrex

Moved to https://github.com/aspnet/AspNetCore.Docs/issues/13919.

@donhuvy Just a quick FYI: If you use the feedback form at the bottom of the topic, it opens the issue with our metadata, which helps us work it. I moved this over to #13919 using that approach ... you can see how it has this nice side-effect of automatically pinging the topic author.


Chore/modify-ua-string: Modifying User agent string

[ ] Tests written and linted ℹ
[ ] Documentation written in Wiki/README
[ ] Commit history is tidy & follows Contributing guidelines ℹ

What this does
Added a variable called USER_AGENT_PREFIX and set it to snyk-user-sync-tool-v3.0.0
Notes for the reviewer
Modified
src/lib/common.ts
src/lib/snykGroupsMetadata.ts
src/lib/snykGroup.ts
More information
Screenshots

Modified


Cookiecutter?

Should we turn this into a cookiecutter project? that way, people can initialize with a single line on the command line, and it will fill in a lot of the info for us (e.g., project name, code owners, etc)

Looks like both copier and cruft allow for easy updating of the template after creation (and cruft at least is compatible with cookiecutter), which seems useful, so look into them.


Bug: sort-union-types broken

Describe the bug
sort-union-types considers an inline comment as part of a union type and breaks it by sorting it
Code example
This case is ok
-    accountTypeIds?: (1 | 2 | 3 | 4 | 5 | 100)[]; // Specific Account type Ids to filter accounts. Example: 2
+    accountTypeIds?: (100 | 5 | 4 | 3 | 2 | 1)[]; // Specific Account type Ids to filter accounts. Example: 2

And here is the problem
-    accountTypeId: 1 | 2 | 3 | 4 | 5 | 100; // Account type ID. Example: 3
+    accountTypeId: 100 | // Account type ID. Example: 3 5 | 4 | 3 | 2 | 1;

ESLint version
8.21.0
ESLint Plugin Perfectionist version
1.5.0
Additional comments
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Thank you for issue. I think I can look into this more on Friday.

Fixed in 95560abdad35184a3d88598fccc7e45956804a4b.
Release is coming soon.

Released in v2.0.0


Handle when user ends auth before finished

When the popup is open, and the user does not complete auth, we aren't handling that in connect. We should do something like:

fire finished() when the component unmounts, and send something to help connect know it was incomplete
implement a new method for something like onCancel

I believe I got caught in this state by mistake and it made for some interesting results.
The full review is here in the forum with the more relevant parts below.

I started to create an account from the "How it works" page, and it never asked me to create an ID (xyz.id.blockstack). Instead, I saw the 12 word phrase and it still let me into the app. Unfortunately, I did not save the phrase as I intended to abandon the process and start over as I typed my review notes.

When I went to Get Started from the main page, I saw my raw ID on the list, and created a second new account. I chose overlyverbose.id.blockstack, but after doing so I was prompted for my email address and logged into the app. I was never presented with a Secret Key - but I was able to post and move around as expected.

I started completely over again by logging out and closing the tab. Now both the raw ID and overlyverbose show on the login screen, and when creating a new account for lessverbose.id.blockstack, the same thing happened. No secret key was displayed and I was prompted to enter my email for the app.

So now I have three accounts and no way to access them. :man_shrugging:

Hey @whoabuddy, thanks for the comments.
You don't see a new secret key, because all of these IDs are "tied" to a single Secret Key. You don't have different Secret Keys for different usernames.
In terms of losing your secret key - your secret key is still stored locally, so we could build a feature where you can recover your secret key, probably from something like the homepage. We don't have a timeline for that at the moment.

Thanks from me as well @whoabuddy.

I started to create an account from the "How it works" page, and it never asked me to create an ID (xyz.id.blockstack)

Is it possible you closed out the popup after first opening it and before getting to the username prompt? That seems like the most likely scenario here in having your Secret Key generated – and your session subsequently started – without ever seeing the username prompt.
We don't currently handle the case well when a user starts onboarding but doesn't get to the point of submitting a username. But I've created an issue to track that here: https://github.com/blockstack/app/issues/214
As @hstove mentions, we also don't currently have a way to see your Secret Key again in the app having signed up already. I've created an issue to track that need here: https://github.com/blockstack/app/issues/215

@hstove Thank you for the quick reply, that makes more sense now, thank you!
@markmhx Yes, that is definitely possible and likely what happened. I was doing a quick demo while in the middle of typing my notes on the other sections, so I didn't intend to fully sign up, just to show off the basics of the interface. Thank you!


Update Huggy Link

The Hugging Space have some problems with Unity demos.
Since a lot of people are starting the course I want them to be able to play correctly with Huggy so I added it to itch.io and updated the links.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Link RL statement and visitor

Pull Request Test Coverage Report for Build 1997

5 of 30 (16.67%)  changed or added relevant lines in 11 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0002%) to 59.131%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-proxy/sharding-proxy-frontend/sharding-proxy-frontend-mysql/src/main/java/org/apache/shardingsphere/shardingproxy/frontend/mysql/command/query/binary/prepare/MySQLComStmtPrepareExecutor.java
0
1
0.0%

sharding-proxy/sharding-proxy-frontend/sharding-proxy-frontend-postgresql/src/main/java/org/apache/shardingsphere/shardingproxy/frontend/postgresql/command/query/binary/parse/PostgreSQLComParseExecutor.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/core/visitor/ParseTreeVisitorFactory.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/MySQLVisitorFacade.java
0
1
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/OracleDMLVisitor.java
0
4
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-postgresql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/PostgreSQLDMLVisitor.java
0
4
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sql92/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQL92DMLVisitor.java
0
4
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sqlserver/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQLServerDMLVisitor.java
0
4
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/MySQLDMLVisitor.java
0
5
0.0%

Totals

Change from base Build 998:
0.0002%

Covered Lines:
12120

Relevant Lines:
20497

💛  - Coveralls

Codecov Report

Merging #4624 into master will increase coverage by <.01%.
The diff coverage is 16.66%.

@@             Coverage Diff              @@
##             master    #4624      +/-   ##
============================================
+ Coverage     55.52%   55.52%   +<.01%
  Complexity      421      421
============================================
  Files          1130     1130
  Lines         20492    20497       +5
  Branches       3792     3793       +1
============================================
+ Hits          11378    11381       +3
- Misses         8396     8398       +2
  Partials        718      718

Impacted Files
Coverage Δ
Complexity Δ

...phere/sql/parser/sql/statement/rl/RLStatement.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...gsphere/sql/parser/visitor/SQL92VisitorFacade.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ere/sql/parser/sql/statement/tcl/TCLStatement.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ere/sql/parser/sql/statement/dcl/DCLStatement.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...re/sql/parser/visitor/PostgreSQLVisitorFacade.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ere/sql/parser/sql/statement/dal/DALStatement.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...sql/parser/sql/statement/AbstractSQLStatement.java
100% <ø> (ø)
1 <0> (?)

...ere/sql/parser/visitor/SQLServerVisitorFacade.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...sphere/sql/parser/visitor/OracleVisitorFacade.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ere/sql/parser/sql/statement/ddl/DDLStatement.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

... and 12 more

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 0ce016e...31d7679. Read the comment docs.


Typesetting would be crate <B  ^__^

I do not know how so somebody else needs to do it

Um was gehts?

@etschgi1 https://docs.python.org/3/library/typing.html


fix Ink not updating on TextField newline

Summary
As documented in the Ink class, a layout LayoutChangedNotification needs to be dispatched.
When a FormField receives inputs it may change its size which causes box decorations (which are based on InkWells) on other elements not to update. To address this issue the text form field now dispatches the notification when it needs to be rebuild which must happen on a size change.
This change is tested by adding an Ink below a text field to which a new line is added.
Fixes #138219
Old behavior:

New behavior:

Pre-launch Checklist

[X] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[X] I read the Tree Hygiene wiki page, which explains my responsibilities.
[X] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[X] I signed the CLA.
[X] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[X] I added new tests to check the change I am making, or this PR is test-exempt.
[X] All existing and new tests are passing. (in text_form_field_test.dart)

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Is it an oversight that we've never added SizeChangedLayoutNotifier to EditableText before?

I am not entirely sure. My guess is that this was just an oversight and nobody thought about this before.


[TabView] Sluggish transitioning animation of the Tabview item header

Describe the bug
The animation seems sluggish when transitioning the tab item header from a long title to a shorter one
Steps to reproduce the bug

Create a WinUI3 project
Add a Tabview to the window and set the size TabWidthMode to "SizeToContent"
Add a tab item from code behind with a long title
Chang the title of the existing tab item to a short one

Expected behavior
The duration of  the transition animation should be very short
Screenshots
https://github.com/microsoft/microsoft-ui-xaml/assets/3145858/804f45b0-9e72-44b5-a696-9d1b7fc7c978
NuGet package version
WinUI 3 - Windows App SDK 1.3.1: 1.3.230502000
Windows version
Windows 11 (22H2): Build 22621
Additional context
No response

Pretty sure this was fixed in 2.8.4, in #8430, which was originally tracked at #8384

@Jimex, thank you for reporting this. As @zadjii-msft commented, this appears to be the same issue as #8384 , which has now been fixed. Please let us know if you still repro the issue with latest WinAppSDK.


Promise resolution when Terminal is ready?

From @jabooth on August 24, 2016 16:58
I'm trying to use this new terminal API to implement a Run File In Terminal command for the Python plugin: https://github.com/DonJayamanne/pythonVSCode/issues/261. In this use case, I need to instantiate a terminal and immediately send commands to it.
Perhaps I'm a little confused here, but on todays Insider build:
c4f365235906cd689e55376e44cc106003ac06b3
2016-08-24T06:02:40.440Z

the following code
const terminal = (<any>vscode.window).createTerminal('Test');
terminal.sendText('echo hello world');

leads to an error:
 Error: Terminal has not been initialised yet

as the terminal is asynchronously initialised in the first line.
As a test, manually adding a delay
const terminal = (<any>vscode.window).createTerminal('Test');
setTimeout(() => {
    terminal.sendText('echo hello world');
}, 1000);

lets everything work.
Looking into this I was going to propose that this API should be promise-based, but then I see this commit from 5 days ago:
https://github.com/Tyriar/vscode-terminal-api-example/commit/8a7820e86052090c59e85322f9207e00f19654c1 API no longer deals with promises
and the related work on vscode itself (https://github.com/Microsoft/vscode/commit/65888050c33641a5bf9d581f19a7f4094c6feb53)
which moves away from an asynchronous paradigm to this.
@Tyriar I don't see a public API for being told that the terminal is ready for input in the current draft, and I right in thinking this or am I missing something?
Copied from original issue: Tyriar/vscode-terminal-api-example#2

@jabooth will try get this fixed up soon, thanks for testing it out 👍


Tex reformat

Issue #8
@aakash1104 can you take a look at these changes? I tried to standardize the formatting a bit, or at least pull similar things (definitions, lists, etc.) into custom environments so we can modify the formatting all at once if necessary. Also tried to make the TeX a bit more readable (IMO), but that may have screwed up the formatting in some areas. I also may have switched some \Leftrightarrows with \iffs or \leftrightarrows accidentally/in the wrong places, so if you see any of those feel free to tag it and I'll fix them!

@matthewsot I can't seem to spot any \iffs in the wrong places. The formatting seems fine and I hope I haven't overlooked any errors. I think everything looks good. If you want to double check one more time and let me know, I can go ahead and merge the PR.


Add support for security manager endpoints

Fixes #2529

Thank you, @valbeat and @raynigon !
Merging.


⚠ Minecraft has degraded performance

In 3ab6256, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1183 ms

Resolved: Minecraft performance has improved in a98834c.


Can I use it without composer?

Hi, thanks for your work! I would like to know if there is a way to use the script without installing composer.
I need to use this script in shared hosting and unfortunately I can't use Terminal or install extensions.
Thanks!

The code is supposed to be run with Terminal commands, so putting it on a web server without Terminal access would probably not work well. However, it will probably run as long as the vendor folder exists and the environment variables are set.
As for the composer requirement it can be circumvented by running composer install on a computer with composer installed before uploading the code to the server. It will prepare the vendor folder, which will make the code work. I often use a CI server or deployment pipelines to accomplish this.

Thanks for your quick answer! I tried to run composer on Xampp and successfully created the vendor folder.
Unfortunately now I enconter an issue when I try to call create_timelapse.php.
I get the following error:
Fatal error: Uncaught Intervention\Image\Exception\NotReadableException: Image source not readable in C:\xampp\htdocs\timelapse\vendor\intervention\image\src\Intervention\Image\AbstractDecoder.php:346
I've installed php 8.0.7, ffmpeg as requested (no imagick), GD 2.1.0 and inserted 2 test jpg in "images-src" folder. What am I doing wrong?
Thanks again

First: Apologies for not catching such exceptions and providing an error message. Luckily, it is a reasonably understandable error message. Seems to me the script finds the folder, finds image files inside the folder, but chokes when trying to open one. This could be due to:

Errors in the file itself (not very likely)
Lacking permissions for the files inside images-src or to the folder itself. Does the running user (httpd or www, perhaps?) have read permissions to the files, and execute permissions for the overlying folder?
If you specified a logo file, is that readable?
Low memory limit for PHP. I would run with at least 128M as memory_limit

Also, the running user must have access to create folders and files alongside the images-src folder.


Rename tight coincidence

Rename tight_coincidence_channel into tight coincidence removes tight_coincidence_channel field from data.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 85.442%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
6051

Relevant Lines:
7082

💛  - Coveralls

Sure I do not see why not.


Robustness - SecretBox has wrong message authentication code (MAC)

BitKeep Android 7.3.4 appears to have a bad implementation for Ping Pairing.  Session Pairing in their UI works.   Just logging for better error handling on the package side.   Not a high priority for me.
Steps to reproduce:

Connect to BitKeep
Perform a Pairing Ping from our client (DAPP).

Ping appears to send find from DAPP
BitKeep response is possibly malformed, and the WalletConnectV2 package gets an Unhandled Exception

Expectation:

Malformed packets/messages are dropped  and event reported via onRelayClientError

pairing.dart - need to catch the error from core.crypto.decode
    // Decode the message
    String? payloadString = await core.crypto.decode(
      event.topic,
      event.message,
      options: DecodeOptions(
        receiverPublicKey: receiverPublicKey?.publicKey,
      ),
    );

I/flutter ( 7523): 19:45:52 🐛 HomePage        - Ping Pairing BitKeep
I/flutter ( 7523): 19:45:52 🐛 WalletConnctSvc - pingPairing - topic: 031a9eb332dfdb563c081c05bed258dd1d55cbd104514a0cd3909bba8ec7c3d7
I/flutter ( 7523): 19:45:57 🐛 HomePage        - AppLifecycleState: AppLifecycleState.inactive.
I/flutter ( 7523): 19:45:58 🐛 HomePage        - AppLifecycleState: AppLifecycleState.paused.
I/flutter (28973): _handleMethod:
I/flutter (28973): _handleMethod:
I/flutter (28973): _handleMethod:
E/flutter ( 7523): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: SecretBoxAuthenticationError: SecretBox has wrong message authentication code (MAC)
E/flutter ( 7523): #0      DartCipherState.convert (package:cryptography/src/dart/dart_cipher.dart:163:7)
E/flutter ( 7523): <asynchronous suspension>
E/flutter ( 7523): #1      DartCipherWithStateMixin.decrypt (package:cryptography/src/dart/dart_cipher.dart:421:23)
E/flutter ( 7523): <asynchronous suspension>
E/flutter ( 7523): #2      CryptoUtils.decrypt (package:walletconnect_flutter_v2/apis/core/crypto/crypto_utils.dart:145:22)
E/flutter ( 7523): <asynchronous suspension>
E/flutter ( 7523): #3      Crypto.decode (package:walletconnect_flutter_v2/apis/core/crypto/crypto.dart:182:28)
E/flutter ( 7523): <asynchronous suspension>
E/flutter ( 7523): #4      Pairing._onMessageEvent (package:walletconnect_flutter_v2/apis/core/pairing/pairing.dart:604:29)
E/flutter ( 7523): <asynchronous suspension>

I would actually expect this to throw an error at the pairing level, not the relay client error.
The pairing is the one with the information on the MAC and how to encode/decode messages, the relay client just forwards the information. Thus the pairing is aware of the error, and not necessarily the relay client.
Perhaps a new event should be created when the pairing fails to decode a message, as shown above.
Does that sound reasonable? Maybe not, it might just be another event to listen in on...
This could also be an issue with my encoder/decoder, however if it only exists for a pairing ping, I doubt that is the case.

Flexible on where / how to report.   If it's a malformed message, the less processing the better.  But it is still important to be able to account for all the bytes sent/received  with the relay.   So if it was a valid relay packet, but invalid as a Pairing Ping your logic makes sense to throw an error at the pairing level.
I'll update this issue if I see it from other wallets.  I suspect it's just a bug on the wallet side as we all get out V2 versions out.
I am still working on the best way to handle both kinds of pings(Pairing and Session) and keep counters for success/timeouts/other errors.  I can open a separate issue to clarify those questions after I have done more homework.

I remember seeing this, but have not seen it in some time.
I didn't intentionally fix it, but I have been working with different auth things. I may have inadvertently resolved it.
If you haven't seen it since the 2.0.6 update, would you mark this as closed for me please?

Tested wit 2.0.7 against Bitkeep 7.3.5 and no longer see any errors thrown from the WalletConnect package.  Closing.

Tested with 2.0.8 today and I see the error.    I was testing incorrectly before (Session ping from wallet) so disregard my closing comments.
What I wanted to use pairng ping to do

Verify my DAPP communication to the WalletConnect Relays is up.

What I did:

core.pairing.ping(topic: pairingTopic);
The paring topic was taken from the pairingInfo.topic returned from core.pairing.getPairings();

The testing wallets showed an incorrect/expired URI error.   The error returned via the protocol triggers the SecretBox error on my DAPP.
Question:

Could the error returned be in plaintext?
Did I use the incorrect topic?

Attempted to test with WalletConnect package 2.1.4.
The Bitkeep wallet is no longer listed on the Wallet Connect Cloud Registry
https://walletconnect.com/explorer?search=bitkeep
Closing this issue as no longer testable. (reproducible)

They were "using an ancient version of the WalletConnect repo"
They also rebranded to BitGet I think.
Thanks for cleaning up the issue @bobwith2bees !


3rd-party components with "Layout" in the name generate incorrect import paths in components.d.ts

[x] I have read the [troubleshooting section] before opening an issue.
[x] I have tried upgrading iles and vite.

Description 📖
So far I've been able to reproduce this with both Vuetify (VLayout) and Inkline (ILayout). This is with the appropriate resolvers (Vuetify3Resolver, InklineResolver) from unplugin-vue-components/resolvers.
// iles.config.ts
import { defineConfig } from 'iles'
import { Vuetify3Resolver } from 'unplugin-vue-components/resolvers'

export default defineConfig({
	vite: {
		ssr: { noExternal: ['vuetify'] }
	},
	components: {
		dirs: ['src/components', 'node_modules/vuetify'],
		resolvers: [Vuetify3Resolver()]
	}
})

 // components.d.ts
declare module '@vue/runtime-core' {
  export interface GlobalComponents {
    // ...
    VLayout: typeof import('./src/layouts/v.vue')['default'] // expected: typeof import('vuetify/components')['VLayout']
    // ...
    VList: typeof import('vuetify/components')['VList']
    VListItem: typeof import('vuetify/components')['VListItem']
    VNavigationDrawer: typeof import('vuetify/components')['VNavigationDrawer']
  }
}

Reproduction 🐞
Available here. This reproduces it with Vuetify, but the same pattern is apparent with Inkline's ILayout, where it points to src/layouts/i.vue instead of src/layouts/v.vue.

Dependencies Info
Run npx iles info and pnpm list (or npm list) and provide the output:
npx iles info: iles v0.9.2 vite v4.1.4
npm list:
<EMAIL_ADDRESS>/home/rsek/Development/GitHub/iles-layout-bug-repro
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>

Thanks for reporting @rsek, fixed in 0.9.4 .

Great, thanks for the fix! :) :rocket:


Design Form Select

goal: Design the Form Select to match the new inputs at #737
Current v1.5.x

Tasks

[ ] Define UX elements and behavior
[ ] Visual Design
[ ] Document results in the wiki

See https://select2.github.io/examples.html

@xak
With this one we only are designing the normal Dropdown.
From the page you sent, the first example. The others examples on that page are more complex components with search, etc.
Once we finished this one we can create a variant or a new component with those features.

@joseegm Please note that the example is not a normal dropdown. It's a custom Select2 implementation. The styling will require pulling in vendor CSS so it may be more effective to handle all dropdown cases as once.
There is no way to style the dropdown part in CSS so is the outcome here to style only the select element with up/down arrows? Does this also apply to ul list menus used for navigation? "Dropdown" is a little bit misleading.
Be clear in the final documentation if all select elements get transformed to Select2 boxes or can a project use default selects without the styling for the options. There are JS implications here that need to be considered by development esp. whether the custom styling is global or opt-in.
Thanks.

The decision of using or not Select2 is a development decision. We had massive problems maintaining Select2 in v1.5.x so I'll like to avoid it.
There is no new design yet.

What you mean with "up/down" arrows?
What other ul list for navigation you are referring?
What you mean by "custom styling is global or opt-in"?

Changing Dropdown to Form Selectto be more clear
This is referring to the default select via css; and custom HTML markup component if more features are needed. We are not going to depend on Select2 for style.

Tagging @saadmhybris for reference.
I agree about Select2, but it will not be possible to support the various select controls in use now without Select2, Selectize.js or other solution. CSS alone simply cannot do it.
The best CSS can do is this ...

The options have limited styling capabilities (that vary across browsers) so you get the good ol' system dropdown ...

So for the default select, we can only style the select element to look like an input with the addition of the "open/close" indicator on the right (what I meant by up/down arrows). The form-group lock up with a label, help and errors should be the same as inputs.
To your other questions ...
What you mean by "custom styling is global or opt-in"?
Should end implementors have the choice of using the browser default select (see image 1 above) or will ALL select elements get transformed into the Select2 look. I assume the latter will be the case, where the init is $('select').select2();. I'd like to understand the exact pain points around Select2. Has global transformation been a problem or is it more about integration challenges?
What other ul list for navigation you are referring?
A navigation ul menu is used for links and is not part of a form. Select2 transforms the form element into a list and binds to the original form element. This isn't incredibly important right now but it's good to know when building so we can test against both elements just like a tn-button needs to be tested against button and a elements. The styling could be the exact same but one is a select and the other a ul.
Hope that helps. If this is only about the default select that's cool, I just wanted to point out that there is a lot more going on behind this one element.

I'm aware of the CSS styling limitations. That will be take in count when doing the design.
The details of the implementation that consumers of the library are doing is out of our scope, we only provide the corresponding CSS styles and HTML markup.
You can ping @saadmhybris about the problems we had with Select2 in the past.
The ul lists used for navigation have nothing to do with the select.
We will make the styles for the default component. If we find other use cases that need a different design then we will see how to implement them.
Thanks for your input.

The problem we had with select2 in the past is when doing one off customizations. Both select2 and angular directive that builds that component was owned and managed by another team so the UI devs didn't have access to it. Also, select2 attaches to the body tag (at least the version at that point)  so it's not possible to customize individual instances. This was a big pain point as it was used all over the modules and there were UX asks to make modification on a one off basis.

@saadmhybris Do we have a good understanding of the one-off customizations? That would be good for UX to know.

The one-off customization had to do with just some basic style overrides. Like with our expandable button pattern, where builder teams were using select2 component instead of a dropdown list, we couldn't easily customize select2 dropdown to match the expandable button pattern.

The case @saadmhybris mentioned is a valid one but it is out of the scope of this issue.
This issue is not concerned with the Select2 use. It is for the simple form select. Other custom cases will be defined, if needed.

@xak

@LeoT7508 This looks good but one thing ... would it be a big deal to make the indicator — V — a 90° angle or use a Unicode character? That way I don't have to use an icon.
The checkbox worked well since the make uses a 90° angle meaning I could make a box with borders on two sides and rotate it to look like a checkmark. https://github.com/SAP/techne/pull/876/files#diff-9b2d847fc5562e255de1ce5050bf5c6bR113

It would be great to do that here as well if it works for you.

@xak
Depends on the line weight. Do you have an example?

We could use 2px weight ... here's what the checkbox looks like at 10x10 with 2px border width.

Minor tweak.

Looks like 1px will be closer to what we have in the original. The check will look more like a check though correct? Not like a chevron?

The checkbox will not change. I just used that as an example to show what the rotated chevron would look like.

@xak
Lets give it a shot and see how it looks.


Remove the default engine behavior in internal applicative eval functions

Context
Following a change https://github.com/nubank/nodely/pull/38 in how dependencies are provided, services importing Nodely (indirectly through nodely-helpers) began facing errors. This occurred because services were not explicitly providing promesa as a dependency, even if they were not using it.
The issue stemmed from Nodely's applicative engine eval functions, which would default to using the promesa engine if no engine was specified. This resulted in promesa being included as a required dependency.
Since applicative eval functions are intended for internal use only and are accessed through the API, which sets the core-async engine when no engine is provided, the decision was made to remove the behavior of having a default engine at applicative eval functions.

Going from providing a 2-arity and a 3-arity to only providing a 3-arity signature is a breaking change: clients who called the 2-arity version get failed when they were supported in the past.
Could we make an applicative engine that didn't rely on promesa lib (or any other of our optional libs) and continued to support the 2-arity version?

I converted the PR to draft, so we can still work on that


[ENHANCEMENT] Changing calico.vethuMTU

Is your enhancement related to a problem? Please describe.
I wanted to use Jumbo frames within Harvester and wanted to change MTU of Calico but noticed that it can be done only by changing  charts.
Describe the solution you'd like
There should be possibility to change calico.vethuMTU.
It could be set in 90-harvester-server.yaml but this is overwritten in boot.
Describe alternatives you've considered
I changed value directly to the deployed chart, but it will be lost during upgrade.
Additional context

Describe the solution you'd like
There should be possibility to change calico.vethuMTU. It could be set in 90-harvester-server.yaml but this is overwritten in boot.
Describe alternatives you've considered
I changed value directly to the deployed chart, but it will be lost during upgrade.

Hi @tahvane1,
Can you share the operation you mentioned? Maybe they could apply to Cloud-Native Node Configuration .

@tahvane1 you mean you can configure the veth_mtu value in 90-harvester-server.yaml ? I didn't see any related field. I only found the MTU value in /etc/cni/net.d/00-multus.conf as a default value.
Do you want to change the default MTU value all calico or you just need to specify it for particular resource?

I changed default calico  MTU via changing RKE2 yaml files (by setting calico.vethMTU in RK2-canal/calico/multus yaml file)

Thanks. The keyveth_mtu refers to the CNI_MTU environment variable from configMap (rke2-canal-config).
kubectl edit daemonset rke2-canal -n kube-system can see but don't know the CNI_MTU value
Then
kubectl describe configmap rke2-canal-config -n kube-system will have the veth_mtu value 1450
But the struct cni_network_config which renders the content of  10-canal.conflist, still needs to refer the CNI_MTU, I may need to check how to modify it.
kubectl edit configmap rke2-canal-config -n kube-system and edit the veth_mtu value from 1450 a specified value. The command can work across reboots just as what you did.
So you would like the change can persist for each reboot and the version upgrade, right?

That would be great

After evaluation, lots of complication will be introduced for version upgrade.
@tahvane1 Please refer to https://docs.harvesterhci.io/v1.3/troubleshooting/harvester/#access-embedded-rancher-and-longhorn-dashboards which can do MTU size configuration, and then reboot or restart the rke2-canal daemonset should make it work.
Please verify if it meets your request. Thanks


[Network] fix wrong deprecating info of flow-log show

[Network] network watcher flow-log show: fix wrong deprecating info.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

add to S167


onClick hit test area is shifted in an AR scene half the time, half the time it's fine

[x] Review the documentation: https://docs.viromedia.com/
[x] Search for existing issues: https://github.com/viromedia/viro
[x] Use the latest ViroReact release: https://docs.viromedia.com/docs/releases

Environment
Please provide the following information about your environment:

Development OS: Mac, Windows, Linux
Mac

Device OS & Version: What Android OS version (7.0, 8.0, etc.) or iOS version (11.0, 11.2) is your device running on?
iOS 13.6

Version: ViroReact version and React Native version
rn: 0.63.2, viro-react: 2.17

Device(s): What device(s) are you are seeing the issue on (i.e. iPhone X, Samsung Note 8, etc)
iPhone 7, Android moto x, iPhone 11

Description
The onClick event sometimes doesn't fire on an ImageMarker or any content contained in an image marker. But, clicking below the image marker and it's content or most often, down and right fires the event. I've tried any number of nesting objects and moving the click handler around, but can't get onclicks to work all of the time. It seems to always work if I switch the event to hovering or dragging.
My image markers are remote image URLs. Not sure if that is part of the issue. I prefetch them in react, but not sure if that caching carries over.

I feel like it may have to do with PixelRatio not initializing correctly because of a race condition or something. I tried using performARHitTestWithPoint but there's no way to tell if the hit test touched an image marker unless I'm missing something.

Hi @Vednus, did you manage to find any workaround ? I'm encountering the same strange behavior.

Hey @teabow yeah, I figured out that it was a bug in the Viro/virocore package. I made a fix, but didn't put up a pr because I think my fix only works if you're using the entire screen for the AR and not a part of it. You can see the relevant changes in the VROViewAR.mm file here.
https://github.com/Vednus/virocore/compare/master...sl/fix/ios-tap-fix
But, you'll need to setup the viro repo and virocore repo in the same directory and go through the build steps for virocore that are outlined in their README to get it to work.

Thanks a lot @Vednus i'll try that !
Did your full screen AR scene was part of a navigation stack ? I'm using react-navigation and I'm wondering if I will have to change the way of navigating to the scene.

@teabow mine is visually in a react-navigation stack, but because of all the workarounds I was trying and some other funky stuff, it is a component in a tab bar. But, I load a black page in the navigation stack and then show the ARView, so the user thinks it's embedded in the stack.

Hey @teabow yeah, I figured out that it was a bug in the Viro/virocore package. I made a fix, but didn't put up a pr because I think my fix only works if you're using the entire screen for the AR and not a part of it. You can see the relevant changes in the VROViewAR.mm file here.
<EMAIL_ADDRESS>But, you'll need to setup the viro repo and virocore repo in the same directory and go through the build steps for virocore that are outlined in their README to get it to work.

Thanks, it work for me.
I rebuild this and upload in https://github.com/developBeetSoftD1/react-viro. Hope to be of help to someone here


Feature/1930 fixed bug btn create post when responsive

What does this pull request aim to achieve? 💯
Please be concise and link any related issue(s):

[x] I have checked that no one else is working on similar changes.
[x] I have read the latest README and understand the development workflow.
[x] The name of this branch is: feature/<branch_name>, or hotfix/<branch_name> if this is a hotfix, where <branch_name> briefly describes the issue(s) resolved and is prefixed with the issue number(s) (e.g. 1127-1130-update-git-branching-model). The branch is created in a cloned version of this repo, not a forked repo.
[x] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[x] There are no merge conflicts.
[x] There are no console warnings and errors.
[x] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".
[x] The title describes the issue(s) being addressed using format: <issue_numbers> - <brief_issue_description> (e.g. #1127/#1130 - Update Git Branching Model)

Duplicate #1946

let's focus on this one for further changes, sorry @rabbi08 about this back and forth

@juandelima
The 'Create a Post' is not displayed correctly on 'Profile page of ORG'
Please refer below screenshot.

duplicated of #1985 , close


Code not working even when typed correctly!

So I tried to do this, and the feature that you have added might have broken!

I have made some changes to this while you were away...
Try git pull and see if that works!

No still not working!

I see the problem now, I am using deprecated unstable conditions!

I will fix src/WPMProgram.java!

oK!


Add phonetic token filter

Add a phonetic token filter.

Codecov Report

Merging #17 (9d26f2f) into main (a18e39d) will decrease coverage by 5.48%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   93.83%   88.34%   -5.49%
==========================================
  Files          11       16       +5
  Lines        4445     4721     +276
==========================================
  Hits         4171     4171
- Misses        274      550     +276

Impacted Files
Coverage Δ

src/lib.rs
100.00% <ø> (ø)

src/phonetic/beider_morse.rs
0.00% <0.00%> (ø)

src/phonetic/daitch_mokotoff.rs
0.00% <0.00%> (ø)

src/phonetic/double_metaphone.rs
0.00% <0.00%> (ø)

src/phonetic/generic.rs
0.00% <0.00%> (ø)

src/phonetic/mod.rs
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.


Error: 002043 (02000): SQL compilation error: Object does not exist, or operation cannot be performed

Provider Version
0.22.0/0.25.36
Terraform Version
1.1.7
Describe the bug
Hi there, I had the following code running fine.
terraform {
  required_providers {
    snowflake = {
      source  = "chanzuckerberg/snowflake"
      version = "0.22.0"
    }
  }
}

provider "snowflake" {
  alias = "sys_admin"
  role  = "SYSADMIN"
  username = "tf-snow"
  private_key_path = "..."
  region = "ap-southeast-2"
  account = "..."

}

resource "snowflake_warehouse" "star_warehouse" {
  provider       = snowflake.sys_admin
  name           = "STAR_WAREHOUSE"
  warehouse_size = "XSmall"

  auto_suspend = 60
}

resource "snowflake_database" "STAR_DB" {
  provider = snowflake.sys_admin
  name     = "STAR_DB"
}

resource "snowflake_schema" "public" {
  provider            = snowflake.sys_admin
  database            = "STAR_DB"
  name                = "public"
  data_retention_days = 1
}

resource "snowflake_table" "sales" {
  provider            = snowflake.sys_admin
  database            = "STAR_DB"
  schema              = "public"
  name                = "sales"

  column {
    name     = "id"
    type     = "int"
  }
}

Then I decided to rename "STAR_DB" to the lower case version "star_db".
It then started giving me these errors.
Error: 002043 (02000): SQL compilation error: Object does not exist, or operation cannot be performed.
with snowflake_schema.public
on snowflake.tf line 33, in resource "snowflake_schema" "public":

Error: 002043 (02000): SQL compilation error: Object does not exist, or operation cannot be performed.
with snowflake_table.sales
on snowflake.tf line 40, in resource "snowflake_table" "sales":

I tried reverting the code, removing the code, removing the underlying assets, rebuilding the underlying assets manually, I tried deleting the file and I even tried running a terraform destroy!
Additional context
I have a bunch of AWS assets in the state as well and now I will need to remove them all manually, this is a big deal breaker any production environment.
Please let me know if you have any advise.
I was initially using 0.22.0, but it also failed for 0.25.36
Note: I am using terraform cloud.

Hi @hugh-nguyen lowercasing names in Terraform does generally not work nicely, as Snowflake's naming convention is uppercase for some resources.
When writing SQL in the Snowflake GUI this is not an issue, but it is an issue when writing Terraform code.
The SQL executed by the provider tries to execute the query with a plain lowercase name and this, in fact, does not resolve against Snowflake's internal resource mapping.
You need to keep some names uppercase explicitly or at least pass them with the TF upper() function.

Hi the @thenaturalist, that's good to know. So I've been playing around and I found that adding
depends_on = [snowflake_schema.public] to the snowflake_table removes the table level error!
But placing
depends_on = [snowflake_database.STAR_DB] on the snowflake_schema did not fix the issue.
I also tried,
depends_on = [snowflake_warehouse.star_warehouse, snowflake_database.STAR_DB]
depends_on = [snowflake_warehouse.star_warehouse]
and also put
depends_on = [snowflake_warehouse.star_warehouse] on the database resource
but no luck.
Any advice?

Hi @hugh-nguyen, it's not possible to exactly tell what is going on without seeing your exact code, but here is my guess.
You mention Then I decided to rename "STAR_DB" to the lower case version "star_db". but without specifying whether you renamed the TF resource name or the Snowflake internal name.
For example:
This should work & is conventional TF resource naming:
resource "snowflake_database" "star_db" {
  provider = snowflake.sys_admin
  name     = "STAR_DB"
}

This should work but isn't conventional TF resource naming:
resource "snowflake_database" "STAR_DB" {
  provider = snowflake.sys_admin
  name     = "STAR_DB"
}

This should not work due to Snowflake's internal resource naming:
resource "snowflake_database" "star_db" {
OR
resource "snowflake_database" "STAR_DB" {
  provider = snowflake.sys_admin
  name     = "star_db"
}

So when you add depends_on = [snowflake_database.STAR_DB] but the Snowflake name in snowflake_database.STAR_DB is still lowercase, the same problem persists.
Generally, I would advise you to minimize using strings and assign values with resource data tags. If you do that, TF can automatically infer resource dependencies and depends_on is obsolete. For example:
resource "snowflake_database" "star_db" {
  provider = snowflake.sys_admin
  name     = "STAR_DB"
}

resource "snowflake_schema" "public" {
  provider            = snowflake.sys_admin
  database          = snowflake_database.star_db.name
  name                = "public"
}

resource "snowflake_table" "sales" {
  provider            = snowflake.sys_admin
  database          = snowflake_database.star_db.name
  schema            = snowflake_schema.public.name
  name                = "sales"

I changed it to use resource data tags, and can confirm that
select * from star_db."public"."sales";
works
But I'am still getting the error.

I find it interesting that when defining the database resource, you do not need to specify the data warehouse, I wonder how it know which data warehouse to target.

I ended up removing the resources from the state file and it works now.

not sure how it "sits" in the terraform mindset, but in Snowflake, you really should avoid wrapping object names in double quotes unless by default you want spaces or case sensitive handling of your object names.
Without double quotes all strings are cast to upper case. so bob, Bob , and BOB are the same as each other, but "BOB" is not the same as "bob". There is a session parameter to ignore the case of quoted strings QUOTED_IDENTIFIERS_IGNORE_CASE.
But if tools/users can avoid double quotes, then things work safer (until you put spaces and such, or try use object names as presentation tokens).

I noticed this as well. It is easy to accidentally get a broken terraform state which is a lot of work to clean up if lowercase names of objects are used. This can easily happen if you are concatenating other things onto the names in order to build them out of other strings. If lowercase names are not supported, then the provider should include a regex validator for all object names to assure that they conform with the expected format so that the user finds out earlier. However, I don't see a technical reason why the provider should be using case sensitive matching.

We are closing this issue as part of a cleanup described in announcement. If you believe that the issue is still valid in v0.89.0, please open a new ticket.


Avatar begins walking to the northwest automatically--cannot control character

What happened?
I've seen this on numerous games using unity when two monitors are present. For some reason, the controls get pegged and forced into one direction and you are not able to reset or center the character--they always keep walking in the same direction.
Steps to reproduce
Need dual monitor setup to engage: load game and the character will begin walking automatically.
https://user-images.githubusercontent.com/24481500/165451694-5fdd8208-9002-4783-bfb6-a74cb9f9e72a.mp4
Unplugging one monitor halts the problem usually.
Operating System
Windows 10  (10.0.22593) 64bit
Wallet
Metamask
CPU
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz
GPU
NVIDIA GeForce RTX 3070 8043MB
RAM
48972MB
Kernel Version
1.0.0-2227136955.commit-c4944ec
Unity Version
desktop-main.commit-7b08a86
Realm
baldr
NameTag
MrFrostyCakes#5209

@careychico since you opened this issue many changes have been applied, could you please verify if this is still happening?


Fix zendesk pipeline object parsing as supported by zenpy

@Redi
Please resolve the fix as described in the feedback from Marcin
https://github.com/dlt-hub/pipelines/pull/74

this is done but needs further improvement as described in #148


How we evaluate transaction if it fraud or not we need the labels of the patterns

Sorry for the late response.
I added descriptions of how to find out SAR accounts and transactions in this Wiki page


[stable/prometheus-operator] Fix several issues

What this PR does / why we need it:

Adds annotations for admission hooks management jobs, which allows disabling istio auto-injection
Adds node selector for admission hooks management jobs, which allows running them in mixed node clusters
Fixes rules not being provisioned on GKE due to it having a pre-release semver
Adds documentation for requirements in private GKE clusters

Which issue this PR fixes

fixes #16237
fixes #16147
fixes #16263

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test
/lgtm


do broadway listeners have priority?

why and why not?

What is your use case?

i was just curious since symfony event listeners do have priority if that thing works in broadway too or if there is any thing like that similar or any usage.

I don't really know that there's a need for one without specific use cases. The listeners are projectors and processors so I don't know that a priority is really of any use

yeah I guess we don't have a use case really. Closing then, big thanks :+1: :older_man: you are the saga :8ball:

In my use case I need to send an email after an event is dispatched. In the email I need some information created after a projection handle by the same event. In my case have a priority could be useful

@francescotrucchia That is in the nature of eventual consistency. There is no promise which projector or processor get's the event first. There are quit some good resources about this: https://www.google.nl/search?q=event+sourcing+eventual+consistency

@francescotrucchia In addition to the eventual consistency topic you may need a process manager aka Saga when your email depends on information created by a projection

@wjzijderveld @codeliner thanks for suggestions :+1:

@wjzijderveld I understand the problem of eventual consistency, but in an environment request/response where the process are synchronous I think that the ventual consistency doesn't exists and in this case could be more pragmatic using the priority on listener.

@francescotrucchia I can follow your argument, but eventual consistency depends on the transaction boundary not on PHP's request/response environment.