ContractVerifierFilter is dead code?

I can't see any implementations of this interface or any calls to its method. Also no uses of MessageNotMatchingException.

That interface was supposed to be implemented by people when they want to do manual messaging. But actually it's pointless. We can delete it.


Offline after latest update

Describe the issue you are experiencing
After upgrade, the CCU Matic dont comes up.
Describe the behavior you expected
The OS can boot
Steps to reproduce the issue

Upgrade
Reboot

...
What is the version this bug report is based on?
3....
Which base platform are you running?
rpi3 (RaspberryPi3)
Which HomeMatic/homematicIP radio module are you using?
RPI-RF-MOD
Anything in the logs that might be useful for us?
Running sysctl xargs Bad message
...
Irqbalance: start-stop-daemon can't not execute ....
Rngd: start-stop-daemon can't not execute ....
System Message Bus: fail to open config
Iptables error in line 9: iptables-restore: Bad message

On the boot the screen, the last message is:
Starting HMserver.........

Additional information
No response

Sorry, but this isn't a discussion fora here but a bug and issue tracker. So please discuss sich local issue in an appropriate user discussion fora instead of raising a bug/issue ticket which certainly isn't a bug in RaspberryMatic.
However, your problem clearly reads like an issue with your used sd card, which in fact might be broken after the update. So please use a completely fresh sd card and install a fresh RaspberryMatic on it and then restore your latest sbk backup. And then please male sure to use an industry like sd card in future (max endurance).
But as said, this is the bug/issue ticket here and no discussion fora. So please route such support request to an appropriate fora instead in future (e.g. homematic-forum.de)


Author box Bio too long

The "see all posts by xx" will not stretch. This usually occurs first in smaller screen size.

fixed, will be available on v0.1.2


Following server

I've opened this draft so you can take a look at the code and open a discussion to add changes, improvemnet, etc. Later, I'll create a new PR with all the changes.
The following server is currently working, as seen in this demo video I made: https://youtu.be/YaQQm7aWb-Y
I'm working on the following new features:

Add the local costmap so the robot can avoid obstacles (this would be an optional parameter). In some clutter spaces, like our lab, sometimes the controller generates a trajectory directly to the tables 😅. I'm thinking that part of this behavior to avoid collisions and the trajectory visualizer can be shared from the graceful controller: by adding it to the smooth control law or share the already implemented class to avoid collisions from the controller. That way we can share code and reduce errors.
Implement some kind of recovery mecanism for when the robot loses sight of the object to follow.

As the following and docking server share a large part of the code. I'll try to extract the code into an ApproachServer class (I'm really bad with the names :/) so both server can inherit from it.

Something with the smoothness would be great too! I think the thread in https://github.com/ros-navigation/navigation2/issues/4115 covers that intention a bit on the controller side. Otherwise, I think your other descriptions make sense. I actually filed a ticket when migrating this over to Nav2 (https://github.com/ros-navigation/navigation2/issues/4404) on the collision checking bits, I agree that it would also be applicable here too!

Great. I'll keep going then.
Because the docking is ported to nav2, should I make the PR here, merge here and then port it to nav2 or make it directly to nav2?

Probably just leave it here for the moment. The Nav2 version has some refactoring that we can wait to address until we're done with a POC

@SteveMacenski I'd like to start finishing this. Due to recent changes in docking and the deprecation of this repo, do you mind if I move this PR to a new PR in the nav2 repo?

Sure! Though, I think some of this work is blocked by the Graceful Controller's selection of the path point to be smooth https://github.com/ros-navigation/navigation2/issues/4115
So, I'd argue perhaps leaving this here until we're unblocked might be a good choice.

Sure! Though, I think some of this work is blocked by the Graceful Controller's selection of the path point to be smooth ros-navigation/navigation2#4115
So, I'd argue perhaps leaving this here until we're unblocked might be a good choice.

No problem, it was to make everything easier with the final steps, testing, etc.
Although any improvements made to the Graceful Controller must also be applied here, this server only uses the control law.


[TT-834] update eth clients to latest versions

geth: 1.13.5 -> 1.13.10
erigon: 2.54.2 -> 2.56.0
besu: 23.10 -> 24.1
nethermind: 1.22.0 -> 1.25.1

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (32d3586) 17.67% compared to head (227780a) 17.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #820   +/-   ##
=======================================
  Coverage   17.67%   17.67%
=======================================
  Files          33       33
  Lines        4227     4227
=======================================
  Hits          747      747
  Misses       3405     3405
  Partials       75       75

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


Make a first pass at documenting the parameters available in the conf…

…iguration yaml file.

This needs to be tested to make sure the yaml will work before mergins.


When Camera Open That time show this ?

I have a question i want to set crop image box when my camera view is open and then click on button just box content get as a image so can i perform this task using this plugin in .net maui

I don't think that this is possible with this package, as that would be a completely custom camera application, of which this is not.


Update Readme.md : removed dead link

The link was leading to a 404

@Zacaria i would have also appreciated if you would have added an alternative to this rather than removing it, thanks anyway!


feat: Add -w parameter to build.sh to disable Filamat build

Adds the -w option to the script, which excludes filamat from the CMake build (-DFILAMENT_BUILD_FILAMAT=OFF).

Hey @romainguy - I think I might've missed something when creating this PR, I thought filamat was fully modular and could easily be decoupled if needed (to shave off the MBs of the binary) - is that not how the Android Maven artefact is set up as well, with filamat being an optional extra dependency?

Filament itself never links against filamat. Filamat is however used by matc to compile materials, so it's always built for the host anyway. This compile-time flag is only used to not build the Android versions of filamat. But like you said, it's a separate AAR anyway, so if you never use it, you'll never pay for its cost.


Missing information that are available on the website

So far I made good progress with a nodejs relication, thanks for the very useful documentation. Some information that I cannot find though by using API that are available on the website are detailed battery informations:
The URI here returns it:
https://www.bmw-connecteddrive.de/api/vehicle/navigation/v1/
The resulting json is:
{
"latitude" : ...,
"longitude" : ...,
"isoCountryCode" : "...",
"auxPowerRegular" : ...,
"auxPowerEcoPro" : ...,
"auxPowerEcoProPlus" : ...,
"soc" : ...,
"socMax" : ...,
"pendingUpdate" : false,
"vehicleTracking" : true
}
the values soc and socMax are very interesting because they mean the state of charge in kilowatts and the maximum respectively.
Any idea how I could retrieve these values by using the API?
Thanks!

just call the url as the other api calls with your VIN at the end. You have to add the authorization header and it works ...

thanks, I noticed these further calls while F12 inspecting the connected website, but did not try the same auth header and adding VIN...
should be added to the readme.md, shouldn't it?

Please add stuff to the README.md 😁
I don't have an i3 any more, so can't check. But I trust you.

@edent I did!


WIP: terraform: remove CloudFlare, add GCP

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Question: nodejs compatibility?

I haven't tested this personally, but can this lib be consumed in nodejs/typescript?

Re node: well the unit/integration tests for the client library run under node with mocked react native modules. And the keyserver runs on node via AWS lambda functions.
Re typescript... I'm not very familiar with it myself. But are there any special requirements to consume regular ES6 modules?

no, if it runs under nodejs then it will most likely run under ts.
but is there a way to just throw in this lib as a dependency in nodejs project with minimal efforts (like, mocking RN modules)?

Are you thinking of backing up the LNDhub private key? If you already have a keystore solution you really only need the keybackup module. The only dependencies there are Frisbee (which runs under node anyway) and cloudstore which would need to be mocked with a cloud backend of your choice. There is a PR for the Gdrive REST api in the works. But hard to guess what the best solution is without better understanding your use case.

no I was thinking about, for example, starting new backend project and having all classes available to import mnemonics and create transactions.
no relation to Lndhub

Ok cool. Well let me know if I can support you in anyway. Photon has profited immensely from your great work at BlueWallet. So whatever I can do to repay the favor :)

thanks! :-)
so long story short, I wanted bluewallet code to be isomorphic, to reuse it on servers and in browsers, but BW grown too complex.
one day I tried to setup nodejs project and add BW as a dependency just to reuse some classes without copy-paste, and failed miserably.
so that would be really nice to have a project (like photon-lib) that can only be used as a dependency, and instantly provide some of bluewallet's classes to either nodejs, browser tab, react-native, or whatever (think javascript on microcontrollers).

bitcoinjs-lib already works everywhere, but its a bit too low-level

so long story short, I wanted bluewallet code to be isomorphic, to reuse it on servers and in browsers, but BW grown too complex.

That makes sense. I got the sense that's why you created the blue_modules folder?

so that would be really nice to have a project (like photon-lib) that can only be used as a dependency, and instantly provide some of bluewallet's classes to either nodejs, browser tab, react-native, or whatever (think javascript on microcontrollers).

Totally! That was precisely the idea behind the photon-lib. I think there would be tremendous value in consolidating library code between photon-lib and bluewallet. That way both projects share the maintenance cost and profit from security reviews, etc.
If that is valuable for you, I'm happy to share org access to photon-sdk with BlueWallet and we could start with consolidating the wallet modules, which both projects use?

the idea behind blue_modules was that sometimes pieces of code work better as OOP (with all that incapsulation and inheritance stuff), and sometimes code is organized well enough with plain modules. also, some external dependencies can be moved outside of node_modules into blue_modules to better keep an eye on them.
consolidating library code is not something I would do immediately for BW, but there is certainly value in making it work seamlessly on nodejs/rn/browser, this way next time I'll start a side project I won't have to copy-paste a bunch of code from BW.

the idea behind blue_modules was that sometimes pieces of code work better as OOP (with all that incapsulation and inheritance stuff), and sometimes code is organized well enough with plain modules. also, some external dependencies can be moved outside of node_modules into blue_modules to better keep an eye on them.
consolidating library code is not something I would do immediately for BW, but there is certainly value in making it work seamlessly on nodejs/rn/browser, this way next time I'll start a side project I won't have to copy-paste a bunch of code from BW.


[BUG] Unknown plugin option jsonLogic

Environment
Please provide as many details as you can:
Direct use the  formio/core render library in dist folder, found report error:
Unknown plugin option jsonLogic.
Then trying to build from source, get same error after invoke "npm install"
BTW, there no detail information about how to use the core library. like like to load plugin by usePlugin.

Hosting type

[ ] Form.io
[ ] Local deployment

Version:

Formio.js version: N/A
Frontend framework: N/A
Browser: Chrome
Browser version: 92.0.4515.107

Steps to Reproduce

npm install

Expected behavior
No error.

@travist I think this plugin should register under rules, is this understand correct? I guess that we should apply the same logic with "rules".

@lane-formio
What about this? It's extremly important

I am having the same issue

@brendanbond @AlexeyNikipelau

This is expected to be resolved in the next major release. We've seen the error but it hasn't blocked us from anything. 🤔

Agent Jeriah Henley linked Freshdesk ticket 48981 for this issue.

Looks like this has been fixed in by these commits...
https://github.com/formio/core/commit/7018209ce0f78df7a20b6bb2329b86480ec6191d
https://github.com/formio/core/commit/8042210d39db250b5a668a9bb7c2d3bf03deb0fd


Add support for GN build

Chromium is moving from GYP to GN to generate "ninja" build. We currently only support GYP for now. We should wait for the GN move to be completed first.

Done since https://github.com/Samsung/ChromiumGStreamerBackend/tree/55.0.2851.2.0_gst


Setup build & docker for release

Description
We need to configure a process of build and publish that will be used for all packages inside the repo. The idea is build a Docker image on Github Registry on each release that can be used altogether with the binaries within the Github Release and publish on Crates.io.
This process is very similar with what already have been doing on Fuel Core.
Tasks

[x] Configure Docker structure for build (use this as base standard)
[x] Add step on Release action for build and publish a Docker image
[x] Add step on Release action for build and publish Rust binaries within the automated Github Release
[x] Add step on Release action for build and publish on Crates.io

Closed #31


WIP

@Theodus I'm going to leave the nits as is as they are copied directly from existing. I'll add an issue pointing back to the PR that someone can address.


Suggestion: Storage Blocks Features

Storage blocks (1k, 4k, 16k, 64k) should keep their configuration when broken.
**When breaking storage blocks the whitelist, priority and settings (void excess, etc..) aren't kept.
Storage blocks with items in them (which aren't part of the white list) can be a pain to manage for the OCD storage nerds among us.  Perhaps allowing them to work like a storage disk in the disk manipulator would be the perfect solution?
**Having to build a secondary refined storage network specifically for the purpose of moving these objects to a disk in order to use in a disk manipulator seems like extra unnecessary steps.
Unless of course I'm missing a simple solution to these issues...

Storage blocks (1k, 4k, 16k, 64k) should keep their configuration when broken.

Use a Wrench in Dismantling mode for this in 1.11 ... except don't because it will void your items (this is already fixed :P)
But Storage Blocks should work with Disk Manipulator yeah, I'm changing your issue title to reflect this.

I changed my mind on this. It's called disk manipulator for a reason. The storage blocks can remain "stupid" blocks.


Derive type information for scala Map

It looks like the deriveTypeInformation macro does not work for Map[K, V] types. It fails with error
magnolia: could not infer serializers.Typeclass for type Map[........
Is this expected?
If Map[K, V] is indeed not supported currently, are there any plans do add such support?

Obviously I can use TypeInformation.of(classOf[Map[K, V]]) instead of deriveTypeInformation , but then it comes back as GenericType<scala.collection.immutable.Map> without any type information for key or value. I guess, in this case the serialization will be not as efficient as it would be with the official scala API which can in fact derive such information.
Is there any workaround?

Could you show small code example which raises that Magnolia error?
I am not sure about pure Map[K,V] support as type T for deriveTypeInformation macro. Maybe it is a bug.
Can you try to infer TypeInformation via implicitly? For example:
import org.apache.flinkx.api.serializers._

case class SimpleClass1(a: String, b: Int)

implicitly[TypeInformation[Map[String, SimpleClass1]]]

Thanks Alexey,
Using of 'implicitly' indeed eliminated that error.
However, now I am facing more issues getting TypeInformation for the types I need.
In our code we deal with Avro objects; the object type that needs to be serialized is a case class that has a few avro fields.
I need to get TypeInformation for those avro classes and have them as implicit values, so the system will pick them up when making serializers for the containing class.
Here is an example. I have a class 'com.liquidnet.sea.order.analytics.avro.CalcIssue', which is a java class created by Avro compiler.
If I do
implicitly[TypeInformation[CalcIssue]]
I get the following error:
could not find implicit value for parameter e: org.apache.flink.api.common.typeinfo.TypeInformation[com.liquidnet.sea.order.analytics.avro.CalcIssue]
Would you have any examples of how to deal with Avro in flink-extended?
FYI: currently we use the official flink scala api, and it works fine there, with createTypeInformation macro. I wanted to experiment with flink-extended because flink scala support is deprecated, and also we may need to move on to newer scala versions in the future. It would be great if I could make flink-extended work for our projects.

Ok, thanks for confirming.
As for the Avro generated Java class, it is a bit more complicated. Since it is not a product type nor case class, flink-extdended will fill to derive implicit TypeInformation.
Better way would be to compile Avro code to Scala case classes. Then it should work (need to check to be sure).
See this project as an example:  https://github.com/novakov-alexey/flink-savepoint-scala212/
Here is the compiler trigger: https://github.com/novakov-alexey/flink-savepoint-scala212/blob/main/build.sbt#L52
This generates Scala case class similar to the following.
Input:
{
    "type": "record",
    "namespace": "com.github.novakovalexey",
    "name": "Event",
    "fields": [
        {
            "name": "number",
            "type": "int"
        },
        {
            "name": "count",
            "type": "int",
            "default": 1
        }
    ]
}

Generated output:
import scala.annotation.switch

final case class Event(var number: Int, var count: Int = 1) extends org.apache.avro.specific.SpecificRecordBase {
  def this() = this(0, 1)
  def get(field$: Int): AnyRef = {
    (field$: @switch) match {
      case 0 => {
        number
      }.asInstanceOf[AnyRef]
      case 1 => {
        count
      }.asInstanceOf[AnyRef]
      case _ => new org.apache.avro.AvroRuntimeException("Bad index")
    }
  }
  def put(field$: Int, value: Any): Unit = {
    (field$: @switch) match {
      case 0 => this.number = {
        value
      }.asInstanceOf[Int]
      case 1 => this.count = {
        value
      }.asInstanceOf[Int]
      case _ => new org.apache.avro.AvroRuntimeException("Bad index")
    }
    ()
  }
  def getSchema: org.apache.avro.Schema = com.github.novakovalexey.Event.SCHEMA$
}

object Event {
  val SCHEMA$ = new org.apache.avro.Schema.Parser().parse("{\"type\":\"record\",\"name\":\"Event\",\"namespace\":\"com.github.novakovalexey\",\"fields\":[{\"name\":\"number\",\"type\":\"int\"},{\"name\":\"count\",\"type\":\"int\",\"default\":1}]}")
}

Although that project is for official scala-api, I think it should work with flink-extended as well. It would be great if you could check it.

Thanks, maybe I will look at this later.
I understand that flink-extended will work much better with real scala classes.
Our problem is, we have a big project that relies on flink kafka support. That includes special kafka sources and sinks, support for schema registry, etc. Naturally, such kafka sources produce Java Avro classes. If we want to have Scala Avro classes instead, we either need some other library or create our own serializers and such. Just a lot of work ath this point.


feat: allow setting # of retries

This change allows users to set the number of no-response retries for GCP.

Codecov Report

Merging #45 into master will decrease coverage by 1.87%.
The diff coverage is 77.77%.

@@           Coverage Diff           @@
##           master   #45      +/-   ##
=======================================
- Coverage   97.87%   96%   -1.88%
=======================================
  Files           1     1
  Lines          47    50       +3
  Branches        8     9       +1
=======================================
+ Hits           46    48       +2
- Misses          1     2       +1

Impacted Files
Coverage Δ

src/index.ts
96% <77.77%> (-1.88%)
: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 78f57e0...b9056a5. Read the comment docs.

@ofrobots @JustinBeckwith @stephenplusplus PTAL

Just curious - why?

Please do let me know if we should open this back up :)


Gemma-1.1 train error

TypeError: GemmaForCausalLM.__init__() got an unexpected keyword argument 'rope_scaling'
They updated config here, so now google/gemma-1.1-2b-it not working with unsloth

@Theodotus1243 Oh ye saw the new model! I'll take a look!

@Theodotus1243 Fixed! For Colab / Kaggle instances, no need to update Unsloth. For local machines, use pip install --upgrade --force-reinstall --no-cache-dir git+https://github.com/unslothai/unsloth.git

Thanks, now training launches smoosly

@danielhanchen Sorry, reopening
For gemma-2b-it no error
For gemma-1.1-2b-it (with rope_scaling == null) new error
RuntimeError: Tensors of the same index must be on the same device and the same dtype except step tensors that can be CPU and float32 notwithstanding
I'm not sure if it's a problem of model init on transformers or unsloth
But reporting here for now

When just added "rope_scaling": null to the gemma-1.1-2b-it config it starts working
So for now I treat it as config error in gemma, not a problem with unslosh

@Theodotus1243 Ill take a look at it again!

Also you can add rope_scaling argument while loading:
model_name = "google/gemma-1.1-7b-it"
model, tokenizer = FastLanguageModel.from_pretrained(
    model_name = model_name
    rope_scaling=None,
)

Oh I think I fixed this!! :) Ie I auto remove rope_scaling if it errors out


Grpc server thread pool is full, rejecting the task

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[x] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?
We have two oapServer, one of it report "Grpc server thread pool is full, rejecting the task" after running some days with fix GrpcServer's threadPoolSize and queueSize.

Bug

Which version of SkyWalking, OS and JRE?
Skywalking 6.3.0
CentOS 7.x, JDK 1.8, 8C CPU, 16G RAM

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.
Following solution of #3405, fixed 6.3.0 according to master branch(6.4.0 under testing) by setting GrpcServer's threadPoolSize and queueSize, oapServer seems ok within one day(no grpc-default-executor-* exponential growth and no heap OOM). but after running two ~ three days, one of our oapServer continue report "Grpc server thread pool is full, rejecting the task"(屏幕快照 2019-09-09 10 09 53) till now(屏幕快照 2019-09-09 10 10 36) in log.
in jConsole, difference between two oapServer is the one reported  "Grpc server thread pool is full, rejecting the task" has no grpc-default-executor-*(屏幕快照 2019-09-09 10 11 42).

Requirement or improvement

Please describe about your requirements or improvement suggestions.

fixed these files.

We added thread size control to make sure you wouldn't OOM. But two many connections are not acceptable. You should not have so many connections.

@wu-sheng fully upgraded to 6.4.0 and running almost three days, the problem reappeared: continue report "Grpc server thread pool is full, rejecting the task" error from grpc-default-worker-ELG-.
in jvisualvm, no grpc-default-executor-, looks like these threads are all destoried somehow.
pls look into this problem.

 Something blocks, I can't be sure why. Most likely, your storage crashed or facing performance issue, then, the grpc can't write data to memory cache, which could trigger this issue.

What do you mean thread destoried?

from jvisualvm, can't find any thread named "grpc-default-executor-", as if they have been destroied somehow. but another server still running ok has several "grpc-default-executor-".

That is new to me. Could you find why? How to make grpc thread exit

strange thing is "grpc-default-executor-" is from the default thread pool in grpc, which use name in format of "grpc-default-executor-%d", but in 6.4.0, a ThreadPoolExecutor is set in GRPCServer "nettyServerBuilder...executor(executor)" with given threadPoolSize and queueSize, so it make no sense still using the default thread pool.

In 6.4.0, we use our own thread pool. Because in some cases, too many grpc thread created. That is expected.

@wu-sheng elasticsearch[scheduler][T#1], this thread blocking all grpc-default-executor-*, and that thread is continue in waiting status.

If they are blocked, that is expected from design level, You need to find out why elasticsearch doesn't response. Check your ES server status and monitoring tool.

yes i will investigate our ES, but what i think is the data keeper should be the lowest priority task compare to others, especially those to save metrics/trace or register service/serviceInstance into ES, so can the data keeper modified to delete older datas only in "Idle" time?

yes i will investigate our ES, but what i think is the data keeper should be the lowest priority task compare to others, especially those to save metrics/trace or register service/serviceInstance into ES, so can the data keeper modified to delete older datas only in "Idle" time?

Delete? The 6.4.0(6.3.0+) delete only happens in middle night, and once per day. You may read the delete logs, but in ElasticSearch implementation, it would execute like that.

ps., after running oapServer on two servers, our ES become heavy loaded all the time, cpu/heap/load always off the chart.

that's strange, from the log, i can only surpose it is running the data keeper executor before thread pool is full.

Read ElasticsearchStorageTTL, if you will find the delete wouldn't happen in every 5 mins. This is only the check running every 5 mins.

we plan to use skywalking on our production environment, but for the moment it can been done without solving this problem first, could you give use some advice on ES heavy load after running oapServer, for example, to disable some functionality that we may not use.
or where to find a stable version of skywalking?

Really, there is nothing I can do.
There are plenty of users using the project in production environments. Some of them are supporting several TB traces per day.
We can't provide the ElasticSearch service for you. ElasticSearch is a stable, powerful and widely used data platform, so you should be able to find the solution by yourself.
The only thing I could say is, either you don't set your ElasticSearch right(we highlighted some in our FAQ, https://github.com/apache/skywalking/blob/master/docs/en/FAQ/ES-Server-FAQ.md), or your ElasticSearch cluster is too small.

i have the same question,how do you resolve this probleam? @jadezhong

today, we also meet the same problem @wu-sheng

@DongWeiMa @wz2cool
Sorry for the delayed reply.
never solved, i duplicated the "thread pool is full" scenario again, and using jvisualvm i saw the L*_AGGREGATION thread is dead due to each 00:00 we start to run a daily ElasticSearch index removal job, this will make the ElasticSearch reject tasks at that time, i suppose this will make those threads dead one after one.
Later we separate ElasticSearch used by Skywalking out, and never do index removal job, for almost a month, Skywalking seems works fine.


Use Cow to hold the contents of packet inside Packet

I think cleaner solution can be found from #46.

PR #46 won the beauty contest.


Ability for JCSDA members to create branches in atlas?

Hi @mmiesch @benjaminmenetrier
Is this repository open for creation of branches?

Can you try again? I just added writing rights to the JEDI team.

Thanks @benjaminmenetrier !


Volume control not working

Volume control is not working for me.  Is this a bug - or needs more config ?

Maybe clarify more about how you are using the program. Rest api? Socketio? (The companion module uses socketio.) what version of Spotify controller?

What version of the companion module and what version of Spotify controller?

Companion 3.2.1 (3.2.1+6676-stable-fa981d25)
Controller 0.3.0

Click the ? Icon next to the module instance in companion. It will report the module version.

Spotify Controller v2.0.1
this ?

The latest version should be v2.1.0. So download a beta build of companion and try to recreate this issue again. If it still happens you can re open this issue.

Ok - Thanx


【Hackathon 6th No. 41】PIRATENETS_code_reproduction_rfc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Allow changing the USB vendor ID, product ID, manufacturer, and product name using build settings

Presently, it's not possible to change the USB identifiers without forking the SDK and modifying src/rp2_common/pico_stdio_usb/stdio_usb_descriptors.c. This can be a challenge when shipping a product or even just developing multiple RP2040-based devices at the same time, since it's very difficult to figure out which device is which when multiple are connected.
This ideally should be possible to change through CMakeLists.txt like this:
target_compile_definitions(project PUBLIC USBD_VID=0xCAFE USBD_PID=0xCAFE USBD_MANUFACTURER="Widgets, Inc" USBD_PRODUCT="Widget")

Merged into develop


projects: Use osc.username if bugowner and maintainer are not informe…

…d (#84)
Signed-off-by: Marcos Paulo de Souza<EMAIL_ADDRESS>

@boiko the first commit only create the person tags if necessary. The other two are just fixes related to a more flexible use of put_meta call.
@crazyscientist I hope that it fixes #84 since now all optional arguments are handled correctly.


fix(coralogix): don't mask error time

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

v2.25.4
v2.25.4
GitHub release

Your semantic-release bot :package::rocket:


Clarified purpose of running w/Bot Emulator

@msftdherron,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[$250] Workspace - Unable to open Workspace setting when open member chat page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.81-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to Profile> Workspaces> Create Workspace
Go to Members> Invite member> Invite any email address
Click on invited member section to open Profile page
Select message user
Go to Profile> Workspaces
Click on the workspace to open settings page

Expected Result:
Workspace settings page should open
Actual Result:
Workspace settings page can not be open, nothing happens when click on workspace
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/8cef2c43-3b0e-413f-a64a-04c8aadf7ef2
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0134016f887e04d471
Upwork Job ID:<PHONE_NUMBER>719432391
Last Price Increase: 2024-06-18

Issue OwnerCurrent Issue Owner: @ahmedGaber93

How bizarre! I'm able to reproduce, moving it on.

Waiting on proposals

Waiting on proposals

I am Michał from Callstack - expert contributor group. I’d like to work on this job.

@trjExpensify We can assign @MrMuzyk from callstack.
@MrMuzyk after assigning, please post your findings here daily and propose how to fix it to weigh in before creating a PR. thanks!

Proposal
Please re-state the problem that we are trying to solve in this issue.
App not navigating user to selected screen
What is the root cause of that problem?
In this scenario, in many of many checks before navigating user, we check if desired state is different from current state.
This is done via getPartialStateDiff function. Next, action from this diff is extracted and dispatched causing a navigate
https://github.com/Expensify/App/blob/b278b8f2ec103eed98215b7068c14cc8d029deea/src/libs/Navigation/linkTo/index.ts#L125-L131
In this function we are checking not only if screen name is different but also if params are different. Because that wasn't the case here - params are the same as before - user wasn't getting redirected to proper screen
https://github.com/Expensify/App/blob/b278b8f2ec103eed98215b7068c14cc8d029deea/src/libs/Navigation/AppNavigator/getPartialStateDiff.ts#L70-L79
What changes do you think we should make in order to solve the problem?

I'd remove shallowCompare check from the condition in line 76. If screen names are different then it's good enough reason to navigate the user already. There is no need to check for params difference as well.
What alternative solutions did you explore? (Optional)
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

In this function we are checking not only if screen name is different but also if params are different.

@MrMuzyk I think we need for that because in nested navigation state, params will contain screen name

@MrMuzyk I think we need for that because in nested navigation state, params will contain screen name

@ahmedGaber93 I don't think it matters anyway. All parts of this condition have to be true because of logical AND.
If it was the case that params would have different name but screen name stored in .name field of stateTopmostFullScreen and templateStateTopmostFullScreen were the same, the outcome would've been the same.
I still believe that checking only for name difference should be sufficient to navigate the user
     (stateTopmostFullScreen &&
         templateStateTopmostFullScreen &&
         stateTopmostFullScreen.name !== templateStateTopmostFullScreen.name &&
         !shallowCompare(stateTopmostFullScreen.params as Record<string, unknown> | undefined, templateStateTopmostFullScreen.params as Record<string, unknown> | undefined))

@MrMuzyk I think if we remove shallowCompare the diff state for the example below will ignore and ScreenC not open
const stateTopmostFullScreen = {
  name: 'ScreenA',
  params: {},
}

const templateStateTopmostFullScreen = {
  name: 'ScreenA',
  params: {
    screen: 'ScreenB',
    params: {
      screen: 'ScreenC',
    },
  },
}

It wouldn't open anyway because name properties are the same which would case whole condition to return false regardless of what comes after
&& stateTopmostFullScreen.name !== templateStateTopmostFullScreen.name &&

Yeah, that is correct true, so I think the condition should be

if names are different, return true without check params
if names are the same, check the params

// stateTopmostFullScreen.name !== templateStateTopmostFullScreen.name && !shallowCompare
stateTopmostFullScreen.name !== templateStateTopmostFullScreen.name || !shallowCompare

What do you think?

Ahh I get it now 😅 I agree - if screen name can come inside params then it would make sense to change the condition like on the line above you've posted

I think this part of the code affected in many places. Could you please test it through the app and update your proposal?

Yes, will do!

Can I proceed with PR then?

CC @Beamanator

Opened a PR, tomorrow Im OOO. If anything comes up I will take care of this on friday

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[x] [@ahmedGaber93] The PR that introduced the bug has been identified. Link to the PR: N/A
[x] [@ahmedGaber93] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
[x] [@ahmedGaber93] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] [@ahmedGaber93] Determine if we should create a regression test for this bug.
[x] [@ahmedGaber93] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Go to Account setting > Workspaces > Create Workspace
Go to Members > Invite member > Invite any email address
Click on the invited member item to open Profile page then click on message user
Go to Account setting > Workspaces again
Click on any workspace to open its settings page and Verify it open successfully

Do we agree 👍 or 👎

Can you provide a bit more context on all the "N/As" in the checklist, please @ahmedGaber93?

@trjExpensify updated.

Cool, thanks. Paid!


Integrate  JDToaster

Replace alerts with JDToaster.

Can I take it?

Yes, you can, but let’s discuss it first in this issue @artyom2206.

So i installed it to Papr and implemented instead of each alertAction, but there is a problem with JDToaster on new IOS 13.2 version. When the toast shows from the bottom, then it disappears(check GIF). Toast duration also not works, I called it manually. Should I do pull request?
(https://user-images.githubusercontent.com/47772683/69388436-25c74e80-0cc9-11ea-935a-3855fe85161a.gif)

This certainly does not look as I expected 🤔
Would you like to investigate the issue in the JDToaster?

Ok, I will look at it

Adding a strong reference to the UIWindow will solve the issue. @artyom2206

The issue is fixed! I will do a release soon.

Hey @jdisho
Is this issue already fixed ? If not I'd like to contribute on that. Let me know at the earliest.

I don't know if this is relevant. As you may see, I'm not maintaining this app anymore 😴


PlatformData is stored in two different places?

Running the following code:
    bgfx::Init init;
    init.resolution.width = width();
    init.resolution.height = height();
    init.resolution.reset = BGFX_RESET_VSYNC;

    bgfx::PlatformData data;
    data.nwh = reinterpret_cast<void*>(winId());
    bgfx::setPlatformData(data);
    bgfx::init(init);

Produces the message:
bgfx/src/bgfx.cpp (3173): BGFX bgfx platform data like window handle or backbuffer is not set, creating headless device.

This is actually incorrect. BGFX doesn't end up creating a headless device. The reason this message is printed is because the check for this checks the _init.platformData structure (which is of course all NULL) instead of the g_platformData structure set by bgfx::setPlatformData().
My question is: Why are there two ways to set the platform data?

bgfx::setPlatformData is legacy, but didn't want to remove it because a lot of people using it. Preferred way to set platform data is via bgfx::Init structure. Also not setting would create headless device on certain renderers where that's possible, but it's not supported everywhere.

Alright, thanks!

bgfx::setPlatformData is legacy

Examples don't reflect/document this and the documentation is not clear. Can the docs be updated? Because I'm hitting the headless error over and over and can't get a simple context with e.g. SDL, even though it's pretty much copied directly from the examples.


Draft PR for checking out +body isue

@DavinciDreams, Thanks, I am going to fix this style.

Closing this since I think we're done


chore(deps): upgrade jenkins-x/jx3-pipeline-catalog to version 3.10.77

from: https://github.com/jenkins-x/jx

/retest

/retest

Ooops, jx-gitops is broken. trying to fix with jenkins-x-plugins/jx-gitops#926


Allow minimising in 2d

Objective

We can't minimise if there's a 2d camera because ??? there legally must be a 2d target.
Fixes https://github.com/bevyengine/bevy/issues/4526

Solution

Make it not crash in those cases, just do nothing
Seems to work ¯\(ツ)/¯
See also the companion commit in https://github.com/bevyengine/bevy/pull/3597 - 503c24717321bb2bb2681b358020ad1bcbef510e

I also wonder why this should panic instead of just returning an error. The Node::run are supposed to be fallible.

the panic is thread 'main' panicked at 'view entity should exist: QueryDoesNotMatch(0v0)', crates\bevy_core_pipeline\src\main_pass_2d.rs:45:14, so panicking on the expect there. Should it return a NodeRunError in this case?

A similar change was made to main_pass_3d.rs in #3330, which Cart approved.

can't reproduce the initial crash, but the code change makes sense

Why is there no view when the window is minimised?

Also, how come this only affects 2d?

I don't know why this happens whilst minimised. Presumably we have some code that prevents creating swapchain textures of size 0.
And as @devil-ira mentioned, #3330 added the same thing for 3d, hence that not crashing.

@DJMcNab, #4747 does the same thing for UI cameras. Can you roll in that fix?

I can do if someone makes a pr against the branch for it :laughing:
I'm not in a context where I can do any manual git operations at the moment

I just made a pr against your branch. 👌

I guess we should add a UI camera to the minimising and resizing examples too

soon © (https://github.com/bevyengine/bevy/pull/4745) the ui camera will be a component on existing cameras


Document annotation interaction with distributed SchedulerPlugins

Currently, the method for specifying annotations is documented, but their interaction with distributed SchedulerPlugins is not.
It would be useful to demonstrate some simple (but powerful) interaction use cases.
I'd like to take this on as I'll be working with SchedulerPlugins over the next while.

Any ideas on the best place for this?

dask documentation
distributed documentation
a blog post?

I imagine distributed documentation and a blog post would be a good approach.

@sjperkins Checking in here, I was wondering if the documentation/blog about the annotation interaction with distributed SchedulerPlugins is still something you'd like to take on?


Correct date return month_start, month_end

Changes solve the problem of the incorrect date to "def current_month()". The error occured due to incorrect date format month_start.

Thanks for the PR!
I am just going to fix the failing travis builds (unrelated to this) before merging.


Hello world

Hello world

Please fix all pylint issues first.

Fixed.

Great! Please create a new PR.


Problem when training with multiple gpu

When training on my own dataset and I set "gpus": 2 in conv3d_sep2,it raises ValueError.
ValueError: Index out of range using input dim 0; input has only 0 dims for 'trainnet/strided_slice_10' (op: 'StridedSlice') with input shapes: [], [1], [1], [1] and with computed input tensors: input[3] = <1>.
But when setting "gpus": 1 ,it works well.
If I want to use multiple GPU training, do I need to modify other parameters?

Multi-gpu training with TrackR-CNN is not supported. We produced all results using a single gpu. It may work if you try setting "own_dataset_per_gpu": true.

Thanks,I will try it later.

Closing this for now, please reopen if needed


JSX element class does not support attributes because it does not have a 'props' property.

I am working with "react-github-btn" and getting this error in my webstorm linter but the code works successfully.
TS2607: JSX element class does not support attributes because it does not have a 'props' property.
TS2786: 'GitHubButton' cannot be used as a JSX component.
  Its instance type 'GitHubButton' is not a valid JSX element.
    Type 'GitHubButton' is missing the following properties from type 'ElementClass': render, context, setState, forceUpdate, and 3 more.

I cannot reproduce your issue. I created a brand new project with npx create-react-app my-app --template typescript and added react-github-btn. No issue is reported by my tsserver linter.
Please try creating a new project with npx create-react-app my-app --template typescript and see if you can reproduce this issue with a brand new project with latest version of all dependencies.

I cannot reproduce your issue. I created a brand new project with npx create-react-app my-app --template typescript and added react-github-btn. No issue is reported by my tsserver linter.
Please try creating a new project with npx create-react-app my-app --template typescript and see if you can reproduce this issue with a brand new project with latest version of all dependencies.

This is likely an issue of webstorm. Closing.


Settings fragment

Added a Settings fragment to set the wakelock on/off and choose theme mode

better merge into dev


Add DPG: Digital Health Access

Automatic addition of a new digital public good submitted through the online form available at https://digitalpublicgoods.net/submission

Duplicate of #1252


BUG on NAPALM [FIX IN DESCRIPTION]

Issue type
[ ] Feature request
[x] Bug report
[ ] Documentation

Environment

Python version: 3.5.2
NetBox version: 2.3.2

Description
By using pip 10 you CAN'T INSTALL NAPALM
You have to downgrade it to atleast pip version 9
sudo pip install pip==9
Once you do that you are able to install napalm just make sure to run it with  --no-cache-dir

See #2008


Aptos type is labeled as evm-base

Aptos type is labeled as evm-base, but it's not an EVM as far as I'm aware.
https://github.com/cosmos/chain-registry/blob/eadb5fa2acad607eb779d57a835b664f29f98dd5/_non-cosmos/aptos/assetlist.json#L20

This has been changed to type unknown. As communities grow more types can be added.
https://github.com/cosmos/chain-registry/blob/0260a8e654869f723ee3f85194bae04192dc5f65/_non-cosmos/aptos/assetlist.json#L20


disable ShaDa when running tests

This prevents the tests from altering the user's own shada history if
he/she runs the tests on a local install.
Fixes #16

I also re-ordered the arguments passed to nvim, so they're all consistent.
One issue though. When disabling ShaDa for tests, they take a very long time to run. I am not sure if that's a side-effect of disabling it or something in the tests themselves that cause it.

Wow yeah this change takes the test suite from 11s to >2min for me. The simplest repro step I can find is time rspec spec/neovim_spec.rb -e ".attach_child". The test is 1 second slower if you add -i NONE to the spawn arguments. Note that getting rid of either the -i or -u options fixes the delay. @justinmk do you have any thoughts? Sorry I'd look into it deeper but I'm on vacation for the next week.
@jpleau this seems like a good opportunity to dry up some of these tests. How do you feel about adding a Support.child_argv helper or something to spec/support.rb and using that in tests rather than repeating them?

@alexgenco Added 6e16d0f6230dadcd3bdfab262da893dded210af0 that adds a function called Support.child_argv like you suggested.
Note: I haven't done ruby in a while, don't hesitate to nit picks anything I could have done better !
I made it "customizable" so if we want to run an entire command we can still use this function, add extra stuff (like --headless or +q like some tests already did)

Will fix the failing tests with ruby 1.9 tomorrow, shouldn't change the code too much

I think I fixed the performance problem with https://github.com/alexgenco/neovim-ruby/commit/b31bee127c072f4a2dbaeee7d942da38ce824971. Go ahead and rebase onto master.

All done, would you prefer me to rebase to only have one commit ?

No need to squash if each commit passes rake. Thanks!

Thanks as well.
By the way, this shada issue was discovered while packaging neovim-ruby for Debian. Forgot to let you know in advance that I planned to package this, so here we go ;)

Awesome!


[NOT TO BE REVIEWED OR MERGED] Move predict to end of test_tqdm_progress_bar_disabled_when_not_rank_zero

What does this PR do?
Repro of #11713
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Marked as ready for review to see full CI


maximum_consecutive_dry_days returns zero for all NaN input

xclim version: '0.12.3'
Python version: 3.7
Operating System: linux

Description
If a whole year of data is NaN xclim.indices.maximum_consecutive_dry_days (and maybe others) returns 0. I would rather expect NaN?
What I Did
import numpy as np
import xclim
import xarray as xr

data = [0, np.NaN, 0, np.NaN]
time = xr.cftime_range("2006-01-01", periods=4, freq="A")
attrs = dict(units="kg m-2 s-1")
da = xr.DataArray(data, dims=["time"], coords=dict(time=time), attrs=attrs)
xclim.indices.maximum_consecutive_dry_days(da)

NaNs are handled by Indicator classes:
from xclim import atmos
atmos.maximum_consecutive_dry_days(da)

will fail saying input should be daily. If you pass daily series with missing values, the output with missing data should be set to NaNs. This was a deliberate design decision to keep indices simple, and manage missing values at a higher abstraction level.

Ok, thanks for the info, that works indeed as I would have expected. However, atmos is very hard to find but it should probably be the default? Also atmos is not a very intuitive name for what it does.
I did import xclim <tab>: oh there is indices, let's use this. Naively I would expect the atmos category in indices and the unchecked ones in indices_no_check (or something like this).

The idea was to have two layers, one with basic calculations (indices), and the other with metadata formatting and missing value checks.
We thought to put these indicators in categories reflecting the cmip realm they belong to (atmos, land, ocean, seaIce).
One issue we have is that those indicators are class instances, and it's not yet clear how to document those in sphinx. So you're right, for the moment they're not very visible.
Suggestions on how to improve on the current setup are most welcome !
One action we could take is clean the scope of xclim, so we have only ICCLIM, atmos, land, indices, seaIce and utils.

On my todo list as well. Need to improve the docs to explain atmos (and class indicators) vs indices better. There is a small section explaining the differences in the workflow examples but this definitely needs a better structure and more examples.

Sounds good to me - one small thing you could also do is to make atmos discoverable from xclim directly, i.e. enable xclim.atmos (such that it does not need to be imported as from xclim import atmos).

That's now in master, with a release planned Friday.


[libidn2] Fix invalid macros in code

Fixes https://github.com/microsoft/vcpkg/issues/31011
./stdint.h:89:5: error: expected value in expression
#if
    ^
1 error generated.
In file included from rawmemchr.c:26:

Fix macro GNULIBHEADERS_OVERRIDE_WINT_T not working on Windows and osx.
Because there is no GNULIBHEADERS_OVERRIDE_WINT_T macro in Windows and osx, the code compilation and replacement fails. Add platform judgment in the code. When the platform is Windows or osx, replace the value of the macro GNULIBHEADERS_OVERRIDE_WINT_T with 0.
Compilation tests pass in the following triplet:
x64-osx
x86-windows
x64-windows
x64-linux

[X] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

Duplicate to https://github.com/microsoft/vcpkg/pull/34234.


Merge 1.7.x into 1.8.x (DEV)

There were no conflicts :partying_face:

No UA necessary, just pulling existing changes.


I need to connect my uvc camera with the object detection application (android).

Could someone tell me how to config or where I should config?

Any updates regarding this issue? Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

@cp0418,
Could you please check out the object detection guide for mobile devices from this link. Thanks!

@amahendrakar the link u shared doesn't correspond to the questions. currently, TensorFlow demo object detection app uses Camera2 API to get a frame from the app. Hence, he is asking if it's possible to use uvc libraries to get the frame fed into the object detection model.


[REQUEST] add a screenshot button

this would come so much in handy when only displaying gyms for tactical purposes or creating maps for events

windows and android aswell are able to take screenshots.. on windows press print then open paint and there press strg+v or open snippingtool to cut a part from your desktop..
on android i think it was volume up+powerbutton or on cyanogenmod go to settings buttons und aktivate screenshot... now press the powerbutton for a second and choose screenshot..

On a mac, you can press [Command] [Shift] [3] to take a screenshot and airdrop it to your phone. The whole process should only take about 30ish seconds.

all good sugestions but a quick screenshot button for the pc version makes it a bit more nicer :)


Add Azure Storage support

We need to provide basic support to Azure Storage, including

[x] Tiers
[x] Kinds
[ ] Public/Private
[ ] VNET Service Endpoints

Add private endpoint capability

[ ]  Private DNS Zone
[ ]  Private DNS Zone Vnet Link
[ ]  Private Endpoint
[ ]  DNS Record
[ ]  Block all other incoming traffic to storage account


cant create claim in fact check form

Cant reproduce the issue

So the issue was that user was trying to hit okay button which was part of the modal and not the create claim form because of which the claim was not being created, but the modal was closing so it was appearing as if the request has been sent. To solve this issue we removed the okay and cancel button at the bottom of the modal.


[Bug]: cannot see workspaces when I fork a template

Is there an existing issue for this?

[X] I have searched the existing issues

Description
when I find a template the I like on the website and try to fork it, I cannot see my workspaces.
There is a workaround - I need to expand the workspaces dropdown twice in order to see the values.
https://user-images.githubusercontent.com/94514895/210748113-4b162eb4-0991-464b-9474-4d22feacf3ab.mov
Steps To Reproduce
see in the attached video or follow these steps

go to https://www.appsmith.com/template/slackbot
click Use this template
expand the dropdown Select Workspace
--> the dropdown is empty
click somewhere else in the modal and expand the dropdown again
--> this time it works, I can see my workspaces

I think this should be at least a medium impact bug because although there is an easy workaround, a new user (explorer or trial) may not be aware of this workaround and me get confused/frustrated and hence in danger of bouncing.
Public Sample App
No response
Issue video log
No response
Version
Cloud

@rohan-arthur I'm unable to reproduce this. Is this reproducible for you?

@ginilpg you're right this is working now, thanks! I'll close the issue.


[ENH] Update OOC Queue for more information

🟢 How does GSE currently work
The Out of Combat Queue doesn't show all actions that are currently being processed in the queue.  Show excluded items.


when installing keeps asking for github name and pass and cannot finish installation

please help boss_ fatal: Authentication failed for 'https://github.com/Cyb0r9/SocialBox.git/

md siyam ahammed<EMAIL_ADDRESS>md siyam ahammed<EMAIL_ADDRESS>

md siyam ahammed<EMAIL_ADDRESS>

just press enter without typing your email and password it will work


[bug] unable to set up ssl connection to AWS rds

I was trying to use the docker image to set up ssl connection to rds, I hit into some issues:

I realised in run_goose.py there are environment variables set up for setting up ssl. However, the environment variable is not set in docker-compose file.
with the environment variable MF_METADATA_DB_SSL_ROOT_CERT in run_goose.py, we need to parse in the local file path of the certificate, would it be possible to set up a folder in this repo to store some common ca certificates for database so that we can parse in the certificates easier?

This seems like an oversight in the docker-compose file, partly as most deployments do not rely on this to spin up the services.
Having fixed a similar issue in a deployment recently, unless you are using custom certificates, getting the connection to a postgres RDS instance working should only require adding MF_METADATA_DB_SSL_MODE=prefer
I'm hesitant to adding any certificates as part of the repo, as these are more deployment specific. The preferred way would be to mount these for the container in question.


Usage of one to many with key directive - getting Global secondary index exception

Note: If your question is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

Which Category is your question related to?
Amplify Schema
Amplify CLI Version
4.18.1
You can use amplify -v to check the amplify cli version on your system
What AWS Services are you utilizing?
Provide additional details e.g. code snippets
Given a Schema
type CampaignAdmin @model
@key(fields: ["email"])
{
    email: AWSEmail!
    title: String!
    campaigns: [Campaign] @connection(keyName: "postsByProfile", fields: ["email"])
}

type Campaign @model
@key(name: "postsByProfile", fields: ["userEmail"]) {
    id: ID!
    userEmail: AWSEmail!
    user: CampaignAdmin @connection(fields: ["userEmail"])
    title: String!
    tasks: [Task] @connection(keyName: "commentsByPost", fields: ["id"])
}

type Task @model
@key(name: "commentsByPost", fields: ["postID"]) {
    id: ID!
    postID: ID!
    campaign: Campaign @connection(fields: ["postID"])
    content: String!
}

mutation MyMutation {
  createCampaignAdmin(input: {email<EMAIL_ADDRESS>title: "k"}) {
    email
  }
createCampaign(input: {userEmail<EMAIL_ADDRESS>title: "POST1", id: "p1"}) {
    id
  }
 createTask(input: {postID: "p1", content: "COMMENT1", id: "c1"}) {
    id
  }
}

query MyQuery {
  getCampaignAdmin(email<EMAIL_ADDRESS>{
    campaigns {
      items {
        title
        id
        user {
          email
        }
        tasks {
          items {
            id
            postID
            content
          }
        }
      }
    }
  }
}

Error
✔ Generated GraphQL operations successfully and saved at src/graphql
Error while executing Local DynamoDB
{
    "version": "2017-02-28",
    "operation": "Query",
    "query": {
        "expression": "#connectionAttribute = :connectionAttribute",
        "expressionNames": {
            "#connectionAttribute": "taskCampaignId"
        },
        "expressionValues": {
            ":connectionAttribute": {
                "S": "p1"
            }
        }
    },
    "scanIndexForward": true,
    "filter": null,
    "limit": 10,
    "nextToken": null,
    "index": "gsi-CampaignTasks"
}
ValidationException: The table does not have the specified index: gsi-CampaignTasks
    at Request.extractError (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/protocol/json.js:51:27)
    at Request.callListeners (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:683:14)
    at Request.transition (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:685:12)
    at Request.callListeners (/Users/name/.nvm/versions/node/v14.0.0/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:116:18) {
  code: 'ValidationException',
  time: 2020-05-03T05:07:24.073Z,
  requestId: '07e9b94e-df54-48a0-83f8-1f7aab6a8d3e',
  statusCode: 400,
  retryable: false,
  retryDelay: 49.65671333788136
}

Fix
BTW appologies for the bad key names - i was incrementally editing this example .
https://docs.amplify.aws/cli/graphql-transformer/directives#belongs-to
I did post an issue trying to mimic my schema using the one shown in example https://github.com/aws-amplify/amplify-cli/issues/4154
Bad Solution (not sure why this work)
if you change Task to Comment like this - things work :| am I missing something? is there some black magic in keynames? (BTW I did try to change key names too - and I am stuck)
type CampaignAdmin @model
@key(fields: ["email"])
{
    email: AWSEmail!
    title: String!
    campaigns: [Campaign] @connection(keyName: "postsByProfile", fields: ["email"])
}

type Campaign @model
@key(name: "postsByProfile", fields: ["userEmail"]) {
    id: ID!
    userEmail: AWSEmail!
    user: CampaignAdmin @connection(fields: ["userEmail"])
    title: String!
    comments: [Comment] @connection(keyName: "commentsByPost", fields: ["id"])
}

type Comment @model
@key(name: "commentsByPost", fields: ["postID"]) {
    id: ID!
    postID: ID!
    content: String!
    campaign: Campaign @connection(fields: ["postID"])
}

query MyQuery {
  getCampaignAdmin(email<EMAIL_ADDRESS>{
    campaigns {
      items {
        title
        id
        user {
          email
        }
        comments {
          items {
            id
            postID
            content
          }
        }
      }
    }
  }
}

{
  "data": {
    "getCampaignAdmin": {
      "campaigns": {
        "items": [
          {
            "title": "POST1",
            "id": "p1",
            "user": {
              "email"<EMAIL_ADDRESS>            },
            "comments": {
              "items": [
                {
                  "id": "c1",
                  "postID": "p1",
                  "content": "COMMENT1"
                }
              ]
            }
          }
        ]
      }
    }
  }
}

Ok this is nonsense
type CampaignAdmin @model
@key(fields: ["email"])
{
    email: AWSEmail!
    title: String!
    campaigns: [Campaign] @connection(keyName: "postsByProfile", fields: ["email"])
}

type Campaign @model
@key(name: "postsByProfile", fields: ["userEmail"]) {
    id: ID!
    userEmail: AWSEmail!
    user: CampaignAdmin @connection(fields: ["userEmail"])
    title: String!
    task: [Task] @connection(keyName: "taskByPost", fields: ["id"])
}

type Task @model
@key(name: "taskByPost", fields: ["postID"]) {
    id: ID!
    postID: ID!
    content: String!
    campaign: Campaign @connection(fields: ["postID"])
}

Changing the name of the key makes it work - Did i not read things correctly? can someone clarify?
Also this doesn't work if I name task as tasks  (I wanna name things plural since it's an array)
type CampaignAdmin @model
@key(fields: ["email"])
{
    email: AWSEmail!
    title: String!
    campaigns: [Campaign] @connection(keyName: "postsByProfile", fields: ["email"])
}

type Campaign @model
@key(name: "postsByProfile", fields: ["userEmail"]) {
    id: ID!
    userEmail: AWSEmail!
    user: CampaignAdmin @connection(fields: ["userEmail"])
    title: String!
    tasks: [Task] @connection(keyName: "tasksByPost", fields: ["id"])
}

type Task @model
@key(name: "tasksByPost", fields: ["postID"]) {
    id: ID!
    postID: ID!
    content: String!
    campaign: Campaign @connection(fields: ["postID"])
}

BTW I was trying to change this schema to that
type User @model
@key(fields: ["email"])
{
    email: AWSEmail!
    title: String!
    posts: [Post] @connection(keyName: "postsByProfile", fields: ["email"])
}

type Post @model
@key(name: "postsByProfile", fields: ["userEmail"]) {
    id: ID!
    userEmail: AWSEmail!
    user: User @connection(fields: ["userEmail"])
    title: String!
    comments: [Comment] @connection(keyName: "commentsByPost", fields: ["id"])
    pictures: [Picture] @connection(keyName: "picturesByPost", fields: ["id"])
}

type Comment @model
@key(name: "commentsByPost", fields: ["postID", "content"]) {
    id: ID!
    postID: ID!
    content: String!
    post: Post @connection(fields: ["postID"])
}

type Picture @model
@key(name: "picturesByPost", fields: ["postID"]) {
    id: ID!
    postID: ID!
    post: Post @connection(fields: ["postID"])
}

This works :( :( I feel so stupid - not sure if I am missing some minute detail or if there is a bug?

@gautamkshitij Mock does

if you change Task to Comment like this - things work :| am I missing something? is there some black magic in keynames? (BTW I did try to change key names too - and I am stuck)

Amplify mock creates table and indexes initially and can't change the indexes once they get created. This might be because of that. One solution to this problem is to nuke the amplify/mock-data folder which has the local dynamodb tables.

@yuth i always nuke my mock-data (always) i combined it with an alias (mock api + rm -r /mock-data).
And this is a weird thing. Lol naming my variable singular works but plural throws the error.
Happy to get on a zoom link to show this :|

I am confirming again and again - plural variable names for arrays didn't work for me lol - singular variable names don't throw any error. Fixing all my problems by that.

Closing as duplicate of #4157


create Shelter mongoDB model

create a shelter MongoDB model with the fields as
name, location, capacity, then reference pets and a description

https://github.com/jjackson0228/Pawsitive-Connection/pull/28


[Discussion] About training models myself

Below is the content from Here and I've read it.
Train model
For training your own model, you need:

A dataset of separated files such as musDB.
Dataset must be described in CSV files : one for training and one for validation) which are used for generating training data.
A JSON configuration file such as this one that gathers all parameters needed for training and paths to CSV file.
Once your train configuration is setup, you can run model training as following :

spleeter train -p configs/musdb_config.json -d </path/to/musdb>

From the command above, I notice that I need to provide:

A musdb_config.json file;
A musdb

Question 1
A musdb_config.json file look like below, copy from here
{
    "train_csv": "configs/musdb_train.csv",
    "validation_csv": "configs/musdb_validation.csv",
    "model_dir": "musdb_model",
    "mix_name": "mix",
    "instrument_list": ["vocals", "drums", "bass", "other"],
    "sample_rate":44100,
    "frame_length":4096,
    "frame_step":1024,
    "T":512,
    "F":1024,
    "n_channels":2,
    "n_chunks_per_song":40,
    "separation_exponent":2,
    "mask_extension":"zeros",
    "learning_rate": 1e-4,
    "batch_size":4,
    "training_cache":"cache/training",
    "validation_cache":"cache/validation",
    "train_max_steps": 200000,
    "throttle_secs":1800,
    "random_seed":3,
    "save_checkpoints_steps":1000,
    "save_summary_steps":5,
    "model":{
        "type":"unet.unet",
        "params":{
               "conv_activation":"ELU",
               "deconv_activation":"ELU"
        }
    }
}

But where can I get the full documentation of it? For example, what does T and F mean? for instrument_list, can I only use ["vocals", "other"]? where can I get the full documentation of all these config options?

Question 2
I've downloaded the musdb from musdb18.zip and extract the zip file, I found that it is a folder containing 2 folders: train and test(see the screenshot below)

Inside the train and test folder, there are all mp4 files(use mp4 instead of mp3 or aac is because mp4 can contain more than one track in it)

I've listened the mp4 file in train folder and test folder in musdb18.zip, it seems they are no any difference, they are all songs.
So in my understanding, they are no difference. Assuming that I have 150 song file, can I choose 100 of them for training and the rest for validation?

Question 3
I use ffprobe to check the mp4 files mentioned above, I found it has many tracks, the first track is a mix of all audio tracks, the other audio tracks are separated tracks(vocals, drum, bass, .etc), and the last track is a video track, but in fact it has no video, it's a still png image.
ffprobe -hide_banner -i  Young\ Griffo\ -\ Pennies.stem.mp4
[mov,mp4,m4a,3gp,3g2,mj2 @ 0x7f8bfc808200] stream 0, timescale not set
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'Young Griffo - Pennies.stem.mp4':
  Metadata:
    major_brand     : isom
    minor_version   : 1
    compatible_brands: isom
    creation_time   : 2017-12-16T17:34:20.000000Z
  Duration: 00:04:37.80, start: 0.000000, bitrate: 1288 kb/s
  Stream #0:0(und): Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 256 kb/s (default)
    Metadata:
      handler_name    : SoundHandler
      vendor_id       : [0][0][0][0]
  Stream #0:1(und): Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 256 kb/s
    Metadata:
      handler_name    : SoundHandler
      vendor_id       : [0][0][0][0]
  Stream #0:2(und): Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 256 kb/s
    Metadata:
      handler_name    : SoundHandler
      vendor_id       : [0][0][0][0]
  Stream #0:3(und): Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 256 kb/s
    Metadata:
      handler_name    : SoundHandler
      vendor_id       : [0][0][0][0]
  Stream #0:4(und): Audio: aac (LC) (mp4a / 0x6134706D), 44100 Hz, stereo, fltp, 256 kb/s
    Metadata:
      handler_name    : SoundHandler
      vendor_id       : [0][0][0][0]
  Stream #0:5: Video: png, rgba(pc), 512x512 [SAR 20157:20157 DAR 1:1], 90k tbr, 90k tbn, 90k tbc (attached pic)

Now I have vocals.m4a and bg-musics.m4a, how can I merge a vocal and it's corresponding bg-music and a album cover image to a mp4 file by using ffmpeg?

Question 4
From question1 we can know that we also need 2 csv file: musdb_train.csv and musdb_validation.csv.
I notice that the musdb_train.csv file has 6  columns:
mix_path | vocals_path | drums_path | bass_path | other_path | duration

If I only needs 2stems, it that I only need to provide these 4 columns in this csv file?
mix_path | vocals_path | other_path | duration

Hi! I am now working on the training too. you may find the the answer  here. I hope it helps.

Hi @xiebruce,
For your first question, you have some information about parameters set in config files in the wiki. It is possibly a bit incomplete though. Regarding the instrument list, it should be set according to the dataset you want to train on. For instance with musdb, you can possibly use ["vocals", "instrumentals"] as you have an both instrumentals.wav and vocals.wav file for every track and that they will sum up to
mixture.wav.
For your second question, spleeter was not made for dealing with the multi-stem format *.mp4, so you should use the multiple waveform version of musdb.
You can do a different split than the original proposed musdb one, which is only provided for algorithm comparison puposes. So if you don't plan to compare your model with other model on the test set, you can possibly use songs of the test set in your training.
The third question concerns musdb and not spleeter, so this is not the right place for asking/answering it.
For your 4th questions, indeed you can provide only two columns if you'd like to perform 2 stems separation. As mentioned in the answer to question 2, you need to ensure that the provided stems sums up to the mix (e.g. the sum of the stems is equal to the mix). With musdb, you can do it with the instrumentals stem and the vocal stems.

Hello! I am doing the same but with Beethoven Cello Sonatas.
How many hours of samples/data are you using to train spleeter?
Thanks!

@isolepinas Sorry, I still don't know how to do it yet. But I think this is depends on your computer's performance and the data size. Can your share the whole process, the step that you are doing? I prefer use examples and screenshots rather than just describing, thank you in advance.

Dear Bruce,
Just like you I am in the first steps of the process. My idea is to feed to
spleeter 3 versions of a performance (piano solo),(cello solo) and (piano
and cello together), I aim to train spleeter to understand whats a piano
and whats a cello and when together, it should be able to only extract
cello without losing vibrato, portamento and other characteristics.
Some studies have used 44 samples or 1h and 14 minutes such as here
https://veleslavia.github.io/conditioned-u-net/
Maybe you will find it interesting!
lets keep in touch so we can share our findings and processes!
On Sat, 26 Mar 2022 at 10:50, Bruce @.***> wrote:

@isolepinas https://github.com/isolepinas Sorry, I still don't know how
to do it yet. But I think this is depends on your computer's performance
and the data size. Can your share the whole process, the step that you are
doing? I prefer use examples and screenshots rather than just describing,
thank you in advance.
—
Reply to this email directly, view it on GitHub
https://github.com/deezer/spleeter/issues/740#issuecomment-1079660733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYNKUM3RWJD55X6KK73LQNLVB3TVXANCNFSM5QYFB33Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

@isolepinas OK, thank you.


fix(select): edit * imports

Codecov Report

Merging #874 into rc0.13.0 will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           rc0.13.0     #874   +/-   ##
=========================================
  Coverage     93.98%   93.98%
=========================================
  Files            85       85
  Lines          3575     3575
  Branches        560      560
=========================================
  Hits           3360     3360
  Misses           90       90
  Partials        125      125

Impacted Files
Coverage Δ

packages/select/helper-text/index.tsx
95% <100%> (ø)
:arrow_up:

packages/select/Option.tsx
100% <100%> (ø)
:arrow_up:

packages/select/EnhancedSelect.tsx
94.04% <100%> (ø)
:arrow_up:

packages/select/BaseSelect.tsx
90.16% <100%> (ø)
:arrow_up:

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 7649f90...bc39d10. Read the comment docs.


fix: apps/tx-builder/package.json to reduce vulnerabilities

The following vulnerabilities are fixed with an upgrade:

https://snyk.io/vuln/SNYK-JS-SSRI-1085630

Thank you for the contribution @affiliatedkat!
This was fixed in https://github.com/gnosis/safe-react-apps/commit/f190ac7a39cfc8bd83e25844b6e1d3716a1d88ad ✅


Add documentation for the CUPiD API

We'd like to include documentation for the code within the CUPiD API. This can be automatically generated and updated with sphinx. We may also want to consider pydoc or other tools.

@rmshkv mentioned that "We should also include some more extensive documentation for people developing notebooks about including the serial parameter and logic with the local cluster, maybe in a separate page about parallelization."


Bug/Feature: Suffix requests

Overall this is fairly straightforward - replicate the set of fixes, additional structs (ContentRange) and some of the error-reporting states.
There is, however, one important deviation when running inside a browser: 'Content-Range' is not a CORS-safelisted property :grimacing: : https://developer.mozilla.org/en-US/docs/Glossary/CORS-safelisted_response_header .
The upstream behaviour treats this as an error (rightly, given the HTTP spec mandates support for the Range header -> support for the Content-Range header, and CORS is inapplicable outside the browser) - in the browser case, this is probably a no-go, given the paucity of sites that include it in Access-Control-Expose-Headers.
Since the meta.size field is not an Option type, it must be set to something - the only problem with that is the natural fallback value (what object store used to do, namely use the Content-Length field) is likely to be wildly inaccurate.

An additional ClientOptions, exposed on the object store itself and the Wasm wrapper, is needed here.
Actually determining at runtime whether a HEAD is required (only in situations where the size of the file actually matters (cases where it doesn't matter: formats like parquet essentially describe themselves once you have the last 8 bytes))... that can be somewhat tricky if all you have is a range and a size that might be the size of the file or just the range.
This is one area where this repo will have to diverge from upstream (probably permanently) - each of the stores authored here will need an extra method (get_range_conditional_head, maybe get_ranges_conditional_head) that does:

internal client::get_opts, with the (very simple) wrapping logic present in the ObjectStore trait.
Based on inspection of the result of get_opts from the internal client (which will need a marker of some sort for internally indicating that the reported sizes are inaccurate), dispatch a second get_opts with a HEAD request (or, if it isn't subject to the same restrictions, a GET zero-length). The results of that second call should be returned alongside the first, with the overall file size extracted from the HEAD and some sort of clear indicator to avoid relying on subsequent calls to non-conditional-head methods.

Since the meta.size field is not an Option type, it must be set to something

With respect to Parquet specifically, I think the ParquetObjectReader shouldn't require an ObjectMeta in its constructor (or it should be optional). This is something I want to fix in https://github.com/apache/arrow-rs/issues/5979#issuecomment-2234204999 but I haven't gotten around to it yet. Happy to collaborate on that if you want.

Yeah, I was thinking along the same lines, that the main consumer (parquet-wasm, geoarrow-wasm) will likely be (transitively) refactored at the earliest by ~September (I agree, it really does seem to only be relevant for the FileMetadata extraction), thus removing most of the motivation for this.

Well arrow 52.2.0 was just released, so we have about ~1 month to get this into parquet, or else it will sit till December


Typo "Azure notebook"→"Azure Notebook"

https://docs.microsoft.com/en-us/azure/cognitive-services/personalizer/tutorial-use-azure-notebook-generate-loop-data

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


drivers/usbdev: fix complile break about mtp class

Summary
drivers/usbdev: fix complile break about mtp class
Impact
fix compile break
Testing
testing vela

Right, understood! Thank you Xiang!


packager detects 'electron-prebuilt' instead of 'electron' in the package.json

Note As of version 1.3.1, this package is published to npm under two names: electron and electron-prebuilt. You can currently use either name, but electron is recommended, as the electron-prebuilt name is deprecated, and will only be published until the end of 2016.

But packager currently detects 'electron-prebuilt' not 'electron' in the package.json.
Installed packager with npm install electron-packager -g.

Thanks for filing an issue! In order to help you with your problem, we're going to need more information about it. In particular:

Console output when you run electron-packager with the environment variable DEBUG=electron-packager. Please include the stack trace if one exists.

What command line arguments are you passing? Alternatively, if you are using the API, what
parameters are you passing to the packager() function?

Hi. Thanks for replying.
I can only describe what happened but I cannot run any command at the moment coz I just don't have access to a computer. QAQ
I was running 'electron-packager ./ --platform=win32 --arch=x64' (something like that for Windows 10 packaging.... Not remembering quite clear.)
After running I get the message that cannot get the version of the electron in the package.json
So I dig into the package.json and found that the proj included electron instead of electron-prebuilt.
After I changed electron to electron-prebuilt, the package process goes as expected without errors.
So I guess the packager was checking electron-prebuilt but as said in the readme this name is deprecated.
Hope it helps.

My suspicion is that you have an old version of Electron Packager installed, that's why I asked for more details.

Hmm, the latest commit was 2176894a5be5da17a362bf3f20678641a78f4b69.... Indeed an old version.
Console ouput with debug on.
Unable to determine application name or Electron version. Please specify an application name and Electron version.
Cannot find module 'package.json' from '/home/ubuntu'(Properties not found yet: dependencies.electron-prebuilt,devDependencies.electron-prebuilt)

Upgrading to @7.7.0.... Then I got this again.
If changing electron": "^1.3.4" to "electron-prebuilt": "^1.3.4", it works.
repo link: https://github.com/Moeditor/Moeditor
But it seems that other people who package this repo aren't facing this strange thing..... Wonder why,

Strange. Please run DEBUG=electron-packager electron-packager ./ --platform=win32 --arch=x64 and paste the output. The debug output should be useful in figuring out what your problem is.

Still

Unable to determine application name or Electron version. Please specify an application name and Electron version.
Cannot find module 'package.json' from '/home/ubuntu'(Properties not found yet: dependencies.electron-prebuilt,devDependencies.electron-prebuilt)

I can share the c9.io environment if you offer an account.

I find it highly odd that you're not getting any debug output. Could you please run the following commands, and paste the output of each?

npm list electron-packager
npm list -g electron-packager

:+1:
Finally I got it from  npm list -g electron-packager.
Only from this I learned that my version in -g is wrong( it was 7.1.0).
:sob: Hope it will be easier to check the version in the future.(such as -v or -V)
Thank you for your help in so many days. I'm closing this issue now.
:tada:

I've never liked the fact that --version doesn't give the electron-packager version. I think in a future major version I'm going to change it so it's more like other command line tools (I've reopened #154 as a result).


Handle errors in MWU/ASEGs, comment-out ASEGs

Returns None for Mann Whitney U calculations or positive_rates (used by ASEGs) where one or both of the series is empty.  This might occur if a subgroup has no positive records (e.g. in our evaluation pipeline, most subgroups have toxic and non-toxic records, but not all of them might have records for toxicity subtypes like "obscene").
Comments out the ASEG calculations from per_subgroup_aucs, as this slows down the function considerably and is not needed just yet.
Adds a default value to the new parameter in per_subgroup_auc_diff_from_overall so that it doesn't break existing callers
This PR also has a bunch of little diffs due to removing whitespace and adding a newline at the EOF, sorry if that's annoying to read although I think the new changes are what we want.

Just made that change, thanks


Update CONTRIBUTING.md

Fixes #<issue_number_goes_here> (it's a good idea to open an issue first for context and/or discussion)

Thanks, but the CLA is not signed and that header is intended to be the title.


Add the possibility for folders and paging

If I understand your description correct, this project collects all photos on one page. I would be interested in different collections.  And can there be a paging mechanism for a  larger amounts of photos?

If I understand your description correct, this project collects all photos on one page.
Yes, this is the current state.

I would be interested in different collections.
Collection meaning grouping of different photos? Something like categories, where you add some photos to the category e.g. "London 2018" and a separate collection with this name and all the photos that are associated with this category appear?

And can there be a paging mechanism for a larger amounts of photos?
This is definitely something worth considering.

yes, collections as categories. the same you could do in photos from Google. Maybe this could  simply be solved with the collection from eleventy. But what about the actual central folder for adding new images? What about placing a folder inside this central upload-folder that is named like the respective category it should be displayed in?
I am just trying to imagine how I could use your tool with more than just a handfull of images.


Players cannot replay quests

I want the players on my server to be able to replay a quest after a day and one large quest be replayable after a week. Is this possible? Is a replayable quest possible even without the fancy time outs?

Figured it out lol


provides initial docker options support

closes #265. Provides a field in all.yml for a string of docker options. This string is applied to each network_plugin template if defined. I found that the Docker daemon only respected a single definition of DOCKER_OPTS, etc., so the if statement is present in the initial line for the template. Works as expected on CoreOS in AWS.
As an aside, need to revisit the templates themselves, as the Calico template seems to be pointing toward flannel_mtu values and other flannel things. I have not fooled with Calico, so it may deploy just fine, but it may be worth someone fixing these inconsistencies for clarity.

We should probably have another PR to remove the contents of the docker template for calico.  All three conditionals can't happen from the calico plugin.  They all include kube_network_plugin == flannel.  This will always generate an empty file.  I think calico needs something like weave.  Also not related to this PR, it seems like each file needs a OS section to generate the correct environment Variable name.

Totally agree @galthaus , we have to do some cleanup.
For instance the condition kube_network_plugin == ... isn't required anymore.
What do you mean by

it seems like each file needs a OS section to generate the correct environment Variable name.
What files exactly ? and What section ?

For the docker env file, the variable seems to be different by OS.  It seems like if we could get the flannel variable earlier (but we can't), we could create a single file that merged options by OS.
OPTIONS vs DOCKER_NETWORK_OPTIONS vs DOCKER_OPTS

Merging this PR and opening https://github.com/kubespray/kargo/issues/271


Ollama Docker Rocm facing constant issues

What is the issue?
So although I have a consistent working Rocm install in Fedora 40- gfx1030 target, also I already have llama.cpp working perfectly in Rocm and Vulkan targets and even Ollama in CPU only mode, but have been consistently getting docker: layers from manifest don't match image configuration. error when trying to run ollama:rocm image in docker. For reference normal ollama/ollama  in Docker works fine on CPU only.
See 'docker run --help'.

OS
Linux, Docker
GPU
AMD
CPU
AMD
Ollama version
0.3.6

I am having the exact same issue on Fedora 40. For me the issue started after 0.1.42-rocm.
 Getting image source signatures Copying blob 4767b6298c18 skipped: already exists   Copying blob 2d473b07cdd5 skipped: already exists   Copying blob bd707b33a96a skipped: already exists   Copying blob b3800af90b90 skipped: already exists   Copying blob 93191fa4e1f8 skipped: already exists   Copying blob ec6b9c70f06a skipped: already exists   Copying blob 10645e0d079c skipped: already exists   Copying blob f35b80e1d7b6 skipped: already exists   Copying blob ba086e88d2e3 skipped: already exists   Copying blob 13bfe13f7f56 skipped: already exists   Copying blob efc528f475fe skipped: already exists   Copying blob 3c9d729a222e done   |  Copying blob 1db71b1d7c67 done   |  Copying blob c630fdf8b6e2 done   |  Copying blob 35d951731348 skipped: already exists   Error: writing blob: storing blob to file "/var/tmp/container_images_storage1414294462/2": pigz: skipping: <stdin>: corrupted -- crc32 mismatch: exit status 1
The command being run is podman pull ollama/ollama:rocm
$ podman version Client:       Podman Engine Version:      5.2.1 API Version:  5.2.1 Go Version:   go1.22.5 Built:        Tue Aug 13 18:00:00 2024 OS/Arch:      linux/amd64

I looped over each version backward from 3.7.0-rc6-rocm and found that the most recent version that pulled correctly was 0.2.5-rocm.
Looks like a bunch of changes came through between 0.2.5 and 0.2.6: diff. But the only change I can see that relates specifically to the Docker image build process is the upgrade from rocm version 6.1.1 to 6.1.2.
This is about as far as my knowledge allows me to investigate this issue.

There is a problem with the fedora pigz package, If you just uninstall it and pull then docker image. As well as this issue seems to be a duplicate of #5979.

There is a problem with the fedora pigz package, If you just uninstall it and pull then docker image, It should work without problem. As well as this issue seems to be a duplicate of #5979.

Hey, it was indeed the problem, i now have it working correctly.

I wish I had the option of uninstalling pigz, but I am on an immutable distro.
As a note, I was able to pull ollama:latest, just not :rocm. I also wasn't able to pull it on an Alma Linux 9 machine.

Let's track this under #5979
@talliansaotome I don't know if this will work, but you could try working around this by pulling on another system where you can uninstall pigz, then docker save and docker load between the systems.


CI: use namespace.so

theoretical 13 second setup time with namespace

when pulling without building

unit tests in under 3 minutes: https://github.com/jnewb1/openpilot/actions/runs/7564279399/job/20598138983?pr=72
breakdown:
35s total setup (git,lfs,docker,build), 2:08 for tests

moving off my fork https://github.com/commaai/openpilot/pull/31056


Upgrade requirements

Codecov Report

Merging #55 (5e9c02a) into main (aa7c499) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #55   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           28        28
  Branches         8         8
=========================================
  Hits            28        28

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 aa7c499...5e9c02a. Read the comment docs.