Any example about writing and deploying a substrate smart contract?

We tried a lot but failed to deploy our smart contract on substrate. I think any document or examples will be very helpful.
Much appreciated!

Please stop spamming the same issue. You got your answer in the last issue.

Did you really check that issue? You just duplicate my last issue to a issue which will be never fixed.
Is this your style you guys to resolve the issue?

I think Basti is right, duplication of the same issue is not helpful for us. I marked it as duplicate for the sake of linkness, so whenever we get documentation we could either post it in one place or crawl to every linked issue. I'm very sorry to hear that you think that that issue will never be fixed.
I'm gonna go ahead an at least lock this issue, and may be even delete this. However, you are welcome to ask further questions in the original issue.


Failed installation at Phase 2 - Missing dependenciy newuidmap

Not sure what's happening, here's my setup.log
Any help appreciated.

I'm running Zorin OS, which is based off of Ubuntu. In case anyone has this issue in the future, I fixed this by running:
sudo apt install uidmap


Bias-corrected VAR running issue on eigenvalues non-convergence

https://github.com/dake-li/lp_var_simul/blob/b7bd3ad0d10ff5d18c517351a2bb2e5c009fe923/Estimation_Routines/VAR_CorrectBias.m#L67

I agree that the signs in lines 13 and 62 seem off. We should check the code on a DGP where we know the Pope correction in closed form, such as an AR(1), or a VAR(1) with independent series.
It's still not clear why the eigenvalue can't be computed, though. Let's see what the server output looks like.

I agree that the signs in lines 13 and 62 seem incorrect. I also checked the Kilian (1998) paper again and could not find a problem with our implementation. In terms of next steps, I thus agree with Mikkel that we should (i) wait for the server output and (ii) check the code for a case where the bias correction is known.

could not find a problem with our implementation

But it seems likely that the sign is a mistake, so we should just confirm this quickly with a special case.


[Bug] Calls to IOTA network significantly slow down in debug

Bug description
Examples and tests seems to massively slow down when Rust is compiled in debug on the dev branch, even more so than would be expected from debug mode. This does not occur on the main branch.
Language version

Language: Rust
Perhaps noticeable in WASM?

Update: no slowdown was noticeable with WASM due to the iota-client not computing proof-of-work locally when the wasm feature is enabled, which changed in https://github.com/iotaledger/iota.rs/pull/480.

IOTA Identity version

Version (version number, commit, or branch): dev branch, commit cb4bd724e51d85eee4be3e067a0159f4cce1abd3 at time of writing

Steps To reproduce the bug
Explain how the maintainer can reproduce the bug.

Run any example in debug mode. E.g.

cargo run --example account_basic

This was resolved incidentally in PR #292 / commit 4e4bd0ecc535c370677719b538a1cf13df3916b7 by @l1h3r.
Specifically, the iota-client builder was previously performing proof-of-work locally for each message added to the tangle rather than having one of the nodes on the network do it. The above commit changes this default to remote proof-of-work.
const DEFAULT_LOCAL_POW: bool = false;

[...]

impl ClientBuilder {
  /// Creates a new `ClientBuilder`.
  pub fn new() -> Self {
    Self {
      nodeset: false,
      network: Default::default(),
      builder: iota_client::ClientBuilder::new().with_local_pow(DEFAULT_LOCAL_POW),
    }
  }

On the dev branch the examples consistently take a few seconds (~5 instead of >25) and do not have excessive CPU usage, which stays below 10% on my local machine compared to 100% for the entire duration before.
This issue may be closed.

Local PoW is very slow in debug mode, but that's not a bug

Yup, there was just some confusion as to why the examples noticeably slowed down between the dev and main branches here.
It was due to a change in identity-iota: previously the local proof-of-work option in iota-client was turned off by default since the "wasm" feature was always enabled, which changed recently.
Before (main branch):
[features]
# default = ["iota-client/async", "iota-client/futures", "iota-client/tokio"]
default = ["iota-client/wasm"]

# Enables Web Assembly support
wasm = ["iota-client/wasm"]

After commit 2ebba4ba46e78b80bf8fe96eb707f09af2fcd3f3:
[features]
default = ["async"]

# Enables async runtime support (Tokio)
async = ["iota-client/async"]

# Enables Web Assembly support
wasm = ["iota-client/wasm"]


making optional logs upload and slack notification

making optional logs upload and slack notification
Description

errors in build for missing env vars
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ ] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #5366 (6ead0ee) into master (f717619) will decrease coverage by 52.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #5366       +/-   ##
==========================================
- Coverage   57.37%   4.52%   -52.86%
==========================================
  Files         233     240        +7
  Lines       14143   14407      +264
  Branches      597     601        +4
==========================================
- Hits         8115     652     -7463
- Misses       6028   13755     +7727

Impacted Files
Coverage Δ

.../main/scala/org/apache/openwhisk/core/WarmUp.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/scala/org/apache/openwhisk/spi/SpiLoader.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...scala/org/apache/openwhisk/common/time/Clock.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...scala/org/apache/openwhisk/core/FeatureFlags.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...scala/org/apache/openwhisk/http/CorsSettings.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...la/org/apache/openwhisk/http/BasicRasService.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...pache/openwhisk/http/LenientSprayJsonSupport.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ache/openwhisk/core/database/DocumentFactory.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/openwhisk/core/controller/Namespaces.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...che/openwhisk/core/invoker/LogStoreCollector.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 155 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Seems S3-related tests are failing.

Yes true. AWS S3 tests are NOW failing, and I guess this is happening simply because someone removed the AWS_BUCKET secret that was there before. I used the bucket to upload logs there and I am sure he was not happy.

Hi I made a few changes, mostly I made the s3 test skipped it the bucket is not available, fixed a compilation error and now the build works. Can you merge?

forget about this - I misunderstood a few things - restarting the patch


Drive health

Resolves https://github.com/sillsdev/machine/issues/141 and https://github.com/sillsdev/serval/issues/243.

This change is

samples/EchoTranslationEngine/EchoTranslationEngine.csproj line 16 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I don't think this is needed.

Correct.  Removed.

samples/EchoTranslationEngine/TranslationEngineServiceV1.cs line 3 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
You should update csharpier so that this get formatted correctly.

It's the new net8.0 primary constructor.

src/Serval.ApiServer/Startup.cs line 45 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I don't think that the null-forgiving operators are needed here.

net8 was complaining about it - is there a better way?  Should I use a ?  or a if statement?  If it is null, I would want to crash out of the code.

src/Serval.ApiServer/Startup.cs line 45 at r1 (raw file):
Previously, johnml1135 (John Lambert) wrote…
net8 was complaining about it - is there a better way?  Should I use a ?  or a if statement?  If it is null, I would want to crash out of the code.

I see what I was doing - I was confusing the null forgiving operator in CSharp (!) with the not-null assertion operator in Kotlin (!!).  I will change them to the ?? throw syntax.

src/Serval.ApiServer/Startup.cs line 47 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
Please avoid using the null-forgiving operator.

Done

src/Serval.ApiServer/Startup.cs line 216 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
There is an extra semicolon here.

fixed.

src/Serval.ApiServer/Controllers/StatusController.cs line 50 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I would call this something else. Maybe ping.

ping is ok.

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 108 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
This should be snake case.

It should be ok - we recast as integers, not as strings.

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 109 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
Do we really need this?

It is in the standard health check - and there is no particular reason not to include it.

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 110 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
What is this used for?

This is used for each of the sub-categories of checks:
    "SmtTransfer": {
      "status": "Healthy",
      "duration": "00:00:01.2095377",
      "description": "SmtTransfer",
      "exception": "System.Exception",
      "data": {
        "S3 Bucket": "Healthy: The S3 bucket is available",
        "Mongo": "Healthy: ",
        "Hangfire": "Healthy: ",
        "SMT Engine Storage Capacity": "Healthy: ",
        "ClearML Health Check": "Healthy: ClearML is available"
      }
    },

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 111 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I would rename this to error or more generally description.  Remember this is defining an API for engines. An engine could be implemented on any tech stack. We don't need to, nor should we match the C# health check models exactly. All we really need is status and description.

The exception literally captures exceptions - which sometimes can be used for debugging.  They would be on the logs - but again, if it is already included, why not?

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 163 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
These should be all uppercase in order to be consistent with the other enums.

Done

src/Serval.Shared/Configuration/IServalBuilderExtensions.cs line 46 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
Please avoid using the null-forgiving operator. Throw an exception if necessary.

Done

src/Serval.Shared/Controllers/ServiceUnavailableExceptionFilter.cs line 21 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
We already pass the exception to the Log method. This might be redundant.

I think it was needed.  I changed it because net8.0 complained about passing the exception.  I needed to switch the order of exception and message to make the function happy - done.

src/Serval.Shared/Utils/WriteHealthCheckResponse.cs line 5 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
This is only used in Serval.ApiServer, so you can move it there.

I used it more places before - you are right.

src/Serval.Shared/Contracts/HealthReportDto.cs line  at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
Why have these DTOs been moved?

I thought we needed them more places - for sharing with machine - when I was doing the rest endpoints.  I'll move them back.

I ran formatting - I'll check the files again.  They changed more than I was expecting and I will revert at least the ldml.

samples/EchoTranslationEngine/TranslationEngineServiceV1.cs line 3 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I know. The line is going over the 120 character limit without wrapping. If you reformat this file with CSharpier, it should fix it.

csharpier doesn't handle primary constructors in C#12 yet - it can only handle C#11.  I am reverting all the new primary constructors to preserve csharpier working.

src/Serval.ApiServer/Startup.cs line 45 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I would do this instead:
var dataFileOptions = new DataFileOptions();
Configuration.GetSection(DataFileOptions.Key).Bind(dataFileOptions);

What about for connection strings?  Shouldn't we fail if one is not actively defined?

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 108 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
Yes, but it is inconsistent with the rest of the API definition.

updated  - it works magically, making the C# classes names as camel case already.

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 109 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
It is in the standard health check for .NET, but this API could be implemented by some other tech stack. I would like to make this as easy as possible to be implemented.

Is making it optional good enough?

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 109 at r1 (raw file):
Previously, johnml1135 (John Lambert) wrote…
Is making it optional good enough?

It isn't needed at all - the Health check function creates it's own duration independent of any data passed.  I'll remove it.

src/Serval.Grpc/Protos/serval/translation/v1/engine.proto line 111 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
By calling it exception, we are assuming a particular language construct. One that some languages might not have. This API is intended to be language-agnostic.

Error is fine enough.  We can map it to exception at the C# Health Check level.

src/Serval.ApiServer/Startup.cs line 50 at r3 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
You missed removing the null-forgiving operator here.

Fixed

samples/EchoTranslationEngine/EchoTranslationEngine.csproj line 16 at r4 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
Why was this added back?

By accident - I removed the commit that did all the mis-formatting of files and must have removed that change from it.

src/Serval.Shared/Configuration/IServalBuilderExtensions.cs line 48 at r3 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I generally try to avoid throwing Exception. InvalidOperationException would be fine in this case. Also, please avoid the ?? throw syntax. I don't think it is as clear as a normal if statement.

Done.


I have the error No such file or directory when launching a connection with redpwn

I have a jail using python
#!/usr/local/bin/python

import random, time

x = random.randint(80, 120)

random.seed(time.time())

ll = [random.randrange(0,4294967294) for _ in range(624)]

togive = {"given":ll}

print(togive)

and this is the Dockerfile
FROM python:slim AS app
RUN pip install --no-cache-dir randcrack

FROM pwn.red/jail
COPY --from=app / /srv

ENV JAIL_POW=0 JAIL_PORT=1337 JAIL_TIME=20

COPY app.py /srv/app/run

WORKDIR /srv/app

RUN chmod +x run

and this is the compose file
version: '3.7'

services:
  predict:
    container_name: PyREDICTOR
    restart: unless-stopped
    build:
      context: ./challenge
      dockerfile: Dockerfile
    ports:
      - "5000:1337"
    privileged: true

but i have this error when launching a connection
PyREDICTOR  | [I][2024-05-11T12:50:57+0000] Setting 'cpu.max' to '100000 1000000'
PyREDICTOR  | [I][2024-05-11T12:50:57+0000] Executing '/app/run' for '[::ffff:<IP_ADDRESS>]:42914'
PyREDICTOR  | [E][2024-05-11T12:50:57+0000][1] newProc():225 execve('/app/run') failed: No such file or directory
PyREDICTOR  | [F][2024-05-11T12:50:57+0000][1] runChild():483 Launching child process failed
PyREDICTOR  | [W][2024-05-11T12:50:57+0000][1] runChild():503 Received error message from the child process before it has been executed
PyREDICTOR  | [I][2024-05-11T12:50:57+0000] pid=12 ([::ffff:<IP_ADDRESS>]:42914) exited with status: 255, (PIDs left: 0)

I'm unable to reproduce your issue with the code provided.

#!/usr/local/bin/python

import random, time

x = random.randint(80, 120)

random.seed(time.time())

ll = [random.randrange(0,4294967294) for _ in range(624)]

togive = {"given":ll}

print(togive)
for i in range(x):
    try:
        inpp = input('> Gimme a number: ')
        if(len(inpp) > 11):
            print('What are you trying to do? Bye!')
            exit(0)
        inp = int(inpp)
    except:
        print('Wrong! try again')
        print('Exiting...')
        exit(0)
    ran = random.randrange(0,<PHONE_NUMBER>)
    if inp == ran:
        print('Correct! continue')
    else:
        print('Wrong! try again')
        print('Exiting...')
        exit(0)
print('done')
exit(0)

here is the full code snippet

I've tested the code you provided and also could not reproduce the error you're seeing. Perhaps try rebuilding the image?

still the same
none@none:~/doc$ sudo docker compose build
WARN[0000] /home/weber/PyREDICT/compose.yml: `version` is obsolete
[+] Building 0.7s (13/13) FINISHED                                                                               docker:default
 => [predict internal] load build definition from Dockerfile                                                               0.0s
 => => transferring dockerfile: 265B                                                                                0.0s
 => [predict internal] load metadata for pwn.red/jail:latest                                                               0.5s
 => [predict internal] load metadata for docker.io/library/python:slim                                                     0.4s
 => [predict internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [predict app 1/2] FROM docker.io/library/python:slim@sha256:2be8daddbb82756f7d1f2c7ece706aadcb284bf6ab6d769ea695cc3ed  0.0s
 => [predict internal] load build context                                                                                0.0s
 => => transferring context: 28B                                                                                0.0s
 => [predict stage-1 1/5] FROM pwn.red/jail:latest@sha256:ee52ad5fd6cfed7fd8ea30b09792a6656045dd015f9bef4edbbfa2c6e672c28  0.0s
 => CACHED [predict app 2/2] RUN pip install --no-cache-dir randcrack                                                      0.0s
 => CACHED [predict stage-1 2/5] COPY --from=app / /srv                                                                    0.0s
 => CACHED [predict stage-1 3/5] COPY app.py /srv/app/run                                                                  0.0s
 => CACHED [predict stage-1 4/5] WORKDIR /srv/app                                                                          0.0s
 => CACHED [predict stage-1 5/5] RUN chmod +x run                                                                          0.0s
 => [predict] exporting to image                                                                                0.0s
 => => exporting layers                                                                                0.0s
 => => writing image sha256:6238b829024ca11cc6a8b53d29bc8785b2b8038304144ddaf76a228978c8f41e                               0.0s
 => => naming to docker.io/library/pyredict-predict                                                                        0.0s
none@none:~/doc$ sudo docker compose up
WARN[0000] /home/weber/PyREDICT/compose.yml: `version` is obsolete
[+] Running 1/1
 ✔ Container PyREDICTOR  Recreated                                                                                0.4s
Attaching to PyREDICTOR
PyREDICTOR  | [I][2024-05-12T20:08:05+0000] Mode: LISTEN_TCP
PyREDICTOR  | [I][2024-05-12T20:08:05+0000] Jail parameters: hostname:'app', chroot:'', process:'/app/run', bind:[::]:1337, max_conns:0, max_conns_per_ip:0, time_limit:20, personality:0, daemonize:false, clone_newnet:true, clone_newuser:true, clone_newns:true, clone_newpid:true, clone_newipc:true, clone_newuts:true, clone_newcgroup:true, clone_newtime:false, keep_caps:false, disable_no_new_privs:false, max_cpus:0
PyREDICTOR  | [I][2024-05-12T20:08:05+0000] Mount: '/' flags:MS_RDONLY type:'tmpfs' options:'' dir:true
PyREDICTOR  | [I][2024-05-12T20:08:05+0000] Mount: '/srv' -> '/' flags:MS_RDONLY|MS_NOSUID|MS_NODEV|MS_BIND|MS_REC|MS_PRIVATE type:'' options:'' dir:true
PyREDICTOR  | [I][2024-05-12T20:08:05+0000] Mount: '/proc' flags:MS_RDONLY|MS_NOSUID|MS_NODEV|MS_NOEXEC type:'proc' options:'' dir:true
PyREDICTOR  | [I][2024-05-12T20:08:05+0000] Uid map: inside_uid:1000 outside_uid:1000 count:1 newuidmap:false
PyREDICTOR  | [I][2024-05-12T20:08:05+0000] Gid map: inside_gid:1000 outside_gid:1000 count:1 newgidmap:false
PyREDICTOR  | [I][2024-05-12T20:08:05+0000] Listening on [::]:1337
PyREDICTOR  | [I][2024-05-12T20:08:29+0000] New connection from: [::ffff:<IP_ADDRESS>]:42730 on: [::ffff:<IP_ADDRESS>]:1337
PyREDICTOR  | [I][2024-05-12T20:08:29+0000] Setting 'memory.max' to '5242880'
PyREDICTOR  | [I][2024-05-12T20:08:29+0000] Setting 'memory.swap.max' to '0'
PyREDICTOR  | [I][2024-05-12T20:08:29+0000] Setting 'pids.max' to '5'
PyREDICTOR  | [I][2024-05-12T20:08:29+0000] Setting 'cpu.max' to '100000 1000000'
PyREDICTOR  | [I][2024-05-12T20:08:29+0000] Executing '/app/run' for '[::ffff:<IP_ADDRESS>]:42730'
PyREDICTOR  | [E][2024-05-12T20:08:29+0000][1] newProc():225 execve('/app/run') failed: No such file or directory
PyREDICTOR  | [W][2024-05-12T20:08:29+0000][1] runChild():503 Received error message from the child process before it has been executed
PyREDICTOR  | [F][2024-05-12T20:08:29+0000][1] runChild():483 Launching child process failed
PyREDICTOR  | [I][2024-05-12T20:08:29+0000] pid=11 ([::ffff:<IP_ADDRESS>]:42730) exited with status: 255, (PIDs left: 0)

this is the exact code
#!/usr/local/bin/python

import random, time

x = random.randint(80, 120)

random.seed(time.time())

ll = [random.randrange(0,4294967294) for _ in range(624)]

togive = {"given":ll}

print(togive)
for i in range(x):
    try:
        inpp = input('> Gimme a number: ')
        if(len(inpp) > 11):
            print('What are you trying to do? Bye!')
            exit(0)
        inp = int(inpp)
    except:
        print('Wrong! try again')
        print('Exiting...')
        exit(0)
    ran = random.randrange(0,<PHONE_NUMBER>)
    if inp == ran:
        print('Correct! continue')
    else:
        print('Wrong! try again')
        print('Exiting...')
        exit(0)

print('Here is your flag: flag')
exit(0)

and this is the Dockerfile
FROM python:slim AS app
RUN pip install --no-cache-dir randcrack

FROM pwn.red/jail
COPY --from=app / /srv

ENV JAIL_POW=0 JAIL_PORT=1337 JAIL_TIME=20

COPY app.py /srv/app/run

WORKDIR /srv/app

RUN chmod +x run

this is the terminal to interact with it
none@none:~$ nc <IP_ADDRESS> 5000
^C

it haven't responded so I killed the NC command.


used "sh" language for syntax highlighting

Those code blocks are not pure shell scripts, because of the $, so the highlight won't work correctly. We just prefer not to have any highlighting of these kinds of simple commands.

10-4, thanks.


Cannot import name 'hostappid'

After doing fetchcord --update (as the distro I'm using was added) and trying to launch it, I get this and it won't launch:
Traceback (most recent call last):
  File "/home/alex/.local/bin/fetchcord", line 7, in <module>
    from fetch_cord.run_rpc import main
  File "/home/alex/.local/lib/python3.6/site-packages/fetch_cord/run_rpc.py", line 12, in <module>
    from fetch_cord.testing import desktopid, termappid, hostappid
ImportError: cannot import name 'hostappid'

commit: 60016d33f535ae03b8fd8db4d169c178b8567ec3 should fix that issue, please try doing  --update again.

It worked, thanks.


Why is there a difference in manifest here and on add-on site

I'm on pcxFirefox which is 64-bit. Some of the last few add-ons I've tried to install won't install and give a "...appears to be corrupt" message. Same for this add-on from the Firefox site.
I noticed a difference and copied the manifest.json from this version into the Firefox version and it installed perfectly and is working fine. This is an amazing add-on! Thank you to the developer.
The lines missing in the Firefox version manifest appears to be the following:
"applications": {
"gecko": {
"id"<EMAIL_ADDRESS>}
Can someone please explain why there is a difference and why it is missing in the Firefox version?
Update: When I tried downloading from here and zipping the contents and renaming as a xpi file, I get the same error message as before, i.e., can't install.
Another Update: I think this is a problem in the current version of Firefox I am using (version 50.1.0). I believe it has been fixed in Firefox 51.
According to this website:
https://8pecxstudios.com/Forums/viewtopic.php?f=5&t=1663&start=10

Hey thanks for liking my addon. I didn't include an ID because I thought that I didn't need it. So the AMO website just auto generated an ID for me. Now that it appears that I need to put that auto generated ID in the manifest to suppress some warnings with the addon validator. This will come in the next version.


Fix #1285 Spring Cloud activates unexpected profiles when some conditions met

https://github.com/spring-cloud/spring-cloud-commons/issues/1285

I recently upgraded to version 3.1.4 and encountered an issue with retrieving active profiles from different PropertySources.

I recently upgraded to version 3.1.4 and encountered an issue with retrieving profiles from different PropertySources. I want to know if this is considered a bug, as previous versions did not have this problem.

Not quite aware of your problem, coz I only found this issue comparing 3.1.7 and 3.1.6. You can check if your issue persists in the latest version and possibly provide a POC.

You will need to change the PR to be against the 3.1.x branch and also fix the checkstyle errors.

Code style issues fixed and rebased to 3.1.x.


Resource Links

Describe the bug
API Reference link redirects to google.com
To Reproduce
Steps to reproduce the behavior:

Go to Demo Section
Click on API Reference
See error

Expected behavior
It should redirect to the actual API reference.
Screenshots

Additional context
Kindly provide the link to the actual reference of the API.
@Akshatchaube01, I request you to assign me this issue under GSSoC'24.

Kindly provide the resource link for API reference to be attached.

I am going to change this section soon, so there is no need for it.


get to UTF-8, otherwise insync-headless barfs on some error messages

Per subject of the commit, I needed UTF-8 to read some error messages from insync headless.
I didn't want to install locales-all, but generating the en_US.UTF-8 kept giving me errors.  If you can figure out a better way, but all means...

Another githubber requested some locale functions on one of my other images. I'll take a peek into looking for a way to embed Locale Changes into my base (Alpine,Debian,Ubuntu) images in coming weeks. I mostly avoided it due to size but perhaps I can come up with a hack.

Adding the following to my docker-compose.yml resolved this for me without needing to make any changes to the Dockerfile:
environment:
    LANG: 'C.UTF-8'
    LANGUAGE: 'C.UTF-8'
    LC_ALL: 'C.UTF-8'


Combine all of Wagtail’s global and layout stylesheets into one + refactorings (#8419)

Addresses #8419. This is a rather large PR – I split it in two commits, which I’d recommend viewing separately, so the actual style changes are more apparent. I’d recommend rebase-merging this with the two commits, so we retain as much history as possible even though the files have changed locations.
This combines almost all of our stylesheets to be loaded via core.css only. The reasoning for this is that having a single stylesheet makes it much simpler to know whether a given component is compatible with the rest of Wagtail’s CSS. If CSS is loaded the same everywhere, then we know components will display the same everywhere.
There are still per-layout overrides, which I’ve had to refactor so they still only apply to their intended layout. This refactoring can’t be done in a fully safe way (as we’re having to increase the specificity of those overrides), but I’ve done my best to keep those changes to a minimum.
Stylesheets left as-is

Userbar – should remain separate
Styleguide – should remain separate
Draftail – not sure, doesn’t really need to be combined with the rest
Typed table block – same
StreamField – not sure, might combine it later, or not, as part of page editor style changes

Additional refactorings
Beyond the scope of #8419, I took some time to review how we actually use jQuery UI and found out / confirmed it’s only for tagit and the "manual page sorting". In particular it seems the images and icons that ship with jQuery UI do nothing in Wagtail – both widgets look the same before and after (as long as we keep a background). So I’ve removed the images / icons.
Next steps
After this is merged, I would like to deprecate the insert_editor_css hook. In this PR, it does the exact same thing as insert_global_admin_css. I don’t want to do this in the same PR because the changes needed are very different (almost entirely in Python + documentation).
Testing
I will highlight which parts have changed the most with code comments.

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?
~[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?~
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[x] Please list the exact browser and operating system versions you tested: Chrome, Safari, Firefox on macOS 12.3
[x] Please list which assistive technologies [^3] you tested:  N/A

[ ] For new features: Has the documentation been updated accordingly?

Thank you @Scotchester 🤘 a few more changes were needed for those new image listing styles, which I’ve gone ahead with.


fixed improper use of aria-expanded by moving to link instead of cont…

…ainer
See https://w3c.github.io/aria/#aria-expanded

This is highly needed, to have the correct structure, especially considering the recent accessibility requirements.


[gmime] Update to version 3.2.3.

Thank you for cleaning up the port so much when you updated it.  This looks good to me!


No XP generated

It seems that when you die the game refreshed so fast that no xp is generated.
Browser: Chrome
Bot version: Auto updater
Cell name: Changed from default
Facebook: Connected

For me Facebook still works, and I believe it works because my computer is really fast.

Not sure what you mean with that respond - the thing is, that when you die the game loads too fast. I never passes the "gain xp" step.

That's not actually the reason for the xp not updating since the xp is done server side. The error happens when the script is first loaded into the page.
Try using the bot.user.js and launcher.user.js instead it might fix the problem.

Same problem with the bot and launcher scripts.

Hm - it might work after all. I will return later.

I can confirm it works with the bot.user.js and launcher.user.js.

Oh sweet! I'll have to figure out how to fix it for the loader.


[Preview] HHH-13925 Upgrade to Gradle 6.3

https://hibernate.atlassian.net/browse/HHH-13925
This is a preview: I ran the build locally to some extent, but I'd like a build from the CI. Also, I still need to run other builds, especially on JDK13 and 14, and to test publishing.

Everything seems in order:

The CI build for JDK8 succeeded.
The CI build for JDK11 failed on this PR, but that's a transient failure caused by problems transfering Maven artifacts. I relaunched the build.
The CI build for JDK13 succeeded: https://ci.hibernate.org/blue/organizations/jenkins/hibernate-orm-personal-yoann-jdk13/detail/hibernate-orm-personal-yoann-jdk13/6/pipeline
The CI build for JDK14 succeeded: https://ci.hibernate.org/blue/organizations/jenkins/hibernate-orm-personal-yoann-jdk14/detail/hibernate-orm-personal-yoann-jdk14/2/pipeline
I managed to publish to my local Maven repository successfully
I managed to publish artifacts to bintray successfully: https://bintray.com/yrodiere/hibernate-release-testing/hibernate-orm/999.4.0.Final
I tested the hibernate-gradle-plugin with an application using Gradle 4.4: it still works, despite the plugin being compiled against Gradle 6 APIs.

Merged, thanks!


Remove calculations from OFFSET/FETCH

Instead using calculated values for OFFSET/FETCH, use direct integer values.  The current format:
OFFSET (@PageNo - 1) * @PageSize ROWS
FETCH NEXT @PageSize ROWS ONLY

Proposed format:
OFFSET @Offset ROWS
FETCH NEXT @Fetch ROWS ONLY

Not all code using OffsetFetch will be based on page numbers and page size values.  Because the formula is hard-coded, code that has start & length values will have to reverse calculate the page number and page size values so they can be converted back to the value as used in SQL.
Ideally the OffsetFetch<TFilter, TResult> (string, string) method would handle the new use and a new function for PageSizeNumber<TFilter, TResult> (string, string) could replace the existing calls.  This would be a breaking change but it might be early enough in its use that it wouldn't cause too much pain.
Thoughts?

What about introducing an additional optional parameter on OffsetFetch(string, string, bool computePage = true), so that the computation could be turned off. This would avoid a major version increment.

This issue is resolved in commit: e43d069c6ae4962127cba3910b9425423201ca7d. Review commit notes for implementation. If we agree on those changes I will queue up a release.


AdamWarmup not working in transformers

Hi,
I tried to wrap AdamWarmup in TFRobertaModel, which use tensoflow 2.0 and tf keras.
Even if I set environment variable
TF_KERAS = 1
I got the following error:
Could not interpret optimizer identifier:', <keras_bert.optimizers.warmup.AdamWarmup object at 0x000002891F6D3E80>.
Could that be because in the 'backend.py',
if TF_KERAS:
    from tensorflow.python import keras

Should be changed to?
if TF_KERAS:
    from tensorflow import keras

I wrote a customized callbacks in keras to bypass the issue. Could you help review if that mimic the behavior of AdamWarmup? Thanks!


Tokenizer fails with empty strings or \n

Describe the bug
I'm trying to write a converter for TfIdfVectorizer when analyser=='char'. I use the regular expression to split a sentance into characters with '.' (does not work) but '[^\n]' works. With the first one, I get the following error:
INVALID_ARGUMENT : Input shape must have either [C] or [B,C] dimensions where C > 0 and B > 0
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows
ONNX Runtime installed from (source or binary): master branch from github
ONNX Runtime version: master branch from github
Python version: 3.7

To Reproduce

clone https://github.com/xadupre/sklearn-onnx/tree/tfc
run test test_model_tfidf_vectorizer11_dot in tests/test_SklearnTfidfVectorizerConverterChar.py

Expected behavior
onnxruntime should return outputs.

@xadupre I will look into the issue promptly. There are few things I want you to note.

If you want to split something into individual characters the Tokenizer spec provides for a char tokenization mode whichis enabled when you supply a single empty string as separator.
Per spec Tokenizer accepts UTF-8 strings for both input and attributes. This applies as well to StringNormalizer and TfIdfVectorizer. So you python script wants to s.encode(encoding='utf-8') before feeding the input. Not sure if there is a difference for latin characters in python. I know it'd produce bytes object, not python unicode string.

I have submitted https://github.com/Microsoft/onnxruntime/pull/617 to address this.

I tried but that does not fix the issue. StringNormalizer can handle empty strings, Tokenizer too, but not TfidfVectorizer. It should handle empty strings and returns an empty result.

Another issue maybe introduced with changes #617 is the following.
corpus = numpy.array([
        'This is the first document.',
        'This document is the second document.',
        ]).reshape((2, 1))
vect = TfidfVectorizer(ngram_range=(1, 1), norm=None,
                       analyzer='word', token_pattern=".{1,2}")
vect.fit(corpus.ravel())
pred = vect.transform(corpus.ravel())
model_onnx = convert_sklearn(vect, 'TfidfVectorizer',
                             [('input', StringTensorType([1, 1]))])

The ONNX pipeline include a couple of nodes which produce the following outputs
-------------- normalized (StringNormalizer)
['this is the first document.']
-------------- tokenized (Tokenizer)
[['th' 'is' ' i' 's ' 'th' 'e ' 'fi' 'rs' 't ' 'do' 'cu' 'me' 'nt' '.']]
-------------- flattened (Flatten)
[['th' 'is' ' i' 's ' 'th' 'e ' 'fi' 'rs' 't ' 'do' 'cu' 'me' 'nt' '.']]
-------------- tfidfvectorizer (TfidfVectorizer with regex = '.{1,2}'
2019-03-14 18:08:44.0281985 [E:onnxruntime:InferenceSession, inference_session.cc:541 onnxruntime::InferenceSession::Impl::Initialize] Exception during initialization: onnxruntime\onnxruntime\core\providers\cpu\nn\tfidfvectorizer.cc:371 onnxruntime::TfIdfVectorizer::TfIdfVectorizer (before_insert + ngrams) == impl_->str_set_.size() was false. poll_strings duplicate 1-grams detected

I found two bugs while converting a TfIdfVectorizer. The first one comes from scikit-learn which creates ambiguities when trying to retrieve n-grams if the tokens contain spaces. The other bug comes from the tokenizer which loses one dimension if the expression to tokenize is empty. The code is the following:
corpus = np.array([
        'This is the first document.',
        'This document is the second document.',
        'And this is the third one.',
        ' ',
        ]).reshape((4, 1))
vect = TfidfVectorizer(ngram_range=(1, 2), norm=None)
vect.fit(corpus.ravel())
pred = vect.transform(corpus.ravel())
from skl2onnx import convert_sklearn
from skl2onnx.common.data_types import StringTensorType

model_onnx = convert_sklearn(vect, 'TfidfVectorizer',
                             [('input', StringTensorType([1, 1]))])

I then checked the intermediate outputs for two strings, an non-empty string and an empty string (differencs in yellow). When the string is empty, the function return an null dimension.
  if (max_tokens == 0) {
    output_dims.push_back(0);
    TensorShape output_shape(output_dims);
    ctx->Output(0, output_shape);
    return Status::OK();
  }

But if mark_ is true, we would expect to have that string surrounded by start/end markers (this code is after the first one).
  if (mark_) {
    max_tokens += 2;  // Start/end markers as separate tokens
  }

I compared the results for the intermediate nodes (graph is below)
with input: [array(['And this is the third one.'], dtype='<U37')]
-> node 'normalized'
['and this is the third one.']
-> node 'tokenized'
[['and' 'this' 'is' 'the' 'third' 'one']]
-> node 'flattened'
[['and' 'this' 'is' 'the' 'third' 'one']]
-> node 'variable'
[[1.9162908 1.9162908 0.        0.        0.        0.        1.2231436
  1.2231436 1.9162908 0.        0.        1.2231436 0.        0.
  1.9162908 1.9162908 1.9162908 1.2231436 0.        1.5108256]]

with input: [array([' '], dtype='<U37')]
-> node 'normalized'
[' ']
-> node 'tokenized'
2019-03-15 17:37:20.3345248 [W:onnxruntime:Default, bfc_arena.cc:201 onnxruntime::BFCArena::AllocateRawInternal] tried to allocate 0 bytes
[]
-> node 'flattened'
2019-03-15 17:37:20.3356089 [W:onnxruntime:Default, bfc_arena.cc:201 onnxruntime::BFCArena::AllocateRawInternal] tried to allocate 0 bytes
2019-03-15 17:37:20.3356310 [W:onnxruntime:Default, bfc_arena.cc:201 onnxruntime::BFCArena::AllocateRawInternal] tried to allocate 0 bytes
[]
-> node 'variable'

for the following pipeline...

@xadupre I think we discussed this sufficiently with Wei-Sheng. Pls, talk to me if you feel there still an issue with regards to empty strings.

Supporting empty input shape in TfidfVectorizer looks reasonable to me. For example, if we tokenized a document (i.e., a very long string) into a [C]-tensor, but unfortunately remove all the elements in StringNormalizer, and finally TfidfVectorizer gets [0]-tensor. What should be the TFIDF-representation of this document? I'd guess that representation is a [D]-vector where all elements are zeros, because the count of every vocabulary is 0.


Source error when installing with fisher

Env
Fish
❯ fish --version
fish, version 3.1.2

OS/distribution
macOS 11.1

Fisher output
fisher install version 4.1.0
Fetching https://codeload.github.com/rafaelrinaldi/pure/tar.gz/HEAD
Installing rafaelrinaldi/pure
           /Users/joe/.config/fish/functions/_pure_check_for_new_release.fish
           /Users/joe/.config/fish/functions/_pure_format_time.fish
           /Users/joe/.config/fish/functions/_pure_get_prompt_symbol.fish
           /Users/joe/.config/fish/functions/_pure_parse_directory.fish
           /Users/joe/.config/fish/functions/_pure_parse_git_branch.fish
           /Users/joe/.config/fish/functions/_pure_place_iterm2_prompt_mark.fish
           /Users/joe/.config/fish/functions/_pure_print_prompt.fish
           /Users/joe/.config/fish/functions/_pure_prompt.fish
           /Users/joe/.config/fish/functions/_pure_prompt_beginning.fish
           /Users/joe/.config/fish/functions/_pure_prompt_command_duration.fish
           /Users/joe/.config/fish/functions/_pure_prompt_current_folder.fish
           /Users/joe/.config/fish/functions/_pure_prompt_ending.fish
           /Users/joe/.config/fish/functions/_pure_prompt_first_line.fish
           /Users/joe/.config/fish/functions/_pure_prompt_git.fish
           /Users/joe/.config/fish/functions/_pure_prompt_git_branch.fish
           /Users/joe/.config/fish/functions/_pure_prompt_git_dirty.fish
           /Users/joe/.config/fish/functions/_pure_prompt_git_pending_commits.fish
           /Users/joe/.config/fish/functions/_pure_prompt_git_stash.fish
           /Users/joe/.config/fish/functions/_pure_prompt_jobs.fish
           /Users/joe/.config/fish/functions/_pure_prompt_new_line.fish
           /Users/joe/.config/fish/functions/_pure_prompt_ssh.fish
           /Users/joe/.config/fish/functions/_pure_prompt_ssh_host.fish
           /Users/joe/.config/fish/functions/_pure_prompt_ssh_separator.fish
           /Users/joe/.config/fish/functions/_pure_prompt_ssh_user.fish
           /Users/joe/.config/fish/functions/_pure_prompt_symbol.fish
           /Users/joe/.config/fish/functions/_pure_prompt_system_time.fish
           /Users/joe/.config/fish/functions/_pure_prompt_vimode.fish
           /Users/joe/.config/fish/functions/_pure_prompt_virtualenv.fish
           /Users/joe/.config/fish/functions/_pure_set_color.fish
           /Users/joe/.config/fish/functions/_pure_set_default.fish
           /Users/joe/.config/fish/functions/_pure_string_width.fish
           /Users/joe/.config/fish/functions/_pure_uninstall.fish
           /Users/joe/.config/fish/functions/fish_greeting.fish
           /Users/joe/.config/fish/functions/fish_mode_prompt.fish
           /Users/joe/.config/fish/functions/fish_prompt.fish
           /Users/joe/.config/fish/functions/fish_title.fish
           /Users/joe/.config/fish/conf.d/_pure_init.fish
           /Users/joe/.config/fish/conf.d/pure.fish
source: Error encountered while sourcing file './functions/_pure_uninstall.fish':
source: No such file or directory
Installed 1 plugin/s

I just ran into the same issue when installing manually. It appears to be due to this commit @edouard-lopez

Seems unrelated to any manager. Having the same problem after updating using fundle.
Removed the line as a workaround.

Apologies,
I will have a look tonight, in the meantime you can install the previous version:
fisher install<EMAIL_ADDRESS>

Fixed in v3.3.3:
fisher install rafaelrinaldi/pure

@edouard-lopez Why extract the event handler https://github.com/rafaelrinaldi/pure/commit/3090c67b600cad81d62f0a51e18cc46422277a87#diff-0728e51023b9a9b70caf9b61545c9996c007bd0cd70479a4b6be89d5069bc0c2R10? Seems unnecessary and roundabout. I'd just keep it in the snippet.

@edouard-lopez Why extract the event handler https://github.com/rafaelrinaldi/pure/commit/3090c67b600cad81d62f0a51e18cc46422277a87#diff-0728e51023b9a9b70caf9b61545c9996c007bd0cd70479a4b6be89d5069bc0c2R10? Seems unnecessary and roundabout. I'd just keep it in the snippet and put code in the uninstaller to erase the function, but I guess it's up to you.

I prefer pure_uninstall, etc., as originally introduced in my PR (https://github.com/rafaelrinaldi/pure/pull/243). Extracting the function into functions/ and hardcode-sourcing it in conf.d/_pure_init.fish is fragile and would break if Fisher changes the cp order (unlikely, though) or Pure users customize their $fisher_path (more likely). The problem is assuming Pure is installed to $__fish_config_dir.
The safe and generic way to avoid these caveats is to handle events as originally introduced in https://github.com/rafaelrinaldi/pure/commit/ba468eb821f140645dc44dddde8d0811c304cf2a.

Thanks @jorgebucaran, I will see what feedback I get on this to prioritize this modification

+1 on @jorgebucaran's suggestion. I know fundle is not supported but it underlines that the assumed install directory may not be correct (~/.config/fish/fundle/rafaelrinaldi/pure in my case).

https://github.com/rafaelrinaldi/pure/pull/251


postgres to alpine and runs on to latest ubuntu

appending -alpine to postgres in various doc-files and .github/workflows/elixir.yml (ref #1569)
switching to ubuntu-latest instead of ubuntu-18.04 for .github/workflows/elixir.yml

I've reverted the commit that bumps the ubuntu image to latest. It does not seem to be compatible with the setup-elixir action and I'd prefer to use 20.04 instead of latest. Will change this in a separate commit.

@adriankumpf looks like the ubuntu-latest was reverted to ubuntu-18.04 but the postgres:13-alpine part for postgres was not applied.
Want me to create a separate PR for that? :)

Hi @tobiasehlert, unfortunately the alpine images don't work on the Raspberry Pi because its package
sources contains an old version of libseccomp, which causes the time to be at 1970. I had to revert it in 6cfb2a24cf2a04d75f941f35c3950d9055cad919.

Hi @tobiasehlert, unfortunately the alpine images don't work on the Raspberry Pi because its package
sources contains an old version of libseccomp, which causes the time to be at 1970. I had to revert it in 6cfb2a2.

Ah alright! I thought it was only the builder part that broke things.. but oke :)


Bad quotes in __done_humanize_duration?

When a long-running process finishes, these messages show up in the terminal:
(standard_in) 1: syntax error
(standard_in) 1: syntax error
test: Missing argument at index 2
test: Missing argument at index 2

Also, the title of the notification is just "Done in", not e.g. "Done in 20s".
I think the problem is on lines 159 and 160 of done.fish, which read:
set -l seconds (math --scale=0 "$milliseconds/1000" % 60)
set -l minutes (math --scale=0 "$milliseconds/60000" % 60)

Running one of those lines in a terminal fails.  I think Fish doesn't like the %.  If I move the quotes to this...
set -l seconds (math --scale=0 "$milliseconds/1000 % 60")
set -l minutes (math --scale=0 "$milliseconds/60000 % 60")

...then the error messages go away, and the notification shows the runtime.

Hi @aselker, thank you for reporting this. Would you be up for submitting a pull request?

Which version of fish are you using? I can't reproduce this issue.

I updated to Fish 3.2.2, from version 2.something, and the problem seems to be gone.

OK, so it seems to depend on the version

Closing this as it seems to only be fixed in newer version of fish. Thanks for reporting!


[Snyk] Upgrade yarn from 1.22.19 to 1.22.20

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade yarn from 1.22.19 to 1.22.20.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2023-11-14.

Release notes

Package name: yarn

1.22.20 - 2023-11-14WarningThis release is missing a couple of artifacts (the .msi/.rpm/.deb/.asc files); we're working on fixing this.

Important: Punycode is now embed within the bundle, as it has been deprecated by Node.js and will be removed in a future version.

A message will be displayed when Yarn 1.22 notices that the local project has a package.json file referencing a non-1.x Yarn release via the packageManager field. The message will explain that the project is intended to be used with Corepack.

The yarn-error.log files won't be generated anymore, as we don't process non-critical 1.x bug reports (we however process all bugs reported on https://github.com/yarnpkg/berry; we just released the 4.0.2 release there).

The yarn set version x.y.z command will now install the exact x.y.z version (prior to this change it used to first install the latest version, and only in a second step would it downgrade to x.y.z; this was causing issues when we bump the minimal Node.js version we support, as running yarn set version 3.6.4 wouldn't work on Node 16).

Prevents crashes when reading from an empty .yarnrc.yml file.

1.22.19 - 2022-06-08
Adds compatibility with WebAuthn on the npm registry

    from yarn GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

already changed on the recent PR merge. Hence Closing this PR


Theme stuck?

Mac OS Catalina 10.15.4 (19E287). I seem to be stuck on the first theme I selected (Material Oceanic) even after removing the theme, restarting slack and confirming default coloring, and then installing a new theme.

True, I need to write a doc about it but I have other priorities at the moment
The apply method has actually completely changed, even though there are still options to the console they will not do anything anymore as it has been replaced by the Loop over installed themes button in the title bar.

It looks like no matter which theme is chosen during the apply theme, the Material Oceanic is always applied.
MacOS Catalina 10.15.4
Slack: 4.5.1

following up; there is no Loop over installed themes button in the title bar.

Facing the same problem, the Material Oceanic is the only theme applied.
Mac OS: 10.15.5 (19F101)
Slack Version 4.6.0

@frackleton launch slack with developer mode and it should be there
export SLACK_DEVELOPER_MENU=true; open -a /Applications/Slack.app

Same problem...

Please make sure you have the latest update

Recently installed and is not working, output is:
? Please select an command Apply theme
Removing theme...

Successfully removed!

current theme is now Material Oceanic
Adding Theme Code to Slack...

Slack successfully themed. Please restart Slack to apply the theme.

Can't choose...

https://github.com/mallowigi/slack-themes#loop-over-selected-themes
--
Elior
On 6 Jul 2020, 19:19 +0300, Diego Ulloa<EMAIL_ADDRESS>wrote:

Recently installed and is not working, output is:
? Please select an command Apply theme
Removing theme...
Successfully removed!
current theme is now Material Oceanic
Adding Theme Code to Slack...
Slack successfully themed. Please restart Slack to apply the theme.
Can't choose...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


OAuth parse token array-type claims

closes #15

Added logic where we check if more than one claim with the same type exists, and if yes, group these under the same dictionary entry in claims using LINQ.
Created a second JWT for testing, which contains an array-type claim. Also added the JWK version of the public key of this into JwkKeys.json, and the PEM version of both the private and public key into README in case the JWT needs to be modified in the future for testing.
Added a test case for the handling of array-type claims.

Summary by CodeRabbit

New Features

Introduced support for handling array-type claims in token parsing, enhancing user role and permission management.
Added a testing file with a Bearer token example for improved validation of array-type claims.
Expanded JSON Web Keys (JWKs) structure to support multiple keys, facilitating better key management and security.

Bug Fixes

Improved handling of claims within the token parsing method to avoid duplicates and accommodate multiple values.

Documentation

Updated README to include examples of key material for testing JWTs, enhancing clarity for developers.

Grouped this change under 1.2.0 along with changes made in #18


Unable to resolve action , repository not found, part 2

I have everything set up correctly.
2 repos
patricia-hartnett/gitHubActions_test1 - it will call the Java code in the other repo
patricia-hartnett/gitHubActions_testQuarkus5 - it has the Quarkiverse Java code in it.
patricia-hartnett/gitHubActions_testQuarkus5  looks like:

In patricia-hartnett/gitHubActions_test1 I have an action whose workflow looks like:

When this is run I get the repo not found error:

I have tried pointing to both my own patricia-hartnett/gitHubActions_testQuarkus5 and gsmet/my-github-action. I get the same error message.
One of the things I noticed was that in the actions.yml file in patricia-hartnett/gitHubActions_testQuarkus5 , there is a line mentioning 'github=https://maven.pkg.github.com/patricia-hartnett/gitHubActions_testQuarkus5/....

I tried going to this URL but I got a 404.
In fairness I also tried going to https://maven.pkg.github.com/gsmet/my-github-action/ (as mentioned here:https://github.com/gsmet/my-github-action/blob/9101a475431954e586525a565a3dd8ef75e48b7b/action.yml#L14) and still got a 404.
I have tried with and without a step doing actions/checkout@v3 in the action in patricia-hartnett/gitHubActions_test1  (where it attempts to call the Java code in the other repo), but I am still getting the same error message.
Both of my repos are set up as private but accessible from repositories owned by the user 'patricia-hartnett'.
Am I doing something wrong ?

(See bottom of the message for what I think is your problem)
It's normal you have a 404, GitHub doesn't generate an index page for the Maven repositories. What you should have is something like this:

What looks suspicious to me is your uses: github.com:patricia-hartnett.... Where does the github.com: prefix come from?
See https://github.com/gsmet/github-action-playground-target/blob/main/.github/workflows/run-action.yml#L24 for how I use my action.

Thank you !
That worked .
Sorry for putting in the extra bit in front of the run action uses.
I am now getting:

Yeah so that's probably because you changed the groupId and artifact after the action creation. You need to update your action.yml to target the proper jar (in the JBang call).

Great thanks will do that now and get back to you.

Sorry I don't see a problem with what I have .
I am only using the default settings.
On the root page of my repo (where the Quarkiverse Java code lives) it has:

And in the actions.yml it has:

Same groupId and artifactId ?

yes
I mean if you look at the Build and Push action in the repo where the Java code lives

So you can see at line 1878, the built jar is going to:
https://maven.pkg.github.com/patricia-hartnett/gitHubActions_testQuarkus5/org/acme/my-github-action/999-SNAPSHOT/my-github-action-999-20230118.180945-1.jar

ok will add --verbose, one min

so line 14 in the actions.yml becomes
run: ~/.jbang/bin/jbang --repos 'github=https://maven.pkg.github.com/patricia-hartnett/gitHubActions_testQuarkus5/' --repos 'mavencentral' org.acme:my-github-action:999-SNAPSHOT --verbose
?

add it just after jbang

thanks will do that now

I personally suspect it will be a permission issue due to the repository being private. Not exactly sure how permissions work for Maven repositories of private projects.

I think it might be that alright
I can see:
Caused by: org.eclipse.aether.resolution.ArtifactResolutionException: Could not transfer artifact org.acme:my-github-action:pom:999-SNAPSHOT from/to github (https://maven.pkg.github.com/patricia-hartnett/gitHubActions_testQuarkus5/): status code: 401, reason phrase: Unauthorized (401)

i can send you screengrabs of the full stack trace if you like ?

Not really needed.
Can you try making the action repository public to see if that's what causes the issue?

Sorry was at lunch
Will try to use public repositories.
Will get back to you soon.


license updates

fixed gosqlmock path in .rat-excludes
added license missing in traffic_portal files

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/146/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-whatchanged/1/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-whatchanged/17/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/148/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/107/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/109/


Guardian of Tirisfal quest cannot be completed without destroying Dalaran

Book of medivh require dalaran's capital destruction, while Guardian of Tirisfal requires book to be completed.

Could we make it so that Dalaran Heroes only need to meet the level requirements?

@TracyTheBoy Yep, that would seem to make sense.

I'm surprised it's not this already


CalenStyle not working on Phonegap

Was testing CalenStyle demo pages, it works flawlessly on web browsers. Great job. Out of curiosity tried running it on phonegap, but all I see is a blank page. Is there a different configuration for phonegap?

I'm using the mobile demo, do I have to use the regular one?

In Phonegap Desktop App, Socket.io is used for logging console messages and this error was occurring while logging large object. When I commented console.log() which were used to log large objects, this error disappeared.


tls: Remove tls certificates from reonciled secret (PROJQUAY-2606)

Prevent old tls certs from being included in new secret generated by controller

@jonathankingfc don't forget to cherry pick this into 3.6


Adding sync push-image workflow form github-config

Summary

This PR syncs the push-image.yml file from github-config, as this PR has now been merged https://github.com/paketo-buildpacks/github-config/pull/900/files
Use Cases

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@mhdawson


fixed MV load bug, added mv create button, handled error case when po…

Description
fixed MV load bug
added mv create button
handled error case when post async query fails
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@sumukhswamy Can you please update the PR title and make it shorter, these title directly go in to the release notes. You can use description for more details.


Tool for converting LVL files to strings

This issue is not related to anything else, and does not require any knowledge of PICO-8, Lua, or LaserTank for that matter. This is simple IO and parsing of a binary file.
I'm thinking a simple command line tool (Python?) that takes a .LVL file as input, and outputs a text file with the level data as strings.
.LVL files are just a series of levels, one after another, where each level has the following format:

256 bytes: Tile data
31 bytes: Title in ASCII
256 bytes: Hint in ASCII
31 bytes: Author in ASCII
1 byte: Difficulty
1 byte: Always a NUL byte?

The easiest would just be to dump that as follows, one after another in a text file:
levels={
  {
    "Tile data goes here, as hex characters in a simple string, for example 000100020f04 etc.",
    "Title goes here",
    "Hint goes here",
    "Author goes here",
    -- Difficulty goes here in something that's not a string possibly, just a decimal number is fine
  },
  -- more levels go here
}

Then I can just copy and paste the contents of that file into the .P8 file and we're ready to go.

Since PICO-8 has a small resolution and no built-in capabilities for formatting strings:
It would be nice if the level hints could be split into several lines (ie. just insert \n in the string), where each line is no longer than 32 characters long. However, the string should be split on word boundaries (otherwise I could just do it in the display code).
If no word boundaries are fine, just insert -\n instead (counting the dash as one of those 32 characters). We don't need to get too fancy.


Issue with symfony 2.8 in the profiler

see https://github.com/symfony/symfony/pull/17179

Temporary fix is to add the following in config.yml
iltar_http:
    router:
        mapped_properties:
            Symfony\Component\HttpFoundation\Request.request: uri


output formats

investigate how to output formats other than HTML.  I'm pretty sure gitbook supports this.  If so, generate them using the same hooks we're using for deployment and link to them for download.

If you mean PDF/Mobi and epub  you need to enable it in your book settings:

after this you get an download dropdown button:

yes, but we're not deployed on gitbook.com, we're self-hosting.

gitbook {pdf,mobi,epub} does the trick.  they are output to the current folder.  It would be nice to modify the language landing page to offer links to each + HTML version as a choice.

The only way would be to do make an folder for them write an nodejs  build script for this and then link the output files directly to the landing page.
There seems to be an GitBook plugin for this:
download-pdf-link
I started with GitBook also recently and a great example for an landing page and also Leanpub compatible format is: https://github.com/thewhitetulip/web-dev-golang-anti-textbook
I hope this helps.

Just a heads up you should take a look at GitBook-beta there are changes coming for multilang support and landing page modifications in the next release.


Add token balances to homescreen

Description
Between the notifications and transaction list on the homescreen, the user can see their total balance of all tokens. If they only have one token, they are shown info for that one balance, otherwise they see their total balance and a link to see a breakdown of all their token balances.
Tested

How others should test
View the homescreen when the user has 0, 1 or multiple token balances
Related issues

Fixes #1062

Backwards compatibility
yes

Also removed the BalancesDisplay from the sidebar!


Edit tags will toggle instead of resetting all tags of a person

E.g. Tommy is listed as index 1 and has tags school and classmates.
edit 1 t/school t/jackass will remove school from Tommy's tags and add jackass to his list of tags.

Coverage increased (+0.04%) to 90.244% when pulling d7624fb8e5af31e4c7b3130b70a66e80edcc6f1f on ZY-Ang:EditTagsToggle into 998e2cd21bbf99a8b73c19a965994d9ecebe97b9 on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.04%) to 90.244% when pulling d7624fb8e5af31e4c7b3130b70a66e80edcc6f1f on ZY-Ang:EditTagsToggle into 998e2cd21bbf99a8b73c19a965994d9ecebe97b9 on CS2103AUG2017-W11-B2:master.

Coverage increased (+0.04%) to 90.244% when pulling 8b0fafb4ea5396e998ab3a5a0a216b02d2a2f68a on ZY-Ang:EditTagsToggle into 998e2cd21bbf99a8b73c19a965994d9ecebe97b9 on CS2103AUG2017-W11-B2:master.


Removed unnecessary commas in JSON examples

Some of the commas provided in the sample JSON objects broke the json syntax. The commas have been removed.

Summary

Ticket Link

Hello @Peytob,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.
Per the Mattermost Contribution Guide, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
Once you have signed the CLA, please comment with /check-cla and confirm that the CLA check is green.
This is a standard procedure for many open source projects.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

/check-cla

Thanks for this fix @Peytob :+1:
Looks like there are stilol some issues with the CLA. Have you made sure to have it signed?

/check-cla

@mickmister Yesterday I forgot to sign the CLA. Now I signed it, but /check-cla didn't do anything. After signing, I received an email attached to the message.
It seems to me that I did not do all the necessary steps. What could I forget?

Awesome, thanks @Peytob!


Function calls with wrong number of arguments don't throw errors

Describe the bug
In Clojure, calling a function with more arguments that it takes throws an ArityException. This is not the behavior experienced.
To Reproduce
In the REPL, try the following:
> (defn arity-test [n] (+ n n))
> (println (arity-test 1))
2
> (println (arity-test 1 1))
2

Expected behavior
In a Clojure REPL, do the same as above:
user=> (defn arity-test [n] (+ n n))
#'user/arity-test
user=> (arity-test 1)
2
user=> (arity-test 1 1)
clojure.lang.ArityException: Wrong number of args (2) passed to: user/arity-test [at <repl>:3:1]
user=>

However, I know that this project doesn't try to stay 1:1 with Clojure's behavior, so I'm left wondering if this is expected behavior or not :)

Its expected for the moment. No exceptions here. I'll leave this issue open though for now.


question

Sorry that I am writing to you in Issues. I I wrote to you by mail but you did not answer.
Question: How can I install your plugins?

Sorry that I am writing to you in Issues. I I wrote to you by mail but you did not answer.
Question: How can I install your plugins?

What client mod do you use?

Sorry that I am writing to you in Issues. I I wrote to you by mail but you did not answer.
Question: How can I install your plugins?

What client mod do you use?

Hi! I am using BetterDiscord

Sorry that I am writing to you in Issues. I I wrote to you by mail but you did not answer.
Question: How can I install your plugins?

What client mod do you use?

Hi! I am using BetterDiscord

This plugin does not support BetterDiscord unfortunately, there are no compatibility plugins for BetterDiscord that will allow this plugin to work, either.

Also note: this plugin is only available for Vizality (which wasn't relased btw), so you need to wait until relase.
By the way, don't forget to close this issue if you don't have more questions

Hey Manix, didn't receive your email, but as others have said this plugin is meant to be used with the Vizality client mod which is currently undergoing closed alpha testing, which just began last week. To find out more info and to possibly become eligible to beta test when the option becomes available, I recommend you join our Discord server at https://invite.vizality.com.
Thanks.


[Qudits] Add support to cirq.channel, cirq.apply_channel, and cirq.mixture

For #933. Part 3 after #1771.

cirq.apply_channel checks for the qid_shape protocol and use properly sized numpy arrays.
cirq.channel and cirq.mixture should just work.

Qudit features not in this PR are simulators, measurements, optimizers.

I am still adding tests for non-qubit channels and mixtures.

This incorrectly extends channel matrices.


Snowflake dialect missing Insert and Delete values

Describe the bug
Hi, I parse some logs of queries and I stumbled upon this bug:
When using the parser with snowflake dialect, select and create works fine, but delete and insert returns empty lists- meaning no tables detected.
To Reproduce
Steps to reproduce the behavior:
In [11]: ddl = """INSERT INTO TDM . CLEANSED . I_TRACKING (ID, BATCH_SIZE, SOURCE, SNOWFLAKE_BATCH_COMMIT_TS, BATCH_MIN_SOURCE_TS, BATCH_MAX_SOURCE_TS, MIN_OFFSET, MAX_OFFSET, K_PARTITION, IR, DUP_K_DROPS, TABLE_NAME ) VALUES ( ? )"""

In [12]: DDLParser(ddl).run(output_mode="snowflake")
Out[12]: []

In [15]: ddl =  """DELETE FROM SL_CHANGE WHERE A_CHANGE_ID = ?"""

In [16]: DDLParser(ddl).run(output_mode="snowflake")
Out[16]: []

Expected behavior
I expect for the insert case, to have TDM.CLEANSED.I_TRACKING as a table, containing these columns:
ID, BATCH_SIZE, SOURCE, SNOWFLAKE_BATCH_COMMIT_TS, BATCH_MIN_SOURCE_TS, BATCH_MAX_SOURCE_TS, MIN_OFFSET, MAX_OFFSET, K_PARTITION, IR, DUP_K_DROPS, TABLE_NAME
as a dict
for the delete case the same thing -
SL_CHANGE as the table_name and A_CHANGE_ID as the column
Desktop:

OS: Mac
Version 1.5.1

Additional context
I tried changing the ? for another value but it didn't make any change
for the insert ddl specifically, I also tried removing the spaces in the table name but it didn't make any change either

@HalfonA, hi, thanks for opening the issue, problem, that insert & delete is not supported. Also, DELETE & INSERT is not a part of DDL - it is a DML, so DML is not supported at all.

Hi Halfon A. can I suggest you to look at the role of this python module : https://github.com/tobymao/sqlglot it seems the good one for you.

@HalfonA no ) it wasn’t planned, only ddl. I agree with @dmaresma, better to take a look on sqlglot

Thank you!


feat: add q8 of tpch

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #863
Closes #

Checklist

[x] Code follows style guide (ruff)
[x] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below.

@MarcoGorelli While I was implementing q8, I asked myself if it is worth implementing tests to validate that the outputs of the TPC-H queries are as expected. What do you think? 🤔


fix: suggest windows-compatible typescript install command

Specifying version range in single quote doesn't work in windows CMD and it returns:

The system cannot find the file specified.

Done. Sorry for the delay.


[Feature Request] Disabling the broadcasting button on account level

Hello,
I am looking for an advanced setting to disable the broadcoasting button on account level.
Why?
If you are using Sparrow for external funding of lightning channels you don't have to broadcast the signed transaction by our own. You have to transmit the tx to lnd. If you accidentally broadcast the tx there is a risk of loosing funds.
Of course you could disconnect Sparrow from your server but this is a bad workaround, because sometimes you need the connection for loading the history.
Imho the best way to prevent accidental broadcasts is to disable the button for the funding account.

This is an odd feature, in the sense that it doesn't fit in well with any current UI. Is there a way this kind of transaction might be detected so a warning can be displayed instead?

+1 for a No Broadcast option or even an additional dialog box after clicking the broadcast button.

Here is some lost 1 BTC with a channel opening
https://stacker.news/items/436720


TypeScript: Improved types for UA, Session and Constants

Hi,
This PR improves the types for SIP.js in the following ways:

Uses real EventEmitter class from node.js "events" module to extend classes, to reflect the real implementation.
Adds all causes to the constants ("C") namespace.
Adds missing return types to various functions, which has lead to compiler errors in projects with strict tsconfig.json configurations.
Adds peerConnection to SessionDescriptionHandlers as described in in this SIP.js guide.
Inital types for ServerContext
Adds the field "id" to session.
Fixes the signature of session.dtmf() to the documented behaviour.

We use SIP.js at sipgate and would like to contribute more types in the future, so feedback is welcome.
Thanks.

This looks great! Had some small feedback items that I commented on. If those get addressed we can absolutely take this

I can't find your comments. What did you comment on?

You should be able to see it now.
Just as a heads up, we are actively transitioning the entirety of the code to typescript/es6. This will hopefully make the types folder unnecessary, but at minimum the new code will match what is in the types folder at the time of the transition getting onto master.

I adjusted the types according to your feedback. Thanks.


load library upload in sub folder

i try to use upload library in controller on sub folder
load in construct
public function __construct() {
    parent::__construct();
    $this->load->library('upload');
}

load in function
public function uploadLandingPage () {
                $this->load->library('upload');
		$landingName = $this->input->post('landingName');

		$config['upload_path'] = './assets/';
		$config['allowed_types'] = 'jpg|png';
		$this->upload->initialize($config);

		$this->upload->do_upload('file');
}

load in construct and function it not work. and error 500 (Internal Server Error)
and i try to load in autoload.php is work :")
i not sure is a bug but i try to help.

Post this on the forums, not here. 9 times out of 10, when you're not sure if it's a bug - it's not.


Fix tabbed panel colors

Feature branch PR
Note: this PR is against feature-focus branch not main. Style fixes related to interaction state (focus, hover, current) will be collected in small increments into feature-focus before being merged into main.
Figma mismatches this PR corrects
Upon re-examining the design in Figma, I realized that I got the colors for the tabbed panel slightly wrong. This PR corrects them. cc @smeragoel
The Figma specifies that the tabbed panel background color should be set to on-background (in light mode, this color is the same white as background, but in dark mode on-background is lighter than background). Also the inactive tabs cannot share the same background color variable for both light and dark mode. In light mode, the background color for the inactive tabs should be surface, but in dark mode, it should be on-background, as the following screenshot shows:

I was scared by the code block living in

Cross-reference:

https://github.com/pydata/pydata-sphinx-theme/pull/1555
https://github.com/Quansight-Labs/czi-scientific-python-mgmt/issues/44


Add zstd-stream support

Discord apparently supports zstd-stream gateway compression now; see ex. discordjs/discord.js#3223

https://github.com/luben/zstd-jni
https://github.com/airlift/aircompressor/blob/master/src/main/java/io/airlift/compress/zstd/ZstdDecompressor.java

(note: there is no guarantee Discord will keep this, so no need to rush into an implementation)

It seems that zstd-stream support has been confirmed to stick around and should be added to the docs soon(tm).

It's been a year and this isn't documented. This can be reopened if zstd comes back to relevance in some form.


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>

Coverage remained the same at 80.631% when pulling cd0200d1d9f5f423b3153c5173e840b5614d1150 on template-sync-1 into 7cf85d589246ca6bbecd2895d8959324078e93e7 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


MSID: 88659 Version: 1 DOI: 2<IP_ADDRESS>3997

"msas": "Computational and Systems Biology", "Structural Biology and Molecular Biophysics"
"msid": "88659"
"preprintDoi": "10.1101/2<IP_ADDRESS>3997"
"Reviewed Preprint posted": "2023-06-26"
"Sent for peer review": "2023-04-26"
"Posted to bioRxiv": "2023-06-07" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>3997v1")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=88659)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/88659/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: https://github.com/elifesciences/enhanced-preprints-data/pull/111
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews
Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>3997)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/88659/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/88659)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/88659

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/88659

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/88659/v1/88659-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/88659)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Corresponding author information needs fixing

Fix in https://github.com/elifesciences/enhanced-preprints-data/pull/120


Do not publish except in official builds

dotnet/aspnetcore#22368

nit: Remove extra $(_PublishArgs) variable

Just to clarify, this still means that bits will be published for the master branch on the internal CI right?

Just to clarify, this still means that bits will be published for the master branch on the internal CI right?

Absolutely


Adding csv files

Prerequisites

[X] I am an Open Source Contributor
[X] I checked to make sure that this issue has not already been filed

Description
In the datasets, we have tsv files. I would like to convert them into CSV and add them to the dataset. This would give us the option of working with either tsv or CSV
Screenshots
No response
Code of Conduct

[X] I follow Contributing Guidelines of this project.

I would like to work on this issue

No, we dont require options since we have to work in the code with only one type of file. It will confuse others if both types are present. @karthikbhandary2

@Rakesh9100 ok understood!

You can create new unique issues anytime and I will assign @karthikbhandary2


Support long QPIGS responses of the PIP6048MT

Added an example of PIP6048MT QPIGS response
Made PIPSOLAR_READ_BUFFER_LENGTH bigger to fit this response

I have the 4024MT version this is my log, i can t use the 2424mse1 branch i have this error
logs_easun_logs.txt

@lucasimons Please a select entity to your configuration or use the example configuration from this feature branch. It contains entities which must be part of the configuration at the moment and aren't optional. If this doesn't work out of the box please perform a clean build:
esphome clean yourconfig.yaml
esphome run yourconfig.yaml

Thanks now it works fine


Add a basic stack-tracking mechanism for easier debugging.

@bklimt are you intending to update this?
If not I will clone your branch, make the modifications and submit as a new PR.

AFAIK he's not. I've also tried using this internally and I get OOM exceptions.

Hmm OK, I'll give it a spin and some point see if it's worth continuing,

*internally within our codebase that uses bolts, not just bolts on its own.

Thank you for your pull request.  As you may know, we require contributors to sign our Contributor License Agreement, and we don't seem to have you on file and listed as active anymore.  In order for us to review and merge your code, please email<EMAIL_ADDRESS>with your details so we can update your status.


Provide better control over filtered replication

In the current resharding workflow, once you've reached the filtered replication phase, there's no straightforward way of going back to a previous phase. There is also no way to pause or resume replication.
Proposed vtctl commands:

StopFilteredReplication
StartFilteredReplication
AbandonFilteredReplication: This will allow you to restart SplitClone. We can also consider implementing this as an additional worker flag: -abaondon_filtered_replication.

Should we call?:
FilteredReplicationPause
FilteredReplicationStart
FilteredReplicationStop(for the abandon)

Rather than this, can we start having options rather than different commands.
FilteredReplication <start|stop>
each command having additional options.

This was fixed through the various vreplication refactors.


error in leaflet package!!! it's "台湾“ and not "中华民国“

we can see "Taiwan" in google map

however, '中华民国' in leaflet map

That's not Leaflet but the tiles (background images) your are using. It's seems to be OSM: http://openstreetmap.org/

Duplicate of https://github.com/Leaflet/Leaflet/issues/4332, https://github.com/Leaflet/Leaflet/issues/3896

great! you are right
2016-06-22 14:12 GMT+08:00 Yohan Boniface<EMAIL_ADDRESS>
That's not Leaflet but the tiles (background images) your are using. It's
seems to be OSM: http://openstreetmap.org/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet/issues/4661#issuecomment-227652859,
or mute the thread
https://github.com/notifications/unsubscribe/AOrXbOXBbLSO7nCaX_UHZp2BGNPC8O3tks5qONJcgaJpZM4I7aer
.


Uhuru login redirects to blank screen

Tried logging in to Uhuru on the main login page on https://beta.uhuru.me/
and just getting a blank screen after the redirect

Known issue: https://github.com/threefoldtech/twin_aydo/issues/302


[Misc] Standardize RoPE handling for Qwen2-VL

https://github.com/huggingface/transformers/issues/33401 has been fixed in Transformers v4.45.2, but the devs have clarified that M-ROPE is intended to be configured as rope_type="default". To avoid future compatibility issues, this PR updates the RoPE code in vLLM to follow their specification.

@ywang96 PTAL when you have time.

Can mrope be completed using specialized CUDA operators? According to the profile results, this part is quite time-consuming.

Can mrope be completed using specialized CUDA operators? According to the profile results, this part is quite time-consuming.

Can you open a new issue for this? Thanks

Can mrope be completed using specialized CUDA operators? According to the profile results, this part is quite time-consuming.

Can you open a new issue for this? Thanks


Ignore main packages

Breaking changes in a main package aren't breaking changes (they can't be imported).

TODO add a test to this to ensure it's ignoring top level main packages, but even those in sub directories (such as ignore cmd/name/main.go).

Done 0619ea94767cb0ce52456356bec4f54983ec5c7f


[SPARK-39259][SQL][3.2] Evaluate timestamps consistently in subqueries

What changes were proposed in this pull request?
Apply the optimizer rule ComputeCurrentTime consistently across subqueries.
This is a backport of https://github.com/apache/spark/pull/36654.
Why are the changes needed?
At the moment timestamp functions like now() can return different values within a query if subqueries are involved
Does this PR introduce any user-facing change?
No
How was this patch tested?
A new unit test was added

Can one of the admins verify this patch?

There are some Python errors in the build, which seem to me like there is a problem with pandas or python versions in the build pipeline. Anyway, I really doubt that I broke those tests

There are some Python errors in the build, which seem to me like there is a problem with pandas or python versions in the build pipeline.

Looking at GAs for commits in https://github.com/apache/spark/commits/branch-3.2, they have been broken for some time already. cc @HyukjinKwon @ueshin @itholic

Let me fix it

https://github.com/apache/spark/pull/36759

Let's wait for https://github.com/apache/spark/pull/36759 and then re-trigger the build via an empty commit:
$ git commit --allow-empty -m "Trigger build"

@olaky Could you re-trigger builds, please.

Here is the PR to fix Scala 2.13 issue.

https://github.com/apache/spark/pull/36762

When updating this PR, let's also pull in my changes from https://github.com/apache/spark/pull/36765 . When merging this, we should probably pick it all the way back to 3.0 (since it looks like a correctness issue that would also impact those versions).

I cherry-picked 583a9c75bbb35387169d4f0cf763ef566d899954

I guess the failure is not related to PR's changes:
[info] - check simplified (tpcds-v1.4/q4) *** FAILED *** (945 milliseconds)
[info]   Plans did not match:

+1, LGTM. Merging to 3.2 and trying to merge to 3.1/3.0.
Thank you, @olaky and @JoshRosen @dongjoon-hyun for review.

@olaky The changes cause conflicts in branch-3.1. Could you PRs w/ backports to 3.1 and 3.0, please.

Merging is blocked because of a test failure that also surfaces in https://github.com/apache/spark/pull/36386

@MaxGekk since you closed this, should I still work on propagating this to 3.1 and 3.0? And how should we deal with the test failures happening on branch-3.2?

@olaky This PR has been merged to branch-3.2 already, see https://github.com/apache/spark/commit/d611d1f66761bd39fee850ca3f435027f9fc1e3c
Please, open separate PRs against branch-3.1 and branch-3.0

@MaxGekk 3.1 already does not have any transform with subqueries function, so I would have to backport this as well. I personally feel that this could be a risky endeavour not worth doing to get this fix, what do you think?

3.1 already does not have any transform with subqueries function ...

Could you list required PRs, please. Is it possible to extract only needed functions from them?

Transforming with subqueries comes from https://github.com/apache/spark/commit/1a356858577aa13012f56fdd4b41f8ea6fe230ce and pruning comes from https://github.com/apache/spark/commit/3db8ec258c4a8438bda73c26fc7b1eb6f9d51631.
Both of these are fairly big, and I would not just want so cherry pick them.
That said Id did create: https://github.com/apache/spark/pull/36818

And for 3.0 we have: https://github.com/apache/spark/pull/36822 (looks to me like there is an infrastructure problem with building it)

And for 3.0 we have: #36822 (looks to me like there is an infrastructure problem with building it)

Thank you for making a backporting PR to branch-3.0 together. However, branch-3.0 is EOL currently because Apache Spark 3.0.0 was two years ago. Please see here.

https://spark.apache.org/versioning-policy.html


Check if symbol has been created in eosio.token::open

Add check to eosio.token::open action to assert that the provided symbol been created in the contract before creating a new zero-balance table row for that symbol.

Resolved by #102.


feat: nft shape

What?
NFT Shape with picture frame
Why?
One small step for man...

It is on purpose, because we don't have accounts yet for the builder, no NFT could be shown in the builder if we add it.


feat(deps): update tubenhirn/semantic-release-github docker tag to v4

This PR contains the following updates:

Package
Update
Change

tubenhirn/semantic-release-github
major
v3.0.0 -> v4.0.0

⚠ Dependency Lookup Warnings ⚠
Warnings were logged while processing this repo. Please check the logs for more information.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 3.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


kaizen: add equals-ignore-case pattern

addresses: #186
This adds support for the equals-ignore-case pattern. The automaton-building is a bit complex but reasonably well tested.
What is controversial is that this PR contains Quamina's first generated code - the case-mappings are pulled from the file CaseFolding.txt in the Unicode Character Database. The generated code is in case_folding.go.  It is built by code_gen/build_casefolding_table.go which is in package main and has main()
The problem is that the CaseFolding.txt is regularly changed with Unicode releases. So build_casefolding_table checks to see if the case_folding.go file is older than 3 months and if so, rebuilds it.  There is a Makefile to take care of building and running the code generator.  So if you're doing a PR you should at some point type make and if you get a new case_folding.go then check it in with your PR.
There are a couple of problems. First, I've never done generated-Go code before and maybe there's a much better well-known approach.  Second, I haven't figured out how to unit test the code generator. The coverage tools ignore it because it's not in the quamina package. Implicitly, it's tested by the tests in monocase_test.go and elsewhere having the desired results but I'm going to open an issue to figure out how to test properly.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 97.50000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 96.59%. Comparing base (daef7fd) to head (407f600).
Report is 4 commits behind head on main.

Files
Patch %
Lines

monocase.go
96.07%
1 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #340      +/-   ##
==========================================
+ Coverage   96.56%   96.59%   +0.03%
==========================================
  Files          18       19       +1
  Lines        1864     1941      +77
==========================================
+ Hits         1800     1875      +75
- Misses         36       37       +1
- Partials       28       29       +1

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

Hmm, surprised by the benchmark failures, I didn't think I changed anything that should affect an existing benchmark.  Will investigate. Could be trailing fallout from the Q-numbers work.

@timbray Go itself (the x package is semi-official and comes close to the standard library) appears to do it this way:
https://cs.opensource.google/go/x/text/+/master:gen.go
with generated code like this: https://cs.opensource.google/go/x/text/+/master:cases/tables15.0.0.go

This is annoying; the same problem with the same benchmark showed up on the Q numbers PR, and I accepted the numeric-match slowdown in the interests of correctness. So I changed the CI/CD to warn but not fail, and then after everything was clean I put the CI/CD back to fail mode, assuming that would reset the cached previous value.  Apparently this did not happen; I have poked around and failed (so far) to figure out how/where the benchmark tool caches previous results.
Anyhow, for now, I'm going to reset the CI/CD again not to fail on slowdown.


fix(notifier): disable for json args

Should we put the update notification via stderr instead of stdout to avoid polluting output?

Good question, every update notifier that I've used pipes to stdout, but I agree this feels like something that may be a bit annoying to maintain in the future so maybe we consider stderr

Nah it shouldn't go to stderr. The real answer is to have two internal writers to stdout; one for structured text, one for normal output.


getToken() never return null

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

That is correct. Thanks!


Aggregate function "min" on checks does not work

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

create a check with aggregate function min
the check is displaying min value in gui but uses last value for rule evaluation
...

@harel-ecoplant just trying to understand the issue a bit more. If you create the check from scratch does it behave as you would expect, or are you seeing if when you update the aggregate function on a check?

@harel-ecoplant just trying to understand the issue a bit more. If you create the check from scratch does it behave as you would expect, or are you seeing if when you update the aggregate function on a check?

it happens when i create a new check from the API ( POST https://us-west-2-1.aws.cloud2.influxdata.com/api/v2/checks ).
im not able to reproduce the error if i use the regular UI (might be wrong using different test case)

Would you be able to provide the post body request so that I can try to reproduce it :)