Unioned type for function param emits object instead of inline type for type that is based on a typedef

What package is the bug related to?
typedoc-plugin-markdown
Describe the issue
When a function parameter accepts a union type which contains a type that is based on a typedef, the type that was created via typedef emits as object instead of the inline type.
Repro: https://stackblitz.com/edit/vitejs-vite-gkbey1
Steps to Reproduce

Open repro
npm run docs:md
View files in docs-md/functions

Actual Result
Generated markdown for function handleFoo which accepts the union Foo | null contains object for the type of Foo
• **a**: `null` \| `object`

Additional Information

The issue does not occur when generating html files (npm run docs:html)
The issue does not occur in the following situations:

handleFooOnly - Does not contain a union, param a is type Foo
handleFoo2 - Defines a union type FooExplicit separately and uses it for the type parameter for a instead of the inline union
handleFooExplicit - Uses a union type inline but does not use typedef

TypeDoc configuration
typedoc: 0.26.11
typedoc-plugin-markdown: 4.2.10
typescript" 5.6.3
Expected behavior
The type Foo should be generated inline similar to the way handleFooOnly which does not have a union type emits.
• **a**
• **a.a**: `string` = `''`
• **a.b**: `string` = `''`

Thanks for detailed description.
Fix in<EMAIL_ADDRESS>It was a bit tricky to format this in a decent way but currently is produces this output.

https://github.com/typedoc2md/typedoc-plugin-markdown-scratchpad/blob/main/issues/720/docs/md/functions/handleFoo.md
Please note output for parameter listings has changed a bit for readability.

Hi @tgreyuk -
Thanks for the quick response on this, grealty appeciated!
Understood that the formatting gets complicated in this scenario, however I think the current in 4.3.0 is missing a key part of the type by not having a | between the types.
Repro: https://stackblitz.com/edit/vitejs-vite-8b5crt?file=src%2Findex.ts
TypeDoc configuration
typedoc: 0.27.0
typedoc-plugin-markdown: 4.3.0
typescript" 5.7.2
In the case of handleFooExplicit, the generated result is:

However, in the case of handleFoo, the generated result is:

Even thought the types are defined in an identical way with the only difference being their "upstream" definitions:
export const handleFoo = (a: Foo | null) => {};
export const handleFooExplicit = (a: FooExplicit | null) => {};

In the HTML version, the result is:

Do you think it makes more sense and would it be possible to delimit null and {a: string;b: string; } with a | and just put on the same line?

Yes makes sense. I assume example 1 is preferable to example 2?
Example 1:

Example 2:

Thanks @tgreyuk.  Yeah, my preference would be Example 1 as it mirrors what the output is in the FooExplicit scenario and also the HTML equivalent.
A few additional notes:

I realized that the typedoc 0.27.0 repro from my last message wasn't saved so I've updated it to ensure the lastest typedoc, typedoc-plugin-markdown & typescript are referenced in package.json
Another scenario where this same formatting issue arises is in the following - I've added handleFooInline to cover this so that you have a repro

export const handleFooInline = (a: { a: string; b: string } | null) => {};

Beyond the | formatting, the current output emits a.a and a.b which differs from the generated HTML for the same function which does not include the additional definition of a since it's already inlined.  I'm not sure why the extra definition is emitted so possibly there is a good reason it behaves the way it does.  That said, it does differ from what the HTML equivalent generates and in the case where there are multiple complex types in the union, things could get rather confusing I fear.  For example, the following:

export const handleFooInlineMulti = (
  a: { a: string; b: string } | { x: string; y: string } | null
) => {};

Generates the following MD which has two issues:

It does not include {'x': 'string';'y': 'string'; } in the definition of a itself (yellow highlight)
It includes additional definition of each complex type of a but since they are already documented for a itself this results in duplicates (red highlight):

While the HTML equivalent generates:

I believe the {x: string, y:string} should be emitted for a itself here and possibly the additional definition a.a, a.b, a.x & a.y isn't needed in this scenario?

@techfg thanks for the comments - this kind of detailed feedback is invaluable and greatly appreciated.

Another scenario where this same formatting issue arises is in the following - I've added handleFooInline to cover this so that you have a repro

Will review thank you.

Beyond the | formatting, the current output emits a.a and a.b which differs from the generated HTML for the same function which does not include the additional definition of a since it's already inlined.

You are correct, the html theme does not include further inlined declarations unless there are associated doc comments, in which case it does render the detailed declaration.

This is a pattern i have considered implementing too but i found it more deterministic just retain the same behaviour regardless if comments are present. However on reflection there is obviously a case not to render this extended view if there is nothing useful to show (ie associated comments), and aligning with the html theme in this regard is probably the way to go.

That said, it does differ from what the HTML equivalent generates and in the case where there are multiple complex types in the union, things could get rather confusing I fear. For example, the following:

Yes i thought about this too which is partly why the initial implementation was in blocks, however i think it can be implemented in a way that works, and if we are not displaying inline details all the time it becomes less of an issue.

Happy to try to help @tgreyuk, glad your finding the feedback useful :)
Makes sense about where you initially took the approach to always emit vs. only when comments are present.  I do think that trying to mirror the HTML equivalent as much as possible may be the best course but understood if that presents challenges and I don't think it's worth bending over backwards to try to make that happen if its not straightforward enough.
Just want to make sure you saw the comment about the issue where in the following, { x: string; y: string } is not emitted for a (yellow highlighted area):
export const handleFooInlineMulti = (
  a: { a: string; b: string } | { x: string; y: string } | null
) => {};

Thank you!

As discussed - fixes in<EMAIL_ADDRESS>

Looking good in 4.3.1, thank you @tgreyuk!
Only feedback I'd offer is that there is a space between certain areas (green arrows) and no space between other areas (red arrows).  For consistency, possibly add a space where the red arrows are?

spacing updated in<EMAIL_ADDRESS>

Looks great, thanks for the quick response @tgreyuk!


Boundary-value is not tokenized properly

If tests/text_large.txt is given, I think 「その他	名詞,代名詞,一般,,,*,その他,ソノタ,ソノタ」 should be returned, but following tokens are returned.
そ	名詞,特殊,助動詞語幹,,,,そ,ソ,ソ
の	助詞,連体化,,,,,の,ノ,ノ
他	名詞,非自立,副詞可能,,,,他,ホカ,ホカ

😅

This happens only when the input text is larger than inner buffer size and the text has no punctuation (like this example.)
https://github.com/mocobeta/janome/blob/master/tests/text_large.txt
For now, I do not come up with good idea for this issue. Anyone has smart solutions or suggestions?

I wanted to fix this , but for now I have no idea...


Click event won't get fired on iOS 8.

I'm using foundation 5.4.6 which has Fastclick and Modernizr as dependencies already. However, I'm experiencing that the click event isn't fired. So, after I clicked a link it just fire hover state and not navigate to anywhere. Has anybody experiencing any issue like this?

have you solved? how?


Unnecessary new line for assignment.

Add close message to close obsolete PR.
This closes #60

Main reason for this PR is to close the lingering PR

merging this ...


Feature/11 client library for convenience

Client Library and example usage added to Login-Service.
Library gives users the ability to:

Fetch Tokens
Fetch PublicKey (As String or JWKS)
Refresh Tokens
Decode and Verify Access Tokens
Parse Claims

A very Basic Example of utilizing the library has also been implemented with documentation.
Please let me know if there should be any other functionality added.
Closes #11

@dk1844 Made some changes for the above comments. Just need some help on one thing.
what Nexus should we use to publish the client Library too? I would also need credentials to be able to publish to that nexus, I'm assuming?

@dk1844 I've pushed the library to the sonatype nexus and referenced it in the project for the example and that works locally.
However, it's failing build on Github since the github builder seems to use repo1.maven.org instead of the Sonatype nexus. Should we push it there as well? Or do we need to add the Sonatype nexus to our github builder?

@dk1844 Made some changes for the above comments. Just need some help on one thing.
what Nexus should we use to publish the client Library too? I would also need credentials to be able to publish to that nexus, I'm assuming?

@dk1844 I've pushed the library to the sonatype nexus and referenced it in the project for the example and that works locally.
However, it's failing build on Github since the github builder seems to use repo1.maven.org instead of the Sonatype nexus. Should we push it there as well? Or do we need to add the Sonatype nexus to our github builder?
So for now, just using the Library Internally so that it builds

If it is possible, how about just publishing locally to ~/.m2 (sbt publishM2) or ivy (sbt publishLocal) and the artifact would depend on that? If all fails, referencing just the module is okayish, too. At least now we have the distinction between library-producing module and example usage module.

@dk1844 Sorry, tried to hide the comment in order to change the credentials quickly but accidently deleted it. Can you ask the question again about getting the git builder to publish the library?

@dk1844 Just had to do a force push to rollback the commit that had the directory name refactoring in order to put that into it's own branch. This branch is only focused on the client Library now and should be much easier to review now. Client Library should be ready now after removing Decoder class

A bit confused on what's wanted for the JWTDecoder? Previous implementation essentially created a decoder class that automatically fetches the public key and verifies Access Tokens which seems to be what your asking for now @dk1844 ? Unless I'm misunderstanding. Because now the current implementation just points the user to use Nimbus directly which is what I assumed we wanted? Sorry, I must be misunderstanding somewhere.

A bit confused on what's wanted for the JWTDecoder? Previous implementation essentially created a decoder class that automatically fetches the public key and verifies Access Tokens which seems to be what your asking for now @dk1844 ? Unless I'm misunderstanding. Because now the current implementation just points the user to use Nimbus directly which is what I assumed we wanted? Sorry, I must be misunderstanding somewhere.

Currently, the ClaimsParser can pickup claims from JWT and check if JWT is valid in terms of content (expiration and token type). It does not check JWT signature. Checking the signature can be done using NimbusJwtDecoder, but the current implementation shows this in the example app only, not in the client library part. Therefore, it would be on the client to implement this, they could not use this easily from the client library.
The suggestion is: Use NimbusJwtDecoder inside clientLibrary module (perhaps in class ClaimsParser), so that the integrators of the client library can use this class not only to extract claim values and check those values (individually or via verifyDecodedAccessToken), but also to check that the signature is correct.
As far as I can see, it is now done only as an example implementation in the clientexample/Application.

Added JWTDecoderProvider. Gives you easy access to the NimbusDecoder which can be interacted with directly, allowing the user to make full use of the Nimbus Lib

All Comments have been addressed as far as I can see @dk1844 . Please review again when time allows

@dk1844 Hi
made some additions and changes also merged in your PR and worked off of it. Hopefully should be good now.

Hi @dk1844 Merged in the Suggestion PR. Is there anything else required?


Refactor: rename pandastable and things called "data", "metadata" and similar

Our namespace was getting a bit degenerate with many things called some variant of "table", "data", "metadata", or some permutation thereof, causing repeated miscommunications.
So I renamed a lot of stuff, expanding into virgin nameterritory. Hopefully the new names will enable clearer communication as well as improve readability and reduce the cognitive load of absorbing all this for a newcomer.

PandasTable --> TableDataFrame as previously discussed.
TableData --> ComplementaryTableInfo. Bit of a mouthful, not my favourite, but "TableData" just had to go. Objects of this class are attached to dataframes and are "responsible for storing any table information not stored by native dataframes" (docstring) so this seems like a fair enough name.

Also renamed multiple functions, methods, arguments etc. to be consistent with the above changes. Updated docstrings and pdtable_demo notebook as well.

I'd prefer not to use a nondescript name such as Context or Info because... then why have a name at all :-)
For me, the fact that it isn't exposed makes me more likely to accept long, cumbersome, but descriptive names. Because it won't be cumbersome to the user -- but will reduce cognitive burden for the code reader.

Thanks for going through this, JF -- looks like a good solution. My inner monologue on reading the summary was pretty much the same played out by you and Thomas above...


Which better? FP16 OR BF16?

When using torch amp, which one is better?
Thanks.

We train with bf16, idk about fp16.


Hide remove all option on bank fillers

Is your feature request related to a problem? Please describe.
I get fustrated when i am orginising my bank with fillers and i missclick the all option when removing the ones that i dont use
Describe the solution you'd like
An extension in menu entry swapper or bank option that allows removing the option from the client.
Describe alternatives you've considered
alternetivly i've tought that the runelite developers could add a warning when you try to click the all option, i know this one would take alot of more work ive worked with some java before.
Additional context
n/a

It already takes 2 clicks to clear all fillers

yes but i want the clear-all option gone since it is easy to misclick while orginising bank

Duplicate of #6910


global: suffix strip of minter and fetcher name

It might be nice for easier use with https://github.com/inveniosoftware/invenio-records/pull/109
--
WDYT @lnielsen ?

LGTM, and I agree it a shorter name is better. It will however impact Records-REST + possible other modules (e.g. example app/tests where RECORDS_REST_ENDPOINTS is specified). Probably a grep is needed to find all the places.

@lnielsen travis is :white_check_mark:


Is there an efficient way to walk a vector n elements in?

although this is possible:
 vector >> drop(n) >> take(m);

Given that this is an random access container I'd suggest implementing an slice / advance operator that is random-access iterator aware.


🥝 support a list of helm chart repos 🥝

What is this PR About?
Support a List of helm chart repos to create.
This is similar to -stable and -incubator repos in the docs:
https://helm.sh/blog/new-location-stable-incubator-charts
and supports common use cases for deploying charts to different nexus helm chart repos (-dev, -test, -staging)
How do we test this?
Tested using helm template into new namespace
helm template foo . | oc apply -f-

cc: @redhat-cop/day-in-the-life

I don't have merge rights on this repo so someone else will have to click 👍

@eformat ; not sure, error suggests a version has not been bumped and has already been released, but I thought the linter checks for version bumps.


chore(TextArea): rows logic

It will be very useful to have y scrollbar depending not only on rows or maxRows values, but on height of the TextArea.
now it is possible to pass height value via TextAreaProps.className to see y-scrollbar

may be it is extra now.
I did not check


terraform orchestration support

ref #54
is writter in Go https://github.com/hashicorp/terraform

The idea is to make Terraform create the functions?

change github.com/gofn/gofn/iaas/digitalocean by terraform digitalocean provider[1]
Create adapters (github.com/gofn/gofn/iaas/digitalocean) to terraform providers
[1] https://github.com/terraform-providers/terraform-provider-digitalocean

Upgrade issue description

On it.

I mean, we could to this steps via code, but I think that the purpose of using terraform is to remove the direct calls to the provider APIs.

@gtsalles
On terraform used
variable "do_token" {}
variable "pub_key" {}
variable "pvt_key" {}
variable "ssh_fingerprint" {}

provider "digitalocean" {
  token = "${var.do_token}"
}

to define auth on cloud, with that we don't have to worry about creating the key, TerraForm manages it

@avelino yes, how to use the keys is clear to me, what it isn't is how we integrate terraform in the way the process is defined today.
Now, we check for existing ssh keys on DO, and create a new one (when it doesn't exist), then upload it to the account. With terraform we can't make this check for existing keys. We can't assume that we always will have a key in there, neither create a new one for every machine creation.

After some research[1], I came to the conclusion that for now, it'll be too much work to get to a working solution for using terraform as a package.
As answered to me [2] in the terraform mail list, it would be easier to make shell calls in the code, and I don't think that's the more suitable solution for this. I think we can close this for now.
[1] https://github.com/hashicorp/terraform/issues/17904
[2] https://groups.google.com/forum/#msg/terraform-tool/7clPX0J7FmU/OqL1lcuWCwAJ


csharp: remove List/Entry[] allocs from Metadata

Memory analysis shows that a lot of allocations come from Metadata - having Metadata is fine, but that then allocates a List<Entry> each time, which also allocates an Entry[] as soon as the first item is added:

The fact that we have 40k Metadata and 40k List<Metadata.Entry>, but only 10k Metadata.Entry[] tells us that most of those Metadata don't actually have any entries in them. Further analysis shows that most of the time, with default options, there's only a single Metadata.Entry in them - namely "user-agent". So; I propose two things:

don't allocate a List<T> unless we need it
we can special-case the single-item scenario by using an object field
detect the 3 scenarios (null, Metadata.Entry, List<Metadata.Entry>) in the relevant methods and act accordingly

Result:

here you can see 40k of Metadata, 10k of Metadata.Entry, but zero of either List<Metadata.Entry> or Metadata.Entry[]. Success!
Because this touched quite a few lines, I beefed of the metadata tests, too.

An interesting optimization, but I'm wondering if it makes things a bit too complex. Would it make sense to just make the List lazy and set it to null by default, which would avoid many allocations in cases where Metadata is empty (that happens often) and then try to figure out the case where we only have one metadata entry (the user-agent) separately (perhaps the solution would be different here - e.g. caching the whole metadata entry or so)?

Possibly, but I was also noting here that in grpc-dotnet on Kestrel there's also usually a single response headers too ("server", IIRC - perhaps I should find out and add as a well-known-key).
The problem with user agent is that it isn't an easy one to cache unless you have a logical session concept - you could presume that different clients will have different UAs, and doing something simple like a dictionary is a DDOS vector :(


Error with watch mode in Windows 10 ubuntu bash

TypeScript Version:  2.0.0 (@beta)
Trying to use bash in Windows 10 for all my development. When running tsc -w on a Project with a tsconfig.json i get an error. This error exists even on an empty Project.
Expected behavior:
tsc -w should build the Project as normally. tsc works fine
Actual behavior:
an error is thrown:
emilekberg@ALDERAAN:/mnt/e/dev/test$ tsc --init
message TS6071: Successfully created a tsconfig.json file.
emilekberg@ALDERAAN:/mnt/e/dev/test$ tsc --w
fs.js:1431
    throw error;
    ^

Error: watch /mnt/e/dev/test EPERM
    at exports._errnoException (util.js:1012:11)
    at FSWatcher.start (fs.js:1429:19)
    at Object.fs.watch (fs.js:1456:11)
    at Object.watchDirectory (/usr/lib/node_modules/typescript/lib/tsc.js:1475:32)
    at Object.executeCommandLine (/usr/lib/node_modules/typescript/lib/tsc.js:38757:43)
    at Object.<anonymous> (/usr/lib/node_modules/typescript/lib/tsc.js:39121:4)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
emilekberg@ALDERAAN:/mnt/e/dev/test$

This doesn't really look like a TypeScript issue -- we're just calling fs.watch. What happens when you run require('fs').watch('/mnt/e/dev/test') from the node REPL?

You seem to be correct, the same behaviour is met.
emilekberg@ALDERAAN:/mnt/e/dev/test$ sudo nodejs
> require('fs').watch('.')
Error: watch . EPERM
    at exports._errnoException (util.js:1012:11)
    at FSWatcher.start (fs.js:1429:19)
    at Object.fs.watch (fs.js:1456:11)
    at repl:1:15
    at sigintHandlersWrap (vm.js:32:31)
    at sigintHandlersWrap (vm.js:96:12)
    at ContextifyScript.Script.runInContext (vm.js:31:12)
    at REPLServer.defaultEval (repl.js:308:29)
    at bound (domain.js:280:14)
    at REPLServer.runBound [as eval] (domain.js:293:12)

Is this a config issue with what bash can access or? I tried with sudo as well, same result.

Check this uservoice out: https://wpdev.uservoice.com/forums/266908-command-prompt-console-bash-on-ubuntu-on-windo/suggestions/13469097-support-for-filesystem-watchers-like-inotify.


use:*: invalid tag

What
I got a error message, it is
$ exec $SHELL
use:*: invalid tag
use:*: invalid tag
Static loading...

My zplug's configure file is:
# --- valid ---
zplug "tarrasch/zsh-bd"
zplug "zsh-users/zsh-syntax-highlighting", nice:19
zplug "b4b4r07/emoji-cli"
# --- valid ---

# --- invalid ---
zplug "sachin21/81cb1e8a65845bbb8f7c", from:gist, as:command, use:"*", hook-build:"chmod +x *"
zplug "sachin21/eecd241ecf420323fe388de155a02658", from:gist, as:command, use:"*", hook-build:"chmod +x *"
# --- invalid ---
# The above will get a error message.

if ! zplug check --verbose; then
  printf "Install? [y/N]: "
  if read -q; then
    echo; zplug install
  fi
fi

zplug load --verbose > /dev/null

I don't understand why it is invalid.
My environment:

Darwin sachin21dev.local 14.5.0 Darwin Kernel Version 14.5.0: Tue Sep  1 21:23:09 PDT 2015; root:xnu-2782.50.1~1/RELEASE_X86_64 x86_64 i386 MacBookPro10,1 Darwin
https://github.com/sachin21/dotfiles/tree/master/zsh.dot
zsh 5.2 (x86_64-apple-darwin14.5.0)

Thanks for reporting! I'll fix it.

Thanks for your the quick reply! I'm looking for forward to it 😄

Hmm... I can't seem to reproduce this on my Gentoo box running zsh 5.2. Does this happen even after you delete the cache with zplug clear?

yeah I'm already executed zplug clear. but I reproduce this problem.
uhh, sorry I forget to write zplug version.
zplug version is zplug 0.1.9

I see. The old tag name for use is of. Unless you have a reason not to, you should upgrade to v2.

Thank you! it solved with v2.
I didn't know I am using old version. I will this issue.


changes to discussion of "constant" or "singleton"

See the python-ideas list for too much discussion of this :-)
Note that this change should probably be reconciled with the section:
None has its own type
They are a bit repetitious now

This is very computer sciencey compared to the original.  You probably don't want to introduce the word "Singleton".  You certainly don't want to use it with that definition; the document has previously told the reader that None is an object, not a type, and you then heavily imply it's a type.  This is only going to cause confusion.

I think this is a great improvement. Simplifying is important, but not when it reaches actual misinformation. I think Chris's edit walls that line well.


navigationItemRelated

Hey :)
I'm trying to set up the navigation Item Related Model but when I do, this error shows up and I cannot get the Models fields then.
Idk if I'm doing it wrong? I will be very grateful for your help, Thanks :)

This is the error!

Exact same bug form me.

Closing issue due to merged fix. Please reopen if needed.


Update reth and support x86_64-darwin

I don't have x86_64-darwin machine, so didn't test it. I can remove it from platforms, though.
Also, reth-0.2.0-beta.1 seems to be changed, as I can't reproduce cached build.

Hi @i1i1 thanks for contributing. And secondly, apologies for taking ages to review this PR. Would you mind double checking if still applies for this beta release and, if it's deprecated, to update it to the latest?

if still applies for this beta release and, if it's deprecated, to update it to the latest

WDUM? So far, there are no more releases of reth available at the moment. You can check it out here: https://github.com/paradigmxyz/reth/releases.

@i1i1 just exactly that (I should have checked the release by myself). Merging it now. Thanks.


Create open edx course video models and populate video duration to the mart

What are the relevant tickets?

close https://github.com/mitodl/hq/issues/3227
Description (What does it do?)

Adding int__mitxonline__courserun_videos and int__mitxpro__courserun_videos to populate the video duration with the data in int__ovs__videos .

Modifying video_duration in marts__mitxonline_video_engagements to populate from the above tables. If it can't be populated from OVS (e.g. due to some data issues with course structure https://github.com/mitodl/hq/issues/4465), use the tracking log duration as default.

No need to create a new mart for the moment as the video structure is incorporated in marts__mitxonline_video_engagements
How can this be tested?

dbt build --select int__mitxonline__courserun_videos
dbt build --select int__mitxpro__courserun_videos
dbt build --select marts__mitxonline_video_engagements

Thanks. I removed the not_null test for video_duration in the mart. Per our slack conv, the other two warnings are not related to this PR but the side effect of running dbt build --select + marts__mitxonline_video_engagements


Joy-Con as controller inputs

I tried to also add soft support for other controllers, but currently ALVR seems to assume the controller inputs are always Quest controllers so I had to do some less-optimal fiddling with the bindings for X/Y/dpad.
Currently, Joy-Con don't support motion, I was planning to make them work via the separated controller profile (hold home+screenshot for 4s), but it's missing a lot of inputs on top of missing motion.
Haptics are supported! Except for frequency, for now.

Merging, I'll fix input profiles in a separate PR


Peer dependencies have stopped matching minor versions of react and react-dom - i.e. 15.0.1

The change from "^15.0.0" to "^15.0" as a required version has stopped 15.0.1 / 15.0.2  etc being matched.
Suggest reverting just the peerDependencies for react and react-dom to "^15.0.0".
I can submit a PR for this, but wasn't sure if it was intentional or already on your radar. I'm sure by the time you have read this you will have fixed it anyway.

This change was made upstream: https://github.com/JedWatson/react-select/commit/33a1af11a28ee4b2581956a2c8caa7fa300cda7f I'd prefer not to deviate from the upstream on matters unrelated to the features added by this fork.
Is it possible that you're using an old version of npm? If you use the [http://semver.npmjs.com/](npm semver calculator), enter "react" as the package and "^15.0" as the range, you'll see that 15.0.1 and 15.0.2 are indeed matched.

Very strange, it is now working as expected. I honestly tried it several times and got the same error that the requirement of ^15.0 was not satisfied by 15.0.2 each time so assumed that not specifically including the minor version in the requirement caused the error.
I am on node 4.4.4 (npm 2.5.1) and definitely haven't updated it since trying it earlier. It is possible I have had PhpStorm open for several weeks and so it's terminal was not running the latest... I wouldn't have thought so though.
Apologies for the false issue :)


Cannot get the numbers

Hi, I click "Download Data " button, then it returns a json file. But the the numbers of tweets per day are None value.In fact their values are "NoneType". Am I the only one experiencing this problem?

This is expected behavior for ngrams with a rank below 1 million. Our database only stores ngram ranks if they are more than the 1 millionth most-used word on Twitter. For rare ngrams, you may see 'null' in the json on dates where the rank exceeded 1 million. For example, the json data returned for "🦠" will include null values for all dates before June 2018, because this ngram was not commonly used on Twitter prior to that time.

Thank you for this detailed answer Jane. Now it makes sense. I got the issue. Best.


ka.json fails to parse in Firefox

Not sure what FF doesn't like about it, it loads fine in Chrome.  Need to investigate further when there's time.

This file also totally breaks jshint - which might in fact provide our answer: its encoding might be at fault.

you are correct. should be okay now.


[SUPPORT] a NPE error when reading MOR table in spark datasource

Describe the problem you faced
Hello guys, I meet a NPE when I use spark dataSource API to read a MOR table.  The stacktrace is attached at the end of post.
Then I tried to find suspicious code by online debugging. The result being observed is shown as below.
Let's start from the method buildFileIndex in MergeOnReadSnapshotRelation.
https://github.com/apache/hudi/blob/master/hudi-spark-datasource/hudi-spark/src/main/scala/org/apache/hudi/MergeOnReadSnapshotRelation.scala#L136
Firstly, file status of all parquet files is fetched from InMemoryFileIndex at line 137.
Then all parquet files is fetched as base file from HoodieTableFileSystemView at line 145 as latestFiles.
After that, logic goes into groupLogsByBaseFile in HoodieRealtimeInputFormatUtils.
https://github.com/apache/hudi/blob/master/hudi-hadoop-mr/src/main/java/org/apache/hudi/hadoop/utils/HoodieRealtimeInputFormatUtils.java#L139
At line 158, all fileSlices are returned with some having a parquet base file while others not (not be compacted).
At line 166, for every file slice, hudi try to get base file for it by look up the map which only contains parquet base file id.
When a file slice has not have a parquet base file yet , such looking up will result in NPE.
Could any one please kindly point out which step has an unexpected result?
To Reproduce
The code I used to query is quite simple.
SparkSession spark = SparkSession.builder()
                .appName("hudi-read_guanziyue")
                .enableHiveSupport()
                .config("spark.serializer", "org.apache.spark.serializer.KryoSerializer")
                .config("spark.sql.hive.convertMetastoreParquet", "false")
                .config("spark.driver.allowMultipleContexts", true)
                .config("spark.dynamicAllocation.enabled", true)
                .config("spark.executor.memory", "30g")
                .config("spark.executor.cores", "4")
                .getOrCreate();
        Dataset<Row> queryDF = spark
                .read()
                .format("hudi")
                .option(DataSourceReadOptions.QUERY_TYPE_OPT_KEY(), DataSourceReadOptions.QUERY_TYPE_SNAPSHOT_OPT_VAL())
                .load(warehousePath + "/*");
        queryDF.createOrReplaceTempView("table");
        queryDF.show();

Environment Description

Hudi version :0.6.0

Spark version :3.0.1

Stacktrace
org.apache.hudi.exception.HoodieException: Error obtaining data file/log file grouping: hdfs://mytablePath/20210308
	at org.apache.hudi.hadoop.utils.HoodieRealtimeInputFormatUtils.lambda$groupLogsByBaseFile$16(HoodieRealtimeInputFormatUtils.java:162)
	at java.util.HashMap$KeySet.forEach(HashMap.java:932)
	at org.apache.hudi.hadoop.utils.HoodieRealtimeInputFormatUtils.groupLogsByBaseFile(HoodieRealtimeInputFormatUtils.java:131)
	at org.apache.hudi.MergeOnReadSnapshotRelation.buildFileIndex(MergeOnReadSnapshotRelation.scala:139)
	at org.apache.hudi.MergeOnReadSnapshotRelation.<init>(MergeOnReadSnapshotRelation.scala:73)
	at org.apache.hudi.DefaultSource.createRelation(DefaultSource.scala:98)
	at org.apache.hudi.DefaultSource.createRelation(DefaultSource.scala:53)
	at org.apache.spark.sql.execution.datasources.DataSource.resolveRelation(DataSource.scala:342)
	at org.apache.spark.sql.DataFrameReader.loadV1Source(DataFrameReader.scala:279)
	at org.apache.spark.sql.DataFrameReader.$anonfun$load$2(DataFrameReader.scala:268)
	at scala.Option.getOrElse(Option.scala:189)
	at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:268)
	at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:214)
	at hudiReadExample.main(hudiReadExample.java:32)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.apache.spark.deploy.yarn.ApplicationMaster$$anon$2.run(ApplicationMaster.scala:742)
Caused by: java.lang.NullPointerException
	at org.apache.hudi.hadoop.utils.HoodieRealtimeInputFormatUtils.lambda$null$15(HoodieRealtimeInputFormatUtils.java:151)
	at java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:184)
	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
	at java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:175)
	at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
	at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1374)
	at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
	at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
	at java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:151)
	at java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:174)
	at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:418)
	at org.apache.hudi.hadoop.utils.HoodieRealtimeInputFormatUtils.lambda$groupLogsByBaseFile$16(HoodieRealtimeInputFormatUtils.java:149)

CC @n3nash : Can you take a look at this hbase index related issue. I remember seeing a similar issue relating to a fileGroup w/ just a log file w/o base file. Not sure if its related.

@guanziyue This issue is the same as https://github.com/apache/hudi/issues/2633. After some analysis, we have found the underlying issue as pointed in that particular ticket. Once a PR is raised for that issue, this issue should also be addressed.

@n3nash : add severity label if you have sense of it. just reminding.

I also meet the problem  when i use spark/hive to read the mor rt table writed by flink.I think it's a critical bug.
Hope you solve this problem first。

@hushenmin Thanks for the cross comment. Yes, it looks like for _rt view, for indexType = global when there are no base data files, this exception is thrown. We will take this up soon.

@hushenmin if you need a rapid but not good fix. I could offer a temp branch. This can be fixed by temporarily disable some feature of canIndexLogFile. However, such action is opposite to the roadmap of community. I have checked it works well in prod.

@hushenmin if you need a rapid but not good fix. I could offer a temp branch. This can be fixed by temporarily disable some features of canIndexLogFile. However, such action is opposite to the roadmap of community. I have checked it works well in prod.

Here is the PR raised on this behalf: https://github.com/apache/hudi/pull/2902
@n3nash : I see there were 2 jiras filed and later one was closed as duplicate. So, can you confirm that the PR fixes the issue raised in this ticket.

@nsivabalan @n3nash  Actually this problem is not totally equivalent to the issue 2633. Just like I mentioned before, the method buildFileIndex in MergeOnReadSnapshotRelation assume all file group must have a base file. Even though issue 2633 is fixed, an index with canLogIndexFile will still generate file group w/o base file. I think the logic in MergeOnReadSnapshotRelation should have a small fix. Method groupLogsByBaseFile of HoodieRealtimeInputFormatUtils may group logs by fileID rather than Base file.

@guanziyue Thanks for the note. Would you mind filing a JIRA and opening a PR for the same ?

@guanziyue Thanks for the note. Would you mind filing a JIRA and opening a PR for the same ?

Hi na3nash, I find https://github.com/apache/hudi/issues/2874. I think they are actually the same problem because HoodieRealtimeInputFormatUtils is shared by both hive and spark. It seems that @danny0405 has taken over this problem. Anything I can contribute to this problem?

@guanziyue that PR looks good. Should be landed soon. May be you can verify after #3032 is landed and see if all is well?

I have verified the patch and I am able to run snapshot query on MOR table with latest master and Spark 3.0.1.

Closing this as we have verified. Please do re-open if you run into issues. Thanks for raising it.


Missing return in missing overrides quick fix when class body is on one line

Create a dart file with the following content:
abstract class A {
  void m();
}
class B extends A {}

Select B, then select the quick fix "Create 1 missing overrides". There should be an extra carriage return after the opening brace.

https://codereview.chromium.org/2256463002

https://github.com/dart-lang/sdk/commit/010578b88aa9cdd43bc4b43dcb55ee56a3394f90


Create static plugin frameworks build ios-framework --static

Just the plugins part of https://github.com/flutter/flutter/issues/52217.  Build the plugin xcframeworks as static frameworks.  Flutter.xcframework and App.xcframework are still dylibs.
Previous behavior creates dylibs dynamically linked shared library:
$ flutter build ios-framework
$ $ file build/ios/framework/Debug/url_launcher_ios.xcframework/ios-arm64/url_launcher_ios.framework/url_launcher_ios
build/ios/framework/Debug/url_launcher_ios.xcframework/ios-arm64/url_launcher_ios.framework/url_launcher_ios: Mach-O 64-bit dynamically linked shared library arm64

With --static the plugins are ar archive random library
$ flutter build ios-framework --static
$ file build/ios/framework/Debug/url_launcher_ios.xcframework/ios-arm64/url_launcher_ios.framework/url_launcher_ios
build/ios/framework/Debug/url_launcher_ios.xcframework/ios-arm64/url_launcher_ios.framework/url_launcher_ios: current ar archive random library

I tested this manually by:

Adding a plugin url_launcher: to the sample project pubspec.
Adding a button that launches a URL to the dart code:

@override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: ElevatedButton(
          onPressed: () async {
            await launchUrl(Uri.parse('https://www.google.com'));
          },
          child: Text('Launch URL'),
        ),
      ),
    );
  }

Calling GeneratedPluginRegistrant.register(with: self.flutterEngine!); in the AppDelegate
Linking (not embedding!) the static url_launcher_ios and FlutterPluginRegistrant xcframeworks
in the Xcode project

		F733C128283DD5C7005295FB /* url_launcher_ios.xcframework */ = {isa = PBXFileReference; lastKnownFileType = wrapper.xcframework; name = url_launcher_ios.xcframework; path = "Flutter/$(CONFIGURATION)/url_launcher_ios.xcframework"; sourceTree = "<group>"; };
		F733C12B283DD6C4005295FB /* FlutterPluginRegistrant.xcframework */ = {isa = PBXFileReference; lastKnownFileType = wrapper.xcframework; name = FlutterPluginRegistrant.xcframework; path = "Flutter/$(CONFIGURATION)/FlutterPluginRegistrant.xcframework"; sourceTree = "<group>"; };

			isa = PBXFrameworksBuildPhase;
			buildActionMask =<PHONE_NUMBER>;
			files = (
				F733C12C283DD6C4005295FB /* FlutterPluginRegistrant.xcframework in Frameworks */,
				F77F88F1255396AD00309E51 /* App.xcframework in Frameworks */,
				F77F88F2255396AD00309E51 /* Flutter.xcframework in Frameworks */,
				F733C129283DD5C7005295FB /* url_launcher_ios.xcframework in Frameworks */,
			);
			runOnlyForDeploymentPostprocessing = 0;

etc.
Run the app, tap the button, the URL launches.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Can I enable this feature on Flutter 3.0?

This is available on 3.1.0-9.0.pre, currently in the beta channel.


Tensorflow using Intel GPU instead of NVIDIA GPU

Please make sure that this is an issue related to performance of TensorFlow.
As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:performance_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 Home 64-bit
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 2.2.0
Python version: 3.8.3
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: 10.1/7.6
GPU model and memory: NVIDIA GTX 1050 (4 GB)

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
TensorFlow, by default is using GPU 0, which on my PC is my built-in Intel GPU while I have a dedicated NVIDIA GPU and have CUDA installed yet TF is using Intel.
Describe the expected behavior
TensorFlow should display current device to be NVIDIA GPU
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@tamimmirza
Can you list all gpus by following the below link and then manually place the gpu you need.
https://www.tensorflow.org/guide/gpu#setup
Thanks!

I did that before and it shows me that it has detected one GPU despite me having two as you can see in the task manager windows above:

Which is weird considering I have installed up-to-date drivers and CUDA Toolkit 10.1 Update 2 (along with cuDNN 7.6)

Here is the verification that CUDA is installed via Visual Studio 2019 IDE

@tamimmirza
Please try install TensorFlow gpu using the below command
pip install tensorflow-gpu==2.2.0 and check if you are still facing the same issue. Thanks!

I did that prior to opening the issue
I uninstalled tensorflow and installed tensorflow-gpu and I still got the same result
I also did a fresh install of Miniconda

I installed TensorFlow 2.3.0 along with a fresh install of Miniconda3 and it solved my problem and correctly identifies the proper GPU


Intermittent SSL certificate failures on cdn.freecodecamp.org

Is anybody else seeing this issue? It has happened to me multiple times over a period of time. It affects everything that uses the CDN, i.e. the test script and forum login page (script and SVG resource fail to load).
It seems very intermittent and random so I'm not really sure what is going on.

SSL Server Test:

Hmm looks like the CDN is erroneously picking up the Cloudflare Certificate at times than the one from netlify.
I have been unable to reproduce this, and going to rule this out as a temporary DNS cache issue?
Please re-open if it persists.


minikube: 1.20.0 -> 1.21.0

Motivation for this change
https://github.com/kubernetes/minikube/releases/tag/v1.21.0
Things done
$ NIXPKGS_ALLOW_UNFREE=1 nixpkgs-review rev minikube-1.21.0
[...]
2 packages updated:
docker-machine-kvm2 (1.20.0 → 1.21.0) minikube (1.20.0 → 1.21.0)
[...]
2 packages built:
docker-machine-kvm2 minikube

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 128549 at 7a935a5b run on aarch64-linux 1

2 packages built successfully:

docker-machine-kvm2
minikube


Python 3ify Loqui

Ran modernize
Add travis for running python tests
Make setup.py run tests

Still WIP just want to get travis running for future commits.

@gmjosack tests pass on Py2 and 3 now.

API tests pass using this, in Python 2.


Add a doc for the garbage collector

cc @lavalamp @johndmulhausen
@pwittrock do you know how to add the doc to the navigation sidebar?

http://kubernetes.io/editdocs/ has instructions on left nav editing.

Addressed the comments and updated the nav bar.

@caesarxuchao Ping

@pwittrock I pushed another version. PTAL. Thanks!

Thanks for the tech review @pwittrock and thanks for the PR @caesarxuchao ! Merging so we can see this staged; passes the test structurally. :)


pod start fails sporadically

We sometimes are missing the qmp notification:
E1122 10:13:06.986580    1287 qemu.go:1004] Unable to launch qemu: exit status 1
E1122 10:13:06.986712    1287 qemu.go:1006]
F1122 10:13:07.815658    1287 main.go:731] Could not start pod: Did not receive the pod started notification

@sameo have you been able to reproduce it recently ? I think we fixed this one long time ago, but wanted your confirmation before I close it.


Export definitions and views

Research how to do this.

Maybe look at  the HTML export functie

This is a dublicate


fix allocation length in listener patch removal

Addresses one minor comment in https://github.com/istio/istio/pull/36404

@hzxuzhonghu fixed PTAL

fixed


Cloud flare clearance failed, please wait one minute and try again

Hi,
I've been getting the following error when trying to log in with procurement:

Cloud flare clearance failed, please wait one minute and try again

CloudFlare is that extra layer of authentication GGG recently added, to try to prevent DDoS attacks.  You might have noticed the brief screen that loads when you go to the login page on the pathofexile.com website.  I've noticed it can get temperamental sometimes with Procurement.
Make sure the Session ID value you're using is still valid (you're still logged in on the web site).  Also, it might help to wait longer than a minute (maybe ten, unfortunately), or log out and back in on the website to get a new Session ID value.

Yeah, had to logout and log in again. Was trying to write a script to find duplicate itens on my stash tabs, but never managed to get a response back. Maybe I got my sessionID blacklisted by them or something like that :(

Duplicate items in what way?  Maybe it's something Procurement can do now, or we can add it.  I've been thinking about ways and reasons to sort/search stash items.

Basically I want to get rid of some of the 150 tabs I have on standards, lol
What I wanted to do is find all the duplicate uniques I have (hoader and collector here), so I can remove most of them and keep a single copy only.

Ah, yeah, I don't think Procurement can help with that task (right now).  But all I've got to say is, maybe hold off until the new tab is announced this week/next week; it's supposed to be suited for collectors. :smiley:

Wasn't aware of that announcement! Thanks for sharing!
Btw, managed to get my script running :)
Used cloudscraper to bypass the Cloudflare check.


Xavier_init bugs when normorlize is enable?

In fpn structure, the repo uses xavier_init for convs.
However, when normorlize is enable (I try to use nn.GroupNorm), It produces error "Nonetype object has no attribute fill_".
I found this occured when initializing bias which does not exist.
In face, it seems that in the model/utils/weight_init.py, the "hasattr(module, 'bias')" expression is always right

Thanks for reporting the bug! Initialization methods in mmcv.cnn.weight_init is more reliable.


Where's the equivalent page for the "Windows 10 app package manifest schema"?

Where's the equivalent page for the "Windows 10 app package manifest schema"?  Why are the links taking me to WIndows 8 documentation?

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

ID: e3d3549b-6f25-f819-b917-d921cf635f02
Version Independent ID: d204096e-ebe1-6436-f51d-b7d27001b1cb
Content: Windows 8 app package manifest schema - Windows UWP applications
Content Source: winrt-related-src/schemas/appxpackage/appxmanifestschema/schema-root.md
Product: uwp
Technology: winrt-reference
GitHub Login: @GrantMeStrength
Microsoft Alias: jken

The Windows 10 package manifest schema is documented in this section. If I have misinterpreted your question, please feel free to reopen this issue with more details.


EDMFX diagnostic and explicit

Purpose

Implement physics closures in EDMFX and run explicit simulations.
Implement diagnostic EDMFX.

Cost/Benefits/Risks

Cost: Not having EDMFX costs us dearly.
Benefit: This is required before working on implicit EDMFX.
Reducing risk: Diagnostic EDMFX is our backup for implicit EDMFX.

People and Personnel

Lead: @trontrytel
Collaborators: @dennisYatunin @nefrathenrici @szy21 @jiahe23 @costachris
Reviewers: @tapios

Components

Part 1: Implement physics closures in EDMFX
Part 2: Implement diagnostic EDMFX
Part 3: Reproduce the canonical single column cases for diagnostic and explicit setups.

Inputs

Re-use as much as possible of the old EDMF closures (update_aux, etc).
Documentation of solved equations in overleaf and numerics in ClimaAtmos docs.
Calibration pipeline for calibrating EDMFX in ClimaAtmos.jl.
Turbulence/convection science support in making the simulations stable.

Results and Deliverables

Simulations reproducing single column canonical cases that we have in the CI (Bomex, Dycoms, etc)
A spherical moist baroclinic wave simulation that can be run for a couple of steps with EDMFX.
Documentation added to ClimaAtmos docs

Milestones

[ ] Run the prognostic edmf explicitly for bomex, dycoms, and rico
[ ] Run the prognostic edmf explicitly on the sphere
[ ] Run the diagnostic edmf (with TKE equation) explicitly for bomex, dycoms, and rico
[ ] Run the diagnostic edmf explicitly on the sphere
[ ] Run the diagnostic edmf with implicit vertical diffusion on the sphere (?)

Task Breakdown And Schedule
Part 1: Implement physics closures (@trontrytel, @dennisYatunin)

[ ] #1421 :
[x] non-hydrostatic pressure
[x] buoyancy
[x] subdomain buoyancy exchange term
[ ] shear
[x] saturated/unsaturated gradients
[x] buoyancy gradients
[x] mixing length
[ ] microphysics
[x] SGS fluxes to the grid mean

Part 2: Implement diagnostic EDMFX (@szy21 , @jiahe23)

[x] Write equations documentation for diagnostic EDMFX
[x] #1722
[x] #1655
[x] #1724
[ ] #1800
[x] #1806
[x] #1743
[x] #1896
[x] #1897
[x] #1840

Part 3: Simulations

[ ] Bomex
[ ] GABLS
[ ] Soares
[ ] Dycoms
[ ] Rico
[ ] TRMM
[ ] Nieuwstadt
[ ] lifecycle
[ ] spherical moist baroclinic wave

QA and testing (@jiahe23 , @szy21 , @trontrytel )

[ ] #1600
[ ] Test for momentum equations (without RHS terms?) in a bubble setup.
[ ] Add LES profiles to CI plots

Calibration pipeline. (@nefrathenrici, @costachris )

[x] https://github.com/CliMA/ClimaAtmos.jl/issues/1340
[ ] Calibratmos.jl package for a single SCM EDMF case in CA.jl - Perfect model setting.

SDI Revision Log

CC: @tapios @simonbyrne @cmbengue

Superseded by #1974 (and other SDIs to be opened).


Azure OpenAI: update codegen

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.AI.OpenAI


Version 3.7 Does Not Support Firefox

I am using the zoom-in effect from animate.min.css and firefox (desktop) is not responding to it.
Here are my test results:
Firefox latest: 3.5.2 works fine, 3.7 doesn't work
Chrome latest: both versions work
Please let me know if this is a glitch on my part. Thanks.

Hello.
Can you elaborate more? I just tested on FF 64.0.2 and 65.0 on Windows and everything worked just fine.

Hi. I am using Firefox 65.0 (64-bit) on Windows and the only variable seems to be the version of my animate.css file.
I have tried the following:

alternate betweed CDN and local file (doesn't work)
use animate.css instead of animate.min.css (doesn't work)
start in safe mode with extensions disabled (doesn't work)

I can confirm that 3.7 works with FF 64.0.2 on my other computer. Just not sure about 65.
It works on FF 65 as soon as I swap 3.7 for 3.5.2.

I'm pretty sure this is related to #831. Please check your configurations on Windows to see if animations are off.
(Go to: "Configuration > Screen > Show animations on Windows" and check if this on)

Thank you. It turns out that Firefox is just being presumptuous.


chore: decrease bundled font size

Description
This PR decreases the size of fonts when Faency is bundled for production usage.
Dependency changes

Dependency
Version
State

rollup-plugin-url
@rollup/plugin-url
Replaced

rollup-plugin-typescript2
^0.30.0 -> ^0.31.2
Updated

How to test?

Bundle an app using Faency with and without those changes.
(This app must use the FaencyProvider so fonts are imported)
If app is a Next.js one the diff can be seen in the First Load JS shared by all section

Good PR checkboxes

[x] Change has been tested
[ ] Added/Updated tests
[ ] Added/Updated stories
[x] PR follows conventions
[x] Labels are set
[x] Project is linked

Good Review checkboxes

 ℹ  Copy the snippet and paste in the review field to fill it
- [ ] I've tested the changes
- [ ] I've agreed on the unit tests (soon to come)
- [ ] I've checked the stories
- [ ] I've read the code and understood it
- [ ] I don't have any more questions
- [ ] I've described any optional improvements
- [ ] I checked PR follows [conventions](https://github.com/traefik/faency#how-to-contribute)

@matthieuh Documenting "How to replace a remote package by a local one" would help review IMO.
https://stackoverflow.com/questions/15806241/how-to-specify-local-modules-as-npm-package-dependencies

How to test section is too complex for review. Please, either provide a package to run the tests or ask to skip testing.

I updated the How to test? section giving more explanations.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve build and fix bug

Clean up ./target/build when compilation succeed.
Fix codegen executable path

I am gonna to close this PR. The changes will come along with the next PR of flock.


Make TreeMap scroll corresponding grid to show selection

When a record is selected in the TreeMap, the corresponding Grid should automatically scroll the corresponding record into view.

👍


[automated] Re-Apply Coding Standards

Automated changes by create-pull-request GitHub action

Codecov Report
Base: 93.97% // Head: 93.97% // No change to project coverage :thumbsup:

Coverage data is based on head (f93a991) compared to base (509beac).
Patch coverage: 37.50% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #162   +/-   ##
=========================================
  Coverage     93.97%   93.97%
  Complexity      478      478
=========================================
  Files           109      109
  Lines          1643     1643
=========================================
  Hits           1544     1544
  Misses           99       99

Flag
Coverage Δ

phpunit
93.97% <37.50%> (ø)

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

Impacted Files
Coverage Δ

src/metadata/src/YamlOperator.php
77.69% <37.50%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


🐛 firebase_auth_web - Firebase Authentication State Reset Issue When Running Flutter Web App

Bug report
Describe the bug
When debugging a Flutter web application that uses Firebase for authentication, the Firebase authentication state is reset each time the web app is run in debug mode. This issue does not occur on iOS or Android.
Steps to reproduce
Steps to reproduce the behavior:

Start a Flutter web project with Firebase authentication integration.
Use the following code snippets in the project:

Query user at the start:final firebaseUser = await FirebaseAuth.instance.authStateChanges().first;

Stream to handle the user's authentication state:_auth.authStateChanges().listen((firebaseUser) {
  _firebaseUser.add(firebaseUser);
});

Logging in with the following method:var result = await _auth.signInWithEmailAndPassword(
  email: email, password: password
);

Debug the web app, stop it, and then rerun it.
Observe that the user is automatically logged out upon rerunning the web app.

Expected behavior
The expected behavior is that the Firebase authentication state should persist when rerunning the web app in debug mode, just as it does on iOS and Android.

Additional context
firebase_core: ^2.15.1
firebase_auth: ^4.9.0
firebase_core_web: ^2.7.0
firebase_auth_web: ^5.8.0

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[✓] Flutter (Channel stable, 3.13.2, on macOS 13.5.2 22G91 darwin-arm64, locale en-DE)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.81.1)

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
- _flutterfire_internals 1.3.5 [collection firebase_core firebase_core_platform_interface flutter meta]
- cloud_firestore_platform_interface 5.16.0 [_flutterfire_internals collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 3.7.0 [_flutterfire_internals cloud_firestore_platform_interface collection firebase_core firebase_core_web flutter flutter_web_plugins js]
- cloud_functions_platform_interface 5.5.0 [firebase_core flutter meta plugin_platform_interface]
- cloud_functions_web 4.6.0 [cloud_functions_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- firebase_analytics_platform_interface 3.6.5 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_analytics_web 0.5.4+5 [_flutterfire_internals firebase_analytics_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- firebase_app_check_platform_interface 0.0.9+2 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_app_check_web 0.0.11+2 [_flutterfire_internals firebase_app_check_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- firebase_auth_platform_interface 6.18.0 [_flutterfire_internals collection firebase_core flutter meta plugin_platform_interface]
- firebase_core_platform_interface 4.8.0 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_crashlytics_platform_interface 3.6.5 [_flutterfire_internals collection firebase_core flutter meta plugin_platform_interface]
- firebase_dynamic_links_platform_interface 0.2.3+25 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_platform_interface 4.5.6 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_web 3.5.6 [_flutterfire_internals firebase_core firebase_core_web firebase_messaging_platform_interface flutter flutter_web_plugins js meta]
- firebase_storage_platform_interface 4.4.5 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_storage_web 3.6.6 [_flutterfire_internals async firebase_core firebase_core_web firebase_storage_platform_interface flutter flutter_web_plugins http js meta]

Hi @Yetispapa, do you launch a new instance of chrome whenever you run the app? If that is the case then this is not a firebase_auth issue.

That's the issue. Instead of running using the debugger, you can run the following command flutter run -d web-server --web-port=<specify_port_here(ignore angle brackets)> and then launch the app on that port in your browser


Can't Arm/Disarm alarm

Was anyone able to send a request to Aram/Disarm the alarm at HA?
I think I might need to be able to add the alarm pin code somehow but I'm not sure how to make it work.

Solved by adding:
{
"code": pincode
}


Move away from Chromium to stop the Google's monopoly on web standerds.

Solution You'd Like
I would like Beaker Browser to move away from Google's technology which Is trying to centralize the web.
Alternatives You've Considered
Mozilla's Gecko engine Is the way to go I think, other projects like Tor Browser have had success with Gecko.

I appreciate the motive but we lack the resources for a move like this, or to even build on gecko. Beaker depends on electron to be feasible given our resources.

There are numerous other options that might be worth checking out, these avoid Electron and its bundled Chrome:
https://github.com/sudhakar3697/electron-alternatives#js


realpath usage fails when options.cwd != process.cwd()

the result is something like
Error: ENOENT, lstat '<actual-cwd>/<first-file-in-options.cwd>'

since fs.realpath does path.resolve() on the path, glob needs to pass absolute paths, not just the match results.
from node fs.js:
fs.realpath = function realpath(p, cache, cb) {
  if (typeof cb !== 'function') {
    cb = maybeCallback(cache);
    cache = null;
  }

  // make p is absolute
  p = pathModule.resolve(p);

Yep, that's a bug.  Good find!


Save color settings

Right now the color settings (style guide) reset for each time I restart Sketch. Would be great if those could be saved in the general settings of the plugin.
Thanks.

Same goes for the design resolution settings, would be helpful if the plugin remembered user settings with an option to reset to defaults?

@utom Any update here? :)

try new~

@utom Ok, there are a bunch of changes, yes - but I don't find the settings anymore, the text layer has now 2px character spacing and every time I use the plugin it creates a hidden config file in my artboard, which doesn't give me any control.
Can you please check? The initial problem is still existing and now the tool is not usable anymore at all.


Combined reports

We would need a report that combines all reports together in one big export to be used in another software for further analysis. It would make sense to add possibility to include and exclude some reports if needed.
We have an alpha version here:
https://github.com/richinfante/iphonebackuptools/commit/dc101b12959e75ce20fcdb832b6105f581aadf51
Should we do pull request for this version or you implement it yourself?

Hmm, I didn't think of this! This might be best implemented inside of #6, where each report could be either printed to stdout, or possibly written to disk in a separate file. The format of the individual reports could be dictated by the user selected output formatter. I'll explore implementing this in that PR.

I would be using it with stdout, but why not add the files. My idea was that all the reports form one json object. The report names could be used as keys. Then you could push it further to other software as one data bundle. To stdout or file.

That sounds like a good idea, I'll explore it.


Explain rel=noreferrer as well as rel=noopener

What's changed, or what was fixed?

Adds information on rel=noreferrer following GoogleChrome/lighthouse#4920

[ ] This has been reviewed and approved by (?)

[x] I have run gulp test locally and all tests pass.

[ ] I have added the appropriate type-something label.

[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

:tada: This has been pushed live to https://developers.google.com/web/


Add Dispatcher loom based module Implementation

Progress in #2928

Proposed Changes

Loom-based dispatcher module implementation
Unit Test for them

/retest

Hey @matzew, I have added the comment.

/retest-required


Ant Design support

Hi, i'm trying to use stitches with the ant design ui lib, but unfortunately it doesn't work.
It doesn't generate the class names properly on the Ant Design Components.
example:
my code

    ©2020 Created by Gustavo Gualbano

what generates:
©2020 Created by Gustavo Gualbano

it puts this atoms toString on the class.
if i use on a native html element likes this

  ©2020 Created by Gustavo Gualbano

works just fine, but if i use directly on a ant design component, it doesn't.

hi @gsgualbano ,
The reason for this is that we we depend on the toString method of the return object to be called. This is normally called in all circumstances where a string is expected, but for some reason ant does not follow this convention.
If you were to String(css.compose(css.textAlign('center'))), you would be okay.
You have a link to the project? We could create a PR which ensures this behaviour, as it should really work that way.

Thank you @christianalfoni,
Wrapping the compose on a String function worked really well. I don't have a link to my project because it's just a side project that i made for studying purposes, anyway thanks again.

Oh, I meant to Ant :-) so that we can fix the issue on their side. Glad it
worked! :-)
On Fri, 24 Apr 2020 at 18:41, Gustavo Gualbano<EMAIL_ADDRESS>wrote:

Thank you @christianalfoni https://github.com/christianalfoni,
Wrapping the compose on a String function worked really well. I don't have
a link to my project because it's just a side project that i made for
studying purposes, anyway thanks again.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/christianalfoni/stitches/issues/23#issuecomment-619122081,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6GBQO5MDEVRZ3MH6XNJTDROG6M5ANCNFSM4MPNDJXA
.

lol 😂, I’m sorry I misunderstood, here is the link https://github.com/ant-design/ant-design


Evaluation Code is Missing

Hi, may I know whether the evaluation code related to the metrics mentioned in the paper has been released? i.e.: FID

haah


[Feature] Change alpha when item is completed and add LineThrough to text decoration

Context
This improvements will help user to take more focus to uncompleted items

Completed on PR #31


fix preloader position

PR Checklist
Preloader posisiton central
PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

thank you very much.
would it better to merge into one commit, it would be cleaner in log and easier to revert. Also, please follow the contributing guide


Open multiple files

My pipeline function
function openhtml(done){
    return gulp.src('dist/**/*.html')
        .pipe(open({app: 'chrome'}, done));
}

First file will open, but next files this error ocurres:
Opening my.html using the app chrome
The following tasks did not complete: dev, openhtml
Did you forget to signal async completion?

Just published v2.0.0, try with that version

Sweet, thx

:smile:


docker-airflow for version 1.8.0

Hi @puckel ,
are you planning a new docker container for the new Airflow 1.8.0 that will be out in a couple of days?
I have been using you airflow docker container for months now. great job and thanks

Hi,
Yes, I follow the Airflow release.


Build failing at npm run build

My build is failing when I run the step   > npm run build
out put is
0 info it worked if it ends with ok
1 verbose cli [ 'C:\Program Files\nodejs\node.exe',
1 verbose cli   'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js',
1 verbose cli   'run',
1 verbose cli   'build' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prebuild', 'build', 'postbuild' ]
5 info prebuild<EMAIL_ADDRESS>6 info build<EMAIL_ADDRESS>7 verbose unsafe-perm in lifecycle true
8 info<EMAIL_ADDRESS>Failed to exec build script
9 verbose stack Error<EMAIL_ADDRESS>build: npm run clean && ./node_modules/.bin/gulp && ./node_modules/.bin/assemble --file assemblefile.js
9 verbose stack Exit status 1
9 verbose stack     at EventEmitter. (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:217:16)
9 verbose stack     at emitTwo (events.js:87:13)
9 verbose stack     at EventEmitter.emit (events.js:172:7)
9 verbose stack     at ChildProcess. (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:24:14)
9 verbose stack     at emitTwo (events.js:87:13)
9 verbose stack     at ChildProcess.emit (events.js:172:7)
9 verbose stack     at maybeClose (internal/child_process.js:829:16)
9 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:211:5)
10 verbose pkgid<EMAIL_ADDRESS>11 verbose cwd c:\Temp\metis7
12 error Windows_NT 6.1.7601
13 error argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "build"
14 error node v4.5.0
15 error npm  v2.15.9
16 error code ELIFECYCLE
17 error<EMAIL_ADDRESS>build: npm run clean && ./node_modules/.bin/gulp && ./node_modules/.bin/assemble --file assemblefile.js
17 error Exit status 1
18 error Failed at the<EMAIL_ADDRESS>build script 'npm run clean && ./node_modules/.bin/gulp && ./node_modules/.bin/assemble --file assemblefile.js'.
18 error This is most likely a problem with the bootstrap-admin-template package,
18 error not with npm itself.
18 error Tell the author that this fails on your system:
18 error     npm run clean && ./node_modules/.bin/gulp && ./node_modules/.bin/assemble --file assemblefile.js
18 error You can get information on how to open an issue for this project with:
18 error     npm bugs bootstrap-admin-template
18 error Or if that isn't available, you can get their info via:
18 error
18 error     npm owner ls bootstrap-admin-template
18 error There is likely additional logging output above.
19 verbose exit [ 1, true ]
any help would be appreciated.
thanks
Colin

@OldProgramCoder
What is the result of the following commands, respectively?
npm run clean
./node_modules/.bin/gulp
./node_modules/.bin/assemble --file assemblefile.js

Not 100% sure of how you want these run but ....    If I have done it wrong let me know.
Colin@COLINS-M6800 MINGW64 /c/temp/metis3 (master)
$ npm run clean
><EMAIL_ADDRESS>clean C:\temp\metis3
> rimraf public dist
Colin@COLINS-M6800 MINGW64 /c/temp/metis3 (master)
$ npm run clean   ./node_modules/.bin/gulp
><EMAIL_ADDRESS>clean C:\temp\metis3
> rimraf public dist "./node_modules/.bin/gulp"
Colin@COLINS-M6800 MINGW64 /c/temp/metis3 (master)
$ npm run clean  ./node_modules/.bin/assemble --file assemblefile.js
><EMAIL_ADDRESS>clean C:\temp\metis3
> rimraf public dist "./node_modules/.bin/assemble" "assemblefile.js"
Further information
I stalled npm v6.6 today and here are the messages I get throughout the process
Colin@COLINS-M6800 MINGW64 /c/temp
$ npm install
npm ERR! registry error parsing json
npm WARN deprecated<EMAIL_ADDRESS>ReDoS vulnerability parsing Set-Cookie https://nodesecurity.io/advisories/130
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Upgrade to<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
><EMAIL_ADDRESS>preinstall C:\temp\metis3
> bower install              << this started automatically which was a change from npm 4.5
< everything runs well for a while with good out put...   and then>
bower rangy-1.3#master                      invalid-meta The "main" field has to contain only 1 file per filetype; found multiple .js files: ["uncompressed/rangy-core.js","uncompressed/rangy-cssclassapplier.js","uncompressed/rangy-highlighter.js","uncompressed/rangy-selectionsaverestore.js","uncompressed/rangy-serializer.js","uncompressed/rangy-textrange.js"]
bower rangy-1.3#master                          resolved https://github.com/noap/rangy-1.3.git#d2b26285b8
< everything runs well  and the directory seems to be out put and then command returns with >
npm WARN optional Skipping failed optional dependency /chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>Colin@COLINS-M6800 MINGW64 /c/temp/metis3 (master)
here I run  > npm run build
Colin@COLINS-M6800 MINGW64 /c/temp/metis3 (master)
$ npm run build
><EMAIL_ADDRESS>build C:\temp\metis3
> npm run clean && ./node_modules/.bin/gulp && ./node_modules/.bin/assemble --file assemblefile.js
><EMAIL_ADDRESS>clean C:\temp\metis3
> rimraf public dist
'.' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "build"
npm ERR! node v6.6.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: npm run clean && ./node_modules/.bin/gulp && ./node_modules/.bin/assemble --file assemblefile.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'npm run clean && ./node_modules/.bin/gulp && ./node_modules/.bin/assemble --file assemblefile.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the bootstrap-admin-template package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run clean && ./node_modules/.bin/gulp && ./node_modules/.bin/assemble --file assemblefile.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs bootstrap-admin-template
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls bootstrap-admin-template
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\temp\metis3\npm-debug.log
Colin@COLINS-M6800 MINGW64 /c/temp/metis3 (master)
$

@OldProgramCoder
Check it out now.
https://github.com/puikinsh/Bootstrap-Admin-Template/blob/master/package.json#L33-L36

Works good. Much appreciated.   The only thing that needs to done is fix issue 67 which affects the bower.json  file.   Then this would build right out of the box.
Nice site by the way.   I really like it.
https://github.com/puikinsh/Bootstrap-Admin-Template/issues/67   << bower.json


BroadcastMessage() function to send to all receivers

I found the smpptest server extremely useful for testing transmitters but not so helpful for testing receivers. In the receiver_test for example, receipt of a message is achieved by hijacking the Receiver.conn, rather than coming from the actual test server.
So I've added a BroadcastMessage function to write a given test message to all bound receiver connections.

Cool, thanks!


new post loading indicator

Purpose
The goal of this PR is to improve the UX when posting
Closes #85
Solution
Add post loading indicator to the frontend after a post is created.
Change summary

Added skeleton post instead of spinner

Steps to Verify

Create a new post, see the indicator.

Additional details / screenshot
https://github.com/user-attachments/assets/0c9d77dc-d401-4b2c-8146-99f6f0c8c95f

Question... The loading indicator stops when it has been sent to the chain, but the user doesn't really get any feedback that it's not going to show up for a little longer becuase we have to wait for the watcher to retrieve it. How do we want to handle that UX, or is this good for now?


Provider React and NextJS providers and hooks

To ease the integration in React and NextJS app, it would be interesting to provide providers and hooks.
render(
  <JunoProvider satelliteId={satelliteId}>
    <App />
  </JunoProvider>,
  document.getElementById('root'),
);

For example I think Solana does that in their WalletProvider. Another example is the Cleck auth.

Let's do it! Done. ✅
As a next, do you want to instantiate the sub-folder/project for this feature?

Let's do it.
I think I can initialize the folder and set up the basic package structure:

src/

components/

JunoProvider.tsx
useJuno.ts

contexts/

JunoContext.ts

hooks/

useCollection.ts
useDoc.ts

test/

components.test.tsx
hooks.test.tsx

README.md
package.json
tsconfig.json
Let me know, if this is still in line with the vision.

Sounds good to me!
Given that this repo is a npm workspace I would just not initiate the src at the root but in plugins/react-hooks or any better idea?

Plugin/react-hooks is way better, I just moved in the plugins folder.
Currently reading the Juno documentation, I want to see all the functions and propably use the dependencies if necessary.

Coolio. Ping me any time if you ever got questions.


Rename base package

Resolves #150
Note : After merging PR, rebuild intellij project
And I branched off my delete-attendance-command branch so there are commits related to it here. I'll pull from delete-attendance-branch over if there are any amendments to be made to that PR branch

Codecov Report

Merging #154 into master will increase coverage by 0.10%.
The diff coverage is 84.41%.

@@             Coverage Diff              @@
##             master     #154      +/-   ##
============================================
+ Coverage     80.57%   80.68%   +0.10%
- Complexity      920      934      +14
============================================
  Files           136      138       +2
  Lines          2693     2770      +77
  Branches        309      322      +13
============================================
+ Hits           2170     2235      +65
  Misses          440      440
- Partials         83       95      +12

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/tutorspet/AppParameters.java
75.00% <ø> (ø)
9.00 <0.00> (?)

src/main/java/tutorspet/Main.java
0.00% <ø> (ø)
0.00 <0.00> (?)

src/main/java/tutorspet/MainApp.java
0.00% <ø> (ø)
0.00 <0.00> (?)

src/main/java/tutorspet/commons/core/Config.java
68.18% <ø> (ø)
7.00 <0.00> (?)

.../main/java/tutorspet/commons/core/GuiSettings.java
69.23% <ø> (ø)
5.00 <0.00> (?)

...c/main/java/tutorspet/commons/core/LogsCenter.java
78.37% <ø> (ø)
10.00 <0.00> (?)

src/main/java/tutorspet/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (?)

src/main/java/tutorspet/commons/core/Version.java
82.50% <ø> (ø)
19.00 <0.00> (?)

.../main/java/tutorspet/commons/core/index/Index.java
100.00% <ø> (ø)
10.00 <0.00> (?)

...et/commons/exceptions/DataConversionException.java
100.00% <ø> (ø)
1.00 <0.00> (?)

... and 130 more

Continue to review full report at Codecov.

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


Update testing Python versions

Pull requests:

https://github.com/senzing-garage/sz-sdk-python-grpc/pull/147


Update set-browser-ttl.md

Hi team! We can benefit from warning the customers that setting a high Browser Cache TTL value can be dangers; because after that the cached data can only be purged locally by the website visitor on its device.

في الأربعاء، ١٩ يوليو ٢٠٢٣ ١١:٢٩ ص angelampcosta @.***>
كتب:

Closed #9429 https://github.com/cloudflare/cloudflare-docs/pull/9429.
—
Reply to this email directly, view it on GitHub
https://github.com/cloudflare/cloudflare-docs/pull/9429#event-9859539778,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWV6NCTNECA3SPL2OK7K5MLXQ6LF3ANCNFSM6AAAAAAZK4LLTA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
com>


renamed branch has broken dist info

Summary
renamed branch has broken dist info
Reproduction Steps
n/a
Code
n/a

Expected Results
pycharm does not think it is https://pypi.org/project/selfcord.py/
Actual Results
pycharm thinks it is https://pypi.org/project/selfcord.py/
System Information
Relavant Info:

Python v3.11.0-final
selfcord.py v2.1.0-alpha

selfcord.py metadata: v2.1.0a4372+g550e0cdd

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
this is very annoying for pycharm users

Unsure on how to fix that, I can probably look into it
Funny that the package on PyPi has such a similar README lol


Same memonic give different wallet adress.

I truly need an explanation.
same mnemonic gave different wallet address  on myTonWallet (https://mytonwallet.io/) and this SDk.
I have used same mnemonic here:
let wallet = WalletContractV4.create({ workchain, publicKey: keyPair.publicKey });
console.log(wallet.address);

this address I get is:  EQCpWkFHCsBCBsmSchzF23Uq0tIA1_RtvVkqUiKys7cMa4tV
converted to non-bounce is: UQCpWkFHCsBCBsmSchzF23Uq0tIA1_RtvVkqUiKys7cMa9aQ
but the same mnemonic gave wallet on myTonWallet: UQCY5izJWx2NxdZub41y9JUWE2MXDHVxW_PPU4u38h96uBL4
all these look strange as I wish to programmatically get the balance of my Ton Wallet address and it return 0 even though funded.
can someone explain this?

jut found that my mnemonic has multiple wallet, so how do i programatically select the wallets, since its showing one as default..

In ton blockchain all wallets are in the form of smart contracts so you have to make instance of wallet w5 instead of v4 to get address of w5 version .


[SWE637 FA23]: Fix body handler flaky test caused by invalid cast of string to java list

In this change set, I addressed an issue related to handling form data and generating a response in a specific JSON-like format. The issue was causing a ClassCastException when attempting to cast a java.lang.String to a java.util.List. The change set aims to handle this situation more gracefully and ensure that the response is generated correctly.
Purpose of this PR:
The purpose of this pull request is to address an issue where a java.lang.ClassCastException was occurring in a particular section of the code. The issue was caused by attempting to cast a value to a List without checking its type, leading to a runtime error. This PR aims to fix the issue and improve the code's robustness.
Why the test fails and how to reproduce the test failure:
The test was failing due to a type mismatch when trying to cast a value to a List without checking its type. This occurred when the map contained non-List values. The failure can be reproduced by running the affected test case, which is trying to convert a map of form data into a JSON-like string representation.
Expected results:
The expected result is that the test case should pass without any exceptions, and the resulting resp string should accurately represent the form data in a JSON-like format.
Actual results:
Prior to this fix, the test was failing with a java.lang.ClassCastException when trying to cast a non-List value to a List. This resulted in an incomplete or incorrect resp string.
Description of the fix:
In this PR, the code was modified to first check the type of the value before attempting to cast it to a List. If the value is not a List, it is treated as a single value and converted to a string using toString(). If the value is a List, it is processed by mapping its elements and joining them with a comma to form a valid JSON-like representation. This change ensures that the code handles both List and non-List values correctly, preventing the java.lang.ClassCastException and producing the expected result.

@sunkarnamritishh Thanks a lot for your help.


Uncaught TypeError: winston__WEBPACK_IMPORTED_MODULE_0__.format.errors is not a function

Please tell us about your environment:
I am using winston in an Electron application, based on VueJS framework, typescript class style

winston version?

[ ] winston@2
[x] winston@3 <EMAIL_ADDRESS>

node -v outputs:
v12.4.0
Operating System? (Windows, macOS, or Linux)
CentOS 7
Language? (all | TypeScript X.X | ES6/7 | ES5 | Dart)
TypeScript 3.5.3

What is the problem?
when running, there is an error: logger.ts:9 Uncaught TypeError: winston__WEBPACK_IMPORTED_MODULE_0__.format.errors is not a function
my code:
import { createLogger, format, transports } from 'winston';

const logger = createLogger({
    level: 'info',
    format: format.combine(
        format.timestamp({
            format: 'YYYY-MM-DD HH-mm-ss'
        }),
        format.errors({ stack: true }),
        format.splat(),
        format.json(),
    ),
    defaultMeta: { service: 'my-service-name' },
    transports: [
        new transports.File({ filename: 'test-file-log-err.log', level: 'error' }),
        new transports.File({ filename: 'test-file-log-all.log' })
    ]
});

in vscode, when i type in format.err, vscode will suggest me errors, auto-completion looks good.
typescript compilation has no error.
but when i running this app, there is an error thrown out:

logger.ts:9 Uncaught TypeError: winston__WEBPACK_IMPORTED_MODULE_0__.format.errors is not a function
    at Module../src/logger.ts (logger.ts:9)
    at __webpack_require__ (bootstrap:790)
    at fn (bootstrap:150)
    at Module../node_modules/cache-loader/dist/cjs.js?!./node_modules/babel-loader/lib/index.js!./node_modules/ts-loader/index.js?!./node_modules/cache-loader/dist/cjs.js?!./node_modules/vue-loader/lib/index.js?!./src/components/HelloWorld.vue?vue&type=script&lang=ts& (app.js:1030)
    at __webpack_require__ (bootstrap:790)
    at fn (bootstrap:150)
    at Module../src/components/HelloWorld.vue?vue&type=script&lang=ts& (HelloWorld.vue?5a2d:1)
    at __webpack_require__ (bootstrap:790)
    at fn (bootstrap:150)
    at Module../src/components/HelloWorld.vue (HelloWorld.vue?f177:1)

What do you expect to happen instead?
there shouldn't be an error related to errors function

I think this is related to webpack and your project's configuration. I tested this in a newly created create react app project and I get the same issue. In the same project, if I run node -e "const { format } = require('winston'); console.log(format.errors);" I can see that the function does indeed exist. When running the react app, I get the error in the browser's console.

Related: https://github.com/winstonjs/logform/issues/97

@rhyek today i tried to reproduce this bug, and like you said, format.errors function is not available in electron renderer process even when nodeIntegration is true


MTBenchEvaluator and MMLUEvaluator should be/have static methods

Evaluator objects shouldn't be reused- once we've evaluated a checkpoint or model, we want to save the score and move on to the next. This motivates a reasonable design change, implementing something like:

class MMLUEvaluator(Evaluator):

    def __init__(self):
        # optional empty initialization
        ...

    def run(self):
       ...

    @staticmethod
    def run(self, model, tasks, few_shot, batch):
        ...

@JamesKunstle if we're outputting the evaluation results and saving it to a file or variable in memory, why not reuse the evaluator class?

@JamesKunstle do you have a PR for this already?


Make JsonCodec.Encoder.schemaEncoder and JsonCodec.Decoder.schemaDecoder public

We are planning to switch to zio-schema for codec-related things, and having the ability to convert Schema[A] to JsonEncoder[A] and JsonDecoder[A] would be valuable for us for two reasons:

interoperability with other projects (like Tapir)
the ability to cache encoders instead of computing them on every serialization/deserialization call

@pwliwanow Happy to merge this, can you update with the base branch?

@jdegoes I have just rebased it on top of the main branch. Thanks a lot!


Pull in updated frameworks for new DOS3 specialists

For this trello ticket.
See this PR on the frameworks repo.
Three new specialist roles have been added.

I've pushed a TMP commit on top of this one which will work with the current pre-merge version https://github.com/alphagov/digitalmarketplace-frameworks/pull/531 in case anyone wants to test it.

TMP-ness removed, ready to go.


Add an API to create text with icons

The most notable thing in this PR is the addition of the CustomizableTextContainer class.
This is a subclass of TextFlowContainer that allows easily interspersing icons with regular text.
There are three flavors of use I chose to support:
text.Text = "This [HeartIcon] is a permanent health upgrade";
text.Text = "Press [KeyboardKeyIcon(65)] to advance text."; // <- 65 = keycode for "A"
text.Text = "I can escape [[0]] too!";

var phIndex = text.AddPlaceholder(someComplexDrawable);
text.Text = $"Something [{phIndex}] very special!"

The way that users should use CustomizableTextContainer is by either subclassing it and adding named factories (such as HeartIcon or KeyboardKeyIcon in the example above), essentially like this:
class Custom : CustomizableTextContainer
{
    public Custom()
    {
        AddIconFactory("HeartIcon", <factory-method>);
        AddIconFactory("KeyboardKeyIcon", <factory-method>);
    }
}

Or as in the 3rd use example, by explicitly adding placeholders up-front and referencing them using the provided numerical id.
For a more complete example, check TestCaseTextFlow where everything I described here is actually done.
This is somewhat of an initial draft, as best as I could. There are a few things I'm unhappy with, and if anyone has better solutions than the ones I implemented, I'd be glad to get around to improving those bits.
Notably, there's the overloads of AddIconFactory and the fact that I end up calling factory methods using DynamicInvoke for lack of a more efficient scheme.

this kind of overlaps with #975

#975 looks to be wedded to formatting SpriteText instances. This is supposed to allow for arbitrary icons embedded in the text. Sure, there is definitely some overlap in the way the API is presented, but they are two distinct features.

And right now it is near impossible to have both features at the same time

Right now its impossible to have either feature because neither feature is merged. Itd be trivial to have the parser fallback to any other (including formatting/markdown) implementation for everything that does not satisfy the placeholder syntax.


Set-SPOListItem within a foreach for item in items

Reporting an Issue or Missing Feature
Please confirm what it is that your reporting
format-default : The collection has not been initialized. It has not been requested or the request has not been executed. It may need to be explicitly requested.
+ CategoryInfo          : NotSpecified: (:) [format-default], CollectionNotInitializedException
+ FullyQualifiedErrorId : Microsoft.SharePoint.Client.CollectionNotInitializedException,Microsoft.PowerShell.Commands.FormatDefaultCommand
Expected behavior
Set-SPOListItem in a for each, updates a list item
Actual behavior
get the error above
Steps to reproduce behavior
connect to SPO
Get a list
Get items in that list
foreach item in items set-spolistitem (a vaule for a field)
What version of the PnP-PowerShell Cmdlets are you using?
latest as of post
How did you install the PnP-PowerShell Cmdlets?
installer

this only seems to be happening when i use the -Query option

Hi! Thank you for your report
Can you provide me with the PowerShell snippet so I can try to reproduce what you're doing? Are you trying to access any fields in your foreach loop? If so, do you include those fields in your query? alike
<View><Query><Where><Eq><FieldRef Name='Title'/><Value Type='Text'>Text</Value></Eq></Where></Query><ViewFields><FieldRef Name='Title' /><FieldRef Name='OtherField' /></ViewFields></View>

/Erwin

Hi, yeh it was in a for each, and I was including the fields, ill get you the code I was using ASAP

$url = "https://mytenant.sharepoint.com/

Connect-SPOnline -Url $url  -Credentials MyAccount

$web = Get-SPOWeb

## Write Web to console
$web

############
# AD Creds #
############
$secpasswd = ConvertTo-SecureString "password" -AsPlainText -Force
$mycreds = New-Object System.Management.Automation.PSCredential ("domain\username", $secpasswd)

################
# Process List #
################

## Connect to Subweb
$web = Get-SPOWeb -Identity HRReporting

## Get user tracking list
$list = Get-SPOList -Identity "User Tracking" -Web $web

## Find items created today
$items = Get-SPOListItem -List $list -Web $web -Fields "Created", "Id" , "Author" , "EmployeeNumber"  -Query "<View><Query><Where><Eq><FieldRef Name='Created' /><Value Type='DateTime'><Today /></Value></Eq></Where></Query><ViewFields><FieldRef Name='Created' /><FieldRef Name='Author' /></ViewFields><FieldRef Name='Id' /></ViewFields><FieldRef Name='EmployeeNumber' /></View>"

## for each item get the created by, and connect to AD to find the employee number

foreach ($item in $items)
{
    try
    {
    $email = $item["Author"].Email
    $empID = Get-AdUser -Server MyDC -Credential $mycreds  -Filter 'mail -eq $email' -Properties * | select EmployeeNumber

     Set-SPOListItem -List $list -Web $web -Identity $item.Id -Values @{"EmployeeNumber" = $empID}
    }

    catch
    {
    Write-Host "error"
    }

}

I'm having the same Issue Erwin.  I'm doing very similarly the same thing.  Getting Information from one list based on Data passed through the Foreach Loop, and when I try to update the List item In questions I get the Error the Set-SPOListItem is not a valid Cmdlet.  This worked less than a month ago.
Now I want you all to know while I was typing this, I was trying the code again, and it worked as designed.  I'm not sure what happened in the 3 minutes it took to type this, but It works for me.
Nicholas

Hi Simon,
Sorry for the delay. I actually think this was caused due to an issue with the format definitions which output the list item info. Can you please check again with the latest version of the cmdlets? I tried it here but fail to reproduce it. I will for now close this issue but do not hesitate to reopen it if the issue is still present.

I still see this error. here is what I am trying to do.

I am trying to get the webpart from webpart gallery and update its group to a new group

$webpartnam = linkfile name = .webpart or .dwp
here is my code below
function UpdateWebpartGalleyWithProperGroup($web,$webpartName)
{
$webpartGallery = (Get-PnPContext).Web.Lists.GetByTitle("Web Part Gallery")
Get-PnPProperty -ClientObject $webpartGallery -Property RootFolder
Execute-PnPQuery
$webpartGallery = Get-PnPList -Identity "Web Part Gallery" -Web $web

if($webPartGallery -eq $null)
{
Write-Host("Unable to retrieve Webpartgallery");
}
$query = "{0}" -f $webpartName
$webpart= Get-PnPListItem -List $webPartGallery -Web $web -Fields "Group"  -Query $query
foreach ($item in $webpart)
{
    try
    {

         Set-PnPListItem -List $webPartGallery -Identity $item -Values @{"Group" = "Test Group"} -Web $web
    }

    catch
    {
       Write-Host $_.Exception
    }

}

}
Am I missing something or is it still a problem and not fixed yet?

I'm having the same issue with the newest PnP PowerShell version


Asyncio support?

Will this lib support async await usage?
And, hope complete documentation will come soon.
Thanks for your effort, this lib is pretty good.

If you do add async/await support, I hope it'd be for the general async/await API, so it can support structured concurrency libraries like Trio instead of only the asyncio library.


chore: update GrantBirki/comment action

I saw some deprecation warnings and thought to update this action.

bors r+