[RFC][DISCUSS] Tuple-related Fusion

Recently, there are a few problems arise wrt to the fusor and tuples. The main problem is the incompatibility of the calling convention when dealing with tuples.

At the lowest level, the primitive function is not aware of the tuple, and we simply flatten everything including inputs and outputs when there is a tuple input or return values.
The function that calls into the primitive function is aware of tuples.

This creates tension between at which point we should run the fusion algorithm. Originally, both Tuple and TupleGetItem are opaque, which means we should not fuse them. Then throughout the course, we started to fuse tuple-related node because they are useful as intermediate values, and can be optimized away in primitive operations.
However, tuple itself is bad as return values and can cause a bunch of problems for low-level code gen. In particular, we can get functions as follows(note the duplication in return values)
fn (%data: Tensor[(1, 32, 32, 3), float32]) -> (Tensor[(1, 32, 32, 3), float32], Tensor[(1, 32, 32, 3), float32]) {
  %3 = fn (%p0: Tensor[(1, 32, 32, 3), float32], __dict__=meta[StrMap][0]) -> (Tensor[(1, 32, 32, 3), float32], Tensor[(1, 32, 32, 3), float32]) {
    %0 = log(%p0)
    %1 = (%0, %0)
    %1
  }
  %4 = %3(%data)
  %5
}

Ideally, what we want is to fuse TupleNode to follow up consumer nodes if they are intermediate nodes, but not do so when they are the return values. So we won't suffer from this problem(as TupleNode itself cannot be the final master node).
Let us use this thread to do some consolidated discussion on the related topic

@tqchen where is %2?

@junrushao1994

@tqchen where is %2?

There might be some code emitted, but the idea is to the problem when dealing with duplicate values in return tuples.

why is the example bad for codegen

The output tensor is scheduled twice in compute_engine here:
https://github.com/dmlc/tvm/blob/552d4aa3587aa3a0443d050ceb324386489ff793/src/relay/backend/compile_engine.cc#L128
There are some problems and discussions in #2932

We can update TOPI to prevent scheduling the same op twice. It is partially done in #1556

@masahi Can we prevent from passing duplicated tensors instead? It looks that we otherwise need to change all schedules for all targets in topi, right?

@zhiics I don't know a solution for your problem without breaking other parts (like breaking calling convention as you and @vinx13 said). I'll trying fixing the fuser so that a tuple like (%0, %0) won't happen.

It seems #2412 is also related to this issue. @srkreddy1238 mentioned that enabling fusion caused a LLVM error.
When fusion support for tuple was added in #2187, I handled the case where

the tuple is root of its group
the tuple is fused with other ops before it

by making a new function that returns the tuple (see my comment). Is this what needs to be changed? @tqchen @zhiics @vinx13 ?

@zhiics can you replace this block with this
  Expr VisitExpr_(const TupleNode* tuple) {
    auto* ret_group = gmap_.at(tuple)->FindRoot();
    if (ret_group == gmap_.at(tuple)) {
      return ExprMutator::VisitExpr_(tuple);
    }
    Array<Expr> new_fields = GetNewArguments(tuple->fields, ret_group);
    // This tuple is an intermediate node in the group
    return TupleNode::make(new_fields);
  }

and try your PR?

one possible way to make the fusor smarter. So that we forbid fusing of a node into TupleNode if that is the last in the group, but allow fuse TupleNode into subsequent injective ops, then in the second iteration the other node can fuse into tuple node because the group no longer have the tuple as return value

We should be aware that if we disable tuple fusion when tuple is the return value, we might lose some efficiency gain.
So this function
fn (%x: Tensor[(64, 64), float32]) -> (Tensor[(32, 64), float32], Tensor[(32, 64), float32]) {
  %2 = fn (%p0: Tensor[(64, 64), float32], __dict__=meta[StrMap][0]) -> (Tensor[(32, 64), float32], Tensor[(32, 64), float32]) {
    %0 = strided_slice(%p0, begin=[0, 0], end=[32, 64], strides=[1, 1])
    %1 = strided_slice(%p0, begin=[32, 0], end=[64, 64], strides=[1, 1])
    (%0, %1)
  }
  %2(%x)
}

will become this
fn (%x: Tensor[(64, 64), float32]) -> (Tensor[(32, 64), float32], Tensor[(32, 64), float32]) {
  %0 = fn (%p0: Tensor[(64, 64), float32], __dict__=meta[StrMap][0]) -> Tensor[(32, 64), float32] {
    strided_slice(%p0, begin=[0, 0], end=[32, 64], strides=[1, 1])
  }
  %1 = %0(%x)
  %2 = fn (%p01: Tensor[(64, 64), float32], __dict__=meta[StrMap][1]) -> Tensor[(32, 64), float32] {
    strided_slice(%p01, begin=[32, 0], end=[64, 64], strides=[1, 1])
  }
  %3 = %2(%x)
  (%1, %3)
}

BTW, I am not certain that stopping fusing return tuple will fully solve the problem because it looks to me that we will still have two identical tensor in the tuple, right? Am I missing something?

@zhiics It looks like the tuple with duplicated tensors is only problematic if it is the return value of a subfunction (i.e. a function that is lowered to topi and compiled by TVM). If we lift the tuple out of a subfunction and put it under the global function, it seems to work fine. The test below works on my local.
import tvm
from tvm import relay

data = relay.var("data", relay.ty.TensorType((1, 32, 32, 3), "float32"))
log = relay.log(data)
func = relay.Function([data],  relay.Tuple(tvm.convert([log, log])))
func = relay.ir_pass.infer_type(func)
with relay.build_config(opt_level=3):
    graph, lib, params = relay.build(func, target="llvm")

The tuple is now lifted out of subfunction %0.
fn (%data: Tensor[(1, 32, 32, 3), float32]) -> (Tensor[(1, 32, 32, 3), float32], Tensor[(1, 32, 32, 3), float32]) {
  %0 = fn (%p0: Tensor[(1, 32, 32, 3), float32], __dict__=meta[StrMap][0]) -> Tensor[(1, 32, 32, 3), float32] {
    log(%p0)
  }
  %1 = %0(%data)
  (%1, %1)
}

@masahi I see, thanks. Another option is probably using a copy operator if there are duplicates.

@tqchen I'm looking into the following approach:

Initially, the tuple and its fields are marked as Injective, so that they can be fused as much as possible.
Later, if we detect that a tuple is the root of its group in this line, we do all necessary patchwork to make each tuple fields become root in its group and return the tuple itself as a isolated node.

I think this approach requires minimum code change. The patchwork only includes updating some pointers (parent, root_ref, etc).

@masahi In your exmaple of two strided_slice, given that we do not yet cooperatively fuse the two parallel strided_slice together atm, the second version is mostly as efficient as the first one.

implemented in #3092


[BUG] Failed to upload artifacts to Maven in recent releases

Describe the bug
See https://github.com/streamnative/kop/runs/4746589549?check_suite_focus=true.
Error:  Rule failure while trying to close staging repository with ID "iostreamnative-1043".
Error:
Error:  Nexus Staging Rules Failure Report
Error:  ==================================
Error:
Error:  Repository "iostreamnative-1043" failures
Error:    Rule "javadoc-staging" failures
Error:      * Missing: no javadoc jar found in folder '/io/streamnative/pulsar/handlers/kafka-payload-processor/<IP_ADDRESS>'
Error:
Error:

It causes kafka-payload-processor and oauth-client cannot be uploaded to Maven repository.
To Reproduce
Expected behavior
The release workflow should pass.

The other failed actions are caused by 403 error
https://github.com/streamnative/kop/runs/4631085873?check_suite_focus=true
Error:  Failed to execute goal org.sonatype.plugins:nexus-staging-maven-plugin:1.6.8:deploy (injected-nexus-deploy) on project pulsar-protocol-handler-kafka-parent: Execution injected-nexus-deploy of goal org.sonatype.plugins:nexus-staging-maven-plugin:1.6.8:deploy failed: 403 - Forbidden -> [Help 1]

https://github.com/streamnative/kop/runs/4655363839?check_suite_focus=true
Error:  Failed to execute goal org.sonatype.plugins:nexus-staging-maven-plugin:1.6.8:deploy (injected-nexus-deploy) on project pulsar-protocol-handler-kafka-parent: Execution injected-nexus-deploy of goal org.sonatype.plugins:nexus-staging-maven-plugin:1.6.8:deploy failed: 403 - Forbidden -> [Help 1]

We should also migrate #1018 to master branch for future versions like 2.10.0.


0.1.14

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

It looks like the author of the library ( :stuck_out_tongue_winking_eye:) removed the setuptools dependency in https://github.com/scopatz/xo/commit/22ee7ed745c87f1b7a5d9022f5d4e6a10a90fa10
You need to remove --single-version-externally-managed from the recipe too.

Wow, I should really have words with that developer! Thanks for catching this @ocefpaf!


memory use rising high

Version of protoc (protoc --version)
2.6.1
Version of ProtocolBuffers.framework
2.4.3
.proto file to reproduce
syntax = "proto2";
message Person {
    required string name = 1;
    required bytes data=2;
}

Description
I  use 2.4.1,and write a simple test case, the memory usage keep rising, is that memory leaks? and code as following
func beginTest() {
var i = 0;
let fileManager = NSFileManager.defaultManager()
let mydir = NSHomeDirectory() + "/Documents/myfolder/test/"
let d:NSString = "AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA"
do{
try fileManager.createDirectoryAtPath(mydir , withIntermediateDirectories: true, attributes: nil)
let pathString = mydir + "q.txt"
        while i < 100000000 {
            i = i+1;
            let a = Person.Builder()
            a.setName(String(i))
            a.setData(d.dataUsingEncoding(NSUTF8StringEncoding)!)
            try a.build().data.writeToFile(pathString, atomically: true)
        }
    }catch{
        print(error)
    }
}

Swift language has memory leak when objects are bridge to objc from swift and backward. This example is not obvious.


Loading model from disk uses cpu_predictor instead of gpu_predictor

System: Windows 10 x64 Professional
Source GPU build from 0.72 release branch
Using R as follows:
> sessionInfo()
R version 3.5.0 (2018-04-23)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)

Matrix products: default

locale:
[1] LC_COLLATE=English_United States.1252  LC_CTYPE=English_United States.1252    LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C                           LC_TIME=English_United States.1252

attached base packages:
[1] parallel  stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] stringi_1.1.7     xgboost_0.71.2    doParallel_1.0.11 iterators_1.0.9   foreach_1.4.4     raster_2.6-7      rgdal_1.3-3
[8] sp_1.3-1

loaded via a namespace (and not attached):
[1] Rcpp_0.12.17      lattice_0.20-35   codetools_0.2-15  grid_3.5.0        magrittr_1.5      data.table_1.11.4 Matrix_1.2-14
[8] tools_3.5.0       compiler_3.5.0

Running this script:
require(xgboost)
data(agaricus.train, package='xgboost')
data(agaricus.test, package='xgboost')
dtrain <- xgb.DMatrix(agaricus.train$data, label = agaricus.train$label)
dtest <- xgb.DMatrix(data = agaricus.test$data, label = agaricus.test$label)
param <- list(max_depth=2, eta=0.1, nthread = 8, tree_method="gpu_exact", predictor="gpu_predictor", objective = "gpu:binary:logistic")
xgbmodel <- xgb.train(param, dtrain, nrounds = 5, verbose = 2)

xgb.save(xgbmodel, fname = "xgboost_model")
xgbmodel_from_disk <- xgb.load(modelfile = "xgboost_model")

#Run model from memory
set.seed(123)
head(predict(xgbmodel,dtest,predictor="gpu_predictor"))
head(predict(xgbmodel,dtest))

#Run model from disk
set.seed(123)
head(predict(xgbmodel_from_disk,dtest,predictor="gpu_predictor"))
head(predict(xgbmodel_from_disk,dtest))

Gives me this output:
> require(xgboost)
> data(agaricus.train, package='xgboost')
> data(agaricus.test, package='xgboost')
> dtrain <- xgb.DMatrix(agaricus.train$data, label = agaricus.train$label)
> dtest <- xgb.DMatrix(data = agaricus.test$data, label = agaricus.test$label)
> param <- list(max_depth=2, eta=0.1, nthread = 8, tree_method="gpu_exact", predictor="gpu_predictor", objective = "gpu:binary:logistic")
> xgbmodel <- xgb.train(param, dtrain, nrounds = 5, verbose = 2)
[08:45:43] Allocated 7MB on [0] GeForce GTX 1070, 6769MB remaining.
[08:45:43] G:\xgboost\src\tree\updater_prune.cc:74: tree pruning end, 1 roots, 6 extra nodes, 0 pruned nodes, max_depth=2
[08:45:43] Allocated 1MB on [0] GeForce GTX 1070, 6767MB remaining.
[08:45:43] G:\xgboost\src\tree\updater_prune.cc:74: tree pruning end, 1 roots, 6 extra nodes, 0 pruned nodes, max_depth=2
[08:45:43] G:\xgboost\src\tree\updater_prune.cc:74: tree pruning end, 1 roots, 6 extra nodes, 0 pruned nodes, max_depth=2
[08:45:43] G:\xgboost\src\tree\updater_prune.cc:74: tree pruning end, 1 roots, 6 extra nodes, 0 pruned nodes, max_depth=2
[08:45:43] G:\xgboost\src\tree\updater_prune.cc:74: tree pruning end, 1 roots, 6 extra nodes, 0 pruned nodes, max_depth=2
> xgb.save(xgbmodel, fname = "xgboost_model")
[1] TRUE
> xgbmodel_from_disk <- xgb.load(modelfile = "xgboost_model")
> #Run model from memory
> set.seed(123)
> head(predict(xgbmodel,dtest,predictor="gpu_predictor"))
[08:45:44] Allocated 0MB on [0] GeForce GTX 1070, 6767MB remaining.
[1] 0.4284496 0.5864953 0.4284496 0.4284496 0.3037330 0.5104644
> head(predict(xgbmodel,dtest))
[08:45:44] Allocated 0MB on [0] GeForce GTX 1070, 6767MB remaining.
[1] 0.4284496 0.5864953 0.4284496 0.4284496 0.3037330 0.5104644
> #Run model from disk
> set.seed(123)
> head(predict(xgbmodel_from_disk,dtest,predictor="gpu_predictor"))
[1] 0.4284496 0.5864953 0.4284496 0.4284496 0.3037330 0.5104644
> head(predict(xgbmodel_from_disk,dtest))
[1] 0.4284496 0.5864953 0.4284496 0.4284496 0.3037330 0.5104644

As can be seen above, only the xgboost from memory runs on the GPU, the one from the disk only runs on the CPU.

Sorry, but I do not have an account there and would prefer to not sign up for more accounts than necessary.  Perhaps since you have an account there you could point to this issue from there for me?  I would appreciate it as this is preventing us from doing some of our work (USGS).
Thanks,
Kory

You can use your GitHub account to sign in.


Saver (sharded) ignores number of checkpoints to keep

tf.train.Saver() has a default limit of 5 most recent checkpoints to keep (max_to_keep=5). When I use a Saver with sharded=True option, only one machine performs this check and keeps ALL the checkpoints without removing any of them.
Environment info
Operating System: Ubuntu 14.04 LTS
Built from source; commit hash: aa2cacd6627ffb296bedc910c957a0fd4a2f957f
Steps to reproduce

Init a sharded saver and pass it to the Supervisor for distributed training and auto-model saving after every save_model_secs.

saver = tf.train.Saver(sharded=True)
sv = tf.train.Supervisor(is_chief=is_chief,
                                 logdir=FLAGS.train_dir,
                                 init_op=init_op,
                                 summary_op=None,
                                 global_step=global_step,
                                 saver=saver,
                                 save_model_secs=FLAGS.save_interval_secs)

Start two workers and one ps on machine 0. Start two more workers and one ps on machine 1.
The saver keeps the 5 most recent sharded checkpoints on machine 0 (where the chief worker also runs). But machine 1 keeps all sharded checkpoints from beginning.

I can't immediately tell why, looking at the code.  Call stack: Saver.save() calls _MaybeDeleteOldCheckpoints(), which will inspect max_to_keep.
Could you add some debug logging to both these methods, and relaunch on the two machines?

Can you please guide on how to add the debug logging?

print("in save()") and print("in _MaybeDeleteOldCheckpoints()")?  Your goal is to figure out if these two functions are actually called?

Automatically closing due to lack of recent activity. Please reopen if further information becomes available.


Add wifi support on RPI Zero Wifi

Connected to https://github.com/resin-os/resin-os-device-support/issues/4
 Front conversations

Reopening this as it seems that it was not fixed only by having the firmware available. It needs dtb too.

Relates to https://github.com/resin-os/resinos/issues/231

Realtes to #245

Relates to raspberrypi/firmware#785

does the latest resin image ie: 2.0.0+rev2-4.1.1-42b3340f8c2c contain support for pi zero w? since i'm just try it and doesn't seems so.

No @lfarkas . This issue will be updated when that happens.

Will this be fixed by v2.0.0+rev3?

Any updates on this? I'm still not able to use a Pi Zero W with Resin.

@RyanBalfanz we are in the process of releasing. The new 2.0.3 will include Pi Zero W. The update should hit the Pi family later today.

@telphan That's great news! Thanks for the update!

@RyanBalfanz Just want to let you know that we have release on Friday as scheduled. Enjoy the new Pi Zero W support.

@telphan Confirmed! I tested the latest release this past weekend on a Zero W using WiFi and it worked perfectly. Thanks for the update!

Which dev image can be downloaded that will support Pi Zero W?

@hellais Sorry for the late answer. I confirm that is indeed the correct image.


Clear Request After Reconnect Doesn't Work

Problem
After reconnecting to the IPC gateway, I am unable to clear my current presence using the clear_activity method. This occurs if I attempt to reconnect before sending the clear request. Here's the code that reproduces the issue:
use discord_rich_presence::{activity, DiscordIpc, DiscordIpcClient};

fn main() -> Result<(), Box<dyn std::error::Error>> {
    let mut client = DiscordIpcClient::new("1076049094281277531")?;
    client.connect()?;

    let payload = activity::Activity::new().state("Test");
    client.set_activity(payload)?;

    client.close()?;
    client.reconnect()?;

    std::thread::sleep(std::time::Duration::from_secs(5));

    client.clear_activity()?;

    client.close()?;
    Ok(())
}

Expected Behavior
The presence should be cleared when calling clear_activity after reconnecting.
Observed Behavior
While the presence is set correctly on initial connection, calling clear_activity after reconnecting does not clear it.
Additional Information

Operating System: Linux (x86_64)

Am I missing something in my implementation, or could this be a bug?

Could it be Discord's update rate?
Try sleeping a lot between clear_activity and close

Thank you very much, that actually worked! I will set the issue as closed. But I still have a question, maybe someone can help me :D
Is there a way to keep the rpc alive even after the program has ended and to clear the rpc later by another program? Because as far as I've seen, the rpc is terminated as soon as you end the current program where you initialized it, even if I don't call the close method or if I set the PID to a constant value instead of the program's PID.

The reason the stream closes when you stop your program even tho .close is never called is because Files (on windows) and UnixStream on Linux / Mac(i believe mac uses theses too) auto-close on drop.
Even Box::leaking it will probably get caught by your os and be closed.
The only way is to keep your program running.

Ok alright, that makes sense, thank you very much. I appriciate that fast and kind response :+1:


Will not install on 10.13.5 - Error OSStatus error 99999

Attempt to install on new 10.13.5 system - get "The operation couldn't be completed. (OSStatus error 99999.)"

I installed it ok on same OS version from the download link.

@sdmactech Best guess this was probably caused by a file missing from the bundle lost during decompression of the ZIP file or the bundle being transferred from one drive to another. Best suggestion for a fix is to download a new copy.

I'm getting the same thing on one of my computers, but not the other one.  Happening on both this and Butterflight's configurator, but nothing else that I've seen.  My system is 10.13.6.


Sketch Form Tool tip deactivation

Steps to reproduce:

Open an analysis > Tablview> Right Click> Select Tooltip> Set Default Tooltip> Select Sktch Form > Edit and Close
The table view now we can see the sketch form tooltip
Now want to turn off or change to default tip
There is no option to deactivate the sketch form tool and always shows the sketch form tooltip

Expected results: There should be an option to turn off sketch form tool tip and switch to default tooltip.
Actual results:  There is no option to deactivate the sketch form tool and always shows the sketch form tooltip.

Duplicate of this ticket -https://github.com/datagrok-ai/public/issues/758 and you may close this as duplicate.

Close as duplicate


setting up Sanity with Nuxt 3

Hello there, this is a great plugin and a lot of respect for that.
I have a short question where I'm exploring the new Nuxt 3 beta.
Is it possible to use this plugin in Nuxt 3 now ?
Has anyone have some example to share ?

v1 of @nuxtjs/sanity has been released with support for Nuxt 3 and Nuxt Bridge. Let me know how it goes, and feel free to raise any issues! ❤

@OrhanOzkercin Please create a new issue with a reproduction and I'll take a look 👍


Creating an invoice doesn't work on legend.lnbits.com Paywall extension

Describe the bug
I am building an app using legend.lnbits.com as the backend to make a lightning paywall. Everything up until this point is working perfectly. The problem is that I cannot seem to create an invoice in the Paywall extension. I am  receiving  {"detail":"Not Found"} when I run curl -X POST https://legend.lnbits.com/paywall/api/v1/paywalls/<paywall_id>/invoice -d '{"amount": <integer>}' -H "Content-type: application/json"
To Reproduce
Steps to reproduce the behavior:

create a paywall using: curl -X POST https://legend.lnbits.com/paywall/api/v1/paywalls -d '{"url": "https://www.test.com", "memo": "test memo", "description": "test desc", "amount": 100, "remembers": true}' -H "Content-type: application/json" -H "X-Api-Key: 788b......8656ed"
Try to create an invoice for that paywall using curl -X POST https://legend.lnbits.com/paywall/api/v1/paywalls/<paywall_id from step 1>/invoice -d '{"amount": 100}' -H "Content-type: application/json"
observe that the result of step 2 is  {"detail":"Not Found"}

Expected behavior
invoice is created after doing the command from step 2
Screenshots
I'm just using the Paywall extension in legend.lnbits, here is a screenshot of my screen:

Desktop (please complete the following information):

LNbits version: 9467a8da07bdf19bc9146d890098268b8979a9cf
Database: na

Additional context
Note I have the same problem (receiving "Not Found") when I try to check invoice status: curl -X POST https://legend.lnbits.com/paywall/api/v1/paywalls/<paywall_id>/check_invoice -d '{"payment_hash": <string>}' -H "Content-type: application/json"

if i could figure out how to programmatically check to see if a paywallId has been paid or not then i'd be unblocked.

I figured out the problem. It looks like the API documentation on lnbits site is wrong.
Here is the documentation that doesn't work on legend.lnbits.com:

And here is the documentation in swagger docs that works:
https://legend.lnbits.com/docs#/Paywall/api_paywall_create_invoice_paywall_api_v1_paywalls_invoice__paywall_id__post

Notice how the URLs are different

thanks for the report! fixed it here: https://github.com/lnbits/paywall/commit/a7530a04f85c3f79b20bb284a38939b332b3d846


java.lang.NoSuchMethodError | Whitelist error

I am attempting to use this mod on a small whitelisted server I'm running with friends and whenever a player joins I get this error and the server crashes.
java.lang.NoSuchMethodError: 'java.util.concurrent.CompletableFuture me.lucko.fabric.api.permissions.v0.Permissions.check(com.mojang.authlib.GameProfile, java.lang.String)' at Not Enough Crashes deobfuscated stack trace.(1.19.2+build.28) ~[?:?] at net.minecraft.server.dedicated.DedicatedPlayerManager.modifyReturnValue$ddn000$vanilla-permissions$vanillaPermissions_addBypassWhitelistPermission(DedicatedPlayerManager:1019) ~[?:?] at net.minecraft.server.dedicated.DedicatedPlayerManager.isWhitelisted(DedicatedPlayerManager:124) ~[?:?] at net.minecraft.server.PlayerManager.checkCanJoin(PlayerManager:423) ~[?:?] at net.minecraft.server.network.ServerLoginNetworkHandler.acceptPlayer(ServerLoginNetworkHandler:122) ~[?:?] at net.minecraft.server.network.ServerLoginNetworkHandler.redirect$bga000$fabric-networking-api-v1$handlePlayerJoin(ServerLoginNetworkHandler:563) ~[?:?] at net.minecraft.server.network.ServerLoginNetworkHandler.tick(ServerLoginNetworkHandler:73) ~[?:?] at net.minecraft.network.ClientConnection.tick(ClientConnection:248) ~[?:?] at net.minecraft.server.ServerNetworkIo.tick(ServerNetworkIo:183) ~[?:?] at net.minecraft.server.MinecraftServer.tickWorlds(MinecraftServer:886) ~[server-intermediary.jar:?] at net.minecraft.server.dedicated.MinecraftDedicatedServer.tickWorlds(MinecraftDedicatedServer:289) ~[?:?] at net.minecraft.server.MinecraftServer.tick(MinecraftServer:819) ~[server-intermediary.jar:?] at net.minecraft.server.MinecraftServer.runServer(MinecraftServer:665) ~[server-intermediary.jar:?] at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer:257) ~[server-intermediary.jar:?] at java.lang.Thread.run(Thread.java:833) [?:?]
It only crashes for other players, the reason it doesn't crash for me I think is that I joined and had my permissions set created as to say before I added this mod. I think the error originates from it trying to get permissions for a player who doesnt have any permissions made yet. I'm using luck perms to manage permissions as well as several other mods, but nothing that would impact permissions.
Game version is 1.19.2
Fabric version is 0.14.21
Im using the latest version of this mod
Server is running with java 17 openjdk

Hey, sorry for the late response. This problem is likely caused by fabric-permissions-api (a library this mod uses), because multiple different versions of it are running on your server and the wrong (older) version gets picked by fabric-loader, because the dependency didn't properly increase it's version number.
Can you try adding this file to your mods folder (it's the latest dependency version with a bumped version number)

Fixed in the latest 1.20.2 release (bumped fabric-permission-api version)


Compile root as newData.parent()+ for multi-path updates

In write and validate rules, when updating multiple paths at once, it is more appropriate to use newData.parent().parent() ...  to compare all the new values of the write instead of using root.... which represents the prior values.
@tomlarkworthy DO NOT MERGE - I need to rebase this on the koss-generics-2 branch.

Merged directly.


Shorten long logs

Some datasets may have unexpectedly long features/types (e.g. if the files are not formatted correctly).
In that case we should still be able to log something readable

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Toggling low power mode doesn't work

Workflow version
2021.1
Alfred version
4.6.4 [1294]
macOS version
12.3.1 (21E258)
Debugger output
[11:00:13.393] Low Power Mode[Script Filter] Queuing argument '(null)'
[11:00:13.619] Low Power Mode[Script Filter] Script with argv '(null)' finished
[11:00:13.621] Low Power Mode[Script Filter] {"items":[{"title":"Battery’s Low Power Mode is On","subtitle":"Action to turn it off","arg":"battery|off","icon":{"path":"icon_battery_on.png"}},{"title":"AC’s Low Power Mode is Off","subtitle":"Action to turn it on","arg":"adapter|on","icon":{"path":"icon_adapter_off.png"}}]}
[11:00:15.058] Low Power Mode[Script Filter] Processing complete
[11:00:15.059] Low Power Mode[Script Filter] Passing output 'battery|off' to Run Script
[11:00:17.332] ERROR: Low Power Mode[Run Script] /Users/tim/Library/Caches/com.runningwithcrayons.Alfred/Workflow Scripts/787A3C96-FABE-4607-9E6E-95DB4D3BA6F6: execution error: Error: Error: Objekt kann nicht gefunden werden. (-1728)
[11:00:17.339] Low Power Mode[Run Script] Processing complete
[11:00:17.340] Low Power Mode[Run Script] Passing output '' to Run Script

More details
No response

Please don’t make simultaneous duplicate posts in multiple places. I’ve hidden the one in the forum so we can keep the conversation focused.
The problem is that toggling requires some GUI scripting (which is why the System Preferences open briefly) and your system in not in English. I’m considering possible solutions.

2022.1 should fix it.


Larger #area to let user zoom out

can't have permission to push branch
in order to let user zoom out and keep png background when he has large DB
I do not have any large background with tecno used inside this repo so i have some issues to make it running local, and i was wondering if it could be possible for one of you to fix that little css and update it on www ?
/* inside /styles/material-inspired.css*/
#area {
        position: relative;
        background: transparent url(../images/back_dim.png);
        width: 3000px;
        height: 3000px;
        opacity: 0.7;
 }
/* to*/
#area {
        position: relative;
        background: transparent url(../images/back_dim.png);
        width: 100%;
        height: 100%;
        opacity: 0.7;
 }

and
/* inside /styles/original.css*/
#area {
        position: relative;
        background: transparent url(../images/back.png);
        width: 3000px;
        height: 3000px;
 }
/* to*/
 #area {
        position: relative;
        background: transparent url(../images/back.png);
        width: 100%;
        height: 100%;
 }

Hi @v2lrf , sorry for the delay.
What exactly is your goal here? I tried adjusting the size as suggested and it broke the app in an expected way: the #area is now smaller, so when you scroll around (you can, because the SVG canvas is still 3000x3000), you have ugly white background (and not the proper background raster).
It might be better to describe a problem you are encountering, instead of suggesting a way to fix it. I believe we would be able to figure something out, provided we have a proper understanding of the issue you ran into.


Migrate master branch to ROS Jazzy

Change master branch to use ROS 2 Jazzy, ROS 2 Humble support moves to the "humble" branch.

Change master branch to use ROS 2 Jazzy, ROS 2 Humble support moves to the "humble" branch.

To make this transition seemless, this PR should be merge only after the sources has been updated in the rosdistro repo (I will do it, but humble branches would be necessary for the two kroshu upstream repos as well)

okay, I will create a humble branch for the upstream repos too

I created the humble branches for the upstream repos and also created a PR for adjusting the upstream.repos file on the humble branch in this repo to use the humble branch of the upstream repos. https://github.com/kroshu/kuka_drivers/pull/176
Still, updating the upstream repos for jazzy is needed before merging the PR.


chore: use @typescript-eslint/prefer-optional-chain

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
594

Relevant Lines:
594

💛  - Coveralls


fix: actually resolve the ref in resolvePath

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSDoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
ACTUALLY resolve the path parameter refs.
Turns out I missed the whole point in #39. I guess that's what I get for rushing a push on a Friday afternoon.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Shouldn't be necessary with the getter syntax:
pathParams: () => ({
  id: id.value
})

But I trust you this change is necessary in your use-case. Accepted.


Documentation for code authors

Include advice for whether you should start adding these annotations to your code.
Specific documentation for:

Authors whose code has no preexisting nullness annotations
Authors whose code has other nullness annotations whose semantics are not enforced by any tool
Authors whose code has other nullness annotations whose semantics are enforced by specific tools, including some subset of:

FindBugs/SpotBugs
NullAway
Checker Framework
IntelliJ

May include updating https://github.com/jspecify/jspecify/wiki/adoption, or replacing that with one or more documents.

One thing to address here is the idea of "sprinkling" annotations into code that is not @NullMarked. This can apply for code that already has non-JSpecify annotations sprinkled in it, or sprinkling can be a first step for a project that currently has no annotations.
Background: We got some feedback that the current User Guide suggests that @Nullable is valid only within @NullMarked code. This may have been based on the phrasing:

JSpecify includes a @NullMarked annotation. In code covered by that annotation, String x means "x is a reference to a string object", and @Nullable String x means "x is either null or a reference to a string object"

This is written in the docs-1.0 branch, which we'll merge into main soon.


Add ServletContextListener to send an event when the applet is shutting down

Ended up implementing as overridden destroy method on httpcontext.


HA core 2024.1.0b0 update some errors

The problem
Hi, running 0.3.12 and updated HA Core 2024.1.0b0 and there were some warnings in the log for this integration:
2023-12-27 15:25:56.309 WARNING (MainThread) [homeassistant.const] TEMP_CELSIUS was used from nest_protect, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please create a bug report at https://github.com/imicknl/ha-nest-protect/issues

2023-12-27 15:25:56.309 WARNING (MainThread) [homeassistant.const] TEMP_CELSIUS was used from nest_protect, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please create a bug report at https://github.com/imicknl/ha-nest-protect/issues

What version of this integration (ha-nest-protect) has the issue?
0.3.12
What version of Home Assistant Core has the issue?
2024.1.0
Device / Model
Nest Protect Battery
Diagnostics information
No response
Home Assistant log
Logs
```
Copy/paste any log here, between the starting and ending backticks (`)
```

Additional information
No response

This error is there on the HA update 2024.1.0. Last working HA version without this error was: 2023.12.5

Thank, we will fix this. This is a warning by the way, not an error.

@iMicknl any chance we can get this merged?

Fixed in v0.4.0b5


fix(docs): 404 pages second iteration

Description
Fixing more 404 Errors. Because it is a joy.

Finally, that's it.


Adding user expired error

The purpose of this PR is to make a specific error for "User account has expired" available in the go client and thus in the CLI.

outdated


Problem running Tutorial1/Example7

Hello,
I have downloaded your DACE package yesterday and successfully compiled it on Ubuntu 22.04.
The tutorial examples compiled as well (except Example11DARK78.cpp which is ignored), but when running Tutorial1/Example7, I get the following message:
`Evaluate sombrero function
terminate called after throwing an instance of 'DACE::DACEException'
what():  daceRoot: Zero-th root does not exist (ID: 645)
Aborted (core dumped)
`
I am wondering if this could be linked to the warning message appearing when compiling the whole package:
[ 96%] Built target c_core_contrib .../dace/interfaces/cxx/DA.cpp: In static member function ‘static void DACE::DA::init(unsigned int, unsigned int)’: .../dace/interfaces/cxx/DA.cpp:69:27: warning: catching polymorphic type ‘class DACE::DACEException’ by value [-Wcatch-value=]    69 |     } catch(DACEException ex) {       |                           ^~ .../dace/interfaces/cxx/DA.cpp: In static member function ‘static void DACE::DA::init(unsigned int, unsigned int)’: .../dace/interfaces/cxx/DA.cpp:69:27: warning: catching polymorphic type ‘class DACE::DACEException’ by value [-Wcatch-value=]    69 |     } catch(DACEException ex) {       |                           ^~ [ 96%] Built target c_core
All other executables  run w/o error messages.
Thanks a lot for any help,
Gilles

Hi Gilles,
this is not a compilation error, it is the expected behavior of example 7. Mathematically the Sombrero function is not defined in (0,0). Therefore, when you try to evaluate it the DACE library rises an exception, which in the very simple code of the example 7 is not managed. Try adding a constant to one of the element of x and the code will correctly evaluate the sombrero function.
Closing the issue as this is not an error in the code.
Mauro.


Use a variable for data source

Hello,
I found your dashboards while looking for Grafana dashboards for graftorio2. I'm trying to adapt them to my Grafana cloud instance, but unfortunately there is a hardcoded datasource ID (see here for example). This data source doesn't work in my Grafana instance.
Are you able to turn this instance into a variable, so we can choose the data source?

Thanks for this hint.
will look into that!

hey @pgrunm,
I moved my dashboards to a separate repository

https://github.com/Kariton/graftorio2-dashboards.git

I removed the hardcoded datasource ID and the dashboards are still working as intended.
hope this fixes your problem.

Thank you very much, I'm going to try this out! :)


Getting an error when adding Windows Node

In mixed deployment mode when we try to add a Windows Node it proceeds and tests the Window Node correctly but after that doesn't proceed further and times out after a certain time.
We tried this multiple times and on different PC's and got the same result. We are using K3S deployment

Hi @miteshkumarmitra,
I am closing this issue due to inactivity. Please feel free to reopen by sharing the issue-required information.
Thanks,
Francisco


commiting to my pledge

@KaranSantra That's nice! please also revert package-lock.json

@Abhishek-kumar09  reverted package-lock.json

Btw can you give me suggestion on how to improve my technical communication skills (like for slack,github etc)

Btw can you give me suggestion on how to improve my technical communication skills (like for slack,github etc)
@KaranSantra You are already good at it.
Generally, Just try to be polite, because everyone in the world is once in their learning phase and can do some mistakes that looks stupid to you or etc, but remember that you had also done the same. Else experience will tell you everything. ...and ask anything smallest to biggest things, but that should be to the point.
For technical things google first before personally asking, because the person you are asking will also google the same thing (in case he is unsure about their answer) and this way you will be wasting everyones time and efforts.

@Abhishek-kumar09 Thank you for your reply. This was my first pr and I have learnt so many new things already. Enjoying this journey so far :)


Dockerfile deployment Issues

Trying to use the Dockerfile and get the following error:
 ---> Running in 8bb1dcf92b93
/usr/local/lib/python3.9/site-packages/huggingface_hub/file_download.py:1132: FutureWarning: `resume_download` is deprecated and will be removed in version 1.0.0. Downloads always resume when possible. If you want to force a new download, use `force_download=True`.
  warnings.warn(
[nltk_data] Downloading package averaged_perceptron_tagger to
[nltk_data]     /root/nltk_data...
[nltk_data]   Unzipping taggers/averaged_perceptron_tagger.zip.
[nltk_data] Downloading package cmudict to /root/nltk_data...
[nltk_data]   Unzipping corpora/cmudict.zip.
Traceback (most recent call last):
  File "/app/melo/init_downloads.py", line 5, in <module>
    from melo.api import TTS
  File "/app/melo/api.py", line 18, in <module>
    from .download_utils import load_or_download_config, load_or_download_model
  File "/app/melo/download_utils.py", line 4, in <module>
    from cached_path import cached_path
  File "/usr/local/lib/python3.9/site-packages/cached_path/__init__.py", line 13, in <module>
    from ._cached_path import cached_path
  File "/usr/local/lib/python3.9/site-packages/cached_path/_cached_path.py", line 15, in <module>
    from .schemes import (
  File "/usr/local/lib/python3.9/site-packages/cached_path/schemes/__init__.py", line 6, in <module>
    from .r2 import R2Client
  File "/usr/local/lib/python3.9/site-packages/cached_path/schemes/r2.py", line 18, in <module>
    class R2Client(SchemeClient):
  File "/usr/local/lib/python3.9/site-packages/cached_path/schemes/r2.py", line 20, in R2Client
    botocore.exceptions.HTTPClientError,
AttributeError: module 'botocore.exceptions' has no attribute 'HTTPClientError'

pls refer to https://github.com/myshell-ai/MeloTTS/pull/124

Creating a Dockerfile for MeloTTS, see https://github.com/Meiye-lj/Dockerfiles/blob/76c88309a4bb7b7070441bed3b4b72231f5349b8/MeloTTS/Dockerfile


lost about 32% of data during writing redis

I tried lettuce on spark cluster, But it lost about 32% of data during writing redis.
Here is my code:
.......
spark.read.load(filePath).repartition(Conf.rePartNum).foreachPartition((it :Iterator[Row])=>{
      val mixColids=123456
      val redisConn=new RedisConn(Conf.redisHosts)
      val async=redisConn.async
      it.foreach(r=>{
        val userId=r.get(0).toString
        val userColsKey=keyPrefix + userId //
        val recommendations: Seq[(Int, List[(Int, Int, Float, String)])] =r.getAs[Seq[Row]]("recommendations").map({
          case Row(categoryid:Int,recSeq:Seq[(Int,Int,Float,String)]) => {
            val genreid=categoryid
            val recSong=recSeq.asInstanceOf[Seq[Row]].map({ // resRank,itemid,gbdtScore,engine
              case Row(r:Int,i:Int,g:Float,e:String) => (r,i,g,e)
            }).toList
            (categoryid, recSong)
          }
        })
        if(recommendations.nonEmpty ){
          var indexCount=0
          recommendations.foreach(v=>{ //
            val colSongsKey=keyPrefix + userId+"_"+ mixColids(indexCount) //
            indexCount=indexCount+1
            val recSongs: Seq[(Int, Int, Float, String)] =v._2.sortBy(_._1)
            var recommendList:Array[String]=Array()
            recSongs.foreach(s=>{ //
              val recJson = new JSONObject()
              val itemId=s._2
              val pop=s._3
              recJson.put("itemid", s._2)
              recJson.put("pui", s._3)
              recommendList = recommendList :+ recJson.toString
            })

		  async.del(colSongsKey)
		  async.rpush(colSongsKey, recommendList:_*)
		  async.expire(colSongsKey,expireSecond)
          })
          async.set(userColsKey, mixColids.take(indexCount).mkString(","))
          async.expire(userColsKey,expireSecond)
        }
      })
      redisConn.flushClose()
    })
.......
	def flushClose(): Unit ={
		async.flushCommands();
		**Thread.sleep(1*1000) // If I do not sleep here, I will lost about 32% data.**
		conn.close()
		clusterClient.shutdown();
	}
........

I have 249067 keys, but only 169590 keys were  written successfully. And I tried sleep 1 second before close redis connect, my data were all written successfully.
Thank you!

flushCommands should not be used in highly asynchronous arrangements as the synchronization can easily lead to command loss.

flushCommands should not be used in highly asynchronous arrangements as the synchronization can easily lead to command loss.

Thank you for your Reply! but I tried without 'flushCommands()' just now, the result was the same as before.


Transparent icon backgrounds are no longer allowed on Google Play

https://developer.android.com/google-play/resources/icon-design-specifications

No warning on latest promote of a version so closing this one.


Import WIP to menus template

Describe your changes
Import the WorkInProgress Component into the menus template and render the component instead of the current div and placeholder section.
Screenshots - If Any (Optional)
Link to issue
Closes #105
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] Followed the repository's Contributing Guidelines.
[x] I ran the app and tested it locally to verify that it works as expected.
[x] I have checked my code with an automatic accessibility tool such as Axe Dev Tools or Wave
and it shows no errors.

Additional Information (Optional)

Thank you so much for your contribution!


The compiler reports an error during making test_ hello_ world_test

Error reported when compiling helloworld
“error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]”
what should I do？And there’s another question here. How do I solve it
‘tensorflow/lite/micro/tools/gen_micro_mutable_op_resolver_test/person_detect’: No such file or directory
_
g++ -std=c++11 -fno-rtti -fno-exceptions -fno-threadsafe-statics -Wnon-virtual-dtor -Werror -fno-unwind-tables -ffunction-sections -fdata-sections -fmessage-length=0 -DTF_LITE_STATIC_MEMORY -DTF_LITE_DISABLE_X86_NEON -Wsign-compare -Wdouble-promotion -Wshadow -Wunused-variable -Wunused-function -Wswitch -Wvla -Wall -Wextra -Wmissing-field-initializers -Wstrict-aliasing -Wno-unused-parameter  -DTF_LITE_USE_CTIME -Os -I. -Itensorflow/lite/micro/tools/make/downloads/gemmlowp -Itensorflow/lite/micro/tools/make/downloads/flatbuffers/include -Itensorflow/lite/micro/tools/make/downloads/ruy -Igen/linux_x86_64_default/genfiles/ -Igen/linux_x86_64_default/genfiles/ -Itensorflow/lite/micro/tools/make/downloads/kissfft -c gen/linux_x86_64_default/genfiles/tensorflow/lite/micro/examples/hello_world/hello_world_model_data.cc -o gen/linux_x86_64_default/obj/core/gen/linux_x86_64_default/genfiles/tensorflow/lite/micro/examples/hello_world/hello_world_model_data.o
g++ -std=c++11 -fno-rtti -fno-exceptions -fno-threadsafe-statics -Wnon-virtual-dtor -Werror -fno-unwind-tables -ffunction-sections -fdata-sections -fmessage-length=0 -DTF_LITE_STATIC_MEMORY -DTF_LITE_DISABLE_X86_NEON -Wsign-compare -Wdouble-promotion -Wshadow -Wunused-variable -Wunused-function -Wswitch -Wvla -Wall -Wextra -Wmissing-field-initializers -Wstrict-aliasing -Wno-unused-parameter  -DTF_LITE_USE_CTIME -O2 -I. -Itensorflow/lite/micro/tools/make/downloads/gemmlowp -Itensorflow/lite/micro/tools/make/downloads/flatbuffers/include -Itensorflow/lite/micro/tools/make/downloads/ruy -Igen/linux_x86_64_default/genfiles/ -Igen/linux_x86_64_default/genfiles/ -Itensorflow/lite/micro/tools/make/downloads/kissfft -c tensorflow/lite/micro/kernels/fully_connected.cc -o gen/linux_x86_64_default/obj/kernels/tensorflow/lite/micro/kernels/fully_connected.o
tensorflow/lite/micro/kernels/fully_connected.cc: In function ‘void tflite::reference_integer_ops::FullyConnected(const tflite::FullyConnectedParams&, const tflite::RuntimeShape&, const int16_t*, const tflite::RuntimeShape&, const int8_t*, const tflite::RuntimeShape&, const AccumScalar*, const tflite::RuntimeShape&, int16_t*) [with AccumScalar = long int; int16_t = short int; int8_t = signed char]’:
tensorflow/lite/micro/kernels/fully_connected.cc:192:1: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
}  // namespace tflite
^
tensorflow/lite/micro/kernels/fully_connected.cc: In function ‘void tflite::reference_integer_ops::FullyConnected(const tflite::FullyConnectedParams&, const tflite::RuntimeShape&, const int8_t*, const tflite::RuntimeShape&, const int8_t*, const tflite::RuntimeShape&, const int32_t*, const tflite::RuntimeShape&, int8_t*)’:
tensorflow/lite/micro/kernels/fully_connected.cc:192:1: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
In file included from ./tensorflow/lite/kernels/internal/types.h:24:0,
from ./tensorflow/lite/micro/kernels/fully_connected.h:22,
from tensorflow/lite/micro/kernels/fully_connected.cc:16:
./tensorflow/lite/kernels/internal/runtime_shape.h: In function ‘TfLiteStatus tflite::{anonymous}::Eval(TfLiteContext*, TfLiteNode*)’:
./tensorflow/lite/kernels/internal/runtime_shape.h:61:31: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
TFLITE_DCHECK_LT(i, size_);
^
./tensorflow/lite/kernels/internal/runtime_shape.h:61:31: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
TFLITE_DCHECK_LT(i, size_);
^
./tensorflow/lite/kernels/internal/runtime_shape.h:61:31: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
TFLITE_DCHECK_LT(i, size_);
^
tensorflow/lite/micro/kernels/fully_connected.cc:79:14: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
TfLiteStatus Eval(TfLiteContext* context, TfLiteNode* node) {
^
tensorflow/lite/micro/kernels/fully_connected.cc:79:14: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
In file included from ./tensorflow/lite/kernels/internal/types.h:24:0,
from ./tensorflow/lite/micro/kernels/fully_connected.h:22,
from tensorflow/lite/micro/kernels/fully_connected.cc:16:
./tensorflow/lite/kernels/internal/runtime_shape.h:61:31: error: assuming signed overflow does not occur when assuming that (X - c) > X is always false [-Werror=strict-overflow]
TFLITE_DCHECK_LT(i, size_);
^
cc1plus: all warnings being treated as errors
tensorflow/lite/micro/tools/make/Makefile:686: recipe for target 'gen/linux_x86_64_default/obj/kernels/tensorflow/lite/micro/kernels/fully_connected.o' failed
make: *** [gen/linux_x86_64_default/obj/kernels/tensorflow/lite/micro/kernels/fully_connected.o] Error 1
`

@TomMichey: Your second question is related to the problem in this currently unsolved issue #1644.
‘tensorflow/lite/micro/tools/gen_micro_mutable_op_resolver_test/person_detect’: No such file or directory

@TomMichey: Your second question is related to the problem in this currently unsolved issue #1644.
‘tensorflow/lite/micro/tools/gen_micro_mutable_op_resolver_test/person_detect’: No such file or directory

Although this warning "person_detect" is not related to my "helloworld", I want to know whether you can run your program after skipping this warning. thank you

Although this warning "person_detect" is not related to my "helloworld", I want to know whether you can run your program after skipping this warning. thank you！

help

Hi @TomMichey, on which gcc version and platform are you seeing this? I'm unable to reproduce it with the two latest gcc lines, 10.4.0 or 12.2.0, on x86_64 Debian.

Hi @TomMichey, on which gcc version and platform are you seeing this? I'm unable to reproduce it with the two latest gcc lines, 10.4.0 or 12.2.0, on x86_64 Debian.
It was “gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.12)”. I failed to compile.


Fix use of undeclared variable in rmodels.

Broken by de173a93c86fbda92b7d93a5fc202e80d4a16f24.

@rcorre Thanks! Just send a fix before seing this PR!


Issues need labels, milestones and assignments

I added the labels and assignments. The current milestones are:

Requirements
Initial Plan
Run 1
Analysis
Someday

I think it might help to have more granularity/clarity in this. We could make it

Requirements
Prototype generation, processing
Final generation, processing
Analysis
Someday

or break it down by component:

Prototype 1.0 Data Generation
Prototype 1.0 Data Processing
Prototype 1.0 light curve extraction and analysis 1.0
Final 1.0 Data generation
Final 1.0 Data Processing
Final 1.0 light curve extraction and analysis 1.0
Someday

I think I want fewer milestones rather than more (so that we don't have to so much changing later ;-) ) We have a deadline for the RQ document from the RoadMap, and I'd suggest we make just two others, one for Run 1 data generation (ImSim) and another for Run 1 data processing (DM). Maybe we could complete these, and then meet (at SLAC in March?) to design, issue and milestone the final Twinkles 1 DG and DP runs?
Then there'll be other milestones that crop up as required - like today's ANL2015 Hack Day one :-) What do you think?

What do we plan to have done by the March meeting?

Hi @DarkEnergyScienceCollaboration/twinkles ,
I'm doing some issue management, as you have seen. Initial data generation experiments are going very well and I am confident of getting an useful requirements document (initial draft) done by Christmas. It's great to see :-)
Looking ahead to the New Year, and inspired by Michael's question above, I suggest we define "Run 1" as the dataset that we generate and analyze before the collaboration meeting, so that we can present our progress to the group for feedback. What do you think? To see what this looks like, check out our new-look milestone page :-)
I don't think we need to define what the Run 1 dataset need look like yet: let's just keep developing and see how things look around mid-January. If we aim to have a reasonable Run 1 dataset in place by the end of Jan, we can then spend a month focusing on its analysis (both DM and then DESC post-processing). We'll need to do some early science analysis at that point, which will be fun :-)
Comments welcome!

I think that having "Run 1" generated and analyzed by March is a good and feasible goal.  However, I would like to see the document describing what goes into Run 1 updated with our latest thinking on all of the ingredients (i.e., enigma_1189 instead of OpSim 3.61, 6 filters instead of just r and g for Twinkles 1.0 as in the SRM) so that the workflows group has a definite set of goals/requirements to shoot for.  I realized these are moving targets, and will evolve, but it would be good to keep them updated as our thinking changes so everyone knows where we stand.

I agree: this must be kept current. Proposing changes to the Run 1 design
means proposing edits to the Run1 document.
On Fri, Dec 4, 2015 at 5:01 PM, James Chiang<EMAIL_ADDRESS>wrote:

I think that having "Run 1" generated and analyzed by March is a good and
feasible goal. However, I would like to see the document
https://github.com/DarkEnergyScienceCollaboration/Twinkles/blob/master/doc/Run1.md
describing what goes into Run 1 updated with our latest thinking on all of
the ingredients (i.e., enigma_1189 instead of OpSim 3.61, 6 filters
instead of just r and g for Twinkles 1.0 as in the SRM) so that the
workflows group has a definite set of goals/requirements to shoot for. I
realized these are moving targets, and will evolve, but it would be good to
keep them updated as our thinking changes so everyone knows where we stand.
—
Reply to this email directly or view it on GitHub
https://github.com/DarkEnergyScienceCollaboration/Twinkles/issues/44#issuecomment-162120737
.


Error correction level

Hi,
What is the error correction level used for the generated qr codes?
Is there a way to specify it?
Thanks,
:panos

The default error correction is ErrorCorrection.Low, you may set the errorCorrection on your QRCode instance.


gitignored symlink is followed and indexed

In my project, I have a gitignored location which is a symlink to a folder outside of the project directory. Yet, this symlink is followed and indexed.
I would expect the ignored symlink to not be indexed.
Thanks!

Reproduced on 1.4.0. This causes certain versions of Meteor projects, for instance, which symlink a local .meteor/dev_bundle folder to a snapshot of a Meteor build that contains 10k+ files, to bog down on search. Screenshots:
https://www.dropbox.com/s/2pmezw3pn5n78cv/Screenshot 2016-12-06 17.17.49.png?dl=0
https://www.dropbox.com/s/zx87jhmulfavz61/Screenshot 2016-12-06 17.24.03.png?dl=0

As a workaround, though, core.followSymlinks can be set to false; this is a blunt instrument turns off following symlinks entirely, even those which aren't gitignored, but may do the trick.
I'm actually thinking this is more of a problem in core, not in this package. If one of the maintainers could confirm and perhaps point me in the right direction, I'd be happy to file the bug there.


Open zh PDF file show that unreadable text.

(you don't have to strictly follow this form)
How to reproduce
open flowing link:
https://clickhouse.yandex/docs/zh/single/clickhouse_zh.pdf
Then displays unreadable text, not chinese text.

it is ready.

Yes, fixed a while ago, but forgot to close in time.


(Raise in Ubuntu, but not raise in Mac) VideoUnavailable: This video is unavailable.

When attempting to run pytube in the droplet of DigitalOcean (Ubuntu 18.04.3), VideoUnavailable was raised for most of the videos. However, there was no problem at all when it was run on my Mac.
For example,
https://youtube.com/watch?v=0zGcUoRlhmw
this url input raise the exception in the droplet, but fine in MacOS.
Interestingly,
https://youtube.com/watch?v=Lg0SBpR2NGE
this url input works in both the droplet and MacOS.
Does anyone have ideas of what is going on?

Solving the issue with changing the region from Singapore to San Francisco.


Proposed changes to test suite

Hi cookiecutters,
I noticed that the test suite in the project is using the unittest style tests for pytest (setup, teardown, etc.). I think these tests should be changed to use pytest fixtures and other pytest style tests. It also looks like the test function for click's CliRunner is written for both pytest and unittest, I propose just having one function in order to avoid duplication. Another thing I noticed is that the test file ends with if dunder name == dunder main. I propose that this be removed since unittest is discoverable with python -m unittest discover. One last thing, which I think can be debated, the tests directory is in the project slug's root directory. I propose moving this directory inside the package directory. Please let me know what you think and I will be happy to submit a pull request for any of the approved changes.
Proposed changes:

Convert pytest tests from unittest style to pytest style
Remove duplication of the cli test function (CliRunner)
Remove if dunder name == dunder main since unittest is discoverable
Move tests directory inside package directory

Hm, I'd rather keep the tests directory where it is.
What's your motivation for moving them to inside the package?
The other changes sound okay to me, though I wonder if removing dunder name == dunder main could break python setup.py test -- that's something to check.
In any case, the best to way to move forward is to submit a PR and we can discuss it there. ;-)

Hi @eliasdorneles, since the tests are testing the CLI application that is in the package, it makes more sense to me to move the directory inside the package directory. I'll submit a PR and we can go from there.
Thanks!

Well, both layouts are common practices and while I understand some people might like it different, I see no point on changing the current one -- which is valid, has been tested, and users are already familiar with it.

Great points, I'll leave it be and focus on the other changes.

Closed by #255

This article details the various Python directory layouts:  https://blog.ionelmc.ro/2014/05/25/python-packaging/


Disable RichHTML Without-Bounce Test

Due to a known swipe resistance detection bug, the unit tests of testComponentsOnLocallyLoadedRichHTMLWithoutBounce and testLongTableOnLocallyLoadedRichHTMLWithoutBounce are temporarily disabled.

I don't see these unit tests anywhere in EarlGrey. Did you report this issue to the wrong repository?

sorry for the late reply. They are in my latest PR:
https://github.com/google/EarlGrey/pull/178

Disabled in https://github.com/google/EarlGrey/pull/179


Don't send no-op updates to the dashboard

Avoids sending a resource update that has no actual changes compared to the last value sent.
This can happen because of how we process services/endpoints. When they are updated, we attempt to apply that updated data to existing DCP containers/executables. Sometimes this results in no actual difference, so we can drop these updates and avoid sending them over the wite and triggering UI refreshes.
As an example, when loading dashboard for the eShopLite sample solution, I see up to six redundant updates that are now avoided.
Microsoft Reviewers: Open in CodeFlow

@JamesNK is there anything else you wanted on this PR?

Test failure in Aspire.Azure.Storage.Queues.Tests.ConformanceTests.TracingEnablesTheRightActivitySource, which is unrelated to this PR as far as I can tell. Re-running CI.

/azp run

@drewnoakes are you still planning on merging this?

@adamint In yesterday's meeting we decided to hold off on this for now - due to the maintenance overhead of keep the equals check up to date as new properties are addded. This may need to wait till we find an optimal way to incorporate tests for this.


Infinite Loader

🙋 Feature Request
Would it be possible to use this in an infinite loader setting? This would mean the following:

Should display rows from a streaming API (think WebSocket) where new rows would be added on top.
Manages the DOM by controlling the number of rows displayed on the device. More specifically, this would include add/removing row elements from the DOM at both the top and bottom as the user scrolls
Efficiently manage memory

Collection components support dynamic data, asynchronous loading, and infinite scrolling. Lists are virtualized, which should satisfy your efficient memory management item.

Thanks for the response @reidbarber
Do you have an example that we can test?
I've looked at a lot of these types of packages, and the authors always make bold claims, but most of these things fail under close inspection.
Some of the bad ones include:
https://medium.com/@mitino/react-hook-for-rendering-large-lists-7761966aa93d
https://github.com/bvaughn/react-window
thanks!


checkParams() should run on defaults once sequence is chosen

The default values for visualizer parameters may not be valid in the context of a particular sequence.  This could be checked by checkParams(), but it appears that right now checkParams() is not run until the user touches the parameters.  For example, if a turtle visualizer or a difference visualizer wants to use the first N terms by defalut, but the sequence at hand has fewer than N terms (there are some sequences with very few terms in the OEIS!), such a check may fail.  If it isn't checked, an error is thrown later on in the code when we try to fetch too many terms.

Hmmm, I am confused. Scope.vue creates a visualization by calling Specimen.fromQuery. That calls validate() on first the sequence and then the visualizer. And validate() in turn calls checkParameters(). So I don't see how a validation can ever get on the screen without going through checkParameters(). Can you provide an explicit recipe for something to go wrong and an explanation of what exactly goes wrong? I tried Differences with all defaults (wants 20 terms)  on A371958 (only has 15 terms) and all seemed to go fine. Admittedly, Differences tolerates its parameter called "n" being larger than the number of entries in the sequence and just deals with it in draw(), so it doesn't seem like anything can really go "wrong" there, but maybe the issue you are identifying can cause real problems in another context?

Ok, well, I noticed this while working on the turtle-enhance branch, so it's always possible it's actually something in that branch.  For now, here's how to reproduce:

On turtle-enhance branch
Go to http://localhost:5173/?name=errordem&viz=Turtle&pathLength=10000&growth=1&seq=OEIS+A156660
Observe that this has 10000 terms (which the sequence allows)
change OEIS sequence to A371606 (for me I've used this sequence before, so it's in the sequence chooser)
now the turtle ends after 16 steps, but it shows that it is using 10000 terms.  If you put the mouse in the 10000 terms input box and try to type, the box will auto-correct to 16.

My essential observation is that it doesn't auto-correct to 16 upon loading the sequence, only upon trying to update that box.  I'll try to look into this more when I have time.

Will try to debug when I get the chance, but sounds like it could possibly be a missing refreshParams() call in one of the places the number of terms is being changed. Basically any time a visualizer changes a parameter itself it should immediately call refreshParams().

I am thinking this will be fixed by the fix for #403; we shall have to see after that lands.

@katestange same comments as for #403. Thanks for looking into the current status of this.

So we can play with this issue in Chaos, which has a field for the number of terms.

Load http://localhost:5173/?name=OEIS+A071642&viz=Chaos&first=1&last=3605&seq=OEIS+A071642
Switch sequence to A371958 (which has only 15 terms)
Observe that the field for number of terms has gone down to 15, which is good, but the URL still says 3605.

So the behaviour has changed in ui2, but is still kind of weird.  In many overhauled visualizers, they'll just use as many terms as available by default, so it won't even be in the url -- like for FactorFence.  In some, we may want to limit the number of terms artificially, in which case this issue may (?) return/remain.  Which brings up the question of the best way to handle such a situation (Turtle Enhance might want to limit the number of terms below the full number available, for example).

With the current handling of sequence bounds in ui2, none of these things are reproducible. Closing.


Singletons and Companion Object

Both Scala and Kotlin has the notion of singleton and companion objects. Perhaps you can consider adding them to Pony as these have been very valuable in these languages.

Pony already has primitives which I believe effectively serve the same purpose as singletons and companion objects.
Primitives are single instance types, ie every time you call create() on a given primitive type you actually get back the same object. Methods on a primitive type are thus effectively static / global. Primitives cannot have data fields because these are then global state which is bad for several reasons, including totally messing up capabilities.
If you are suggesting something that is not covered by primitives please be much more specific.

The purpose of the companion object is to have state and methods which are accessible from other objects to which is a companion to. This loosely like all static fields and methods pulled into an object of its own but it has an apply and unapply method which can be used use the class itself like a function. Generally it is use to create a class which is companion to. E.g. (pseudo syntax)
class Bat
class ABat ....
class BBat

object BatFactory
   fun apply[T :< Bat](implicit f : String => T): T = f(name)

Since a constructor is like a function taking arguments and returning a object assuming this can be implicitly matched.


[Event Request] table 5964 "Service Contract Line" - Event for trigger OnInsert()

Can we get the following event for the "Service Contract Line" table: OnBeforeValidateServContractHeader
trigger OnInsert()
    var
        ///
        IsHandled: Boolean; ///NEW
    ///-
    begin
        TestField(Description);
        GetServContractHeader;

        ///
        OnBeforeValidateServContractHeader(ServContractHeader, IsHandled); ///NEW
        if not IsHandled then begin ///NEW
            ///-

            ServContractHeader.TestField("Customer No.");
            ServContractHeader.TestField("Contract No.");
            ServContractHeader.TestField("Starting Date");
            if "Service Item No." <> '' then begin
                ServContractHeader.TestField("Service Period");
                ServContractHeader.TestField("First Service Date");
            end;
        end;

    [IntegrationEvent(true, false)]
    local procedure OnBeforeValidateServContractHeader(ServContractHeader: Record "Service Contract Header"; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

The events were implemented incompletely in the Update BC16.5. We need the sender:
[IntegrationEvent(true, false)]


API without ApiResource?

Hey,
Do I need to define ApiResource if I making only one API server which is protected by IDS4? Can I just use IdentityResources only?
The API server is also auth server which integrates IDS4. So like ALL-IN-ONE solution :)
Regards,
TheAifam5

Sorry, my bad. I think I misunderstood the definition of "ApiResource". :)


revert: "fix(executor): resource patch for non-json patches regression…

… (#5951)"
This reverts commit 80f8473a13482387b9f54f9288f4a982a210cdea.
Signed-off-by: Alex Collins<EMAIL_ADDRESS>

So it was not that PR then.


Create a way to add C++ attributes to generated classes or functions

It would be useful to me to be able to add the [[nodiscard]] attribute to certain generated types, like for Result-like types. Could we add a new Rust attribute that will cause the C++ codegen side to add the given attribute to a type or function declaration, for example #[cxx_attribute="[[nodiscard]]"]?
One caveat to a general "apply any C++ attribute" Rust attribute is that I'm uncertain whether any C++ attributes exist that might change the behavior of a type or function in ways that would violate CXX's expectations.

I'll close this since I don't think I would want to do an arbitrary-purpose "paste this blob into the generated file" which is what #[cxx_attribute="[[nodiscard]]"] would imply.
I filed #930 to propose specifically doing #[must_use]⟶[[nodiscard]] and #[must_use = "..."]⟶[[nodiscard("...")]].


Biometric manager

add Biometric Manager
needs additional Plug-In : git@github.com:Loxone/cordova-plugin-android-fingerprint-auth.git @

@LoxoneSeb why you close PR?


Reuse the unwatch function for the build tasks

Depends on https://github.com/mozilla/tofino/pull/649
Only 6b8417e is relevant.
@Mossop

Coverage remained the same at 42.868% when pulling 6b8417ee4aa10aad85b8f83a06bb9f343e0950c1 on victorporof:reuse-unwatch into 0b453f03769f9bdaf5293bd7f8e84c7436c26d88 on mozilla:master.

Coverage remained the same at 42.836% when pulling ed69e07bc63b319682929b24770c3093e9022458 on victorporof:reuse-unwatch into b64557aa649f129f42fc9ff012feb9fa67924c86 on mozilla:master.

Coverage remained the same at 42.836% when pulling ed69e07bc63b319682929b24770c3093e9022458 on victorporof:reuse-unwatch into b64557aa649f129f42fc9ff012feb9fa67924c86 on mozilla:master.

Coverage decreased (-0.06%) to 42.836% when pulling a655ca64a7d51d45e551a69589e51a9d7fe7c098 on victorporof:reuse-unwatch into 15ead0436cd8d9911f49fd76045ace42a88e5ff7 on mozilla:master.

Coverage remained the same at 42.899% when pulling a655ca64a7d51d45e551a69589e51a9d7fe7c098 on victorporof:reuse-unwatch into 15ead0436cd8d9911f49fd76045ace42a88e5ff7 on mozilla:master.

Coverage remained the same at 42.821% when pulling 68c198c9871881c35961da2bccb1f3e178e27d37 on victorporof:reuse-unwatch into 65a84d3bc40aea6a33f25e01e12651409c8733d4 on mozilla:master.

@Mossop gentle review ping.


camelCase and snake_case in JSON

It should be possible to configure if the JSON output is in camel or in snake_case.
We already discussed this in #60

we have to use dashes now #91 closing this ticket


No way to delete image from comment via edit form

m.686623

Comment edit form doesn't allow to edit attachments. As you can see on the screen the attached image is out of the edit form (under Post button). It's displayed in attachment section of the comment itself.


Improving unit tests

Hi,
I'd like to suggest a few things to improve the unit tests:

we need to have unit test which do not use the datasets... you need a procedure that directly tests the functions; also, you need to have one test for each type of error!

we also need to keep the tests on the datasets! and there, we need to improve the datasets to improve the tests:

we need one fully valid dataset (maybe several ones with different characteristics)
we need to construct datasets that deviates from the valid dataset but violate one and only one spec (one dataset for each type of error)
we need one or several datasets that violate several types of errors

@Slowblitz question: are all the datasets included in the repo really useful for the current version, or are the old ones useless now? (for ex because the specs have evolved)

@Slowblitz have you updated the datasets? (I forgot...)

No not yet , added to TODO list at https://github.com/INT-NIT/AnDOChecker/projects/1

ok!
then, for the datasets, I'm not sure you need to have both a directory like ds001, ds002 etc., and a subdirectory data inside each of them... having just ds002/exp-Landing would be enough (and better); and maybe renaming ds by dataset would be worth it also...
at the end, "dataset002/exp-Landing"? WDYT?

Yes seems good to delete the "data"  directory level of each dataset  and renaming 'ds001' by 'dataset001' seems a good idea too.

[ ] Deleting all "data" subdir for each dataset

[ ] update "test_isAnDO.py"

Hi @Slowblitz ! I'm coming back to this issue now ;)

It seems that in the latest version, none of the datasets available for testing are actually valid AnDO datasets... Could you please add one? (one that is not trivial, e.g that maybe has several subjects and several sessions per subject)

Also, maybe it would be better to have one non-AnDO dataset for each type of error? (because for now, the datasets 1-10 seem to test very different things, that are mixed etc.)


VSCode SITL debug updates

Running combined gzserver & gzclient doesn't work anymore, so now we start gzserver standalone and you can optionally run the gui with the "gazebo client" task.

added gazebo VTOL tailsitter debug target
improved jmavsim debug target and helper tasks to work properly (build and launch directly instead of through the jmavsim_run.sh script)
improved output console handling for helper tasks (eg gazebo and jmavsim build and run, followed by cleanup after debug)

To launch the gazebo gui select "Tasks: Run Task" from the command palette, then select "gazebo client"

The other little improvement included here is convenient access to a serial terminal.
From Tasks: Run Task select miniterm.py.
If it's available on your system (included with pyserial) you'll see the terminal open with an option to select a port.


chore: point to linked events dev env

Description
Issues
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

Federation Router branch is deployed to platta: https://events-graphql-federation-hobbies-pr674.dev.hel.ninja :rocket::rocket::rocket:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr674.dev.hel.ninja :rocket::rocket::rocket:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr674.dev.hel.ninja :rocket::rocket::rocket:

Events-Helsinki branch is deployed to platta: https://tapahtumat-pr674.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-events-pr674.dev.hel.ninja :rocket::rocket::rocket:

Federation Router branch is deployed to platta: https://events-graphql-federation-sports-pr674.dev.hel.ninja :rocket::rocket::rocket:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr674.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://liikunta-pr674.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://tapahtumat-pr674.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://harrastukset-pr674.dev.hel.ninja :laughing::tada::tada::tada:


Fix getHelperFtn issue

- it needs due to https://github.com/dotnet/coreclr/pull/15003 change
- synchronization with the VM(CoreCLR) version

@dotnet/arm32-corert-contrib please review

@BredPet Thank you!


"FileBasedEventStorageCustomLocationTest" test not passed in Windows

The current "FileBasedEventStorageCustomLocationTest " unit test uses "base-context-custom-files.xml" resource file as its application context. In the resource file the default TEMP directory assumed to be /tmp which is not compatible with Windows.
To solve this, the corresponding property(filePath) setting in .xml file was removed and instead the temp directory was prepared in the test class.
This change passed all  tests: "mvn test"

Nice work @mahmoodm2, and thank you.


Digital Twin KIT Release 23.12 Miscelleaneous Acceptance Criteria

Release 23.12 Miscellaneous

[ ] Version under Review

Pull request for KIT version under review must be placed minimum one week prior to your reserved Quality-Gate timeslot (to enable formal reviews by relevant BOs, BDAs, DevSecOps System Team)
Prerequisite: IP approval by all companies is available!

Artifact Repository:

Eclipse Tractus-X repo of KIT

Contacts:

Responsible: PO
Review by:

T-X Committer / Project Lead
DevSecOps System Team

[ ] Approval of Acceptance Criteria

Product Manager / Business Owner confirms:

All acceptance criteria met (as defined in PI documentation). Relevant evidence is available.

Best Practice:

PO / PCA to obtain BO / BDM OR PM / SA approval prior to Gate review.

Artifact Repository:

JIRA solution Level

Contacts:

Responsible: PO/PCA
Consult: KIT-CONTACT1, KIT-CONTACT2
Review by: BO / BDM (from relevant Domain) OR PM / SA

[ ] Test and Approval of Developer Journey

recommended; not mandatory
Business Owner / Product Owner should conduct a case study with at least 1 provider (ideally outside) of the consortium to test the developer journey.

Best Practice:

PO / BO to obtain feedback from case studies prior to Gate review.

Artifact Repository:

Case Study

Contacts:

Responsible: PO/BDA
Consult: KIT-CONTACT1, KIT-CONTACT2
Review by: Case Study Team

[ ] Provide change log page in KIT

Each KIT team must provide a KIT-specific release note/change log for developers

Best Practice:

For good practice visit: keepachangelog

Artifact Repository:

Provide change log page in KIT - example

Contacts:

Responsible: PO/BDA
Review by: Release Management

Version under Review --> Kit for R23-12 shall have v1.2 as no breaking changes occurred but features got added
Approval of Acceptance Criteria --> @bosserf  - can you write a quick comment that you're happy with the DT-Kit to be released with R23-12?
Test and Approval of Developer Journey --> Optional, no explicit Developer journey in DT Kit
Provide change log page in KIT --> provided here

Approval of Acceptance Criteria: I agree with the DT-Kit to be released with R23-12

@danielmiehle @maximilianong kindly provide your approval.


Pad charts with an extra period for some series

Automatically detects which charts need padding rather than having it set on the charts themselves

Can navChart implement same kind of zoom function? Pan and zoom functionality may need to be split.

Took this, leaving this here for now.

Continued in #277


New version? 🤔

https://github.com/ninja-build/ninja/compare/v1.11.1...master shows 90 commits since last release which was year ago.
I think that it would be good to start prepare release new version.
Any plans to do that soon? 🤔

Unlikely, unless there's something I need personally. See https://groups.google.com/g/ninja-build/c/HqLF-JzsEdc/m/XMT8F760AQAJ

Half year later .. any update? 🤔
https://github.com/ninja-build/ninja/compare/v1.11.1...master shows now almost 150 commits since last release.


Move admin Checks to Admin Dashboard

Maybe we should move all popups for admins into the admin dashboard as a table with things to fix and links to how to fix them instead of leaving it a popup. We could then point to the dashboard and let people see that table instead.
Would clean up the popups for new installations and give a better way to see what we actually check. Good checks could be listed as well :-)

I can probably take this, I've got a decent sized list of things to add and will write up nice descriptions for them all.

That's great, thanks! I will check your PR when it's ready, maybe I have some additional ideas by then.
Was just a quick idea I had. It may be useful to add Wili pages for those helper links that we don't link across the wiki itself.

I am almost done with this, if you have any requests now is the time :+1:

Awesome! Was just thinking about this issue ;-)
I'd like to make it as easy as possible to extend the admin checks. Very much like the admin settings, which creates the settings page from an array, we may look into splitting up admin checks into functions/includes that can be used easily. To add more checks, each one would need a short name, description, help link and the actual function to run the check. If those can be included dynamically and automatically on the dashboard, we'd have a clean way to expand this in the future.

I did it using includes (pages/admin/checks) and moved all of the current checks into that format. Using a call from a class or creating a simple model/loading would probably clean it up a good bit, and I will add field validation too - would like your input. I would like to write some additional checks as well.

Excellent! I'd add more colors (warning yellow, critical red) and maybe one box for each check? Not sure how that would look like. It may become overloaded, so yours may be better. For variables, they should be put into a code kinda block to highlight it's  code (I think Bootstrap has something for that?). Last but not least, maybe supply an explanation what the check means without sending people to the Website?
Great work, that's exactly what I had in mind :-)

I will add error level ie notice/warning/omg why; colored for each type,
I can clean it up with a model/create new objects in each test, I was just worried about dealing with scoping. All of them will have longer descriptions, just haven't moved them in yet :-) Another hour or two of polish.

I don't think we need objects for those simple checks, just make sure they can be included easily would be good enough for me. Thanks a lot man!

How is this? Each test will have a similar description beneath it.
I realize it's not immediately obvious but the config value listed in the pre links to the wiki page for the value/section.

That's great! Love it.

That's all the current tests with extended descriptions. Open to input on the wording/etc, and new test ideas.

It's like you read my mind :-) Good stuff, wording is fine. If you commit a PR please update the CHANGELOG as well and point it against development. Once I get a chance I will test it out myself!

https://github.com/xisi/php-mpos/blob/29f879a9c1bdd77f69c33c2eea93b283bdaa4d73/include/pages/admin/setup.inc.php#L30-L36
I think it would be way easier to work with http://php.net/manual/de/function.glob.php
Returns only the files matching a pattern, no need to do file name parsing to find the proper files.

We should somehow indicate that a coin daemon needs to fully sync before the software is fully usable.

For the stratum/daemon checks maybe?

Good idea, usually that one won't work until fully synced.

Noted it in the daemon/stratum checks.

Will try it out sometime this weekend. Merged into development in case others are willing to test it.

Tested it out, the only change I can think of is placing the To disable these checks, set skip_config_tests to true in global.inc.php string at the footer of the box instead of in the header.

Merged into master (with already existing merged data that I didn't pay attention to). We have now made it 1.0.4.


Retry EOF errors by the spoofing client

This is for connection/network errors. For example, when a gateway is temporarily down, like in HA tests. In that case it can throw an error like this:
The service at http://kourier-gateway-h-a-hhqgngvs.example.com didn't return success: response: <nil> did not pass checks: Get "http://kourier-gateway-h-a-hhqgngvs.example.com": EOF

/assign @chizhg
Please, review.

/lgtm
/approve

@vagababov @chizhg thanks for reviews!


Voice message generation

Request flow seems to be as follows:
POST => https://discord.com/api/v9/channels/{CHANNEL_ID}/attachments
BODY:
{
  "files": [
    {
      "filename": "voice-message.ogg",
      "file_size": 8987,
      "id": "2"
    }
  ]
}

RESPONSE <=
{
  "attachments": [
    {
      "id": 2,
      "upload_url": "https://discord-attachments-uploads-prd.storage.googleapis.com/{UUID}/voice-message.ogg?upload_id={UPLOAD_ID}",
      "upload_filename": "{UUID}/voice-message.ogg"
    }
  ]
}

The upload_url is then used in a PUT call:
PUT => {upload_url} from before
content-type: audio/ogg
host: discord-attachments-uploads-prd.storage.googleapis.com
user-agent: Discord-Android/173023;RNA

The contents of the PUT is the ogg file from earlier
After that succeeds, it is followed up by this API call to send it out:
POST => https://discord.com/api/v9/channels/{CHANNEL_ID}/messages
BODY:
{
  "content": "",
  "channel_id": "{CHANNEL_ID}",
  "type": 0,
  "flags": 8192,
  "attachments": [
    {
      "id": "0",
      "filename": "voice-message.ogg",
      "uploaded_filename": "{URL PATH}",
      "duration_secs": 2.48,
      "waveform": "AFZJL0FIITA3MysmLjYgNz5KNiosKSA="
    }
  ],
  "nonce": "{NONCE}"
}

Not sure how waveform is generated, but the rest makes sense enough.

will this be included in the new bluecord update? I'm looking forward to it


update nuget v2 nuspecs to point to github instead of codeplex

@yishaigalatzer @emgarten can this be merged in? (and am I targeting the correct branch?)

@xavierdecoster it should go into the 2.9 branch. I'm not sure we want another build of that, or if we should modify the current 2.9 packages.

Will reopen against 2.9 branch. Metadata of latest releases on NuGet.org has been updated already.


Receiving Images Help

I'm having a hard time finding much documentation on this so I decided to just ask here. How can I have my bot see and download any images sent on my discord server?

this project's dead but the DiscordMessage object has a DiscordAttachment object inside of it which then has a URL property which should link directly to the image.
https://github.com/suicvne/DiscordSharp/blob/master/DiscordSharp/Objects/DiscordMessage.cs

One last Question, I have everything working except for sending images. Any Help?


[FEATURE] Add Cache instances details

Add cache instance pricing informations to the website.
It is a lot similar to the rds scarping approach.

There is still a bug in the ouput website, and I would like some help to get over it before merging :)

Hi @V0lantis thanks for this. What bug are you seeing? I can assist with this.

As you can see in the screenshot I join with this comment, the table showed are printing some columns that I don't want (EBS THROUGHPUT for example).
Also, the filtering and sorting feature are not working. I must admit being a little bit lazy on this part, because I don't know how to easily correct the bug. Could you try to fix it on your side?

I have the following error in my console :

Maybe, this would help you find out where the issue is. If you don't have time, Ill try to find some on my spare time, but not sure I will be able to...

@V0lantis I just opened a PR for this with the two fixes you called out. The console error was caused by the datatables default sort being initialized on a column that didn't exist in the cache pricing page. It's actually strange because it does not appear to me that any default sort order is being applied to the live site, much less sorting on linux price which is what the comment in default.js says it is doing. I'll investigate this later.


fix: increase timeout and use tearDownNodes

Problem
There are some tests that consistently fail during the CI at teardown (stopping nodes)
Solution
Added a bigger timeout and used the tearDownNodes function that better handles this job

Seems that it helps a little but doesn't fix the issue fully. Still 1/3 go waku jobs failed

Did you check the nwaku logs see if there is any indication of the slow shutting down?

Did you check the nwaku logs see if there is any indication of the slow shutting down?

No, I just saw that the default timeout of 2000 ms is hit and increasing it to 10000 helps.
However even with 10000 ms there's something wrong happening sometimes, because it fails during the test with empty payload
My guess is that is somehow related to the stop of the AsyncIterator because this is the only test that uses this function
cc @weboko

My guess is that is somehow related to the stop of the AsyncIterator because this is the only test that uses this function

Might be a bug in the AsyncIterator, need to investigate

@danisharora099 https://github.com/waku-org/js-waku/issues/1560

@danisharora099 #1560

My guess is that is somehow related to the stop of the AsyncIterator because this is the only test that uses this function

Might be a bug in the AsyncIterator, need to investigate

Had a 2nd look and turns out that stop was fired imediately after the message was send and that was the reason for the failures. Adding a small delay seems to fix the issue


Integrated terminal for Python Debugger

Type: Feature Request
When starting a debugging session, it will automatically start one terminal, which is not consistent with the terminal that has already been started and set with environment variables
Extension version: 2024.4.0
VS Code version: Code 1.88.1 (e170252f762678dec6ca2cc69aba1570769a5d39, 2024-04-10T17:41:02.734Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Remote OS version: Linux x64 5.15.0-76-generic

Thanks for the feature request! We are going to give the community 60 days from when this issue was created to provide 7 👍 upvotes on the opening comment to gauge general interest in this idea. If there's enough upvotes then we will consider this feature request in our future planning. If there's unfortunately not enough upvotes then we will close this issue.

dup of https://github.com/microsoft/vscode-python-debugger/issues/88


hostPort and hostIP config fails to create iptables rules/chains

Version:
k3s version v1.18.3+k3s1 (96653e8d) running on Ubuntu 20.04 (master and target agents are AMD64 and there is also a ARM64 agent but unused for this deployment)
K3s arguments:
export INSTALL_K3S_EXEC='server --tls-san <IP_ADDRESS>' so using default containerd and not docker.
Describe the bug
Deployed unifi-video-controller and tried to map 2 ports to the secondary network of an agent. The pod is deployed but I cannot see any KUBE-HOSTPORTS or KUBE-HP- prefix iptables chains on the agent (as of https://github.com/rancher/k3s/blob/master/vendor/k8s.io/kubernetes/pkg/kubelet/dockershim/network/hostport/hostport.go)
I can telnet to the port 6666 probably because the agent opens that port to keep it busy but I don't think the traffic is forwarded to the pod.
I only have the following rules related to KUBE:
root@nuc6:/var/log# iptables --list-rules | grep KUBE
-N KUBE-EXTERNAL-SERVICES
-N KUBE-FIREWALL
-N KUBE-FORWARD
-N KUBE-KUBELET-CANARY
-N KUBE-PROXY-CANARY
-N KUBE-SERVICES
-A INPUT -m conntrack --ctstate NEW -m comment --comment "kubernetes service portals" -j KUBE-SERVICES
-A INPUT -m conntrack --ctstate NEW -m comment --comment "kubernetes externally-visible service portals" -j KUBE-EXTERNAL-SERVICES
-A INPUT -j KUBE-FIREWALL
-A FORWARD -m comment --comment "kubernetes forwarding rules" -j KUBE-FORWARD
-A FORWARD -m conntrack --ctstate NEW -m comment --comment "kubernetes service portals" -j KUBE-SERVICES
-A OUTPUT -m conntrack --ctstate NEW -m comment --comment "kubernetes service portals" -j KUBE-SERVICES
-A OUTPUT -j KUBE-FIREWALL
-A KUBE-FIREWALL -m comment --comment "kubernetes firewall for dropping marked packets" -m mark --mark 0x8000/0x8000 -j DROP
-A KUBE-FORWARD -m conntrack --ctstate INVALID -j DROP
-A KUBE-FORWARD -m comment --comment "kubernetes forwarding rules" -m mark --mark 0x4000/0x4000 -j ACCEPT
-A KUBE-FORWARD -m comment --comment "kubernetes forwarding conntrack pod source rule" -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT
-A KUBE-FORWARD -m comment --comment "kubernetes forwarding conntrack pod destination rule" -m conntrack --ctstate RELATED,ESTABLISHED -j ACCEPT

To Reproduce
The deployment YAML includes the following ports setup and even with or without hostIP the hostPort doesn't work:
          ports:
            - name: rtmp
              containerPort: 1935
              protocol: TCP
            - name: inbound
              containerPort: 6666
              protocol: TCP
              hostPort: 6666
              #hostIP: <IP_ADDRESS>
            - name: talkback
              containerPort: 7004
              protocol: UDP
            - name: http
              containerPort: 7080
              protocol: TCP
            - name: management
              containerPort: 7442
              protocol: TCP
              hostPort: 7442
              #hostIP: <IP_ADDRESS>
            - name: https
              containerPort: 7443
              protocol: TCP
            - name: rtmps
              containerPort: 7444
              protocol: TCP
            - name: httpvideo
              containerPort: 7445
              protocol: TCP
            - name: httpsvideo
              containerPort: 7446
              protocol: TCP
            - name: rtsp
              containerPort: 7447
              protocol: TCP

The container is privileged:
      containers:
        - name: controller
          securityContext:
            allowPrivilegeEscalation: true
            privileged: true
            capabilities:
              add: ["SYS_ADMIN", "DAC_READ_SEARCH"]

Expected behavior
Traffic to be forwarded from cameras towards the pod.
Actual behavior
No traffic is forwarded from cameras towards the pod, I have not tried to use tcpdump but I reckon the missing iptables chains should be enough.
Additional context / logs

I've uninstalled the agent and reinstall to use docker and results are the same with no hostPort and hostIP config.

Looks like it records with docker under the agent. Is it only supposed to work with docker or it should work with containerd?
Thank you

Most of the rules are in the nat table:
iptables --list-rules --table nat | grep KUBE

It worked. Thank you


Support new version of .lnk which targets Microsoft Store App

It seems a new version/format of .lnk comes out in Windows 10. With Windows' built-in file Properties dialog, I cannot modify key properties of .lnk file which targets Microsoft Store App.
I want to figure out full target information which is to be used in XLaunchpad to create a launch item.
To create that kind of .lnk, see Create Desktop Shortcut for Store app in Windows 10

Same here. Wanted to edit a game link file but it actually destroyed Microsoft Store games .lnk file :(

Found an alternative tool, perfectly working and able to add command line arguments for .lnk files created by XBox app (Microsoft Store games):
https://flint-inc.ru/eng/info/linkinfo.html

Hi. I am very sorry for the delay in offering a response, but I have had this project quite forgotten and abandoned.
Responding to your question, that kind of link from Microsoft Word is not a "new" link format, it is a link composed by Item Identifiers and Identifier Lists.
Easy Link File Viewer was aware of item identifiers for shell folders, but it was not working as expected for some kind of links with other item identifiers such as for example a link created by an MSI installer.
The mentioned third party tool, LinkEditor, it also does not work properly (it can't retrieve the program target) for those kind of link files from Windows Installer products, such as link files created by Microsoft Word or Nero Burning ROM MSI installers.
I've implemented experimental functionality in Easy Link File Viewer to try properly read/write those kind of links, please see change log of release v1.6.
If you have more questions feel free to continue asking in this thread. For now, I consider it completed.
And sorry again for the delay in my answers.
Sincerely,
Elektro.

Same here. Wanted to edit a game link file but it actually destroyed Microsoft Store games .lnk file :(

Hi. Again I need to say: I am very sorry for the delay in offering a response, but I have had this project quite forgotten and abandoned.
And I feel sorry for the application to have corrupted your link file, but I've made updates applying consistent security changes providing various recent releases, ¡please give it a try for one last time!: https://github.com/ElektroStudios/Easy-Link-File-Viewer/releases
Sincerely,
Elektro.


Add PR Creation stage configuration

This PR adds support for a new prCreation configuration option, with default value 'immediate' (existing behaviour). It also supports these two options:

status-success: only create PR once branch status is "success"
not-pending: create PR once statuses are no longer pending (maybe success or failure)

Codecov Report

Merging #168 into master will decrease coverage by 2.43%.
The diff coverage is 10%.

@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
- Coverage   68.59%   66.16%   -2.44%
==========================================
  Files          19       19
  Lines         952      990      +38
  Branches      161      169       +8
==========================================
+ Hits          653      655       +2
- Misses        299      335      +36

Impacted Files
Coverage Δ

lib/config/definitions.js
100% <ø> (ø)
:arrow_up:

lib/api/github.js
95.12% <0%> (-3.62%)
:arrow_down:

lib/api/gitlab.js
2.42% <0%> (-0.3%)
:arrow_down:

lib/workers/branch.js
77.77% <100%> (ø)
:arrow_up:

lib/workers/pr.js
73.91% <20%> (-26.09%)
:arrow_down:

Continue to review full report at Codecov.

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


Add registry settings for separate scroll bindings

Add registry settings to bind separate 'scroll up/scroll down' buttons, instead of 'emulate 3-button point ctrl'

I'll probably split out the pointctrl button handling as a generic 3-to-4 handler; some people have issues with the oculus ring finger pinch being confused with the middle finger pinch - supporting either 3 or 4 'buttons' regardless of input method would be useful.

Done in 851a99ec6e053203a39a6fdc564c59f317cc5867 for v1.1