[stable/nginx-ingress] fixing pdb selectors to match pod labels

What this PR does / why we need it:
The PDB's currently will not match the pods they are intended to by default.
I'm not sure why the PDB had an if statement around the new label as it gets applied to every pod regardless of what's set for .Values.controller.useComponentLabel.
Which issue this PR fixes

fixes #21949

Special notes for your reviewer:
Checklist

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

/assign @taharah

/ok-to-test

/retest

/assign @ChiefAlexander - sorry for adding another assignee, just looking to get this resolved.

/assign @ChiefAlexander
sorry for adding another assignee, just looking to get this resolved.

Thank you!
/lgtm


chore(docs): Gatsby 버전 추가

:tada: This PR is included in version 1.2.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Reduce involvement of shared parental leave in pay leave for parents

This PR removes multiple outcomes from the Pay Leave For Parents flow around shared parental leave.
It removes

logic and tests for birth due dates prior to 2015-04-05
questions related to shared parental leave
several partials related to shared parental leave
mother earnings question for employed and self-employed mothers with less than 26 weeks worked

It adds

copy for unemployed mother earnings and weeks worked questions
copy for paternity leave and pay information for outcomes where they are NOT eligible

Trello

@cbaines Whats the status of this PR?

@cbaines Whats the status of this PR?

As far as I'm aware, the changes are still with the content team/department.
I'm also not the best person to talk to about this any more, @thomasleese might have more up to date information.

As far as I'm aware, the changes are still with the content team/department.
I'm also not the best person to talk to about this any more, @thomasleese might have more up to date information.

Yeah, we're still waiting on content.

I've reviewed from 0c41710 onwards, on the assumption that Chris B's commits beforehand have already been reviewed.
A couple of comments below, but the commit in question is actually my only blocking comment - there seems to be a lot of change in the pay-leave-for-parents.rb file in that commit, and not necessarily the tests to back it up 🤔

Thanks Chris - in fact all these changes have previously been reviewed. I've hopefully answered the questions below, the changes are not as sizeable as they appear but hard to follow due to the very large conditionals we have in this flow.


[Deprecation] Remove type specific methods in Arguments

For future knowledge, encourage the use of generics versus type-specific casting. If anything, getObject is still public.

Adding the deprecated annotation would probably be better in the meantime

Adding the deprecated annotation would probably be better in the meantime

Adding the deprecated annotation would probably be better in the meantime

Breaking changes have been introduced multiple times before and I don't think it's going to stop with this PR.

Adding the deprecated annotation would probably be better in the meantime

Breaking changes have been introduced multiple times before and I don't think it's going to stop with this PR.


Service - Detail Page

Based on #583, we need the detail page of this resource.
DetailTop should include, but the rest of the page should look like edit page

Item

Type

Created

@westlywright After reviewing service detail page, it would be nice to have the selector tab be able to list out the pods that match the selector.
Current tab:

Addition to tab:
Pods Section - H3 Heading
Table with these columns

Header
Item Info

Status
State of Matching Pod

Pod
Name of pod with link to pod detail page

Workload Type
Name of workload with link to workload detail page

Image
Image Name(s) of Containers inside pod

Node
Node Name of where the pod is running

@deniseschannon we should hold off on adding the pod counts for the same reasons we discussed with the create/edit pages. I opened the ticket for a new API mechanism to fetch related resources more efficiently here.

Session Affinity Tab

[ ] Instead of None, can we have text that says There is no session affinity configured. None seems too short and not informative and we're trying to move to a style of instead of showing a bunch of empty headers to consolidate what could be done into a There is no BLAH configured.

Issues found so far:
#800
#804

Rancher version master-head (07/09/2020) commit 8a032e73b
dashboard-index -> latest
Details page checked for the following service types:

L4 Loadbalancer (Using an AWS service provider)
Headless (Selector, Alias)
Workload
External IP
Cluster IP
External Name (Hostname in Ember UI)

Steps followed for each service type:

Keep the Browser Developer Tools open in the network tab for analyzing the v1 API responses.
Create the service type
Navigate to the Service List
Click in the new created service.
Save the API response in an Editor
Validate each data field displayed by the Service Details Page comparing with the v1 data returned by the API.
Navigate through the different Tabs in the service details

Selectors (where applicable)
Listener IPs
Session Affinity (where applicable)
Labels
Annotations

Navigation issue found:
https://github.com/rancher/dashboard/issues/820


Authenticated user broken

>>> u=network.get_authenticated_user()
>>> u.get_id()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/gnumdk/Téléchargements/pylast-1.2.1/pylast/__init__.py", line 3435, in get_id
    doc = self._request(self.ws_prefix + ".getInfo", True)
  File "/home/gnumdk/Téléchargements/pylast-1.2.1/pylast/__init__.py", line 1306, in _request
    params = self._get_params()
  File "/home/gnumdk/Téléchargements/pylast-1.2.1/pylast/__init__.py", line 3671, in _get_params
    return {"user": self.get_name()}
  File "/home/gnumdk/Téléchargements/pylast-1.2.1/pylast/__init__.py", line 3676, in get_name
    doc = self._request("user.getInfo", True, {"user": ""})    # hack
  File "/home/gnumdk/Téléchargements/pylast-1.2.1/pylast/__init__.py", line 1308, in _request
    return _Request(self.network, method_name, params).execute(cacheable)
  File "/home/gnumdk/Téléchargements/pylast-1.2.1/pylast/__init__.py", line 1136, in execute
    response = self._download_response()
  File "/home/gnumdk/Téléchargements/pylast-1.2.1/pylast/__init__.py", line 1127, in _download_response
    self._check_response_for_errors(response_text)
  File "/home/gnumdk/Téléchargements/pylast-1.2.1/pylast/__init__.py", line 1154, in _check_response_for_errors
    raise WSError(self.network, status, details)
pylast.WSError: No user with that name was found

I'm authenticated as scrobbling works...

Strange, I get this:
>>> pylast.__version__
1.1.0
>>> u=network.get_authenticated_user()
>>> u.get_id()
u'1051899'

Something to check: in _download_response in pylast/__init__.py, print out data and response_text to see what parameters are sent and what comes back.
Removing keys, I get:
data: sk=SNIP&api_sig=SNIP&api_key=SNIP&user=hvk&method=user.getInfo
response_text:
u'<?xml version="1.0" encoding="utf-8"?>
<lfm status="ok">
<user>
    <name>hvk</name>
    <realname></realname>
    <image size="small"></image>
    <image size="medium"></image>
    <image size="large"></image>
    <image size="extralarge"></image>
    <url>http://www.last.fm/user/hvk</url>
	<id>1051899</id>
	<lang>en</lang>
    <country></country>
    <age></age>
    <gender>n</gender>
    <subscriber>1</subscriber>
    <playcount>62709</playcount>
    <playlists>9</playlists>
    <bootstrap>0</bootstrap>
    <registered unixtime="1081980493">2004-04-15 00:08</registered>
	<type>subscriber</type>
</user></lfm>'

I get the same if I paste the parameter string at the end of http://ws.audioscrobbler.com/2.0/?
I created network like:
network = pylast.LastFMNetwork(
    api_key=API_KEY, api_secret=API_SECRET,
    username=lastfm_username, password_hash=lastfm_password_hash)

What do you get?

gnumdk@debian:~$ python3
Python 3.4.3+ (default, May  7 2015, 23:10:14)
[GCC 4.9.2] on linux
Type "help", "copyright", "credits" or "license" for more information.

import pylast
API_KEY = "7a9619a850ccf7377c46cf233c51e3c6"
network = pylast.LastFMNetwork(api_key = API_KEY, username='lollypop_gnome', password_hash=pylast.md5("password"))
u=network.get_authenticated_user()
u.get_id()

    No user with that name was found

 method=user.getInfo&user=&api_key=7a9619a850ccf7377c46cf233c51e3c6
Traceback (most recent call last):
  File "", line 1, in
  File "/usr/lib/python3/dist-packages/pylast.py", line 3410, in get_id
    doc = self._request(self.ws_prefix + ".getInfo", True)
  File "/usr/lib/python3/dist-packages/pylast.py", line 1305, in _request
    params = self._get_params()
  File "/usr/lib/python3/dist-packages/pylast.py", line 3646, in _get_params
    return {"user": self.get_name()}
  File "/usr/lib/python3/dist-packages/pylast.py", line 3651, in get_name
    doc = self._request("user.getInfo", True, {"user": ""})    # hack
  File "/usr/lib/python3/dist-packages/pylast.py", line 1307, in _request
    return _Request(self.network, method_name, params).execute(cacheable)
  File "/usr/lib/python3/dist-packages/pylast.py", line 1135, in execute
    response = self._download_response()
  File "/usr/lib/python3/dist-packages/pylast.py", line 1126, in _download_response
    self._check_response_for_errors(response_text)
  File "/usr/lib/python3/dist-packages/pylast.py", line 1153, in _check_response_for_errors
    raise WSError(self.network, status, details)
pylast.WSError: No user with that name was found

Thanks. So you're not using an API secret when creating the network. I can reproduce it if I remove api_secret:
network = pylast.LastFMNetwork(
    api_key=API_KEY,
    username=lastfm_username, password_hash=lastfm_password_hash)

But this works:
network = pylast.LastFMNetwork(
    api_key=API_KEY, api_secret=API_SECRET,
    username=lastfm_username, password_hash=lastfm_password_hash)

Please can you try with your API secret? Find it here: http://www.last.fm/api/accounts

Can i put the secret key in my software? Why it is called secret?

Good to hear it works.
It's generally best to leave the secret (and key, but especially secret) out of software, especially open source software. Load them in from config files or environment variables, or import another Python file that just has those constants in them.
Someone with access to the key and secret can make another app using them, and if users have authorised your app to do stuff with their account, then someone else could also do nasty stuff to their accounts.

As soon i add secret key, scrobbler fails ...
Traceback (most recent call last):
File "", line 1, in
File "/usr/lib/python3/dist-packages/pylast.py", line 4345, in scrobble
"s": self._get_session_id(),
File "/usr/lib/python3/dist-packages/pylast.py", line 4285, in _get_session_id
self._do_handshake()
File "/usr/lib/python3/dist-packages/pylast.py", line 4272, in _do_handshake
server, params, self.network, "GET").execute().split("\n")
File "/usr/lib/python3/dist-packages/pylast.py", line 4205, in execute
self._check_response_for_errors(response)
File "/usr/lib/python3/dist-packages/pylast.py", line 4223, in _check_response_for_errors
raise BadAuthenticationError()
pylast.BadAuthenticationError: Bad authentication token

BADAUTH
sk=6612a4810c12853030b3637943178985&p=1.2.1&t=1432567980&hs=true&c=tst&a=94cf0df5512fe576afb397bcc2fc8230&u=lollypop_gnome&v=1.0&api_key=SNIP

Ok, it works with network.scrobble  ...

Good to hear!
Some of my pylast-powered tools are here, you can check for examples: https://github.com/hugovk/lastfm-tools
And also the tests are basically example code too: https://github.com/pylast/pylast/blob/develop/tests/test_pylast.py


SQL Error

I added Supertable.
Now I wanted to create a new Field, which results in this:

CDbCommand couldn't excecute SQL-Statement: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'buttonLink-superTableBlockType:3' for key 'craft_fields_handle_context_unq_idx'

???
I didn't add supertable before in this project so there can't be a Duplicate entry of this.
Craft CMS 2.6.2968 | Supertable 1.0.4 | PHP 7.0.17 |

Can you try re-saving the Super Table field? It seems like something may have gone awry in the field creation.
Are you also using this in a Matrix super, or just purely with Super Table on its own?

Closing due to inactivity.


fixing readme to pass md lint

Saw DSCResource/212 and thought the linter was interesting. So I fixed up the README.MD to pass the linter.  Haven't added anything automatic but if we do the README should now pass.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Possible performance issues

Bug Report
I am getting a slow performance generating my blog, detailed here:
https://github.com/getzola/zola/issues/2508
It's possible Tabi plays a part here so I hope you don't mind I pull you in.
Do you have any idea what in the page or atom template could be causing slowness?
I have gone live with my zola and tabi based blog yesterday by the way, thanks for all your help! I like it:
https://blog.startifact.com

I've done some experimenting, and if I stop using the evaluate_setting_priority macro the build time goes from 8+ seconds on my laptop to under 1 second. I'm not sure why this happens, but it's pretty drastic.

Thanks to the zola devs, this change speeds up the build times massively. It takes less than 700 ms now on the same machine.  In evaluate_setting_priority change:
 {%- set current_section = get_section(path=last_ancestor.0) %}
to:
{%- set current_section = get_section(path=last_ancestor.0, metadata_only=true) %}

Thank you so much! Sorry I didn't have time to check this earlier, but I'm really happy you managed to solve it! Merging the PR now.


fix: contract class splitArgsAndOptions

Motivation and Resolution
fix type splitArgsAndOptions
Usage related changes
type updated to represent better what it is
Checklist:

[ ] Performed a self-review of the code
[ ] Rebased to the last commit of the target branch (or merged it into my branch)
[ ] Linked the issues which this PR resolves
[ ] Documented the changes in code (API docs will be generated automatically)
[ ] Updated the tests
[ ] All tests are passing

OK i belave at least now i fixed type

Yes, that is a more accurate representation.
I believe the spread Calldata branch can also be removed ([...Calldata] | [...Calldata, ContractOptions] from the reworked ArgsOrCalldataWithOptions type). It is supported by splitArgsAndOptions itself, however, splitArgsAndOptions is in all its usages followed by the getCalldata() utility that filters the argument based on the __compiled__ marker which isn't propagated if the Calldata is spread. This means that the only spread input that should be able to work is method(...Calldata, { parseRequest: false }).
I don't see any utility in restoring the full support, so refactoring the code to exclude the supported scenario seems preferable, it should also simplify detecting whether a ContractOptions object is used.

I agree focused on this pr to fix type and new one to propose solution


deleted files that shouldnt be there

fixes #224

dont really need a review, im just removing the files.


Tls13 support for greater than net5

Thank you @samisil for this!  Merging now and will update NuGet momentarily!

Thanks again @samisil
NuGet: https://www.nuget.org/packages/CavemanTcp/1.3.9
Commit: https://github.com/jchristn/CavemanTcp/commit/72da1caf8e2dd9f19534ab0f8895ec9bebc0322c


Callback until long press end

Hi, thank you for this awesome library
Is there a way to to have callback until long press finish ? cause i use backend call

Hi, @OuSS-90. Thanks for the kind words. I'll add a longPressEnded callback (that executes immediately after a long press gesture ends) in the next release.

Thank you 👍

@OuSS-90 Check out v2.4.0 when you have a moment.

Thank you very much 👍

I fixed some code for longPress.
When there is long press, usually we don't want increment and decrement to be called so i added this code in both incrementValue() and decrementValue():
// Increment function
if longPressTimer == nil {
   delegate?.KWStepperDidIncrement?()
    incrementCallback?(self)
 }

// decrement function
if longPressTimer == nil {
     delegate?.KWStepperDidDecrement?()
     decrementCallback?(self)
}

basically i call delegate only if there is no longpress

Thanks for the update. If that’s what you’re after, a flag to indicate when a long press gesture is happening would make more sense. Then, you can guard against doing additional work (such as hitting the backend) in your callbacks. Or, you can try https://github.com/kyleweiner/KWStepper/pull/7#issuecomment-301321280, which avoids needing to concern yourself with the long press gesture entirely. For the former, please feel free to open another issue or submit a pull request. I'm not certain this feature is within the scope of the project though.


fig consistency

in our multiscale power maps, we have 3 different axis labels:

in fig 1, we write " X scales"
in fig 3, we write "# of X Neighbors"
in fig 4, we write "X Neighbors"

let's pick one and use it for all 3?
i vote we try:

"# of X Neighbors", but if it doesn't fit, we revert to:
"X Neighbors (#)", then
"# X Neighbors", then,
"X Neighors"

what do you think?

X Neighbors looks good enough.
Changed all figures accordingly


Type support for RAML 1.0

Hello,
I have noticed that parser does not support parsing examples/definitions of responses with new "Type" that was introduced in RAML 1.0, however it still support schema only which is deprecated and will be eventually removed.
Since I want to use RAML 1.0 with all new feature fork will be created and this functionality(Type definitions) implemented.
However for sure it will be useful for someone else too, so can we define and discuss everything regarding this so the output of the commit will be best generic code.
/test:
  get:
    responses:
      200:
        body:
          application/json:
            type: TestType

types:
  TestType:
    type: !include type/index.raml
    example: !include example/index.json

Example is pure json and just need to be included, but type is an object and should be parsed with all it's parameters.
type: object
properties:
    id: integer
    name: string
    created:
        type: datetime
        format: rfc3339
    modified:
        type: datetime
        format: rfc3339

Basically it should support keys defined in specification:

https://github.com/raml-org/raml-spec/blob/master/versions/raml-10/raml-10.md#type-declarations

Just saw a Merge request regarding this, will test it out and give input on this.
https://github.com/alecsammon/php-raml-parser/pull/107

@vbartusevicius What is your progress on this? As it seems it just needs to be linked to https://github.com/alecsammon/php-raml-parser/blob/master/src/Body.php#L87 and Types should be parsed into the Response object.
Can I help you with something or do you have things done locally already?

Just ran into this myself, I have been working on implementing this for a project at my work and when I wanted to validate the requests and responses against the spec I discovered empty bodies. I am guessing this has to do with the missing 1.0 feature implementation as this is what I am using in my spec too. I will test the pull request and give feedback on it as well.

To add a fully functional support of Type into the library following examples (and more) should be supported.
Simple Type defintion
types:
  TestType:
    type: object
    properties:
        id: integer
        name: string
        created:
            type: datetime
            format: rfc3339
    example: !include example/test.json
/test:
  get:
    responses:
      200:
        body:
          application/json:
            type: TestType
  post:
    body:
      type: TestType
    responses:
      200:
        body:
          application/json:
            type: TestType

Using Traits
traits:
  secured:
    usage: Apply this to any method that needs to be secured
    description: Some requests require authentication.
    headers:
      access_token:
        description: Access Token
        example: 5757gh76
        required: true
  paged:
    queryParameters:
      numPages:
        description: The number of pages to return
types:
  TestType:
    type: object
    properties:
        id: integer
        name: string
        created:
            type: datetime
            format: rfc3339
    example: !include example/test.json
/test:
  is: [ secured ]
  get:
    is: [ paged ]
    responses:
      200:
        body:
          application/json:
            type: TestType

Using Traits and ResourceTypes
resourceTypes:
  searchableCollection:
   get:
      queryParameters:
        title:
          description: Return values that have their title matching the given value
traits:
  secured:
    usage: Apply this to any method that needs to be secured
    description: Some requests require authentication.
    headers:
      access_token:
        description: Access Token
        example: 5757gh76
        required: true
  paged:
    queryParameters:
      numPages:
        description: The number of pages to return
types:
  TestType:
    type: object
    properties:
        id: integer
        name: string
        created:
            type: datetime
            format: rfc3339
    example: !include example/test.json
/test:
  type: searchableCollection
  get:
    is: [ secured, paged ]
    responses:
      200:
        body:
          application/json:
            type: TestType

Support for injectining variables should be implemented too
traits:
  secured:
    queryParameters:
      <<tokenName>>:
        description: A valid <<tokenName>> is required
/books:
  get:
    is: [ secured: { tokenName: access_token } ]

Specification states:
A resource type, like a resource, can specify security schemes, methods, and other nodes. A resource that uses a resource type inherits its nodes. A resource type can also use, and thus inherit from, another resource type. Resource types and resources are related through an inheritance chain pattern. A resource type definition MUST NOT incorporate nested resources. A resource type definition cannot be used to generate nested resources when the definition is applied to a resource. A resource type definition does not apply to its own existing nested resources.

A trait, like a method, can provide method-level nodes such as description, headers, query string parameters, and responses. Methods that use one or more traits inherit nodes of those traits. A resource and resource type can also use, and thus inherit from, one or more traits, which then apply to all methods of the resource and resource type. Traits are related to methods through a mixing pattern.

Functions should be understood too defined at functions.
Examples were taken from specification and tested with npm raml2html library which gave expected output.

@alecsammon what do you think of completely breaking 0.8 compatibility and redoing library to support raml 1.0 only?

From my point of view we need 4 different objects(Type, Trait, QueryParameter and ResourceType) which holds all available definitions and then has to be applied in correct order to the current Body that is being processed.
Order will probably be parsed to and read inside Resource object since it's endpoint/method specific.

Hi @AnzeDspot - sorry I'm not very active in this project at the moment. My view would be to just switch to supporting 1.0, and ensure that we create a major version. We can then update the README so that anyone needing support for 0.8 knows to use < v3 of this library.

@AnzeDspot I agree with @alecsammon redoing the entire library looks like a bit too much to me. Also, how much is the 1.0 spec breaking 0.8? As far as I can tell there seems to be mostly adding of functionality and some small tweaks. I did only briefly look at the full spec though so I might have missed some.
I did however already find that we need to do some fundamental changes to the library to get the new objects (like you mentioned) in and be able to access them later by reference when processing the bodies.

@InfopactMLoos I did not mean to redo the whole library since it supports a lot of things that still needs to be supported at 1.0 like JsonSchema, XML, ..., however if it's easier to break some current functionalities to support 1.0 features we should do that.
Currently I fall back to RAML 0.8, since it's supported and works, but still can help to make it compliant with  RAML 1.0.
My thoughts are that we should start by making 2 types of tests(0.8 and 1.0), leave 0.8 as is and hopefully they still all pass when library is extended/redone, but for 1.0 we should merge all used fixtures together in one documentation and make it 1.0 compliant.
After that library can be changed and tests run against that 1.0 documentation which will have to be fully supported and some even extended(Types, Traits, ...).
Could be a little more work to do, but in my opinion it's worth it since we are talking about 1.0 version. Old 0.8 schemas are in json/xml and validated in that way so for 1.0 validation will have to be done in different way(not using justinrainbow json validator library).

Ok great then we are on the same page, just a misunderstanding. As @alecsammon suggested we should commit to a new v3.0 branch and update the documentation that v3.0 branch implements RAML 1.0, possibly breaking compatibility with 0.8. Of course we'll try to be backwards compatible but if BC-breaks are unavoidable I too favor RAML 1.0 functionality since most features are needed in enterprise environments in my opinion.
Small side-note to keep in mind, JSON/XML schemas are deprecated NOT yet removed in 1.0.
I have already put some work in implementing, continuing on the pull request of @vbartusevicius. Right now I have implemented all the transformation functions and copied the test fixtures from your comments. I still have to write unittests (I know, should have done that beforehand but oh well).
I had also made a small POC for the validating. I looked at the way the justinrainbow JSON validator works and it seems to me that if we convert the parsed RAML types to json with the json_encode function we can reuse the same validation code. Maybe a bit dirty but if it works we won't have to completely rewrite the validating of all the types.
I will push my code as soon as I have tidied up my work and properly unit tested it. Then I can really use your (and others) feedback to make this a great improvement.

Sorry for the long radio silence. After re-evaluating the RAML type validation I determined that the reusing of the justinrainbow JSON validator wasn't a good enough choice. It seems that the validation done by RAML types is, although compatible with JSON schema, slightly different (more refined). Since validation is an important part of an API, I decided that an as accurate as possible implementation was required. Also the fact that for precise validation of the RAML file I have to implement the types anyway, it made all the more sense to do so.
At this moment I have implemented all types and have successfully parsed the RAML into the right types. The only thing left now is adding the ability for response / request body validation, which would mean giving all types a validation method. I am commiting and pushing my current progress so other can check it out. I will then start implement the validating calls into the types.

Hm ok it seems I am not able to create a new branch on the upstream (this) repo. @alecsammon , can you create a v3.0 branch? Then I can send in my pull request. I don't want to pollute the master branch.

@AnzeDspot while awaiting @alecsammon maybe you can already review my current work? See my fork. I have updated the README with a quick notice about a work-in-progress 1.0 specification and created a small TODO list with missing chapters from the specification. Also validation has been implemented but only in its most basic form, validate returns true or false. What I am now planning to do is implement the storing of errors while validating and then create an getErrors method on the validator which should wrap around the type's validate methods. Current implementation does have not such a wrapper yet.
What I am currently considering is what to do with the old validator classes, which were used for the 0.8 deprecated schema element. I am thinking about replacing them with a new validator class for the types. I can think of two reasons: one, since the XML and JSON schema's have been incorporated inside the types we can redirect the logic to them (I have already copied the validation logic there). Reason two would be that for namespace and class naming Validator would be logical and therefore conflicting with the current class and namespace used by schema (Raml\Validator). And I don't feel like creating something like TypeValidator because that might be confusing.

Any update on this @InfopactMLoos ?

I have written all the validator class code last week, still need to clean up the code a bit before pushing to github. Sadly I have received little feedback on my previous work so not sure if I am going the right way for the community at large here. However since my company needs this code to work I will make it work the way we need it in the best way I can. I will try to finish my current code tonight but I am busy with other work all week so it might be a couple days later. If you can give some feedback on my code that will be great!

@InfopactMLoos  I'm happy to look over what you have done when I have some free time, I'm not sure what overall direction we are wanting this library to take with 1.0 spec.
We are in the same situation where we are implementing the 1.0 spec for a new framework, however this library isn't quite there yet.

RAML 1.0 support is provided in version series 3
RAML 0.8 support is provided in version series 2

Any update on this? I just ran into this issue today and I saw that there was already an issue for this. I'm running version 4.3.2 at the moment and the error is still thrown.

@kanduvisla what is the error you get?
Can you provide samples?

PHP Fatal error:  Uncaught Raml\Exception\InvalidQueryParameterTypeException:
The type BusinessId is not a valid query parameter type.
in [...]/vendor/raml-org/raml-php-parser/src/NamedParameter.php:483

The RAML File has the following content:
... (ignored) ...

types:
  BusinessId:
    type: string
    maxLength: 50
    description: The ID provided to identify your company.

... (ignored) ...

/checkouts:
  get:
    queryParameters:
      businessId:
        required: true
        type: BusinessId

... (etc) ...

The error makes sense, because if you look at the class, the method in question checks the property $validTypes:
    /**
     * Set the type
     *
     * @param string $type
     *
     * @throws \Exception
     */
    public function setType($type = 'string')
    {
        if (!\in_array($type, $this->validTypes, true)) {
            throw new InvalidQueryParameterTypeException($type, $this->validTypes);
        }

        $this->type = $type;
    }

... and $validTypes is a predefined list that is not manipulated by other processes:
    /**
     * Valid types
     *
     * @var string[]
     */
    protected $validTypes = [
        self::TYPE_STRING,
        self::TYPE_NUMBER,
        self::TYPE_INTEGER,
        self::TYPE_DATE,
        self::TYPE_BOOLEAN,
        self::TYPE_FILE,
        self::TYPE_DATETIME_ONLY,
        self::TYPE_DATE_ONLY,
        self::TYPE_TIME_ONLY,
        self::TYPE_DATETIME,
        self::TYPE_ARRAY,
    ];


Replace stub for mongoDB lesson (create many records)

Add hints
Add solution

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉
IMPORTANT INFORMATION!!!
@ailyntang Until further notice, we are not going to be accepting/merging any new PRs for changes to the Guide articles in the certifications folder. There will be an announcement at a later date on the forum about how to update the curriculum related articles, but for now, just hold on to your solutions. We will work on approving the ones you have already created though.
Thank you.


[YUNIKORN-979] Add e2e test coverage for the admission controller

What is this PR for?
Add the e2e test for the admission controller to increase test coverage.
What type of PR is it?

[ ] - Bug Fix
[x] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
YUNIKORN-979
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

@lowc1012 can you have a look at the two lint issues?
The kClient is a strange one to get. Not something you would expect. It is linked to not using the Hungarian notation. The go lint code specifically picks up on the k which stands for kind in that case. Rename the variable to kubeClient or just to client can be used.

One other thing: should we check that the admission controller is up and running by calling the health check before we run these tests or is that called earlier in the e2e tests?

@wilfred-s Thanks for your explanation.
About the second comment, Do we have the health check API endpoint for admission controller? If not, should I implement it in the PR or create another JIRA for that? Thank you!

I have no idea why the test for namespaceBlacklist failed.
In the test, I created a pod in kube-system namespace, it should not be mutated by admission controller.
YUNIKORN-1084 may address this problem.

@craigcondit can you please check if the e2e test failed as expected and that this will work in combination with your changes. If we need changes for YUNIKORN-1084 I think we need to commit this first and add changes to that PR.

@craigcondit can you please check if the e2e test failed as expected and that this will work in combination with your changes. If we need changes for YUNIKORN-1084 I think we need to commit this first and add changes to that PR.

Yes, this test will fail currently because the changes from YUNIKORN-1084 have not yet been merged (the existing behavior is not correct, but this test assumes it is). I don't think we need changes to this, but YUNIKORN-1084 needs to go in first to get the admission controller to exhibit the correct behavior.

@lowc1012 can you re-base this on current master? I think all the admission controller fixes are in now, so hopefully the tests pass.


App does not show the correct amount to the user when price has fractions

Buying an item for 1.01 shows 1.00 to the user although the correct amount is processed internally. Bug can be found here: https://github.com/mposSVK/acceptSDK-Android/blob/master/demo/demo/sample/src/main/java/de/wirecard/accept/sample/AbstractPaymentFlowActivity.java#L179

Yes, wrong preparing mock data in demo app.
pls use:
new Bigdecimal("1.01")...for add amount of item, and
AcceptSDK.getPaymentTotalAmount().scaleByPowerOfTen(amountCurrency.getDefaultFractionDigits())
for preparing amount units
Demo App is using amount "10.0" ;)
thx. Good point
Will be fixed in new version soon.


Add every HTML5 attribute and event emitter builder

See:

https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes
https://developer.mozilla.org/en-US/docs/Web/Events

Added a bunch of new events and attributes in v0.8.0. :)

All attributes on the doc list should be added by #16.
Some more events where added as well, but I see now that the doc list has more that are missing.


move deps in root cargo.toml

closes #134

Current dependencies on/for this PR:

main

PR #195   👈

This stack of pull requests is managed by Graphite.

@pepyakin can you review this one? The changes look like they'd work, to me, but does it accomplish what you wanted it to?


crypto: export errors

Exports errors from the crypto module per #1140 for the following submodules:

[x] armor
[x] ed25519
[x] encoding
[x] merkle
[x] sr25519
[x] xchacha20poly1305
[x] xsalsa20symmetric

On an unrelated note: I noticed that the following break statement is ineffectual if the intention is to break out of the outer loop. I am not sure if this posses a problem.
https://github.com/cometbft/cometbft/blob/77fa8a00c524474a055d95bd5d522491c3bdbea3/crypto/merkle/proof.go#L207-L217

PR checklist

[x] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[x] Updated relevant documentation (docs/ or spec/) and code comments

Please also run the linter (make lint from the root of the repo) prior to committing and pushing changes.

You're welcome, @thanethomson!
Sorry about that. I was under the impression that a different naming convention is used for structured errors (which also what the standard library appears to do). But I've pushed the requested changes.

Please also run the linter (make lint from the root of the repo) prior to committing and pushing changes.

The linter is flagging the use of naked returns which is unrelated to the changes I made in this PR. Do you want me to fix it anyway?

The linter is flagging the use of naked returns which is unrelated to the changes I made in this PR. Do you want me to fix it anyway?

That's odd - I don't get any linter errors when running make lint on main. Any idea as to why that'd be?

That's odd - I don't get any linter errors when running make lint on main. Any idea as to why that'd be?

I don't know. I am guessing the lint was added after the last change to the file? I think the checks are only done on modified files instead of the entire repository which would explain why it could only pick it up after a change was made albeit in an unrelated part.

The linter can be a little finicky. If it fails, please do fix the lints.

I pushed the fix so it should pass now.


Set PATH in boot2docker example shell function

I installed nsenter following the instructions given in Chris Jones'
post to get setup with boot2docker on Mac OS X. I added his example
script to run /bin/bash if none is given (which is based on the shell
function in nsenter's README), and upon trying to enter my container,
got this error message:
/var/lib/boot2docker/docker-enter: line 43: importenv: not found
error in run: exit status 127

Evidently, /var/lib/boot2docker/importenv is not on the path, so this
change sets PATH when SSHing to the boot2docker VM and successfully
enters the target container with no error message.

I was going to leave a comment on that post, but comments on it are disabled now. Hopefully I've tracked down the right Chris Jones...hey, @yaychris! Thanks for the great post, it was really helpful (until I hit this error anyway :wink:).

I think #70 solves this issue; does it?

Looks like it does! I guess I made the mistake of trying to fix a problem before seeing if it already had a different solution (I didn't look at the open issues here). Thanks!

No problem! Thank you!
On Sat, May 23, 2015 at 9:42 PM, Michael Brandt<EMAIL_ADDRESS>wrote:

Looks like it does! I guess I made the mistake of trying to fix a problem
before seeing if it already had a different solution (I didn't look at the
open issues here). Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/jpetazzo/nsenter/pull/71#issuecomment-104978684.

--
@jpetazzo https://twitter.com/jpetazzo
"Is it safe to run applications into containers?"
http://www.slideshare.net/jpetazzo/docker-linux-containers-lxc-and-security


Use python dataclasses for Repo info and Issue

/rebase

Thanks for the contribution, @djanmamur! 🙌🏼


WIP: Infinite Pan/Tilt presets

This addresses #1070
There's still a bug: Most fixtures have this channel order:

Pan
Tilt
Pan Continuous Rotation
Tilt Continuous Rotation
...

The EFX function sets channel 3 and 4 then (which is incorrect, as it should access the angle by setting 1 and 2). I don't know how to fix this – maybe by changing something in qlcfixturehead.cpp#L171 or efxfixture.cpp#L98?

Ok, I'm gonna merge this one and fix caching precedence.

@fxedel  I have just tried ADJ Asteroid in Basic and Standard mode and EFX controls the right channels (1 and 2)
Where did you see the wrong behaviour ?


WebView load  file

Now this  i have a file  eg:'hello.html'  ,i want to load this file from webview.so there is a good case?

You can use the source prop on WebView: http://facebook.github.io/react-native/docs/webview.html#source

Hey and thanks for posting this!
There's an awesome place to post code / ask question first before filing an issue: StackOverflow. It's the best system for Q&A. Many people from the community hang out there and will be able to see and answer your question. This lets us use the github bug tracker for bugs.
I'm posting this here because github issues haven't been working very well for us and because StackOverflow is so much better. Thanks for reading! :)

Closing this as it's a question rather than a bug and questions should be asked on StackOverflow.
@jiangqqlmj I have meta question. Did you see the New Issue Guidelines when creating this issue? Is there anything we could improve there? Is the text too long or unclear for example?


fix: add support for imagePullPolicy for installer daemonsets (#395)

This PR implements support for ImagePullPolicy which is present in the CRD but not actually used (default Always).

@stevenhorsman @fidencio would you be able to re-run the required tests for this PR?
This is ready to be merged, but pending since quite some time..

@stevenhorsman @fidencio would you be able to re-run the required tests for this PR? This is ready to be merged, but pending since quite some time..

I've tried closing and re-opening, but it might need a rebase as the jobs have changes since then

@teodor-pripoae would you be able to rebase the PR? Sorry for the delay :-(

Otoh, may be rebase is not needed. All required tests are passing and this can be merged..


Enable checking email address

@rogerwangcs might want to consider passing in the user email if we want to have some user to have early access to dataproc in prod.

Is this for supporting identity overrides? I didn't need to make additional change to UI flagsmith logic to get it to pick up email_address segment overrides

Is this for supporting identity overrides? I didn't need to make additional change to UI flagsmith logic to get it to pick up email_address segment overrides

ya because I only implemented the trait things in the ui :p


Release v1.47.0

Seems like bugfixes and workarounds for kernel bugs are slowing down, so it seems like time to release a new version? I think this dropped support for some old platforms, so it would be a minor release in semver, and not just a patch release. Any thoughts or things people are eager to include?
Changes: https://github.com/libuv/libuv/compare/v1.46.0...v1.x

https://github.com/libuv/libuv/pull/4187 - hopefully uncontroversial

#4075 isn't ready, IMO. Left a review. Don't hold up the release for that, it can wait until v1.48.0.

II forgot only @bnoordhuis and @saghul and authorized to make releases now on the ftp server https://github.com/libuv/libuv/issues/4081#issuecomment-1628913340

I have access myself as well. From this comment it seems you should have access? https://github.com/nodejs/build/issues/3353#issuecomment-1547578054

Not sure, since I couldn't get past the firewall. But I think we publish to GitHub now, so I just need to update the release script next week to do that instead

Fair enough. Now that I remember, there was a jump host you have to point to to access the actual server. Let me know if you need that info and I'll send an email with the details

1.47.0 was released a month ago, but it's still not included in Node.js because of some issues: https://github.com/nodejs/node/pull/50650
Will you release 1.47.1 with the fixes in this year?
To fix this bug https://github.com/nodejs/node/issues/48673 finally.


QuickFix list support cycle select

For example, in following scene, I just need to press ↑ to select the last item.

Duplicate of #55111
That is the standard MacOS context menu. VS Code can't change it's behavior


Bug: i18n error on startup

I just clone and started up the project
The i18n seems like not recognize the locales.

Sorry, I refactored this app not long ago, and I don't have time to modify it yet. Just modify the i18n key. Maybe it will work.
I'm looking for a better i18n solution that can read i18n resources from the backend.
I will work on completing the react version first, then the vue version.


Switching To Debian Based Installation System (PAL Mini)

Ported PAL-USB installation to Debian from a script-based system. Debian source files are maintained at https://github.com/DreamVu/debian_installations_files/tree/3d-object-detection-tracking and https://github.com/DreamVu/Firmware/tree/I006_Tracking_JPP/deb.

I didn't know the new way of creating lib files, but that lib file need change.


Suddenly typed characters was doubled automatically some lines below (incl. screencast)

Description
This funny issue isn't reproducable for me and i don't know if it is a feature or a bug ;)
However, at some point Atom began to double my typing input some lines below.
I recorded a screencast https://www.youtube.com/watch?v=TmNzQKavbQY&feature=youtu.be
It suddenly began and it stopped after switching from the IDE-Window to youtube for uploading the screencast and returning back to IDE.

bQY
Steps to Reproduce
not possible. It suddenly began and it (possible) stops with a switch between IDE-Window and Chrome and back.
Versions
$ atom --version
Atom    : 1.7.3
Electron: 0.36.8
Chrome  : 47.0.2526.110
Node    : 5.1.1

OS: OSX El Capitain.

Are you sure you aren't cmd-clicking on other lines by accident?  That creates an extra cursor.


Adding check for lists of columns with a column not in the index

What do these changes do?

Related issue number

[x] passes git diff upstream/master -u -- "*.py" | flake8 --diff
[x] passes black --check modin/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-Performance-PRB/175/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Modin-PRB/237/
Test PASSed.

Fixes #335


load balancer does not seem to work on centos7

@iwilltry42 this repo is a very useful k3d demo, with many tips packed.
Thanks for sharing it !
It works flawlessly on my ubuntu 20.04 machine.
However I noticed that on a Centos7 VM the load balancer does not work.

If you run make demo and stop after the first stage (cluster creation) on Ubuntu I get:

> curl localhost:8080
404 page not found

On a Centos7 machine:

> curl localhost:8080
Empty reply from server

The following steps do not change this output.
Inspecting container k3d-demo-serverlb logs and processes I could not pinpoint the problem.
At least pinging from one container to another using theirs name works:
#### inside k3d-demo-server-0
/ # ping k3d-demo-agent-0
PING k3d-demo-agent-0 (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.077 ms
64 bytes from <IP_ADDRESS>: seq=1 ttl=64 time=0.080 ms

I wonder if this is a tiny config glitch, or some serious low level limitation...
What do you think ?

Well, well, I found the cause for the issue.
It is completely independent of k3d and embarrassingly basic..
The Centos7 VM is behind a corporate firewall.
The pods in name space kube-system could not load their images.
That became clear doing:
kubectl get -A

Closing.


adds temporalBaselineDays search keyword

Adds temporalBaselineDays search keyword, provides exact matches
(See #198 )

Codecov Report

Merging #199 (b11690f) into master (f0df348) will decrease coverage by 0.09%.
The diff coverage is 92.30%.

:exclamation: Current head b11690f differs from pull request most recent head 7f0aede. Consider uploading reports for the commit 7f0aede to get more accurate results

@@            Coverage Diff             @@
##           master     #199      +/-   ##
==========================================
- Coverage   89.58%   89.49%   -0.09%
==========================================
  Files          51       51
  Lines        1930     1933       +3
==========================================
+ Hits         1729     1730       +1
- Misses        201      203       +2

Flag
Coverage Δ

unittests
89.49% <92.30%> (-0.09%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

asf_search/ASFSearchOptions/validator_map.py
58.82% <ø> (ø)

asf_search/CMR/field_map.py
100.00% <ø> (ø)

asf_search/search/search.py
100.00% <ø> (ø)

asf_search/search/search_generator.py
95.90% <91.66%> (-1.58%)
:arrow_down:

asf_search/CMR/subquery.py
100.00% <100.00%> (ø)

asf_search/download/download.py
65.00% <100.00%> (ø)


plugin doesn't work if port 9650 not opened - 1.9.0

Hello,
Since 1.9.0 version, root cause analysis is communicating with performance analyser through port 9650. If this port is not open, the plugin crash. Anywhere in your documentation you mentioned this port. Users who not having this port opened by default will face the crash.
Thanks,
Julien Elk

I have also stumbled on this. Unfortunately, opening 9650 is not an option.
I'm getting a ton of log spam in /tmp/PerformanceAnalyzer.log because of this, which is filling up the partition.
Is there any way to disable this behavior?
I'm on opendistro 1.10.1.

Ping?
This is still an issue, and spam logs in are filling up /tmp.
What is port 9650 used for? There is, as far as I can tell, no documentation about it. How can I configure it to use TLS, and how can I configure the logs to go somewhere other than /tmp?
If it's not possible, how can I disable the plugin completely, to avoid having it filling up /tmp ?

I also hit this issue. Port 9650 is blocked in firewall but on the other side there is nothing listening on it on the destination server?
What should we do?
(attached a log output)
Additionally there seems to be some files missing (see systemd-output attached)
ODFE-205.txt

I have managed to stop and disable opendistro-performance-analyzer (systemctl stop opendistro-performance-analyzer), which stops the log spam, at least. It also disables the performance analyzer though.

Hey zeising@
You can set up the new log directory in the file log4j2.xml which is present in directories:
/usr/share/elasticsearch/plugins/opendistro_performance_analyzer/pa_config
/usr/share/elasticsearch/performance-analyzer-rca/pa_config
The parameter File Name with value PerformanceAnalyzerLog contains the path for the PerformanceAnalyzer.Log
and with value StatsLog contains the path for the performance_analyzer_agent_stats.log
By default both of them are set to /tmp/.... You can modify it to whatever location you like and update both log4j2.xml in both the directories.
Restart the ES process should make the plugins pick up the new log directory. This solves the log directory problem.
We use the port number 9650 for internal node to node communication to transfer data between the nodes. Currently this is non configurable. We already have this on our roadmap to make it configurable.
Thanks,
Aditya

Hi @aditjind!
I have disabled the performance monitor for now, but I'll look into changing where it stores log.
Will it also be possible to protect the traffic on 9650 using TLS in a future update?

Hey zeising@
We currently have encryption in transit support for client and server communication (Rest and GRPC calls). We will support client authentication as well in the upcoming release.
The support to have the GRPC port as configurable will also be added. This PR is currently in review.
Thank you!

I can't see how the link you provided tells me how to protect 9650 using TLS.

Apologies, updated the link.


Text function: Block use of format string when input is non-numeric at compile time

Today we allow Text to be called with a string, Boolean, GUID, etc. along with a format string.  The format string appears to be ignored.   Canvas does the same.
> Text( "123.456", "$ ##.####" )
"123.456"

> Text( true, "$ ##.####" )
"true"

> Text( GUID(), "$ ##.####" )
"b9866596-378c-4d19-b79b-b967f39996a7"

This is dangerous because Excel supports coercion from string to number in this scenario.  Which is fraught with peril as this is a runtime check:

Where we are different from Excel is that we are strongly typed.  We can use that to provide a better, less error prone experience.
The desired experience:

For all types, Text without a format string works as it does today.
For number, date, date/time, and time - no change.  Format string works as it always has.
For all other types, where a format is provided, we will generate a compile time error.  This avoids us needing to support the string coercion to number that Excel supports today, something that we can add in the future if the need arises as we would be removing a portion of this block.

There are tests for this change in https://github.com/microsoft/Power-Fx/blob/gregli/text-excel/src/tests/Microsoft.PowerFx.Core.Tests/ExpressionTestCases/Text_ExcelCompat.txt.

Fixed in 1711


convnet with cuda (v11) support not working

hello there,
I am trying to make convnet cuda example work with CUDA v11.8 and it doesn't work.
It either ignored cuda and runs on CPU or it crashes with index out of range [-1]:
/gorgonia/examples/convnet_cuda$ time go run -tags='CUDA'  main.go -epochs 1 2
Epoch 0 4 / 600 [>----------------------------------------------]   0.67% 04m58spanic: runtime error: index out of range [-1]

goroutine 27378 [running]:
gorgonia.org/gorgonia.(*maxPoolDiffOp).f64s(0x8?, 0x64, 0x80, 0x3?, 0x3?, 0x31, 0x9, 0x9, {0xc002200000, 0x99200, ...}, ...)
        /go/pkg/mod/gorgonia.org/gorgonia@v0.9.15/op_nn.go:1075 +0x365
gorgonia.org/gorgonia.(*maxPoolDiffOp).do(0xc0000da600, {0xeba638, 0xc01d298800}, {0xeba638, 0xc01d30c300}, {0xeba638, 0xc01d30c400}, {0xeba638, 0xc01cd20900})
        /go/pkg/mod/gorgonia.org/gorgonia@v0.9.15/op_nn.go:1023 +0x368
gorgonia.org/gorgonia.(*maxPoolDiffOp).UsePreallocDo(0xc0000be0f0?, {0xeb23d0, 0xc01d298800}, {0xc01cff4040?, 0x1?, 0x1?})
        /go/pkg/mod/gorgonia.org/gorgonia@v0.9.15/op_nn.go:973 +0xc7
gorgonia.org/gorgonia.(*execOp).exec(0xc01b455a40, 0xc0000be0f0)
        /go/pkg/mod/gorgonia.org/gorgonia@v0.9.15/vm_tape_nocuda.go:60 +0x43a
gorgonia.org/gorgonia.(*tapeMachine).runall(0xc0000be0f0, 0x0?, 0x0?)
        /go/pkg/mod/gorgonia.org/gorgonia@v0.9.15/vm_tape.go:235 +0x102
created by gorgonia.org/gorgonia.(*tapeMachine).RunAll
        /go/pkg/mod/gorgonia.org/gorgonia@v0.9.15/vm_tape.go:208 +0x158
exit status 2

real    0m3.220s
user    0m8.269s
sys     0m0.492s

I have ran codagen successfully and I am wondering if I simply should switch to CUDA v10.2 since it is already tested against as stated on the website.
Relates to #522 #515 and even potentially as far as #376.

for now, switch to cuda 10.2 . CUDA11 is still being worked on and it's been difficult for me given that I'm sick fairly often nowadays

I was wondering if you could give any pointers (code, walk-though, previous PR...) as to what is needed. If it is not insanely convoluted I have some spare time, I could give a try for educational purposes.


What the difference between  “.dependencies.json” and “.json”  in ./data?

Hi  cyrillef
I am trying to make this project work in my localhost to learn the forge service.
there is a problem when I uploading files.
In 'lmv-porject.js'    line 311~316
            function (err) { //- All tasks are done
                    if (err !== undefined && err !== null) {
                        console.log(err)  // I add one line here to see the err message
                        console.log ('Something wrong happened during upload') ;
                        callbacks2 (err, 3) ;
                        return ; `

the output windows shows
 async uploading(s)
async uploading 5681152-rst_basic_sample_projectrvt
5681152-rst_basic_sample_projectrvt
Failed to upload 5681152-rst_basic_sample_projectrvt
{ Error: ENOENT: no such file or directory, open 'D:\DB\coding\forgeweb\data\5681152-rst_basic_sample_projectrvt.json'
    at Error (native)
  errno: -4058,
  code: 'ENOENT',
  syscall: 'open',
  path: 'D:\\DB\\coding\\forgeweb\\data\\5681152-rst_basic_sample_projectrvt.json' }
Something wrong happened during upload

I didn't know the difference between “.dependencies.json” and “.json” in ./data,
I am freshman in coding. Hope you can help.
Otherwise, is this project updated the live demo in http://extract.autodesk.io/?

Sorry for the delay in replying, I rewrote the sample to use the Forge v2 API so I do not need this dependencies file anymore. That file was to describe the various dependencies between files, for example an ipt being a dependency of an iam. In Forge v1, you had to describe the dependencies, in v2 just zip all files together, and submit the zip.
I also rewrote the sample using v2, so the error above should not happen any more. Let me know if you still got issue,
-cyrille


Add istio deployment

Fixes: #585

 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.

 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.

Looks like Istio is installed and passes the test in the E2E/test for platform


Cross tenant limitation

If I understand correctly, it is a limitation of the product, not just of this article.
This article requires that the Azure AD tenant configured for authentication is same as the tenant for the subscription that holds your AKS cluster.
It is misleading to say that the procedure in this article cannot be done, but other procedures can. If this is a limitation of the product, please state it clearly.

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

ID: 42afd8de-b697-100c-98c7-ae8e6a6f1b92
Version Independent ID: 81674054-0ea2-210b-2e65-18a0f15617c1
Content: Use Azure RBAC for Kubernetes Authorization - Azure Kubernetes Service
Content Source: articles/aks/manage-azure-rbac.md
Service: azure-kubernetes-service
GitHub Login: @palma21
Microsoft Alias: jpalma

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

@torumakabe
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


added Smith College

Our major in Statistical & Data Sciences was approved by the faculty yesterday!

@beanumber congrats, that is exciting


Lepiter: hide title left spaces

If users type header like # Title 1, we want to hide #  (including spaces) part, not only #.

Cursor in place:

Cursor outside:


Enzyme output

In my unit tests, I used to render the components using the Enzyme shallow() function. The output printed by debug() looked like a regular tree component:
<TouchableOpacity accessibilityComponentType="button">
   <View style={{...}} onLayout={[Function: bound onViewLayout]}>
      <Image source={{...}} style={{...}} />
   </View>
</Component>

Using react-native-mock-render and jsdom, the tree nodes are not named anymore:
<Component accessibilityComponentType="button">
   <Component style={{...}} onLayout={[Function: bound onViewLayout]}>
      <Image source={{...}} style={{...}} />
   </Component>
</Component>

Is it intentional and how can I retrieve each node using the find() function in this case?

I'm running in the same issue. Is it an expected behavior? If it is, for instance, it mean that I can't use find('TextInput') :-1:

@untemps Ey man, I have created a pull request to fix it! #39

Hi all, thanks for posting this issue!  You should be able to work around this in the short term by finding components by their testID.
I'm happy to help get code changes merged to help address this issue as well. @dabit1, I can reopen that PR for you if you update it to resolve the errors that @mbutsko pointed out.


Added nav at the bottom of the docs

Coverage remained the same at 90.95% when pulling b4e62d3fd88218480fc568fc6472ac0a39352e10 on develop into 8bc0afcf2410b4075126333f8082c6f4eb177113 on master.

Coverage remained the same at 90.95% when pulling b27b8f20a9dc80c1b89f6f81bfd1e88fa03bcd15 on develop into 8bc0afcf2410b4075126333f8082c6f4eb177113 on master.

Done a bit more work on the documentation

Coverage remained the same at 90.95% when pulling 4de16d22850f910c282fee8fa878f303181de451 on develop into 8bc0afcf2410b4075126333f8082c6f4eb177113 on master.


Extract a table as JSON format with a JSON field

Is your feature request related to a problem? Please describe.
I have a JSON field in a table, and I would like to download all the table with the JSON format. The problem is that the JSON field is not recognized as a JSON. Is treated as a text, so is all quoted and unusable.
Describe the solution you'd like
I would like that the JSON field was recognized as JSON, and unquoted.
Describe alternatives you've considered
Don't really know how to fix it inside the database. I tried with regexes on the export itself, but I don't think this is the focus of the question.
Additional context
An example just to reproduce all this and be clear about what I'm asking:
create table test.example (
	race text,
	animal json
)

insert into test.example
values ('dog','{"name":"fuffi"}'::json)

select *
from test.example

I then extract the data in test.example and the output is this
{
"select *\nfrom test.example": [
	{
		"race" : "dog",
		"animal" : "{\"name\":\"fuffi\"}"
	}
]}

instead of
{
"select *\nfrom test.example": [
	{
		"race" : "dog",
		"animal" :
				{
					"name" : "fuffi"
				}
	}
]}

@solnaranu Hi, could you provide information about the database?

Sure! My specs:
Database: PostgreSQL 14.4
DBeaver: Community Edition <IP_ADDRESS>212041619
OS: Ubuntu 18.04.6
Driver: PostgreSQL JDBC 42.2.5

@solnaranu Hi, could you provide information about the database?

I suppose you meant @sofiarosati :smile:

You DBeaver guys are awesome (mean it!), but this might not be in your top priority.
So for anyone really needing it, this is my weird workaround with regex.
I extract using this query
select race, '#' || regexp_replace(animal::text, '"', '§', 'g') || '#' as animal
from test.example

(Any other special character combination you know won't be in your file is ok.)
The output is this
{
"select race, '#' || regexp_replace(animal::text, '\"', '§', 'g') || '#' as animal\nfrom example": [
	{
		"race" : "dog",
		"animal" : "#{§name§:§fuffi§}#"
	}
]}

And then I use this command to "clean" the JSON
sed -E 's/"?#"?//g' animals.json | sed -E 's/§/"/g'

So that it becomes like this
{
"select race, '' || regexp_replace(animal::text, '\"', '"', 'g') || '' as animal\nfrom example": [
	{
		"race" : "dog",
		"animal" : {"name":"fuffi"}
	}
]}

(it breaks the query part, but I don't need it, sorry lol)

Thank you for your request.

This feature would be very appreciated!

+1

This feature would be very appreciated!

+1

one alternative will be
SELECT json_agg(t.*)
FROM test.example t
;

but this add unnecesary escapes and splitting large set in several files


package:integration_test constraint on vm_service is preventing upgrade of dwds, vm_service

This is blocking https://github.com/flutter/flutter/issues/67322 as well as a dwds update that should add better handling for several web errors.
https://github.com/flutter/flutter/pull/69067/files
Since the latest package:integration_test depends on an old vm_service, updating vm_service forces a downgrade of package:integration_test

@dnfield

Should be fine to roll integration tests constraint forward

Could you take care of this?

This is a bit of a pickle. When I try to update it it complains that flutter_driver wants me to use 4.2.0, and version solving failed.
I guess we could use dependency overrides, but .. is there a better way?

ToT flutter_driver does not depend on vm_service: https://github.com/flutter/flutter/blob/master/packages/flutter_driver/pubspec.yaml
though of course it eventually will.. I think this sort of circular dependency issue implies that either package:integration_test needs to be in the SDK or it needs to be banned from use in the SDK.

One way for this to potentially work is that package:integration_test could take extremely wide constraints on any shared dependencies with the SDK...

I don't remember how but I'm pretty sure we fixed this.


Follow redirects

What kind of change does this PR introduce?

Add follow_redirects=True.
All other Supabase libs follow redirects by default.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.88%. Comparing base (641c24e) to head (6ef4698).
Report is 39 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #257      +/-   ##
==========================================
- Coverage   84.36%   82.88%   -1.49%
==========================================
  Files          12       12
  Lines         467      485      +18
==========================================
+ Hits          394      402       +8
- Misses         73       83      +10

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


bootstrap

Is it possible I use bootstrap instead of tailwind css? if so, which part should I change?

Hi @robertnicjoo.
You could export all views and replace Tailwind classes with Bootstrap.
Hope it helps.

Hi @robertnicjoo.
You could export all views and replace Tailwind classes with Bootstrap.
Hope it helps.


Added Rajabhat Maha Sarakham University for supporting domain rmu.ac.th

Please add Rajabhat Maha Sarakham University for supporting domain rmu.ac.th
webside:
https://www.rmu.ac.th

@karrung Pull request merged. Thank you!

@karrung Pull request merged. Thank you!

@karrung Pull request merged. Thank you!

@karrung Pull request merged. Thank you!


Testing Data

How can i Test model ?

@bansimendapara
Hi!Sorry to bother you !
I am not able to find NDSB data. So can you share with  me ?
Looking forward to your reply!
Thanks!

The data is removed from kaggle..
You can test on luna 16 challenge data


getting a "Cannot read properties of null (reading 'struct')" when clicking on the ketcher-canvas

I am using ketcher-react 2.8.0 and an instantiate it in my create-react-app using
import {RemoteStructServiceProvider} from 'ketcher-core'
import {ButtonsConfig, Editor} from 'ketcher-react'
import {Ketcher} from 'ketcher-core'
import 'ketcher-react/dist/index.css'
import styled from '@emotion/styled'
import {StandaloneStructServiceProvider} from "ketcher-standalone";

const GridWrapper = styled('div')`
  height: 100vh;
  width: 100%;
  max-height: 100vh;
  max-width: 100%;
  overflow: hidden;
  display: grid;
  grid-template-columns: 1fr 270px 320px;
  grid-template-rows: 1fr;
  gap: 0px 0px;
  grid-template-areas: 'Ketcher Panel Output';
  & > div {
    border: 1px solid grey;
  }
`

const KetcherBox = styled('div')`
  grid-area: Ketcher;
  height: 500px;
`
const structServiceProvider = new StandaloneStructServiceProvider();

const MoleculeSketcher = () => {
    return (
        <GridWrapper>
            <KetcherBox>
                <Editor
                    key={1}
                    staticResourcesUrl={process.env.PUBLIC_URL}
                    structServiceProvider={structServiceProvider}
                    errorHandler={(err) => console.log(err)}
                    onInit={(ketcher: Ketcher) => {
                        ;(global as any).ketcher = ketcher
                    }}
                />
            </KetcherBox>
        </GridWrapper>
    )
}
export {MoleculeSketcher};

when clicking on the canvas of catcher I getting this error:
Cannot read properties of null (reading 'struct')
TypeError: Cannot read properties of null (reading 'struct')
at Handler.onElementEdit [as f] (http://localhost:3000/static/js/bundle.js:124200:40)
at PipelineSubscription.dispatch (http://localhost:3000/static/js/bundle.js:229805:65)
at propsDialog$1 (http://localhost:3000/static/js/bundle.js:112312:38)
at RGroupAtomTool.click (http://localhost:3000/static/js/bundle.js:112292:9)
at useToolIfNeeded (http://localhost:3000/static/js/bundle.js:115580:26)
at Handler.f (http://localhost:3000/static/js/bundle.js:115626:24)
at DOMSubscription.dispatch (http://localhost:3000/static/js/bundle.js:229827:61)
Any help/suggestions much appreciated!

Hello @andreassteffen , thanks for creating this issue, can you please describe the exact steps required to reproduce the issue. Do you select any particular tool when clicking on the canvas? Are you clicking on an empty canvas or on a structure? etc.
Thank you!


check number of servers in configuration comparator

Resolves #945
Fix for the problem when Nginx is not synced & reloaded when you update an ingress with more than one host rule. The condition to check if the old and new configuration is unchanged returned true always, as the check for len(servers) was not there.

This change is

/lgtm


What is the experimental setting in the paper?

Hi, in Sec 4.1, the paper said that Unicorn in four tasks uses the same model parameters, does this mean that the Unicorn model is trained in a multi task manner? Meanwhile, I found that in the ablation study in Sec 4.6, the four results of  Unification in the Single task part of Table 7 are different from  the results in Tables 1, 3, 4, 6, what is the difference between the Unification setting in Table 7 and those in Table 1,3, 4, 6?

Hi, yes, Unicorn is jointly trained on datasets from four tasks and there is only one set of parameters across four tasks.
Table 1, 3, 4, 6 use ConvNeXt-Large as the backbone. While we use ConvNeXt-Tiny as the backbone for ablations.

Hi, yes, Unicorn is jointly trained on datasets from four tasks and there is only one set of parameters across four tasks. Table 1, 3, 4, 6 use ConvNeXt-Large as the backbone. While we use ConvNeXt-Tiny as the backbone for ablations.

Thank you for your reply.


Rule Proposal: no-zero-timeout

Meteor.setTimeout(foo, 0) can be used to defer the execution of foo. Meteor actually provides another function that does exactly this: Meteor.defer(foo).
The rule should warn when Meteor.setTimeout is used with a timeout of 0.

Added in https://github.com/dferber90/eslint-plugin-meteor/commit/d9824737a1d100ed34a1e51a221b193bf14cf7ac.
Commit got squashed with others while setting up semantic-release unfortunately.


Failures on [tests] Remove triple shift test skips from status files

There are new test failures on [tests] Remove triple shift test skips from status files.
The test
co19/LanguageFeatures/Triple-Shift/Constants_A01_t02 CompileTimeError (expected Pass)

is failing on configurations
analyzer-asserts-strong-linux
analyzer-asserts-weak-linux

cc @scheglov

cc @srawlins

@nshahan I'm working on a fix. But to clarify, the CL you linked to is an effort to delete or reduce status files? And in this case, the result was new test failures? I'm all for that; I didn't know we still had status files :P Just wanted to clarify.

The status files do still exist but only as a mechanism to skip running tests (and maybe to mark tests as slow and needed more time, I think). Since some of these tests for the triple shift operator have been around for a few years at some point we started skipping them.
Now that the feature is becoming a reality I wanted to start running these tests by deleting the lines in the status files that caused them to be skipped. Once they started running and a few started failing.


[BUG] Crash with future

Some time ago I was able to use 3arth and future, but it crashed and since then I haven't been able to use it anymore
I already deleted the "earthhack" folder
I already tried it with "-noverify"
I await your response thanks.
---- Minecraft Crash Report ----
WARNING: coremods are present:
Contact their authors BEFORE contacting forge
// My bad.
Time: 5/7/22 11:40 AM
Description: Initializing game
java.lang.ExceptionInInitializerError
at me.earth.earthhack.impl.managers.chat.CommandManager.init(CommandManager.java:100)
at me.earth.earthhack.impl.managers.Managers.load(Managers.java:80)
at me.earth.earthhack.impl.Earthhack.init(Earthhack.java:27)
at net.minecraft.client.Minecraft.handler$zzl000$initHook2(Minecraft.java:44655)
at net.minecraft.client.Minecraft.func_71384_a(Minecraft.java:536)
at net.minecraft.client.Minecraft.func_99999_d(Minecraft.java:378)
at net.minecraft.client.main.Main.main(SourceFile:123)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
Caused by: java.lang.NullPointerException
at net.minecraft.network.PacketBuffer.constant$zjd000$f$I(PacketBuffer.java:47180)
at net.minecraft.network.PacketBuffer.func_150793_b(PacketBuffer.java:315)
at net.minecraft.network.PacketBuffer.func_150791_c(PacketBuffer.java:361)
at me.earth.earthhack.impl.commands.KitCommand.(KitCommand.java:249)
... 13 more
A detailed walkthrough of the error, its code path and all known details is as follows:
-- Head --
Thread: Client thread
Stacktrace:
at me.earth.earthhack.impl.managers.chat.CommandManager.init(CommandManager.java:100)
at me.earth.earthhack.impl.managers.Managers.load(Managers.java:80)
at me.earth.earthhack.impl.Earthhack.init(Earthhack.java:27)
at net.minecraft.client.Minecraft.handler$zzl000$initHook2(Minecraft.java:44655)
at net.minecraft.client.Minecraft.func_71384_a(Minecraft.java:536)
-- Initialization --
Details:
Stacktrace:
at net.minecraft.client.Minecraft.func_99999_d(Minecraft.java:378)
at net.minecraft.client.main.Main.main(SourceFile:123)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:497)
at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
at net.minecraft.launchwrapper.Launch.main(Launch.java:28)
-- System Details --
Details:
Minecraft Version: 1.12.2
Operating System: Windows 10 (amd64) version 10.0
Java Version: 1.8.0_51, Oracle Corporation
Java VM Version: Java HotSpot(TM) 64-Bit Server VM (mixed mode), Oracle Corporation
Memory: 760641904 bytes (725 MB) /<PHONE_NUMBER> bytes (1760 MB) up to<PHONE_NUMBER> bytes (4096 MB)
JVM Flags: 9 total; -XX:HeapDumpPath=MojangTricksIntelDriversForPerformance_javaw.exe_minecraft.exe.heapdump -Xmx4G -XX:+UnlockExperimentalVMOptions -XX:+UseG1GC -XX:G1NewSizePercent=20 -XX:G1ReservePercent=20 -XX:MaxGCPauseMillis=50 -XX:G1HeapRegionSize=32M -Xverify:none
IntCache: cache: 0, tcache: 0, allocated: 0, tallocated: 0
FML: MCP 9.42 Powered by Forge <IP_ADDRESS>59 Optifine OptiFine_1.12.2_HD_U_G5 4 mods loaded, 4 mods active
States: 'U' = Unloaded 'L' = Loaded 'C' = Constructed 'H' = Pre-initialized 'I' = Initialized 'J' = Post-initialized 'A' = Available 'D' = Disabled 'E' = Errored
| State  | ID        | Version      | Source                        | Signature                                |
|:------ |:--------- |:------------ |:----------------------------- |:---------------------------------------- |
| LCHIJA | minecraft | 1.12.2       | minecraft.jar                 | None                                     |
| LCHIJA | mcp       | 9.42         | minecraft.jar                 | None                                     |
| LCHIJA | FML       | <IP_ADDRESS>    | forge-1.12.2-<IP_ADDRESS>59.jar | e3c3d50c7c986df74c645c0ac54639741c90a557 |
| LCHIJA | forge     | <IP_ADDRESS>59 | forge-1.12.2-<IP_ADDRESS>59.jar | e3c3d50c7c986df74c645c0ac54639741c90a557 |

Using GL 1.3 texture combiners.
Using framebuffer objects because OpenGL 3.0 is supported and separate blending is supported.
Shaders are available because OpenGL 2.1 is supported.
VBOs are available because OpenGL 1.5 is supported.

This is an issue on Futures end. Fortunately, because most people don't have that problem it might be one of Futures modules so if you find it you can turn it off.

Fine, while I hope they fix it here's a fix that will make you not crash but breaks the KitCommand: https://github.com/3arthqu4ke/3arthh4ck/actions/runs/2287200703

thanks


KAFKA-12898; Owned partitions in the subscription must be sorted

While investigating https://issues.apache.org/jira/browse/KAFKA-12896, I have noticed that the leader was always sending the same subscribed partitions but not always in the order. The group coordinator compares the provided subscription with the store subscription based on their bytes representation. So if the subscribed partitions are not in the same order, the group coordinator would consider that they are different and rebalance the group.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

I forgot to update few tests.

@ableegoldman Thanks for your comments. I have updated the PR and fixed the tests.

Failed tests are not related:
Build / JDK 11 and Scala 2.13 / org.apache.kafka.connect.integration.ConnectorClientPolicyIntegrationTest.testCreateWithNotAllowedOverridesForPrincipalPolicy
Build / JDK 11 and Scala 2.13 / kafka.server.RaftClusterTest.testCreateClusterAndCreateListDeleteTopic()
Build / JDK 11 and Scala 2.13 / kafka.server.RaftClusterTest.testCreateClusterAndCreateListDeleteTopic()
Build / JDK 11 and Scala 2.13 / org.apache.kafka.streams.integration.QueryableStateIntegrationTest.shouldNotMakeStoreAvailableUntilAllStoresAvailable
Build / JDK 16 and Scala 2.13 / kafka.server.RaftClusterTest.testCreateClusterAndCreateListDeleteTopic()
Build / JDK 16 and Scala 2.13 / kafka.server.RaftClusterTest.testCreateClusterAndCreateListDeleteTopic()

Merged to trunk and 2.8.


test: run virtual against screen reader acceptance tests

To build confidence that virtual is "spec compliant" it should be validated against a trusted set of acceptance tests.
Consider from:

https://a11ysupport.io/
https://aria-at.netlify.app/

TBH maybe getting WPT as a submodule as running against actual browser a11y tests would be reasonable here.
There’s some copy paste WPT in the codebase, but not scalable to maintain and port that way.
See https://github.com/eps1lon/dom-accessibility-api/tree/main/tests for an example of how this could work


HARP-7104: Theme build tool

Codecov Report

Merging #1259 into master will decrease coverage by 0.04%.
The diff coverage is 22.58%.

@@            Coverage Diff             @@
##           master    #1259      +/-   ##
==========================================
- Coverage   52.88%   52.83%   -0.05%
==========================================
  Files         260      261       +1
  Lines       23364    23391      +27
  Branches     5171     5177       +6
==========================================
+ Hits        12355    12358       +3
- Misses      11009    11033      +24

Impacted Files
Coverage Δ

@here/harp-theme-tool/src/cli-build-theme.ts
0% <0%> (ø)

@here/harp-mapview/lib/ThemeLoader.ts
82.54% <87.5%> (-0.15%)
: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 9bc68e4...15dd314. Read the comment docs.


Update authentication-conditional-access.md

Learn Build status updates of commit 0fdba7f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/external-identities/authentication-conditional-access.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@msmimart
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@msmimart feel free to rephrase if necessary.

Hi @ChristianJBergstrom, I'm reviewing the changes with the product team. Thanks for your suggestions!

@msmimart any update from the team yet? 😊

Hi @ChristianJBergstrom, not yet so I want to make sure I understand the context or issue behind these suggested changes. I ask because in this section of the doc we're describing how you can scope a Conditional Access policy to B2B direct connect users with the new Assignments > Guest or external users > B2B direct connect users option (also described in the linked Conditional Access user assignments). So a B2B direct connect user can be in-scope for CA policies, which work together with trust settings to determine access. Can you help clarify the proposed changes?

Hi @msmimart ! You’re spot on. What I’m proposing is that one could get the idea that when selecting B2B direct connect users you can choose whatever CA setting available. But direct connect users are only affected by the three trust settings as you mention. Not everyone knows this. I.e. One can complete a config as if the user was a proper B2B user with a sign-in frequency policy as an example but will not take effect. Hope that explains it better 😊

@msmimart By any chance you could fetch a table of the settings one actually can use with direct connect? Just tried a Terms of use with CA and it worked for the shared channel. That hasn’t worked before but awesome that it does. I believe the preview is causing some confusion, what works, whats to come etc. I did get a copy like a year ago from Teams PG and it basically said only the three trust settings will work with CA. Thanks again and hope you don’t get reminders for all edits ☺

@ChristianJBergstrom thank you so much for testing all of this out. Your point is well taken, and we are working on ways to outline the policies you can apply in B2B direct connect scenarios. If you don't mind, I'd like to hold off on these changes until we can make the updates we have in the works. In meantime, we can close this PR but still track the thread in case there are any other findings you want to add. Again, thanks for your dedication to helping us get this right!

@msmimart Sure thing! Please let me know as soon as possible when that "outline of policies" is completed. I'm going crazy over here 😅

#please-close

@msmimart It was the UserType "Member" instead of default "Guest" on a proper B2B guest account that did it. So I've actually used a session CA with MDFCA blocking downloads for B2B direct connect users in shared channels if being on an unmanaged device by using that property. Please send a thumbs up so I know you've read this. Thanks.

@msmimart Didn't make sense with the UserType so continued. From what I can tell it's must be the field "Company name" with the full external domain name that is the trigger, it's the only thing that differs. I must say it isn't consistent and MDFCA doesn't inform you about being "monitored" as in normal cases, but if you look at the link one can see mcas involved and the file is blocked from download, not all the time though. I.e. one second you can download and the other not on the same file.
Managed to use 'Terms of use' and 'app enforced restrictions' too. The latter is probably the preferred way right now as you don't even see the download option and is consistent.


Isolate channel logic in a channel.qnt spec

The reference implementation has a channel keeper (like the bank keeper). I think we also want something like that since handling channels has been a pain.

Related: the packet fields in ChainState currently cause some confusion (https://github.com/informalsystems/quint/pull/865#discussion_r1187323389)
If we want to be entirely truthful to the implementation, we would have only outPackets and inAcknowledgements in ChainState, and the remaining relayer magic is done in the stateful part of the spec, or another module.
We should check with @josef-widder how far we want to take this idea.

I am not sure I totally understand the question: are we discussing whether/how we should write a spec that, e.g., makes sure a packet sent is received or generates a timeout?

No, this issue is more of a refactoring: Currently the relaying logic is hidden/spread across the ChainState type, and the various actions.
I think there would be a cleaner way to do it, by isolating all the channel/relayer logic into a separate module. But also that adds more code, and is probably detrimental to understanding ICS 20.

We have made some nice improvements on channel topology manipulation inside the specs after I opened this issue. I'm not sure if it makes sense anymore.


(keepass-langfiles) release 2.57 fixes #2527

Description
Updated README.md regex to https and changed the zip extension to 7z.
Added all lngx files in keepass_2.x_langfiles.7z.
Motivation and Context
Language files are outdated since 5 years.
https://github.com/chocolatey-community/chocolatey-packages/issues/2527
How Has this Been Tested?
Referred to README.md
No testing environment
Types of changes

[x] Update
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[x] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] I have updated the package description and it is less than 4000 characters.
[x] All files are up to date with the latest [Contributing Guidelines] (https://github.com/chocolatey-community/chocolatey-packages/blob/master/CONTRIBUTING.md)
[ ] The added/modified package passed install/uninstall in the Chocolatey Test Environment (https://github.com/chocolatey-community/chocolatey-test-environment/).
[x] The changes only affect a single package (not including meta package).

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.


Fix server parameter documentation

Rasa NLU version: master branch
Issue:
rasa_nlu documentation has problems and its information is old and not working with master branch.

in https://nlu.rasa.com/http.html#server-parameters, phrase below is wrong:

phrase: Eg: python -m rasa_nlu.server -p project1 project2 --path projects

Will causes: `server.py: error: unrecognized arguments: -p`

--server_model_dirs (option also removed from server arguments and does not work.)

in https://nlu.rasa.com/http.html#serving-multiple-apps , phrase below is wrong (Its because config file doesn't support path configuration option) :

by default the server will load all projects found under the path directory defined in the configuration.

I'll create PR.

@wrathagom I'll create a PR.

I made a PR #1163


AngularJS scope - value not in $scope

Hi,
Looking at the example at: https://ag-grid.com/angular-grid-cell-rendering/index.php (section complex html, with Angular JS) - there is a mysterious "{{value}}" binding which suggests "value" would be in $scope. This is however not the case, all I can see is the "data" object. Is it a bug?
The "data" object is generally fine, as long as we know precisely what we are looking for. But for general renderers it reduces reusability.

I think the documentation is incorrect. There's only a single scope per row, so a generic value property wouldn't make sense.
You could do something like this:
column.cellRenderer = function (params) {
  return '{{data[\'' + params.colDef.field + '\']}}';
};

Thanks with the tip, it should work.
But with the scope it isn't true - there is separate scope for cells - look e.g. here: https://ag-grid.com/javascript-grid-cell-template/index.php - all the cells have the standard .ngScope

The angular plugin for Chrome Tools isn't working for me on the example page, but I do see the ng-scope class on each cell. But each cell is using the same $scope since the $id property is the same. If there was a separate scope for each cell, I think the grid would be much slower.

Hmm, it seems you are right.
Tracked down here: https://github.com/ceolter/ag-grid/blob/d6b340ceb2cda946a4beb930b7c2a90b90889f54/src/ts/rendering/renderedRow.ts#L534
So clearly a documentation bug :)

i don't know how you got the link above, it's old data, the correct cellRendering docs are:
https://ag-grid.com/javascript-grid-cell-rendering/index.php
as per discussion above, there is one scope per row. there was never a scope per cell.
if you want to reuse your cellRenderer, bind it with the data, eg:
function myRenderer(field, params) {
  return '{{data.'+field+'}}';
}

colDef = {
  cellRenderer: myRenderer.bind(this, 'firstName');
}


Bug 60595 - add a SpashScreen at the start of JMeter GUI

Bug 60595 - Add a SplashScreen at the start of JMeter GUI

Hi,
Thanks to your PR
I think your IDE have reformated the code in src/core/org/apache/jmeter/JMeter.java and we can't easily check your modification.
Can you upgrade the file with the original format?
Thank
Antonio


Switch to go mod compatible version of operator-sdk

make images does not work if not in $GOPATH (and it requires go mod vendor).
https://github.com/operator-framework/operator-sdk/pull/1475

Looks like 1475 has been merged, let's wait for the next sdk release


Show how-to video inline on the readme page.

This sort of fakes it so people can see what it looks like before they watch the whole thing.

lgtm


Make mandelbrot sample more colourful and more blurry

Making it more colourful will overcome some limitations of image diffing algorithms with respect to black and white.
Making it more blurry will make comparisons less sensitive.

Fixed via #362


What is the size limit for a comment?

I tried to make a 2040 chars long comment and I got the error below. If there is an upper limit it would be better to show a more informative message.

Me neither.

I still have issue with wrong comment size calculation due to the treatment of EOL. I will try to fix it :-)

There is an issue in WHATWG about newline characters and others: whatwg/html#1467. It's an old bug in Webkit, which was fixed in Chrome but in Safari wasn't.
Honestly, we can implement max length using JS, but I'm sure it would be slower. But what's more important is the fact that it would affect all users of Remark42. I think that this bug with max length in Safari can found maybe tens of users, but all others shouldn't use slow UI because of JS implementation.
So, I think that we just need to be patient and wait till this bug in Safari is fixed ¯_(ツ)_/¯.


Dev

Codecov Report

Merging #48 (2367e0a) into master (6ca6d79) will increase coverage by 31.70%.
The diff coverage is 58.97%.

@@             Coverage Diff             @@
##           master      #48       +/-   ##
===========================================
+ Coverage    5.74%   37.44%   +31.70%
===========================================
  Files           7        9        +2
  Lines         209      227       +18
  Branches        9        9
===========================================
+ Hits           12       85       +73
+ Misses        197      142       -55

Impacted Files
Coverage Δ

opensentiment/data/utils.py
0.00% <0.00%> (ø)

opensentiment/data/make_dataset.py
33.33% <25.00%> (+33.33%)
:arrow_up:

opensentiment/models/predict_model.py
30.00% <33.33%> (+30.00%)
:arrow_up:

opensentiment/models/train_model.py
22.47% <33.33%> (+22.47%)
:arrow_up:

opensentiment/api/fast/serve_api.py
100.00% <100.00%> (ø)

opensentiment/api/fast/serve_api_helper.py
100.00% <100.00%> (ø)

opensentiment/utils.py
100.00% <0.00%> (+100.00%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


/auth/google 404 Not Found

Steps to reproduce

feathers generate app with rest and sockets
feathers generate authentication select google, facebook, and local
fill out default.json configuration with mongodb connection and google/facebook clientID/secrets
tested users.create and feathers.authenticate({strategy: 'local'}) in client is working
started working on google signin and discovered that /auth/google and /auth/facebook are 404 not found.  not found at /users/auth/... or /authentication/... either

Expected behavior
the routes should exist
Actual behavior
the routes are not found
System configuration
package.json
...
"dependencies": {
    "body-parser": "^1.18.1",
    "compression": "^1.7.0",
    "feathers": "^2.2.0",
    "feathers-authentication": "^1.2.7",
    "feathers-authentication-hooks": "^0.1.4",
    "feathers-authentication-jwt": "^0.3.2",
    "feathers-authentication-local": "^0.4.4",
    "feathers-authentication-oauth2": "^0.2.6",
    "feathers-configuration": "^0.4.1",
    "feathers-errors": "^2.9.2",
    "feathers-hooks": "^2.0.2",
    "feathers-hooks-common": "^3.7.3",
    "feathers-mongodb": "^2.9.0",
    "feathers-rest": "^1.8.0",
    "feathers-socketio": "^2.0.0",
    "helmet": "^3.8.1",
    "mongodb": "^2.2.31",
    "passport-facebook": "^2.1.1",
    "passport-google-oauth20": "^1.0.0",
...
}
...

Tell us about the applicable parts of your setup.
config/default.json
...
  "authentication": {
    "secret": "...",
    "strategies": ["jwt", "local"],
    "path": "/authentication",
    "service": "users",
    "jwt": {
      "header": {
        "type": "access"
      },
      "audience": "https://yourdomain.com",
      "subject": "anonymous",
      "issuer": "feathers",
      "algorithm": "HS256",
      "expiresIn": "1d"
    },
    "local": {
      "entity": "user",
      "usernameField": "email",
      "passwordField": "password"
    },
    "google": {
      "clientID": "...",
      "clientSecret": "...",
      "successRedirect": "/",
      "scope": ["profile email"]
    },
...

NodeJS version:
v6.11.1
Operating System:
windows 10
Browser Version:
chrome
I've tried debug logging with winston but it only shows this right now which i don't believe is related to this 404 problem:

info: error: authentication - Method: create: Missing credentials
debug: hook.data strategy=local
debug: hook.params , provider=socketio, , ,
error:  BadRequest: Missing credentials
at Error.BadRequest (C:\Users\Travis\Documents\Code\Projects\vuthers3\node_modules\feathers-errors\lib\index.js:93:17)
at C:\Users\Travis\Documents\Code\Projects\vuthers3\node_modules\feathers-authentication\lib\hooks\authenticate.js:102:31
at process._tickDomainCallback (internal/process/next_tick.js:135:7)
info: (404) Route: /auth/google - Page not found

Really appreciate any advice.  Thank you.

After removing feathers notFound() and handler() middlewares from src/middleware/index.js, and using one of my own, I was able to discover that this was an error with my google oauth2 configuration which was being hidden by feathers.  I was able to get it working after adding localhost:3030/auth/google/callback to the list of allowed redirect URIs.
  // app.use(notFound())
  // app.use(logger(app))
  // app.use(handler())
  function errorHandler(err, req, res, next) {
    res.status(500)
    res.render('error', { error: err })
  }
  app.use(errorHandler)h

Somehow this allowed me to see the error message returned by google.  Still not sure why other than I prevented the notFound page from showing up.


update horizon image's branding from aurora/bluefin to horizon

[x] "Welcome to Horizon" in the terminal MOTD
[ ] Logos
[ ] Wallpaper

i cant be bothered doing all that