drivers: modem: ublox-sara-r4: autodetection of modem and APN

Implemented:

automatic detection of modem type (Sara R410 or Sara U201)
automatic setting of APN, depending on inserted SIM card

Automatic detection of the modem type:
After hardware initialisation of the modem, the type of modem
is detected. After this, further initialisation and functionality
is chosen depending on the detected modem type.
To enable this functionality, set following bool in Kconfig:
MODEM_UBLOX_SARA_AUTODETECT_VARIANT
Automatic setting of APN:
During communication initialisation, the IMSI of the inserted SIM
card is evaluated to determine the APN. This is done by comparing
the first 5 characters of the IMSI to a list of known providers.
The list can be given in Kconfig.
To enable this functionality, set following bool in Kconfig:
MODEM_UBLOX_SARA_AUTODETECT_APN
To set a list of providers, set following string:
MODEM_UBLOX_SARA_AUTODETECT_APN_PROFILES
If the provider can not be found in the list, the APN given in
following entry is used as a fallback:
MODEM_UBLOX_SARA_R4_APN
Signed-off-by: Hans Wilmers<EMAIL_ADDRESS>

Caveat: When autodetection is chosen, it is presumed that initialisation of the hardware can be identical for R410 and U201. On my hardware this is the case, but probably it is not true generally.

ping @mike-scott

@mike-scott does this look ok to you?

@jukkar I'll play test today.  Thanks for the reminder.

@hwilmers After reviewing this, I think this should be 2 commits:

automatic detection of modem type (Sara R410 or Sara U201) changes
and then automatic setting of APN, depending on inserted SIM card

I also am questioning the complexity of the #ifdefs in this driver. If we can come up with a creative solution for the different pin init between U2 and R4, then we could do away with the manual selection of the modem type entirely.
But let's get the 2 changes into their own commits first.

@mike-scott Changes are now divided in two commits. Would you have any suggestions about how to reduce complexity of the #ifdefs? I am reluctant to introduce dead code, but if it would be ok to introduce some, complexity could be reduced.

@hwilmers Thanks for splitting this up.  I was thinking about the exact same thing with the #ifdef complexity.  If we could consolidate the init process for both R4 and U2, then we would remove the 3 #ifdefs which are really the only sticking point to removing the variant #ifdefs entirely.  It would increase the code size of the driver by an unknown amount (not a huge amount) -- but would also provide good code coverage.

@hwilmers Let me look at the Application notes power on/off sequence for both the R4xx and U2xx and see if we can conslidate.

@mike-scott Is there a chance to get this in before the merge window closes?

I think we could try to get this to 2.3, @mike-scott please ack if ok.

@jukkar I'd need at least one approval for this to go in. I will add @rlubos as well in case he can take a look.

I'll take another look

@hwilmers @jukkar @carlescufi I think this PR still needs a bit of discussion:
My current issues are:

The R4 vs. U2 differences in pin_init() are not addressed.  I think in it's current form, this PR would probably break R4.
The second patch might be mergeable as a stand-alone patch, but it only applies to U2 which uses the AT+UPSD command.  R4 relies on AT+CGDCONT to set APN.

1. The R4 vs. U2 differences in `pin_init()` are not addressed.  I think in it's current form, this PR would probably break R4.

Yes, I see that there is a problem. But it is the other way around, since with this patch (but only with MODEM_UBLOX_SARA_AUTODETECT_VARIANT set), for both modems the pin_init() is done as though they were R4. I have no better solution for this, and because on my hardware pin_init() is merely ignored, I can not test for this case.

2. The second patch might be mergeable as a stand-alone patch, but it only applies to U2 which uses the AT+UPSD command.  R4 relies on AT+CGDCONT to set APN.

If R4 is detected, AT+CGDCONT is used to set the APN. I apologise for the complicated #ifdef logic that makes this less obvious than it could be.

@hwilmers

1. The R4 vs. U2 differences in `pin_init()` are not addressed.  I think in it's current form, this PR would probably break R4.

Yes, I see that there is a problem. But it is the other way around, since with this patch (but only with MODEM_UBLOX_SARA_AUTODETECT_VARIANT set), for both modems the pin_init() is done as though they were R4. I have no better solution for this, and because on my hardware pin_init() is merely ignored, I can not test for this case.

I would also hate to break U2 users.  I don't think we can merge the first patch until we figure out the pin_init changes. :/

2. The second patch might be mergeable as a stand-alone patch, but it only applies to U2 which uses the AT+UPSD command.  R4 relies on AT+CGDCONT to set APN.

If R4 is detected, AT+CGDCONT is used to set the APN. I apologise for the complicated #ifdef logic that makes this less obvious than it could be.

Oh you are correct.  The mdm_apn value is also used for AT+CGDCONT (somehow I missed this initially).  So the 2nd patch can probably be merged.  Do you want to submit that as a separate patch in a new PR or perhaps remove the first patch from here and re-post that patch as a new PR so it can continue to develop?

@hwilmers The first patch is going to need the following:

A U2 user to confirm whatever pin_init changes are working for them
I can validate the R4 pin_init changes on my end

@mike-scott About the first patch: I may be missing something here, but there are no changes in pin_init() at all. If you choose to set MODEM_UBLOX_SARA_U2 or MODEM_UBLOX_SARA_R4, it will behave exactly as before.
But I don't want to force this onto anyone. If you think it is better, I can remove this patch from the PR.

@mike-scott About the first patch: I may be missing something here, but there are no changes in pin_init() at all. If you choose to set MODEM_UBLOX_SARA_U2 or MODEM_UBLOX_SARA_R4, pin_init() will behave exactly as before.
But I don't want to force this onto anyone. If you think it is better, I can remove this patch from the PR.

@hwilmers I should have worded my original response this way:
I wouldn't want to merge the first patch and have users try it with unexpected results.  I would rather merge it when it's ready for all users.
You are correct in that by itself it doesn't actually break anyone.  As long as they don't swap to using MODEM_UBLOX_SARA_AUTODETECT_VARIANT.
I'm curious how you are able to use the first patch as it is now.  You mention that for your hardware pin_init() is ignored?  What HW is that?  If there is a valid user of this patch, then I would be more inclined to merge it now and then work to cleanup pin_init().

I'm curious how you are able to use the first patch as it is now. You mention that for your hardware pin_init() is ignored? What HW is that? If there is a valid user of this patch, then I would be more inclined to merge it now and then work to cleanup pin_init().
I am using cellular XBee modules made by Digi. The hardware init of the modem engines is done by by some middle layer made by Digi, but after hardware init they allow to communicate directly to the modem engines (in this case U201 or R410). This is why pin_init() works identical for both types - it is merely ignored.

@hwilmers if you say that this PR makes the cellular XBee modules made by Digi usable with Zephyr.  Then I'm ok with +1 for merge in 2.3.


Add dynamic diagnostic

Draft for dynamic diagnostics for design discussion.
Implements #259

Do you agree with DynamicDiagnostic name?

How generic should this class be?
I tried using generic types with defaults:
struct DynamicDiagnostic<D = String, C = String, ...>
where
    D: Debug + Display,
    C: Debug + Display,
    ...
{
   pub description: D,
   pub code: Option<C>,
   ...
}

But this was  ambiguous if using None values:
let diag: DynamicDiagnostic<&str, _, ...> = DynamicDiagnostic {
   description: "",
   code: None
};

is cargo readme from #124 not longer working?

is cargo readme from #124 not longer working?

@gavrilikhin-d cargo make readme

looks like you have a clippy thing too

Issues #259 and #201 will be resolved by this pr

Why tests fail on windows? Looks like it can't find local arguments and assumes they are named arguments

I think 1.56.0 is before format argument interpolation was stabilized, now that I think about it

@zkat how can I fix it? See no way to conditionally compile code depending on rust version.
My best guess is to introduce new default feature format_args_capture

Now it will pass workflow (checked in my fork)

Yay! It's in!!


Improved Documentation

This is improved Documentation because your file did not had any comments or documentation. Just a small contribution to the OpenSource community

Hi @UdhavKumar,
We would absolutely welcome useful contributions, but in this case your changes are:

Not useful, and
Syntactically invalid, causing the page to render incorrectly,  and
Will in any case not get you a free T-shirt.

  Maybe you could tackle fixing one of the many bugs that we have in our bug tracker?

Sir sorry for your inconvenience I just wanted to do help to big tech
company like google and I dont want that t-shirt hacktoberfest is much more
like #shittoberfest one more time sorry
On Thu, Oct 15, 2020 at 9:36 PM Christopher Allen<EMAIL_ADDRESS>wrote:

Hi @UdhavKumar https://github.com/UdhavKumar,
We would absolutely welcome useful contributions, but in this case your
changes are:

Not useful, and
Syntactically invalid, causing the page to render incorrectly, and
Will in any case not get you a free T-shirt
https://github.com/digitalocean/hacktoberfest/pull/596.

 Maybe you could tackle
fixing one of the many bugs that we have in our bug tracker
https://github.com/google/CodeCity/issues?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/CodeCity/pull/431#issuecomment-709428207, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOUARAW4JCENRMZEBDZDN3LSK4MWTANCNFSM4SSFNSQA
.


Some Stone Tagged Unbreakable (1.19.2)

I'm not totally sure what the bug's logic is, but when I have origins installed it tags most (not all) stone and stone-adjacent blocks (so far ores are exempt) as unbreakable, no matter what I have equipped. They behave like bedrock where I can swing at them infinitely without them taking damage. I did a mod purge to discover this mod as the source, but I do have 110~ mods installed, so it may have been a compatibility issue somewhere.
I did see some mention of Curios incompatibility and I do have that installed, but upon removing that mod as well it didn't fix the issue.
Minecraft: 1.19.2
Forge: 43.3.5
Origins: <IP_ADDRESS>
Optifine: HD_U_I2
If there's a work around anyone might know that would be great, but I unless there's a fix I'll have to stick to building in creative or playing without the mod.
Thank you for your time!
UPDATE: Upon further testing it appears to be an issue with the feline origin, I'm going to see what I can do to get around/fix it.

Oh man do I feel silly! I went into the config and set weak_arms to "false", which solved my issue. so sorry for the waste of time!


Quick fix for indentation causes lint issue with DCL inside PRocedures

I used the auto-convert to convert an entire program that was mixed format. the D specs were in column format. After conversion, i used quick fix to fix the LINT indent errors, and the DCL statements inside the procedure all were indented at 0 and cause a LINT error. I moved them to position 4 and the LINT message went away

@davidknittle Can you share a sample before you do the quickfix so I can recreate the issue?


Missing some validations for ConfigAPI

What happened:
The kubebuiler or controller-gen markers wouldn't work well since the ConfigAPI is not CRD.
So, we need to implement validations by ourselves, but we don't have some validations at the following APIs:

[ ] MultiKueue: https://github.com/kubernetes-sigs/kueue/blob/5401a3b55c15098795d22d0c61ddae40bdc6bf25/apis/config/v1beta1/configuration_types.go#L205-L225
[ ] InternalCertManagement: https://github.com/kubernetes-sigs/kueue/blob/5401a3b55c15098795d22d0c61ddae40bdc6bf25/apis/config/v1beta1/configuration_types.go#L271-L284
[ ] QueueVisibility: https://github.com/kubernetes-sigs/kueue/blob/5401a3b55c15098795d22d0c61ddae40bdc6bf25/apis/config/v1beta1/configuration_types.go#L335-L345

What you expected to happen:
Once the KueueConfig with invalid parameters is created, the kueue-controller-manager raises the errors and stops working during booting manager.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
Kueue version (use git describe --tags --dirty --always):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/assign


Add Docker publish GitHub Action

Contributor Comments
This introduces invoke tasks and a GitHub Action to publish to Dockerhub.
Pull Request Checklist
Thank you for submitting a contribution to easy_sast!
In order to streamline the review of your contribution we ask that you review and comply with the below requirements:

[ ] Ensure that make build completes successfully
[ ] Keep our reports up-to-date by committing the output of make report
[ ] Rebase your branch against the latest commit of the target branch, which likely should be master
[ ] If there is an issue associated with your Pull Request, align your PR title with this documentation

Opened #48 to replace this


Can you add Caloric Intake?

Can you add this param?
Thanks

Hello @gabamnml, even though the base library tormoder/fit I am using here does support caloric intake, I have not found a solution to show this inside of Garmin Connect after importing.
It seems that Garmin does not display the weight values together with the calories.
It would also help to have a better understanding of your requirements and where you want these values to show up - since I haven't found the right place yet.
Regards,
David

Hello @davidkroell do you see the caloric intake in this weight view?

Thanks

@gabamnml Do you also have a problem with uplod to Garmin: https://github.com/davidkroell/bodycomposition/issues/13
I can't wait to check the functionality from version 2.2.0-rc1 with Xiaomi scale :-)

@gabamnml any updates on this?

Unfortunately it does not work (I compiled a new version of the fix base: https://github.com/abrander/garmin-connect/pull/13)

@RobertWojtowicz I created a new release 2.2.0-rc2 to reflect the changes made in #13.
I know, it does not work for me either. There must be something wrong with a dependent library

The last version I uploaded is the version with an EOF error. I checked the package bodycomposition_2.2.0-rc2_Linux_x86_64.tar.gz
It works on my compilation with fix .

Sorry for the delay @davidkroell but dont work. Here is the output

Hello @davidkroell sorry for the delay. The RC dont work here is the output.
If I put password after the flags

If I send password as a param flag:

@gabamnml As I wrote above, the compiled version includes old version without the patch.
If you are using Linux I suggest you download the correct version from me: https://github.com/RobertWojtowicz/miscale2garmin/blob/master/bodycomposition

It does not show the uploaded calories in Garmin Connect, it's ignored. I used the binary file.
Maybe the "Caloric Intake" value from the "Weight" section is from the "Calories in/Out" section (MyFitnessPal import) and cannot be uploaded?


📢 Deprecating sfpowerkit:project:diff

sfpowerkit diff is no longer being updated as this is being provided by other open source tools such as: https://github.com/scolladon/sfdx-git-delta
This only applies to the sfpowerkit:project:diff and not to the sfpowerkit:project:orgdiff command

@Caitlyn-Mills @azlam-abdulsalam is the deprecation still planned for this year ?

@adrianioana  Yes it is will be  part of December release

@adrianioana Yes it is will be part of December release

Thanks for confirmation.


Update

Latest ruamel.yaml needs this version.

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


Python version compatibility

Hi， Can python 3.8 work on your code？ I need a 3-rd party lib but it has a conflict with python 3.10.

Hi, I don't think there should be a problem with using python 3.8 apart from a few minor changes.


feat(Order/CompleteBooleanAlgebra): Himp in terms of sSup

Himp can be expressed in terms of sSup. Does this fit into mathlib?

There you go:
theorem isGreatest_himp {α} [GeneralizedHeytingAlgebra α] (a b : α) :
    IsGreatest {w | w ⊓ a ≤ b} (a ⇨ b) := by simp [IsGreatest, mem_upperBounds]

theorem isLeast_himp {α} [GeneralizedCoheytingAlgebra α] (a b : α) : IsLeast {w | a ≤ b ⊔ w} (a \ b) := by)
  simp [IsLeast, mem_lowerBounds]

I added these two in the middle:
`lemma isGreatest_sSup {α} [CompleteSemilatticeSup α] (a : α) (s : Set α) :
IsGreatest s a → sSup s = a :=
fun h ↦ IsLUB.sSup_eq (IsGreatest.isLUB h)
lemma isLeast_sInf {α} [CompleteSemilatticeInf α] (a : α) (s : Set α) :
IsLeast s a → sInf s = a :=
fun h ↦ IsGLB.sInf_eq (IsLeast.isGLB h)`
Are they to trivial or should I do a seperate PR with these two (probably in a more sensible location)?


Fix init_max_accept_queue_size

File /proc/sys/net/core/somaxconn ends with a \n, but not \0. So, after calling to strtol, *end would be \n. This results in s_max_accept_queue_size always being SOMAXCONN.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

is there any body to take a look?
@ctiller @murgatroid99

@markdroth @ncteisen

I think @sreecha is the right person to review iomgr changes while Craig's out.

Hmm, I get
$ cat /proc/sys/net/core/somaxconn
128
$ hd /proc/sys/net/core/somaxconn
00000000  31 32 38 0a                                       |128.|
00000004

So I'm seeing a newline terminator; this PR would break for me.

Jenkins, this is ok to test

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

@vjpai If this PR is OK, please help to take a look at the UT failures.

Hi, sorry to have let this sit unmerged for a while. Unfortunately, conflicts have arisen because of the recent C-to-C++ change. Would you like to rebase with the current master and resolve conflicts? Thanks.

@vjpai rebasing done.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

@vjpai Please merge ASAP, in case of conflicts.

Thank you for your pull request. Before we can look at your contribution, we need to ensure all contributors are covered by a Contributor License Agreement.
After the following items are addressed, please respond with a new comment here, and the automated system will re-verify.

User @dutor isn't covered by a CLA. They will need to complete the form at https://identity.linuxfoundation.org/projects/cncf

Regards,
The Linux Foundation CLA GitHub bot

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

done


chore) remove vendored files; require sync

Current coverage is 20.54% (diff: 100%)

No coverage report found for master at a6c14a9.

Powered by Codecov. Last update a6c14a9...dbe4509

approved


(BKR-542) remove hosts object from log about execution against empty …

…host list

Can one of the admins verify this patch?

@pondohva, thanks for submitting this patch!
while I agree that printing block_hosts in the empty array case doesn't look useful, I like having the hosts array printed to get a reminder of what was cut.  Why cut hosts as well?

@kevpl I was too lazy to do that. Ok, I'll look into hosts dump

@pondohva I was actually referring to not cutting as much, no additional work needed.  So that string would become something like:
against an empty array of hosts (filtered from #{hosts})

I think that we can keep the hosts dump, but change the message to an info?

Well, finally fix it, sorry for delay. BTW, I commented to jira with loooong logger string

@pondohva ok, you definitely get different output than we do.  Rather than creating the array implicitly, you could make it explicit with something like this:
hosts.map { |h| h.name }

If you do decide to continue fixing this, please make sure to add a spec test as well.
Thanks,
Kevin

@kevpl sure, I'll try to fix this as soon as I can. Could you merge this PR with severity decreasing, please? I'd like to see it in next beaker release :)

@puppetlabs-jenkins test this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/1675/
Test PASSed.


Error:  MAX_CLIENTS=6

This shows up in my error log.  Some time recently Harmony increased the number of devices to 15 from 6.  Can this be increased?
But even at that when I click GET hubs in the web UI it shows me 6 hubs in the JSON output.  But on the web page it only lists 5 of the 5 hubs.

One interesting thing to note is that I do not have this error when using harmony-api under an Ubuntu VM but I do have the error when running harmony-api as a docker in unRAID.

Here is more detail on this error from the log:
`events.js:141
throw er; // Unhandled 'error' event
^
Error: MAX_CLIENTS=6
at Connection.onStanza (/usr/src/app/node_modules/harmonyhubjs-client/node_modules/node-xmpp-client/node_modules/node-xmpp-core/lib/Connection.js:355:17)
at StreamParser. (/usr/src/app/node_modules/harmonyhubjs-client/node_modules/node-xmpp-client/node_modules/node-xmpp-core/lib/Connection.js:226:10)
at emitOne (events.js:77:13)
at StreamParser.emit (events.js:169:7)
at SaxLtx. (/usr/src/app/node_modules/harmonyhubjs-client/node_modules/node-xmpp-client/node_modules/node-xmpp-core/lib/StreamParser.js:58:14)
at emitOne (events.js:77:13)
at SaxLtx.emit (events.js:169:7)
at SaxLtx._handleTagOpening (/usr/src/app/node_modules/harmonyhubjs-client/node_modules/ltx/lib/parsers/ltx.js:31:18)
at SaxLtx.write (/usr/src/app/node_modules/harmonyhubjs-client/node_modules/ltx/lib/parsers/ltx.js:105:26)
at StreamParser.write (/usr/src/app/node_modules/harmonyhubjs-client/node_modules/node-xmpp-client/node_modules/node-xmpp-core/lib/StreamParser.js:123:17)
at Connection.onData (/usr/src/app/node_modules/harmonyhubjs-client/node_modules/node-xmpp-client/node_modules/node-xmpp-core/lib/Connection.js:305:17)
npm info<EMAIL_ADDRESS>Failed to exec start script
npm ERR! Linux 4.9.8-unRAID
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
npm ERR! node v4.8.4
npm ERR! npm v2.15.11
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: node app.js
npm ERR! Exit status 1
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node app.js'.
npm ERR! This is most likely a problem with the harmony-api package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR! node app.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR! npm bugs harmony-api
npm ERR! Or if that isn't available, you can get their info via:
npm ERR! npm owner ls harmony-api
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR! /usr/src/app/npm-debug.log`
But if I try to start again manually it works and this is what is in the log:
`npm info it worked if it ends with ok
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm info prestart<EMAIL_ADDRESS>npm info start<EMAIL_ADDRESS>
<EMAIL_ADDRESS>start /usr/src/app
node app.js

Starting discovery.
Starting discovery.
Hub discovered: Office at <IP_ADDRESS>.
Hub discovered: Bedroom at <IP_ADDRESS>.
Hub discovered: TV Room at <IP_ADDRESS>.`

One more note:  This error is mentioned in the harmonyhubjs-client repository at:
https://github.com/swissmanu/harmonyhubjs-client/issues/34


Allow get on id arrays

Is there plan for supporting get on id arrays?
conversations?ids[]=12&ids[]=14

You can now (v0.7.27) ;)
conversations?id=12&id=14


Set network namespace in the container

From all the namespaces specified in the container spec, youki only sets the user and pid namespaces.
Other namespace, like the nerwork namespace, are ignored.
See https://github.com/containerd/runwasi/pull/364 and this slack thread for context.

Thanks for creating the issue. Why did they think that? I think youki apply the rest of the namespace here:
https://github.com/containers/youki/blob/a6735b1c7e055d68f263624de767e952279fb5bc/crates/libcontainer/src/process/container_init_process.rs#L299

You are 100% correct.
Youki is setting the network namespace, but for some reason it's not taking effect.
It might be related to what point in the container lifecycle the call is done, maybe the runwasi shim needs to create the container earlier.
I need to dig dipper into this.

Pods share a network namespace among the containers, right? If it is right, perhaps runwasi is responsible for creating a network namespace for a pod. But I'm not sure...

Pods share a network namespace among the containers, right? If it is right, perhaps runwasi is responsible for creating a network namespace for a pod. But I'm not sure...

This is how containerd-shim works. The network namespace is created upfront and the OCI runtime (runc in this case) will join the network namespace. youki implements all namespaces, both create and join existing. I did not dig into the containerd/runwasi codebase, but the idea would be for the containerd level to create the network namespace upfront, and pass the path of the network ns to youki as part of the runtime spec. youki should then join the network namespace correctly. Let me know if there is any issue.

Close this issue since youki does correctly setup the network namespace.


Pre-made Effects Missing

The are no pre-made effects available. I ran "find the culprit". With all other modules except for dependencies deactivated, the issue remain. Also did this manually with only one other dependent module activated. The problem still persist.

Created a sandbox world with only DFreds convenient effects module and it worked fine.
Foundry v12 build 331
D&D v3.3.1
No related error found in the console.

I have the same issue, somehow strange dfreds effects appeared as items in the Items pannel, I deleted them all and afterwards I noticed that there are no dfreds effects at all.
I disabled the module, uninstall, install, enable, still the same issue. It seems the issue it's with the  world because on other world I can see the effects.

I'm not quite sure why this has been happening. Maybe related to refreshing the game world too early or something. Basically, on startup, it creates all the effects once. But anyway, run game.dfreds.effectInterface.resetSystemInitialization() in the console (F12).
Next version of the module will have a button that does it for you

Thanks, mate. That worked for me. Was about to export/import them from my sandbox.

It worked. Thanks!


[9.x] Use provided table name as foreign key if present

TL;DR — getForeignKey() method does not respect the $table property of model. To improve this, we use the power of getTable() method, also it has fallback support for the current approach ( resolving from class basename ).
Why?
Because, when you need to use a specific table name, you have to manually type the actual foreign key in each relation. I thought this make things more complicated, so I came up with this solution.
What’s wrong about with tests?
As you can see from the tests, relation columns have named based on its models base class name, but at the same time, these models have a specific table name. As a result, column names would not match the provided table name.
To fix this, we could add the foreign keys to relations, but that's not the point of these tests, so I have decided to change these with proper convention as it should be.
Example
class Reservation extends Model
{
  protected $table = 'flight_reservations';

  public function passengers()
  {
-    return $this->belongsToMany(Passenger::class, null, 'flight_reservation_id');
+    return $this->belongsToMany(Passenger::class);
  }
}

This is probably breaking change for users who don’t follow proper naming conventions, but I believe that most people will get rid of these unnecessary definitions.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Exact Change Problem Issue

Challenge Name

https://www.freecodecamp.com/challenges/exact-change
Issue Description

I had problem with this problem due to inaccurate calculations while working with "0.01" value. Sometimes returning value of calculation had wrong value after 7 or 8 decimal places (for example 1.01 - 0.01 = 0.999999996).
Did anyone else had similar problem?
I managed to solve it but still took me a lot of time to do it due to this issue that shouldn't happen... Anyone have any idea why it is so. My opinion is that it is due to way that js saves integer and float values...
Browser Information

Browser Name, Version: Google Chrome, 52.0.2743.82 (64-bit)
Operating System: Linux Ubuntu 16.04
Mobile, Desktop, or Tablet: Laptop

Your Code

Screenshot

Hi, @FatKoala. You can read all about floating point arithmetic (and good ways to handle it) here and other places online.
And a fun fact: JavaScript doesn't make a type distinction between floats and integers. 1 and 1.00000 are the same number in JS (though this isn't strictly the cause of the floating point precision errors you experienced).
It might be worthwhile to include a resource about floating point arithmetic (though not necessarily the one I linked to above) in the Helpful Links of this challenge as it is probably the first one which requires users to understand the precision issues it raises.
\cc @FreeCodeCamp/issue-moderators for feedback on adding such a link (and which link to add)

Yeah this is a very interesting topic and should be broached IMHO.
In JavaScript there is just on numeric Data type that is number and it's usually a 64 bit floating point (considering how Browsers (JS Engines to be precise) are implementing them).
Let's tag this as Help wanted for some one in the community to link up a good resource. I'll try and look up as well, but yeah it's better if it comes from someone who perhaps is a Language expert.

Thanks for the link, it was helpful... got really frustrated about problem while I was coding... now it has sense... I agree that it should be linked somewhere or explained earlier that those things happen...

I poked around a bit and @BKinahan's link seems to be at least as good if not better than most and seems to be a straight-forward site. I vote we use his link. I'm happy to do the PR unless someone else wants to.

@dhcodes please go ahead, :wink:


Exception when running flutter upgrade

Just ran flutter upgrade inside the gallery:
Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter upgrade
exception
The null object does not have a getter 'length'.


Use protocol types in vscode extension

This is a blocker for the release. Is this now feature-complete/ready to merge?


Orcasound extra .ts files in bucket

While working on Orcasound data I've noticed that buckets contain some "extra" files if you can call them so.
Examples:

last file in bush_point/1624170621 is live2158.ts while live.m3u8 lists live2150.ts as the last one.
similar with bush_point/1624192222, last one is live2158.ts while live.m3u8 lists live2151.ts as the last one.

The problem with these very last files (live2158.ts) is that they are empty (0 bytes). Honestly only noticed them because ffmpeg failed when processing them, I wasn't even looking at live.m3u8 before.
Files live2151-2157 seem to be valid ones.
@paulcretu, I think you can help with this/did the setup of this? For now I just drop the last file (will probably read only files listed in the playlist eventually) but we probably shouldn't store these files at all or should include them in .m3u8 (except the last one)?

Good find! The reason these streams end is because the node gets shut down, so I assume this has something to do with that, especially that very last file being empty.
I'm not sure why the last 8 files aren't in the m3u8. I did set up the streaming, but that was a while ago and @mcshicks has made a lot of changes since then. He wrote this python script to handle uploads, but it's not obvious to me looking at it why it would cause the missing entries. I know the older versions of the code used to intentionally cut out the last few files, it seems likely that Bush Point is still running the older code.
So for now @Molkree I agree with just dropping the last file as you're doing. @scottveirs or @mcshicks do you know what version of the code Bush Point is currently running? I think figuring that out is the next step here.

I'm pretty sure it's pre upload script.  I think upgrading to a more recent versions was one of @scottveirs goals for his hydrophone updates, which was why we wanted to get a working image in docker hub.  But I since it's only a few files at the end it certainly seems like it's probably related to the cron restart.  I'm not sure it won't happen with the new script since we still I think we are still restarting, but I think since we guarantee in order writes i'm suspecting that ffmpeg closes the .ts file before doing updating the m3u8.  With the s3fs upload we can't guarantee the uploads are in order.

For now this was solved on my end by reading playlist .m3u8 file and processing only files in it.


chore: ScalaStyle fixes

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Fix all existing scalastyle warnings and make them errors for the future.
Since we use scalastyle as a gate, having a thousand warnings in the output makes it cluttered and annoying to find errors. We should make everything an error and just require annotations to suppress.
I also improved some error messages and relaxed some length requirements a little.
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ x] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

/azp run

Codecov Report

Merging #1716 (88a9e80) into master (f2e88fd) will decrease coverage by 78.37%.
The diff coverage is 9.72%.

@@            Coverage Diff             @@
##           master   #1716       +/-   ##
==========================================
- Coverage   85.69%   7.32%   -78.38%
==========================================
  Files         277     277
  Lines       14495   14546       +51
  Branches      777     768        -9
==========================================
- Hits        12422    1065    -11357
- Misses       2073   13481    +11408

Impacted Files
Coverage Δ

...soft/azure/synapse/ml/cognitive/AudioStreams.scala
0.00% <0.00%> (-87.88%)
:arrow_down:

...osoft/azure/synapse/ml/cognitive/AzureSearch.scala
0.00% <0.00%> (-87.77%)
:arrow_down:

...ft/azure/synapse/ml/cognitive/AzureSearchAPI.scala
0.00% <0.00%> (-89.62%)
:arrow_down:

...t/azure/synapse/ml/cognitive/BingImageSearch.scala
0.00% <ø> (-89.29%)
:arrow_down:

...zure/synapse/ml/cognitive/DocumentTranslator.scala
0.00% <0.00%> (-19.45%)
:arrow_down:

...se/ml/cognitive/MultivariateAnomalyDetection.scala
0.00% <0.00%> (-89.52%)
:arrow_down:

...azure/synapse/ml/cognitive/SlimStorageClient.scala
0.00% <0.00%> (-44.74%)
:arrow_down:

...crosoft/azure/synapse/ml/cognitive/SpeechAPI.scala
0.00% <0.00%> (-70.00%)
:arrow_down:

...oft/azure/synapse/ml/cognitive/SpeechSchemas.scala
0.00% <ø> (-100.00%)
:arrow_down:

...t/azure/synapse/ml/cognitive/SpeechToTextSDK.scala
0.00% <0.00%> (-91.38%)
:arrow_down:

... and 319 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

/azp run

/azp run


how can i use this to recognition chinese word in image?

Do  you test to use this to  recognition chinese word in image?

Hey @Jayhello , I haven't tested it on Chinese words. Could you specify the problem you have encountered?

thank you fo reply !!
To do chinese words recognition in the image.
my question is ：
chinese words is about 3K, and more complicated than English words. can this be possible ( with high a accuracy)

I may need lots of samples , i don't know how to train and define network
how to prepare training images , for example the character " 你" in chinese, how many image about this
char to prepare, and the image form ( the image i want to recognize which has a complicated background,like the image below . and how the trained image size is provided)

You can synthesize images to train your network. Synthesizing images like the example one shouldn't be hard. You can use blend random natural images with text rendered with random color, font, etc.

@bgshih
Thank very much for you reply.
And I also want to ask you
The image you train is gray and the width is 100, and you use 0~9 a~z A~Z in utilities,and DatasetLmdb.lua file.
how to change these file for RGB image and the chinese words?

https://github.com/bear63/sceneReco

@Jayhello hello, have you successfully trained the model for Chinese characters?

@Jayhello @LiangHao92 have you successfully trained the model for chinese characters?


[FIX] harmonize 'file name' into 'filename'

Text is using both 'file name' and 'filename' pretty much to the equal amount
ATM (see git grep outputs below).  Code uses 'filename', and wikipedia has
https://en.wikipedia.org/wiki/Filename and prefers to use 'filename' in it. So
I decided to harmonize  into  'filename'.
$> git grep  'file name' | grep '\.md' | grep -v MACRO | nl
	 1	src/02-common-principles.md:1.  **File extension** - a portion of the file name after the left-most
	 2	src/02-common-principles.md:are compulsory. For example a particular file name format is required when
	 3	src/02-common-principles.md:saved under a particular file name specified in the standard. This standard
	 4	src/02-common-principles.md:A file name consists of a chain of *entities*, or key-value pairs, a *suffix* and an
	 5	src/02-common-principles.md:`subject`, the file name MUST begin with the string `sub-<label>_ses-<label>`.
	 6	src/02-common-principles.md:If the `session` level is omitted in the folder structure, the file name MUST begin
	 7	src/02-common-principles.md:key/value pair MUST also be included as part of the file names themselves.
	 8	src/02-common-principles.md:produces a human readable file name, such as `sub-01_task-rest_eeg.edf`.
	 9	src/02-common-principles.md:It is evident from the file name alone that the file contains resting state
	10	src/02-common-principles.md:Entities within a file name MUST be unique.
	11	src/02-common-principles.md:For example, the following file name is not valid because it uses the `acq`
	12	src/02-common-principles.md:label, but must be included in file names (similarly to other key names).
	13	src/02-common-principles.md:meaning of file names and setting requirements on their contents or metadata.
	14	src/02-common-principles.md:to suppress warnings or provide interpretations of your file names.
	15	src/03-modality-agnostic-files.md:This file is REQUIRED if `sample-<label>` is present in any file name within the dataset.
	16	src/04-modality-specific-files/05-task-events.md:Where `<matches>` corresponds to task file name. For example:
	17	src/04-modality-specific-files/06-physiological-and-other-continuous-recordings.md:In the template file names, the `<matches>` part corresponds to task file name
	18	src/05-derivatives/02-common-data-types.md:is used to prevent clashing with the original file name.
	19	src/06-longitudinal-and-multi-site-studies.md:and [file names](02-common-principles.md#file-name-structure)
	20	src/99-appendices/03-hed.md:screen or the file name of the stimulus image.
	21	src/99-appendices/03-hed.md:       "LongName": "Stimulus file name",
	22	src/99-appendices/04-entity-table.md:[file name structure](../02-common-principles.md#file-name-structure),
	23	src/99-appendices/06-meg-file-formats.md:that not only the file names, but also the internal file pointers will be
	24	src/99-appendices/09-entities.md:[file name structure](../02-common-principles.md#file-name-structure).
	25	src/CHANGES.md:-   \[FIX] Clarify use of session entity in file names [#532](https://github.com/bids-standard/bids-specification/pull/532) ([Moo-Marc](https://github.com/Moo-Marc))
	26	src/CHANGES.md:-   \[FIX] Specify marker file names for KIT data (MEG) [#62](https://github.com/bids-standard/bids-specification/pull/62) ([monkeyman192](https://github.com/monkeyman192))
	27	src/CHANGES.md:-   Added missing `sub-<participant_label>` in behavioral data file names.
	28	src/pregh-changes.md:-   Added missing `sub-<participant_label>` in behavioral data file names.

$> git grep 'filename' | grep '\.md' | grep -v MACRO | nl
	 1	CONTRIBUTING.md:Make sure that all filename format templates, entity tables, and entity definitions are correct
	 2	src/02-common-principles.md:(with the same filename as the `.nii[.gz]` file, but with a `.json` extension).
	 3	src/03-modality-agnostic-files.md:      "filename": ("REQUIRED", "There MUST be exactly one row for each file."),
	 4	src/03-modality-agnostic-files.md:filename	acq_time
	 5	src/04-modality-specific-files/02-magnetoencephalography.md:which saves the MEG sensor coil positions in a separate file with two possible filename extensions  (`.sqd`, `.mrk`).
	 6	src/05-derivatives/01-introduction.md:    status. Any modification of raw files must use a modified filename that does
	 7	src/05-derivatives/01-introduction.md:    not conflict with the raw filename. Further, any files created as part of a
	 8	src/05-derivatives/01-introduction.md:    derivative dataset must not match a permissible filename of a valid raw
	 9	src/05-derivatives/01-introduction.md:    dataset. Stated equivalently, if any filename in a derivative dataset has a
	10	src/05-derivatives/01-introduction.md:-   Each Derivatives filename MUST be of the form:
	11	src/05-derivatives/01-introduction.md:    `source_entities` MUST be the entire source filename, with the omission of
	12	src/05-derivatives/01-introduction.md:    the source suffix and extension. One exception to this rule is filename
	13	src/05-derivatives/01-introduction.md:-   There is no prohibition against identical filenames in different derived
	14	src/05-derivatives/03-imaging.md:filename.
	15	src/99-appendices/04-entity-table.md:specification, and establishes a common order within a filename.
	16	src/99-appendices/08-coordinate-systems.md:The `scanner` coordinate system is implicit and assumed by default if the derivative filename does not define **any** `space-<label>`.
	17	src/99-appendices/11-qmri.md:filenames will remain the same; however, the optional metadata (third column) may
	18	src/CHANGES.md:-   \[SCHEMA] Use macro for filename templates in file collections appendix [#787](https://github.com/bids-standard/bids-specification/pull/787) ([tsalo](https://github.com/tsalo))
	19	src/CHANGES.md:-   \[FIX] Accidentally swapped Neuromag/Elekta/MEGIN cross-talk & fine-calibration filename extensions [#621](https://github.com/bids-standard/bids-specification/pull/621) ([hoechenberger](https://github.com/hoechenberger))
	20	src/CHANGES.md:-   \[INFRA] SCHEMA: Declare entities by concept names, add entity field for filename components [#616](https://github.com/bids-standard/bids-specification/pull/616) ([effigies](https://github.com/effigies))
	21	src/CHANGES.md:-   \[FIX] Common principles: Fix filename in inheritance principle [#261](https://github.com/bids-standard/bids-specification/pull/261) ([Lestropie](https://github.com/Lestropie))
	22	src/CHANGES.md:-   \[FIX] Example for IntendedFor was missing session indicator in the filename [#129](https://github.com/bids-standard/bids-specification/pull/129) ([yarikoptic](https://github.com/yarikoptic))
	23	src/schema/README.md:the entity tables, entity definitions, filename templates, and metadata tables.
	24	src/schema/README.md:-   `entities.yaml`: Entities (key/value pairs in folder and filenames).
	25	src/schema/README.md:This file contains a dictionary in which each entity (key/value pair in filenames) is defined.
	26	src/schema/README.md:they appear in filenames _and_ their full names.
	27	src/schema/README.md:For example, the key for the "Contrast Enhancing Agent" entity, which appears in filenames as `ce-<label>`,
	28	src/schema/README.md:since many entities (such as `ce`) have very short filename elements.
	29	src/schema/README.md:The `entity` field is the entity as it appears in filenames. For example, the `entity` for `ceagent` is `ce`.
	30	src/schema/README.md:Given that all entities appear in filenames, they should all be strings and the `type` field should always be `string`.
	31	src/schema/README.md:For example, `run` should have an index, so a valid key-value pair in a filename would be `run-01`.
	32	src/schema/README.md:Keys are the filenames (without file extensions),
	33	src/schema/README.md:-   `datatypes/*.yaml`: Files in the `datatypes` folder contain information about valid filenames within a given datatype.
	34	src/schema/README.md:    Each dictionary contains a list of suffixes, entities, and file extensions which may constitute a valid BIDS filename.
	35	src/schema/README.md:-   `entities.yaml`: This file simply defines the order in which entities, when present, MUST appear in filenames.
	36	src/schema/README.md:Each dictionary corresponds to a group of suffixes that have the same rules regarding filenames.
	37	src/schema/README.md:**NOTE**: The order in which entities appear in these dictionaries does not reflect how they should appear in filenames.
	38	src/schema/README.md:This file contains a list of entities in the order in which they must appear in filenames.

If we don't want them reverted, we'll need to edit the pr titles.

If we don't want them reverted, we'll need to edit the pr titles.

I just did that

Thanks @yarikoptic


Add option to run Circuitbreaker for each LND Node

Add the ability to run a custom version of circuitbreaker alongside each LND node in warnet. Circuitbreaker can be run using the following command:
circuitbreaker --network=regtest --rpcserver={lnd host:lnd port} --tlscertpath={path to LND cert} --macaroonpath={path to LND admin.macaroon}
For my purposes, I'll need to be able to copy in pre-populated databases into the circuitbreaker containers, so it would be very helpful to be able to provide a path to the database that needs to be mounted. This database needs to be in place before circuitbreaker starts (otherwise it'll create its own, empty database), not sure how that works in a kubernetes world.
The database is located at ~/circuitbreaker/circuitbreaker.db in the using AppDataDir to determine the home directory. This base directory can also be customized with --configdir if required.

Hey @carlaKC can you confirm that this linked circuitbreaker the exact version you want to be running in warnet?

Hey @carlaKC can you confirm that this linked circuitbreaker the exact version you want to be running in warnet?

This is the version I'd like to run, but will it be difficult to update? Totally possible that I need to do a bugfix between now and running the simulations as we test things out.

Ah no that will be fine.
I was just more wondering "are they using some custom patched version of this, that we need to build docker images for dynamically, or can I rely on the existing images". It sounds like we should be able to rely on pulling a pre-built image from a registry though.

Gotcha, you pull from carlakirkcohen/circuitbreaker:endorsement-experiment. Definitely don't need to worry about building docker images.


BUGFIX: Support get arguments in API request

This change allow to call ElasticSearch with get arguments:
$this->getIndex()->request('GET', '/_search/scroll?scroll=1m');

This is required because some ElasticSearch API argument are only
available as GET request arguments.

@dfeyer, @daniellienert, @kitsunet: can we merge this? Else we will need to use our fork again just for that feature... We need to pass query_type=count so that the query_cache can be used (ES 1.7)...

Merged the PR with previous votes.


feat: use href for buttons

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


sensor noise parameter control dialog

I implemented the sensor noise parameter control dialog for flight test and analysis.
https://youtu.be/5V67fofzN9s
The idea was inspired by @hamishwillee while add random walk noise for gps in this simulation.
(refer to https://github.com/PX4/jMAVSim/pull/64)
I hope that it is useful to analyze and understand the px4 control code.
please check it @bkueng and @TSC21
Thanks in advance.
SungTae Moon

@stmoon is there any way you could actually integrate this with the UI of jMAVSim, instead of having a new window for it?

If you think it is better than now, I will try it again.

I think it will look better but if it is too much work you don't have to spend time on it. As is, is more than enough.

Thanks @TSC21
Actually, I think it spend some time. How about using the current version ?
(However I will have plan to insert the dialog to main UI for next time)

Let's wait for @bkueng review.

Thanks @TSC21

Based on advice of @TSC21, I integrated the sensor parameter control panel with the main UI of jMAVSim (not dialog)
the shortcut key is same ('d' key)
Thanks for your advice of @TSC21  ;-)

@stmoon can you share us a new video with it working? Thanks

please check the video
https://youtu.be/e3Tppt9nteM
Thanks.

@bkueng,  please check it


categorise and reformat UG

Codecov Report
Base: 51.77% // Head: 55.08% // Increases project coverage by +3.30% :tada:

Coverage data is based on head (4a6d617) compared to base (246b87e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #230      +/-   ##
============================================
+ Coverage     51.77%   55.08%   +3.30%
- Complexity      568      601      +33
============================================
  Files           129      129
  Lines          2416     2447      +31
  Branches        310      316       +6
============================================
+ Hits           1251     1348      +97
+ Misses         1078      994      -84
- Partials         87      105      +18

Impacted Files
Coverage Δ

.../address/logic/commands/buyer/AddBuyerCommand.java
100.00% <ø> (ø)

...dress/logic/commands/buyer/DeleteBuyerCommand.java
100.00% <ø> (ø)

...address/logic/commands/buyer/EditBuyerCommand.java
92.94% <ø> (ø)

...ress/logic/commands/buyer/FilterBuyersCommand.java
0.00% <ø> (ø)

...ddress/logic/commands/buyer/FindBuyersCommand.java
55.55% <ø> (ø)

...ddress/logic/commands/buyer/ListBuyersCommand.java
25.00% <ø> (ø)

...ddress/logic/commands/buyer/MatchBuyerCommand.java
0.00% <ø> (ø)

...ddress/logic/commands/buyer/SortBuyersCommand.java
0.00% <ø> (ø)

...ss/logic/commands/property/AddPropertyCommand.java
41.66% <ø> (ø)

...logic/commands/property/DeletePropertyCommand.java
100.00% <ø> (ø)

... and 41 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Clicking 'Save' a second time after adding a new menu item in the UI builder results in an error.

@eugeneingram new version that contains fix has been released as 1.0.0-beta.17. Test an close the ticket please.

Thanks, 1.0.0-beta.17 fixes the issue.


LMDB OOM's for a Large Dataset

Current Behavior
Currently exploring if we can get one of the SAIL implementations to scale to the use cases we have (>= single-digit Billion Triples in some cases. The LMDB SAIL seems like it may be able to handle this (https://github.com/eclipse-rdf4j/rdf4j/discussions/3706#discussioncomment-2285945); However, I am getting an OOM error on some (not all queries).
More specifically we are using the SP2B benchmark to test this: https://dbis.informatik.uni-freiburg.de/forschung/projekte/SP2B/ using the bundled generator to populate the store.
The query that we first ran into was Q2:
PREFIX rdf:     <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
PREFIX rdfs:    <http://www.w3.org/2000/01/rdf-schema#>
PREFIX swrc:    <http://swrc.ontoware.org/ontology#>
PREFIX foaf:    <http://xmlns.com/foaf/0.1/>
PREFIX bench:   <http://localhost/vocabulary/bench/>
PREFIX dc:      <http://purl.org/dc/elements/1.1/>
PREFIX dcterms: <http://purl.org/dc/terms/>

SELECT ?inproc ?author ?booktitle ?title ?proc ?ee ?page ?url ?yr ?abstract
WHERE {
	?inproc rdf:type bench:Inproceedings .
	?inproc dc:creator ?author .
	?inproc bench:booktitle ?booktitle .
	?inproc dc:title ?title .
	?inproc dcterms:partOf ?proc .
	?inproc rdfs:seeAlso ?ee .
	?inproc swrc:pages ?page .
	?inproc foaf:homepage ?url .
	?inproc dcterms:issued ?yr
	OPTIONAL {
	     ?inproc bench:abstract ?abstract
	}
}
ORDER BY ?yr

Initially I ran this with the default JVM heap etc. and it OOM'd after a period of time. I whacked up the heap space to 48G on my 96G machine and it hasn't OOM'd so far.
Expected Behavior
Given the iterator design I would've expected that the query may be slow but shouldn't OOM during evaluation, is that understanding not correct?
Steps To Reproduce

Generate and load LMDB Store with 1 Billlion dataset using SP2B: https://dbis.informatik.uni-freiburg.de/forschung/projekte/SP2B/
Run SP2B Q2 over dataset using default JVM settings

Version
4.3.11
Are you interested in contributing a solution yourself?
Perhaps?
Anything else?
The store was able to load 1 Billion on my machine in ~5.5-6 hrs using write-batches of around a 1000 triples which was really nice!

The current Implementation in 4.x.x is experimental and I've made some fixes and enhancements in the develop branch.
Those could be backported to 4.x.x but I don't know the correct procedure for doing this.
@hmottestad Could you give some advice here?

You can create a PR with the fixes you want to backport and we can merge them into main. If the code ends up being identical between main and develop then there shouldn't be any problems. If not then we will need to be a bit more careful when merging main into develop later.

@benherber How do you execute the queries? Are all results materialized?

@benherber How do you execute the queries? Are all results materialized?

I just iterate through the result set, counting the number of triples:
try (final TupleQueryResult res = query.evaluate()) {
	for (final BindingSet set : res) {
		for (final var ignore : set) {
			count++;
		}
	}
}

OK, that looks good. Could you investigate the memory usage with VisualVM while running the query?

OK, that looks good. Could you investigate the memory usage with VisualVM while running the query?

Yea I'll probably get to doing that later this week if I get the chance. Will update once I do


Is Momentum dead?

Hey guys, thanks for your work! The package hasn't seen updates for 2 years, should it be considered dead or are there any development plans?
Best,
Kris

Hey @krishaamer -- we aren't developing it (although it seems to work pretty well as is). If you'd like to take it over, that'd be great.


npm clean-install: Execution aborted, a subcommand failed!

MAMP PRO 5.3
OSX 10.11.6
PHP 7.2.14
node v10.15.1
Xcode 8.2.1

I followed these install instructions: https://docs.shopware.com/en/shopware-platform-dev-en/getting-started/system-installation-guides/mac-os-x. An installation (guided by the quick install guide) worked for me (docker installation)

The clean-install command is available since NPM v.6.5 as described here.
Probably you need to update your npm version.
So what NPM version are you using?

I ran into the same error. A quick update of npm to the newest version did the trick for me.

Thanks for verifying, we'll update the installation guide accordingly.

NPM v6.5 fixed this error for me, thank you!

We'll have to up the required npm version in our package.json, thanks for pointing it out


cant level up

when i create a new char it says max level at 1, old char are now stuck at there level aswell, im on the 18-09-2016 version 0055CFI1
i tested some more its only in the dail-world001.map , the other 2 are working normal

I am also having the same issue, I can level in the other two custom maps but in the regular world my character is at max level

the problem is somewhere in the dail.arz in the mods-dail-database but cant seem to figure it out hope this somewhat helps

I also have the same problem since the update 55

Just for completed : i have selectionned option B for installation and test X and i'ts correct level up
(sorry for my english)

And here I thought 54 was the most messed up version to date. Now we're stuck at level 1 in v55. Oh well, gonna try reinstalling mode other than B to see if it helps, but I'm not too optimistic..

Tried reinstaling with X mode - game crashes to desktop after hitting START button. Wonderful.

Tried leveling up the Voidcaller in X's Survival mode (aka the only thing that I managed to launch), but the game crashed after I used voidcaller's charge skill on Warden Krieg in 10th round..

Seems to be a problem with the (S) install option, i did (ASX) and i can lvl on the first world mod.

Same problem. B option. v0055CFI1

TheDerpest, how do you fix it? Cant level up on last 55HF2 version...


Integrate with sequel migrations

Currently rom-sql provides no integration with any migration mechanism, hence you need to use either AR's migrations (which works just fine but is obviously not a final solution) or plug in sequel migrations, it's probably a good idea to extract sequel migrations part from sequel-rails and integrate it with rom-sql.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

Will take care of it, or try ;] you can check progress: https://github.com/gotar/rom-sql/tree/%23130-migrations

@gotar w00t!

i think we can close this ticket @solnic

Right!!


opensora/utils/ckpt_utils.py does not have OpenSora 1.1 in pretrained_models

How do we specify the --ckpt-path argument to run the inference script for opensora 1.1?

for all who want to load opensora 1.1 model, please just add the following lines into https://github.com/hpcaitech/Open-Sora/blob/e836befc0810c627413fdb8abd400e10b00c9eac/opensora/utils/ckpt_utils.py#L264
elif ckpt_path.endswith(".safetensors"):
    from safetensors.torch import load_file
    state_dict = load_file(ckpt_path)
    missing_keys, unexpected_keys = model.load_state_dict(state_dict, strict=False)
    print(f"Missing keys: {missing_keys}")
    print(f"Unexpected keys: {unexpected_keys}")

for all who want to load opensora 1.1 model, please just add the following lines into
https://github.com/hpcaitech/Open-Sora/blob/e836befc0810c627413fdb8abd400e10b00c9eac/opensora/utils/ckpt_utils.py#L264
elif ckpt_path.endswith(".safetensors"):
    from safetensors.torch import load_file
    state_dict = load_file(ckpt_path)
    missing_keys, unexpected_keys = model.load_state_dict(state_dict, strict=False)
    print(f"Missing keys: {missing_keys}")
    print(f"Unexpected keys: {unexpected_keys}")

Thank you, works now

We have added now.


thomasgauvin: First draft at KV tutorial to serve static & dynamic re…

…sponses from KV assets

Love this use-case of KV, but this is essentially Workers Sites, right? But handling the assets very manually, without the nice kv-asset-handler wrapper?
https://developers.cloudflare.com/workers/configuration/sites/
Is this something intended for folks to still be doing? What about Cloudflare Pages? Or the --assets binding within wrangler?

@Cherry, yep, this is essentially Workers Sites (some of the code in here is taken from Workers Sites & KV assets handler code). The intention is to show directly how KV can be used (without the KV assets handler abstraction) directly. It serves as a tutorial for a common use of KV to make it more easily discoverable from KV docs.
There is a call-out at the top of https://e37d08d7.cloudflare-docs-7ou.pages.dev/kv/tutorials/workers-kv-to-serve-assets/ that indicates that Pages is the platform to use for serving assets of web applications.
This tutorial demonstrates how to access data and assets from KV and use them to generate static and dynamic responses. If you want to serve static assets of a frontend web application, consider using [Cloudflare Pages](https://e37d08d7.cloudflare-docs-7ou.pages.dev/pages/) which provides a purpose-built experience for web applications with built-in environments, CI/CD integration and more.

I'll make sure to align with the roadmap for Workers Sites. It was also my intent to include https://github.com/cloudflare/workers-sdk/tree/85a22faa75c9d33e7790aa78e49e5c18982a179f/packages/kv-asset-handler as well, I'll make sure to add that to this PR.

[...] how to access data and assets from KV and use them to generate static and dynamic responses. If you want to serve static assets of a frontend web application, consider using Cloudflare Pages [...]

Wondering if we want to call out dynamic content with Pages too? Pages isn't just static content, you can also serve dynamic content with Functions (and without needing to use a manual KV implementation).

Considering that Workers Sites docs indicate 'Do not use Workers Sites for new projects.', I'm inclined to leave the PR as is (not including a mention of Workers Sites of kv-assets-handler).
I will edit the Pages guidance to include the fact that it supports dynamic responses, but given folks land here for 'serving static assets for KV', I think the guidance from here to Pages for serving static assets for a web application is appropriate.

@hyperlint review

Closing as done in https://github.com/cloudflare/cloudflare-docs/pull/16358


Feature/cli add syntax option

After butchering https://github.com/sasstools/sass-lint/pull/391, this should be clean.
As per #381, this adds a --syntax option to the CLI that overrides the filename extension based syntax detection.
DCO 1.1 Signed-off-by: Donald Abrams<EMAIL_ADDRESS>

Update this and it's good to go. :+1:

Hey @donabrams would you mind updating this with the latest develop branch. Happy to merge it once it's up to date.


This should improve flask result for json output

If flask does not receive the X-Requested-With: XMLHttpRequest header on a request which responds with jsonify, it will format the output, slowing dow the response time quite a lot. This PR fixes this. Flask bench should be faster, now.

I'm not sure, @dimaqq The PR is seating still for some time now. What should be done to have it accepted?

Fork it and own it :)

Good point @dimaqq Although sad. Will do.
@klen @gi0baro @hawkowl please, take a look.


Gj

nice to see that all the new types of stealers out there are using my project for anti-vm-blacklist
Good stealer btw.

I apologize for not realizing that the blacklist strings belonged to you. I collected them from open-source stealers. I'll credit you.

Its fine, most of people dont credit me. But its not needed. Thanks anyways.

The comparing picture says everything. There are numerous different methods to analyze the file. The most common is static and dynamic. This method stops most of the dynamic analyze process. This means your file wont be runned fully and it will not expose it's malicious actions. Very easy to understand.  You wouldnt be able to name any online vm that changes everything

Anyways this list is for online vms. As the name says its an anti virustotal list

It would be probably easy to implement timing attack in python using ctypes/windll

The ctypes library is able to interact with the system's low-level API

I will test this thing out in a hyper v ENV.


Does not work with goreleaser to build darwin and linux

I'm not sure which library caused the problem. I wrote a demo to verify, it said
   ⨯ release failed after 0.53s error=failed to build for darwin_arm64: exit status 2: # golang.design/x/clipboard
../../../go/pkg/mod/golang.design/x/clipboard@v0.5.1/clipboard.go:90:14: undefined: read
../../../go/pkg/mod/golang.design/x/clipboard@v0.5.1/clipboard.go:110:18: undefined: write
../../../go/pkg/mod/golang.design/x/clipboard@v0.5.1/clipboard.go:125:9: undefined: watch

demo
Any ideas?

You must specify CGO_ENABLED=1 whereas your goreleaser config is:
builds:
  - env:
      - CGO_ENABLED=0

I change CGO_ENABLED= 1 . output:
   ⨯ release failed after 4.12s error=failed to build for linux_386: exit status 2: # runtime/cgo
linux_syscall.c:67:13: error: implicit declaration of function 'setresgid' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
linux_syscall.c:67:13: note: did you mean 'setregid'?
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/unistd.h:593:6: note: 'setregid' declared here
linux_syscall.c:73:13: error: implicit declaration of function 'setresuid' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
linux_syscall.c:73:13: note: did you mean 'setreuid'?
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/unistd.h:595:6: note: 'setreuid' declared here

It seems to be a problem with my local environment, I will try by myself

Although in the end I still can't build the linux version, but this is a bug of musl-cross on Apple silicon,thx a lot

Reopen to give a better error message.


release: look for CLs with +2 but no Trust+1

As part of the transition to the release freeze, we should look for CLs that have a +2 but no Trust+1.  Those CLs should all be reviewed for Trust+1 purposes, and, if no problems are found, submitted.
CC @dmitshur @cagedmantis @toothrot

@toothrot has previously suggested this Gerrit search query:
https://go-review.googlesource.com/q/label:Code-Review%253D%252B2+label:Untrusted%253Dreject
There are 3.5~ pages of results (at 25 CL/page) at this time.

https://go-review.googlesource.com/q/label:Code-Review%253D%252B2+label:Untrusted%253Dreject+repo:go+branch:master narrows it down to just the main repository, main branch. There are 58 matching open CLs.

Many CLs have been submitted since last time. I've looked over some of them and reviewed for Trust where it was the only missing thing. By now, the same query finds 15 CLs (down from 58). Many of those 15 have remaining work or need attention from someone, although maybe not all.

The query above (slightly modified to additionally filter out CLs that are waiting on author or have a -2 review) by now finds only 8 CLs. Of those, 6 are sent by people who have the ability to grant Trust+1 themselves. The other 2 were being actively evaluated for readiness for submission.
Given we're in the freeze and the week to submit in-flight changes has passed, I don't think there's more to do here.

I was hoping that this issue would become a recurring issue that we kept open for each release.

@cagedmantis I was previously thinking this was needed for the Go 1.16 release only because the Trust requirement is new, and some CLs that could previously be submitted now needed an extra vote. But it does seem such a situation can continue to happen in the future if a CL is sent by someone without ability to self-Trust+1, and reviewed only with a single Core-Review+2. Thanks for pointing this out.
Maybe in the future this can be factored out to happen continuously as part of normal code review and not need to be tied as closely to the start of the release freeze, but I guess it'll work to make this issue recurring until that happens.

Just a note that this is a steady ongoing problem that we ought to address somehow.  It doesn't have anything to do with releases.  We have a somewhat patchy and unreliable system for getting reviewers to look at CLs, but we don't have any system at all for getting CLs that have a +2 over the finish line to submission.

Ian came up with an idea for us to add a weekly task for someone on the Go team (we have a weekly rotation we can make use of) to look for CLs that have been reviewed but just need trust to become submittable, and try to review them for trust.
https://go-review.googlesource.com/q/label:Code-Review%253D%252B2+label:Untrusted%253Dreject+-label:Code-Review%253D-2+-hashtag:wait-author+-age:1w finds CLs with CR+2 but no trust (without CR-2 or wait-author hashtag) that are no more than a week old. Perhaps it's a good place to look for the purpose of a weekly task. CC @toothrot.


Create suntech.txt

@STC-EDU Please provide us with:

the school official website URL,
an URL of a page or another proof showing that the school recognizes the domain which you are submitting as an official email domain for the students.

https://suntech.ac.jp/
(japanese)
https://suntech.ac.jp/en/index.html
(english)
https://campus.suntech.jp/
(not accessible from other domains)

@STC-EDU Could you please tell me how can we check that the university recognizes suntech.jp as an official domain for their students? Do you have any paper, .pdf., internal instruction or other data to prove that? Thank you.

@STC-EDU You might miss my previous comment. Please react.

@STC-EDU Could you please answer the question about official document to prove the domain recognition by the university?


"Promise" is undefined error with IE11

Hey,
I was testing my app with some different browsers and the only one that got an issue is IE11 (I didn't test other versions yet).
So I've got this error :
`[Vue warn]: Error in created hook: "ReferenceError: « Promise » is undefined"
found in
--->  at ...\MyProject\node_modules\vuetable-2\src\components\Vuetable.vue`
This error shows up just when i load a vuetable component, or a component containing it.
I tried with Chrome / Firefox / Edge, not problem on these browsers, only on IE.
Do yo heard about a similar problem ?
Maybe I miss a dependency ?
Thnaks for any help !

There might be a need to use a promise polyfill https://github.com/taylorhakes/promise-polyfill

@cristijora  I don't know if it will have any side effect to the others. If not, we can just add it as another dependency. But if unsure, I will just put a note regarding this on the README.
@bellissi1u Can you please try adding this dependency to your project and see if it fixes your problem? If you confirm the fix and do not see any side effect, I'll add the dependency to Vuetable.

@cristijora @ratiw Thanks for help, I installed promise-polyfill and just followed the setup guide.
It seems to work now, I don't see any side effects for now.
I'll tell you if I found something.

I've got another issue with IE9.
I can't get the Access-Control-Allow-Origin header to work, maybe because of this.
Do you know if the jquery module specified in the first answer can be used here, or an alternative one ?

@bellissi1u Thanks. If you're certain that no side effect is occurred, please let me know and I'll add this dependency to the package.
For the jquery plugin for CORS, I really have no idea. You may have to try it or consider leave IE9 out if possible.

I'll leave IE9, I don't think i'll absolutely need it.
Thanks for help. Considering the setup of promise-polyfill :
import Promise from 'promise-polyfill if (!window.Promise) {     window.Promise = Promise }
I think there will be no side effects at all. And I didn't find any so far.

@bellissi1u Can you please make a PR for this?

@ratiw Sorry I'm pretty a newbie in github, so I don't actually understand the working of pull-request and collaborative work ...
I cloned the repository and made some changes, now I do I create my own branch ?

@bellissi1u You start by "forking" vuetable-2 project on GitHub web page, then you'll have your own vuetable-2 in your repositories. You, then, clone "your" vuetable-2 to your machine, making changes, in this case adding promise-polyfill dependency and those import statement (I think it should be inside index.js). Commit everything and push to your repo.
After the push is successful, go back to "your" vuetable-2 repo on Github, you should see something like "Submit Pull Request", press the button and follow further instruction (if any), then you're good.
But if this seems a little bit daunting to you, don't worry, it will not mess up anything. It's a good experience to give it a try. It's quite hard to mess anything up.

@ratiw Thanks a lot for help, it's pretty easy after all
I made the pull-request, should work now for IE10/11


feat: Changing finite field arithmetic in wasm to 29 bits for multiplications (second try, disabled AVM build in wasm)

Now when we compile for wasm we use 9 29-bit limbs to perform all multiplications. In wasmtime this results in -10% for multiplication benchmark and -35% in squaring. This makes a -7% impact on wasm client_ivc.
Other changes:

Most of the changes are just additions to tests to handle the new montgomery form, because we have a lot of hardcoded internal representations. If we get a new one, we should rewrite those to use non-montgomery form. Also we need to recompute all parameters.
Added squarings to univariate and used in some relations
Added a script to run wasmer instead of wasmtime on the benchmark (but you need to change arguments slightly to add "--")
For some reason wasm build has become really slow (especially on relations and protogalaxy).
Added docs

Benchmark results
Metrics with a significant change:

batch_insert_into_append_only_tree_32_depth_ms (1024): 598 (+19%)
node_history_sync_time_in_ms (10): 34,506 (+16%)
note_trial_decrypting_time_in_ms (64): 121 (+410%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
772
772
772

l1_rollup_calldata_gas
6,820
6,832
6,820

l1_rollup_execution_gas
587,301
587,313
587,301

l2_block_processing_time_in_ms
1,408 (-3%)
5,122 (-2%)
10,157 (-2%)

note_successful_decrypting_time_in_ms
151 (-24%)
523 (+1%)
1,110 (+4%)

note_trial_decrypting_time_in_ms
40.8 (-52%)
20.8 (+11%)
:warning: 121 (+410%)

l2_block_building_time_in_ms
18,693
73,037 (-1%)
145,697

l2_block_rollup_simulation_time_in_ms
18,485
72,281 (-1%)
144,108

l2_block_public_tx_process_time_in_ms
8,448
33,117 (-1%)
65,860 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,911 (+2%)
:warning: 34,506 (+16%)

note_history_successful_decrypting_time_in_ms
1,320 (-6%)
2,566 (-5%)

note_history_trial_decrypting_time_in_ms
74.2 (-34%)
113 (+5%)

node_database_size_in_bytes
20,709,456 (+8%)
37,646,416 (+5%)

pxe_database_size_in_bytes
29,868
59,425

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
167 (+6%)
47,763
31,978

private-kernel-inner
189 (+7%)
80,794
31,978

private-kernel-ordering
148 (+8%)
57,732
49,313 (-1%)

base-parity
9.61 (+2%)
128
265

root-parity
67.4 (-10%)
1,244
265

base-rollup
12,045
110,926
957

root-rollup
51.1 (+3%)
4,551
821

public-kernel-app-logic
1,579 (+1%)
62,017
26,202

public-kernel-tail
4,407 (+4%)
168,566
3,812

merge-rollup
3,254 (+1%)
2,760
957

public-kernel-teardown
122 (+3%)
62,017
34,711

public-kernel-setup
133
62,017
34,711

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.5 (-6%)
17.4 (-1%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.7
31.8
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.611 (-5%)
0.535 (-2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
50.8 (-1%)
74.9 (-7%)
243 (-6%)
:warning: 598 (+19%)
921 (-6%)
1,816 (-6%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.519 (-1%)
0.462 (-7%)
0.442 (-7%)
0.560 (+19%)
0.436 (-7%)
0.433 (-6%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
59.5 (-2%)
112 (-6%)
350 (-6%)
808 (+9%)
1,372 (-6%)
2,734 (-6%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
208
692
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.524 (-2%)
0.502 (-6%)
0.475 (-7%)
0.555 (+9%)
0.473 (-7%)
0.475 (-6%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
65.4 (-3%)

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
108 (-1%)

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.574 (-2%)

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
51,411
536,920 (-1%)

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
897
1,145
1,345

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,231 (-1%)
922 (+6%)
3,503 (-1%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
887 (+6%)
765 (-1%)
1,360
907 (-1%)
2,077 (-1%)
1,090 (-1%)


How to specify 'open source only'  scan

I like to run the scan for the 'open source only',  How do we achieve this using this appscan-gradle-plugin?

The Gradle plugin does not currently run open source only scans, but it's something we hope to add soon.

cool, thanks a lot. I wanted to add all the parameters we set at the appscan-config.xml in this Gradle plugin.


NavMenu使用路由功能内容显示错误

router-view显示的内容为当前组件内容+router-view内容

hello @weitao901127 I want to ask you for help. I want to have different URLs for each menu-item.
I try to get the http://router.vuejs.org/ work together with el-menu but could not sort it out. The elem UI docu mention route attributes (on menu:router and menu-item:route), what my lead to the conclusion, that it is prepared for vue-router, but I did not get it all together in my App.vue file.
Where to put the router instance?
Is a el-menu@select method needed somehow?
Do you have an example App.vue from where I can grab it?

请给出可复现问题的 demo


added cpp code of swapping two integers without using third variable

Remove the .exe file

I have removed the exe file.


When there are multiple Expanded in the row, if one has enough space, the other can occupy it. Like the flex layout of the web.

When there are multiple Expanded in the row, if one has enough space, the other can occupy it. Like the flex layout of the web.
In the picture, the content of text is uncertain, and a text can be displayed completely with just a little more space. The flex layout on the web is completely fine and easy to implement.
Details
Target Platform: android
Target OS version/browser: 33
Devices: Pixel_3a_API_34_extension_level_7_arm64-v8a
Logs
Row(
                  children: [
                    ClipRRect(
                      borderRadius: BorderRadius.circular(1000),
                      child: Image.network(
                        'https://cdn.pixabay.com/photo/2023/08/05/23/40/bird-8171927_640.jpg',
                        fit: BoxFit.fill,
                        height: 30,
                        width: 30,
                      ),
                    ),
                    Expanded(
                      child: Container(
                        margin: const EdgeInsets.fromLTRB(10, 0, 10, 0),
                        child: Text(
                          'aaaaaaa',
                          overflow: TextOverflow.ellipsis,
                          style: const TextStyle(fontSize: 16, color: Colors.black),
                        ),
                      ),
                    ),
                    Image.network(
                      'https://cdn.pixabay.com/photo/2023/08/05/23/40/bird-8171927_640.jpg',
                      fit: BoxFit.fill,
                      height: 20,
                      width: 20,
                    ),
                    Container(
                      child: Text('回复'),
                      margin: EdgeInsets.fromLTRB(10, 0, 10, 0),
                    ),
                    Expanded(
                      child: Container(
                        margin: const EdgeInsets.fromLTRB(0, 0, 10, 0),
                        child: Text(
                          'c',
                          overflow: TextOverflow.ellipsis,
                        ),
                      ),
                    ),
                    Text('2022-02-22')
                  ],
                ),

Hi @zhou26192619
This issue isn't a performance issue and also quite not clear about the expected result. Please update the issue or open a new issue (and close this respectively) corresponding to Bug issue template so that we may verify the issue properly. Thank you!

Hi @zhou26192619 This issue isn't a performance issue and also quite not clear about the expected result. Please update the issue or open a new issue (and close this respectively) corresponding to Bug issue template so that we may verify the issue properly. Thank you!

Thank yout, I put it to Bug issue.


Lars changes mar22

Hi @rnikutta,
I just pushed some text and formatting changes that Lars made to Notebooks 5 and 9. Could you please check?
Thanks!

Hi @DavidVargasMora Sorry, was out of town. Thanks for the PR, merging.


Tweak PhotometryFilters table

Right now, we require Telescope and Instrument in the PhotometryFilters Table but I think that's redudant since those fields should be associated with a particular observation. For example, lots of different instrument/telescope combos use the SDSS filter set.  I think we should remove those columns from the PhotometryFilters table and only require band.

Also seems like if we're gonna keep UCDs, they should be in the Photometry Filters table and not in the Photometry table.

Ok, so, the official filter names from the SVO usually include the specific instrument name so there's not a lot of duplicate use of filter names. This tells me that using the UCD (e.g., em.opt.J) is useful so that you can run a query to get all the J-band data without needing to know all the various names for J band filters. (e.g, 2MASS.J, WFCAM.J, VISTA.J, UKIDSS.J).
Having instrument and telescope on both the Photometry and PhotometryFilters table is still redundant. But I'm not sure which table to keep it on.

We're leaning towards removing Telescope and Instrument from PhotometryFilters.

But we want to keep UCD (e.g., em.J) in the PhotometryFilters table so that we can generate J-K plots, for example, which include lots of different J filters (MKO.J and 2MASS.J).
Also want to keep effective_wavelength, but it doesn't need to be perfectly accurate.

The website uses effective wavelength to construct colours to ensure all are blue - red
if photfilters.at['effective_wavelength', bandtrue] >= \
                    photfilters.at['effective_wavelength', nextbandtrue]:  # if not blue-red
                continue

Can connect an approximate central wavelength in the PhotometryFilters table to allow this grouping of bands

Remove instrument from Photometry table
Remove instrument and telescope from PhotometryFilters

Remove ucd from Photometry
Add ucd to PhotometryFilters


Using customvision onnx on Nvidia (TensorRT/Deepstream)

I am experimenting to get a customvision model (exported as ONNX) to run on a nvidia device using deepstream SDK (TensorRT engine for accelerating ONNX).

I was able to follow the steps in this repository and train a model using CustomVision.AI and run it on Nvidia device.
This works great for a object detection type of model.

When i am using a classification model trained using customvision and exported as ONNX, i get several warnings during conversion:
[W1]:Your ONNX model has been generated with INT64 weights, while TensorRT does not natively support INT64. Attempting to cast down to INT32.
[W2]:Calling isShapeTensor before the entire network is constructed may result in an inaccurate result.

Even though the TensorRT model gets generated, its not working and i get incorrect results (possibly due to the weights cast down).

So wanted to check if Microsoft has tried this setup? Any information on how customvision uses ONNX internally?

TensorRT doesn't support INT64 and will convert INT64 weights to INT32. Usually we don't see accuracy loss due to the conversion, but it depends on how big the INT64 value is. You might also try OnnxRuntime for onnx model inferencing. You can run the model by different execution providers to check the latency and accuracy.


Unit test infrastructure setup

Originator(s): @mwaxmonsky
Summary (include the keyword ['closes', 'fixes', 'resolves'] and issue number): Sets up the basic infrastructure to start enabling unit testing.  Addresses #82
Describe any changes made to the namelist: N/A
List all files eliminated and why: N/A
List all files added and what they do:
A       .github/workflows/code-coverage.yaml
A       test/unit-test/CMakeLists.txt
A       test/unit-test/include/ccpp_kinds.F90
A       test/unit-test/tests/CMakeLists.txt
A       test/unit-test/tests/utilities/CMakeLists.txt
A       test/unit-test/tests/utilities/test_state_converters.pf

Initial round of infrastructure to start enabling unit tests.

List all existing files that have been modified, and describe the changes: N/A
(Helpful git command: git diff --name-status development...<your_branch_name>)
List any test failures: N/A
Is this a science-changing update?   No

couple small questions.
Also, I know you've documented this stuff on the new doc page (thanks!), but can I also request a README in the test/unit-test directory that describes, at minimum, how to run the tests?

Added a README with a few instructions and pointer to the gh-docs page but just let me know if there's any other details you think we should add.


Adjust version detection to support react@15

Hey there!
I've been testing react@15 (which came out today) with my project, seems like everything works fine apart from the version detection in react-utils.js which doesn't cater to the fact that react is now using major versions.
This PR fixes the above issue - have smoke tested and seems to be running fine with react@15 and<EMAIL_ADDRESS>Thanks!

Looks great to me.  :+1: Thank you @samhogg

Thanks!


[Enhancement] Use real key type size as the initial agg state size

What type of PR is this：

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #13880

Use the real key type size as initial agg state size instead of 16;
Split aggregator allocator from aggregator header;

Problem Summary(Required) ：

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_be_unittest

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test