Remaining Koalas Series.str functions

As discussed in #281, this PR completes most of the remaining Series.str functions for manipulating strings in Koalas Series.
Functions not implemented:

cat() - returns scalar string, not a Koalas series
decode(), encode() - BinaryType not supported in pyspark < 2.4
extract(), extractall() - returns dataframe of varying dimensions
partition(), rpartition()- returns dataframe of varying dimensions
get_dummies() - returns dataframe of varying dimensions

Codecov Report

Merging #496 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #496      +/-   ##
==========================================
+ Coverage   93.71%   93.78%   +0.06%
==========================================
  Files          28       28
  Lines        3803     3875      +72
==========================================
+ Hits         3564     3634      +70
- Misses        239      241       +2

Impacted Files
Coverage Δ

databricks/koalas/typedef.py
80.5% <100%> (ø)
:arrow_up:

databricks/koalas/strings.py
98.37% <100%> (+2.29%)
:arrow_up:

databricks/conftest.py
92.85% <0%> (-3.58%)
:arrow_down:

databricks/koalas/__init__.py
89.65% <0%> (-3.45%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks @HyukjinKwon !

Thanks, @hjoo.


Invalid class name being generated (first character is number) causing styles to fail

It appears babel-plugin-styled-components is assigning component classnames based on folder/file path structure. For example I have a component
/14-offer-radbtn/index.js

that file contains a component
const HoverIndicator = styled.span`
    opacity: 0;
`;
const StyledLink = const StyledLink = styled.a`
    &:hover {
        ${HoverIndicator} {
            opacity: 1;
        }
    }
`;

const BA14OfferRadbtn = () => (
    <StyledLink>
        <HoverIndicator />
    </StyledLink>
);

export default BA14OfferRadbtn;

However the hover effect doesn't work. On closer inspection, it appears that the generated class name is invalid as it starts with a number and is being ignored by the browser

generated CSS rule:

It appears that the class name is generated by looking at the folder path of the file. As the containing folder begins with "14" the class name follows suit invalidly.
System:

OS: macOS High Sierra 10.13.4
CPU: x64 Intel(R) Core(TM) i7-6920HQ CPU @ 2.90GHz
Memory: 361.34 MB / 16.00 GB
Shell: 5.3 - /bin/zsh

Binaries:

Node: 8.9.4 - ~/.nvm/versions/node/v8.9.4/bin/node
npm: 5.8.0 - ~/.nvm/versions/node/v8.9.4/bin/npm

npmPackages:

babel-plugin-styled-components: ^1.5.0 => 1.5.1
styled-components: ^3.2.3 => 3.2.6

Expected Behavior
Valid CSS class name generated.
Actual Behavior
Invalid CSS classname generated, starting with a number, which is ignored by the browser

Hmm it seems we do have some code that handles this: https://github.com/styled-components/babel-plugin-styled-components/blob/master/src/visitors/displayNameAndId.js#L100
Maybe it's not being applied in all situations?

Hmm I've tried turning the SSR option on and off with no joy.
My setup is using "@storybook/react": "^3.2.18" with a custom .babelrc as follows
{
    "presets": ["env", "react"],
    "plugins": [
        "transform-class-properties",
        "transform-object-rest-spread",
        "styled-components",
        "macros",
        ["transform-runtime", {"polyfill": false,"regenerator": true}]
    ]
}

Give displayName: false a try for the moment.

Oh that works.

["styled-components", {"displayName": false}],

But at the expense of the obfuscated names

Yup. I'm looking into a fix for this issue in the meantime.


Avoid parameter assignment in circuit_to_instruction when there are no parameters

What is the expected enhancement?
The circuit_to_instruction function calls the method QuantumCircuit.assign_parameters with the explicit argument inplace=False. This is the expected behavior, since the conversion should not affect the original circuit.
However, when the given circuit is not parametrized, this is redundant. The returned circuit is not changed in any way in the rest of the conversion. Thus, we can avoid calling assign_parameters, and simply use the input circuit for the rest of the function. This will improve runtime significantly for bigger circuits.

This is a good point, thanks.  I wonder if we'd actually be better inverting the order of the whole circuit_to_instruction function - build up the definition first, and then call QuantumCircuit.assign_parameters(instruction.definition, inplace=True). I think that ought to involve fewer allocations in both cases, without needing the extra login in circuit_to_instruction to skip assign_parameter.
I'd be happy to accept a PR on this topic, if you'd like to work on it?


[kong] add missing root object type to CRD OpenAPI v3 schema

What this PR does / why we need it:
Currently all installed CRDs are throwing violations warnings due to missing root type values.
kubectl describe crd kongconsumers.configuration.konghq.com
...
Status:
  Accepted Names:
    Kind:       KongConsumer
    List Kind:  KongConsumerList
    Plural:     kongconsumers
    Short Names:
      kc
    Singular:  kongconsumer
  Conditions:
    Last Transition Time:  2021-01-20T14:41:12Z
    Message:               spec.versions[0].schema.openAPIV3Schema.type: Required value: must not be empty at the root

See the following for more information on this topic:

https://github.com/kubernetes/kubernetes/issues/81445
https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definitions/#specifying-a-structural-schema

Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
OpenAPI will not be published to kubectl explain unless we also set preserveUnknownFields: false or set the apiVersion: apiextensions.k8s.io/v1 (it is v1beta1 today).
kubectl explain kongingress
KIND:     KongIngress
VERSION:  configuration.konghq.com/v1

DESCRIPTION:
     <empty>

See https://github.com/kubernetes/kubernetes/issues/95702 for more info on this topic.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] PR is based off the current tip of the next branch and targets next, not main
[x] New or modified sections of values.yaml are documented in the README.md
[x] Title of the PR and commit headers start with chart name (e.g. [kong])

Thanks @Starefossen for your contribution.
Could you please make this change instead to the CRDs defined in this file in the Kong/kubernetes-ingress-controller repo? You'll also need to run the hack/build-single-manifests.sh scritpt there when you're done.
We have some changes there not yet promoted to the helm chart. Also, it's the KIC repo that's the source of truth for CRDs.


Investigate the option of offloading RBAC to the database when fetching records

Currently, record access is checked on Corteza which unfortunately prevents us from optimising operations like counting records.
Figure out if possible and how we could do it.
Identified issues:

different resource specificity
contextual roles
resource hierarchy (planned feature)

shmuel suggested we could have different levels to this -- the more trivial checks could be added to the query and the more complex  checks would be done later by Corteza

@tjerman ,
I'm open to taking a crack at this. I would love some guidance if possible.
Thanks!

Without any guidance, my approach would be the following:
The rule of marginal gains

Selective and clean optimization. Choose the optimizations that will maximally improve performance.
Use benchmarks before deciding which things to modify
Don't change things unless it either improves performance or would be inconsistent to leave it unchanged with the changes I do make. e.g. only target bulk read compose records that are not subject to contextual roles.

High level strategy

Upon request for a resource, determine if the request is bulk. If the request is not bulk, or not part of a bulk context (e.g. fetching related data), then use today's version of permission checking
Define a set of permission checking abilities per rbac rule in combination with data source (e.g. bulk read in combination with an sql data source is something we can optimize)
If any rule set can use the optimized version, run the optimized rules first since it can greatly reduce the set of resources that the unoptimized rules need to iterate over

Lower level strategy

Transform rbac rules into sql joins so that one "list" request produces approximately 1 database query as opposed to n database queries where n is the size of the dataset to be displayed
Allow those same sql queries to be fed into a subquery of an aggegrate query that will return number of rows so those could be displayed to users as well without having to process the whole module server-side.


Added a --zoom option for setting default zoom

These changes add the ability to declare a default zoom factor for an application with the --zoom flag. While it doesn't fix #125 directly, it at least seems like a better workaround than the one mentioned in the issue.

Thanks so much! Could you update the documentation as well to specify this new flag?

I have now added information about the flag to docs/api.md. Is there anything else that should be updated?

Looks good, thank you!!


[WIP] Add some missing task specifications

Adds specifications for some tasks that were missing them.

Changing to WIP because I think I might need to fix the specifications for translation tasks.

I think this should be OK now @pfliu-nlp !

I'm going to go ahead and merge this as it'll make testing translation support in ExplainaBoard a bit easier...


Uat feedback

Fix #94
Fix #95
Fix #96 (peek-a-boo fix only - mobile description fix still needed - separate PR for that as part of the Slider issue)
🔗 Test URLs:

Before: https://main--caesars-careers--hlxsites.hlx.live/careers/
After: https://uat-feedback--caesars-careers--hlxsites.hlx.live/careers/

📝 Description:
UAT Feedback - mostly link fixes, Heading sizing and padding issues. Also added Peek a boo fix for sliders and support for multiple buttons on the same line in cards

Fyi - There seems to be a slider that contains text that it should not (it's not on the original website):


Missing caveat that AAD cannot be enabled on an existing cluster

Hi all,
Documentation does not make it clear that you cannot add AAD integration to an existing cluster which was not created with AAD integration.
Is it possible to add this as a caveat/warning, as I am pretty sure that was present on the previous version we did call this out (to prevent any confusion)?
Thanks!
Adam

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

ID: bc86c3af-f6c7-4c48-e157-54abbc5958ff
Version Independent ID: 3611b478-6df6-f5cd-563d-2fff6cfec9ef
Content: Integrate Azure Active Directory with Azure Kubernetes Service (legacy) - Azure Kubernetes Service
Content Source: articles/aks/azure-ad-integration-cli.md
Service: container-service
GitHub Login: @TomGeske
Microsoft Alias: thomasge

thanks for your feedback. Yes, it should be called out in our limitation section. Just made a PR.
You may want to consider our new managed AAD integration experience. It allows you to enable AAD integration on existing RBAC enabled cluster. https://aka.ms/aks/managed-aad

going ahead closing this one. Feel free to keep commenting if you have further feedback. Doc change should go live latest in 24 hours.
#please-close

Thanks @TomGeske for the doc update. :)

Thank you @TomGeske !
Should we add the same caveat here for managed AAD:
https://docs.microsoft.com/en-us/azure/aks/managed-aad
If so, shall I raise a new issue on that page instead?

@AdamSharif-MSFT
Enabling managed AAD Integration on existing cluster with RBAC enabled is supported. Do you have any issues doing it?


fixed missing flag for publishing port 6006 in docker run step

docker run -it -p 8888:8888 p 6006:6006 tensorflow/tensorflow bash searches for docker image p.  I updated the p to -p so that the container publishes port 6006 as desired.

Thank you!


Add precision on README.md for Windows support

Just a few notes since I am a Windows guy. Yes.
I wanted to try out discurses on Windows and I had a sad surprise with it. The reason is I wanted to hack on the project from my main OS and since I cannot try out my modifications on-the-go through Windows, I'll have to use my Linux system instead (which is way better anyway!).
So here's just a not to point out that it is useless to try to install discurses on Windows.

yeah, this is kinda unfortunate. I think I hadsuccess installing it in the linux subsystem, you may might want to take a look at that


Install overwrites custom rancher-system-agent.env file

It appears that the installer is overwriting the .env file at
https://github.com/rancher/system-agent/blob/main/install.sh#L804
This is causing issues with a few users who are behind a corporate proxy. Is there a way to set a proxy to be used when defining a cluster?

The expectation is that proxies required by the node are defined by the user shell/service shell, and are set in a global location on the node. If the user has the corresponding HTTP_PROXY etc. environment variables set, they should be set for the system agent.

It's important to note that Rancher doesn't really support communication-through-proxy in the first place, due to potential performance implications. We don't want to open WATCH requests through a corporate proxy for example.


CupertinoTabScaffold current index

Set tab bar current index if exist
Fixes: flutter/flutter#15969

I signed it!

Ah, sorry, didn't notice you were already working on this.
I ended up doing the same thing. Approach looks good. Just needs docs updates and tests.
Please take a look at #16040


getButtonProps is not a function

As of version v1.31.1 we get getButtonProps is not a function.
There supposedly is an alias since the change tonight to getToggleButtonProps, but something is not working as expected, looking through the code right now, will update if I find the problem.

The aliasing wasn’t not done properly (ref to Line 552) as the getButtonProps don’t get passed down along with getToggleButtonProps to the render function.
This change was introduced as minor patch and need to be rectified as soon as possible.
Would you like to makeapullrequest.com @gsimone?

Working on the PR @donysukardi

I'm terribly sorry about that. Thank you for the quick PR!


Unable to access the website

Hey there fellows,
For some reason, I'm unable to access the documentation page on your website.
Docs.

Problem fix


Request for Openpunk

https://openpunk.com
my site may not be updated yet, as it updates hourly but the commit for the added webring logo & link is here:
https://git.openpunk.com/OpenPunk/OpenPunk-Hugo/commit/55683b0e2f47a0688710eb3884db66357d29a1ce

Hey CPunk!
It's nice to see a PR from you, but your website in its current state does not qualify for the webring. There are no projects documented on-site(besides the site-builder itself), also using such a standard hugo template goes against the hand-crafted requirement of the webring. Sorry, feel free to open another one when the site is further down the line :)


Eloquent model relationships are loaded after attributes when unserializing in queue

Laravel Version: 6.x
PHP Version: 7.4
Database Driver & Version: MySQL 8

Description:
When passing an Eloquent model to the queue Job with some loaded relationships, it tries to reload them before the model attributes are loaded.
In my use case, it was a problem, because I used some attributes in the relationship function (maybe I was wrong to assume it should work?)
So, since the relationship was loaded already, using empty attributes, and was equal to NULL, then later in the code, when I tried to use it, it was not reloaded and caused issues.
Steps To Reproduce:
Have relationship something like I had:
class MyModel extends Eloquent {
        .....
        public function myRelation(): HasOneThrough
       {
            return $this->hasOneThrough(
                Related::class,
                Through::class,
                'id',
                'through_table_id',
                'related_table_id',
                'id'
             )
                ->where("some_column", $this->some_attribute);
       }
}

Load this relationship in the model, and then pass the model as a parameter to the Queue Job class.
Explanation
When the Job starts, the model is reloaded from DB, and all relationships too.
But, in some reason, when i's loading this relationship, $this->attributes equals [0] on the model.
This causes the relationship not to load, but still be created on the object as $this->myRelation = null.
Should be
(per my assumption) - loading relationships after $this->attributes is filled

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to open up a new issue with a link to the original one and we'll gladly help you out.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!


Update Rspamd and base image

Proposed Changes

(Describe the changes and rationale behind them)

Related Issues

(Github link to related issues or pull requests)

Does not build


Improve UI/UX

[ ] Increase font size
[ ] have different modes

e.g. fix certain things to be hidden
timer
input answers


Gustavolbs/feat/create user register page

Description
Creates user register page

Creates Form
Creates Page
Adds validation
Adds Mirage

Related Issues
Closes Criação de tela de cadastro de usuário
Closes Publicar frontend
Progress
Use checkboxes to show that you did and what you have to do. For example:

[x] New endpoint created;
[ ] Update organizations;
[ ] Create tests;

How to test it

Access homepage
Click at "Cadastro" at navigation bar. You should be redirected.
You can test filling the fields with invalid data or with the Happy Path. Both should be working
Register will not work for now, since we don't have API endpoint yet, but you should be available to test locally thanks to MIrage.js.

Visual reference

Change logs

Updates eslint
Adds packages
Creates SignUpForm
Adds Mirage.js
Creates SignUp Page
Creates string validation helper

It is showing two snackbars
Screencast 2022-11-10 15:05:07.webm

It is showing two snackbars
Screencast.2022-11-<IP_ADDRESS>.webm

No, it's just one snackbar, but once you click it removes the message and changes the severity, so it appears to be a second snackbar. I'll fix this.

Fixed your comments, please, take a look again


TEST: travis now tests 1.0, 1.1 and nightly

@sglyon this release doesn't fix anything. master was compatible with 0.7, 1.0, 1.1.
I think the problem is I never released it. I just did with a minor number increment.

Actually, it does fix something: update travis and drop 0.7 compat, which I'm happy to do. So I'm merging that one.

Ok great, thanks for following up and releasing


Fonts are not applied when using AppCompat and a theme

When using Calligraphy with AppCompat, by default you cannot apply a font to some widgets such as Button and EditText using a theme.
For example, if you define the following theme, your Buttons will use the default font:
<style name="Theme.FancyTheme" parent="Theme.FancyTheme">>
    <item<EMAIL_ADDRESS></style>

<style name="Widget.Fancy.Button" parent="android:Widget">
    <item name="fontPath">font/fancy_font.otf</item>
</style>

This is because AppCompat injects itself in the layout inflation process and replaces some components such as Button with AppCompat versions (e.g. AppCompatButton).
As a workaround, you can register the AppCompat versions of these widgets as having "custom styles" in Calligraphy like so:
CalligraphyConfig.initDefault(new CalligraphyConfig.Builder()
                        .setDefaultFontPath("fonts/fancy_font_default.otf")
                        .setFontAttrId(R.attr.fontPath)
                        .addCustomStyle(AppCompatButton.class, android.R.attr.buttonStyle)
                        .build()
);

Since this isn't an immediately obvious problem or solution, I believe that Calligraphy should register these by default, as it does with the framework Button classes and such.
This also doesn't work with text appearances for theme styles. For example, if I reference a text appearance style with a custom font  from Widget.Fancy.Button, the above workaround does not work- the button will still have the default font.

Well the latter issue is separate I would raise that as a separate issue. As for the former. No. That would create a dependency in the project that would require checking if the AppCompat lib is available when not everyone would include it.
I will take a PR mentioning it in the docs though.

You are already including AppCompat in the project and are checking if the user is using it.
I can add documentation surrounding this a little later though if that is the preferred method.

@bherbst Where do you register the "initDefault" function? Is it in the overwriting Application class?

@hongzhou85 Yes, I registered it in my Application class.

should be fixed now in #191 there is a 2.1.1 snapshot with those changes for you to check.

Guy How can i get the snapshot for 2.1.1

@Nammari  add:
maven {
            url "https://oss.sonatype.org/content/repositories/snapshots"
        }

to your gradle buildScripts{repositories{` closure.


Add GameDig monitor

⚠⚠⚠ Since we do not accept all types of pull requests and do not want to waste your time. Please be sure that you have read pull request rules:
https://github.com/louislam/uptime-kuma/blob/master/CONTRIBUTING.md#can-i-create-a-pull-request-for-uptime-kuma
Tick the checkbox if you understand [x]:

[x] I have read and understand the pull request rules.

Description
This pull request adds support for using GameDig as a monitor, this allows rich game server monitoring rather than the apparently unreliable method of using the TCP monitor.
Fixes #(issue)
#1053
#479
#525
Type of change

User interface (UI)
New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the style guidelines of this project
[x] I ran ESLint and other linters for modified files
[x] I have performed a self-review of my own code and tested it
[ ] I have commented my code, particularly in hard-to-understand areas
(including JSDoc for methods)
[x] My changes generate no new warnings
[ ] My code needed automated testing. I have added them (this is optional task)

Screenshots (if any)
Please do not use any external image service. Instead, just paste in or drag and drop the image here, and it will be uploaded automatically.

https://github.com/gamedig/node-gamedig/blob/master/games.txt
This monitor will be perfect if the game field is a dropdown instead of a free text field.

Autofill in the port may not be necessary. When you tell gamedig the game type, it will automatically resolve the port to use (defaults and through SRV records).
A game dropdown would be great though, I guess the dropdown list would have to be an object with an ID and a displayname so it is easy to select a game by name without knowing the ID.

A gamedig monitor would we great.
@WhyKickAmooCow Are you working on the game dropdown?
gamedig's games.txt already contains an id and a human readable name, the id is what you'd pass to the query as type.
Regarding the port: I'd say letting gamedig guess the port every single time is not ideal. Especially because gamedig uses a port offset for some games, which it always applies (see lib/QueryRunner.js#L33). For a Battlefield 4 server using query port 47200, gamedig

attempts to query the server on given port (47200) + query port offset (22000)
attempts to query server on given port (47200)
attempts to query server on default game port (25200) + query port offset

Since server admins generally know their query port settings (or can find out rather easily). Thus I would say a much better option is to fill the port field with a default (which can be determined based on data from games.txt) and use the givenPortOnly option for the query. If the port is wrong, the monitor runs a single query that fails. If the port is correct (and the game server is up), the monitor runs a single query that succeeds.
Numer of queries sent (Battlefield 4 example)

givenPortOnly
Server up
Server down

false (default)
2
3

true
1
1

A gamedig monitor would we great.
@WhyKickAmooCow Are you working on the game dropdown?
gamedig's games.txt already contains an id and a human readable name, the id is what you'd pass to the query as type.
Regarding the port: I'd say letting gamedig guess the port every single time is not ideal. Especially because gamedig uses a port offset for some games, which it always applies (see lib/QueryRunner.js#L33). For a Battlefield 4 server using query port 47200, gamedig
* attempts to query the server on given port (47200) + query port offset (22000)

* attempts to query server on given port (47200)

* attempts to query server on default game port (25200) + query port offset

Server admins generally know their query port settings (or can find out rather easily). Thus I would say a much better option is to fill the port field with a default (which can be determined based on data from games.txt) and use the givenPortOnly option for the query. If the port is wrong, the monitor runs a single query that fails. If the port is correct (and the game server is up), the monitor runs a single query that succeeds.
Numer of queries sent (Battlefield 4 example)
givenPortOnly 	Server up 	Server down
false (default) 	2 	3
true 	1 	1

Thanks, maybe I try to add today.

Nice, it is more easy to use with the dropdown now.

Really nice! Do you plan on adding the givenPortOnly option to the query? I'll gladly open a PR for that if need be, but it seems a bit overkill for one line of code.

Oh I forgot to add this. Should be ok now: https://github.com/louislam/uptime-kuma/commit/9cc3bd0de416bb12dcb1b156953b203b2fcd3fa8

Just seen this has been added. Great news. Thank you


clj-sqs-ext should use a single client object to share http connections

Story details: https://app.clubhouse.io/motiva/story/7713

fixes #112


Routing against oneway on oneside - both direction cycle track.

It looks like there are problems with routing on cycleway=track against direction on oneway streets.
Tagging:
highway=primary
oneway=yes
cycleway:right=track
cycleway:right:oneway=no
(oneway:bicycle=no)
Observation:
Bicycle routing is only working in the direction of the oneway road.
cycleway:right:oneway=no do not show any effect !!!    I think this should be enough for 2 direction cycle routing!
(oneway:bicycle=no also have no effect but maybe the dataset is still not updated after 2 days.)

https://graphhopper.com/maps/?point=52.423377%2C9.444177&point=52.424113%2C9.451455&profile=bike&layer=Omniscale
https://www.openstreetmap.org/directions?engine=graphhopper_bicycle&route=52.42365%2C9.44420%3B52.42395%2C9.45180#map=17/52.42313/9.44668

https://wiki.openstreetmap.org/wiki/Tag:cycleway%3Dtrack

Since 2020 there is an addional wiki page: https://wiki.openstreetmap.org/wiki/Key:cycleway:right:oneway
Altough it states:

If the way has oneway=yes and you are allowed to cycle against that oneway flow add also oneway:bicycle=no to make it easier for bicycle routers to see that the way can be used in two directions.

I think that we could support cycleway:right:oneway=no only as well quite easily as well.

osm documentation: https://wiki.openstreetmap.org/wiki/Key:cycleway:right:oneway

oneway:bicycle=no solves the routing problem but as expected (and imO absolutely correct) causes a wrong visualization in CyclOSM.  Now it looks, like the carriageway can be used by bicycle in opposite direction.
It would be great if cycleway:oneway=no, cycleway:left:onway=no, cycleway:right:oneway=no, cycleway:both:oneway=no would be implemented as an exception from general oneway=yes.


Generate docs automatically

Committing doc rebuilds manually is getting really old. It's also contributing to bloat of the repo.
Acceptance criteria: there is an automated build and publish of docs (i.e. with CircleCI or GitHub Actions) so that changes to the restructured text / docstrings don't need doc rebuild commits and we can remove/git-ignore the built docs.

Follow-up: I just had the the realization that not doing this is the best way to go. The reason for it is that building and committing the HTML docs at a given version provides a snapshot of the docs for those cases where someone has to use an older version (i.e. a system still stuck on Python 2.7 or 3.5).
In such a case, it won't matter if the docstrings and Sphinx config has some incompatibility with Sphinx at that version (I ran a very similar issue frequently before Hashicorp's knowledge base started featuring versioned documentation) because the portable self-contained HTML docs are already there and there is no need to run any commands to generate it.


Request SCHED_RR using CAP_SYS_NICE and add Python to nix dev shell

Describe your PR, what does it fix/add?

Add missing python3 dep to nix develop
Change Hyprland process scheduling strategy to SCHED_RR using pthread

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Also require the maintainers to add SYS_CAP_NICE capability to the package
Is it ready for merging, or does it need work?
I don't think that adding python3 directly into devShell is a good solution, but, works.

why

why

It makes Hyprland more responsive when the computer is under CPU stress, and python is required because libudis86 needs it installed
Example:
$ nix develop
$ meson setup build
# ...
| -- Could NOT find Python3 (missing: Python3_EXECUTABLE Interpreter)
| Building itab.c/itab.h...
| -- Configuring done (1.3s)
| CMake Error at libudis86/CMakeLists.txt:30 (add_library):
| Cannot find source file:
| itab.c

| Tried extensions .c .C .c++ .cc .cpp .cxx .cu .mpp .m .M .mm .ixx .cppm .h
| .hh .h++ .hm .hpp .hxx .in .txx .f .F .for .f77 .f90 .f95 .f03 .hip .ispc

| CMake Error at libudis86/CMakeLists.txt:30 (add_library):
| No SOURCES given to target: libudis86
# ...

https://blog.martin-graesslin.com/blog/2017/09/kwinwayland-goes-real-time/
https://github.com/swaywm/sway/pull/6994

See also https://github.com/swaywm/sway/pull/6994

Replacing SCHED_RESET_ON_FORK with pthread_atfork (like Sway) makes it work on FreeBSD with mac_priority(4). For example, cursor in Hyprland no longer decelerates when running poudriere in background.
Sway copypasta
--- a/src/init/initHelpers.cpp
+++ b/src/init/initHelpers.cpp
@@ -4,12 +4,36 @@ bool Init::isSudo() {
     return getuid() != geteuid() || !geteuid();
 }

+static void child_fork_callback(void) {
+    struct sched_param param;
+
+    param.sched_priority = 0;
+
+    int ret = pthread_setschedparam(pthread_self(), SCHED_OTHER, &param);
+    if (ret != 0) {
+        Debug::log(WARN, "Failed to reset scheduler policy on fork");
+    }
+}
+
 void Init::gainRealTime() {
-#ifdef SCHED_RESET_ON_FORK
-    const int   minPrio = sched_get_priority_min(SCHED_RR);
-    sched_param param   = {.sched_priority = minPrio};
-
-    if (pthread_setschedparam(pthread_self(), SCHED_RR | SCHED_RESET_ON_FORK, &param))
-        Debug::log(WARN, "Failed to change process scheduling strategy");
-#endif
-}
\ No newline at end of file
+    int prio = sched_get_priority_min(SCHED_RR);
+    int old_policy;
+    int ret;
+    struct sched_param param;
+
+    ret = pthread_getschedparam(pthread_self(), &old_policy, &param);
+    if (ret != 0) {
+        Debug::log(WARN, "Failed to get old scheduling priority");
+        return;
+    }
+
+    param.sched_priority = prio;
+
+    ret = pthread_setschedparam(pthread_self(), SCHED_RR, &param);
+    if (ret != 0) {
+        Debug::log(WARN, "Failed to set scheduling priority to %d", prio);
+        return;
+    }
+
+    pthread_atfork(NULL, NULL, child_fork_callback);
+}

if (pthread_setschedparam(pthread_self(), SCHED_RR, &param)) {
+        Debug::log(WARN, "Failed to change process scheduling strategy");
+        return;
+    }

hm, I think that if-guard SCHED_RESET_ON_FORK can be removed, because pthread_atfork method also works on Linux, but SCHED_RESET_ON_FORK looks more integrated with Linux, what do you think?

Up to @vaxerski. pthread_atfork gives slightly more control, and dropping SCHED_RESET_ON_FORK can reduce maintenance while avoiding build/runtime bugs on non-Linux.

I don't care

whats the status on this? ready or not

@fufexan


Added - Select Pixel font

Added code to allow Grand9KPixel pixel font to be selected via FONT Device Variable
resolves balenalabs/inkyshot#6

@balena-ci rebase

@SamEureka merged as well, thanks again for the contributions :)


Added code to generate permalink

This may probably need to be moved to a better location in index.html... Or maybe add a share button...

This may probably need to be moved to a better location in index.html... Or maybe add a share button...

Yeah, this is workable now, but not particularly visually appealing. I'm not great at that side of things.. I'll poke someone else in the discord, see if anyone wants to take a stab at styling it better

I can take a stab at styling this up a bit differently. I think the link should be up in the form probably above the graph - maybe with a copy to clipboard button next to it.

@joseamirandavelez Thanks for kicking this off!
Permalink button is now live :)

@joseamirandavelez Thanks for kicking this off!
Permalink button is now live :)

It seems youn modified the code, the permalink looks like this now:
https://turnipprophet.io/?prices=93...115.127.95.......&pattern=3
The zeros are not written. Still works, though.

Did you add the code on a different fork? I dont see my name in the contributors list.


fix(size): change density to size

First stem in moving to using sizing tokens

add this to the list of removals (enlarged number field)


fix(esbuild): disable identifier minification for source maps

closes https://github.com/esbuild-kit/core-utils/issues/7

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update package name "ubuntu 17.10" -> "ubuntu 18.04"

Ubuntu 17.10 reached End of Life.
The supported upgrade path from Ubuntu 17.10 is via Ubuntu 18.04 LTS.
Currently package file name is "powershell-preview_6.1.0~preview.4-1.ubuntu.17.10_amd64.deb".

I think we already added 18.04.  We should add 18.10

Please!

Maintaining packages for the non-LTS releases has been painful.  We are adding community support for 18.10 and 19.04 via the snap package.


Autojoin filter when the predicate isn't freemappable from the source

Fixes https://github.com/quasar-analytics/quasar/issues/2366
Completes part of https://github.com/quasar-analytics/quasar/issues/3021

@alissapajer Looks like even more tests are passing ;)

@wemrysi hm, so demo/nestedSelectWithReduce.test didn't pass on the previous build, but passed on this build for spark and ML. I can't get it to pass locally either though; it's returning results but incorrect ones. It's very strange that it passed for four connectors this last build. If it's going to be flaky, then unfortunately I think we have to skip it :/

@alissapajer Oh, bummer 😞

@wemrysi this is ready to go!


is numeric check

Tasks:

[x] done
[ ] todo
[ ] got feedback

__Informations: product save bugfix

Q
A

Bug fix?
yes

New feature?
no

Tests pass?
yes

Fixed tickets
https://github.com/massiveart/POOL-ALPIN/issues/1277

Doc
none

choose a meaningful PR title and use [FEATURE]/[BUGFIX]/...

unfortunately this does not solve the real issue and is just a quick fix, since the real problem is that the entity seems to return strange id's.
ACK anyway


Logging debug, verbose, and warning output.

Package dependencies and other information are displayed only through debug level messages. This information would be useful to have without cluttering the terminal output. This patch creates a log file for additional debug, verbose, and warning output during bulk runs.

By the way there is a bigger problem here in that the entire bulk console is not logged anywhere. The plan has been that normal usage of bulk would shift over to poudriered. That effort has stagnated a bit though. I do agree we need an easier way to get the debug log and the console log. Having the console log available from the web interface has been requested a few times.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Name changed not updated.

Windows Defender Exploit Guard should be changed to Microsoft Defender Exploit Guard in the GPO section.

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

ID: 673970ca-2f79-a288-978c-ceb4845c5062
Version Independent ID: 2a58449f-b0d4-bf17-1209-d7b8ec0481b6
Content: Turn on network protection
Content Source: microsoft-365/security/defender-endpoint/enable-network-protection.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@senorleung Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9975
Once the author approves, the changes will be live.
Thanks
Sri

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


Make .. a pattern

Per https://github.com/rust-lang/rfcs/pull/2707.

Implemented in dab8097c143a17f1ad99ca4d4d59165467683cf0.


PasswordValidator Null Argument Exception Bug

Even though I set the passwordRequireLength at ConfigurationService, when I call the PasswordValidator in the controller, null argument exception will throw if the password is empty.
private UserManager<AppUser> _userManager;
        private IUserValidator<AppUser> _userValidator;
        private IPasswordValidator<AppUser> _passwordValidator;
        private IPasswordHasher<AppUser> _passwordHasher;

        public AdminController(UserManager<AppUser> userManager, IUserValidator<AppUser> userValidator, IPasswordValidator<AppUser> passwordValidator, IPasswordHasher<AppUser> passwordHasher)
        {
            _userManager = userManager;
            _userValidator = userValidator;
            _passwordValidator = passwordValidator;
            _passwordHasher = passwordHasher;
        }

IdentityResult validPassword = await _passwordValidator.ValidateAsync(_userManager, user, password);
                if (!validPassword.Succeeded)
                {
                    AddErrorsFromResult(validPassword);
                }
                else
                {
                    user.PasswordHash = _passwordHasher.HashPassword(user, password);
                }

ArgumentNullException: Value cannot be null.
Parameter name: password
Microsoft.AspNetCore.Identity.PasswordValidator.ValidateAsync(UserManager manager, TUser user, string password)

In the controller action did you first check for ModelState.IsValid? If not, then it is probably not safe to call into Identity to do the password checks because the data entry was already invalid.

Hi. We're closing this issue as no response or updates have been provided in a timely manner and we have been unable to reproduce it. If you have more details and are encountering this issue please add a new reply and re-open the issue.


Replace "EIGIBILE USERS" with "ELIGIBLE USERS"

Mispelling with "EIGIBILE USERS" instead of "ELIGIBLE USERS"

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

ID: b4bb8dcf-9211-591e-4ddc-a82ba53b2807
Version Independent ID: 4b501c46-bd2e-457d-c891-2ebea3e44967
Content: How to deploy Dynamics 365 App for Outlook
Content Source: ce/outlook-app/deploy-dynamics-365-app-for-outlook.md
Service: dynamics-365-cross-app
GitHub Login: @mduelae
Microsoft Alias: mkaur

Thanks for the feedback. I fixed the typo. You should see the change within 15 minutes.
Thanks,
Mints


修改版本

Describe what this PR does / why we need it
Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Spring Cloud Gateway Version is 2.1.2.RELEASE in Greenwich.SR2, refer https://repo1.maven.org/maven2/org/springframework/cloud/spring-cloud-dependencies/Greenwich.SR2/spring-cloud-dependencies-Greenwich.SR2.pom.

the latest version of Spring Cloud Gateway is 2.1.2.RELEASE, refer: https://mvnrepository.com/artifact/org.springframework.cloud/spring-cloud-gateway-dependencies


Encountered warning:TF_REPEATED_DATA run RNE in Neotic

Hi, when i run RNE in Neotic i encounted the warning : "TF_REPEATED_DATA ignoring data with redundant timestamp for frame base_footprint at time 61.723000 according to authority unknown_publisher". Although it does not affect the operation of the program, I would like to know what the problem is. It may be a version problem. Do you have any ideas to solve this problem?

Hey, this seems to be a problem with Noetic and the geometry package rather than this package according to this ROS answers thread.
Further discussion can also be found here.
So, I would say, nothing to worry about. :)


Travis CI Support

Enable Travis CI builds for the unit tests for the Python versions we support (2.6+, 3.3+)

Added configuration at https://travis-ci.org/amznlabs/ion-python for Ion Python to be enabled.
Configuration tested on my user fork in #21 and can be seen at https://travis-ci.org/almann/ion-python

Implemented.


Refactor Args

This is a part of #718
Goals:

Unification of yaml entities and cli options as both do the same there was no reason to keep them separately.
Reduction of redundant code between AndroidArgs and IosArgs.
Convertion of AndroidArgs and IosArgs to data classes.
Extraction of factory methods and validators.

Test Plan

How do we know the code works?
As refactor  of cli options, yaml config, AndroidArgs and IosArgs. It doesn't bring any features or changes so the existing unit tests should cover possible regression.

Checklist

[x] Documented
[x] Unit tested
[ ] release_notes.md updated

Codecov Report

Merging #831 into master will increase coverage by 1.05%.
The diff coverage is 93.41%.

@@             Coverage Diff              @@
##             master     #831      +/-   ##
============================================
+ Coverage     80.09%   81.14%   +1.05%
+ Complexity      674      646      -28
============================================
  Files           151      165      +14
  Lines          3084     3235     +151
  Branches        442      461      +19
============================================
+ Hits           2470     2625     +155
- Misses          337      342       +5
+ Partials        277      268       -9


[Bug]: Disallow never type wild-card-binding-pattern

Description
As per this statement mentioned in the spec , It is an error if the type-descriptor in the typed-binding-pattern is never., We need to disallow never type wild-card-binding-pattern.
Steps to Reproduce
function bar() returns never {
   panic error("Error");
}

public function main() returns error? {
   never _ = bar(); // this should be an error
}

Affected Version(s)
Ballerina 2201.2.0
OS, DB, other environment details and versions
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

This seems to be fixed. The given code snippet produces the following error message.
expression of type 'never' or equivalent to type 'never' not allowed here(BCE3033)

Closing the issue because this is already fixed in U8.


Fix intersphinx refs for various packages

The intersphinx references for a few packages are missing or incorrect (don't point to the directory or URL containing the objects.inv file). We need to go through the documentation, find the broken links, and fix the intersphinx call in docs/conf.py.

Issue moved to CosmiQ/solaris #20 via ZenHub


fix assert during compact IDs pass

During the compact IDs optimization pass, the result IDs of some
basic blocks can change. In spite of this, GetPreservedAnalyses
indicated that the CFG was preserved. But the config relies on
the basic blocks having the same IDs. Simply removing this flag
resolves the issue by preventing the config check.
However, the config retrieval would update the config if this bit
is set. So it's not necessary to skip the config test even when it
has been changed. So I've removed that early exit too
Addresses https://github.com/KhronosGroup/SPIRV-Tools/issues/1648

I'll make a suggestion on how to do the test tomorrow.  You can build the analyis using a call in the ir context before running the pass.

Heh. seems @s-perron ended up contributing more than I did anyway. Thanks for humoring me just the same. :smiley: I'll look into testing as well. I would like to get this in ASAP though since it's affecting testing of dxc. So maybe we can add the test separately.


Escape dollars in names to prevent env-var expansion

Dollars in filenames need to be escaped so they aren't interpreted as variables for expansion

Having $ in filenames is something that is common in Remix (dynamic segments)

Better to escape other special chars, e.g. %, \ as well using fnameescape(), see https://github.com/hrsh7th/cmp-cmdline/pull/122


I do have signed the CLA and have an email address added to my GitHub account

URL to the linked Repo/Org:
https://github.com/Semmle/ql/pull/3090
Steps to reproduce the problem:
I tried to submit a pull request to https://github.com/Semmle/ql but got the error showing "beichent seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account."

I did accept the CLA as shown in the screenshot below:

I still got the warning below even after clicking "recheck"

I didn't see the blue question mark to the right of my username in the guide

Please help. Thanks.

Hi @luchua-bc  , Thank you very much for reaching out to us.
It looks like this commit "Address sensitive info logging"  is not recognised by GitHub because you need to add the email address to GitHub which you used for this commit in your local git configuration.

@luchua-bc here is the email address, you have used for the commits<EMAIL_ADDRESS>

Thanks a lot @KharitonOff. The issue was resolved after I changed the email associated with my commit.


latest version (1.6.2) not published on npm

the release was created 2 weeks ago but it is not on npm:
https://www.npmjs.com/package/@discord/embedded-app-sdk?activeTab=versions

Where are you seeing the 1.6.2 release? I'm not finding it in tags or releases on github.


UnsupportedJavaRuntimeException.javaDeprecationWarning() should fail gracefully

The UnsupportedJavaRuntimeException.javaDeprecationWarning() method calls JavaVersion.current() which throws a runtime exception in previous Gradle versions when executed on Java 9. This is the root cause why the Tooling API lost backward compatibility on that platform (#3317).
Expected Behavior
Instead of throwing an exception, the method should log the parse error when an unknown Java version is found.
Current Behavior
The current state is fine but not future-proof. If a new version scheme is introduced by Java then #3317 will happen again.

We can either fix this particular problem or adjust JavaVersion such that it returns an UnknownVersion if the version string parsing fails.

Oops, this is closed automatically by comments in my commits.


IDEA plugin not supporting @With annotation for Java 16 records

Short description
I have a java16 record with an @With annotation. When trying to use any of the generated with...() methods, the plugin tells me they don't exist, even though it compiles just fine.
Expected behavior
The plugin should show me the withLength(...) method exists. Instead it tells me it doesn't.
Steps to reproduce
   @With
    record Rectangle2(int length, int width) {}

    var rect = new Rectangle(12, 16);
    var rect2 = rect.withLength(19);

It will indicate that withLength doesn't exist.
Sample project
see code above
Version information

IDEA Version: 2021.2
JDK Version:  AdoptOpenJDK-16.0.1+9
OS Type & Version: Linux, distribution: Ubuntu 20.04.2 LTS (focal), kernel: 5.11.0-34-generic
Lombok Plugin Version: 212.4746.92
Lombok Dependency Version: 1.18.20

Additional information
-
Stacktrace
-

Hi, my additional observations.
Source
public record Foo(String name, @With int size) {}
compiles, but IntelliJ IDE warns
Wither needs constructor for all fields
If I add @AllArgsConstructor, the warning is gone.
Nonetheless, IntelliJ IDE won't recognize the generated Foo.withSize(int size) method. Maven build succeeds without problems though.
Lombok version: 1.18.22
Platform: Corretto-<IP_ADDRESS>.1 (build 17.0.2+8-LTS)
Plugin version: 213.6777.52
IMHO, @With annotation is a perfect companion for the Java record types - yet another example of lombok providing the essential missing piece of functionality that really should be part of Java language.


Fix #172 by adding parsing step to save_metadata

creates a correctly formatted chapter.txt file
with titles from audible-cli .json
uses this chapter.txt instead of ffprobe output
in case audible-cli data should be used

@fabh2o tested again with the changes from your review applied and it works as expected.

Thanks for the contribution!

Thanks for merging and thanks to @fabh2o for his original idea with jq that helped a lot!


docstring fix

OCD fix
(maybe should also be RE=RE. also would suggest emphasizing that this is not necessary and RE.subscribe can be called after the fact as well)

Codecov Report

Merging #883 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #883   +/-   ##
=======================================
  Coverage   85.34%   85.34%
=======================================
  Files          18       18
  Lines        4981     4981
=======================================
  Hits         4251     4251
  Misses        730      730

Impacted Files
Coverage Δ

bluesky/callbacks/zmq.py
75.19% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

It actually must be RE=RE as it is a keyword-only arg!

urgh yes you're right :-| oops!

Just noticed this is your first bluesky PR! :tada:

:laughing:


Deprecate PRAGMA x=y in favor of SET x=y

Fixes #1753
@Mytherin we discussed deprecating PRAGMA in the documentation where possible. This is what you had in mind?

Yes, thanks!


Is it possible to run it with hex file?

Hi! Thanks for awesome emulator! Is it possible to specify a hex file on start, not by drag-n-drop?

Hi! The command line works similar to the URL parameters, with "param=value" syntax, e.g.: ./Ardens file=game.hex.
Leaving this issue open until I document this in the readme.

#37


Can anyone please provide a working example of this on codepen?

Hi Mike,
here you go: http://codepen.io/guylabs/pen/GJmwMw. I also added it to the README such that others also right away are able to try it live.
Can I then close this issue?
Thanks and regards,
Guy


Cleanup ruby dependency

Triggered by https://github.com/conda-forge/libsdformat-feedstock/pull/70#issuecomment-1272497559 .
Changes:

Move the ruby dependency from build to host, as for cross-compiled builds the ruby install should be the target one
Remove the hard specification of ruby on Windows run dependencies (add added ignore_run_exports to ensure these are not added), as this packages does not compila anything against ruby ffi, so any recent ruby is ok to be installed at runtime

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender


Add search_bar example from the upstream C repository.

This example pull request is apart of #36, and adds a recreation of the original C example that demonstrates how to use the SearchBar and SearchEntry.
It recreates the example almost identically:

Thanks!
- Grant

Sorry for the nitpick, but please don't use the dark variant for screenshots for two reasons:

The "dark" theme isn't what the app developers often think is, it's just half broken implementation that only work per application basis and people should often just avoid it until there's a proper dark theme support implemented per platform (say in GNOME/KDE & all)
To keep the screenshots consistent

Oops yeah I've got it, I just forgot to change themes before screenshoting :)

Everything looks good here except for the CI, which at least looks good for the examples runner, I think it's ready to merge.

Everything looks good here except for the CI, which at least looks good for the examples runner, I think it's ready to merge.

Yes the CI fails due to a change in rust-nightly which broke one of gtk4-rs dependencies. I will have a look at how should we handle it tomorrow

Do you mind rebasing your branch on top of master? there were some changes that makes the CI fails

I merged the upstream branch to my fork, but it doesn't seem to show up here and rerun the CI.

Okay everything looks good, seems ready to merge.

Thanks @grantshandy


Drag and drop nodes in a type tree chart

What problem does this feature solve?
Allowing drag and drop of individual nodes  in a type: 'tree' chart in Echarts.
What does the proposed API look like?
series-tree. draggable
boolean

Hi any news on this topics? I have tried the draggable property. the however mouse pointer changed accordingly but the node doesn't move at all.


feat(terraform_docs): Drop support for terraform-docs <0.12.0

Description of your changes
Cleanup of old awk hacks introduced long time ago when terraform-docs wasn't able to work with terraform 0.12+.
How can we test changes
repos:
  - repo: https://github.com/antonbabenko/pre-commit-terraform
    rev: e40ffc76df4d24fa617c46094c934847334b12c0
    hooks:
      - id: terraform_docs
        args:
          - "--args=--lockfile=false"
          - --hook-config=--create-file-if-not-exist=true # Boolean. true or false

This PR is included in version 1.95.0 :tada:


Add Home Assistant integration information to the wiki

Explain how to create an MQTT binary sensor to respond to the motion messages.
platform: mqtt
device_class: motion
name: "Front door motion"
state_topic: aimotion/triggers/FrontDoor
payload_on: "on"
payload_off: "off"
value_template: "{{value_json.state}}"

done


Fix crd test flakeness

This should fix the test flakeness and potential timeout for pkg/config/crd. At least this should remove
the timeout failure which hides the actual problem if there are.

similar to #1210, creates the watcher instance earlier, to resolve the race condition between invocation of WatchFunc (of creating watcher) and put (use of watcher)
replace FakeWatcher by RaceFreeFakeWatcher. Otherwise, put may be blocked due to write of channel with no listeners.
restructure TestCrdsRetryAsynchronously a bit; make the put/watch/get testing after verifying the store creates the cache. Otherwise, the cache isn't ready yet, and the test can fail.
minor cleanup to cancel the context for Init after the test case finishes.

Release note:

none

This change is

/assign @guptasu

@jmuk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
8b212caaa39fbfd5e237c63233bcf268f7a5f8c4
link
/test mixer-e2e-smoketest

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

@jmuk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
8b212caaa39fbfd5e237c63233bcf268f7a5f8c4
link
/test mixer-e2e-smoketest

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

/assign @mandarjog

/lgtm

@jmuk: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixer-e2e-smoketest.sh
e0450e5f473a31281b13ed4bb2d3f084a51213e2
link
/test mixer-e2e-smoketest

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

@mandarjog
the smoketest succeeds with my private GKE cluster, as follows:
% ./tests/e2e.sh --mixer_hub=gcr.io/istio-testing --mixer_tag=e0450e5f473a31281b13ed4bb2d3f084a51213e2
(snip)
ests Summary
PASSED: //tests/e2e/tests/mixer:go_default_test
PASSED: //tests/e2e/tests/bookinfo:go_default_test

Please merge this, as we discussed offline.


feat(wash-cli): write out generated wadm

Feature or Problem

Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

wash dev E2E test is failing in CI -- looking into it!


Wider and bigger clock on Large screens

The clock has to get big, center of attention on laptops

on it


Add tests for round-trip serialize then deserialize DateTime and DateTimeOffset values

Addresses https://github.com/Azure/azure-sdk-for-net/issues/35944
Adds tests for serialization/deserialization of DateTime and DateTimeOffset values.

API change check
API changes are not detected in this pull request.

Addressed in a different PR


Add option for file output

Users may want to extract all generated commands / text into a text file, e.g, for sharing.

Solved by 7b3cd8e0d9b073721929057e37f34fa6fa49ba7c.


StackNavigator triggers transition twice when double tapping fast

Hi,
if you use a StackNavigator and double tap on an item, the transition is triggered twice. You then have to go back two times as well to get back to the starting point. This behaviour can be observed in the current NavigationPlayground using expo. I was able to reproduce it on iOS (iPhone 6) and on Android (Samsung Galaxy S8).
Reproducing this bug on real devices can require a few attempts, you have to double tap really fast. I was not able to reproduce it on the iOS simulator.
Current Behavior

Fast double tap on StackNavigator item triggers transition to next Screen two times, user must go back two times

Expected Behavior

When the tap was recognized, another tap should not trigger the transition again

Environment

Current NavigationPlayground using Expo
iPhone 6 on iOS 10.3.2
Samsung Galaxy S8 on Android 7.0

This is a known issue, see https://github.com/react-community/react-navigation/issues/271. There is discussion in other issues for potential fixes/workarounds. See also https://github.com/react-community/react-navigation/issues/135.

This issue fixed with latest version, I guess after v2.0 or try upper 👍 @murtuzamac

more info about how 2.0 fixed it here: https://reactnavigation.org/blog/#navigateroutename-in-stacknavigator-is-less-pushy


How to disable port forwarding when using default driver and provider

Issue Type

Question

Desired Behaviour
Now the default ansible_port is forwarded from the vm. How to make vagrant using host-only network,
and the ansible_host / ansible_port is directly from the guest hosts.

I need further information.

now, for each vagrant/virtualbox  instance, the host is always <IP_ADDRESS>,  ssh port is a forwarding port (22->2222 or 22--> 2200, etc), and the network mode of the vm nic is nat.
I want to change the network mode to bridge or host-only, and the generating ansible_host in inventory file is the real ip of the vm nic, and the generating ansible_ports are all 22.

@gzm55 That is not possible.  However, you can assign extra interfaces to your VMs through molecule.yml.  This would bring up a host-only network eth1 on both instances.
instances:
  - name: vagrant-test-01
    interfaces:
      - network_name: private_network
        type: dhcp
        auto_config: true
  - name: vagrant-test-02
    interfaces:
      - network_name: private_network
        type: dhcp
        auto_config: true

@retr0h you solution works, thx.


Add POST methods to async client

When creating new collections+items via an API (e.g. in stac-siphon-rs), the JSON is POSTed. If we provide a .post() method on our client, downstreams could use that to async-write to APIs.

Closed by #130.


fix polyfit changing the original object

[x] Closes #5644
[x] Tests added
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

~New functions/methods are listed in api.rst~

I don't think the small change in the curvefit doctest is caused by this PR.
I get the same error using main.
I've updated what's new in a separate commit, but looks like GH PRs are having difficulties at the moment.


actionlib: 1.11.13-0 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository actionlib to 1.11.13-0:

upstream repository: https://github.com/ros/actionlib.git
release repository: https://github.com/ros-gbp/actionlib-release.git
distro file: lunar/distribution.yaml
bloom version: 0.6.2
previous version for package: 1.11.12-0

actionlib
* [bugfix] added missing boost/thread/reverse_lock.hpp include (#95 <https://github.com/ros/actionlib/issues/95>)
* Contributors: Robert Haschke

Holding for synchronized release...


Contribute a copy of Dex protection source code

Source code
code.zip
https://www.yjaq.xyz

[!TIP]

Details

https://github.com/luoyesiqiu/dpt-shell
@developer-krushna

Hi ..I not understand what you need.. based on my understanding you need Dex encryption project right?? Currently I don't have any projects on it as I lost many of my codes because of destruction of my sdcard where I saved my important project data second,Even I lost my phone where I created many new projects..all lost sad

Hi ..I not understand what you need.. based on my understanding you need Dex encryption project right?? Currently I don't have any projects on it as I lost many of my codes because of destruction of my sdcard where I saved my important project data second,Even I lost my phone where I created many new projects..all lost sad

I have a source code from this project, can you build the serverless apk file for me? Thank you
ApkProtector.zip
https://www.rustore.ru/catalog/app/com.mcal.dexprotect

Sorry I can't build..I don't have much time+ I don't have a perfect device to build such big projects

Sorry I can't build..I don't have much time+ I don't have a perfect device to build such big projects


Different behavior in desktop, mobile vs. aframe, three.js

If the sourceWidth, sourceHeight, displayWidth, and displayHeight are set the same in the aframe wrapper and the pure three.js version, in desktop (win10 x64, chrome 56.0.2924.87) both produce nearly identical outputs as you would expect (3js on left, aframe on right):

In mobile (OnePlusOne, android 6.0.1), however, the same two webpages produce drastically different outputs (3js on left, aframe on right):

Note that, in addition to the different aspect ratio and cropping, the aframe version seems to be at a lower resolution.
The two webpages are:
https://jeromeetienne.github.io/AR.js/three.js/examples/mobile-performance.html
https://hypogeum.info/misc/aframe_arjs_lowquality.html
The aframe example is exactly the example code from the readme, except with the settings changed to mirror the three.js mobile example:
<a-scene stats embedded artoolkit='sourceType: webcam; sourceWidth: 240; sourceHeight: 180; maxDetectionRate: 30; detectionMode: mono'>

i think i understand what is happening.
three.js examples got the display issue fixed (cc @blq) but the aframe doesnt have it yet.

Discussion with fredrick - open the webcam with the displayWidth, not the sourceWidth

this.parameters.displayWidth
maxWidth: this.parameters.sourceWidth
source is influence resolution/performance of videoTexture (e.g. refraction, marker cloak)
so full screen for normal demo, but when performance related (aka when using video as texture) then tune down the resolution

TODO in context, change sourceWidth/sourceHeight by canvas height

reevaluate all the options in aframe-ar

@PyryM it should work now. tell me how it goes
<a-scene stats embedded artoolkit='sourceType: webcam; sourceWidth: 240; sourceHeight: 180; maxDetectionRate: 30; detectionMode: mono'>
change sourceWidth into canvasWidth, and the same for canvasHeight

i did a mobile-performance for a-frame too
https://github.com/jeromeetienne/AR.js/blob/master/aframe/examples/mobile-performance.html

Now it looks pretty similar in both threejs vs aframe on mobile: in landscape they are identical, and in portrait they seem to be at the same base resolution, but with slightly different cropping.

perfect. thanks for reporting this


Document what's new in .NET Standard 2.1

2.1 is coming, We need to document what's new.
Should we cover in the same topic like we do for .NET Framework or split like we do for C# or .NET Core?
@terrajobst is this something you could help us put together? Thanks!

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

ID: de130a75-fed2-0567-6e3d-07a384900eec
Version Independent ID: a2993c26-93f8-7281-d6ca-5fcc899e95f3
Content: What's new in the .NET Standard
Content Source: docs/standard/whats-new/whats-new-in-dotnet-standard.md
Product: dotnet
Technology: dotnet-standard
GitHub Login: @mairaw
Microsoft Alias: dotnetcontent

While updating for Core 3.1 I ran across this page. I've updated the comment metadata to remove ron.

Thanks @Thraka. @terrajobst can you help to write the what's new in .net standard 2.1 content?


StreamId Map should be cleared when connection is stopped

Currently we don't clear the internal dictionary for mapping stream ids to active streams when the client disconnects. It's a silent issue because we will override any stale values with new streams, but regardless, we should be clearing out the map on connection close.

And removing on stream closes?

We could. We'd have to have another map though correlating invocation ids to their stream ids. So we'd have to push the invocation id into the checkUploadStreamlogic where we allocate the stream Ids so we could group them together.  I don't feel to strongly about this tbh.

Sure, but it's technically a memory leak on the client.

*sighs in agreement

Consider for preview 8, relatively low priority.


Supporting HA Master with Slaves

Not an issue, but more of a call for assistance!
Is there a way with ansible-redis (I assume using Sentenial) to have a High Availability setup for Master, and with multiple slaves connecting to the active master.

C1, C2, C3 are all slaves of M1 or S1 (depending which is the active master)
Thanks,
Adam

Hey @adamstrawson. Did you get anywhere with this? I haven't tested this specific architecture, but it should be possible. I'll have to give it some thought and I can try to put together an example playbook for you.

@adamstrawson This is more of a limitation of Redis than of this role, but it doesn't appear possible (within reason) to always have your dedicated slaves pointed to the active master.
You can set redis_slave_priority: 0 on each of the client nodes to prevent them from becoming master, but I just don't see a way to always point them to the current master without some sort of wizardry outside of what Redis provides. If you know of something I don't, let me know and I can accommodate that use case in this role.


[TUBEMQ-237]add docker image to maven build

add docker image to maven build

Codecov Report

Merging #153 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #153   +/-   ##
========================================
  Coverage      7.62%   7.62%
  Complexity      436     436
========================================
  Files           235     235
  Lines         26535   26535
  Branches       4424    4424
========================================
  Hits           2023    2023
  Misses        24065   24065
  Partials        447     447

Continue to review full report at Codecov.

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


Find synergies with tsfresh / tspreprocess

Hi @rtavenar,
I am impressed with the number of algorithms you already implemented in tslearn.  Actually, I had the idea for the same package, even the same name :D :D. Some of the algorithms I implemented in the past.
I have been working with time series models for the last years and there is definitely a lack of python tools or ecosystems for time series analysis. So, I designed both packages https://github.com/blue-yonder/tsfresh and https://github.com/MaxBenChrist/tspreprocess.
If you are interested, I would like to have a chat with you to hear your ideas and vision for tslearn. We have a really good development team in tsfresh. Maybe we find a way to synchronize our efforts and  support you.
Best, Max

Yes, that would be great. Would next Wednesday (during Western Europe office hours) be OK for you? I could maybe join your gitter.

Next Wednesday sounds good. Can you send me a personal message on gitter?

So, after the call, @MaxBenChrist has published a list of github repositories that provide tools for the analysis of time series.
The idea is that, at some point, it would be nice to gather people from this repos and see if it seems feasible to propose a uniform representation for time series across all these tools.
If this works well, further collaboration could be considered.

So, after the call, @MaxBenChrist has published a list of github repositories that provide tools for the analysis of time series.
The idea is that, at some point, it would be nice to gather people from this repos and see if it seems feasible to propose a uniform representation for time series across all these tools.
If this works well, further collaboration could be considered.

Thanks again for the nice call.
Yes, I am planning to extend the list to get an overall overview. Regarding the collaboration, as you said, I think we should focus on the formats for now.

In #149 , we have added conversion functions between several Python libs for time series, including tsfresh. Already available in dev branch, will be included from version 0.3


Improved search results page form on mobile

Fixes #777 #776 #774
Fixes some of the issues with the search form on the results page on mobile screens.

Developer Certificate of Origin Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors. 1 Letterman Drive Suite D4700 San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or
(b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or
(c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it.
(d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.

I think that's ok because we don't have anyone using Firefox on mobile. But feel free to create a separate issue to fix that


Add jamxl(zijianjoy) to KFP ci-team

Officially getting start at KFP team.

/lgtm
/approve
Welcome!

/lgtm
/approve
Welcome!

Thank you Yuan!

Thank you Yuan!


svg use produces "multiple root elements must be wrapped in an enclosing tag" error

It seems strange to me that the following markup produces the multiple root elements must be wrapped in an enclosing tagerror message:
return html`

        <svg>
          <use xmlns:xlink="http://www.w3.org/1999/xlink" href="#reply"></use>
        </svg>

    `;

I'm not clear if you say that this code should get the error message or if it is getting the error message and it shouln't.

Sorry, a bit unclear there. This code is getting this error. I don't see why it does.

Not on laptop right now so can't debug - sometimes this error is
incorrectly thrown because tags are closed when they shouldn't be - or a
trailing closing tag is added when it isn't needed. Could you try out a
variation of these? Thanks!
On Tue, Jun 20, 2017, 14:49 Martin Folkeseth<EMAIL_ADDRESS>wrote:

Sorry, a bit unclear there. This code is getting this error. I don't
see why it does.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yoshuawuyts/choo/issues/514#issuecomment-309742727,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWlekFL6Rav6v5BA28-7TtK-ouGmD0oks5sF7_sgaJpZM4N_hnQ
.

This is a bel issue not choo specific. and is not actually an issue, it happens to me when I had <path></path> in my svg, it fixed changing it to <path />. So, in your case it should fix with
return html`

        <svg>
          <use xmlns:xlink="http://www.w3.org/1999/xlink" href="#reply" />
        </svg>

    `;

Thanks, that fixes my problem. I'll close this.


Hard coded relay fee

Relay fee was hard coded using txrules.StakePoolTicketFee issue 54.
it seems like a better idea to rather retrieve it from dcrd rpc, hence this pr.

Relay fee was hard coded using txrules.StakePoolTicketFee issue 54.

Confusing.

it seems like a better idea to rather retrieve it from dcrd rpc, hence this pr.

Why does it "seem" like a better idea?


Promise is rejected: Error: 2 UNKNOWN: error starting container: API error (404): {"message":"network build-blockchain-insurance-app_default not found"}

I've just download git repository and during build_ubuntu.sh, I've got the follow error message:

user@ubuntu:~/Insurance/build-blockchain-insurance-app$ docker logs web

<EMAIL_ADDRESS>serve /app
cross-env NODE_ENV=production&&node ./bin/server

/app/app/static/js
Server running on port: 3000
Default channel not found, attempting creation...
Successfully created a new default channel.
Joining peers to the default channel.
Chaincode is not installed, attempting installation...
Base container image present.
info: [packager/Golang.js]: packaging GOLANG from bcins
info: [packager/Golang.js]: packaging GOLANG from bcins
info: [packager/Golang.js]: packaging GOLANG from bcins
info: [packager/Golang.js]: packaging GOLANG from bcins
Successfully installed chaincode on the default channel.
error: [client-utils.js]: sendPeersProposal - Promise is rejected: Error: 2 UNKNOWN: error starting container: API error (404): {"message":"network build-blockchain-insurance-app_default not found"}
at new createStatusError (/app/node_modules/grpc/src/client.js:64:15)
at /app/node_modules/grpc/src/client.js:583:15

Fatal error instantiating chaincode on some(all) peers!
Error: Proposal rejected by some (all) of the peers: Error: 2 UNKNOWN: error starting container: API error (404): {"message":"network build-blockchain-insurance-app_default not found"}
at /app/www/blockchain/utils.js:248:15
at Generator.next (<anonymous>)
at step (/app/bin/blockchain/utils.js:102:191)
at /app/bin/blockchain/utils.js:102:361
at <anonymous>

npm ERR! code ELIFECYCLE
npm ERR! errno 255
npm ERR<EMAIL_ADDRESS>serve: cross-env NODE_ENV=production&&node ./bin/server
npm ERR! Exit status 255
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>serve script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!
Edit build-blockchain-insurance-app/peer-base.yaml file. Search for the following:
CORE_VM_DOCKER_HOSTCONFIG_NETWORKMODE=build-blockchain-insurance-app_default
Replace the line with:
CORE_VM_DOCKER_HOSTCONFIG_NETWORKMODE=buildblockchaininsuranceapp_default
Run ./build_ubuntu.

The docker network name in the peer-base.yaml file is different from docker network name when setup in Ubuntu. Editing the file and renaming it will fix the problem.

Thanks a lot @parsiya .... It is working now.....

Much appreciated, @parsiya!
That sorted that fixed the issue for me and the chaincode was able to initiate.

ERROR: pull access denied for shop-ca, repository does not exist or may require 'docker login'
I followed the mentioned steps @parsiya  but i am getting the above mentioned error, any help would be appreciated,Thank you.

Having the same issue,  even if I run a prior commit.

pattern is updated so issue went away


Add back button to webreport

works only together with https://github.com/e-m-b-a/emba/pull/185


Missing Flux study ...

Many times release the flux in the atmosphere.

Azanor, don't look my mistake ...

The author is busy and will look when he can.


Local video player

Hi,
I wonder how I could add a video in my presentations. I have one constraint however: the video is not on a server (not on youtube, not elsewhere). I can't (I mean, I don't want to) upload it on any server. I want to keep it local. So I guess I must use an embedded player.Do you have any idea about how I could do that ?

Did you try the HTML5 video tag?
You may use more or less any HTML you want in your slides, to do stuff Markdown doesn't let you do.

Please re-open if this is still unresolved.


Clean up the torch and torch vision python 3.12 gating when PyTorch is 3.12-compatible

Clean up the patches introduces by:

https://github.com/rerun-io/rerun/pull/4146

Look for TODO(#4704) in the repo...

Related pytorch issues:

https://github.com/pytorch/pytorch/issues/110436


[ECS] Conflict in PSR-2 set

There are few lines (same) that are being changed always, after each run:
    ---------- begin diff ----------
--- Original
+++ New
@@ -48,7 +48,7 @@
             self::API_CREDENTIALS_USERNAME,
             self::API_CREDENTIALS_PASSWORD,
             $this->countryVariables->marketing['etargetClientId'],
-            );
+        );
         $this->client->setLogger($this->logger);
         $this->client->setVerboseLogger($this->verboseLogger);

    ----------- end diff -----------

Applied checkers:

 * PHP_CodeSniffer\Standards\PSR2\Sniffs\Methods\FunctionCallSignatureSniff.Indent

    ---------- begin diff ----------
--- Original
+++ New
@@ -48,7 +48,7 @@
             self::API_CREDENTIALS_USERNAME,
             self::API_CREDENTIALS_PASSWORD,
             $this->countryVariables->marketing['etargetClientId'],
-        );
+            );
         $this->client->setLogger($this->logger);
         $this->client->setVerboseLogger($this->verboseLogger);

    ----------- end diff -----------

Applied checkers:

 * PhpCsFixer\Fixer\FunctionNotation\MethodArgumentSpaceFixer

Was able to debug and solve the issue.
Resolved by adding manually this service:
services:
    PhpCsFixer\Fixer\FunctionNotation\MethodArgumentSpaceFixer:
        ensure_fully_multiline: false

Originally there was true value which was causing that change (i do not know why, though 😄 )

Thanks for sharing the solution :+1:


Add more Docker + ASP.NET Core samples

We frequently get asked to build features into our Docker images that are already possible and easy to do without needing changes to the base images.
Instead, we would like to address this by adding more samples and docs for these commonly asked features/questions.

How to use multi-stage builds
Using NodeJS 9
Add NodeJS so JavascriptServices works at runtime
Using powershell in Linux images or NanoServer 1709
Running tests in docker
Writing a Dockerfile to building a solution or multiple .csproj files
Debugging remotely into a docker container (vsdbg)

cc @glennc
[Edit - append by @rick-anderson]
Create basic windows/MacOs dev/productions samples using HTTPS from https://github.com/dotnet/dotnet-docker/tree/master/samples

What I am thinking here is something akin to a Docker recipes page. Including examples for each of the ones you listed as well as how to do libgdiplus in Docker.
I think we could link to this from the top of our scaffold-ed Dockerfiles.
//cc @richlander

//cc @spboyer

Here is an additional request for doc on ASP.NET Core SPA templates - https://github.com/dotnet/dotnet-docker/issues/393

One use case that I don't even know if it's supported, and at least not documented, is to be able to just recompile the application without needing to rebuild and restart the container. Right now I have to do CTRL+F5 for the entire docker-compose, even if I've just made changes to one of the projects, which results in a very slow feedback cycle. It would be nice to get the same live update experience for compiled code, as I do for static files. Perhaps using dotnet watch, or, failing that, using some post-build script that updates the running container with new binaries or something like that.

I just opened the new Visual Studio 15.7, created a new ASP.NET Core 2.1 project with the Angular template, added Docker Support started the application and ... got an exception 👎
This all is now a release candidate!!!
Honestly, I would get into trouble making software like that

@dapalmi  This repro is for document issues. Did you want to open an issue with the product?  If so, see the ASP.NET How to contribute
It's helpful to add the steps to reproduce the problem, what happened, and what was expected.

Ok. Sorry. I thought this is the continuation of this issue

@dapalmi  no problem. Does the workaround solve your issue?

@Rick-Anderson The workaround works. Thanks. It just doesn't fit into the otherwise very nicely flowing ASP NET Core workflows. In Visual Studio Angular Templates are offered and you can add Docker support, so me as a normal user would expect this to work without having to google a workaround first

@dapalmi  agreed.
@scottaddie  can you work with the PU to get a fwlink on the wizard to warn folks. The fwlink could point an H2 in our Angular doc, which in turn could point to that issue.

@natemcmaster This is good way to install nodejs in linux based docker image. But there is one issue with docker integration with visual studio because visual studio mount local source in docker container but all native nodejs components (like node sass) are built for windows not for linux :disappointed:

@natemcmaster 	@glennc 	do you have any samples/content we can start with?

I have the same issue: https://github.com/Microsoft/aspnet-docker/issues/164


[2.1.1] Use correct TaskCompletionSource ctor

These are test only changes. It's the combination of the following PRs backported for release/2.1:

https://github.com/aspnet/KestrelHttpServer/pull/2531
https://github.com/aspnet/KestrelHttpServer/pull/2567
Parts of https://github.com/aspnet/KestrelHttpServer/pull/2526
Parts of https://github.com/aspnet/KestrelHttpServer/pull/2547

These changes have been in dev for a while and have improved test reliability. This comment from @pakrym explains why:

The common issue I discovered in our tests is completing TCS inside the appfunc and then awaiting it just before disposing the server.
This causes continuation to be run inline in appfunc and prevents graceful shutdown making tests slower and possibly more flaky.
There is also a race it that if TCS get's completed before being awaited we get different behaviour then if was awaited first and then completed.


Added simple player controls and screen collision

Thanks for taking the time to create this PR.
Merged! :)

Great, thanks !
How barebone do you wnat this template to be, like, how much will be too
much ?
2016-07-07 22:24 GMT+02:00 Ollie<EMAIL_ADDRESS>
Thanks for taking the time to create this PR.
Merged! :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ojdon/pico8-boilerplate/pull/1#issuecomment-231196335,
or mute the thread
https://github.com/notifications/unsubscribe/AD-jyaIFJvF_2qCIGndS7XELb6hRCBb6ks5qTWCCgaJpZM4JG6nW
.

--
briac

I'm not too sure myself, to be honest! :sweat_smile:
I sort of want to make sure that the template code itself does not obstruct the game developer's actual game creation time too much.
For example, I don't want them to have to sit there for 5 or 10 minutes stripping out the bits they don't need, as I feel that a project like Lib-Pico8 would be better suited for larger code.

Yeah, I understand. Maybe it could be nice to have different templates for
different kind of games (shoot 'em up, arcade, etc.)
2016-07-07 23:19 GMT+02:00 Ollie<EMAIL_ADDRESS>
I'm not too sure myself, to be honest! 😅
I sort of want to make sure that the template code itself does not
obstruct the game developer's actual game creation time too much.
For example, I don't want them to have to sit there for 5 or 10 minutes
stripping out the bits they don't need, as I feel that a project like
Lib-Pico8 https://github.com/clowerweb/Lib-Pico8 would be better suited
for larger code.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ojdon/pico8-boilerplate/pull/1#issuecomment-231210672,
or mute the thread
https://github.com/notifications/unsubscribe/AD-jyX6aTDdAazzrpxru1antCgGMcrpFks5qTW1vgaJpZM4JG6nW
.

--
briac


Rotate only and do not Scale and Scale only and do not rotate

How can i achieve that  if the rotate event is currently active then scale should not happen and when scale event is currently active then rotate should not happen?

You could/should use a flag in your activity which you can set to a certain state (i.e. rotating or scaling)  from within your eventhandler and test for a flag state in the other eventhandler preventing it from handling if another state is currently active...


Feature/carter qol -- finetuning cancelation support and retry logic

retry logic for data synthesis (sometimes the models dont emit valid json), and adding a signal handler to cancel finetuning when you exit the program

Right now even if one datapoint has an issue (leads to unparseable JSON), the whole process in restarted. I think this is fine for now, but maybe in the future we want to have all the other datapoints that are still formatted correctly to still be used and just the bad datapoints ignored.

while testing, I have been stuck doing generation for last 10 minutes because one datapoint has a parsing issue

@abhigya-sodani, thats some terrible luck! its set to fail after two tries by default, did you hit that limit or ?

I’m happy to add some more robust formatting tricks to this pr.  Or are you comfortable with this being merged in as is ?

Cntrl+c quitting was not working for me

Ok that’s weird.  What os, terminal, and shell are you using?

Anaconda Prompt, windows 10

Ok I think interrupts/signals aren’t supported on windows.  I’ll check out if there’s a simple equivalent. If it’s easy for you to whip up a handler for native windows , I’ll stack that on


Add Header UnicodeEncodeError

Add Header fails when scandinavian characters are used in argument.
PycURLLibrary.Add Header    Username:ääkköset
Gives an error "UnicodeEncodeError: 'ascii' codec can't encode characters in position 9-10: ordinal not in range(128)"

Modifying all string handling into Unicode


Code tags don't appear in output anymore

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #14479

Description

The solution was to make the regex lazy i.e. matching as little as possible. I also changed matching to global so that it would match more than one time.

LGTM. Both changes are needed for the corner case in question. A simpler regex could be used, but this way the change is minimal.

Thanks!

Tested locally and LGTM. Thanks @techstonia . Keep up the good work! 👍 💯 🌮


Install cargo-dist to generate releases

This PR was generated by running cargo dist init and selecting all the default options. It seems to have pretty sane behaviour out of the box.
This requires the repository URL fix in https://github.com/mozilla/cargo-vet/pull/599 to work correctly.
Fixes https://github.com/mozilla/cargo-vet/issues/484

For anybody following along on this, https://matrix.to/#/!dRHkXGXAEhtBOpnLTY:mozilla.org/$4_H4pbu-FftonOnqfHJLXWZzd1ZNHSDdtptBk1IW_r4?via=mozilla.org


feat(borg): :arrow_up: Upgrade borg to Version 1.2.4

:tada: This PR is included in version 1.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Upgrade translation store

Refactor code to

remove duplicated variables such as global isLoading
remove the unneeded interface
simplify private methods

Provide t and isError service
Modify jest to test new features
Update document according to new features

@NishuGoel I'm not sure whether I should tag you for this PR... It'll be delightful to have this change for my project.


Reports showing duplicate entries on test success

Apologies in advance if this has already been reported, I thought I saw a similar issue but can't seem to find it.
I have implemented Maester through GitHub Actions using the documentation. The majority of our tests are showing duplicate responses in the results, in both markdown and HTML. Clicking into the failed result shows it was actually a success. This is version 0.2.0 of the Action but I also tested the latest preview.

I can look at the script too if someone with deeper knowledge of the tool is able to point out where the failure could be :)

I found issue #71 reported this issue fixed but I still have the problem, happy to help out where I can.

Trying to provide more information where I can, but I've looked at the logic in one of the simpler checks (EIDSCA.AF01) which seems to all be correct.
I'm not sure if the problem here comes down to the 'Add-MtTestResultDetail' script, as in our environment every result is duplicated. Once for success, once for failure. If the test is something we have failed, a failure is reported twice.

It is happening to me as well for all the tests, it started to happen once I have changed the main.yml file in the pipeline (and unfortunately I was never been able to restore the single occurrence of tests)

I have found more info on this, as below.

This looks to show for most, if not all, EIDSCA checks though I'm not sure why this is happening within the module.

Can you please try running again? I just published a new version of the module that should have this cmdlet.

Still seems a problem unfortunately. Here's a sample output we have from the JSON and a visual from the step summary. Whilst now reporting success, the results are still duplicated. All I can notice from the output is how the ScriptBlock line is slightly different.
{
      "Name": "EIDSCA.AF01: Authentication Method - FIDO2 security key - State.",
      "HelpUrl": "https://maester.dev/docs/tests/EIDSCA.AF01",
      "Tag": [
        "EIDSCA",
        "Security",
        "All",
        "EIDSCA.AF01"
      ],
      "Result": "Passed",
      "ScriptBlock": "\r\n        <#\r\n            Check if \"https://graph.microsoft.com/beta/policies/authenticationMethodsPolicy/authenticationMethodConfigurations('Fido2')\"\r\n            .state = 'enabled'\r\n        #>\r\n        Test-MtEidscaControl -CheckId AF01 | Should -Be 'enabled'\r\n    ",
      "ScriptBlockFile": "/home/runner/work/maester/maester/private-tests/tests/EIDSCA/Test-EIDSCA.Generated.Tests.ps1",
      "ErrorRecord": [],
      "Block": "Authentication Method - FIDO2 security key",
      "ResultDetail": {
        "SkippedReason": null,
        "TestResult": "\r\nWell done. The configuration in your tenant and recommended value is *'enabled'* for *policies/authenticationMethodsPolicy/authenticationMethodConfigurations('Fido2')*\r\n",
        "TestSkipped": "",
        "TestDescription": "Whether the FIDO2 security keys is enabled in the tenant.\r\n\r\nenabled\r\n\r\n#### Test script\r\n  \r\nhttps://graph.microsoft.com/beta/policies/authenticationMethodsPolicy/authenticationMethodConfigurations('Fido2')\r\n.state = 'enabled'\r\n  \r\n\r\n#### Related links\r\n\r\n- [Open in Graph Explorer](https://developer.microsoft.com/en-us/graph/graph-explorer?request=policies/authenticationMethodsPolicy/authenticationMethodConfigurations('Fido2')&method=GET&version=beta&GraphUrl=https://graph.microsoft.com)\r\n- [fido2AuthenticationMethodConfiguration resource type - Microsoft Graph v1.0 | Microsoft Learn](https://learn.microsoft.com/en-us/graph/api/resources/fido2authenticationmethodconfiguration)\r\n\r\n\r\n"
      }
    },
    {
      "Name": "EIDSCA.AF01: Authentication Method - FIDO2 security key - State.",
      "HelpUrl": "https://maester.dev/docs/tests/EIDSCA.AF01",
      "Tag": [
        "EIDSCA",
        "Security",
        "All",
        "EIDSCA.AF01"
      ],
      "Result": "Passed",
      "ScriptBlock": "\n        <#\n            Check if \"https://graph.microsoft.com/beta/policies/authenticationMethodsPolicy/authenticationMethodConfigurations('Fido2')\"\n            .state = 'enabled'\n        #>\n        Test-MtEidscaControl -CheckId AF01 | Should -Be 'enabled'\n    ",
      "ScriptBlockFile": "/home/runner/work/maester/maester/public-tests/tests/EIDSCA/Test-EIDSCA.Generated.Tests.ps1",
      "ErrorRecord": [],
      "Block": "Authentication Method - FIDO2 security key",
      "ResultDetail": {
        "SkippedReason": null,
        "TestResult": "\r\nWell done. The configuration in your tenant and recommended value is *'enabled'* for *policies/authenticationMethodsPolicy/authenticationMethodConfigurations('Fido2')*\r\n",
        "TestSkipped": "",
        "TestDescription": "Whether the FIDO2 security keys is enabled in the tenant.\r\n\r\nenabled\r\n\r\n#### Test script\r\n  \r\nhttps://graph.microsoft.com/beta/policies/authenticationMethodsPolicy/authenticationMethodConfigurations('Fido2')\r\n.state = 'enabled'\r\n  \r\n\r\n#### Related links\r\n\r\n- [Open in Graph Explorer](https://developer.microsoft.com/en-us/graph/graph-explorer?request=policies/authenticationMethodsPolicy/authenticationMethodConfigurations('Fido2')&method=GET&version=beta&GraphUrl=https://graph.microsoft.com)\r\n- [fido2AuthenticationMethodConfiguration resource type - Microsoft Graph v1.0 | Microsoft Learn](https://learn.microsoft.com/en-us/graph/api/resources/fido2authenticationmethodconfiguration)\r\n\r\n\r\n"
      }
    }

I have the same issue, maybe you can change the "motd" to only MAESTER, and include the version (including build version) as normal text.
Currently it returns the big banner with v0.2 (which probably is the v0.3 version)

I think this bug is related to the import of the Github action, with duplicate tests.
So I run my own tests, and it's combined with all the tests from the main maester project as well.
For instance when I delete one of my own tests, the count of tests only goes down by one.
I think when I delete all my tests, it will still run all the maester supplied ones.

Can confirm this is the case. The default run for the GitHub action includes a checkout for all public tests. Setting include_public_tests to false fixes that!

@BenStreet365 shouldn't this be documented then somewhere?

@Oppedijk Yes, thinking about it that would be a good idea. I'll look at how I can tweak the documentation tomorrow for the GitHub Actions setup.