Site data

Hello and thank you for this integration !!!
I have 2 Fronius Symo connected in daisy chain and a Fronius smartmeter.  I set up this integration 3 times, for slave 1 and 2 as the inverters and 240 for the meter.
Data is coming through and all seems good.
However i am struggling to find out how to get the Site data.  Fronius modbus docs specify that
"The following registers can be used to query power and energy data from all inverters
connected to this Fronius Datamanager via Fronius Solar Net.
These values are displayed in Watt (W) or Watt hours (Wh) and do not require scale factors.

F_Site_Power(500–501): Power
F_Site_Energy_Day(502–505): Daily Energy
F_Site_Energy_Year(506–509): Yearly Energy
F_Site_Energy_Total(510–513): Total energy of the entire system."

This data is available when doing a http call to the meter: "http://ipaddress/solar_api/v1/GetPowerFlowRealtimeData.fcgi". The response contains:

Basically what i am trying to get is the Site Power (P_Load)  - aka the total power in Watts for the house and the P_Grid - power from Grid
Is this possible please ?
Thank you

I figured it out :) just needed to do some calculations to extract that data from the existing one.
Cheers :)


Avoid docutils.nodes.reprunicode

docutils.nodes.reprunicode was until recently just a subclass of str. It was removed in this commit: https://sourceforge.net/p/docutils/code/9415/ Therefore path = nodes.reprunicode(path) just converted a string to a string and can therefore be removed.
This is an alternative to fixing docutils to an old version (as done in #69).
Related discussion on a similar issue with another Sphinx project: https://github.com/orgs/sphinx-doc/discussions/13020#discussioncomment-10930769
Fixes #68

Bumping (@CrossNox)—can we get this merged for a new release? This project is unusable for me until this PR is merged 🙏


add coverage test to normal test runs

Coverage test is only in travis tests but not locally
so add it to the normal test suite.
Fixes https://github.com/kedgeproject/kedge/issues/333

Works for me :) LGTM


SW: RTC and timezones

I spent a minute trying to figure out how one of my nodes had gotten three hours off today, only to eventually put together that it was indeed 10:20 US/Pacific... I'm not sure what exactly the best policy is here, but I think it makes sense to program the RTC on the board with UTC time as programming the RTC is ideally rare and could happen anywhere, so that should be globally consistent.
We should probably add something to the data processing as well, but I think "which time zone was this device in while recording" is more of a metadata problem.

This has been resolved with the latest check-in.


feat: pattern validation on input event

Addresses #3744
Summary
The validation of the pattern regexp eventually attached to a string field is currently triggered when the user save an entry. Instead, if pattern validation is triggered onChange, the UX would be enhanced and would help user to make necessary corrections to satisfy pattern prior to saving itself.
To create this functionality, handleOnChange function is created inside EditorInterface component.
handleOnChange = () => {     this.controlPaneRef.validate(); };
Above function is called inside handleChangeDraftField function inside Editor.js file which is triggered when onChange is invoked.
Test plan
yarn test was run to verify all testcases passed.
Non mandatory cute animal

@erezrokah @magomimmo I noticed a few issues.

When the Editor is opened for the first time, the title field and the body show error that they must not be empty and are highlighted in red.

When the title field is first filled and then emptied, it doesn't show error immediately but shows only when other field is selected and something is typed.

IMHO I feel that when need to perform validation on keyUp event which solves the 1st issue. Like @barthc suggested, we could also trigger validation on blur event if we need better performance.

Hi @BA1RY, IMHO onBlur event seems to be safer and a good trade-off.

@erezrokah I was thinking about approaches to implement validation on blur. I noticed that we are passing validate function to each Widget and we perform setActiveStyle and setInactiveStyle on focus and blur event. So it would be better, we could wrap setInactiveStyle and validate into one function which can be called on blur event. What do you say?
If we follow the above approach, how can we perform perform validation on blur for nested and editor widget. That part I'm not able to understand.

Hi @BA1RY, have you tried using the onBlur approach?
I think it has two issues at the moment:

The user won't see the "changes" indication until the input loses focus (gif shows expected behaviour):

Since widgets have their value passed from the redux store without calling handleChangeDraftField on each change the widget value won't get updated.

Regardless, looks like there is a delay with the current implementation:

@erezrokah I only tried onBlur approach for 'widget-string'. I used validate function present inside Widget.js  to perform validation for that individual field. I wrapped validate and setInactiveStyle of Widget.js into one function and passed it to handle onBlur and it worked well. But, I didn't have any idea on how to do the same for 'nested objects' and 'markdown' widgets. Also, I didn't want to change any code of core components without consulting you. https://github.com/netlify/netlify-cms/blob/397857855b2c8514c2f7ce83756af6b6698abc3d/packages/netlify-cms-core/src/components/Editor/EditorControlPane/Widget.js#L285
https://github.com/netlify/netlify-cms/blob/397857855b2c8514c2f7ce83756af6b6698abc3d/packages/netlify-cms-core/src/components/Editor/EditorControlPane/Widget.js#L292
I think the above issues you raised could be solved by using onChange for that specific functionality only. What do you think?

@BA1RY, good points. Let me dig in into this a little more.
Can you commit you onBlur changes and I'll check what can be done about nested fields/markdown editor?

@erezrokah I've added validation on onBlur functionality for string, text and list widgets.
For 'date' and 'datetime' widgets shouldn't date be invalid if date is empty string?
https://github.com/netlify/netlify-cms/blob/43ef28b5dce32542e9c2d44cc9b2fc031232a13b/packages/netlify-cms-widget-datetime/src/DateTimeControl.js#L74
https://github.com/netlify/netlify-cms/blob/43ef28b5dce32542e9c2d44cc9b2fc031232a13b/packages/netlify-cms-widget-date/src/DateControl.js#L80

@BA1RY, sorry again for the late reply.
What do you think about de-risking and de-scoping this PR just to the request in the issue?
I think we can do something like https://github.com/netlify/netlify-cms/commit/69e79bdbd72fa74382197fb980cae7a60ebb51f7 and cover almost any valid use case.

@erezrokah That's a good point. I think that would work out well.

It should only happen if a field has patten configured.
I'm using this config:
collections:
  - name: 'posts'
    label: 'Posts'
    label_singular: 'Post'
    folder: '_posts'
    create: true
    fields:
      - {
          label: 'Title',
          name: 'title',
          widget: 'string',
          tagname: 'h1',
          pattern: ['\d', 'Title must contain a digit'],
        }
      - { label: 'Body', name: 'body', widget: 'markdown', hint: 'Main content goes here.' }

Replaced by https://github.com/netlify/netlify-cms/pull/3922


Fixes HTML  issues

Passes HTML proofer tests to make Circle CI happy :blush:

Could you describe the issues with Circle CI a bit more please? :)

Yeah I don't think my review counts towards anything 😄

Changes merged in #27


Take into consideration active record time zone aware flag

When active record has the flag config.active_record.time_zone_aware_attributes setted to false time zone aware should not used, and same case when attribute class is not included on classes with time zone aware defined by AR too.

Thanks for this. This PR confused me at first but it demonstrates a few issues that exist in the gem which need fixing.
Which are:

That AR is expected in the validator class despite this gem offering ActiveModel support rather than AR specifically
That the gem implicitly expects a timezone aware attributes, which is not intended
The gem doesn't check time_zone_aware_types at all which is problematic if you want to control that

But your implementation is not quite what I'd like, so I'm going to review whether some of this AR specific code can be moved to an ORM extension.

Thank you @adzap, I will close this in favor of an ORM extension that I think is a better approach than this.


[Feature Proposal] .justify-left-center, .justify-center-right

Perhaps this is out of scope for TailwindCSS but I'd like to propose the following two classes which allow flexbox positioning of two elements to either left-center or center-right without needing to use justify-between and an empty div (something I've found myself doing somewhat reguarly). See this post.
.justify-left-center {
    @apply justify-between;
    &:first-child::after {
        content: '';
    }
}

.justify-center-right {
    @apply justify-between;
    &:first-child::before {
        content: '';
    }
}

On second thought, this is surely too specific for Tailwind. Sorry to create this issue!


update for new flags structure

Description
Update flowmetric samples that uses flags as https://github.com/netobserv/flowlogs-pipeline/pull/747 changed the decoding of tcp flags
Dependencies

n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[ ] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labeled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[ ] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

/approve


Drop easy_install, everyone now uses pip

Everyone now uses pip, drop reference to easy_install

What do we gain by dropping reference to something that still works? Most tools still allow easy_install based installation as documented method, don't they?

So the other reason to keep it around is because pip cannot install eggs, so if you're working with legacy systems or have a specific requirement to install from an egg, you'd still need it. I would say that pip should be featured first, with an additional note that easy_install is available. It's not anything I'm passionate about though, so I'm fine with this. Within the confines of folks I know, everyone utilizes pip and the only reason anyone used easy_install was to install pip (before get_pip.py was a thing).

Keep in mind that this is purely a docs change, not a functional change.
I work primarily in the python ecosystem, and pip has become the blessed package manager in the community. No one I know uses easy_install for anything other than when they need to bootstrap pip.
Anyone who actually needs easy_install for things like installing eggs will likely be aware of it (and if not, this one-line change will not help them understand the difference between eggs, wheels, etc.)
So my thought on this change was to reduce confusion for 99% of users who don't need to know about easy_install.

Honestly, that's my preference as well. There isn't much use in keeping around install information for a bunch of package managers when the people who need to know about it already do.

Agree with everything you said and understand - as someone who also uses
python for his "day job". As I said originally, because it's a doc not a
functional change I see nothing wrong with this.
On Aug 29, 2017 16:48, "Jeff Widman"<EMAIL_ADDRESS>wrote:

Keep in mind that this is purely a docs change, not a functional change.
I work primarily in the python ecosystem, and pip has become the blessed
package manager in the community. No one I know uses easy_install for
anything other than when they need to bootstrap pip.
Anyone who actually needs easy_install for things like installing eggs
will likely be aware of it (and if not, this one-line change will not help
them understand the difference between eggs, wheels, etc.)
So my thought on this change was to reduce confusion for 99% of users who
don't need to know about easy_install.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sorin-ionescu/prezto/pull/1436#issuecomment-325814499,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXql_E0TnU_vTZzmU8hke28wGB2yhq3ks5sdIdKgaJpZM4O8PBZ
.

Alright, merged. Thanks to everyone for weighing in!

:) I only know because I've had the "fortune" of working on legacy systems,
but agree that OSS involvement definitely expands my knowledge about things
I'd never had known (issues with prompts being a good example since I'd
never looked at the zsh man pages up to that point).
On Aug 29, 2017 18:16, "Jeff Widman"<EMAIL_ADDRESS>wrote:

BTW, I actually didn't know (or had forgotten) that pip didn't support
eggs... thanks for mentioning that trivia above @johnpneumann
https://github.com/johnpneumann, stuff like that is why I like
contributing to OSS, learn something new every time!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sorin-ionescu/prezto/pull/1436#issuecomment-325831380,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXqlyGg6XNwzd-JVqdNrBvJhC-3xrvBks5sdJvQgaJpZM4O8PBZ
.


Netflix error (some exceptions in the log)

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

General infomration
Prerequisites

[x] Are you running the latest version?

Description
Netflix addon won't start. Shows just "Netflix error: check the log for more information" message.
There are two exceptions raised by the Netflix addon in the log: the first is a "index out of range" and the second "urlopen error [Errno 111] Connection refused". See the attached log for more details.
Context (Environment)
Installation

[ ] I installed the plugin via zip from the Releases page
[x] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[x] Linux (ARM)
[ ] Android

Additional informatin on the environment
OSMC with latest Kodi Leia on Raspberry Pi
Debug log
https://gist.github.com/madadam/b2c3284898be9599d40f63d1219022d5

I got the same exception.

OS: Ubuntu 18.04 x86-64
Kodi: 2:18.0+git20180526.0201-8a6c6c7-0bionic
Netflix plugin:  0.13.5, but 0.13.4 and 0.13.2 give the same error.

Are there still issues with 0.13.8? or can we close this issue report?

All good here, closing.


endpoints

provide an endpoint for to display the current sessions access token
provide an endpoint to test if a session is expired

raised PR #38


Add endorser protocol support for revocation functionality

Adds endorser protocol support for revocation functionality such as:

publishing a revocation registry definition to the ledger
publishing a revocation entry to the ledger
It is based on #1230, so that PR should likely be merged first.c.c: @ianco

Currently, if endorsing is requested the agent will NOT handle the generation and rotation of the revocation registry for the user. Improvements to the usability are necessary, see #1238 for reference.
A set of BDD tests validating the end-to-end flow has been added.

Please note: I have NOT added extra unit tests for the new endorsement handling in the revocation endpoints since it would take significant time and we are anyway going to be refactoring the code fairly soon, based on #1238
If more unit tests are required let me know and I'll try and sort them out though.

Codecov Report

Merging #1246 (f00919b) into main (72b42b7) will decrease coverage by 0.24%.
The diff coverage is 52.17%.

@@            Coverage Diff             @@
##             main    #1246      +/-   ##
==========================================
- Coverage   98.47%   98.23%   -0.25%
==========================================
  Files         466      466
  Lines       26562    26674     +112
==========================================
+ Hits        26157    26203      +46
- Misses        405      471      +66


Stages on Map: Pin doesn't centre to screen when tapped on iOS

Description of issue
When I tap on a stage pin the pin remains in its current position
Expected behaviour
When I tap on a stage pin I'd expect the pin to move to the center of the screen / frame
Steps to reproduce
Navigate to Parade > Map
Drag screen to make a stage pin off-center
Tap on stage pin
Stage pin remains in position
OS / Browser / Device model
Jun's iPhone X
Screenshots
none

This is an iOS problem. Android exhibits the correct behaviour.


Rebased user checkout flow

[ ] Generic Error Message

Something went wrong please select the proper variant and try again
Need to be more specific

[ ] Add more details to Cart Page

Add Quantity
Price
Usable Design

is this integrated ??

I start with guest session and then login with my credentials
will the guest cart be linked as my cart ??

[ ] When proceeding to checkout, ask to Login first

[ ] On failure on add_to_cart on Product Show Page

Last Variant selection should persist

[ ] Add a cancelled state next to Order in Listing when all line_items cancelled

[x] Warning
the :name option in form_for/inputs_for is deprecated, please use :as instead
(phoenix_html) lib/phoenix_html/form_data.ex:47: Phoenix.HTML.FormData.Plug.Conn.to_form/2
(phoenix_html) lib/phoenix_html/form.ex:234: Phoenix.HTML.Form.form_for/4
(ex_shop) web/templates/user/product/show.html.eex:21: ExShop.User.ProductView."show.html"/1
(ex_shop) web/templates/layout/app.html.eex:26: ExShop.LayoutView."app.html"/1

When user logs in when asked for checkout, should be redirected to Cart Page and not Address Page

so that he get to see items before he starts pay process

Please check cart_not_empty on all steps in checkout and show cart page, if empty


Upgrade to Using Environment Files - set-output Deprecated

I'm receiving the following warning when running the action.

The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

We're using @actions/core v1.10.0, which per https://github.com/actions/toolkit/blob/main/packages/core/RELEASES.md#1100, should be using the new output format.


Fix fast restore

This has regressed countless times:

(recently) https://github.com/fsprojects/Paket/pull/3598
https://github.com/fsprojects/Paket/pull/3013
https://github.com/fsprojects/Paket/pull/3200
https://github.com/fsprojects/Paket/pull/3201
https://github.com/fsprojects/Paket/pull/3206

The last regression was due to the change of the caching file to only contain hashes. This is tricky to fix in msbuild, but there is https://docs.microsoft.com/en-us/visualstudio/msbuild/getfilehash-task?view=vs-2019 which seems to produce the exact same hash we already generate in C# (when using SHA256 in hex format).
I guess this also replaces https://github.com/fsprojects/Paket/pull/2870, lets see if this actually works for everyone, if not we are doomed ;)
Therefore we:

[x] Add (failing) integration test for #2684.
[x] Fix the issue at hand

/cc @TheAngryByrd Please check if this regresses on https://github.com/fsprojects/Paket/issues/2796

Seems to be working for me!
Only issue I seemed to have is having an older FAKE version causes paket.restore.cached to be in a old format and give this new Targets file errors.  Might need to recommend people update their FAKE version if they hit this error
.paket/Paket.Restore.targets(101,13): error MSB4184: The expression ""System.String[]".GetValue(1)" cannot be evaluated. Index was outside the bounds of the array.

I think we should fix that, if we cannot read the cache file we should probably call restore and consider it outdated

Seems the testsuite agrees ;)

is this ready?

Should be ready if it turns green. I addressed the failing tests and the issue @TheAngryByrd pointed out (including a test for this scenario as none existed)
Notice that this also fixes https://github.com/fsharp/FAKE/issues/2348 which apparently is a feature now in the SAFE-Stack. Instead of adding an integration test into FAKE, I think it should be enough that we now have one here, time will tell.

I accidentally pushed into origin and then deleted the branch, therefore we now have 4 instead of 2 CI builds and two are red because they can't find the branch :)

@matthid you always find novel ways to confuse me ;-)

That's why I added a comment ;).
push origin is just to much muscle memory ;)

You could point origin to your fork and name the common fork upstream
Matthias Dittrich<EMAIL_ADDRESS>schrieb am Di., 2. Juli 2019,
19:47:

That's why I added a comment ;).
push origin is just to much muscle memory ;)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/fsprojects/Paket/pull/3608?email_source=notifications&email_token=AAAOANBLLF5ZD6S5OU5UVRDP5OIBZA5CNFSM4H4WQFWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZCBRCQ#issuecomment-507779210,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAOANEASK632R7RJFAVCZLP5OIBZANCNFSM4H4WQFWA
.

That's what I do, too terrified I'll flub things up by happenstance

Had to fix another test anyway, so back to normal ;)


FIx #457 : Add ToPreciseTraversable type class and tests

In the absence of an accessible Monoid definition, this should be ok for now.
The name is debatable (I tried to find a name close to ToTraversable since both type classes perform similar work).
Tests could be enhanced, especially if I were able (or aware of a way) to express "equality" of Coproducts (e.g. expressing that A :+: B :+: CNil == B :+: A :+: CNil).

You should be able to assert equality of Coproduct types using shapeless.ops.coproduct.Basis.

Could you rebase this against master? To support a Scala.js CrossProject(CrossType.Full) everything under core/src has been moved to core/shared/src.

Done (great news by the way !)

This looks pretty good :-)
I think we need a better name though. Given that the distinguishing feature from to[M] and ToTraversable is that the element type of the resulting collection is a Coproduct rather than the LUB then I think maybe toCoproduct[M] and ToCoproductTraversable would be better.

Ok.
I had considered that in the beginning, but feared that it would be slightly misleading (mostly due to the words order). I felt that it would be unsettling that toCoproduct[M] does not return a Coproduct, at least for newcomers.

I think those are all very sensible concerns, but I think this is the better option.

LGTM and Travis is green! Many thanks ... merging :-)

All the pleasure we for me.
(look Mom, I contributed to open source software ! :-) ).

Would you mind adding your details to CONTRIBUTORS and README.md?

From the peanut gallery here, I think toCoproduct[List]/toCoproduct[Vector] looks great.


Outdated Keycloak image in docker compose prevents docker compose up

The docker compose on the main branch should be usable without any manual changes!
Options:

Update Keycloak image to one that works
Make Keycloak containers not start by default (e.g., by assigning them to a different compose profile)

Updated the keycloak image, such that it can be pulled and does not prevent the docker compose from starting


Add details about FIPS compliance

Please add details whether this extension will work on servers with FIPS enabled.
As example: https://docs.microsoft.com/en-us/azure/azure-monitor/platform/log-analytics-agent#installation-and-configuration
Based on my research - it does not fully support FIPS on Linux VMs because extension management script avme_common.sh uses "openssl dgst -r" to calculate configuration file hash and this won't be allowed by FIPS.

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

ID: 747a2c52-c59c-dedc-9c72-6b6352ea2cab
Version Independent ID: 07e64d3e-26da-be39-de53-6e259c90545f
Content: Built-in vulnerability scanner in Azure Security Center
Content Source: articles/security-center/built-in-vulnerability-assessment.md
Service: security-center
GitHub Login: @memildin
Microsoft Alias: memildin

@dc-isafonov Thank you for your inputs . We will engage the content authors on this to see how best this can be incorporated in the document .

#reassign:bwren

#label:"CreateAzMonWIFeb2021"

Thank you for your contribution to the Azure Monitor documentation. We have created a workitem for this issue in our internal backlog database. We will prioritize it and make any necessary changes as we are able.
#please-close


Add missing tests for edit_ip_description

Coverage increased (+0.7%) to 100.0% when pulling a0facb489c34cc6b9cd5bf23596387e942c86866 on dclaisse:master into f1fbe0b5aa55f0a7b055ef788d93b4114a61b072 on criteo:master.


Error when I'm trying to login with GitHub or GitLab

InvalidStateException in AbstractProvider.php line 200:

in AbstractProvider.php line 200
at AbstractProvider->user() in AuthController.php line 66
at AuthController->handleProviderCallback('github')
at call_user_func_array(array(object(AuthController), 'handleProviderCallback'), array('provider' => 'github')) in Controller.php line 55
at Controller->callAction('handleProviderCallback', array('provider' => 'github')) in ControllerDispatcher.php line 44
at ControllerDispatcher->dispatch(object(Route), object(AuthController), 'handleProviderCallback') in Route.php line 203
at Route->runController() in Route.php line 160
at Route->run() in Router.php line 559
at Router->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 30
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in RedirectIfAuthenticated.php line 32
at RedirectIfAuthenticated->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in SubstituteBindings.php line 41
at SubstituteBindings->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in ShareErrorsFromSession.php line 49
at ShareErrorsFromSession->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in StartSession.php line 64
at StartSession->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in AddQueuedCookiesToResponse.php line 37
at AddQueuedCookiesToResponse->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 102
at Pipeline->then(object(Closure)) in Router.php line 561
at Router->runRouteWithinStack(object(Route), object(Request)) in Router.php line 520
at Router->dispatchToRoute(object(Request)) in Router.php line 498
at Router->dispatch(object(Request)) in Kernel.php line 174
at Kernel->Illuminate\Foundation\Http\{closure}(object(Request)) in Pipeline.php line 30
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Debugbar.php line 51
at Debugbar->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in CheckForMaintenanceMode.php line 46
at CheckForMaintenanceMode->handle(object(Request), object(Closure)) in Pipeline.php line 148
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 102
at Pipeline->then(object(Closure)) in Kernel.php line 149
at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 116
at Kernel->handle(object(Request)) in index.php line 52

This happen with GitHub in my digital ocean server... using ubuntu 16.04

You have to put your app credentials into your .env file and run php artisan config:cache.
Did you already executed this steps?


refactor: handle errors in redeem retry

Signed-off-by: Gregory Hill<EMAIL_ADDRESS>

Maybe you can open an issue on the appropriate partity repo to get more specific errors?

https://github.com/paritytech/substrate/issues/8790


JSON syntax error when node label name contains "-"

We confirmed that this is not a bug, but a valid syntax error. The node labels should be surrounded by quotes in config:
'person-sus': 'f41a',

The default config should follow this rule (even when they do not contain -).

Now the default config does not contain the icon entry. The future examples should use '' appropriately.


Use Terminal Reporter when running in TeamCity

This simply runs a terminal reporter with no color when team city is detected. This ensures that jasmine-node can be used on a teamcity build.

The Teamcity runner as it was written should work with Teamcity 6.5 and later. Was there a particular reason that you reverted back to a Terminal Reporter. I had originally written the Teamcity reporter in the https://github.com/larrymyers/jasmine-reporters repo so that individual tests would be reported as per Teamcity's documentation at http://confluence.jetbrains.net/display/TCD7/Build+Script+Interaction+with+TeamCity#BuildScriptInteractionwithTeamCity-ReportingTests

Interesting,
Based on what I observed is that the TeamCity reporter was not available (undefined) in jasmine-node. Perhaps the issue is that jasmine-reporters is required as a dependency?

You are correct! I just looked at the packages.json file and it is looking at an old version of that dependency, which explains your trouble. See pull request #136.

Thanks!


NullPointerException related to filename of .env

When using this plugin, i got a java.lang.NullPointerException, and after some fiddling around, it seemed to be related to the filename of the .env i was using: ".env.local".  When i changed it to "dev.env", it worked fine.

Oh, that's a nice catch! Thanks! I will release new version soon with fix for this.

Unfortunately I wasn't able to reproduce the issue. @smcoll can you please provide more details if you still able to reproduce it?

@Ashald no longer seems to be a problem, using 2016.1.4.  Maybe it was a PyCharm bug that has since been fixed.

@smcoll thanks for the update.


{Compute} Update vm test with new versioned image aliases

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

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

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

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute


KIWI-1719: Set alarms to off default in dev

Add missing Is notDevelopment condition

Proposed changes

IsnotDevelopment condition does not exist for setting the alarm on non-dev environments

What changed

Why did it change

Issue tracking

KIWI-1719

Checklists
PII logging

[ ] Verified that no PII data is being logged

Environment variables or secrets

[ ] No environment variables or secrets were added or changed

[ ] Documented in the README
[ ] Added to deployment repository
[ ] Added to local startup repository

Other considerations

[ ] Update README with any new instructions or tasks

Looks alright to me, please could you attach test evidence to the PR then I'll give it a ✅ from kiwi

Updated the PR with the proof of successful deployment without warning or critical alerts on the slack channel.


Add collider:moveKinematic API

Hand physics can be implemented in a simpler way: hand:moveKinematic(pos, rot, dt)

🤔 Should people pass the dt from lovr.update or the fixed timestep from lovr.physics.newWorld?

It should be. Just like the current calculation, if you use Lovr's FixedStep, then whether you use moveKinematic or not, you can't calculate the exact position because you can't get the correct physics delta time.
I implemented fixed step myself in my project, without using Lovr's

Yeah, I think we either need to add a fixed update callback or remove the fixed timestep.  Either way, it doesn't affect this PR.  Thanks for the patch!


Bar chart grouped isn't working in bokeh

Expected behavior
Bar chart displaying vertical bars grouped by similar category
Actual behavior

Steps to reproduce the behavior
Cluster by a relevant field, select grouped.

@chetnawarade the values you selected produces a very very large number of bars. each bar ends up being so thin you cannot see them. if you use the bokeh zoom tool and zoom in to a specific city you will see the bars. if you try more appropriate values you will bee the grouping:

example
keys: Property Type
values: Beds
cluster by: Price

example
keys: Property Type
values: Beds
cluster by: Baths


Add split text boxes for address for applicant address details form

What
Link to story

Add address_data JSON column to the Person model.
Add feature flag SPLIT_ADDRESS
Update applicant address form to separate address column into several text boxes and save in address_data form, when feature flag SPLIT_ADDRESS is enabled.

New applicant  address details form look like the screenshot below:

Checklist
Before you ask people to review this PR:

[x] Tests and rubocop should be passing: bundle exec rake
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[x] The PR description should say what you changed and why, with a link to the story.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

More context here: https://design-system.service.gov.uk/patterns/addresses/


fix: Skip undefined props in snapshots

It seems like the snapshots where overwritten when updating to Enzyme 3 and React 16.

Good catch! I'll merge it

The fix has been released in v3.2.1 🎉


When using some mods such as Quark and Chunk Pregen, it will incorrectly activate all their controls regardless of which wheel bind was clicked

mc 1.16.5, caused when using a sub category

duplicate of #113


more sensible way to copy the image icons (no functional change)

uses the caches dir to store the images, and gives them more sensible (predicatble) filesnames

@morenoh149 just tidied up some of the iOS code (I'm learning!)

lgtm

Thank you.


Logout and login in auth issues

When logging out there are errors. Additionally, when logging in it does not redirect you to the homepage and the user is not given a logout option unless the refresh the page

login and log out are working  wich page refreshing


Add func StatusText(code int) string

This function allows to retrieve the according status text similar to net/http. In addition I optimized the IsStatusValid function with the new statusText map.

Thanks for adding this. I would correct the text description of each status to use the exact text in Appendix 1 of the Gemini specification. That way there can be one name for everything across the board. Keeping PascalCase instead of all caps is fine though.
Also I'd like to remind you what the README says, which is:

This is mostly a personal library. You might want to check out go-gemini (no relation) for more features.

I'm fine to add this because it seems useful and isn't much effort to maintain, but that's just something to be aware of if you weren't.

Thanks for the reminder, I'll keep it in mind. Yet I do not need a lot of fancy features, so I am going to stick with this module, at least for a while.

Merged, thanks!


[Support] RN 0.40.0

RN 0.40.0
iOS native headers moved
This change affects all native modules and application code on iOS that refer to react native .h files
After e1577df, Native code on iOS must refer to headers out of the react namespace. Previously the following would work:
#import "RCTUtils.h"

But now all headers have been moved:
#import <React/RCTUtils.h>

Would the only change be updating the import headers in the .h and .m files? If so, this is easy, but what about backwards compatibility?

My understanding is that this is the only change required.  As near as I can tell from looking at other projects, backwards-compatibility is difficult or impossible.  Most projects are releasing a new major version with 0.40.0 support and updating their README's to suggest that people using older versions of RN should stick with older versions of the library.

Does anyone have a pull request already with the RN 0.40 changes? The other React Native packages I'm using are not supporting backwards-compatibility and recommend sticking to an older version of their packages if need be.

I just submitted one: https://github.com/zmxv/react-native-sound/pull/98
If you need it to work now, it's really simple to fix locally.

@veddermatic @Beingbook The master branch now has this fixed. I'll push to npm as soon as I have access to do so.

Closing as 0.9.0 includes this feature.


Fontification color

Hi,
I just installed this using the nbstata instructions, and the usual green fontification isn't working but rather a weakish red/magenta color:

In your demo gif it looks dark green, which is definitely more readable.
On the same jupyter instance, the python code is highlited as expected (dark/bold green).

I just reverted to jupyterlab_stata_highlight and things work as expected.  Are there downsides to not using v2?

My original impetus for making a new version was this: https://github.com/kylebarron/jupyterlab-stata-highlight/issues/12#issue-1405224223
There are also some relatively minor changes to command/function highlighting, where I'm being more conservative in what to highlight. Most of the changes are shown in this abandoned pull request: https://github.com/kylebarron/jupyterlab-stata-highlight/pull/13/files
But the colors are the main difference is the colors. I'm targeting the Stata IDE's highlighting rather than the default Jupyter Python highlighting. I haven't figured out how to match the colors exactly yet (as a CSS noob). But that may not be the main cause of the "weak" color: the Stata IDE's color-scheme just has less contrast than Jupyter's default, at least to me (as I'm red-green color-blind, so not the best judge of colors).
The ultimate goal is for this extension to provide a couple options for the colors as themes, one matching the original jupyterlab-stata-highlight and one matching the Stata IDE precisely. But I won't be able to do that in the short term.

Here's a diff that shows all the substantial changes (as of this date):
https://www.diffchecker.com/44NFIGa9/

In your demo gif

Oh, and I should say: that demo gif is taken directly from https://github.com/kylebarron/stata_kernel, just to show an example of a Stata notebook. The functionality shown is the same, now, though, apart from the highlighting. I do plan to create my own demo gif soon, though.

Ok, thanks.


Graphing Calculator sometimes falsely shows "The expression is empty"

Describe the Bug:
The Graphing Calculator sometimes falsely shows the error stating, "The expression is empty".
Steps to Reproduce:

Open Calculator. Switch to Graphing mode.
Copy the following into the Expression Box:

sin x +cos x

OR
sin x+cosx

You may also enter this sequence using your keyboard. But DO not alter the spacing.
Otherwise you may not see the error.
You will see an error stating that the expression is empty.

Close all running instances of the Calculator, and then open it again.
Type first expression as 'sin(x)+cos(x)' and press Enter.

This time the graphing calculator is able to plot the function.
Edit
This error also occurs in case of the following function:
h(x) = sin x/cos x

Expected Behaviour:
The Calculator should readily plot this function.
Device and Application Info:
OS Build: 10.0.19041.264
Architecture: x64
Application Version: 10.2005.23.0
Region: India (EN-US)
Dev Version Installed: No
Requested Assisgnment:
I'm just reporting this problem. I don't want to fix it.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Thanks for the detailed report! This error definitely appears when I copy those strings into the equation box, but I am not able to reproduce when typing the expression in manually, so that might be intermittent.

#1278

Thanks, we believe this might be a different underlying issue than what we addressed in #1278.

There is indeed this problem. A similar problem occurs when inputting the vertex formula of the quadratic function. The calculator will not output the correct image, but inputting the corresponding expansion of the quadratic function can output the correct image.The message also showed: The expression is empty.Device and Application Info:App Ver 10.2012.21.0Version: Windows 10 Education Insider Preview
Version number: 2004
<!--!Installation date :2021/1/29--!>
Operating system version: 21301.1000
Experience: Windows Feature Experience Pack 221.501.0.3

![屏幕截图 2021-01-30 101109](https://user-images.githubusercontent.com/64402861/106344683-48e68880-62e6-11eb-8859-fc060f54444d.png)

There is indeed this problem. A similar problem occurs when inputting the vertex formula of the quadratic function. The calculator will not output the correct image, but inputting the corresponding expansion of the quadratic function can output the correct image.The message also showed: The expression is empty.Device and Application Info:App Ver 10.2012.21.0Version: Windows 10 Education Insider Preview
Version number: 2004
<!--!Installation date :2021/1/29--!>
Operating system version: 21301.1000
Experience: Windows Feature Experience Pack 221.501.0.3

![屏幕截图 2021-01-30 101109](https://user-images.githubusercontent.com/64402861/106344683-48e68880-62e6-11eb-8859-fc060f54444d.png)

I'm having this issue even with simple expressions sometimes such as trying to enter a very simple expression such as x or x * 2 or x^2 even when typed out this is when trying to graph multiple functions simultaneously

Also occurred for me with y=(100/91)x+200

Happens with e^x^3  /6 [arrow right]*e^-x +1 (exactly these keys pressed)

This error still occurs on, for example, y=(x-1)(x-2) but y=(x-1)1(x-2) is fine:

This is still an issue, though it can somehow be fixed by retyping part of the expression string for some reason

It's still an issue, bug was not fixed.

Still an issue :(

For me this issue happens basically every second time I enter whatever function.

It's still an issue.


Caching

Epoxy should cache routes that are visited to avoid repeated database calls for the same route in a short time frame.

Note: this currently wouldn't work well with our usage as we need to check the user agent in the route handler, I propose that instead of the route being set to a function in the returned object, it can also instead be set to an object that has a condition property that is a function that must return true to continue, that isn't cached, and then a handler property that the result of is cached.
Alternatively we could have a condition function for all the routes, that also receives the route and works the same way

I was thinking something similar where the result from the epoxy config is an array of objects with handlers. However, instead of the condition property, I was thinking a key property. That key property is a function with the signature Express.Request -> [any]. Then we cache against the result from that function.
So for example, in stevent we could have
export default {
  '/event/:eventid': {
     handler: eventDetailHandler,
     key: req => [req.params?.eventid, req.get('User-Agent')]
  }
}


installing busy with yarn install failed. The engine "node" is incompatible with this module

hi,
i tried to install busy locally on my laptop as explained in your readme file.
and i got the following error:
~/busy$ yarn install
yarn install v1.5.1
[1/5] Validating package.json...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=7.10.1".
error An unexpected error occurred: "Found incompatible module".
info If you think this is a bug, please open a bug report with the information provided in "/home/pav/busy/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
thanks for your help

You need to use a newer version of Node.js. The minimum is 7.10.1.

you right it worked. thank you!!


Microsoft.IdentityModel.Clients.ActiviteDirectory.AuthenticationContext Constructor missing when updated

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
C# 4.4.5 on .net core 2.2
Describe the bug
The Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationContext constructor used by Microsoft.Bot.Connector is wrong if that nuget package is installed in an updated version. It gives an error that the method is not found in Azure.
To Reproduce
Steps to reproduce the behavior:

Create a new bot template project
Install the nuget package Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationContext version 5 or higher
Deploy the bot to Azure
The bot will not work and an error will be thrown when you use turnContext.SendActivityAsync

Expected behavior
The updated version requires an IHttpClientFactory instead of an HttpClient.
Additional context
Exception message:
Method not found: 'Void Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationContext..ctor(System.String, Boolean, Microsoft.IdentityModel.Clients.ActiveDirectory.TokenCache, System.Net.Http.HttpClient)'.
Stack trace:
   at Microsoft.Bot.Connector.Authentication.AdalAuthenticator..ctor(ClientCredential clientCredential, OAuthConfiguration configurationOAuth, HttpClient customHttpClient)
   at Microsoft.Bot.Connector.Authentication.MicrosoftAppCredentials.<>c__DisplayClass8_0.<.ctor>b__0()
   at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
--- End of stack trace from previous location where exception was thrown ---
   at System.Lazy`1.CreateValue()
   at Microsoft.Bot.Connector.Authentication.MicrosoftAppCredentials.GetTokenAsync(Boolean forceRefresh)
   at Microsoft.Bot.Connector.Authentication.MicrosoftAppCredentials.ProcessHttpRequestAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at Microsoft.Bot.Connector.Conversations.ReplyToActivityWithHttpMessagesAsync(String conversationId, String activityId, Activity activity, Dictionary`2 customHeaders, CancellationToken cancellationToken)
   at Microsoft.Bot.Connector.ConversationsExtensions.ReplyToActivityAsync(IConversations operations, String conversationId, String activityId, Activity activity, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.BotFrameworkAdapter.SendActivitiesAsync(ITurnContext turnContext, Activity[] activities, CancellationToken cancellationToken)
   at Microsoft.Bot.Builder.TurnContext.<>c__DisplayClass22_0.<<SendActivitiesAsync>g__SendActivitiesThroughAdapter|1>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.Bot.Builder.TurnContext.SendActivityAsync(IActivity activity, CancellationToken cancellationToken)
   at EchoBot1.Bots.EchoBot.OnMessageActivityAsync(ITurnContext`1 turnContext, CancellationToken cancellationToken) in C:\Users\kross\source\repos\EchoBot1\EchoBot1\Bots\EchoBot.cs:line 23

[bug]

The workaround is to use version 4.5.1 of  Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationContext.

Closing the issue as it was resolved by rosskyl.


Prevent Re-Weaving via Java Destination Directory Deletion

I observed the following build error if I didn't execute a clean task along with build
Caused by: java.lang.RuntimeException: bad WeaverState.Kind: -115.  File was :::0
at org.aspectj.weaver.WeaverStateInfo.read(WeaverStateInfo.java:170)
at org.aspectj.weaver.AjAttribute.read(AjAttribute.java:105)
at org.aspectj.weaver.bcel.Utility.readAjAttributes(Utility.java:101)
at org.aspectj.weaver.bcel.BcelObjectType.ensureAspectJAttributesUnpacked(BcelObjectType.java:383)
... 84 more
In the Maven AspectJ plugin, the potential for encountering this issue is accounted for by segregating output directories to prevent this error from occurring. See http://mojo.codehaus.org/aspectj-maven-plugin/weaveDirectories.html for more details.
        <!-- Modifying output directory of default compile because non-weaved classes must be stored
             in separate folder to not confuse ajc by reweaving already woven classes (which leads to
             to ajc error message like "bad weaverState.Kind: -115") -->

I think the more appropriate fix would be to mimic the design of the Maven plugin, but a temporary solution is to delete the destination directory before invoking the compiler

@esmasui Do you see any concerns with this temporary approach; and do you have any plan (or see the need) to update such that the behavior is similar to that of the Maven plugin?


Roadmap for sanity-ruby gem

Great initiative on creating a Ruby gem for Sanity :)
I am researching which content platform to build a fairly large project with—wondering if you guys have any thoughts about the direction of this gem or some kind of roadmap?
Would be very helpful to know a little more!
Have a great week, best Oliver

@olivertedwards apologies for the delay. There's no current roadmap for this gem. This gem was built to help our team manage interactions between Rails apps and our Sanity desk. TMK most available features available through Sanity's API are supported via this gem.
Are there features here that are missing or that you'd like to see us work on? Always open to suggestions

Closing for now.


feat: Implemented Design for Home Banner, Footer Menu, and Responsive Navbar Across All Pages

This pull request addresses the implementation for the Home Banner, Footer Menu, and The Responsive Navbar across all pages.
1. Improved navigation experience with a responsive Navbar.

When closed :

When opened :

Fixed Issue: #104
Responsive design updates for Home Banner.

Fixed Issue: #64
Enhanced responsiveness in Footer Menu.

Fixed Issue: #69

@ItsRoy69 Please Review this PR

LGTM, thanks for contributing. That was huge PR.

LGTM, thanks for contributing. That was huge PR.

Can you please add the label IWOC for this PR?

@ItsRoy69  In This PR the Vercel Preview was fine. But after merging this PR the trusted by developers section is gone break. Can you please tell me the reason why this issue is happening?

Merge conflict issues

Merge conflict issues

@ItsRoy69
Can you please suggest me something to follow so that I can avoid merge conflict issues?

@YashNuhash before committing from localhost, try to git pull to keep your local codebase updated


Adds TLS SNI to ghidra client connections

This PR adds the SNI extension to the TLS requests, which enables proxying by domain.  It should be ignored/unused for direct connections, since the Ghidra server-side doesn't look at SNI.

Adds SNI to RMI sockets (13100 and 13101)
Adds SNI to Block Stream socket

Had you encountered issues with the canonical hostname resolution issue #4924 in conjunction with the use of a proxy server?

No, but it could be used to work around the issue in a cloud environment.  AWS allows you to configure reverse DNS for Network Load Balancers with EIPs assigned.
This PR was originally created to allow for a single AWS SNI-based proxy to be in front of multiple Ghidra Server instances using the same RMI ports.  SNI aware proxies can route IP traffic to different backends solely based on FQDN and subdomain used to connect to the server.

@dtshepherd I think this PR suffers from the same problem that the canonical hostname issue raises. The key point is that unless a different IP is used for each connection, you cannot distinguish between connections. You cannot use a proxy because regardless of the servername the user entered when creating the project, Ghidra will always connect with (and send as the SNI) the canonical hostname for the IP.
So this PR in itself doesn't really help at all, and cannot be used to work around the canonical hostname issue.

This PR is really dependent on correcting the canonical hostname transformation issue raised by #4924.

Is this change still neccessary given that the canonical hostname transformation issue raised by #4924 has been resolved?


[SPARK-35282][SQL][FOLLOWUP] Make config  maxShuffledHashJoinLocalMapThreshold fallback to advisoryPartitionSizeInBytes

What changes were proposed in this pull request?

Tune spark.sql.adaptive.maxShuffledHashJoinLocalMapThreshold default value.
Why are the changes needed?

Make config spark.sql.adaptive.maxShuffledHashJoinLocalMapThreshold fallback to spark.sql.adaptive.advisoryPartitionSizeInBytes so that we can convert SMJ to SHJ in AQE by default.
Does this PR introduce any user-facing change?

Yes, default value changed.
How was this patch tested?

Pass CI.

Test build #139001 has started for PR 32683 at commit 381379d.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43519/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43519/

cc @hvanhovell @maryannxue @JkSelf @viirya @maropu

@ulysses-you can you make a new JIRA ticket? This will be a big decision.

created SPARK-35540

Test build #139001 has finished for PR 32683 at commit 381379d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139001/

Test build #139016 has started for PR 32683 at commit 38251d0.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43533/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43533/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43533/

Test build #139016 has finished for PR 32683 at commit 38251d0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139016/

At present, AQE uses the compressed size to do conditional judgment. In some extreme cases, such as when the compression ratio is relatively high, if smj is converted to shj when enable AQE, it is very likely that the oom issue will happen when building the hash table in shj.

Another concern is the test coverage of shuffle hash join, as it's disabled by default.
Can we add a testing config to forcibly apply shuffle hash join, and then test different join algorithms in golden file tests? e.g., in outer-join.sql, we have
--CONFIG_DIM1 spark.sql.autoBroadcastJoinThreshold=10485760
--CONFIG_DIM1 spark.sql.autoBroadcastJoinThreshold=-1,spark.sql.join.preferSortMergeJoin=true
--CONFIG_DIM1 spark.sql.autoBroadcastJoinThreshold=-1,spark.sql.join.preferSortMergeJoin=false

But preferSortMergeJoin=false doesn't guarantee to use shuffle hash join, as we still have conditions like streamside must be 3 times larger than build side.
The same applies to the TPCDS result checking, and by default we probably only test the broadcast join, cc @maropu

What I saw is advisoryPartitionSizeInBytes is not big (64MB), is OOM likely a concern here? What I concern is maxShuffledHashJoinLocalMapThreshold is silently bound to advisoryPartitionSizeInBytes by default. If users don't notify that and change advisoryPartitionSizeInBytes, they might not be aware of the fact that they are changing join strategy.

If users don't notify that and change advisoryPartitionSizeInBytes, they might not be aware of the fact that they are changing join strategy.

This is a good point. We can probably hardcode the default value as 64mb

+1, I like 64mb as default instead of fallback.

And just prepare to run tpcds-sf100 to check if there has some issue or improvement with/without smj -> shj.

This is a good point. We can probably hardcode the default value as 64mb

Sounds good.

In the production environment, we have encountered oom issue when AQE is turned on to convert smj to bhj. The reason is that the compression ratio of the data is too large, the size after compression is 16M, but the data after decompression is 2GB when building the hash table. I think this concern still exists in shj.

Also this is a big changes and it is better to have a config to control this feature. So even if OOM appears, users can bypass this problem by disabling this switch.

Also this is a big changes and it is better to have a config to control this feature. So even if OOM appears, users can bypass this problem by disabling this switch.

we can tune this config to 0 to disable the feature.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43577/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43577/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43577/

Test build #139056 has finished for PR 32683 at commit ce27e6a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139056/

Here list some queries with obvious performance differences. Note that, seems query q72 has a perf regression .
tpcds sf100 perf number(seconds) with two value of spark.sql.adaptive.maxShuffledHashJoinLocalMapThreshold:

query
64mb
0
diff

q2
72.224
109.094
0.337965424

q11
87.811
106.552
0.175885952

q14a
269.834
298.432
0.095827525

q14b
200.496
212.351
0.05582738

q32
24.37
27.066
0.099608365

q37
31.595
36.413
0.132315382

q57
40.86
46.324
0.117951818

q69
42.891
52.873
0.188792011

q72
124.234
98.2
/

The same applies to the TPCDS result checking, and by default we probably only test the broadcast join, cc @maropu

Yea, it would be nice to add test paths for the other join strategies in these TPCDS tests.

Here list some queries with obvious performance differences. Note that, seems query q72 has a perf regression .

Do you know why the regression happens there? And, does the regression occur even if a different value (e.g., 32mb, 128mb, ...) is set?

@maropu  After taking a deep look, I think it's about compression ratio that the extreme case @JkSelf  metioned above.
Here is some pictures of q72.  We can see the 60MB data contains 4.9million rows.

@ulysses-you What is the data size in your TPC-DS test ?

@ulysses-you What is the data size in your TPC-DS test ?

@JkSelf 100GB, as I mentioned sf100.

I think the only blocker to this change is we need to track precise shuffle data size in AQE.

At present, AQE uses the compressed size to do conditional judgment. In some extreme cases, such as when the compression ratio is relatively high, if smj is converted to shj when enable AQE, it is very likely that the oom issue will happen when building the hash table in shj.
I think AQE is Based on metrics dataSize,and dataSize is the actually size, rather than the compressed size. Can you point this why?


Fix fn calls being treated as methods in object literal

Before:

After:

Thank you, I also add another commit to make sure . is in correct group. Maybe your fix will not necessary(preceding look up hurts performance). Will take more test to confirm.

I think you are right, this is not needed after @189d0b6. Great that we can avoid the cost of another lookbehind, and my fix isn't very good anyway because it isn't semantic.
Few more cases to consider:
let foo = {
  bar: baz(42).
       and(fuzz)  // not a method
}

let foo = {
  bar: baz(42)
       .
       and(fuzz)  // not a method
}

let foo = {
  // Comment ends with dot.
  and(fuzz) { }
}

All of these will work as expected if you add skipwhite skipempty to javascriptDotNotation:
@@ -239,1 +239,1 @@
- syntax match   javascriptDotNotation           /\./ nextgroup=javascriptProp,javascriptMethod
+ syntax match   javascriptDotNotation           /\./ nextgroup=javascriptProp,javascriptMethod skipwhite skipempty


[Bug] create table with DUPLICATE KEY can not use select(*)

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.2.3-rc02-de2bc98
What's Wrong?
select count(*) from test;
ERROR 1105 (HY000): errCode = 2, detailMessage = (<IP_ADDRESS>)[INTERNAL_ERROR]failed to initialize storage reader. tablet=6828716.594421327.184494b44ff63ed3-ad900306f7f054aa, res=[E-3111], backend=<IP_ADDRESS>
What You Expected?
fix it
How to Reproduce?
CREATE TABLE test (
data_source varchar(190) NOT NULL
) ENGINE=OLAP
DUPLICATE KEY(data_source)
COMMENT 'OLAP'
DISTRIBUTED BY HASH(data_source) BUCKETS 16
PROPERTIES (
"replication_allocation" = "tag.location.default: 3",
"in_memory" = "false",
"storage_format" = "V2",
"disable_auto_compaction" = "false"
);
insert into test values('1');
select count(*) from test;
ERROR 1105 (HY000): errCode = 2, detailMessage = (<IP_ADDRESS>)[INTERNAL_ERROR]failed to initialize storage reader. tablet=6828716.594421327.184494b44ff63ed3-ad900306f7f054aa, res=[E-3111], backend=<IP_ADDRESS>
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Maybe the tablet is damaged， show tablet 6828716;


SharePoint List Query has broken filters

The errors seem to be in the src/PropertyFieldSPListQueryHost.tsx file. The query is being built with "$filters" instead of "$filter", which is causing the query to be incorrect and results in all list items being returned. There are still other issues, like "AND" should be "and", and operator keys need to be updated. For example, "Ge" should be "ge", "Le" should be "le", and so on. Once these strings are all fixed, the filters seem to be working again. Is this a bug, or should the filters work as is?

I have submitted a pull request to fix the OData query errors.
Thanks

Thanks a lot @macica2 for your bug report and @TravisGilbert for the fix. The new version 1.3.4 has now this fix included.


Angular Formly with dynamic custom datetime field is not showing validation message on form submit

`ngOnInit(): void {
this.setFields();
}
private setFields() {
this.fields = [
{
key: "dateOfBirth",
type: "custom-date",
templateOptions: {
label: "FamilyPay.CreateHolidayClubOrder.EmployeeBirthDate",
type: "date",
minAge: this.model.dateOfBirth ? null : 18,
maxAge: this.model.dateOfBirth ? null : 80,
translate: true,
disabled: !!this.model.dateOfBirth,
required: true,
},
},
];
}

`
Datetime validation is working with Min and Max range but I want to add another validation which validates the date when no value for datetime is selected so It should show a regular message "Date of birth is required"
That would be great if somebody can suggest something on it. Thanks

You'll need to define a custom validation type, please check the doc and examples section:

https://formly.dev/docs/examples/validation/custom-validation
https://formly.dev/docs/guide/validation


Show the HUD without any image or Activity Indicator

I wanted to show the HUD with just text inside the HUD and it shouldn't the Activity Indicator and image inside.
Also can you please make a method to show custom images inside the HUD like SVStatusHUD.

It's called a progress HUD because it's presented during an ongoing task. The activity indicator is there to show that it's still in progress. Without it there's no way to know whether the app is just hanging.
Adding custom image support to SVProgressHUD was in the plans, but I ultimately decided not to implement it. Can you describe your use case to me so I can re-evaluate the need for such a method? You want custom image support for the success/error one of just in replacement of the activity indicator?

I was planning to show a HUD for when the internet was unavailable. For this i wanted to show just the text without activity indicator or maybe show a WIFI icon or something like that.

That sounds like a use case for SVStatusHUD. Feel free to fork it to suit your needs. In all cases I don't think this is a case to be supported by SVProgressHUD, as it's not really a "task" but more of a "status".

ya it makes sense. Thanks buddy. I'm using statusHUD also now


Reading the dataset gives Error

Line no. 2018 of helper.py :  info["duration"] = float(container.duration) / av.time_base results in run time error. The 'container.duration' comes as null/empty which results in  'NoneType object is not subscriptable' later.
As a temporary fix: comment line no. 2018.

Quick fixed after 41f0017


Error after login (The report server has encountered a configuration error. (rsServerConfigurationError))

Hi All, I am try custom authen on power bi report server, but after I login, I had error (The report server has encountered a configuration error. (rsServerConfigurationError)).
I Do not  know how fixed this error. can Someone help me

this event log

And I run portal, in log file by
RSPortal!reportserverwebapp!RSPortal.exe!3!08/27/2017-15:44:37:: e ERROR:  fe80::9577:fb4f:e7e2:5586%12: GET  - 0:00:00.0960627
Exception: System.Web.HttpException (0x80004005): Unable to validate data.
at System.Web.Configuration.MachineKeySection.EncryptOrDecryptData(Boolean fEncrypt, Byte[] buf, Byte[] modifier, Int32 start, Int32 length, Boolean useValidationSymAlgo, Boolean useLegacyMode, IVType ivType, Boolean signData)
at System.Web.Security.FormsAuthentication.Decrypt(String encryptedTicket)
at Microsoft.ReportingServices.Portal.WebHost.Common.RsRequestContext.get_User()
at LVReportingServiceExtension.AuthenticationExtension.GetUserInfo(IRSRequestContext requestContext, IIdentity& userIdentity, IntPtr& userId)
at Microsoft.ReportingServices.Portal.WebHost.Services.AuthenticationService.GetUserInfo(RsRequestContext requestContext)
at Microsoft.ReportingServices.Portal.WebHost.Owin.CustomAuthenticationMiddleware.Invoke(IOwinContext context)
at Microsoft.Owin.Security.Infrastructure.AuthenticationMiddleware`1.d__0.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.ReportingServices.Portal.WebHost.Owin.RequestLoggingMiddleware.d__0.MoveNext() | RequestID = s_73204c0c-c5c9-4ed1-b91d-7f3d7ab4d96a

How did you resolve it?


Improve docker swarm configuration exemple

Improve to use the unix socket as this is what is enabled by default. Also fetch metrics from docker daemons.
Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

I don't know enough about Docker Swarm to really review the content of this change, but since this is documentation about a feature we introduce in 2.20, I'd really like to include this in the release.
Could you rebase on top of the release-2.20 branch? Once this is in, I'll cut the rc.1.

@SuperQ @darkl0rd does this look good to you?

On my hand I have tested this again an it works as I expect it to work.

Let's give it an hour or so for someone to comment. Otherwise, I'll merge and cut the rc.1. (It's "only" an RC, after all. And it's "only" documentation. ;o)

LGTM

Thanks everyone.
Merging without waiting for the circleci build as this change doesn't even touch it.

How to use it into docker?
docker-compose.yml
services:
  prometheus:
    image: prom/prometheus:v2.20.1
    command:
    - --config.file=/etc/prometheus/prometheus.yml
    - --storage.tsdb.path=/prometheus
    volumes:
      - prometheus:/prometheus
      - /var/run/docker.sock:/var/run/docker.sock:ro

scrape_configs:
  - job_name: 'prometheus'
    static_configs:
      - targets: ['localhost:9090']

  - job_name: 'swarm-nodes'
    dockerswarm_sd_configs:
      - host: unix:///var/run/docker.sock
        role: nodes

Get error
level=error ts=2020-09-03T06:51:21.184Z caller=refresh.go:98 component="discovery manager scrape" discovery=dockerswarm msg="Unable to refresh target groups" err="error while listing swarm nodes: Got permission denied while trying to connect to the Docker daemon socket at unix:///var/run/docker.sock: Get \"http://%2Fvar%2Frun%2Fdocker.sock/v1.24/nodes\": dial unix /var/run/docker.sock: connect: permission denied"


[html] hovers in embedded JavaScript should look like in TypeScript

Screenshot:
in JS:

in HTML:

I hope to make the hover prompt in HTML and JS consistent, Thanks!

Hello! I would be interested in working on this issue. I was wondering if anyone would mind leading me in the right direction. Im thinking that since the feature is already there for typescript files I wouldnt have to re write the feature. Im just not quite sure where to look. Thanks!

@amaust You find the code for hovers in embedded JavaScript here: https://github.com/microsoft/vscode/blob/1c0d05691547300f8e9ffd69704eadc1e3c2e0f5/extensions/html-language-features/server/src/modes/javascriptMode.ts#L140
The code for hover in TypeScript/JavaScript is here: https://github.com/microsoft/vscode/blob/1c0d05691547300f8e9ffd69704eadc1e3c2e0f5/extensions/typescript-language-features/src/languageFeatures/hover.ts#L32
The first one uese the TypeScript service, and the second one the TypeScript server. The object representing the QuickInfo is not the same but kind of similar.

Awesome thank you for that, could you also tell me where the HTML hover functions are? @aeschli

Not sure what you mean by the 'HTML hover' functions. The first link is for the HTML hover, specifically when in JavaScript embedded in HTML. Check the code in that folder for css and htmnl element hovers.

Ah I see now, my apologies for the confusion.

@aeschli I fixed it by doing this. But I think this maybe has a better way to do it?

I also tried the solution, and it worked.

Thanks @chenjigeng !


Minor release v2.1.0

Signed-off-by: laurentsimon<EMAIL_ADDRESS>

Signed-off-by: laurentsimon<EMAIL_ADDRESS>
What will the release contain? The new OSV scanner?

correct. ouch. I probably need to update the go.mod

correct. ouch. I probably need to update the go.mod

Already done.

Pre-release run succeeded


On Mac unable to start REPL

I get the following error trying to start the REPL:
This is SBCL 2.2.4, an implementation of ANSI Common Lisp.
More information about SBCL is available at http://www.sbcl.org/.
SBCL is free software, provided as is, with absolutely no warranty.
It is mostly in the public domain; some portions are provided under
BSD-style licenses.  See the CREDITS and COPYING files in the
distribution for more information.
WARNING: System definition file #P"/Users/dbotton/.quicklisp/dists/quicklisp/software/cl-json-20220220-git/cl-json.asd" contains definition for system "cl-json.test". Please only define "cl-json" and secondary systems with a name starting with "cl-json/" (e.g. "cl-json/test") in that file.
debugger invoked on a SB-KERNEL:CASE-FAILURE @70088F2FA0 in thread
#<THREAD "main thread" RUNNING {70086E0423}>:
:LOCAL-NICKNAMES fell through ECASE expression.
Wanted one of (:USE :MIX :REEXPORT :USE-REEXPORT :MIX-REEXPORT :IMPORT-FROM
:SHADOWING-IMPORT-FROM :NICKNAMES :DOCUMENTATION :SHADOW
:EXPORT :INTERN ...).
Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.
restarts (invokable by number or by possibly-abbreviated name):
0: [RETRY                        ] Retry ASDF operation.
1: [CLEAR-CONFIGURATION-AND-RETRY] Retry ASDF operation after resetting the
configuration.
2:                                 Retry ASDF operation.
3:                                 Retry ASDF operation after resetting the
configuration.
4: [CONTINUE                     ] Ignore runtime option --eval "(asdf:load-system :alive-lsp)".
5: [ABORT                        ] Skip rest of --eval and --load options.
6:                                 Skip to toplevel READ/EVAL/PRINT loop.
7: [EXIT                         ] Exit SBCL (calling #'EXIT, killing the process).
(ASDF/PACKAGE-INFERRED-SYSTEM::PACKAGE-DEPENDENCIES (DEFPACKAGE :ALIVE/POSITION (:USE :CL) (:EXPORT :CREATE :LINE :COL :LESS-THAN :LESS-OR-EQUAL :FROM-WIRE) (:LOCAL-NICKNAMES (:TYPES :ALIVE/TYPES))))
; File has been modified since compilation:
;   SYS:CONTRIB;ASDF;ASDF.LISP.NEWEST
; Using form offset instead of character position.
source: (ECASE OPTION
((:USE :MIX :REEXPORT :USE-REEXPORT :MIX-REEXPORT)
(DOLIST (P ARGUMENTS) (DEP (STRING P))))
((:IMPORT-FROM :SHADOWING-IMPORT-FROM)
(DEP (STRING (FIRST ARGUMENTS))))
((:NICKNAMES :DOCUMENTATION :SHADOW :EXPORT :INTERN :UNINTERN
:RECYCLE)))
0]

That's an issue with ASDF being an older version. If you upgrade it to at least 3.3.3, it should work.

Not easily finding instructions on upgrading asdf, any pointers?
On Thu, Jun 9, 2022 at 3:56 PM nobody-famous @.***>
wrote:

That's an issue with ASDF being an older version. If you upgrade it to at
least 3.3.3, it should work.
—
Reply to this email directly, view it on GitHub
https://github.com/nobody-famous/alive/issues/90#issuecomment-1151560699,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYUEFP5EQHCYKSRSBTYMTTVOJEADANCNFSM5YKZSXGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Got it. just git clone https://github.com/fare/asdf.git info ~/common-lisp
Now working thank you.


Evaluate on NuScenes test set

hey, I tried to evaluate the performance of a pretrained PointPillars model on the NuScenes test set. I followed the instruction in the tutorial and changed the path like this:
https://github.com/open-mmlab/mmdetection3d/blob/master/configs/_base_/datasets/nus-3d.py#L132
However, I got the error as follows:
load checkpoint from local path: ./outputs/pointpillars_nuscenes_default/epoch_24.pth
[                                                  ] 0/6008, elapsed: 0s, ETA:/Data/Ubuntu_Default/anaconda3/envs/openmmlab/lib/python3.8/site-packages/torch/functional.py:568: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at  /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/TensorShape.cpp:2228.)
  return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
[>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>] 6008/6008, 4.1 task/s, elapsed: 1481s, ETA:     0s[[B^[[B^[[B^[[B^[[B^[[B^[[B
Formating bboxes of pts_bbox
Start to convert detection format...
[>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>] 6008/6008, 46.0 task/s, elapsed: 131s, ETA:     0s
Results writes to /tmp/tmpyvjypall/results/pts_bbox/results_nusc.json
Evaluating bboxes of pts_bbox
Traceback (most recent call last):
  File "tools/test.py", line 260, in <module>
    main()
  File "tools/test.py", line 256, in main
    print(dataset.evaluate(outputs, **eval_kwargs))
  File "/home/shimingwang/workspace/mmdetection3d/mmdet3d/datasets/nuscenes_dataset.py", line 509, in evaluate
    ret_dict = self._evaluate_single(result_files[name])
  File "/home/shimingwang/workspace/mmdetection3d/mmdet3d/datasets/nuscenes_dataset.py", line 403, in _evaluate_single
    eval_set=eval_set_map[self.version],
KeyError: 'v1.0-test'

Do you know how to solve this?

Could you tell me how did you solve it? I want to use the same dataset for testing, but data_annotations.json file for v1.0-test has just an empty array inside, like this: []


Renamed Cancel() event to Cancelled()

Keeping similar names as of Defaulted, Finished, Cancelled.

Done


Parsing of Temperature data from HDS units and handling of unknown sensor types

Implemented parsing of Temperature data from HDS units.
Added handling of unknown sensor types.

Looks pretty good to me, if it passes testing over the weekend I am fine with merging it.

We've been using this successfully in Cololmbia with no issues. I'll push it out to UCM who initially had the error this was made to fix. I think we can merge.


Revert the lookup on documentView since introduces regression

With this issue, the lookup on the documentView was introduced. The issue is that, the lookup is slower than what we usually had - just lookup on node. So, we should recommend that for global events, that are not on HTMLElement prototype, the lowercase format should be used.
Reverts the fix for #1589
Removes -16B 😄

Coverage remained the same at 100.0% when pulling d1285e232c40be106f2d3f08867e6ea0824d01d3 on revert/Revert-the-documentView-lookup into 38c4f641174b281d923d0dfb15b7689233b6baae on master.


Outdated Content

Hello
Network load balancer deployment guide leads here:
https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-server-2008-R2-and-2008/cc754833(v=ws.10)
Which then leads to what looks like it might have been a design guide for NLB on server 2003?
https://technet2.microsoft.com/windowsserver/en/library/4d24fdac-edf3-4ac5-937a-273867f46fcf1033.mspx?mfr=true
Any updated design guides available?

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

ID: 62f953b5-5ea4-4435-37b9-53484f75b316
Version Independent ID: 6394da95-c5cb-2e17-2ae7-82e203b16c9e
Content: Network Load Balancing
Content Source: WindowsServerDocs/networking/technologies/Network-Load-Balancing.md
Product: windows-server
Technology: networking
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

Hello @mr-fujnz,
We'd like to express our gratitude for your commitment to our documentation. Support for Windows Server 2008 and 2012 has ended, and we don't update documentation for those previous versions, so all the linked pages in that table are unsupported and are for historical purposes only. Unfortunately, we don't have an updated "Deployment Guide" per se for NLB.
For updated information, you could consult the Load Balancer Documentation, or the Q&A Page for Load Balancer.
Here are some other channels for engaging with Microsoft for support questions, product feedback, or other feedback:

Bug submittal or feature requests
 - From Windows, use Windows Feedback Hub or press Windows + F. You can also provide feedback with Feedback.
Open a ticket with Microsoft Support.
Search or engage with Microsoft TechNet.
Search or engage with the Microsoft Developer Network.
Search or engage with the Microsoft Support Community.

We offer our apologies for any inconvenience we’ve caused, and welcome any further suggestions for improving our documentation to provide a more satisfactory experience in the future. Thanks for your understanding and continued support.
#please-close


Update NFT page with new UI

Summary
You can click next without filling out both required. This should be rethought
Design: https://www.figma.com/file/EN7CZTekcSo3sHyaGt6cC0/JBX---Project-Creation-Flow---High-Fidelity---Updated-Jan-2023?node-id=3285%3A163287&t=zTZOrZQKv7M1QqcJ-1
Thread: https://discord.com/channels/939317843059679252/1068601323475456051/1069477650432806973

https://u3302489.ct.sendgrid.net/ls/click?upn=5jn-2BNVr6mWrJbcZ1z4kro3nySutAgQPt34lNFAjvKx18J6EtfN0IydFNhmyIvmzCHOPswZDDC9HiJ1xNZRSYE9To7oxVzXsvtVXueE8Z8gTCRc6kN-2FCM6PVaZBpbm55gqES-2BFFa2vuOlTmTHBc8zkxiwwOi9wFpsgN82tmG6gy4-3DK5Vv_uLUa1W0cH0Aq-2BecF6inFavBFSDG891nrsrNm5Y6aeClkX5dN5OlXf2pjGcbNjCkbYqH5t2B7Ea9u6ivNmdHrjhd-2FrnmqA6s-2F3r1rbQU9fTGYZ8DHbnlMee8riC6Jq9LhVW-2Bfu35N-2BPL4s1K4qDLMCV-2FA0Ueu4LiJp2nEzFMT3cCuh4rH0z-2F1ZgonHfM1Gj2aLjNfBHy-2Fw59S0pPezYem8zEWDnWUJy0uwNYRQZDsVHZHTgTEdE0aqjSjkjN309yK

@wraeth-eth maybe can bundle this in here too: https://github.com/jbx-protocol/juice-interface/issues/2775

@wraeth-eth maybe can bundle this in here too: #2775

sounds good to me @tomquirk what do you think is a reasonable increase?

cc @nnnnicholas re individual NFT description size increase

bump @tomquirk @nnnnicholas re NFT description size

sounds good to me @tomquirk what do you think is a reasonable increase?
Maybe 10k characters? Doesn't cost us much and never know how someone might want to use that space.


Support tweet entities like media, urls, mentions, hashtags

Support the following entities returned from JSON as "models" in the twitter4r API:
http://dev.twitter.com/pages/tweet_entities

See pull request #19


VS Code: Refactor does not appear on editor context menu

Steps to Reproduce

VS Code ver 1.63
Flutter (Channel stable, 2.8.1, on Linux Mint 20.3 5.4.0-96-generic, locale en_US.UTF-8)
flutter doctor - all is ok

Logs

 Refactor does not appear on editor context menu.No refactor menu if press ctl+. OR ctl+space OR mouse right click on widget

Hi @andim27, can you share a screen recording that shows the issue you're experiencing?

Hi @andim27, can you share a screen recording that shows the issue you're experiencing?

I can mention what if I open project this menu appears from ~15 min of working.And  I saw it happens when gradle doing many tasks in background

I'm not entirely sure that the flutter plugin for vscode has any control over that context menu you're seeing.
Regardless, the vscode plugin lives in its own repo over here.
Kindly file this issue there so it may be properly addressed.
Closing from here.
Thank you


API key remains in the DB and is not actually disabled

Link to reproduction
https://github.com/franciscolourenco/payload/commits/reproduce-api-key-not-being-disabled/
Describe the Bug
When disabling an API key, enabledAPIKey is set to false, but the apiKey property is not removed for the user document, so it's still possible to authenticate using it.
To Reproduce

pnpm dev _community
Manually enable API key
Copy the API key
Save the user document
Disable the API key
Save the user document
Test if API key is still working, with eg. curl 'http://localhost:3000/api/users/me' -H "Authorization: users API-Key [replace-with-api-key]" | jq

Payload Version
2.12.1
Adapters and Plugins
No response

There was an attempt to fix this in https://github.com/payloadcms/payload/pull/5145/files, however the fix doesn't seem to be working.
The tests passe because authentication isn't actually working, even if the API Key is enabled.

Thank you for the report @franciscolourenco . We'll look into this.

const encryptKey: FieldHook = ({ req, value }) =>
  value ? req.payload.encrypt(value as string) : null

The change in line 10 also had an unexpected side effect: If the apiKey property is not provided in data (for example because it was removed in a hook), the ke will set to null. This deviates from the normal behavior which is that undefined properties are left unchanged.

@franciscolourenco Thanks to your comment I investigated a little further and made more improvements.
First off, I added a test specifically for the thing you called out. It wasn't broken because behind the scenes payload makes an index field for looking up the API key user in a secure encrypted way so that separation meant it was not removing the ability to login. I left the test in though as a safeguard for any future issues with this.
Then I did make changes to the index hook to make it copesetic. Now an update with apiKey: null or enableAPIKey: false will strip the index back to null so that a user cannot login with that key any longer.
Cheers!

First off, I added a test specifically for the thing you called out. It wasn't broken because behind the scenes payload makes an index field for looking up the API key user in a secure encrypted way so that separation meant it was not removing the ability to login. I left the test in though as a safeguard for any future issues with this.

@DanRibbens not completely sure which of the things you are referring to in this one.
I gave the PR a try, and can confirm that the apiKey is correctly disabled 👍

However the issue with the key being set to null when not present still exists.
Here is some reproduction code.
Integration Test:
 FAIL  test/_community/int.spec.ts (7.782 s)
  _Community Tests
    ✓ the hook should prevent further changes to the username (5 ms)
    ✕ should not delete API key when updating other fields (4 ms)

  ● _Community Tests › should not delete API key when updating other fields

    expect(received).not.toBeNull()

    Received: null

      66 |     })
      67 |
    > 68 |     expect(foundUser.apiKey).not.toBeNull()
         |                                  ^
      69 |
      70 |     const updatedUser = await payload.update({
      71 |       collection: 'users',

      at Object.toBeNull (test/_community/int.spec.ts:68:34)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 passed, 2 total
Snapshots:   0 total
Time:        7.803 s

Manual test:
https://github.com/payloadcms/payload/assets/208149/6741ee75-a512-44c8-8172-3de43a9b84d9
Returning undefined in this line seems to be sufficient to solve the issue and make all tests pass:
const encryptKey: FieldHook = ({ req, value }) =>
  value ? req.payload.encrypt(value as string) : undefined


add lookup

It came into my head that lookup is a left fold, of a piece with index, find and so on. It is trivial but may be of interest as helping get beginners to see the range of things that can be represented as left folds.

Great idea!  Thanks :)


Should allow for providing credential JSON in different ways

This library currently cannot work on hosting providers that have ephemeral file systems like Heroku. It would be handy if there were a way to provide this object directly, rather than by file path.
Since these systems have an ephemeral fs you cannot easily include files there without having them checked into the git repo.

I believe this should already work:
const GoogleAuth = require('google-auth-library');
const googleAuth = new GoogleAuth();

googleAuth.fromJSON("your JSON key", (err, authClient) => { .... });

@askdesigners, @logicoder, @fhuel: does this address your use cases?
I can adjust the docs which don't seem to provide details on these. I would also suggest checking out google-auto-auth which provides a higher level API to work with google auth that might be a bit more convenient to use.

Fixed in #220


Type inference doesn't work when indexing a generic type

TypeScript Version:  2.4.0 / nightly (2.5.0-dev.201xxxxx)
Code
enum CFieldType {
  NUMBER,
  TEXT,
};
class CTextValue {
  fieldType: CFieldType.TEXT;
  value: string;
}
class CNumberValue {
  fieldType: CFieldType.NUMBER;
  score: number;
}
type CValue = CTextValue | CNumberValue;

class Aggregate<V extends CValue> {
  test(fieldType: V['fieldType']): string {
    return CFieldType[fieldType];
  }
}

(In TS playground: https://goo.gl/hctJPv)
Expected behavior:
In the test method, fieldType should have type CFieldType.
Actual behavior:
In the test method, fieldType has type CFieldType Value["fieldType"].
This causes an error:
Type 'V["fieldType"]' cannot be used to index type 'typeof CFieldType'.

Do not see any errors on latest.

Can confirm, it looks fixed in 2.6, still broken in 2.5
$ tsc -v
Version 2.5.2
$ tsc ts-repro.ts
ts-repro.ts(17,12): error TS2536: Type 'V["fieldType"]' cannot be used to index type 'typeof CFieldType'.
$ npm install -g typescript@next
$ tsc -v
Version 2.6.0-dev.20170916
$ tsc ts-repro.ts

It can also be reproduced in TS playground https://goo.gl/hctJPv

Playground is still on 2.5


abacus-debug does not work with debug 2.4.x

The debug module changes signature for formatArgs which breaks our custom formatting.
We also fail to change the logging function with clog

Proposed a fix in debug with https://github.com/visionmedia/debug/pull/379


Support of slice for Program code

Now Program::code is ArrayVec type, which contains its own data.
I think almost all of use cases are to program "static" one, and it is not necessary to allocate memory every time.
So I suggest to add slice support for Program::code.
Furthermore, I want pio_proc assembler macros to support slice as well.
The macros returns ProgramWithDefines<> type, which has dynamic type parameter, so it is not possible to use it as static variable.
I am able to work with this change, but I am not sure if it is good idea or not.

As pio programs are only a few bytes long, I wonder if it's worth to spend effort on optimizing memory usage.
Do you have an example where such a change would help?

Hello, @jannic.
Your indication is correct; the performance problem is not so significant.
I don't like that the process is not minimal.
However, I think that is more convenient since slice is available as a primitive type.

In almost all use cases, the program is a static value, so we can use a static slice.

The difference is little, but a simple and fast way is preferable.

When dynamically generating programs, we don't need to convert the array to ArrayVec.

The code I first confront this issue is like this:
fn main() {
  //
  pio0.install(&PROGRAM).unwrap();
  //
}

// Here the program is compile-time-static value
// but this approach is not available.
static PROGRAM: Program<32> = pio_proc::pio_asm![
  ".wrap_target",
  "  set pins, 1",
  "  set pins, 0",
  ".wrap",
].program;

I get some errors because ArrayVec does not work in statics.


Update akkasls to 0.1.0

Hi @fernandoiury,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
https://www.lightbend.com/contribute/cla


Correctly using shader

Using the 'Converting the Object-Space forward to World-Space' results in only 3 sides of the spite being viewed.
I am wondering where does Normalize connect too? Thanks!

Edit: I've got the shader working but it seems the 'is moving'  or any addressable tiling is not inside the downloadable shader

This repo unfortunately doesn't have the changes mentioned in the second article: https://www.noveltech.dev/sprite-change-camera-direction-2/
I copied the two shader graphs using the images in the article. Getting the animation to work was simple. I just had to connect the output of the elapsed time to the X coordinate of the sprite selector (where the Y coordinate changes the viewing angle of the character).
However I haven't been able to get 'Converting the Object-Space forward to World-Space' to work. It outputs a Vector3 and I'm not sure where to connect it to. From the article it sounds like the shader graph for selecting the viewing angle of the character has been changed so there's probably a lot more I would need to modify.
@StatueTheShaman Could you show me a screenshot of your shader graph?