Love this!

Would you want to get this into the embark-theme org?

yes! that would be great :D

Repo created here I'd like you to get credit for the commit so if you can open a PR there that would be great!


Add riscv64 & loong64 for goreleaser

@znley I don't think we actually use goreleaser for our release process (a pity I'd say, but that's the way it is now).
believe tha the goreleaser config is a remnant of a past attempt to use goreleaser.

@fredbi
Now  I get it! So what method is used now for release?

I believe that is the tool used by @casualjim  to cut releases: https://github.com/go-openapi/github-release

@fredbi @casualjim
The  github-release seems to be used for upload binaries and gorelease is used for building, is right?


Add Stratos chain

Hey! Thanks for this. We recommend using the following to add chains on the fly!
import { suggestChain } from "@cosmsnap/snapper";
let chain: Chain = {};
await suggestChain(chain);

or
const chainInfo: ChainInfo = {}
window.cosmos.experimentalSuggestChain(chainInfo)

Let us know if you need any help :)

@schnetzlerjoe thank you for reply. As I understood, this is when a dapp wantd to add a chain to cosmos snap, but we want to add just our chain to be in the list of you snap app (as we are not building dapp, we adding a network itself). Is it possible to add our chain just in the defined list which I provided in PR?

So the list you added your chain to is just the list for initialization. The many users who already downloaded the snap would not have access to your chain just by adding it there. Only the new downloaders.
This is why we suggest doing so the way above!
However, with Dynamic UI in the extension rolling out we will make it so you can one click install any chain in the chain registry. Maybe that will help!

@schnetzlerjoe so as I understand if a dapp using cosmos snap and wants to have our network, they should use the snippet above right?
Dynamic UI will it be a sort of https://chains.keplr.app/ ?

@BoThe1K yep exactly! To both your questions.

@schnetzlerjoe could our chain also be a part of list initialization (even it will be for the new users only)? And later when Dynamic UI will be available, we also want to apply there.


Proposal: Doccomments

OCaml supports doccomments, which could be added here too.
For example:
(** Generation of fresh Diffie-Hellman keys. Returns a pair (kpriv,kpub) *)
letfun DHKeygen() =
new s:scalar;
(s,dh(s,G))
.

Expectation of what vscode then should display:

The doccomment should also be highlighted appropriately by the TextMate grammar.

Resolved in #34


i want a demo app to know the real experence but i get 404

Tnx for report some url is wrong in docs website i will fix them soon. Also you can send my wrong pages and i can do this fast.
Until i fix wrong links you can use:
Download demo app from release
https://github.com/WinUICommunity/WinUICommunity/releases
Or explore source code
https://github.com/WinUICommunity/WinUICommunity/tree/main/Samples

done


fix encoding issue for feature table metadata

Fixes #751

Thanks for the pull request @joseph-kaile!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!
Maintainers, here's how to check for a submitted CLA.

:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium for Unreal Samples works.

Looks good @joseph-kaile. Please update CHANGES.md and I'll merge this.


Fix changed download url

I can not download from previous DOWNLOAD_URI (fastdl.mongo). Instead, it appears that download links are valid on this new URI (downloads.mongo).
/!\ I did not test it for every platform. still, it should works for win.

Downloading from fastdl.mongodb.org with your changes from https://github.com/winfinit/mongodb-download/pull/20 works for me on Debian 9. Using this new http://downloads.mongodb.org URL instead of https://fastdl.mongodb.org triggers this error for me :


Premature narrowing causing type error

TypeScript Version:  3.6
Search Terms: narrowing generic
When inferring/extracting the array type from an inferred rest arg, it seems that the rest arg is widened which causes the widened type to get passed down to the type and may be too wide to match the declared type.
In the repro case below, the rest type is being inferred as [number] instead of [1] this causes the ValueFromArray to return number and when attempting to assign number to Num fails since number is much larger than Num
Code
class Observable<T> {
  value!: T
}

type ValueFromArray<T extends any[]> = T extends Array<infer U> ? U : never

declare function of<A extends Array<any>>(
  ...args: A
): Observable<ValueFromArray<A>>;

type Num = 1|2;

const working: Observable<Num> = of(1 as const); // Works
const broken: Observable<Num> = of(1); // Error: Type 'Observable<number>' is not assignable to type 'Observable<Num>'.

Expected behavior:
const broken: Observable<Num> = of(1) should compile correctly
Actual behavior:
Type error: Type 'Observable<number>' is not assignable to type 'Observable<Num>'.
Playground Link
Related Issues: https://github.com/ReactiveX/rxjs/pull/5027

I think at one point we were going to make the form as written illegal since it doesn't really make any sense (the rest arg type is not truly specializable)

Please don't =(
Sometimes, we do want it to infer a tuple type for the type parameter! The order of the input arguments is important

While that works, it would be nice to get the specific args from the invocation, in RxJS this would allow the piped map() to have a callback type of
of(true, 'test').pipe(
  map(callback)
)

// callback has type
type Callback = <T>(elem: Boolean, index: 0): T | <T>(elem: string, index: 1): T

It would also allow us to keep track of how large the Observable is so that something like
of(1, 2).pipe(take(4))

Can fail at compile time.
Is there a way to get that behavior and not have the widening issue mentioned above?

This isn't really a question and is more of a bug report / feature request (or removal of the "feature" request) since at the moment having rest args extending any[] is supported but isn't properly inferred when extracting the Array's generic

Sometimes, we do want it to infer a tuple type for the type parameter! The order of the input arguments is important.

Agreed.  It’s also the only way to capture parameter names when writing a function whose sole purpose is to forward a call, a la Function.prototype.call.

@RyanCavanaugh I guess I'm confused a little by your responses in here. This next release of RxJS is a bid to try to fix all of our current typings woes for our users. I really want to get this right.
Would it be possible to collaborate directly with the TS team to shore this up and make sure we don't have regressions when there are new versions of TypeScript? I think that RxJS just happens to trip all of your edge cases. Google catches a lot of those, but the feedback loop is slow:  1) TypeScript releases new major, 2) Google syncs TS into google3 and sees breaking things 3) RxJS team hears about it and investigates 4) Feedback given to TS team.
It would be nice to get things straightened out together if possible, even in an advisory capacity, and then work on something where maybe TS team can run dtslint tests that match features we're using so we prevent regressions or at least know more about breaking changes up front.

Somewhat related question, how big/widely used must a library be before they receive special attention?

@benlesh RxJS is already a part of our user test suite, so we're basically aware of the changes at the point that they happen. Conversely, our nightlies are available on a nightly basis, so if you'd like to field things from your side, that avenue is available. A bug report with "xxx change breaks RxJs in yyyy scenario" is something we'd treat as very actionable as long as the relevant definitions from the .d.ts were inlined so we can comprehend what's happening without hours of digging.
For this specific case, we could work from a more demonstrative example to figure out what your use cases are and determine the best path going forward. Striking the balance between a representative repro and something simplified to the point that it looks like a question/misunderstanding is difficult, so I appreciate the patience and engagement on that front.

how big/widely used must a library be before they receive special attention?

"Special attention" is probably overly broad, but any notable project that is something we can build as part of our user test suite is welcome to add themselves. The line for inclusion there is basically "yeah, we've heard of that one".
When we do encounter a potential break in those suites, the question is more about representativeness (i.e. is this form of break going to be commonly observed) than popularity, though popularity itself is a de facto form of representation due to usage.

For this specific case, we could work from a more demonstrative example to figure out what your use cases are and determine the best path going forward.

RxJS includes a number of APIs - of, startWith and endWith - that can be used to declare observables that emit multiple values of different types. One use case would be streams that emit multiple Redux actions:
// A pretend Observable
class Observable<T> {
  constructor(public value: T) {}
}

const init = { type: "init" as const };
const nav = { type: "navigate" as const };

If of is declared like this, actions of different types cannot be specified (see https://github.com/microsoft/TypeScript/issues/30824):
declare function of<T>(...args: T[]): Observable<T>;
const actions = of(init, nav); // ERROR: '"navigate"' is not assignable to type '"init"'

So of is instead declared like this:
type ValueFromArray<T extends any[]> = T extends Array<infer U> ? U : never;
declare function of<T extends any[]>(
  ...args: T
): Observable<ValueFromArray<T>>;
const actions = of(init, nav);

Which works fine, except in the situation mentioned above, in which literal arguments are widened.
For me, there are two issues:

the widening of literal arguments - which is what this issue was initially about; and
whether or not "the form as written" (in the second of declaration in this comment) is something that is encouraged, supported or is likely to change.

The latter of these is what concerns me most, as all of the N-args (i.e. a single, rest-parameter signature instead of multiple overload signatures, each with a different number of parameters) support in RxJS is predicated upon this form.

Seems this was closed prematurely, it was a little heavier than a simple question, and I don't see a resolution. cc/ @RyanCavanaugh
We're still uncertain what the recommended solution should be for these scenarios.


After running test_pgn.py i am getting this error.

@minar09 I am not getting output can you help me.

@minar09 I am able to generate image-parse using this https://github.com/Engineering-Course/LIP_SSL.
This is how the image-parse looks.

This is the TOM tryon result

I observed one thing whenever I use the updated segmentation that is the image-parse-new file the final result is better

For this updated segmentation

The final try result looks like this below,

Hi @Sumukh0512 , yes, that is one of the contributions of CP-VTON+. CP-VTON used segmentation which was missing the neck region label, so we added the label and got better results. Now that, PGN can generate better segmentation with neck-torso label, so we recommend generating segmentation with PGN. It seems you have CUDA/GPU issue. If you don't have a GPU, either you can remove the .cuda from the codes, or you can try to utilize colab for running with GPU.

@minar09  thank you so much, I got it.


feat(curriculum): add book inventory lab

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

I went through all the steps.
The project is looking great 🎉
Would the demo be included in this lab for campers?

I think we should add one user story between the second and third that says something like "Your table should have at least four rows, the first for the column headings and the rest filled with book information."

Done :+1:


Move "ImGui's files" from right in the middle, to a dear-imgui/ git subtree

Hello,
In this pull request, I have moved the Dear ImGui's files form "in the middle of all AllenDang's files" to a new completely independent and self-contained dear-imgui/ directory.
(this is a step in addition to the split you already done this year, by splitting this repo from https://github.com/AllenDang/giu)
This new dear-imgui/ directory is a git subtree directory, which means that :

you can super easily pull latest "Dear ImGui" into your dear-imgui/ subdirectory (git subtree pull)
you can super easily "extract" local modification to "Dear ImGui" code, if you were to propose changes to (C++) Dear ImGui, and thus super easily create a pull request to "Dear ImGui" code.

Hint : I needed that because, indeed, I have some bug in "Dear ImGui" base code (https://github.com/ocornut/imgui/pull/3795) that:

occurs in a Go program, which uses https://github.com/AllenDang/giu
I was willing to identify the bug and fix it in "Dear ImGui" imgui.cpp
I do not wanted to re-create a sample app in C++
I wanted to fix https://github.com/AllenDang/giu (or rather fix it's dependency), which meant fixing the imgui.cpp code (official "Dear ImGui") in https://github.com/AllenDang/imgui-go
I wanted to be able to "push upstream" the modification to imgui.cpp, to Dear ImGui (git subtree push)

(which mean "git subptree push" the dear-imgui/ folder to my own ocornut/imgui's fork, and create a pull request from there)

So I forked your github.com/AllenDang/imgui-go repo, and created a folays.imgui-v1.85 branch.
Here are the commands that I did to create it :
grep IMGUI_VERSION imgui.h
-> 1.85

git checkout -b folays.imgui-v1.85 # because you are currently based on v1.85

git subtree add  --prefix=dear-imgui https://github.com/ocornut/imgui.git v1.85 --squash
git subtree pull --prefix=dear-imgui https://github.com/ocornut/imgui.git v1.85 --squash # no-op
-> # From ssh://github.com/ocornut/imgui
   #  * tag                 v1.85      -> FETCH_HEAD
   # Subtree is already at commit 55d35d8387c15bf0cfd71861df67af8cfbda7456.

diff -ru dear-imgui/ ./ | grep -r 'Only in dear-imgui/'
-> (standard input):Only in dear-imgui/: .editorconfig
   (standard input):Only in dear-imgui/: .gitattributes
   (standard input):Only in dear-imgui/: .github
   (standard input):Only in dear-imgui/: backends
   (standard input):Only in dear-imgui/: docs
   (standard input):Only in dear-imgui/: examples

diff -ru dear-imgui/ ./ | grep -- ---
-> --- dear-imgui/.gitignore	2021-12-23 08:30:27.000000000 +0100
   --- dear-imgui/imgui.cpp	2021-12-23 08:30:27.000000000 +0100
   --- dear-imgui/imgui.h	2021-12-23 08:30:27.000000000 +0100
   --- dear-imgui/imgui_internal.h	2021-12-23 08:30:27.000000000 +0100
   --- dear-imgui/imgui_widgets.cpp	2021-12-23 08:30:27.000000000 +0100

diff -ru dear-imgui/ ./ | grep -v ^Only > ../PATCH_imgui-go_diff_to_imgui_v1.85.patch

# remove all ROOT/ files which are already in dear-imgui/
( cd dear-imgui/ && find . -type f -print0 ) | xargs -0 -n1 -- git rm

git checkout HEAD LICENSE.txt .gitignore # restore some stuff

# below : do not "symlink" but rather "#include", because Go Modules (go mod download) IGNORE symlinks.
for name in $(cd dear-imgui/ && ls -1f *.{cpp,h}); do printf '#include "dear-imgui/%s"\n' $name > $name ; done
for name in $(cd dear-imgui/ && ls -1f *.{cpp,h}); do git add $name ; done

# re-add local modifications to dear-imgui/ (modifications exclusives to imgui-go)
patch -d ./dear-imgui/ -p1 < ../PATCH_imgui-go_diff_to_imgui_v1.85.patch

git checkout dear-imgui/.gitignore # restore probably no-more-wanted local modifications
git add dear-imgui/

git commit # ...

I also  git add a a new file, /git_pull_dear-imgui.sh which you can use to "pull" new Dear ImGui version from upstream.
Here is what I could do if I wanted to update to Dear ImGui v1.86 (released... yesterday?)
git checkout folays.imgui-v1.85 # fetch this very branch from this pull request

git checkout -b folays.imgui-v1.86
sh ./git_pull_dear-imgui.sh v1.86 # "git subtree pull" from Dear ImGui v1.86

go clean -cache
go build ./ # check if it compiles

# Hint 1) It does not compile, there is some SMALL changes in ImGui's (`imgui.h`) ImGuiListClipper struct,
#  (1 struct field removed, 2 or 3 added) which need to be reflected in the wrapper ListClipper.{cpp,h},
#  and while I could update those manually, I feel that you have some sort of procedure to update them automatically?

# Hint 2) For whatever reason, it works even if it SHOULD NOT, if you did not "go clean -cache". Dunno why...

# ... Fix ListClipper.{cpp,h} so that it compiles

git commit
git push origin folays.imgui-v1.86
# or git tag whatever tagname you use to be compliant w.r.t. Go Modules.

Also, I think that you are substantially derived from inkyblackness/imgui-go, but I could not figure your thought process.
I'm somewhat curious :

I guess that you expose an API (the Go API thus exposed) probably adapted to https://github.com/AllenDang/giu needs
Way more curious : You adopted a different way to generate the pure "C++ to C" wrappers ?

Full Swig automation ?
Swig was used only as a "working base", and then from now on, you do all updates to those wrappers manually ?

Hope you understand that the purpose of this change is to ease :

pulling from Dear ImGui upstream (which was not really an hassle)
more importantly : being able to fork your github.com/AllenDang/imgui-go repo, fix imgui.cpp directly, while being able to still track and pull upstream changes, and independently also being able to "extract" local changes (with "git subtree push") to propose patches (via a pull request) to Dear ImGui (ocornut).

Regards,

Another example, after that, you could :
git remote add upstream_imgui https://github.com/ocornut/imgui.git

# don't fetch tags (--no-tags), but yes, fetch them, but in a specifc refspec
# (to not clutter your own repos tags, with the tags of a 3rd party repos)
git fetch --no-tags upstream_imgui '+refs/tags/*:refs/tags_upstream_imgui/*'

$ cat .git/refs/tags_upstream_imgui/v1.85
55d35d8387c15bf0cfd71861df67af8cfbda7456 # <- commit of Dear ImGui v1.85

git diff refs/tags_upstream_imgui/v1.85 folays.imgui-v1.85:dear-imgui/
# OR :
# - git diff refs/tags_upstream_imgui/v1.85 HEAD:dear-imgui/ # if your HEAD is "near" v1.85

Which would display ONLY you local changes to officlal Dear ImGui sources.
In this case, the git diff would reveal :
$ git diff refs/tags_upstream_imgui/v1.85 folays.imgui-v1.85:dear-imgui/ |grep -vE '^(---|\+\+\+) ' | cut -c1 | sort | uniq -c
  48
  55 +
   2 -
   8 @
   4 d
   4 i

Your 55 lines added, 2 removed, on 4 files (8 sections) compared from Dear ImGui sources.
Regards,


The document generation using the third party plug-in is not successful

Search terms

Typedoc-Webpack-Plugin
95% emitting unnamed compat pluginWarning: No compiler options set. This likely means that TypeDoc did not find your tsconfig.json. Generated documentation will probably be empty.
95% emitting unnamed compat pluginWarning: No compiler options set. This likely means that TypeDoc did not find your tsconfig.json. Generated documentation will probably be empty.
Expected Behavior

Generate API documents
Actual Behavior

No API document was generated
Steps to reproduce the bug

Environment

Typedoc version:0.20.23
TypeScript version: 3.9.7
Node.js version:12.16.1
OS:Windows10

The webpack plugin isn't maintained here, and looking at the source, it hasn't been updated in years... I'm surprised it gets as close as it does.
I should probably go ahead and remove this plugin link from typedoc.org since it is broken. I recommend running typedoc separately with the CLI.


Use new introspection engine binary

CLI should use the new introspection engine binary.
@mavilein can provide more information when upload to S3 is done via https://github.com/prisma/prisma-engine/issues/26

Upload is done.


Ineffectual assignment (possible bug?)

Hey @cznic -- github.com/gordonklaus/ineffassign flagged this line as an ineffectual assignment: https://github.com/cznic/b/blob/6955404bf550e1eae1bf83121739078b027f4547/btree.go#L304
I haven't examined this closely enough to figure out whether the line can just be deleted or whether it's a bug.

Thanks for the report!

Thanks for the quick fix!


update(tests/falco): cover new skip-if-unknown-filter semantics

Follow-up to https://github.com/falcosecurity/falco/pull/2749 for increasing coverage on the improved semantics of skip-if-unknown-filter.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 9a1ef4a355fd72d4194860d560ca0b0e052c2ba7

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


add Never type

closes #20
currently broken due to https://github.com/python/mypy/issues/11903

https://github.com/python-poetry/poetry/issues/5002


Set a suitable background-color in light theme

What is your color theme? the default light?

Atom One Light Theme @Gerrnperl

I didn't reassign the background color due to differences in different color themes.
In new version v0.5.2 two items in configuration: outline-map.color : visibleRange and focusingItem have been added
Now you can specify the background-color.


Relax shader strictness in RELEASE mode

Disables strictness and warnings as errors for custom pixel shaders in
RELEASE. Windows terminal is not telling the user why the shader won't
compile which makes it very frustrating for the shader hacker.
After trying the recent preview none of my shaders loaded anymore in
Windows Terminal Preview which made me very sad. I had no idea what was
wrong with them. After cloning the git repo, building it, fighting an
issue that prevent DEBUG SDK from being used I finally was able to
identify some issues that were blocking my shaders.

error X3556: integer modulus may be much slower, try using uints if possible.
error X4000: use of potentially uninitialized variable (rayCylinder)

While the first one is a good warning I don't think it is an error and
the tools I use didn't flag it so was hard to know.
The second one I was staring at the code and was unable to identify what
exactly was causing the issues, I fumbled with the code a few times and
just felt the fun drain away.
IMHO: I want it to be fun to develop shaders for windows terminal.
Fighting invisible errors are not fun. I am not after building
production shaders for Windows Terminal, I want some cool effects. So
while I am as a .NET developer always runs with Warning as errors I
don't think it's the right option here. Especially since Windows
Terminal doesn't tell what is the problem.
However, I understand if the shaders you ship with Windows Terminal
should be free of errors and silly mistakes, so I kept the stricter
setting in DEBUG mode.
Validation Steps Performed
Loaded Windows Terminal in RELEASE and DEBUG mode and validated that
RELEASE mode had reduced strictness but DEBUG retained the previous more
restrictive mode.

(Thanks for doing this, and sorry for the super opaque error message  to begin with! :D)

(Thanks for doing this, and sorry for the super opaque error message to begin with! :D)

No problems at all. I was a bit sad but your comment on twitter about strictness of the compiler suggested the solution and seemed like a simple enough thing to suggest.

Current status

I'm sorry for causing this issue. 😓
You can fix the formatting error by running the following with PowerShell from the root of the repository:
Import-Module -Force .\tools\OpenConsole.psm1
Invoke-CodeFormat -IgnoreXaml

Oh, sorry, I just pushed a fix! :D

@msftbot merge this in 1 minute


Make @zkfs/examples run against the latest (@next) deployed packages

Since stackblitz doesnt work well with npm workspaces, we can't run @zkfs/examples in stackblitz while using locally built and linked @zkfs/contract-api.
Stackblitz shows @zkfs/examples from the develop branch, but relies on the @next release of our packages. @next packages are released every time a push to develop is made and there are code changes to deploy.
This means there is room for inconsistencies across our CI/CD pipeline, if a deployment of a @next package fails, then stackblitz will point to the latest develop examples but will run older @next packages.
We should make the CI/CD pipeline run @zkfs/examples after publishing @next packages, so that we can be aware of the working status of the stackblitz example.
It won't unpublish/fix the underlying inconsistency, but it will alert us as developers to fix the actual issue with the pipeline.

After recent updates, develop is published as @latest (+ canary) and the stackblitz example points to that release as well. However the underlying consistency issue still persists.


[BUG] ArmClient.GetManagementGroups().ExistsAsync() fails unexpectedly with incorrect error message

Library name and version
Azure.Core <IP_ADDRESS>
Azure.ResourceManager <IP_ADDRESS>
Describe the bug
Calling ArmClient.GetManagementGroups().ExistsAsync() returns a 403 Forbidden error when accessing a scope that the relevant credentials have read permissions on. In addition, the given scope in the error message is malformed.
Expected behavior
The method properly considers the scope and returns true or false appropriately.
Actual behavior
The following code (called within a ASP.NET6 controller action):
logger.LogInformation(scopeIdentifier);
result.Exists = await _armClient.GetManagementGroups().ExistsAsync(scopeIdentifier);

Produces this output and exception (trimmed down from the full output):
...
info: application.Controllers.AzureManagementController[0]
      /providers/Microsoft.Management/managementGroups/NonProduction
fail: application.Controllers.AzureManagementController[0]
      Failed Azure request - most likely the given scope is valid, but the resource it points to does not exist.
      Azure.RequestFailedException: The client '<my AAD account>' with object id '<my AAD object ID>' does not have authorization to perform action 'Microsoft.Management/managementGroups/Microsoft.Management/NonProduction/read' over scope '/providers/Microsoft.Management/managementGroups/providers/Microsoft.Management/managementGroups' or the scope is invalid. If access was recently granted, please refresh your credentials.
      Status: 403 (Forbidden)
      ErrorCode: AuthorizationFailed
...

Note that the scope in the error message - /providers/Microsoft.Management/managementGroups/providers/Microsoft.Management/managementGroups - is two copies of the first three segments of the given scope argument appended together. In my opinion, this is not a reasonable scope to deduce from the argument value (see log output just before the error message for comparison).
The type of scopeIdentifier is Azure.Core.ResourceIdentifier, so maybe something is going on with its implicit string conversion operator in this scenario.
az account management-group show --name NonProduction works just fine with the same credentials:

Reproduction Steps
Repro code is above.
Environment
VS code version information:

Output of dotnet --info:
$ dotnet --info
.NET SDK (reflecting any global.json):
 Version:   6.0.400
 Commit:    7771abd614

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.19044
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\6.0.400\

global.json file:
  Not found

Host:
  Version:      6.0.8
  Architecture: x64
  Commit:       55fb7ef977

.NET SDKs installed:
  2.1.818 [C:\Program Files\dotnet\sdk]
  3.1.418 [C:\Program Files\dotnet\sdk]
  5.0.411 [C:\Program Files\dotnet\sdk]
  6.0.400 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.All 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.24 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 3.1.28 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.24 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 3.1.28 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.24 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 3.1.28 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.8 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Download .NET:
  https://aka.ms/dotnet-download

Learn about .NET Runtimes and SDKs:
  https://aka.ms/dotnet/runtimes-sdk-info

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Extensions:0.13063328,Storage:0.091699325,Event Grid:0.08414881'

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

To add some additional context - to workaround the above error, I am using this call:
var existsTest = await _armClient.GetManagementGroupResource(scopeIdentifier).GetAsync();
result.Exists = existsTest != null;

This produces the same exception and 403 Forbidden error if the management group does not exist, but if it does exist, the call completes successfully. Again, same credentials and everything.

Okay, so after running into a similar issue with a different call and finally finding and reading this documentation, this appears to be user error. I guess I got confused about which methods/API calls take full resource identifiers and which ones just take the id because they already have the scope or can infer it.


No archived logs found at /opt/log

Hi,
I get the error No archived logs found at /opt/log. I do noticed, that the logs are not contained in the container, because $ docker-compose exec goaccess ls /opt/log do not show me any log file. I have mapped the volume in my docker-compose.yml like this
 volumes:
        - ./data/logs:/opt/log

On my host, there are plenty of files in data/logs

@leon1995 are there any other errors shown? Can you send me a screenshot of the output when running the container?
Thanks.

no, there arent any errors. here is the output with debug True:
goaccess     |
goaccess     | GOAN v1.1.8
goaccess     |
goaccess     |
goaccess     | NGINX SETUP...
goaccess     |
goaccess     | NGINX BASIC AUTH
goaccess     | -------------------------------
goaccess     | None
goaccess     |
goaccess     |
goaccess     | NPM INSTANCE SETTING UP...
goaccess     |
goaccess     | LOADING NPM PROXY LOGS
goaccess     | -------------------------------
goaccess     |
goaccess     |  Adding proxy logs...
goaccess     |  Found (0) proxy logs...
goaccess     |
goaccess     |  SKIP ARCHIVED LOGS
goaccess     |  -------------------------------
goaccess     |  FALSE
goaccess     |  No archived logs found at /opt/log...
goaccess     |
goaccess     | EXCLUDE IPS
goaccess     | -------------------------------
goaccess     | <IP_ADDRESS>
goaccess     |
goaccess     | Setting GeoIP Database
goaccess     | -------------------------------
goaccess     | DEFAULT
goaccess     |
goaccess     | DEBUG
goaccess     | -------------------------------
goaccess     | OFF
goaccess     |
goaccess     | RUN NPM GOACCESS
goaccess     |
WebSocket server ready to accept new client connections

and I can understand, there he cannot find any logs, because it seems that they arent copied to the docker container. But I dont know why, because I set up the directory mapping...

@leon1995 you are correct the logs are not copied into the container. Can you try referencing the entire directory rather than just ./data/logs ?

What do you mean exactly?

/root/something/npm/data/logs

not working. I know that ./data/... works, because I use it also on other services

also it says that
goaccess     | DEBUG
goaccess     | -------------------------------
goaccess     | OFF

but I have DEBUG=True set

can you provide your docker compose file so I can look that over?

version: '3'
services:
  app:
    image: 'baudneo/nginx-proxy-manager:latest'
    container_name: app
    healthcheck:
      test: ["CMD", "/bin/check-health"]
      interval: 10s
      timeout: 3s
    ports:
      - '80:80'
      - '81:81'
      - '443:443'
    environment:
      # crowdsec censored stuff
      DB_MYSQL_HOST: "db"
      DB_MYSQL_PORT: 3306
      DB_MYSQL_USER: "npm"
      DB_MYSQL_PASSWORD: "npm"
      DB_MYSQL_NAME: "npm"
    volumes:
      - ./conf.d/:/etc/nginx/conf.d/
      - ./data:/data
      - ./letsencrypt:/etc/letsencrypt
      - ./logs:/var/log/letsencrypt/
    restart: always

  db:
    image: 'jc21/mariadb-aria:latest'
    container_name: db
    environment:
      MYSQL_ROOT_PASSWORD: 'npm'
      MYSQL_DATABASE: 'npm'
      MYSQL_USER: 'npm'
      MYSQL_PASSWORD: 'npm'
    volumes:
      - ./data/mysql:/var/lib/mysql
    restart: always

# crowdsec stuff

  goaccess:
    image: xavierh/goaccess-for-nginxproxymanager:latest
    container_name: goaccess
    restart: always
    environment:
        - TZ=Europe/Berlin
        - SKIP_ARCHIVED_LOGS=False #optional
        - DEBUG=True #optional
        - EXCLUDE_IPS=<IP_ADDRESS> #optional - comma delimited list
        - LOG_TYPE=NPM #optional - more information below
    ports:
        - '7880:7880'
    volumes:
        - ./data/logs:/opt/log/

Looks right.
Maybe try adding this to goaccess but that shouldn't matter if the files are already there.
depends_on:
- app

Yes there are 301 files in ./data/logs...
I added the depends_on but it didnt work

Having a similar issue.  If I jump into the containers console I can tail output the log files bind mounted into /opt/logs.  Haven't figured out anything else yet.

@leon1995 or @Sperryfreak01 any luck with this?

I'll be honest I forgot what this was and what I was trying to do.
On Tue, Feb 14, 2023, 7:56 PM Xavier @.***> wrote:

@leon1995 https://github.com/leon1995 or @Sperryfreak01
https://github.com/Sperryfreak01 any luck with this?
—
Reply to this email directly, view it on GitHub
https://github.com/xavier-hernandez/goaccess-for-nginxproxymanager/issues/106#issuecomment-1430717761,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJDFI6ZJSSZN64JEM3CTGTWXRHYNANCNFSM6AAAAAARATAQXE
.
You are receiving this because you were mentioned.Message ID:
<xavier-hernandez/goaccess-for-nginxproxymanager/issues/106/1430717761@
github.com>

No... but it would be nice if there was a solution...

@leon1995 what I've somewhat learned is that sometimes GoAccess silently discards logs starting with errors. Can you send me a sample of the logs? Post them on PasteBin or something similar.

I habe updated the docker container and now the log files are found and everything looks fine!


STONEBLD-1085 Merge all build steps into a single step

This provides a significant memory saving, as the pipeline pods now require less that 50% of the memory they required before.
A standard build was consuming close to 3G before this change, now it is slightly over 1G.

/test jvm-build-service-in-repo-apicurio-e2e

This has significantly better memory usage but fails on JDK7 builds as JDK7 can't run the preprocessor/deploy tasks.

So this breaks the JDK7 builds, because JDK7 builds are still UBI7.
I am not really sure how to address that, JDK7 is a bit of a nightmare.


1.3.0: --dburl default:...

Hello,
I'm trying to automate the redeployment of modoba.
When using the new modoboa-admin.py parameters, especially the new
--dburl default:postgres://USER:PASS@HOST:PORT/DB
and
--dburl amavis:postgres://USER:PASS@HOST:PORT/DB
That should be right, shouldn't it?
When running the modoba-admin.py deploy dir and the parameters above, I get:
"You must define a 'default' database" Error.
What's up?

The --dburl option must appear only once but it accepts multiple arguments :
--dburl defaut:<...> amavis:<...>
Am I clear ?

Thank you +1:
Could you place a note to the docs, because it was not clear?

I will.


Neo4j tests take a long time on TC

Should be fixed with https://github.com/opencypher/cypher-for-apache-spark/pull/558
Oddly enough the time is still higher with 44minutes.
The build of this PR is from an AWS m3.large machine. A previous 31 min build for PR https://github.com/opencypher/cypher-for-apache-spark/pull/552 was run on an AWS m3.2xlarge machine from a pool that doesn't seem to exist anymore. It's plausible that it could take 50% longer with only ~25% of the resources.
Given that, I suspect that the regression is probably fixed, but it's hard to verify with different worker pools building our PRs.

Closing, creating a separate issue for TC worker pools.


Exceptions can not be pickled

I've been using betfairlightweight in conjunction with multiprocessing (via ProcessPoolExecutor), and occasionally an uncaught exception will bubble up to the parent process, usually resulting in:
...
  File "/home/test/betfairlightweight/exceptions.py", line 93
    error_data = response.get('error')
AttributeError: 'str' object has no attribute 'get'

I just assumed this was some obscure betfairlightweight issue, but turns out the underlying problem is just that the custom exceptions can't be pickled:
exc = None
try:
    trading.account.list_currency_rates(from_currency='XYZ')
except Exception as e:
    exc = e

import pickle
pickle.loads(pickle.dumps(exc))
# => AttributeError: 'str' object has no attribute 'get'

The current design of all the exception classes is to format a message from arbitrary objects (eg requests response objects etc), but all that gets stored is the final message. You can see some of the possible solutions here - https://stackoverflow.com/questions/16244923/how-to-make-a-custom-exception-class-with-multiple-init-args-pickleable.
I've tended to go the route of passing the unaltered arguments directly to super() so that it can be re-constituted easily:
class MyException(Exception):
    def __init__(self, arg1, arg2):
        super(MyException, self).__init__(arg1, arg2)
        self.arg1 = arg1
        self.arg2 = arg2

The message can still be formatted afterwards, and would need a custom __repr__, or perhaps just build on the fly in each subclass' __repr__ method.  An upshot to this is not having to grovel around in the formatted string when more detail is needed.
Alternatively it is possible to address by using a custom __reduce__ method in the base class, but it needs to do some contortions to avoid calling constructors:
class BetfairError:
    def __reduce__(self):
        tup = super(BetfairError, self).__reduce__()
        return (tup[0].__new__, (tup[0],), {'args': tup[1]})

Happy to submit a pull request one way or the other - what is your thinking?

Not something I have encountered before but happy to accept a PR on what works best.

Adding __reduce__ is in some ways the least risky - shouldn't affect anything except the pickle/unpickle use case which didn't work before.
That said my preference is probably to go the other route as there's times when it's useful to get underlying error data (eg errorCode from a ListenerError etc). I have an except clause currently doing if 'Exception: ConnectionError' in str(e) which is icky. That shouldn't be a breaking change in that str() and repr() will remain the same, there'll just be more some attributes. The only other difference would be that BetfairError.args won't contain a tuple with the formatted message. If that's alright I can look to put it together.


Is there a list of supported sites?

Which domains or webpages can be scraped?

from the code -- these items cannot be scraped.
BAD_CHUNKS = ['careers', 'contact', 'about', 'faq', 'terms', 'privacy',
'advert', 'preferences', 'feedback', 'info', 'browse', 'howto',
'account', 'subscribe', 'donate', 'shop', 'admin']
BAD_DOMAINS = ['amazon', 'doubleclick', 'twitter']

I'm surprised to see I haven't found a news site that this can't scrape. It's met my needs anyways.

Yes, Newspaper has a lot of flexibility.  I recently started putting together a detailed Newspaper3k usage document that I publicly share. The document is available here: https://github.com/johnbumgarner/newspaper3_usage_overview. Please let me know if you see anything that is missing or needs more clarification.

u can't scrape javascript rendered pages

u can't scrape javascript rendered pages

What javascript rendered site are you trying to scrape?


Sync data from Aurora Serverless to Elasticsearch

** Which Category is your question related to? **
data sources, @searchable graphql transform
** What AWS Services are you utilizing? **
Aurora Serverless, ElasticSearch
I would like to sync data from Aurora Serverless to Elasticsearch for text based search by users. It seems like Dynamodb does this automatically, but how would this be done with Aurora Serverless?
Thanks

Amplify CLI does not have support for ElasticSearch support for Aurora Serverless. In case of DynamoDB, the data modelling and creation of the tables are managed Amplify CLI as defibed by the GraphQL Schema. When you import Aurora Serverless, the schema is generated based on the Database schema. CLI does not have support for any directives for the imported data source.
I am going to mark this as a feature request, and add it to our backlog


It plays really bad with vim-latexsuite

It seems that this bundle really disables all normal mode key bindings in vim-latexsuite, which makes it half less useful. I don't know which part does this, though

When you run the haskell-vim-now installer it backs up your old vim config and puts a totally new one in its place. This new vim config won't include the tex suite, but here's how you can add it back.
Create a file called ~/.vim.local/plugins.vim containing
Plug 'gerw/vim-latex-suite'

Then start vim and run :PlugInstall. You may have to restart vim, but you should be able to use the tex suite again.

No, I have it from distro package, and things like <++> works. Just any normal mode features fail (like\ll). If I go with vim -u /etc/vimrc, those features work again. So it has to be something about vim-now

I can think of two more possible causes:

/etc/vimrc contains mappings which are not being set when using another vimrc. Solution: ln -s /etc/vimrc ~/.vimrc.local. Haskell vim now will load whatever is in ~/.vimrc.local after it has finished loading itself.
HVN changes the leader key to ,. So the command might be ,ll rather than \ll. Again this might get solved by linking the vimrc.

We'll add leader key override soon. So if you need that, wait a bit please.

latex-suite has both , leading and \ leading commands. For example, ,li is insert the list environment, \ll is compile.

@carldong or @SX91 are either of you interested in sending a pull request to do leader override?


Added Codecov to github actions

Made the following changes:
Added coverage.xml to setup.cfg
Added codecov action to workflow.yml

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@91627b3). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   96.34%
=========================================
  Files             ?        6
  Lines             ?      246
  Branches          ?        0
=========================================
  Hits              ?      237
  Misses            ?        9
  Partials          ?        0

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.


Episodes not ordered by time

Why are the episodes sorted by name? Wouldn't it make more sense to leave them in the order they were called? After all, it's just a sequence of remote requests. Or am I misunderstanding what the concept of episodes means?

@petrbela No real reason, but mainly due to the fact that a single rspec may test multiple endpoints of some API, hence a cassette may contain different episodes and they are easier to distinguish if they are sorted by endpoint url. In addition mr_video allows you to delete episodes so having episodes sorted by time may be hard to follow. Maybe the table should be sortable?

Hmm, that's a good reason, although I feel that sorted by time is a much more common use case. You could certainly have a test case where you fetch data from multiple places but I think it still makes more sense to order them chronologically just by the simple fact that a cassette is a recording of events. Actually, if you change the order, VCR would fail because it expects the replay in a given sequence.

This is mainly a front-end concern and doesn't affect the underlying cassette, I'm closing this. If anyone wants to add a sorting mechanism to the table it would be appreciated.


Events/Groups: attached video isn't correctly displayed in FF

Issue is reproduced in Firefox 50.1.0, it's reproduced in about 50% of page loads.

https://www.boonex.com/forums/topic/video-upload-issue-with-Firefox-layout-issue-.htm


Update request - how can I get yesterday's date

Summary of the update request
How can I get yesterday's date?
https://devblogs.microsoft.com/scripting/how-can-i-get-yesterdays-date/
I see q lot of questions like this
NEeds to be re-written for PowerShell

The article should explain the DateTime class plus show how to use the methods.

Seems like it would be nice to sync this up with a new PSKoans foundations learning block on datetime and datetimeoffset along with datetime math ($_.LastWriteTime -gt (Get-Date).AddDays(-1))

Should also definitely link tot he docs on those classes.

Thanks for trhe comments.
It is my intention that each post should contain a bit of background to the question. So in this case, this article should talk about system.datetime class (and properties, as well as the Get-Date cmdlet.

@Szeraax I am not personally familiar with PSKoans - feel free to suggest how to integrate it. Seems a great idea.

I think that we would need to create a new PSKoan script module and then ask Vexx to add it to the module. I like that Hashtable is in the foundations area and think datetime would fit too. Maybe DateTimeOffset and TimeSpan aren't as important and should be cut out.
I just spent a few minutes creating a really quick partial sample of what it could be. Someone completing this module would need to edit all the instances of $__ to something else that makes the unit tests not fail:
using module PSKoans
[Koan(Position = 122)]
param()
<#
    Dates

    In this module, you will test your understanding of DateTimes,
    TimeSpans, DateTimeOffset's, and date/time math.

    A date/time is called a datetime and points to a local time. Each
    datetime object points to a specific time, but each DateTimeOffset
    object points to a specific global moment in time.

    You will need to understand the ramifications of both to complete
    these challenges.
#>
Describe 'Datetimes' {

    It 'allows you to point to a time' {
        <#
            A DateTime object can be created many ways

            Refer to MS Docs for more help:
            https://docs.microsoft.com/en-us/dotnet/api/system.datetime.-ctor
        #>
        [datetime]"$__" | Should -Be (Get-date 2000-01-01).Date
        [datetime]::new($__) | Should -Be (Get-Date 2001-02-03).Date
    }

    It 'can be compared and operated on' {
        # Which one, which one
        # Does it really matter?
    $One = [datetime]"2020-01-01"
    $Two = [datetime]"2020-01-02"
    $__ -gt $__ | Should -Be $true
    $__.AddDays(-7) -lt $__ | Should -Be $true
    }

}

Describe 'TimeSpans' {

    It 'is a length of time' {
        [timespan]"$__" | Should -Be [timespan]::New(2, 15, 3)
    }
    # Add more
}

Describe 'DateTimeOffsets' {
    It 'is always the same instant' {
        # Add more
    }
    # Add more
}

As this article is now up on the blog, I will close this PR.
I don't want to lose the comment regarding PSKoans. Can I suggest that @Szeraax post the issue and suggestion to their issue page


add Api modules to /platform

I'm not sure if it makes sense (im consider my self very new on effect and fp paradigms), but I would like the @effect/RPC API to eventually be as similar to this HttpApi as possible. While RPC is currently very composable, I understand that it is not as declarative as this (HttpApi) in terms of separating the blueprint or declaration (with the help of schemas) from the implementation.
Great work, congratulations! 🚀


Techborn patch 1

Changes to allow for elements in compact mode to be expressed as a primitive value without requiring the textKey attribute when no other attributes are present. Undefined and null elements are skipped unless the fullEmptyTag option is passed. This change only affects the js->xml conversion
So, this (in yaml):
example:
  __text: 'value'

is allowed to be expressed like this:
example: 'value'

@techborn Your work is merged and published in v1.5.1.
Thanks for your contribution.


Issue 2334

Added an extra column to detailed csv report which shows the student mark/ total possible mark for the assignment

@binuri I would rather see a total number of marks earned by itself, rather than both the mark and the total possible.
I.e., show "10" rather than "10/15".

Thanks, @binuri!


[FEATURE] Add zk address config for proxy to find available broker

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

In featurehttps://github.com/streamnative/aop/issues/130, we have add this inclueded.


Updating instructions for H14 creation

Fixing https://github.com/rom1504/clip-retrieval/issues/288 and https://github.com/rom1504/clip-retrieval/issues/286
Updated documentation for H14 index setup:

aria2c command saving parquet files
other typos

thank you


Duplicate items when dropping on nested list

That's it, I have a nested set of sortables and I drag an item into some other as child it dupplicates (so I can see two identical items inside), or sometimes the item is inside and when I drag it out of its list it duplicates (leaving a copy inside the original container).
http://jsfiddle.net/Ln9e8/
In the example I reproduce the second case

Had the same problem described on this thread and the $(el).remove();  code replacement worked for me...


Add a generic Dart library for notifications

Is your feature request related to a problem? Please describe.
As described in this document, there is an opportunity to provide an end-to-end encrypted, peer-to-peer, surveillance free notification service.
Describe the solution you'd like
The approach needs to be discussed in the architecture call.
Adding to the backlog so we can plan accordingly.
Describe alternatives you've considered
This could be accomplished as a widget only, but a library for sending notifications for IoT or other applications may also makes sense.

@nickelskevin Now that notify and monitor have been implemented, do we want to revisit this and think about making a stand-alone library that developers can include just for this purpose?

@gkc start thinking about this again

@gkc should we re-prioritize to high for PR43 sprint planning?

I've flagged it for architecture discussion. The question to be answered is whether we want a standalone library just for sending / receiving notifications.


Routing State not propagating out of view models properly

createRoutingState and getRoutingState are not handling situations where routing state is returned. We want to support returning routing state as well as assigning routing state to the passed in state param. returned values will take precedence. We also need to updating the saveRoutingState default to return the state param.

actually we don't return anything from saveRoutingState, we don't want to force a return value, only allow users to return the state instead of editing in place.


Question for using kafka-client

【kafka-python-0.9.4】
from kafka import KafkaConsumer
File "/private/var/folders/jb/smm0c4k544j2fs_vk3d_7ykc0000gn/T/tmpC5rDzV/resources/logging_example/line_spout.py", line 4, in
from kafka import KafkaConsumer
ImportError: No module named kafka
It is ok when I load the kafka module in python console.
Python 2.7.9 (default, Dec 10 2014, 23:46:04)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.56)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

import kafka

Could you be a bit more precise? Are you running your topology locally or on a Storm cluster? Are you running the vanilla logging example?


HIVE-28422: Iceberg: Added missing awaitility dependency in iceberg-catalog

What changes were proposed in this pull request?

HIVE-28364 only added the org.awaitility dependency in iceberg root pom, but this dep is used by iceberg-catalog, so we need to add it in iceberg-catalog moudle. Otherwise, the Intellij idea will regard Awaitility as unrecognized code in TestHiveTableConcurrency which may make people confused, though we can build iceberg project successfully.
Why are the changes needed?

Does this PR introduce any user-facing change?

No
Is the change a dependency upgrade?

No
How was this patch tested?

Run the existing test TestHiveTableConcurrency.

cc @deniskuzZ

@zhangbutao it's already there, now declared twice

@zhangbutao it's already there, now declared twice https://github.com/apache/hive/blob/master/iceberg/iceberg-catalog/pom.xml#L100-L104

My bad.. I didn't reload my project after rebasing the lastest code.
Closing this ticket.


Loading multiple animation files with one player

Is it possible to write a single player to load multiple animation files rather than for each animation file. Is it possible to use one player to play multiple animations?
Here's my current code.
 var bmAnim = document.querySelector('#animation');

    var animData1 = {
            container: sun,
            name: 'privacy',
            renderer: 'svg',
            loop: false,
            autoplay: false,
            path: '_animation/sun.json',
    };

    var animData2 = {
            container: payment,
            name: 'payment',
            renderer: 'svg',
            loop: false,
            autoplay: false,
            path: '_animation/moon.json',
    };

    var animData3 = {
            container: passcode,
            name: 'passcode',
            renderer: 'svg',
            loop: false,
            autoplay: false,
            path: '_animation/stars.json',
    };

var anim = bodymovin.loadAnimation(animData1, animData2, animData3);

What I'm thinking is something like
 var bmAnim = document.querySelector('#animation');

    var animData1 = {
            container: passcode,
            renderer: 'svg',
            loop: false,
            autoplay: false,
            path: '_animation/sun.json',
            path: '_animation/moon.json',
            path: '_animation/stars.json',
    };

var anim = bodymovin.loadAnimation(animData1);

Thanks

if you load all three animations in a single html element (passcode), they would all overlap between each other.
Do you want a single player in order to control them all at once or have them all synced between each other for example?

I was trying to get them all synced to each other on hover which I figured out.
I was more trying to figure out if there was a shorter way of writing the bodymovin.loadAnimation()  code since if I had 10 animations on a page, I'd have to write loadAnimation 10 times which takes up quite a bit of space.
I'm also trying to get each animation to be next to each other but for some reason, no matter how much I set the width or display properties, they are all sitting on top of each other. Any ideas?

have you tried using display: inline-block to the containing div?
regarding code repetition, you can add a function that loads the animation and only pass the needed params to it.

Yup. Got the positioning figured out. Thanks.
Back to the multiple animations issue, all my animations are in one js file and I'm now calling them through an array.
I want to pause each array element individually but instead, every animation I click is only playing the last item in the array. I know I can probably write individual functions with eventlisteners for each array item, but is there a way to do it globally?
Here's the code snippet.
var elementsArr = ['map', 'privacy', 'payment', 'rewards', 'cat', 'passcode'];
var element;

function buildIllus(){
    for(var i=0;i<illustArr.length;i+=1){
        illust = document.getElementById(illustArr[i]);
        var params = {
            container: illust,
            name: 'passcode',
            autoplay: false,
            loop: false,
            animationData: animations[illustArr[i]],
            renderer: 'svg'
        };

        var anim = bodymovin.loadAnimation(params);

        illustArr.forEach(function (e) {
            document.getElementById(e).addEventListener('click', function() {
        anim.goToAndPlay(0, true)
        }, false);}
        )
    }
};

this is related to Javascript scoping. When you declare all the click functions inside your loop, that function gets scoped with your variable declaration. So once the event is dispatched, the anim value is always the last value of the loop.
You need to create a separate scope for each function invoked in the loop in order to get the anim variable scoped correctly.
I didn't test it, but something like:
var elementsArr = ['map', 'privacy', 'payment', 'rewards', 'cat', 'passcode'];
var element;

function addListenerToElement(e, anim){
  document.getElementById(e).addEventListener('click', function() {
    anim.goToAndPlay(0, true)
  }, false)
}

function buildIllus(){
    for(var i=0;i<illustArr.length;i+=1){
        illust = document.getElementById(illustArr[i]);
        var params = {
            container: illust,
            name: 'passcode',
            autoplay: false,
            loop: false,
            animationData: animations[illustArr[i]],
            renderer: 'svg'
        };

        var anim = bodymovin.loadAnimation(params);

        illustArr.forEach(function (e) {
          addListenerToElement(e, anim)
          })
    }
};

Could someone help me out with a similar issue please. I'm trying to get these 3 stars to play next to each other.

My code is in columns in the backend of Wordpress which I can add custom CSS elements to but for the life of me I just can't get them to float next to each other instead of under each other.

My test page is this:
https://motionvideos.uk/bodymovin-test/
My code is this:
`
`
Hope someone can help.
Thanks in advance.


[feat][stream] Add Stream Connector, print rowData RowKind in first C…

add print rowkind like below picture.

Because of conflicting with branch 'master', maybe you should adjust the code.


formula_cellar_checks: exclude llvm from Python linkage check

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

The LLDB Python bindings embed a Python interpreter into LLDB, so
explicit linkage to a Python interpreter is needed. [1]
This change will allow us to build the Python bindings for LLDB. This
improves feature-parity with system LLDB, which does provide Python
bindings. Currently, these bindings are not built because they fail the
check_python_framework_links audit.
[1] https://blog.tim-smith.us/2015/09/python-extension-modules-os-x/

Review period will end on 2021-03-30 at 00:00:00 UTC.

The LLDB Python bindings embed a Python interpreter into LLDB, so explicit linkage to a Python interpreter is needed.

It makes sense for the interpreter to be linked to LLDB itself but less so for it to be linked to the supporting Python module.
How well does it work when used outside of LLDB's embedded interpreter (using Python directly)? If it doesn't work well, then we can set LLDB_PYTHON_RELATIVE_PATH so that it is not in the default Python path (and won't be triggered by this audit).

It makes sense for the interpreter to be linked to LLDB itself but less so for it to be linked to the supporting Python module.

I agree with you there, except:
❯ ls -l $(brew --prefix llvm)/lib/python*/site-packages/**/*.so
lrwxr-xr-x 22 carlocab 28 Mar 06:06 /usr/local/opt/llvm/lib/python3.9/site-packages/lldb/_lldb.so -> ../../../liblldb.dylib

We could give this exception a lighter touch by excluding only any Python modules that belong to the llvm formula and refer to LLDB in its path, just in case other LLVM sub-projects start shipping Python bindings too. But I'm not hugely convinced this is necessary.

How well does it work when used outside of LLDB's embedded interpreter (using Python directly)? If it doesn't work well, then we can set LLDB_PYTHON_RELATIVE_PATH so that it is not in the default Python site-packages path (and won't be triggered by this audit).

I haven't tested it much, but the first couple of examples in the LLDB docs works for me. However, since llvm is keg-only, it's already not in the default Python site-packages path. You need to add $($(brew --prefix llvm)/bin/lldb -P) to your PYTHONPATH to be able to import lldb in Python.

Incidentally, Apple's LLDB module also has explicit framework links, so you can use it only with system Python:
❯ ls -l $(/usr/bin/lldb -P)/**/*.so
lrwxr-xr-x 52 root 10 Mar 15:19 /Library/Developer/CommandLineTools/Library/PrivateFrameworks/LLDB.framework/Resources/Python3/lldb/_lldb.so -> ../../../liblldbPluginScriptInterpreterPython3.dylib
❯ otool -L $(/usr/bin/lldb -P)/**/*.so
/Library/Developer/CommandLineTools/Library/PrivateFrameworks/LLDB.framework/Resources/Python3/lldb/_lldb.so:
        @rpath/liblldbPluginScriptInterpreterPython3.dylib (compatibility version 0.0.0, current version 0.0.0)
        @rpath/LLDB.framework/Versions/A/LLDB (compatibility version 1.0.0, current version 1200.0.44)
        @rpath/Python3.framework/Versions/3.8/Python3 (compatibility version 3.8.0, current version 3.8.0)
        /usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 904.4.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1292.0.0)

The differences between Apple's LLVM and upstream's version is limited nowadays (main difference now is the different branching points) so I'm not surprised they share the same bugs.

I haven't tested it much, but the first couple of examples in the LLDB docs works for me.

I worded it wrong. It will work fine with brewed Python because that's the very Python it is linked against. But it probably won't work if you load it from a Python 3.9 installed elsewhere.

since llvm is keg-only

This change is probably ok in that case, if we don't accept bug reports from people using the lldb Python module with non-brewed Pythons.
Keg-only can be overridden with brew link --force on a non-default prefix, but that's hopefully rare.
I would add a comment to the code though explaining exception, mentioning that we have acknowledged the trade-offs/limitations.

But it probably won't work if you load it from a Python 3.9 installed elsewhere.

Yep, agree.

if we don't accept bug reports from people using the lldb Python module with non-brewed Pythons.

I was thinking of adding a caveat to this effect. But I initially only thought of showing it when you install Homebrew into a non-default prefix, as I'm pretty sure even brewed Python 3.9 won't help you there.

But it probably won't work if you load it from a Python 3.9 installed elsewhere.

Yep, agree. It definitely does not work.

This is why this audit exists. I'm not sure I understand why it shouldn't apply to llvm but should anything/everything else? Have (m)any users requested this functionality?

This is why this audit exists. I'm not sure I understand why it shouldn't apply to llvm but should anything/everything else?

My point is that I think the audit is wrong in this case. The Python bindings embed an interpreter into lldb and for that, you do need explicit linkage.
Let me clarify one thing, though. What won't work if you install Homebrew into a non-default prefix is calling lldb from Python. I think what should still work is calling Python from lldb and, in my opinion, that's the valuable part of enabling the Python bindings.

Have (m)any users requested this functionality?

Not many that I know of, but there is me 👋 I've actually spent a bit of time trying to work out if there's an upstream fix for this, but there don't seem to be any simple ones.

My point is that I think the audit is wrong in this case.

I wouldn't say "wrong". Just "less of a problem" since the primary use case is using Python within LLDB.
Hence my suggestion to move the module out of site packages which will dodge the audit on its own.

there don't seem to be any simple ones.

Yeah, the fix would be to completely separate the module code from the interpreter code, rather than have it live in the same library. I don't think there's anyone interested in tackling this at the moment.

Let me clarify one thing, though. What won't work if you install Homebrew into a non-default prefix is calling lldb from Python.

Before or after this change?

I think what should still work is calling Python from lldb and, in my opinion, that's the valuable part of enabling the Python bindings.

Homebrew's Python, the system Python, both or neither?

Before or after this change?

I'm not sure I understand the question. Without this change (or an audit dodge as suggested by Bo), the Python bindings can't be built, so you can't call Homebrew LLDB from (any) Python, default prefix or not. (You also can't call system LLDB from Homebrew Python.)
With this change, we can build the LLDB Python bindings. This will allow you to call Homebrew LLDB from Homebrew Python if you've installed Homebrew into the default prefix.

Homebrew's Python, the system Python, both or neither?

Homebrew's Python.

With this change, we can build the LLDB Python bindings. This will allow you to call Homebrew LLDB from Homebrew Python if you've installed Homebrew into the default prefix.

What happens if you try to do so with a non-Homebrew Python or non-default-prefix?

Segfault.
❯ /usr/bin/python3 -c 'import lldb'
[1]    84055 segmentation fault  /usr/bin/python3 -c 'import lldb'

It's not pretty, but you need to jump through some hoops (e.g. set your PYTHONPATH to llvm's site-packages directory) to get to it, so I'm not overly concerned about users stumbling upon it.

It's not pretty, but you need to jump through some hoops (e.g. set your PYTHONPATH to llvm's site-packages directory) to get to it, so I'm not overly concerned about users stumbling upon it.

Ok. Given that: fine with me but let's have an actual allow-list here rather than an opt-out for a specific formula (as it feels like this list will only grow with time).

as it feels like this list will only grow with time

I don't want this list to grow at all. The only reason I'm somewhat ok is because it's not in Python's path by default.
Either we skip this for all keg-only formulae or adjust LLDB to not install to site-packages. The latter means it won't be a problem with brew link --force.

adjust LLDB to not install to site-packages. The latter means it won't be a problem with brew link --force.

Feels nicer to me 👍🏻

I don't want this list to grow at all.

👍🏻

adjust LLDB to not install to site-packages

Will look into this.

LLDB_PYTHON_RELATIVE_PATH is probably what you want. It's set to lib/pythonX.Y/site-packages by default.

Yup, I've seen it. Thanks for the hint.

Review period ended.

Opened https://github.com/Homebrew/homebrew-core/pull/74425.


Pulling the same story multiple times

Maybe an edge case - but you can currently publish the same PMP story to Wordpress multiple times.  Wondering if we should call this is a "buyer beware" kind of thing, or a bug.

Update: saved searches will automatically de-dup content (you can only pull a pmp-guid once).  If a WP user wants to display the Post in multiple pages of their site, they should probably use categories.
I'm thinking we should do the same with 1-off pulls.  Maybe throw up some sort of warning that the story was already pulled, and link you to it.

This is done!

Confirmed!


hide Read More button for entirely displayed posts

Considering the aesthetics and consistency of the index, I think we'd better not hide any Read More button. 😄


npm script with regex glob

This is my npm script:
"cross-env UV_THREADPOOL_SIZE=100 BABEL_DISABLE_CACHE=1 NODE_ENV=test nyc mocha --require test/helper.js \"test/spec/**/*.test.+(js|jsx)\""

After upgrading to 4.0.0, I get this error:
/bin/sh: -c: line 0: syntax error near unexpected token `('
/bin/sh: -c: line 0: `nyc mocha --require test/helper.js test/spec/**/*.test.+(js|jsx)'

Is this a bug with v4 or do I need to change how I am passing my args to mocha?

This might be related to #99

Definitely #99. @adamgruber could you try \\\"test/spec/**/*.test.+(js|jsx)\\\"?

I tried different combinations of escaping but none have worked so far. Here's a simplified script:
"cross-env mocha \\\"test/spec/**/*.test.+(js|jsx)\\\""

and the error:
sh: -c: line 0: syntax error near unexpected token `('
sh: -c: line 0: `cross-env mocha \"test/spec/**/*.test.+(js|jsx)\"'

I tried escaping the parenthesis:
"cross-env mocha \"test/spec/**/*.test.+\\(js|jsx\\)\""

but then I get this:
/bin/sh: jsx): command not found

Applying the same analysis as there.
original script

The original text / what we can read in package.json: "cross-env mocha --require test/helper.js \"test/spec/**/*.test.+(js|jsx)\""
The actual JSON value / what the shell sees:  cross-env mocha --require test/helper.js "test/spec/**/*.test.+(js|jsx)"
The shell-parsed value / what cross-env gets as process.argv:

[ 'mocha',
  '--require',
  'test/helper.js',
  'test/spec/**/*.test.+(js|jsx)' ]

Spawn call: spawn('/bin/sh', ['-c', 'mocha --require test/helper.js test/spec/**/*.test.+(js|jsx)'])
What the spawned shell sees: mocha --require test/helper.js test/spec/**/*.test.+(js|jsx)

Regexp is not quoted so it is interpreted by the shell. That why it says /bin/sh: -c: line 0: syntax error near unexpected token ('`.
suggested fix

The original text / what we can read in package.json: "cross-env mocha --require test/helper.js \\\"test/spec/**/*.test.+(js|jsx)\\\""
The actual JSON value / what the shell sees:  cross-env mocha --require test/helper.js \"test/spec/**/*.test.+(js|jsx)\"
The shell-parsed value / what cross-env gets as process.argv:

[ 'mocha',
  '--require',
  'test/helper.js',
  '"test/spec/**/*.test.+(js|jsx)"' ]

Spawn call: spawn('/bin/sh', ['-c', 'mocha --require "test/helper.js test/spec/**/*.test.+(js|jsx)"'])
What the spawned shell sees: mocha --require "test/helper.js test/spec/**/*.test.+(js|jsx)"

So it should work. I suspect nyc might be adding an additional processing layer here.
@adamgruber could try without nyc?

I guess you could also try test/spec/\\*\\*/\\*.test.+\\(js\\|jsx\\).

@hgwood My example above was without nyc.
Taking your suggestion and escaping not just the parenthesis but also the pipe in my regex fixes it. So my working script is now:
"cross-env mocha --require test/helper.js \"test/spec/**/*.test.+\\(js\\|jsx\\)\""

Thanks for the help.


Synthesis failed for Google.Cloud.PubSub.V1

Hello! Autosynth couldn't regenerate Google.Cloud.PubSub.V1. :broken_heart:
Here's the output from running synth.py:
Cloning into 'working_repo'...
Switched to branch 'autosynth-Google.Cloud.PubSub.V1'
fatal: ambiguous argument 'synth.metadata': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
Running synthtool
['/tmpfs/src/git/autosynth/env/bin/python3', '-m', 'synthtool', 'synth.py', '--']
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/apis/Google.Cloud.PubSub.V1/synth.py.
On branch autosynth-Google.Cloud.PubSub.V1
nothing to commit, working tree clean
HEAD detached at FETCH_HEAD
nothing to commit, working tree clean
Cloning into '/tmpfs/src/git/autosynth/working_repo/gapic-generator-csharp'...
Cloning into 'googleapis'...
Building existing version of Google.Cloud.PubSub.V1 for compatibility checking
Generating Google.Cloud.PubSub.V1
Running mid-micro-generation script for Google.Cloud.PubSub.V1
sed: can't read googleapis/google/pubsub/v1/pubsub.proto: No such file or directory
sed: can't read googleapis/google/pubsub/v1/pubsub_grpc_service_config.json: No such file or directory
sed: can't read googleapis/google/pubsub/v1/pubsub.proto: No such file or directory
sed: can't read googleapis/google/pubsub/v1/pubsub_grpc_service_config.json: No such file or directory
synthtool > Failed executing /bin/bash generateapis.sh --check_compatibility Google.Cloud.PubSub.V1:

None
synthtool > Wrote metadata to synth.metadata.
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.1/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.1/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 205, in _call_with_frames_removed
  File "/tmpfs/src/git/autosynth/working_repo/apis/Google.Cloud.PubSub.V1/synth.py", line 20, in <module>
    hide_output = False)
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/git/autosynth/env/lib/python3.6/site-packages/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.1/lib/python3.6/subprocess.py", line 418, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '('/bin/bash', 'generateapis.sh', '--check_compatibility', 'Google.Cloud.PubSub.V1')' returned non-zero exit status 1.

Synthesis failed

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


Add "shift + mouse scroll" action for carousel

Describe the feature
I don't know if I should ask for it here or in the original project, but it would be nice if the user could scroll through the contents of the carousel using the scroll wheel (as is already implemented in all browsers for horizontal overflow).

Hi;
Haven't tested it with a mouse, only the trackpad, but there is a plugin you need to add in order to achieve this: https://www.embla-carousel.com/plugins/wheel-gestures/
Does that meet your needs?

Closing as this is outside the scope of this project and a plugin for embla-carousel can be used if this feature is required.


Styles are not working, 9/10 times.

Description
Hello I have a setup a monorepo with an expo app and have initialized gluestack in the monorepo. When I start the app, most of the times the styles don't work. That's like 9 times out of 10. But once in a while, the styles will load correctly, but that's 1/10 times if not less.  Could someone tell me what I'm missing? Please note that if you start the web app before starting expo, the styles always work.
CodeSandbox/Snack link
https://github.com/gablabelle/gluestack-ui-bug
Steps to reproduce

Clone the repo
pnpm install
pnpm run:native
Choose IOS simulator

gluestack-ui Version
latest (installed using --all yesterday, Dec 4th)
Platform

[X] Expo
[ ] React Native CLI
[ ] Next
[ ] Web
[ ] Android
[X] iOS

Other Platform
No response
Additional Information
⚠ Please note that if you start the web app before starting expo, the styles always work.
If it doesn't work you'll see this

But if it works you'll see this

Hey @gablabelle, Please upgrade to stable version. you can find a blog about here

Works

Works

Hi @Viraj-10 In case I have already initialized it before, how can I upgrade the version? Thank you.

Hi @Viraj-10 In case I have already initialized it before, how can I upgrade the version? Thank you.

Good question, I had to check the source code of their CLI to figure it out. You need to delete the folder of the ui/gluestack-ui-provider folder in order to be able to install.
IMO this is a bad way to check that the lib is installed.
But anyways after removing that folder you should be able to run the init again.

Works, thanks.


[graphviz] Update auto update installer match string

Description
Not tested but looking at https://www.graphviz.org/download/ the stable_ prefix was dropped since version 2.50
Motivation and Context
automatic updates are not working, this should fix it

:white_check_mark: Package verification completed without issues. PR is now pending human review


Type checking question

Hello,
I would like to know the important of the structure let x = blabla in when you do the type checking. For instance, it seems there are some difference in term of type check between
let a:good_symbol = { name = "enc" ; arity = 2; cat = Constructor [n;n] n }

and
let a:good_symbol =
  let x = { name = "enc" ; arity = 2; cat = Constructor [n;n] n } in
  x

My type good_symbol is a refinement of an other type symbol that just check that the size of the arity is the same as the size of the list in the Constructor. But with the first line of code, it does not type check. I have the error Expected type "good_symbol"; got type "symbol". However, with the second way, Fstar succeeds to type check. So I was wondering how important is the 'let' for type checking. Wouldn't it possible that he manages to type check even without it so that the first line of code (the most natural one in my opinion) works ?
The full code is below.
Best regards,
Vincent Cheval
(************************
***       Types       ***
*************************)

module Term

type symbol_cat =
  | Tuple : symbol_cat
  | Constructor : list term -> term -> symbol_cat
  | Destructor : list term -> term -> symbol_cat

and symbol =
  {
    name : string;
    arity : nat;
    cat : symbol_cat
  }

and term =
  | Func : (symbol * list term) -> term
  | Name : string -> term

(******* List ******)

val length: list 'a -> Tot nat
let rec length l = match l with
  | [] -> 0
  | _::q -> 1 + (length q)

(*********************************
***        Predicates          ***
**********************************)

(** Arity and arguments **)

type good_symbol = s:symbol{ match s.cat with
  | Tuple  -> true
  | Constructor list_arg _ -> length list_arg = s.arity
  | Destructor list_arg _ -> length list_arg = s.arity
  }

let n:term = Name "n"

(* Does type check *)
let a:good_symbol =
  let x = { name = "enc" ; arity = 2; cat = Constructor [n;n] n } in
  x

(** Does not type check
let a:good_symbol = { name = "enc" ; arity = 2; cat = Constructor [n;n] n }
*)

FYI, type inference is no longer bidirectional. So, both my pair examples work well now ... I've added this to our regression suite.


Added parameter to Display tool version.

Closes #146
Possible approach which opens the possibility of displaying additional relevant information such as: version of Component Detection being used. Feedback would be appreciated as I wasn't sure if a workflow should be added to accomplish this task.
Usage:
sbom-tool info
Output:

@microsoft-github-policy-service agree

I would prefer to have --version parameter instead of info. It is more commonly used across cli tools

Changed naming and added a shortcut so that the action can be invoked using either:
sbom-tool version or sbom-tool --version


chore: Remove minor/patch versions from examples

Description

Remove minor/patch versions from examples
Update CI workflows to use latest
Update pre-commit version to latest

Motivation and Context

Closes #446
Closes #450

Breaking Changes

No

How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects
[x] I have executed pre-commit run -a on my pull request

This PR is included in version 5.2.2 :tada:


Right Ctrl key support for Find My Mouse

Description of the new feature / enhancement
New setting in mouse utilities for Find My Mouse: toggle between left and right Ctrl key as the effective keyboard shortcut.
Scenario when this would be used?
As a left handed mouse user, I want to use the right Ctrl button on my keyboard to highlight the position of my mouse, so I can keep my hand on the mouse while locating it and my right hand in a natural typing position.
Supporting information
No response

Duplicate of #14651

Specifically, you might want to try this comment

/dup  #14651


crash

Crash on GLTFDocument document = DeserializeJson(*stream);
Unhandled exception at 0x76A308F2 in WindowsMRAssetConverter.exe: Microsoft C++ exception: Microsoft::glTF::InvalidGLTFException at memory location 0x00AFC2F4. occurred
This when loading a gltf file

Can you please post the file? Also the exception indicates that the file is invalid GLTF. Can you please try validating with http://github.khronos.org/glTF-Validator/ ?

Pinging @ehtick - Would love to get to the bottom of your issue.

I'm closing this since we didn't have any answers from @ehtick in more than a month. Feel free to reopen with more information. Thanks!


Error typescript

Hi, I was testing the component with the example you have in the docs, and I get this error:
Warning: Cannot update a component (VerticalTimelineElement) while rendering a different component (InView). To locate the bad setState() call inside InView, follow the stack trace as described in https://fb.me/setstate-in-render     in InView (created by VerticalTimelineElement)     in VerticalTimelineElement (at Jobs.tsx:10)     in div (created by VerticalTimeline)     in VerticalTimeline (at Jobs.tsx:9)     in section (created by styled.section)     in styled.section (at Jobs.tsx:8)     in Jobs (at MainLayout.tsx:15)     in MainLayout (created by Context.Consumer)     in Route (at App.tsx:21)     in Switch (at App.tsx:19)     in Router (created by BrowserRouter)     in BrowserRouter (at App.tsx:17)     in Suspense (at App.tsx:16)     in App (at src/index.tsx:12)     in IntlProvider (at WrapperLang.tsx:48)     in Wrapper (at src/index.tsx:11)     in StrictMode (at src/index.tsx:10)
my environment is the following
reactjs
 "@fortawesome/fontawesome-free-solid": "^5.0.13",     "@fortawesome/fontawesome-svg-core": "^1.2.32",     "@fortawesome/free-brands-svg-icons": "^5.15.1",     "@fortawesome/free-solid-svg-icons": "^5.15.1",     "@fortawesome/react-fontawesome": "^0.1.11",     "@testing-library/jest-dom": "^4.2.4",     "@testing-library/react": "^9.3.2",     "@testing-library/user-event": "^7.1.2",     "@types/jest": "^24.0.0",     "@types/lodash": "^4.14.161",     "@types/node": "^12.0.0",     "@types/react": "^16.9.0",     "@types/react-bootstrap": "^0.32.24",     "@types/react-dom": "^16.9.0",     "@types/react-responsive": "^8.0.2",     "@types/react-router": "^5.1.8",     "@types/react-router-dom": "^5.1.5",     "@types/react-vertical-timeline-component": "^3.0.0",     "axios": "^0.20.0",     "bootstrap": "^4.5.2",     "faker": "^5.1.0",     "lodash": "^4.17.20",     "react": "^16.13.1",     "react-bootstrap": "^1.3.0",     "react-dom": "^16.13.1",     "react-flexbox-grid": "^2.1.2",     "react-intl": "^5.8.4",     "react-responsive": "^8.1.0",     "react-router": "^5.2.0",     "react-router-dom": "^5.2.0",     "react-scripts": "3.4.3",     "react-vertical-timeline-component": "^3.3.0",     "styled-components": "^5.2.0",     "typescript": "^3.8.2"
And functional component look :

Could you provide a full example please ?

That's a React error message and has nothing to do with TypeScript I think. You could try these suggestions: https://stackoverflow.com/questions/62336340/cannot-update-a-component-while-rendering-a-different-component-warning


Release

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@60ebbc4). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #31   +/-   ##
=======================================
  Coverage        ?   46.25%
=======================================
  Files           ?       10
  Lines           ?     1042
  Branches        ?        0
=======================================
  Hits            ?      482
  Misses          ?      560
  Partials        ?        0

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 60ebbc4...1360b6f. Read the comment docs.


Typo in RKRequestDelegate

- (void)request:(RKRequest*)request didReceivedData
``
should be

(void)request:(RKRequest*)request didReceiveData:

Fixed on development via a separate commit. Did not want to merge this pull request because of the extraneous Rakefile changes.


Add nominee: Open Foris (OF) Collect and Collect Mobile

Automatic addition of a new nominee submitted through the online form available at https://digitalpublicgoods.net/submission

Looks good @nathanbaleeta, ready for your formal review, approval and merge. Thank you! 🙏


added support to set minute, second and hour

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Issue tracker returns 404 when the repository is empty.

Gogs version (or commit ref): <IP_ADDRESS>1
Can you reproduce the bug at https://try.gogs.io:

[X] Yes (https://try.gogs.io/squarefractal/test-repository/issues)
[ ] No
[ ] Not relevant

Description
When a repository is empty ("empty" as in nothing has been pushed yet to a newly created repository), trying to view the issue tracker results in a 404.

Hi, this is expected behavior.


Understanding systemd

Article Checklist
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

NOTE: (1st time contributors - submit ONLY one article to get familiar with the process and style-once published, you can then submit any number of articles.)
Grammar Checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical Checks

[ ] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution Guidelines
For first-time contributors and for more details, see Contributing Guidelines

@hectorkambow please can my article be reviewed?

@hectorkambow please can my article be reviewed?

@hectorkambow, I'll review this!

@hectorkambow, I'll review this!

@hectorkambow I'll review this article!

@hectorkambow I'll review this article!

@hectorkambow I'll review this article!

Alright @jumamiller, thanks.

@hectorkambow I'll review this article!

Alright @jumamiller, thanks.

@Bashorun97, are you there?

@Bashorun97, are you there?

@Bashorun97, are you there?

Yes, I am. I'll get it fixed today.Thanks

@Bashorun97, are you there?

Yes, I am. I'll get it fixed today.Thanks

Hello @jumamiller, can you please review now? I've made all necessary fixes. Thanks you.

Hello @jumamiller, can you please review now? I've made all necessary fixes. Thanks you.

@Bashorun97  what issue # does this article tie into?

@Bashorun97 what issue # does this article tie into?

It's tied to #1386

@jumamiller I've made the fixes.

Add your author details(refer to authors directory for examples).
Kindly update that single typing error before I submit your article for final review!

Wow.. Thanks.
I've made the appropriate changes now.

Rename the image to "avatar" NOT index, read the instruction given above

@Bashorun97, author details include your profile photo. In the authors' directory, add your photo and rename it to 'avatar.(your image extension)'. Ensure the image fits our website. 600x400 range would be a good one

Ok. That was an oversight. Done.

@Bashorun97, author details include your profile photo. In the authors' directory, add your photo and rename it to 'avatar.(your image extension)'. Ensure the image fits our website. 600x400 range would be a good one

Ok. That was an oversight. Done.

Yeah, that has been fixed.

@hectorkambow, this article is ready for final review!

@Bashorun97
👋  Hello and welcome to the EngEd program.
I will be giving your articles a final review before we publish them.
This PR has multiple files attached to it and will cause a merge conflict if we merge.

The PR should only have the files you are adding. For example, the hero image .md (markdown) file - in this case since it is you 1st article a headshot and bio file.
Best way would be to create a new fresh PR with just the files (needed) are added.

Alright. But I really don't get how to go about this.
Do I have to separate both the article and bio files then make individual
commits and PR for them?
On Mon, Feb 8, 2021 at 4:12 PM hectorkambow<EMAIL_ADDRESS>wrote:

@Bashorun97 https://github.com/Bashorun97
👋 Hello and welcome to the EngEd program.
I will be giving your articles a final review before we publish them.
This PR has multiple files attached to it and will cause a merge conflict
if we merge.
[image: image]
https://user-images.githubusercontent.com/60800358/107238521-2be83e80-69e5-11eb-8522-ae1827588c7b.png
The PR should only have the files you are adding. For example, the hero
image .md (markdown) file - in this case since it is you 1st article a
headshot and bio file.
Best way would be to create a new fresh PR with just the files (needed)
are added.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/1465#issuecomment-775217384,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFHRXE3UJ2O4NFKONVJV7GTS575NZANCNFSM4WLIPWIA
.

@Bashorun97, I think at this stage you have "messed" up your PR, reach me on Slack for assistance!

Okay, I will.
On Tue, Feb 9, 2021 at 6:31 AM jumamiller<EMAIL_ADDRESS>wrote:

@Bashorun97 https://github.com/Bashorun97, I think at this stage you
have "messed" up your PR, reach me on Slack for assistance!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/1465#issuecomment-775677048,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFHRXEZSNAASYH3ZTNVDN33S6DCCDANCNFSM4WLIPWIA
.


[$125] Dot separator in the report preview component appears bottom aligned

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.3.83-10
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @trjExpensify
Slack conversation: (internal)
Action Performed:

Global create > request money
Create a request
As the payer, click Pay elsewhere to settle up
Observe the "Cash - Paid elsehwere" dot separator position in the report preview component is bottom-aligned

Expected Result:
The dot separator should be center-aligned:

Actual Result:
It looks bottom-aligned:

Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Yes, visual design bug.
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01765a32b3ae401177
Upwork Job ID:<PHONE_NUMBER>859080704
Last Price Increase: 2023-10-13

@shawnborton @dannymcclain @dubielzyk-expensify for vis and confirmation!

Also, when testing this I noticed that Distance has Done after the dot separator, but it should use the appropriate payment method like all other requests. E.g Paid or Paid elsewhere. We may as well fix that here as well. CC: @neil-marcellini @JmillsExpensify not sure when or why that happened.

@tienifr nice! Can you also include the change for the Done? Asking @trjExpensify what should go there instead

@tienifr for the Done let's remove distance from this check - https://github.com/Expensify/App/blob/a05c3c5ed002b2e23d216bd899ff3238b0d86b20/src/components/ReportActionItem/MoneyRequestPreview.js#L178-L180

Or we can remove styles.mb1 in both text component and add styles.mb1 to the wrapped view

This makes the most sense to me. I like the idea of making sure that entire view has the margin bottom, and making sure that entire view uses a centered vertical alignment.
Also, I would be curious to see the dot indicator icon bumped up to being 8x8.

@tienifr @rushatgabhane regression https://github.com/Expensify/App/issues/29888 from this issue PR.

👋 @rushatgabhane can you finish the checklist, please? Also looks like we've got a regression from this PR, right?

Removing the hold from the title and Awaiting payment as we need to fix the regression this caused.

I believe this is ready for payment now

We're still awaiting the checklist. Also, can we confirm the number of regressions here?
It seems like one so that takes us to $62.5 as the bounty due to @rushatgabhane & @tienifr, right?

Bump!

WIP ⚠

The PR that introduced the bug has been identified. Link to the PR:

The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Determine if we should create a regression test for this bug.

If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again

Do we need to issue payouts as per this? https://github.com/Expensify/App/issues/29543#issuecomment-1821861202