Decide how to handle person names

Problem: There are many different used to represent person names. See this w3 article for a detailed discussion and IFLA - Names of persons for a comprehensive overview for each nation. A few examples:

Iceland: Björk Guðmundsdóttir with "given" name Björk and "family" name Guðmundsdóttir, although the latter is more a description/patronymic than a proper family name (it means daughter of Guðmundur). People prefer to be called by their given name (Björk) or by the full name. Sorting is done by given name. Also an article written by Björk Guðmundsdóttir would be cited by an in-text citation of the form Björk 2020 (and not Guðmundsdóttir 2020).
China:  毛泽东 (Mao Ze Dong) the family name is 毛 Mao. The given name is Dong. The middle character, 泽 Ze, is a generational name and is somewhat part of the given name, i.e. you would say "Mr Mao" and "I'm a good friend of Ze Dong". There is no space between the characters in the name.
Spain: María José Carreño Quiñones where Carreño Quiñones are multiple family names, and María José multiple given names. There might be additional particles like Carreño de Quiñones.
America: Often people prefer to have their middle name abbreviated, e.g. John Q. Public (This is however not common eg in the UK)
Philippines: Abbreviate double-middle names with only one letter, e.g. Maria D. Go for Maria Dela Cruz Go.
India: Sometimes includes further information in the name, e.g. Kogaddu Birappa Timappa Nair follows the order villageName-fathersName-givenName-lastName
Vietnam: Names such as Nguyễn Tấn Dũng follow the order familyName-middleName-givenName. They are referred to using the given name, ie. Mr. Dũng, not Mr. Nguyễn.
Netherlands: Last names with a van particle, like van Beukering should be sorted under B; but if the person is Belgian, sort it under V

Takeaways:

We cannot take the "family" name to get an in-text citation (e.g. Iceland)
We cannot take the initial letters from the middle name to get the initials (e.g. Philippines)
There is no consistent way to sort last names without further information which particles should be included/omitted (e.g. Netherlands vs Belgium)

This should be confronted with the following requirements for citation:

For the references, one usually needs the full name.
Sometimes an abbreviated name (i.e. with initials) is required in the references by some styles.
Need a way to consistently sort names.
For in-text citations one usually needs a short name.

Overview of existing solutions:

CSL:

“family”: surname minus any particles and suffixes
“given”: given names, either full (“John Edward”) or initialized (“J. E.”)
“suffix”: name suffix, e.g. “Jr.” in “John Smith Jr.” and “III” in “Bill Gates III”
“non-dropping-particle”: name particles that are not dropped when only the surname is shown (“de” in the Dutch surname “de Koning”) but which may be treated separately from the family name, e.g. for sorting
“dropping-particle”: name particles that are dropped when only the surname is shown (“van” in “Ludwig van Beethoven”, which becomes “Beethoven”)

biblatex

family: The family name(s), know as ‘last’ in BibTeX. If a name consists of a single part only (for example, ‘Aristotle’), this part will be treated as the family name.
given: The given name(s). Note that given names are referred to as the ‘first’ names in the BibTeX file format documentation.
prefix: Any name prefices, for example von, van, of, da, de, del, della, etc. Note that name prefices are referred to as the ‘von’ part of the name in the BibTeX file format documentation.
suffix: Any name suffices, for example Jr, Sr. Note that name suffices are referred to as the ‘Jr’ part of the name in the BibTeX file format documentation.

ORCID

"given-names": The given name(s) of the researcher or contributor.
"family-name": The family (last) name of the researcher. This element is optional, because some cultures only use given names.
"credit-name": The full name of the researcher as they prefer it to appear.

Citation-file-format

To be consistent with JabRef I would suggest using the biblatex one.

Decision: use CSL
In the long-term I would like to move to a more inclusive standard instead of the western-focused first + last name format. However, this would require a huge effort and changes in all involved parties (Jabref Desktop, CSL, Biber, ...). So let's do this later.
We choose the CSL format since it contains more information (namely if a prefix is "dropping" or not) and there is lossless conversion from CSL to biblatex, while the opposite conversion biblatex > CSL needs additional info.


Should not send unbind to broker for a Binding that was never successful

If a binding is not ready because BindCallFailed, then the broker has never successfully handled the binding on its end.  If the Binding object is then deleted from the service catalog, then currently the svc cat controller still sends an unbind call to the broker, which then will fail since the binding was never actually created in the broker.
This results in a state where the Binding is stuck in a retry loop on unbind that cant succeed and the object never gets deleted.
Conditions after creating the Binding:
status:
  conditions:
    - lastTransitionTime: '2017-06-20T15:28:03Z'
      message: >-
        Bind call failed. Error creating Binding
        "jenkins-ephemeral-nqw3d-5knrd/test" for Instance
        "test/jenkins-ephemeral-nqw3d" of ServiceClass "jenkins-ephemeral" at
        Broker "template-broker": Unexpected status code from broker response:
        400
      reason: BindCallFailed
      status: 'False'
      type: Ready

Conditions after DELETE on the Binding:
status:
  conditions:
    - lastTransitionTime: '2017-06-20T15:28:03Z'
      message: >-
        Unbind call failed. Error unbinding Binding
        "jenkins-ephemeral-nqw3d-5knrd/test" for Instance
        "test/jenkins-ephemeral-nqw3d" of ServiceClass "jenkins-ephemeral" at
        Broker "template-broker": There is no binding with the specified service
        instance id and binding id
      reason: UnbindCallFailed
      status: 'False'
      type: Ready

cc @pmorie @bparees

Looks like we can edit the client. It sort of depends on what the answer to  "If we're trying to delete a binding, and it doesn't exist, is that an error, or can we just continue as if it was successfully deleted?"
We don't check that the binding exists before trying to delete it, so we could also check that.
I was looking at the call to DeleteServiceBinding and the contents of that method.

#954 fixes this by checking for the 410 isgone.

@jwforres can we close?

ping @jwforres

When i tested this with #954 it seemed to fix it so closing this


Update godotcpp version to 3.5.2

Add godotcpp version 3.4.5/3.5/3.5.1/3.5.2.
source: https://github.com/godotengine/godot-cpp/tags

please pus to dev branch to trigger ci next

please pus to dev branch to trigger ci next

okay, I will remember this from now on.


Add Automated Linting

Description
We should use GitHub actions to lint the styles and js once #7 is in place.

Closed by #9


Laravel 5 support

Laravel 5 support
Rely on the configuration repository contract to read configuration values
Inject the configuration repository in the service provider
Follow new coding conventions
Update the README file
Add extra information in the configuration file

Travis was failing because we now should require PHP 5.4

@thomaswelton please add this PR.

+1

@thomaswelton Do you intend to continue to maintain this package? If not, I'd suggest marking it as deprecated so someone else can pick it up.

@AntoineAugusti I should now be staying on top of the maintenance of this and my other packages. However if not, and in appreciation of your contributions I have made you a collaborator on this repo.
Thanks.


Formatting and Wording of Main/CMain

In particular, replace “must be declared” by “is typically declared,” as many variants are possible.

The specification should be exact. If there are valid main signatures missing, they should be added.

The specification should be exact.

Agreed.

If there are valid main signatures missing, they should be added.

Trying to be absolutely technically correct with grammar here would be madness. The reason is that there are simply too many syntactic ways to express the main function semantically correctly.

The return type, if present,

can be an Identifier which is an alias in-effect to void, int, or noreturn;
can also be a typeof(Expression) if the Expression has one of the allowed types;
can be an enum type that is backed by int;
can be any of the above with const or immutable qualifiers attached.

The parameter, if present,

need not be named;
in practice, may have any qualified version of char[][];
may have storage classes;
may have its whole declaration be an alias.

My best take:

MainFunction:
    StorageClasses? MainReturnType main MainParameters? FunctionAttributes? MainFunctionBody
    StorageClasses main MainParameters? FunctionAttributes? MainFunctionBody

MainReturnType:
    void
    int
    Identifier

MainParameters:
    ( MainParameter )
    ( MainParameter , )

MainParameter:
    MainParameterDeclaration
    MainParameterDeclaration = AssignExpression

MainParameterDeclaration:
    ParameterAttributes? MainParameterBasicType MainParameterTypeSuffixes? Identifier?

MainParameterBasicType:
    char
    TypeCtor ( TypeCtors? MainParameterBasicType MainParameterTypeSuffixes? )
    Identifier

MainParameterTypeSuffixes:
    [ ]
    [ ] [ ]

MainFunctionBody:
    SpecifiedFunctionBody
    ShortenedFunctionBody

(Essentially, I ended up repeating the FuncDeclaration grammar, pruning it where it can’t be semantically correct.)
The Identifier in MainReturnType must be an alias to void, int, or noreturn, or an enum backed by int — that is near-impossible to express with syntax.
In MainParameterDeclaration, the MainParameterBasicType and MainParameterTypeSuffixes must agree to produce a type that is semantically a dynamic array or dynamic array of char, qualified in any shape or form, including mutable, inout, and shared.
For some reason, the parameter of main can have a default argument, but not be var-arg.
The fact that the parameter type can both be char[][] and string[] is wild. The only way this can be right is if the backing array is indeed mutable and converts to string[] because it’s unique.
Doing the above again for the extern(C) main function:

CMainFunction:
    StorageClasses? extern ( C ) StorageClasses? MainReturnType main CMainParameters? FunctionAttributes? MainFunctionBody

CMainParameters:
    CMainParameterList
    CMainParameterList ,

CMainParameterList:
    CMainFirstParameter , CMainNextParameter
    CMainFirstParameter , CMainNextParameter , CMainNextParameter

CMainFirstParameter:
    CMainFirstParameterDeclaration
    CMainFirstParameterDeclaration = AssignExpression

CMainNextParameter:
    CMainNextParameterDeclaration
    CMainNextParameterDeclaration = AssignExpression

CMainFirstParameterDeclaration:
    ParameterAttributes? CMainFirstParameterBasicType Identifier?

CMainFirstParameterBasicType:
    int
    TypeCtor ( TypeCtors? CMainFirstParameterBasicType )
    Identifier

CMainNextParameterDeclaration:
    ParameterAttributes? CMainNextParameterBasicType CMainParameterTypeSuffixes? Identifier?

CMainNextParameterBasicType:
    char
    TypeCtor ( TypeCtors? MainParameterBasicType CMainParameterTypeSuffixes? )
    Identifier

CMainParameterTypeSuffixes:
    *
    * *

I’ll translate that into the DDoc syntax if you think that’s valuable. Otherwise I’d rather remove the grammar part and just explain the declaration with prose.

@ntrel, this should be good to go. Please have a look and if it’s good, merge.

I prefer Dennis's suggestion, I think that's clearer.
(I don't have merge rights).

The current formulation:

The main function must have either no parameters or a single parameter. If provided, the parameter must be a possibly qualified version of char[][], and can optionally have the ParameterAttributes in, return or scope. It is customary to use immutable(char)[][], i.e. string[], but, in particular char[][] is also allowed. The argument passed to a main function with parameter is mutable and unique, which is why it converts to immutable, inout, or shared.

What exactly do you want to replace by “Other formulations that result in a main with the same or a covariant function type as one of the above are also allowed.”?

I think that sentences can be appended without replacing existing sentences in that paragraph.


SPFx generator references "clientsideinstance.xml" when file is sentence case (fails build on Linux)

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
I am not sure if this is the right Repo to report this issue.
I tried to create a new spfx extension using the yo generator @microsoft/generator-sharepoint
The generator includes this File: package-solution.json
Within this file the ClientSideInstance.xml is written as lower case.
If you build this solution on a Linux based system (e.g. Azure DevOps Build using Ubuntu as this is the default for NPM Builds), you will run into an error
Found Element Manifest: elements.xml
Error - 'package-solution sub task errored'
Could not find Element Manifest: sharepoint/assets/clientsideinstance.xml

If you alter the elements.xml using the exact file name, the build will be successful.
"assets": {
          "elementManifests": [
            "elements.xml",
            **"ClientSideInstance.xml"**
          ]
        }

Steps to Reproduce

Install Sharepoint Generator for yo (install yo if not installed)
npm install @microsoft/generator-sharepoint -g

Create a new extention using the yo SharePoint Generator
yo @microsoft/sharepoint --solution-name "solution-name"  --component-type "extension" --component-description "List Command Bar Extention" --environment "spo" --extension-type "ListViewCommandSet" -- skip-feature-deployment  --component-name "component-name"

Within the elements.xml the filename will be lower case. Builds on Linux Systems will fail.

Good catch... renamed the issue to be more specific on what's going on...
FYI - @patmill @VesaJuvonen, flagged for next SPFx drop?

@andrewconnell. assigned to dev for triage... will let you know soon if we can make it for the next drop ;-)

This fix has been merged and will be available in next SPFx drop cc: @lucabandMSFT.


Print current values of stat counters as well.

For some reasons unknown to me the current values of stat counters are never printed.
This makes is quite hard to use printing during the debugging in the middle of the program run.

I'll merge this but I will probable let the current column follow the freed column instead. Thanks!

I'll merge this but I will probable let the current column follow the freed column instead. Thanks!


⚠ Bofrost.at has degraded performance

In ac234f9, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1214 ms

Resolved: Bofrost.at performance has improved in 17a2970 after 15 minutes.


Document how to use merger tool

Requires:

a gltf with no animations to be the rig
several gltfs with a clip (can there be more than one clip?)
Run the dll via dotnet WonderMedia.glTF.ModelClipMerger.dll
clips with the same name are overwritten in some order.

Currently there's a hard coded path too.

Well, I would have to figure it out myself again, it has been a while since I worked on this ;-)

I figured it out, but we're not using it. So closing bug.


docs: add note to getting started ref lowercase name

Checklist

[x] commit message follows commit guidelines.

[x] website builds and runs in production - for information on how to test locally go here.

Description

Appsody build fails if you have a uppercase letter in your directory, this pr adds a note at the create directory step

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

Thanks for your contribution @andrewhughes101 🙂This issue has been resolved at will be landed in appsody 0.2.5 (see: https://github.com/appsody/appsody/issues/73) so I'm hesitant on landing this PR.
However, you have pointed out that we should find a place for limitations of appsody somewhere central. @mhamwala do we have a section like this currently?

Would it be worth landing this now and @andrewhughes101 opening a follow-up PR to remove the note that is ready to land after the release? So that anyone following the Quick Start instructions in the meantime does not hit this issue?


Update microsoft-edge-devtools-extension.md

Patch line 358

Docs Build status updates of commit 69eff1b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/visual-studio-code/microsoft-edge-devtools-extension.md
:white_check_mark:Succeeded
View

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


add version commands and add version to state

fixes #31
fixes #33
fixes #39

Current coverage is 62.07%

Merging #45 into master will increase coverage by +0.36% as of 81fb011

Powered by Codecov. Updated on successful CI builds.

LGTM


Hate to Spread Negativity

I think that the concept behind 'TokenScript' is honestly one of the most redundant, useless ideas I've seen that also has so much effort behind it in terms of development.
That's my honest assessment. I'm not saying this to cut the project down even though that's exactly what this message does. I just want to provide my feedback because I feel like nobody else has ever said that to you guys before and this is how I genuinely feel.
You All are Talented and Needed Though
This feedback doesn't mean you guys aren't brilliant. What you did with AlphaWallet is absolutely superb and this space will forever remain indebted to you all for it.
My dream is to revisit this project / entity behind both 'TokenScript' & AlphaWallet and find that you all have re-shifted focus and decided to put all of your energies behind the direction pursued by AlphaWallet.
That's not an order. You don't have to listen to me and I don't expect you to. My criticism is going to sound biting, but its an honest opinion.

Thanks for the feedback! To be honest I myself ask me this question once in a while, and I've been with this project since its begining! I wish to dissect this with you. Which one of the following is closer to your view?

The vision that future web3 is decentalised thanks to tokens as trust anchors is implausible.
The mechanism to reach that vision, that web3 websites runs code from multiple tokens of different domains and work together† is not attenable
The above mechanism is attenable but not through a token oriented archtect that is not blockchain
The above is attenable through blockchain but you are making it wrong.

† (compared to today websites runs components of the same domain - I use the word domain abstractively, basically meaning same group of builders with different interest, not 'web domain')
It may appear like dissecting a dead cat but the outcome is different. If 1 or 2 is ultimately true, we better close the project or the company. If 3 is true we showed up at a wrong time and should go to academic research. If 4 is true we still have time to pivot.
Wrong or not, we never was/will be at a point we can claim "we are fine and we don't your help/advice".

Je to konec cesty


[Bug] Read only mode

How to disable selecting days or ranges.
I used IgnorePointer but this will disable scrolling between months and I need to enable it.

anyone know how to disable selection?


Unable to make debug build due to Task :react-native-image-picker:testDebugUnitTest FAILED

Hi, I am using "react-native": "0.60.4" & "react-native-image-picker": "1.0.1" and my gradle settings look like,
buildToolsVersion = "28.0.3"
minSdkVersion = 16
compileSdkVersion = 28
targetSdkVersion = 28
supportLibVersion = "28.0.3"
There is no issue when I am making a release build, but in debug build I am getting the following error:

Task :react-native-image-picker:testDebugUnitTest

com.imagepicker.testing.media.ImageConfigTest > testOnImmutable FAILED
java.lang.ClassCastException
com.imagepicker.testing.media.ImageConfigTest > testParsingOptions FAILED
java.lang.ClassCastException
com.imagepicker.testing.media.ImageConfigTest > testUseOriginal FAILED
java.lang.ClassCastException
com.imagepicker.testing.media.ImageConfigTest > testGetActualFile FAILED
java.lang.ClassCastException
com.imagepicker.testing.ImagePickerModuleTest > testCancelTakingPhoto FAILED
java.lang.ClassCastException
5 tests completed, 5 failed

Task :react-native-image-picker:testDebugUnitTest FAILED

Please help me to fix this issue.

Hi, I have the same errors with :
"react-native": "0.60.0",
"react-native-image-picker": "^1.0.1"

and
buildToolsVersion = "28.0.3"
minSdkVersion = 16
compileSdkVersion = 28
targetSdkVersion = 28
supportLibVersion = "28.0.0"

Same issues here!
Task :react-native-image-picker:testDebugUnitTest FAILED
com.imagepicker.testing.ImagePickerModuleTest > testCancelTakingPhoto FAILED
java.lang.ClassCastException
com.imagepicker.testing.media.ImageConfigTest > testOnImmutable FAILED
java.lang.ClassCastException
com.imagepicker.testing.media.ImageConfigTest > testParsingOptions FAILED
java.lang.ClassCastException
com.imagepicker.testing.media.ImageConfigTest > testUseOriginal FAILED
java.lang.ClassCastException
com.imagepicker.testing.media.ImageConfigTest > testGetActualFile FAILED
java.lang.ClassCastException
5 tests completed, 5 failed

Downgrading open-jdk version 11 to 8 fixes my issue.

This still persist on version 1.1.0, any news?

with:
buildToolsVersion = "28.0.3"
minSdkVersion = 21
compileSdkVersion = 28
targetSdkVersion = 28

and
"react-native": "0.60.4"

I am getting this error on all versions of react-native-image-picker that I have tried so far:
1.1.0
1.0.1
0.28.0

I am going to have to remove this package from my project until this issue is fixed.
Is there a way to disable the tests so I can get it to at least compile?

Same issue here:
https://github.com/react-native-community/react-native-image-picker/issues/1121

I don't know it will help others or not,
I'm facing this too, I'm just run cd android; ./gradlew clean; cd ..; npx react-native run-android
my build is successful again

To summarise, follow instructions here to downgrade Java from v11 to v8 (v1.8*) fixed it for me:
https://stackoverflow.com/a/24657630/2280866
as @dsgithub2018 suggested above, then ./gradlew clean in the android folder as @rynvva suggests

This still seems to be an issue with 2.3.4 and Java 11.0.2.
I worked around this by disabling tests for this library:
project(':react-native-image-picker').tasks.withType(Test) {
    enabled = false
}

Where do you do this?

Walkaround! I fixed this issue, use my fork instead by adding this line to package.json.

"react-native-image-picker": "git+https://github.com/martin-przystalka/react-native-image-picker.git#develop"

cc @sadikyalcin

Where do you do this?

at the bottom of your projects android/build.gradle. No need to for a fork.

Walkaround! I fixed this issue, use my fork instead by adding this line to package.json.

"react-native-image-picker": "git+https://github.com/martin-przystalka/react-native-image-picker.git#develop"

cc @sadikyalcin

It works well !!!
Thanks a million.


When destroying application sends onAssetChanged and Data is null

https://github.com/playcanvas/engine/blob/d3d84301bf9463f0b3f606f238514381a2601713/src/framework/components/component.js#L48

Hi @lukasharing - would you have any repro project for this? That would make it a lot easier to fix. Thanks!

I also noticed that you are on engine 1.46.5. Is this issue reproducible on a more recent version of the engine?

It is reproducible on the '1.51.5' version

The destroy function of an animation should first remove every emitter and then unload the asset

The lines:
https://github.com/playcanvas/engine/blob/d3d84301bf9463f0b3f606f238514381a2601713/src/framework/components/animation/component.js#L283-L289
The fix:
if ((attribute === 'resource' || attribute === 'resources') && this.data) {

Would you have a repro project we could investigate this on?

You could try the game here:
https://playcanvas.com/project/666521/overview/color-bee

Just had a look at the project posted and cannot reproduce the issue when calling pc.app.destroy() in the console

I manage to fix it by checking this:
if ((attribute === 'resource' || attribute === 'resources') && this.data) {

I can reproduce it by playing on our emulator

Sorry, I don't understand how you've fixed it on the project side?

https://github.com/playcanvas/engine/blob/d3d84301bf9463f0b3f606f238514381a2601713/src/framework/components/animation/component.js#L283-L284
Yes i have override this method on my side :
window.pc.AnimationComponent.prototype.onAssetChanged = function(asset, attribute, newValue, oldValue) {
        if ((attribute === 'resource' || attribute === 'resources') && this.data) { /// now we check if this.data exists


fix: Add proper support for Nvidia

You have to use ublue-os-nvidia kickstarter file, otherwise it won't work.
Offtopic:

I'm not someone who is interested in i3 type of workflow, which this image delivers, but I really like to look at other projects which base on startingpoint.  It is a really good work & I will help occasionally when I notice things like I did here.

Thank you.

Much appreciated, thanks! I don't have Nvidia hardware so I never tested the output of those images.


Implement partial DDI-CDI model.

Need a python implementation. #3
Need a protocol buffer implementation. #4
The implementation will cover the class dataset.

Partial implementation complete, further development done on other issues.


Adjust reminder event validation

Motivation and Context

Currently reminder events that are in the past can not be updated,
which is a valid validation, but this check should only apply if the
collection exercise has been executed. This PR adds a check for CE
state before validating the reminder event has passed. Allowing a user
to update reminder events that have passed but not actually created any
notification files/emails.
What has changed

Only validate existing reminder event hasn't passed if collection exercise has been executed.
How to test?

Ran unit tests, integration tests and acceptance tests.
Links

 Add events validation for creating and updating events


query fix for issue 42

Overview/Summary
Update AzFw DDoS protection query to address issue #42  https://azure.github.io/Azure-Proactive-Resiliency-Library-v2/azure-resources/Network/azureFirewalls/#configure-ddos-protection-on-the-azure-firewall-vnet
Related Issues/Work Items
#42
This PR fixes/adds/changes/removes

Update query 1b2dbf4a-8a0b-5e4b-8f4e-3f758188910d

Breaking Changes

None

As part of this pull request I have

[ ] Read the Contribution Guide and ensured this PR is compliant with the guide
[ ] Checked for duplicate Pull Requests
[ ] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[ ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Ensured PR tests are passing
[ ] Performed testing and provided evidence (e.g. screenshot of output) for any changes associated to ARG queries
[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)


ParallelRange with Sliced Matrix

If you slice a matrix in the ParallelRange matmul does not detect the change in dimension and fails:
res = af.constant(0,20,20,1000 )
left = af.constant(0, 20, 40)
right = af.constant(0, 40,20, 1000)

af.matmul(left, right[:,:,0]) # this works
for ii in af.ParallelRange(10):
            res[:,:,ii] = af.matmul(left, right[:,:,ii]) #will fail

@georgh This needs https://github.com/arrayfire/arrayfire/pull/1898 to be merged.
But if that is merged, you can simply do res = af.matmul(left, right)

Are you sure?
left has a fixed size here, for me that fails (even with the patch with)
Expected: lDims.ndims() == rDims.ndims()
But even if you add this to matmul, the problem is seems not matmul but ParallelRange.
Another example: I would like to add to the diagonal part of all the matrices in my batch
Something like
for ii in af.ParallelRange(10):
            for jj in af.ParallelRange(10):
               for kk in af.ParallelRange(10):
                   res[ii,jj,kk] +=tmp[kk]

would be cool

@georgh You can't nest ParallelRange. But don't need ParallelRange for this. You can do the following:
tmp = af.moddims(tmp, 1, 1, tmp.shape[0])
res += af.tile(res.shape[0], res.shape[1]) # tile does not allocate additional memory

Ok its really nice to know that tile doesn't need more memory :)
But still, shouldn't
for kk in af.ParallelRange(10):
                   res[:,:,kk] += tmp[kk]

work?

What are the sizes of res and tmp ?

Sorry my bad, beside the first case ParallelRange works fine.
But I still don't think the first case is a problem with matmul. Full example:
import arrayfire as af
af.set_backend('cuda')

res   = af.constant(0, 20,20,10)
right = af.constant(0, 20,20,10)
fixed = af.constant(0, 20,20)

af.matmul(fixed, right[:,:,0]) # this works
for ii in af.ParallelRange(10):
      res[:,:,ii] = af.matmul(fixed, right[:,:,ii])
#this does not work even though it is only a multiplication of two matrices with the same size

I have #1898 merged in my build.
Error:
Traceback (most recent call last):
  File "simple.py", line 10, in <module>
    res[:,:,ii] = af.matmul(fixed, right[:,:,ii])
  File "/home/ghiero/anaconda3/envs/intel/lib/python3.6/site-packages/arrayfire/blas.py", line 57, in matmul
    lhs_opts.value, rhs_opts.value))
  File "/home/ghiero/anaconda3/envs/intel/lib/python3.6/site-packages/arrayfire/util.py", line 79, in safe_call
    raise RuntimeError(to_str(err_str))
RuntimeError: In function af_err af_matmul(void**, af_array, af_array, af_mat_prop, af_mat_prop)
In file src/api/c/blas.cpp:131
Invalid dimension for argument 1
Expected: lDims.ndims() == rDims.ndims()

Or even simpler:
import arrayfire as af
af.set_backend('cuda')

res  = af.constant(0, 20,20,10)
xx = af.constant(0, 20,20)
yy = af.constant(0, 20,20)

for ii in af.ParallelRange(10):
      res[:,:,ii] = af.matmul(xx, yy)
s = af.sum(res)
print("work finished, got {}".format(s))


Add owners of each dandisets as contributors

This is a one-time fix operation that depends on #425 being solved first.
The fix should, for each dandiset, add any owners as contributors if they are not already present in the contributor list.

is this issue resolved?

I don't think so: look at this dandiset for example https://gui.dandiarchive.org/#/dandiset/000050.
New dandisets do have the creator added as a contributor, and I don't think adding an owner to a dandiset adds them as a contributor (nor am I sure that we want that to happen--@satra?).
This issue is not really about those policies though--this has to do with adding owners as contributors to current draft dandisets. We need some planning over exactly what should be done, and then the work will be done as a maintenance script on the Django side of things (at least, that's my understanding).
@dchiquito, I'm assigning this issue to you, but as I mentioned it still requires a bit of planning before we can really do anything.

i think new dandisets add the owner/creator as contributor + contactperson and i think that is correct.
regarding older dandisets, we should do the same for all dandisets missing a contributor.

Do we add all owners as contributors? Which one do we mark as the contact?

for the pass through old dataset, i would say add all and make all contactPerson. they can change it if they want. this should only be done for those dandisets that have no contributors.


MAINT: improve special.gammainc in the a ~ x regime.

Before gammainc would hang in the a ~ x regime because the methods
used to compute it converged very slowly. This adds an asymptotic
series tailored for that region.
It also adds a folder _precompute to special which contains the code
used to precompute the coefficients of the asymptotic series as well
as tests to verify that these coefficients are correct. The idea is to
ensure reproducibility and aid debugging.

If this requires linking both fortran and c++ code to the same shared library, we may be in for problems because distutils doesn't support that --- currently there's a workaround via passing function pointers via Cython.

IOW, it may fail compilation on Windows, which probably should be tested with mingwpy or ifort :/

Shoot. Is there an example of the workaround in special? Unfortunately testing on Windows is a little trickier than virtualizing since I don't own the OS...

The current workaround is to have separate _ufuncs and _ufuncs_cxx Cython modules, and move function pointer from the latter to the former via Cython export API. IOW, they're available in _ufuncs.pyx with names scipy.special._ufuncs_cxx._export_faddeeva_dawsn etc., but there's no existing plumbing to route them back to C. It probably is possible to export these function pointers back to C.
Now, it may be that things work out with Windows without the workarounds now that we're not using g77 any more. But it requires Windows installation / VM to check plus some digging in order to find out how to setup the mingw toolchains. I have a VM, but it would take some time to get it done.

If you don't need to call the specfun fortran code, you can maybe put this code to _ufuncs_cxx, just adding the necessary files to ufuncs_cxx_src in setup.py...

Embarrassed to admit it, but I missed that there's an implementation of erfc in Cephes (in fact, it's the one that gets used in the ufunc for real arguments). Thought everything was using Faddeeva... The upshot is that there's no mixing of C++ and C now.

Thanks, merged in 6b10f5af6a with some plumbing fixes, cf https://github.com/scipy/scipy/compare/08173bc066...6b10f5af6aca


Update to official release of the Settings System

I have been working on a massive overhaul to the Settings.lua script over the last week, which makes it much much more flexible and easy-to-use, while at the same time being more powerful. ModernGadgets will convert to this release of the system once it is finished, and this will come with a few key changes:

A complete rewrite of the checkboxes and radio buttons
Deprecation of RainRGB in favor of a custom Rainmeter-based color picker that I designed
Solution for massive settings skin being too tall for 720p screens

This issue will be updated with a to-do once more concrete plans are laid out.

This will be finished with the next beta.


LR Scheduler doesn't work

Information

Model I am using (ListenAttendSpell, Transformer, Conformer ...): All
LR Scheduler doesn't work (all schedulers)

To reproduce
Skip
Expected behavior

https://github.com/PyTorchLightning/pytorch-lightning/discussions/8042

This is pytorch-lightning bug. https://github.com/PyTorchLightning/pytorch-lightning/issues/8057

Hey @sooftware,
This was a mis-use of Pytorch Lightning.
See the correct way to setup your schedulers,
https://github.com/PyTorchLightning/pytorch-lightning/issues/8057#issuecomment-865058609
Best,
T.C

@tchaton Thanks, I'll try!

Fix


[Bug]: Maps widget currently does not handle string values and need to be typecast to number

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When Maps widget is bound to a selected row of a table widget that contains latitude and longitude values in string format, then Maps widget fails to derive a location pin.

Workaround is to typecast as number as follows

Steps To Reproduce

Go to app at link https://release.app.appsmith.com/applications/620a06c4a68a2619dc8d0caa/pages/620a06c4a68a2619dc8d0cac/edit
Observe that since latitude and longitude values are in string, the first map widget does not display a pin, whereas the second map widget that's been typecast displays location pin appropriately.

Environment
Production
Version
Cloud

@dilippitchika I don't think this should be fixed,
The Structure of the marker is this  Array<{ "lat": "number Min: -90 Max: 90 Required", "long": "number Min: -180 Max: 180 Required", "title": "string", "color": "string" }>
A number is required, best that can happen would be to throw the validation error

@Tooluloope @dilippitchika I think we should assist the user by trying to typecast these strings into integers where ever possible. Small features like these should be helpful to our users.
Do you see any issues with implementing this?

We do that to some extent in a few widgets, for example chart widget, but don't do it everywhere across widgets. We can add it here, but we also need to think about other widgets.

@riodeuno @dilippitchika This can be really tricky. One of the reasons we have issues with the defaultValue for select widget was that we were trying to help the users in typecasting.
I feel it's better to be predictable, So the users know that this expects a Number and not a Stringified Number.
I think it's best to show adequate messages in the evaluation popup so the user is clear of what needs to be done.

@Tooluloope @dilippitchika I agree with both your points.

@Tooluloope If we have a technical hurdle in supporting such automatic typecasting, let's figure this out by laying out the RCA
@dilippitchika Agreed. From a DX perspective, it makes sense to support expected scenarios like these, and is something we should include in all scenarios if it exists in one scenario.


Flakey Active Tab Slider Styling

Bug type
Component
Component name
MudTab
What happened?
The Tab slider randomly does align correctly with the tab. The issue is flakey, as sometimes the slider will align correctly.

However, after some random number of initializations the slider stops aligning correctly

This alignment then gets worse on successive initializations.

Then will go back to being properly aligned.

Additionally, when the alignment is working correctly the active slider first renders the full width of the tab.

But when it does not work, it first renders less than the full width of the tab.
And the misalignment occurs on all tabs, and gets successively worse for each tab (going left to right).
Expected behavior
The slider show remain aligned.

Reproduction link
https://try.mudblazor.com/snippet/nZA9C8IwEIZ3_4ThdhUcnNqC1I5Fkezlaq5YCElJUr9-vWltilOGZsrd8_LccVVVYqu2Bj_arJOyFxxrywpJD3StVinsgV11rwSJFJzpCdix6-S7aBq6Oct1rpXzBjIBX1CRzCVam0KHmwNkK-bf5B4p4_RyKfiSnRVNgTnkWTZYSbkBJ7vQ_Hl2f6Komj91TO3xcvXdUHTvMbBYf2ot1pIEzL_puJGJIRkfGkqbfQE
Reproduction steps

Load rows into a mud table.
A Mud Dialog is opened containing a tab panel.
The issue is random, after many successive loads and closes of the dialog the alignment stops work. Then will start working again. All random and flakey

Relevant log output
No response
Version (bug)
6.0.7
Version (working)
No response
What browsers are you seeing the problem on?
Chrome, Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Hello!
I have exactly the same issue with Tab Slider that.
I am working on version 6.0.11.


fix(cosmetic): Fix Datasource Modal Out Of Box

SUMMARY

remove 'overflow: visible' from DatasourceModal component, to fit with the height limit, and show the scrollbar.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before

After

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[x] Has associated issue: Fixed #20236
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks for the contribution! Actually this css snippet can be found in the context of this issue, so we probably can't remove it easily, but if we verify that the issue is working correctly, I think it's okay to remove it. I think the problem is probably related to this issue. cc @rusackas

Thanks for the contribution! Actually this css snippet can be found in the context of this issue, so we probably can't remove it easily, but if we verify that the issue is working correctly, I think it's okay to remove it. I think the problem is probably related to this issue, which is about maxHeight of the modal. cc @rusackas

Thanks for reviewing.
I think other modals may facing the same issue(when actual height is more than max height).

FWIW, when I run this locally, I'm not seeing the issue that @AAfghahi was tackling in his earlier PR, but I'm spinning up an ephemeral environment, and pinging him for a review, just in case :)

/testenv up

Looked at the other two modals that allow overflow, and they don't seem so problematic. I think this fix is the better way forward than the revert, but I'll let it simmer for a moment for discussion.

Thanks for fixing this. The Save Chart modal has overflow visible, but looks OK since it's short. Same with the Dashboard's Refresh Interval Modal. We can do this same fix on either, but it makes matters worse on the Refresh Interval modal. If you do this fix on the Save Chart modal, it works fine, but looks a bit goofy as the bottom Select menu goes upward to cover the Input above it. I'll leave them both alone, but I'm open to ideas/consideration.


W-11064555 - Fixing Import Selected DI button to work with batches

Previously the new 'Import Selected Data Import Records' button would throw an error if a Data Import record in a batch was processed.  This was because the query being used did not retrieve the batch number field from the linked Data Import Batch record.
This commit:

Adds the missing field to the query used for the import selected di button
Modifies unit test so that it covers Data Import records in a batch
fixes a small unrelated typo found in the Notification Recipient Email field label on the Opportunity.

Critical Changes
Changes
Issues Closed

Known Issue: NPSP (Summer 22) - Utilizing "Import Selected Data Import Records" button to Import Data Import Records results in SOQL Error and Failure

Community Ideas Delivered
Features Intended for Future Release
Features for Elevate Customers
New Metadata
Deleted Metadata

@callen-sfdo let me know when you've updated the code to use the custom label on this one and I'll re-review

https://github.com/SalesforceFoundation/NPSP/pull/7040 created in the place of this PR.


Avoid depending on a global configuration file

I can see why you would want to supply additional paths for .db files, however you limit the use of your tool by having these in a global config file. These should be supplied either as environment variables or additional CLI options at runtime.

The use case I'm working with is this:
One user has one FoundryVTT installation and one FoundryVTT data directory. In this directory they have one more more modules they use to share compendiums between worlds. They understand basic CLI tools like "mv" and "ls" but not much more. They want to quickly move around files and folder in their FoundryVTT data directory.
Please provide a use case where global environmental variables would be better than a config file. I might consider implementing global environment variables support with config file configuration as a fallback.
As for supplying them with CLI arguments: The goal is to make the program feel like mv and quickly move around files and folders. Supplying the additional paths at runtime with additional CLI arguments would make the usage clumsy in my opinion. The same thing as above applies:
Please provide a use case where CLI arguments would be better than a config file. I might consider implementing it.

One user has one FoundryVTT installation and one FoundryVTT data directory. In this directory they have one more more modules they use to share compendiums between worlds. They understand basic CLI tools like "mv" and "ls" but not much more. They want to quickly move around files and folder in their FoundryVTT data directory.

This is precisely the use case where my suggestion is applicable. The issues are many:

The config file must be created externally and the tool cannot be used until the config file is created; this is a UX problem.
It is configured to live in a privileged location (/etc/). This requires root to create, and root to update, and the location contains many other files that if incorrectly modified could damage the OS itself.

If it is your intention to memoize the paths to the .db files then a config file does the job well. In principle, I am not opposed to a config file — this is the standard way of saving state for many utilities — however I would suggest the following:

Have the config file in the user directory so that privileged access is not required; i.e. ~/.local/fvttmv.conf.
Create the config file with default values if it doesn't already exist at runtime.

If this involves prompting the user to say where the paths are then prompt on first use.

Provide the means to override what is specified in the config file at runtime, for example by checking environment variables, or an optional command line flag. The purpose of this is if you want to make a once-off change to your environment, for example if you made a copy of your world and wanted to try running fvttmv on the copy without risking damage to the original and needed to temporarily update the config to do that. In this event it is less effort to use a CLI flag than to change the variable in config and change it back afterward.

If the config file does not exist and an override is supplied (either args or env) then use that value when creating the config file instead of prompting or using the default.

The result is a guided experience rather than the user having to backtrack and debug what has gone wrong and generally make your tool more predictable: i.e a command that is run on one system will produce the same result when run on another.

Thanks for the tips, I know they're meant well. I don't have a lot of time on my hand right now. I might implement some of your suggestions in the future, but probably not in the coming weeks. For now I'll put them on my list of possible improvements.


Pass data between content view when calling openContentViewControllerForMenu

In one of my content view (1st view), I have a button to switch to another content view (2nd view), how do I pass data to the 2nd view when calling openContentViewControllerForMenu. Thanks a lot.

Implement - (void)prepareForSegue:sender: method in your left/right table view controller.
- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender
{
    if ([segue.identifier isEqualToString:@"someContentSegue"])
    {
        MyViewController *vc = [segue.destinationViewController viewControllers][0];
        //pass data to vc
    }
}

Thanks for the prompt response. So I have to pass the data from 1st content view to the left menu table view controller first and then pass along to the 2nd view controller?

Yes.
And for the 2nd option you can pass your data through shared instances.

Got it thanks. Btw, I swipe right to reveal the left menu and I can tap on anywhere in the displayed content view to close the menu, how can I configure so that I swipe left to close the menu too?


site_ids for wake-word detection on mqtt base don't work

Hi,
I am not sure in which repository to post this issue, because I could not find, the place where it is actually handled.
my setup:
base .. doing all the work (including wake word detection)

and several raspberry satellites .. doing only audio in and out, the rest is connected via mqtt

the wake word detection (using snowboy on the base) only works for one satellite, if I add exactly that one name into the comma-separated list like this:

If i leave it blank or add multiple satellite names, the wake-word detection is no longer working for any of them. also enabling or disabling wake-word detection on the satellites, seems not to make any difference in that behavior.
Also I am not yet sure what the setting for "Output siteId" does in audio recording - for those satellites it's currently empty.
version 2.5.8
mqtt broker: rhasspy
I hope you can help me - best regards, and thanks again for that great piece of software!

Several strange things here.
Your base should be mqtt external as all your satellites, with base IP set
Dialogue management should be disable on satellites
Better ask on https://community.rhasspy.org/ instead of reporting issue/bug ?

Base uses rhasspy's internal mqqt broker and satellites are connected to that using the external setting. This part this part does not make problems.
I think i tried disabling dialog managent before, but that didn't really make a difference.
As the wrong behaviour was easy to recreat i assumed it is a bug, not question.

I though I fixed that, but this problem still persists.

I though I fixed that, but this problem still persists.

Hi @h4de5 , can you confirm this fixed or still an issue? Seems to be a while ago

are you cleaning up :)
I do not have a working setup at the moment, but the last time I checked, it did work for multiple satellites sending wake words.

Yup, cleaning up a bit. To much old stuff. Thanks for closing


feat: add support for notation inspect

duplicate of notation #528
squashed changes to fix commit author and signature.

Codecov Report

Merging #556 (00896e4) into main (54b42cb) will decrease coverage by 1.98%.
The diff coverage is 21.16%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #556      +/-   ##
==========================================
- Coverage   36.34%   34.36%   -1.98%
==========================================
  Files          30       32       +2
  Lines        1607     1848     +241
==========================================
+ Hits          584      635      +51
- Misses       1002     1192     +190
  Partials       21       21

Impacted Files
Coverage Δ

cmd/notation/main.go
0.00% <0.00%> (ø)

internal/envelope/envelope.go
25.92% <0.00%> (-74.08%)
:arrow_down:

cmd/notation/inspect.go
14.21% <14.21%> (ø)

internal/tree/tree.go
100.00% <100.00%> (ø)

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

Closing this as #556 is merged


fix: should probably use irys gateway instead of arweave

Irys uploader returns arweave URLs by default which works in mainnet I guess, but not if the uploader is configured with devnet provider.

Thank you very much!

Somebody merge this please!


Add 'tier created' as a webhook option

I'm working with a Collective that is mainly fundraising through a series of crowdfunding campaigns created as tiers with goals, and this community uses Slack heavily. It would be really great if tier created and goal reached were webhook options.

For this one, we don't yet have an activity that would allow us to use it with a webhook.
We could introduce a new collective.tier.created activity for that.

I think this need is now better solved through the activity log feature

Well, since we don't yet have an activity for it new tiers created are not currently surfaced in the activity log.


Attempt to fix #1810

rclone doesn't support 2 SSH key exchange algorithms diffie-hellman-group-exchange-sha256 and diffie-hellman-group-exchange-sha1 due to the upstream package crypto didn't support it.
The upstream lib crypto has recently added support for those algorithms in this commit
However, since diffie-hellman-group-exchange-sha256 and diffie-hellman-group-exchange-sha1 are not recommended, see this proposal. They need to be opt-in explicitly by the client, in this case, rclone.
This PR attemps to opt-in diffie-hellman-group-exchange-sha256 and diffie-hellman-group-exchange-sha1 when the flag --sftp-use-insecure-cipher is set to true.

[x] I have read the contribution guidelines.
[ ] I have added tests for all changes in this PR if appropriate.
[ ] I have added documentation for the changes if appropriate.

[ ] I'm done, this Pull Request is ready for review :-)

I have no idea why the test failed on make racequicktest.  I have tried to run this command on my Mac, it worked.
The error reported is in the file pool_test, which I think has no connection with the change I make here. so would a rerun solve the test issue? @ncw

The build is fine but you may also wanna update the docs and the message here:
https://github.com/ncw/rclone/blob/5433021e8b0d517c134da6a81d51c787403db1ac/backend/sftp/sftp.go#L89

@buengese I have updated the doc as you suggested. Let me know if anything else needs to be done

Yeah the docs here need to be updated too.
https://github.com/ncw/rclone/blob/master/docs/content/sftp.md#--sftp-use-insecure-cipher

@buengese The doc file is now updated as well : )


Add Click event to Chart body

Dears,
how i can add URL to chart body??
Thanks,

To add a url to individual points on the chart, you can include the urls in the data array. For instance:
$data = array(
    array('x' => 8, 'y' => 67, 'url' => 'http://url.com/1'),
    array('x' => 53, 'y' => 09, 'url' => 'http://url.com/2'),
);

and in your series configuration:
...
"series" => array(array(
    "data" => $data,
    "cursor" => "pointer",
    "point" => array(
        "events" => array(
            "click" => "js:function(){ location.href = this.options.url; }"
        )
    )
))
...

Here's the official Highcharts documentation for that, with a working example.

Thanks, but i need to add link to hole chart
Not series only is there anyway?
Sent from Samsung Mobile
Milo Schuman<EMAIL_ADDRESS>wrote:
To add a url to individual points on the chart, you can include the urls in the data array. For instance:
$data = array(
array('x' => 8, 'y' => 67, 'url' => 'http://url.com/1'),
array('x' => 53, 'y' => 09, 'url' => 'http://url.com/2'),
);
and in your series configuration:
...
"series" => array(array(
"data" => $data,
"cursor" => "pointer",
"point" => array(
"events" => array(
"click" => "js:function(){ location.href = this.options.url; }"
)
)
))
...
Here's the official Highcharts documentationhttp://api.highcharts.com/highcharts#plotOptions.series.point.events.click for that, with a working example.

Reply to this email directly or view it on GitHubhttps://github.com/miloschuman/yii-highcharts/issues/15#issuecomment-119716841.

According to this StackOverflow answer, the chart itself also has a click event.
http://api.highcharts.com/highcharts#chart.events.click
So:
...
"chart" => array(
    "events" => array(
        "click" => "js:function(){ location.href = 'http://moo.com'; }"
    )
)
...

I tried it but I get JavaScript error after click on chart:
<EMAIL_ADDRESS>Code screenshot
<EMAIL_ADDRESS>From: Milo Schuman<EMAIL_ADDRESS>Sent: Wednesday, July 08, 2015 11:27 PM
To: miloschuman/yii-highcharts
Cc: Ahmad Abujoudeh
Subject: Re: [yii-highcharts] Add Click event to Chart body (#15)
According to this StackOverflow answerhttp://stackoverflow.com/a/16792514, the chart itself also has a click event.
http://api.highcharts.com/highcharts#chart.events.click
So:
...
"chart" => array(
"events" => array(

    "click" => "js:function(){ location.href = 'http://moo.com'; }"

)

)
...
—
Reply to this email directly or view it on GitHubhttps://github.com/miloschuman/yii-highcharts/issues/15#issuecomment-119721398.

@ajoudeh, I can't help you without more information. The code above works without error for me. If Highcharts is throwing an error, they normally include a link in their error message, which you can follow for more info and troubleshooting. Let me know if you are unable to find a solution. I am on vacation, so it may take me a while to respond. Good luck.

@ajoudeh, sorry, I didn't notice you attempted to attach a screenshot, but it failed to attach. You may have to reply on GitHub directly.

I tried it but I get JavaScript error after click on chart:

Code screenshot

I am still unable to reproduce the error. It looks like there is an error loading your debug toolbar, and perhaps that is causing a ripple effect on subsequent jQuery events. Try fixing the debug toolbar error first (maybe by clearing your browser cookies or even disabling the debug toolbar) and see if you still get the error on click.

Hello,
i'm trying to insert a horizontal plotline in my chart in the value 0.42, when my range in yAxis is[-1.2,1.6] , with the following code:
` 'yAxis' => array(
'plotLines' => array(
'color' => '#FF0000',
'width' => 2,
'value' => 0.42
),
'title' => array(
'text' => 'Results',
),
'allowDecimals' => true,
'min' => -1.2,
'max' => 1.6,
            ),`

So when i put the plotline code, the chart shows only the axes and the line doesn't show at all.
However when i remove only the code about the plotline everything works just fine.
Is there a chance to let me know what am i doing wrong and how could i solve this problem?
Thank you  in advance,
Rania


consider adding presage keyboards to base install

Today in the meeting ljo mentioned something about him rewriting functionality we need from pressage. I have no idea what this is all about but i know for sure i need text prediction in Greek. And other languages in general.

Yes, @ljo mentioned it, indeed - I saw from the logs. Its excellent news and it sounds like its proceeding well. I am not sure whether you will need unicode screen handling, the current setup may just work.
But with https://github.com/sailfish-keyboard , you "just" need to get your hands on representative text. Note that we have already few languages covered at OpenRepos

PS: That representative text should be rather large. I think we explained the procedure somewhere (TMO or TJC in addition to READMEs in the projects). Can't recall where though

http://talk.maemo.org/showthread.php?t=100266 this ???

Yes. Look into the thread it also explains how to make new languages possible, as for Greek

Closing it without adding. Its in general considered to be better as bare bones as possible with ports. Users can add keyboard later


Tooling: CI builds and image-publishing

Add this repository to https://drone.pelo.tech/
Configure .drone.yml to run the test and lint steps for new PRs
Configure .drone.yml to publish an image to docker hub when a PR is merged
Add branch protection to master that requires CI to pass before merging a PR

If #8 has already been done, we'll need to update the example .drone.yml stanza in README.md so it shows the actual plugin image.

@josmo I've assigned this issue to you because I don't have sufficient perms to complete it—I can't activate repos on drone.pelo.tech and I can't set branch protections on this repo.

protected master and builds/publish in place

√ Added to our Drone server
√ Image is published as pelotech/drone-helm3
√ Drone config was done in 6452a01
√ Branch protection added


Issue with Neuba config file

Upon running the python demo_attack.py --config_path /configs/neuba_config.json, there's the following error:
File "demo_attack.py", line 61, in <module>
    main(config)
  File "demo_attack.py", line 42, in main
    backdoored_model = attacker.attack(victim, poison_dataset)
  File "/attackers/neuba_attacker.py", line 36, in attack
    victim_config = config["victim"]
TypeError: 'NoneType' object is not subscriptable

Same with configs/por_config.json;
File "demo_attack.py", line 61, in <module>
    main(config)
  File "demo_attack.py", line 42, in main
    backdoored_model = attacker.attack(victim, poison_dataset)
  File "/attackers/por_attacker.py", line 37, in attack
    victim_config = config["victim"]
TypeError: 'NoneType' object is not subscriptable

Thanks for pointing that out. We have fixed the bug.


Many parameters under Server are missing components

I noticed this while writing the issue for #1802 -- many of the Server.XYZ config params are missing at least one server component. For example:
name: Server.ExternalWebUrl
description: |+
  A URL indicating the Pelican web interface and internal web APIs address as it appears externally.

  Port number will be stripped if it's 443, from `Server.WebPort` or directly set through `Server.ExternalWebUrl`.
type: url
default: https://${Server.Hostname}:${Server.WebPort} (for ${Server.WebPort} != 443)
components: ["origin", "director", "registry"]

Caches also have a WebUI affected by this, but the cache as not listed as a component.
If there are Server.XYZ config params that really don't apply to one of our server components, we might need to think about this naming convention.

Another note -- we should mark Server.Modules as hidden because the CLI tool it mentions (pelican serve) is also hidden. It also doesn't affect all components, so that should be corrected for our internal docs.


Terraform 0.12.x compatibility

Currently this repo only works with legacy Terraform 0.11.x versions, it should be updated for Terraform 0.12.x.

The principles should be the same, you'd just need to use a Terraform 0.12 image and config.

I thought the modules in the repo also needed some updates to work with 0.12.x but maybe my memory is failing me and it was just the version pinning that had to be updated...
Regardless, an official training module probably shouldn’t be recommending people install 0.11 anyway.


PluginClean removes Vundle.vim

And then, of course, it's all broken and you have to re-install Vundle.

Same happened here.

@JESii @willthemoor
Do you write the following in .vimrc?
Plugin 'VundleVim/Vundle.vim'

Ah, no. Thank you. I still had this:
Plugin 'gmarik/vundle'

Plugin 'VundleVim/Vundle.vim' helps to avoid to remove self, but the command looks waste.
I have put vundle.vim at ~/.vim/Vundle.vim and added the directory to rtp.

Shoulda closed this long ago; thanks, @syossan27


Without selecting a task, it is not recorded in the active file.

Hello. Thank you for developing the plugin. It's interesting to see what features you are adding—it makes me reflect on my own tracking practices. And special thanks for considering my request about tracking in the active file.
However, it seems that since version 1.1.2, I can't record the log directly in the active file. It works if a task in this file is selected. But if no task is selected, a log entry will appear in the daily file. I can't figure out if I'm doing something wrong or if this is intended behavior.
I'll be glad for your advice

Hey @theotheo, I apologize for the breaking change that was introduced in the recent update, which shifted the tracking context from file-based to task-based. This change is why you now need to select a specific task to log into the active file.
I am looking for a way to address the situation where no task is selected, and I'll let you know as soon as I get a solution

I could work on a fix if it's convenient.

Sure, that would be great!


Event addEventListener not triggered in iOS on lost internet connection

Environment
System:
    OS: macOS 11.2.3
    CPU: (8) x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
    Memory: 48.01 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 14.16.0 - /usr/local/bin/node
    Yarn: 1.22.11 - /usr/local/bin/yarn
    npm: 7.18.1 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.1 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
    Android SDK:
      API Levels: 25, 28, 29, 30
      Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.0, 30.0.2, 30.0.3
      System Images: android-29 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom, android-30 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: 4.1 AI-201.87<IP_ADDRESS>42882
    Xcode: 12.5.1/12E507 - /usr/bin/xcodebuild
  Languages:
    Java: 1.8.0_282 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: ^17.0.2 => 17.0.2
    react-native: ^0.64.2 => 0.64.2
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Platforms
iOS
Versions

iOS: 14.4
react-native-netinfo: 6.0.1
react-native: 0.64.2
react: 17.0.2

Description
Since upgrading react-native-netinfo to version 6.0.1, I am having problems with the event addEventListener in iOS (I am testing this in a real device and not an emulator). Basically, the event is not triggered when the internet connection is lost. I did not detect any problem in Android at the moment. I had to downgrade to the previous version (6.0.0), which is working as expected.
Is anyone having this problem?
Thank you for your help and time!

I suspect this https://github.com/react-native-netinfo/react-native-netinfo/commit/ac0ed659014838d49dfa617000ed22f981dc20a8 is having an impact on the event subscriptions

@leecrossley you link an android patch, original poster mentions iOS. :thinking:

The top file is android, the bottom is ios :) - see RCT_EXPORT_METHOD in ios/RNCNetInfo.m

Oh dang! You are right of course, sorry @leecrossley - in that case that is almost certainly the issue and adding a stub appears incorrect however the event emitter infrastructure in react-native changed completely (as in, they actually changed the inheritance hierarchy and thus what methods are called!) between 0.64 and 0.65 so this probably needs a solid look and some testing before a PR, or perhaps just reverting the iOS side of the changes is correct?
Happy to merge+release anything that appears well-reasoned

Am facing the same issue

Same issue here

Workaround: stay on 6.0.0 or patch out the 2 methods added in the above linked commit via patch-package
PRs happily accepted!

If the semantic release bot is working, there will be a release in a moment with the iOS changes made for react-native 0.65 compatibility reversed, thanks to @xHeinrich 🏆 -
would love to hear status on new release of
a) iOS listener behavior,
b) android listener behavior
c) whether there are listener warning on android or iOS while using react-native 0.65
Thanks!

I just installed the new version, and everything seems to be fine on iOS and Android. At least I did not detect any problem at the moment. Thanks!
However, since I am not using react-native 0.65 because I'm having issues with a package for this react-native version, I can't comment on c).
Thank you once again!

Found the root cause of regression. Needs PR from community (and testing!) to fix, sketch outlined as last comment on #486


DPOP Validation does not seems to check the DPOP Token is signed with the private key matching the access token public key

DPOP Validation does not seems to check the DPOP Token is signed with the private key matching the access token public key.
The DPOP validation does not seems to check that the CNF of the access token matches the CNF of the key used to sign the dpop token.
If true, you would be able to sign a DPOP token with your own private key (not the one from the client) and it will be accepted by the API

Thanks for raising this issue! It should be fixed in the linked PR.

Thanks for fixing it this quickly, it looks good !


Intel C++ compiler and performance library fail to install

A brief description

I am trying to install the Intel Parallel Studio suite of compilers and tools (mostly the C++ compiler, MKL, Intel TBB and Intel Daal). However the setup fails as soon as it is launched using the install.sh script

Expected results

Intel compilers installed under /opt/intel

Actual results (with terminal output if applicable)

Setup fails immediately with the following output:
Initializing, please wait...
Something wrong with activation module. Please go to
http://www.intel.com/software/products/support/ for your support options.
Press "Enter" key to quit

Your Windows build number

Windows 10 Pro version 1607 Build 144393.447

Steps / All commands required to reproduce the error from a brand new installation

Download Intel Parallel Studio for Linux.
Start Install

Strace of the failing command

No traces available. Silent fail.

Required packages and commands to install

See our contributing instructions for assistance.

Hi @aludin -- thanks for reporting this!  It sounds to me like a duplicate of #794 .  Could you take a look at that issue and comment on whether you think it's the same thing?

Hi @aseering , yes indeed this is a duplicate of #794. ifort is also part of Intel Parallel Studio. Please note that I have also raised the issue on Intel forums. It seems however that in order to solve this specific issue, a concerted effort between MS and Intel would be required.

I guess this is also related to https://github.com/Microsoft/BashOnWindows/issues/1373. Hopefully this will be addressed. The test binary built with Intel tools seems to run with more than twice the speed as when built with g++ so it would be good to have support for that.

Marking this as a duplicate. @aludin please close this issue if you feel your concerns are addressed in the other issues mentioned above.

closing this issue and updating issue #794 to include the Intel c/c++ compilers


ARROW-17177: [C++][Docs] Re-Organize the Existing ACERO Streaming Engine Documentation

This PR re-organizes the streaming execution engine documentation.
Created sub-pages and partitioned the exampels so that the reading is more comfortable.
Also splitted the overview section, substrait and examples in different sub-sections.
Note: This effort only re-organized the structure not the content.

cc @lidavidm @pitrou @ianmcook @wjones127 @westonpace
Organized the documentation to an extent. It needs more work on the content.
There is a follow up PR for that. Merely organized the content for readability and easy browsing.

closing this since updated here: https://github.com/apache/arrow/pull/14185


flake8 warning: B031: repeat usage of generator

I'm not complete certain whether this is a true-positive or a false-positive, but it looks like the pre-commit flake8 checks in continuous integration are warning about generator-reuse here.
flake8...................................................................Failed
- hook id: flake8
- exit code: 1

scripts/generate_identifier_pattern.py:33:18: B031 Using the generator returned from `itertools.groupby()` more than once will do nothing on the second usage. Save the result to a list, if the result is needed multiple times.
scripts/generate_identifier_pattern.py:34:15: B031 Using the generator returned from `itertools.groupby()` more than once will do nothing on the second usage. Save the result to a list, if the result is needed multiple times.
scripts/generate_identifier_pattern.py:34:24: B031 Using the generator returned from `itertools.groupby()` more than once will do nothing on the second usage. Save the result to a list, if the result is needed multiple times.

Environment:

Python version: Unknown
Jinja version: 85e5ad85d10eaf119dc0f17f245118c9f78bb864

090a5a9


docs: check for typos and grammar issues all the markdown files

Description
Check for typos/ captalization and grammar issues all the markdown files
There are 9 markdown files.

one README.md file
rest 8 markdown files are in .docs/

I'd like to try this

Hi, Can I work on this @ashuvssut ?

This issue will be always open. Its for anyone who can find flaws in the docs. But, this issue will assigned on first-come, first-serve basis (ref contributing guidelines).

@iatharva you can pickup other .md files

Hi, I can take a look at some unassigned docs pages when iatharva is finished

@dejmedus go through the top issue description. only 2 pages are done. no one is working currently. assign yourself any doc page you want

Sounds good, I will work on contributing.md then

I want to work on CODE-OF-CONDUCT.md

I will look over the project-structure.md

@dejmedus @Pratham-Mishra178 if you are interested do more PRs in this, then there are 3 more markdown files remaining (see description)

@dejmedus @Pratham-Mishra178 if you are interested do more PRs in this, then there are 3 more markdown files remaining (see description)

okay sure, looking into it

I want to work on STYLE-GUIDE.md

I want to work on README.md

@Asmi-1 you can work on 2 PRs at a time.
Take PROJECT-GUIDELINES.md + README.md

I want to work on STYLE-GUIDE.md

@Pratham-Mishra178 please read issue description carefully before starting.

Ok, sure

This issue can be reopened if you want to contribute to this issue


Exclude buergertest.ecocare.center from link test

This PR addresses the issue reported in https://github.com/corona-warn-app/cwa-website/issues/3415#issuecomment-1455528374. It is an alternative resolution to PR https://github.com/corona-warn-app/cwa-website/pull/3417.

It adds the hostname buergertest.ecocare.center to the list noCheckHosts in cypress/e2e/hybrid/check_links.cy.js, since https://buergertest.ecocare.center/#c734 responds with a 404 (Not Found) error.

$ curl -I https://buergertest.ecocare.center/#c734
HTTP/2 404
x-ua-compatible: IE=edge
content-type: text/html; charset=utf-8
date: Tue, 07 Mar 2023 08:20:57 GMT
server: Apache

@brianebeling
This would be a quick, easy and relatively clean fix. I don't think it is worth spending too much time on making a decision on this. There are currently more important issues to be resolved.

Brian-J. Ebeling
This would be a quick, easy and relatively clean fix. I don't think it is worth spending too much time on making a decision on this. There are currently more important issues to be resolved.

Hey Mike, its not that I'm spending a lot of time on this, more so that I don't want to be hasty, because there is a lot to do. I already noticed that leads to bad decisions.
I was also waiting for an answer from other participants, if we can make an exception updating the blogs in this case. The answer is yes, we can. I'd say we merge this PR and wait for the banner indicating EOL. Then we don't need to change the blogs at all, for now. (unless another link gets hijacked again, in this case I advocate to just remove it)


Add Elixir to the example

I'm using this script in internal LINE Thailand products that i take care. So I would like to share this script to this repository. :)

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

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

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

Thanks for your contribution! 💯


Update keyboard-maestro to 7.3.1

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

There is already an update pull request at #26094
recommend this one close.

@pegurnee oh, sure, have not checked before


[BUG] FsHealthService can cause node drops if disk utilization reaches 100%

Describe the bug
FsHealthService fails with java.io.IOException: No space left on device in case the disk utilization on the node reaches 100%. Disk full issue can arise due to multiple reasons such as issues with disk watermark configuration or disk initialisation check not kicking in-time due to  cluster.info.update.interval.  This can end up in the node drop.
2021-08-03T05:02:05,428][ERROR][o.o.m.f.FsHealthService  ] [28c54db4129917b0371c73f05bfb9d1b] health check of [<****>/nodes/0] failed
java.io.IOException: No space left on device
        at java.base/sun.nio.ch.FileDispatcherImpl.write0(Native Method)
        at java.base/sun.nio.ch.FileDispatcherImpl.write(FileDispatcherImpl.java:62)
        at java.base/sun.nio.ch.IOUtil.writeFromNativeBuffer(IOUtil.java:113)
        at java.base/sun.nio.ch.IOUtil.write(IOUtil.java:79)
        at java.base/sun.nio.ch.FileChannelImpl.write(FileChannelImpl.java:280)
        at java.base/java.nio.channels.Channels.writeFullyImpl(Channels.java:74)
        at java.base/java.nio.channels.Channels.writeFully(Channels.java:97)
        at java.base/java.nio.channels.Channels$1.write(Channels.java:172)
        at java.base/java.io.OutputStream.write(OutputStream.java:122)
        at org.opensearch.monitor.fs.FsHealthService$FsHealthMonitor.monitorFSHealth(FsHealthService.java:186)
        at org.opensearch.monitor.fs.FsHealthService$FsHealthMonitor.run(FsHealthService.java:160)
        at org.opensearch.threadpool.Scheduler$ReschedulingRunnable.doRun(Scheduler.java:226)
        at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:765)
        at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at java.base/java.lang.Thread.run(Thread.java:834)

To Reproduce
Steps to reproduce the behavior:

Update cluster setting and remove disk water mark
Run indexing workload and allow disk to get full
Node with 100% utilization is kicked out of the cluster
Above error seen in the data node log

Expected behavior
Exceptions related to disk space full should be consumed by the FsHealthService and node should still remain part of the cluster.

Duplicate of #1040 This is not a bug as this can manifest even with lag detector kicking the node out on disk full

Closing as dup.

@dblock In my opinion this not by design but a bug since the intent FsHealthService  is to detect grey disk issues such as read only file system, which is currently failing and kicking out nodes on disk being full. I have seen this happening even if indexing traffic has stopped, and node is unable to join back the cluster. My take on this is that the exception related to disk space full should be consumed by the FsHealthService and node should not be kicked out of the cluster as suggested above.
Also this is similar to #1040, however #1040 is more specific version of this issue where issue happens post segment merges. I see there are other ways how disk full scenario can occur, such as combination of change in disk water mark setting and shard relocation.

I agree that #1040 is a narrower example of this issue, so we can leave that one open and when fixed we should re-examine this one and make sure that's fixed too. Or do you think we should reopen this one specifically?
@Bukhtawar saying this is not a bug, you two should agree ;)

@dblock I am fine either ways. However the fix proposed for both the issues are different though.
I will let @Bukhtawar comment and decide if this is a concern wrt to FsHealthService behaviour, or I am missing out something here.

Few things to consider

Firstly a node should very rarely run full on disk, segment merges is something we plan on fixing in Lucene to not make an attempt if the disk is low or fail gracefully when merge runs out of disk
If the node is running full for a pro-longed interval, writes would anyways fail, failing the shard on the node(this is by design). The reads can work until the next cluster state update
Once the cluster state update happens and the node is unable to write the new metadata onto the disk, master starts a Lag Detector to kick out lagging nodes in the cluster. This is imperative to ensure that cluster metadata on nodes aren't stale, should it be stale it might turn out poisonous for the entire cluster
The Lag detector today will flip-flop the node causing a loop of joins and disconnects with every cluster state update
The idea of #1040 is to detect a persistent nature of failure where all attempts to relocate the shards off the node have failed and the node cannot cannot operate normally mandating an external intervention.


Navigator.pushReplacementNamed does not update address bar

$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.10.14-pre.15, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[✓] Android Studio (version 3.4)
[✓] IntelliJ IDEA Community Edition (version 2019.2)
[✓] VS Code (version 1.38.1)
[✓] Connected device (4 available)

This commit shows where I'm working the navigator logic: https://github.com/flutter/devtools/pull/1144/commits/2759f5918a9b9d450ae27eafc727d0aa77106870
popAndPushNamed does update the address bar with the new route, but pushReplacementNamed does not.

We are not handling the replacement of routes in the web engine (see code here).
Thanks for filing the issue, I'll have a PR ready soon.


Progress for image upload

When uploading images, there should be a progress shown.

Reload when image fully uploaded.

Progress not possible, but we wait until it's transferred. We should wait (loading-screen), not sure though.


Outdated instructions in multiple learning paths

Contact Details
No response
What happened?
Learning Path 2
Lab 1
Exercise 2
Mitigate Attacks with Microsoft Defender for Endpoint
Task 2
Step 2 references evaluations and tutorials, which have been retired by Microsoft and are now deprecated: https://techcommunity.microsoft.com/t5/public-sector-blog/january-2024-microsoft-365-us-public-sector-roadmap-newsletter/ba-p/4034768 [p1].
Learning Path 3 - Lab 1 - Exercise 2 - Mitigate threats using Microsoft Defender for Cloud
Task 1: Explore Regulatory Compliance
After Step 9, add this step:
Click on Click here, next to Welcome to the new security policies management page. To switch back to the previous experience [p2],
Lab
Lab 02 Exercise 02 Mitigate Attacks with Microsoft Defender for Endpoint
Relevant screenshots
[p1]

[p2]

Do you want to help us? 👏

[ ] Create a Pull Requestt by following this project's Contribution Guide

We know about both of these. And they duplicate issues #235 and #243. Hope to have them resolved next week

Both fixed.


MR SDK updates

add image data structure
add thing API endpoint
update urls for the SDK

Is there a WME GitHub account where this Repo can be moved to?
As a WME client, it looks like WME don't support this code because it's on a single person's account. Also, should this Repo fall within our SLA agreement?

It's not falling within our SLA, also we are planning to move this to WME Github at some point.

Looks good.


Add a Blog and Education Section in Portfolio

There is no blog and Education section in portfolio

No blog and Education Section

Hey, I would like to work on this! Can you let me know what information do you want me to put in blog and education section? Kindly assign me.

Assigined please go ahead

Hey, I want to contribute to this repo, Can u let me know what content needs to add to the blog and ed section? please assign me this task!

Hey, I want to contribute to this repo, Can u let me know what content needs to add to the blog and ed section? please assign me this task!

Assigined

Bro , can u let me know what I have to fill in education and blog section
of this project ?
On Sun, 3 Oct, 2021, 9:11 pm Adithya Pai B, @.***>
wrote:

Assigned #38
https://github.com/adithyapaib/Neomorphic-PortfolioTemplate/issues/38
to @dubeyadarsh https://github.com/dubeyadarsh.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/adithyapaib/Neomorphic-PortfolioTemplate/issues/38#event-5399977357,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR73SZLAG5RV2TCPNGJDGJTUFB2R7ANCNFSM5FGUJUNA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hi, I would like to work on this, can you please assign me this task, and also let me know what needs to be added to this new section.


Unable to use inline CSS after switching to mini-css-extract-plugin

Since recently (https://github.com/zeit/next-plugins/pull/228), it is no longer necessary to add <link rel="stylesheet" href="/_next/static/style.css" /> to _document.js because <link rel="stylesheet" href="/_next/static/css/styles.HASH.chunk.css"/> is appended automatically. This is easy for developers, but comes at a cost for users. In my case, it blocks rendering by ≈1 second and shows the following warning in the lighthouse report:

External stylesheets are blocking the first paint of your page. Consider delivering critical CSS via
<style> tags and deferring non-critical styles.

See original discovery of this performance issue in https://github.com/zeit/next-plugins/issues/11#issuecomment-376125174.
It'd be great to be able to use <style>...inline...</style> instead of <link rel="stylesheet" /> (perhaps, as an opt-in). This approach would be especially useful in a project with styled-components, where pretty much anything except normalize.css and a couple of global CSS rules is added to the body of a page by that library. How could we use inline styles in _document.js?

Same problem here.
Having a way to import styles and inline them would be very helpful to improve perf.
Any update on this?

For anyone still having this issue, this is the solution I came up with.
import Document, { Head, Main, NextScript } from "next/document"
import { readFileSync } from "fs"
import { join } from "path"

class InlineStylesHead extends Head {
  getCssLinks() {
    return this.__getInlineStyles()
  }

  __getInlineStyles() {
    const { assetPrefix, files } = this.context._documentProps
    if (!files || files.length === 0) return null

    return files.filter(file => /\.css$/.test(file)).map(file => (
      <style
        key={file}
        nonce={this.props.nonce}
        data-href={`${assetPrefix}/_next/${file}`}
        dangerouslySetInnerHTML={{
          __html: readFileSync(join(process.cwd(), ".next", file), "utf-8")
        }}
      />
    ))
  }
}

export default class CustomDocument extends Document {
  render() {
    return (
      <html>
        <InlineStylesHead />
        <body>
          <Main />
          <NextScript />
        </body>
      </html>
    )
  }
}

Note the use of data-href prop, that is necessary to avoid fetching the CSS file unecessarily, see webpack-contrib/mini-css-extract-plugin#90

This is a great finding @arianon! Here's a small improvement from my side:
  import Document, { Head, Main, NextScript } from "next/document"
  import { readFileSync } from "fs"
+ import { memoize } from "lodash";
  import { join } from "path"

+ const doGetContent = (file) =>
+   readFileSync(resolve(process.cwd(), ".next", file), "utf8");
+ const getContent =
+   process.env.NODE_ENV === "production" ? memoize(doGetContent) : doGetContent;

  class InlineStylesHead extends Head {
    getCssLinks() {
      return this.__getInlineStyles()
    }

    __getInlineStyles() {
      const { assetPrefix, files } = this.context._documentProps
      if (!files || files.length === 0) return null

      return files.filter(file => /\.css$/.test(file)).map(file => (
        <style
          key={file}
          nonce={this.props.nonce}
          data-href={`${assetPrefix}/_next/${file}`}
          dangerouslySetInnerHTML={{
-           __html: readFileSync(join(process.cwd(), ".next", file), "utf-8")
+           __html: getContent(file)
          }}
        />
      ))
    }
  }

  export default class CustomDocument extends Document {
    render() {
      return (
        <html>
          <InlineStylesHead />
          <body>
            <Main />
            <NextScript />
          </body>
        </html>
      )
    }
  }

By using memoize or any similar wrapper function in production, you significantly reduce the number of times readFileSync gets invoked. CSS files are actually read only during the very first render of _document and their content is kept in memory until the process is stopped. Fewer synchronous calls to the filesystem → less frequent blocks of the main thread → quicker response times! 🎉
One caveat I'm still keeping in mind is related to use of process.cwd(). Its presence means that the app can only be started from the project root and not any other folder. I know that that's an edge case, but it'd be still great to leverage Next's config → distDir somehow. I could not find this value in this.context or elsewhere while inside _document; if anyone knows how to get it, feel free to share!

This is a great solution!
The only issue is when scss gets modified, there is no automatic refresh. The only way is F5. Any idea how to resolve that?

It works for me in v7 https://github.com/badpenguin/nextjs-static-generator-boilerplate
But in v8.1 there is some weird things happening with this.context._documentProps.files  https://github.com/zeit/next.js/issues/7641

For anyone still having this issue, this is the solution I came up with.
import Document, { Head, Main, NextScript } from "next/document"
import { readFileSync } from "fs"
import { join } from "path"

class InlineStylesHead extends Head {
  getCssLinks() {
    return this.__getInlineStyles()
  }

  __getInlineStyles() {
    const { assetPrefix, files } = this.context._documentProps
    if (!files || files.length === 0) return null

    return files.filter(file => /\.css$/.test(file)).map(file => (
      <style
        key={file}
        nonce={this.props.nonce}
        data-href={`${assetPrefix}/_next/${file}`}
        dangerouslySetInnerHTML={{
          __html: readFileSync(join(process.cwd(), ".next", file), "utf-8")
        }}
      />
    ))
  }
}

export default class CustomDocument extends Document {
  render() {
    return (
      <html>
        <InlineStylesHead />
        <body>
          <Main />
          <NextScript />
        </body>
      </html>
    )
  }
}

Note the use of data-href prop, that is necessary to avoid fetching the CSS file unecessarily, see webpack-contrib/mini-css-extract-plugin#90

@arianon This solution works in my local server, but when i try to deploy it to my now server Im getting following error Error: ENOENT: no such file or directory, open '/var/task/_next/static/css/commons.b2e899ec.chunk.css' Do you know a workaround for this or faced a similar situation

@HimanJusta I tried to replicate your error (I don't use Zeit Now) but it works fine. https://himanjusta-issue-238-comment.arianon.now.sh/
Thought that file path in the error does seem weird to me, shouldn't the path be /var/task/.next/static/css/commons.b2e899ec.chunk.css? note the . in .next

@HimanJusta I tried to replicate your error (I don't use Zeit Now) but it works fine. https://himanjusta-issue-238-comment.arianon.now.sh/
Thought that file path in the error does seem weird to me, shouldn't the path be /var/task/.next/static/css/commons.b2e899ec.chunk.css? note the . in .next

Im facing this issue when i deploy to Zeit now, if I just use npm start and see the prod build on localhost it works. Regarding the error in file path I did notice that, but looks like zeit now uploads the next build to _next below is the snapshot of the final folder stucture in now

@HimanJusta same issue for me, too. Have you had any luck?

that won't work if your server load your css files from an external source like s3

A got a notification from github and was surprised to discover that this was my issue 😅
Nearly two years since creating it, I can say that I’ve solved this problem for myself by using styled-normalize. Example:

GlobalStyle.tsx
_document.tsx

Thus, in my recent apps, all styles come from styled-componets and there is not a single css file in the whole folder tree.
That said, I think it's worth keeping the issue open because it is still not possible to inline 'classic' css in Next. Just sharing this update to help folks who can use the same trick as me 🙌

@kachkaev yes, it's built-in, but it doesn't inline it on the page and I ended up here from googling on how to do it :)
I'm using a
import '../styles/all.scss';

in my _app.js and it ends up adding my CSS-file twice (once with preload, assumingly comes from it automatically preloading other routes or something?)
<link rel="preload" href="/_next/static/css/63b012e1eea8af8f9e91.css" as="style"/><link rel="stylesheet" href="/_next/static/css/63b012e1eea8af8f9e91.css"/>

I simply have an old fashioned global SCSS-file - Ideally I would just embed it in a <style>-block on the page - does anyone know if this is possible?

The solution in https://github.com/vercel/next-plugins/issues/238#issuecomment-432211871 still works well today (Next.js 9.5, July 2020).
It gives a warning, though I don't think it'd cause any impact:

Warning: next-head-count is missing. https://err.sh/next.js/next-head-count-missing

On v9.5.2 which released today August 11, 2020. The solution at #238 returns the following error:
Unhandled error during request: TypeError: Cannot destructure property 'assetPrefix' of 'this.context._documentProps' as it is undefined.

You should now get this two variables from this.context to make it work again ✅
const { assetPrefix, files } = this.context

In v9.5.3, this.context does not contain the files key so the solution outlined in #238 doesn't generate embedded styles anymore. It appears the previous files list has moved into the buildManifest.pages key where each page has its own list of files.
This minor change I did seems to fix it for me (YMMV). I'm still getting my feet wet with Next so maybe someone with a little more experience can chime in to see if this is an appropriate fix?
        __getInlineStyles() {
-               const { assetPrefix, files } = this.context;
+               const { assetPrefix, buildManifest } = this.context;

-               if (!files || files.length === 0) return null;
+               if (!buildManifest.pages || buildManifest.pages.length === 0) return null;

-               return files.filter((file) => /\.css$/.test(file)).map((file) => (
+               const pageKeys = [];
+               for (const key of Object.keys(buildManifest.pages)) {
+                       if (buildManifest.pages[key]) {
+                               pageKeys.push(buildManifest.pages[key]);
+                       }
+               }
+               const mergeDedupeFiles = (arr) => {
+                       return [ ...new Set([].concat(...arr)) ];
+               };
+
+               return mergeDedupeFiles(pageKeys).filter((file) => /\.css$/.test(file)).map((file) => (
                        <style
                                key={file}
                                nonce={this.props.nonce}

My updated __getInlineStyles function
	__getInlineStyles() {
		const { assetPrefix, buildManifest } = this.context;

		if (!buildManifest.pages || buildManifest.pages.length === 0) return null;

		const pageKeys = [];
		for (const key of Object.keys(buildManifest.pages)) {
			if (buildManifest.pages[key]) {
				pageKeys.push(buildManifest.pages[key]);
			}
		}
		const mergeDedupeFiles = (arr) => {
			return [ ...new Set([].concat(...arr)) ];
		};

		return mergeDedupeFiles(pageKeys).filter((file) => /\.css$/.test(file)).map((file) => (
			<style
				key={file}
				nonce={this.props.nonce}
				data-href={`${assetPrefix}/_next/${file}`}
				dangerouslySetInnerHTML={{
					__html: fs.readFileSync(path.join(process.cwd(), '.next', file), 'utf-8')
				}}
			/>
		));
    }

Hope this helps someone!

In 9.5.3, files are passed as argument to getCssLinks so no need to go through buildManifest:
getCssLinks({ allFiles }) {
  return allFiles
    .filter((file) => file.endsWith('.css'))
    .map((file) => (
      <style
        key={file}
        nonce={this.props.nonce}
        dangerouslySetInnerHTML={{
          __html: fs.readFileSync(path.join('.next', file), 'utf-8'),
        }}
      />
    ));
}

@jsiddiqui How do I do it in TypeScript?
When I click on Head in VSCode, I get the following:
getCssLinks(files: DocumentFiles): JSX.Element[] | null;

And DocumentFiles is defined as:
declare type DocumentFiles = {
    sharedFiles: readonly string[];
    pageFiles: readonly string[];
    allFiles: readonly string[];
};

Right now, I'm using any like:
getCssLinks({ allFiles }: { allFiles: any }) {
	const { assetPrefix } = this.context
	if (!allFiles || allFiles.length === 0) return null

	return allFiles
		.filter((file: any) => /\.css$/.test(file))
		.map((file: any) => (
			<style
				key={file}
				nonce={this.props.nonce}
				data-href={`${assetPrefix}/_next/${file}`}
				dangerouslySetInnerHTML={{
					__html: fs.readFileSync(path.join(process.cwd(), '.next', file), 'utf-8'),
				}}
			/>
		))
}

But why not export DocumentFiles so I can do the following instead:
getCssLinks({ allFiles }: DocumentFiles) {
  ...
}

@deadcoder0904 Specifying a type here is unnecessary as Typescript is able to infer the correct type due to extending the base class.
For example if you do this:
getCssLinks({ allFiles, test }) {

I get the following Typescript error:
src/pages/_document.tsx:9:3 - error TS2416: Property 'getCssLinks' in type 'InlineStylesHead' is not assignable to the same property in base type 'Head'.
  Type '({ allFiles, test }: { allFiles: any; test: any; }) => any' is not assignable to type '(files: DocumentFiles) => Element[]'.
    Types of parameters '__0' and 'files' are incompatible.
      Property 'test' is missing in type 'DocumentFiles' but required in type '{ allFiles: any; test: any; }'.

Therefore, I think that there is no need to export this.

Hi, thanks for creating an issue. We currently recommend using https://nextjs.org/docs/basic-features/built-in-css-support as zeit/next-css and zeit/next-sass have been deprecated in favor of the built-in support.