Prep commits for new avatar_url handling (#4157)

In a revision of my work for #4157, I noticed that there were some necessary changes to some test files, and some of these files haven't been type-checked or taken advantage of the work we've put into src/__tests__/lib/exampleData.js.
Rather than continue editing these files in this unstable state, pave the way for better edits by type-checking them and using that example data. 🙂  And fix a few bugs I've noticed.
The revision that informed these changes was one where the type for avatar_url on the Message type (not just that field on the User type) was changed. I think this might be a direction we'll go in (e.g., the type might be a union of all subclasses of AvatarURL except IdleAvatarURL); I'm thinking of the discussion around here. In any case, that explains why fetchActions-test.js was touched (fetchMessages is in there).
There will be some more of this; I know at least src/webview/__tests__/webViewHandleUpdates-test.js has some stuff that I'm still figuring out.

Just resolved some conflicts; awaiting review. 🙂

Just rebased again, following the RN v0.61 upgrade (no conflicts), awaiting review. 🙂

Ah, oops: Just found a few things to do better, so I pushed again; hopefully this didn't interrupt a review. I'll move on to something else now so you don't have to worry about possible interruptions again, and I'll post here if for some reason I need to make more changes.

Mm, looks like there are some conflicts—fixing those now.

And now more conflicts, from commits in https://github.com/zulip/zulip-mobile/pull/4187—I'll fix these now.

Thanks for the review! Just pushed those changes. You may want to see my note at https://github.com/zulip/zulip-mobile/pull/4171#discussion_r459224854 about the omit, but it's not Earth-shattering; I'll merge as-is in a day or two if you don't get around to reading it.

Actually, since you said at https://github.com/zulip/zulip-mobile/pull/4171#pullrequestreview-453847396 that it otherwise looks good, I'll go ahead and merge the changes to the fetchActions tests; that'll be helpful for me to get #4205 to a more reviewable state. The one comment you made on that (https://github.com/zulip/zulip-mobile/pull/4171#discussion_r459226185) was straightforwardly addressed (thanks for catching it!).

Actually, since you said at #4171 (review) that it otherwise looks good, I'll go ahead and merge the changes to the fetchActions tests; that'll be helpful for me to get #4205 to a more reviewable state. The one comment you made on that (#4171 (comment)) was straightforwardly addressed (thanks for catching it!).

OK, I ended up merging all the commits except for two you commented on that I don't have an immediate need for, so you can see my responses to them when you get a chance (https://github.com/zulip/zulip-mobile/pull/4171#discussion_r461234078 and https://github.com/zulip/zulip-mobile/pull/4171#discussion_r461235748).

I said:

There will be some more of this; I know at least src/webview/__tests__/webViewHandleUpdates-test.js has some stuff that I'm still figuring out.

OK, just pushed this. It was a little tricky getting prevProps and nextProps working with Flow, but I think I've found a good balance. I've decided not to bother faking the intl property of props._, but we don't currently have tests that use that, and I don't think we will soon.

The new commit lgtm!
Re omit, see reply above about a quick comment to add. Then please merge at will. Thanks for making all these tests better!

OK, done, thanks for the review!


bug: Variable Id { idx: 23 } was already renamed

Bug Report
Cairo version:
2.5.2

Current behavior:
Compiling the codebase works fine.
However, running scarb test -p contracts in the following branch fails. As such, we're blocked and cannot upgrade our codebase to 2.5 and are stuck using a nightly release.

Expected behavior:
There should be no issue.

Steps to reproduce:
Clone https://github.com/kkrt-labs/kakarot-ssj, checkout the branch of this pull request https://github.com/kkrt-labs/kakarot-ssj/pull/723 and run scarb test -p contracts

please check on 2.5.3 - should be solved on it.


[WIP] Replace use of impossible idiom with proof by reflection

The purpose of this WIP pull request is to slowly remove all usage of the impossible idiom by proof-by-reflection. A new development in Agda has made it possible to do this without using instance search.

[x] define product and unit types as records in Connectives;
[x] explain difference between data and records w.r.t. definitional equality in Connectives;
[X] define True in Decidable;
[X] add exercise for defining False in Decidable;
[X] replace S constructor in Lambda with a version which checks the inequality implicitly;

Explain how you can write x - y where x and y are known variables. I presume we can write something like
minus : (m n : ℕ) (n≤m : n ≤ m) → ℕ
minus m n n≤m = - m n {f n≤m}
for a suitable choice of f?

This technique is really only useful when both arguments to _-_ are literals. I'd prefer not to pretend that it has any value if you have any evidence that n ≤ m for two variables n and m, as the whole point is the fact that Agda can compute the answer for you, which goes out the window if you have to provide explicit evidence.
Furthermore, _-_ is defined in terms of minus, so  your code here ties a strange loop?

The exercise refers to True, but the text refers to T.

True is defined just before the exercise.

@wadler This is now done. Could you review the changes?


Do not crash after compile error

Right now we continue parsing/codegen after reporting errors and almost always end up crashing buzz.
Go over each error reporting and see how it must be handled: either stopping everything or providing enough details so buzz does not crash going forward.

Further cases should be reported in there own issue


Show preview of data on Accessor repr

As a user, I wish I could see the schema along with a preview of the Data when looking at the WoodworkTableAccessor's repr.

Currently, the repr is just that of the Schema class., but we'd like to show some of the data from the DataFrame along with the typing information as a multi index.

Current behavior --> https://woodwork.alteryx.com/en/latest/start.html#Creating-a-DataTable

this needs more discussion


Add a tutorial

This project is useful and would be easier to use if a step by step tutorial would guide the users on how to set this up on all backends and test it.

It would also be great to to know how one can check the cluster status.  For the native clustering and clusterer plugin, rabbitmqctl cluster_status checks.   Is it sufficient to check if Console sees the right number of nodes to know the cluster status?
Likewise, the other two clustering methods are known to rely on host reachability via DNS or /etc/hosts.  Is this also a requirement for autocluster?   The hope with using it is to eliminate needing to push unique configs onto each RMQ instance, and instead have them all point to the same Consul instance be done.

Second this

This plugin was forked by the RabbitMQ team and is now part of RabbitMQ. More information can be found @ https://github.com/rabbitmq/rabbitmq-autocluster


#320  use property setter/getter invocation instead of il.Emit to fix…

I know this will most likely be rejected but I wanted to ask you why do you need to use DynamicMethod instead of simple Getter/Setter invoke? This also fixes the #320

ILEmit is theoretically a LOT faster than reflection... that's why it's used.

And, in my tests, Expressions are fast as Emit. Current version uses Expressions (and a little of Emit to specific case)


Add "DOLBY AUDIO-DD+ +DSUR" sound mode

see https://github.com/home-assistant/core/issues/105514#issuecomment-1910534944

@ol-iver failed check seems unrelated to this PR


TCP retransmition is not correct

Description / Steps to reproduce the issue
tcp retransmission has an exponential regression mechanism. For example, if no ack is received for 1.5 seconds, the first retransmission will be sent. If no ack is received, the second retransmission will be generated after 3s. If an ack is received, the exponential retreat timer resets. But timing doesn't work that way. After receiving an ack for the first retransmission, no reset was performed. The tcp interval becomes larger when the subsequent transmission becomes retransmission.
On which OS does this issue occur?
[Linux]
What is the version of your OS?
ubuntu22.04
NuttX Version
unkown
Issue Architecture
[arm]
Issue Area
[Networking]
Verification

[X] I have verified before submitting the report.

@wengzhe please take a look

@Meissi-jian Do you have any idea about this? I saw you were modifying retransmission these days https://github.com/apache/nuttx/pull/13120

@ChenChuang-23 You can have a try with https://github.com/apache/nuttx/pull/15118 . And I think it will resolve you probrem.


Publishing the package to PyPI

Hi @thefab, @Hawker65 and @thebaptiste,
thanks for conceiving this excellent program. I just found it indirectly through [1].
Extending my research, there is also

cronwrap by @lowks, @imankulov and @desktable on behalf of @Doist
croncoat by @MatthiasKauer with contributions from @lowks, @KeyMasterOfGozer and @imankulov
Another cronwrap by @jonyesno written in Ruby
lockrun by @SJFriedl written in Bash
... and probably more.

cronwrap and croncoat are already available on PyPI [2,3] and there is also another tool called cronwrapper [4]. However, the release package of cronwrapper available on PyPI apparently does not even include any code. After looking inside, I found it might have been conceived by Banio Carpenter (@baniobloom) and is currently maintained by @ranamalo, who also hosts its source code at [5].
However, all the Python implementations apparently didn't receive any commits within the last two years. I didn't investigate thoroughly if some features might be missing compared to the other tools, but I believe your variant to be pretty complete and advanced from my perspective.
So, it would be really nice to see your variant of cronwrapper to be published to PyPI in any way. However, as the name is already taken on PyPI, we might either ask @baniobloom or @ranamalo whether they want to carry on with their variant of cronwrapper or you might think about giving your variant a different name.
With kind regards,
Andreas.
[1] https://askubuntu.com/questions/179943/getting-notified-of-failed-cron-jobs
[2] https://pypi.org/project/cronwrap/
[3] https://pypi.org/project/croncoat/
[4] https://pypi.org/project/cronwrapper/
[5] https://github.com/ranamalo/cronwrapper

cron-wrapper? (with a -)

https://pypi.org/project/cron-wrapper/
=> available

[x] rename repo
[ ] merge #18
[ ] add github action to build/test/publish to pypi

published! @amotl: can you check it's ok for you?

Hi Fabien,
thanks a stack. Installing the package from PyPI works like a charm. I appreciate your responsiveness on this excellent ride.
Have a good weekend and with kind regards,
Andreas.

I'm a little late to the party but one of the reasons I created cronwrapper originally was to be notified on failed cron jobs on boxes that did not have mail configured by sending those errors from a uniform error email account.  Those errors were sometimes non-zero exit statuses and sometimes scripts that exited with status 0 but had output to stderr.


Add if modifier to validates

Allow validation to be dynamically run or skipped depending on return value of proc passed to an if key in .validates options.
e.g.
validates :postcode, presence: true, if: -> (person) { person.country == "US" }

TODO

[X] implement if logic
[X] tests passing
[X] readme updated with usage docs
[ ] refactor modifier implementation to reduce responsibilities in Subvalid::Validators module. It's getting a bit crowded in there.

cc @amanda08 . Here's a quick implementation I worked on over the weekend based on what we were talking about on Friday

👍  Nice one @madlep !


changes to backtrace crate

https://travis-ci.com/rust-lang-nursery/error-chain/jobs/205173078
|
100 |             Symbol::Dladdr(ref s) => return s.addr(),
|                                             ^^^^^^^^ expected enum libc::c_void, found enum std::os::raw::c_void
|
= note: expected type core::option::Option<*mut libc::c_void>
found type core::option::Option<*mut std::os::raw::c_void>
error[E0308]: mismatched types

I think this is fixed now by the minimum version bumps?


New auto-generated D bindings

Updates the bindings-d.lua file to generate code for the new BindBC-bgfx bindings, which will be replacing the old bindings from now.

Why suddenly we need 17 .d files?

They house types to link against bgfx’s fake enum class types: bgfx::Type::Enum
It was more convenient for me to give each one its own file, but I can put them all in 1 file if you prefer. :)

Now it's just 2 files again. :)

Now it's just 2 files again. :)

Can we go to 1, and have only bgfx.d?

Now it's just 2 files again. :)

Can we go to 1, and have only bgfx.d?

Unfortunately no, there needs to be 2 files to avoid symbol name clashes.

Ok. What about renaming fakeenums.d to something different? Like enums.d?

Ok. What about renaming fakeenums.d to something different? Like enums.d?

Uh, is that unnecessary? It contains fake enums, I don't want anyone to think it has the real enums in it.

What fakeenums mean?

What fakeenums mean?

It contains enums that are only used to make the function bindings omit the correct symbols. I chose "fake" to stop people from wasting their time reading it, because the enums in there aren't for end-users.

Instead:
enum Access: bgfx.fakeenum.Access.Enum{
	read = bgfx.fakeenum.Access.Enum.read,
	write = bgfx.fakeenum.Access.Enum.write,
	readWrite = bgfx.fakeenum.Access.Enum.readWrite,
	count = bgfx.fakeenum.Access.Enum.count,
}

can you just generate?
enum Access {
	read = 0,
	write = 1,
	readWrite = 2,
	count = 3,
}

Instead:
enum Access: bgfx.fakeenum.Access.Enum{
	read = bgfx.fakeenum.Access.Enum.read,
	write = bgfx.fakeenum.Access.Enum.write,
	readWrite = bgfx.fakeenum.Access.Enum.readWrite,
	count = bgfx.fakeenum.Access.Enum.count,
}

can you just generate?
enum Access {
	read = 0,
	write = 1,
	readWrite = 2,
	count = 3,
}

No, then it wouldn’t be implicitly castable to the other type.

Can you think of way to remove bgfx.fakeenum completely, and have everything in one file?
That would be ideal.

Can you think of way to remove bgfx.fakeenum completely, and have everything in one file?
That would be ideal.

I mean, if you write and maintain a D library that can create Itanium and MSVC manglings from functions. Otherwise, it’s just the simplest way to make everything nice for the users. Also there were always 2 files for no reason until now, so I don’t see why you’re demanding something that anyway? If there could’ve easily been one file, I would’ve used only one file.
The only other possibility might be that declaring them as extern(C++, bgfx) (with bgfx not as a string) wouldn’t clash within the same file, but it might not work. I’ll investigate that when I’m home, but don’t get your hopes up. What I’m doing there is pretty weird, but it’s for the sake of pleasing the linker while maintaining D-like syntax. Personally I think the result will be bindings that 99% feel like they’re just D code

I'm not familiar with D, so just asking questions... Anyhow this is like 4th rewrite of D bindings. Just trying to make it final... :)

I'm not familiar with D, so just asking questions... Anyhow this is like 4th rewrite (or 5th, seems like a lot for language that's close to C and C++) of D bindings. Just trying to make it final... :)

D is syntactically close to C and C++, yes.
C and C++ get new standard features and standard library changes less than once a year, they’re planned well in advance, and by default (most if not all?) C/C++ compilers disable all new features until you add a flag.
D is forwards looking. There are preview flags for major upcoming features (or ones that need more time to be completed). If you update your compiler, you’ll get the newest non-preview language/standard library features unconditionally, and anything that was removed won’t come back unless you backdate your compiler. Most regular D users will either stick to one frontend version  or update as often as possible.
With how I wrote these bindings, I can dramatically change the generated code without having to modify the auto-generator because they use mixins that are generated by another BindBC library, so hopefully it won’t be necessary to majorly change them unless bgfx’s API changes a lot. :)
I’ll see if I can use D playgrounds to test that hypothesis of mine before this is merged…

This seems to work:
extern(C++, bgfx) extern(C++, RendererType){
  enum Enum{
    x,
  }
}

extern(C++, "bgfx") enum RendererType: bgfx.RendererType.Enum{
  x = bgfx.RendererType.Enum.x,
}

I’ll test it properly when I’m home.

Unfortunately, that method I found does not work in practice.
The issue with it is that it creates a bgfx "namespace", but it has the same name as the bgfx module, so anything trying to access bgfx.* always accesses the namespace instead of the module:
mixins.d(1391,21): Error: `namespace bgfx.topologySortTriList` cannot be resolved

There's no nice way around this issue, to my knowledge, so I guess it'll have to stay at 2 files...

Why there is discrepancy between what you generated, and when I generated IDL?
https://github.com/bkaradzic/bgfx/commit/c1afaa88e7466182ef20546197d949fb665abf19

Why there is discrepancy between what you generated, and when I generated IDL?
c1afaa8

It looks like all that happened is some structs were reordered? Maybe non-sequential Lua tables don’t have a fixed iteration order, and/or we’re using different Lua versions?

Can you make it deterministic. This didn't happened before, and I rely that when I invoke IDL if there are no changes there won't be anything changed with generated files.

Can you make it deterministic.

This should do it: https://github.com/bkaradzic/bgfx/pull/3118


Templates - Add SVGs to shape panel

This task is to go through each template and add the relevant SVGs documented here:
https://www.figma.com/file/RTzSw47oi6pEZXEfP2JoBo/Templates
under shapes & bring them into the shape panel in the editor:

https://github.com/google/web-stories-wp/pull/2760


Fix typo in struct.py documentation

What does this PR do?
Fixes typo in flax/struct.py documentation.
Checklist

[x] This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
checks if that's the case).

Codecov Report

Merging #3209 (0a4b0d6) into main (881c449) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3209   +/-   ##
=======================================
  Coverage   82.33%   82.33%
=======================================
  Files          54       54
  Lines        6062     6062
=======================================
  Hits         4991     4991
  Misses       1071     1071

Impacted Files
Coverage Δ

flax/struct.py
92.18% <ø> (ø)

Thanks!


How to disable using disk storage?

How can we disable using disk storage?

The easiest way is probably to put the "applet.jar" into the classpath, or with Maven have a dependency to the apfloat-applet artifact.

How must I call the getContents method in my initialization method?

https://github.com/mtommila/apfloat/blob/d709027ace3c15b218a4575cea2adc55ad8d5607/apfloat-applet/src/main/java/apfloat.java#L46

Ok, I wikll try it.


[1.18.2-Forge] Crashes when loaded on servers.

Title ^
Forge: 40.0.45
Mod Version: 1.18.2-1.4.0 | https://www.curseforge.com/minecraft/mc-mods/amethyst-tools-mod/files/3744989
Crash-Report: https://gist.github.com/Gbergz/7326cf75c09cc6817fce0e72201e8f74
- Gbergz

I'm sorry. I'll fix it today. Wait for 1.4.1 update.

problem resolved. just update mod. good luck <3


Set default value of LIMIT_HPU_GRAPH to True

What does this PR do?
Set default value of LIMIT_HPU_GRAPHS as True. This will disable HPU graphs for prefill operation. It is expected to be done for larger model and/or sequences, but it also fixes the accuracy issue in continuous batching scenario.
Update README with current limitations and batch size controlling method.

@kdamaszk There is a small merge conflict in the README, should be straightforward to solve

@kdamaszk There is a small merge conflict in the README, should be straightforward to solve

Done


Adding start and end date to builds on the Zuul source

Output for a build now looks like this:
Status: FAILURE
Start Time: 2022-10-20 13:39:31
End Time: 2022-10-20 14:54:32

'End Time' will not be printed if the build is still running.

It seems from the tests that maybe datetime.fromisoformat is not available for py36?

Ugh, 'fromisoformat' was added on Python 3.7. I will have to use a third party library to parse this then.

I have uploaded both requested changes.


set-samaccountpasswordhash : Access is denied

hi
i can execute the Get-ADReplAccount without an issue. the user being used is an ad domain admin
set-samaccountpasswordhash -domain westworld -samaccountname adadminuser -nthash ba17e001e5467d85d16ae7247947929c -server W8AAAADS01
set-samaccountpasswordhash : Access is denied
At line:1 char:1

set-samaccountpasswordhash -domain westworld -samaccountname adadminuser  ...

  + CategoryInfo          : NotSpecified: (:) [Set-SamAccountPasswordHash],
 UnauthorizedAccessException
  + FullyQualifiedErrorId : System.UnauthorizedAccessException,DSInternals.P
 owerShell.Commands.SetSamAccountPasswordHashCommand

 any ideas on why this is happening or how to solve it ?

Hard to tell. Is the powershell.exe elevated (Run as Administrator)?

Hard to tell. Is the powershell.exe elevated (Run as Administrator)?

yes

OK. What about Get-SamPasswordPolicy -Domain westworld, does it work? And net user /domain? Had NetCease been applied to that environment? Or any other hardening? Does the Security log on the DC tell you enything, if you enable all Advanced Auditing categories?

MinPasswordLength           : 8
ComplexityEnabled           : True
ReversibleEncryptionEnabled : False
MaxPasswordAge              : 31.00:00:00
MinPasswordAge              : 8.00:00:00
PasswordHistoryCount        : 24
net user also works fine, i even changed the password expiry with wmic and same domain admin user
its a red team so i actually stopped the auditing, i found another way to set the hash with smbpasswd but i am just confused as to what would block your set-samaccountpasswordhash

That is strange. I only have a limited AD lab, just re-tested the cmdlet and had no issues. If you figure it out, keep me posted, pls. I would also be curious what mimikatz lsadump::setntlm does, as it seems to be using the same function.

do you know what type of permissions are needed by set-samaccountpassword hash , maybe i can check the permissions or something

Only the Reset password permission should be required. Just tested it in a clean AD environment with a fully updated Windows Server 2022 21H2 DC:

Command:
Set-SamAccountPasswordHash -SamAccountName joe -Domain contoso -NTHash e19ccf75ee54e06b06a5907af13cef42 -Server dc.contoso.com


feat(StrongTopology): add lemmas about IsVonNBounded

Also add UniformOnFun.ofFun here and there to avoid type defeq abuse.

[x] depends on: #15215

Let's try again
bors merge


disscuss about decompress image after download

It is not a issue, I just have some test about how "decompress image" will affect with the performance, and then the result is confused.
I test with 2 code:
A: I just call initWithContentOfFile: at background thread, and then setImage in the main thread.
B: I use SDWebImage to test the fps after the image was all downloaded.
The fps was almost the same at instrument, so I doubt the effect of decompressing image.
Then I test the time consuming of "initWithContentOfFile" and "drawImage" with 2 images, the one was undecompress, the other was decompressed(I call decodedImageWithImage function and save to file).
The result show that with different images, the "drawImage" consume the same time, but the "initWithContentFile" has a different time consuming.
Is it mean that the "decompress image" is occurred at "initWithContentOfFile", not the time we call "setImage" at main thread?
And, is it mean that we have no need to decompress image after download while we always call the "initWithContentOfFile" function at a background thread?
This confused me very much, I need your advice.

When I get fps of instrument, it always be high(bigger then 50) while scrolling. And some times I can see the delay, is it possible that the instrument is not exact.
And how can I get the true fps while scrolling, any advice?

@bpoplauschi can you help with this issue?

@luoyibu for your fps question you can connect your device to XCode and open instruments and select Core Animation.

@luoyibu indeed, you can use CoreAnimation for real FPS measurements. Regarding the difference between initWithContentsOfFile and drawImage, I didn't take any measurements here, so I can't really have an opinion. The documentation on SDWebImage was written by @rs, so he should have the info on that. You can check out https://github.com/path/FastImageCache#the-problem, as it explains the classic problem with image decompression.


MagellanTV.com

[x] I'm reporting a new site support request
[x] I've verified that I'm running youtube-dl version 2021.03.03
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] I've searched the bugtracker for similar site support requests including closed ones

Would be awesome if someone could add MagellanTV.com.
Example URLs

Sample video: https://www.magellantv.com/video/the-eternal-notredame-4k
Sample series: https://www.magellantv.com/series/harem-pleasure-politics-and-power
Sample episode: https://www.magellantv.com/series/harem-pleasure-politics-and-power/rise-of-the-ottoman-harem
Sample playlist: https://www.magellantv.com/playlist/the-greatest-heroes-legacy-of-dday

Description
MagellanTV is a documentary streaming service with docuseries, short films and movies.

@dstftw I know it was a duplicate, but the original issue had no information.
That's why I figured I'd open this one in case somebody wants to put together an extractor for it.


Move ChunkingCookieManager to a separate Sources project

@Tratcher @rynowak

Hi @kichalla, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Wait, you plan to use this allocatey code elsewhere? Where I must know 😄

@davidfowl  in CookieTempDataProvider to chunk potentially large cookies as typically this happens:
cookie value = Base64Encode(DataProtect(BinaryJson(actual-data)))

Have you tried plugging the new sources package into MVC yet?

@Tratcher yes and it builds and tests run fine.

@davidfowl in CookieTempDataProvider to chunk potentially large cookies as this happens:
cookie value = Base64Encode(DataProtect(BinaryJson(actual-data)))

That is terrible. That looks hella inefficient. Where is that code.

In PR currently, https://github.com/aspnet/Mvc/pull/5257/files#diff-8edc4d422c5102070cbdb6f993605c20R75

:up: :date:

One other consideration. Because this writes directly to the headers it will bypass the CookiePolicy middleware (https://github.com/aspnet/Security/tree/dev/src/Microsoft.AspNetCore.CookiePolicy)

🆙 📅

🆙 📅


Nothing happens on OSX Catalina

I built the code fine and I ran ./sample on two consoles. I get the following in the two consoles but nothing else. Do I need to do something else? Or is there some permission issue on Mac?
0x10ed34dc0:Main thread

It looks good. You can input some command on a console like sdp1, ice1and else, like follow the link.
https://github.com/llamerada-jp/webrtc-cpp-sample/blob/master/README.en.md

I didnt read the instructions. it works fine. Thanks.


channel.default_exchange doesn't behave like default exchange

The default exchange should (as I understand - please correct if I'm wrong) route a message to a queue specified by the routing_key even if it is not explicitly bound (ala https://www.rabbitmq.com/tutorials/amqp-concepts.html under "Default Exchange")
So something like:
channel.default_exchange.publish('message', routing_key: 'test_queue', persistent: true)
should always send the message 'message' to the queue 'test_queue', even with no explicit binding, assuming that that queue exists.
This doesn't seem to happen with BunnyMock.

A possible solution (at least for my usage):

make a subclass of direct exchange for the default exchange and return that from channel.default_exchange
override the publish method to look for the queue specified by the routing key before doing anything else - if found, bind it there. This would at least produce the illusion of all queues being automatically bound to the default exchange for publishing purposes.

Another possibility:

make a singleton subclass of direct exchange for the default exchange and return that from channel.default_exchange
on queue creation automatically bind to that exchange

If I get some time I'll see if I can work one of those up and see if it works for me and if so put in a pull request. If not, happy for any sort of solution.

Sorry for reviving and old (but open) issue. Here's how I "solved" the issue:
```# Override BunnyMock::Queue#subscribe to mimic the default exchange behavior
of RabbitMQ. Also add the missing opposite BunnyMock::Queue#cancel.
module BunnyMock
class Queue
alias original_subscribe subscribe
def subscribe(*args, &block)
  channel.default_exchange.add_route(name, self)

  original_subscribe(*args, &block)
end

def cancel
  channel.default_exchange.remove_route(name, self)
end

end
end
Can I open a PR to include this functionality in BunnyMock?

of course @arnodirlam - make a PR and we can talk about migration


如何在下游任务微调？

请问，在下游任务比如qm9上的微调脚本方便提供吗？

Hi, I have released the fine-tuning code of QM9. The fine-tuning code of PDBBind will also be released soon~ Please stay tuned.


"Gherăești", "Gherăeștii Noi" and "Tețcani" are listed in the drop-down menu

Description:
"Gherăești", "Gherăeștii Noi" and "Tețcani" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Gherăești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Gherăești, NEAMȚ (Gherăești)" and "Comuna Gherăești, NEAMȚ".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Gherăeștii Noi" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Gherăeștii Noi, NEAMȚ (Gherăești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Tețcani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Tețcani, NEAMȚ (Gherăești)".


HDFS-17503. Unreleased volume references because of OOM.

Description of PR
Refer to HDFS-17503.
When BlockSender throws an error because of OOM，the volume reference obtained by the thread is not released，which causes the thread trying to remove the volume to wait and fall into an infinite loop.
I found HDFS-15963 catched exception and release volume reference. But it did not handle the case of throwing errors. I think "catch (Throwable t)" should be used instead of "catch (IOException ioe)".
How was this patch tested?
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

_ Prechecks _

+1 :green_heart:
dupname
0m 00s

No case conflicting files found.

+0 :ok:
spotbugs
0m 01s

spotbugs executables are not available.

+0 :ok:
codespell
0m 01s

codespell was not available.

+0 :ok:
detsecrets
0m 01s

detect-secrets was not available.

+1 :green_heart:
@author
0m 00s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m 00s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
94m 08s

trunk passed

+1 :green_heart:
compile
6m 41s

trunk passed

+1 :green_heart:
checkstyle
5m 12s

trunk passed

+1 :green_heart:
mvnsite
7m 34s

trunk passed

+1 :green_heart:
javadoc
6m 25s

trunk passed

+1 :green_heart:
shadedclient
159m 41s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 57s

the patch passed

+1 :green_heart:
compile
3m 48s

the patch passed

+1 :green_heart:
javac
3m 48s

the patch passed

+1 :green_heart:
blanks
0m 01s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
2m 30s

the patch passed

+1 :green_heart:
mvnsite
4m 26s

the patch passed

+1 :green_heart:
javadoc
3m 41s

the patch passed

+1 :green_heart:
shadedclient
168m 39s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
asflicense
6m 47s

The patch does not generate ASF License warnings.

452m 05s

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/6782

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
MINGW64_NT-10.0-17763 06e84c9dc0f5 3.4.10-87d57229.x86_64 2024-02-14 20:17 UTC x86_64 Msys

Build tool
maven

Personality
/c/hadoop/dev-support/bin/hadoop.sh

git revision
trunk / b25c35089fe854166b4f7a57171fa21abeb99a5f

Default Java
Azul Systems, Inc.-1.8.0_332-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch-windows-10/job/PR-6782/1/testReport/

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch-windows-10/job/PR-6782/1/console

versions
git=2.45.0.windows.1

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@Hexiaoqiao Hi~ sir. Could you please help me review this PR when you are free? Thanks.

Merged. Thanks @zhuzilong2013 for your contribution.

Thanks @ZanderXu for your review and merge~


Use lighter and more neutral colors for report and log html page

Current colors are very bright/flashy, I think it would be better to user lighter and more neutral colors. To be consistent we can use Twitter Bootstrap's color scheme: http://getbootstrap.com/css/#less-variables
An other approach would be to use (sparingly) the robotframework.org color scheme to emphasize important information.
Goal

Reduce the number of colors used
Replace yellow as hover color with grayscale color
Replace grey-blue table header with grayscale color
Replace purple and (default) blue for links with a more balanced blue
Use labels for status (error, warn, fail...)
Use monospaced font for test executions output (more readable in most case)
Replace black border with grayscale color (black is too intense and make other colors look pale)

We are starting to create alpha1 release now. I will move this to alpha2. That should give us still plenty of time to test the changes before final release.

@Mogztter, are you still interested in this? As we commented in PR #1940, we really liked the new styles in general but there were some issues that needed to be fixed. If you aren't interested or don't have time, we can take a look at this ourselves. We'd like to get the new styles into 2.9 beta 1 to give users more time to comment them before the final release. We try to get that beta released already early next week.

Sorry I didn't have much spare time lately. I don't think I will have time
to fix this issues this week but feel free to adapt my work.
Guillaume
Le 22 juin 2015 12:12 PM, "Pekka Klärck"<EMAIL_ADDRESS>a
écrit :

@Mogztter https://github.com/Mogztter, are you still interested in
this? As we commented in PR #1940
https://github.com/robotframework/robotframework/pull/1940, we really
liked the new styles in general but there were some issues that needed to
be fixed. If you aren't interested or don't have time, we can take a look
at this ourselves. We'd like to get the new styles into 2.9 beta 1 to give
users more time to comment them before the final release. We try to get
that beta released already early next week.
—
Reply to this email directly or view it on GitHub
https://github.com/robotframework/robotframework/issues/1943#issuecomment-114057741
.

So the colors we used now are still a bit more flashy than the suggested ones, but we also wanted them to be easy to differentiate for a colorblind person, and to still look good on an older monitor. These seem to work and they are still quite a bit less flashy than the originals.
Comments are welcome!

@jussimalinen and I went through the changes proposed in PR #1940. We did same/similar changes on a separate branch, fixed some problems, and committed changes separately. As Jussi already commented above, we didn't use exactly same colors as suggested and there were also some other differences. The commits above contain more details.
In the end we are very happy with the results. Big thanks for providing the original PR @Mogztter! I hope you don't mind me adding you to AUTHORS.txt.

Thanks for the follow up.

I hope you don't mind me adding you to AUTHORS.txt.

Not at all!

Have you @Mogztter looked how logs and reports look now? We are interested in feedback about the new styles in general, but it would be especially nice to get comments from the originator of this issue.

Not yet, I need to setup a new local environment to view the changes, I will let you know :smile:

As a colour blind person, I think the new colours are not good, its very hard for me to differentiate between the colours. Maybe text saying PASS / FAIL instead of TEST would be better. btw I am only slightly colour bind in the red/green spectrum, so this may be worse for people that are more colour blind on this spectrum.

Interesting. Do you mean the proposed colord or the ones that are nowadays in use? We actually had a color blind person to test the colors and also used a tool to visualize how people with different color blindness see them. I guess the problem is that there are so many different color blindness variations. If you have problems with the current colors, please submit a new issue about that. Even better if you have a proposal for new colors or other enhancements.

I ran a 'failed' report through an accessibility checker (against WCAG v2 AA) and nothing was returned against the following criterion... "1.4.3 Contrast (Minimum): The visual presentation of text and images of text has a contrast ratio of at least 4.5:1, except for the following: (Level AA)" which is the one which springs to mind.
You're right though, there are different types of colour-blindness (I have red-green colour blindness myself, but can see the reports fine).  If you are looking at changes, consider this tool... http://gmazzocato.altervista.org/colorwheel/wheel.php (from http://www.w3.org/WAI/ER/tools/)


implement lazy sessions

Here we've reworked our session implementation to avoid loading sessions until they're actually used. In other words, these are "lazy" sessions.
We enable this by colocating the session store with the session.
Doing so provides some noteworthy benefits:

The overhead of loading from a backend store in the middleware is entirely eliminated--this crate is "infinitely" faster now insomuch as the middleware itself is concerned.
Because stores are colocated with sessions, we are now able to read from and write to sessions from e.g. handlers--this will allow us to address concurrent data access with some backends, but also relates to the first point: we only pay for what we actually use.

However, this also introduces some important changes to the interface, most obviously the fact that session methods are now async and fallible (for stores that can result in an error).
Ultimately moving in this direction ensures we can scale the middleware. In other words, this is likely needed to claim "web scale" (for some measure of that dubious term). Prior to this we've worked around the load cost with e.g. caching stores, but at some point it becomes desirable to simply not load unless the session is used.
We must also point out that this is how Django has designed their session middleware1: at no point is the session loaded and instead this is cleverly deferred by leaning on Python's magic methods and property setters.
As per ergonomics, because async-await is ubiquitious in our API, there is a general degradation to note here: we must now await all these methods. At the same time, the proliferation of await points should be beneficial to highly concurrent applications.
That said, this design is generally easier to use correctly because the costs are moved to the public API callsites as opposed to the prior design, which obscured specifically the load cost within the middleware.
Finally it's also important to point out that session stores are now used as trait objects, which means that the error type has been changed: it's now a concrete type owned by this crate. Implementers should map their backend errors to this new error enum.
For more details on the exploration of this design, please see this discussion thread: https://github.com/maxcountryman/tower-sessions/discussions/101

cc @benwis: this completely eliminates store loading overhead (we just do not load from the store in the middleware now), so re your much earlier questions over on the axum-sessions repo, you might find this helpful.

cc @benwis: this completely eliminates store loading overhead (we just do not load from the store in the middleware now), so re your much earlier questions over on the axum-sessions repo, you might find this helpful.

Very cool, love the work that's going on!


Private repositories search does not work on Ubuntu 14.04

It works fine on OSX, but does not work on Ubuntu 14.04
I migrated from gogs.
List of private reps is visible, but filter does not make any changes on it.
Sorting works fine, but the search does not.

Hi, could you send some screenshot to show your issue?

@romanvbabenko Any Update on this?

I can't guess any cause why this should not work on ubuntu...

Just gave it a quick try on try.gitea.io and searches for orgs and users seem to work.
The search for repos gives zero hits! Try fx a search on 'rails' https://try.gitea.io/explore/repos?q=+rails

Just close this since it cannot be reproduced and need feedback. Please feel free to reopen it if any further informations.

Just gave it a quick try on try.gitea.io and searches for orgs and users seem to work.
The search for repos gives zero hits! Try fx a search on 'rails' https://try.gitea.io/explore/repos?q=+rails

Remove the + and you will find rails, we don't trim whitespaces within the search.

Maybe we need to trim the search keyword? If so, I can send PR.


Bug in function getFolders()

This problem is in during using IMAP in ovh. The imap_list function return
 [0] => (string) {ssl0.ovh.net:993/imap/ssl/novalidate-cert/readonly}INBOX
 [1] => (string) {ssl0.ovh.net:993/imap/ssl/novalidate-cert/readonly}INBOX.INBOX.Trash
 [2] => (string) {ssl0.ovh.net:993/imap/ssl/novalidate-cert/readonly}INBOX.INBOX.Drafts
 [3] => (string) {ssl0.ovh.net:993/imap/ssl/novalidate-cert/readonly}INBOX.INBOX.Sent
 [4] => (string) {ssl0.ovh.net:993/imap/ssl/novalidate-cert/readonly}INBOX.INBOX.Junk
 [5] => (string) {ssl0.ovh.net:993/imap/ssl/novalidate-cert/readonly}INBOX.Trash

but function getFolders() return only:
array(2) (
  [INBOX] => array(1) (
    [Trash] => array(0)
  )
  [] => null
)

Any ideas how to fix this?

Seems to be an issue with the multi-dimensional array. I'll look into it.

If you config contains
$imap = new ImapClient([ ...     'mailbox' => [         'mailbox_name' => 'INBOX',     ], ... ]);
try this
$imap->getFolders('.',1);
and sends results

Yes, my configuration contains maiblox_name. I used your parameters and got it
array(6) (
  [0] => (string) INBOX
  [1] => (string) INBOX.INBOX.Trash
  [2] => (string) INBOX.INBOX.Drafts
  [3] => (string) INBOX.INBOX.Sent
  [4] => (string) INBOX.INBOX.Junk
  [5] => (string) INBOX.Trash
)

Seems to work.
but without parameters in gmail I have other structure.
array(8) (
  [INBOX] => array(0)
  [[Gmail]/Kosz] => array(0)
  [[Gmail]/Oznaczone gwiazdka] => array(0)
  [[Gmail]/Spam] => array(0)
  [[Gmail]/Wazne] => array(0)
  [[Gmail]/Wersje robocze] => array(0)
  [[Gmail]/Wszystkie] => array(0)
  [[Gmail]/Wyslane] => array(0)
)

I think this function should recognize and return same structure folder for various email account.

If possible, send to a second server (gmail) imap_list() function.

Okay, no problem.
imap_list() function in gmail
array(8) (
  [0] => (string) {imap.gmail.com:993/imap/ssl/novalidate-cert}INBOX
  [1] => (string) {imap.gmail.com:993/imap/ssl/novalidate-cert}[Gmail]/Kosz
  [2] => (string) {imap.gmail.com:993/imap/ssl/novalidate-cert}[Gmail]/Oznaczone gwiazdka
  [3] => (string) {imap.gmail.com:993/imap/ssl/novalidate-cert}[Gmail]/Spam
  [4] => (string) {imap.gmail.com:993/imap/ssl/novalidate-cert}[Gmail]/Wazne
  [5] => (string) {imap.gmail.com:993/imap/ssl/novalidate-cert}[Gmail]/Wersje robocze
  [6] => (string) {imap.gmail.com:993/imap/ssl/novalidate-cert}[Gmail]/Wszystkie
  [7] => (string) {imap.gmail.com:993/imap/ssl/novalidate-cert}[Gmail]/Wyslane
)

Has corrected, we wait when they will take the pull request. And need use getFolders() method without 'mailbox_name' property in config, like this [ ... 'mailbox' => [ 'mailbox_name' => '', ], ... ].

Well, I've fixed something now, you can just use the getFolders () method, regardless of the configuration. But you need to wait for the pull request.

I have merged the pull request. Let me know if the error persist.

OP is inactive. Marked as inactive.

Seems to works good :)

Thank you!


Testcases for services in chaos_experiment directory

Inside the /chaos_experiment/ at location choascenter/graphql/server/pkg, the service.go file does not have any testcases.
Testcases can be added for this file

Hello @SohamRatnaparkhi , I'm looking to participate in the LFX mentorship this fall, I would like to work on this issue to get familiar with the project 🙂

Welcome @Freedisch ,
This issue is a part of current LFX mentorship program (i.e. term - 2) and I am working on it. Feel free to pick some other issue. If you have started working on it then sorry for the inconvenience.


"_fke.csv"

kit dataset originally had "_fke.csv" file, or was it generated by data preprocessing? If it was generated by preprocessing, could you please send me the code

I got the same question, did you fix it? I can only find files end with "_quat.csv" and "_quat.fke"

Hello both @1390806607 @maplect,
As explained in the readme, you should run the code from Ghosh et al or JL2P to process the data. It will create the files which ends with "fke.csv". The code links are in the readme.
Tell me if you have any issues

Hello both @1390806607 @maplect,
As explained in the readme, you should run the code from Ghosh et al or JL2P to process the data. It will create the files which ends with "fke.csv". The code links are in the readme.
Tell me if you have any issues

Many thanks for your rapid reply. I got the problem, I firstly run the code from JL2P, and the output of find datasets/kit/* -name . -o -prune | head -20 is
datasets/kit/00001_annotations.json datasets/kit/00001_meta.json datasets/kit/00001_mmm.xml datasets/kit/00001_quat.csv datasets/kit/00001_quat.fke datasets/kit/00001_quat.rifke datasets/kit/00001_raw.c3d datasets/kit/00002_annotations.json datasets/kit/00002_meta.json datasets/kit/00002_mmm.xml datasets/kit/00002_quat.csv datasets/kit/00002_quat.fke datasets/kit/00002_quat.rifke datasets/kit/00002_raw.c3d datasets/kit/00003_annotations.json datasets/kit/00003_meta.json datasets/kit/00003_mmm.xml datasets/kit/00003_quat.csv datasets/kit/00003_quat.fke datasets/kit/00003_quat.rifke
While I run the code from Ghosh, the output of find datasets/kit/* -name . -o -prune | head -20 is
datasets/kit/00001_annotations.json datasets/kit/00001_ax.csv datasets/kit/00001_fke.csv datasets/kit/00001_meta.json datasets/kit/00001_mmm.xml datasets/kit/00001_quat.csv datasets/kit/00001_raw.c3d datasets/kit/00001_rifke.csv datasets/kit/00002_annotations.json datasets/kit/00002_ax.csv datasets/kit/00002_fke.csv datasets/kit/00002_meta.json datasets/kit/00002_mmm.xml datasets/kit/00002_quat.csv datasets/kit/00002_raw.c3d datasets/kit/00002_rifke.csv datasets/kit/00003_annotations.json datasets/kit/00003_ax.csv datasets/kit/00003_fke.csv datasets/kit/00003_meta.json
and I can find the files which end with "fke.csv". So I think there are some differences between these two data generation script.

Hi @maplect,
I will remove JL2P from the README then, thanks for pointing it out!


litan2/osu_latency

Added OSU Micro-Benchmarks as a new test.

I cannot see your comments on GitHub, if you are referring to
Divyansh's suggestion to use node.os.install_packages to install
package, node.os.install_packages gives me some errors when checked
using nox -vRs mypy. As such, I kept the old way to install package.
Thanks.
--
Li
On 12/7/22, LiliDeng @.***> wrote:

@LiliDeng commented on this pull request.

+def _install_osu_bench(node: Node) -> str:

wget = node.tools[Wget]
tar = node.tools[Tar]

path = wget.get_tool_path(use_global=True)
extracted_path = tar.get_tool_path(use_global=True)
if node.shell.exists(path / OSU_BENCH_FILE_NAME):

   return

download_path = wget.get(

   url=OSU_BENCH_LOCATION, filename=str(OSU_BENCH_FILE_NAME),

file_path=str(path)

)
tar.extract(download_path, dest_dir=str(extracted_path))

node.execute("apt-get install -y python3.8-dev", shell=True,
sudo=True)

why resolves this comment?
--
Reply to this email directly or view it on GitHub:
https://github.com/microsoft/lisa/pull/2403#discussion_r1041809535
You are receiving this because you authored the thread.
Message ID: @.***>

Can you please tell the error?

Can you please paste the error?

I used up all my codespace and cannot retrieve the complete error. According to my memory, it says: node.os does not have function install_packages().
I went to check all the exising LISA tests and found they are using this.node.os.install_packages, but when I put this in the front, there are new errors. So eventually I stick with node.execute()

Can you please paste the error?

I used up all my codespace and cannot retrieve the complete error now. According to my memory, it says: node.os does not have function install_packages().
I checked all the existing LISA tests and found they are using this.node.os.install_packages, but when I put this in the front, there were new errors coming. So eventually I stick to use node.execute()

Can you please try this:
 assert isinstance(
            node.os, Posix
        ), f"unzip: unsupported OS {self.node.os.name}"
 node.os.install_packages(<package-name>)

Can you please paste the error?

I used up all my codespace and cannot retrieve the complete error now. According to my memory, it says: node.os does not have function install_packages().
I checked all the existing LISA tests and found they are using this.node.os.install_packages, but when I put this in the front, there were new errors coming. So eventually I stick to use node.execute()

Can you please try this:
 assert isinstance(
            node.os, Posix
        ), f"unzip: unsupported OS {self.node.os.name}"
 node.os.install_packages(<package-name>)

Updated and all nox tests passed.

LGTM! @squirrelsc / @LiliDeng Can you please take a look?
@darkwhite29 Please fix any pipeline issues

You mean all issues from "nox -vRt all"? There are no issues.

LGTM! @squirrelsc / @LiliDeng Can you please take a look?
@darkwhite29 Please fix any pipeline issues

You mean all issues from "nox -vRt all"? There are no issues.

They seem to be fixed now. The last PR had one test failing.

Test results
OSU Bench.run_osu_bench: PASSED

osu_bench.log

@darkwhite29 Can you please resplit the commit into two separate commits: change in infiniband and osu test? You can do it by first doing a soft reset with git reset --soft HEAD~2 and then reading the files into two separate commits. Please feel free to ping me if you are facing any problem

@darkwhite29 Can you please resplit the commit into two separate commits: change in infiniband and osu test? You can do it by first doing a soft reset with git reset --soft HEAD~2 and then readding the files into two separate commits. Please feel free to ping me if you are facing any problem

Done.


[ionic4beta13] AlertController/Alert Usage and NgZone

Bug Report
Ionic Info
Ionic:

   ionic (Ionic CLI)          : 4.1.2 (C:\Users\rtpha\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework            : @ionic/angular 4.0.0-beta.13
   @angular-devkit/core       : 0.7.5
   @angular-devkit/schematics : 0.7.5
   @angular/cli               : 6.1.5
   @ionic/ng-toolkit          : 1.0.8
   @ionic/schematics-angular  : 1.0.6

System:

   NodeJS : v8.12.0 (C:\Program Files\nodejs\node.exe)
   npm    : 6.4.1
   OS     : Windows 10

Describe the Bug
If you look in this thread you will see the discussion. It's not clear to me if this is a bug / something unfinished with the AlertController or if its something that's not covered by the docs:
https://forum.ionicframework.com/t/alertcontroller-ionic-4-beta-7-navigation-triggered-outside-angular-zone-did-you-forget-to-call-ngzone-run/142332/10
To summarise, the handlers for buttons in an Alert has some confusing usage issues.
In one scenario attempting to use this.navController.navigateForward("/"); will actually redirect to the destination but an error appears in the log:
Navigation triggered outside Angular zone, did you forget to call 'ngZone.run()'?

In another scenario I had a delete function called from the confirm button handler of an Alert and on the ok it would delete that item from the data but the ui wasn’t being refreshed. This meant that it was still in the list of cards in the UI but no longer in the data.
In both cases using the NgZone solution mentioned in the forum post got things working.
I did some research and actually didn’t expect this to work as the Angular docs say zone is for when you update the data that you don’t want Angular to monitor and update, but I wanted the opposite. It worked anyway.
I'm not clear on what NgZone is really for so thats why I'm not sure if this is a bug in my code, in the Alert or just that the docs are missing saying this is needed?
Related Code
First example I described (the solved code that got it working):
  async buttonCancel() {
    const alert = await this.alertController.create({
      header: 'Reset',
      message: 'Do you want to reset back to the homepage?',
      buttons: [
        {
          text: 'No',
          role: 'cancel',
          cssClass: 'secondary'
        }, {
          text: 'Yes',
          handler: () => {
            // https://forum.ionicframework.com/t/alertcontroller-ionic-4-beta-7-navigation-triggered-outside-angular-zone-did-you-forget-to-call-ngzone-run/142332/8
            this.zone.run(async () => {
              this.beerbargainService.resetOffers();
              this.navController.navigateForward("/");
            });
          }
        }
      ]
    });

    await alert.present();
  }

Second example I described (again the solved issue):
  async buttonRemoveOffer(i) {
    const alert = await this.alertController.create({
      header: 'Delete Offer',
      message: 'Are you sure you want to delete this offer?',
      buttons: [
        {
          text: 'No',
          role: 'cancel',
          cssClass: 'secondary'
        }, {
          text: 'Yes',
          handler: () => {
            this.zone.run(async () => {
              this.deleteOfferByIndex(i);
              this.saveOfferToBeerOffers();
            });
          }
        }
      ]
    });

    await alert.present();
  }

The unsolved versions of both of these is just the removal of the this.zone.run(async () => { line and its closing });.
Expected Behavior
I'm looking for guidance here.
Additional Context
Original snippets were both adapted from the docs at https://beta.ionicframework.com/docs/api/alert/

👋 Hi there. This was an issue last release that I though was fixed by @manucorporat via https://github.com/ionic-team/ionic/commit/f63c0f5ccc8dad9958861b13e465432ddd8cd89b
Can you confirm that you have beta 13 installed?

I have beta 13 installed.
This is from my packages.json:
"dependencies": {
    "@angular/common": "~6.1.1",
    "@angular/core": "~6.1.1",
    "@angular/forms": "~6.1.1",
    "@angular/http": "~6.1.1",
    "@angular/platform-browser": "~6.1.1",
    "@angular/platform-browser-dynamic": "~6.1.1",
    "@angular/router": "~6.1.1",
    "@ionic-native/core": "5.0.0-beta.15",
    "@ionic-native/splash-screen": "5.0.0-beta.15",
    "@ionic-native/status-bar": "5.0.0-beta.15",
    "@ionic/angular": "^4.0.0-beta.13",
    "core-js": "^2.5.3",
    "rxjs": "6.2.2",
    "zone.js": "^0.8.26"
  },
  "devDependencies": {
    "@angular/cli": "~6.1.1",
    "@angular/compiler": "~6.1.1",
    "@angular/compiler-cli": "~6.1.1",
    "@angular/language-service": "~6.1.1",
    "@angular-devkit/architect": "~0.7.2",
    "@angular-devkit/build-angular": "~0.7.2",
    "@angular-devkit/core": "~0.7.2",
    "@angular-devkit/schematics": "~0.7.2",
    "@ionic/ng-toolkit": "^1.0.0",
    "@ionic/schematics-angular": "^1.0.0",
    "@types/jasmine": "~2.8.6",
    "@types/jasminewd2": "~2.0.3",
    "@types/node": "~10.9.2",
    "codelyzer": "~4.4.2",
    "jasmine-core": "~2.99.1",
    "jasmine-spec-reporter": "~4.2.1",
    "karma": "~3.0.0",
    "karma-chrome-launcher": "~2.2.0",
    "karma-coverage-istanbul-reporter": "~2.0.0",
    "karma-jasmine": "~1.1.1",
    "karma-jasmine-html-reporter": "^0.2.2",
    "protractor": "~5.4.0",
    "ts-node": "~7.0.0",
    "tslint": "~5.11.0",
    "typescript": "~2.9.2"
  },

In another scenario I had a delete function called from the confirm button handler of an Alert and on the ok it would delete that item from the data but the ui wasn’t being refreshed. This meant that it was still in the list of cards in the UI but no longer in the data.

Hello, I am still seeing this issue with beta15.
It seems that the handler: ()  function runs OUTSIDE the angular zone

Can confirm these issues still exists, we have them in action sheets and such. Not only navigation is problem but also updating the page itself does not happen. runnning beta 16

Hello.
This problem still exists in 4.0.1 but only when navigating tabs back and forth.  For example:
this.navCtrl.navigateForward('/tabs/subtab');  doesn't result in a ngZone warning.
Hitting the
and then calling...
this.navCtrl.navigateForward('/tabs/subtab');  results in a ngZone warning.
core.js:16824 Navigation triggered outside Angular zone, did you forget to call 'ngZone.run()'?
Regards


Create an 'About' section

Create an 'About' section that describes on-going activities, so that visitors to TeachOSM.org get a sense of what TeachOSM is actively engaged in.

@osmlab/teachosm self-assigning

@geomantic - which route would you like to go with this?  examples being...

About describing the project mission/purpose of TeachOSM OR
About describing current TeachOSM initiatives linking to workspaces OR
About being an event calendar OR
something else

The first two, but not the third (or fourth). Don't think we have enough yet to warrant a calendar and besides, descriptive stuff could also include dates for events.

Thanks @geomantic, I will fork & start working on this.

OBE


[SPARK-21027][ML][PYTHON] Added tunable parallelism to one vs. rest in both Scala mllib and Pyspark

What changes were proposed in this pull request?
Added tunable parallelism to the pyspark implementation of one vs. rest classification. Added a parallelism parameter to the Scala implementation of one vs. rest along with functionality for using the parameter to tune the level of parallelism.
I take this PR #18281 over because the original author is busy but we need merge this PR soon.
After this been merged, we can close #18281 .
How was this patch tested?
Test suite added.

Test build #81350 has started for PR 19110 at commit 24f4499.

Test build #81350 has finished for PR 19110 at commit 24f4499.

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

Merged build finished. Test FAILed.

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

Jenkins, test this please.

Test build #81352 has started for PR 19110 at commit 24f4499.

Test build #81352 has finished for PR 19110 at commit 24f4499.

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

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

Merged build finished. Test PASSed.

Test build #81381 has started for PR 19110 at commit fc6fd5e.

Test build #81381 has finished for PR 19110 at commit fc6fd5e.

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

Merged build finished. Test PASSed.

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

LGTM
Pinging on https://github.com/apache/spark/pull/16774 first to coordinate merging

Test build #81436 has finished for PR 19110 at commit 7d0849e.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #81438 has finished for PR 19110 at commit edcf85c.

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

Merged build finished. Test PASSed.

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

FYI I went ahead and merged #16774 - the doc on the shared param trait is a little more detailed there which I slightly prefer. @WeichenXu123 you will just need to resolve the small merge conflict that introduces.

@MLnick Conflict resolved. Thanks!

Test build #81456 has started for PR 19110 at commit 7a1d404.

Test build #81456 has finished for PR 19110 at commit 7a1d404.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class NettyMemoryMetrics implements MetricSet

Merged build finished. Test PASSed.

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

LGTM!

Thanks @MLnick @BryanCutler . Would you mind helping review another similar PR #19122 ? We need some other features but blocking on that PR. Thanks!

Other than that 1 item, this looks ready

Test build #81653 has started for PR 19110 at commit c24d4e2.

Test build #81653 has finished for PR 19110 at commit c24d4e2.

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

Merged build finished. Test PASSed.

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

LGTM
Merging with master
Thanks!


Improve the tests to check Python 2.7 and 3.6 on all platforms

Making an internal PR to this fork to confirm the Travis and Appveyor configs are correct, then will PR to the Uber repo.


Litestar DI

Is it possible to use Litestar DI at job functions like:
    def job(product_service: ProductService):
           pass

?

Thinking, it is possible via worker context via self.context, but plugin implementation does not permit such things

It's not currently possible; this would be an awesome addition though.


how to get the build-status and build-commiter

i installed the gitlab-ci 3.0. after run the project build, it will show us a list the build info. But what i want is that how    to get the build-status and build-commiter info by gitlab-ci api or someway else,the reason is that i want to use those above info in the build script.
PS: Recently i do not know how to send email to the project commiter or the owner when the build is failed.

@yinxiaomei In the current version of GitLab CI Webhooks are present.


Dialog is having ripple effect

Dialog in android is showing ripple effect. I believe this will make the change.

I am going to close this since I think Dialog can have RippleFeedback. We can remove RippleFeedback directly in RippleFeedback component if there is no onPress method.

Sorry for the delay, i was out, how about we pass one prop to Dialog container  that will disable the native  that falls under RippleFeedback container.


Add support for Elyra Pipelines

Add support for Elyra Pipeline files: https://github.com/elyra-ai/elyra
Changes proposed:

[x] Add
[ ] Delete
[ ] Fix
[ ] Prepare

Hi @bourdakos1, is this icon original? I cannot see it in the Elyra documentation. It also seems like a very generic extension, which could potentially be tied to other kind of pipelines. That's why I'm asking about the origin of the icon. Thanks for the PR.

Hi @robertohuertasm, this icon was originally created for @elyra/canvas which is the core of Elyra Pipelines, but also used in a couple other IBM pipeline related products. The @elyra/canvas package can be used by anyone to build a pipeline editor, so the icon was designed to be generic. Hope this helps :)

Thanks @bourdakos1 for the explanation. It helps a lot!


Filter DocMaps index by publisher.account.id

Goal

index endpoint lets user filter by publisher.account.id
use group slug in publisher.account.id
query param takes full url that is exposed in publisher.account.id
query param is called publisheraccount

Biorxiv usecase
Biorxiv may need to update their data for a single group, perhaps if something went wrong at their end.

https://sciety.org/docmaps/v1/index?publisheraccount=https://sciety.org/groups/rapid-reviews-covid-19 shows 89 docmaps for Rapid Reviews.


Download URL to python-yadis package on PyPI broken

Please update the download link to the python-yadis source dist on http://pypi.python.org/pypi/python-yadis/ or for the love of god upload it to PyPI. kthxbye

Until someone bothers to fix this, I happen to have a cached package squirreled away.
http://static.crossway.org/_tmp/python-yadis-1.1.0.tar.gz

Please fix this problem. I, like many people, are having problems when trying to install with PIP:
Downloading/unpacking python-yadis>=1.1.0 (from -r requirements.txt (line 7))
Downloading python-yadis-1.1.0.tar.gz
Exception:
Traceback (most recent call last):
File "/usr/lib/pymodules/python2.7/pip/basecommand.py", line 130, in main
self.run(options, args)
File "/usr/lib/pymodules/python2.7/pip/commands/install.py", line 223, in run
requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
File "/usr/lib/pymodules/python2.7/pip/req.py", line 918, in prepare_files
self.unpack_url(url, location, self.is_download)
File "/usr/lib/pymodules/python2.7/pip/req.py", line 1024, in unpack_url
return unpack_http_url(link, location, self.download_cache, only_download)
File "/usr/lib/pymodules/python2.7/pip/download.py", line 448, in unpack_http_url
unpack_file(temp_location, location, content_type, link)
File "/usr/lib/pymodules/python2.7/pip/util.py", line 465, in unpack_file
untar_file(filename, location)
File "/usr/lib/pymodules/python2.7/pip/util.py", line 401, in untar_file
tar = tarfile.open(filename, mode)
File "/usr/lib/python2.7/tarfile.py", line 1676, in open
return func(name, filemode, fileobj, **kwargs)
File "/usr/lib/python2.7/tarfile.py", line 1727, in gzopen
raise ReadError("not a gzip file")
ReadError: not a gzip file

This repo is being archived. Closing issue.


ReferenceError: Can't find variable: MutationObserver

Just installed the ng-Inspector extension and getting this error when I enable it.
Safari 5.1 on Windows.
ReferenceError: Can't find variable: MutationObserver           ng-inspector.js:827

@jbeckton ng-inspector relies on the MutationObserver specification. This was not introduced into Safari until 7.1 (http://caniuse.com/#feat=mutationobserver). You'll have to upgrade to a newer version to use the extension.

@jbeckton, @DrewML is correct, in a future update the extension won't rely on MutationObservers, but it will probably break at some other point in the execution :(
I'm surprised you were able to even install it!


use java

Can soupout java?

You can refer to https://github.com/amitshekhariitbhu/RxJava2-Android-Samples for Java sample with RxJava2
RxJava2 and RxJava3 are quite similar. Although I recommend kotlin language as its 4year not since google announced first class support to kotlin language.
Please close if required. cc: @amitshekhariitbhu


#2615 Adding nudge in wallet auth

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[ ] Bugfix
[x] Feature
[ ] Refactoring

What's new?

[x] PR closes #2615
[ ] Requires deployment <rubick/magick_issue>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address:
Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 100 usd

😍 Perfect, I’ve sent the payout
💵 $100 @ 68.86 USD/KSM ~ 1.452 $KSM
🧗 EzGc4s9PgCPx1YnF3fqzhLzVHpHMTL4LWPScwpDrR8JKgSU
🔗 0x22441a79e05a9520033a80a7f0b86e432c4014376e27a067ef7932f230eaeb01
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Issue with gradient part

I think these piece of code:
self.gradInput[1] = (n - p):cmul(self.Li:gt(0):repeatTensor(1,a:size(2)):type(a:type()) * 2/N)
self.gradInput[2] = (p - a):cmul(self.Li:gt(0):repeatTensor(1,a:size(2)):type(a:type()) * 2/N)
self.gradInput[3] = (a - n):cmul(self.Li:gt(0):repeatTensor(1,a:size(2)):type(a:type()) * 2/N)

should be:
(if we calculate loss gradient with respect to a,p,n), then this one will come)
self.gradInput[1] = (n - p):cmul(self.Li:gt(0):repeatTensor(1,a:size(2)):type(a:type()) * 2/N)
self.gradInput[2] = (- a):cmul(self.Li:gt(0):repeatTensor(1,a:size(2)):type(a:type()) * 2/N)
self.gradInput[3] = (a):cmul(self.Li:gt(0):repeatTensor(1,a:size(2)):type(a:type()) * 2/N)

Please correct me, if i am wrong..

See Wikipedia.

OK, just putting this here, for reference. But it's high school calculus...

yeah its a high school calculus, but its my badness that I have assume that generally we use norm layer before triplet loss, and if we use norm layer, then I think this sinmple high school calculus will become more easier :)
(a-p)T(a-p)= 2ap
(a-n)T(a-n)=2an
derivative of loss with respect to p=2a
derivative of loss with respect to n= -2a
Thanks for high school calculus..

What is (a-p)T(a-p)= 2ap supposed to mean? I am not following...

its 2-2ap,
When the embeddings of the images are normalized to have unit l2 norm, the hinge loss function can be simplified to:
l(a, p,n) = max{0, g − 2ap+ 2an}
But, thanks for help...

And what's g?

Also (a-p)ᵀ(a-p) = a² + p² + 2aᵀp...

Oh, I see now what you're taking about. Sure...
I've implemented the triplet loss and its gradient. There is no assumption about the input embedding.

Yeah thanks.. its my badness I confused it with normalised input embeedings :) a^2, p^2 will become 1 (unit normalized embeeding) .. Thanks a lot..


Unable to move resource files in Java Projects view

A resource file cannot be drag and dropped to any folder in the Java Projects view, some works, some others not.
Step to reproduce
I create a file .env at the root of a Maven project (near the pom.xml) and then I try to move it to different folder:

the .vscode folder that contains a launch.json -> doesn't work
a newly created folder my-folder -> doesn't work
the src\main\resources\ folder -> work as expected
After moving it to src\main\resources\ folder, I try to move it back to the root, this doesn't work.

Guess it's related with:

https://github.com/microsoft/vscode-java-dependency/blob/main/src/views/DragAndDropController.ts#L238
https://github.com/microsoft/vscode-java-dependency/blob/main/src/views/DragAndDropController.ts#L276


wip: initial work on configstore

Not done with this but made enough progress to get feedback on parts of it.

Re. from #65

A couple methods that may be needed:

take all the records for one type of config and reduce them to one object
query all config records of all types and return the big config object that we get in a mapeosettings file

I don't think either of these are needed - on mobile we actually split up this single object and turn presets and fields into arrays. We just need to add a getMany() or list() function to the Datastore that will also be used for presets and fields (we will need this for observations too).
It could potentially be useful to join fields to presets on the server, although I don't see a huge issue if we do the join on the client (e.g. making multiple request, first for the preset, then for each field). Since we have very low latency I don't think this should be an issue?
I am wondering about the preset <-> fields relationship: should we maybe store the field versionId as well as the field docId? This would be useful if we are using historical presets to match old data: it would ensure we knew that data was collected with a previous version of a field definition. Not sure about this though, needs more thinking...

Fixes #65


When request is aborted, cancel event is not propagated to downstream services.

When a request is aborted by the client, spring-cloud-gateway is aware of it by Subscription.cancel(). Adding a filter can easily comprove this behavior, where
override fun apply(config: Config?): GatewayFilter {
    return GatewayFilter { exchange, chain ->
        chain.filter(exchange)
                .log()
                .doOnCancel { log.info { "ABORTED" } }
    }
}

when a request is canceled, generates
2020-06-01 11:20:03.824 INFO t=boundedElastic-7 reactor.Mono.LiftFuseable.1 - | onSubscribe([Fuseable] ScopePassingSpanSubscriber)
2020-06-01 11:20:03.824 INFO t=boundedElastic-7 reactor.Mono.LiftFuseable.1 - | request(unbounded)
2020-06-01 11:20:04.639 INFO t=reactor-http-epoll-2 p.s.g.s.filter.GatewayLogFilter - ABORTED
2020-06-01 11:20:04.640 INFO t=reactor-http-epoll-2 reactor.Mono.LiftFuseable.1 - | cancel()

But if it was routed to a downstream service, SCG does not notifies the service. I don't know if this should be default but, at least, should be configurable.
Looking at NettyRoutingFilter, only when a response is recieved, CLIENT_RESPONSE_CONN_ATTR is set as an exchange attribute, later used in NettyWriteResponseFilter#(cleanup) to dispose the call using doOnCancel event.
Version:

spring-cloud-gatway: 2.2.1

I also came across this situation mainly with mobile browsers when they go to the background and a request is in progress..

@joaobologna are you asking for logging in doOnCancel()?

@spencergibb no, I would like to notify the downstream service that the request was canceled. As for now, the service processing the routed request will compute it even though it was canceled by the client.

how would you do such a thing?

if the connection used on NettyRoutingFilter was disposed, wouldn't that solve the trick?

@violetagg does this make sense?

waiting for feedback

@joaobologna If I understand the issue correct, we have the following:
client -> gateway -> target system
       ^
       | here client drops the connection

If the client drops the connection before gateway has a chance to send a request, then gateway should not send it at all.
If the client drops the connection while gateway is sending the request, then gateway should cancel that operation.
Is my understanding correct?
Currently I do not see a public API provider by Spring Framework that can be used for obtaining the connection when on the server side.
@rstoyanchev Is there a way Gateway to understand that the connection between client -> gateway was dropped?

On the other hand if the connection is closed Reactor Netty will definitely propagate the cancellation to Spring Framework.

That's what I would expect. Isn't that what @joaobologna demonstrated in his original comment with a log message in doOnCancel?

@violetagg that's correct.

If the client drops the connection before gateway has a chance to send a request, then gateway should not send it at all.

This scenario probably already happens, since the connection was dropped and gateway is aware of it, no further code is executed to reach a point where the request is routed.

If the client drops the connection while gateway is sending the request, then gateway should cancel that operation.

This is what is not happening and I would like to. I tried to dispose the connection used in NettyRoutingFilter but unfortunately, it didn't work.

@rstoyanchev yes, it is.

@joaobologna So if you include the code below, it does not cancel the request?
Connection connection = exchange.getAttribute(CLIENT_RESPONSE_CONN_ATTR);
if (connection != null) {
	connection.dispose();
}

@violetagg CLIENT_RESPONSE_CONN_ATTR is only available if there is a response from the downstream service, so it is not possible to depose it during the call.
I did something like:
nettyOutbound.withConnection(connection -> exchange.getAttributes().put("conn", connection));

responseFlux
					.timeout(responseTimeout, Mono.error(new TimeoutException(
							"Response took longer than timeout: " + responseTimeout)))
					.onErrorMap(TimeoutException.class,
							th -> new ResponseStatusException(HttpStatus.GATEWAY_TIMEOUT,
									th.getMessage(), th))
					.doOnCancel(() -> cleanup(exchange));

@violetagg CLIENT_RESPONSE_CONN_ATTR is only available when there is a response from the server, so if connection is aborted before that, it will always be null.
I did save the connection on the exchange attributes on:
https://github.com/spring-cloud/spring-cloud-gateway/blob/cde4d564172f9eb87d8f74a221b1fa5dde42abc7/spring-cloud-gateway-core/src/main/java/org/springframework/cloud/gateway/filter/NettyRoutingFilter.java#L143
nettyOutbound.withConnection(connection -> exchange.getAttributes().put("conn", connection));

While configuring the Flux response, I added an doOnCancel event to dispose de connection:
https://github.com/spring-cloud/spring-cloud-gateway/blob/cde4d564172f9eb87d8f74a221b1fa5dde42abc7/spring-cloud-gateway-core/src/main/java/org/springframework/cloud/gateway/filter/NettyRoutingFilter.java#L197
...
		.doOnCancel(() -> cleanup(exchange));
	}
		return responseFlux.then(chain.filter(exchange));
}

private void cleanup(ServerWebExchange exchange) {
	Connection connection = exchange.getAttribute("haha");
	if (connection != null) {
		connection.dispose();
	}
}

On my tests, my downstream service is still unaware of it :disappointed:
I assumed since spring-cloud-gateway receives a cancel signal from the browser since the call originated from there, it could be reproduced with netty HTTP clients that are used in the project.
PS: downstream service is using webflux as well, and if the request is directly made to it (not going through gateway) it receives the cancel signal as well.

@joaobologna just to check that either you have log level trace or you invoked .withConnection outside of the if (log.isTraceEnabled()) {

haha I made that mistake at first @violetagg, but it is outside since it was tested as mentioned in https://github.com/spring-cloud/spring-cloud-gateway/issues/1751#issuecomment-638238213

@violetagg @spencergibb just checking if you have any idea how to solve the mentioned problem.

@joaobologna Please excuse me for the delayed reply, but I made one change in Reactor Netty that might help here https://github.com/reactor/reactor-netty/commit/c61e556fed1c7d28cd7d389a4126ded0b74a51a0
Will you be able to try it? (0.9.10.BUILD-SNAPSHOT)

Hi guys, is this issue resolved? it seems that still exists, is there any workaround?

Hi again, We tried the following code, but it seems that either doOnCancel or connection.dispose(); function.  Connection seems to be always null as joaobologna allready mentioned, If there another way to achieve the following ?
@Override
public GatewayFilter apply(Object config) {
private void cleanup(ServerWebExchange exchange) {
Connection connection = exchange.getAttribute(CLIENT_RESPONSE_CONN_ATTR);
if (connection != null) {
connection.dispose();
}
}
return (exchange, chain) -> {
        return chain.filter(exchange).doOnCancel(() ->
                { cleanup(exchange) ;
                    log.info("Cancel connection");
                }
        );
    };
}

Hi @violetagg,
I am also interested in this functionality, I would like to drop the connection to my downstream APIs in the event the client drops the connection with the gateway.
Any updates on this function?

:+1:

Is there any progress?


Build SPIRV-Tools from source without Python

The OSS project I am working on uses "build thirdparties from source" policy, but I don't want to introduce Python as build requirement either.
Is it viable to somehow pre-generate and deploy all the stuff generated via scripts? Any pitfalls?
I assume that any specific version of SPIRV-Tools+SPIRV-Headers will always yield the same artifacts, but maybe I am wrong and they are somehow compiler- or platform- dependent?
Also, would it be an acceptable PR if I made an option to cache codegen artifacts, or this project is not interested in supporting this usecase?

This is the draft of the solution I did for my project, in case someone have similar issue:
https://github.com/rbfx/SPIRV-Tools/commit/bf0d80190f38e36807b225cad1c7109c0fc570ab
I didn't test it much, but it works okay-ish.
Re-generation is ugly, I have to delete cache folder, configure, build, and configure again.
But when the cache folder is ready (which is always unless I update my thirdparties) it works fine.

Sorry, but we are not interested in supporting this.

@s-perron That's understandable, I'll keep the branch in my fork alive if anyone ever needs this behavior.

Thanks for asking in advance.  :-)


[BUG] Failing to Add Threat Intelligence when environment is upgraded.

What is the bug?
We have upgraded two of our environments (one from 2.15 and another one from 2.11) to 2.16. When trying to create a Threat Intelligence source local (using the Documentation sample and also a custom stix2), it fails.
The error message:
[exception] Error occurred while ingesting IOCs to [.opensearch-sap-iocs-ux1uv5ebhudpy2h-aqmh-1723747035697] with an error failure in bulk execution: [0]: index [.opensearch-sap-iocs-ux1uv5ebhudpy2h-aqmh-1723747035697], id [dfdc4b83-6b2c-4bc6-b06c-e4b30158f542], message [[.opensearch-sap-iocs-ux1uv5ebhudpy2h-aqmh-1723747035697] IndexNotFoundException[no such index [.opensearch-sap-iocs-ux1uv5ebhudpy2h-aqmh-1723747035697]]] [1]: index [.opensearch-sap-iocs-ux1uv5ebhudpy2h-aqmh-1723747035697], id [d3706759-5567-4551-ac28-59823d4b8140]
However, the index ends up being created (empty, but it gets created).
This goes a very long list for each IOC in the STIX json.
How can one reproduce the bug?
Steps to reproduce the behavior:

Ensure the environment is an Upgraded environment. ( The error doesn't happen in a fresh install )
Go to 'Security Analytics / Threat Intelligence'
Click on 'Add Threat Intel Source'
Select Local File Upload
Upload the file
Select all checkboxes and add a name
Click on Add Threat Intel Source

What is the expected behavior?
It should completed the creation of the TI Source and show in the list. The IOCs should be ingested in the index .opensearch-sap-iocs-randomindex.
What is your host/environment?

OS: [Windows 11]
Version [2.16 - Ugpraded]

Do you have any screenshots?

Do you have any additional context?
In the OSD logs, I can also see the error (besides the one above) - Alerting - MonitorService - searchMonitor: StatusCodeError: [alerting_exception] Configured indices are not found: [.opendistro-alerting-config]

Adding more info.
My cluster has auto index creation denied, after enabling it I was able to create.

[Weekly Catch All Triage - 1]

index creation denied, a

Hi @givilleneuve
Is the issue fully resolved?

you can message me at https://opensearch.slack.com/team/U052WCKACH3 on OpenSearch Slack where I can better understand your set up over a call
Am unable to reproduce this


[Feature-2932][Pipeline] Flink CDC pipeline supports transform

This is a draft pull request, and the functionality has not been completed yet.
Please discuss the details, implementation, and optimization under this PR.
I will complete the remaining code. thx

What is the more appropriate way to obtain the field name and data type of dataChangeEvent. after()?

Change the parameter definition of transform to the following:
transform:
  - source-table: mydb.app_order_.*
    projection: id, order_id, TO_UPPER(product_name)
    filter: id > 10 AND order_id > 100
    description: project fields from source table
  - source-table: mydb.web_order_.*
    projection: CONCAT(id, order_id) as uniq_id, *
    filter: uniq_id > 10
    description: add new uniq_id for each row

@lvyanquan Thanks for your review. I will fix these issues tonight.

There are only two bugs that have not been fixed yet. I will fix them again today.
1.A doc or more tests is necessary to show what kind of Projection we support after this pr. For example, can we support CAST Function now?
2.We need to support more types, such as some time types and float types.

Hi @aiwenmo , I  met Invalid signature file digest for Manifest main when testing this transform in E2e environment.
Can you help to fix it when building flink-cdc-dist jar?

Overall, look good to me, and can you fix the remaining minor issue with CI?

Completed features:

Support projection.
Support filter.
Support automatic evolution of projection's column.
Support SQL operators, functions, constants, and metadata columns.
Support restoring jobs from save points.

Todo:

SchemeRegistry saves the routing rules used to restore the TransformDataOperator.
Support UDF.
Support type conversion.
Support CASE WHEN.
Support the strategy of schema evolution.


Vectorize lnprobfn call

I added the capability to make lnprobfn() call the entire vector p all at once, instead of using map() to iterate over p. For custom lnprobfn() functions that are already highly vectorized, this can bring a large speed-up (for my custom use-case I get over an order of magnitude in speed-up).

I should note that the edits are in ensemble.py, and the vectorized lnprobfn() is toggled on or off via the keyword-argument "vectorize" which is set to False by default.

Current coverage is 72.63% (diff: 75.00%)

Merging #208 into master will decrease coverage by 0.02%

@@             master       #208   diff @@
==========================================
  Files            10         10
  Lines           823        826     +3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            598        600     +2
- Misses          225        226     +1
  Partials          0          0

Powered by Codecov. Last update ce41764...3bade53

Hi! I've implemented smth similar in PR#53. It really speeds up sampling sometimes.

Cool. Can you have a go at adding a usage example, some notes in the docs (maybe in "advanced patterns"), and some tests for this?

It should test that it performs properly under the correct conditions but it would be interesting to also see what happens if it is used incorrectly. Are useful exceptions thrown at the right times or does it just return garbage?
What if the results returned by the lnprob function are the wrong shape?
Does this support blob vectors too?
Is "vectorize" the right name for this argument?

Thanks and let me know if you need a hand getting this set up.

If I understand correctly, vectorize passes off all points into the lnprob function simultaneously. Since the loadbalance (and runtime_sorting) option is implemented assuming single evaluations, the vectorize and the loadbalance options are likely to be mutually exclusive. If so, a simple check that both options are not enabled might be a good idea.

Hi there,
@dfm: yes, I'll look into making some tests and checking compatibility with blob vectors. About the name of the on-off flag, I thought of "vectorize" because it is operating on the whole p vector (allowing custom-lnprobfn functions that are by design vectorized to be exploited) rather than iterating over the p vector with the MAP function. But it could also be something else. Let me know if you have any ideas.
@manodeep: I'll look into that, shouldn't be hard.
Here is a benchmark for my personal use-case (I've written personal functions that wrap around emcee's run_mcmc but they are in effect just calling run_mcmc). I think that in other cases vectorize=True does not lead to a speed up and may actually lead to producing more overhead. But I'll look into making a generic use-case.

Best,
Nick


fixed - search for "process/online" [835]

https://github.com/participedia/usersnaps/issues/835

LGTM 👍


Statistics Panel needs to be added

Related to https://github.com/PRIDE-Archive/xi-web-api/issues/3
Statistic panel

[x] - Number of projects
[x] - Number of Species
[x] - Number of proteins
[x] - Number of peptides
[x] - Number of spectra

Plots

[x] Number of projects per species
[x] Number of peptides per protein

done


add support for ASCII transfer mode

Hi,
I don't like ASCII mode but it's a basic piece of RFC 959 so I think we should support it.
This implementation is based on pyftpdlib, please search self._current_type here.
For the converter I evaluated this library too, if you change the included benchmark you can see that it is slower than the proposed implementation, for example using this code:
    c := crlf.NewReader(readerCRLF)
    _, err := io.Copy(ioutil.Discard, c)
    panicOnError(err)

you have this result:
BenchmarkASCIIConverter-12          802696          1309 ns/op        8368 B/op           4 allocs/op

replacing it with this code:
    c := newASCIIConverter(readerCRLF, convertModeToLF)
    _, err := io.Copy(ioutil.Discard, c)
    panicOnError(err)

the result is this one:
BenchmarkASCIIConverter-12         1611764           775 ns/op        4255 B/op           3 allocs/op

I did some testing using filezilla as client and some big text files such as the ones here
I also tested the file obtained using this command:
yes "Some text entry here. This will be on each line..." | head -n 10000000 > bigfile.txt

it seems fine, however more testing would be better :) thank you!

Thank you very much @drakkan, this looks perfect !
To be honest I was planning on never implementing it. But anything that brings us closer to the FTP standards is welcome.

thank you for merging this, reading the RFC 959 seems we only miss STOU, STRU and MODE to be fully compliant. I don't know how these commands are relevant today but if you want I can implement them too in the same way as pyftpdlib: basically we just reply to the commands both STRU and MODE will be not configurable, for STOU I have never seen a modern client that use it but implementation should be quite easy, please let me know what do you think about, thanks

For the ASCII Converter, I don't think performance is important here and avoiding having too many dependencies is. So it's perfect as it is.

Thank you very much @drakkan, this looks perfect !
To be honest I was planning on never implementing it. But anything that brings us closer to the FTP standards is welcome.

thank you for merging this, reading the RFC 959 seems we only miss STOU, STRU and MODE to be fully compliant. I don't know how these commands are relevant today but if you want I can implement them too in the same way as pyftpdlib: basically we just reply to the commands both STRU and MODE will be not configurable, for STOU I have never seen a modern client that use it but implementation should be quite easy, please let me know what do you think about, thanks

For the ASCII Converter, I don't think performance is important here and avoiding having too many dependencies is. So it's perfect as it is.

The MODE command would be nice! See https://github.com/fclairamb/ftpserverlib/issues/451.


Change provider submit() to require an exception on failure, rather than None

Is your feature request related to a problem? Please describe.
When a parsl provider cannot start a requested job, it can either raise an exception or return a False-like object (such as None).
See docstring here:
https://github.com/Parsl/parsl/blob/896514ff211678a0cc9161da34406f8720044d02/parsl/providers/base.py#L66
Code invoking provider submit, in that second case, does not get much information about why a submission failed, meaning that further exception messages cannot report much more than: ScalingFailed(self, "Attempt to provision nodes did not return a job ID"). While parsl.log often contains further details (for example, submit command stderr), that is not propagated through the exception mechanism to the end user.
I would like to change this code to report all errors as exceptions, never as None, giving a clearer error path.
This can also tidy up some slight ambiguities in whether a job ID or failure is returned: is the integer 0 a failure or a job ID?
Code calling submit already has to handle submit raising an exception, so shouldn't need changes. Provider code will need changing to raise a suitable exception rather than defaulting to (usually) None, but that drives better error reporting.

This seems a very sensible change.  I'm a :+1:.


Service Request by David Lounsbury about programming Units for 'Other' measures

[ ] Problem NOT RESOLVED
[ ] Problem PARTIALLY RESOLVED
[ ] Problem FULLY RESOLVED and Issue is Closed

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

@jbazinet1111 Can you help us figure out a way to better units for Other measures?
Currently, I think any user can and an Other measure of their own, but that feature does not prompt for 'Units'.
Maybe we should only allow Administrators to ADD/define Other measures?  Can that be arranged?
Even if we use List Options to edit units for Other measures, it does not seem to properly update data in the Tracker.
Thoughts?
Best,
David

First, let me see if we have a bug with the units in List Options. If that is working fine, then perhaps we can force the users in that direction.

So there is only a single OTHER MEASURE defined within the list options and it is for Expand MOUD Availability.
I would suggest that perhaps there should be others defined within the Options (which is Admin only) such that the units are clearly set up.
Maybe look at what the users have typed in as their "other measures" in the ad-hoc specifications process and create the same measures (w/ units) in the Options. That might make things easier for everyone.
If that doesn't seem feasible then we can look at alternatives but since that functionality is already there, we might as well try it.

This issue links with #39, too.
DL

I think we may need to restrict users from ADDING a new 'Other' measure. Somehow, we need to inform users that, to add an other measure, they need to send a Tracker TA Service request, or find someone with Admin privileges who would know to add the Other measure and update the units for it.
@jbazinet1111 What do you advise to address this matter?

This would certainly work. Removing that functionality we built previously will take about a two hours of effort.

Very good! Okay, let's go with this!
THANK YOU Jeff

This has been deployed

Thanks Jeff!
Donrie and Nick, we need to document the Discontinue (a strategy) feature in our User's Manual. #2

What bug?

This looks to be resolved, thanks Jeff!

thanks all.
DL


traverse/mapM for sequence and <$>

Hlint suggests to replace sequence . fmap f  and sequence (fmap f a) with mapM, but it doesn't for sequence (f <$> a). Is there any reason not to do so or this case is just accidently missed?

 Closing the issue.

Sorry for disturbing you. I really have an old version.