Multi peer malicious validator set is not detected

When running the bisection/multi-peer/malicious-validator-set.json both primary and witness manage to verify the light block without detecting the malicious validator set. In this case the hashes or headers from two peers are wrong so a fork is detected and the test passes but neither of the two peers are marked as faulty as they do not fail on verification.

@Shivani912 Is that the expected behavior here?

@OStevan @romac I think we badly need to discuss what goes in the description of these tests - it's ambiguous or should at least say what the description is meant for. On the test, so there's a trusted_validator_set for initial few heights and then it changes after a certain height. When you ask the light client to verify header at height=11 with trusted_header at height=1 in bisection mode, primary_provider gives a header with set of validators that is different than the one we have in our trusted_store. But, witness gives a header signed off by the exact same validator set we already trust (from trusted_validator_set). Now this is the case where old validator set starts to misbehave. So this can be seen as a fork too, because both headers are valid. This could also result into an error that no witness found for the data we got from primary and should abort - that's what Go Light Client does. So it's hard to say what specific error it must fail on because this is very implementation specific. That is one major reason for not having expectations on specific error types in these tests.

@Shivani912 @romac Sorry, I made a mistake when comparing. There was a bug in my code. However, I agree it would be useful to have some more details in the tests why they fail.


Trigger Agent regex

Why is Regexp::IGNORECASE option set by default? Is there any way if I want the regex to be case sensitive.

Try prefixing your pattern with (?-i), or surrounding your pattern with (?-i: and ), and it will be treated as case sensitive pattern.

Interesting! Thank you.


[Q/BUG] Handling connection loss

Describe your system

odbc Package Version: 2.4.7
ODBC Driver: IBM i Access ODBC Driver
Node.js Version: 18.15.0

Question/Describe the bug
I currently do not see a way to handle the connection being lost.
When you create a pool and connect it and then disconnect (lose internet connection or similar) there is no way to check if that connection has been lost (event or property/function).
Because of this there is no way to handle connection loss before doing a query, meaning it will simply timeout.
It does not seem to reconnect on it's own if you reconnect the internet connection.
The current connected getter on Connection seems to always return true.
Expected behavior
An event or a method to check the current connection state.
To Reproduce
Steps to reproduce the behavior:

Create a pool
Connect the pool
Cut off any communication with the database
Reconnect the connection to the database
Query/call procedure fails
Creating another pool fails as well, you need to make a new connection

Additional context
Preferably we would like to check the connection state or act upon an error instead of timing out and only then handling the issue.

Bug/Question remains

Would like some guidance on progress of this as well!

I would love to see a solution to this.  This has been a pain point for us, too.

I would like to see this added as well. im having trouble reusing a connection without knowing the status of that connection.
Specifically, looking for a way to check the status of a current connection, (open, closed, stuck) etc.
Also, would like to see some methods added for connection events. ie connection.on("open", function...) or connection.on("close", function...)

I have also encountered this problem, have you solved the problem that the connection in the connection pool will be disconnected, in fact, just modify pool.js can, but why this project has not been updated

We came up with a way to handle this via trapping for specific sql codes/states that indicate the connection issue. Our applications are all dockerized within k8s, so we handle the connection dropping/failure by exiting the application and continuing to restart until reconnection. This has been a good approach until this is officially handled.

This has not been resolved yet. There hasn't been a commit on the main branch in 8 months.


Use of autocomplete needs more work and guidance regarding its use in survey questions where the integrity of the data could suffer from pre-population

Use of autocomplete needs more work and guidance regarding its use in survey questions where the integrity of the data could suffer from pre-population
Originally posted by @jrbarnes9 in https://github.com/ONSdigital/design-system/issues/163#issuecomment-470123980

Dupe of #275


District Heating build-out

Our district heating build out is specified in config.yaml and stays the same for all scenarios:
sector:
  district_heating:
    potential: 0.4
    progress:
      2020: 0.0
      2025: 0.15
      2030: 0.3
      2036: 0.45
      2040: 0.6
      2045: 0.8
      2050: 1.0

Compared to the other models we have much less variations in DH

Option A:
I think REMIND models the DH build-out endogenously. We could take over the DH potential and progress from their runs and only let PyPSA decide how to supply the heat.
Option B:
We could also double check our assumption, find a source for the total DH potential, and specify different progress trajectories for each scenario in order to get more variation.

BMWK says

Mit einem deutlich beschleunigten Ausbau der Wärmenetze soll sich die Anzahl der angeschlossenen Gebäude bis 2045 gegenüber heute in etwa verdreifachen, d.h. es sollen mittelfristig jährlich mindestens 100.000 Gebäude neu an Wärmenetze angeschlossen werden (https://www.bmwk.de/Redaktion/DE/Downloads/Energie/0612-erklaerung-fernwaeme-gipfel.pdf?__blob=publicationFile&v=8).

At the moment the DH share in urban areas is ~11.6% in the 2020 model.
heat_demand = networks[-1].statistics.withdrawal(
        bus_carrier=["urban central heat","urban decentral heat"],**kwargs
    ).filter(like=region).groupby("carrier").sum().get(["urban central heat","urban decentral heat"])
heat_demand["urban central heat"] / heat_demand.sum()

And goes up to ~37.2% in 2045. That's a bit more than triple, and i am somewhat puzzled how it relates to the 0.4 specified in the config. But it's at least roughly the right order of magnitude.
 Finde ich eine sinnvolle Annahme, Frage wäre dann nur welche Zahlen wir dafür annehmen.

@nworbmot @cpschau Maybe you have some useful input

at the moment district heat share for Germany in 2020 is 0.077 [specified in districit_heat_share.csv]
Where does this number come from? And what exactly does it mean?
Since the BMWK aims for a tripling by 2045, maybe we could just triple that number to arrive at a DH potential?

Number comes from build_district_heat_share and is a conservative estimate of existing capacities. 0.077 is the fraction of total heat demand. The fraction of urban heat demand in DE would be around 0.1 -> tripling that we arrive at 0.3.

Sounds good to me!

I'm still a bit puzzled by the DH share of 7.7%. According to BDEW, it's 14.2% (including Nahwärme).

The district_heat_share.csv of the PyPSA-Eur master branch indicates 13.8% for Germany (https://github.com/PyPSA/pypsa-eur/blob/master/data/district_heat_share.csv).
Statistics about the heat sector should be carefully examined to determine whether they refer to the entire heat demand, including process heat, or just building heat. In the case of Germany process heat contributes another 600 TWh FED (UBA 2021: https://www.umweltbundesamt.de/daten/energie/energieverbrauch-fuer-fossile-erneuerbare-waerme).

I reduced the potential to 0.3, and for Current Policies and Worst Case scenarios to 0.1
59333db0059a7ac4a72aa8814c649774528bf5de


Building fail

I have this error then building:
ld: library not found for -llibomp
clang-9: error: linker command failed with exit code 1 (use -v to see invocation)
make[2]: *** [render] Error 1
make[1]: *** [CMakeFiles/render.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
Undefined symbols for architecture x86_64:
"TiXmlDocument::LoadFile(char const*, TiXmlEncoding)", referenced from:
TiXmlDocument::LoadFile(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, TiXmlEncoding) in MusculoSkeletalSystem.cpp.o
"TiXmlDocument::TiXmlDocument()", referenced from:
MakeMuscles(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, std::__1::shared_ptr&) in MusculoSkeletalSystem.cpp.o
"Ipopt::IpoptApplication::IpoptApplication(bool, bool)", referenced from:
FSM::FSM(std::__1::shared_ptr const&, std::__1::vector<int, std::__1::allocator > const&, int) in FSM.cpp.o
HighLevelController::Initialze(Eigen::Matrix<double, 3, 1, 0, 3, 1> const&, Eigen::Matrix<double, 3, 1, 0, 3, 1> const&, std::__1::shared_ptr const&, std::__1::vector<std::__1::pair<std::__1::pair<dart::dynamics::BodyNode*, Eigen::Matrix<double, 3, 1, 0, 3, 1> >, Eigen::Matrix<double, 3, 1, 0, 3, 1> >, std::__1::allocator<std::__1::pair<std::__1::pair<dart::dynamics::BodyNode*, Eigen::Matrix<double, 3, 1, 0, 3, 1> >, Eigen::Matrix<double, 3, 1, 0, 3, 1> > > > const&, std::__1::map<std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator >, Eigen::Matrix<double, 3, 1, 0, 3, 1>, std::__1::less<std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > >, std::__1::allocator<std::__1::pair<std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const, Eigen::Matrix<double, 3, 1, 0, 3, 1> > > > const&, int, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, int, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, bool, double) in HLC.cpp.o
LowLevelController::LowLevelController(std::__1::shared_ptr const&) in LLC.cpp.o
"TiXmlNode::~TiXmlNode()", referenced from:
TiXmlDocument::~TiXmlDocument() in MusculoSkeletalSystem.cpp.o
"TiXmlElement::Attribute(char const*) const", referenced from:
MakeMuscles(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, std::__1::shared_ptr&) in MusculoSkeletalSystem.cpp.o
"TiXmlNode::FirstChildElement(char const*) const", referenced from:
TiXmlNode::FirstChildElement(char const*) in MusculoSkeletalSystem.cpp.o
"TiXmlNode::NextSiblingElement(char const*) const", referenced from:
TiXmlNode::NextSiblingElement(char const*) in MusculoSkeletalSystem.cpp.o
"vtable for TiXmlDocument", referenced from:
TiXmlDocument::~TiXmlDocument() in MusculoSkeletalSystem.cpp.o
NOTE: a missing vtable usually means the first non-inline virtual member function has no definition.
ld: symbol(s) not found for architecture x86_64
Do you guys have any idea what is causing this? Any help is appreciated!

hi,@vittorione94
Can you provide more detailed information, such as: the build environment and software version?

Thanks for replying! Sure it was built on a MacBook with macOS Catalina.
gcc -v return:
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 11.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin19.0.0
Thread model: posix
In case you need anything else let me know!

the error message is quite descriptive. you are missing a library on your build system which is required libomp. maybe this helps https://stackoverflow.com/questions/44579856/ld-library-not-found-for-lomp

the error message is quite descriptive. you are missing a library on your build system which is required libomp. maybe this helps https://stackoverflow.com/questions/44579856/ld-library-not-found-for-lomp.

As you said the problem was related to libomp and not tinyxml directly. Thanks for the comment!


Unable to save robot descriptor from live discovering

-S --Save flag does not work at all.

the -s -S aliases aka '--save' '--Save' collides: it seems character case is not supported by yargs for aliases
the -S (save robot descriptor) alias has been renamed to --save-desc


Mesos container logs aggregation

With "EXECUTOR_FILE_PATH": "/tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stdout, /tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stderr, /var/log/*.INFO, /var/log/*.WARNING" docker inspect of logstash executors is
MacBook-Pro-2:temp vik$ docker inspect dd2e3bd2ae4f
[
{
    "Id": "dd2e3bd2ae4f756c17e95e35118deb52254e07a489b58e07e09ad68e55125738",
    "Created": "2016-01-20T14:08:30.205507381Z",
    "Path": "/tmp/start-executor.sh",
    "Args": [
        "dummyArgument"
    ],
    "State": {
        "Status": "running",
....
        "FinishedAt": "0001-01-01T00:00:00Z"
    },
    "Image": "cd0a3f5d3304308ab0c6677d10f0d1c771840763a31728fdbd044f025f64298a",
    "ResolvConfPath": "/mnt/sda1/var/lib/docker/containers/dd2e3bd2ae4f756c17e95e35118deb52254e07a489b58e07e09ad68e55125738/resolv.conf",
    "HostnamePath": "/mnt/sda1/var/lib/docker/containers/dd2e3bd2ae4f756c17e95e35118deb52254e07a489b58e07e09ad68e55125738/hostname",
    "HostsPath": "/mnt/sda1/var/lib/docker/containers/dd2e3bd2ae4f756c17e95e35118deb52254e07a489b58e07e09ad68e55125738/hosts",
    "LogPath": "/mnt/sda1/var/lib/docker/containers/dd2e3bd2ae4f756c17e95e35118deb52254e07a489b58e07e09ad68e55125738/dd2e3bd2ae4f756c17e95e35118deb52254e07a489b58e07e09ad68e55125738-json.log",
    "Name": "/mesos-e0baf934-fffe-426b-b576-b450aec02c78-S2.916d49cb-30c8-40ed-9551-5425bd3296b8",
    "RestartCount": 0,
....
    "AppArmorProfile": "",
    "ExecIDs": null,
    "HostConfig": {
        "Binds": [
            "/:/logstashpaths:ro",
            "/tmp/mesos/slaves/e0baf934-fffe-426b-b576-b450aec02c78-S2/frameworks/e0baf934-fffe-426b-b576-b450aec02c78-0002/executors/executor.0a516626-4495-4f84-a59e-fa7429100fb6/runs/916d49cb-30c8-40ed-9551-5425bd3296b8:/mnt/mesos/sandbox"
        ],
        "ContainerIDFile": "",
....
        "RestartPolicy": {
            "Name": "no",
            "MaximumRetryCount": 0
        },
        "SecurityOpt": null,
        "ReadonlyRootfs": false,
        "Ulimits": null,
        "LogConfig": {
            "Type": "json-file",
            "Config": {}
        },
        "CgroupParent": "",
        "ConsoleSize": [
            0,
            0
        ],
        "VolumeDriver": ""
    },
    "GraphDriver": {
        "Name": "aufs",
        "Data": null
    },
    "Mounts": [
        {
            "Source": "/",
            "Destination": "/logstashpaths",
            "Mode": "ro",
            "RW": false
        },
        {
            "Source": "/tmp/mesos/slaves/e0baf934-fffe-426b-b576-b450aec02c78-S2/frameworks/e0baf934-fffe-426b-b576-b450aec02c78-0002/executors/executor.0a516626-4495-4f84-a59e-fa7429100fb6/runs/916d49cb-30c8-40ed-9551-5425bd3296b8",
            "Destination": "/mnt/mesos/sandbox",
            "Mode": "",
            "RW": true
        }
    ],
    "Config": {
        "Hostname": "dd2e3bd2ae4f",
        "Domainname": "",
        "User": "",
        "AttachStdin": false,
        "AttachStdout": true,
        "AttachStderr": true,
        "Tty": false,
        "OpenStdin": false,
        "StdinOnce": false,
        "Env": [
            "LIBPROCESS_PORT=0",
            "MESOS_CHECKPOINT=1",
            "MESOS_DIRECTORY=/tmp/mesos/slaves/e0baf934-fffe-426b-b576-b450aec02c78-S2/frameworks/e0baf934-fffe-426b-b576-b450aec02c78-0002/executors/executor.0a516626-4495-4f84-a59e-fa7429100fb6/runs/916d49cb-30c8-40ed-9551-5425bd3296b8",
            "MESOS_EXECUTOR_ID=executor.0a516626-4495-4f84-a59e-fa7429100fb6",
            "MESOS_FRAMEWORK_ID=e0baf934-fffe-426b-b576-b450aec02c78-0002",
            "MESOS_NATIVE_JAVA_LIBRARY=/usr/lib/libmesos-0.25.0.so",
            "MESOS_NATIVE_LIBRARY=/usr/lib/libmesos-0.25.0.so",
            "MESOS_RECOVERY_TIMEOUT=15mins",
            "MESOS_SLAVE_ID=e0baf934-fffe-426b-b576-b450aec02c78-S2",
            "MESOS_SLAVE_PID=slave(1)@<IP_ADDRESS>:5051",
            "JAVA_OPTS=-Xmx128m -Dmesos.logstash.logstash.heap.size=256m",
            "MESOS_SANDBOX=/mnt/mesos/sandbox",
            "MESOS_CONTAINER_NAME=mesos-e0baf934-fffe-426b-b576-b450aec02c78-S2.916d49cb-30c8-40ed-9551-5425bd3296b8",
            "PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin"
        ],
        "Cmd": [
            "dummyArgument"
        ],
        "Image": "mesos/logstash-executor:latest",
        "Volumes": null,
        "WorkingDir": "",
        "Entrypoint": [
            "/tmp/start-executor.sh"
        ],
        "OnBuild": null,
        "Labels": {}
    },
    "NetworkSettings": {
        "Bridge": "",
.....
        "IPv6Gateway": "",
        "MacAddress": "02:42:ac:11:00:0b",
        "Networks": {
            "bridge": {
                "EndpointID": "94d61157a2729b5cf372525a544c47df79069b2c63ead470c30b80bdaedb3de6",
                "Gateway": "<IP_ADDRESS>",
                "IPAddress": "<IP_ADDRESS>",
....
                "MacAddress": "02:42:ac:11:00:0b"
            }
        }
    }
}
]

Therefore this logstash configuration will not be working as /logstashpaths/tmp/ is not available in logstash executor container
14:08:33.646 [Thread-3] INFO  o.a.m.l.executor.LogstashService - Starting subprocess: LS_HEAP_SIZE=256m HOME=/root /opt/logstash/bin/logstash --log /var/log/logstash.log -e input {
file {
"path" => [ "/logstashpaths/tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stdout", "/logstashpaths/tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stderr", "/logstashpaths/var/log/*.INFO", "/logstashpaths/var/log/*.WARNING" ]
}
}
filter {
mutate {
"add_field" => {
"mesos_slave_id" => "e0baf934-fffe-426b-b576-b450aec02c78-S2"
}
}
}
output {
elasticsearch {
"host" => "<IP_ADDRESS>"
"port" => 33000
"protocol" => "http"
}
}

I'd like to review the code to check if paths in  /tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stdout, /tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stderr, /var/log/*.INFO, /var/log/*.WARNING can be analysed to extract the most specific non-wild-card path.
Going from
/tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stdout,
/tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stderr,
/var/log/*.INFO,
/var/log/*.WARNING

to
/tmp/mesos/slaves,
/var/log

and mapping those to the container with /logstashpaths prefix

From what I can see --enable.file=true --executor.file-path=/var/log/mesos/mesos-*.INFO is sufficient?

Damn docker-outside-docker of minimesos. In my tests logstash executors run on host, not on mesos agent :rage:
will re-structure the tests.... to be continued

Yeah. Hindsight I always found that move a bit a pain in the A…

It's working.
With "EXECUTOR_FILE_PATH": "/Users/vik/projects/cs/devbox/sources/mesos-elk-demo/build/elk.log, /Users/vik/projects/cs/devbox/sources/mesos-elk-demo/build/*/*.log" newly created logs files were picked up by logstash

:+1:

Just requires testing on real mesos cluster with paths like /tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stdout, /tmp/mesos/slaves/*/frameworks/*/executors/*/runs/latest/stderr

currently blocked by #94 ...

You should be able to continue with MESOS_ROLE=* in your envs

Tested on Mesos Cluster and it works.
Description of the tests is in https://github.com/ContainerSolutions/mesos-elk-demo

:+1:


fixes OpenAPI spec security definitions

SpiceDB does not use ApiKeyAuth authentication, but Bearer authentication, where
the type of bearer token is an API Key, as defined by the OpenAPI v3 spec.
However, the OpenAPI v2 Spec, which is the one supported by grpc-gateway,
does not support bearer authentication:
https://swagger.io/docs/specification/v2_0/authentication/authentication/
Still, the grpc-gateway maintainers indicated in
https://github.com/grpc-ecosystem/grpc-gateway/issues/1089
that bearer is actually supported in grpc-gateway generator.
In https://github.com/authzed/authzed-go/pull/255 a user reported
trying to generate Java client code out of the SpiceDB OpenAPI spec,
but had errors because the generated error did not properly provide
the preshared key with the expected Authorization: Bearer <psk>
format.
I'm not 100% sure if this is a legit intermediate state
between v2 and v3 we can leverage, but the current generated
client code is clearly broken anyway.
See https://swagger.io/docs/specification/v3_0/authentication/api-keys/
See https://swagger.io/docs/specification/v3_0/authentication/bearer-authentication/
The generated Swagger Spec looks like this
"securityDefinitions": {
    "bearer": {
      "type": "apiKey",
      "description": "SpiceDB preshared-key, prefixed by Bearer: Bearer \u003cpreshared-key\u003e",
      "name": "Authorization",
      "in": "header"
    }
  },
  "security": [
    {
      "bearer": []
    }
  ],

Closing in favor of https://github.com/authzed/api/pull/120, see https://github.com/authzed/authzed-go/pull/255#issuecomment-2470218649 for an explanation.


Small bug report

## 🐛 Bug Report
Theres a buddy in my server, hes the owner of the server, Old School Bot will allow him to set his rsn but it wont acknowledge that its valid username when he attempts to +stats. We have tried changing his name but it never works. We are out of options and we are coming to you for help

have you considered that the name they are trying to use isnt valid

it would help having the RSN so it can be checked @THCinabag

No further information was provided. Close issue.


adds YUIDoc comments to orm/collection.js

for #658
I can see that the Travis failure I'm experiencing currently is also on at least one other PR. All tests pass locally, not sure what the Travis error is about (something about lodash?).

Yep, thanks I got that fixed.
YOU ROCK! Thx again for all this awesome docs work!
🎶  this girl is on 🔥 🔥 🔥  🎶


ODBC fails when using TRIM(datname)

With the following SQL statement run against a PostgreSQL or RedShift database:
SELECT TRIM(datname) FROM pg_database

I get the following error:
ERROR: MethodError: `copy!` has no method matching copy!(::Type{WeakRefString{UInt8}}, ::ODBC.Block{UInt8}, ::Array{Int64,1}, ::Array{WeakRefString{UInt8},1}, ::Int64, ::Int64)WARNING: Error showing method candidates, aborted

 in stream! at /home/ubuntu/.julia/v0.4/ODBC/src/backend.jl:167
 in stream! at /home/ubuntu/.julia/v0.4/ODBC/src/backend.jl:145
 in query at /home/ubuntu/.julia/v0.4/ODBC/src/backend.jl:256

This only happens when calling TRIM on pg_database.datname, so the following queries work:
SELECT TRIM(name) FROM stv_tbl_perm

SELECT datname FROM pg_database

Other info:
julia> Pkg.status("ODBC")
 - ODBC                          0.4.2

julia> VERSION
v"0.4.5"

Some more info... in backend.jl, Data.stream!,
SELECT datname FROM pg_database

has rb.fetchsize === 5
while
SELECT TRIM(datname) FROM pg_database

has rb.fetchsize === 1

More info, it looks like rb.indcols[col] is of type Array instead of NullableArray, and this is what causes the problem.

I tried changing it to a NullableArray, but that did not fix the issue.

Hey @bluesmoon, have you had a chance to try and reproduce this on Julia 0.5 using ODBC 0.5.1?

No, we stopped using ODBC.jl and instead I wrote a wrapper around PyCall to call a database specific driver depending on the database we use.  We are also unable to upgrade to Julia 0.5 at this point, and are unlikely to do so for at least another year as our customers are change averse.

Understood. Plans are underway for Julia 1.0 to be released this summer, so that could be something to put on the radar as it will be the most feature-complete, long-support release for a while. Many packages are also already planning on targeting long-term support release to coincide.

Thanks, that is good to know.  We might go right from 0.4 to 1.0.

At this point, that's what I would recommend. 0.6 will be released imminently and 1.0 is next up, slated for this summer (maybe late summer before it gets all finalized).


Mocks in monorepo strategy

Posted to discordapp here:
https://discordapp.com/channels/102860784329052160/103622435865104384/696809510412484719
I have a monorepo with several packages in it, many with their own __mocks__ folders. I run jest from the monorepo root to test all packages inside of it.
I don't want @scope/package-a to pick up the __mocks__ in @scope/package-b, nor have to be aware of them so as to call jest.unmock() on all of those mocked modules.
Is there a best practice or strategy when it comes to a setup like this?

I think automocking may just not be for you with this setup. And with manual mocking, since you'd normally only ever import or jest.mock the top level @scope/package-a and not paths within it, only the main / index.js file of the package having an __mocks__ equivalent or not will matter.

@jeysal Sorry about the late reply. I don't mind this particular issue being closed. It might be considered a duplicate of this one, and that's my bad: https://github.com/facebook/jest/issues/9904


Ato 626 porting

Proposed changes:

Moving the commits from PR https://github.com/RasaHQ/rasa/pull/11902/commits into 3.4.x.

@sanchariGr I would hold off from merging this PR until Monday because of this thread.

@sanchariGr I would hold off from merging this PR until Monday because of this thread.

Ah thanks for pointing me to the thread @ancalita , i'll hold off.


Should react be devDependancy?

I wanted to play a bit with react-router, namely enhace getComponents handling to actually pass the state there, as the docs say.
I created new project with react in node_modules.
Then I cloned your repo into node_module/react-router and ran npm install inside and built it with scripts/build.sh
What happened was, when I tried to webpack my project it used react from node_module/react-router/node_modules/react (not from node_module/react).
This caused error very hard to debug, which root cause was having two reacts in one app. Error message looked like this:
Uncaught TypeError: Cannot read property '_currentElement' of nullReactCompositeComponentMixin._updateRenderedComponent @
Um.js:11903ReactCompositeComponentMixin._performComponentUpdate @
Um.js:11885ReactCompositeComponentMixin.updateComponent @
Um.js:11801ReactPerf.measure.wrapper @
Um.js:3587ReactCompositeComponentMixin.performUpdateIfNecessary @ Um.js:11698
...

One way I could get rid of this bug was to wrap
  this.setState({ isTransitioning: true });

    runTransition(this.sta....

inside Router._updateState() in setTimeout(..., 0)
The other way was to delete node_module/react-router/node_modules/react
I'm not sure if there's any way to prevent all this. I'm sure tests use this devDependancies react.

I've just checked it and it works fine for me. Instead of copying the react-router repo to node_modules of your app and running build.sh I would recommend just specifying the path to your local react-router in the package.json of you app like this:
"react-router": "file:PATH_TO_YOUR_LOCAL_VERSION"

Here are more options on how to do it.

Using npm link should work as well

Thanks for the advice!


when git clone url，there are some errors!

requests_deps directory can not download.

I am not sure what problem you're facing, or what you've tried, or if this is related to vundle in any way.


A question about the training dataset

I have a question when I make the training dataset. I want to use my own dataset, but my dataset is  in coco format，So I converted the dataset to voc format and use the class VOCDataSet. But when I begin to train, there was a strange problem. It fills up all the video memory，but not use and not report any errors.I am eager to get your help.

First, you could reduce the input size to avoid Out-of-Memory issue. Then, you are expected to check the dataloader can access the PASCAL VOC data.


⚠ Bofrost.lu has degraded performance

In cbcd20c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1024 ms

Resolved: Bofrost.lu performance has improved in 42f8f75 after 2 hours, 7 minutes.


Set Content-Type when upload a file

I successfully upload a file to my bucket in GCStorage.
The question is that I do not see how can I set the max-age parameter in the metadata using the PHP API.
Here is the code I use:
$file = fopen($_SERVER['DOCUMENT_ROOT']."/uploads/example_50.jpg", 'r');
$bucket = $storage->bucket("my_bucket");
$object = $bucket->upload($file, [
          'name' => "example_50.jpg"
       ]);
$object = $bucket->object("example_50.jpg");
$object->update(['acl' => []], ['predefinedAcl' => 'PUBLICREAD', 'max-age' => 36000]);

As you can see, I have tried it but no success.
Any idea?

I think it's more appropriate to file this on google-cloud-php repo. Can you open an issue there?


ENG-1466: Pull in updates from upstream repo

Description
Summary of changes:

Remove Travis-CI pipeline in favor of GitHub Actions Workflow
Rename scheduler tag variable name from resources_tag to scheduler_tag
Changes to variable naming in Lambda function
Bump minor version of Lambda function archive file (from 3.0.1 -> 3.1.3)
Bump versions of several Python package dependencies
Rename examples/instance-schedule directory to examples/instance-scheduler
Replace custom lambda_logging_policy with AWSLambdaBasicExecutionRole role policy attachment
Fix lambda_logging role policy collision
Add missing tags to AWS resources

Motivation and Context
We haven't pulled in upstream changes since Nov 19, 2020, so this was due for an update.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have linted my code locally prior to submission.
[x] I have updated the documentation accordingly.

Related ClickUp Task
ENG-1446

Task linked: ENG-1466 Pull in upstream changes on twopt/terraform-aws-lambda-scheduler-stop-start

LGTM, but I do see there is a merge conflict.

Fixed. I'm gonna clean up my commits before merging and open a PR on the upstream repo.

LGTM, but I do see there is a merge conflict.

Fixed. I'm gonna clean up my commits before merging and open a PR on the upstream repo.

@Andrew-Kulpa nevermind. I ran into some issues with that fix. The merge conflict will have to be resolved manually while merging the PR.

Why do we need our own fork of this?

@bnweiss Thank you for asking this! We actually don't need it anymore! I was so focused on figuring out what to do about the merge conflict that I didn't stop to question whether this fork was still necessary. The reason I created this in the first place was to patch a permissions issue that was preventing the Lambda from creating CloudWatch log streams. The patch replaced the then broken lambda_logging_policy with the AWS managed service-role AWSLambdaBasicExecutionRole. 8461925 fixed the permissions issue and is actually better than what I changed a year ago because it scopes down the logging permissions to just the CloudWatch log group this module creates (AWSLambdaBasicExecutionRole's scope is too permissive because it lets the Lambda create any kind of logging resources it wants).

Why do we need our own fork of this?

@bnweiss Thank you for asking this! We actually don't need it anymore! I was so focused on figuring out what to do about the merge conflict that I didn't stop to question whether this fork was still necessary. The reason I created this in the first place was to patch a permissions issue that was preventing the Lambda from creating CloudWatch log streams. The patch replaced the then broken lambda_logging_policy with the AWS managed service-role AWSLambdaBasicExecutionRole. 8461925 fixed the permissions issue and is actually better than what I changed a year ago because it scopes down the logging permissions to just the CloudWatch log group this module creates (AWSLambdaBasicExecutionRole's scope is too permissive because it lets the Lambda create any kind of logging resources it wants).

@Andrew-Kulpa are you cool with me closing this PR and replacing anything that's using this fork with the upstream module?

Why do we need our own fork of this?

@bnweiss Thank you for asking this! We actually don't need it anymore! I was so focused on figuring out what to do about the merge conflict that I didn't stop to question whether this fork was still necessary. The reason I created this in the first place was to patch a permissions issue that was preventing the Lambda from creating CloudWatch log streams. The patch replaced the then broken lambda_logging_policy with the AWS managed service-role AWSLambdaBasicExecutionRole. 8461925 fixed the permissions issue and is actually better than what I changed a year ago because it scopes down the logging permissions to just the CloudWatch log group this module creates (AWSLambdaBasicExecutionRole's scope is too permissive because it lets the Lambda create any kind of logging resources it wants).

@Andrew-Kulpa are you cool with me closing this PR and replacing anything that's using this fork with the upstream module?

So sorry for not seeing this lol. Yes, definitely!


Error on subtleBrackets.bracketPairs

I wanted to report an error I get when trying to customize the subtleBrackets.bracketPairs setting.
As soon as I copy the default setting over to the user setting, I get the error message:

Incorrect type.Expected "string[]"

What I wanted to do is extend subtle-brackets to handle:

double quotes "
single quotes '
tags < >

Hi! Thanks for reaching out and sorry for the delay. The incorrect type error should now be solved (after updating to the latest Subtle Brackets version).


tts: 0.0.12 -> 0.0.13

Motivation for this change
https://github.com/coqui-ai/TTS/releases/tag/v0.0.13
❯ ./result/bin/tts --text "To help with the large amounts of pull requests, we would appreciate your reviews of other pull requests, especially simple package updates. Just leave a comment describing what you have tested in the relevant package/service. Reviewing helps to reduce the average time-to-merge for everyone. Thanks a lot if you do\!"
 > Downloading model to /home/hexa/.local/share/tts/tts_models--en--ljspeech--tacotron2-DDC
 > Downloading model to /home/hexa/.local/share/tts/vocoder_models--en--ljspeech--hifigan_v2
 > Using model: Tacotron2
 > Generator Model: hifigan_generator
Removing weight norm...
 > Text: To help with the large amounts of pull requests, we would appreciate your reviews of other pull requests, especially simple package updates. Just leave a comment describing what you have tested in the relevant package/service. Reviewing helps to reduce the average time-to-merge for everyone. Thanks a lot if you do!
 > Text splitted to sentences.
['To help with the large amounts of pull requests, we would appreciate your reviews of other pull requests, especially simple package updates.', 'Just leave a comment describing what you have tested in the relevant package/service.', 'Reviewing helps to reduce the average time-to-merge for everyone.', 'Thanks a lot if you do!']
 > Processing time: 9.314172506332397
 > Real-time factor: 0.4426849905259117
 > Saving output to tts_output.wav

https://shells.darmstadt.ccc.de/~hexa/tts-0.0.13.wav
Things done

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

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

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

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

1 package built:

tts

I tested also the 2 new models. Here are examples: tts_output.zip


Print the HTTP status & the elapsed time in the console.

it's merged. Thank you very much!

https://github.com/Bailador/Bailador/commit/8e2a7ba0e6b4718cfc50fa80c53c85d0d73d297e


Typo in gisticOncoPlot

> gisticOncoPlot(gistic, showTumorSampleBarcodes=T)
Error in text(x = 1:nrow(nm), y = par("usr")[3] - 0.2, labels = rownames(nm),  :
  object "SampleNamefontSize" not found

I assume that meant fontSize alone, according to the function signature.

Ah, sorry.. code migration from ggplot to base graphics has led to some discrepancies. I have fixed it. Could you please try again from GitHub ?

Thanks, this is fixed, however a new problem surfaced (will open a separate issue for it).


Full game save / persistence / snapshot mega-issue

This is a collection of issues/tasks that need to be addressed for full game saves to work properly. Some of these are engine issues, but a decent number of the problems are in content, so I've opened it here.
See also:

space-wizards/RobustToolbox/issues/2406
space-wizards/RobustToolbox/issues/4856

Issues:

[ ] Nullspace entity and multi-map support space-wizards/RobustToolbox/pull/5572
[ ] NodeNets like power and atmos need to be turned into entities.
[ ] Several prototypes are marked as non-saveable via MetaDataComponent.MapSavable, which includes most mobs. I don't know specifically why they are marked as such, but that needs to be fixed.
Some system data is not serialized / not stored on entities

[ ] Device Nets #26424
[ ] Wire layouts?
[ ] Game Ticker / Round flow

Might've already been fixed? but it and various rule systems used to have fields referencing specific entities/rules or maps.

Some components likely store data in non-serialized fields

[ ] Hands? #20776
[ ] Need to check that mind/actors work well when loading entities.

Might largely be fixed by null-space entity auto inclusion #34020
See also #20773
What happens if an entity with a player attached is saved, deleted, and then loaded again?
What happens if it gets loaded multiple times?
What if the player disconnects and reconnects during the deletion & loading?

[ ] space-wizards/RobustToolbox/issues/3119
[ ] How do we handle BUIs?

If we save them we need to ensure all the BUI state is yaml-serialisable.
If we don't save them, for consistency we should probably first close all buis before saving.

[ ] Prototype uploads

The full-save system should add an extra entry for listing all uploaded prototypes

[ ] Resource uploads

The save file should also reference any uploaded resources, and the saving system should write them to files alongside the save file.

CTRL+F & fix:

[ ] Timer.Spawn()

Maybe fine for some "out-of-game" stuff like votes or ghost-role raffles?

[ ] // TODO full game save

And then we need to thoroughly test this somehow. Catching all the odd bits and bobs that aren't saved properly will be difficult. If admins are feeling sadistic they could just save & reload an ongoing round and collect the hate mail. Though we should probably start by saving a real round and trying to load it locally.

mobs

i think mostly its so you dont map mobs and instead use the spawner, having a tested do not map category would help with this
but when i tested it out, saving a urist wrote all its bodyparts etc in the yml even if they arent damaged or anything, could just be a mapinit thing

game will throw if its not at least NetSerializable, i dont think any states arent Serializable too

I've edited the point to clarify: I meant yaml-serializable, not net-serializable. We could try use net-serialization, but that would generally prevent loading an older save with a newer version, for the same reason the replay client version should to be the same as the actual game client version for loading a replay (or at least, needs the same registered net serializable C# types).


[2.0.0-alpha.7] https import missing in es modules build

The https import seems to be stripped out of the es modules build:
<EMAIL_ADDRESS>So any https url requests are throwing ReferenceError: https is not defined

@TimothyGu you might want to take a look at this, seems like after #243 I no longer see https in the index.es.js build. Am I missing some installation steps before npm publish?

Well, I have absolutely no idea why it's failing. But it doesn't matter anymore, since I'm tired of pretending Node.js core modules were designed to be loaded with import, so I just changed all the imports to require()s for core modules.

released as alpha.8, thx all involved!

Why not just switch to native ES6 with node 6 requirement? Node 8 will be LTS in few days, but mixed import/require syntax looks horrible...

@tinovyatkin because import is what causes this issue, and no one can figure out why rollup/babel decides to strip a particular native module import (in this case https)
So yes, we decide having thing work is way better than having thing look nice but fails for unknown reasons.

Not using import at all is probably a more sane choice ultimately. ES Modules really aren't a thing and are generally just transpiled out by babel/typescript/fooTranspiler. I guess webpack and rollup have their own magic but since the implementation spec is still wayyyyyyy up in the air.. and when es modules finally hit node they will be via .mjs file extension.. and since this is a node.js package... just trying to get to the logical conclusion...

@jkantr Yes I know the semantics difference between Babel and the spec and I know how the ESM loader in Node.js will work (in fact I helped implement a part of it). The decision to use Rollup here is purely so that when we publish the module we only publish one file, which helps for memory usage etc., while keeping our internal source tree intact. Browserify does this too, but it adds a bunch of function wrappers that are really unneeded.


ENH: Allow generation of multiple vascular territories maps

Enable generation of several Vascular Territory label maps, which can be selected from GUI widget.
This fixes #217
Fixes #240

Changes in the widget .ui file will be syncronized with #232 and #230 when merging this PR to develop branch.

This fixes ALive-research/Slicer-Liver#217
This fixes ALive-research/Slicer-Liver#240

I ge the following errors in the python console with this PR
[VTK] vtkMRMLColorTableNode::GetColor: requested entry -1 is out of table range: 0 - 257, call SetNumberOfColors
[VTK] ERROR: SetColorName, index was out of bounds: -1, current size is 257, table name = SlicerLiverColorMap

Thanks Rafael! I will fix the error msg. and update the PR.


Panic in regex_syntax, found by fuzzing

While fuzzing the regex_syntax target from https://github.com/rust-fuzz/targets, I hit this panic:
thread 'main' panicked at 'tried to unwrap expr from HirFrame, got: Group { old_flags: None }', /root/.cargo/registry/src/github.com-1ecc6299db9ec823/regex-syntax-0.6.2/src/hir/translate.rs:203:18

Here is the regex:
regex::Regex::new("(((?x)))");

playground

Nice! It looks like ((?x)) also panics. (?:(?x)) does as well. As does ((?x))x.


New fixture - Chauvet-Rogue-RH1-Hybrid

http://www.chauvetprofessional.eu/products/rogue-rh1-hybrid/

Awesome, thanks !
Just one minor thing. When Pan and Tilt are present, the Focus type should be "Head" and not "Fixed":
<Focus Type="Fixed" PanMax="540" TiltMax="260"/>


Automator: update common-files@master in istio/istio@master

Generated by Automator - 2023-06-09T01:55:25+00:00

/retest

/rebase

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
576067aa2c27341082413a98638940bd42dad1f8
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Can't use compiler in strict mode

What problem does this feature solve?
We cannot use the compiler in strict mode because it uses the with statement.
https://github.com/vuejs/vue/blob/master/src/compiler/codegen/index.js#L47
I have to compile templates in the browser at run time in strict mode.  Anybody have a solution for this?
What does the proposed API look like?
Change all with statements to another solution.

The compiler works fine in strict mode. Or maybe you mean you have a strict CSP? Either way, you need to provide more information or a reproduction because the in-browser compilation works regardless of strict mode or not.

@yyx990803  I posted this issue before seeing https://github.com/vuejs/vue/issues/3832.  That gave me much more clarity, and I did the hack that was suggested there which caused my code to work. However I did not try the new Function syntax, does that make strict mode work?

I’m also working in salesforce on the same problem as described above in the other issue.   I am using run-time compilation.  I was getting the error because of the with keyword.   If I add new Function around the with keyword to the library will that work? And is there a better practice way to do this?

@yyx990803 I changed the library to this:
return {
    render: new Function("with(this){ return " + code + "}"),
    staticRenderFns: state.staticRenderFns
  }

Salesforce still threw an error saying "Strict mode code may not include a with statement"


Team 1 death screen statistics

Description
Updated the UI for better visibility of the text.
Fixes / Closes # (issue)
#541
Type of change

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

The game crashes when the player die in the first room, please review your code

doesn't happen anymore, added a null check


[ISSUE#2080] Add unit test for LocalPluginController

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

welcome

I'm


//fix command crash!!

I try to fix skywars from Java edition of i say //Wand the PocketMine was crash plss help

can you provide more information? like a crashdump file...

I can. Its due to:

Fixed in Pre-Release branch.


Warning when accessing meta robots

salut h0gar ;)
using your great lib, I want to say you I got a warning while extracting a meta tag :
<meta name="robots" content="index,follow" />

$doc = new \H0gar\Xpath\Doc($html, 'html');
$meta_robots = $doc->item('//meta[@name="robots"]/@content');
echo $meta_robots->text();

Everything woks well since it extracts text but I get this warning :
Warning: DOMNode::appendChild(): Couldn't append node in /lib/h0gar/xpath/Node.php on line 218

J'espère que tu vas bien par ailleurs, je suis sous l'eau au niveau du taf ! A bientôt
Hugo

Hey Hugo
Nice to see you are using my libs :)
Thanks for opening an issue. Just fixed it.
Otherwise you could also do:
echo $doc->item('//meta[@name="robots"]')->attr('content');

Bon courage et à bientôt !

Thank you !
Just another question relative to your code update. When doing :
php composer.phar update h0gar/xpath
Loading composer repositories with package information
Updating dependencies (including require-dev)
Nothing to install or update

My composer.json : "h0gar/xpath": "*",
Is it because your library version hasn't changed or update is not already available for composer ? I'll try later or directly add github repo in composer.json. Do you have any idea ?

Probably because I had not published a new release yet. Try again. Otherwise, if you were using the dev compatibility mode it would have used the last commit (but it's less stable).

Update Successful, thank again !


Rename package to eslint-plugin-svelte

This PR rename the package from @ota-meshi/eslint-plugin-svelte to eslint-plugin-svelte.
There is the following remaining work.

[x] Add migration guide.
[x] Drop support for Node12. #161
[x] Remove deprecated rules. #162

Related to #157

Thank you for your review. I changed this PR. So please check it again.


linux_ck: init at 5.12.0

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 128897 at a8a70f58 run on aarch64-linux 1

45 packages failed to build:

linuxPackages_ck.acpi_call
linuxPackages_ck.akvcam
linuxPackages_ck.anbox
linuxPackages_ck.apfs
linuxPackages_ck.batman_adv
linuxPackages_ck.bcc
linuxPackages_ck.can-isotp
linuxPackages_ck.cryptodev
linuxPackages_ck.ddcci-driver
linuxPackages_ck.digimend
linuxPackages_ck.ena
linuxPackages_ck.evdi
linuxPackages_ck.fwts-efi-runtime
linuxPackages_ck.gcadapter-oc-kmod
linuxPackages_ck.hid-nintendo
linuxPackages_ck.jool
linux_ck (linuxPackages_ck.kernel)
linuxPackages_ck.lttng-modules
linuxPackages_ck.mba6x_bl
linuxPackages_ck.mbp2018-bridge-drv
linuxPackages_ck.mwprocapture
linuxPackages_ck.mxu11x0
linuxPackages_ck.netatop
linuxPackages_ck.oci-seccomp-bpf-hook
linuxPackages_ck.odp-dpdk
linuxPackages_ck.openafs (linuxPackages_ck.openafs_1_8)
linuxPackages_ck.openafs_1_9
linuxPackages_ck.openrazer
linuxPackages_ck.r8125
linuxPackages_ck.r8168
linuxPackages_ck.rtl8192eu
linuxPackages_ck.rtl8723bs
linuxPackages_ck.rtl8812au
linuxPackages_ck.rtl8821cu
linuxPackages_ck.rtl88x2bu
linuxPackages_ck.rtw88 (linuxPackages_ck.rtlwifi_new)
linuxPackages_ck.sch_cake
linuxPackages_ck.systemtap
linuxPackages_ck.tuxedo-keyboard
linuxPackages_ck.v4l2loopback
linuxPackages_ck.veikk-linux-driver
linuxPackages_ck.vhba
linuxPackages_ck.xpadneo
linuxPackages_ck.zfs (linuxPackages_ck.zfsStable)
linuxPackages_ck.zfsUnstable

2 packages skipped due to time constraints:

linuxPackages_ck.dpdk-kmods
linuxPackages_ck.rtl8821ce

6 packages built successfully:

linuxPackages_ck.cpupower
linuxPackages_ck.hyperv-daemons
linuxPackages_ck.perf
linuxPackages_ck.stdenv
linuxPackages_ck.tmon
linuxPackages_ck.usbip

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

What to do with the kernel?

It is not great that so much fails to build with it.

It is not great that so much fails to build with it.

Close PR?


Is magic ui still in Beta?

Hi @dillionverma ,
I have noticed the badge beta beside the project name, was that kept intentionally, or just forget to remove it?
Thank you,

Hi! Thanks for noticing this :)
I would say it's still in beta for now until we've reached a stable state for the components and their type definitions.
We're still actively working towards that for now but will announce when out of beta!


[Win] Generate "FocusLost" events when window is grabbed and moved?

AfaIk there's no way to handle this problem (at least on Windows) at the moment. It would be really useful.
The only workaround I know is the way Rickety Raquet (http://www.ricketyracquet.de/) does it:
Prevent the user from using the mouse completely, so he's not able to move the window until he opens up the menu.
Is there a more simple solution? And if not, is it worth to become a feature?

Could you clarify this a bit? What's the actual issue? Do you get FocusLost in case you move the window around? Or do you want to get the event if you try to move it?
Can you provide a minimal source code example to demonstrate the issue? Closing this till then. You should discuss issues on the official forum and open a bug once you've confirmed it's indeed a problem.

Ok, I'll move this to the forum.

Ok, feel free to post here again once there's been some consensus, so we can reopen/update this.


KeyError: 'RANK' occered when trained in single gpu

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
A clear and concise description of what the bug is.
Reproduction

What command or script did you run?

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
What dataset did you use?

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!
I try to reference https://mmdetection.readthedocs.io/zh_CN/v2.21.0/2_new_data_model.html began training, but error occurred:
D:\python_code\mmdet_2\mmdetection>python ./tools/train.py
D:\ProgramData\Miniconda3\envs\mmdet_2\lib\site-packages\mmengine\utils\dl_utils\setup_env.py:46: UserWarning: Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
warnings.warn(
D:\ProgramData\Miniconda3\envs\mmdet_2\lib\site-packages\mmengine\utils\dl_utils\setup_env.py:56: UserWarning: Setting MKL_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
warnings.warn(
Traceback (most recent call last):
File "./tools/train.py", line 135, in
main()
File "./tools/train.py", line 124, in main
runner = Runner.from_cfg(cfg)
File "D:\ProgramData\Miniconda3\envs\mmdet_2\lib\site-packages\mmengine\runner\runner.py", line 443, in from_cfg
runner = cls(
File "D:\ProgramData\Miniconda3\envs\mmdet_2\lib\site-packages\mmengine\runner\runner.py", line 353, in init
self.setup_env(env_cfg)
File "D:\ProgramData\Miniconda3\envs\mmdet_2\lib\site-packages\mmengine\runner\runner.py", line 656, in setup_env
init_dist(self.launcher, **dist_cfg)
File "D:\ProgramData\Miniconda3\envs\mmdet_2\lib\site-packages\mmengine\dist\utils.py", line 84, in init_dist
_init_dist_pytorch(backend, init_backend=init_backend, **kwargs)
File "D:\ProgramData\Miniconda3\envs\mmdet_2\lib\site-packages\mmengine\dist\utils.py", line 101, in _init_dist_pytorch
rank = int(os.environ['RANK'])
File "D:\ProgramData\Miniconda3\envs\mmdet_2\lib\os.py", line 675, in getitem
raise KeyError(key) from None
KeyError: 'RANK'

do you fix it? in what way?

Hi,
Did you find the fix?


StringSliceFlags has strange behavior

When supplying StringSliceFlag with a default Value, the default value remains in the returned slice even when a value is manually applied.  Example:
Using:

cli.StringSliceFlag{
        Name: "Hostnames, n",
        Usage: "`HOSTNAME`s to listen to.",
        Value: &cli.StringSlice{"localhost"},
}

Running invocation -n somehost returns [localhost somehost] but I would expect it to return only [somehost].

This can be fixed with a simple enough hack by using context.IsSet, but it turns out there is a bug here too.  If I use the above invocation of invocation -n somehost then context.IsSet("Hostnames") == false, however, context.IsSet("n") == true.  This is different from the behavior of the default values, such as Int or String where IsSet would return true whether the alternative switchname was used or not.

That also got me super confused today, glad I'm not the only one :sweat_smile:

Same problem on my end. It also prevents me from using IsRequired: true as it will sometimes fail for the same reason.

Can someone put together a feature request or bug report (templates are here) describing was should actually be done here? An issue titled "X has strange behavior" is hard to triage.

As far as I can tell, this is no longer a problem on current v2. Modified from @michaeljs1990 example:
package main

import (
        "fmt"
        "log"
        "os"

        "github.com/urfave/cli/v2"
)

func main() {
        app := &cli.App{
                Flags: []cli.Flag{
                        &cli.StringSliceFlag{
                                Name:    "Hostnames",
                                Aliases: []string{"n"},
                                Usage:   "`HOSTNAME`s to listen to.",
                                Value:   cli.NewStringSlice("localhost"),
                        },
                        &cli.StringFlag{
                                Name:    "lang",
                                Aliases: []string{"l"},
                                Value:   "english",
                                Usage:   "language for the greeting",
                        },
                },
                Action: func(c *cli.Context) error {
                        fmt.Println("Hostname is set ...", c.IsSet("Hostnames"))
                        fmt.Println("n is set ...", c.IsSet("n"))

                        fmt.Println("land is set ...", c.IsSet("lang"))
                        fmt.Println("l is set ...", c.IsSet("l"))
                        return nil
                },
        }

        if err := app.Run(os.Args); err != nil {
                log.Fatal(err)
        }
}

$ go run ~/tmp/urfave-cli-790b.go -Hostnames somehost -l hi
Hostname is set ... true
n is set ... true
land is set ... true
l is set ... true

Please holler if this problem is still affecting you on v2 🙇🏼


ODataModel.getProperty needs an additional maxDepth Parameter

I have a model with potential circularities (Resource has Assignments, Assignments belong to Demands, Demands have Assignments, Assignments belong to Resource).
A call to ODataModel.getProperty with bIncludeExpandEntries = true could result now in an infinite recursion, since the implementation operates on the data stored in the model.
So if I do expand from Resource to AssignmentsSet/Demand and then in the Detail an expand from a Demand to AssignmentsSet/Resource there can be now a circle in the data in the model.
So please provide a maxDepth Parameter to prevent an infinite recursion.
An alternative would be to provide an expand Parameter to be filled with an expand string like the one used with bindings/reads.

Hi,
i had a brief talk with a responsible developer.
It sounds like a good enhancement.
I will forward it to the product management.
tracked internally<PHONE_NUMBER>
Thanks for pointing this out.
Best regards,
Tobias

I just ran into the same issue in my application:
Uncaught RangeError: Maximum call stack size exceeded
on calling getProperty with bIncludeExpandEntries = true.
So +1 from my side!

Should this be fixed by these commits?
https://github.com/SAP/openui5/commit/b16598a62a01b27b750b3748473789a4b8afc53d
https://github.com/SAP/openui5/commit/3d8660456e821492836be3f9209ed6e3e1f294b3
https://github.com/SAP/openui5/commit/3d8660456e821492836be3f9209ed6e3e1f294b3

Yes, those commits seem to solve this issue.
As the same problem was also reported internally this issue wasn't updated.


How do I put RMT Encoder in IRAM?

In the official documentation for RMT, it is highly recommended to put the encoding function into IRAM:

The encoding function is running in the ISR context. To speed up the encoding session, it’s high recommend to put the encoding function into IRAM. This can also avoid the cache miss during encoding. -- https://docs.espressif.com/projects/esp-idf/en/v5.1/esp32/api-reference/peripherals/rmt.html

But I don't understand how to do this. Neither the esp_idf_hal::rmt module docs, nor anything in esp_idf_sys or the Rust on ESP Book mentions IRAM at all.
How can I move arbitrary Rust functions to IRAM? And will this be possible with the iterator approach that TxRmtDriver is taking?

So the CONFIG_RMT_ISR_IRAM_SAFE config option will handle the C side of putting the required methods in IRAM, so we then just have to worry about the rust side when we detect this option. This isn't trivial at all tbh, it's quite difficult to ensure that all the code/subroutine calls inside a function are all in IRAM too. In my experience its dependent on a number of things, including optimization levels as to whether the functions actually end up in IRAM.

Interesting... it looks like if you can get the function into a section whose name is in the form .iram1.#, there's some magic that copies that function into IRAM.
If I'm reading this correctly, CONFIG_RMT_ISR_IRAM_SAFE only allocates data structures in IRAM:

By adding MALLOC_CAP_8BIT to RMT_MEM_MALLOC_CAPS
By adding ESP_INTR_FLAG_IRAM to RMT_INTR_ALLOC_FLAG

Actually putting a function into IRAM is done by decorating it with IRAM_ATTR, e.g. [static void IRAM_ATTR rmt_tx_mark_eof(rmt_tx_channel_t *tx_chan)](https://github.com/espressif/esp-idf/blob/master/components/driver/rmt/rmt_tx.c#L531). IRAM_ATTR is defined in esp_common/include/esp_attr.h as:
#define IRAM_ATTR _SECTION_ATTR_IMPL(".iram1", __COUNTER__)
#define _SECTION_ATTR_IMPL(SECTION, COUNTER) __attribute__((section(SECTION "." _COUNTER_STRINGIFY(COUNTER))))
#define _COUNTER_STRINGIFY(COUNTER) #COUNTER

I can't find the magic that does the copying, though. Maybe it's in the ROM? @MabezDev, do you know where this magic happens? (grep didn't turn up anything other than linker scripts for me.)
Both C and Rust have the issue of calling from IRAM into non-IRAM; C is just a bit more predictable in its code generation here. The C code does take care not to accept callback parameters it can't verify as being IRAM-safe, e.g. in rmt_tx_register_event_callbacks.

The copying code will be buried deep in the startup code for esp-idf, but provided we also put our Rust functions there, i.e by using #[link_section = ".iram1"] then it should work, provided the code within that function is also in IRAM.

Great research both of you. I will experiment a bit with this. Do you know how to verify what functions are placed in IRAM for a given compiled firmware/app?

Do you know how to verify what functions are placed in IRAM for a given compiled firmware/app?

Yes, you can use nm to find function names and their address, I'm not sure what chip you're using but you'll need to ensure that the address of the function is in the IRAM address space.

Adding on to this, as a note for myself and future spelunkers:
The code that is responsible for moving the function into IRAM is in the second stage bootloader, in process_segment, whose signature is:
process_segment(int index, uint32_t flash_addr, esp_image_segment_header_t *header, bool silent, bool do_load, bootloader_sha256_handle_t sha_handle, uint32_t *checksum)

esp_image_segment_header_t just has a load address and a length:
typedef struct {
    uint32_t load_addr;     /*!< Address of segment */
    uint32_t data_len;      /*!< Length of data */
} esp_image_segment_header_t;

This calls process_segment_data, which bootloader_mmaps the flash data, memcpys it into the destination, then bootloader_munmaps it.
The segment headers are created by the flash utility (e.g. espflash's elf.rs for translating the ELF binary into the firmware format).
I'm a bit surprised with the amount of code around memory mapping into the flash. It handles virtual to physical address translation cases, which I didn't think were a thing on any ESP32 chips; I thought addresses were directly mapped, and the flash accessible by the SPI bus (with cache possibly mediating accesses).
If you're using ESP32-C3/-C6 direct boot (no second-stage bootloader, just ROM into your app code) or have a custom second-stage bootloader, you're responsible for doing all of this yourself.

Original question was answered. Please create a new separate issue for enhancement proposals etc.


miota withdrawal disabled on binance

Issue description
miota withdrawal disabled on binance
Firefly version
v1.2.0
OS details
windows 10
Can the issue be reliably reproduced?
Yes
Steps to reproduce the issue
1.miota withdrawal disabled on binance
2.
3.
Expected behaviour
miota withdrawal disabled on binance
Actual behaviour
miota withdrawal disabled on binance
Errors
miota withdrawal disabled on binance

It's unrelated to Firefly.

according to binance its unrelated to Binance. where is the problem?

Yeah this is Binance's fault, has nothing to do with firefly.

Not an issue with firefly


libusb-compat: fix up the .so so it can find libusb1

Motivation for this change
Applications using libusb-compat do not run after
https://github.com/NixOS/nixpkgs/pull/48434 as libusb-compat is no longer able
to find libusb1.
With this in place, g15daemon is running again.
Things done

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

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

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

Cc: @Infinisil & @lschuermann who worked on the referenced PR.

Thanks for pointing this out. Weird that it now requires setting the RPATH manually, I could've sworn to have tested libusb-compat with the old PR. However that was 2018, my brain may just confuse some things.

Thanks for looking into this @lschuermann. I'm merging this into staging due to the rebuilds.


The training speed is very slow

Hello, thank you so much for your excellent work！
I made my own dataset using colmap, but training this dataset was very slow (30,000 iterations took one day), what could be the reason for this phenomenon? What should I do? I look forward to your reply. Thank you very much.

I hope you can help. Thank you very much

I had a similar training speed on one machine where the training was CPU limited, so you should check your GPU load. If it only reaches the expected value intermittently, as opposed to hovering above 95% most of the time, then this is likely the problem.
Try launching NeuS on a different machine if this indeed is the case.

I had a similar training speed on one machine where the training was CPU limited, so you should check your GPU load. If it only reaches the expected value intermittently, as opposed to hovering above 95% most of the time, then this is likely the problem.
Try launching NeuS on a different machine if this indeed is the case.

你好，感谢你的回复！在训练我自己的数据集的时候，GPU和CPU都一直保持着较低的占用率。只有运行内存占用较高，大约60%（25G）。因此推测可能是因为训练模型没有调用GPU而导致训练速度慢。但是我不知道在制作数据集的过程中的什么原因为会导致“训练速度慢”这个问题的出现。你能帮助我吗?

Hello, thank you for your reply! While training my own data set, both the GPU and CPU were kept low. Only the running memory usage is higher, about 60% (25G). Therefore, it is speculated that the training speed may be slow because the training model does not call the GPU. But I don't know what caused the "slow training" problem during the production of the data set. Can you help me?

Well, you've found the reason then, more or less. You need to supply the data to the GPU such that it remains fully loaded. Ultimately my solution was just to use the same environment on a different machine, so I can't point you to anything precise.
Does your environment does match the requirements.txt, by the way? If not, try to build one that does.
Also you could try increasing the batch size. This didn't work for me. If for you it doesn't work either, you'll have to profile the code to see which parts consume the most time, and then try to cut down on them.

Well, thank you for your answer. It's very helpful to me.

Well, thank you for your answer. It's very helpful to me.谢谢你的回答对我很有帮助。

I am also facing a similar situation as you. Have you solved the problem？


Add turtlebot3_navigation as a dependency in the robot workspace for rviz

Add turtlebot3_navigation as a dependency in the robot workspace for rviz
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This requires change to the CMakeLists to add the rviz model into a discoverable location.


config enable quantile_state type by default

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.94 seconds
load time: 444 seconds
storage size:<PHONE_NUMBER>8 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221118114957_clickbench_pr_48693.html


[MM-29403] migrate string-refs in custom_theme_chooser

Summary

Replace this.refs to using React.createRef in custom_theme_chooser.jsx
Ticket Link

Fixes https://github.com/mattermost/mattermost-server/issues/15842
JIRA ticket: https://mattermost.atlassian.net/browse/MM-29403
Related Pull Requests

#5753

Hello @filipghorbani,
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.

@filipghorbani Can you also remove /* eslint-disable react/no-string-refs */ instances from the file

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich

@filipghorbani sorry it took me a while to come back to your PR, looks good but it is now conflicting with another PR removing jquery. can you update this to fix the conflict?

@filipghorbani let me know if I can help you with the conflict somehow! I'm happy to help

closing for inactivity, feel free to ask to work on it again and thanks for the efforts :)


Create integration test for AppRun-Leaflet

Create the integration tests for all the components to this version

I have this issue when I run the server

@Dangelo-JAN, please could you push the changes to the branch so I can have a look.

Push done


Added bash_aliases to fileTypes

It appears the format/structure has changed. will resubmit to the new format.


Improve tasks on pre-commit hook

Changes

Upgrade husky to v7
Upgrade lint-staged to v11
Avoid checking types on pre-commit
Format files other than JS/TS on pre-commit hook

Context

Closes #211

I'd say we're good to go merge now?


Put ENABLE_GOOGLE_AUTH setting back

Fixes:

#267
#345
#349

Ready to merge as of 04c27c28d804bd31753b01d05dc818d52ed432ca?

lgtm! :rocket:


Can we add Java support

Category
Cedar language features
Describe the feature you'd like to request
I would like to use cedar in Java
Describe the solution you'd like
Cedar working in Java natively
Describe alternatives you've considered
I have tried nothing
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

We do have java bindings available, see: https://github.com/cedar-policy/cedar-java
We do not have the cedar evaluator implemented natively in java. Is this sufficient?

Ah I missed that, thank you!


OAIHarvest: resumptionToken fix

FIX Fixes the parsing of resumptiontoken in incoming OAI-PMH XML
which could fail when the resumptiontoken was empty.

Signed-off-by: Jan Aage Lavik<EMAIL_ADDRESS>See master version in: #3297
@aw-bib

Tested-by: CDS :shipit:

Superseded by #3325

Well, I don't think it is superseded per se. This patch applies to the legacy branch and #3325 applies to the master branch.
Should we keep this one open and pending to integrate in 1.2.2?

May I raise a hand that it should also go to 1.1-maint?

Yes you are right, I'm changing the milestone and reopening
cc @tiborsimko

@egabancho See https://github.com/inveniosoftware/invenio/pull/3297 for the legacy PR I created at the same time as this one. So if it's okay for you we can close this again :)

@egabancho Nevermind, I mixed it up. This is indeed the right PR and #3297 can be closed. Sorry for the mishap :boom:

@jalavik can you rebase and add Tested-by: Esteban?

@kaplun Done. #2793 is also ready.

Merged to maint-1.1, maint-1.2, legacy.


set pure=False in call to Dask to avoid hashing buff

This is a fix for https://github.com/limix/pandas-plink/issues/35.

I'm about to release a version that tackles that. Thank you for the help!


fix(lambda): module dependency had changed the output name of lambda arn

Hey @phundisk 👋 Would be great to get this merged, if you have time 🙇
The module is in a broken state at the moment


Problems in pure-op-based mode

$ LDB_MODE=pure_op_based make shell

If I do this and wait long enough, I see:
[warning] Unhandled info message: check_resend

Ignoring that, I can't use a gcounter:
1> ldb:create(a, gcounter).
ok
2> 2> ldb:update(a, increment).
17:35:15.028 [error] gen_server ldb_pure_op_based_backend terminated with reason: no function clause matching ldb_pure_op_based_backend:encode_op(increment) line 415
17:35:15.041 [error] CRASH REPORT Process ldb_pure_op_based_backend with 1 neighbours exited with reason: no function clause matching ldb_pure_op_based_backend:encode_op(increment) line 415 in gen_server:terminate/7 line 826
** exception exit: {{function_clause,
                        [{ldb_pure_op_based_backend,encode_op,
                             [increment],
                             [{file,
                                  "/home/user/Downloads/ldb/_build/default/lib/ldb/src/ldb_pure_op_based_backend.erl"},
                              {line,415}]},
                         {ldb_pure_op_based_backend,handle_call,3,
                             [{file,
                                  "/home/user/Downloads/ldb/_build/default/lib/ldb/src/ldb_pure_op_based_backend.erl"},
                              {line,154}]},
                         {gen_server,try_handle_call,4,
                             [{file,"gen_server.erl"},{line,629}]},
                         {gen_server,handle_msg,5,
                             [{file,"gen_server.erl"},{line,661}]},
                         {proc_lib,init_p_do_apply,3,
                             [{file,"proc_lib.erl"},{line,240}]}]},
                    {gen_server,call,
                        [ldb_pure_op_based_backend,
                         {update,a,increment},
                         infinity]}}
     in function  gen_server:call/3 (gen_server.erl, line 212)

Changing encode_op and decode_op seems to solve it.
%% @private
encode_op({add, E}) ->
    {1, E};
encode_op(Op) ->
    Op.

%% @private
decode_op({1, E}) ->
    {add, E};
decode_op(Op) ->
    Op.

About the check_send, it seems that its handler is handle_cast and it should be handle_info.
However, doing that change, seems to break the ldb:update.

Ok. Will take a look at it.

#12


Compatibility with IDA 7.4 and Python3

The 'idc.Demangle' function can be replaced by 'idc.demangle_name' and the 'GetLongPrm' function by 'get_inf_attr'. However, more things such as 'xrange' -> 'range' should be fixed, and the UI objects are missing functions or attributes. Could you change the missing things such that the plugin works with IDA 7.4 and Python3?

Thank you for pointing it out.
But unfortunately, I don't have a license for latest IDA Pro 😢
If you create PR for this issue, I appreciate it.

I created a pull request with the script that seems to work in my IDA 7.4 installation! :)


Update issue templates

This PR was closed  because it did not meet the branch naming convention. A new one has been raised.


XDM shared state APIs

A follow up to https://github.com/adobe/aepsdk-core-ios/pull/421
This PR implements XDM shared state APIs as described in this doc and based on feedback in PR #421: https://wiki.corp.adobe.com/pages/viewpage.action?spaceKey=adms&title=XDM+Shared+States
These APIs will be used for extensions to write XDM data to shared state, then the Edge extension will read this data and attach it to outgoing Platform events.
Note: The added tests are pretty light, I feel that we have adequate tests for SharedState logic, however, I've added a test to ensure that we are reading/writing to the correct SharedState instance based on the SharedStateType.

@pfransenadb do you want to quickly review the change in this PR?

@pfransenadb  @emdobrin
The reason I don't want to make the parameter optional or provide another overload API, is 1) to reduce the chance a developer might make a mistake where they only update one of the shared states, and 2) also keep the Core API clean and simple.

Let's keep the old API and add the other two for xdm support.

@emdobrin most of your feedback is based on the premise that we will add separate APIs for the XDM shared state. Given the current state of our conversation, it doesn't sound like @shalehaha is onboard. I'm going to hold off on addressing your feedback until we can come to a consensus.


dist folder missing

Unlike vue-table (Version 1), it doesn't have the dist folder. If I want to add your script directly to my html page, how can I do that?

I think it will be in the next release. You could look on the dev branch in the dist folder and use the js from there for now.

@cristijora how would i go about getting a bootstrap version of that dist ?


[WIP] [DO NOT MERGE] Adding Support for SCL Apache

Added the config/apache.yml file
Leverage MiqApache module from manageiq-gems-pending
Updated references to /etc/httpd using MiqApache module methods
Updated rspecs

PRs for ManageIQ SCL Apache Support:

Gems Pending - https://github.com/ManageIQ/manageiq-gems-pending/pull/114
ManageIQ - https://github.com/ManageIQ/manageiq/pull/14693
Appliance - https://github.com/ManageIQ/manageiq-appliance/pull/117
Appliance Build - https://github.com/ManageIQ/manageiq-appliance-build/pull/208

closing in preference to keep using distribution apache with the following config file change: https://github.com/ManageIQ/manageiq-appliance/pull/126


External repositories not needed anymore in README

Currently, the following code can be found inside the README:
  <repositories>
    ...
    <!-- BEGIN: Dapr's repositories -->
    <repository>
      <id>oss-snapshots</id>
      <url>https://oss.sonatype.org/content/repositories/snapshots</url>
    </repository>
    <repository>
      <id>oss-release</id>
      <url>https://oss.sonatype.org/content/repositories/releases/</url>
    </repository>
    <!-- END: Dapr's repositories -->
    ...
  </repositories>

As far as I can see and have tested, these repositories are no longer needed to download the SDK using Maven, as Dapr has now been uploaded to Maven Central.
I'd suggest removing this entry then from the README.

@delvh Please, create a PR.

I have seen that the same might apply to Gradle:
    ...
    // Dapr repositories
    maven {
      url "https://oss.sonatype.org/content/repositories/snapshots"
	  mavenContent {
	    snapshotsOnly()
	  }
    }
    maven {
	  url "https://oss.sonatype.org/content/repositories/releases/"
    }

Can someone test whether its unnecessary there too, please?
I am not too familiar with gradle. I'd assume it to be true there too, but I don't know for sure whether it is.
Because if that is the case, I'd remove both at once.


Internal compiler error with conv2d_transpose

Hi everyone,
I'm trying to accelerate an autoencoder network on an EdgeTPU and have run into some issues compiling a model that uses a transpose convolution for upsampling. The Coral documentation says transpose convolutions are supported as of the compiler runtime version 13.0, which I have installed. TFLite is able to build a quantized graph, but when I run the EdgeTPU compiler, it fails with an "Internal compiler error."
Some pertinent system information:
tensorflow version: 1.15.0, python3.7
OS: Ubuntu 18.04
edgetpu_compiler version: 13.0
The following code reproduces the error with some dummy quantization values added. Tensorflow is able to build the TFLite model, but it fails when trying to compile for the EdgeTPU:
import os
os.environ['TF_CPP_MIN_LOG_LEVEL'] = '3'

import tensorflow as tf

if __name__ == '__main__':
    batch_size = 1
    tflite_file = "test.tflite"

    input_tensor = tf.compat.v1.placeholder(tf.float32, [batch_size, 32, 32, 6], name="input_tensor")
    output = tf.contrib.slim.conv2d_transpose(input_tensor, 32, (1, 1), 2, padding="SAME", activation_fn=tf.identity)

    inputs = [input_tensor]
    outputs = [output]

    with tf.compat.v1.Session() as sess:
        sess.run(tf.compat.v1.global_variables_initializer())

        converter = tf.lite.TFLiteConverter.from_session(sess, inputs, outputs)
        converter.inference_type = tf.uint8
        converter.default_ranges_stats = (0, 255)
        converter.quantize_weights = True
        converter.change_concat_input_ranges = True
        input_arrays = converter.get_input_arrays()
        converter.quantized_input_stats = {input_arrays[0]: (127.5, 127.5)}  # mean, std_dev

        tflite_model = converter.convert()
        open(tflite_file, "wb").write(tflite_model)

        os.system("edgetpu_compiler -s {}".format(tflite_file))

And the output (tensorflow warnings removed):
Edge TPU Compiler version 2.0.291256449

Internal compiler error. Aborting!

Also, here are the tflite model itself, as well as the output of visualize.py: edgetpu_test.zip
It's possible this is a user error on my end, especially since the transpose conv layer is untrained. However, compilation works fine if I replace the conv2d_transpose with a regular conv2d.
Thanks for your help!

From what I have found, I believe quantization for transpose convolutions are not supported in TF 1.15. I found that the quantization parameters for conv2d_transpose are only added when using Tensorflow 2.2, as per this Github issue, so it seems like the edgetpu compiler was crashing when it couldn't find those parameters. When I used tf-nightly with post-training quantization, I was able to quantize the graph and compile it properly for the EdgeTPU.


the GenericDialog should be centered on the screen

At the moment the GenericDialog is off 100px horizontal and vertical from the center of the screen.

Until the 08.02.12 you can find a screenshot of the problem here: http://uploadpie.com/YFIX4


data smooth index check needs to be fixed

currently when doing an update(), the code writing to the data smooth index handling looks to see if it is buffer size  - 1 and then sets it to 0.   Unfortunately, if you create this with new, or you are putting it into stack memory that has already been used, the index initially can be outside the range 0 - buffersize -1.
To fix this the safest thing would be before it writes to the buffer, the index should be checked to ensure it is >= 0 and < buffer size and if it is outside that range, set to 0,.
An alternative would be to initialize the index to 0 on construction, but this does leave the possibility of some other bug setting it out of range and the update() continuing to overwrite memory it doesn't own.

Hi
Sure, I can include some code to check that the readIndex is within range as you suggest.
But could you please explain in a little bit more detail how the variable readIndex can be "initially out of range"? (I'm not familiar with the "new" operator)

Got it, thanks a lot for the explanation!
I have actually had this problem before with local variables, so I usually initialize those.
As a fix I think I will initialize the index in the class declaration. We could in addition to that also do a range check of the index inside the function as you first suggested, that should be bullet-proof.
You also mention initializing of the data array, but is that necessary? The whole data array will be overwritten with fresh values quickly and unlike the index, old garbage should not make any serious problems?

Implemented Initializing of the index counter in the class declaration: readIndex = 0


Release @google-cloud/tasks v0.5.0

This pull request was generated using releasetool.

Release is at https://github.com/googleapis/nodejs-tasks/releases/tag/v0.5.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Duplicate word on paragraph #64

Plus minor formatting proposals

the README was drastically changed, this PR is no longer applicable.


Adding HPC logging and extending logger to support different WG

Dear Sir/Madam,
I have extended the existing package checker to be able to support other WG's such as HPC (only HPC and training are supported yet, where HPC is WIP). This PR does not touch the evaluation or checking logic but makes use of the already existing usage parameter and extends the compliance checker and other relevant scripts such as the rcp checker with that usage parameter. Mostly what was changed was renaming the directories named  into _. That way, the checker can pick up the right compliance and rcp scripts from the correct WG.
I am happy to discuss the changes here but I open the PR to get things started. HPC submission deadline is end of September and we would like to have a working submission checker stack by then.
Note that if the usage parameter is not passed, it defaults to training so it should in principle not break any BW compatibility. I tested the script on the NVIDIA training submission for 1.0.0 and it passed, but feel free to verify on your own in case I missed subtle details.
Some things which could be improved:

the dictionaries containing the allowed benchmarks and the required number of runs are in the python scripts, but ideally those should be read from a YAML file as well. I haven't done that yet because I wanted to keep the changes minimally invasive, but I am happy to do that.
HPC scoring has an additional metric which I am not yet clear about of how to incorporate into this stack and thus it is missing yet. I will think about it a bit and will open another PR for this later on.

Let me know what you think
Best and thanks
Thorsten

Hi Thorsten,
Thank you for your contribution and comments.
Adding HPC to the mainline training logging mostly makes sense, but also
comes as a surprise.
At least from my side I would like to know more of HPC training plans, what
you expect from the submission infra, if and for how long you can
contribute, etc.
Can you please attend our weekly meeting one time, on Tuesday morning
10:30PDT?
What is the best e-mail address so we send you the invitation?
Thanks,
Elias
On Mon, Aug 16, 2021 at 7:50 AM github-actions[bot] <
@.***> wrote:

MLCommons CLA bot:
Thank you for your submission, we really appreciate it. We ask that you
sign our MLCommons CLA
https://github.com/mlcommons/systems/blob/main/mlcommons_cla.txt and be
a member before we can accept your contribution. If you are interested in
membership, please contact @.*** .
0 out of 1 committers have signed the MLCommons CLA.
❌ @Awesom https://github.com/Awesom E. Dev
Awesom E. Dev seems not to be a GitHub user. You need a GitHub account
after you become MLCommons member. If you have already a GitHub account,
please add the email address used for this commit to your account
https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/#commits-are-not-linked-to-any-user
.
You can retrigger this bot by commenting recheck in this Pull Request
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mlcommons/logging/pull/151#issuecomment-899573176,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKN472PQVFPHCLXYN4WV6HTT5EQR7ANCNFSM5CH4MQJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Hello Elias,
I am happy to join. Can you send me an invitation, maybe for tomorrow?
Btw, I did a rebase to fix the author name + email issue. My email is<EMAIL_ADDRESS>Best regards
Thorsten

@emizan76 I created a PR to this PR's source branch (https://github.com/azrael417/logging/pull/3) "FileNotFoundError" that you saw. The root cause was that the paths in enqueue_config inside rule yaml are not updated to reflect the split between training and HPC. @azrael417 once you merge https://github.com/azrael417/logging/pull/3, @emizan76 and I can validate this branch again.

Hi all,
the error you mentioned was fixed in my updated PR already. Can you please re verify that the commands work? Btw, the verify training script as-is does not work, because it does not pass --rcp_bert_train_samples to the package checker, thus causing the RCP checks to fail. If you run the package checker manually with that flag specified, it works fine, same for result summarizer. Pleaser verify the latest push.

Let me summarize what I found running the tests:

fails on Google and NVIDIA submissions because --rcp_bert_train_samples is not passed to package checker in that script. But this is an intended fail, not a bug in the scripts.

python3 -m mlperf_logging.package_checker training_results_v1.0/Google/ training 1.0.0 --rcp_bert_train_samples works fine, w/o the last flag it fails Rcp test

Works fine

Works fine

Works fine

Both work fine

Let me know what you think.

Hi all,
the error you mentioned was fixed in my updated PR already. Can you please re verify that the commands work? Btw, the verify training script as-is does not work, because it does not pass --rcp_bert_train_samples to the package checker, thus causing the RCP checks to fail. If you run the package checker manually with that flag specified, it works fine, same for result summarizer. Pleaser verify the latest push.

The FileNotFoundError are from places like  https://github.com/mlcommons/logging/blob/5f08a416ab4cff5035063f7b578eaa3dc62af002/mlperf_logging/compliance_checker/training_1.0.0/closed_resnet.yaml#L11 where it should be  enqueue_config('training_1.0.0/closed_resnet_{}.yaml'.format(v['value'])) instead because you renamed the directories of those rules. Checking the Files changed list of this PR as of now, they don't seem to be fixed.
--rcp_bert_train_samples is not needed for Google's v1.0 results since their results were reporting number of samples in the "right" place.

Hi guys, any update on this?

@xyhuang @emizan76 we'd like to merge this PR now then add in HPC specific things in later PRs.

LGTM

Thanks everybody for the quick feedback and help. Who is able to authorize the merge?

Thanks everybody for the quick feedback and help. Who is able to authorize the merge?

@xyhuang has the authorization to merge. Although the weird thing is that I'm seeing "At least 1 approving review is required by reviewers with write access." on this webpage. @emizan76 I thought you and I have write access before?

Any update on this? It would be good if we can merge this soon?

looks like my approval didn't help either - i must not have write access either, so we need someone who has write access.  @mwawrzos , do you have write access as well?


[JB plugin] Chat doesn't work

Chat doesn't have any input place to interact.
PhpStorm 2024.1.4
Build #PS-241.18034.69, built on June 21, 2024
Runtime version: 17.0.11+1-b1207.24 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Linux <IP_ADDRESS>-microsoft-standard-wsl2
Plugin version 1.3.3

Issue closed, it's working ok on plugin version 1.3.4

Same issue:
PhpStorm 2024.2.4
Build #PS-242.23726.107, built on October 23, 2024
Runtime version: 21.0.4+13-b509.26 amd64 (JCEF 122.1.9)
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Toolkit: sun.awt.X11.XToolkit
Linux <IP_ADDRESS>-microsoft-standard-wsl2
GC: G1 Young Generation, G1 Concurrent GC, G1 Old Generation
Memory: 4096M
Cores: 20
Registry:
ide.browser.jcef.jsQueryPoolSize=200
ide.browser.jcef.gpu.disable=true
debugger.new.tool.window.layout=true
run.processes.with.pty=TRUE
ide.experimental.ui=true
Using plugin Refact 5.1.0

Wow....Thanks for report, we will fix that asap


gdb-arm-none-eabi not found

After a
sudo apt-get install gdb-arm-none-eabi
got
E: Unable to locate package gdb-arm-none-eabi
can you help me ?

solved. Need to find the bin code from a repository and write it to /dev/bin

how to get bin code? please!

Please, see my blog (portuguese)
I report how to solve that
https://mynewt-ninabx.blogspot.com/2019/02/programando-nina-b112-com-apache-mynewt.html


Set default values for make_timeseries

This makes it easier to quickly create a non-trival dataframe for
testing or training.
These values are what we use by default in dask-examples

[x] Tests added / passed
[x] Passes flake8 dask
[ ] Fully documented, including docs/source/changelog.rst for all changes
and one of the docs/source/*-api.rst files for new API

Merging later today if there are no further comments.


Paladin (mod of Pendragon 5,2)

Changes / Comments
New sheet for Paladin: Warriors of Charlemagne (2018 Nocturnal Media/Chaosium), based on the Pendragon 5.2 sheet by Cassie Levett. This is a beta with the first sheet looking pretty close to done. The appendix pages are untouched.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[x] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Hi! It's my first project on Roll20 so this is a very basic question, but when should I expect to be able to see it on Roll20?
Thank you in advance!
Sophie

You should see it tomorrow after our weekly code push!

Thank you! It's silly but I'm excited. ^_^
Sophie Lagacé
On Mon, 30 Mar 2020 at 13:57, Olivia Kristiansen<EMAIL_ADDRESS>wrote:

You should see it tomorrow after our weekly code push!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/Roll20/roll20-character-sheets/pull/6051#issuecomment-606245713,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO7GFA2AUZW556H7J7RU553RKEBU7ANCNFSM4LWK67SQ
.


Charts JS not work with loaded by AJAX .php

Hello, this plugin not work when I am using jquery ajax function .load() How to solve this problem?This is my .php code:

And this is my functions.js:
jQuery(document).ready(function() {
var randomScalingFactor = function(){ return Math.round(Math.random()*100)};
var lineChartData = {
labels : ["January","February","March","April","May","June","July"],
datasets : [
{
label: "My First dataset",
fillColor : "rgba(220,220,220,0.2)",
strokeColor : "rgba(220,220,220,1)",
pointColor : "rgba(220,220,220,1)",
pointStrokeColor : "#fff",
pointHighlightFill : "#fff",
pointHighlightStroke : "rgba(220,220,220,1)",
data : [randomScalingFactor(),randomScalingFactor(),randomScalingFactor(),randomScalingFactor(),randomScalingFactor(),randomScalingFactor(),randomScalingFactor()]
},
{
label: "My Second dataset",
fillColor : "rgba(151,187,205,0.2)",
strokeColor : "rgba(151,187,205,1)",
pointColor : "rgba(151,187,205,1)",
pointStrokeColor : "#fff",
pointHighlightFill : "#fff",
pointHighlightStroke : "rgba(151,187,205,1)",
data : [randomScalingFactor(),randomScalingFactor(),randomScalingFactor(),randomScalingFactor(),randomScalingFactor(),randomScalingFactor(),randomScalingFactor()]
}
]
}

window.onload = function(){
var ctx = document.getElementById("canvas").getContext("2d");
window.myLine = new Chart(ctx).Line(lineChartData, {
responsive: true
});
}
});
Why it is not working? Please help.

Do you get any errors? This question is probably better suited for Stack Overflow.

Firebug (Firefox plugin) returned classical 404 error.

@Artimal looks like you have a typo. The file is 'Chart.js'