Client Core API

The new Hot Chocolate client API is based on the work GitHub has done with its OctoClient. We will start by refactoring the client core to use our parser and remove dependencies on GitHub itself.
resolves #775

@PascalSenn @willwolfram18 just added you two to keep you in the loop on this one so you can add your ideas as we move this forward.

I have added the core so far and will start removing GitHub specific code .....

Something that would be good to add is the ability to provide HTTP headers for a request that is made. This could be done with a Headers (or RequestHeaders) property on the IQueryRequest or as its own parameter to an overloaded Run method. I know it's not an uncommon practice to include metadata in headers for an HTTP request (request Ids, access tokens, etc.), and while it would be possible to use the DefaultRequestHeaders property on the HttpClient for some items, some requests may require distinct values for each individual request such as a request Id. This would mean the DefaultRequestHeaders wouldn't work in that scenario.
The abstract HttpHeaders class implements IEnumerable<KeyValuePair<string, IEnumerable<string>>> so this would likely be the best type to expose.

Speaking of headers, not just being able to include them in the request but being able to access them from the request might be nice. Not sure how well that fits into this though, especially since GraphQL itself doesn't describe a specific protocol.

@willwolfram18 I think if we use IHttpClientFactory you are able to provide handler that can provide headers....
regarding reading response headers, this might be more difficult. Since we are mapping to an object model. The question is how do we store them or map them to that model. I mean with a handler you can also grab these, but how do we bring them together with the data?

@michaelstaib  Would it make sense to have a response object?  Like the way Apollo does it.
{
data: The recevied data
reponse: HttpResponse
... otherProps
}

Let me have a look at how relay does it.

OK, just talked to our guys.... relay does not do that.
I think we have to look at this a different way. For what do we want to have access to the raw message? What do we actually want to do with the response headers. Are those part of the content that we want to consume?
If so we should extend the local schema and do some schema stitching on the client.

Just to explain why I am thinking like that ....
I think the resulting graph should be very clean and very close to the structure that I want to consume. There should be no need to pass on the message.
I mean there are ways to intercept the message baked in the http client and we can also allow interception in the connection layer but passing it on feels not good. I think it is very important that the new API feels very clean.

I was just thinking this would have lower potential for breaking changes.
Once you need some kind of context around the data you will not be able to add it to the response object

@PascalSenn but I also would bind it tightly to HTTP and this would kind of render the connection abstraction useless. I can follow your argument but I just think this would expose to much implementation details.

i guess handling this in a HttpFactory would be the better way (when using http as transport protocol at all)

That is what I think... having HttpFactory allows us to handle and http relevant stuff with handlers... the one thing we additionally need is a way to map data like response headers to the object model.....
I favor something like the relay local schema extension... what if we had also local schema extensions like we allow to extend the schema:
extend type Foo {
   someLocalData: String
}

and then we allow users to write resolvers on top of these extensions. This would lead to a consistent object model on which we can do queries. The fact that some data is computed on the client side is oblivious to the user. It would be clean.

OK, lets talk more about the API. After looking a lot at the GitHub API I think we can move faster by starting from scratch but preserve their idea.
So, what do you think about an approach like this.
IQuery<MyFoo> query =  Schema.Query.Select(q =>
    new MyFoo  {
        q.Foo,
        q.Bar.Select(b => {
              b.Baz
        }
    }).Compile();

var myResult = query.ExecuteAsync(connection);

If you guys have other ideas pleas post them.

This is very close to what the oktoclient is.

IQuery<MyFoo> query =  Schema.Query.Select(q =>
    new MyFoo  {
        q.Foo,
        q.Bar.Select(b => new MyBar {
              b.Baz
        }
        q.Qux.OneOf(t =>
        {
             t.On<MyTypeA>....
             t.On<MyTypeB>....

        })
    }).Compile();

var myResult = query.ExecuteAsync(connection);

https://github.com/ChilliCream/octokit.graphql.net

We will close this one In favor of a different approach.

For the new Client API approach look here #1011


Stimulants are overtuned

Description
Stimulants remain the most useful purchase in a nukies arsenal, and arguably even for a syndicate agent. Not even mentioning you can currently dupe them infinitely due to how smoke and foam mechanics are currently. The ability to move faster than any crew member, with instant stun recovery, and critical health recovery is just too insane. Five minutes is enough time to kill a third of the crew by yourself, not mentioning you can just buy more if you don't wanna dupe em.
I've frequently seen stimulants used to great effect with any combo of weapon. The newest trick is to use it with the double bladed energy sword, meaning you are effectively immune to disabler shots meaning you must be stun batoned before being shot. This would work just fine with the normal esword or even best with the fists of the northstar since you don't drop them when stunned.
I'm not sure if they should just have a price increase, a nerf, or a combo of the two. I'd argue they should either lose the speed buff, or lose the stun recovery effect. Nerfing the speed would probably be the better of the two options. I believe the experimental stim should be what you have to purchase if you want a combo of speed and stun recovery since the cost of that one is death.

Think removing the stim purchases or giving all stims a damage debuff like the experimental stim and nerfing its speed boost is prob the way to do it. Right now, stims are just super meta as a nukie and the desword made it way worse to fight against (oops).

I'd need to see the interaction with stimulates and stunning/disablers/etc and then also with dswords. FWIW dswords also take up two hands so the nukie will have to get into melee range to attack, which means you're highly susceptible to being stunbatonned.

It doesn't really matter that much if you stun baton someone on stimulants. You can't cuff them, so unless you have perfect reactions by stunning them, turning off harm mode, either spam clicking where the sword is or right clicking and finding the sword in a menu. A very skilled and lucky officer might get a few bullets in while they're flip flopping off the floor but that is really it. They run so much faster than a hardsuited security officer they shouldn't even be getting stunned in the first place if they're trying to kite. (Of course this is all in a 1v1 scenario which doesn't happen often, especially during nuke ops)

They were nerfed.


BigQuery: Jobs with null parameters throw InvalidArgumentException

Environment details

OS: all
PHP version: all supported
Package name and version: google/cloud-bigquery 1.22

Steps to reproduce
Try creating a job with a parameter that contains a null value throws an InvalidArgumentException:
Unrecognized value type NULL. Please ensure you are using the latest version of google/cloud.

Code example
$client
->query(
    'SELECT * FROM `bq.transactions` WHERE `device_id` = ? AND `QTY` IS ?;'
)
->parameters([
    123,
    null,
])
;

Executing the query directly inside the SQL Workspace works fine:
SELECT * FROM `bq.transactions` WHERE `device_id` = 123 AND `QTY` IS null;

@dwsupplee Thanks for adding the labels, but this is not really a question but a reported shortcoming of the google/cloud-bigquery package, which makes it more of a feature request or a bug (depending on how you look at it).

@bramus fair point!

BQ Team said we can pass string as type for the NULL value. Will write some tests to understand this and make a PR soon.

This issue shows like IS is an operator and NULL is an operand, which is not the case. IS NULL and IS NOT NULL are the operators and they can not be divided like this in the bug description.
Additionally, = NULL under WHERE clause is not valid. So, I would suggest to please use the IS NULL operator directly in the query string.
Further, the valid cases for NULL to be passed are Insert and Update statements. I don't see the library allows parameters like we do in the query for those operations.
Therefore, I think we are good to close this issue.


Add Colab about how to convert TFLite compatible seq2seq model

Describe the feature and the current behavior/state.
Relevant information

Are you willing to contribute it (yes/no): yes
Are you willing to maintain it going forward? (yes/no): yes
Is there a relevant academic paper? (if so, where): no
Is there already an implementation in another framework? (if so, where): don't know
Was it part of tf.contrib? (if so, where): no

Which API type would this fall under (layer, metric, optimizer, etc.)
seq2seq
Who will benefit with this feature?
Everyone who wants to use RNN models for NLP, TTS etc
Any other info.
This issue will have link to the colab later (WIP)

this is really helpful, i'm working on Seq2Seq based-on tf-addon and i want tf_addons has official tutorial about convert TFLite for seq2seq module. Pls take a look @guillaumekln @seanpmorgan

https://github.com/tensorflow/addons/pull/1964

TensorFlow Addons is transitioning to a minimal maintenance and release mode. New features will not be added to this repository. For more information, please see our public messaging on this decision:
TensorFlow Addons Wind Down
Please consider sending feature requests / contributions to other repositories in the TF community with a similar charters to TFA:
Keras
Keras-CV
Keras-NLP


Stability of mirror system for Dart VM

Stability consideration for mirror system since Dart 1 mirror system is considered as unstable. There was been another major bump of Dart as Dart 2 was released, were in mirror system is also considered as unstable, before stable release it's been a bit unclear will mirroring be introduced also in Dart 2 or not, now we have it. But can you, please, clarify the state of the mirror system for Dart 2 and what future for it is planning? Mainly I speak here about Dart VM.

You mention that the mirror system was considered unstable in Dart 1, can you elaborate on that with specifics of what is unstable.
In Dart 2 we intended to maintain the same support that we had for dart:mirrors, there could be some issues that we overlooked. Could you elaborate on what you have found.

Mirrors support was marked experimental for dart2js as far as I remember and
removed entirely in Dart 2 dart2js and Flutter.
Only the server VM supports mirrors in Dart 2.

It might be relevant to know the following: In a setting where 'dart:mirrors' is not supported you can do most of what 'dart:mirrors' will do using code generation with package reflectable.

@a-siva that was right in the documention of 'dart:mirrors' package, here is fragment from that comment:

At this point we have no plans for enhancing dart:mirrors library.
Please see https://github.com/dart-lang/sdk/issues/44489 which is investigating a path towards discontinuing support for dart:mirrors


Missing Results when Search Phrase Includes a Decimal Number and UsingSynonymsImproved() is Used

Describe the bug
With a search phrase such as "1.5 ton cased" when using UsingSynonymsImproved(), our search results do not show all expected results. We have narrowed this down to the use of both UsingSynonymsImproved() and a search phrase that includes a decimal number like "1.5" and is at least 3 words.
To Reproduce
Our search code:
ProductSearchService.For(term)
    .InField(x => x.WebDescription)
    .InField(x => x.GetDisplayName()) //this is the product name
    //... additional .InField's here
    .InField(x => x.GetAggregatedBulletsForSearch())
    .WithAndAsDefaultOperator()
    .UsingSynonymsImproved()
    .ApplyBestBets()
    .WildcardSearch(term,
        (x => x.Code, boostCode),
        (x => x.CrossItems, boostCrossItems),
        (x => x.GetDisplayName(), boostName));

Actual vs Expected Behavior
We expect products with the following names to be returned, however, only #1 is returned:

AIRQUEST 1.5 TON CASED A COIL - EAM4X18L14A
GOODMAN 1.5 TON/2.0 TON UPFLOW/DOWNFLOW INDOOR CASED COIL
GOODMAN 1.5 TON TO 2.0 TON HORIZONTAL "A" FLOWRATOR R22/R410A INDOOR CASED EVAPORATOR COIL

If we remove or comment out UsingSynonymsImproved(), or just search for "1.5 ton" (i.e. a 2 word search phrase) all three results are returned as expected.
Additionally, I tried adding the exact search phrase, "1.5 ton cased", to each item in a searched field. Again, only #1 was returned while also using UsingSynonymsImproved().
Additional information

Find and CMS version:

Find: 13.4.7
CMS: 11.20.9
Find Toolbox: 1.1.1

List of synonyms:

Only one synonym - Unidirectional: "refill, replaces" > "replacement"

Additional context
As expected, we see this across multiple environments. It's easily reproducible by adding and removing the use of UsingSynonymsImproved().
We do use more synonyms in our production environment which is why we use UsingSynonymsImproved(), but this is reproducible as noted when we just have one or even zero synonyms.
When doing other searches that involve numbers, like "2 way switch", results seem to be as expected. The decimal format of the number seems to be a common denominator as well as searching for 3 words in the title.
I have looked into the index for the items noted above and nothing looks wrong, for example the product name for #2 is correctly indexed as:

"GetDisplayName$$string": "GOODMAN 1.5 TON/2.0 TON UPFLOW/DOWNFLOW INDOOR CASED COIL"

Finally, thank you for your work with this project. Despite this issue, the use of this code notably improves our search results.

Thanks Jason. Will look into this.

Fix for searching with , and . in decimal numbers will be pushed next week and along with custom search providers for the CMS UI that make use of Toolbox - improving the overall search experience in the UI as well. Thanks for your feedback and kind words.

Please try out the new 1.3.0 preview release

I've added the 1.2.1 prerelease version to my solution but am getting the following exception when searching:
System.TypeLoadException
HResult=0x80131522
Message=Could not load type 'EPiServer.Find.Cms.MultiFieldQueryStringQueryExtensions' from assembly 'EPiServer.Labs.Find.Toolbox, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
Source=ChadwellSupplyEpiserver.Find
StackTrace:
at ChadwellSupplyEpiserver.Find.Services.FindProductService.GetProductTerm(String term) in C:\stuff\code\chadwell\EpiServer Commerce Web\WebsiteSolution\ChadwellSupplyEpiserver.Find\Services\FindProductService.cs:line 204
at ChadwellSupplyEpiserver.Find.Services.FindProductService.GetProductResults(SearchRequestModel filter, List1 activeWarehouseCodes, List1 validStates, Boolean isLTLShipMethod, UserAccount userAccount) in C:\stuff\code\chadwell\EpiServer Commerce Web\WebsiteSolution\ChadwellSupplyEpiserver.Find\Services\FindProductService.cs:line 367
Any idea on how to resolve?


gitlab-ci.yml not working

To host hugo on gitlab the gitlab-ci.yml needs to be modified. The settings on
https://gohugo.io/tutorials/hosting-on-gitlab/
are no longer correct. Please correct them to the lattest version which is from the gitlab website:
https://gitlab.com/pages/hugo
'''
image: alpine:3.4
before_script:

apk update && apk add openssl
wget https://github.com/spf13/hugo/releases/download/v0.18.1/hugo_0.18.1_Linux-64bit.tar.gz
echo "e8ba723c508e87912970f132534c64dc  hugo_0.18.1_Linux-64bit.tar.gz" | md5sum -c
tar xf hugo_0.18.1_Linux-64bit.tar.gz && cp ./hugo_0.18.1_linux_amd64/hugo_0.18.1_linux_amd64 /usr/bin
mv /usr/bin/hugo_0.18.1_linux_amd64 /usr/bin/hugo
hugo version

test:
script:

hugo
except:
master

pages:
script:

hugo
artifacts:
paths:

public
only:

master
'''

nvm, i was wrong


Changelog in pdf build lacks description

The changelog in html looks great, but when building a pdf, the changelog only has the tag and title, but lacks the descriptive text of the changelog.

I have to admit that I never looked at anything else than the pdf version. Would that be something you would consider contributing ?

When you say PDF, you mean html? Being able to contribute to a repository requires quite some corporate effort, which is likely not reasonable.
I did have a look at the code, but nothing obvious seemed there which could cause the observed behavior.

When you say PDF, you mean html?

yes sorry.

Being able to contribute to a repository requires quite some corporate effort

Well either that or contributing on your own time, which might be even more unreasonable, thus this. Just wanted to mention that, while I admit I have no idea what the proportion is, a large proportion of overall open-source work (and about 100% of this repo) is volunteer work, with none of the development time being compensated. This repo is me thinking that the project should exist and making it for no other goal than being satisfied that it exists (and able to use it in other similar projects). While I acknowledge that there are a lot of issues in the fact so much open-source work is unpaid work on top of which companies can build profit, I find your wording odd (though I'm not taking any offense or anything) that you tie open-source solely to corporate effort. And again, you're totally entitled to have a personal life free of any open-source volunteer commitments, the only thing I'm noticing is your wording describing this as a fact where open-source is, as far as I can tell, one of humanity's most impressive collaborative achievement that is empowering everyone and (somewhat) not solely led by companies or "traditional" politics.

I highly value your contribution with this package. I do not expect you to fix any issues I am posting. But reporting an issue I am observing is contributing to your project in the way that is possible for me. It would be appreciated if you can state if this is something that you are intending to fix or not.

It would be appreciated if you can state if this is something that you are intending to fix or not.

Yep, though I have no idea whether this will happen in a foreseeable timeline or not.
It probably helps if you have a reproducer. Not having one doesn't mean I won't work on it either. I mainly don't know.

Ah here's a reproducer: https://docs.openhwgroup.org/_/downloads/openhw-group-core-v-xif/en/latest/pdf/
Perfect.

Ok, the reason why it happens is that the description HTML is passed to the parser as raw, so it's untouched. If the output format is html, it works. If it's any other format it doesn't.
How to fix that: we need to parse either the HTML or the GitHub-flavored Markdown or the html into docutils nodes. I think myst-parser should be able to do it but after an hour searching, I don't understand how.
I'm not sure I'm passionate enough to spend more time on that right now but if anyone has an idea, feel free.
I've asked in the myst-parser discussions.

For the record, I posted here a script to setup a repo that uses sphinx-github-changelog, and here are the additional commands one may use to reproduce this issue in particular:
You'll need a latex distribution supporting tlmgr. Also, it works on MacOS and I believe it works on Linux.
Here are the tlmgr plugins you'll need:
sudo tlmgr install latexmk tex-gyre fncychap wrapfig capt-of framed needspace tabulary varwidth titlesec

Build with
make -C docs SPHINXOPTS=-EW clean latexpdf

And then open docs/build/latex/sphinx-github-changelog-devenv.pdf.
I believe the same fix will fix both issues #120 and #122.

which system are you running this on? I tried to get this running on WSL Ubuntu-22.04, but it seems non-trivial, as the tlmgr version is sort of old.
Is what you are describing a workaround or will this be the long term solution?

Looking into this a bit further, it seems that debian strongly discourages texlive in favor of apt.
While it is probably possible to patch up a working version of texlive under ubuntu and somehow get this running, this might lead to a too high hurdle for people to be able to reproduce a build (and also getting it running on readthedocs), and is actually unnecessary.
On Ubuntu 22.04, all of the mentioned packages are contained in texlive-latex-extra, which i have installed. Still, using your command, the changelog does not build properly.

which system are you running this on?

I ran it on macOS. Yeah, on an ubuntu (or WSL), I guess APT makes sense.

Is what you are describing a workaround or will this be the long term solution?

Neither. It's a way to setup one's computer if we needed to launch the code locally so as to study it in details. Note that it's perfectly reasonable to reimplement the way Docutils nodes are produced, and then test it on ReadTheDocs directly, without installing the latex stack locally.
As I said:

here are the additional commands one may use to reproduce this issue in particular

Sorry for the confusion.

Hi @ewjoachim, thanks for this amazing project. Is there any progress going on with this issue? Or what kind of help is needed to fix it?


Py 3.7 Syntax Error on 0.7

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/me/virtualenv/pymonctl/lib/python3.7/site-packages/pymonctl/__init__.py", line 22, in <module>
    from ._main import (getAllMonitors, getAllMonitorsDict, getMonitorsCount, getPrimary,
  File "/Users/me/virtualenv/pymonctl/lib/python3.7/site-packages/pymonctl/_main.py", line 643
    _updateScreens: Optional[_UpdateScreens] = None
    ^
SyntaxError: annotated name '_updateScreens' can't be global

which seems to have been fixed in 3.8?
https://github.com/python/cpython/commit/de2aea0ff02fa9486365ce9d215bef150fae3a0b
If you are able to repro on your end it might be worth considering removal of the 3.7 classifier

Hi! First off, sorry for my late reply. I've been very busy these last months, so I completely missed your issue. Secondly, thanks a lot for your help!
I will remove 3.7 classifier in next version. Thank you again!


Cannot mutate JSONB column via GraphQL

Describe the bug
When trying to update a JSONB column via GraphQL you get the error: "Invalid input for JSON type". This wasn't an issue as of roughly 2022-10-12
To Reproduce
Steps to reproduce the behavior:

Create table "Event" with the default id and created_at columns and JSONB column "Body"
Add a new record via the Supabase UI, setting the Body field to { "rating": 1 }
Via GraphQL, try to update the field:

mutation {
  updateEventCollection (
    filter: {
      id: {
        eq: 1
      }
    },
    atMost: 1,
    set: {
      Body: {
        rating: 2
      }
    }
  ) {
    affectedCount
  }
}

End result:

{
  "data": null,
  "errors": [
    {
      "message": "Invalid input for JSON type"
    }
  ]
}

Expected behavior
Expect the value of the Body column to update to { "rating": 2 }
Versions:

PostgreSQL: [e.g. 14.1]
pg_graphql commit ref: ea2ab60

Additional context
The most recent record I see in my table that had the value set successfully was in 2022-10-12, so I suspect something (pg_graphql, other library) updated and broke it. If there is a different syntax that I should be using, please let me know. Note that you cannot put { "rating": 2 } in the GraphQL mutation as it will give you a different error.

The JSON  GraphQL type represents a json/jsonb column serialized as a string
For example
Given a setup:
create table event(
    id serial primary key,
    body jsonb
);

insert into public.event(body)
values
    (jsonb_build_object('rating', 2));

This show how the data looks when you select it out (serialized)
select graphql.resolve($$
    {
      eventCollection {
        edges {
          node {
            id
            body
          }
        }
      }
    }
$$);

                                          resolve
--------------------------------------------------------------------------------------------
 {"data": {"eventCollection": {"edges": [{"node": {"id": 1, "body": "{\"rating\": 2}"}}]}}}

And you can similarly update the contents with a json serialized string
select graphql.resolve($$
    mutation {
      updateEventCollection(
        set: {
          body: "{\"rating\": 2}"
        }
        filter: {
          id: {eq: 1}
        }
      ) {
        records {
          id
          body
        }
      }
    }
$$);

                                         resolve
------------------------------------------------------------------------------------------
 {"data": {"updateEventCollection": {"records": [{"id": 1, "body": "{\"rating\": 2}"}]}}}

but raw json will raise the error you experienced
select graphql.resolve($$
    mutation {
      updateEventCollection(
        set: {
          body: {rating: 2}
        }
        filter: {
          id: {eq: 1}
        }
      ) {
        records {
          id
        }
      }
    }
$$);

                                resolve
------------------------------------------------------------------------
 {"data": null, "errors": [{"message": "Invalid input for JSON type"}]}

This change was first present in 0.5.0 which roughly lines up with timeline you mentioned (Oct-22). There were several increases in strictness in that release. When a new version of pg_graphql rolls out to the supabase platform it is pre-installed on the database server to make it "available" but it doesn't take effect until a user runs
drop extension pg_graphql;
create_extension pg_graphql;

or toggles the extension in studio. New projects also default to the latest version. So if any of those are applicable you'd have received the update at that time.
Sorry to hear this caused an issue for you. Now that pg_graphql is out of beta and > 1.0 the stability guarantee is much stronger.

@olirice is there documentation about json being string-serialized + examples on how to query sub-objects (and filter) ?

thanks, it was not, added in #341
its live on docs https://supabase.github.io/pg_graphql/api/#json


IGNITE-18991 Remove zoneId field from RebalanceRaftGroupEventsListener

A small fix corresponding to the note in original PR's IGNITE-18991 Move stable/planned/pending assignments from table to distribution zone root keys discussion

zoneId field is removed from RebalanceRaftGroupEventsListener:

in 3 tablesCounterKey callings zonePartitionId is passed;
in 2 logging lines pass local zoneId var from zonePartitionId.zoneId().

In RebalanceUtils are numerous changes like:

partId to zonePartiotionId changes in args and javadocs;
javadoc of zonePartiotionId args are changed to corresponding meaning;
tablesCounterKey overloading with ZonePartitionId arg for point 1 fix with zoneId removing.

Changes are patched in original PR's commit f997ff4


Don't depend on num's default features; removing rand requirement

requirement for rand comes from num's default features which are all unused in chrono
not including the default features means that that build no longer requires rand (and hence does not pass it on to others importing chrono)
Not quite sure why cargo test still wants rand, but it's to do with the serde_json dev-dependency

Good observation. I never thought that Chrono transitively depends on rand until this point. Thank you for the PR!


Inherited tests from base class are ignored

Description

It would be fantastic if MSTest2 would allow you to derive from another class and inherit tests from that base class.  This actually works if both classes are within the same assembly, but breaks across assemblies.  This is detailed on this StackOverflow post.

Steps to reproduce
Base class in Assembly 1:
[TestClass]
public abstract class TestingBase
{
    [TestMethod]
    public void TestBase()
    {
        Assert.IsTrue(true);
    }
}

Derived class in Assembly 2:
[TestClass]
public class Testing : TestingBase
{
    [TestMethod]
    public void TestTwo()
    {
        Assert.IsTrue(true);
    }
}

Expected behavior

When running the "Testing" test class, both TestBase and TestTwo should be run.

Actual behavior

Only TestTwo is run.

Environment

Running the latest MSTest2 NuGet package and adapter with Visual Studio 2015.

Would love to see this fixed, and am willing to submit a pull request if someone wants to point me in the right direction in regards to what code I'd want to start looking at.

Just found this pull request that looks like it might fix this bug!  Let's get that thing merged!!!

@MikeChristensen : This is #23 and @ajryan helped with fixing this already. This should be part of the nuget release this week. However, this is currently under a setting EnableBaseClassTestMethodsFromOtherAssemblies as #144 describes. We plan to make this the default as soon as we fix an issue that popped up post this change that is being discussed in #23.
Thanks for willing to fix this however 👍 . Will close this issue.


Fix collision with entities that has 'simple' in their name

Running the command ludown parse toluis produces an error if the .lu file being parsed contains an entity with 'simple' on its name confusing it with the built-in 'Simple' entity type.
This change fixes the error Error: Line #n is not part of a Intent/ Entity/ QnA that gets thrown instead of getting the entity parsed correctly.
Before applying the fix, the mocha test added for this case fails

The line 179 is comparing if the entityType contains the word 'simple', giving a wrong comparison if the entity name is a compound name like 'simple-test'.
With the fix in place, the test now passes correctly

Pull Request Test Coverage Report for Build 398

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

Totals

Change from base Build 397:
0.0%

Covered Lines:
3609

Relevant Lines:
5375

💛  - Coveralls

Closed since #790 is merged and the fix is in place.


Add documentation for DROP SHARD, DROP SERVER, SHOW SERVERS

DROP SHARD:

Relevant version: 0.11
Adds description to the query language reference and database management docs
Fixes https://github.com/influxdata/docs.influxdata.com/issues/339

DROP SERVER:

Relevant versions: 0.10,0.11
Adds description to the query language reference doc and adds a link to the cluster setup doc
Adds META and DATA to keywords in query language reference doc
Fixes https://github.com/influxdata/docs.influxdata.com/issues/186

SHOW SERVERS:

Relevant versions: 0.10,0.11
Adds description to the query language reference doc

@e-dard can you please review?
@jwilder who else should review the Docs PR for DROP SHARD, DROP SERVER, and SHOW SERVERS?

LGTM.  Drop server is going to be removed.  show servers likely will be as well too.

:+1: Ship it.

Thank you!
@e-dard, best ship it message yet.


[ALLUXIO-1819] Used diamond operator instead of Guava's shorthand

https://alluxio.atlassian.net/browse/ALLUXIO-1819
This JIRA is still work in progress and needs more PRs.

LGTM

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9493/
Test PASSed.

LGTM, thanks for the upgrades


set_default_x expects None if x_shape not explicitely specified upon snpe object instantiation

Hey,
The x_shape is marked as optional in the docstrings of SNPE_C. However, if I do not specify it, i.e. it defaults to None, the I get the following error, when I want to call posterior.set_default_x:
Traceback (most recent call last):   File "/home/ga26jey/infer-decision-making/experiments/fit_five/visualize_five.py", line 59, in <module>     file_name_postfix=postfix)   File "/home/ga26jey/infer-decision-making/experiments/fit_five/utils.py", line 53, in generate_plots     posterior.set_default_x(x_o_parameter_values)   File "/home/ga26jey/anaconda3/envs/idm/lib/python3.7/site-packages/sbi/inference/posterior.py", line 133, in set_default_x     processed_x = process_x(x, self._x_shape)   File "/home/ga26jey/anaconda3/envs/idm/lib/python3.7/site-packages/sbi/user_input/user_input_checks.py", line 488, in process_x     f"Observed data shape ({input_x_shape}) must match " AssertionError: Observed data shape (torch.Size([1, 5])) must match the shape of simulated data x (None).
Or put in little more visual way:
This works:
snpe_c = SNPE_C(simulator=simulator,                         prior=prior,                         x_shape=x_o.shape,                         simulation_batch_size=key['simulation_batch_size'],                         density_estimator=density_estimator)
This gives me the described error:
snpe_c = SNPE_C(simulator=simulator,
prior=prior,
simulation_batch_size=key['simulation_batch_size'],
density_estimator=density_estimator)
Not sure if it's intentional, but It's definitely not handy, as that means I need to pass x_o explicitely whenver calling posterior.sample, posterior.log_prob, etc.

Also note #255

Thanks! Possibly this could be addressed as part of https://github.com/mackelab/sbi/pull/256


Add Terasology Twitch Listing

As is discussed in MovingBlocks/DestinationSol#419, Destination Sol has its own Twitch entry, although not much in there. But Terasology, as it turns out, doesn't have one at all!
So, your task will be to:

Create a twitch entry for Terasology
Create/find a nice box art for Terasology
Get the box art associated with the twitch entry

Quest hints:

You can view the requirements for a game entry to appear on Twitch here
You need to create a Terasology entry at the GiantBomb game wiki

Make sure to provide as many details about Terasology there as you can

Lots of info can be found just by googling
There is some info on Wikipedia (List of Open Source games)
You can probably find a lot of stuff in our forum

You might want to provide a box art for there too

You probably stand the highest chance of finding a suitable existing box art by scourging through our forum
To get the box art associated on twitch, you will need to ping someone with official terasology email to do it. Cervator is probably a pretty safe choice, but you can try asking in our chat too for other people.

Just to add: our official Twitch is http://www.twitch.tv/terasology

Getting a 600x800 boxart for Terasology is important. I can help make one if I find the time later.


Fix support for .NET 7 public releases

Update the agent to support determining .NET 7 versions from the public release notes.
It might be the case that this can't be merged until 7.0.0 RTM as it would interfere with your own testing before that ships, but without these changes I can't get an app targeting .NET 7 RC1 to work correctly because it tries to use the daily builds feed and then fails to get the right version numbers automatically.

Is the error you are getting about a 6.0.10 runtime package not resolved? Otherwise can you show me a way to repro what's happening?

I don't have a public repro for this at the moment, but I can make one tomorrow if the following details aren't useful.
So we have an app with 7.0.100-rc.1.22431.12 in its global.json targeting net7.0.
Initially it was trying to use .NET 6.0.9, but I found the docs to force it to use net7.0 with the application.framework setting.
When crank runs as-is once the framework is overridden, it tries to resolve the latest RTM build from the nightlies and also installs a .NET 8 SDK. This then fails to restore the NuGet packages it tries to use because the ones it finds aren't in NuGet.org.
[11:39:33 INF] Installing dotnet runtimes and sdk
[11:39:33 INF] Detected target framework: 'net7.0'
[11:39:33 INF] Patching project file with Framework References
[11:39:35 INF] Checking package: https://dotnetbuilds.azureedge.net/public/Runtime/7.0.0-rtm.22480.10/dotnet-runtime-7.0.0-rtm.22480.10-linux-x64.tar.gz
[11:39:36 INF] Runtime: 7.0.0-rtm.22480.10 (Latest - From 7.0 feed)
[11:39:36 INF] SDK: 8.0.100-alpha.1.22480.9 (Edge)
[11:39:36 INF] Checking package: https://dotnetbuilds.azureedge.net/public/aspnetcore/Runtime/7.0.0-rtm.22501.6/aspnetcore-runtime-7.0.0-rtm.22501.6-linux-x64.tar.gz
[11:39:37 INF] ASP.NET: 7.0.0-rtm.22501.6 (Latest - From 7.0 feed)
[11:39:37 INF] Patching existing global.json file
[11:39:37 INF] Installing SDK '8.0.100-alpha.1.22480.9' ...
[11:39:37 INF] Checking package: https://dotnetbuilds.azureedge.net/public/Sdk/8.0.100-alpha.1.22480.9/dotnet-sdk-8.0.100-alpha.1.22480.9-linux-x64.tar.gz
[11:41:38 INF] Installing Runtime '7.0.0-rtm.22480.10' ...
[11:41:38 INF] Checking package: https://dotnetbuilds.azureedge.net/public/Runtime/7.0.0-rtm.22480.10/dotnet-runtime-7.0.0-rtm.22480.10-linux-x64.tar.gz
[11:41:45 INF] Installing ASP.NET runtime '7.0.0-rtm.22501.6' ...
[11:41:45 INF] Checking package: https://dotnetbuilds.azureedge.net/public/aspnetcore/Runtime/7.0.0-rtm.22501.6/aspnetcore-runtime-7.0.0-rtm.22501.6-linux-x64.tar.gz
[11:41:52 INF] Working directory: /tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app
[11:41:52 INF] Command line: /tmp/benchmarks-agent/benchmarks-server-1044/f1ocy0ny.vhn/dotnet publish some-app.csproj -c Release -o /tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/published /p:MicrosoftNETCoreAppPackageVersion=7.0.0-rtm.22480.10 /p:MicrosoftAspNetCoreAppPackageVersion=7.0.0-rtm.22501.6 /p:MicrosoftNETCoreApp50PackageVersion=7.0.0-rtm.22480.10 /p:GenerateErrorForMissingTargetingPacks=false /p:MicrosoftNETPlatformLibrary=Microsoft.NETCore.App /p:RestoreNoCache=true --framework net7.0 --self-contained -r linux-x64
[11:41:57 INF] Processing job 'application' (1) in state Failed
[11:41:57 INF] Driver stopping job '1'
[11:41:58 INF] Processing job 'application' (1) in state Failed
[11:41:58 INF] Driver deleting job '1'
[11:39:25.776] Running session 'e6b354f5a5c7464998427e53c4bb6b05' with description ''
[11:39:25.863] Starting job 'application' ...
[11:39:26.096] Submitted job: http://localhost:5010/jobs/1
[11:39:29.206] 'application' has been selected by the server ...
[11:39:29.213] 'application' is now building ... http://localhost:5010/jobs/1/buildlog
[11:41:57.417] 'application' failed on agent, stopping...

Command:
dotnet publish some-app.csproj -c Release -o /tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/published /p:MicrosoftNETCoreAppPackageVersion=7.0.0-rtm.22480.10 /p:MicrosoftAspNetCoreAppPackageVersion=7.0.0-rtm.22501.6 /p:MicrosoftNETCoreApp50PackageVersion=7.0.0-rtm.22480.10 /p:GenerateErrorForMissingTargetingPacks=false /p:MicrosoftNETPlatformLibrary=Microsoft.NETCore.App /p:RestoreNoCache=true --framework net7.0 --self-contained -r linux-x64
MSBuild version 17.4.0-preview-22479-02+88f89b105 for .NET
  Determining projects to restore...
/tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj : error NU1102: Unable to find package Microsoft.NETCore.App.Runtime.linux-x64 with version (= 7.0.0-rtm.22480.10)
/tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj : error NU1102:   - Found 98 version(s) in Artifactory [ Nearest version: 7.0.0-rc.1.22426.10 ]
/tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj : error NU1102:   - Versions from aspnet-contrib were not considered
/tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj : error NU1102:   - Versions from local were not considered
/tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj : error NU1102: Unable to find package Microsoft.AspNetCore.App.Runtime.linux-x64 with version (= 7.0.0-rtm.22501.6)
/tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj : error NU1102:   - Found 98 version(s) in Artifactory [ Nearest version: 7.0.0-rc.1.22427.2 ]
/tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj : error NU1102:   - Versions from aspnet-contrib were not considered
/tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj : error NU1102:   - Versions from local were not considered
  Failed to restore /tmp/benchmarks-agent/benchmarks-server-1044/v33drppj.pba/some-app/src/some-app/some-app.csproj (in 2.43 sec).
Exit code: 1

I did try explicitly specifying all the versions (runtime, ASP.NET Core and SDK), but because it looked in the nightly feed for that it couldn't validate they existed and failed too.
Once I made all the changes in the commit, it started to work as I'd expect.

You can see the incorrect behaviour here.

because the ones it finds aren't in NuGet.org

The agent is generating a custom NuGet.config with these internal feeds, I will try to understand why your NuGet.config file at the root is taking precedence. Maybe by design and because there is a <clear /> inside.
But I believe that what you need is a way to use the "current" channel instead of edge, at least by using the argument on the command. And maybe that's what your PR is fixing ... looking


unix: use vfork instead of fork for uv_spawn

When using vfork the calling thread is suspended until the child calls
execve, and they share the memory space, so we can replace the error
reporting mechanism from using a pipe to just writing to a variable.
I hope I understood vfork vs fork right. While tests pass here locally, lets discuss and have Jenkins test it. FWIW, the spawn benchmark improves significantly (average of 3 runs):

v1.x: 733 spaws/sec
vfork: 967 spaws/sec

R=@bnoordhuis

Forgot to mention, I ran the tests on Linux.

https://jenkins-iojs.nodesource.com/view/libuv/job/libuv+any-pr+multi/54/

I don't think we can apply this change as-is.  Here is what the spec says about vfork():

The vfork() function shall be equivalent to fork(), except that the behavior is undefined if the process created by vfork() either modifies any data other than a variable of type pid_t used to store the return value from vfork(), or returns from the function in which vfork() was called, or calls any other function before successfully calling _exit() or one of the exec family of functions.

In other words, you cannot do anything after the call to vfork() except call execve() or _exit().  It's not legal to call chdir(), setuid(), etc.  In those cases, we would have to fall back to fork().
Linux relaxes the standard restrictions a little but there is an important catch: fork handlers registered with pthread_atfork() don't run when you use vfork().  I think that alone is a significant enough change that it's probably better to just stick with fork().
We could perhaps add a UV_PROCESS_USE_VFORK flag but it should come with a big fat warning that you shouldn't use it unless you know what you're doing.

Damn, too good to be true, I guess :-( Thanks for having a look either way Ben! I don't feel like adding UV_PROCESS_USE_VFORK, it would make the code more complex for probably little gain. I just explored this idea out of curiosity.

Oh, for a bit of historic background: node used to use vfork() pre-libuv.  It was causing no end of trouble and the real world speed up was mostly negligible; the real cost center is normally the dynamic linker.

I see. Thanks again Ben. Closing then!


Update Visual Studio Code to version 1.23.0 fails

vscode-inno-updater.log
Issue Type: Bug
Visual Studio Code fails to update using the tooling provided in the menus. The error in the log file is Access Denied (on Code.exe)

Launch VS Code via Cortana Search; hit enter to select matched result.
Click Gear Icon; if already checked, click Install Update, otherwise, click Check for Updates and then Install Update.
If necessary, progress though dialog(s).
Error occurs.

Does this issue occur when all extensions are disabled?: Yes
Failure behavior is the same across at least 10 attempts to install.
VS Code version: Code 1.22.2 (3aeede733d9a3098f7b4bdc1f66b63b0f48c1ef9, 2018-04-12T16:38:45.278Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz (8 x 2712)

Memory (System)
15.89GB (8.19GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

vsliveshare
ms-
0.3.93

Is similar to #48000 but not the same generated log file. Also, #48000 is already closed... and obviously this is still happening.

Rendering in Maximized mode is broken.
A window titled "Welcome - folder_name - Visual Studio Code" takes over the top 12% or so of the window and hides the IDE display. The window does not respond to Max/Min/Close buttons.
Version 1.23.0
Commit 7c7da59c2333a1306c41e6e7b68d7f0caa7b3d45
Date 2018-05-03T16:44:55.614Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
Windows 10 x64 Professional

Exact same thing for me.

💩  <- added a poop emoji because I recently learned the vscode team filters for that.

/duplicate #47841


Secrets in Pulumi state file

Hi,
I am not sure whether this feature request / issue should be here or the main repo (https://github.com/pulumi/pulumi).
I am working with a customer who has got a lot of complaince requirements when provisioning infrastructure.
when using pulumi, what i have observed is that the statefile contains the secrets for example, the primary key, the secondary key, connection strings of the infrastucture components.
we have an option using the custom resource option by providing a list of properties to be encrypted in the statefile using AdditionalSecretOutputs .
This will do the job in case of smaller infrastructure with less number of components.  But incase of a huge infrastructure, where there are a lot of people contributing , this will be bit cumbersome to the reviewers to make sure that the all the correct properties are provided as list for encryption.
Is there a way we can setup encrypting for all the output properties ?
It can be either an environment variable like
encryptAllOutputPropertiesInStateFile = True which will make sure all outputs are encrypted.
If there is a possibility to encrypt the statefile itself, that would be great.
The Main usecase here is below :
If the statefile is compromised, the entire infra secrets are availble in clear text and the attacker can gain access to all the components.

Hi @midhun-mohan,

If there is a possibility to encrypt the statefile itself, that would be great.

If you're using the Pulumi Service backend, the statefile itself is already encrypted in transit and at rest.
If you're using a self-managed backend (e.g. Azure Blob Storage), you should be able to configure it to use encryption.

Is there a way we can setup encrypting for all the output properties ?

There isn't aside from listing all the outputs in AdditionalSecretOutputs. This is an interesting feature request, though, so I'm going to transfer this issue to the pulumi/pulumi repository to track the request.
However, there are a couple things you could consider doing in the meantime to enforce the use of secret outputs:

You could write custom CrossGuard policies that ensure output properties of particular resources are included in AdditionalSecretOutputs.

You could write abstractions around the raw resources (such as a ComponentResource wrapper or helper functions), that handle creating the underlying resources and setting the AdditionalSecretOutputs for the properties that must be treated as secrets. Developers in the organization would then use these abstractions instead of creating the raw underlying resources directly, so nobody forgets to set AdditionalSecretOutputs on the properties that need to be secrets. CrossGuard policies could be used in conjunction with this approach for enforcement.


Is it possible to change the colors of whitespaces?

When I toggle the whitespace rendering in vscode while using the Material theme, those dots that are rendered are hardly visible. Could I change the greyish color of those whitespace dots to a grey which gives more contrast to the background?

Ok got it, there is a global color setting in vscode, that I found in one of the issues here

For easier reference:
    "workbench.colorCustomizations": {
        "editorWhitespace.foreground": ""
     }


Enumerate template specializations in OpDescription via macro

Remove the template <> const OpDescription &OpDescription::get<...>() specializations and use some macros instead.
I abstained from adding more complicated macros for now to handle the Intrinsic:: prefix similarly for the non-array and array cases. Open for suggestions.

Added dedicated macros.


github build script needs to point at correct opensha repo

gradle.yml uses opensha/opensha but should use GNS-science/opensha so that we have better control over which opensha version we use.
This was correct until 17/12/2021

done


Update access-api.mdx

Closes: #???
Description

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Also I just noticed this page which might benefit being linked from here: https://developers.flow.com/learn/concepts/networks


Determine steps to publish Java JAR to Maven repo

See https://central.sonatype.org/publish/publish-guide/#introduction


change storage setting in default backend application.yml to h2

…lasticsearch
Signed-off-by: zhengyangyong<EMAIL_ADDRESS>Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues

Bug fix

Bug description.
Default storage setting in default backend application.yml is elasticsearch not h2,may better let newbie startup(quick start) without any config.
In windows environment also difficult to know where problem is for collector failed because the Console windows will blink close without any log.
Screenshot with prt sc key :

How to fix?
change storage setting in default backend application.yml to h2

New feature or improvement

Describe the details and related test reports.

Coverage remained the same at 19.497% when pulling 18b07e656aad77656a683d0f8f66052320f9f8c7 on zhengyangyong:update_default_app_yml into 6ba42f364a12ae1a0d80f7fd7cd6d3157a4119c4 on apache:master.

First of you, welcome and thank you to contribute your thoughts. By using h2 as default storage implementations, will help the newbie. I agree with that, and you can find our, our 3.2.x releases are following this issue. The reason you can't find this in master branch is we are not ready. Even this pr merged, no H2 implementation can be provided.
You can check all H2 DAOs to find out.
BTW, H2 implementation will not be provided in 5.0.0-alpha release. Because no committer is working on that.
And for

In windows environment also difficult to know where problem is for collector failed because the Console windows will blink close without any log.

I agree this is not a good experience, but I don't know any way to improve that, seems like a Windows design issue. If you have any suggestion, I am very glad to discuss with you about that, and work with a pr with you.

I see,thanks


PinToTop, pinning a window to the top

Description of the new feature / enhancement
Basically a keyboard shortcut to pin a window to the top.
Scenario when this would be used?
It's a feature I really miss in Windows. For instance, Sticky Notes, why on earth can't we pin a sticky note to the top?? I want to keep my notes on my screen and another window below it so I don't have to shrink the main window... Or a small music player app that you want to pin on the screen so you can easily access it while working on something else. Or if you want to watch a video while working in Word. You can basically have a Picture In Picture experience.
Supporting information
I think something like [Ctrl]+[SPACE] would work well. There already is a way using AutoHotkey, and yes this works well, but I would like to see this feature in PowerToys as it's such a basic feature in my opinion.

Tracked with #13


deps: upgrade to<EMAIL_ADDRESS>

Related to #4483
Related to https://github.com/camunda/camunda-modeler/issues/4467
Proposed Changes
Since electron@32 introduces breaking changes which require to modify multiple parts of the app (cf. https://github.com/camunda/camunda-modeler/issues/4483#issuecomment-2311778398), for now we can keep electron@31. The version in this PR contains a fix for https://github.com/camunda/camunda-modeler/issues/4467
Checklist
To ensure you provided everything we need to look at your PR:

[x] Brief textual description of the changes present
[ ] Visual demo attached
[ ] Steps to try out present, i.e. using the @bpmn-io/sr tool
[x] Related issue linked via Closes {LINK_TO_ISSUE} or Related to {LINK_TO_ISSUE}

This is not closing https://github.com/camunda/camunda-modeler/issues/4483 as we will face it once we upgrade to v32.


SP5 Label: changing font name or style should blow the cache

Issue: Label.cs strings are not shown using the requested FontName.
When using WPF, Title / XLabel / YLabel are always shown using the font "Segoe UI", even when a different font is set.
Expected result:

Actual result:

ScottPlot Version: v5.0.18
Code Sample:
// XAML: <sp:WpfPlot Name="WpfPlot1" />
WpfPlot1.Plot.Title("测试");
WpfPlot1.Plot.XLabel("测试");
WpfPlot1.Plot.YLabel("已占用");
WpfPlot1.Plot.Style.SetBestFonts();
WpfPlot1.Refresh();
WpfPlot1.Plot.SavePng("bug.png", 300, 300);

Possible root cause:
The root cause seems to be that the Skia typeface is cached in Label.cs. (Property CachedTypeface)
Once it is cached, it is never updated, even when the Label.cs:FontName property is changed.
It still works in Avalonia as long as all FontNames are set before the first Render() call is made by the user code.
But after that, the FontName property is ignored.
It does not work in WPF since the WPF control seems to make an automatic Render() call before the start of the user code. This locks in the default FontName.
WPF render() call:
https://github.com/ScottPlot/ScottPlot/blob/9905ff5bc27feb584790d1d6106b55e2b2631338/src/ScottPlot5/ScottPlot5 Controls/ScottPlot.WPF/WpfPlot.cs#L42
When changing the Label.cs properties as below, the fonts render as expected:
private SKTypeface? _cachedTypeface = null;
public SKTypeface? CachedTypeface
{
    get
    {
        return _cachedTypeface;
    }
    set
    {
        Debug.WriteLine($"CachedTypeface set: Label text = '{Text}', typeface = '{value?.FamilyName}'");
        _cachedTypeface = value;
    }
}

private SKTypeface Typeface
{
    get
    {
        if (_cachedTypeface == null)
        {
            Debug.WriteLine($"Label.cs:Typeface.get(): _cachedTypeface == null. Text = '{Text}'");
        }
        return CachedTypeface ??= FontStyle.CreateTypeface(FontName, Bold, Italic);
    }
}

private string fontName = Fonts.Default;
public string FontName
{
    get => fontName;
    set
    {
        fontName = value;
        if (CachedTypeface != null)
        {
            CachedTypeface.Dispose();
            CachedTypeface = FontStyle.CreateTypeface(FontName, Bold, Italic);
        }
    }
}

NOTE: This is just a quick test, and not a well designed implementation. At a minimum, the Bold and Italic properties should probably also trigger updates?

5beef84865a0868884593ebd515f4bd2468a413d should fix it!
If it doesn't, we can try disabling caching with this new flag:
formsPlot1.Plot.Axes.Left.Label.UseCachedTypefaces = false;

v5.0.19 works for my test case, while v5.0.18 displays the problem. Nice work getting it pushed out so fast!


mesa: 20.3.2 -> 20.3.3

The "util: Disable memstream for Apple builds" patch got backported:
https://gitlab.freedesktop.org/mesa/mesa/-/commit/6a006c35675636cb0d2cbfdfb1681b6639e0664d

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.


Add missing typings for device type

Change-type: minor
Signed-off-by: Stevche Radevski<EMAIL_ADDRESS>
Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

@balena-ci I self-certify!


pull request for e2e test at 01/08/2017 05:26:12

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


Could include JSX syntax?

I know now I can enable JSX highlighting with package of Babel, but in Sublime Text, I must ignore default JS package first to make that work, that means I will lose all features (such as auto complete) provided by default JS package.

Use https://github.com/borela/naomi instead, it'll enable JSX support + other javascript extensions.

This isn't specifically an issue of the syntax plugin since we can't bundle an entire JS syntax implementation. You can optionally open issues with Babel/Sublime to add functionality that you might be missing with each packages.


Cluster-wide local cache invalidation support

At the moment, there is no way to invalidate the local cache in other cluster nodes. We have to wait till the cache expiry time for that.
This feature provides support for immediate invalidation of the local caches in all the in a cluster when one node removes/modifies a cache entry.

Implemented as part of https://github.com/wso2/carbon-kernel/pull/1424


Climate missing override parameter?

When you start the climate in the Tesla app with a door open it prompts for an override. Presumably using the same manual_override parameter that exists in other commands like Defrost?
That override is present on some commands in the SDK but is missing here: https://github.com/teslamotors/vehicle-command/blob/e8634aa8ac74282ab195edc462a86b1f87f451de/pkg/vehicle/climate.go#L38

Hello, can I control the air conditioner only through ble? I look at the code as if it is calling the Internet method

@stx The Manual Override bool is defined in the protobuf built by the method you linked: https://github.com/teslamotors/vehicle-command/blob/main/pkg/protocol/protobuf/car_server.proto#L142. Be careful about setting it to true; IIRC, it also overrides dog mode and low SoC, and so, as the name suggests, should never be set automatically without explicit manual confirmation.
There's a pending PR that will likely conflict with a quick fix, so I'd prefer to wait it gets merged.
@zhy2020 Please keep comments on topic. The example BLE program shows how to accomplish what you're after.

@stx The Manual Override bool is defined in the protobuf built by the method you linked: https://github.com/teslamotors/vehicle-command/blob/main/pkg/protocol/protobuf/car_server.proto#L142. Be careful about setting it to true; IIRC, it also overrides dog mode and low SoC, and so, as the name suggests, should never be set automatically without explicit manual confirmation.
There's a pending PR that will likely conflict with a quick fix, so I'd prefer to wait it gets merged.
@zhy2020 Please keep comments on topic. The example BLE program shows how to accomplish what you're after.

Makes total sense. Thank you!


Support TLS 1.2 In PublishSymbolsV2 Task

Task name:
PublishSymbolsV2
Description:
Add TLS Helper module to support TLS 1.2
Documentation changes required: (Y/N)
N
Added unit tests:
N
Attached related issue: (Y/N)
#15139
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

@fadnavistanmay would you be able to review this PR as a code owner for this task?

/azp run

/azp run


Closing add a message screen dismisses the full story.

whenever you drag down it dismisses the whole story instead of only dismissing the add a message overlay

Please follow the document and do all the steps correctly
For example
https://github.com/shirvanie/flutter_story/blob/master/example/lib/main.dart


Add warning to connectWithEmoji

Summary
Add warning to connectWithEmoji

"Window", not "windows"

In react the note is not viisble

I think that this text shold not be written upper case because  it is too annoying. Let's consider Do not close this window!

@JustynaBroniszewska @xitronix fixed for react

@xitronix @rzadp


C: Address remaining gcc compiler compatability issues

Description
This PR addresses all the remaining gcc issues C compiler compatibility issues.
In line with the previous issues we've reviewed, the majority of the fixes are either (a) fixing test cases with syntactically invalid C code; or (b) including a different expected results file for different compilers, because the results are morally equivalent but textually different.
The sole exception in this PR is STR37-C. This rule prohibits the use of the <ctype> functions. However, the C standard allows these "functions" to be implemented either as functions or macros, and we only detected the former. Detecting "arguments" to macros is tricky, because they are textually expanded and may appear in multiple places in the AST generated from the macro, with no easy way to determine which aspects came from the argument itself. We handle this by looking for specific AST patterns corresponding to known, supported compilers, and identifying a "unique" element to report as the argument itself.
Fixes https://github.com/github/codeql-coding-standards/issues/246.
Finally, as an additional "bonus" I've updated Rule 2.4 to excluded template parameters.
Change request type

[ ] Release or process automation (GitHub workflows, internal scripts)
[ ] Internal documentation
[ ] External documentation
[x] Query files (.ql, .qll, .qls or unit tests)
[ ] External scripts (analysis report or other code shipped as part of a release)

Rules with added or modified queries

[ ] No rules added
[ ] Queries have been added for the following rules:

rule number here

[x] Queries have been modified for the following rules:

Rule 2.4
Rule 2.6
Rule 21.1
DCL41-C
Rule 8.2
STR34-C
STR37-C

Release change checklist
A change note (development_handbook.md#change-notes) is required for any pull request which modifies:

The structure or layout of the release artifacts.
The evaluation performance (memory, execution time) of an existing query.
The results of an existing query in any circumstance.

If you are only adding new rule queries, a change note is not required.
Author: Is a change note required?

[x] Yes
[ ] No

🚨🚨🚨
Reviewer: Confirm that format of shared queries (not the .qll file, the
.ql file that imports it) is valid by running them within VS Code.

[ ] Confirmed

Reviewer: Confirm that either a change note is not required or the change note is required and has been added.

[ ] Confirmed

Query development review checklist
For PRs that add new queries or modify existing queries, the following checklist should be completed by both the author and reviewer:
Author

[x] Have all the relevant rule package description files been checked in?
[x] Have you verified that the metadata properties of each new query is set appropriately?
[x] Do all the unit tests contain both "COMPLIANT" and "NON_COMPLIANT" cases?
[x] Are the alert messages properly formatted and consistent with the style guide?
[x] Have you run the queries on OpenPilot and verified that the performance and results are acceptable?As a rule of thumb, predicates specific to the query should take no more than 1 minute, and for simple queries be under 10 seconds. If this is not the case, this should be highlighted and agreed in the code review process.
[x] Does the query have an appropriate level of in-query comments/documentation?
[x] Have you considered/identified possible edge cases?
[x] Does the query not reinvent features in the standard library?
[x] Can the query be simplified further (not golfed!)

Reviewer

[ ] Have all the relevant rule package description files been checked in?
[ ] Have you verified that the metadata properties of each new query is set appropriately?
[ ] Do all the unit tests contain both "COMPLIANT" and "NON_COMPLIANT" cases?
[ ] Are the alert messages properly formatted and consistent with the style guide?
[ ] Have you run the queries on OpenPilot and verified that the performance and results are acceptable?As a rule of thumb, predicates specific to the query should take no more than 1 minute, and for simple queries be under 10 seconds. If this is not the case, this should be highlighted and agreed in the code review process.
[ ] Does the query have an appropriate level of in-query comments/documentation?
[ ] Have you considered/identified possible edge cases?
[ ] Does the query not reinvent features in the standard library?
[ ] Can the query be simplified further (not golfed!)

🤖 Beep Boop! clang/cpp/x86_64 Matrix Testing for this PR has been completed but I didn't find anything to test!

🤖 Beep Boop! gcc/cpp/x86_64 Matrix Testing for this PR has been completed but I didn't find anything to test!

🤖 Beep Boop! clang/c/x86_64 Matrix Testing for this PR has been completed. See below for the results!

TEST_PASS            : False
PACKAGE              : Strings2
RULE                 : STR37-C
COMPILE_PASS         : True
QUERY                : ToCharacterHandlingFunctionsRepresentableAsUChar
SUITE                : CERT-C
COMPILE_ERROR_OUTPUT :
TEST_DIFFERENCE      : --- expected
                       +++ actual
                       @@ -1,28 +1,28 @@
                       -| test.c:7:3:7:9 | call to isalnum | $@ to character-handling function may not be representable as an unsigned char. | test.c:7:11:7:12 | * ... | Argument |
                       -| test.c:8:3:8:13 | isalpha(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:8:11:8:12 | * ... | Argument |
                       -| test.c:10:3:10:9 | call to isblank | $@ to character-handling function may not be representable as an unsigned char. | test.c:10:11:10:12 | * ... | Argument |
                       -| test.c:11:3:11:9 | call to iscntrl | $@ to character-handling function may not be representable as an unsigned char. | test.c:11:11:11:12 | * ... | Argument |
                       -| test.c:12:3:12:13 | isdigit(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:12:3:12:13 | (...) | Argument |
                       -| test.c:13:3:13:13 | isgraph(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:13:3:13:13 | (...) | Argument |
                       -| test.c:14:3:14:13 | islower(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:14:3:14:13 | (...) | Argument |
                       -| test.c:15:3:15:13 | isprint(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:15:3:15:13 | (...) | Argument |
                       -| test.c:16:3:16:9 | call to ispunct | $@ to character-handling function may not be representable as an unsigned char. | test.c:16:11:16:12 | * ... | Argument |
                       -| test.c:17:3:17:13 | call to __isspace | $@ to character-handling function may not be representable as an unsigned char. | test.c:17:11:17:12 | * ... | Argument |
                       -| test.c:18:3:18:13 | isupper(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:18:3:18:13 | (...) | Argument |
                       -| test.c:19:3:19:10 | call to isxdigit | $@ to character-handling function may not be representable as an unsigned char. | test.c:19:12:19:13 | * ... | Argument |
                       +| test.c:7:3:7:13 | isalnum(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:7:3:7:13 | (...) | Argument |
                       +| test.c:8:3:8:13 | isalpha(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:8:3:8:13 | (...) | Argument |
                       +| test.c:10:3:10:13 | isblank(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:10:3:10:13 | (...) | Argument |
                       +| test.c:11:3:11:13 | iscntrl(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:11:3:11:13 | (...) | Argument |
                       +| test.c:12:3:12:13 | isdigit(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:12:3:12:13 | (...) | Argument |
                       +| test.c:13:3:13:13 | isgraph(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:13:3:13:13 | (...) | Argument |
                       +| test.c:14:3:14:13 | islower(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:14:3:14:13 | (...) | Argument |
                       +| test.c:15:3:15:13 | isprint(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:15:3:15:13 | (...) | Argument |
                       +| test.c:16:3:16:13 | ispunct(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:16:3:16:13 | (...) | Argument |
                       +| test.c:17:3:17:13 | isspace(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:17:3:17:13 | (...) | Argument |
                       +| test.c:18:3:18:13 | isupper(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:18:3:18:13 | (...) | Argument |
                       +| test.c:19:3:19:14 | isxdigit(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:19:3:19:14 | (...) | Argument |
                        | test.c:21:3:21:9 | call to toupper | $@ to character-handling function may not be representable as an unsigned char. | test.c:21:11:21:12 | * ... | Argument |
                        | test.c:22:3:22:9 | call to tolower | $@ to character-handling function may not be representable as an unsigned char. | test.c:22:11:22:12 | * ... | Argument |
                       -| test.c:70:3:70:9 | call to isalnum | $@ to character-handling function may not be representable as an unsigned char. | test.c:70:11:70:11 | t | Argument |
                       -| test.c:71:3:71:12 | isalpha(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:71:11:71:11 | t | Argument |
                       -| test.c:73:3:73:9 | call to isblank | $@ to character-handling function may not be representable as an unsigned char. | test.c:73:11:73:11 | t | Argument |
                       -| test.c:74:3:74:9 | call to iscntrl | $@ to character-handling function may not be representable as an unsigned char. | test.c:74:11:74:11 | t | Argument |
                       -| test.c:75:3:75:12 | isdigit(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:75:3:75:12 | (...) | Argument |
                       -| test.c:76:3:76:12 | isgraph(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:76:3:76:12 | (...) | Argument |
                       -| test.c:77:3:77:12 | islower(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:77:3:77:12 | (...) | Argument |
                       -| test.c:78:3:78:12 | isprint(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:78:3:78:12 | (...) | Argument |
                       -| test.c:79:3:79:9 | call to ispunct | $@ to character-handling function may not be representable as an unsigned char. | test.c:79:11:79:11 | t | Argument |
                       -| test.c:80:3:80:12 | call to __isspace | $@ to character-handling function may not be representable as an unsigned char. | test.c:80:11:80:11 | t | Argument |
                       -| test.c:81:3:81:12 | isupper(a) | $@ to character-handling function may not be representable as an unsigned char. | test.c:81:3:81:12 | (...) | Argument |
                       -| test.c:82:3:82:10 | call to isxdigit | $@ to character-handling function may not be representable as an unsigned char. | test.c:82:12:82:12 | t | Argument |
                       +| test.c:70:3:70:12 | isalnum(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:70:3:70:12 | (...) | Argument |
                       +| test.c:71:3:71:12 | isalpha(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:71:3:71:12 | (...) | Argument |
                       +| test.c:73:3:73:12 | isblank(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:73:3:73:12 | (...) | Argument |
                       +| test.c:74:3:74:12 | iscntrl(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:74:3:74:12 | (...) | Argument |
                       +| test.c:75:3:75:12 | isdigit(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:75:3:75:12 | (...) | Argument |
                       +| test.c:76:3:76:12 | isgraph(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:76:3:76:12 | (...) | Argument |
                       +| test.c:77:3:77:12 | islower(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:77:3:77:12 | (...) | Argument |
                       +| test.c:78:3:78:12 | isprint(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:78:3:78:12 | (...) | Argument |
                       +| test.c:79:3:79:12 | ispunct(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:79:3:79:12 | (...) | Argument |
                       +| test.c:80:3:80:12 | isspace(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:80:3:80:12 | (...) | Argument |
                       +| test.c:81:3:81:12 | isupper(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:81:3:81:12 | (...) | Argument |
                       +| test.c:82:3:82:13 | isxdigit(c) | $@ to character-handling function may not be representable as an unsigned char. | test.c:82:3:82:13 | (...) | Argument |
                        | test.c:84:3:84:9 | call to toupper | $@ to character-handling function may not be representable as an unsigned char. | test.c:84:11:84:11 | t | Argument |
                        | test.c:85:3:85:9 | call to tolower | $@ to character-handling function may not be representable as an unsigned char. | test.c:85:11:85:11 | t | Argument |
                       [1/1 comp 37.8s eval 1.6s] FAILED(RESULT) /__w/codeql-coding-standards-release-engineering/codeql-coding-standards-release-engineering/codeql-coding-standards/c/cert/test/rules/STR37-C/ToCharacterHandlingFunctionsRepresentableAsUChar.qlref

TEST_PASS            : True
PACKAGE              : Declarations2
RULE                 : DCL41-C
COMPILE_PASS         : True
QUERY                : VariablesInsideSwitchStatement
SUITE                : CERT-C
COMPILE_ERROR_OUTPUT :
TEST_DIFFERENCE      :

TEST_PASS            : True
PACKAGE              : Strings3
RULE                 : STR34-C
COMPILE_PASS         : True
QUERY                : CastCharBeforeConvertingToLargerSizes
SUITE                : CERT-C
COMPILE_ERROR_OUTPUT :
TEST_DIFFERENCE      :

TEST_PASS            : True
PACKAGE              : DeadCode
RULE                 : RULE-2-4
COMPILE_PASS         : True
QUERY                : UnusedTagDeclaration
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
TEST_DIFFERENCE      :

TEST_PASS            : True
PACKAGE              : DeadCode
RULE                 : RULE-2-6
COMPILE_PASS         : True
QUERY                : UnusedLabelDeclaration
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
TEST_DIFFERENCE      :

TEST_PASS            : True
PACKAGE              : Preprocessor4
RULE                 : RULE-21-1
COMPILE_PASS         : True
QUERY                : DefineAndUndefUsedOnReservedIdentifierOrMacroName
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
TEST_DIFFERENCE      :

TEST_PASS            : True
PACKAGE              : Declarations4
RULE                 : RULE-8-2
COMPILE_PASS         : True
QUERY                : FunctionTypesNotInPrototypeForm
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
TEST_DIFFERENCE      :

🤖 Beep Boop! Matrix Testing for this PR has been completed. If no reports were posted it means this PR does not contain things that need matrix testing!

🤖 Beep Boop! clang/cpp/x86_64 Matrix Testing for this PR has been completed but I didn't find anything to test!

🤖 Beep Boop! gcc/cpp/x86_64 Matrix Testing for this PR has been completed but I didn't find anything to test!

🤖 Beep Boop! clang/c/x86_64 Matrix Testing for this PR has been completed. See below for the results!

TEST_DIFFERENCE      :
TEST_PASS            : True
RULE                 : STR37-C
QUERY                : ToCharacterHandlingFunctionsRepresentableAsUChar
COMPILE_PASS         : True
PACKAGE              : Strings2
COMPILE_ERROR_OUTPUT :
SUITE                : CERT-C

TEST_DIFFERENCE      :
TEST_PASS            : True
RULE                 : DCL41-C
QUERY                : VariablesInsideSwitchStatement
COMPILE_PASS         : True
PACKAGE              : Declarations2
COMPILE_ERROR_OUTPUT :
SUITE                : CERT-C

TEST_DIFFERENCE      :
TEST_PASS            : True
RULE                 : STR34-C
QUERY                : CastCharBeforeConvertingToLargerSizes
COMPILE_PASS         : True
PACKAGE              : Strings3
COMPILE_ERROR_OUTPUT :
SUITE                : CERT-C

TEST_DIFFERENCE      :
TEST_PASS            : True
RULE                 : RULE-2-4
QUERY                : UnusedTagDeclaration
COMPILE_PASS         : True
PACKAGE              : DeadCode
COMPILE_ERROR_OUTPUT :
SUITE                : MISRA-C-2012

TEST_DIFFERENCE      :
TEST_PASS            : True
RULE                 : RULE-2-6
QUERY                : UnusedLabelDeclaration
COMPILE_PASS         : True
PACKAGE              : DeadCode
COMPILE_ERROR_OUTPUT :
SUITE                : MISRA-C-2012

TEST_DIFFERENCE      :
TEST_PASS            : True
RULE                 : RULE-21-1
QUERY                : DefineAndUndefUsedOnReservedIdentifierOrMacroName
COMPILE_PASS         : True
PACKAGE              : Preprocessor4
COMPILE_ERROR_OUTPUT :
SUITE                : MISRA-C-2012

TEST_DIFFERENCE      :
TEST_PASS            : True
RULE                 : RULE-8-2
QUERY                : FunctionTypesNotInPrototypeForm
COMPILE_PASS         : True
PACKAGE              : Declarations4
COMPILE_ERROR_OUTPUT :
SUITE                : MISRA-C-2012

🤖 Beep Boop! gcc/c/x86_64 Matrix Testing for this PR has been completed. See below for the results!

TEST_PASS            : True
RULE                 : STR37-C
QUERY                : ToCharacterHandlingFunctionsRepresentableAsUChar
TEST_DIFFERENCE      :
COMPILE_PASS         : True
SUITE                : CERT-C
COMPILE_ERROR_OUTPUT :
PACKAGE              : Strings2

TEST_PASS            : True
RULE                 : DCL41-C
QUERY                : VariablesInsideSwitchStatement
TEST_DIFFERENCE      :
COMPILE_PASS         : True
SUITE                : CERT-C
COMPILE_ERROR_OUTPUT :
PACKAGE              : Declarations2

TEST_PASS            : True
RULE                 : STR34-C
QUERY                : CastCharBeforeConvertingToLargerSizes
TEST_DIFFERENCE      :
COMPILE_PASS         : True
SUITE                : CERT-C
COMPILE_ERROR_OUTPUT :
PACKAGE              : Strings3

TEST_PASS            : True
RULE                 : RULE-2-4
QUERY                : UnusedTagDeclaration
TEST_DIFFERENCE      :
COMPILE_PASS         : True
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
PACKAGE              : DeadCode

TEST_PASS            : True
RULE                 : RULE-2-6
QUERY                : UnusedLabelDeclaration
TEST_DIFFERENCE      :
COMPILE_PASS         : True
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
PACKAGE              : DeadCode

TEST_PASS            : True
RULE                 : RULE-21-1
QUERY                : DefineAndUndefUsedOnReservedIdentifierOrMacroName
TEST_DIFFERENCE      :
COMPILE_PASS         : True
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
PACKAGE              : Preprocessor4

TEST_PASS            : True
RULE                 : RULE-8-2
QUERY                : FunctionTypesNotInPrototypeForm
TEST_DIFFERENCE      :
COMPILE_PASS         : True
SUITE                : MISRA-C-2012
COMPILE_ERROR_OUTPUT :
PACKAGE              : Declarations4

🤖 Beep Boop! Matrix Testing for this PR has been completed. If no reports were posted it means this PR does not contain things that need matrix testing!


Exception from HRESULT: 0x80131515

When I attempted to import:
PS > Import-Module PolicyFileEditor -Force
Add-Type : Could not load file or assembly
I've unblocked the DLL.  Turns out I can load the DLL with:
Add-Type -Path C:\users...\PolFileEditor.dll -ErrorAction Stop
....
Why is the module not able to load the dll?

It works fine with me, no error.

Does that continue to happen after you start a fresh PowerShell session?  There's some caching that happens with regard to blocking / unblocking a file, as I recall.

This does appear to be the fix.
For some reason I had to open a new powershell and then I was able to import the module.  Maybe it is as you said, because I had to unblock the DLL.
Thank you.


Added more helpful error message for animation player

Fixes #16567

Let's reopen and improve the error message.

Thanks!

Cherry-picked for 3.2.4.


baresip: 3.8.0 -> 3.8.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/baresip/baresip/releases.
meta.description for baresip is: A modular SIP User-Agent with audio and video support
meta.homepage for baresip is: https://github.com/baresip/baresip
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.8.1 with grep in /nix/store/m9izp4a4xhlzvlaqscfjnhj0r6vqklai-baresip-3.8.1
found 3.8.1 in filename of file in /nix/store/m9izp4a4xhlzvlaqscfjnhj0r6vqklai-baresip-3.8.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
baresip

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/m9izp4a4xhlzvlaqscfjnhj0r6vqklai-baresip-3.8.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A baresip https://github.com/r-ryantm/nixpkgs/archive/76384d4d3b167772bfea28416bfa99d532021e68.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/76384d4d3b167772bfea28416bfa99d532021e68#baresip

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/m9izp4a4xhlzvlaqscfjnhj0r6vqklai-baresip-3.8.1
ls -la /nix/store/m9izp4a4xhlzvlaqscfjnhj0r6vqklai-baresip-3.8.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

baresip

Maintainer pings
cc @elohmeier @7c6f434c for testing.

@ofborg eval


Integrated terminal in separate window

I'd like to have the integrated terminal in fullscreen mode on my laptop screen next to my desktop monitors.

It is possible to use an external terminal, but all features of the intergrated terminal would be lost (notably file links).
It is possible to manually resize the vscode window to extend beyond the bounds of one monitor, but it would require a lot of tinkering with tab/window placement, and the end result would still not compare to a fullscreen terminal.
It is possible to duplicate the vscode window, but file links would open in the same window as the terminal.

A dock/undock feature or similar would be greatly appreciated.

There's a tip in https://github.com/microsoft/vscode/issues/193267#issuecomment-2148129764

You can open a terminal as editor in the editor area and then drag that editor out into a separate window, can you try?

@bpasero @gjsjohnmurray Thank you for input. I've tested it and it works as advertised 👍 I would still prefer a cleaner solution where nothing but the terminal area is shown on screen, but I like it overall.


Improving Accuracy

Hi! This project is sooo awesome! It sure has made face recognition very easy. However, I have a few concerns.

After testing this on a few of my images, there were a few results that were incorrect. Assuming the my images have different dimensions, the results were often inaccurate. Such as comparing two different persons would say that the unknown face was the known face, when in reality they were completely different persons.

I tried comparing a colored image of a person to a different person that was B&W. It also produced inaccurate results. The image size was also different.

So far those 2 things are what I've noticed. My images contains different sizes. Also, my images are BGR and some B&W.
Any suggestions on improving my results? Could it be caused by my inconsistent image dimensions, or their colors?
Thanks!

Are you able to share some examples that failed?
In general, just having images of different sizes shouldn't be an issue (unless the images are very low resolution or something like that).

How should I share them?

If you are ok with them being public, you can drag and drop them in the comment box here. Otherwise I guess you could email them to me at<EMAIL_ADDRESS>

I think I'd prefer emailing them. Btw, I managed to improve the accuracy results by lowering the tolerance. From 0.6 to 0.5.

I'll send some of the pictures I used to your email.

You can also try setting jitter=100 when generating embeddings for a very slight improvement in accuracy in trade for slower runtime.


Made weather agent dependent on user location

I'm not sure if this is the best way to do it... I have zero experience with huginn and haven't written much production Ruby. So feel free to make suggestions for improvements.

I just tried exactly that (user location in weather agent), but not even knowing ruby or json, this is quite a rough start.
So far, I managed to get my location via mqtt in a JSON string (?).
{
"topic": "owntracks/",
"message": {
"_type": "location",
"tid": "3g",
"t": "u",
"conn": "w",
"lat": 66.6,
"lon": 66.6,
"tst":<PHONE_NUMBER>,
"acc": 10,
"batt": 99
},
"time":<PHONE_NUMBER>
}
how would I get the lat and lon from this event into the location field of the weather agent?

@Azimoth can you please open a new issue to discuss it?


LLDP probe

run skydive-functional-tests-backend-elasticsearch

run skydive-functional-tests-backend-elasticsearch

Starting that code on my 4 nodes led to a segfault when starting the capture : http://pastebin.test.redhat.com/653969

Running on c9d0dbe : if lldapd is not running on the node, the graph doesn't report anything. But thanks to your latest commit, the output is fine in that case.

Just a question. If the switch is updating an existing field, like a mtu switch from 1500 to 9000, do we update the field value ?
There is cases where having an interface up/down should update/remove what we know about it, but data we know could change without any interface up/down event.
Are we compatible with that ?
Thx for cool work !

rerun tests

run skydive-cdd-overview-tests

run skydive-functional-tests-backend-elasticsearch

run skydive-functional-tests-backend-orientdb

run skydive-functional-tests-backend-elasticsearch


fix: asset filter query test

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.332%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4422

Relevant Lines:
5437

💛  - Coveralls


Update expressroute-about-virtual-network-gateways.md

@duongau

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

Okay will do!
Thank you,
Mekayla Moore
PM ExpressRoute

From: Jak Koke @.>
Sent: Thursday, September 7, 2023 2:54 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Mekayla Moore @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update expressroute-about-virtual-network-gateways.md (PR #114453)
@duongauhttps://github.com/duongau

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-teamhttps://github.com/orgs/MicrosoftDocs/teams/public-repo-pr-review-team
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/114453#issuecomment-1710814674, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BCNTIOKUT7HG2KMXLF3LEHLXZI7B5ANCNFSM6AAAAAA4POR6Z4.
You are receiving this because you were mentioned.Message ID: @.***>


Generated beta models and request builders using Kiota

This pull request was automatically created by the GitHub Action, create pull request.
The commit hash is e478bf4a638584cb54bb9dc896b25375614e42ee.
Important Check for unexpected deletions or changes in this PR.

Superseded by #14


Repo seems to be offline

I cannot access http://objectbox.net/beta-repo/
Trying to import objectbox 1.0.0
buildscript {

    repositories {
        google()
        jcenter()
    }
    dependencies {
        classpath 'com.android.tools.build:gradle:3.0.0-beta4'
        classpath 'io.objectbox:objectbox-gradle-plugin:1.0.0'
    }
}

allprojects {
    repositories {
        maven { url "http://objectbox.net/beta-repo/" }
        google()
        jcenter()
    }
}

AndroidStudio is giving me this message
Error:Could not find io.objectbox:objectbox-gradle-plugin:1.0.0.
Searched in the following locations:
    file:/Applications/Android Studio 3.0 Preview.app/Contents/gradle/m2repository/io/objectbox/objectbox-gradle-plugin/1.0.0/objectbox-gradle-plugin-1.0.0.pom
    file:/Applications/Android Studio 3.0 Preview.app/Contents/gradle/m2repository/io/objectbox/objectbox-gradle-plugin/1.0.0/objectbox-gradle-plugin-1.0.0.jar
    https://dl.google.com/dl/android/maven2/io/objectbox/objectbox-gradle-plugin/1.0.0/objectbox-gradle-plugin-1.0.0.pom
    https://dl.google.com/dl/android/maven2/io/objectbox/objectbox-gradle-plugin/1.0.0/objectbox-gradle-plugin-1.0.0.jar
    https://jcenter.bintray.com/io/objectbox/objectbox-gradle-plugin/1.0.0/objectbox-gradle-plugin-1.0.0.pom
    https://jcenter.bintray.com/io/objectbox/objectbox-gradle-plugin/1.0.0/objectbox-gradle-plugin-1.0.0.jar

I also cannot access the repo in the browser, it's giving me a "403 Forbidden" error.
Is the repo offline or am I doing something wrong?

The issue seems to have been resolved now, without changing my code access works


Search and replace does not find multiline strings

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:32:36.023Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

html-snippets
abu
0.1.0

project-manager
ale
0.23.0

path-intellisense
chr
1.4.2

bracket-pair-colorizer
Coe
0.10.14

xml
Dot
1.9.2

php-intellisense
fel
2.0.1

file-icons
fil
1.0.7

auto-close-tag
for
0.5.3

code-runner
for
0.8.4

output-colorizer
IBM
0.1.2

csharpextensions
jch
1.3.0

mssql
ms-
1.2.1

python
ms-
0.8.0

cpptools
ms-
0.14.2

csharp
ms-
1.13.1

PowerShell
ms-
1.5.1

team
ms-
1.122.0

debugger-for-chrome
msj
3.5.0

color-highlight
nau
2.3.0

material-icon-theme
PKi
3.1.0

vscode-icons
rob
7.18.1

code-spell-checker
str
1.5.1

vim
vsc
0.10.5

html-css-class-completion
Zig
1.14.0

Steps to Reproduce:
Description:
When searching for multi line strings it appears that the default behavior of the search is to strip newlines from the paste. This prevents the search function from matching the string and therefore prevents multi line search and replaces.
Steps to reproduce:

Copy code in "Code used" and paste into a new empty file in vs code several times.
Ctrl F and paste the code in Search.
0 matches found since newlines are stripped.

Text to be searched.

Search and Replace Box:

Code used:
`# tmp = ""
for r in relationships:
if r == relationships[-1]:
tmp = tmp + "\n\tnew ListItem() { Value = "" + r + "", Text = "" + r + "" }\n"
else:
tmp = tmp + "\n\tnew ListItem() { Value = "" + r + "", Text = "" + r + "" },"
out = "return new ListItem[] { " + tmp + " };\n"
print(out)`

Reproduces without extensions: Yes/No

/duplicate of https://github.com/Microsoft/vscode/issues/13155


build-toolchain.sh should include isl

build-toolchain.sh currently sets up the gcc build to include gmp, mpc and mpfr but not isl which is also a dependency of gcc.

related issue #312

I thought isl is optional?

@Jan200101 Are you using WSL1 or WSL2 (given you have linked that issue)?!

I thought isl is optional?

Doesn't seem to be from looking at the gcc configure.ac script.
https://gcc.gnu.org/install/prerequisites.html also doesn't mention it as optional (though it doesn't mention anything being optional, just that they could be).
On a CI runner I ran into Unable to find a usable isl.  See config.log for details.
I've managed to recreate this on a fresh Debian 12 install in a VM

Are you using WSL1 or WSL2 (given you have linked that issue)?!

Not using WSL at all, this occured on a Fedora based runner.
I linked the issue because it ran into the same error message

OK I think the reason I've never realized this is that isl is a dependency also of gmp. So if you install gmp through a package manager, you get isl. And even if you uninstall it to test the gmp source build, there's still isl in your system :)
I guess we need to amend the script to also download and copy isl then.

This has been implemented for Mac for now.


Add sdl2 package

Note that the package depends on sdl2 >= 2.0.3 via pkg-config.

This looks good, I'm just figuring out the system library issues before merging. If you know the best way to install system libraries for Ubuntu 12.04, that would be helpful.

Simply building and installing SDL2 from upstream has worked out for me. It should also be possible to bundle the .pc file for pkg-config and instruct it to link the library statically.

It would be far better if there was a prebuilt package or PPA available to
use, as we try to automate and standardize these things as much as possible.
On Thu Feb 12 2015 at 5:01:45 PM Gabríel Arthúr Pétursson <
<EMAIL_ADDRESS>wrote:

Simply building and installing SDL2 from upstream has worked out for me.
It should also be possible to bundle the .pc file for pkg-config and
instruct it to link the library statically.
—
Reply to this email directly or view it on GitHub
https://github.com/fpco/stackage/pull/444#issuecomment-74084341.

Here are some PPA: https://launchpad.net/~zoogie/+archive/ubuntu/sdl2-snapshots

Thanks, merged. I've just kicked off a nightly build with sdl2 included, hopefully all will work correctly :)


Add Table Builder inside Custom Field  Not work

Package
filament/filament
Package Version
v3.0.0
Laravel Version
v10.0.0
Livewire Version
v3.0.0
PHP Version
8.2.4
Problem description
I try to insert table builder inside Custom Field
i get this error Property [$table] not found on component
Field:
class TableField extends Field implements HasForms, HasTable
{
    use \Filament\Forms\Concerns\InteractsWithForms;
    use \Filament\Tables\Concerns\InteractsWithTable;

    public function getView(): string
    {
        return 'forms.components.table-field';
    }

    public function table(Table $table): Table
    {
        return $table
            ->columns([
                TextColumn::make('name'),
            ])
            ->query(User::query());

    }
}

View:
<x-dynamic-component
    :component="$getFieldWrapperView()"
    :field="$field"
>
    <div x-data="{ state: $wire.$entangle('{{ $getStatePath() }}') }">

        {{$this->table}}

    </div>
</x-dynamic-component>

Expected behavior
To work normally
Steps to reproduce
i try add this comand insid getView but not work
 public function getView(): string
    {
        $this->table =$this->table($this->makeTable());
        return 'forms.components.table-field';
    }

Reproduction repository
https://github.com/mstfkhazaal/test-table-field
Relevant log output
No response

👀

the TableBuilder inside a custom field works well (display rows and even reorder/field edit from a relationship are correctly saved on blur)
BUT it prevent the rest of the form to work :  the form 'submit' button doesn't trigger any network activity
demo : https://github.com/olivM/test-table-field
php artisan migrate db:seed
php artisan serve

-> http://<IP_ADDRESS>:8000/admin/users/1/edit


SimpleCov configuration

disable combined RSpec & Cucumber coverage reporting
exclude app/admin folder from coverage reporting
configure SimpleCov & Coveralls to both send coverage data to Coveralls
and to create a report locally

Wrong repo...