allow pass manager options

Signed-off-by: Kevin O'Brien<EMAIL_ADDRESS>

For debugging, its useful to be able to see the IR after each pass. This PR enables taht.

Hi Kevin, the onnx-mlilr executable was conceptualized to be a simple user-facing frontend, that's why it contains a short help message description (https://github.com/onnx/onnx-mlir#using-onnx-mlir). I suspect this patch may obfuscate that concise help message.
OTOH, I feel like the proper thing to do may be to register our usual compiler pass pipelines in onnx-mlir-opt, which is supposed to be an onnx-mlir debugger, although I understand that as we amass more compiler passes, it is increasingly impossible to type out all the passes executed by our main compiler frontend as command-line options, therefore the need to register them as pass pipelines mentioned earlier.
OTOH again, if you still think this is the right thing to do, maybe we should think instead about merging onnx-mlir and onnx-mlir-opt into one, given this patch will make two utilities overlap more in functionalities.

I assume that this change is to enable all the pass related flags defined in mlir for onnx-mlir.  The flags include IR printing, timing, statistic and crash reproducer. They are the debug flags provided by MLIR.  I tried to use them once and the flags did not work. I will be glad to have these flags enabled.
As for Tian's concern about the interface, I think we will not, and should not, put these flags in our command line document. We just add a link to MLIR and say onnx-mlir accepts them.  It is part of MLIR functionalities.

@tjingrant The way I see it, onnx-mlir-opt is usefule for running one or two passes, maybe starting from an intermediate stage in the compile process. This is certainly a good tool to have, and I don't think we should eliminate it. The other use-case is running the complete compile (or running from ONNX input up to some desired point), this is what onnx-mlir does. When running like this and trying to find a problem in the series of transformations, it is useful (actually necessary) to be able to print out the IR after all or some subset of passes so that one can inspect it to find erroneous transformations.
I don't know how to eliminate the extra output from the --help flag, which I agree is problematic.
Somewhat tangentially, I think this is what @CHUNERnubi is asking for in issue #506 .

@tjingrant Maybe another solution would be to have onnx-mlir write out a pipeline description file which onnx-mlir-opt could then read in.

Jenkins Linux s390x Build #121 [push] allow pass manager optio... started at 18:22

Jenkins Linux amd64 Build #121 [push] allow pass manager optio... started at 17:22

Jenkins Linux amd64 Build #121 [push] allow pass manager optio... passed after 42 min

Jenkins Linux s390x Build #121 [push] allow pass manager optio... passed after 43 min


[RFC] experimenting with dynamic coin types

Playing with two different ideas for more dynamic coin types (and soliciting ideas for others)--feedback welcome!

Goal: Trying to solve specific problems for DeFi builders that need to work with heterogenous collections of coins and find the strong types of Coin painful
Non-goal: introduce a new coin standard understood by the wallet/explorer, or that is broadly used outside of the (I think relatively narrow) needs of these DeFi builders. A new coin standard could happen in the long term and would likely be dynamic, but would likely want a lot more features that aren't ready yet (coin metadata v2, private balances, traits or similar to allow custom data inside coins, restricted transfers, dynamic loading of per epoch-immutable or immutable objects like coin metadata, possible consolidation with closed loop tokens, ...)
Two approaches: Coins container w/ magic native functions, for split, join, etc. designated DCoin type + conversion functions from strongly typed coins. I think the second one is more appealing/more flexible, and can already be implemented with no Sui Framework changes. However, a framework version of DCoin would surely help with interoperability/evolution toward wallet and explorer support if that make sense

Hey @sblackshear
Thanks for this!
I would favor the second approach as it is easier to reason as the API is 1:1 with the coin. Also, the concept of having a Registry to mint/burn the DCoin is similar to ETH <> WETH, which most DeFi builders are familiar with.
P.S. - I do support adding the decimals as well 💯 . Decimals are highly used in DeFi and can save function argument(s) slots.
I cannot wait to use this!


Random comics are not really random

Everytime I open the app and click on random, same comics are generated as random.
Steps to reproduce:

Open the app.
Click on random - some comic is loaded (let's call it C1).
Click on random - some comic is loaded (let's call it C2).
Click on random - some comic is loaded (let's call it C3).
etc.
Close the app.
Open the app again, and click on random.

Actual result:
I see comic C1, then C2, C3, etc.
Expected result:
A see random images, not the same ones all over again.
Workaround:
I have not found a workaround, but the set of shown comics gets randomly regenerated every time new comic is posted.

Looks a lot similar to this issue https://stackoverflow.com/questions/73475522/kotlin-random-always-generates-the-same-random-numbers


Can't get 6RD to work with Tweak NL

I was following  #1501 and am trying to get 6RD working with TweakNL.

Thanks for the quick reply, I have installed the kernel and the patches:
root@firewall:~ # uname -a
FreeBSD firewall.diekgait.nl 11.1-RELEASE-p11 FreeBSD 11.1-RELEASE-p11  bdf43282b19(master)  amd64

And the interface is looking good:
wan_stf: flags=4041<UP,RUNNING,LINK2> metric 0 mtu 1280
        inet6 2a02:58:c9:8100:: prefixlen 56
        nd6 options=1<PERFORMNUD>
        groups: stf

Also when I send a ping from an IPV6 source to 2a02:58:c9:8100:: I can see the packets comming in from the 6RD Border Relay:
root@firewall:~ # tcpdump -i em1 host <IP_ADDRESS>
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on em1, link-type EN10MB (Ethernet), capture size 262144 bytes
16:29:30.294579 IP <IP_ADDRESS> > 84-245-9-129.dsl.cambrium.nl: IP6 diekgait-1-pt.tunnel.tserv18.fra1.ipv6.he.net > 2a02:58:c9:8100::: ICMP6, echo request, seq 1, length 64

However it does not reply and when I try a ping from the firewall I get:
root@firewall:~ # ping6 google.com
PING6(56=40+8+8 bytes) 2a02:58:c9:8100:: --> 2a00:1450:400e:807::200e
ping6: sendmsg: Network is down
ping6: wrote google.com 16 chars, ret=-1

I think you may have installed the kernel but skipped the base update that is required for ifconfig to send the 6RD options.
# opnsense-update -vb

This says 18.1.11, or?

root@firewall:~ # opnsense-update -vb
18.1.11-amd64

Okay, right, you need this too...
# opnsense-update -br 18.7-stftest -n "snapshots\/dummy"
# /usr/local/etc/rc.reboot

Cheers,
Franco

That gives me no prefix on the interface and no route:
root@firewall:~ # ifconfig wan_stf
wan_stf: flags=4000<LINK2> metric 0 mtu 1280
        nd6 options=21<PERFORMNUD,AUTO_LINKLOCAL>
        groups: stf
root@firewall:~ # netstat -rn6
Routing tables

Internet6:
Destination                       Gateway                       Flags     Netif Expire
::1                               link#4                        UH          lo0
fe80::%em0/64                     link#2                        U           em0
fe80::21b:21ff:fe90:3b22%em0      link#2                        UHS         lo0
fe80::%em1/64                     link#3                        U           em1
fe80::21b:21ff:fe90:3b23%em1      link#3                        UHS         lo0
fe80::%lo0/64                     link#4                        U           lo0
fe80::1%lo0                       link#4                        UHS         lo0
fe80::6ef0:49ff:fede:8ece%ovpns1  link#8                        UHS         lo0
fe80::%em0_vlan50/64              link#9                        U      em0_vlan
fe80::21b:21ff:fe90:3b22%em0_vlan50 link#9                      UHS         lo0
                 link#4                        UH          lo0
fe80::%em0/64                     link#2                        U           em0
fe80::21b:21ff:fe90:3b22%em0      link#2                        UHS         lo0
fe80::%em1/64                     link#3                        U           em1
fe80::21b:21ff:fe90:3b23%em1      link#3                        UHS         lo0
fe80::%lo0/64                     link#4                        U           lo0
fe80::1%lo0                       link#4                        UHS         lo0
fe80::6ef0:49ff:fede:8ece%ovpns1  link#8                        UHS         lo0
fe80::%em0_vlan50/64              link#9                        U      em0_vlan
fe80::21b:21ff:fe90:3b22%em0_vlan50 link#9                      UHS         lo0

I've disabled 6rd and reconfigured it but that makes no difference.

The ifconfig output suggest you still have the wrong ifconfig binary. :( You could also try the 18.7-RC1 image. Things are a bit confusing at the moment. Lots of different states that will sync up in the final 18.7.... almost there though... end of July.

Cheers again! Installing 18.7-RC1 does the trick, after that I'm able to reach google. There are still some issues though:
When I configure the lan interface as a tracker interface it picks the first /64 from my customer prefix. This /64 contains the default router and therefor all ipv6 connectivity breaks.
I've now configured it staticly with the 2nd /64 from my prefix. And this works if I staticly configure my client because for some reason radvd won't start so no router advertisements are being sent out.

If you are tracking WAN from LAN, change its IPv6 Prefix ID from 0 to something else so the first prefix isn't used by LAN :)

That makes sense :) RA's are working now when I set it to tracking. If with 6RD the first /64 can never be used it might be a good idea to default the prefix-id to 1? What also would be nice is to add an auto option to the 6rd configuration so it takes the configuration from v4 dhcp option 212. Is that already on the feature request list?

Historically, Prefix 0 is ok, because sometimes the PD is /64 so you only have one in the first place. I wanted to improve this a bit because of you have two LANs the second one is set to 0 as well so that's a bit iffy.
Let's close this ticket and you can create a new one with the dhcp request and also mention the prefix id oddity there? I didn't know DHCP can advertise 6RD. It's certainly handy to get the values directly from the request but I'm unsure how much work it is to auto-configure using it so we'll have to discuss and see. Ok?


Disable quick save on new designs

Description (what's changed?)
Removed the Quicksave button when you're creating a new design. This solves the issue.
Testing instructions

Clear access tokens from local storage. Try to create a new design as a guest. You won't see the quick save button. You have to go through Finish&Share which will prompt you to login if you're a guest

Task linked: CU-86aya5mef [Demo] Creating a design and clicking 'Save' without logging in returns 401


An OpenSSL.crypto.Error occurs when a certificate contains an extension not known to OpenSSL

The problem occurs e.g. with ey.com and office365.com
The proposed fix is to catch the error and show the extension in hex.
It is not ideal, as the extension name is shown UNDEF instead of the object ID. I regard this as an deficiency of the X509Extension class and didn't find an easy way around.

Thanks again, these bug fixes are extremely helpful.
As you've noticed, the PyOpenSSL API is less than ideal here: a bad value raises a generic OpenSSL.crypto.Error error, while a bad name returns the fixed string UNDEF. Perhaps it'd be better to parse the certificates using cryptography.x509.load_pem_x509_certificate, from a more carefully designed library.
And it's a small change since we already depend on cryptography as a transitive dependency from  pyopenssl.

Thanks again, these bug fixes are extremely helpful.
As you've noticed, the PyOpenSSL API is less than ideal here: a bad value raises a generic OpenSSL.crypto.Error error, while a bad name returns the fixed string UNDEF. I couldn't find a way to get the raw extension type either. Perhaps it'd be better to parse the certificates using cryptography.x509.load_pem_x509_certificate, from a more carefully designed library.
And it's a small change since we already depend on cryptography as a transitive dependency from pyopenssl.

Great idea


SPGO Populating Workspace Keeps Spinning

Note: for support questions, please use stackoverflow and tag the post with #spgo.

I'm submitting a ...

[ ] bug report
[ ] feature request

What is the current behavior?

If the current behavior is a bug, please

Your spgo.json file (You may replace corporate PII with "xxx")
Steps to reproduce

What is the expected behavior?

Please tell us about your environment:

OS and Version
SharePoint Location (Online vs. On Premise)
Authentication Type (e.g. Digest, NTFS, ADFS)

Hello @chrishasz ,
my organization just migrated to SharePoint SE and we are trying to connect via Visual Studio. I created the json file but when I try to populate the workspace it keeps spinning. Our servers are on premises so I chose NTML as the authentication type.
{
    "sourceDirectory": "src",
    "sharePointSiteUrl": "https://bc-net",
    "workspaceRoot": "c:\\Users\\jangelogianopulos\\Desktop\\BCNetSE",
    "publishWorkspaceOptions": {
        "destinationFolder": "/",
        "globPattern": "c:\\Users\\jangelogianopulos\\Desktop\\BCNetSE\\src\\**\\*.*",
        "localRoot": "c:\\Users\\jangelogianopulos\\Desktop\\BCNetSE\\src"
    },
    "publishingScope": "SaveOnly",
    "authenticationType": "NTLM",
    "remoteFolders" :[
        "/_catalogs/masterpage/**/*.*"
    ]
}

Please let me know what I can do to fix this.
Thanks


Feature: Chording

OS
macOS Sonoma 14.3
LinearMouse
0.9.5
Is your feature request related to a problem?
I have an old mechanical ADB trackball that I absolutely love and have gotten to work with my M1 Mac via a converter box, but because it's so old, it has only two buttons and no means of scrolling built into the hardware (scroll wheel, etc). It has forced me to revert to interacting with the GUI as I would have many years ago, and I've become keenly aware that some of the changes and additions to input-device conventions have been major improvements.
Describe the solution you'd like
I'd like to be able to "Assign Actions to Mouse Buttons" in which multiple mouse buttons (plus whatever modifier keys) have to be pressed to trigger the action. In the case of my trackball (only 2 buttons), both the Left and Right buttons would be pressed at the same time. This would allow for an even wider range of customization. So, for example:

Chording Left and Right could be treated as a click lock (like holding down the Left button, or the same as Tap and Hold on a trackpad)
or as a scroll lock (the trackball input registers as scrolling until the chord is performed again)
Control + Left Click + Right Click might be used to bring up Mission Control. (as is currently possible with just one mouse button)
Maybe the user assigns Shift + Control + L + R to bring up the Launchpad
Command + L + R brings up the App Switcher

And so on. Allowing for chording multiple buttons allows the user to assign many more actions to mouse buttons.
Describe alternatives you've considered
Again, my exposure to this was Kensington Mouseworks for Mac OS 8.
Additional context
Having all of the options (that I'm posting across 3 feature request threads today) with Kensington in the OS 8 days was really great: they massively improved my productivity within the GUI.

Update: as far as I can tell, this post is asking for the same thing, I think.


plsql if

MyBatis version
3.x.x
Database vendor and version
Test case or example project
Steps to reproduce
Expected result
Actual result

Looks empty.


support cast for decimal data type

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
In the https://github.com/apache/arrow-datafusion/issues/122#issuecomment-975036335, we need  to support decimal for cast and try_cast
In the first step, we just support signed numeric data type to decimal, and other casting rules related to decimal will be supported in the feature.
future

[x] signed numeric to decimal https://github.com/apache/arrow-rs/pull/1044
[x] unsigned numeric to decimal @alamb do we need to support this? My opinion is that unsigned data type is not in the standard SQL data type, and we no need to support this.
[x] UTF-8 to decimal https://github.com/apache/arrow-rs/pull/3281
[x] decimal to signed numeric data type https://github.com/apache/arrow-rs/pull/1073
[ ] decimal to UTF-8
[x] decimal to decimal https://github.com/apache/arrow-rs/pull/1084

enhancement

[x] optimize iter the decimal array and use the from iter to construct Int、Float Array https://github.com/apache/arrow-rs/issues/1083

[x] add benchmark result for cast decimal data type

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

please assign this issue to me.
Thanks

unsigned numeric to decimal @alamb do we need to support this? My opinion is that unsigned data type is not in the standard SQL data type, and we no need to support this.

@liukun4515  I personally think it would be valuable to support unsigned numeric (aka PrimitiveArray<T>) to / from the decimal type, though we can also leave it as a follow on PR / issue (doesn't have to be in the first implementation).
I suspect once you implement PrimitiveArray<T> to/from DecimalArray for signed variants the incremental cost to supporting the unsigned numeric variants will be quite low.

Ok, I will implement signed numeric first, and leave other as the follow-up PR.

On commit ab48e69099f2d7c4178 building with nightly-2021-12-05-aarch64-apple-darwin I am getting:
---- record::api::tests::test_convert_double_to_string stdout ----
thread 'record::api::tests::test_convert_double_to_string' panicked at 'assertion failed: `(left == right)`
  left: `"1e-15"`,
 right: `"0.000000000000001"`', parquet/src/record/api.rs:1097:9

---- record::api::tests::test_convert_float_to_string stdout ----
thread 'record::api::tests::test_convert_float_to_string' panicked at 'assertion failed: `(left == right)`
  left: `"1e-15"`,
 right: `"0.000000000000001"`', parquet/src/record/api.rs:1084:9

The string formatter is:
https://github.com/apache/arrow-rs/blob/ab48e69099f2d7c4178a75f7e1cf8b223ecf8d76/parquet/src/record/api.rs#L716-L728
Where are the requirements for this?

Where are the requirements for this?

I do not know @chadbrewbaker
Some code archeology in  https://github.com/apache/arrow-rs/blame/ab48e69099f2d7c4178a75f7e1cf8b223ecf8d76/parquet/src/record/api.rs#L717 suggests that there has been a special case for checking that value for quite a while
Given that 0.000000000000001 is right on the boundary, perhaps we need to adjust the code to reflect differences in floating point on aarch64?

Same errors on x86.
This seems to be the last commit that changed the rounding.
https://github.com/apache/arrow-rs/commit/21cb780808407fc9fca3d028106181c5c9ae54ee

@chadbrewbaker  On a mac, the tests pass for me using cargo test -p parquet (using stable)
$ cargo --version
cargo 1.57.0 (b2e52d7ca 2021-10-21)

I also get the diff using nightly:
(arrow_dev) alamb@MacBook-Pro-2:~/Software/arrow-rs$ cargo +nightly --version
cargo 1.58.0-nightly (294967c53 2021-11-29)

If you want to propose a patch to allow the tests to pass on both nightly and stable that would be 👍

Bah - my stupidity not running stable. I'll diff what they changed in nightly. Seems to be cross-platform.

I'll try to upstream our test for the "1e-15" boundary. Doesn't seem to be one now.
https://github.com/rust-lang/rust/blob/c5ecc157043ba413568b09292001a4a74b541a4e/library/alloc/tests/fmt.rs#L137-L155

TIL
cargo bisect-rustc --start 2021-10-01  -- cargo test test_convert_double_to_string

I bisected the 1e-15 conversion regression down to this changeset between nightlies.
https://github.com/rust-lang/rust/compare/1af55d19c...efd048394

I bisected the 1e-15 conversion regression down to this change-set between nightlies.
<EMAIL_ADDRESS>
From your range of commit you can see that there is a commit named Automatic exponential formatting in Debug that added an internal logic that switch to the exponential representation in some cases when using the Debug formatter.
Rust make no promises about Debug formatting and can change it's output anytime for any reason but this is not the case for the Display formatter which has guaranties for how it display thing.
I would suggest changing your formatting to the Display (ie write!(f, "{:?}", value) -> write!(f, "{}", value)) formatter.

@alamb we need to speed up reviewing these two pull requests https://github.com/apache/arrow-rs/pull/1044 https://github.com/apache/arrow-rs/pull/1073, and they can be merged into the release of arrow-rs 6.5.
They will be used in the datafusion for comparison operation and mathematics operation for decimal type.

@liukun4515  I will do so today

There are bugs for cast decimal to decimal.
If we cast decimal(1256,12,2) to the type of decimal(12,1), the result should be decimal(126,12,1) not the decimal(125,12,1)

👍  thanks @liukun4515  -- maybe it is worth its own ticket to talk about the rounding vs truncation behavior of decimals while casting

Nice! I will look at the commit. For ML workloads, also going to need a version that stochastically flips the last bit. https://twitter.com/id_aa_carmack/status/1296295284994183168

Possibly related https://github.com/apache/arrow-rs/pull/3281


better german translation de.json

wait to merge, till @Fabio1988 review it!

@TurtIeSocks ready to merge this, it's fine :)

Are you sure about battle_changes_poracle and slot_changes_poracle? @Fabio1988 @ReuschelCGN this is meant to be the same as the command used to add slot_changes and battle_changes in Poracle command?

Ahh I'm not sure about battle_changes_poracle.... But slot_changes_poracle is translated with Platzänderung

So those translations have to be synched with poracle?

Yeah. They're for displaying the command in the poracle Manager so they should indicate the same way a user would type the command into Poracle.

https://github.com/KartulUdus/PoracleJS/blob/6dd7e748a2800f18e32e44f35a0c19c8604137f4/locale/de.json#L490
"slot changes": "Platzänderung"
That should be updated to singular form pls @ReuschelCGN
But I think "battle_changes" is missing in poracle translations


Feat: Add Bulma, Headless UI, Laravel, PHP

Changelog:

Add the following terms to the dataset: Bulma, Headless UI, Laravel, PHP
One could argue that React is a library — stated on their homepage and some people wrote about it as well. NextJS is a framework based on React.
One could also argue that Bootstrap is a CSS framework — it provides utilities similar to Tailwind CSS. Its implementation e.g. React Bootstrap might be a UI component library.

I do not plan to add php or other server side scripting languages.
As for framework and library, I suggest you can read up on the differences more. I do not plan to change existing categories.

Upon a closer look, I agree that React can be considered a library on its own.

Upon a closer look, I agree that React can be considered a library on its own.

With that being said, I would still like to categorize it in the same category as Vue and Angular just because they are often compared (with the whole ecosystem considered, hence more on the framework side).
If you have good suggestions on how to do it, please let me know below in comments.

With that being said, I would still like to categorize it in the same category as Vue and Angular just because they are often compared (with the whole ecosystem considered, hence more on the framework side).
If you have good suggestions on how to do it, please let me know below in comments.

@paradite Ah, understood. I decided to leave it back as it is to reduce confusion until there's a better term. Seems like most comparison blog posts commonly refer them as a web framework (example 1, example 2).
I've also removed PHPs.

Yes. Exactly. Most of the items look good. I will merged and makes some minor edits.


Some proposals on semanticdb protos and Scala3 semanticdb extractor

I think it might be better to do some refactoring around semanticdb protos in both scalameta and Scala3 before working on the improvement of Scala3's semanticdb extractor.
What do you think about this? @tgodzik @bishabosha @olafurpg
some disclaimers :)

I'm not sure this is an appropriate place to discuss (because this issue is related to both scalameta and scala3 (and lsif-java and every library that use semanticdb) if there's a better place to talk about it, let me know :).
I don't mean to blame the decision of copy-and-pasting generated code in Scala3; I believe it was a good decision for Scala3's semanticdb extractor for moving the project forward (actually, there were not so many case classes are needed for extracting semanticdb's symbol occurrences).

Background

The data structures for semanticdb in Scala3 are copy-and-pasted (and some edits so Scala3 doesn't need to have unnecessary dependencies to protobuf).

https://github.com/scalameta/metals/discussions/2593#discussioncomment-529949

The semanticdb protobuf schema is not included in semanticdb's jar like common protos
The semanticdb proto in lsif-java is copy-and-pasted with some modification

Current problem with Scala3 and Semanticdb
Since the data structures for semantidbdb in Scala3 (e.g. TextDocument.scala) are copy-and-pasted, there're some problems:

If we want to add more semanticdb support to scala3 and need additional data structures (such as ClassSignature, MethodSignature, TypeRef,... everything), we have to do copy-and-paste and some modifications for all those generated case classes, which is tedious.
Scala3 have to manually catch up the original protobuf schema's update (by generating scala code and copy-and-pasting)
Cannot catch up scalapb's update (if there's an update in generated code).

Proposal
publish semanticdb proto
It might be better to publish semanticdb proto file (and generated scala code) as common-protos do, so that lsif-java and Scala3 can consume the semanticdb protos without copy-and-pasting.
Make Scala3's semanticdb extractor depends on the published semanticdb proto
I think it would be easier to maintain the semanticdb data structures in the Scala3 if it generates the code from semanticdb protos instead of copy-and-pasting generated Scala codes.
The problem with this is, if scala3 relies on semanticdb proto and its generated code, Scala3 will have a dependency on scalapb, which is unpreferable (because compilers should have as small dependencies as possible). I guess it is inevitable as long as we develop a semanticdb extractor as part of the Scala3 compiler and use scalapb in it.
It could be better to separate the semanticdb extraction phase as a compiler plugin as the semanticdb plugin (for scala2) does; this enables us to detach the dependency to scalapb from scala3.

I think it would be easier to maintain the semanticdb data structures in the Scala3 if it generates the code from semanticdb protos instead of copy-and-pasting generated Scala codes.

The problem with this is, if scala3 relies on semanticdb proto and its generated code, Scala3 will have a dependency on scalapb, which is unpreferable (because compilers should have as small dependencies as possible). I guess it is inevitable as long as we develop a semanticdb extractor as part of the Scala3 compiler and use scalapb in it.

It could be better to separate the semanticdb extraction phase as a compiler plugin as the semanticdb plugin (for scala2) does; this enables us to detach the dependency to scalapb from scala3.

I would prefer to avoid making it again a compiler plugin, since the issue here is that we would need to publish it for every version of the compiler and those will come out every 4 weeks. Additionally, it's currently super easy to have semanticdb in Metals, just add the additional flag, which makes it possible to release Metals within a day.
The alternative approach would be to have the semanticdb schema in Scala 3 compiler and generate protobuf files from it. We could use the protobuf java library here instead of scalap or maybe using scalap wouldn't be that big of a problem if it's just a compile dependency. We don't need to have it in the runtime (?) but just to generate the sources. @bishabosha what do you think? Any other ideas on how to avoid having to copy paste the code?

Thank you for starting this discussion! I’m excited that you’re looking into this :)
I think the best solution is to continue to copy paste the relevant parts of the generated code into Dotty. However, SymbolInformation requires significantly more code than SymbolOccurrence so I can understand this is painful. My reasoning for copy pasteing the code is that you want to be able to compile the compiler codebase from source. While it’s tedious to copy paste and adjust the generated code, it’s a one time thing (Maybe it can be automated with Scalafix?). SemanticDB as a format has been pretty stable so you it’s not something that needs to be done often.

Thank you for the comments @tgodzik @olafurpg
It's good to know the background of why we embedded SemanticDBExtractor to the compiler instead of separating it as a plugin.
As @olafurpg mentioned, this process is a one-time thing, it would be a decent option to keep copy-paste-and-adjust the generated code at this moment (and automating the process with Scalafix is a good idea!)
Therefore, I'm going to work on a project that generates from semanticdb.proto and adjust it for Scala3 codebase (using Scalafix).

Though it's a bit crappy, I think I could generate and adjust semanticdb data structures for Scala3!
https://github.com/tanishiking/semanticdb-for-scala3/tree/main/output/src/main/scala/generated/dotty/tools/dotc/semanticdb
I'll try to build scala3 with these generated codes

Since the generated Scala code is merged into Scala3 repository, and it works really well to update the schema, I'm closing this issue.

https://github.com/lampepfl/dotty/pull/12780
https://github.com/lampepfl/dotty/pull/13239

If there's more to talk about the semanticdb schema, let's create another issue and discuss there :)
Thank you so much all the comments @olafurpg @tgodzik @bishabosha !


Add transcript like the one in Youtube.

I just add the transcript module to look up the word and excerpt quickly. It works like the one on Youtube.

It will change currentTime when clicking on the transcript text.
It always highlights the text of currentTime.
Transcript text will scroll to the text of the currentTime when the pointer does not hover on the transcript.

I think the transcript is a good selling point for some language learners like me.
By the way, I limit adding the subtitle to only one video so I don’t need extra code to make transcript work on multiple videos. I don’t think any user will watch two videos with subtitles at the same time. If you don't like it, I will change it.
Edited:
More features:

support more websites(can work on videos in a nested iframe).
delete in the video list if the video is removed(sometimes the advertisement has the video tag and will be removed)
fix some warnings on vue-eslint/recommend and format some files using prettier.
change the popup position on click
scroll to the center of the video when hovering on the video card if the video is not in the viewport(learn from substital)

This is made mainly for my own purpose. It you don’t like it, just close the pull request and I will move it to my own branch.

Sounds really interesting.
I will have a look and will give you feedback.

I REALLY like the transcript feature, but it seams that it does not work correctly with the offset.

Nice, thanks for the feedback.
Just fix the offset problem in the transcript window by changing the parsed to withOffsetParsed.

I know the meaning of the excerpt component now. I just don't know how to use it previously. I think it's also very useful.


Wider Version Support

Wider Version Support

This will fail with Laravel v 6.X & 7.X as the CartItem class contains references to the array helpers


Prevent concurrent GitHub workflow runs

Current dependencies on/for this PR:

main

PR #15   👈

This comment was auto-generated by Graphite.

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (ebe7e37) 92.42% compared to head (bd39813) 92.42%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   92.42%   92.42%
=======================================
  Files           5        5
  Lines         198      198
=======================================
  Hits          183      183
  Misses         15       15

Flag
Coverage Δ

unittests
92.42% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Merge activity

Oct 11, 6:00 PM: @sumeet-revefi started a stack merge that includes this pull request via Graphite.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_8b_hs37fvf/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_8b_hs37fvf/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_8b_hs37fvf/clone/recipe/meta.yaml:31: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_8b_hs37fvf/clone/recipe/meta.yaml:39: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_8b_hs37fvf/clone/recipe/meta.yaml:39: http_url: http://github.com/capless/python-jose-cryptodome is not https
WARNING: /tmp/abs_8b_hs37fvf/clone/recipe/meta.yaml:39: missing_description: The recipe is missing a description
Errors were found


html video play controls from tusd url

Question
Using tusd urls as a source for html video?
It works for mp4 files by adding the type attribute:
<video preload="auto"
                controls
                src={tusdVideoFileURL}
                type='video/mp4'></video>

but the timeline control doesn't work, could this be missing metadata in the file that is on the .info?
I would like to make the timeline controls work if anyone has succeeded at this advice would be much appreciated, will keep trying on my end and keep this updated
Setup details
Please provide following details, if applicable to your situation:

Operating System: Windows
Used tusd version: v1.4.0
Used tusd data storage: disk store
Used tusd configuration: none
Used tus client library: tus-js-client

tusd supports GET requests but does not support the Range header in GET requests which is used by video players to seek to certain time stamps. We currently have no intention of implementing support for this feature since tusd is not meant to be used as a file server. Instead, it is best to use a proper tool like nginx for this.

tusd supports GET requests but does not support the Range header in GET requests which is used by video players to seek to certain time stamps. We currently have no intention of implementing support for this feature since tusd is not meant to be used as a file server. Instead, it is best to use a proper tool like nginx for this.

how can I make nginx serve the video files with Range header support? Will look into it, thanks @Acconut

how can I make nginx serve the video files with Range header support? Will look into it, thanks @Acconut

I think nginx supports it by default and there is no additional option required to enable the Range header, as far as I know.

I think nginx supports it by default and there is no additional option required to enable the Range header, as far as I know.


p2p: eventually evict disconnected peer from address book

Description

during longevity testing, p2p would emit error logs for nodes that we replaced during tests. an error log is printed for each disconnected peer every X seconds where X ranges from 10-90 seconds.
and this is for every peer the node lost connection ever. this produced ~500 error logs every 10 min and lasted till the end of the test.
it would appear that disconnected peers are not removed from the address book.
log snippet
2022-08-01 12:40:05
{"L":"ERROR","T":"2022-08-01T15:40:05.955-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWHfVBhsnXsCEXEA8CjU1Pn8vWVDbppiKzuxRMem1s3cdu","errmsg":"no good addresses","name":"p2p"}
2022-08-01 12:40:05
{"L":"ERROR","T":"2022-08-01T15:40:05.957-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWSofbFHnoyNkmgVMTQ5bjmuNsEuaHwesFU3q2D8MqvUGp","errmsg":"failed to dial 12D3KooWSofbFHnoyNkmgVMTQ5bjmuNsEuaHwesFU3q2D8MqvUGp:\n  * [/ip4/<IP_ADDRESS>/tcp/7513] dial backoff","name":"p2p"}
2022-08-01 12:40:05
{"L":"ERROR","T":"2022-08-01T15:40:05.957-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWMK8uyp8dJjXNn8mpEN3fvjoLwQSJtmv3c7xpfHfm1qWa","errmsg":"no good addresses","name":"p2p"}
2022-08-01 12:40:05
{"L":"ERROR","T":"2022-08-01T15:40:05.959-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWBtKW2hD2T2MdviXJ5qWqaX4ZbnVfLcbrJrZoF2Twm6cL","errmsg":"failed to dial 12D3KooWBtKW2hD2T2MdviXJ5qWqaX4ZbnVfLcbrJrZoF2Twm6cL:\n  * [/ip4/<IP_ADDRESS>/tcp/7513] dial backoff","name":"p2p"}
2022-08-01 12:40:05
{"L":"ERROR","T":"2022-08-01T15:40:05.960-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWGXxqnC1kxzk6LSXo8KkWkM2bUUwFMETcHQBbavvqj6AY","errmsg":"failed to dial 12D3KooWGXxqnC1kxzk6LSXo8KkWkM2bUUwFMETcHQBbavvqj6AY:\n  * [/ip4/<IP_ADDRESS>/tcp/7513] dial backoff","name":"p2p"}
2022-08-01 12:40:05
{"L":"ERROR","T":"2022-08-01T15:40:05.960-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWCdmeu4GwP36L9dgx15rADhANX6uR9ek8TemKXFLXY9Dw","errmsg":"failed to dial 12D3KooWCdmeu4GwP36L9dgx15rADhANX6uR9ek8TemKXFLXY9Dw:\n  * [/ip4/<IP_ADDRESS>/tcp/7513] dial backoff","name":"p2p"}
2022-08-01 12:40:05
{"L":"ERROR","T":"2022-08-01T15:40:05.960-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWCThvyPGHe5mUTPWanCNRCX12ieRZiEPKF6uLYgZJYS3t","errmsg":"failed to dial 12D3KooWCThvyPGHe5mUTPWanCNRCX12ieRZiEPKF6uLYgZJYS3t:\n  * [/ip4/<IP_ADDRESS>/tcp/7513] dial backoff","name":"p2p"}
2022-08-01 12:40:05
{"L":"ERROR","T":"2022-08-01T15:40:05.965-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWEXqEv6UoU2edY6gKz7ugpSy5SKXohaCeGC1FXkAXjSKW","errmsg":"failed to dial 12D3KooWEXqEv6UoU2edY6gKz7ugpSy5SKXohaCeGC1FXkAXjSKW:\n  * [/ip4/<IP_ADDRESS>/tcp/7513] dial backoff","name":"p2p"}
2022-08-01 12:40:10
{"L":"ERROR","T":"2022-08-01T15:40:10.966-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWGMaAHMoVB6NjymmZnT5Sny1AF5dyqd1TYMrX8tLYZhzk","errmsg":"failed to dial 12D3KooWGMaAHMoVB6NjymmZnT5Sny1AF5dyqd1TYMrX8tLYZhzk:\n  * [/ip4/<IP_ADDRESS>/tcp/7513] dial tcp4 <IP_ADDRESS>:7513-><IP_ADDRESS>:7513: i/o timeout","name":"p2p"}
2022-08-01 12:40:10
{"L":"ERROR","T":"2022-08-01T15:40:10.967-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWPXBnzFxVtRh1PfQTiisw8t7teGoTkq9Nogf9qi8Vb9F2","errmsg":"failed to dial 12D3KooWPXBnzFxVtRh1PfQTiisw8t7teGoTkq9Nogf9qi8Vb9F2:\n  * [/ip4/<IP_ADDRESS>/tcp/7513] dial backoff","name":"p2p"}
2022-08-01 12:40:15
{"L":"ERROR","T":"2022-08-01T15:40:15.957-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWHfVBhsnXsCEXEA8CjU1Pn8vWVDbppiKzuxRMem1s3cdu","errmsg":"no good addresses","name":"p2p"}
2022-08-01 12:40:15
{"L":"ERROR","T":"2022-08-01T15:40:15.957-0400","N":"b4141.p2p          ","M":"failed to connect to peer","node_id":"b414165507ca0823b2fd71c9bad03a15732b8c1b9db47c0a93a40f020bc53ed0","module":"p2p","peer":"12D3KooWMK8uyp8dJjXNn8mpEN3fvjoLwQSJtmv3c7xpfHfm1qWa","errmsg":"no good addresses","name":"p2p"}

screenshot at the beginning of the TestStepReplaceNodes.

screenshot at the end of the test.

works items:

instrument address book to track how many addresses were removed (total addresses gauge, remove addresses counter, maybe a hist to track how long it takes to remove a peer after it failed connection the first time)
based on the above metrics evaluate the efficiency of the current eviction logic using basic longevity test
come up with a better heuristic for evicting peers that are in unknown state (e.g if a peer gave already dead addresses)

Summary of my observations so far:
When the bookchecker queries for peers that are unavailable for X period of time, it checks the LastSeen property of a knownAddress.  This property is periodically updated by the crawler in crawler::query() when it receives a list of peers known to the other node.
When I changed the bookchecker to query unavailable peers by looking at the LastSuccess property instead, it started to evict stale peers properly. However, they are soon added back to the address book by the aforementioned logic of the crawler.
I need to think about how to avoid re-adding just evicted peers back to the address book. @dshulyak suggested that nodes should not broadcast peer addresses that are stale for some time (LastSuccess older than X time).


[DCOS_OSS-1003] Admin Router auth module: require a literal 'false' for disabling the module

High Level Description
The DC/OS oauth_enabled configuration parameter value determines whether Admin Router's authentication module is activated or not. This patch ensures that the auth module is deactivated only if a literal 'false' (YAML string type) is provided and that it is enabled in all other cases.
As of now, the auth module is enabled only if a literal 'true' (YAML string type) is provided and it is disabled in all other cases. This implies an unnecessarily large risk for unintentionally deactivating the authentication module and therefore is a security risk.
Kudos to @lordnynex for the report via Slack and via DCOS_OSS-1003.
Related Issues

DCOS_OSS-1003 oauth_enabled cfg param: only the literal 'false' should disable AR's auth module

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: this is quite a bit of work, tracked via DCOS_OSS-1005
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

A note for any users finding this PR in the future, if you want to patch the issue on a running cluster without an upgrade you can run the following on all masters in your DCOS cluster.
Change config value
$ echo 'ADMINROUTER_ACTIVATE_AUTH_MODULE=true' | sudo tee /opt/mesosphere/etc/adminrouter.env

Restart Adminrouter
$ sudo systemctl restart dcos-adminrouter.service

@jgehrcke is this gonna require docs changes?

Re-running aws-deploy:

An error occurred (RequestLimitExceeded) when calling the DeleteKeyPair operation (reached max retries: 4): Request limit exceeded.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

The downstream tests passed (manually verified that one test failure was transient). I am not really sure about the severity of the test failures in integration-test/deploy-vpc-cli, but they are unrelated.

Enterprise PR failure is only in permissive mode, and it failed after all the tests passed:
267 passed, 33 skipped, 6 deselected, 31 xfailed, 4 xpassed in 2976.95 seconds


Workspace avatar reverts to default after reopening the app

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

Action Performed:

Open iOS/Android/Desktop App or go to staging.new.expensify.com
Go to existing or create new workspace
Upload workspace avatar and tap 'Save'
Close and re-open app or reload web page

Expected Result:
Avatar saved and showing as workspace avatar
Actual Result:
Workspace avatar is not saved and reverts to default after reopening the app
Workaround:
Unknown
Platforms:

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/7b5ceb06-2255-436f-a712-384642141e91
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014fa14191ff031aab
Upwork Job ID:<PHONE_NUMBER>956208128
Last Price Increase: 2023-07-03

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Didn't have time to investigate further, but I'm removing the deploy blocker label as this is reproducible on prod.
https://github.com/Expensify/App/assets/10736861/09a1abe9-0c60-4f5c-a90d-9aacb7e47eb3
https://github.com/Expensify/App/assets/10736861/9d4de49c-7016-46bb-876b-16446ed4f3a5

Ok, I found the root cause and it's in the backend.
When we are returning the policySummaryList it has the key of avatar and not avatarURL. Easy fix.

@0xmiroslav Just a heads up that I don't think we need any help testing here as there won't be any NewDot PR

We can close this now as solution been deployed to prod.


Revise API reference for v1.29

This PR also includes a version bump of the bootstrap css files.

/retitle Revise API reference for v1.29

/area web-development

/lgtm
Thank you.
Should we use the same CSS file for the older API references too?

/lgtm
Thank you. Should we use the same CSS file for the older API references too?

I'll go through the reference to other versions and clear them when possible.
I'm not inclined to rebuild the reference for older versions and push the new js/css there. It will work but we gain nothing from it.

It would be great if this PR is approved quickly as there are more issues being raised for the broken links
#44385
#44387

This PR is ready for approval.

/lgtm

/approve


Fix Image Path

Please accept this image path update, thanks.
Fix PR:
https://github.com/solana-labs/token-list/pull/23819/files
OG PR:
https://github.com/solana-labs/token-list/pull/23802

Hello?


Add vim documentation

They're missing. :help autoformat returns built-in docs.

I don't think we should overwrite that vim-docs entry, as it still provides valuable information. Vim-autoformat does not obsolete the builtin functionality, it kind of extends it.
That being said, I wouldn't hurt to have our own vim-docs entry. We could use https://github.com/xolox/vim-tools to export our README.md to vim-docs format. I would be open to a PR implementing this. The README should get a paragraph with instructions about this though, since with every contribution to the README, the vim-docs would have to be updated as well.


Techniques for browser's back button closing modal window using History API?

Hello, I have written a SPA which utilises the History API for all its navigation. This adds each page to the history stack when you navigate to it.
My modal windows do not currently use the History API. They only popup more details of the card you've clicked on via an AJAX call. I have set up no modal-specific 'hashbang' system or fragment identifier and there is no URL you could enter which would take you to that page with the modal open. So at the moment my modal windows work independently from the usual page generator.
Because of this, if a modal is open and the user hits the browser's back button it will take the user back to the previous page. What I would like ideally is for the back button to just close the modal window.
I can see two solutions but each has issues.

Simplest. Detect back button click. If modal open then close it and ignore browser's attempt to go to previous URL. The problem with this is that I don't think it's possible to stop the browser doing this.

Add hash to url when window opens, and add to History API. If you close the modal normally then add the page's URL to the History API, else if you clicked the Back button it would just reload the background page, without the modal. The problem with this is that the page needs to reload.

So I was wondering if there were any strategies you have used to somehow get the browser's back button to mimic a standard modal close? Is there an easy solution I just haven't though of?
Thanks in advance.
John..

Colorbox has a close method that you can bind to the popstate event (back navigation). I would use Colorbox's onOpen/onCleanup options or the global event hooks to do setup and teardown of subscribing to popstate:
$(document).bind('cbox_open', function(){
  window.addEventListener('popstate', $.fn.colorbox.close);
});

$(document).bind('cbox_cleanup', function(){
  window.removeEventListener('popstate', $.fn.colorbox.close);
});

Thanks, Jack. I never thought of that way. Much appreciated.


Default to opening story in browser if there is no article

I'm used to hitting a particular keyboard shortcut to open in my browser the article associated with a story.  But some stories, e.g. "Ask HN" don't have an associated article.  In that case, I would prefer for the story page to be open in the browser.  That way, I don't have to switch mental model (and keyboard shortcut) whenever I get to an "internal" story.
In other words, "Open in browser the article associated with the focused story" should default to "Open in browser the focused story" if there is no article associated with the story

Hi @huyz. Sorry for the delay, with the new version 0.10.1 released, this behaviour should be implemented with "open article in browser" command.
Please note that there are some changes in the list of default shortcuts in the new version, which may break your current config file.

Thanks @aome510


Runscript error

I am seeing this error when running rswift in Xcode 9.1.  I am using
"$PODS_ROOT/R.swift/rswift" generate "$SRCROOT"
to run rswift.   The unknown argument is my $SRCROOT directory
Unknown Arguments: /Users/me/mySrcRoot

Arguments:

    outputDir - Output directory for the 'R.generated.swift' file.

Options:
    --version [default: false] - Prints version information about this release.
    --edge [default: false] - Enable stable features that will be in the next major release.
    --import [default: ] - Add extra modules as import in the generated file, comma seperated.
    --accessLevel - The access level [public|internal] to use for the generated R-file.
    --rswiftignore [default: .rswiftignore] - Path to pattern file that describes files that should be ignored.
    --xcodeproj [default: PROJECT_FILE_PATH] - Path to the xcodeproj file.
    --target [default: TARGET_NAME] - Target the R-file should be generated for.
    --bundleIdentifier [default: PRODUCT_BUNDLE_IDENTIFIER] - Bundle identifier the R-file is be generated for.
    --productModuleName [default: PRODUCT_MODULE_NAME] - Product module name the R-file is generated for.
    --buildProductsDir [default: BUILT_PRODUCTS_DIR] - Build products folder that Xcode uses during build.
    --developerDir [default: DEVELOPER_DIR] - Developer folder that Xcode uses during build.
    --sourceRoot [default: SOURCE_ROOT] - Source root folder that Xcode uses during build.
    --sdkRoot [default: SDKROOT] - SDK root folder that Xcode uses during build.

Are you using the latest version of R.swift?
Note that the command line arguments have changed between R.swift 3.x and 4.0

Closing due to inactivity. Feel free to reply with any further comments.


Signed Decimals

Requires https://github.com/CosmWasm/cosmwasm/issues/1710

Out of curiosity has there been any progress on this enhancement since I posted there in January?

Development wise, nope. But I got confirmation from at least 2 collegues now that we don't want signed integers that are based on an Positive/Negative enum or a struct with a signed field as many users suggested before. Instead we'll go for traditional integers based on two's complement (#1710) that do not use more than 128bit/256bit/512bit in memory. Once we have Int128 and Int256 we can make a signed Decimal and Decimal256.


The first run will report an error, prompt httplive.db file does not exist

# ./httplive_linux_amd64
open /home/travis/gopath/src/github.com/gencebay/httplive/httplive.db: no such file or directory

#HotFix
Db creation strategy is updated.  I hope this will solve your problem.
Latest release link https://github.com/gencebay/httplive/releases/tag/20201202124733-201502a


Moved: Layer.LSTM not performing as well as Architect.LSTM

This bug has been moved from Neataptic.
https://github.com/wagenaartje/neataptic/issues/25

Currently I'm rewriting all the built-in networks code to construct networks form layers instead of groups and nodes. However, using groups/nodes to construct a LSTM network seems to always outperform using LSTM layers.
Simple one-layer LSTM network with layers
function LSTM(inputSize, hiddenSize, outputSize){
  var input = new Layer.Dense(inputSize);
  var hidden = new Layer.LSTM(hiddenSize);
  var output = new Layer.Dense(outputSize);

  input.connect(hidden, Methods.Connection.ALL_TO_ALL);
  hidden.connect(output, Methods.Connection.ALL_TO_ALL);

  // option.inputToOutput is set to true for Architect.LSTM
  if(true)
    input.connect(output, Methods.Connection.ALL_TO_ALL);

  return Architect.Construct([input, hidden, output]);
}

So:
var network = new LSTM(1, 6, 1);

// is the equivalent of

var network = new Architect.LSTM(1,6,1);

However, there is one key difference: Layer.LSTM consists of an output block itself. So when you connect Layer.LSTM to an output layer, there will be a group of size in between (this is needed if you connect two LSTM layers with each other). Architect.LSTM also does this, except for the output layer:
var outputBlock = i == blocks.length-1 ? outputLayer : new Group(block);

However, there is no way a Layer.LSTM can know that the next layer is a regular dense layer - if it would know, then no outputBlock would be needed.
Visualisation

Left: layer LSTM - Right: architect LSTM
You can clearly see the lack of gates/extra nodes in the second image. All these extra nodes and gates require a lot of extra iterations to converge to a solution.
Current solution idea
Make the Layer.Dense.input() function detect if the connecting layer is a LSTM. If it is, remove its outputBlock. This requires a workaround of the connect() function though.
This issue will not be fixed soon.

@luiscarbonell and I collaborated on a partial fix in eb465b3b5381915235bb49a812ddd06e8f34f38f

Good work. I've added some small comments.


Add Backing File format

Adds specification of backing file format header extension
closes #9

Hi @dypflying ,
I have made the necessary changes, Do let me know if anything else needs to be changed.
Thank you.

Hi @dypflying,
Just wanted to enquire if any changes are needed ?

this looks good to me


Use demo with finetuned checkpoint

Is it possible to use a finetuned checkpoint in the demo, specifically for ActivityNet-QA?
Right now the API for finetuned models is kinda locked away in the train and eval scripts, which makes it hard to play with. I'm impressed by the zero-shot capabilities exhibited in the demo and would like to interact with a finetuned model to see how well it understands the time position embeddings (questions like: what happened in the last N frames or T seconds).
Can I adapt the Blip2VicunaInstruct_MALMM  class to accept one of the finetuned checkpoint in saved_models.tar from the README?

Hello, I have updated the demo.ipynb.
So the model loads the default config from lavis/configs/models/blip2/blip2_instruct_vicuna7b.yaml.
If you want to load a finetuned checkpoints, you need to first set the load_finetuned=True and specify the finetuned checkpoint path in the yaml config and reload the model again.


Adding Hash partitioner

I checked that code segment. It's used for reading a graph from a file since in Mahen's Laptop Kafka queue is not working. So I added my code from line 582-612 in AcaciaFrontendServiceSession.x10


Create more functions

Codecov Report

Merging #2 (00a965f) into main (1413167) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #2   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           80        95   +15
=========================================
+ Hits            80        95   +15

Impacted Files
Coverage Δ

sliceutils.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


omnisharp-roslyn: 1.39.7 -> 1.39.8

Description of changes
Update omnisharp-roslyn to its latest version.
CHANGELOG: https://github.com/OmniSharp/omnisharp-roslyn/releases/tag/v1.39.8
This releases bundles an important fix of the LSP.
cc @tesq0 @ericdallo @corngood @mdarocha
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 244119 run on x86_64-linux 1

1 package built:

omnisharp-roslyn

Result of nixpkgs-review pr 244119 run on aarch64-darwin 1

1 package built:

omnisharp-roslyn

@mdarocha can we tag this to be backported to 23.05 or will the bot not trigger now?
Pkg is broken in 23.05 and needs this fix too.


Checkpoints for other parts

I noticed that when trying to perform try-on with lower garments, such as pants or skirts, the outputs are not satisfactory. From what I understand, the mask predictor in the demo code seems to be set as "upper," which likely means the model is trained specifically for upper garments. Are there plans to share checkpoints for other garment types in the future?

Thank you for your interest in our project!
I have just released the model trained on DressCode, https://huggingface.co/franciszzj/Leffa/blob/main/virtual_tryon_dc.pth, which offers better performance for lower body and similar cases. This is because the previous virtual try-on model was trained only on the VITON-HD dataset and had never encountered any lower body datasets.

@joekendal The DC model is trained with upper body, lower body, and dresses together.
However, the current mask generator is implemented using the CatVTON approach, which slightly differs from the mask used during internal model training. This discrepancy results in suboptimal performance.
This means that if you can generate masks suitable for dresses, it will be ok to achieve dress try-on.

If you have any recommendations for mask generators, feel free to suggest them. I will integrate them into the demo.

I also tried dresses and found it not work. I will adapt another mask generator to improve the performance.


Fix encoding issue in SpringEncoder for binary data

Fixes gh-1698

I will try. Give me more time.

@tjuchniewicz NP!

@spencergibb Test added. Some test fail on CI but I believe this is not my fault. All tests pass on my machine.

Codecov Report

Merging #1699 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1699      +/-   ##
==========================================
+ Coverage   75.12%   75.12%   +<.01%
==========================================
  Files         198      198
  Lines        6179     6180       +1
  Branches      774      775       +1
==========================================
+ Hits         4642     4643       +1
  Misses       1198     1198
  Partials      339      339

Impacted Files
Coverage Δ

...ork/cloud/netflix/feign/support/SpringEncoder.java
65.11% <100%> (+0.83%)
:white_check_mark:

Continue to review full report at Codecov.

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


Change checkbox default state in A class registration to unchecked

As a TSC staff member I want the checkboxes to NOT be checked by default So That seedlots do not erroneously have the TSC listed as an extractory agency when, in reality, the user just hasn't done the data entry yet
Additional Context

Currently, the default behavior for a few checkboxes in the A Class registration flow is set to "TRUE" / checked. We'd like to change the default behavior of the checkboxes to unchecked.

The checkboxes can be found on these steps of the registration flow:
Step 1:

[x] Use applicant agency as collector agency

Step 2:

[x] Use applicant agency as owner agency

Step 3:

[x] Use applicant collector agency as interim storage agency

Step 6:

[x] The extractory agency is the Tree Seed Centre (TSC)
[ ] The seed storage agency is the Tree Seed Centre (TSC)

One more item - Collection information:
Step 1 has data prefilled in the "Collection Information" which is causing data integrity issues for the TSC:
Number of containers
Volume per Containers
Volume of cones

[x] These fields currently have a default value of 1. Please change from 1 to NULL (no default value)
[x] Change "Volume of cones (HL)" to instead read: Volume of cones (hl)
[x] Change "Volume per Containers (HL)" to instead read: Volume per container (hl)

Acceptance Criteria

The checkboxes listed above default to unchecked as a user works through the seedlot data entry workflow

Definition of Done

[ ] Ready to Demo in Sprint Review
[ ] Does what I have made have appropriate test coverage?
[ ] Documentation and/or scientific documentation exists and can be found
[ ] Peer Reviewed by 2 people on the team
[ ] Manual testing of all PRs in Dev and Prod
[ ] Merged

Marking as done


Add support for audio recording

See if audio recording can be supported through advanced. If not, look into implementing parameters for it.

macOS - https://trac.ffmpeg.org/wiki/Capture/Desktop
Linux - https://trac.ffmpeg.org/wiki/Capture/ALSA
Windows - https://medium.com/@colten_jackson/simple-free-screen-capture-with-ffmpeg-3aeddbeb161b

Hello! Thanks for the gem!
Is it possible to record video with audio now? I added settings to 'advanced' but got an error 'Unable to find a suitable output format for ***'. If its possible, I dont undestand where and what settings need to be set to do this. I use linux, if its important.

Hi @sh3p42d, can you share the advanced configs (code for it) that you have tried? There is an example showing the audio configs on Linux over here: https://trac.ffmpeg.org/wiki/Capture/Desktop

If you need audio too, you can use ALSA (see Capture/ALSA for more info):
ffmpeg -video_size 1024x768 -framerate 25 -f x11grab -i :0.0+100,200 -f alsa -ac 2 -i hw:0 output.mkv

I'll look into updating the gem with this feature very soon.

Thank you for efficiency.
I want to run this command "ffmpeg -video_size 1024x768 -framerate 25 -f x11grab -i :0.0+100,200 -f pulse -ac 2 -i default output.mp4"
This is my configs:
advanced = {
    input: {
        video_size: '1024x768',
        framerate: '25',
        ac: '2'
    }
}
@recorder = ScreenRecorder::Desktop.new(input: ':0.0+100,200 pulse default', output: 'output.mp4', advanced: advanced)

How can i set the advanced configs for '-f pulse -ac 2 -i default'? Every time I got 'Unable to find a suitable output format for'.

@sh3p42d I just pushed some code which will allow you to record audio with the following configs:
advanced = { 'f': 'pulse', 'ac': 2, 'i': 'default' } # Uses default sound output device
@recorder = ScreenRecorder::Desktop.new(output: 'output.mp4', advanced: advanced)

You can test it in your current project by adding this to your Gemfile:
gem 'screen-recorder', git: 'https://github.com/kapoorlakshya/screen-recorder.git', ref: 'correctly_parse_advanced_settings'

And you don't need to provide an input value if you're not using multiple screens. The default is set to :0 which should work on most systems. You can check your input number by executing echo $DISPLAY in the terminal.
I'll implement a better solution to record audio when I have more free time. Let me know if you run into any problems.

I have pushed some more code to properly support and document the configs for capturing audio: https://github.com/kapoorlakshya/screen-recorder/compare/15_capture_audio
Will merge it after testing it on Windows and macOS over the weekend.


add quotation around ports in docker-compose.yml

Without quotation around ports in docker-compose.yml, an error will occured when docker-compose up:
Creating network "test_gitea" with the default driver
Creating volume "test_gitea" with local driver
Creating test_server_1 ...
Creating test_server_1 ... error

ERROR: for test_server_1  Cannot create container for service server: b'invalid port specification: "601342"'

ERROR: for server  Cannot create container for service server: b'invalid port specification: "601342"'
ERROR: Encountered errors while bringing up the project.

According to yaml's specification, : is a special mark, which denotes map, so quotation is necessary to avoid ambiguity.

Codecov Report

Merging #3090 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3090      +/-   ##
==========================================
- Coverage   33.26%   33.25%   -0.01%
==========================================
  Files         273      273
  Lines       39954    39954
==========================================
- Hits        13290    13288       -2
+ Misses      24768    24767       -1
- Partials     1896     1899       +3

Impacted Files
Coverage Δ

models/repo_indexer.go
45.54% <0%> (-3.47%)
:arrow_down:

models/repo.go
38.15% <0%> (-0.19%)
:arrow_down:

models/issue.go
45.05% <0%> (+0.19%)
:arrow_up:

models/error.go
32.72% <0%> (+0.61%)
:arrow_up:

models/repo_list.go
67.18% <0%> (+1.56%)
:arrow_up:

models/issue_indexer.go
67.53% <0%> (+2.59%)
:arrow_up:

Continue to review full report at Codecov.

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

@bibaijin any rule reference link from docker?

Here is the example from docker-compose, although not described explicitly, all of the example port items is enclosed by quotation. And https://github.com/docker/compose/issues/3109 describe this same problem, the solution is add quotation.

LGTM

LGTM


added namespace

Description
added description to change namespace in lens
Related issues
#76
Type of change
What types of changes does your code introduce to this project?

[ x] Bug fix (non-breaking change which fixes an issue)

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

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

@JeremyTheocharis Can you check the CLA error here ?


license - lgpl instead of gpl ?

Hi there,
Can you relicense this great work under LPGLv3 so it can actually be used by Wow and Wow addons ? (unless somehow I misinterpret the GPL but it seems pretty clear I am afraid?)
Thanks!

Though I do not think GPL will affect WoW addon to use LibDeflate, I can change the license to LGPLv3, to make it used by more people
The main reason that LibDeflate was licensed under GPLv3, because LibCompress (https://www.wowace.com/projects/libcompress) is also GPLv2, and I do not think this affects the WoW addon development.
The official WoW addon policy (https://us.forums.blizzard.com/en/wow/t/ui-add-on-development-policy/24534) requires

Add-on code must be completely visible.
The programming code of an add-on must in no way be hidden or obfuscated, and must be freely accessible to and viewable by the general public.

This means that the Lua code running inside World of Warcraft must be open source, although you could obfuscate the source code to make it hard to read. You could make it close-source for helper programs not written in Lua that running outside World of Warcraft, similar a database auto updater, which is not affected by GPLv3 even if you use any Lua GPL library. In a word, WoW addon policy requires addon Lua code to be "GPL" anyway, so GPL is not a problem for you to use GPL library.
I did use some GPL code in the testing part though, I will license testing code in GPL, but LibDeflate.lua itself in LGPLv3
P.S.: I am not playing WoW for more than a year, sorry for very late reply. Trying to remember how to how to do lua packaging and Wow packaging stuffs.

Will update the license and the version number in several days after setting up my Lua development environment and after downloading a WoW client for further testing

@mooreatv  @Stanzilla   Version 1.0.1 has been released. I have replaced every mentions of GPL of LibDeflate to LGPL, including the source code, README, documentation, Luarocks page, Curseforge page. Report if there is still any GPL leftovers.


troubleshooting: BACKUP too large

Finalize: https://cockroachlabs.atlassian.net/wiki/spaces/TS/pages/1171947732/DRAFT+BACKUP+too+large

This guidance is finalized.

This guidance is finalized.


dynamic kfold

thanks for this @emattia !
i'm not certain we want this in the same dir/level as the lesson NBs.
what about creating another dir in /notebooks for us to put stuff like this?


Add secret key to personal hub edit form

We should add some notes:

The secret key is used to sign notebooks have been written by you and it should not be shared.
If you change the secret, previous notebooks written by you that used secrets will need to grant access to secrets again.
We should probably add a "Generate random".

There are some ways we can tackle this. One option is to add at the end of the "Sessions" bit something like:

Once you grant access to a secret in a notebook, the secret name is encrypted as part of the notebook with your private key.

Private key is a link that opens up a modal with the information above and the field to change it.
WDYT?

There are some ways we can tackle this. One option is to add at the end of the "Sessions" bit something like:

You mean the secrets panel in sidebar? The private key is only relevant for personal hub, so otherwise we wouldn't mention the private key?

Sorry, I mean here:

Ah that makes more sense. We could also have a third section like "Stamping" with the notes and private key? Especially if we use it for other purposes than secrets in the future.

It is up to you. :)

@josevalim


Docs.MS Release Updates for Azure.Messaging.EventHubs

Docs.MS Release Updates for Azure.Messaging.EventHubs

Docs Build status updates of commit 3f6a539:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/overview/azure/messaging.eventhubs-readme.md
:white_check_mark:Succeeded
View

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


[ci] Don't emit warnings for absence of failure logs

Describe the pull request

What does your PR fix?
This PR removes the pointless warnings which appear on GH ("0 errors / 1 warnings") and AZP ("Directory 'C:\a\1\a\failure-logs' is empty. Nothing will be added to build artifact 'failure logs for x86-windows'.") for build jobs without failure logs.

Which triplets are supported/not supported? Have you updated the CI baseline?
--

Does your PR follow the maintainer guide?

--

This is awesome! I was wondering how to do this.

Got an in-person review from @ras0219 / @ras0219-msft

I chopped off the "proof of concept failure" commit and will merge this when we have a clean build back.

Thanks again for the awesome!


Cannot Capture properly take a screen capture of ax::ui::ImageView

Hi,
We are trying to perform a screen capture an ax::ui::ImageView. Unfortunately so far we are unsuccessful.  The ImageView is part of an axmol::ui::ScrollView.
After analyzing the saved file, we noticed that actually the saved file has the correct dimensions than the original image (1320x1412).
width   = 1320
height  = 1412
The issue is that with the saved image the graphic in the saved file doesn't occupy the full size of the available png file. The graphic size in the png file is about 1059x563
The graphic should have been 1320 x1412. It should fully occupy the image size.
We are not 100% sure if this is an Axmol bug or our code is not correct. Another possibility would be that this is just not possible do  right now with the Axmol API that we have.
Any help will be appreciated. This is our code.
auto pImage = ax::ui::ImageView::create(“textImage.png");

if(pImage != nullptr)
{
        auto contSize = parentLayer->getContentSize();
        parentLayer->setPosition(Vec2(contSize.width, contSize.high));
        parentLayer->addChild(pSentRequestedFbFieldBackGroundImage);

	std::string strFilename(“myCaptureTextImage.png”);

        scheduleOnce([=](float)
        {
            // Get the size of the ImageView
            Size imageViewSize = pImage->getContentSize();

            AXLOG("ImageView Size - Width: %f, Height: %f", imageViewSize.width, imageViewSize.height);

            // Create a RenderTexture with the same size as the ImageView
            RenderTexture* renderTexture = RenderTexture::create(imageViewSize.width, imageViewSize.height, backend::PixelFormat::RGBA8, backend::PixelFormat::D24S8);
            renderTexture->retain();

            // Check the RenderTexture size after creation
            Size renderTextureSize = renderTexture->getContentSize();
            AXLOG("RenderTexture Size - Width: %f, Height: %f", renderTextureSize.width, renderTextureSize.height);

//            // Ensure the RenderTexture is properly initialized
//            if (renderTextureSize.width == 0 || renderTextureSize.height == 0) {
//                AXLOG("Error: RenderTexture not properly initialized.");
//                return;
//            }

            // Calculate the scale factor to fit the ImageView within the RenderTexture while preserving aspect ratio
            float scaleX = renderTextureSize.width / imageViewSize.width;
            float scaleY = renderTextureSize.height / imageViewSize.height;
            float scale = std::min(scaleX, scaleY);

            // Calculate the position to center the ImageView in the RenderTexture
            float posX = (renderTextureSize.width - imageViewSize.width * scale) / 2;
            float posY = (renderTextureSize.height - imageViewSize.height * scale) / 2;

            // Save the original position, anchor point, and scale
            Vec2 originalPosition = pImage->getPosition();
            Vec2 originalAnchorPoint = pImage->getAnchorPoint();
            float originalScaleX = pImage->getScaleX();
            float originalScaleY = pImage->getScaleY();

            // Set the position, anchor point, and scale to fit within the RenderTexture
            pImage->setAnchorPoint(Vec2::ANCHOR_BOTTOM_LEFT);
            pImage->setPosition(Vec2(posX, posY));
            pImage->setScale(scale);

            // Begin rendering to the RenderTexture
            renderTexture->begin();

            // Get the global renderer
            auto renderer = Director::getInstance()->getRenderer();

            // Visit the ImageView to render it to the RenderTexture
            pImage->visit(renderer, Mat4::IDENTITY, false);

            // End rendering to the RenderTexture
            renderTexture->end();

            // Reset the position, anchor point, and scale of the ImageView
            pImage->setPosition(originalPosition);
            pImage->setAnchorPoint(originalAnchorPoint);
            pImage->setScale(originalScaleX, originalScaleY);

            // Save the RenderTexture to an image file
            bool success = renderTexture->saveToFile(filePath, Image::Format::PNG);
            if (success) {
                AXLOG("Screenshot saved to: %s", filePath.c_str());
            } else {
                AXLOG("Failed to save screenshot to: %s", filePath.c_str());
            }

            // Release the RenderTexture
            renderTexture->release();

        }, 0.f, "capture_sv");
 }

Please see one example of the png image output during our attempts.
As you can see, the graphic size is smaller than what it should be and the proportions are not kept.

Thank you for your help,
Stay safe
axmol version: 2.1.4
devices test on: iPhone 11, 12, 14
developing environments
NDK version: n/a
Xcode version: 15.4
Visual Studio: n/a
VS version: n/a
MSVC version: n/a
Windows SDK version: n/a
cmake version: 3.29.0
Steps to Reproduce:

create an ax::ui::ImageView object
Try to capture it
Check the saved image

@asnagni It's always best to provide a minimal test case with the relevant test graphics etc., to prevent wasting time on trying to reproduce what you're seeing.

@rh101 good point, no one should be wasting their time helping 😊. Please see attachment, you have the test case (the code) reproducing exactly the problem. Please let me know if you have any questions or if the previous description is not clear. I removed the iOS build folder to minimize the size of the zip file beside that everything is in the zip file.
HelloCpp.zip
Thank you for helping.

The code is missing the call to Renderer::render(), so see if this fixes the issue by adding this line just before or after the call to saveToFile:
renderer->render();
...
            // Reset the position, anchor point, and scale of the ImageView
            pImage->setPosition(originalPosition);
            pImage->setAnchorPoint(originalAnchorPoint);
            pImage->setScale(originalScaleX, originalScaleY);

            // Save the RenderTexture to an image file
            bool success = renderTexture->saveToFile(filePath, Image::Format::PNG);
            renderer->render();  // <<== add this line

            // the following code is pointless, since `RenderTexture::saveToFile` always returns true
            //if (success) {
            //    AXLOG("Screenshot saved to: %s", filePath.c_str());
            //} else {
            //    AXLOG("Failed to save screenshot to: %s", filePath.c_str());
            //}
...

The result is this:

@rh101 thank you spotting the missing line, I do appreciate. By adding that line ( to render the content) it fixes the issue. Good catch.
Stay safe


Has anyone been able to get MXNet to work with Spark on Mesos?

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System:
Compiler:
Package used (Python/R/Scala/Julia):
MXNet version:
Or if installed from source:
MXNet commit hash (git rev-parse HEAD):
If you are using python package, please provide
Python version and distribution:
If you are using R package, please provide
R sessionInfo():
Error Message:
Please paste the full error message, including stack trace.
Minimum reproducible example
if you are using your own code, please provide a short script that reproduces the error.
Steps to reproduce
or if you are running standard examples, please provide the commands you have run that lead to the error.

What have you tried to solve it?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!

@apache/mxnet-committers: This issue has been inactive for the past 90 days. It has no label and needs triage.
For general "how-to" questions, our user forum (and Chinese version) is a good place to get help.

Suggested Labels :"HowTo", "Question"


Adds support for AsyncStream

This PR:

Adds AsyncStream support to FileMonitor for modern Swift Structured Concurrency support
Bumps the Package build requirement to 5.9, breaking backward compatibility

Benefits:
Users can change callback based system with for await event in fileMonitor.stream { ... }
Drawbacks:
I have forced the Package to use Swift 5.9 instead of using #available or @available. We could use an availability check to avoid breaking backward compatibility. OR we could release a new, breaking release of this Package using Github releases.
I’m open to either approach, but we should pick one.

@petershaw Any interest in taking this change?

Thanks for adding this and sorry for the late reply, somehow we missed it... We will have a look and get back to you, soon!

Hey @mcritz
I am totally  fine with a change to the swift 5.9 dependency!
I'll manage to take a look to the implementation at the beginning of next week.
Pretty cool at the first look! Thank you very much for contribution.

On Linux the FileMonitorTests.testLifecycleChange hangs.
@mcritz can you take a look into it?

@mcritz I split test into two and fixed the problem.
Let's wait for the CI confirmation. ;)

@KrisSimon Thanks for fixing that. Happy to have made a contribution!

@mcritz thank you very much for your contribution.


destory()

Hi,
Is there a destroy() or a cleanup() method?
I dynamically load sub modules of html and need to reload sometimes.

You can stop a monitor in a clean way with myMonitor.stop(). See  http://vissense.github.io/vissense/VisSense.VisMon.html#stop
Does this help you?

I'll try and let you know

Alright. Let me know if you need any help!

Works w/o any destroy or clean up, it just runs again ok. (some .js runs second time a mess)

@puppetMaster3 I cannot figure out what you are trying to tell me :)

It works, I can just leave the page.

(2. but some other projects don't work when you load html/js module 2 times).


🐪 Perl support

Icon: 🐪

This is a bit vague for an issue. Would you like the icon added to the README?

Perfect, thanks. I will take this one.

PR Submitted #1135

Thanks! I'll deliver it as a part of 4.x

Resolved by #1135


simple asset download not working

without flow router
Download it now

will download the test.txt file sitting in the public folder, as it should.
With flow router I'm getting a complete blank html page (Meteors start page).   I've tried other attributes in the anchor and they give me  'route not found errors'.
How do I allow simple hyperlink downloads with Flow-router?

I built a default project with just flow-router and flow-layout .... I don't have the problem....  so its obviously  something of my doing in my own project....


All entries in the CMS disappeared.

Describe the bug
Shown message if Im logged into the CMS.
Failed to load entry: TypeError: Cannot read property 'split' of null
To Reproduce
All my entries disappeared. No content shown in the CMS.

Screenshots

Applicable Versions:

Netlify CMS version: 2.3.3
Git provider: GitLab
OS: Catalina
Browser version  Chrome 78.0.3904.87

Node.JS version:  8

CMS configuration

backend:
name: git-gateway
branch: master # Branch to update (optional; defaults to master)
media_folder: "static/images/uploads" # Media files will be stored in the repo under static/images/uploads
public_folder: "/images/uploads" # The src attribute for uploaded media will begin with /images/uploads
collections:
  - name: "blog" # Used in routes, e.g., /admin/collections/blog

Additional context
I have two Apps running with Netlify CMS all of them are the same now. No CMS entries anymore. Both are connected via GitLab.
Both of them have different setups and all work great round about 10 days ago.  My clients where able to make changes. The error pops up since a couple of days.
I remember that we had the same issue this year regarding the assets folder. I hope the team can solve this like last time. Thank you.

This is a duplicate of #2881
See here https://www.netlifystatus.com/incidents/2r4ndv92qvs9

Thanks for quick reply.

This has been resolved https://www.netlifystatus.com/incidents/2r4ndv92qvs9?u=lrdhk0vbrbm7

I'm having the same problem @erezrokah , any ideas how I may resolve it?

Hi @ChenMorSays, I think you might be experiencing a different issue, as this one was related to a Netlify incident that had been resolved (same goes for your comment here https://github.com/netlify/netlify-cms/issues/1078#issuecomment-600605958).
You could trying asking in out slack #help channel: https://netlifycms.slack.com/join/shared_invite/enQtODE1NTcxODA5Mjg1LTRjYWExM2MyZDJmODA3YmVkMjI2YmQwZDg2ZDUyYTMyM2Y3Zjc1ZTJhNDBkYmMwNjA2ZTkwODY4YjZjNGNlNTE
Or post a new issue following the issue template.


Question: Helm chart deplyoment handle secret values

Hi @seglo
Is there an envisaged way for helm chart deployment to replace sensitive information (such as ssl.keystore.password) in the ConfigMap so that this information does not have to be blank in the configuration map?
I’m thinking of something like envsubst. The configuration file placeholders are replaced before the container starts and then starts from it.

I'm open to suggestions, but it's not something I have on my radar. Configuration is based on typesafe config and it currently relies on environment variable substitution when the config is loaded into the app itself, so I'm not sure preprocessing anything with enbsubst would work like you expect.


Remaining Edits from IPDS

Reviewers were:

Julia Prokopec, WIM
David Selkowitz, Nevada Water Science Center

Edits Requested:

[x] add a sidebar expander that explains "Water Year" somewhere. [ EB drafted one, it's in this PR]
[ ] JS wants us to choose "flow" or "streamflow" and be consistent. I suggest "flow" because it's in the title, but am not married to the decision.
[ ] DS wants to add this in the Sublimation sidebar: "Higher wind speeds can accelerate sublimation, and sublimation losses are much more significant in the more continental areas (Colorado, Wyoming) and less significant in the more coastal mountains."
[ ] JP requests that we put more consistent/clear titles on each of our charts, especially the line chart/ridgeplots which have none.
[ ] DS wants to add this to the last paragraph:

My only other recommendation is to mention that the USGS contribution to snow science also includes the development and production of the Landsat snow covered area product, which is available through Earth Explorer. Additional information can be found here.  https://www.usgs.gov/core-science-systems/nli/landsat/landsat-fractional-snow-covered-area

Other:

[x] On some smaller screens, one cloud image in the splash gets cut off with a hard line. Adjust image sizing to remove that. [EB did this, it's in her branch that hasn't been PR'ed yet as of 4/16]
[x] Need to adjust caption overlay sizing, it sometimes covers up important parts of the diagrams. [EB did a version of this as of 4/19]
[x] update diagram labels with correct terminology (SWE instead of "snowpack thickness") [ EB did this, it's on test but not on -beta]
[ ] Decide on a citation/linking style and implement it
[x] Add a link to download the diagrams
[ ] Update the README.md with final repo description and instructions for accessing data? [ EB is interested, re: conversation at kickoff with JR ]
[ ] Add units to buttons, not to axes
[ ] On mobile, buttons aren't behaving well in regards to size.  CN has already done some work on their branch on this, but just want to flag it.

I think the buttons should be different now based on PRs that have already been made. Have you pulled down recently?


mj-social icons

hey i noticed you guys have outdated icon like Instagram can you please update them

Hi @ncamacho97
Thanks for the issue but as stated in #611 every icons will be updated for the 4.0 release to avoid breaking changes on old templates

thanks man i really like the mjml thanks for being open source :)


💡 DIY Email Newsletter

This could be implemented via Netlify Background Functions I believe.
The function would be called every 10 minutes, for example, scrape the website (or rss feed?) for new posts and if one is found, send an email to any user that subscribed to the newsletter.

Note: this requires some backend to store the emails for users that subscribed to the newsletter.

I think a standard serverless function would be better. It should check the rss feed (/posts/index.xml) for any post in the last 10 minutes, let's say, and then trigger the email to be sent.
The function could be triggered via a webhook when a new PR is merged into master that has modified something under /content.

If i end up using cloudflare functions/workers with a D1 database (which I probably won't and I'll just use a 3rd party service or roll out my own), here is a provisioning script:
#!/bin/bash

# Define the database names
prod_database="mirceanton_prod"
preview_database="mirceanton_preview"

# Function to provision a database and initialize tables
function provision_database {
  database_name=$1
  echo "Provisioning $database_name database..."
  if [[ $(wrangler d1 list | grep "$database_name" | wc -l) -eq 0 ]]; then
    if wrangler d1 create "$database_name" >> /dev/null; then
      echo "$database_name database provisioned successfully."
    else
      echo "Failed to provision $database_name database."
      return 1
    fi
  else
    echo "$database_name database already exists. Skipping..."
  fi
}

function initialize_database {
  database_name=$1
  echo "Initializing tables in $database_name database..."
  wrangler d1 execute "$database_name" --command "
  CREATE TABLE IF NOT EXISTS newsletter_emails (
    address VARCHAR(255) PRIMARY KEY,
    join_date DATE
  );
  INSERT INTO newsletter_emails (address, join_date)
  VALUES<EMAIL_ADDRESS>'2022-01-01'<EMAIL_ADDRESS>'2022-02-01');"

  if [[ $? -eq 0 ]]; then
    echo "Tables in $database_name database initialized successfully."
  else
    echo "Failed to initialize tables in $database_name database."
  fi
}

# Call the function to provision the two databases
provision_database "$prod_database"
provision_database "$preview_database"

# Call the function to initialize the two databases
initialize_database "$prod_database"
initialize_database "$preview_database"


Peak width measurement

What about having a peak width at half heigth measurement tool?

It is already measured by the algorithm ml-gsd that we use.

I wonder how your implementation of GSD works in the ml-gsd. In principle GSD can get peakshapes that are in a range fo shapes with a parameter (a bit like the Voigt parameter for Vogt - combination of Lorentzian to Gaussian - but a braoder range of shapes). Do you search specifically Lorentzian? (meaning the linewidths are those of Lorentzian) or is it up to the GSD alogithm to find the best shape (typically a combination of Gau/Lor), in which case we should get the paramterS of the vogt shape. This will be very important for Julien when he will study complex superposition of peaks...

We are improving this aspect of the code recently. There is now a 'shape' package https://github.com/mljs/peak-shape-generator and we will be able to specify to GSD the shape so that the width is always FWHM


Add validation to the SubRequestType parameter

Fixes #10282
Status
ready
Description
The SubRequestType base spec parameter has been adopted already by a couple of Microservices, and we are going to extend its use in the framework, so we better start validating these values to make sure that the expected values are provided.
If request is created with an invalid value, it fails workflow creation.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
None
External dependencies / deployment changes
None

Tested in my VM. It has to be considered when we resume working on the Lexicon refactoring.

Thanks for this review, Todor. Release 1.5.5.pre4 coming in in a few minutes.