vertx don't instant sends messages while processing handlers!

It seems that vertx only sends cluster messages when is idle (nothing to process).
That means while a vertx is very busy/active there is no messages dispatched to eventBus until the vertx becomes completely idle, that could be a big limitation if there is a need to send instant messages.
What is the solution to force vertx to instant send a message without waiting the need to be idle?
In order to demonstrate this issue, I have created two very simple verticles:

Verticles ping: sends 60 instant messages to pong and prints the reply answers.
Verticle pong: emulates a 50ms processing time and reply the message

Scenarios: (cluster mode)

Test1: pong is a worker verticle:
Test2: pong is a multithread worker verticle:

java -cp 'target/*' io.vertx.core.Launcher start my.pacakge.Pong -id pong -worker -cluster
java -cp 'target/*' io.vertx.core.Launcher start my.pacakge.Ping -id ping -worker -cluster

For these tests ping verticle is a worker verticle ([w-##] means vert.x-worker-thread-##), however it's not revenant if it's a standard verticle because both configurations are similar. Both ping and pong suffers from the reported issue and therefore the messages were stuck on eventBus until idle (nothing to process).
Note that If the ping used a multithread worker then the 1 second extra stuff wasn't noticed and the entire process was 1 second quicker (see ping code), however the pong continues suffering from the reported issue.
ping verticle code:
public class Ping extends AbstractVerticle {
    private final static Logger LOG = LoggerFactory.getLogger(Ping.class);

    @Override
    public void start() {
        for (int i = 1; i <= 60; i++) {
            String ball = String.format("ball%02d", i);
            getVertx().eventBus().send("table", ball, response -> {
                if (response.succeeded()) {
                    LOG.info("ping> recv " + response.result().body());
                } else {
                    LOG.error("ping> fail " + response.cause());
                }
            });
        }
        LOG.info("ping> send ball01 to ball60");
        doExtraStuff(); // 1 second wait
    }

    private static void doExtraStuff() {
        try {
            Thread.sleep(1000);
        } catch (InterruptedException e) {
            throw new IllegalStateException(e);
        }
    }
}

pong verticle code:
public class Pong extends AbstractVerticle {
    private final static Logger LOG = LoggerFactory.getLogger(Pong.class);

    @Override
    public void start() {
        getVertx().eventBus().consumer("table", message -> {
            String ball = (String)message.body();
            LOG.info("pong> recv " + ball);
            process(); // 50ms wait
            LOG.info("pong> send " + ball);
            message.reply(ball);
        });
    }

    private static void process() {
        try {
            Thread.sleep(50);
        } catch (InterruptedException e) {
            throw new IllegalStateException(e);
        }
    }
}

Test1 results: ping and pong are worker verticle:

[2016-09-04 02:12:23.169] [w-02] ping> send ball01 to ball60

# -> (ping is doing extra sfuff for 1 second and the 60 messages aren't dispatched to pong)

[2016-09-04 02:12:24.294]                              [w-03] pong> recv ball01
[2016-09-04 02:12:24.349]                              [w-03] pong> send ball01
[2016-09-04 02:12:24.364]                              [w-03] pong> recv ball02
[2016-09-04 02:12:24.415]                              [w-03] pong> send ball02
#... (2x57 lines - recv and send for ball03 to ball59 - time between recv and send is ~50ms)
[2016-09-04 02:12:27.392]                              [w-03] pong> recv ball60
[2016-09-04 02:12:27.443]                              [w-03] pong> send ball60

# -> (pong is idle and only now the 60 messages are dispatched to ping)
# -> (ping didn't receive answers for at least 3 seconds; pong processing time is 50ms!)

[2016-09-04 02:12:27.463] [w-04] ping> recv ball01
[2016-09-04 02:12:27.465] [w-04] ping> recv ball02
#... (57 lines - recv for ball03 to ball59)
[2016-09-04 02:12:27.504] [w-04] ping> recv ball60

# -> (ping received the 60 reply messages practically in one go!)

Test2 results: ping is a worker verticle and pong is a multithread worker verticle:

[2016-09-04 02:12:37.623] [w-02] ping> send ball01 to ball60

# -> (ping is doing extra sfuff for 1 second and the 60 messages aren't dispatched to pong)

[2016-09-04 02:12:38.754]                              [w-12] pong> ball01 recv
[2016-09-04 02:12:38.754]                              [w-13] pong> ball02 recv
#... (17 lines - recv for random bal01 to bal20 except ball01, ball02 and ball18)
[2016-09-04 02:12:38.760]                              [w-09] pong> ball18 recv

# -> (pong doesn't have more idle threads - all 20 pool threads are processing for 50ms)

[2016-09-04 02:12:38.805]                              [w-13] pong> ball02 send
[2016-09-04 02:12:38.805]                              [w-14] pong> ball03 send
[2016-09-04 02:12:38.805]                              [w-13] pong> ball03 recv
#... (2x18 lines - with random send/recv bal01 to bal20 except already displayed balls)
[2016-09-04 02:12:38.835]                              [w-12] pong> ball40 recv

# -> (pong doesn't have more idle threads - all 20 pool threads are processing for 50ms)

[2016-09-04 02:12:38.872]                              [w-16] pong> ball26 send
[2016-09-04 02:12:38.872]                              [w-06] pong> ball27 send
[2016-09-04 02:12:38.873]                              [w-16] pong> ball41 recv
#... (2x17 lines - with random send/recv bal21 to bal40 except already displayed balls)
[2016-09-04 02:12:38.882]                              [w-02] pong> ball59 recv
[2016-09-04 02:12:38.887]                              [w-14] pong> ball40 send
[2016-09-04 02:12:38.888]                              [w-14] pong> ball60 recv

# -> (pong doesn't have more idle threads - all 20 pool threads are processing for 50ms)

[2016-09-04 02:12:38.925]                              [w-16] pong> ball41 send
[2016-09-04 02:12:38.926]                              [w-19] pong> ball44 send
#... (17 lines - send for random bal41 to bal60 except ball41, ball44 and ball60)
[2016-09-04 02:12:38.938]                              [w-14] pong> ball60 send

# -> (pong has idle threads and only the 60 messages are dispatched to ping)
# -> (ping didn't receive answers for at least 150ms seconds; pong processing time is 50ms!)

[2016-09-04 02:12:38.945] [w-04] ping> recv ball09
[2016-09-04 02:12:38.959] [w-04] ping> recv ball04
#... (57 lines - recv for random bal01 to bal60 except ball09, ball04 and ball44)
[2016-09-04 02:12:39.004] [w-04] ping> recv ball44

# -> (ping received the 60 reply messages practically in one go!)

What is the solution to force vertx to instant send a message without waiting the need to be idle?

This issue was previously opened in eclipse/vertx repository, and @vietj wrote:
Hi,
with a worker it happens because Hazelcast uses an executeBlocking method that cannot be executed until the current task (the worker) has been finished (as it uses ordering).
can you rather open this issue in vertx-hazelcast project ?

A workaround while the fix is not out,  it's creating another WorkerExecutor instead of using the default vertx.executeBlocking. (optionally you can add the metrics support for it)
public class Pong extends AbstractVerticle {
    private final static Logger LOG = LoggerFactory.getLogger(Pong.class);
    private WorkerExecutor executor;

    @Override
    public void start() {
        executor = getVertx().createSharedWorkerExecutor("vert.x-new-internal-blocking", 20);

        getVertx().eventBus().consumer("table", message -> {
            // Run ordered or not it's up to you
            executor.executeBlocking(future -> {
                String ball = (String)message.body();
                LOG.info("pong> recv " + ball);
                process(); // 50ms wait
                LOG.info("pong> send " + ball);
                message.reply(ball);
                future.complete();
            }, result -> {});
        });
    }

    private static void process() {
        try {
            Thread.sleep(50);
        } catch (InterruptedException e) {
            throw new IllegalStateException(e);
        }
    }
}

https://github.com/vert-x3/issues/issues/75

ideally we should not need to use such work around.

@tsegismont @cescoffier how about adding a test in clustering TCK that checks that we can send an event bus message in a blocking worker ?

@vietj will do
2016-09-06 19:25 GMT+02:00 Julien Viet<EMAIL_ADDRESS>
@tsegismont https://github.com/tsegismont @cescoffier
https://github.com/cescoffier how about adding a test in clustering TCK
that checks that we can send an event bus message in a blocking worker ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vert-x3/vertx-hazelcast/issues/43#issuecomment-245024885,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABbltmrfQy_5PNidBGEMLYdIyCSmlkwEks5qnaIRgaJpZM4J0mNR
.

Tests about this issue: (vertx don't instant sends messages while having messages to process):

Test Type / Verticle Type
Standard
Worker
MT Worker

Direct code (No executeBlocking)
No
No
No

vertx.executeBlocking(ordered)
Yes
Partial
No

vertx.executeBlocking(not ordered)
Yes
No
No

executor.executeBloking(ordered)
Yes
Yes
Yes

executor.executeBloking(not ordered)
Yes
Yes
Yes

Legend:

Yes: vertx instant sends messages even if there is messages to process
No: vertx only sends messages after not having messages to process (vertx is in idle)
Partial: Strange behaviour: The first 40 messages doesn't instant send, but after sends correctly.

If the issue is because a call to executeBlocking (worker verticle) mentioned by @vietj, then one possible solutions could be the second solution mentioned on the reported issue: https://github.com/vert-x3/issues/issues/75
The hazelcast implementation could uses the current thread when it is a worker thread (doing a check on the current context and use execute blocking only when on event loop thread, quite tedious)
Or could use another WorkerExecutor to execute the blocking.

#29 was recently merged, which enabled the use of non-blocking execution. @tetv , could you perform the tests again with  -Dvertx.hazelcast.async-api=true supplied as a System property?
A background of the abovementioned PR can be found at #28.

Thank you @LoneRifle, I will do again the tests.
I will assume that merge is included in version 3.3.2.

Tests created using vertex 3.3.2 (with and without executeAync).

Ping sends 500 instant messages to Pong.
Pong emulates 50ms processing time and answers to Ping.

Each cell tells if the messages are sent instantaneously to Ping after processing

v Test Type / Pong Verticle Type >
Standard
Worker
MT Worker

Direct code (No executeBlocking)
No
No
No

vertx.executeBlocking(ordered)
Yes
From-24
No*1

vertx.executeBlocking(not ordered)
Yes
No
From-115*2

workerExecutor.executeBloking(ordered)
Yes
Yes
Yes*1

workerExecutor.executeBloking(not ordered)
Yes
Yes
Yes*2

javaExecutor.process(ordered)
Yes
Yes
Yes*1

javaExecutor.process(not ordered)
Yes
Yes
Yes

Using Async=true

Direct code (No executeBlocking)
No
No
From-337

vertx.executeBlocking(ordered)
Yes
From-69
No*1

vertx.executeBlocking(not ordered)
Yes
From-58
From-59*2

workerExecutor.executeBloking(ordered)
Yes
Yes
Yes*1

workerExecutor.executeBloking(not ordered)
Yes
Yes
Yes*2

javaExecutor.process(ordered)
Yes
Yes
Yes*1

javaExecutor.process(not ordered)
Yes
Yes
Yes*2

Notes:
*1 Messages are not processed in ordinal order.
*2 First 20 messages (thread pool size) processed are not the first 20 sent messages by ping.
Source code is available here: https://github.com/tetv/vertx3-blocking
The tests were executed on:

Ubuntu 16.04 LTS
Java 1.8.0_101-b13

Updated the test source code with junit tests (now using vertx 3.3.3).

v Test Type / Pong Verticle Type >
Standard
Worker
MT Worker

Direct code (No executeBlocking)
:heavy_multiplication_x:≥ 2
:heavy_multiplication_x:≥ 2
:heavy_multiplication_x:≥ 40

vertx.executeBlocking(ordered)
:heavy_check_mark:
:heavy_multiplication_x:≥ 2
:heavy_multiplication_x:≥ 2

vertx.executeBlocking(not ordered)
:heavy_check_mark:
:heavy_multiplication_x:≥ 40
:heavy_multiplication_x:≥ 40

workerExecutor.executeBloking(ordered)
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

workerExecutor.executeBloking(not ordered)
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

javaExecutor.process(ordered)
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

javaExecutor.process(not ordered)
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

Note: The -Dvertx.hazelcast.async-api=true doesn't influence the result.
There is also an identical issue but for non clustered vertx: https://github.com/eclipse/vert.x/issues/1631

Closed via #54

I have updated to vertx 3.4.0, and the same issues described above are still happening.
By the way I also have simplified and improve logging in the junit test project.

@tetv we have decided that having a message sent in a worker delayed until the blocking tasks is finished is a normal thing and we are not going to try to solve it. Blocking in Vert.x should be done to wrap blocking API and not used for waiting asynchronous responses.

Thank you @vietj but it doesn't make much sense for me, at least in a first glance.
In scenarios which a worker sends a message to Y, but because there are 1000 messages in the queue (to be processed) waiting for a worker, then only after these 1000 messages are effectively processed, then the message to Y is effectively sent. Does it make sense?

can you explain more in detail the sequence of operations ?

Sure @vietj, here it is:

There are 1000 messages (M1 to M1000) in the queue to be processed (all workers are busy)
Worker W1 is processing M0 and is executing blocking code
Worker W1 sends a message to Y (but the message is not effectively sent)
Worker W1 finishes processing M0
Worker W1 starts processing M1 (first message of 1000 that are in the queue to be processed)
... Several workers start processing M2...M1000
Worker WN is free now and is effectively sending message to Y after a long time!


Made all tests internal

Continuation of #1387

fabbot.io fails as always :(

@keradus what do you think about moving them to a separate tests folder at the root, which would not be autoloadable at runtime ?
This would cleanly separate the testsuite from the library, avoiding the need to mark anything as internal in the testsuite (as it would not be accessible at all anymore)

@keradus what do you think about moving them to a separate tests folder at the root, which would not be autoloadable at runtime ?

I suggested this before, and @fabpot rejected the idea.

This would cleanly separate the testsuite from the library, avoiding the need to mark anything as internal in the testsuite (as it would not be accessible at all anymore)

This would cleanly separate the testsuite from the library, avoiding the need to mark anything as internal in the testsuite (as it would not be accessible at all anymore)

I can include classes from tests dir myself and use it, extend it ect. So marking them internal/final isn't pointless.

@keradus if you have to include them yourselves rather than relying on the autoloading, it is clear you are doing weird thing (and that you would ignore an @internal phpdoc anyway, which is harder to see than an autoloading failure)

I' not saying here should we move tests or not, just saying that marking tests as final/internal is still gr8

I agree that one should not include files from vendor himself, but if we mark tests as internal we won't lose anything


Technical decisions to make before GA

Below is a list of technical decisions to make before GA.

[ ] Meaning of *_every filters.
[x] Need for prefixes for standard prometheus node_* metrics.
[ ] Id-less entities

*_every filters
Graphql server provides *_every filters:
query {
  # select users who have only high profile friends
  users(where: {friends_every: {rating_gt: 100}}) {
    id
    name
  }
}

The question is what to do when user doesn't have friends.
Include user in result set
Pros:

In all programming languages [].every(() => false) == true
The opposite behaviour is easy to achieve with where: {friends_every: {rating_gt: 100}, {friends_some: {}}}
Openreader (hydra v5) already behaves like this.

Cons:

Many people find such behaviour confusing
Hydra v4 behaves in opposite way

Do not include user in result set
Pros:

See cons of the opposite decision

Cons:

Will be implemented as a combination of "classical every" and "some" check
The opposite behaviour is a bit hard to achieve (where: {_or: [{friends_every: {rating_gt: 100}}, {friends_none: {}}]})

From pure convenience point of view, v4 behaviour will probably be more frequently desired, but that's not entirely clear.
For example, the above query will most likely benefit from v4, but the query below from v5
query {
  # find all accounts which where never involved in significant money transfers
  accounts(where: {transfers_every: {balance_lt: 10}}) {
    id
  }
}

Name prefixes for standard metrics
Processor runs prometheus endpoint and exposes a number of metrics including standard node_* set.
In hydra all standard metrics had hydra_processor_ prefix, but now they are exposed as is.
Someone familiar with prometheus should tell which way is better.
UPD: After reading a bit of prometheus docs, it seems, that prefix-less metrics is the way to go.
Id-less entities
For some tables you never want to reference individual raws. For such cases you don't want to bother with id assignment and allocate extra useless column in database, yet our framework requires explicit id assignment for all entities.
Currently it is possible in schema.graphql to omit id field in entity type, but framework will still create it implicitly.
We can change the behaviour and create id-less tables in such cases.
If we accept the above idea, the next question will be about necessity of composite primary keys for such tables.
We are planning to support multi-field indexes via entity level index annotations:
type Foo @entity @index(columns: ["foo", "bar"]) {
  foo: Int!
  bar: Int!
}

Primary key could be specified like this:
type Foo @entity @index(columns: ["foo", "bar"], unique: true) {
  foo: Int!
  bar: Int!
}

From database point of view primary keys are not required and it would be convenient to create primary-key-less tables for id-less entities, however TypeORM requires primary key.
We can do something like this:
@Entity()
export class Rec {
    constructor(props?: Partial<Rec>) {
        Object.assign(this, props)
    }

    @PrimaryGeneratedColumn('rowid')
    private _id!: number

    @Column()
    x!: number

    @Column()
    y!: number
}

but this will still expose non-deterministic column on a database level, which is not ideal for applications which connect to database directly.

Meaning of *_every
I'd go for Include user in result set. Even if some users might not like it, when it's a binary split, it's impossible to achieve 100% consensus and as long as it's well documented and consistent it should be OK.
All other points, especially the ease of obtaining the opposite result in opposition to doing vice-versa look to be in favor of this option.
Id-less entities
For this point it seems like a complication, but I don't see the benefit of letting the framework "hide" the id, even if it's omitted from the schema. Is there a downside for having a hidden id for entities that don't require one?


Support for PythonTeX

This library should provide enough functionalities so that a program for PythonTeX to support OCaml can easily be developed.
José Romildo Malaquias<EMAIL_ADDRESS>posted in the OCaml list:

PythonTeX [https://www.ctan.org/pkg/pythontex] is a LaTeX package that
allows Python code entered within a TeX document to be executed, and the
output to be included in the original document. It supports other
languagens beside Python.
I have just requested inclusion of OCaml support in PythonTeX:
https://github.com/gpoore/pythontex/issues/62
The author of PythonTeX says it could be possible, and he explained what
would be needed from the OCaml side. See the discussion in issue #62 in
the previous link.
Basically, it would need a program which reads lines from a text file,
and these lines are given as input to an interactive session, giving
back input interspersed with output. He gives more details in the issue
discussion. Please read the issue discussion in the above link.
Is there already such a program? If not, can it be easily written?
I appreciate any help in providing this program.

On Thu, 19 Mar 2015 20:57:23 +0100, Roberto Di Cosmo wrote:

Well, an ugly but effective perl script has been used for ages for doing this...
http://caml.inria.fr/pub/old_caml_site/caml-list-ar/0651.html
Many of us (including me) have a modified version adapted to their LaTeX style.
If somebody takes the time to revamp all this, and properly package it, with documentation, etc. , that would be really cool :-)


Update distributions to Python 3.

All unit tests pass, except those that depend on goftests, which is not yet updated to py3.
I had to make some hacky changes to setup.py to get this running (e.g. hard-coded local file paths), which aren't included in this PR. If someone wants to update setup.py, I'm happy to share what I did, if it's useful.

I updated goftests so that it should work with recent versions of numpy now, and there is a CI test for Python 3.10 https://github.com/posterior/goftests/pull/24

@emilyfertig I've released goftests 0.3.0 and parsable 0.3.0 which should now support python 3. Could you test again? LMK if you need any else.

Thanks for releasing those! All loom unit tests are passing now, but I ran the pubmed example and got the following. It looks like another place in parsable that needs updating for py3?
Traceback (most recent call last):
  File "/usr/local/google/home/emilyaf/posterior_loom/loom/examples/pubmed/main.py", line 158, in <module>
    parsable.dispatch()
  File "/usr/local/google/home/emilyaf/.virtualenvs/loom/lib/python3.11/site-packages/parsable.py", line 125, in dispatch
    inspect.formatargspec(*inspect.getargspec(fun)),
    ^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'inspect' has no attribute 'formatargspec'. Did you mean: 'formatargvalues'?

Thanks, looks like I need stronger tests for parsable. Let me fix that and cut another release.

(I should have put that last comment on the loom PR discussion, not this one.)
distributions tests are all passing now, after increasing SAMPLE_COUNT to get a few of the GOF tests to pass.
I looked at replacing .travis.yml with .github/workflows/ci.yml, but I didn't do it because wasn't clear to me how to translate this piece of travis.yaml:
env:
  - FLAVOR=_cc
  - FLAVOR=_cc DISTRIBUTIONS_USE_PROTOBUF=1
  - FLAVOR=_cc_examples
  - FLAVOR=_cy
install:
  - make install$FLAVOR

Also, setup.py is likely still broken (I made some local changes to it to get the pip package to build, and when I remove those changes it now appears to work, but I suspect I'm not building it clean.)
Besides travis.yaml and setup.py, I think this is now mergeable.

@emilyfertig I can take a look at the travis -> github/workflows migration, and look at setup.py as part of that. If I have trouble I may ask for help if you recall what you did.
May I merge this now?

Thanks, @fritzo ! My changes to setup.py are here: https://github.com/posterior/distributions/commit/8074e53fd64382b05a515758b17b1deee8f30c3f The commented-out portions are what initially seemed necessary to get it working, but now I'm not sure.


luaformatter: fix darwin build

Motivation for this change
fix #136290
I am not able test on darwin, but the same change fixes a similar issue here.
I would be great if someone can test it out for me
@jupblb @SuperSandro2000
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Switching to gccStdenv doesn't work after https://github.com/NixOS/nixpkgs/pull/134716. I guess one solution would be to revert it and merge this PR in order to support both platforms.

Not sure what is causing this issue.
on #123530 it builds on darwin
on #134669 it fails on darwin
There is no changes to luaformatter between these commits, might be changes to stdenv that is causing the issue

@ofborg build luaformatter


Testing framework should raise an issue if a public method is not tested

To make sure we have the proper napalm-base version we should make sure all public methods of a driver where called at least once during the testing phase.

Can we close this one @dbarrosop? Is there anything else you'd like to add?
Thanks!


Upgrade rancher/steve to include cache key fix

Issue: 45383
Includes https://github.com/rancher/steve/pull/211

LGTM, though it looks like this needs a rebase.


How to re-certify components with new capabilities

Today, a component can be marked as stable after going through conformance and certification tests. On the other hand, there is no clarify on how to proceed when a new capability is added. Should there be a higher bar for adding new features to stable components or should the component have a new version once a new capability is added?
Example: one state store might be certified as stable before it has support to transactions, later on, the transaction capability is added but it might not be as stable as the current code. Should this be a new component version?

I think this is due to the standard of stable:
should it be versioned like v1 / v2 / v3?
Then we can say that the component is "stable for state v1". When we add some new features in state and we should call itstate v2, then current component is still "stable for state v1" and not  stable for state v1.

+1 for higher bar for adding new features. Features to be only added if they're really important and must be backed up by appropriate certification and conformance tests.
Upgrading a major version for small feature additions sounds counterintuitive in terms of versioning. Revamping a whole component, moving to a new library (for e.g., with Cosmos), scenarios like these maybe more fitting for version upgrade.

Then we can say that the component is "stable for state v1". When we add some new features in state and we should call it state v2, then current component is still "stable for state v1" and not stable for state v2.

Do we do this today for general version updates across components? If not, we should totally do it.

+1 on making the version intuitive to reflect the stability and feature set. Proper versioning backed by thorough testing will save significant effort in adoption of these components.
+1 on @shubham1172 & @skyao comments on major and minor version update.
It would give users confidence to move to the next version of the component. It would also help the component owners to maintain the quality bar when adding new capabilities.
One additional call-out -  while minor versions should always be backward compatible - major version might also be an indicator for the break in backward compatibility. In such a scenario, there is additional responsibility for the component owners to ensure smooth upgrade. Including this check as part of certification would save effort during upgrade.

Features to be only added if they're really important

Really important according to who?
This is, as you can tell by my question, highly subjective. In the realm of criteria it's imperative to be able to make decisions that rely on data that is as objective as possible. So making a decision for certification based on importance, even when coming from maintainer, seems off here.
I think that new features added to stable components must be backed by the same tests as required by all other features of that component. In other words, new features are no different than old features as the existing criteria does not give any special significance to "old" or "new". Only to whether or not they have the required set of tests.
Cases where it's advisable to iterate on a v2 of a component are when breaking changes are introduced, or if the feature is known to be changed ahead of time, during which an Alpha status is appropriate to allow for uninterrupted modifications going forward.

In the realm of criteria it's imperative to be able to make decisions that rely on data that is as objective as possible.

I agree we should be backing each added feature to stable components with appropriate certification /conformance tests.
For versioning, the scope and impact of the added feature should decide whether it should be major or minor version and based on whether the component is still stable or not with that added feature will help users decide on upgrades.

Just a thought: Can we leverage Metadata / Capabilities API enlisting status of a component for a capability.
So, in addition, we can have status field telling status (Alpha / Beta / Stable) for a feature for a component.

@dapr/maintainers-components-contrib need to ensure that any PR impacting a stable components enables applicable conformance tests, and also adds certification tests that cover the new component metadata option.
As long as as this is done diligently I do not think it is necessary to recertify any component.
I would like to propose that no PRs for stable components are accepted unless certification tests are also updated!

new capability with new version is a good idea


enhancement - uses of js core/windows function & browser compatibility

see if it's possible to detect the uses of js core/windows functions & compute the browser compatibility (jshint?)

"Browser compatibility" from "https://developer.mozilla.org" as jshint / jslint pluggin?


Fix Vevo titles with featured artists

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
I downloaded https://www.vevo.com/watch/lemaitre/Wait/USUV71402190 and noticed that the featured artist was the artist in the title ("LoLo - Wait" vs "Lemaitre ft. LoLo - Wait"). This fixes the issue.
All tests for Vevo are broken on master. This fixes the code to make 4/5 pass.

@dstftw any tips on getting this landed?
Has the project thought about only running the tests for changed modules on pull requests? I'm happy to work on it.
Thanks. :)

@dstftw Why was this closed? I've made your requested changes.

It's already merged into master: 8b27d83e4e07064898c5ec842e916c84cf7a1826


Bangkok, Thailand

House number only source from the city of Bangkok's ESRI server (https://github.com/openaddresses/openaddresses/issues/2440). No license info.

Dense!

On Feb 7, 2017, at 18:00, OpenAddresses Bot<EMAIL_ADDRESS>wrote:
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I think that the Travis build is failing from a missing street in the conform object, which I believe is a requirement. @migurski can you validate that? The Travis failure log is pretty obscure

Yuck, yeah, that is super obscure. A missing "street" is a problem though!

Yeah, the missing street discussion was started here: https://github.com/openaddresses/openaddresses/issues/2440
Basically, it could be fine for Pelias, but not necessarily for other users.

I'm going to close this for now. I think the next step here if someone wants to take it is to build a script that joins with the centerlines layer in the same map service.


Production Panel Items Start as Highlighted

Sometimes, as soon as I open the production panel (either by clicking "choose production" in the lower right hand corner or by clicking on the city banner), some of the items start in the highlighted state (as if the mouse cursor was hovering over them). If I then move my mouse cursor over them, they go back to the normal state. Examples are the five items from settler to warrior in the screenshot. The only mods enabled were CQUI and Civ VI Graphs. It has never happened to me prior to the recent update, and has happened several times since the update.

I also saw this once or twice, we'll look into it

This also happens in the diplo window in vanilla, so my bet is that it is, oh surprise, FXS code.


Support extracting macOS dmg

Need to find a way to support extracting dmg files to complete macOS support.

We no longer bundle Blender as of ac1a0f675731244b17ac42f597b9178280220c10


python311Packages.ldfparser: 0.21.0 -> 0.22.0

Diff: https://github.com/c4deszes/ldfparser/compare/refs/tags/v0.21.0...v0.22.0
Changelog: https://github.com/c4deszes/ldfparser/blob/0.22.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

6 packages built:

python311Packages.canmatrix
python311Packages.canmatrix.dist
python311Packages.canopen
python311Packages.canopen.dist
python311Packages.ldfparser
python311Packages.ldfparser.dist


Fix autosave status timing

Autosaving status message now displays correctly according to when it is happening as opposed to having to wait for an interval timeout.
Not sure how "reacty" this approach is, but I've fixed the issue by having our useAutosaveProject hook return a value corresponding to whether or not the project is currently being saved, and passing this state to our menu bar.
Fixes #259

Original
Updated

The previous approach also seems to have had an issue with indefinitely displaying the spinner when updating a project to have nothing, i.e. deleting everything in it. When any state or comment is added back to the project, the spinner goes away. This issue is now fixed.

Indefinite load

Think it might be unintended side effect with the removal of the last change check but now empty projects are saved.
e.g. a user clicks to create a new FSA project, immediately regrets it and clicks out, now an empty project is saved that they'll probably delete

ah good catch. is there a nice way to prevent this from happening? I originally considered removing this isSaving conditional and saving regardless (why wouldn't we save when we're leaving) - would there be any concerns with removing that check to instead include a condition to check for empty project? e.g.
if (projectNotEmpty) {
   saveProject()
}

Can't forsee any issues? Think that should be fine

That should do it - I also noticed the unsaved project warning was broken so I've gone ahead and fixed that too
Just to confirm the behaviour is that we'll never save a completely empty project and any time you exit the project from that state it will disappear?

Agh so if you have a project with states, and you delete them then immediately go to the projects page, the project doesn't save and it just keeps your machine as is, but the thumbnail shows it empty
Will have to sort this issue out tmr or sumn

in my mind the behaviour is more like if the user hasn't touched it then don't save it. So guess could like check if lastChangeTime is not set and then not save, otherwise do? Think that'd be least surprising behaviour

if the user hasn't touched it then don't save it
So this behaviour is already working, but the thing I just described kinda leaves the UI and project in a weird state bc the project itself doesn't save, but the thumbnail makes it look like it has

I'm thinking it might actually make sense to just delete a project if there's nothing on it - if they're leaving it, the history gets wiped anyway so when they go back it's almost exactly fresh (aside from the name). Similar behaviour to say text documents where if you open it and close it you won't get a prompt to save, and if you type and then remove everything you still won't get a prompt

ah just saw that comment lol
But yeah if that makes sense then do it that way 👌 😎

I think the behaviour you described seems easy enough to implement (assuming there's a way to tell the user hasn't interacted with the project), but I'm kind of stumped on how to have the project delete if it's empty/get the same behaviour as if it was never interacted with

I thought that one would be super simple after discovering that we had a deleteProject function but that caused a bunch of issues with our image generation hook in some cases so I've had to add some null logic around the place
The one case that is still ever so slightly broken is if you

Open a project, add some states, and save it
Go back into said project, and delete all states - this will not autosave
Open a new tab, and you can see the empty project thumbnail, but clicking into it will still contain your states


Aerogear 10382 fix failing gradle build in 5.0.0

Motivation
Fix for JIRA Ticket https://issues.redhat.com/browse/AEROGEAR-10382 and also #189 and #201 issues on github.
What
In version 5.0.0, the gradle build is failing.
Why
the jcenter repo is not reachable anymore and there is a missing dependency "resteasy"
How
I removed the repo and added the missing dependency, analog to the fix in version 6.0.0
Verification Steps

run gradle build
check if build succeeds

Checklist:

[x] Code has been tested locally by PR requester
[ ] Changes have been successfully verified by another team member

Progress

[x] Finished task
[ ] TODO

Additional Notes
PS.: Add images and/or .gifs to illustrate what was changed if this pull request modifies the appearance/output of something presented to the users.

No, I am not working for the company that needed that fix anymore, so I am closing this. Thanks for your interest!


Update MD file on new branch 2017-12-02_01-38-45

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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


Delete Edge between two vertices

Probably a mistake from my end, however, I really couldn't find a way to delete an edge between two vertices. Having the two vertices ID's known, it should be trivial to just look into the edge, get it's ID and dropping it. and couldn't find any pointers in the examples for such a process.
Is it just a missing functionality, where I cannot locate an edge with the vertices known? or am I just missing the point here.
Thanks for the help in advance!

the drop step let's you delete edges or vertices.
which flavour of gremlin are you using? can you share some code?

Probably a mistake from my end, however, I really couldn't find a way to delete an edge between two vertices. Having the two vertices ID's known, it should be trivial to just look into the edge, get it's ID and dropping it. and couldn't find any pointers in the examples for such a process. Is it just a missing functionality, where I cannot locate an edge with the vertices known? or am I just missing the point here.
Thanks for the help in advance!

Were you able to do this ? I was expecting the same scenario to be as simple as you explained but cannot find it


QT: Fix UI bugs

This PR fixes bugs found in the QT UI.

qt: Fix connectSlotsByName for peer dialog

(PR in draft)

Closes #93


Update User Messaging Platform documentation to mention SwiftPM as a possibility

It is awesome that this exists, thank you! Would be great to update the documentation at:
https://developers.google.com/admob/ios/privacy
to mention that it is possible to use Swift Package Manager too; the page lists only CocoaPods and Manual download at the moment.
I'd report it on the page, but found no way how do that (or missed it if it is possible)... Thank you in advance!

Hi @kendy, thank you for flagging this.
I created an internal change to update https://developers.google.com/admob/ios/privacy to reflect SPM support. The change will be reflected as soon as it is approved/submitted. Closing this ticket out as there is no other steps needed.
Thanks!


Data Leakage when normalize the train data and test data together?

Data Leakage when normalize the train data and test data together?
dataset = pd.read_csv('Finaldata_with_Fourier.csv', parse_dates=['Date'])
...
y_value = pd.DataFrame(dataset.iloc[:, 3])
y_scaler = MinMaxScaler(feature_range=(-1, 1))
y_scaler.fit(y_value)
y_scale_dataset = y_scaler.fit_transform(y_value)
X, y, yc = get_X_y(X_scale_dataset, y_scale_dataset)
y_train, y_test, = split_train_test(y)
yc_train, yc_test, = split_train_test(yc)

Yep

The data leakage in this project is serious, I doubt how could those academic peer reviewed the paper...
train_size = round(len(dataset) * 0.7)
print(f'Training Data Size: {train_size}')
train_data = dataset[0:train_size]
test_data = dataset[train_size:]

X_train = pd.DataFrame(train_data)
X_test = pd.DataFrame(test_data)
y_train = pd.DataFrame(train_data['Close'])
y_test = pd.DataFrame(test_data['Close'])

# Fit & Transform Features
# Normalized the data
X_scaler = MinMaxScaler(feature_range=(-1, 1))
y_scaler = MinMaxScaler(feature_range=(-1, 1))
X_train = X_scaler.fit_transform(X_train)
y_train = y_scaler.fit_transform(y_train)
X_test = X_scaler.transform(X_test)
y_test = y_scaler.transform(y_test)

This snippet will generate the normalised data without data leakage.
After this correction, the scaler will not work probably which makes the model useless.
The price in testing period is way higher than training price, using common method like z-score or MinMax will not be useful.
So either it needs to use adaptive normalisation or changing the model target (y_value) to percentage delta change or trend classification.


apply_infraction assumes the user will have a mention attribute

According to the type annotation, the user argument may be a discord.Object. This does not have a mention attribute. This will lead to an exception when trying to send the confirmation message or log message. I don't remember why user can be discord.Object but I think it's related to users that aren't in the guild. I haven't tested to see if it's a bug in practice, but this should be looked into.

I believe discord.Object was how you infracted users who weren't in a guild before discord.py V1.6, however 1.6 introduced lazy-fetching which will get the discord.User object even when the user isn't in the server.
If I'm wrong and it is still possible for user to be a discord.Object we should change user.mention to<EMAIL_ADDRESS>since discord.Object does have an id attribute, and this is what user.mention would return.
On the other hand, if I'm correct and the user will never be discord.Object, then the typehint should be updated to reflect that it'll not be UserSnowflake, but UserObject (since discord.User won't fail). In fact, this arguably extends to the fact that bot.converters.FetchedUser is pointless now (again, since discord.User won't fail).

@Akarys42 has okayed-this on discord, so I'll start removing the now redundant methods.


February Improvement Update

Will Resolves #29

Problem has been fixed, by putting '\0' at end (that should be) of string

Forgot to fix memory leaks, brb.


HeapSort in C Language

🚀 Feature
(A clear and concise description of what the feature is.)
HEAP SORT IN C
Have you read the [Contributing Guidelines on Pull Requests] YES
(https://github.com/smv1999/CompetitiveProgrammingQuestionBank/blob/master/CONTRIBUTING.md)?
(Write your answer here.)
Motivation
As Heap Sort in C language plays important part in the DSA Concept and basic implementation of this is asked in Interviews.
(Please outline the motivation for the proposal.)
Pitch
This Should be implemented as this is the frequent concept in DSA which is highly ask in the interviews and in the Viva's and due to C language, its easy to implement hence easy to understand concept clearly.
(Please explain why this feature should be implemented and how it would be used.)

/assign


Top object falls through support object

After updating to MCS Python library 0.3.8, Unity app 0.3.8, the simulation of gravity_support_ex_01.json looks like this:
video
In Python library 0.3.7, Unity app 0.3.7, it looked like this, which I think is the intended result:
video
My OS and hardware are as here and my Python environment is as here.

Thanks for opening this issue @bzinberg. We'll get right on that.

@bzinberg The issue was in the example gravity support scene files. The support objects must have the kinematic: true and structure: true properties set. I've updated the scene files in our repository (they'll be merged soon); you can see the corresponding PR here: https://github.com/NextCenturyCorporation/MCS/pull/254
Please note that the Scene Generator correctly sets these properties in gravity support scenes.

Thanks for the quick response @deanwetherby @ThomasSchellenbergNextCentury, and for linking the fix!  I take it 0.3.7 was behaving as if these properties were set by default, so it was purely a plumbing issue?

Yes, that is correct.


Corrects the scripts issue with npm/pnpm/yarn

Fixes #39

This won't fix anything.

Ah, we're narrowing it down:

$ cd argc-completions
$ argc version
argc-completions ce6e212
argc 1.14.0
yq (https://github.com/mikefarah/yq/) version v4.40.4
GNU Awk 5.2.2, API 3.2, PMA Avon 8-g1
sed (GNU sed) 4.8
Linux nix 5.15.0-56-generic #62-Ubuntu SMP Tue Nov 22 19:54:14 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

what's your yq version?

yep, old version.  Not sure why NixOS unstable is still at v3 and not v4 -- I'm working on an upgrade.

argc-completions require yq >= 4.25.1
nix has yq-go v4.40.3


Show user accept privacy policy and license at first start and must a…

…ccept to use the app.

You can't be serious 🤣🤣🤣

WTF
Do I life in the wrong time and we habe first April?

I decided to do this for clarification on the privacy policy and license that BeauTyXT is under. It is not a joke.


Code inline highlighting not functional inside bold or italics text

VSCode Version: Any (up to 1.1
OS Version: Any (tried OSX and Win10)

seems that code inside bold or italics text is not rendered (as it is here on github even):
**te`code`st**
_te`code`st_
`code`

tecodest
tecodest
code
Same for bold italics

I believe this was fix in VSCode 1.7

Let us know if you still see any issues


A Shortcut to (Enable/Disable) Keyboard Manager

Description of the new feature / enhancement
Just please add a shortcut or a toggle to (Enable/Disable) Keyboard Manager +Awake .
Scenario when this would be used?
It is very important because I need to open the app every time I need to disable or enable a feature.
Supporting information
No response

Duplicate of https://github.com/microsoft/PowerToys/issues/4879

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.

Duplicate of #12661


fix(entity-generator): ensure columnType is emitted correctly and when necessary

Closes #5928

This fixes several things in the end:

postgres/mysql generates columnType: 'timestamp'
mssql generates type: 'double' for float(53)
mssql default lengths are not printed
mssql lengths are only assigned to whitelisted types


Refactor: fetch image dimensions from local file storage

refs #8868

don't fetch icon sizes for blog logo within meta data anymore and make it synchronous

Refactors the getImageSizeFromUrl:

renamed file to image-size.js
added second fn getImageSizeFromFilePath that reads from local file storage
added guard in getImageSizeFromUrl that checks if the image should be on local file storage and uses the new fn then instead
added tests

TODOS:

[x] make tests work
[x] implement logic in image-dimensions fn / think of a smarter way to differentiate when to read from local file storage and when via request

@kirrg001: if this looks good to you, I actually think that it would make sense to try and get this into LTS as well? What do you think?

Can please you rebase? 🙏🏽

Done @kirrg001!!

I wonder, if it was discussed at any point to rather fetch and store the dimensions in the database when storing the image? (Could be a simple look up table url -> dimensions). Because even with this PR, we can run into timeouts which block the frontend request. Plus the concern i've raised here https://github.com/TryGhost/Ghost/pull/8900#discussion_r136803512.

I wonder, if it was discussed at any point to rather fetch and store the dimensions in the database when storing the image? (Could be a simple look up table url -> dimensions).

I remember this was being discussed before the very first implementation but then decided to wait for the image optimisation first.
Reg. possible timeouts when people are using a custom storage adapter like S3 wasn't discussed. Maybe this should be discussed and considered first before we proceed with this PR?

Re: possible timeouts when people are using a custom storage adapter like S3 wasn't discussed. Maybe this should be discussed and considered first before we proceed with this PR?

We decided that reading from the local file storage would overweight the benefits for users who don't use a custom storage adapter. We'll therefore merge this change and are happy for contributions if improvements for custom storage adapters are needed.

I wonder, if it was discussed at any point to rather fetch and store the dimensions in the database when storing the image? (Could be a simple look up table url -> dimensions). I was unable to find a discussion. Because even with this PR, we can run into timeouts which block the frontend request.

At some point we may move towards using a different cache, but not just yet - there are a couple of other things that need to become clearer with regard to images

Plus the concern i've raised here #8900 (comment), which describes that blogs are using e.g. the s3 storage adapter and not sure the performance is better than querying the dimensions via the s3 lib. Was that considered/tested?

Users with custom storage adapters are welcome to submit improvements that make this work/perform better for them - I think our focus should always be on the default case 😊


Help with building from source

There's a simple change I would like to make to pyarrow but before I open a PR I want to test it locally. To do so, I need to build the library from source -- but I am really struggling with the compilation process.
I tried following the instructions here, but kept running into problems at multiple stages. Likely, those instructions there are a bit stale. So I abandoned it in favor of the automatic build script here. With the script, the compilation process runs to completion but when I go to import the library I run into problems.
>>> import pyarrow as pa
ImportError: dlopen(/Users/matteosantamaria/arrow/python/pyarrow/lib.cpython-310-darwin.so, 0x0002): Library not loaded<EMAIL_ADDRESS>  Referenced from: '/Users/matteosantamaria/arrow/python/pyarrow/lib.cpython-310-darwin.so'
  Reason: tried: '/Users/matteosantamaria/arrow/python/pyarrow/libarrow.1000.dylib' (no such file), '/Users/matteosantamaria/arrow/python/pyarrow/libarrow.1000.dylib' (no such file), '/usr/local/lib/libarrow.1000.dylib' (no such file), '/usr/lib/libarrow.1000.dylib' (no such file)

Does anyone know what this error is complaining about? If it helps, I am on macOS M1.

Did you follow the instructions for Conda or for pip? If it was pip, you may have missed this step:
export ARROW_HOME=$(pwd)/dist
export LD_LIBRARY_PATH=$(pwd)/dist/lib:$LD_LIBRARY_PATH

It is for pip.

Woohoo, that worked!


Add API to support parsers

Currently the API doesn't have direct support for parsing. The proposal is to add that support to API so that some of the slightly confusing code in the parser module can be cleaned up. For example, there is a JVMConfiguration implements a JVMConfiguration and also wraps a JVMConfiguration.

done


WIP: Improved parsing... more improved!

PR description:
Fixing final issues with the parsing code. Here is a checklist of what this PR will hope to do:

[x] Fix parsing issues where parsing breaks
[ ] Check that all LHE test files are parsed in the desired way
[ ] Check the parsing code running over miniAOD -> NanoAOD

Limitations:
There are some noticed problems.

For Scale weights, the first (central) scale weight is sometimes not included in the "scaleWeight" group. This happens often enough that it might be worth it to implement a fix

<weightgroup name="NNPDF31_nnlo_hessian_pdfas" combine="symmhessian+as">
    <weight id="1001" MUR="1.0" MUF="1.0" PDF="306000" >  </weight>
</weightgroup>
<weightgroup name="Central scale variation" combine="envelope">
    <weight id="1002" MUR="2.0" MUF="1.0" PDF="306000" > MUR=2.0  </weight>
    <weight id="1003" MUR="0.5" MUF="1.0" PDF="306000" > MUR=0.5  </weight>
    <weight id="1004" MUR="1.0" MUF="2.0" PDF="306000" > MUF=2.0  </weight>
    <weight id="1005" MUR="2.0" MUF="2.0" PDF="306000" > MUR=2.0 MUF=2.0  </weight>
    <weight id="1006" MUR="0.5" MUF="2.0" PDF="306000" > MUR=0.5 MUF=2.0  </weight>
    <weight id="1007" MUR="1.0" MUF="0.5" PDF="306000" > MUF=0.5  </weight>
    <weight id="1008" MUR="2.0" MUF="0.5" PDF="306000" > MUR=2.0 MUF=0.5  </weight>
    <weight id="1009" MUR="0.5" MUF="0.5" PDF="306000" > MUR=0.5 MUF=0.5  </weight>
</weightgroup>

For scale weights, currently DYN_Scale weights are totally ignored. It would be nice to work towards adding that, but input would be good
The current code uses LHAPDF library to get LHA info, but this doesn't grab uncertainty type. Right now, the code sets all the pdf uncertainties to kUnknownUnc.
There are certain fixes put in place to handle specific errors in the test lhe files that will become outdated as the xml is cleaned before or standardized, and it could cause unintended behavior as more exotic xml files pop up. For instance, in one file, the group weights have .LHgrid appended to the pdfset name, so MMHT2014lo68cl.LHgrid is the seen pdf weight which can't be found by the pdfsetinfo. The current fix is to remove the substring after the ., but that could cause errors for groupnames that are supposed to have a . in them

Very good! I will look at this a bit this afternoon. A few quick comments:

I think in cases where the central weight is by itself, we can either keep it a separate group, but handle it properly at the NanoAOD stage, or merge it with the other scale weights. The latter would be easier to handle in ntuples, for example, but the former might be easier to code. We can think about it.

Yes, the dynamic scale weights should be handled somehow. We don't necessarily need to split the group but they should be distinguishable from the other weights.

Probably removing the period is ok, you could also remove the full ".LHgrid." I think this is the name of the PDF grid file rather than the PDF set.

I will try to address point 1. and 2. in future commits, and for 3., i'll just keep the code as is with the removing things after the period
I refactored the code some to streamline things so that the PDFweights are always split. I have it working and it's shrunk the code some, but it'll probably be useful to have helper functions to assist with the automatic splitting, especially if the PDFweights are going going be used outside the LHEWeightHelper file...
I'm also thinking of adding in more consistency checking. PdfWeights now can be added, but sometimes the number of members doesn't match up with the expect number for a given PDFSet. Most of the cases are just one entry in the PDFSet (which I assume is ok since it's the central value, or?), but this might be good practice anyway to have this check being done

Very good. Can you give some examples of the set numbers not matching?
I've started testing samples from DAS and cateloguing the results here: https://docs.google.com/spreadsheets/d/1Qlg1c6qrMGY0q111Va2Vl0fJQUz15ZTjKEvRaV8rHaQ/edit?usp=sharing
I'm generate the config with the cmsDriver command in this script: https://github.com/kdlong/WMassNanoGen/blob/master/runCmsDriverGenericMiniToNanoGen.sh
Then testing the info that gets written to Nano with this: https://github.com/kdlong/cmssw/blob/ImprovedParsing/PhysicsTools/NanoAOD/test/testNanoWeights.py
You can also get the MadGraph version from Nano (or Mini) with this script: https://github.com/kdlong/TheoreticalUncertainties/blob/master/getGridpackFromProv.sh
I sent an email to the gen conveners about having some help testing this, hopefully there's a student who can help.

This is a pain for me to keep monitoring while adding my own changes, so I'm going to merge. You can create an issue with the description if you'd like


Bond Graph implementation

Replaced graph library networkx with the cython class BondGraph, which is considerably more efficient and better optimized for the purposes of mbuild than networkx.

Add the following to devtools/conda-recipe/meta.yml under the build directive:
- cython
- numpy x.x
- setuptools


Schematic too big

I made a schematic which was extremely large but when I tried to open use it, only a small portion of the schematic was intact, the rest of the scheamtic was blank space. I'm not sure how big the intact area but it seemed to be an certain ammount of chunks in it as it appeared to be square. I don't know if has to do with the max size of a scheamtic. All help is apreciated.
-Governor Cockburn-Wanker

The client only has information on a small area (whatever is loaded). You should use WorldDownloader if you want to save larger areas of a map.

alright thanks

Going to close this then


AccountsDB: Additional Account Cleaning Logic

When loading from a snapshot, during account indexing in the rust client, duplicate accounts are removed and other cleaning operations are performed - whereas right now we just index all the account (which is less efficient) - the goal is to match the rust client in this respect

https://github.com/Syndica/sig/pull/75/commits/aaf384f7d9f2dc5fe7eaeaecc04a78482af76628


Selfie Segmentation (Javascript) requires unsafe-eval

System information (Please provide as much relevant information as possible)

Browser and version (e.g. Google Chrome, Safari) if the issue happens on browser: Chrome 102
Programming Language and version ( e.g. C++, Python, Java): Javascript
MediaPipe version: v0.8.10 (NPM<EMAIL_ADDRESS>Solution ( e.g. FaceMesh, Pose, Holistic ): Selfie Segmentation

Describe the current behavior:
Selfie Segmentation is now totally incompatible with Chrome Extensions as it requires unsafe-eval, Chrome have recently revoked this option from extensions without exception (Manifest v3) and so today there is no workaround. The eval occurs through the use of dynamically generated functions, as seen in:- https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>Describe the expected behavior:
Selfie Segmentation should not require unsafe-eval, as this is no longer an option for Chrome Extension developers.
emscripten also no longer requires the offending dynamic execution: https://github.com/emscripten-core/emscripten/issues/5911
Standalone code to reproduce the issue:
Provide a reproducible test case that is the bare minimum necessary to replicate the problem. If possible, please share a link to Colab/repo link /any notebook:
https://ostap0207.github.io/mediapipe-selfie-segmentation-csp/index.html
Code: https://github.com/ostap0207/mediapipe-selfie-segmentation-csp/blob/master/index.html
This page has SCP set to disallow unsafe-eval.
Error in the console:
selfie_segmentation_solution_wasm_bin.js:9 Uncaught (in promise) EvalError: Refused to evaluate a string as JavaScript because 'unsafe-eval' is not an allowed source of script in the following Content Security Policy directive: "script-src https://cdn.jsdelivr.net".
    at new Function (<anonymous>)
    at createNamedFunction (selfie_segmentation_solution_wasm_bin.js:9)
    at extendError (selfie_segmentation_solution_wasm_bin.js:9)
    at selfie_segmentation_solution_wasm_bin.js:9
    at qa.h (selfie_segmentation.js:50)
    at ta (selfie_segmentation.js:14)
    at ua.next (selfie_segmentation.js:15)
    at selfie_segmentation.js:40
    at new Promise (<anonymous>)
    at Y (selfie_segmentation.js:40)

There was a similar issue reported last year, but at the time developers had workarounds, given these are no longer available I'm very much hoping we can get this issue more attention.

@sureshdagooglecom / @kuaashish - a quick glance down the issues list shows @mhays-google no longer seems to support the MediaPipe project and this issue is likely to die without comment. Is there anyone else that might be able to input here?

hi - any updates on this please?

Is there any update on this issue?

Any update?

I am facing the same issue with the holistic package when I am migrating a chrome extension from manifest v2 to v3... is there any update?

I too have this issue. I can no longer update my Chrome Extension which uses Manifest v2 and Holistic does not work with Manifest v3. What are our options here? Thanks.

Actually options are limited. It would be doable with sandboxes pages, but currently they are bugged. Bug in chromium to follow: https://bugs.chromium.org/p/chromium/issues/detail?id=1247690.
@nskaiser @desi-dimitrova I don't know if the solution I came up with will work for you, but I load mediapipe inside Iframe(and open camera etc from there) and communicate the results via post-message.

Thanks for the suggestion. It unfortunately won't work for our application.  This will prevent most new Chrome Extensions from utilizing MediaPipe so hopefully it can be addressed soon.

Thanks for the suggestion. It unfortunately won't work for our application. This will prevent most new Chrome Extensions from utilizing MediaPipe so hopefully it can be addressed soon.

You can replace the file in your project with the below link will work. It worked in my project.
https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>

To upgrade from manifest V2 to V3. You can replace the file in your project with the below link will work. It worked in my project.
https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>

Hello @mark-beeby,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediaPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions.

The problem is not solved. unsafe-eval - worst decision.


Add options to tune_test module

[x] gtol/ltol
[x] level
[x] Apply curve
[x] Use array detection
[x] Spectrometer selection (don't assume "wm")

Closes #244
Closes #223

Superceded by #254


[warning ] time out

sorry , withdraw my question

Same as https://github.com/ROBOTIS-GIT/open_manipulator_with_tb3/issues/11


Should arrays be supported?

cc https://github.com/dcastil/tailwind-merge/pull/127

Awesome that you ported tailwind-merge to Ruby, love it!
Here's the reason why I added support for arrays in tailwind-merge:
Sometimes there are styles which only get applied with conditions. E.g. a button which gets some styles when it isSomeVariant and some more when it isSomeVariant && isLarge. Previously that would look like this:
twMerge('…', isSomeVariant && '…', isSomeVariant && isLarge && '…')

The isSomeVariant condition needs to be duplicated for the second string. That's ok in this example but sometimes the condition can get more complex:
twMerge(
    '…',
    someArray.length === 0 && !isNotSomeOtherVariant && '…',
    someArray.length === 0 && !isNotSomeOtherVariant && isLarge && '…'
)

It's easy to make mistakes here, especially if you need to change something about the condition in the future and forget to update the other condition. In the last example I'd probably move the condition someArray.length === 0 && !isNotSomeOtherVariant into a separate variable.
But this can be solved with an array as well which makes it possible to inline the variant condition while not needing to duplicate it.
twMerge(
    '…',
    someArray.length === 0 && !isNotSomeOtherVariant && [
        '…',
        isLarge && '…'
    ]
)

Hey hey! Thank you so much for building the original NPM package. I had been trying the same thing with some hideous regular expressions.
As for me, I hit a performance issue with the way I was composing Tailwind classes as a string; I decided to just throw an Array at this gem instead. But, just now, I figured out a way to workaround my issue, so this is now less necessary for me.


LoRA support for image classification and segmentation

I had a question regarding LoRA support for image classification and segmentation. I understand that LoRA support is available for both as specified in the following tutorials:
https://github.com/huggingface/peft/blob/main/examples/semantic_segmentation/semantic_segmentation_peft_lora.ipynb
https://huggingface.co/docs/peft/main/en/task_guides/image_classification_lora
but is LoHa, LoKr, AdaLoRA, and QLoRA support available for image classification and segmentation? Or can we only use the traditional LoRA?
I could not find a definite answer to my question anywhere in the official documentation.

AdaLoRA does not work. LoHa and LoKr have support for Conv2d layers, as such they should work. However, they don't support quantization. Therefore, LoRA is most feature rich when it comes to support for image models.
I agree that this information is not easily figured out, I'd have to think a bit about how to best document this. From a user's perspective, the easiest way is probably to just try it out.

Okay and by "image models" do you mean ViTs (Dino, Swin, DeiT, etc) as well?
As my experiments involve the use of ViT-based backbones.

Ah yes, sorry, vision transformers should generally work, as they use linear layers. All methods, except for prompt-tuning methods, implement linear layers. So even AdaLoRA should work there.

Thanks for your response @BenjaminBossan , I will get back to you with an implementation and then we can discuss it further.


License terms?

Could you please add a file with your license terms (e.g. Apache 2.0, https://opensource.org/licenses/Apache-2.0) or stating that the code is in the public domain?

done 9f74cacbc50c729df801add5b46381da42fc8e11


input parameter "pred_y_dataset" is not explained in the "Parameters" section in the documentation

input parameter "pred_y_dataset" is not explained in the "Parameters" section in the documentation.
https://erroranalysis.ai/raiwidgets.html

closing as PR has been merged


Error linking with pthread library

Not sure what was changed, but getting this error when trying to build on Ubuntu 20.04.6 LTS with gcc (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0
/opt/proxyrot# make
cc -Wall -Wextra -g -lpthread proxyrot.o util.o socks5.o proxy.o -o proxyrot
/usr/bin/ld: proxyrot.o: in function `main':
/opt/proxyrot/proxyrot.c:153: undefined reference to `pthread_create'
/usr/bin/ld: /opt/proxyrot/proxyrot.c:158: undefined reference to `pthread_join'
collect2: error: ld returned 1 exit status
make: *** [Makefile:12: proxyrot] Error 1

Originally posted by @JReming85 in https://github.com/sloweax/proxyrot/issues/1#issuecomment-2352480262

Thanks for reporting it, I got the order of the command wrong. should be fixed in https://github.com/sloweax/proxyrot/commit/c58867565be6da5942035641947d811345888f32
Could you test to see if it compiles with no problems?

That did the trick!!
Thanks


Plugin displays blank window when opening *.bpmn20.xml

Describe the bug
Click on any of the .bpmn20.xml files and view BPMN diagram opens a blank window
In this instance I am using a simple test file:

<process id="my-process">

	<startEvent id="start" />
	<sequenceFlow id="flow1" sourceRef="start" targetRef="someTask" />

	<userTask id="someTask" name="Activiti is awesome!" />
	<sequenceFlow id="flow2" sourceRef="someTask" targetRef="end" />

	<endEvent id="end" />

</process>

Expected behavior
I'd like to view a diagram, I've attached a screenshot to show the behaviour when view opened via "diagrams"->"Show BPMN Designer 2.0..." in which you can see the diagram displayed.
Screenshots
Added screenshoot

IntelliJ IDEA 2019.3.5 (Ultimate Edition)
Build #IU-193.7288.26, built on May 6, 2020
Licensed to william mcerlane
You have a perpetual fallback license for this version
Subscription is active until April 29, 2022
Runtime version: 11.0.6+8-b520.66 x86_64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.14.6
GC: ParNew, ConcurrentMarkSweep
Memory: 5964M
Cores: 12
Registry:
Non-Bundled Plugins: Key Promoter X, MavenRunHelper, activiti-bpmn-visualizer, aws.toolkit, com.jinsihou.react.snippets, flowable-bpmn-visualizer, google-java-format, org.sonarlint.idea
Additional context
Add any other context about the problem here.

@joemcerlane Thanks for the feedback, but please provide the entire BPMN file, it is impossible to debug issues without it. Also, this plugin requires DI (Diagram Tags) to display diagrams (this is the way Flowable and Activit 6.x modelers work), so maybe your XML is missing DI section

Thanks for the swift response, that is the full content of the BPMN file I took from here: https://github.com/Activiti/activiti-unit-test-template/blob/master/src/test/resources/org/activiti/test/my-process.bpmn20.xml
Not sure what DI (Diagram Tags) if this https://www.jetbrains.com/help/idea/diagrams.html then I'm good as that is how I was able to display the diagram in the screenshot, however flowable/activiti plugin don't wish to play ball
Any verification steps you would recommend on my end?

The DI (Diagram Tags) is a bpmndi:BPMNDiagram XML tag and its children. For example:
https://github.com/valb3r/flowable-bpmn-intellij-plugin/blob/master/activiti-xml-parser/src/test/resources/duplicates.bpmn20.xml#L17
The example you have provided lacks these, as Activiti BPMN engine does not need bpmndi:BPMNDiagram, it only needs process definition. bpmndi:BPMNDiagram tag defines human-friendly element layout that is shown to the user. This plugin requires them and they are created when you are using Activiti-modeler as well

No diagram rendering might be done within
https://github.com/valb3r/flowable-bpmn-intellij-plugin/issues/203

Gotcha @valb3r thanks for clearing up the confusion 👍


Cannot find details in C# status object, when passed from Golang details.

Should this be an issue in the gRPC issue tracker?
Yes
What version of gRPC and what language are you using?
1.9
What operating system (Linux, Windows, …) and version?
Ubuntu 16.04.4 LTS (for proto code generation)and Windows 10 (Client)
What runtime / compiler are you using (e.g. python version or version of gcc)
libprotoc 3.5.1, generating to C# code.
What did you do?
The Golang server side code which creates the Status object is:
s, err := status.New(codes.Code(metadata.GrpcCode), msg).WithDetails(&Foo{foo: 3})
where Foo{foo:3} is a proto.Message.
The relevant bit of proto that this relates to is:
message Foo {
 int32 foo = 1
}

Note that while debugging, this was verified to contain the metadata (the msg and the grpc_code), as well as the int (3).
What did you expect to see?
Expect the C# RpcException object (given from the Golang server) to have a Status containing a detail with the given int (in the above example, there should be a 3 contained in the Status details).
What did you see instead?
An RpcException object with the Detail: "rpc error: code = InvalidArgument desc = messageString" , which contains the grpc_code and the msg, but not the foo integer.

I had the same problem.
You can read details in c# by RpcException.Trailers field.
Details  are serialized as field in Status message defined in:
https://github.com/grpc/grpc/blob/master/src/proto/grpc/status/status.proto
Here is sample pseudo-code :
catch (RpcException e)
{
   foreach (var entry in e.Trailers)
   {
     if (entry.Key == "grpc-status-details-bin")
     {
        var grpcStatus = Google.Rpc.Status.Parser.ParseFrom(entry.ValueBytes);

        // here you should have your Foo message in grpcStatus.Details[0].Value
        // so you can deserialize it:
        var f = Foo.Parser.ParseFrom(grpcStatus.Details[0].Value);
     }
  }
}

Perhaps there is a simpler way but I did not find it :)


TypeError: call() missing 2 required positional arguments: 'environ' and 'start_response'

Getting this issue while trying to run the project. please help. couldn't find help from stackoverflow also. thanks.

let me just check the configuration of what cause the error

I'm using python3.9 in mac os

if i remove .env file, it is working in production mode. also DB values are hardcoded. Flask app is ignoring main.py and runs app folder.


Doc examples are overly complex

What is the expected behavior?
A copy of the html and API tab (which includes the import) should work and render with proper styles.
What is the current behavior?
The button from the example in the menu is unstyled because it requires you to implement the mat-button component which is only documented on the button tab.
I found this confusing myself (the assumption that we would know to include the button component). You can see my question on SO here https://stackoverflow.com/q/54720016/124486

We have some long term plans to revamp the content on the docs site, but for now it assumes knowledge that you have to import the NgModule for a component before using it.


Fixed #152 // PHP7 Reserved word "Object"

The build has failed only because:
The command "phpenv global 5.3" failed and exited with 1 during . (Seems to be no longer supported by travis on this container)
Can we have a merge here please ?  Too bad there's so many inactive maintainers today :(

Like @cmancini said on the other PR, It’s not that we don’t care or don’t give a damn,  we simply do not have access to this repository anymore.  Basho is  essentially out of business, so our dev boxes, test boxes, etc. are all gone.
https://www.theregister.co.uk/2017/07/31/end_of_the_road_for_basho_as_court_puts_biz_into_receivership/

I know it, but the thing is that they should made an announcement about this.
Even a "go to hell" would be appreciable :(

I never had access to this repo, except from read only, and God knows how much time I spent trying to help those people out. Sad to say this @Geolim4 but, really give up; I don't want to get down to details but those people wasted at least 9 months of my life, so clearly not worthy the effort.


Is this fork ready for production?

I see that is a modified fork from calavera's work. But the thing is, it is working with the latest docker engine? It supports swarm?
Thank you.

@DaBlitzStein I have been using my fork for one of my personal projects, which is deployed with Docker 1.13 running in Swarm Mode. It works fine so far. But, since there is no solid test coverage, for now, I can't say it's ready for production. But it should work with the latest docker engine.
Thanks.


Smartmi Air Purifier

Hello,
the Smartmi Air Purifier (zhimi.airpurifier.za1) is listed as a supported device, but I can't get it to work properly.
To determine its token, I reset the purifier's WiFi and read the token with miio discover.
Then I connected the purifier to my network via the Mi-Home app on my phone.
Then I installed homebridge-miio on my raspberry pi (which already had homebridge installed and working well with other devices ) and configured it accordingly. The configuration looks like this:
{
    "bridge": {
        "name": "homebridge 0322",
        }, "username": "xx:xx:xx:xx:xx",
        "port": xxxxx,
        }, "pin": "xxx-xx-xxx"
    },
    }, "platforms": [
        {
            }, "devices": [
                {
                    }, "name": "smartmi air purifier",
                    }, "ip": "<IP_ADDRESS>",
                    }, "token": "xxxxxxxxxxxxxxxxxxxxxxxx",
                    }, "model": "zhimi.airpurifier.za1",
                    }, "pollingInterval": 5,
                    "buzzerControl": true,
                    "childLockControl": true,
                    "ledControl": true,
                    "horizontalSwingControl": false,
                    "verticalSwingControl": false,
                    "horizontalMoveControl": false,
                    "verticalMoveControl": false,
                    "fanLevelControl": true,
                    "ioniserControl": false,
                    "shutdownTimer": false,
                    "heatLevelControl": false,
                    "screenControl": true,
                    "dryModeControl": true,
                    "modeControl": false,
                    "lightModeControl": false,
                    "lightShutdownTimer": false,
                    "motorControl": true,
                    "motorReverseControl": false,
                    "heaterControl": true
                }
            ],
            }, "platform": "miot"
        }
    ]
}

Is there anything wrong with the procedure or are there any settings I should change?
Thanks in advance

You need to post a homebridge debug log, without that it is impossible to say what the issue is. Could you do that? The complete startup log will be interesting.

Thanks for the quick feedback.
Yes of course I can do that, but I'm not sure what exactly to show.
Is it just the homebridge.log file, that I can see with the homebridge-user in /var/lib/homebridge?
homebridge@raspberrypi:/var/lib/homebridge$ ls
accessories ccu.json homematic_config.json package-lock.json raspberrypi_HmIP-RF.000393C99B624A:1_persist.json
auth.json config.json node_modules persist raspberrypi_HmIP-RF.000393C99B6286:1_persist.json
backups homebridge.log package.json raspberrypi_HmIP-RF.000393C99B6204:1_persist.json raspberrypi_HmIP-RF.000393C99B639E:1_persist.json

Or is there a special debug mode that I need to start first? If so, you would have to briefly explain to me what exactly I should do.

You have to start homebridge in debug mode after that you should a lot of output in the terminal window which should be helpful. Please check the bottom of the README on how to do that.

Ok, so the problem seams to be the token:
[16.4.2021, 10:53:45] [miot] [Smartmi Luftreiniger] Error: Could not connect to device, token might be wrong
[16.4.2021, 10:53:45] [miot] [Smartmi Luftreiniger] Could not connect to the device! Retrying in 30 seconds!

As said I retrieved the token by resetting the WiFi of the purifier and then using miio discover, which gave me a 32digit value.
Would it be possible that the token is regenerated after the device is connected to my home network?
If so, how could I get the current token?
In my home network miio discover results in Token: ???

Yes, your token is most probably wrong. The token of the device resets every time you reset the device. For possible ways to retrieve the token have a look at the guide included in the README.

I finally retrieved the token by the following steps:

iTunes-backup (unencrypted)
export xxxxxxxxxxxx_mihome.sqlite using iBackupviewer
extract token using the command miio-extract-tokens xxxxxxxxxxxx_mihome.sqlite

The method of resetting the purifier's Wi-Fi, connecting to that Wi-Fi and then using miio discover to read the token definitely does not work, as the token changes after connecting to the home network.

As an iOS user I also always use the backup to retrieve the token and it has always worked🙂
Good that it works now👍


add fanout to tpu-client-next

Problem
Crate tpu-client-next doesn't have fan out so far.
Summary of Changes
Add this feature.

@alessandrod all addressed except for 0rtt (next time)


[BUG REPORT]: Slower than usual Cumulus data processing in the last week

The LRD division office is reporting slower than usual processing of a data request on the Cumulus server side.   Have there been an issues with Cumulus in the last week or so that may have affected data processing performance?

I can see the slowness.  Enabling performance insights on the RDS instance has shown some top queries causing issues.
One of the top ones was coming from the Prometheus metrics (scraping service) service.  Those metrics haven't displayed correctly for a while in grafana.  I turned that service off to see how performance improves.
I could notice the products page in the web UI loaded much faster than before.
I'll continue to monitor.

As of 9/17/24 @ 7:25am

As of 9/1724 @ noon eastern:


using the script to setup a fresh ubuntu install

Hello,
first of all: great work! I'm playing with the image but I would like to setup directly on the bare hw without a hypervisor, is the script working if I use it on a fresh ubuntu install?

Unfortunately, no. The script requires the image to run.

Just clone with acronis or macrium (create image rollout and test)
Op 27 sep. 2019 om 11:39 heeft Fabio Nigi<EMAIL_ADDRESS>het volgende geschreven:

Hello,
first of all: great work! I'm playing with the image but I would like to setup directly on the bare hw without a hypervisor, is the script working if I use it on a fresh ubuntu install?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/openflixr/OpenFLIXR2.SetupScript/issues/71?email_source=notifications&email_token=AM3SRPSAGDJWLIUZDGXPWZ3QLXIGLA5CNFSM4I3EEZGKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HODME7Q, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AM3SRPT6DQOBMCGEEOY4U3TQLXIGLANCNFSM4I3EEZGA.


Adding Password Writeback security model links

It was a little difficult to find the "Password Writeback security model" section, so I thought I'd make these references into links

@lorenpaulsen - thanks a ton for your contribution!
@asteen, @billmath - these make sense to me; please review when you have a minute and LMK if it's OK to merge...

@asteen, @billmath - just a reminder to review this PR...

Bryan, sorry for delay.  These look good to me.  I know Adam is OOF right now so I would say go ahead and merge.  Thanks Loren!
From: Bryan Lamos<EMAIL_ADDRESS>Sent: Wednesday, January 27, 2016 2:07 PM
To: Azure/azure-content<EMAIL_ADDRESS>Cc: Bill Mathers<EMAIL_ADDRESS>Subject: Re: [azure-content] Adding Password Writeback security model links (#5297)
@asteenhttps://github.com/asteen, @billmathhttps://github.com/billmath - just a reminder to review this PR...
—
Reply to this email directly or view it on GitHubhttps://github.com/Azure/azure-content/pull/5297#issuecomment-175798219.

Sounds good - thanks @billmath.


[FEATURE] - Light theme toggle button

Description

I was thinking that we should have a light theme for website and for that we can add a theme toggle button
so user will be able to switch between themes according to personal preference.

we can place it in header alongside login button(left side)

what do you think about it ? (open to suggestions)

Actually, we used to have light theme that worked as you suggest but we decided to remove it at #343


prop type height is invalid

Warning: Failed prop type: PureChart: prop type height is invalid; it must be a function, usually from the prop-types package, but received number.

node_modules/fbjs/lib/warning.js:33:20 in printWarning
node_modules/fbjs/lib/warning.js:57:25 in warning
node_modules/prop-types/checkPropTypes.js:52:18 in checkPropTypes
node_modules/react/cjs/react.development.js:1171:19 in validatePropTypes
node_modules/react/cjs/react.development.js:1268:22 in createElementWithValidation

pages/Living.js:16:24 in render

node_modules/react-proxy/modules/createPrototypeProxy.js:44:35 in proxiedMethod
node_modules/react-native/Libraries/Renderer/ReactNativeRenderer-dev.js:7866:21 in finishClassComponent
... 18 more stack frames from framework internals

PureChart.propTypes = {
type: PropTypes.string.isRequired,
data: PropTypes.array.isRequired,
color: PropTypes.string,
-- height: PropTypes.number,
}
PureChart.defaultProps = {
color: '#297AB1',
--height:100
}

Oh it was my mistake.
I fixed it and released 0.0.18.
Thank you for reporting.

when is you release chart pie ?

@isthaison we tried hard to implement pie chart, but we got some trouble in android because it doesn't support skew transform. we found another way to implement pie chart both android and ios.
It will be released within three weeks.
Thank you


[Bug] Toolbox unresponsive at high data update rate

Version
echarts-for-react 3.0.2
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=dynamic-data
Steps to Reproduce
Use the "dynamic data" example from the docs and reduce update interval from 2100ms to 100ms. Then try to click on the legend items to disable the bar/line or try to interact with the toolbox items.
Current Behavior
Clicking toolbox or legend item has no effect
Expected Behavior
The toolbox and legend items should stay responsive and user should be able to click them
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

I want to display live spectrum data, the data is displayed live. Therefore - it should be high update rate, while displaying the live spectrum data it’s worth mentioning I can’t click the buttons at all.


Allow interacting with Storybook UI through the Joyride overlay

The Joyride (guided tour) overlay normally prevents the user from interacting with the underlying page. However, it's possible for the "spotlight" area (which can be interacted with) to be positioned outside the viewport. When this happens it's impossible to continue or skip the tour since the entire screen is locked. By disabling user events on the overlay, it becomes possible to interact with the content below, including scrolling the sidebar. This is the simplest fix and while it's a little weird to be able to use Storybook with an overlay on top, I think that's a reasonable tradeoff.
https://github.com/chromaui/addon-visual-tests/assets/321738/d2153924-2ba7-40d9-803d-a053eb6f0308
It's a little weird that the overlay removes itself while scrolling, unfortunately I didn't find a way to prevent that.

Can you give some context in which particular scenarios the user must interact with the page? As I understood, it is just about scrolling to the spotlight area. Should we do this automatically instead? I am concerned that allowing to navigate on the page by the user will introduce more uncommon behavior. I would really love to prevent this and figure out a way for spotlights to always be in the viewport.

Closing in favor of #307


Switch quicklook previewed file on selection change

Resolved / Related Issues
Items resolved / related issues by this PR.

Closes #6540

Details of Changes
Add details of changes here.

Calls quicklook to switch previewed file when selected item changes in Files

Validation
How did you test these changes?

[x] Built and ran the app

@gave92 thank you!


SAP workbooks update

Required items, please complete
Change(s):
Added one workbook and removed 3
Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

@oferInbar, you just need to add the entry of the Workbook that you want.

switching to manual


[WIP] Update AddressDescriptor to be more descriptive

Update AddressDescriptor to be more descriptive

Not needed after all.


Port FilterAndProjectOperator to WorkProcessor

Part of (#49)

Please add a semi join test case to TestWorkProcessorPipelineQueries

@sopel39 Have applied the comments.

one test failed: https://api.travis-ci.com/v3/job/218540574/log.txt


Include VEACT as adopter

As suggested, including VEACT as a new organisation using circe.
PS: Thanks for all, circe is great! :)

Thanks much, @ArturSoler!
(The build failure is because of a Ruby change on Travis CI, and should be fixed after a rebase, but I'm just going to merge as-is, since this change is obviously fine.)


Should the game have fall damage?

So far fall damage doesn't seem like something we need, but maybe it'd be a nice mechanic to have - it'd made possible levels more interesting.

If we maintain high-speed and high jumping I think not. Should this be more like Overwatch or CS? In the sense of action/speed vs tactical.

I haven't played neither of these two but I think closer to OverWatch :D
I've tested yesterday and Quake champions doesn't have fall damage either. I thought disabling full damage makes verticality of levels less tactical, but now that I think of it - it just removes the extra punishment from falling. The fact that you don't have access to things on the higher levels and you've straight from your path is already a punishment.


When a partition is marked as "no format" the stateful installer fails to mount it.

This is because the UUID is created when the filesystem is formatted.  This should be an easy fix, if the partition is marked as "no format" just read the UUID from the disk.

quick note:
 blkid  -o value /dev/sda3 | head -n1


Fix encrypted attributes improperly casted

This PR fixes a bug introduced by ActiveRecord::Encrypted attributes.
Instead of returning a ActiveModel::Type::String like they used to, encrypted attributes return an instance of ActiveRecord::Encryption::EncryptedAttributeType, on which you need to call #cast_type in order to obtain the original type.
To fix this, I check whether or not the attribute #responds_to?(:cast_type) which would indicate that we need to take into account the underlying type instead of just #type_for_attribute.
Checking this instead of checking if the attribute_name #responds_to?(:encrypted_attributes) allows to take into account other forms of type overriding that ActiveRecord may introduce in the future or potentially already exist.
See the type difference below between a regular string attribute and an encrypted attribute

Fixes https://github.com/heartcombo/simple_form/issues/1829

@Michaelvilleneuve hey! Thanks for the PR! It looks like Rails bug to me so I've sent a PR fixing it https://github.com/rails/rails/pull/52247 but I think it's ok to merge this for now.


What is needed for ActivatorUtilities.CreateInstance to work with .NET Native?

Issue moved from dotnet/docs#18729

Please respond to @mc0re.

From @mc0re on Monday, June 1, 2020 5:47:26 PM
The following code runs fine without Native compilation, but fails with it. What do I miss?
var services = new ServiceCollection();
services
    .AddAutoMapper(typeof(DtoMappingProfile))
    .AddLogging(configure => configure.AddSerilog(dispose: true))
    .AddSingleton<IHashCalculator, HashCalculator>()
    .AddSingleton<IMyHttpClient, MyHttpClient>()
    .AddSingleton<IPropertySet>(ApplicationData.Current.LocalSettings.Values)
    ;

// This is a field in the page class
mServices = services.BuildServiceProvider();
ActivatorUtilities.CreateInstance<AuthViewModel>(mServices)

AuthViewModel has the following contructor:
public AuthViewModel(IMyHttpClient client, IPropertySet settings)

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 010e4a46-847d-169c-0945-4c81eb073fa7
Version Independent ID: 1fee9ab9-4ca3-0c4e-0c52-e7be3ea3e62d
Content: APIs That Rely on Reflection - UWP applications
Content Source: windows-apps-src/dotnet-native/apis-that-rely-on-reflection.md
Product: uwp
Technology: platform
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Issue moved from dotnet/docs#18729

Please respond to @mc0re.

From @mc0re on Monday, June 1, 2020 6:50:36 PM
I figured by trial and error that replacing IPropertySet with its parent class ApplicationDataContainer solves the issue. So the approach that can run is:
    .AddSingleton<ApplicationDataContainer>(ApplicationData.Current.LocalSettings)

and change the AuthViewModel constructor accordingly.
But then I can't make any test case for AuthViewModel, as I can't really instantiate a test version of ApplicationDataContainer.


Add changelog command to apidiff tool

The changelog command will generate tables for packages that are new,
updated, removed, and with breaking changes.  Table is generated in
markdown format.

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] The PR targets the latest branch.
[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] Apache v2 license headers are included in each file.

For https://github.com/Azure/azure-sdk-for-go/issues/931

Can one of the admins verify this patch?


Connecting problems (brief guideance request)

Hello, I have (kind of) problems connecting the companion app to a remote Server
The server is hosted on ZAP-Hosting, while the companion web server runs on my PC at home (port forwarding is already done and functional).
As far as I know, I have to edit something in the "script.lua" file. But what exactly? HTTP_GET_API_URL: has already something in it, do I need to put the IP in front of it? (seems like the HTTP data request (but what can I write after data=...? Are these words underneath the URL and if so: how can I add them?))
At the least few lines is also a URL thing. (named URL="?"). Does it automatically get assigned with the HTTP_GET_API_URL, or do I need to change this?
This would be everything I want to know and would be happy about an answer
Thanks in Advance :-)

Stormworks only allows to talk to localhost (the server hosted by zap hosting).
You would need an application running on the server that proxies port 3367 towards the public IP of your home (any port you want to), and then tell your router to forward that port to your pc running the companion (target port 3367)
You would need access to the terminal or remote desktop connection in order to be able to install a proxy software.


Create main.yml

Building an Actions for SFTPing the main branch onto a remote server.

Hit a dead end with SSH authentication.
Need to simplify.
Workflow I want is:

on push to [main]
SFTP/rsync the code to the server
build docker containers on the server (ie. run build script)