Avoid errors when Image.run_commands commands are empty strings

Helps avoids a confusing error that a user ran into (empty or whitespace strings produce Dockerfile syntax that looks like RUN  ).
Arguably we could raise here, but given that we already "gracefully" handle an empty list, dropping empty commands feels more consistent.

Resolves MOD-2626

Changelog

Image.run_commands will now skip empty strings or strings containing only whitespace characters, which otherwise produce malformed Dockerfile syntax.

Hmm, an empty list to me makes sense because it is the natural progression of
image.run_commands("echo 1", "echo 2", "echo 3") # 1, 2, 3
image.run_commands("echo 1", "echo 2") # 1, 2
image.run_commands("echo 1") # 1
image.run_commands() #

But not as sure about empty string
>>> subprocess.run(["echo", "0"])
0
CompletedProcess(args=['echo', '0'], returncode=0)
>>> subprocess.run([])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.11/subprocess.py", line 548, in run
    with Popen(*popenargs, **kwargs) as process:
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/subprocess.py", line 1026, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/usr/lib/python3.11/subprocess.py", line 1819, in _execute_child
    executable = args[0]
                 ~~~~^^^
IndexError: list index out of range

or strings containing only whitespace characters

Is this a minor typo in changelog?

Image.pip_install("") also fails with an empty string:
=> Step 1: RUN python -m pip install ''
ERROR: Invalid requirement: ''
Terminating task due to error: failed to run builder command "python -m pip install '' "

Caused by:
    container exit status: 1
Runner failed with exception: task exited with failure, status = exit status: 1
Stopping app - uncaught exception raised locally: RemoteError('Image build for im-q5JWJjKi7EuNMgQHfHigKC failed with the exception:\ntask exited with failure, status = exit status: 1').

Is this a minor typo in changelog?

author-blindness, but i am not seeing the typo?

Image.pip_install("") also fails with an empty string:

True — good point — although this feels more obvious to debug then the one the user reported.

But not as sure about empty string

Is this to advocate for raising instead of silently dropping empty strings?

Is this to advocate for raising instead of silently dropping empty strings?

Yes I think it might be more expected to raise?

I’m not really following the argument for why empty lists should be considered valid but empty strings not.

I also don't really understand why empty strings are an expected API

The argument is pretty simple: the parameter is typed as Union[list, str], so if we allow empty list, allowing empty string makes sense to. As noted in the PR description, raising an error could make sense too. But this is a very minor change, and either approach would be a marginal improvement.

The argument is pretty simple: the parameter is typed as Union[list, str], so if we allow empty list, allowing empty string makes sense to.

I must be crazy because this doesn't make any sense to me, can you please explain more as I have 🤔

I also wrote the API to take a union only as backwards compatibility with previous list APIs. But I don't get why an empty list and an empty string inside a list are the same regardless??


[WIP] (Express) Handle uncaught errors

Pull Request Test Coverage Report for Build 881fa967-192e-4512-a0bb-e3d4a6283204

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.379%

Totals

Change from base Build 8a496211-a16f-4bc9-b05a-185778f23204:
0.0%

Covered Lines:
1263

Relevant Lines:
1768

💛  - Coveralls


feat: use suffixed repo name as package name

Resolves #1057.
Following the recommendation from the discussion there, this uses the repo name suffixed with .rspm as the package name.

From my quick read, it looks like passing package_name will be a noop if the package does not use the package access modifier. Is that correct?

That's my understanding, yes. I believe SwiftPM does this by default, using the package name in Package.swift.

Seems that downstream compilation workers got confused by the @-prefixed package name and that caused tests to fail in CI. I've tweaked this slightly to trim the leading @.

I'm having trouble confirming locally, but it seems like the failing tests are running with Xcode 14.3/Swift 5.8, which doesn't support the -package-name compiler argument. Is the expectation that rules_swift_package_manager should gate on Swift version, or should rules_swift be handling that?

rules_swift should ignore that attribute when it's not supported. If it doesn't, we should fix it.

rules_swift should ignore that attribute when it's not supported. If it doesn't, we should fix it.

@brentleyjones Should I open an issue on rules_swift? Not sure what the next step ought to be here.

I can look at fixing that tomorrow if no one else has time, but rules_swift releases are sort of blocked right now as we get some other stuff is merged so not sure when this repo could pick that change up

Could/should we flag this in this report based on the swift tools version of the Package.swift?

it probably makes more sense to gate the setting of package_name in this repository and not in rules_swift

SGTM. What is the correct gate? The Swift tools version?

Yeah I imagine that's how it works in SPM but haven't checked

Just pushed an update that gates package_name behind the tools version. The semver parsing is naive but hopefully sufficient for this purpose. If not, please lmk!


Integrate Into WebUI's API?

Hello, Auto told me that extensions can be integrated into the webui's api. Could we do this ^_^

I have the same question. when I use txt2img api to generate a image, I found there is no Score field on the meta data of the generated jpg image


Refactor for Zeit 2.0

Replace microrouter with Zeit 2.0's new api routing system.
This new system is currently being used for https://gitographer.com/

@mxstbr Just bumping this in-case your notifications weren't going through.


[Feature] : Design Categories Page for Seller

Description

Current Page is just rending table data by fetching the same from db.
We need to add selection of pre-defined categories : Beauty Products, Gadgets, House Hold etc.
Improve UI/UX of this page

Screenshots
Current Design

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Can i try it ?


refactored based on wes bos advanced react course

https://github.com/wesbos/Advanced-React/tree/master/finished-application/backend

Issues from my end on localhost:4000:
-- Mutations go through, but the changes made aren't showing up in queries


[FEATURE] Form Education

Description
As a User, I want to see a form in which I can add my education.
Create a form with following input elements

[ ] Degree
[ ] School
[ ] GPA
[ ] City
[ ] Country
[ ] Start Date
[ ] End Date
[ ] Description
[ ] Save button
[ ] Cancel button

Acceptance Criteria (AC) for beginners

[ ] Create a component called Education.js and save it in a folder called components in the src directory
[ ] Heading: Education
[ ] Form with above mentioned fields
[ ] Inputfields must be editable
[ ] Input fields are accessible
[ ] Save button does not require functionality
[ ] No stylings but plain html

Additional AC if you are more experienced

[ ] Save data in localStorage

Screenshots
No response
Additional information
Please do not hesitate to ask me about accessible forms in case you are unsure.
Accessibility and your inexperience with this topic should not be the reason you are not work on this issue, but to learn about it.

Hi, I would like to attempt this.


Merge prTesting_201909092349 branch to master branch

Docs Build status updates of commit 238bb74:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:bulb:Suggestion

Details

split/updateTestCon.md

[Suggestion] Missing attribute: author. Add the current author's GitHub ID. NOTE: This Suggestion will be elevated to a Warning on 9/1/2019.
[Suggestion] Missing attribute: title. Add a title string to show in search engine results. NOTE: This Suggestion will be elevated to a Warning on 9/1/2019.

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


Capture ExportTemplate rendering errors more elegantly

Environment

Python version:  3.5.6
NetBox version:  2.5.10

Proposed Functionality
Currently, if an ExportTemplate is saved with invalid template code and then run, it will raise an exception which is handled like any other server error (500). Ideally, we should capture the template rendering error and present it in some more elegant fashion so that the user is immediately aware that it's a bug in the template rather than NetBox itself.
Use Case
This would allow for quicker identification of template errors.
Database Changes
None
External Dependencies
None

I should point out that we are already catching TemplateSyntaxError. This extends that logic to catch any exception raised during rendering of the template.


[question] Aeon Labs Home Energy Meter G2 to much information

One of the devices I have added recently is a Home energy meter.
I have used this device in my previous HA application (Indigo for mac) and zwave2mqtt. On both platforms the sensors made sense. But when added to zwavejs2mqtt there is a lot of sensors that I have never seen before. Like previous values and delta time etc... I sort of just want the energy readings. When created in Home Assistant it creates so long names that it is difficult to make out what is what.
Is it supose to be like this, or is there something faulty with the way zwavejs2mqtt is reading it?

@haavardNO Could you show me the names you are getting? Have you tried to set a custom entity name? @billiaz

@haavardNO Could you show me the names you are getting? Have you tried to set a custom entity name? @billiaz

I get this with my Aeon plug, the previous value is required to store the value when you reset values (the button in your screenshot), the delta entities can just been hidden / disabled (either in the zwavejs side, or the Home Assistant side).

I get this with my Aeon plug, the previous value is required to store the value when you reset values (the button in your screenshot), the delta entities can just been hidden / disabled (either in the zwavejs side, or the Home Assistant side).

@robertsLando no I have set a name and location in the zwavejs2mqtt UI, and I have changed the name in HA afterwards. But it is pain to find the device that gives the correct value since there is so many. This metering device has one clamp. This generates 21 devices. The other metering device I use has 3 clamps, that device generates 36 devices. I guess I can tweak it to figure out what is what.

@robertsLando no I have set a name and location in the zwavejs2mqtt UI, and I have changed the name in HA afterwards. But it is pain to find the device that gives the correct value since there is so many. This metering device has one clamp. This generates 21 devices. The other metering device I use has 3 clamps, that device generates 36 devices. I guess I can tweak it to figure out what is what.

I think I need to configure a way to skeep discovery for some values. @billiaz I think that in case of meter CC some values should be skipped, then we could also provide a way to ignore the discovery of some specific values using the gateway values table

I think I need to configure a way to skeep discovery for some values. @billiaz I think that in case of meter CC some values should be skipped, then we could also provide a way to ignore the discovery of some specific values using the gateway values table

i found a nice way to do this. I will also add the pr here.

i found a nice way to do this. I will also add the pr here.

Made a small patch: #237

Made a small patch: #237


Suggestion: Rename Podcast to All Episodes

I had no idea what I would end up on when clicking "Podcast"

I think for now this will remain as Podcast however I think I might add an additional heading to the podcast page above All Episodes for heading clarity. This will need to be re-thought out if there are ever any additional Syntax branded shows.


Test Besu -> Tessera TLS configuration

Description
The Besu -> Tessera configuration needs to be verified and tested.
Acceptance Criteria
Verify that the TLS connection works as expected.

[x] Tessera <> Tessera
[x] Tessera <> Besu
[ ] instructions for how to generate certs for Tessera to use

TLS - Generating certificates - Do you have instructions for how to generate certs for Tessera? eg here we have instructions for Orion generating certs - https://docs.orion.consensys.net/en/stable/Tutorials/Generating-Certificates/
Tessera team says:
These are instructions to generate any certificate - Tessera should be able to use them. I don’t think anything special need to be done although if running on localhost it’s likely you also need to include the port in the CN.
But let me try this and get back to you

       {
           "app":"P2P",
           "serverAddress":"https://localhost:8081",
           "sslConfig": {
               "tls" : "STRICT",
               "generateKeyStoreIfNotExisted" : "true",
               "serverTlsKeyPath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/key.pem",
               "serverTlsCertificatePath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/cert.pem",
               "serverTrustMode" : "TOFU",
               "clientTlsKeyPath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/key.pem",
               "clientTlsCertificatePath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/cert.pem",
               "clientTrustMode" : "TOFU",
               "knownClientsFile" : "/Users/sallymacfarlane/workspace/besu-local-nodes/knownClients",
               "knownServersFile" : "/Users/sallymacfarlane/workspace/besu-local-nodes/knownServers"
           },
           "communicationType" : "REST"
       }

When I use cert.pem and key.pem from tessera/security/src/test/resources/ the above config runs ok, and 2 tessera nodes can communicate using SSL. However if I run with the certs generated by orion I get an error
2021-03-22 11:29:29.849 [pool-3-thread-1] WARN  c.q.t.p.p.PartyInfoBroadcaster - Failed to connect to node https://localhost:9002/, due to javax.net.ssl.SSLHandshakeException: No name matching localhost found
and if I replace localhost with <IP_ADDRESS> in the config I get this error:
2021-03-22 11:20:52.312 [pool-3-thread-1] WARN  c.q.t.p.p.PartyInfoBroadcaster - Failed to connect to node https://<IP_ADDRESS>:9002/, due to javax.net.ssl.SSLHandshakeException: No subject alternative names present

From Tessera team:
The issue is Orion automatically generate the certificate with something like <IP_ADDRESS>:8080 in the CN field of the certificate. When it comes to IP address Java is stricter that it enforces the certificate contains a SAN (Subject Alternative Names) entry for the IP as having the url in the CN has become obsolete.
so - 2 ways you can get it working
Follow the Orion instruction to generate certs here https://docs.orion.consensys.net/en/stable/Tutorials/Generating-Certificates/#public-dns-is-cn and ensure that the CN=localhost
Follow instruction to generate certs https://docs.orion.consensys.net/en/stable/Tutorials/Generating-Certificates/#ip-address-is-cn and have the SAN configured to include <IP_ADDRESS>
and Tessera should be able to start with those certificates

I followed the instructions as suggested (I tried both ways, and orion can run with the certs generated) but Tessera gives this error:
com.quorum.tessera.ssl.exception.TesseraSecurityException: java.security.spec.InvalidKeySpecException: java.security.InvalidKeyException: IOException : algid parse error, not a sequence
at com.quorum.tessera.ssl.context.ServerSSLContextFactoryImpl.from(ServerSSLContextFactoryImpl.java:49)
at com.jpmorgan.quorum.server.utils.ServerUtils.buildWebServer(ServerUtils.java:44)
at com.quorum.tessera.server.JerseyServer.start(JerseyServer.java:88)
at com.quorum.tessera.launcher.Launcher$1.launchServer(Launcher.java:58)
at com.quorum.tessera.launcher.Main.main(Main.java:88)
Caused by: java.security.spec.InvalidKeySpecException:

need to verify whether we can use PKCS1 key or does it need to be converted to PKCS8

confirming the PKCS1 key works (as generated by instructions in Orion docs) https://docs.orion.consensys.net/en/latest/Tutorials/Generating-Certificates/
PKCS1 key:
               "serverTlsKeyPath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer2.key",
               "serverTlsCertificatePath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer2.pem",

PKCS8 key generated by
openssl pkcs8 -topk8 -inform PEM -outform PEM -nocrypt -in orion_cer2.key -out pkcs8-2.key
               "serverTlsKeyPath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/pkcs8-2.key",
               "serverTlsCertificatePath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer2.pem",

A couple of things to make clear in the docs for

If running on localhost, you can follow the instructions for generating certs in Orion docs. You just need to make sure that the certificates are generated with "localhost" in the CN (ie when you answer the questions, make sure you specify localhost as CN) eg:

$ openssl x509 -in orion_cer.pem -text -noout
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:<PHONE_NUMBER>9262280134 (0xcb56d675d49109c6)
    Signature Algorithm: sha256WithRSAEncryption
        Issuer: C=AU, ST=QLD, L=Brisbane, O=ConsenSys, CN=localhost
        Validity
            Not Before: Apr  1 01:49:10 2021 GMT
            Not After : Aug 14 01:49:10 2022 GMT
        Subject: C=AU, ST=QLD, L=Brisbane, O=ConsenSys, CN=localhost

The issue is Orion automatically generate the certificate with something like <IP_ADDRESS>:8080 in the CN field of the certificate. When it comes to IP address Java is stricter that it enforces the certificate contains a SAN (Subject Alternative Names) entry for the IP as having the url in the CN has become obsolete.
so - 2 ways you can get it working
Follow the Orion instruction to generate certs here https://docs.orion.consensys.net/en/stable/Tutorials/Generating-Certificates/#public-dns-is-cn and ensure that the CN=localhost
Follow instruction to generate certs https://docs.orion.consensys.net/en/stable/Tutorials/Generating-Certificates/#ip-address-is-cn and have the SAN configured to include <IP_ADDRESS>
and Tessera should be able to start with those certificates

If running multiple tessera/besu nodes on a single host - Use separate knownClients and knownServers for P2P and Q2T endpoints in Tessera config. eg

   "serverConfigs":[
       {
           "app":"ThirdParty",
           "serverAddress": "http://localhost:8581",
           "communicationType" : "REST"
       },
       {
           "app":"Q2T",
           "serverAddress":"https://localhost:8881",
           "sslConfig": {
               "tls" : "STRICT",
               "generateKeyStoreIfNotExisted" : "true",
               "serverTlsKeyPath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer1.key",
               "serverTlsCertificatePath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer1.pem",
               "serverTrustMode" : "TOFU",
               "clientTlsKeyPath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer1.key",
               "clientTlsCertificatePath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer1.pem",
               "clientTrustMode" : "TOFU",
               "clientAuth" : "false",
               "knownClientsFile" : "/Users/sallymacfarlane/workspace/besu-local-nodes/knownClients1",
               "knownServersFile" : "/Users/sallymacfarlane/workspace/besu-local-nodes/knownServers1"
           },
           "communicationType" : "REST"

       },
       {
           "app":"P2P",
           "serverAddress":"https://localhost:8081",
           "sslConfig": {
               "tls" : "STRICT",
               "generateKeyStoreIfNotExisted" : "true",
               "serverTlsKeyPath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer1.key",
               "serverTlsCertificatePath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer1.pem",
               "serverTrustMode" : "TOFU",
               "clientTlsKeyPath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer1.key",
               "clientTlsCertificatePath" : "/Users/sallymacfarlane/workspace/besu-local-nodes/orion_cer1.pem",
               "clientTrustMode" : "TOFU",
               "knownClientsFile" : "/Users/sallymacfarlane/workspace/besu-local-nodes/knownClients3",
               "knownServersFile" : "/Users/sallymacfarlane/workspace/besu-local-nodes/knownServers3"
           },
           "communicationType" : "REST"
       }
   ],


chacha20 seems to be broken in OpenSSL 3.0.0alpha6

When using OpenSSL 3.0.0alpha6, OpenSSH versions > 8.2p1 fail to connect anywhere, usually resulting in a "protocol error" while establishing the connection.
Bisecting OpenSSH shows this commit is what triggers the breakage:
https://github.com/openssh/openssh-portable/commit/abe2b245b3ac6c4801e99bc0f13289cd28211e22
Defining HAVE_BROKEN_CHACHA20 in openbsd-compat/openssl-compat.h (like the openssh commit does for some versions of LibreSSL) "fixes" the problem.
Looks like EVP_chacha20 doesn't work as expected by OpenSSH.

Are you in a position to attempt to bisect on openssl versions?
It would be particularly interesting to know if 1.1.1 releases (e.g., 1.1.1h) are functional for this usage.

You might want to read the commits: https://github.com/libressl-portable/openbsd/commit/1a068026c3e1e9cd67a2941795421a8fe38f8401, https://github.com/libressl-portable/openbsd/commit/0f67aa39232d057952bb726a0d8f9db49d0fdec4, https://github.com/libressl-portable/openbsd/commit/61c1088900c0b029354d99ac53a24c39d0105576 and the release notes for libressl 3.1.0: https://ftp.openbsd.org/pub/OpenBSD/LibreSSL/libressl-3.1.0-relnotes.txt
They introduced EVP_chacha20 before RFC7539 was published, while we waited for the RFC. The RFC says it's a 32 bit counter followed by a 96 bit nonce, which is what we do. They did something else.
So libressl in 3.1.0 switched to have the same EVP_chacha20 behaviour as OpenSSL. And the commit you point to seems to suggest that that requires to have HAVE_BROKEN_CHACHA20 defined.

@kroeckx This issue has nothing to do with the LibreSSL commits you're mentioning. LibreSSL is not involved here. The problem occurs with OpenSSL 3.0.0a6.
The only connection to those LibreSSL commits is that - because of its initial broken implementation - OpenSSH already has a builtin workaround for SSL implementations with broken chacha20, and activating that makes it work with OpenSSL 3.0.0a6 too.

In a later commit, they fixed it to say LIBRESSL_VERSION_NUMBER < 0x3010000fL
I can confirm that openssh 8.3p1 works with openssl 1.1.1g but not with master. Running a bisect.

3d5a7578e09a984c6475b1c008f5c76f850328cb is the first bad commit
commit 3d5a7578e09a984c6475b1c008f5c76f850328cb
Author: Shane Lontis<EMAIL_ADDRESS>Date:   Wed Oct 16 16:18:42 2019 +1000
Add ChaCha related ciphers to default provider

Reviewed-by: Richard Levitte<EMAIL_ADDRESS>(Merged from https://github.com/openssl/openssl/pull/10081)

@kroeckx, would you mind to submit a test?

@kroeckx, what are the steps to test this?

What I did was compile openssh against 3.0, and then use the ssh
binary to connect to an sshd that's build against 1.1.1.
<EMAIL_ADDRESS>seems to be the default cipher it
uses, at least talking to 7.9p1 server.

#9524 might be related

I just tried the current openssh-portable development with my 3.0 installation, and it fails in make tests...  seems to be regress/connect.sh failing, and I found this in the end of regress/failed-ssh.log:
debug1: kex_input_ext_info: server-sig-algs=<ssh-ed25519,sk-ssh-ed25519@openssh.com,ssh-rsa,rsa-sha2-256,rsa-sha2-512,ssh-dss,ecdsa-sha2-nistp256,ecdsa-sha2-nistp384,ecdsa-sha2-nistp521,sk-ecdsa-sha2-nistp256<EMAIL_ADDRESS>Bad packet length<PHONE_NUMBER>.
debug3: send packet: type 1
ssh_dispatch_run_fatal: Connection to UNKNOWN port 65535: Connection corrupted
FAIL: ssh proxycommand connect failed

Considering #9524, I did a similar build with pkixssh, where a make tests ends like this:
test_sshbuf: ....................................................................................................... 103 tests ok
test_sshkey: .......................................................................................... 90 tests ok
test_bitmap: .. 2 tests ok
test_conversion: ... 3 tests ok
test_kex: .....Bad packet length<PHONE_NUMBER>.
ssh_packet_next: Connection corrupted

regress/unittests/kex/test_kex.c:73 test #6 "rekeying client"
ASSERT_INT_EQ(r, 0) failed:
           r = -54
           0 = 0

That's worryingly similar...  I haven't dived into the pkixssh logs yet

This is caused by the provider conversion breaking the sequence of multiple EVP_CipherInit() calls where first one sets the EVP_CIPHER and the key and then there are multiple calls to EVP_CipherInit with just an iv set, NULL EVP_CIPHER and NULL key followed by EVP_Cipher call. We need to preserve the previous key set on the EVP_CipherInit call in this circumstance.

This is caused by the provider conversion breaking the sequence of multiple EVP_CipherInit() calls where first one sets the EVP_CIPHER and the key and then there are multiple calls to EVP_CipherInit with just an iv set, NULL EVP_CIPHER and NULL key followed by EVP_Cipher call. We need to preserve the previous key set on the EVP_CipherInit call in this circumstance.


implement rtx test

asdf has the ability to test a plugin, we should do the same for parity: https://asdf-vm.com/plugins/create.html#testing-plugins
This will also help with #182

this command doesn't actually do what I thought it did. Somehow I thought that it would call a binary in the plugin like bin/test so the plugin could perform some checks. It's more manual, and just makes it easier to install a plugin and runtime at the same time which is already pretty easy to do with rtx exec.
That said, it has the ability to use a specific git ref which you can't do with rtx exec. I suspect it might be possible to add this but I'm not sure what the syntax would be. A less good workaround would just be if I implemented the ability to set git refs with plugins install or maybe plugins update.
I'm wondering if it would be better see if asdf might be interested in adding a bin/test feature to the plugins like I was thinking.

this seems to be of less value to me as I explained in #192
I'd be open to adding this if someone can give me a use-case where it is important though. It seems more useful in asdf since there isn't the ability to install a plugin, runtime, and execute in a single command like we can do in rtx


Why does not support one-click confirmation group?

As you can see, marvelousga this site is verified by spelling &verify + ID in the form after the page.

Each task has a unique ID.

So, can you do this function?

I try this, it not work.

My goal for this script is to add new features to these sites, but not to change any features they currently have.  So sorry, but I don't plan to add anything like this.
Also, there's no need to click "Verify" for the tasks at MarvelousGA.  Simply join the Steam Groups, complete the Captcha, and click "Get My Key".  There used to be a single task that did require verification (detailed here), but I haven't seen it in a while.

Would add this to my extension, but there seem to be different task types. I can bypass the gamehag tasks, but full automation would make it break too fast.


Disallow more dangerous characters in paths

'&' in folder name could also end with strange build errors
https://github.com/Cognifide/gradle-aem-plugin/blob/master/src/main/kotlin/com/cognifide/gradle/aem/common/instance/LocalInstanceManager.kt#L512

to be released


Change stream cache eviction to least-recently-used

This is the same patch as https://github.com/praw-dev/praw/pull/1547, but applied to the equivalent files in Async PRAW.
I haven't made any effort to test beyond importing the unit test patch, nor do I know anything about Async PRAW beyond some general knowledge of asyncio.

Upon further review, it seems that a custom async iterator is going to have to replace the function.

Upon further review, it seems that a custom async iterator is going to have to replace the function.

I have helper functions made here: https://github.com/praw-dev/asyncpraw/blob/ad443d6a9218e8868dfcd33d53ae9fd3508c9ce7/tests/integration/init.py#L76-L85
Note that async_next() is not a true implementation of next(). I think using async_list() to convert it to a list would be best as you can then you can use next().

I think the issue here is that in normal PRAW, the function returns a normal iterator (a list). The function therefore should logically return an async iterator. The best idea would be an async list, which inherits from list, and the __anext__ method should yield the list's attributes back.

As a heads up, I'm not planning to work on this patch any further. Feel free to do whatever you like with it.

Pull Request Test Coverage Report for Build d4e22fe65fb1b6fabb0f66677261b5765592b2bf-PR-20

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 9fc6211cffd98188f7a20c712fe205bac9fe2f4c:
0.0%

Covered Lines:
3481

Relevant Lines:
3481

💛  - Coveralls


Update notifications must be translated

Describe the problem
When a new version of Bravo is available, a system notification (toast) is displayed in English, regardless of the UI language.
It should match the UI language.
Bravo version
0.9.1
Tabular model
Power BI Desktop, import mode
Power BI
Any
Steps to reproduce the behavior
No response
Additional context
No response
Screenshots
No response
Anything else ?
No response

Added in version 0.9.5


Trigger Docker builds from main build pipelines

Currently Docker builds are triggered via a schedule separate from the main build pipelines.  This issue is to discuss whether we initiate those builds from the main build pipelines.
Related: https://github.com/AdoptOpenJDK/openjdk-tests/issues/773 - we currently trigger Docker image testing off of the main build pipelines, which is incorrect, as there may not be a new Docker image to test, if the Docker build was not run.

The way I see it, there’s two ways you could do this

Have a check in the build_base to see if we’re building docker or a normal pipeline. Then execute build_all.sh if it is docker or carry on the pipeline as normal if not. Im not sure how transferable this could be however. The parameters for build_docker_multiarch seem simple enough to adapt to the normal pipelines but im not sure if there’s anything in build_all.sh that could interfere or outright break the normal pipelines. Best to get @johnoliver  or @sxa555 to comment on that one.
Adapt the build_all.sh script to be a groovy jenkins script instead of a bash script and have it run parallel with the normal pipelines from build_base. That would require a lot more effort to adapt the script but it might result in a faster and more efficient solution at the end as the whole docker pipeline would have been looked at for bugs and it will mean the docker repo could be fully moved in the build repo.

If we add it to the pipelines just now it will ause a significant delay in the overnight builds due to the length of time required to build all the images (3-24 hours at the moment). I would not be keen to slow the overnight build pipelines down in this way at present ... Thoughts? I am looking at the performance of those pipelines (I have new x64 machines for docker which have speeded those ones up in the last week)

Tagging in @dinogun on this conversation too :-)


add import extensions rule

Resolve #32
Added exception for css

Thank you, if you like this project, please don't hesitate to put your star.


Login: fix error message and OAuth

There are 2 issues in the Login.js component:

Notifications (success or error) are shown 2 times, probably because the <Notifications cmp is added.
Notifications are outside the background.
There should be no notifications in the frontsite, replace them with a box in the login page
It is not possible to override the OAuth config and removing existing OAuth login (e.g. GitHub): probably when merging overridden config, we should use assign instead of merge? To be checked.

See screenshot:

It seems the UI issue is solved; moreover the OAuth component was already made extensible. Will link the PRs when I find them.

The issue is still present. To reproduce, just go to: http://localhost:3000/login?message=error&code=400


cause a 100% CPU use in some condition

My ENV is MAC OSX 10.7, TextMate Version 1.5.10 (1631).
Some time just open textmate for a while, my system become very slow, and CPU of TextMate cause is 100%(also distnoted agent will use another 100%).
I am not very sure it is caused by Minimap. but after I remove Minimap, seems everything goes well.
Buy the way, I like this app very much. Thanks a lot.

Mac OSX 10.12.6. Textmate just began taking over 100% CPU 2 weeks ago. Renders my laptop pretty much inoperable. What can I check to fix this?


Break out ssh-agent

Make tkey-ssh-agent standalone in its own repo.
Probably rename -apps to tkey-ssh-agent to keep the stars for Homebrew.

It's now alone in repo but renaming repo will have to wait until we announce the plans widely.

Any progress on renaming the repo?

Just did it. For context, we were look into if it would give us problems with our brew formula, but apparently GitHub redirects to the new name automatically.


shiboken: win64: wrapInstance: fix pointer size

I think this is fixed in newer versions of pyside. The patch is quite simple:
diff --git a/sources/shiboken/shibokenmodule/typesystem_shiboken.xml b/sources/shiboken/shibokenmodule/typesystem_shiboken.xml
index 6025400..4a0853c 100644
--- a/sources/shiboken/shibokenmodule/typesystem_shiboken.xml
+++ b/sources/shiboken/shibokenmodule/typesystem_shiboken.xml
@@ -17,7 +17,7 @@
         </inject-code>
     </add-function>

-    <add-function signature="wrapInstance(unsigned long, PyType)" return-type="PyObject*">
+    <add-function signature="wrapInstance(unsigned long long, PyType)" return-type="PyObject*">
         <inject-code>
             if (Shiboken::ObjectType::checkType((PyTypeObject*)%2))
                 %PYARG_0 = Shiboken::Object::newObject((SbkObjectType*)%2, (void*)%1, false, true);

Maybe this patch could be added to the feedstock and be applied for `# [win64]
https://bugreports.qt.io/browse/PYSIDE-191

this is marked duplicate of https://bugreports.qt.io/browse/PYSIDE-103 which is still valid.
I won't accept it here as is unless you submit that patch upstream first.

I know the better solution would be to have it fixed upstream, but as I expect not too much work to be done on the qt4 bindings, it would be simpler to patch it within the feedstock.
I will try size_t and upload a package for the freecad channel. thanks for the information.

The bug still exists for PySide2 (dev branch), so you know what to do... : )

fork pyside : ).  I have no idea how to work with qt-projects. If pyside would be accesable via github, this wouldn't be a big problem...
So I will let this open for someone experienced with contributing to qt-projects...

at least you can send an email to the (few) active maintainers

they replied: it was fixed in tree a long time ago!

thanks!

@jschueller there is another problem with shiboken. The diff is coming from @cgohlke who seems to have a working pyside for win10. Maybe this patch can be added to the merged one, as these patches address quite the same issue.
https://github.com/looooo/pyside-feedstock/blob/master/recipe/win_pointer_conversation_fix.patch

is it a new patch or is it already in tree ?

I think it is allready in pyside2 but in a different way:
#if PY_MAJOR_VERSION >= 3
        if (!PyLong_Check(pyobj)) {
            PyErr_SetString(PyExc_TypeError, "Invalid type for unsigned long long conversion");
            return 0;
        }

        return PyLong_AsUnsignedLongLong(pyobj);
#else
        if (PyInt_Check(pyobj)) {
            long result = (unsigned PY_LONG_LONG) PyInt_AsLong(pyobj);
            if (result < 0) {
                PyErr_SetObject(PyExc_OverflowError, 0);
                return 0;
            } else
                return (unsigned PY_LONG_LONG) result;
        } else if (PyLong_Check(pyobj)) {
            return (unsigned PY_LONG_LONG) PyLong_AsUnsignedLongLong(pyobj);
        } else {
            PyErr_SetString(PyExc_TypeError, "Invalid type for unsigned long long conversion");
            return 0;
        }
#endif // Python 2
    }
};

what bug does it fix ?

I don't know which bug it fixes, but the python interface of shiboken has quite some problems without this fix. Without it Overflow Errors are reported on win10. This patch fix this problems.
There is no such issue with win 7 and linux.

@cgohlke do you know what's the problem ?

Somewhere in the PySide headers  PyInt_Check is defined as PyLong_Check on Python 3 for compatibility reasons (Python 3 only has PyLong, not PyInt). So on 64-bit Python 3 the if (PyInt_Check(pyobj)) {...} block is run with 64-bit PyLong values even though the block can handle C long only (because it was meant for Python 2 PyInt). On Windows C long is always 32-bit. Hence the block returns wrong results on Windows when the 64-bit PyLong values are greater than 2**32. That can happen on Windows 7 too, it's just not that likely for pointers. On Python 3 the if (PyInt_Check(pyobj)) {...} block should be removed.  Hope got it right...

Do you have a script that shows the bug ?

https://github.com/looooo/pivy/issues/18#issuecomment-305956542


Does the PCF tile also forward syslog to stackdriver?

Does installing the GCP Stackdriver Nozzle for PCF also install the fluent bosh addon which forwards all syslogs for a PCF foundation to stackdriver? Or is this something that must be done separately?

Currently it is not installed with the tile. A separate BOSH deploy is required


Design 'Review and Confirm' page

The user will be landed on this page to review the shipping and payment information as well as the Items they are ordering.

#18


Update Spanish localization

Translate missing strings to hit full Spanish localization.
Going forward, I would take @elenatorro and @ecreeth 's very sensible suggestions in #6094 to the rest of the strings, but additional community input would be helpful to agree on the preservation (or not) of certain anglicisms (Is it a "post" or a "publicación"? In verbal form, "repost" or "republicar"? Once done, is it a "repost" or a "republicación"? ¿A un usuario "le gusta una publicación" o "le da «me gusta» a un post"? And so on with "Feed", "Discovery", etc.)

Hey! I've also been working on the Spanish localization, but on my own, I'll publish it when I'm done.
I've been using some anglicisms like Post, Repost, Feed, etc because in my opinion trying to use native spanish words with those doesn't fit in well, but I did opt to not use anglicisms in a couple where they fit in, like "Chat" which in my project translated as "Mensajes"
Personally, I don't find the «» quotation marks appealing and instead use "". In the case of "le da «me gusta» a un post" I don't find it necessary to use quotation marks at all. In my localization I worded it as "{user} le ha dado me gusta a tu post"

I agree that "Post," "Repost," and "Feed" are commonly used terms, as well as "Pack" versus "Paquete." I don't have a strong preference since both options are correct, but I think it would be helpful to know which terms people are more accustomed to using nowadays 👍🏽

Hello @reoblauber . Of course, you're welcome to expand on this PR, it now has 100% coverage for the latest release.
All new strings are using "Publicación" but I do agree that "Post" feels more familiar, even in Spanish, and that "Chat" reads much better as "Mensajes".
@madkoding has done a massive effort before, some of their translations with "me gusta" use guillemets («») and some others don't; though it might seem trivial, we could start by standardizing these - in the new strings I went for regular double quotation marks because I feel quoting is necessary to describe someone else's action on one's post, but I'm not married to that idea. What I was wondering about was if it'd be more natural to say a {user} le gusta tu post rather than {user} ha dado "me gusta" a tu post. Do you have an opinion on that?

In the PR I am working on I am mainly fixing some inconsistencies all around, just tidying up things, but I don't know how long it's gonna take so I might just make it its own PR. If I get it done soon enough I will consider just adding it to this one.

some of their translations with "me gusta" use guillemets («») and some others don't

In my opinion, I don't think they are a good fit. I have of course seen them in the past but I don't think I've ever seen them used in the Spanish localization of any other site or app I've worked on, and personally they simply just don't feel natural to me to use them in this case.

What I was wondering about was if it'd be more natural to say a {user} le gusta tu post rather than {user} ha dado "me gusta" a tu post. Do you have an opinion on that?

Both are correct, but I am more partial to the "le ha dado me gusta" (without the quotation marks around "me gusta", i personally don't find them necessary there) since it emphasizes more that the user pressed "Like" on the post, rather than the slightly ambiguous "le gusta tu post" which could mean the user simply thought your post was neat. This is of course just my opinion though and I wouldn't mind whichever you decide to choose.

Absolutely right. It's missing @hogaza 's changes from #6691, I'll pull those. Thank you!
2740:msgstr ""
2839:msgstr ""
3767:msgstr ""
6685:msgstr ""
8222:msgstr ""
8477:msgstr ""
8481:msgstr ""
8896:msgstr ""

I know that Bluesky wants to move to some CAT-tool, but for now the translations can be done in Localazy. Currently for sv, es, an and nl based upon en. I have learned that the Tools -> File management -> Export is only available to persons with role Owner. I will need to do the export; I am sorry, but just drop me an email or ping this issue. I will put the new version then in the branch i18n-es on https://github.com/monty241/social-app/tree/i18-es

Yes, it is a little cumbersome. Hope that the alternative tool becomes available. In case it takes very long I can run a daily script to extract them using the Invantive SQL-driver for Localazy and commit & push them automatically, but a process integrated solution is better for bsky.

Thanks for adding my changes, @codehead ! I'm wondering how to proceed, as I wanted to update other strings but I'd like to do it in a coordinated way with the other contributors. @monty241 is it possible to join the Localazy team while we wait for an official Crowdin integration?

Sure. Just enter request at https://localazy.com/p/bsky. Please put your GitHub name and language in the text.

@hogaza I have added you to ES as translator and changed all first person to enter for a language to reviewer, in this case @codehead . I am currently the only one that can export it seems, but just drop me a note. It takes me just a few seconds and in case Crowdin takes very long I will change it into a SQL-script to dump daily to messages.po.


Add support for PostgreSQL

#43 Hey guys, I added support for PostgreSQL. Note this is only for PostgreSQL! And since it's been asked for, I'm going to be looking into adding support for PostGIS. 🤙

@brianfajardo Thanks for the PR!
One small thing missing, please document how to change the credentials for the Postgres service just like we have it for the MySQL service. https://github.com/Osedea/nodock#change-the-mysql-databaseuserpassword

@philtrep Readme has been updated to show users can change the credentials to their liking!

@brianfajardo Fantastic! Thanks a bunch


Fix bug with publishing OpenOA notebooks to ENTR docs site

Part of the decomposition of this GH issue: https://github.com/entralliance/entr_runtime/issues/27

@ejsimley is this still an issue?


Merge storage_throughput and storage_throughput_vs_cpu benchmarks

The two benchmarks are quite similar, with some differences that can be handled (in hindsight) using command-line options:

throughput_vs_cpu does not test XML vs. JSON, that could be a separate option.
throughput uses fixed sized objects, that can be handled by setting the minimum and maximum object sizes to the same value.
throughput_vs_cpu does not run if using multiple threads on a platform without RUSAGE_SELF, but we need that for throughput.  We can print a warning and zero-out the CPU usage results in such a platform.

This is obsolete, the storage_throughput benchmark is gone.


cssSelectorAncestor not applied to grouped child selectors properly

With multiple videos on page and multiple play/pause controls, the cssSelectorAncestor does not apply properly, causing all play/pause controls after the first in the css selector group to actually control the last-instantiated video element.
Sample configuration:
screen.jPlayer({
     ready: function () {
         $(this).jPlayer("setMedia", {
             title: title,
             m4v: path
         });
     },
     play: function() {
         $(this).jPlayer("pauseOthers");
     },
     swfPath: "/js",
     supplied: "m4v, ogv",
     cssSelectorAncestor: "#foo",
     cssSelector: {
         title: "#title",
         play: ".video__play, .video__controls--play-pause",
         playBar: ".current",
         seekBar: ".progress-bar",
         mute: ".mute",
         unmute: ".unmute",
         fullScreen: ".fullscreen",
         currentTime: ".time-status__now",
         duration: ".time-status__total"
     },
     size: {
          width: "100%",
          height: "auto"
     }
 });

Pertinent part:
screen.jPlayer({
     cssSelectorAncestor: "#foo",
     cssSelector: {
         play: ".video__play, .video__controls--play-pause",
    }
});

So the CSS selector used to find play buttons appears to be #foo .video__play, .video__controls--play-pause and the second in the group (.video__controls--play-pause) is not contained within the #elementId parent object.
This can be worked around by changing the cssSelector.play key to .video__play, #foo .video__controls--play-pause.

Appears to be caused on line 2385:
this.css.cs[fn] = this.options.cssSelectorAncestor + " " + cssSel;

if(cssSel) { // Checks for empty string
    this.css.jq[fn] = $(this.css.cs[fn]);

Seems like it should be simply assigning the cssSelector back, then using the ancestor during the query on line 2388.
ie:
this.css.cs[fn] = cssSel;

if(cssSel) { // Checks for empty string
    this.css.jq[fn] = $(this.options.cssSelectorAncestor).find(this.css.cs[fn]);

Not sure where you went wrong, but jPlayer already works with multiple instances. The cssSelectorAncestor specifies the unique ID and then the nested UI elements have classes that then go with the id to make a unique selector.
Did you not see this demo?
http://jplayer.org/latest/demo-03-video/

This line here should be a class selector too:
title: "#title",

@thepag Hi, its not about having multiple on the page, its allowing grouped selectors for the controls
The current code generates a selector by naïvely gluing them together like this: #ancestor .play, .pause - which for clarity, does this:
#ancestor .play,
.pause

and so only the first of the grouped selectors is scoped inside the ancestor selector. I'm suggesting doing it as two separate queries, one to find the ancestor, the other to find the controls, so you effectively end up with:
#ancestor .play,
#ancestor .pause

Check again mate. It has worked the way you propose for about 2 years or more now.
Otherwise multiple instances on a single page would not work.
Maybe provide a link to your Deb site and I can review tomorrow for the error.

Hi, I think this is just me explaining it badly, I've commented on the PR with a clearer example. I think you're focusing on the class names I used, not the code in jPlayer I highlighted? The code is just gluing two strings together, but grouped css selectors don't work that way, the comma starts a new selector and ignores that the ancestor was glued on the front. I'm not on the project that is using the plugin, I just noticed the issue while I was helping out another dev, so I don't have videos to hand to make an example I'm afraid. I can ask them to set one up somewhere public tomorrow if they have time though, but we've fixed it temporarily by changing
cssSelectorAncestor: "#foo",
cssSelector: {
    play: ".video__play, .video__controls--play-pause",

to
cssSelectorAncestor: "#foo",
cssSelector: {
    play: ".video__play, #foo .video__controls--play-pause",

(ie by manually gluing the ancestor on after each comma) for now though, so I don't know if they'll be working on the videos again soon or not.

I'll open this up again to make the pr more visible.
I'm away atm, and hope to spend some time on jplayer next week, and in the new year.
As long as it does not break backwards compatibility I see no reason why I'd not accept it.


please check support for flv.js（请检查对flv.js的支持）

When I use npm to install flv.js and import flvjs from 'flv.js' normally, and create a flvjs instance in onMounted, if the import {viteCommonjs} from' @ originjs/vite plugin commonjs' is configured in vite.config.ts; And in plugins: [viteCommonjs()], an error message will be prompted
(vite+vue3+ts 当我使用npm install flv.js后正常import flvjs from 'flv.js',并在onMounted中创建 flvjs 实例时如果在vite.config.ts中配置了import { viteCommonjs } from "@originjs/vite-plugin-commonjs";并在plugins: [viteCommonjs ()],则提示错误)
[vite] Internal server error: Failed to resolve import "vertx" from "node_modules.vite\deps\flv_js.js?v=cddc8629". Does the file exist?

Of course, you can also use the official built-in exclusion（当然也可以使用官方自带的排除）
viteCommonjs({
exclude: "node_modules/.vite/deps/flv_js.js"
})
But I still hope that the official compatibility with flv.js（但我还是希望官方兼容一下flv.js）

Your issue should be in this repo. https://github.com/originjs/vite-plugins/tree/main/packages/vite-plugin-commonjs


fix: set empty names as shortened address

What it solves
Resolves #3526
How this PR fixes it
If an empty name makes it's way to the address book reducer (be it by adding or importing), it is set to a shortened version of the given address.
A temporary dispatch sets any existing empty-named address book entries to be the same as the above.
How to test it
Remove the name name of a contact in the locally stored address book. Refresh/navigate to the address book and observe the shortened address be in place of the name.
Screenshots

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 10 (30.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 35.559%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/logic/addressBook/hooks/useAddressBookSync.ts
0
2
0.0%

src/logic/addressBook/store/reducer/index.ts
2
7
28.57%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
3550

Relevant Lines:
9026

💛  - Coveralls

Looks good to me.
The name is fine, it shows also the same in the owners' list. The name still works like any other name, it can be edited just fine, is used if another safe is created with that user as owner...


Sunless Space freezes everything-- except turrets

Sunless space freezes all enemies and makes them stop attacking, but not turrets. Probably due to them being stationary and not staggerable rather than a genuine gameplay/balance decision.

Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Use sunless space on a turret, nothing happens.

This falls a bit too much into a balance change, in my opinion.


Flash attention 2 doesn't work

Hello!
The main (a441a3f) branch of the AQLM repository does not support flash attention 2. The error occurs because QuantizedWeight does not have a weight attribute (closed issue #31). For example, the error occurs in implementations:

MistralFlashAttention2
LlamaFlashAttention2

In issue #31 a solution was proposed in the pre_quantization_dtype branch. The last commit ebe8ece tries to solve the Flash Attention 2 problem for the mistral type model, but it doesn't help. I tried doing the same thing in the main (a441a3f) branch for the llama type model, ie.
if model.config.model_type == "llama":
    layer.self_attn.config._pre_quantization_dtype = torch.float32

and I get error: RuntimeError: FlashAttention only support fp16 and bf16 data type. See the details of the error at the very bottom.
You can quickly reproduce the experiment by executing bash flash_attn2_problem.sh, where
flash_attn2_problem.sh
git clone https://github.com/Vahe1994/AQLM.git

SHA1_HEAD=$(git --git-dir=AQLM/.git rev-parse  HEAD)
echo "SHA1_HEAD ${SHA1_HEAD}"
if [[ $SHA1_HEAD != "a441a3f0ece4cbaa2a91a3421c95a8b7432e4d99" ]]
then
  echo "The main branch has been modified. Issue #128 may not be up to date."
  exit 1
fi

sed -n '1,296p' AQLM/main.py > tmp.txt
echo '            if model.config.model_type == "llama":' >> tmp.txt
echo '                layer.self_attn.config._pre_quantization_dtype = torch.float32' >> tmp.txt
sed -n '297,918p' AQLM/main.py >> tmp.txt
mv tmp.txt AQLM/main.py

pip install -r AQLM/requirements.txt
pip install flash-attn --no-build-isolation

MODEL_NAME=meta-llama/Llama-2-7b-hf
DATASET_NAME=pajama

python AQLM/main.py $MODEL_NAME $DATASET_NAME \
  --nsamples 1 \
  --model_seqlen 16 \
  --nbits_per_codebook 4 \
  --init_max_iter 1 \
  --max_epochs 1 \
  --steps_per_epoch 1 \
  --finetune_max_epochs 1 \
  --attn_implementation "flash_attention_2"

The full output of the error that occurs:
PREPARING TO FINETUNE
LlamaDecoderLayer(
  (self_attn): LlamaFlashAttention2(
    (q_proj): QuantizedLinear(
      (quantized_weight): QuantizedWeight(self.out_features=4096, self.in_features=4096, bits_per_parameter=0.5040283203125)
    )
    (k_proj): QuantizedLinear(
      (quantized_weight): QuantizedWeight(self.out_features=4096, self.in_features=4096, bits_per_parameter=0.5040283203125)
    )
    (v_proj): QuantizedLinear(
      (quantized_weight): QuantizedWeight(self.out_features=4096, self.in_features=4096, bits_per_parameter=0.5040283203125)
    )
    (o_proj): QuantizedLinear(
      (quantized_weight): QuantizedWeight(self.out_features=4096, self.in_features=4096, bits_per_parameter=0.5040283203125)
    )
    (rotary_emb): LlamaRotaryEmbedding()
  )
  (mlp): LlamaMLP(
    (gate_proj): QuantizedLinear(
      (quantized_weight): QuantizedWeight(self.out_features=11008, self.in_features=4096, bits_per_parameter=0.5039516715116279)
    )
    (up_proj): QuantizedLinear(
      (quantized_weight): QuantizedWeight(self.out_features=11008, self.in_features=4096, bits_per_parameter=0.5039516715116279)
    )
    (down_proj): QuantizedLinear(
      (quantized_weight): QuantizedWeight(self.out_features=4096, self.in_features=11008, bits_per_parameter=0.5014989098837209)
    )
    (act_fn): SiLU()
  )
  (input_layernorm): LlamaRMSNorm()
  (post_attention_layernorm): LlamaRMSNorm()
)
Fine-tuning 51584 parameters
The input hidden states seems to be silently casted in float32, this might be related to the fact you have upcasted embedding or layer norm layers in float32. We will cast back the input in torch.float32.
Traceback (most recent call last):
  File "/root/flash_attn2_problem/AQLM/main.py", line 920, in <module>
    main()
  File "/root/flash_attn2_problem/AQLM/main.py", line 894, in main
    quantize_model(model, args)
  File "/root/flash_attn2_problem/AQLM/main.py", line 59, in quantize_model
    results = quantize_aq(model, train_data, val_data, args)
  File "/root/.local/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/root/flash_attn2_problem/AQLM/main.py", line 300, in quantize_aq
    layer = finetune_groupwise(
  File "/root/.local/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/root/flash_attn2_problem/AQLM/src/finetune.py", line 143, in finetune_groupwise
    loss = _compute_mse_on_batch(layer, train_batch_iterators[0], **kwargs)
  File "/root/flash_attn2_problem/AQLM/src/finetune.py", line 268, in _compute_mse_on_batch
    outs_prediction, *_unused = layer(inps_batch, **kwargs)
  File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/.local/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 741, in forward
    hidden_states, self_attn_weights, present_key_value = self.self_attn(
  File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/.local/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/.local/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 490, in forward
    attn_output = self._flash_attention_forward(
  File "/root/.local/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 555, in _flash_attention_forward
    attn_output = flash_attn_func(
  File "/root/.local/lib/python3.10/site-packages/flash_attn/flash_attn_interface.py", line 880, in flash_attn_func
    return FlashAttnFunc.apply(
  File "/root/.local/lib/python3.10/site-packages/torch/autograd/function.py", line 574, in apply
    return super().apply(*args, **kwargs)  # type: ignore[misc]
  File "/root/.local/lib/python3.10/site-packages/flash_attn/flash_attn_interface.py", line 546, in forward
    out, q, k, v, out_padded, softmax_lse, S_dmask, rng_state = _flash_attn_forward(
  File "/root/.local/lib/python3.10/site-packages/flash_attn/flash_attn_interface.py", line 52, in _flash_attn_forward
    out, q, k, v, out_padded, softmax_lse, S_dmask, rng_state = flash_attn_cuda.fwd(
RuntimeError: FlashAttention only support fp16 and bf16 data type

Hi, I found the reason why FlashAttention2 (--attn_implementation “flash_attention_2”) is not working.
On line 296 of the main.py file, you cast the block to the float32 data type and on line  257 of the src/finetune.py file, you cast the input data for the block to the float32 data type. But FlashAttention2 only works with the bfloat16 and float16 data type. Therefore, if you use FlashAttention2, you need to cast the block to the bfloat16 (float16) data type and cast the input data for the block to the bfloat16 (float16) data type as well. This will be enough, it will start working. You should keep in mind that casting to the float16 data type may break the training because of nan. Therefore it is better to use bfloat16.


AnimeSama default source -> colorless

Source name
AnimeSama
Source language
French
Describe your suggested feature
It seems that by default AnimeSama selects the color version of the manga (I only check One Piece). Thus it is not necessarily the most up to date version of the manga.
Would it be possible to take the colored version version instead?
Other details
No
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I have updated the app to version 0.15.3.
[X] I will fill out all of the requested information in this form.

@CriosChan I hate to ping people like this, but I think this should be an easy fix ? I checked the code, I don't know Kotlin but I think I have identified the place :
https://github.com/keiyoushi/extensions-source/blob/eb1b3ebdd51135b5cb8516144d12a110ba148161/src/fr/animesama/src/eu/kanade/tachiyomi/extension/fr/animesama/AnimeSama.kt#L103

It wasn't easy, but it should work in this version, so you can choose which version (color or not) you want to read.

TYSM for your efforts!


Can't type Polish characters in editor

VSCode Version: 1.4.0 (1.4.0)
OS Version: MacOS 10.10.5 (14F1808

Steps to reproduce:

Add Polish Pro keyboard in System Preferences and try type in editor

ALT + z

ALT + c

ALT + n

Expected results:
2. ż
3. ć
4. ń

@PawelGIX Did you install any extensions? Are you perhaps pressing AltGr? What happens when you type the same key combination in the Define keybinding widget (Preferences > Keyboard Shortcuts):

You can then search for the displayed string in the default keybindings (in the left) and identify the bad keybinding.

@alexandrudima
Thanks. That was default keybindings. I just override it with empty command like this:
{
    "key": "alt+c",
    "command": "",
    "when": "editorTextFocus"
}


[mDNSResponder] Update to 878.270.2

Describe the pull request

What does your PR fix?
Fixes https://github.com/microsoft/vcpkg/issues/19966

No feature need to be tested.

Thanks for the version update!


Can't change inactive color

i cant change inactive color on tab, anyone help?

me too

Hi @Hary25 try with this method.
 mBottomBar.setFixedInactiveIconColor(ContextCompat.getColor(context, R.color.primary));
R.color.primary is declared in color.xml, also you need to know that the documentation says:
NOTE: This value is ignored if not in fixed mode.
In my project I have more that 3 tabs and I don't want to use the background color so i use
mBottomBar.setMaxFixedTabs(N);
where N > 3.

your method is not work hmm @edwinperaza

Now possible with the version 2.0. and up.
See here

Thanks @roughike  for the new version. I will try it asap.

Hi @roughike @edwinperaza
Im using 2.1.1 and inactive color don't seem to work.
Tried in xml, like inActiveColor ="#ffffff"
As well as in code:
for (int i = 0; i < mBottomBar.getTabCount(); i++) {
	mBottomBar.getTabAtPosition(i).setInActiveColor(ContextCompat.getColor(this, R.color.white));
}

But don't seem to work.


Any examples of handing SAML and custom logins?

I want to provide my users with the option of using SAML, but the default way they will be signing up and logging into my application will be a custom Rails authentication system.
When using SAML and the user authenticates, do I just then create the regular authentication token and save it to a cookie to fetch the user from my database?  I guess the first time I will somehow pull  certain information provided by SAML like firstname/lastname and email?

When using Devise and SAML, you just have two ways into the application - through the Devise sessions controller and through a SAML controller. Both use the sign_in method if everything goes well. You should be able to follow this example SAML controller and replace any Devise references with your own authentication methods. https://github.com/onelogin/ruby-saml-example/blob/master/app/controllers/saml_controller.rb


Adding Ollama provider Python API

Hello,
I'm developing a personal project that involves working with a LLM with a large context length. I have previous successfully used this wrapper to interact with my OpenAI account, but since I need to do a lot of tests I have purchased a Jetson Orin AGX to have a local machine and use local models during development to reduce the cost.
So far, I've been able to change the provider to Ollama in LWE but I don't know what should I do in order to change the provider through the Python API. In the next image you can see how I successfully change to Ollama provider in LWE but when I run my Python app I cannot see the Ollama provider:

This is the code of the application. So far I've successfully used with OpenAI with ChatGPT. The code allows me to manage the conversations and the only thing I need is to change the provider. I'm not a Python expert, so any help would be greatly appreciated.
Python code of LLM conversation management Flask app
Thanks

https://github.com/llm-workflow-engine/llm-workflow-engine/blob/main/lwe/backends/api/repl.py contains the CLI functions for the API backend
https://github.com/llm-workflow-engine/llm-workflow-engine/blob/main/lwe/core/repl.py contains the general CLI functions.
Those are the places to look for understanding how to perform any operation in Python code that you can perform in the CLI.
You'll be looking for the command_provider method, which, at least right now, lives in these lines of code: https://github.com/llm-workflow-engine/llm-workflow-engine/blob/main/lwe/backends/api/repl.py#L701-L730

Thanks for the reply.
I must have not been able to explain myself properly. I only wanted to know how the existing method "set_provider" in the ApiBackend should work:
Python code:
llmAPI = ApiBackend()
llmAPI.set_provider('chat_ollama')

Output: ProviderManager - ERROR - Provider provider_chat_ollama not found in provider_plugins.
so far when I list the available providers I only get this:
Python code:
llmAPI = ApiBackend()
providers_list = llmAPI.get_providers()
print("LLM PROVIDERS is {}".format(providers_list))

Output: LLM PROVIDERS is {'provider_chat_openai': <provider_chat_openai.ProviderChatOpenai object at 0xffffa9b89ae0>}
The terminal shell commands are great and thanks to them I've been able to change the provider and set the local model I need, but what I need is to be able to use the existing ApiBackend's set_provider method in order to access to my local model through my Python code.

You're making an assumption about how the configuration is loaded. I'm guessing you have a config.yaml, and that's where you're enabling the provider_chat_ollama. The Python module doesn't load the config file by default.
Have a look at how the CLI handles setting up the config: https://github.com/llm-workflow-engine/llm-workflow-engine/blob/main/lwe/main.py#L133-L143

You're making an assumption about how the configuration is loaded. I'm guessing you have a config.yaml, and that's where you're enabling the provider_chat_ollama. The Python module doesn't load the config file by default.

You are right. Python doesn't load that configuration. My question was how Python could load that configuration.
I'm sorry. I don't understand how I can replicate the part you show me with Python. I'll try my chances with a different solution. It's too bad because the only thing that I needed is for the existing ApiBackend's set_provider to work as intended.

from lwe.core.config import Config
config = Config()
config.load_from_file()
llmAPI = ApiBackend(config)

If you have the config in the default location that should do it.


Add more users agents of Yandex bots found in documentation

Actualize all info for Yandex bots by current (2020/11/30) information from https://yandex.com/support/webmaster/robot-workings/check-yandex-robots.xml#robot-in-logs

Thanks a lot! How to fix the CI failure?

@monperrus finally fixed ci, the tests are really good here 🙂

Thanks a lot for your contribution!


[tool_crash] FileSystemException: Exists failed, OS Error: Device is not ready., errno=21

Command
flutter doctor

Steps to Reproduce

...
...
...

Logs
FileSystemException: Exists failed, OS Error: Device is not ready., errno=21
#0      _Directory.existsSync (dart:io/directory_impl.dart:94:7)
#1      ForwardingFileSystemEntity.existsSync (package:file/src/forwarding/forwarding_file_system_entity.dart:43:33)
#2      ErrorHandlingDirectory.existsSync.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:497:22)
#3      _runSync (package:flutter_tools/src/base/error_handling_io.dart:597:14)
#4      ErrorHandlingDirectory.existsSync (package:flutter_tools/src/base/error_handling_io.dart:496:12)
#5      PubDependencies.isUpToDate (package:flutter_tools/src/flutter_cache.dart:102:57)
<asynchronous suspension>
#6      Cache.updateAll (package:flutter_tools/src/cache.dart:677:11)
<asynchronous suspension>
#7      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1293:7)
<asynchronous suspension>
#8      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1183:27)
<asynchronous suspension>
#9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#10     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#11     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>
#12     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#13     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>
#14     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.0.4, on Microsoft Windows [version 10.0.22000.194], locale fr-CI)
    • Flutter version 3.0.4 at D:\src\flutter_windows_3.0.4-stable\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 85684f9300 (2 weeks ago), 2022-06-30 13:22:47 -0700
    • Engine revision 6ba2af10bb
    • Dart version 2.17.5
    • DevTools version 2.12.2

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✗] Visual Studio - develop for Windows
    ✗ Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[✓] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[✓] VS Code (version 1.69.1)
    • VS Code at C:\Users\Octave\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.44.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [version 10.0.22000.194]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 103.0.5060.114
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 103.0.1264.49

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 2 categories.

Flutter Application Metadata
No pubspec in working directory.

possibly related to https://github.com/flutter/flutter/issues/81146

Closing this as a duplicate of https://github.com/flutter/flutter/issues/81146


Set shippingConfirmation email fails

I try to set a confirmation email address for the recipient using the Service Class:
$service = new Services();
$service->setShippingConfirmation('test@test.com');

That call throws the error Attempt to assign property "email" on null.
The problem lies in html /includes/Services.php line 621
$class->shippingConfirmation->email = $this->getShippingConfirmation();
where $class->shippingConfirmation is undefined at this moment.
Adding $class->shippingConfirmation = new StdClass; before should solve the problem.

is fixed now in the new version


mip missing from uf2 for Pico W

Hi guys, thanks for the great work. Is there any way of including mip in the distribution, it seems to be missing? "import mip" will fail using this uf2 but works with the latest standard micropython uf2 for the Pico W

The groundwork is in place to switch us over to mip, but there are breaking changes in MicroPython that have to be addressed before we can build against the latest code. It's coming!
If you're logged into GitHub you should be able to grab a pre-release build here: https://github.com/pimoroni/pimoroni-pico/actions/runs/3875338726

That's great to hear Philip. Thanks for the update and all of the great work.

Builds with mip should now be available here! https://github.com/pimoroni/pimoroni-pico/releases/tag/v1.19.12


Failed Scan

Hello!
One: thank you for this, very useful and I'm excited to use it to showcase Harbor with Concourse.
Two: I'm unable to get the scan functionality to work.  I get the following error from concourse when I try and use the scan https://harbor.gcp.59s.io/api/repositories/harbor.gcp.59s.io%2F59s%2Fadsb-sync/tags/latest.
I have the resource configured as follows in my pipeline.yml:
- name: docker-image
  type: harbor-image
  source:
    username: ((harbor.username))
    password: ((harbor.password))
    skip_pull: true
    repository: harbor.gcp.59s.io:443/59s/adsb-sync
    harbor_host: harbor.gcp.59s.io
    harbor_image: harbor.gcp.59s.io/59s/adsb-sync
    harbor_trigger_scan: true
    harbor_scan_thresholds:  [{"severity": 1,"count": 20},{"severity": 2,"count": 20}]
    notary_enable: false
    insecure_registries: ["harbor.gcp.59s.io:443"]

I tried to recreate the call at the cli on my own machine and got back a 404. When I look at the swagger docs for harbor it looks like the url for initiating a scan is different. /repositories/{repo_name}/tags/{tag}/scan
Anyway, not sure what to do but would love to get this working.

And I looked at https://github.com/virtmerlin/harbor-image-resource/blob/master/assets/out#L328 and it was just the error message that didn't have the right url. The curl command there looks good.


this_will_be_replaced_by_sandstorm needs to be replaced even when escaped in JS strings

From @kentonv on December 2, 2015 2:36
http://ninjility.com/, which appears to be hosted on Sandstorm, has this blob at the top:
    <script type="text/javascript">
        window._wpemojiSettings = {"baseUrl":"http:\/\/s.w.org\/images\/core\/emoji\/72x72\/","ext":".png","source":{"concatemoji":"http:\/\/this_will_be_replaced_by_sandstorm:10000\/wp-includes\/js\/wp-emoji-release.min.js?ver=4.3.2-alpha"}};
        !function(a,b,c){function d(a){var c=b.createElement("canvas"),d=c.getContext&&c.getContext("2d");return d&&d.fillText?(d.textBaseline="top",d.font="600 32px Arial","flag"===a?(d.fillText(String.fromCharCode(55356,56812,55356,56807),0,0),c.toDataURL().length>3e3):(d.fillText(String.fromCharCode(55357,56835),0,0),0!==d.getImageData(16,16,1,1).data[0])):!1}function e(a){var c=b.createElement("script");c.src=a,c.type="text/javascript",b.getElementsByTagName("head")[0].appendChild(c)}var f,g;c.supports={simple:d("simple"),flag:d("flag")},c.DOMReady=!1,c.readyCallback=function(){c.DOMReady=!0},c.supports.simple&&c.supports.flag||(g=function(){c.readyCallback()},b.addEventListener?(b.addEventListener("DOMContentLoaded",g,!1),a.addEventListener("load",g,!1)):(a.attachEvent("onload",g),b.attachEvent("onreadystatechange",function(){"complete"===b.readyState&&c.readyCallback()})),f=c.source||{},f.concatemoji?e(f.concatemoji):f.wpemoji&&f.twemoji&&(e(f.twemoji),e(f.wpemoji)))}(window,document,window._wpemojiSettings);
    </script>

As you can see there is a case of this_will_be_replaced_by_sandstorm that failed to be replaced in there, because the '/'s are for some reason escaped (who knew '/' was a valid escape sequence in Javascript?).
This actually leads to an error reported on the JS console when loading the site.
Copied from original issue: dwrensha/wordpress-sandstorm#15

From @mrdomino on March 1, 2016 17:36
+1

From @dwrensha on November 9, 2016 14:48
The reason we see the strange-looking host this_will_be_replaced_by_sandstorm:10000 has do with our integration with Sandstorm's web publishing feature. When you click the "Rebuild Public Site" button, we run a recursive wget to locally grab the contents of your site and copy them into /var/www/. This relies on some configuration in /etc/hosts. The idea is that after grabbing the content, we can do a find-and-replace to convert these URLs to an appropriate form for external consumption. Unfortunately, the find-and-replace does not catch all cases.

From @dwrensha on November 9, 2016 14:26
Eek, I just noticed that this error shows up in the default configuration. That is, if I create a new WordPress grain, click "Rebuild Public Site", and then visit the site, I see the following error in my browser console:
Blocked loading mixed active content “http://this_will_be_replaced_by_sandstorm:10000/wp-includes/js/wp-emoji-release.min.js?ver=4.4.2”[Learn More]

From @JamborJan on January 16, 2017 6:59
Is there any chance that we get this bug solved? As far as I can see it has impact on other issues and possible solutions there, see issue #22. Thanks a lot!
Update: as far as I can see, my point is caused by this line in the mentioned recursive wget script. I guess the script should replace the url in any case no matter if it is a https or http link.

Did you find a solution for this?  I have the same problem.

@cyberzenk99
I have not looked at this yet, because I'm mostly concentrating to maintaining Wekan https://wekan.github.io .
Currently @JamborJan maintains this WordPress for Sandstorm.
I presume fixing this would require somebody looking at this or related repos source code, and find where to remove that text etc. Contributions welcome :)

I have some experimental stuff but nothing 100% works yet. I’ll create a branche for this bug to share my current status of the bug fix.

Thank you for the quick reply Lauri (@xet7 ).   I understand there's a lot to maintain.  With just a few bugs fixed, Sandstorm would be an amazing platform.  By the way, you're doing a great job with Wekan too.
And thank you @JamborJan .  If you could give me any kind of fix, whether it's editing some code manually, or any other ideas, I would be so thankful.  Even if it's experimental, I can do it the hard way until the final fix is in place.  Any suggestions, I would be appreciative.

@xet7  @JamborJan   As an added note, I would love to give a contribution, just let me know where to send it.  :-)

@cyberzenk99
For Wekan related contributions, you can contact me by email<EMAIL_ADDRESS>

@JamborJan should post his email too.  :-)

Hey @cyberzenk99,
I was able to implement a quick and dirty fix for the java script issue (see this for details). It's not nice but it solves the issue with the replacement of this_will_be_replaced_by_sandstorm.
But in my test cases I now get another error message in the browsers console:
wp-emoji-release.min.js:1 Failed to load resource: the server responded with a status of 404 (Not Found)

Not sure if this is a Sandstorm related issue or not. Can anyone please test with the beta release and let me know the result? Please be aware that this is not much tested. So please don't upgrade any production critical stuff.
For any contributions, questions or whatever: feel free to contact me :-)

Are you sure wp-emoji-release.min.js didn't somehow get dropped from your beta package? It doesn't look like you submitted any chances to the sandstorm-files.list file.

You are right, it seems so. But as I did test with vagrant-spk dev including building pages any missing files should have been added to the file list. Plus the file is in the list and has been there before:
wordpress-read-only/wp-includes/js/wp-emoji-release.min.js

Thanks for your thoughts and input anyway. It always helps to see issues from another perspective.

This still seems to be an issue. Will investigate further.


doxygen stuff

Addresses https://github.com/WSU-CEG-6110-4410/ADAD-Craft/issues/107

Looks good! Good work


VisUI Won't Compile with GWT

When I run "./gradlew html:build", the GWT compilation fails:
:html:compileGwt
Compiling module tech.otter.merchant.GdxDefinition
   Tracing compile failure path for type 'com.kotcrab.vis.ui.util.async.AsyncTask'
      [ERROR] Errors in 'com/kotcrab/vis/ui/util/async/AsyncTask.java'
         [ERROR] Line 46: The constructor Thread(new Runnable(){}, String) is undefined
         [ERROR] Line 51: The method start() is undefined for the type Thread
   [ERROR] Aborting compile due to errors in some input files

I believe this is because GWT doesn't support the Thread class. I am using VisUI version 1.3.0-SNAPSHOT to maintain compatibility with libGDX 1.9.5. After doing some reading, it sounds like this was probably caused by #174
Would it be feasible to use something that is GWT compatible or move this back to the editor package?

Sorry for the delay, I removed it from GWT compilation. Btw you will probably have to use next libgdx snapshot as GWT is broken in 1.9.5 anyways: http://www.badlogicgames.com/forum/viewtopic.php?f=11&t=24379


fix issue #5

Motivation:
fix issue #5
Modification:

upgrade  the client version to 1.4.1;
lookout-ext-os degrade the level  of warning logs;
add metrics which  record data report result status ;

Result:
Fixes #5

Pull Request Test Coverage Report for Build 10

9 of 57 (15.79%)  changed or added relevant lines in 10 files are covered.
18 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.7%) to 64.722%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client/lookout-reg-server/src/main/java/com/alipay/lookout/remote/report/SchedulerPoller.java
0
1
0.0%

client/lookout-ext-os/src/main/java/com/alipay/lookout/os/linux/DiskUsageMetricsImporter.java
0
1
0.0%

client/lookout-ext-os/src/main/java/com/alipay/lookout/os/linux/SystemLoadMetricsImporter.java
0
1
0.0%

client/lookout-ext-os/src/main/java/com/alipay/lookout/os/linux/NetTrafficMetricsImporter.java
0
1
0.0%

client/lookout-ext-os/src/main/java/com/alipay/lookout/os/linux/CpuUsageMetricsImporter.java
0
2
0.0%

client/lookout-ext-os/src/main/java/com/alipay/lookout/os/linux/IOStatsMetricsImporter.java
0
4
0.0%

client/lookout-reg-server/src/main/java/com/alipay/lookout/remote/report/HttpObserver.java
5
43
11.63%

Files with Coverage Reduction
New Missed Lines
%

client/lookout-reg-server/src/main/java/com/alipay/lookout/remote/report/support/ReportDecider.java
1
66.67%

client/lookout-reg-server/src/main/java/com/alipay/lookout/remote/report/HttpObserver.java
17
17.04%

Totals

Change from base Build 9:
-0.7%

Covered Lines:
1919

Relevant Lines:
2965

💛  - Coveralls


Generate aliases

[x] I agree that my contribution may be licensed either under MIT or any version of LGPL license.

Fixes #757

@cameronwhite Do you have remarks regarding the alias support? You can see it's features in the tests here.

I think that looks fine to me.
I was wondering at first whether it should be more of a "strong typedef" that doesn't allow implicit conversions from the primitive type, if the user should mainly be using the alias type, but the implicit conversions are consistent with the C API. I don't have a strong opinion on this

I made it implicit as constant values can't be of an alias type as creating an instance makes them not constant. So I fall back to their base type for rendering as constant.
The implicit cast allows to use the constant primitive value as an parameter to a function which requires the alias struct.
I wonder if it is to much "magic" as I normally don't like those implicit things happening.

That's a good point about constants, I can see that being necessary for pattern matching if there are any aliased types with constants that are useful to compare against

There is currently only GST_CLOCK_TIME_NONE.
It seems to work in a switch statement but not a switch expression. Perhaps there is a way forward to solve this. Will test.


Block bruteforce & timer on 2fa

Block bruteforce attempt based on IP address. Max n-try and wait for x-minutes before retry.

After successful username and password, limit time to x-seconds and x-retry 2fa, before username and password are required again.

This will take of some serverload and a improve UI for users not using 2fa.
@juancarlospaco, this way we still prevent bruteforcing the 2fa due to a timelimit and max retry. Would that be a okay solution?

2FA you only have 30 Seconds to Brute Force, with using >10 chars password, makes it almost impossible,
Is this needed :grey_question:
I think that with 2FA,Recaptcha and Firejail we are pretty much covered on security,
theres other stuff to improve, like the info on the demo page, design, themes, etc.
Lets add code were is needed...

I have no need for the timeout, it was because of our discussion at PR #39 where you linked to the CPU in case of bruteforcing. I would just like to move the 2FA to a "next page", so it's not shown to users, who is not using 2FA.
Yep, it's low priority compared to the topics you listed.

Oh, thats completely different :grey_exclamation: I misunderstood.
Feel free to move 2FA to an intermediate page. :+1:


Boundary cons

Alterations:

Added boundary_conditions class which user can specify which bcs for each marked boundary are implemented through flux.py.
Defaults each boundary to solid wall conditions unless specified.
Added test of well balanced conditions given bcs of manual setup of solid wall conditions with river bc specified.
Added test of all defaulted boundaries

To do before merge:

[ ] Create an outflow option with info RE Onno's email, changing river to inflow

@colinjcotter
see commit: https://github.com/firedrakeproject/flooddrake/pull/15/commits/1f9f44afd25ca497ce12ff2a969f6e36b731abbc
one can now use any mesh markers / tags - not just a square mesh

@colinjcotter How's this looking w.r.t. merging?


Weight issues

When anyone in my lobby picks up an item From LethalThings they forever weigh 100 pounds until they relog.

I've had this happen with a lot of mods after v56; the casino mod and UsualScrap also had this issue, so I'd bet it's the same thing.

ChillaxScrap also suffer from the same bug. This happens on my side if I have any other mod that modify the weight of items (weight reduction mostly). 0 weight Items can randomly be super heavy or have negative weight


high training cost

Hi Edward,
While I was searching for a new research idea, I've found your model and it was interesting in that it can learn code- and visit-level representation from EHRs simultaneously.
Using your model, I'm trying to learn embeddings that can represent measurements other than medical codes. However, the training cost seems quite high (around 150~250) and it doesn't converge(or go below 1) just like other models. I've found that the others have the same range of cost, but I wonder what was the final cost at the end of training.
Is it natural for this model to have this kind of high cost at the end of training? or is something wrong with a setting? I've adjusted the parameters in the model, but cost 170 was the best I could get.
I would appreciate your help.

Hi,
Thanks for taking interest in med2vec.
To be honest, I don't remember what the loss values looked like, since it's been almost 3 years.
But when you say that you want to learn the embeddings for measurements, rather than medical codes, I'm not sure how you are training med2vec.
Med2vec has a skip-gram loss which is basically co-occurrence counting, but I'm not sure how you would do that with continuous measurements.
Maybe you could provide more info?
Best,
Ed

Thank you for your comment.
I've sent you an<EMAIL_ADDRESS>about more detailed information.


Fixing example for virtual wan supported security provider api.

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[x] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

Automation for azure-sdk-for-js

Can one of the admins verify this patch?

@MikhailTryakhov please advise on what target branch should @nalinrajgupta use

@dsgouda Since we just merged Network into master I think it should target master, but change should  be applied to 2018-12-01, not 2018-11-01
@nalinrajgupta please sync your branch with master and re-apply your changes to API version 2018-12-01

@veronicagg Running into linter diff failures for a couple of networking related PRs, who can we contact to take a look into it?

@dsgouda the error indicates an out of memory exception, we should investigate further. For now you can try running this one locally and see if it passes. This change seems small, so it should not be related. I'll open an issue on the linter. Thanks!

Thanks @veronicagg I will verify this locally

@veronicagg  I can run the linter locally and did not see any obvious issues but I don't have the merge permissions, could you do the honors


MangaDogs update on Nine manga extension

Source information
NineManga
Source language
Spanish
Steps to reproduce
When I want to read a chapter of any manga, an image appears that says "urgent! Please update" but instead of being the extension it is Manga dogs.
Expected behavior
Read any manga
Actual behavior
An image asking to update Manga Dogs

Mihon/Tachiyomi version
0.16.5
Android version
Android 14
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Seems like an issue with the site  rather than an issue witht the extension.

Tried opening jn browser to see how it shows there, the same. On clicking the series from homepage of ninemanag it redirects to another website.

https://en.ninemanga.com/chapter/Welcome%20to%20Dungeon%20Hotel/15952260-1-1.html
This one is the url that is being fetched by Tachiyomi.
https://en.ninemanga.com/chapter/Welcome%20to%20Dungeon%20Hotel/15952260.html
This one works fine when opening from the browser.

Created a draft pull with the fix https://github.com/keiyoushi/extensions-source/pull/2570#issue-2263221931
Will need to add other checklist item before it cqn be ready for publish.

Created a draft pull with the fix #2570 (comment)
I dont have android studio to do one of the checks in checklist. If someone can quickly check and progess the draft pull, that would be great.

it's good


How to include environment variable in a snippet definition

Discussed in https://github.com/L3MON4D3/LuaSnip/discussions/475

Originally posted by jimcornmell July  5, 2022
Allow the ability to include any environment variable in a snippet definition json file eg $HOME:
For example:
{
  "A Description": {
    "prefix": "home",
    "body": [
         "$HOME",
    ],
....
}

Will it make sense to use a prefix for that so we avoid collisions with the built-in variables? I'm thinking of something like $ENV_HOME.

Oh yes, ENV_SOMEVAR seems alright.
Also not without precedent, TM_SOMETHING.

Yes, I think the same way, giving that they are not lsp-standard it could be less confusing just to inject them in the setup and maybe resolve unknown environment variables (those not defined by the user or default ones) to "" (not sure what is vscode doing in that case).

like
ls.config.set_config{
...
extra_environ={
    HOME = os.getenv'HOME',
    MY_RANDOM = function(){return math.random();}  ## lazy env that is executed once per snippet
}
}

What do you think of
ls.add_env_resolver(fn(string) -> (table of string|string))

we could maintain a list of those fns, call all in turn on Environ.__index, accept on non-nil return.

like
ls.config.set_config{
...
   extra_environ={
       HOME = os.getenv'HOME',
       MY_RANDOM = function(){return math.random();}  -- lazy env that is executed once per snippet
   }
}

+ very easy to use+readable, metatable should be no problem as well, so that's nice
- a plugin that provides snippets to luasnip might have their own env-variables, adding those in setup wouldn't be as nice I think (could be ugly, especially if multiple do so)

like
ls.config.set_config{
...
   extra_environ={
       HOME = os.getenv'HOME',
       MY_RANDOM = function(){return math.random();}  -- lazy env that is executed once per snippet
   }
}

+ very easy to use+readable, metatable should be no problem as well, so that's nice - a plugin that provides snippets to luasnip might have their own env-variables, adding those in setup wouldn't be as nice I think (could be ugly, especially if multiple do so)

I was thinking about the same :smile:, although given that each snippet has his own Environ instance, could not we add an extra_envirom there too?

So authors of snippets sets don't pollute the global Environ :thinking:

I was thinking about the same :smile:

:D
extra_environ should work for lua-snippets, but vscode/snipmate ones would miss out then.
(+some stuff really makes more sense to define on a global level, like ENV_SOMEVAR)

So authors of snippets sets don't pollute the global Environ ￼

Hmm yeah, that's a point.. it might be enough to put some big warning to use "namespaces" (I agree though, a safe mechanism would be better).
(snippet namespaces? give additional properties based on namespace? not sure if there's anything beside this that could be used for)

wtyt about this one?
-- No namespace, for regular users

ls.extend_env {
       HOME = os.getenv'HOME',
       RANDOM = function(){return math.random();}  -- lazy env that is executed once per snippet
}

-- With namespace for snippets-sets devs
-- has to be used as $MY_HOME

ls.extend_env("MY", {
       HOME = os.getenv'HOME',
       RANDOM = function(){return math.random();}  -- lazy env that is executed once per snippet
})
-- Maybe?
ls.remove_env_namespace("MY")

Oh, using a table is more readable, that's nicer than a function with a big ifelse.
Not completely convinced by the optional namespace, plugins could also just call the function meant for personal use.
Making the namespace required could be annoying too, I can see that... Maybe disable the required namespace by passing a special value, like THIS_IS_FOR_MY_PERSONAL_USE_AND_NOT_A_PLUGIN? xD

True, in any case, users can just use a very short namespace for "local" ones and a longer one for shared snippets sets (to avoid collitions)

Should be alright :+1:
Are you interested in adding this?

I can work on it this weekend.

Sweet, looking forward to it

Just explicit steps for this:
Add
ls.env_namespace("SYS", {vars = os.getenv})

to your config, then you can access any environment-variable as $SYS_VARNAME, eg $SYS_HOME


Frontend: dummy Pages Created

Description
This PR will create the dummy pages according to the issue  Fixes #61
How to test?
Run the project and make sure that components have empty values.
Check the issue 61 and be sure that all components are created.
Checklist

[x] I have performed a self-review of my own code
[x] I have followed the name conventions for CSS Classnames and filenames, Components names and filenames, Style filenames, if you are in doubt check the the project README.MD and here https://github.com/HackYourFuture-CPH/curriculum/blob/master/review/review-checklist.md
[x] I have commented my code, particularly in hard-to-understand areas, if you code was simple enough mark the box anyway
[x] I have made corresponding changes to the documentation, if you code was simple enough mark the box anyway
[x] This PR is ready to be merged and not breaking any other functionality

@simonreddy2001 thank you, Simon.
Please, be assigned before PR, if the classmates don`t make some task at some time.

Hi Simon, CartpageContainer, CategorypageContainer, ConfirmationpageContainer, FavoritespageContainer and all another const would be better if Page will be capital. For example: CartPageContainer, CategoryPageContainer, ConfirmationPageContainer.

Thank you Olena.. for feedback..

Good job, thank you Simon. I have updated Description and How to test?. It was unfinished.

Super... Tak @OlenaKasian


CAMEL-18576: camel-base - Escape a placeholder

Fix for https://issues.apache.org/jira/browse/CAMEL-18576
Motivation
The elastic search queries can contain double braces which are seen as a property placeholder by Camel, we need a way to escape it.
Modifications:

Propose to use the backslash character to escape double braces

Thanks I will look closer tomorrow.


idm authentication config

Add the ability to pull / push the IDM authentication.json

It's pretty unusual to touch the IDM authentication config in cloud, but we could add it as a command. In the meantime you could use the raw command to fetch it by path?

Yep, the raw command can fetch it nicely. There can be cases like custom org admin type or OAuth2 apps accessing custom endpoints via an internal role, instead of a service account. In both cases, the authentication.json needs to be updated - although generally, we'd prefer not to touch it.

Somehow I missed that you can do fr-config-push raw  - so that's should be sufficient.