changing data-parsley-pattern doesn't work

after changing data-parsley-pattern of a field,the field always shows that its validate result is error, even destroy the form before changing the attribute and init after that.
ps: the other attribute such as ' required' worked
code:
$('#login-form').parsley().destroy();
$('#login-by').attr('data-parsley-pattern','/^(13|14|15|17|18)\d{9}$/g');
$('#login-form').parsley();

No need for destroy.
Closing until a working example is provided.


is omnilayer support p2pk tx?

Haven't seen from the Specification?

Hi @gingingin, the specification of the Omni Layer is not up-to-date and does not match the implementation entirely, but it does not support P2PK. There are allowed types:
https://github.com/OmniLayer/omnicore/blob/master/src/omnicore/rules.cpp#L362

as I describe in issue 811,I read omni code,I just wonder the consideration why do not support p2pk as input.because some user may have some p2pk output,so it can't used as omni input


Adding navigation

Wanted to hear you thoughts about adding a basic navigation to the example repo?
IMHO one of the trickiest part about combining these two frameworks is to figure out navigation like Fernando Rojo showed in the lastes nextjs.conf

Oh.. just saw that there already is an open PR for this #1


scheduler takes a long time to shutdown

Thought it'd be fixed by #32, but seems like it wasn't.
At first glance, seems like the context solution doesn't work either way, because K8s doesn't use the cobra command's context - see:

func runCommand(cmd *cobra.Command, opts *options.Options, registryOptions ...Option) error {
	verflag.PrintAndExitIfRequested()

	// Activate logging as soon as possible, after that
	// show flags with the final logging configuration.
	if err := logsapi.ValidateAndApply(opts.Logs, utilfeature.DefaultFeatureGate); err != nil {
		fmt.Fprintf(os.Stderr, "%v\n", err)
		os.Exit(1)
	}
	cliflag.PrintFlags(cmd.Flags())

	ctx, cancel := context.WithCancel(context.Background())
	//                                ^^^^^^^^^^^^^^^^^^^^ should be cmd.Context()
	defer cancel()
	go func() {
		stopCh := server.SetupSignalHandler()
		<-stopCh
		cancel()
	}()

	cc, sched, err := Setup(ctx, opts, registryOptions...)
	if err != nil {
		return err
	}
	// add feature enablement metrics
	utilfeature.DefaultMutableFeatureGate.AddMetrics()
	return Run(ctx, cc, sched)
	//         ^^^ context used here.
}

pointing comments added by me. source: https://github.com/kubernetes/kubernetes/blob/d6fe718e19881399fbb65f693c0c0f29f8f63861/cmd/kube-scheduler/app/server.go#L121-L147

HOWEVER, we're working with K8s 1.23 and I don't particularly want to fork kubernetes to backport a fix. AND this shouldn't matter — K8s is using server.SetupSignalHandler() to cancel the context, which should work anyways.

Checked, this issue still occurs after switching to 1.24 — will investigate further later this week.

Looked into this, tried the fix in this issue, didn't fix it. Going to try the long way (start with base scheduler, gradually add each piece of the plugin until the behavior re-emerges).

The issue was fixed in later K8s versions by this PR: https://github.com/kubernetes/kubernetes/pull/110207. Specifically, this diff:
diff --git a/pkg/scheduler/scheduler.go b/pkg/scheduler/scheduler.go
index 8d9cc2c2763..c165b34ffc2 100644
--- a/pkg/scheduler/scheduler.go
+++ b/pkg/scheduler/scheduler.go
@@ -335,7 +335,16 @@ func New(client clientset.Interface,
 // Run begins watching and scheduling. It starts scheduling and blocked until the context is done.
 func (sched *Scheduler) Run(ctx context.Context) {
        sched.SchedulingQueue.Run()
-       wait.UntilWithContext(ctx, sched.scheduleOne, 0)
+
+       // We need to start scheduleOne loop in a dedicated goroutine,
+       // because scheduleOne function hangs on getting the next item
+       // from the SchedulingQueue.
+       // If there are no new pods to schedule, it will be hanging there
+       // and if done in this goroutine it will be blocking closing
+       // SchedulingQueue, in effect causing a deadlock on shutdown.
+       go wait.UntilWithContext(ctx, sched.scheduleOne, 0)
+
+       <-ctx.Done()
        sched.SchedulingQueue.Close()
 }

Of course, only discovered this after figuring out the fix myself and then finding it had been fixed in master. :disappointed:
Unfortunately, it wasn't backported to 1.24. It is included in 1.25, but upgrading to 1.25 involves more infra work.


skipIdentity: add missing implementation

Currently the skipIdentity flag does nothing. This PR adds the missing implementation of this flag, resulting in the identity check to be skipped and marked as successful if the flag is now set to true.

Hey Florian thanks. Although it's been already implemented in staging and currently under testing, look at this commit.
FYI In general, the lifecycle of the contributions should always be: your branch -> staging -> master
I think we can close this one

Ok, thank you for the info. I was not aware of the staging branch. Certainly think my version is more readable.


Update manifest with evmconnect image

Update manifest.json with latest evmconnect image

Superseded by https://github.com/hyperledger/firefly/pull/1494


Duskwalker Heritage not adding Darkvision to Low-Light Vision Ancestries (tested with Elf)

With no modules at all, made a new character that's an Elf. Gave them Duskwalker Heritage and they didn't get Darkvision even though the heritage says "You also gain Low-Light Vision, or you gain Darkvision if your ancestry already has low-light vision.".

Closed by #12362


feat: sync kcl v0.5.4 APIs and update test cases.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

feat: sync kcl v0.5.4 APIs and update test cases.

add DefaultValue showing for GetSchemaType API
add IncludeSchemaTypePath option for Run API

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

kclvm_test.go

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 11 (36.36%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 62.527%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/kcl/opt.go
1
8
12.5%

Files with Coverage Reduction
New Missed Lines
%

pkg/tools/format/kformat.go
2
54.55%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2281

Relevant Lines:
3648

💛  - Coveralls


Support rich notifications and payload per token

 Support translations

 Add rich notification support to the code

 Merge batch_send changes that landed in master into translation branch

 Implement changes in rich notifications as added by Ophir

 Support translations

@matangover feel free to merge and update the other PR :)


Unable to build on macOS 10.14 (Mojave)

I get the following error when running the first step/cmake on macOS 10.14. Any ideas what may be wrong. I don't have XCode installed, so if that's a dependency I'd be glad to install it.
I use Nix package manager; this may have something to do with that.
$ cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX:PATH=/usr/local $PWD
CMake Error at /nix/store/fhp2mjmbhkjsn3czhd0y8fphs83xxnq1-cmake-3.19.1/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:218 (message):
  Could NOT find PAM (missing: PAM_LIBRARY PAM_INCLUDE_DIR)
Call Stack (most recent call first):
  /nix/store/fhp2mjmbhkjsn3czhd0y8fphs83xxnq1-cmake-3.19.1/share/cmake-3.19/Modules/FindPackageHandleStandardArgs.cmake:577 (_FPHSA_FAILURE_MESSAGE)
  cmake/Modules/FindPAM.cmake:20 (find_package_handle_standard_args)
  CMakeLists.txt:26 (find_package)

-- Configuring incomplete, errors occurred!
See also "/Users/gurjeet/dev/PAM_REATTACH/CMakeFiles/CMakeOutput.log".

Hi,
As far as I can tell, CMake is unable to find libpam, which is a system library and found at /usr/lib/libpam.dylib as far as I know, though it might be that Nix hides these.
On the other hand, on Big Sur, libpam and its headers are provided by Xcode, so you might also check if installing Xcode resolves your issues.

I stumbled upon this as well, and found a solution, although I don't think it will work for long.
@gurjeet did you solve this?
I did through the following modifications:

Found the libpam.dylib in my nix store at /nix/store/8pcyfg4vg2gb6910gwv63khis4g4vr00-openpam-20170430/lib
Modified cmake/Modules/FindPAM.cmake find_path and find_library, adding HINTS /nix/store/8pcyfg4vg2gb6910gwv63khis4g4vr00-openpam-20170430/lib/)
Called cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX:PATH=/usr/local -DCMAKE_OSX_ARCHITECTURES="arm64" . (note that I set the arch to arm64 specifically, not to arm64;x86_64 as in the Readme)
make and sudo make install to install it.

Everything seems to work right now, but I guess as soon as nix decides to update the openpam (whatever that is), I'll need to rebuild pam_reattach as well as it is dynamically linked?
Any better approaches? I'm a beginner Nix user and never really compile C programs from scratch either :smile:

Also ran into this. For Nix on Apple Silicon:
% nix-shell -p cmake -p clang -p openpam
[nix-shell]$ cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX:PATH=/usr/local -DCMAKE_OSX_ARCHITECTURES="arm64" .
[nix-shell]$ sudo make install


Add internals_visible_to to the rules

Part of #150

Doing this on an ARM Mac which doesn't support Bazel or .NET yet.

Doing this on an ARM Mac which doesn't support Bazel or .NET yet.

What could go wrong!

Wording suggestions to align with the other parameter descriptions.


convolution layer followed by RELU : can convolutionlayer's output data tensor be passed to RELU as input data tensor ?

Convolution layer stores its output data to its output tensor’s data buffer, and RELU takes its input from its input tensor’s data buffer (input tensor specified as 1st argument of NEActivationLayerKernel::configure).
In order to pass convolution layer’s output to RELU operation as input, do we have to use 2 separate tensors - one for NEConvolutionLayer output and another one for NEActivationLayerKernel input ? In this case, the data would have to be copied row after row from one tensor's data buffer to the other one.
Or can we directly pass convolution layer’s output tensor as input tensor to NEActivationLayerKernel ? I would prefer this implementation as it would save precious CPU time (zero-copy), but do we have to expect some incompatibility of data buffer layout, due to padding performed in convolution layer (we use 3x3 filters for convolution layer, with 1-element padding on each side) ? If so, is there a way to overcome this incompatibility ?

in my question, I mentioned by mistake NEActivationLayerKernel - I actually meant NEActivationLayer

There is no need to use two different tensors for input and output. As long as the dimensions match you directly reuse the same tensor the output of the first function is written to. The library will handle the padding etc. transparently.

Many thanks !
From: Moritz Pflanzer<EMAIL_ADDRESS>Sent: Monday, April 24, 2017 11:28 AM
To: ARM-software/ComputeLibrary
Cc: Joel Miedzigorski; Author
Subject: Re: [ARM-software/ComputeLibrary] convolution layer followed by RELU : can convolutionlayer's output data tensor be passed to RELU as input data tensor ? (#60)
There is no need to use two different tensors for input and output. As long as the dimensions match you directly reuse the same tensor the output of the first function is written to. The library will handle the padding etc. transparently.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ARM-software/ComputeLibrary/issues/60#issuecomment-296574753, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AaiCl_pDFZNuhLVaIsDGg8IFLAYnP3nHks5rzF0pgaJpZM4NFYWa.

one more question related to this issue :
if I reuse a tensor for input, I understand that I must refrain from calling allocator()->allocate() for this tensor after calling configure - am I right ?
(While for new tensors - not reused ones - I call allocator()->allocate() after configure)

Yes, you need to allocate the tensor only once.
However, to make sure that it will have enough padding for all the functions you want to use it with you need to first configure all functions before allocating.

Assuming question has been addressed. Closing issue. Reopen if required!


json-schema type alias translates to union in GraphQL

Hi, one additional issue... when you have a type definition in json-schema like this:
"DateTimeIso8601": {
  "type": "string",
  "description": "A datetime in ISO-8601 format (YYYY-MM-DDTHH:MM:SS.MMMZ)",
  "format": "date-time"
}

It results in a union of String in graphql, like this: union DateTimeIso8601 = String
This is invalid graphql and results in an error:
Union type DateTimeIso8601 can only include Object types, it cannot include String.

Thanks

Thanks, will fix this!


⚠ Turnos 24x7 has degraded performance

In c78914a, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 978 ms

Resolved: Turnos 24x7 performance has improved in 334b78f.


Message when plugins have been updated

Hi.
Is it possible to get some message, when some installed plugins have been updated?

Not yet, but it is high on our back log.


Cyberware block recipes

Cyberware blocks should use steel and aluminum for their blocks, specificially the surgery chamber which only needs iron. They should also require some sort of circuitry for everything, something like solid state circuits from cyberware itself or maybe super circuit maker parts.

I wouldn't mind revisiting the recipes later, but here is what I have for now.


Update extconf.rb to check for termux

Closing obsolete PR. Please reopen if there is interest in pursuing this.


Create HOWTO on adding images to kata description

Motivated by this post: https://www.codewars.com/kata/5fc5e74d8c53ff0027f00ace/discuss#5fc6cac41da1040023327e7d
I prepared some examples here: http://snippi.com/s/g4a6emp
And this kumite: https://www.codewars.com/kumite/5fc6e48caa10e60018543723?sel=5fc6e48caa10e60018543723 but it turns out not to work too well as a presentation of things related to descriptions :(
If you have any ideas what should (or should not) end up in such article, let me know.

I've been wanting to support image uploads for the description. Once that's done, users can paste the image directly into the editor. But I don't know when I'll be able to work on that yet. I'd also like to support a way to specify image for light/dark theme, similar to the docs.
The description shouldn't rely on images unless it's absolutely necessary though.

Kumite descriptions are rendered by the server and have different behavior, so you should create a draft kata if you want to provide a live example.

So maybe I should hold on with this article and do not create it until uploading is possible?

You can write it. I don't think I can work on it for a while.
When the feature is ready, we can say that's the preferred way, but inlining will continue to work. Just mention not to embed huge images.

Mention drawbacks : layout, theming,  mobile users
Copyright, licensing, IP issues

@codewars/community @ggorlen
I published a PR (#187) with an initial version. Examples are deliberately somewhat sparse, to show how problematic working with images can be, but maybe they should be improved?

From this review comment: https://github.com/codewars/docs/pull/187#issuecomment-748464654

about the data url part, the way to create them should be provided too (maybe just an external link?)
edit: or maybe, put the inlining svg § first, if the underlying goal is to not lead the users to use data urls.

@Blind4Basics what do you mean exactly by way to create them should be provided too (maybe just an external link?) ?
My idea here was not to spoonfeed users, just to point them in a direction and hope they could take it from there. Do you think examples present there are not enough?

Not sure... If the goal of the docs is to actually give all useful info, I think this should be given too. On my side, Idon't know how to create such a data url image, actually.
But again, if you prefere the idea of using the "svg stuff" rather tha "data urls", that's fine. I'd just push that § above the data url one.

You think a link to some tool, like this one: https://ezgif.com/image-to-datauri would be ok?

:+1:

Added some examples of tools, let me know if something needs improvements.
I also thought about mentioning Unicode characters, for example chess pieces to draw a chessboard. Do you think it would be good to have?

Possible, yes, but not in the "image embedding" part x)
Tho, I guess it's a bit specific, so just talking about them with some examples might be enough, yes (I'd put that in the section(s) about writing descriptions, but that's useful in testing feedback too, so....?)


Document changes to /ping API

Changes /ping route to return status code 200 instead of 204 when verbose is set
URL for relevant page?
https://docs.influxdata.com/influxdb/v1.6/tools/api/#ping-http-endpoint
What products and version are you using?
InfluxDB OSS 1.7.0
Where did you look before opening the issue?
https://github.com/influxdata/influxdb/pull/10321
Some older context which is useful to understand more can be found here:
https://github.com/influxdata/influxdb/pull/9784
If the user requests "verbose=true" along with the /ping request, HTTP status code may be different than without the "verbose" setting.

Added to influxdb-1.7 branch. Pending InfluxDB 1.7 release to appear in the docs.

Closed -- verified in 1.7 docs. https://docs.influxdata.com/influxdb/v1.7/tools/api/#verbose-option


Ported to HBL and packaged into a sort-of library format.

The program has been ported into a format that can be loaded in HBL.
The keyboard can now be opened by calling launchKeyboard(&returnedString); (with char returnedString[256] abive it). Once the user presses HOME, the string is returned and can be re-printed to the screen using OSScreenPutFontEx.
Most importantly, any user can easily insert the keyboard into their HBL program by copying draw.c, draw.h, keyboard.c and keyboard.h to their own project.

I haven't tested it, but from what the comment promises, I have to say that I'm impressed! I had not thought that somebody would still enhance this little project!


HIE seems to be able to typechek only one module

I use HIE with LanguageClient_neovim. Versions:
$ ~/.local/bin/hie --version
Version <IP_ADDRESS>, Git revision e159e35a5471bcfa8576d663cb82180dc9fe53ab (dirty) (1555 commits) x86_64 ghc-8.2.2
$ stack --version
Version 1.7.1, Git revision 681c800873816c022739ca7ed14755e85a579565 (5807 commits) x86_64 hpack-0.28.2

I build HIE with $ stack --resolver=lts-11.2 --stack-yaml=stack-8.2.2.yaml install, since we have LTS-11.2 in the project I develop.
The project consists of a library and an executable in separate Cabal files. The executable is built with -dynamic, so I also have to add -fexternal-interpreter to its Cabal file.
However, it seems that HIE can typecheck only the first file I open in nvim. All subsequent files result in didn't get typechecked module. Here's a part of log with --debug --vomit:
2018-07-12 23:45:52.549029635 [ThreadId 4] - ---> {"jsonrpc":"2.0","method":"textDocument/didOpen","params":{"textDocument":{"languageId":"haskell","text":" ...
2018-07-12 23:45:52.550559802 [ThreadId 10] - ****** reactor: got message number:3
2018-07-12 23:45:52.55069708 [ThreadId 10] - ****** reactor: processing NotDidOpenTextDocument
2018-07-12 23:45:52.5508884 [ThreadId 9] - ghcDispatcher:got request 3 with id: Nothing
2018-07-12 23:45:52.749975698 [ThreadId 9] - file mapping state is: fromList [("/../MainWindow.hs",FileMapping {fmPath = "/tmp/ghc-mod24974/MainWindow24973-0.hs", fmTemp = True}),("/.../WalletTree.hs",FileMapping {fmPath = "/tmp/ghc-mod24975/WalletTree24973-1.hs", fmTemp = True})]
2018-07-12 23:45:52.750217547 [ThreadId 9] - ghcDispatcher: top of loop
2018-07-12 23:45:52.750346578 [ThreadId 9] - ghcDispatcher:got request 3 with id: Nothing
2018-07-12 23:45:52.750479733 [ThreadId 9] - ghcDispatcher:Processing request as version matches
2018-07-12 23:45:52.806597695 [ThreadId 9] - ghcDispatcher: top of loop
2018-07-12 23:45:52.806739459 [ThreadId 9] - ghcDispatcher:got request 3 with id: Nothing
2018-07-12 23:45:52.806801621 [ThreadId 9] - ghcDispatcher:Processing request as version matches
2018-07-12 23:45:52.827888132 [ThreadId 9] - setTypecheckedModule: file mapping state is: fromList [("/home/maks/Projects/SRK/ariadne/ui/qt/src/Ariadne/UI/Qt/MainWindow.hs",FileMapping {fmPath = "/tmp/ghc-mod24974/MainWindow24973-0.hs", fmTemp = True}),("/home/maks/Projects/SRK/ariadne/ui/qt/src/Ariadne/UI/Qt/Widgets/WalletTree.hs",FileMapping {fmPath = "/tmp/ghc-mod24975/WalletTree24973-1.hs", fmTemp = True})]
2018-07-12 23:45:52.828050778 [ThreadId 9] - setTypecheckedModule: before ghc-mod
2018-07-12 23:45:52.85245996 [ThreadId 6] - <--2--{"jsonrpc":"2.0","params":{"uri":"...","diagnostics":[...
2018-07-12 23:45:54.599552113 [ThreadId 9] - setTypecheckedModule: after ghc-mod
2018-07-12 23:45:54.600437366 [ThreadId 9] - setTypecheckedModule: Didn't get typechecked module for: "/home/maks/Projects/SRK/ariadne/ui/qt/src/Ariadne/UI/Qt/Widgets/WalletTree.hs"
2018-07-12 23:45:54.600668187 [ThreadId 9] - ghcDispatcher: top of loop
2018-07-12 23:45:54.600838194 [ThreadId 6] - <--2--{"jsonrpc":"2.0","params":{"type":1,"message":"Got error while processing diagnostics: {handle: <file descriptor: 14>}: GHCi.Message.remoteCall: end of file"},"method":"window/showMessage"}

Also, ps indicates that ghc-iserv-dyn process dies after this. How can I debug this further?

I started to get this error as well:
[Error] Got error while processing diagnostics: readCreateProcess: /home/maks/work/haskell-ide-engine/.stack-work/install/x86_64-linux-ncurses6/lts-11.13/8.2.2/libexec/x86_64-linux-ghc-8.2.2/
How can I debug this?
Bump :)

Does nuking .stack-work resolve anything?

I've recently moved to a new laptop, so I had clean .stack-work, but it did not help. In fact, now I'm seeing the new error described in my second comment. This can be due to some changes in HIE or due to the fact that we switched from ghc 8.2 to ghc 8.4.
So I'm sure cleaning .stack-work won't help now and it's a rather long rebuild.

You said you switched to 8.4 but the error message implies that it is using 8.2, could that be a reason?

Sorry, that was actually 8.0 to 8.2, not 8.2 to 8.4 :)
Specifically, we use LTS-11.13. I've build HIE with stack --resolver=lts-11.13 --stack-yaml=stack-8.2.2.yaml.

Does that happen with any project you use or just with this one specifically? Is there any way we can reproduce this problem?

Hm... I crafted a simple project with two modules and -dynamic, hie seems to work fine. I tried adding dependency to qtah (dynamic-only bindings to Qt), but still could not reproduce the error. Can you suggest something I can try on the actual project?

Tried it once again in our project, here's complete HIE log with --vomit: https://gist.github.com/maksbotan/412ea588248d05b52a5f399227c59e92
We have three packages, let's call them A, B and C. B and C depend on A, B is ordinary and C is built with -dynamic. HIE works fine on modules from A and B, but fails on C. I have added -fexternal-interpreter to -dynamic packages as earlier errors recommended.

Hmm... After some experimenting I found that HIE should call ghc-mod with -Wwarn, since it will fail entirely when I have -Werror in my cabal file, but I'd like to see the exact warning in my editor.
I reverder https://github.com/haskell/haskell-ide-engine/pull/374/files changing Wall to Wwarn and it seems to work now.
Also I had to run that long cabal-helper command with stack exec --, seems like HIE runs it with some wrong path. How should it be handled properly?

Thanks for -Werror fix! However, I'm still getting this in log:
[Error] Got error while processing diagnostics: {handle: <file descriptor: 14>}: GHCi.Message.remoteCall: end of file
[Error] Got error while processing diagnostics: ghc-iserv terminated (-11)
[Error] Got error while processing diagnostics: ghc-iserv terminated (-11)
[Error] Got error while processing diagnostics: ghc-iserv terminated (-11)

Overall, hie seems to work rather unstably.

It seems like this issue has to do with the -dynamic flag, right? Have you experienced the same problem when working in other condebases at all?

Yes. I enabled -fexternal-interpreter as suggested by error from ghc-mod and then this happens. Tried right now with killing .stack-work in the relevant package.
I open one file -- everything is OK. I open another -- ghc-iserv-dyn dies. Is there a way to make it output some logs to see what's going on?

@DanielG Do you know anything about this?


Allow team name to be changed

The player who registered a team (lowest ID belonging to that team) should be able to change the team name. This will cut back on the occasional support request to game admins.
Renaming the team should be available to admins as well.

:( wrong repo


[layers] Add trainable feature

Add trainable feature for each layer which allows certain layers to just not train without affecting others
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #255. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


Fix crash on invalid tournament

Not sure yet if it fixes the issue

May I suggest opening a message box if the tournament link doesn't work?

I can confirm that the fix prevents the software from crashing but the end user is not informed that there was an issue with loading the tournament, maybe this can tie in to a fix for Bug #284?


Deleted files are still displayed in assetPicker and only loads the initial first page of 80 data

Describe the bug
bug1:
Deleted files are still displayed in assetPicker and I don't know why
bug2:
Assetpicker only loads the initial first page of 80 data
How to reproduce
I just checked out the latest version of the AssetPicker example project and found the bug
At first, I thought the reason for Bug2 was that Bug1 threw an exception and interrupted the program so that more data was not loaded. However, after I ran the example program and clicked the Video Picker Item, I found that the selected Video also showed only 80 data on the first page
Steps to reproduce the behavior:
1.Just run the Example project
2.Click on all item
Expected behavior
I hope to find out the cause of the bug and fix it
Screenshots (If contains)
bug1:

some log:

bug2:
Only the first page of data is loaded, no screenshots
Version information

Device: redmi k30 5g(MIUI 12.0.5,Android 10), HUAWEI mate 20(HarmonyOS 2.0.0), HUAWEI mate 40 pro(HarmonyOS 2.0.0)
wechat_assets_picker: 7.0.5
wechat_camera_picker: 3.0.1

Additional context
I have tested three Android devices and found these bugs. I hope you can help solve them. Thank you

This has been already tracking at fluttercandies/flutter_photo_manager#734, closing.


chore: migrate lib to use module augmentation

@nythrox I have started this, migrated Either and all the tests pass except two that depends on Maybe

I will revert the change on EitherAsync if need be.
I think it will make sense to break this PRs apart as they tend to be a lot of changes

Oh that's cos of the locks generated, I guess we need to get rid of one of them. Let me know what you think

closed in favour of #44


Don't try to re-read data unless SSL_MODE_AUTO_RETRY is on

We can also retry if we know we have data waiting to be read.
Fixes #18887

This should probably be backported to 3.0 and 1.1.1 but it won't cherry-pick cleanly there - so new PRs will be needed.
I don't plan to merge this until #18132 has been merged to avoid the potential conflict mayhem that will result.

This PR is in a state where it requires action by @openssl/otc but the last update was 61 days ago

Needs rebase

Ping for rebase

ping @mattcaswell is this still relevant?

Abandoning this PR


PyPI Package

Could you push FastMoE to PyPI?
As a workaround, we're currently using git+https://github.com/laekov/fastmoe in our requirements.txt, but a proper pip integration would be very much appreciated.

You're right, that might cause some issues.
DeepSpeed simply ignored a large part of these dependencies and put a note in the readme. Would this be a "solution" here?

This does not sound elegant to me. For example, we act differently when compiling our C code with different versions of PyTorch. So, there should definately be different versions of FastMoE binary according to PyTorch version. That is why I still insist that the users should compile FastMoE in their own environment. Anyway, NVIDIA's apex also requires users to compile locally.
I suppose an elegant way should be creating a script to automatically compile and pack FastMoE with the combinations of dependents. We may use spack to handle different versions of python, cuda, and NCCL. But I still do not have an idea about how to deal with different PyTorch versions. I am not quite motivated to dig deep into this, and looking for contribution from the open source community.


simplebar.js:3780: Unable to preventDefault inside passive event listener invocation.

Hi! Any pointers on what could be causing this error would be much appreciated.
Describe the bug
The following console error when clicking the scrollbar: "simplebar.js:3780 Unable to preventDefault inside passive event listener invocation" (latest Chrome on macOS). Similar error in Firefox.
Actual functionality seems to be unaffected.
To Reproduce

Visit https://hoard.fyi in Chrome
Click scrollbar on homepage carousels

Oh, I've actually just noticed it happens on your demo site also: http://grsmto.github.io/simplebar/

Thanks for your report, it's actually already fixed in #358 and should be merged  soon.

Released.


Use ~/.profile file (fixes #618)

Bash (at least on mac) loads from ~/.profile as well

Could you please find a list of all files both shells load for interactive sessions in their documention and add all of them here to close it once and forever?

Done. Used the list from each shell's man page, from the one I have on my system. I haven't tried the zsh one since I don't use zsh.

You're awesome, man.


breaking: move iteration away from module

merging mcaf-svm-ea and mcaf-svm-csp + reworking the module to make a single subscription (a.k.a. removing the for_each loops inside the module) since creation of multiple subscriptions should be orchestrated from the module caller)

@Blankf changes to output look good to me, ready for merge


Xbox auxiliary optimization for tiling

Make use of XG's CopyFromSubresource and CopyToSubresource methods
Add CalculateSubresource helper to TexMetadata

Test coverage for non-Auxiliary stuff: https://github.com/walbourn/directxtextest/pull/39


Icon for the plugin

Please add some icons to this and other Microsoft's plugins for IntelliJ platform.(e.g. Azure Toolkits for Java) Currently they look like unofficial plugins. :sweat_smile:

Thanks for suggestion, we've done that in #317.


Add a license before pushing code

Most Orcasound projects have defaulted to a permissive MIT license, with copyright to the Orcasound open source community (not a legal entity). Here is an example from the code developed by the all-volunteer Microsoft hackathon teams over the past few years.
Happy to discuss our experience to date with open source licenses, as well as Creative Commons licenses for data, and -- most recently -- emerging licenses for models that address ethical use of AI.

Hey @jonom and @ben-hendricks, I added an MIT license here with this copyright notice:
Copyright (c) 2023 Orcasound, North Coast Cetacean Society, SoundSpace Analytics (v.1)


New-controls

Addresses #103
Addresses #77

[x] Reference the issue your PR is fixing
[ ] Assign at least 1 reviewer for your PR
[ ] Test with run_dwelling.py or other script
[ ] Update documentation as appropriate
[ ] Update changelog as appropriate

Closed this because the PR should be merged into dev, not main.


add basic ui

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

i was getting CORS issues with the latest strategy so moving to a fullstack sveltekit


Authorization failed in Sandbox

Hi,
I'm still testing in the sandbox and I get authorisation failed with API Call Log like:

Call Timestamp
API Call Attempted
API Call Body
Key / Token Match
Response Body to Seller

2018-06-22 07:16:05 AM
Authorization
No Items Selected.
No
Authorization Failed

I did one successful sandbox test using Postman following the instructions at:
https://www.2checkout.com/documentation/sandbox/payment-api-testing
generating the key from the pastebin code and running in postman as advised.
the error could perhaps relate to:
#41 2co.js can't be loaded

2checkout have replied confirming telephone number is a required field.
This was causing failures in the testing since telephone number was not a required field in the woocommerce setup.
Since 2checkout is requiring the phone number but the shop and the shop managers do not want and therefore should not store the phone number (under data privacy rules), then the plugin should implement the solution.
2checkout:
"
The "phoneNumber" parameter is among the required attributes that need to be passed when creating a sale. For more information regarding the required attributes please access our Payment API documentation:
https://www.2checkout.com/documentation/payment-api/create-sale
Our Sandbox environment has limited resources and the only supported currencies are USA, EUR, GBP, therefore testing with a different currency will generate failed transactions.
Also you may find our test data for Sandbox accounts: https://www.2checkout.com/documentation/sandbox/test-data
"

@Jon007 any solution on that? I am using the outdated-official 2Checkout plugin for Woocommerce, but I get the 503 response from https://sandbox.2checkout.com/checkout/api/2co.js 503 (Service Unavailable).

Hi @Jon007 , thanks for the input! I just moved to live environment and enabled Demo, so my credit card is not charged. Now is working.


run browser tests on GHA

Once #4459 lands, the only thing building on Travis-CI will be the browser tests.  I'd like to see those moved as well, but I'm struggling to get it working (example).
This CI config is where I left off:
  test-browser:
    name: 'Test Browsers'
    runs-on: ubuntu-latest
    steps:
      - uses: actions/setup-node@v1
        with:
          node-version: 12
      - name: Get npm cache directory
        id: npm-cache
        run: |
          echo "::set-output name=dir::$(npm config get cache)"
      - uses: actions/checkout@v2
      - name: 'Cache node_modules'
        uses: actions/cache@v2
        with:
          path: '${{ steps.npm-cache.outputs.dir }}'
          key: "test-browser-${{ hashFiles('**/package-lock.json') }}"
          restore-keys: |
            test-browser-
      - name: Update to Latest Chrome Stable
        run: |
          sudo apt-get update
          sudo apt-get --only-upgrade install google-chrome-stable
      - name: Install Dependencies
        run: npm ci
      - name: Open Sauce Connect Proxy
        uses: saucelabs/sauce-connect-action@master
        with:
          username: '${{ secrets.SAUCE_USERNAME }}'
          accessKey: '${{ secrets.SAUCE_ACCESS_KEY }}'
          tunnelIdentifier: 'github-actions-${{ github.run_id }}_${{ github.run_number }}'
          sePort: 4444
      - name: Run All Browser Tests
        run: npm start test.browser

In addition, some changes to karma.conf.js will be necessary, e.g.:
// karma.conf.js:L86

  // configuration for CI mode
  if (env.CI) {
    console.error('CI mode enabled');
    if (env.GITHUB_WORKFLOW) {
      console.error('GitHub Actions detected');
      bundleDirPath = path.join(BASE_BUNDLE_DIR_PATH, env.GITHUB_RUN_ID);
      // correlate build/tunnel with Travis
      sauceConfig = {
        build: `GitHub Action #${env.GITHUB_RUN_NUMBER} (${env.GITHUB_RUN_ID})`,
        tunnelIdentifier: `github-actions-${env.GITHUB_RUN_ID}_${env.GITHUB_RUN_NUMBER}`,
        startConnect: false
      };
      console.error('Configured SauceLabs');
    } else {

For whatever reason, I could not get Karma to try to use a port other than 4444.  This seems to be the default, but the Sauce Connect default seems to be 4445.

I wonder if @christian-bromann is bored and might want to shoot a pointer my way 😇

Let me take a look at that!

Ok, some things I noted:
        [sauceBrowser]: {
          base: 'SauceLabs',
          browserName,
          version,
          platform
        }

these are old capabilities, I recommend to update them to:
        [sauceBrowser]: {
          base: 'SauceLabs',
          browserName,
          browserVersion: version,
          platformName: platform
        }

Then it seems that it tries to run a browser in CI instead of on Sauce Labs. Let me try to setup and example repository to see if I run into the same.

I created a minimal example project that shows the config needed to run the tests on Sauce Labs. Looking at the error from the failing GH workflow above I assume something is not correct with the fact that the tests are suppose to run on Saucelabs.
Here are some references that might help resolve the issue:

Minimal example project: https://github.com/christian-bromann/saucelabs-karma-example
Passing GH workflow with full Karma config: https://github.com/christian-bromann/saucelabs-karma-example/runs/1241406904?check_suite_focus=true

Hope this helps to investigate. Let me know if I can help with anything else.

@christian-bromann Thanks for taking a look.  Karma works fine on Travis-CI as configured, so I was hoping there was some misconfiguration around Sauce Connect in the GH action that you'd recognize.
Karma does not open the connection itself in CI, since we want to reuse the proxy for several runs (otherwise it's slow).
Karma does try to run a browser on the CI agent--it runs headless Chrome regardless of whether or not SauceLabs is used.  If it's in CI, Karma will run headless Chrome + the saucelabs browsers.  If it's not in CI, it just runs headless Chrome.

Anyway, I'll take a look at that example repo and see if I can't get it working.


Import Citizens API | Occupation not populating

What version of Node.js are you using?
v16.13.2
What operating system are you using?
Ubuntu 20.04 LTS
What version of SnailyCAD are you currently using?
v1.63.0 82cb9eb
Describe the Bug
[{
"userId": "clixqhacq00009guf9tlofdaj",
"name": "Test 3",
"surname": "Lockhart",
"dateOfBirth": "1991-05-24",
"gender": "clijqawor05ni9g7c3lvngzr1",
"ethnicity": "clijqaqou05na9g7cg601fprd",
"hairColor": "Black",
"eyeColor": "Green",
"postal": "564",
"address": "Whispymound Drive",
"weight": "120",
"height": "5'6"",
"driversLicenseId": "clijqb61p05nq9g7ca9l99g8d",
"occupation": "Testing 123"
}]
The above json when posted to the import/citizens address creates a citizen correctly, but occupation is still null.
Expected Behavior
I thought that all the fields that are provided when GET a civilian would be able to be specified when importing one using POST.
To Reproduce
I used Postman to generate POST requests to my /v1/admin/import/citizens with the json you can see in the "Describe the Bug" field.

Supported now!

Can confirm this worked :)


Update for SE-0055: Making pointer nullability explicit

See apple/swift#1878. Note: this needs a coordinated merge, i.e. it must go in ASAP after that request!

@mxcl or someone else, would you mind reviewing?


FATA[0040] unsupported restart policy "unless-stopped", supported policies are: []

What is the problem you're trying to solve
FATA[0040] unsupported restart policy "unless-stopped", supported policies are: []
Describe the solution you'd like
It should be same as docker.
Additional context
No response

This is already supported, but you need containerd v1.7 (beta)

https://github.com/containerd/containerd/pull/6744

Good job! By the way, when will containerd v1.7 official be released?

Got it~


postfix: 3.4.14 -> 3.4.16

Motivation for this change
http://www.postfix.org/announcements/postfix-3.5.6.html
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

LGTM.
Result of nixpkgs-review pr 94458 1

6 packages built:
- mailman
- postfix
- python37Packages.mailman
- python37Packages.mailman-hyperkitty
- python38Packages.mailman
- python38Packages.mailman-hyperkitty


ServiceConfiguration.apiToken dropped when doing ClientConfigurations.of

What would the client do with this token?

Ping


Value from bind:value in <Input type="number"/"range"> is "string" and not "number"

In this example code using vanilla Svelte and TypeScript:
<script lang="ts">
    import { Input } from "sveltestrap"
    let someVariable: number
</script>

<Input type="number" bind:value={someVariable} /> <!-- Type 'number' is not assignable to type 'string'. -->

the following error gets thrown by the TypeScript compiler: "Type 'number' is not assignable to type 'string'.", whereas if I would use the vanilla <input> element everything would be fine and the compiler would understand that the value is actually a number.
Environment information:
Sveltestrap 5.4.0
TypeScript 4.3.5
Svelte 3.38.3
Linux (pop os) (kernel 5.11.0-7620)

Thanks, I'm assuming the d.ts here is wrong: https://github.com/bestguy/sveltestrap/blob/master/src/Input.d.ts
Though also assuming string | number is not sufficient.  Open to suggestions and will take a look this week

Thanks!

Hi @marzeq ,
This should be corrected by #329 , and I've published a prerelease version<EMAIL_ADDRESS>if you could try and confirm it corrects your issue.  It looks okay to me locally.

Released in<EMAIL_ADDRESS>


[FLINK-27499] Test PR do not merge

This is a for of @SteNicholas 's work to eliminate some outstanding issues

@SteNicholas I managed to Fix your PR with a simple trick, had to disable the overlapping class test.
I simply copied the Flink 1.15 request bodys and removed the required formatType + added include nonnull
cc @wangyang0918

I know that this is not a pretty solution but anything else involves 5 times as much class / method copying from Flink. The RestClusterClient is not very extensible :/


Move to using pydantic for configuring Symphony Adapter

This should probably be a minor version bump, it breaks backwards compatibility since we now rely on the config


Updated documentation's example code for Popups

Hi, only a small PR here.
I've removed the 'tk.' which is not needed, and I've included the proper imports for the text popup example to work.

Nice. Thank you.


Consider the requirement of a field to be readonly for record/object subtyping

Purpose
$title - if the field in the target type is a readonly field, the relevant field in the source type should also be readonly either explicitly or due to the source type itself having its read-only bit set.
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/23468
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Is the following disallowed by design?
type Person object {
    string name = "John";
    readonly int age;

    function __init(int age) {
        self.age = age;
    }
};

Gives the error
error: .::test.bal:13:9: cannot update 'readonly' object field 'age' in 'Person'

Is the following disallowed by design?
type Person object {
    string name = "John";
    readonly int age;

    function __init(int age) {
        self.age = age;
    }
};

Gives the error
error: .::test.bal:13:9: cannot update 'readonly' object field 'age' in 'Person'

I assume you mean to set the value in the initializer method? If so it has to be init right?
The following doesn't give any errors.
type Person object {
    string name = "John";
    readonly int age;

    function init(int age) {
        self.age = age;
    }
};

The error in your sample is expected, because we don't allow setting values for object fields in methods other than in the init.

Argh.. Forgot we changed it to init :sweat_smile: Even in the ini(), should only be allowed to set it once right?
type Person object {
    string name = "John";
    readonly int age;

    function init(int age) {
        self.age = age;
        self.name = "foo";
        self.age = age + 20;
    }
};

This is probably not related to this PR though

Argh.. Forgot we changed it to init  Even in the ini(), should only be allowed to set it once right?
type Person object {
    string name = "John";
    readonly int age;

    function init(int age) {
        self.age = age;
        self.name = "foo";
        self.age = age + 20;
    }
};

Yeah, I believe the plan is to have this be an error. This is not yet implemented. https://github.com/ballerina-platform/ballerina-spec/issues/532 is the related spec issue and I just created https://github.com/ballerina-platform/ballerina-lang/issues/24119 to track fixing the implementation.


added mypy type hints

:building_construction: updated project to support mypy type annotations. Added init.pyi stub file and py.typed marker file. Updated setup.py to include new files
:memo: removed lint warnings from .md files
The project has been repackaged to allow introducing the new mypy type hint stubs
Closes #21

This looks good, thank you! I don't have whatever tool you're using to format the markdown, so would you mind resetting those?

I just removed some empty spaces to be compliant, nothing fancy.
to test the new stubs, create a new project and install env_flag and mypy. Add a Python file that imports your function and then try to run mypy your_file.py. It should give you an error.
then try to build and install the updated version, go through the same process and the error should disappear

Can confirm it works! Thanks for the PR!

@HitLuca I installed the pyi plugin for the linter and it complained about too many statements per line in the pyi file. To make it happy I moved the types into the source code. It still seems to be working for me in a descendant project. Want to give it a try yourself to make sure?

can confirm it works


CSSOM Injection performance metrics

Im looking for some performance metrics in terms of on a large scale what the difference is between having StyleSheetManager with disableCSSOMInjection turned on and turned off, the same for disableVendorPrefixes as from what i can tell it seems to slow down the initial page load.
If anyone else has any insight into speeding styled-components up even further that would be a great help.

We're actually considering to remove the non-CSSOM injection. The reason for its existence is that previously, a lot of browsers used for development would "lock" styles inserted using CSSOM methods. That meant that in development it became impossible to edit styles with Chrome. This restriction has been lifted a while ago now however.
Ultimately that means that non-CSSOM injection is obsolete, and it was only ever meant to be used in development. That being said, some people enabled it because third-party scripts had a tendency to destroy CSSOM-related scripts. This is an unsupported case for us however. Whenever a third-party script is reading or destroying our DOM nodes, then all bets are off, similar to when the same would happen to React's root DOM node.
The entire code that describes the non-CSSOM injection is located here and very compact: https://github.com/styled-components/styled-components/blob/28b4c28f1ebd1ea5a34717df2fe49634461285b1/packages/styled-components/src/sheet/Tag.js#L61-L99
You'll be able to see that per rule we inject one TextNode. As you may guess from just looking at this, this is extremely slow compared to CSSOM insertion. So slow in fact, that we wouldn't even bother measuring it. We have a refactoring PR that will improve this, but we may only merge it and then delete it in the next major upgrade immediately: https://github.com/styled-components/styled-components/pull/3295

So in theory its best to run disableCSSOMInjection then?

@The-Code-Monkey no, the opposite. In development by default CSSOM is disabled in production it’s enabled. So for performance, it’s best to leave this prop alone and to make sure that, however you build your app, that process.env.NODE_ENV === “production” for production builds (Webpack et al will do that automatically when they’re building for production)


Move to 2.60.2 LTS

Notable changes at: https://jenkins.io/changelog-stable/#v2.60.2

[test]

thanks!
[merge]

[merge]

[test]

[merge]


"Source code" link in pkl packages website broken

The "source code" link on the pkl packages website is broken.
For example, here:
https://pkl-lang.org/package-docs/pkg.pkl-lang.org/pkl-go/pkl.golang/current/GeneratorSettings/index.html
The "source code" links to: https://pkl-lang.org/package-docs/pkg.pkl-lang.org/pkl-go/pkl.golang/current/GeneratorSettings/https%3A//github.com/apple/pkl-go/tree/v0.5.3/codegen/src/GeneratorSettings.pkl
Which does not work.

Thanks!

Looks like a regression in the documentation generator (looks like someone changed a proper URL resolve to string concatenation... 😬): if you select older versions 0.5.1 or 0.5.0, (but not 0.5.2 or 0.5.3) then the link works
(and then you can fixup the URL to version 0.5.3, of course, or notice that the end of the URL https%3A//github.com/apple/pkl-go/tree/v0.5.3/codegen/src/GeneratorSettings.pkl in the broken link can be URL-decoded to the same).

I'm here because I was curious about AppEnvCluster which is showing the same problem.


Set 60s cache timeout on errors

Fix for #99, not yet ready.

Changes Unknown when pulling 4b920d57e8d840d8df38d369f40a07bf9b6de2e3 on error-timestamp into ** on master**.

wb @coveralls, you're weird.

Could just put the skin first, it's still called back anyway.

What do you mean?

I think currently, we process the skin after the cape. Might make more sense to process the skin first.

Actually, couldn't we make both run completely separate of each other, since we're calling back either way? As long as we have the profile, we can request them at the same time.

That's what I'm trying to do, but something is failing

Alright, I'll take a look when I get home. I hated writing that networking sfuff

@jomo don't we have the problem of the cape/skin hash overriding the cached hash, since they're only stored one at a time now?

Nope. I have changed cache.save_hash so that it doesn't set a field in redis to "." if it's undefined.
So if skin_hash is undefined and cape_hash is 3f688e..., it should only set the c field and ignore the s field.
Redis doesn't touch the other fields when setting a new one, as can be seen here:
redis> hmset test foo 1
OK
redis> hmset test bar 2
OK
redis> hgetall test
1) "foo"
2) "1"
3) "bar"
4) "2"

Interesting, nice! :D

It seems like the tests were already broken in some commit in the master branch and only fail 90% of the time.
It's actually only failing at this test which runs 3 times during tests.
I will try to figure out which commit broke it

Apparently they are only braking now due to @ec0840794baee432ad82afd5f75ba76fd9c4c600. So the issue is still within this PR

:shipit:


JWT认证

修改jwt的实现，添加一个从postform中获取token的配置是不是会更完美？

need to follow issue request template and thanks ❤

maybe you  can raise a feature request issue here: https://github.com/hertz-contrib/jwt


Moving Java sample code and instructions to /java/ folder.

@zfong I totally agree. For this PR, I quickly wanted to create the folder structure which will allow us to publish other language samples.
Could you please create the necessary separation of instruction in your PR with Python samples?

Sure, I can do that.


How to run TestContainers within Docker

I'm building my maven project within a docker container itself (it's run on Jenkins). Since my build container does not contain docker environment itself I'm getting:
java.lang.IllegalStateException: Could not find a valid Docker environment
when running tests.
Is there a way to use your test containers within docker build container?

guess you need to take a look something like https://www.docker.com/blog/docker-can-now-run-within-docker/


Fix read-only validation for sata hard-drives

What this PR does / why we need it:
SATA hard-drives, in contranst to SATA cdroms, can't not have read only set because the SATA bus does not support it for disks.
This validation fix avoids running into libvirt sync errors like this:
command SyncVMI failed: "LibvirtError(Code=67, Domain=10, Message='unsupported configuration: readonly sata disks are not supported')"

More information can be found in https://bugzilla.redhat.com/show_bug.cgi?id=915162.
Special notes for your reviewer:
Release note:
Fix read-only sata disk validation

/lgtm


Add chain token to approved tokens

Fixes #317
Changes proposed in this PR:

Add chain token to approved tokens and symbols csv.

Makes sense but there is another thing I forgot to mention. If we don't have them in the CSV files the reward calculation won't include free assets.

I started going down the path of investigating the code that would need changing to allow this.
Then I realized: we don't care what the asset is priced in anymore, at all. Why:

Before, we did, because it was to bias to OCEAN or H2O as liquidity in AMMs.
But now, the token is just a unit of exchange. We don't need to bias to OCEAN or H2O as unit-of-exchange at all.
The blog post says this too actually. (But we just never propagated to code)

Therefore: we can deprecate all the approved tokens stuff! Instead, allow pricing in whatever token, it's ok.
You can do it via this issue / PR (and rename the issue + PR accordingly), or create a new issue + PR (and close the current ones)


Sprite example broken after #12582: Divide the single VisibleEntities list

Bevy version
The release number or commit hash of the version you're using.
5caf085da (from #12582)
Relevant system information
INFO bevy_diagnostic::system_information_diagnostics_plugin::internal: SystemInfo { os: "MacOS 14.2.1 ", kernel: "23.2.0", cpu: "Apple M1 Max", core_count: "10", memory: "64.0 GiB" }
INFO bevy_render::renderer: AdapterInfo { name: "Apple M1 Max", vendor: 0, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Metal }
INFO bevy_winit::system: Creating new window "App" (Entity { index: 0, generation: 1 })

What you did
$ RUST_BACKTRACE=full MTL_HUD_ENABLED= cargo run --example sprite

What went wrong
Looks like this:

Additional information
Works before this commit on 5c3ae32ab:
$ git checkout 5c3ae32ab
$ RUST_BACKTRACE=full MTL_HUD_ENABLED= cargo run --example sprite

#12945 is a fix


Set .class instead of .className in MaterialComponent.ts

An attempt at fixing #1356 (after npm audit fix).

Failed pictures (3): [failed-pictures.zip]

:postbox: Posted by artipost.io, through Travis build 710927232, for commit 6a1f635.

I am sorry but master is not ready for preact X and above. This only worked uptil preact 8


Writing large quantities of data into Hyperdb can be incredibly slow

I have not looked at this since v3.0.0-0 was released, but i dont think this issue has been resolved. Looking at the code now - it looks like the small cache that was present has been removed.
As discussed on gitter last month:

I did a bit of profiling to see where the bottle neck is in importing data into hyperdb - it looks like its raf.read().
 ticks parent  name
 49282   41.9%  node::Read(v8::FunctionCallbackInfo<v8::Value> const&)
 49282  100.0%    v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*)
 49257   99.9%      LazyCompile: *fs.read fs.js:645:19
 49257  100.0%        LazyCompile: *RandomAccessFile._read /Users/x/hyper-graph-cli/node_modules/random-access-file/index.js:92:45
 48824   99.1%          LazyCompile: *Storage.getNode /Users/x/hyper-graph-cli/node_modules/hypercore/lib/storage.js:108:38
 48466   99.3%            LazyCompile: *Storage.dataOffset /Users/x/hyper-graph-cli/node_modules/hypercore/lib/storage.js:70:41

After further analysis it looks like its caused by a cache overflow issue, but I think an unavoidable one unless you store all of hyperdb in memory which is probably not a good idea or do something super clever. The hyperdb writer already has a cache - https://github.com/mafintosh/hyperdb/blob/master/lib/writer.js#L25-L26. But when putting in new entries it looks like it needs to get information about the previous nodes to construct the trie. On a large db this quickly results in evictions from the cache as each new key has to get different nodes for the hypercore.

If you still have it, could you add the benchmark you used to write data?
So we can rerun it in the future (once this becomes a priority) and also
review it
On Thu, Mar 15, 2018, 05:22 Benjamin Forster<EMAIL_ADDRESS>wrote:

I have not looked at this since v3.0.0-0 was released, but i dont think
this issue has been resolved. Looking at the code now - it looks like the
small cache that was present has been removed.
As discussed on gitter last month:
I did a bit of profiling to see where the bottle neck is in importing data
into hyperdb - it looks like its raf.read().
ticks parent  name
49282   41.9%  node::Read(v8::FunctionCallbackInfov8::Value const&)
49282  100.0%    v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*)
49257   99.9%      LazyCompile: *fs.read fs.js:645:19
49257  100.0%        LazyCompile: *RandomAccessFile._read /Users/x/hyper-graph-cli/node_modules/random-access-file/index.js:92:45
48824   99.1%          LazyCompile: *Storage.getNode /Users/x/hyper-graph-cli/node_modules/hypercore/lib/storage.js:108:38
48466   99.3%            LazyCompile: *Storage.dataOffset /Users/x/hyper-graph-cli/node_modules/hypercore/lib/storage.js:70:41
After further analysis it looks like its caused by a cache overflow issue,
but I think an unavoidable one unless you store all of hyperdb in memory
which is probably not a good idea or do something super clever. The hyperdb
writer already has a cache -
https://github.com/mafintosh/hyperdb/blob/master/lib/writer.js#L25-L26.
But when putting in new entries it looks like it needs to get information
about the previous nodes to construct the trie. On a large db this quickly
results in evictions from the cache as each new key has to get different
nodes for the hypercore.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mafintosh/hyperdb/issues/74, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAW_VcZqGHyMojrS7lDoJxj7K6o-v8ziks5tekDogaJpZM4Sr4rF
.

I'm not sure how the issue was for @e-e-e but running pull request https://github.com/mafintosh/hyperdb/pull/83 benchmarking tool, I get very slow performance compared to leveldb.
Some examples:
# batch write numKeys: 10 with hyperdb
ok ~5.38 ms (0 s + 5377136 ns)

# batch write numKeys: 10 with leveldb
ok ~816 μs (0 s + 815669 ns)

# batch write numKeys: 10000 with hyperdb
ok ~507 ms (0 s + 507371342 ns)

# batch write numKeys: 10000 with leveldb
ok ~29 ms (0 s + 29185328 ns)

# single write numKeys: 100000 with hyperdb
ok ~58 s (57 s + 837967694 ns)

# single write numKeys: 100000 with leveldb
ok ~1.99 s (1 s + 989761075 ns)

The performance is "fast enough" with small here and there updates but starts to rapidly breakdown with larger batches. However it's not entirely fair comparison because leveldown is using C++ code and hyperdb is running purely on JS but JS shouldn't be that much slower.
Any ideas whether this is due JS VM or something in the code?

We've only been focused on making reads fast (still work to do there) so we have some easy wins in writes. Note that because of the crypto needed to make this p2p we'll prob never be as fast as level. Having the bench is great tho :)

@mafintosh awesome to hear :+1: I was suspecting that crypto is probably affecting the results and at least based on a very quickly look, there's nothing in the code turning it off.


Problems with esp8266 OTA

When I  try the OTA example found here:
https://github.com/esp8266/Arduino/blob/master/libraries/ArduinoOTA/examples/BasicOTA/BasicOTA.ino
I encounterd when uploading the OTA script didn’t seem to recognize my input IP. I was getting:
14:21:38 [CRITICAL]: Not enough arguments.
It seems on windows you must specify upload port in ” ” like this.
upload_port = “<IP_ADDRESS>”
When I set the IP in " " I  get this output when I do an upload:
esptool v0.4.6 – (c) 2014 Ch. Klippel
setting board to nodemcu
setting baudrate from 115200 to 115200
setting port from COM1 to <IP_ADDRESS>
espcomm_upload_file
stat .pioenvs\nodemcu\firmware.bin success
error: Failed to open <IP_ADDRESS>
error: espcomm_open failed
scons: *** [upload] Error -1
[ ERROR ]
Any Idea? I disabled the firewall and the IP is definitely OK. I tested the UDP port 8266 and it is open.
Also there is no activity on the serial monitor when I try OTA.
I think the problem is still in the IP address somehow it is not recognized properly.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29483823-problems-with-esp8266-ota?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2014790&utm_medium=issues&utm_source=github).

@igorjan  Can you try testing "ESPWebServer/HelloSever.ino" (inarduino ide examples) and see if you can access the mentioned IP-Address and also try to ping the ip-address and see if you get any response.

@itsjustvenky the HelloServer.ino works without a problem. I get "hello from esp8266!" in the web borwser.
I can ping the IP, also I tested the UDP port 8266 on the ESP which is used for OTA and it seems to be open.

@igorjan  which version of platformio are you using ?

@itsjustvenky platformio version 2.6.3

@igorjan try to specify IP as http://docs.platformio.org/en/latest/projectconf.html#upload-port . Don't pass it to console as argument.

Here you can see 3 attempts with difrent definitions of upload-port.
Attempt 1:
C:\Igor_i\ESP8266\Project1>platformio run --target upload --upload-port <IP_ADDRESS>
[12/30/15 16:19:27] Processing nodemcu (platform: espressif, board: nodemcu, framework: arduino)
BeforeUpload(["upload"], [".pioenvs\nodemcu\firmware.bin"])
"C:\Users\igor.platformio\packages\framework-arduinoespressif\tools\espota.py"
--debug --progress -i <IP_ADDRESS> -f .pioenvs\nodemcu\firmware.bin
16:19:31 [CRITICAL]: Not enough arguments.
scons: *** [upload] Error 1
========================= [ ERROR ] Took 3.44 seconds =========================
Attempt 2:
C:\Igor_i\ESP8266\Project1>platformio run --target upload --upload-port "<IP_ADDRESS>"
[12/30/15 16:19:47] Processing nodemcu (platform: espressif, board: nodemcu, framework: arduino)
BeforeUpload(["upload"], [".pioenvs\nodemcu\firmware.bin"])
"C:\Users\igor.platformio\packages\framework-arduinoespressif\tools\espota.py"
--debug --progress -i <IP_ADDRESS> -f .pioenvs\nodemcu\firmware.bin
16:19:49 [CRITICAL]: Not enough arguments.
scons: *** [upload] Error 1
========================= [ ERROR ] Took 2.63 seconds =========================
Atempt 3
C:\Igor_i\ESP8266\Project1>platformio run --target upload --upload-port '<IP_ADDRESS>'
[12/30/15 16:20:16] Processing nodemcu (platform: espressif, board: nodemcu, framework: arduino)
BeforeUpload(["upload"], [".pioenvs\nodemcu\firmware.bin"])
"C:\Users\igor.platformio\packages\tool-esptool\esptool" -vv -cd nodemcu -cb 11
5200 -cp '<IP_ADDRESS>' -cf .pioenvs\nodemcu\firmware.bin
esptool v0.4.6 - (c) 2014 Ch. Klippel<EMAIL_ADDRESS>setting board to nodemcu
setting baudrate from 115200 to 115200
setting port from COM1 to '<IP_ADDRESS>'
espcomm_upload_files
cons: *** [upload] Error -1stat .pioenvs\nodemcu\firmware.bin success
error: Failed to open '<IP_ADDRESS>'
error: espcomm_open failed
========================= [ ERROR ] Took 2.46 seconds =========================

Please read my comment above https://github.com/platformio/platformio/issues/417#issuecomment-168017241. Don't pass IP to terminal. See example http://docs.platformio.org/en/latest/platforms/espressif.html#over-the-air-ota-update
Add this line to your platformio.ini
upload_port = <IP_ADDRESS>

Isn't this a windows/click/esptool problem? If windows tokenizes periods as separate tokens, you need to quote it if you want to pass it in the command line.

@ivankravets sorry I misunderstood you. I was doing it with platformio.ini file all along.

@igorjan please wait for the new development version. I'll ask you to test it. 1-2mins

@igorjan could you paste directly yo terminal this
"C:\Users\igor.platformio\packages\tool-esptool\esptool" -vv -cd nodemcu -cb 11
5200 -cp "<IP_ADDRESS>" -cf .pioenvs\nodemcu\firmware.bin

Does it work?

@ivankravets
c:\Users\igor.platformio\packages\tool-esptool>esptool.exe -vv -cd nodemcu -cb
115200 -cp "<IP_ADDRESS>" -cf C:\Igor_i\ESP8266\Project1.pioenvs\nodemcu\firmwa
re.bin
esptool v0.4.6 - (c) 2014 Ch. Klippel<EMAIL_ADDRESS>setting board to nodemcu
setting baudrate from 115200 to 115200
setting port from COM1 to <IP_ADDRESS>
espcomm_upload_file
stat C:\Igor_i\ESP8266\Project1.pioenvs\nodemcu\firmware.bin success
error: Failed to open <IP_ADDRESS>
error: espcomm_open failed

@igorjan STOP STOP :no_good:
Did you upload OTA sketch before make OTA uploading?

No here problems with IP. Looks like you didn't upload OTA sketch and din't configured it.

damn... we totally forgot the basic question...

@itsjustvenky :) :cocktail: New Year...

esp_OTA.zip

Yes I did have the OTA example uploaded via serial before trying OTA. I even changed some strings in the serial.print so I'm sure that the correct sketch is uploaded.

@igorjan could you make OTA uploading using Arduino IDE? See https://github.com/esp8266/Arduino/blob/master/doc/filesystem.md#uploading-files-to-file-system

@namoaton tried your project it is the same

@ivankravets Will try arduino but have to install it first

@igorjan https://github.com/esp8266/Arduino/blob/master/doc/ota_updates/ota_updates.md
Try this arduino build : Arduino-PR-4107-BUILD-421
else you need to manually set the IP-Address in arduino preference files.

https://github.com/esp8266/Arduino/blob/master/doc/ota_updates/ota_updates.md#arduinoota

OK tried the arduino and it works without a problem. The sketch is the same.

@igorjan print here `platformio --version``

c:\Users\User.platformio\packages\tool-esptool>platformio --version
PlatformIO, version 2.6.3

@igorjan please install the latest development version http://docs.platformio.org/en/latest/installation.html#d-development-version
I've just made a few changes for you.

Now I have: PlatformIO, version 2.7.0.dev12
But the problem is the same
"C:\Users\igor.platformio\packages\tool-esptool\esptool" -vv -cd nodemcu -cb 115200 -cp "<IP_ADDRESS>" -cf .pioenvs\nodemcu\firmware.bin
esptool v0.4.6 - (c) 2014 Ch. Klippel
setting board to nodemcu
setting baudrate from 115200 to 115200
setting port from COM1 to <IP_ADDRESS>
espcomm_upload_file
stat .pioenvs\nodemcu\firmware.bin success
error: Failed to open <IP_ADDRESS>
error: espcomm_open failed
scons: *** [upload] Error -1
[ ERROR ] Took 11.14 seconds
[Finished in 11.882s]

@igorjan I've jot this error. Sorry, hard day is today. :sad:
Please wait for the new build

@igorjan try to reinstall the latest PlatformIO 2.7.0.dev13

esptool doesn't do OTA, its only for  local ports and not ip-address.
espota.py is the tool we need to try.
python espota.py -i <IP_ADDRESS> -p8266 -f .pioenvs\nodemcu\firmware.bin

@igorjan  can you give us the exact command you are trying ?

C:\Igor_i\ESP8266\Project1>platformio run --target upload
[12/30/15 18:06:01] Processing nodemcu (platform: espressif, upload_port: "<IP_ADDRESS>", board: nodemcu, framework: arduino)
(["upload"], [".pioenvs\nodemcu\firmware.bin"])
"C:\Users\igor.platformio\packages\tool-esptool\esptool" -vv -cd nodemcu -cb 11
5200 -cp "<IP_ADDRESS>" -cf .pioenvs\nodemcu\firmware.bin
esptool v0.4.6 - (c) 2014 Ch. Klippel<EMAIL_ADDRESS>setting board to nodemcu
setting baudrate from 115200 to 115200
setting port from COM1 to <IP_ADDRESS>
secons: *** [upload] Error -1spcomm_upload_file
stat .pioenvs\nodemcu\firmware.bin success
error: Failed to open <IP_ADDRESS>
error: espcomm_open failed
========================= [ ERROR ] Took 2.31 seconds =========================

@igorjan please remove double quotes

Thus command works without a problem
python espota.py -i <IP_ADDRESS> -p8266 -f .pioenvs\nodemcu\firmware.bin

Please install again the latest PlatformIO.

@igorjan
the actual command is
platformio run --target upload --upload-port <IP_ADDRESS>

@itsjustvenky for your command I get:
C:\Igor_i\ESP8266\Project1>platformio run --target upload --upload-port 192.168.
1.40
[12/30/15 18:13:43] Processing nodemcu (platform: espressif, upload_port: 192.16
8.1.40, board: nodemcu, framework: arduino)
(["upload"], [".pioenvs\nodemcu\firmware.bin"])
"C:\Users\igor.platformio\packages\framework-arduinoespressif\tools\espota.py"
--debug --progress -i <IP_ADDRESS> -f .pioenvs\nodemcu\firmware.bin
18:13:45 [CRITICAL]: Not enough arguments.
scons: *** [upload] Error 1

@ivankravets if I remove the double qutes. I get the error:
18:13:45 [CRITICAL]: Not enough arguments.
scons: *** [upload] Error 1
The command python espota.py -i <IP_ADDRESS> -p8266 -f .pioenvs\nodemcu\firmware.bin
works without a problem.

Ok, now please revert upload_port: "<IP_ADDRESS>" in platformio.ini.

@igorjan  I think "ivankravets" just fixed something related this ip-address. Please install latest dev. version.

I don't understand behaviour of your machine. Please revert back quotes. If it helps, I'll added them to code.

I don't see difference between "C:\Users\igor.platformio\packages\framework-arduinoespressif\tools\espota.py" --debug --progress -i <IP_ADDRESS> -f .pioenvs\nodemcu\firmware.bin and espota.py -i <IP_ADDRESS> -p8266 -f .pioenvs\nodemcu\firmware.bin

@igorjan Could you try without TCP/IP port? espota.py -i <IP_ADDRESS> -f .pioenvs\nodemcu\firmware.bin. Does it work?

@ivankravets notice that he's running esptool, not espota. Why would that work?
"C:\Users\igor.platformio\packages\tool-esptool\esptool" -vv -cd nodemcu -cb 115200 -cp "<IP_ADDRESS>" -cf .pioenvs\nodemcu\firmware.bin

OK this works too:
python espota.py -i <IP_ADDRESS> -f C:\Igor_i\ESP8266\Project1.pioenvs\nodemcu\firmware.bin

Also in version dev13 I get error
18:18:56 [CRITICAL]: Not enough arguments.
With or without qutes for upload port

I've just pushed dev14. Please try it

Dev 14 is the same
18:18:56 [CRITICAL]: Not enough arguments.
With or without qutes for upload port
If you guys want I can give you team viewer access to the computer so you can test yourself

@igorjan Can you paste your platformio.ini here?

Project Configuration File

A detailed documentation with the EXAMPLES is located here:
http://docs.platformio.org/en/latest/projectconf.html

A sign # at the beginning of the line indicates a comment
Comment lines are ignored.
Simple and base environment
[env:mybaseenv]
platform = %INSTALLED_PLATFORM_NAME_HERE%
framework =
board =

Automatic targets - enable auto-uploading
targets = upload
[env:nodemcu]
platform = espressif
framework = arduino
board = nodemcu
upload_port = "<IP_ADDRESS>"

Yes, please mail<EMAIL_ADDRESS>with TeamViewer Id/pass. Thanks

@igorjan can you post all the text from your console from the starting of platformio run...

@igorjan You should surround your code with three tildes:
~~~
like this
~~~

This seems to fix it:
        upload_port = env.get("UPLOAD_PORT", "").strip("\"\' ")
        if (upload_port and (
                upload_port.endswith(".local") or
                socket.inet_aton(upload_port))):
            env.Replace(
                UPLOADCMD="$UPLOADOTACMD"
            )

Guys, the problem was linked with launching espota.py. On this machine PyCharm IDE handles all Python files by default.
I'll fix it and paste directly current Python interpreter

@igorjan please try the latest dev15

:) This one works without problem
Thank you guys for your help, really appreciate it.

Thank you too that report it. I'm going to release 2.7.0 :christmas_tree:  :blush:

@itsjustvenky, @skorokithakis and @namoaton  - THANKS A LOT FOR THE HELP! :beers:

Thanks for a great project, @ivankravets!


Suggestion: Multilanguage

Hello, thank you very much for all your work, a suggestion, it would be good translations to be multilanguage, including the Spanish that is my language.

Yes it would be nice to be multilingual. I will look into making Rclone Browser able to use different language assets but then it will be up to the wider community to do translations.

Yes it would be nice to be multilingual. I will look into making Rclone Browser able to use different language assets but then it will be up to the wider community to do translations.

Great, what good news! You will see how the community collaborates because you do very good work, RcloneBrowser is spectacular, very good work is what you do.

I would be happy to help with Spanish translation once the ability to use different language assets is added

I would gladly help with the Italian translation ;-)

Unfortunately it is much bigger task than I expected. I don't have time to do it.

A pity to read that, well, thanks alike.

The biggest issue is that it is legacy application with core of it created by previous developer. It was not started with translation in mind and now would require rewriting tones of code.
Hopefully some Qt guru can take this task over - so even I am not able time wise to do it maybe it will happen one day.


Tester feedback: language

Question ID
is there an opposing party?

language
If I am the defendant/respondent, there will have to be a plaintiff/petitioner in this case, so I think we can skip this question.

Other information

Variable being sought
other_parties.there_are_any

Package version
1.1.4

Maturity target
Level 4 - this is purely an enhancement to improve user experience

Tester contact info
mnewsted

@mnewsted what if the case is an "In re" / "In the matter of"?
Maybe there's a way to phrase that more clearly for that situation


output specification in app metadata

This thread to discuss the design of output of apps as specified in their appmetadata, particularly in theproduces field.

closed via #53 .


Type checking issue in map constructing Query expressions for string:Char

Description:
$title.
Steps to reproduce:
function test() {
    [string:Char, int|float][] list = [["s", 1.4], ["s", 2], ["s", 3]];
    map<int|float> & readonly|error mp1 = map from var item in list select item; // incompatible type in select clause: expected [string,any|error], found '[string:Char,(int|float)]'
}

Affected Versions:
2201.2.0-rc1

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/37402.


make install.sh work in windows Git Bash / mingw shell

So I got install.sh to work with Git Bash, which I think is actually mingw shell, and I've tested it on Mac, Windows, and Ubuntu. #68

Not sure why ci / test (ubuntu-latest) failed.  Got to:
Deno was installed successfully to /home/runner/.deno/bin/deno

But then
./install_test.sh: 17: ./install_test.sh: ./install.sh: Permission denied
##[error]Process completed with exit code 126.


feat: Introduce Notifications Configuration CR for managing argocd-notifications-cm

What type of PR is this?

/kind enhancement

What does this PR do / why we need it:
Currently, Argo CD Operator supports Notifications as Beta. One of the primary reasons for this is, Argo CD Operator does not manage the configuration(templates, triggers, services, subscriptions e.t.c.,.) of the the argocd-notifications-cm.
Triggers, Templates, Services and Subscriptions can be so huge at times that it creates a bad user experience if they are added to the Argo CD CR and managed by the Argo CD Reconciler of the Argo CD Operator.
Have you updated the necessary documentation?

[x] Documentation update is required by this PR.
[x] Documentation has been updated.

Test acceptance criteria:

[x] E2E Test

How to test changes / Special notes to the reviewer:
Sample NotificationsConfiguration resource
apiVersion: argoproj.io/v1alpha1
kind: NotificationsConfiguration
metadata:
 name: default-notifications-configuration
spec:
 triggers:
  trigger.on-sync-status-unknown: |
    - when: app.status.sync.status == 'Unknown'
      send: [my-custom-template]
 templates:
  template.my-custom-template: |
    message: |
      Application details: {{.context.argocdUrl}}/applications/{{.app.metadata.name}}.

/lgtm


changes to reflect moving chm files to offline-docs branch

Moved the chm files to the offline-docs branch to keep the master branch .md content only to simplify the loc process.

@davidchesnut,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Offline-Docs/ACVBACon.chm
:white_check_mark:Succeeded

VBA/Offline-Docs/PPVBACon.chm
:white_check_mark:Succeeded

VBA/Offline-Docs/WDVBACon.chm
:white_check_mark:Succeeded

VBA/Offline-Docs/XLVBACon.chm
:white_check_mark:Succeeded

VBA/Offline-Docs/readme.md
:white_check_mark:Succeeded

VBA/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Offline-Docs/ACVBACon.chm
:white_check_mark:Succeeded

VBA/Offline-Docs/PPVBACon.chm
:white_check_mark:Succeeded

VBA/Offline-Docs/WDVBACon.chm
:white_check_mark:Succeeded

VBA/Offline-Docs/XLVBACon.chm
:white_check_mark:Succeeded

VBA/Offline-Docs/readme.md
:white_check_mark:Succeeded

VBA/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Crash in SkyFloatingLabelTextField textHeight

Report
What did you do?
used SkyFloatingLabelTextField in an iOS project using Carthage
What did you expect to happen?
App launch without problem, this mostly happens
What happened instead?
We have one crash reporting using Crashlytics. Crash report attached.
SkyFloatingLabelTextField textHeight crashed when force unwrapping self.font
We were using SkyFloatingLabelText 2.0.1 have upgraded to 3.4.0
Environment
Xcode Version: 8.3.3
SkyFloatingLabelTextField Version: 3.4.0
Deployment Target: 10.0, with crash on 10.3.2 (14F89)
Base SDK: iOS 10.3
Method of Integration: Carthage
Project that demonstrates the issue
We have only one report of this crash.
SkyFloatingLabelTextField crash during autolayout.txt

Hmm are you setting the font yourself our relying on the system font being set by the OS? I agree that the force unwrap is not ideal, however what you would do in its absence is a big question. Just using a line height of 0 is likely not what you'd want

Has this cropped up again or are you any closer to reproducing it?

I've added 9c50bbe just to make sure we can avoid this happening. I'll release it in a while as 3.5.1

Published as 3.5.1


Help

It's the same like yours but that is showing

@mihai200677 you need to enable intents when you create the bot


option: enable message "user wants to merge n commits into branch from branch"

When GitHub is used without refined-github, a message like this appears near the top of each pull request:

user wants to merge n commits into branch from branch

For example, PR #4016 in this repo shows:

yakov116 wants to merge 2 commits into main from markEmAll

When refined-github is used, that message is reduced to:

from branch

Again, for #4016, it shows:

from markEmAll

For some PRs I was working with, I wanted to be certain they were opened using the correct destination branch, but it isn't shown.
I looked through the extension options to find one that might enable the original message, but I couldn't find one.  Please add an option to enable this message.

All of that content is already on the page: https://github.com/sindresorhus/refined-github/issues/3089
The feature clean-conversation-headers can already be disabled, but I don't know why you would want duplicate information on the page.

using the correct destination branch, but it isn't shown.

The default branch isn't shown, it's the default.

Thank you for the pointer and the information.
I spent an hour searching through the extension options for phrases and words like "branch", "merge", "commits", "wants", etc.

Thanks, I updated the description of the feature. This is a recurrent issue


Reorganise notebooks for ease of teaching

Please run this branch and check for any missing dependencies and other snafus

Rectified minor bug and added cell outputs. LGTM.


V2 - CLA Not Signed - Identify Yourself as CLA Manager Designee - Error displayed after creating account

Prerequisite

Create a CLA group with CCLA enabled (I tried the flow where CLA Group is at foundation level and there is only 1 subproject for the foundation)
Add github repository for the CLA group

Steps to Follow

Login to Github as corporate contributor
Navigate to the repository used in prerequisite
Create PR
CLA check fails
Click on CLA NOT SIGNED
Contributor is redirected to Contributor Console
Click on Proceed as a Corporate Contributor
Select Company that has not signed CLA for the project
Click Proceed
Click Yes
Click Proceed
Create account
Complete Verification of email
Contributor is redirected to contributor console
Error is displayed however CLA Manager designee role is assigned. Also observed that CLA Manager designee role is assigned to both Foundation and Child project
Error message contains child project SFID though the CLA Group is at Foundation level

@nirupamav could you kindly as well share the response via console just to quickly identify the issue ?

@wanyaland I tried the flow again and below is the console log

Verified that the issue is fixed


Updated ET locale.

I am an estonian and made the text make more sense.

Thanks for the PR.
On nui_menu.misc.help_message can you change TxAdmin to txAdmin?

Yes done. Sorry for taking this long.

Why is it txAdmini instead of txAdmin?

Thanks again for the PR.
I just modified a few lines to prepare for the v5 update, if you don't mind, can you also translate those?
https://github.com/tabarra/txAdmin/blob/develop/locale/et.json
edit: don't forget to target your PR to the develop branch :)


FIX: fix wald friendman for incompatibility with numpy subtract with boolean

Looks good, although I'm confused why I wasn't having the error before

yeah me too - must be a version issue. Using python3.6 or python3.7 conda?


[BUG] Logstash static role should include the ecs-logstash pattern in index permissions

What is the bug?
Since logstash 8.x has the ecs v8 compatibility mode enabled by default for all ecs compatible plugins and the logstash-output-opensearch plugin sets the default index pattern for the ecs mode to be ecs-logstash-%{+yyyy.MM.dd} then the minimum index permissions for the builtin logstash static role should allow at least the creation and crud operations on all indexes following this pattern.
How can one reproduce the bug?
Steps to reproduce the behavior:
Use logstash in ecs compatibility mode and its builtin static role along with the logstash-output-opensearch plugin to ingest events into OpenSearch.
What is the expected behavior?
Logstash should have enough permissions to create and modify indexes with prefix ecs-logstash
What is your host/environment?
Logstash version: 8.9.0-oss
OpenSearch version: 2.9.0
Logstash output plugin: logstash-output-opensearch
Do you have any screenshots?
Error message:
[INFO ][logstash.outputs.opensearch][main][2a15a812080baab0c6fe9a5ece656b985ecb67109da737d3b3644d70e810668e] Retrying failed action {:status=>403, :action=>["index", {:_id=>nil, :_index=>"ecs-logstash-2023.11.19
", :routing=>nil}

[...]

error=>{"type"=>"security_exception", "reason"=>"no permissions for [indices:admin/create] and User
[name=logstash, backend_roles=[logstash], requestedTenant=null]"}}

[Triage] @dpavlos Thank you for filing this issue and PR. It looks like the role needs to be updated for all default index patterns. Thank you for the details.


Add docs for new Portfolio Item fields

https://projects.engineering.redhat.com/browse/SSP-173
This PR updates the openapi swagger docs with the new fields that we're
gathering on PortfolioItem.

@Hyperkid123 Please test

The service_offering_icon_id attribute is also missing and i will need this one very soon. I'm already working on icon integration, but waiting for some stuff on Topological API.
I don't know what data we store/need but because we are pulling same entities from two sources i think we should try and provide the same response as in Topological inventory API. There is a full list of attributes from their schema: https://github.com/ManageIQ/topological_inventory-api/blob/master/public/doc/swagger-2-v0.1.0.yaml#L1901

@Hyperkid123 service_offering_icon_id docs etc are taken care of in #112, so when that is merged it'll be taken care of.
I think right now we produce most of the same values that topology returns, I'll investigate what we aren't returning and create a story to get that done. - I'll update that the provider name to match topology.

I'll update that the provider name to match topology.

@lindgrenj6  Its supposed to be distributor. My hands wrote something else

@syncrou @gmcculloug @mkanoor  got this updated for merging - this was just updating some docs, could we get some eyes on it?


Pomodoro timer state machine implementation

Introducing Pomodoro Technique state machine.

State changes are implemented using Finite State Machine lib: https://github.com/looplab/fsm
Usage has not changed: ./bin/gopom start "my sweet task"
Pomodoro timer starts with 5sec delay, then indefinitely loops in states pictured above.
Current state times are as follows:

working: 25minutes
resting: 5 minutes
longer break 20minutes

Working and resting states cycles 4 times then enters longer break state.
Timer itself is rendered using refreshed ASCII and sounds are free to use samples generated in Audacity by me

This PR has to be moved to docs. Create new column i project to docs and move this issues  when this pr is merged.

great.


implement parser support for generated column definition

check that opening and closing parantheses are both required or optional, otherwise ack


[BUG] No update due to missing glyphs in monobit.ttf

Hi there. For some reason updates fail due to missing glyphs in monobit.ttf.
Any idea what changed?
Failed config

font: [source /config/esphome/ulanzi-archiv.yaml:76]

  Font monobit.ttf is missing 2 glyphs:

   (b'\\n')
      € (b'\\u20ac').
  - file: _fonts/monobit.ttf
    size: 16
    id: default_font
    glyphs: |-
      !?"%()+*=,-_.:°0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ abcdefghijklmnÖÄÜöäüopqrstuvwxyz@<>ß§€/

Seen in ESPhome on HomeAssistant
Core 2024.12.2
Supervisor 2024.11.4
Operating System 14.0
Frontend 20241127.7
Thanks and cheers

hi,
i think it is the euro sign and the backslash at the end

Thanks. € is one of them. Couldn't locate the other yet. Odd that they are suddenly declared as missing.

Use that
font:
  - file: ulanzi/font/monobit.ttf
    id: monobit_font
    size: 16
    glyphs: >-
      !?"%()+*=,-_.:°µ²³0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ abcdefghijklmnÖÄÜöäüopqrstuvwxyz$@<>/

make sure using >-

|- is OK too... Don't use punctuation at the end like a backslash.  Move all your punctuation to somewhere else.  Make z the last character.

Thanks for the help. Eventually it was just the € character. It now updated without any further error.
I just wonder why the error came up now - I hadn't touched the installation in quite a while and I had various successful updates in the meantime. Oh, well... as long as it works now...
Thanks again and cheers.


Extendend Envers Criteria API to express relation traversion

The api supports different join types on relation traversion. However, implementation is done only for
inner joins currently. Future commits will extend this implementation to support further join types.

Thanks. Is there any JIRA issue related to this? I see there's a significant amount of code change and wanted to see the design decisions behind those.

https://hibernate.atlassian.net/browse/HHH-3555

@ffeisst Does this PR completely supercede https://github.com/hibernate/hibernate-orm/pull/1079 ?

@sebersole Yes,  I've cherry picked from #1079 and changed some API methods accordingly. #1079 should be closed.

I have started the work of integrating this.  So please do not push any more work to this branch (it will get lost)

XYZImplementor in Hibernate generally is used to denote the SPI portion of
a contract (interface).  You seem to follow that convention
with AuditQueryImplementor, but then you name an
implementation AuditAssociationQueryImplementor.  Unless this is some break
in convention followed in Envers, I plan on
renaming AuditAssociationQueryImplementor to AuditAssociationQueryImpl
following the conventions used throughout the rest of Hibernate code base.
Also, for future reference you are not applying the copyright notices.  So
new files you have added have no copyright.  I will take care of the ones
in this PR, but for any future work please apply the copyright header.
Thanks
On Tue, Feb 9, 2016 at 2:08 PM Felix Feisst<EMAIL_ADDRESS>wrote:

@sebersole https://github.com/sebersole Yes, I've cherry picked from
#1079 https://github.com/hibernate/hibernate-orm/pull/1079 and changed
some API methods accordingly. #1079
https://github.com/hibernate/hibernate-orm/pull/1079 should be closed.
—
Reply to this email directly or view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/1258#issuecomment-182040010
.

Yes, AuditAssociationQueryImplementor should be renamed to AuditAssociationQueryImplementor. I'm sorry about the missing copyright notices, I will provide them in future work.

I just applied this work upstream.  Thanks so much Felix!