gen-certs is not a valid identifier

Function names with the dash "-" sign are not allowed in bash, according to bash itself (see error below) as well as the Bash scripting guide at http://tldp.org/LDP/abs/html/gotchas.html.
I'm surprised that the adop script has ever worked before:
++ basename ./adop
+ CMD_NAME=adop
+++ echo ./adop
+++ sed -e 's,\\,/,g'
++ dirname ./adop
+ export CLI_DIR=.
+ CLI_DIR=.
+ export CONF_DIR=.
+ CONF_DIR=.
+ export CONF_PROVIDER_DIR=./conf/provider
+ CONF_PROVIDER_DIR=./conf/provider
+ CLI_CMD_DIR=./cmd
+ main compose gen-certs /Users/oscar.renalias/.foo
+ '[' 3 -lt 1 ']'
+ SUBCOMMAND=compose
+ shift
+ '[' '!' -e ./cmd/compose ']'
+ . ./cmd/compose gen-certs /Users/oscar.renalias/.foo
++ SUB_CMD_NAME=compose
./cmd/compose: line 239: `gen-certs': not a valid identifier

This was the output of sh -x ./adop compose gen-certs ~/.foo.
The attached pull request fixes the problem but please note that the "gen-certs" commands is also renamed to "gen_certs" unless the current command delegation logic is changed. I have not updated any documentation.

If I run sh -x ./adop compose gen-certs /root/.docker/ it fails as above, but if I run bash -x ./adop compose gen-certs /root/.docker/ then it works fine - which is also how we expect it to be run as we use bash across the CLI.
Happy for this to be changed though for compatibility, provided that the readme is updated please.

I messed up and created this PR straight from my master branch, which was getting confusing. I 've opened a new one, PR #76, which also maintains "gen-certs" as the name of subcommand so no updates to the documentation are needed. I'll close this one unless there are any objections.


Show/Hide  options  programaticaly

"Display
Select2 provides options that allow you to directly affect how the container that holds the current selection is displayed."
So please explain me  how  can I  work wit just this  one  sentence
Is  there  any wat to SHOW/HIDE  just ONE (only ONE ) option from dropdown list in whitout making ajax call or  refilling data  into slect ?

+1
Although I would suggest to be able to modify the original select and then properly display (after re-initialising).
I already saw several issues related to the fact that Select2 doesn't care about CSS styling (display:none), and I even noticed that it doesn't even care about the hidden attribute.
CodePen: https://codepen.io/anon/pen/JrJKBq
As you can see the default handling (at least in Chrome) will hide the options which have either display:none in the CSS or have the hidden attribute. At least it follows the disabled attribute.

I'm actually going to close this as a duplicate of #4105.  I am surprised though, that Select2 does not respect the hidden attribute on a hardcoded <option> element.

Hello. Does Select2 still not respect the hidden attribute of the option html element?
How can we make it respect it and hide those options?

Is it still not working?

+1 Would like to hide all options that do not have attribute=value

+1 What reallyslick said.


Verify Installation Is Really Completed (was 2.9.0 Installation failure)

I was doing a fresh installation of Forseti 2.9.0 in an organization where I am org admin. After the installation completed, when I went to run forseti inventory create I get the following.
zberger_scefast_com@forseti-client-vm-9d85e7f:~$ forseti inventory create
Error communicating to the Forseti server.
Please check the status of the server and make sure it's running.
If you are accessing from a client VM, make sure the `server_ip` field inside the client configuration file in the Forseti client GCS bucket contains the right IP address.

I validated the client configuration file and it contains the following:
server_ip: <IP_ADDRESS>

which is the server's IP address:

I've attached the deployment.log files from both the client and the server, but nothing stands out there:
client-deployment.log
server-deployment.log
The server service is in fact running:
zberger@forseti-server-vm-9d85e7f:~$ sudo systemctl status forseti
● forseti.service - Forseti API Server
   Loaded: loaded (/lib/systemd/system/forseti.service; disabled; vendor preset: enabled)
   Active: active (running) since Mon 2018-12-17 21:54:06 UTC; 25min ago
 Main PID: 12165 (forseti_server)
    Tasks: 10 (limit: 4915)
   CGroup: /system.slice/forseti.service
           └─12165 /usr/bin/python /usr/local/bin/forseti_server --endpoint [::]:50051 --forseti_db mysql://root@12

Dec 17 21:54:06 forseti-server-vm-9d85e7f systemd[1]: Started Forseti API Server.

And is listening on port 50051:
zberger@forseti-server-vm-9d85e7f:~$ sudo lsof -iTCP | grep 50051
forseti_s 12165          ubuntu   58u  IPv6  39589      0t0  TCP *:50051 (LISTEN)
forseti_s 12165          ubuntu  112u  IPv6  49035      0t0  TCP ip6-localhost:35850->ip6-localhost:50051 (ESTABLIS
HED)
forseti_s 12165          ubuntu  113u  IPv6  49816      0t0  TCP ip6-localhost:50051->ip6-localhost:35850 (ESTABLIS
HED)

And I can establish a connection to 50051:
zberger_scefast_com@forseti-client-vm-9d85e7f:~$ telnet <IP_ADDRESS> 50051
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.

Workaround: restarting the forseti server service allowed the client to connect.
Here are the server runtime logs:
server-runtime.log

Hi Zach, the server still requires a period of initialization after the installation finishes.  We used to check by ssh into the server and check the /tmp/deployment.log, but the ssh has stopped working recently.  So, we should work on issue #1112 to final an alternate way to verify that the server is up and running, before calling the installation complete.

Thanks @blueandgold, however this persisted for hours

sudo lsof -iTCP | grep 50051

I was doing a fresh installation of Forseti 2.9.0 in an organization where I am org admin. After the installation completed, when I went to run forseti inventory create I get the following.
zberger_scefast_com@forseti-client-vm-9d85e7f:~$ forseti inventory create
Error communicating to the Forseti server.
Please check the status of the server and make sure it's running.
If you are accessing from a client VM, make sure the `server_ip` field inside the client configuration file in the Forseti client GCS bucket contains the right IP address.

I validated the client configuration file and it contains the following:
server_ip: <IP_ADDRESS>

which is the server's IP address:

I've attached the deployment.log files from both the client and the server, but nothing stands out there:
client-deployment.log
server-deployment.log
The server service is in fact running:
zberger@forseti-server-vm-9d85e7f:~$ sudo systemctl status forseti
● forseti.service - Forseti API Server
   Loaded: loaded (/lib/systemd/system/forseti.service; disabled; vendor preset: enabled)
   Active: active (running) since Mon 2018-12-17 21:54:06 UTC; 25min ago
 Main PID: 12165 (forseti_server)
    Tasks: 10 (limit: 4915)
   CGroup: /system.slice/forseti.service
           └─12165 /usr/bin/python /usr/local/bin/forseti_server --endpoint [::]:50051 --forseti_db mysql://root@12

Dec 17 21:54:06 forseti-server-vm-9d85e7f systemd[1]: Started Forseti API Server.

And is listening on port 50051:
zberger@forseti-server-vm-9d85e7f:~$ sudo lsof -iTCP | grep 50051
forseti_s 12165          ubuntu   58u  IPv6  39589      0t0  TCP *:50051 (LISTEN)
forseti_s 12165          ubuntu  112u  IPv6  49035      0t0  TCP ip6-localhost:35850->ip6-localhost:50051 (ESTABLIS
HED)
forseti_s 12165          ubuntu  113u  IPv6  49816      0t0  TCP ip6-localhost:50051->ip6-localhost:35850 (ESTABLIS
HED)

And I can establish a connection to 50051:
zberger_scefast_com@forseti-client-vm-9d85e7f:~$ telnet <IP_ADDRESS> 50051
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.

Workaround: restarting the forseti server service allowed the client to connect.
Here are the server runtime logs:
server-runtime.log

Hi can you please help me how to resolve the issue
i have configured forseti multiple times

@blueandgold to investigate this issue in more detail and follow up.

I was doing a fresh installation of Forseti 2.9.0 in an organization where I am org admin. After the installation completed, when I went to run forseti inventory create I get the following.
zberger_scefast_com@forseti-client-vm-9d85e7f:~$ forseti inventory create
Error communicating to the Forseti server.
Please check the status of the server and make sure it's running.
If you are accessing from a client VM, make sure the `server_ip` field inside the client configuration file in the Forseti client GCS bucket contains the right IP address.

I validated the client configuration file and it contains the following:
server_ip: <IP_ADDRESS>

which is the server's IP address:

I've attached the deployment.log files from both the client and the server, but nothing stands out there:
client-deployment.log
server-deployment.log
The server service is in fact running:
zberger@forseti-server-vm-9d85e7f:~$ sudo systemctl status forseti
● forseti.service - Forseti API Server
   Loaded: loaded (/lib/systemd/system/forseti.service; disabled; vendor preset: enabled)
   Active: active (running) since Mon 2018-12-17 21:54:06 UTC; 25min ago
 Main PID: 12165 (forseti_server)
    Tasks: 10 (limit: 4915)
   CGroup: /system.slice/forseti.service
           └─12165 /usr/bin/python /usr/local/bin/forseti_server --endpoint [::]:50051 --forseti_db mysql://root@12

Dec 17 21:54:06 forseti-server-vm-9d85e7f systemd[1]: Started Forseti API Server.

And is listening on port 50051:
zberger@forseti-server-vm-9d85e7f:~$ sudo lsof -iTCP | grep 50051
forseti_s 12165          ubuntu   58u  IPv6  39589      0t0  TCP *:50051 (LISTEN)
forseti_s 12165          ubuntu  112u  IPv6  49035      0t0  TCP ip6-localhost:35850->ip6-localhost:50051 (ESTABLIS
HED)
forseti_s 12165          ubuntu  113u  IPv6  49816      0t0  TCP ip6-localhost:50051->ip6-localhost:35850 (ESTABLIS
HED)

And I can establish a connection to 50051:
zberger_scefast_com@forseti-client-vm-9d85e7f:~$ telnet <IP_ADDRESS> 50051
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS>.

Workaround: restarting the forseti server service allowed the client to connect.
Here are the server runtime logs:
server-runtime.log

Hi may i know How this issue was resolved

This issue is stale and has not had any recent activity. If this is still a concern please reopen the issue. Thanks!


Please add tests to release tarballs

Please add the tests to the release tarball. It'd be nice for anyone who gets the tarball to be able to run the tests. This could be used to run the tests when building distribution packages, for example.

Since 2.4.6, I've been tagging release commits. If you download those tagged versions, they should contain everything.

Downloading github tags is kind of annoying because of the cset ID appended to the directory name. It also takes more effort to regenerate those tarballs, over the pre-built tarballs you put up for download. Why not include the tests in the latter, too? It's generally pretty useful to be able to test a package before installation or usage in the environment.


Being able to extend state machines through classes (Single inheritance)

Umple allows to extend state machines through traits, but it still doesn't support extension through classes (normal single inheritance).
The merging algorithm has already been developed for merging state machines (coming from traits). It can be reused easily for this case.
Steps:

state machines of a class must be merged with the ones coming from used traits.
get the new set of state machines for the class (merged one)
state machines from the previous step must be merged with the ones coming from superclasses.

@vahdat-ab

I am wondering if you could please provide an example for this behaviour?

The merging algorithm has already been developed for merging state machines (coming from traits). It can be reused easily for this case.

Whereabouts is this algorithm in the code?

Please consider the following example:
class X{
  status{
    s1{
      exit /{System.out.println("t4");}
      e2-> s2;
      s11{
        exit /{System.out.println("t3");}
        s111{
          exit /{System.out.println("t2");}
          s111{
            exit /{System.out.println("t1");}
            e-> Final;
          }
        }
      }

    }
    s2{
      e4-> Final;
    e3-> s3;
    }
    final s3{}
  }

}

Your solution doesn't take care of exits.

@vahdat-ab ^ I will comment on this in my PR

Let's arrange an online meeting for this.
I need to clarify several things.
I'll mostly available during this week except monday from 2:30-4 pm.

@vahdat-ab sounds good. Does 4:30PM EST (1:30PM PST) work for you tomorrow (April 3rd)?

@vahdat-ab sounds good.
I am available at the following times tomorrow for Skype:
1:15PM - 2:15 PST          (4:15PM - 5:15PM EST)
5:45PM - 7:45PM PST   (8:45PM - 10:45PM EST)
Please let me know what works for you. Thanks

@jblang94 let's talk today at 4:15 PM EST on skype.

This is not an issue to be done in a week so we will get another person to work on this.

Traits and classes are different, in that code is actually generated in a class, so the merging for traits to create classes might not work without further thought.
One thing to bear in mid is this: If a superclass X has state machine sm1, then code will be generated for that. And the enums created belong to class X (e.g. the state machine enum in the generated java is X.Sm1, and getSm1() will return a value of X,Sm1)
If a subclass of X called Y is to also have sm1 (with extensions), then compatible code in the subclass needs generating. Currently if the state machine sm1 is declared, the Java state machine is Y.sm1, yet the getSm1() method would return Y,Sm1 ... and the compiler will complain that Y.Sm1 is incompatible with X.Sm1.
Enums are special classes with sets of constants, and each enum is a final class so it can't be overridden in a subclass. However the advice at the following URL might allow a workaround https://www.baeldung.com/java-extending-enums
I wonder if the following could work:
During state machine processing, after the class is complete, check to see if there is already a state machine in the superclass that has the same name. If so copy it entirely into the current class. Then any changes made would be relative to the copied state machine ... it would work just in the same manner as adding a feature in a mixin in the superclass. ... but we would have to deal with the enum problem as described above.


Starred topics using starred messages

Starred topic = topic having at least one starred message.
Only last 2 commits are new here. Rest will be soon merged into master, so I worked on top of them.
TODO: Add tests for it in frontend.

Closing as this idea didn't work out.


Robot is unbalanced

There is more weight on one side of the robot, so it turns, even if we program it to go straight forward. We can either use the IMU (inertial measurement unit), or we can add a counterweight.

OK, that sounds good. How did the countereweight stuff work out?

OK, that sounds good. How did the countereweight stuff work out?

It was never the weight that was the problem. The motor configurations were mixed up.              :|

It was never the weight that was the problem. The motor configurations were mixed up.              :|


More Double -> BigDecimal sequence nums

Rationale
BigDecimal is our preferred representation for sequence numbers; migrate some more cases

In addition to all the BVTs passing, all relevant tests in the five PostgreSQL Daily suites are green. (Just some unrelated, existing Luminex test failures.) The Dailies showed failures previously, pointing out the need for better BigDecimal normalization.


[Add] Snap Update Automation

Integrated the GitHub workflow auto-update.yml located in .github/workflows/ from the https://github.com/ubuntu/desktop-snaps repository. This leverages the GitHub action for automating updates to the snapcraft.yaml file.

Modified the snapcraft.yaml file to enable the update automation for the individual parts.

Could you kindly review this pull request as well, @tillkamppeter?


Respect ApiBehaviorOptions.SuppressInferBindingSourcesForParameters when specified

Fixes https://github.com/aspnet/Mvc/issues/8657

@pranavkm I think I should hold off reviewing 'til you split this in two. LMK if I need to review this up front.

@pranavkm the commit description and the title are way wonky. This is about a new option (AllowInferringBindingSourceForCollectionTypesAsFromQuery).


Xp drops broken

My xp drops arent showing other than skilling drops.

Need more details. What are your plugin settings?

not showing predicted hit

I see. Since you're filtering combat skills there is no xp drop to group the predicted hit with. Therefore you should turn Never group predicted hit ON.

Thanks dood <3 fixed it. Legend


Solving the difficulty of writing program's code beyond 4k with banking

A new keyword ":offset NNNN" will subtract any new addresses and branches assigned after it by the specified NNNN until the compiler met another ":offset", hence one don't need to write separate program in order to do multiple banks of "program code".
For example, when program code located on some long address "0x2000", and one wish to execute it on "0x800", using keyword ":offset 0x1800" will subtract the new addresses by "0x1800":
: main
  i := long codeonlong  load vF
  i := bankedroutine00  save vF
  i := long lockuponlong  load v4
  i := bankedroutine10  save v4
  jump bankedroutine00

: getrandom
  v0 := random 63
  v1 := random 31
  return

:org 0x800
: bankedroutine00
:org 0x900
: bankedroutine10

:org 0x2000 # place the routine at beyond 0xFFF
:offset 0x1800 # subtract addresses and branches by 0x1800 [0x2000-0x800], to run the routine at 0x800
: codeonlong
  loop
    getrandom # call a predefined subroutine below 0x1000
    if v0 == v1 then
      jump outloop # jump to address which meant to exit the loop
    i := hex v0
    sprite v0 v1 5
  again
: outloop
  jump bankedroutine01

:org 0x3000 # place another routine at beyond 0xFFF
:offset 0x2700 # subtract addresses and branches by 0x2700 [0x3000-0x900], to run the routine at 0x900
: lockuponlong
  loop  again  # lockup

The offset could be reset anytime by ":offset 0", as the offset is 0 by default on the start of compilation.
The ":offset" also may accept constant, or any variables defined by ":calc".

In case of some forward references, ":offset" only affects new resolved addresses.

I've given this some thought, and I've decided it's too application-specific. In practice very few people have written programs that need bank-switching, and there are already several techniques available. I'm not adding this feature at this time.


how to implement a sort without a default column id

hi yajra,
since i don't have a cloumn named id in my database as the primary key , instead, the primary key is user_id, in my query, i certainly know how to get myself's sort with orderColumn, but the plugin sort by id by default, can i get any idea to avoid sorting by id by default?
thanks for your advice in advance;

The setting of datatables : sorting: [[ 1, "asc" ]]

@1147832740 answer should be able to help you setting the default sorting. Thanks!


CLI doesn't respect context namespace

As of v1.89.0, the kots CLI will utilize the namespace of the current context if one is not explicitly provided.  This applies to all commands except for the kots install command since the user is prompted for a namespace if one is not provided via CLI flag.
PR: https://github.com/replicatedhq/kots/pull/3286


DeepBind on larger genes

Hi all,
Thanks for this great tool! I have a question on its use though.
Could the user examine the binding specificity of the motifs in your models by querying a sequence > 10000b introducing an upper bound change to the script? Will the algorithm work properly in this case?
In general, I was wondering if you have applied DeepBind on a genome-wide scale
Thank you very much,
Eleni

Hi Eleni, the DeepBind models in most cases do not work at all on a genome wide scale based on internal evaluation. The negative set used to train these models was shuffled versions of sequences at chip-seq peaks (that preserved dinucleotide frequencies) which is not representative of the ~99.9% of the genome that would be the negatives set for a given TF in a genome wide prediction. The actual negative set is much larger more heterogenous so models have to take that into account during training to work genome-wide.
I could recommend some alternatives if you could tell me a little more about the setup of the prediction:

what data can you use to impute TFBS? Do you have DNase-seq data in the target celltype or do you need these using DNA sequence only?
is there a particular false discovery rate (FDR) the predictions need to satisfy, i.e. do they need to limit FDR to something like 10%? 50%?

You might also want to contact the original authors of the DeepBind paper (http://www.nature.com/nbt/journal/v33/n8/full/nbt.3300.html).  I made this repo. awhile ago when I identified a bug in the deepbind code, and I put my fixed version of the code here.  Since then, I have been in touch with the authors of the paper, and they have incorporated my fix into their code-base.

Thank you both for your replies. It seems I have misunderstood some parts of the algorithm. We only have DNA seq data and no requirement yet for FDR. I will think about it and maybe come back here soon.
Best,
Eleni

You can use this code to predict whether a TF of interest will bind to a DNA sequence of interest.  Let me know if you want help figuring out how to do it.
I think that you can find a more recent version of the code here: http://tools.genes.toronto.edu/deepbind/#

Dear Irene,
Thank you for your response. I already got the newest version of the code but I still have some questions:

I have some input sequences that are > 50bp long. The algorithm recommends length up to 50bp, but it still accepts 1,000bp long sequences. Are the predictions reliable for 1,000bp sequences?

Can I give as input a sequence i.e. 2,000bp long by modifying the c code? Will the prediction make sense then?

Thank you very much for your help.
Best regards,
Eleni

Hi Eleni,
Here are my thoughts:

I have not thoroughly tested deepbind on longer sequences.  I would recommend asking the authors this question.  Depending on how the code is set up, it is possible that it might not take a longer sequence as input.

I am not sure if there is a way to modify the code to take a 2,000bp sequence.

In general, convolutional neural networks assume a consistent input size.  If the code can handle sequences of different sizes, it might be worth checking whether the code cuts the sequences down to all being the same size.
Irene

Thank you very much Irene! I already checked; the code does not cut all the sequences down to the same size. So, I will try contacting the authors on whether the predictions on larger genes are reliable.
Best regards,
Eleni


How to run a sudo command?

For example, I would like khd to run "sudo networksetup -setsocksfirewallproxystate Wi-Fi off" when I press a hot key. Will this be possible?

List the process to run via sudo in the sudoers file. Please note that this is a huge security issue.


Documented JSON schema validation

Purpose
Document JSON schema validation based on [1] for APIM 3.0.0.
[1] https://docs.google.com/document/d/14AYpwVtRMpe2-KbFILRqRzFA-_Gt7nlHZCCCPzs0jw0/edit#

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

Fix for https://github.com/wso2/docs-apim/issues/59


add support for private registries

Need the support for private docker registries.
Already triied to manualy login on the registry being logged as root on the node, the pull still fail
{"name":"wings","hostname":"ptero-node-1","pid":1872,"level":50,"err":{"message":"(HTTP code 500) server error - {\"message\":\"Get https://hub.company.com/v2/ptero/docker-csgo-sourcemod/tags/list: denied: access forbidden\"}\n ","name":"Error","stack":"Error: (HTTP code 500) server error - {\"message\":\"Get https://hub.company.com/v2/ptero/docker-csgo-sourcemod/tags/list: denied: access forbidden\"}\n \n    at /srv/daemon/node_modules/docker-modem/lib/modem.js:229:17\n   at IncomingMessage.<anonymous> (/srv/daemon/node_modules/docker-modem/lib/modem.js:256:9)\n    at emitNone (events.js:91:20)\n    at IncomingMessage.emit (events.js:185:7)\n    at endReadableNT (_stream_readable.js:974:12)\n    at _combinedTickCallback (internal/process/next_tick.js:74:11)\n    at process._tickDomainCallback (internal/process/next_tick.js:122:9)"},"msg":"(HTTP code 500) server error - {\"message\":\"Get https://hub.company.com/v2/ptero/docker-csgo-sourcemod/tags/list: denied: access forbidden\"}\n ","time":"2016-12-13T22:47:10.196Z","v":0}

Manual pull is working.

closed in Pterodactyl/Daemon@f7170d6


Is it possible to report PHP errors?

In documentation there is a section explaining how to configure the error report level. Therefore, when i've created a simple controller with echo $randomVar (what would show a undefined variable notice), i haven't received any error in Bugsnag.
I'm using error_reporting at the top of my app.php file, just for testing.
My environment is Symfony v3.1.6 and PHP 7.0.12 inside a Linux 14.04 instance.

Just updating: some errors, like FatalErrorException are not being sent to Bugsnag too. Basically, what i did was run a specific service with a class as argument. I just put "null" instead the class, when allowing errors to display in the page, i've got the errors normally as text in the page. But they' not being sent to Bugsnag.

Interesting. Where abouts did you put this code please. Anything that goes through symfony's error handler should get dealt with by Bugsnag.

Sorry, it wasn't something related specifically to this bundle.
I was using a custom exception controller for Twig and some errors were not being handled properly, blank pages were being shown instead, exactly as posted here and here. The errors were not being logged also.
I'm going to close it. Thanks.


Value is delete sometime unexpectedly

Please consider submitting the following information (if relevant):

Library setup method: file, Carthage, CocoaPods or Swift Package Manager.
Version of the library. Example: 8.0.
Xcode version. Example: 8.3.3.
OS version. Example: iOS 10.3.2.

I used keychain for store identifier device
`
class IOSDeviceManager {
static let shared = IOSDeviceManager()
var identifier: String {
    if let deviceId = keychain.get(deviceIdString) {
        return deviceId
    } else {
        let deviceId = UIDevice.current.identifierForVendor?.uuidString ?? ""
        keychain.set(deviceId, forKey: deviceIdString)
        return deviceId
    }
}

}
`
It worked perfect when I uninstalled then reinstall. But I don't know sometime IOSDeviceManager.shared. identifier get new value.
Do you know any reason?

Hi, thanks for reporting. This could be related to #15. Sometimes the Keychain returns nil, there is a suggested workaround in #78.


chore: Use [].map instead of [...nodes].map

Uses Array.prototype.map directly by providing the NodeList as the this argument.
This is achieved by using the .call method that JS provides.
Array.prototype.map.call([1, 2, 3], console.log);

is the equivalent of
[1, 2, 3].map(console.log)

This is just a minor optimization, but squeezing out as much as we can is not a bad thing.

This is a pretty cool hack and I really can't see why .forEach exists for NodeList but not .map.

I prefer how it used to be. Performance isn't a concern here.

And you shouldn't commit your lock file in PRs.

...you are the one that added the pre-commit hook, it is your doing, not mine
https://github.com/antfu/refined-github-notifications/blob/main/package.json#L37

No, I am using pnpm not npm.

what are you even saying?
how does that relate to you having a pre-commit hook?

and even if it was related in any way, the pre-commit hook uses pnpm

so what you are saying makes no sense even in that scenario

all I did was edit the userscript and make a commit
the commit failed because you have a hook that uses pnpm, i then installed pnpm and retried to make the commit
the hook ran and added the lockfile to the diff

how is that smth I did?
you are the one that added the pre-commit hook in the package.json

do you just not know what a pre-commit hook is?
did you use a template?


Counting specific sound occurances in the audio

I am trying to count the number of specific sound occurrences in the audio
The problem I have is that I am calling TfliteAudio.startAudioRecognition and trying to listen to the events steam I am receiving events every 1 second. And I can't find the possibility to increase events' frequency to receive events every 50-100 ms. Is it possible to decrease interval duration to 50-100 ms?
Another problem I have is that event['recognitionResult'] always returns "1 Result":
result:
{hasPermission=true, inferenceTime=75, recognitionResult=1 Result}
However, there are more than 1 repetitions of sound I am trying to count in each 1-second interval. Should it work like this and what does number "1" means, is this number of the sound in a single audio interval or something else?
Is it possible to implement specific sound counting with this package or I should look somewhere else? Any feedback would be helpful, thanks!

Hi @nazdream,
Looking at your description, it looks like you’re trying to build Sound Event Detection model. Correct me if I’m wrong here.
As for “ 1 Result”, i can check what’s wrong if you’re willing to share your label text file. Let me know if this is possible .

Sure, 5 minutes please, I can upload the label and ts model to google drive and share access to it to this gmail<EMAIL_ADDRESS>Will you be able to access it from there?

@nazdream no problems. It should be fine

I have sent you the invite, can you check if you received it, please?

Here is the code I used for testing the functionality I need:
import 'package:flutter/material.dart';
import 'package:tflite_audio/tflite_audio.dart';

class TestPage extends StatefulWidget {

  @override
  _TestPageState createState() => _TestPageState();
}

class _TestPageState extends State<TestPage> {
  bool _recording = false;
  int _results = 0;
  int _events = 0;
  Stream<Map<dynamic, dynamic>> _result;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: ListView(
        children: [
          const SizedBox(height: 30),
          Center(
            child: Text('Audio'),
          ),
          const SizedBox(height: 30),
          Center(
            child: Container(
              width: 100,
              height: 100,
              decoration: BoxDecoration(
                borderRadius: BorderRadius.circular(100),
                color: _recording ? Colors.red : Colors.blue,
              ),
              child: Center(
                child: Text(_recording ? 'Recording...' : 'Idle'),
              ),
            ),
          ),
          const SizedBox(height: 30),
          Center(
            child: Text('Results: $_results'),
          ),
          Center(
            child: Text('Events: $_events'),
          ),
          const SizedBox(height: 30),
          Padding(
            padding: const EdgeInsets.symmetric(horizontal: 20),
            child: RaisedButton(
              onPressed: _recording ? _stop : _recorder,
              child: Text(_recording ? 'Stop' : 'Record'),
            ),
          ),
        ],
      ),
    );
  }

  void _recorder() {
    if (!_recording) {
      setState(() {
        _recording = true;
        _results = 0;
        _events = 0;
      });

      _result = TfliteAudio.startAudioRecognition(
        numOfInferences: 100,
        inputType: 'rawAudio',
        sampleRate: 44100,
        recordingLength: 44032,
        bufferSize: 22050,
        averageWindowDuration: 10,
        detectionThreshold: 0.6,
        suppressionTime: 10,
        minimumTimeBetweenSamples: 10,

      );

      _result.listen((event) {
        setState(() {
          _events ++;
        });
        if (event['recognitionResult'] == '1 Punch') {
          setState(() {
            _results ++;
          });
        }
      }).onDone(() {
        setState(() {
          _recording = false;
        });
      });
    }
  }

  void _stop() {
    TfliteAudio.stopAudioRecognition();
  }
}

And in the app file I am initializing the model:
void _loadTFModel() async {
    String result = await TfliteAudio.loadModel(
      label: 'assets/ml/labels.txt',
      model: 'assets/ml/soundclassifier.tflite',
      numThreads: 2,
      isAsset: true,
    );
  }

Ah, it seems you are using Google's Teachable Machine.
So, just to clarify a few more things, you want to reduce the recording length from 1000ms to around 50-100ms? Is that correct?

Yes

Or another option is to see how many occurrences of specific sound are in a 1-second interval. If startAudioRecognition event included a number of occurrences of sound in the 1-second interval that also would be perfect.

Yes

In that case, i think it may possible to do so. However, I have yet to test whether it works.
If you don't mind doing the testing for me, I suggest reducing the recordingLength to perhaps half, quarter or one eigth.

Or another option is to see how many occurrences of specific sound are in a 1-second interval. If startAudioRecognition event included a number of occurrences of sound in the 1-second interval that also would be perfect.
Something like this:
 {hasPermission: true, inferenceTime: 71, recognitionResult: 0 Name, occurrences: 5}

This may be possible, but i may need to change the source code around a bit to achieve this effect.

If you don't mind doing the testing for me, I suggest reducing the recordingLength to perhaps half, quarter or one eigth.
I will test this in a moment an provide result here

I have tried reducing recordingLength by 2, 4 and 8 times, but the app is crashing every time with the following error:
E/AndroidRuntime(25618): FATAL EXCEPTION: Thread-7
E/AndroidRuntime(25618): Process: se.boxbollen.flutterapp, PID: 25618
E/AndroidRuntime(25618): java.lang.IllegalArgumentException: Internal error: Failed to run on the given Interpreter: tensorflow/lite/core/subgraph.cc BytesRequired number of elements overflowed.
E/AndroidRuntime(25618):
E/AndroidRuntime(25618): Node number 42 (MAX_POOL_2D) failed to prepare.
E/AndroidRuntime(25618):
E/AndroidRuntime(25618): 	at org.tensorflow.lite.NativeInterpreterWrapper.run(Native Method)
E/AndroidRuntime(25618): 	at org.tensorflow.lite.NativeInterpreterWrapper.run(NativeInterpreterWrapper.java:204)
E/AndroidRuntime(25618): 	at org.tensorflow.lite.Interpreter.runForMultipleInputsOutputs(Interpreter.java:374)
E/AndroidRuntime(25618): 	at org.tensorflow.lite.Interpreter.run(Interpreter.java:332)
E/AndroidRuntime(25618): 	at flutter.tflite_audio.TfliteAudioPlugin.rawAudioRecognize(TfliteAudioPlugin.java:508)
E/AndroidRuntime(25618): 	at flutter.tflite_audio.TfliteAudioPlugin.access$300(TfliteAudioPlugin.java:54)
E/AndroidRuntime(25618): 	at flutter.tflite_audio.TfliteAudioPlugin$4.run(TfliteAudioPlugin.java:452)
E/AndroidRuntime(25618): 	at java.lang.Thread.run(Thread.java:919)
I am using Xiaomi Redmi 7 for testing

Can you tell me your bufferSize? You need to match it with recording length.

I was using 8000 buffer size

Just tried to use some other combinations of recordingLength and bufferSize but the app keeps crashing when I start recording audio. Let me know if you have any ideas on what could be the problem, please

Hi @nazdream,
I will take a look into the source code when i find some spare time. I cannot guarantee a fix, but i will keep you updated.

Thanks!

Hi @nazdream,
After running some tests, i think an inference every 50-100ms (or 10 to 20 inferences per recording) will be extremely taxing for mobile devices. The best i can achieve is around 200ms, and this does not include latency delay for each inference.
What i can do however is increase the sensitivity of bufferSize so that delay for each inference can be minimized.

Hi @Caldarie ! Thanks for the response.
I think that 200ms will not work for me however I can try, maybe that will be enough. I ideally need some solution to be able to detect sound around every 100 ms or less. Is it technically possible to achieve something like this or typical device doesn't have enough resources to achieve such high frequency?

@nazdream I think this may be very difficult to achieve your requirements with GTM models considering its a simple audio classification model.
However, i think it's possible to detect sound every 100ms if you built your own custom model. Though, this may require deep knowledge of machine learning.

How can inference every 50-100ms impact the device? I just need this sound detecting feature to work 30 seconds in a row max, so maybe if that will not impact the device that much.
Can you clarify if I understood you correctly that it's the GMT model which makes processing slow/ power consuming and the custom model can be much faster? Because we can create a custom GMT model if needed, however, I want to understand if this 200ms is defined in the package code or it depends on the model and device benchmark and is unique for every combination or device and tf model?

As I've already explained in comment 13, it's a matter of processing power.
As building a custom model, on second thought, it may also not be possible depending on what you want. Do you want real time results (inference every 100ms?), or do you want 20 results to appear after the recording is finished?

I need real-time results

Ah, in that case,  I think real time results for every 100ms may be very difficult to achieve; even with custom models.
I may be wrong however. You might want to consult the developers for tflite to see if its possible.

I will write them a letter and provide the answer from them when they reply.
Is it possible to detect the number of sounds in 1-second intervals, or it is only possible to count the number of sounds by splitting interval them into smaller intervals and check if there is a specific sound in each of them?

Or another option is to see how many occurrences of specific sound are in a 1-second interval. If startAudioRecognition event included a number of occurrences of sound in the 1-second interval that also would be perfect.
Something like this:
{hasPermission: true, inferenceTime: 71, recognitionResult: 0 Name, occurrences: 5}

Apologies for the late reply.

Is it possible to detect the number of sounds in 1-second intervals

Yes, its possible if you develop your own model. With GTM, its impossible considering it only outputs one result at a time.

it is only possible to count the number of sounds by splitting intervals into smaller intervals and check if there is a specific sound in each of them?

As explained before, this approach is possible but not advisable.

Do I understand that the model output is defined by the model itself and it doesn't depend on the package?

{hasPermission=true, inferenceTime=75, recognitionResult=1 Result}

@nazdream That is correct. You want train a model with multi label classification.
As for the package, I have yet to adapt it for models with multiple outputs. However, I am more than happy to adapt it for you; if you're willing to share your model.


Edited Demons on Holiday Whitelist Token, wrong address added

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

I have opened a new pull request for Editing, should or can I open the old one to edit the address?

Recreateing request

Please remove The added tokenlisting

I have raised a NEW one with the symbol of 'DOHWLT'


Clean up build dependencies

archaius-pull-requests #354 SUCCESS
This pull request looks good

NetflixOSS » archaius » archaius-pull-requests #188 SUCCESS
This pull request looks good


Expose agent's data directory

Proposed changes
Expose the agent's data directory as a property.
Fixes
n/a
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[ ] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
n/a

Codecov Report

Merging #36 (3ed6653) into main (148df5c) will decrease coverage by 0.18%.
The diff coverage is 66.66%.

:exclamation: Current head 3ed6653 differs from pull request most recent head 14674ac. Consider uploading reports for the commit 14674ac to get more accurate results

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   98.36%   98.17%   -0.19%
==========================================
  Files         301      313      +12
  Lines       23856    24893    +1037
==========================================
+ Hits        23465    24438     +973
- Misses        391      455      +64

Flag
Coverage Δ

unittests
98.17% <66.66%> (-0.19%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

aea/skills/base.py
99.77% <66.66%> (-0.23%)
:arrow_down:

aea/__init__.py
100.00% <0.00%> (ø)

packages/fetchai/protocols/gym/dialogues.py
100.00% <0.00%> (ø)

packages/fetchai/connections/gym/__init__.py
100.00% <0.00%> (ø)

packages/fetchai/skills/__init__.py
100.00% <0.00%> (ø)

.../aea-ledger-fetchai/aea_ledger_fetchai/__init__.py
100.00% <0.00%> (ø)

...s/aea-ledger-fetchai/aea_ledger_fetchai/_cosmos.py
91.47% <0.00%> (ø)

packages/fetchai/protocols/__init__.py
100.00% <0.00%> (ø)

packages/fetchai/skills/gym/dialogues.py
100.00% <0.00%> (ø)

...ea-ledger-ethereum/aea_ledger_ethereum/ethereum.py
95.38% <0.00%> (ø)

... and 6 more

Continue to review full report at Codecov.

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


Update LoginPage.md

Fixes issue https://github.com/Badgerati/Pode/issues/703
Description of the Change

Fix file names in the file structure section
Rename filename in the pages section
Add link to the example file in the git repo

Related Issue
https://github.com/Badgerati/Pode/issues/703
Examples
reference: https://github.com/Badgerati/Pode/issues/703#issuecomment-788138562
Additional Context
reference: https://github.com/Badgerati/Pode/issues/703

LGTM! Thanks 😄


[Azure Search] Remap legacy DNS to point to Azure Search Service

There is still large amounts of traffic going to legacy search (App Insights query). Things to do:

Understand CORS policy? Talk to Barry
Consider updating the DNS to point legacy search to azure search. See: https://github.com/nuget/engineering/issues/2841

Raw query:
requests
| where timestamp > ago(1d)
| where name in ("GET /search/diag", "GET /search/query", "GET /query", "GET /autocomplete")
//| where url !contains "/search/query?q=packageid:E2E." // End-to-end tests
| extend dId = tostring(customDimensions.DeploymentId)
| extend Region =
  iff(dId == "a755594844164679a1ca264946f81f18" or dId == "233d868eef7b47ddacbc138a147a50c3", "USNC",
     iff(dId == "86b2d4f718ee498996291ab94a08febf" or dId == "c4e86a9343a44b7ba14593a720e5c13c", "USSC",
       iff(dId == "fe4df56377be435a97eded74a7529f98" or dId == "d293fb80632a41cbbfc535e9adeb5c2c", "EA",
         iff(dId == "15493afe371f43e99924731983887688" or dId == "e3f92c47a56342a0bd36f3f971700184", "SEA", "Unknown")
       )
     )
  )
| extend RegionAndRequestName = strcat(Region, " ", name)
| summarize count() by bin(timestamp, 30m), RegionAndRequestName, Region, name
| order by count_ desc
| render barchart kind=stacked

This investigation needs to be done before we can delete legacy search resources.

Two known internal partners:

Application Insights - internal issue devdiv 1009585
Component governance - internal issue mseng 1640389

This is blocked by https://github.com/NuGet/Engineering/issues/2835.

Both IIS logs and Application Insights corroborate the fact that SEA and EA traffic is extremely low. I will proceed with shutting these down.

CORS is no longer supported. This may be a problem for Azure Portal:
Legacy CORS
Azure Search CORS

Talked offline to Joel:

Use IIS logs to find common referrers
See Joel's notes here
Reach out to Azure Site Extensions and App Insights to confirm domains for CORS policy

If we can convince Barry that we can keep a cors policy for all domains we can probably save a lot of time with emails and analysis

CORS for all domains is deployed to all environments. The remapping can commence.

This is done on both PROD USNC and PROD USSC.


add Let's Encrypt ssl  Prompt for this error，How can I solve this problem

add Let's Encrypt ssl  Prompt for this error，How can I solve this problem
Error: Command failed: certbot certonly --config "/etc/letsencrypt.ini" --cert-name "npm-24" --agree-tos --email<EMAIL_ADDRESS>--domains "*.weerr.cn" --authenticator dns-dnspod --dns-dnspod-credentials "/etc/letsencrypt/credentials/credentials-24"
An unexpected error occurred:
ModuleNotFoundError: No module named 'zope'
Ask for help or search for solutions at https://community.letsencrypt.org. See the logfile /tmp/certbot-log-x1vhtsg9/log or re-run Certbot with -v for more details.
at ChildProcess.exithandler (node:child_process:402:12)
at ChildProcess.emit (node:events:513:28)
at maybeClose (node:internal/child_process:1100:16)
at Socket. (node:internal/child_process:458:11)
at Socket.emit (node:events:513:28)
at Pipe. (node:net:301:12)

hope this get fixed soon, this also shows "Communication with the API failed" error. maybe it got something to do with the api

I'm getting the "Communication with API failed, are you sure NPM is running correctly" error too, I can't request any new certificate nor renew my existing ones. I didn't change anything in my docker compose or anything, this issue just came up by itself. I also can't find any solution on this on the internet.

same for me!

I'm getting the "Communication with API failed, are you sure NPM is running correctly" error too, I can't request any new certificate nor renew my existing ones. I didn't change anything in my docker compose or anything, this issue just came up by itself. I also can't find any solution on this on the internet.

I "solved" the issue by destroying my docker container, destroying the proxmox lxc where npm was hosted, and starting all over again. I downloaded the certificates before, so I applied them as custom now. Took me some time, but luckily I was managing just 15 hosts so not to big of a deal, But I still wonder how can something break, if I don't touch any config file.

Test server reachability do not working
[3/4/2023] [11:26:38 AM] [SSL      ] › ℹ  info      Renew Complete
`QueryBuilder#allowEager` method is deprecated. You should use `allowGraph` instead. `allowEager` method will be removed in 3.0
`QueryBuilder#eager` method is deprecated. You should use the `withGraphFetched` method instead. `eager` method will be removed in 3.0
QueryBuilder#omit is deprecated. This method will be removed in version 3.0
Model#$omit is deprected and will be removed in 3.0.
[3/4/2023] [11:33:24 AM] [SSL      ] › ℹ  info      Testing http challenge for xxx.xxx.cn
Uncaught SyntaxError: Unexpected end of JSON input

FROM
./run: line 19:   617 Trace/breakpoint trap   (core dumped) node --abort_on_uncaught_exception --max_old_space_size=250 index.js

I change to use traefik with let`s encrypt, and it work finesl


Errors in APIM module: resource_group_key & internal vnet configuration

There are a few bugs when I want to deploy API management.

When configuring virtual_network_type as internal, the virtual_network_configuration block is required. But this part of the module is throwing errors:

│ Error: each.value cannot be used in this context
│
│   on /home/vscode/.terraform.cache/kbtest/modules/solution/modules/apim/api_management/module.tf line 218, in resource "azurerm_api_management" "apim":
│  218:         try(each.value.subnet_id, null),
│
│ A reference to "each.value" has been used in a context in which it unavailable, such as when the configuration no longer contains the value in its "for_each" expression. Remove this reference to each.value in your
│ configuration to work around this error.
╵
╷
│ Error: Reference to undeclared input variable
│
│   on /home/vscode/.terraform.cache/kbtest/modules/solution/modules/apim/api_management/module.tf line 219, in resource "azurerm_api_management" "apim":
│  219:         try(var.vnets[virtual_network_configuration.value.lz_key][virtual_network_configuration.value.vnet_key].subnets[virtual_network_configuration.value.subnet_key].id, null),
│
│ An input variable with the name "vnets" has not been declared. This variable can be declared with a variable "vnets" {} block.
...

I think the variable declaration 'vnets' is missing and 'each.value.subnet_id' should be 'virtual_network_configuration.value.subnet_id' in the module code:
dynamic "virtual_network_configuration" {
    for_each = try(var.settings.virtual_network_configuration, null) != null ? [var.settings.virtual_network_configuration] : []

    content {

      subnet_id = coalesce(
        try(each.value.subnet_id, null),
        try(var.vnets[virtual_network_configuration.value.lz_key][virtual_network_configuration.value.vnet_key].subnets[virtual_network_configuration.value.subnet_key].id, null),
        try(var.vnets[var.client_config.landingzone_key][virtual_network_configuration.value.vnet_key].subnets[virtual_network_configuration.value.subnet_key].id, null)
      )
    }
  }

I must set the resource group key in 2 different ways to make it work.

My configuration:
api_management = {
  apim01 = {
    name = "apim-demo"
    resource_group_key = "rg1"
    resource_group = {
      key = "rg1"
    }

    publisher_name  = "test"
    publisher_email =<EMAIL_ADDRESS>
    sku_name = "Developer_1"

    virtual_network_type = "Internal"
    virtual_network_configuration = {
      conf1 = {
        vnet_key   = "vnet01"
        subnet_key = "snet01"
      }
    }

    identity = {
      type                  = "UserAssigned"
      managed_identity_keys = ["mi1"]
    }
  }
}

Could we have some help on this topic

@SergioMiyama can you please have a look? thanks


Define and get feedback on KJ user story

The KJ user story at https://ryanolds.invisionapp.com/freehand/Club-UI-wireframe-pRcN8AczK?v=Hke6bxk5AxtHMnISD960OQ%3D%3D&linkshare=urlcopied is just a stub. Talk with the team in Discord and build out the KJ user story.
What does the KJ see when they first access the room?
How do they manage and advance the queue?
What do they see when they advance the queue?
Should the KJ and singer have a 1-on-1 before being broadcasted to everyone in the room?
What audio controls do they need to improve audio quality?
What controls should we provide to the KJ so they can handle a "singer" ranting about 5G/fluoride?

If you need access to InVision send @ryanrolds your email address.

No longer needed, the karaoke feature is being cut due to a lack of interest.


Fixes tag control multi-value

Drop-downs have multi-select functionality as of https://github.com/ManageIQ/manageiq-ui-classic/pull/114. This fixes an issue where tag control, which is sub-classed from the same base as drop-down, lacks the necessary method for multiple items on the same field.
Depends on https://github.com/ManageIQ/manageiq-ui-classic/pull/729

@d-m-u Let's get a test around this please.

Euwe backport details:
$ git log -1
commit ab0b9c9ae8d9c77bd1636188a280c1c06ec33d1a
Author: Greg McCullough<EMAIL_ADDRESS>Date:   Wed Mar 22 17:49:42 2017 -0400

    Merge pull request #14382 from d-m-u/tagControlMultiFix

    Fixes tag control multi-value
    (cherry picked from commit b9de782ab3bd7c0b5d64821f4539b8a777cacefa)

    https://bugzilla.redhat.com/show_bug.cgi?id=1436223


Some questions about the Useage

Can this toolkit be used for model forward process?

yes, the  soft kmeans algorithm is differentiable through the fixed points.
Please see the corresponding paper.

Thanks a lot, my problem has been solved! This package is convenient and excellent!

Thanks a lot, my problem has been solved! This package is convenient and excellent!
发件人: jokofa
发送时间: 2023年11月23日 18:54
收件人: jokofa/torch_kmeans
抄送: Xiao JH; Author
主题: Re: [jokofa/torch_kmeans] Some questions about the Useage (Issue #8)
Closed #8 as completed.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Add library-format flag and fix Tensile issues

Makes yaml default for Tensile, and msgpack default for TensileCreateLibrary
Adds library-format flag, and new Tensile_LIBRARY_FORMAT cmake variable (will need to be added to rocBLAS)
Works with NewClient: 0, 1, and 2 for both msgpack and yaml

Never mind, looks like TensileCreateLibrary is broken since I fixed the merge conflicts. Something about LibraryFormat being unrecognized. Will leave that for Monday


Add tab_complete packet

im not sure why but bot.end works better than bot.quit

Seems like some tests failed. Was the wait 5s needed ?


Patch some files Bitwarden addon

first of all, thanks for the addon which works very well.
I wanted to contribute by modifying the code to switch to the new base "12" but without success, there is always the error message "s6-overlay-suexec: fatal: can only run as pid 1".

Hi 👋,
Thanks for this contribution ❤. I'm unsure if your changes are working because of your comment. I only accept working and tested PR's. Can you please elaborate if this PR is working or not and what changes you want to make.


Add ability to generate new artifact type

The CNCF wg-wasm has published an OCI artifact format for packaging wasm modules and components.  The artifact can be produced locally by running the --as-artifact flag:
cargo run --bin oci-tar-builder -- --name wasi-demo-oci --repo ghcr.io/containerd/runwasi --tag latest --as-artifact --module ./target/wasm32-wasi/debug/wasi-demo-app.wasm -o target/wasm32-wasi/debug/img-oci-artifact.tar
regctl image import localhost:5000/wasi-artifact:latest target/wasm32-wasi/debug/img-oci-artifact.tar

regctl manifest get localhost:5000/wasi-artifact:latest

Name:                                localhost:5000/wasi-artifact:latest
MediaType:                           application/vnd.oci.image.manifest.v1+json
Digest:                              sha256:7c31e635b3bef8b6c727a316e9a2dae777dbd184318d66a97da040fb11e37d70
Annotations:
  io.containerd.image.name:          ghcr.io/containerd/runwasi/wasi-demo-oci:latest
  org.opencontainers.image.ref.name: latest
Total Size:                          2.006MB

Config:
  Digest:                            sha256:24f30be41b447bbaf3644dad1e1c23dd28b597f36a5f455399657d65945816ea
  MediaType:                         application/vnd.wasm.config.v0+json
  Size:                              235B

Layers:

  Digest:                            sha256:0db51ed1c94837f422b2259c473758f298eef69605eaae6195bc043e25971e94
  MediaType:                         application/wasm
  Size:                              2.006MB

Note: this does not add support to runwasi to run these yet.  Some work in Containerd is required: https://github.com/containerd/containerd/issues/10179

I published an example to https://github.com/users/jsturtevant/packages/container/package/wasi-artifact
Can use the new tooling wkg to pull and run:
wkg oci pull ghcr.io/jsturtevant/wasi-artifact:latest
wasmtime run jsturtevant_wasi-artifact.wasm

This is a song that never ends.
Yes, it goes on and on my friends.

I am not sure why the smoke test is failing, I've run the same tests locally and it doesn't timeout.  I've also not modified any the shim or the test image.

it is also only failing on ubuntu 20.04

The musl build of wasmtime and the image built with these changes is working on ubuntu 20.04 for me locally

/assign @devigned


Usage of "numbering-start-at" unclear.

Hello everybody,
I've waited for Pull#1649 because I need to start my page numbering on page 2 (with number 2), but this is a little unclear to me.
I've made following example.
page.adoc
= Testpage
:toc: macro
:sectnums:

<<<
toc::[]
<<<
== page one
<<<
== page two
<<<
== page three

theme.yml
extends: default

running-content:
  start-at: 2

page:
  numbering-start-at: 2

header:
  height: 0.75in
  line-height: 1.2
  recto:
    center:
      content: '{page-number}'
  verso:
    center:
      content: '{page-number}'

I would except my page numbering starts on page 2 with number 2 but it is still on page 1.
It would work if I set the :title-page: true within the adoc file, but I need to customize it too much.
So my question is: What am I doing wrong? Did I misunderstood the pull  #request?
EDIT:
asciidoctor-pdf --version
Asciidoctor PDF 1.5.3 using Asciidoctor 2.0.10 [https://asciidoctor.org]
Runtime Environment (ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux-gnu]) (lc:UTF-8 fs:UTF-8 in:UTF-8 ex:UTF-8)

That change is not yet available in a release. You'll either need to wait for Asciidoctor PDF 2 to be released or make the case that it should be backported to 1.5.x and released as part of a 1.5.4 release.
The docs that go with the release are here: https://github.com/asciidoctor/asciidoctor-pdf/blob/v1.5.x/docs/theming-guide.adoc#key-prefix-page
In the note, it clearly states that the body offset only works if the document has a title page. In Asciidoctor PDF 2, it will work even if the document does not have a title page.

I thought it was released.
I checked out master and build it like descripted in https://github.com/asciidoctor/asciidoctor-pdf/blob/master/CONTRIBUTING-CODE.adoc
$ bundle --path=.bundle/gems
$ bundle exec rake install
$ gem install pkg/*.gem

Thanks for help!
I close the issue now.


How do you open a .hive file?

How can I open a .hive file in Android Studio so I can view the contents of my Hive boxes? I have tried opening the file as JSON/TXT/XML and nothing works.
Using Android Studio on Ubuntu with Flutter

Hive uses custom binary format for storing data which doesn't supported by any exists editors or tools.

We can use it as a preview version https://studio.hivedb.dev/#/
Unfortunately, there is no scroll in it added comment in https://stackoverflow.com/a/75022792/2462531


Added check for secure connection in all methods.

Every REST API method that needs a jwt token is now checked against
secure connection in the require_token decorator. Every other REST API
method has a require_https decorator. For other request the control is
inside the method.

Just some minor remark (check the comments). Also, please have a warning (or an assert) in the test runner where we make the tests fail if we run them in debug mode due to the likely possibility of false positives.
Thanks!

LGTM


eslint fails on fresh clone

Hi! nice boilerplate, especially the prettier/eslint rules.
I cloned the project and made no changes to give it a try.
Running pnpm lint shows 38 Problems, most related to cypress, the others related to spec files:
> eslint .

/workspaces/angular-boilerplate/cypress.config.ts
   6:25  error  'on' is defined but never used      @typescript-eslint/no-unused-vars
   6:29  error  'config' is defined but never used  @typescript-eslint/no-unused-vars
  18:1   error  Delete `⏎`                          prettier/prettier

/workspaces/angular-boilerplate/cypress/support/component.d.ts
  0:0  error  Parsing error: ESLint was configured to run on `<tsconfigRootDir>/cypress/support/component.d.ts` using `parserOptions.project`: <tsconfigRootDir>/cypress/tsconfig.json
However, that TSConfig does not include this file. Either:
- Change ESLint's list of included files to not include this file
- Change that TSConfig to include this file
- Create a new TSConfig that includes this file and include it in your parserOptions.project
See the typescript-eslint docs for more info: https://typescript-eslint.io/linting/troubleshooting#i-get-errors-telling-me-eslint-was-configured-to-run--however-that-tsconfig-does-not--none-of-those-tsconfigs-include-this-file

/workspaces/angular-boilerplate/cypress/support/component.ts
  9:31  error  Unsafe argument of type `any` assigned to a parameter of type `CommandFn<keyof Chainable<any>>`  @typescript-eslint/no-unsafe-argument

/workspaces/angular-boilerplate/src/app/app.component.spec.ts
  14:9  error  Unsafe call of an `any` typed value  @typescript-eslint/no-unsafe-call

/workspaces/angular-boilerplate/src/app/lib/components/footer/footer.component.spec.ts
  23:9  error  Unsafe call of an `any` typed value  @typescript-eslint/no-unsafe-call

/workspaces/angular-boilerplate/src/app/lib/components/layouts/layout-horizontal/layout-horizontal.component.spec.ts
  20:9  error  Unsafe call of an `any` typed value  @typescript-eslint/no-unsafe-call

/workspaces/angular-boilerplate/src/app/lib/components/logo/logo.component.spec.ts
  20:9  error  Unsafe call of an `any` typed value  @typescript-eslint/no-unsafe-call

To ensure this isn't a local issue on my setup, I decided to use a Github Codespace and installed node 18.10.0.
Ran pnpm install, then pnpm build, and pnpm lint, still the same issues showed up.
Is this expected? What is the fix?
Local setup:
Windows 10, node v18.18.2
Github Codespaces:
Ubuntu 22.04.3 LTS, node v18.10.0

@Clemens-E sorry for the delay and thank you for reporting this 🙂. Will try to fix it today.

Hi, this is still an issue


Update app.yml

Added hindi translator

All changes applied to #1343, i'll keep this PR open for reference until the former is merged.


Runner.run parameter not available

Hi,
In the documentation it's said that it's possible to use options to configure the runner. I would like to use the disableMultipleWindows option
runner.run({ disableMultipleWindows: true });

But I get the message :
Argument of type '{ disableMultipleWindows: boolean; }'
is not assignable to parameter of type 'Partial<RunOptions>'

Is this a bug or is this option deliberately not activated yet?
I am using the latest stable release v1.14.2

Thank you for submitting this issue. I'm happy to help you diagnose this behavior, but I couldn't reproduce it. Please describe your environment in greater detail and share a sample project and tests. Then, we will be able to determine the cause of the issue. I look forward to your response.

Hi,
here is my simple example:
disableMultipleWindows.zip
As I already mentioned in my question I use v1.14.2

Thank you for your example. I reproduced the issue. The disableMultipleWindow option is missing in type definitions. As a workaround, you can specify it in the following way:
    const options: any = { disableMultipleWindows: true };

    return runner.run(options);

Hi @Dmitry-Ostashev,
Thanks for the quick answer. It works well !
I had seen that the option was missing in the type definitions but I didn't know if it was intentional or not. I'm glad I could help you to detect this oversight :)

Fixed in<EMAIL_ADDRESS>


[BUG] Azure.Tables Throws Exception when TableEntity contains enum

I'm calling UpsertEntityAsync with a class that has an enum property.
[JsonPropertyName("type")]
public ImageType Type { get; set; } = ImageType.New;

Getting this error:
An exception of type 'System.NotSupportedException' occurred in System.Private.CoreLib.dll but was not handled in user code: 'Not supported type Lib.ImageType'

Do we support Enum properties?
If yes, how.
If no, what is the workaround?
azsdke2e
@christothes

Looks like this is being thrown here.  Investigating the right approach to support enum values and how this would work cross language.


Investigate if makes sense to create a lambda function to  run the test script to probe the Service control policies (SCP)

Describe the Issue
As a QA, I would like that anybody in the team can easily run the test scripts.
Currently, to run the script you need to download the code and run it from your machine. It also requires to supply the appropriate AWS credentials. This makes it difficult to run, specially for Live (currently the tester has to ask the SysOp to run it , as the tester does not have Live master credentials)
Additional Context
See the first iteration of this ticket for more information #2251
Acceptance Criteria

Check if it is technically possible to run the python scripts as a lambda function without creating security risks
Check with the team if it makes sense to implement such function (easy to trigger the function, function does not interfere with other artifacts, code control of the function....)
If it goes ahead, create the ticket to port the test script to Lambda.

See #2316, unfortunately, it does not make sense to create such lambda function


Scriptables' context is missing elements and options

I'm using a scriptable display property to control when annotations can be shown, based on their options. (This is the same use case I'd originally asked about in #244.) This was working in an earlier copy of chartjs-plugin-annotation (circa December 2020), but in the current 1.0.0-rc, it's giving problems:

The first time display is called, the context parameter's element member is completely missing.
The second time it's called, element is present, but it has no options member.

If display is called again (for example, because the chart updates from showing or hiding a dataset), then element.options is correctly populated.
See https://codesandbox.io/s/chartjs-annotation-context-c95k6 for a demo. The console will show error messages when the page is first loaded. Clicking a dataset's legend item to show or hide the dataset will cause the annotations to be correctly shown or hidden.

@joshkel I think the issue is here:
https://github.com/chartjs/chartjs-plugin-annotation/blob/b9bc2955d20607d9b6d4e336b2fe34dde153a326/src/annotation.js#L65
The display property is used and being in a proxy options, the callback is invoked by chart context type.
Add console.log(context.type); in your callback and you can see the type == chart instead of annotation (the first call).
@kurkle I think it's a bug. The elements are created in the afterUpdate hook which is invoked after  afterDataLimits where the options display is used. Before the afterUdate we must be sure do not invoke any options to avoid this issue. Make sense?
I think it sounds related to this issue and the behavior of the proxy options: https://github.com/chartjs/Chart.js/issues/8623

Thanks, @stockiNail . I tracked down the issue as far as that but realized I didn't understand Chart.js's proxied context implementation well enough (is there documentation on it?), so I wasn't confident of a fix.
Would it work to create (mostly empty?) elements objects and initialize their contexts sooner (e.g., from beforeUpdate, when the annotations array is first set)?
#300 was a similar issue from before the proxy options implementation.

(is there documentation on it?),

I'm not sure about documentation because I have studied it (from the code) for my lib because I didn't recognize at the beginning its behaviors.

Would it work to create (mostly empty?) elements objects and initialize their contexts sooner (e.g., from beforeUpdate, when the annotations array is first set)?
#300 was a similar issue from before the proxy options implementation.

I think your proposal didn't solve completely this issue.
The issue is not related to the missing element, but the callback is invoked with a chart context.
The resolveAnnotationOptions(resolver) function should be executed in beforeUpdate otherwise the callback assigned to display will be invoked always with a chart context, instead of annotation one (proxy behavior).
Nevertheless the workaround is to add a check in your callback to test if the context you are receiving as argument is consistent (annotation type):
const display = (context) => {
  if (context.type !== 'annotation') {
    return true; // checks the scales for all annotation but it should work (I didn't test)
  }
  // your logic here when context is annotation type
};

About the missing options, the callbacks are invoked before the options are added to element (the first call). Maybe that part of code could be reviewed.

I think there are other statements in the code where this issue could occur (where an option is invoked before resolveAnnotationOptions function and could be scriptable):
function getScaleLimits(scale, annotations) {
  const axis = scale.axis;
  const scaleID = scale.id;
  const scaleIDOption = axis + 'ScaleID';
  let min = valueOrDefault(scale.min, Number.NEGATIVE_INFINITY);
  let max = valueOrDefault(scale.max, Number.POSITIVE_INFINITY);
  for (const annotation of annotations) {
    if (annotation.scaleID === scaleID) {    // <-- if 'scaleID' is a scriptable options
      for (const prop of ['value', 'endValue']) {
        const raw = annotation[prop];    // <-- if 'value' and/or 'endValue' are scriptable options
        if (raw) {
          const value = scale.parse(raw);
          min = Math.min(min, value);
          max = Math.max(max, value);
        }
      }
    } else if (annotation[scaleIDOption] === scaleID) {  // <-- if the axis+'scaleID' is a scriptable options
      for (const prop of [axis + 'Min', axis + 'Max', axis + 'Value']) {
        const raw = annotation[prop];  // <-- if the axis+'Min', axis+'Max' and/or axis+'Value' are scriptable options
        if (raw) {
          const value = scale.parse(raw);
          min = Math.min(min, value);
          max = Math.max(max, value);
        }
      }
    }
  }
  return {min, max};
}

display:(context,options) => ...

Also the display property was missed in docs:
https://www.chartjs.org/chartjs-plugin-annotation/guide/options.html#option-context

Thanks, @kurkle.
So, is the current behavior desired? If so, the TypeScript type definitions should be changed?
// Some scriptable options may be invoked before the element is fully initialized.
export interface PartialEventContext {
	chart: Chart,
	element?: Partial<AnnotationElement>,
}

export interface CoreAnnotationOptions extends AnnotationEvents {
  display?: boolean | ((context: PartialEventContext, options: AnnotationOptions) => boolean);

I can submit a PR for this if you like.
I noticed another type-related issue while investigating this issue: context.element.options.type is one of the ...Annotation.id values (e.g., boxAnnotation), while options.type is an annotationTypes / AnnotationTypeRegistry key (e.g., box). However, the TypeScript type definitions say that it's always an AnnotationTypeRegistry value. I can open a separate issue for this or try and open a PR for this as well if you like.

Not sure if its desired. Its a compromise. Earlier, all of the element options were resolved early, before the scales were laid out etc, so you could not get a tick value for example for annotation value. So its desirable to resolve the options that can be resolver later, when then scales are laid out etc.
But then the adjustScaleRanges need to be done before the scales are laid out..
I think the current way covers more use cases, although adding the confusion to options being resolved with different contexts.
Another option would be to create the elements early, but not resolve the options. But I'm not sure an empty element with no valid properties would be useful.
So.. the behavior is there, and the types are not aligned, so I'd welcome a PR for those.
The type issue could be a bug.

Thanks.
I updated the type definitions and added some comments to the docs to try and explain these discussions. Feedback welcome.

I can open a separate issue for this or try and open a PR for this as well if you like.

Separate issue would be good. This can be closed after that?

I created #388 to track the options.type discrepancy. I apologize for the delay in getting back to this. Thanks for your help!


FINAL CHECK: Use MySQL and the Sequelize ORM for the database.

GF and MC set up backend with MySQL and Sequelize


Make figures for Sphinx

This would fi× #174 but I suspect a manual step is also required somewhere.

No problem, this is much better than the current workflow of manually running each and every example.


Improved parsing and formatting of URI components

@carterkozak Please review.

I will review this next week. I already see issues.

@ok2c Should this be merged before Carter's work?

LGTM

I believe the plan was to merge this before my change -- after a rebase it should make my modifications much simpler.

Looking at HttpHost again, it can wipe internal fields as well for a Host instance just like you have done with URIAuthority.

Looking at HttpHost again, it can wipe internal fields as well for a Host instance just like you have done with URIAuthority.

@michael-o What would be the benefit of doing so?

Looking at HttpHost again, it can wipe internal fields as well for a Host instance just like you have done with URIAuthority.

@michael-o What would be the benefit of doing so?

Code reduction and reuse. Plus a clear logical hierarchy (not talking about inheritance).

Code reduction and reuse. Plus a clear logical hierarchy (not talking about inheritance).

@michael-o I am not entirely sure if it is really worth it but so be it. Please review.


Running an image built in previous step doesn't work anymore

This part of the documentation does not work as expected anymore.
- uses: docker/build-push-action@v1
  with:
    repository: test-image
    push: false
- uses: addnab/docker-run-action@v3
  with:
    image: test-image:latest
    run: echo "hello world"

docker/build-push-action gives the below warning while running action.
Warning: Input 'repository' has been deprecated with message: v2 is now available through docker/build-push-action@v2

Thanks. I've updated the README.

@pmoosh: you likely need to specify load: true for build-push-action. For example:
      - name: Build Docker image
        uses: docker/build-push-action@v2
        with:
          tags: <your-tag>
          load: true
          push: false

I discovered this when I was trying to debug the same issue you were having. Running docker images didn't list the tagged image. I happened to decide to read the documentation for docker buildx build and found the following:

Load the single-platform build result to docker images (--load)
Shorthand for --output=type=docker. Will automatically load the single-platform build result to docker images.

In documentation for build-push-action, the default for load is false.
@addnab: maybe the example in the README could be updated?
#### run an image built by a previous step
```yaml
– uses: docker/build-push-action@v2
  with:
    tags: test-image:latest
    push: false
+   load: true
– uses: addnab/docker-run-action@v3
  with:
    image: test-image:latest
    run: echo "hello world"


Error: P1001: Can't reach database server when including large table

Bug description
Bug: I am trying to query a table with around 10,000 rows and join a table with ~100,000 rows on it. Im expecting this query to run successfully or timeout, but it will not even show up in the query log as it will hit the error right after the graphql prisma:client request is printed out. The error does not occur when limiting the request to less rows.
What i have tried: I have tried increasing the timeout or check if something is going wrong at the database level, but the raw query runs fine, when ran manually.
Raw error:
{
    "errorMessage": "\nInvalid `prisma.item.findMany()` invocation:\n\n\n  Can't reach database server at `localhost`:`5432`\n\nPlease make sure your database server is running at `localhost`:`5432`.",
    "errorType": "PrismaClientKnownRequestError3",
    "stackTrace": [
        "Error: ",
        "Invalid `prisma.item.findMany()` invocation:",
        "",
        "",
        "  Can't reach database server at `localhost`:`5432`",
        "",
        "Please make sure your database server is running at `localhost`:`5432`.",
        "    at cb (/Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/node_modules/@prisma/client/runtime/index.js:36374:17)",
        "    at processTicksAndRejections (node:internal/process/task_queues:94:5)",
        "    at async /Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/dist/apps/updatePrices/main.js:219:27",
        "    at async AppService.run (/Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/dist/apps/updatePrices/main.js:1482:23)",
        "    at async handler (/Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/dist/apps/updatePrices/main.js:1910:5)"
    ]
}

How to reproduce

Create a PostgreSQL 12 database with the default image via docker-compose
Create 2 tables (Both with a combined PK and ~20columns)
Link them with a one-to-many relationship
Insert a couple (>100k) rows in both tables
Run a findMany query on the first table and include the second

Expected behavior
Im expecting the query to either resolve or timeout.
Prisma information
  prisma:client:fetcher Error: Can't reach database server at `localhost`:`5432`
  prisma:client:fetcher
  prisma:client:fetcher Please make sure your database server is running at `localhost`:`5432`.
  prisma:client:fetcher     at BinaryEngine.graphQLToJSError (/Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/node_modules/@prisma/client/runtime/index.js:29972:16)
  prisma:client:fetcher     at BinaryEngine.request (/Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/node_modules/@prisma/client/runtime/index.js:29857:24)
  prisma:client:fetcher     at processTicksAndRejections (node:internal/process/task_queues:94:5)
  prisma:client:fetcher     at async cb (/Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/node_modules/@prisma/client/runtime/index.js:36344:26)
  prisma:client:fetcher     at async /Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/dist/apps/updatePrices/main.js:219:27
  prisma:client:fetcher     at async AppService.run (/Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/dist/apps/updatePrices/main.js:1482:23)
  prisma:client:fetcher     at async handler (/Users/nschroeter/Documents/workspace/RamboTrades/hercules-serverless/dist/apps/updatePrices/main.js:1910:5)

Environment & setup

OS: Mac OS
Database: PostgreSQL
Node.js version: 14
Other: NestJS v8

Prisma Version
prisma               : 2.29.1
@prisma/client       : 2.29.1
Current platform     : darwin
Query Engine         : query-engine 1be4cd60b89afa04b192acb1ef47758a39810f3a (at node_modules/@prisma/engines/query-engine-darwin)
Migration Engine     : migration-engine-cli 1be4cd60b89afa04b192acb1ef47758a39810f3a (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine : introspection-core 1be4cd60b89afa04b192acb1ef47758a39810f3a (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary        : prisma-fmt 1be4cd60b89afa04b192acb1ef47758a39810f3a (at node_modules/@prisma/engines/prisma-fmt-darwin)
Default Engines Hash : 1be4cd60b89afa04b192acb1ef47758a39810f3a
Studio               : 0.419.0
Preview Features     : filterJson, referentialActions

Do you run that query right after you added all this data or are these two separate scripts?
Can you provide a repository with the scripts and data you are testing this with?

Do you run that query right after you added all this data or are these two separate scripts?
Can you provide a repository with the scripts and data you are testing this with?

This happens at any point when querying the data. Ive done some digging and it seems related to prisma translating the include statement into 2 seperate SQL statements. The second one being a WHERE IN with a huge amount of data from the first query. Ive setup a minimum reproduction repo here: https://github.com/nwidynski/reproduction-prisma-8878

Thanks so much, that will help us a lot. The error message is probably that misleading as the database connection "dies" for some reason. We will definitely look into that with your reproduction. Thanks!

@janpio any progress on this? It kinda renders prisma unusable for us.

Hey @nwidynski
Sorry for the delay here. I actually forgot to post my findings here. I wasn't able to reproduce this when I attempted this locally. I tried multiple time but it didn't crash for me.
What I will suggest here would be to upgrade to v3 which has some performance improvement and set the QUERY_BATCH_SIZE environment variable to QUERY_BATCH_SIZE=5000 (or depending upon your postgres installation) to limit the number of items in IN clause. Using JOINs for many such queries a performance optimization that we will tackle later on.
If you are still hitting this even after the above, please do let me know.

I think, this is related/identical to #8832 and #9275. A reproduction repo exists in #8832.

Have you tried the fixes mentioned above? I haven't got around to implementing those. It could be related to the database running inside docker, since im also doing that? Also good to know that upgrading to v3 won't fix the issue.

Not the person you replied to, but I don't think docker is the issue. I had the error on a local db and a heroku managed db. #9275

As posted before, closely related to https://github.com/prisma/prisma/issues/9275 but I will keep this one open as some additional information is available here.

I just ran into this and setting QUERY_BATCH_SIZE=5000 fixes it for me. Thanks for saving me a lot of time @janpio and @pantharshit00

Do you have a reproduction by chance @cyrus-za? That would still be super helpful here.

Have you seen the two reproduction repo already posted more then six months ago? https://github.com/prisma/prisma/issues/8832#issuecomment-903251153
https://github.com/prisma/prisma/issues/8878#issuecomment-905291403
What is missing to reproduce?

The one in this thread was tried by @pantharshit00 above and could not be reproduce, hence this issue is back to status bug/0-unknown. Even if the other one in #8832 would be reproducible, situations are often slighty different so when someone pops into a thread saying "I have the same problem and could work around it", asking for their reproduction it still helpful.

I am going to close this as its been 3 months now and there is no activity here.
Please reply if you want us to take another look.

If you stumble upon this issue, please see this comment.

We have closed the original issue in<EMAIL_ADDRESS>please see this comment.


Swap fees from two OSMO/LUNC pools

There are 2 OSMO/LUNC pools, one has a swap fee of 8% and another has a swap fee of 0.2%. The pool that has 8% fee always be picked when trying to swap between Osmo and Lunc
A user reported massage :
"Make swap on swap-tab from LUNC to asset 2. Most optimal swap is misrepresented; screenshots attached. My explanation is as follows > The swap router is optimizing pre-Cost_of_execution, leading to incorrect optimization of swap pathing. 1000 LUNC > OSMO is more often than note being routed through pool 800, an 8% swap fee lunc/osmo pool because the router 'thinks' it's cheaper; until swap fee is factored in it most usually isn't cheaper than directly swapping through pool 561 (.3% lunc/osmo pool)
Below's images do not fully exemplify this 'bug' because the swap tab found the most optimal route; this is not always the case however it is quite a large spread because of the 8% swap fee's skew."

There are 2 OSMO/LUNC pools, one has a swap fee of 8% and another has a swap fee of 0.2%. The pool that has 8% fee always be picked when trying to swap between Osmo and Lunc
A user reported massage : "Make swap on swap-tab from LUNC to asset 2. Most optimal swap is misrepresented; screenshots attached. My explanation is as follows > The swap router is optimizing pre-Cost_of_execution, leading to incorrect optimization of swap pathing. 1000 LUNC > OSMO is more often than note being routed through pool 800, an 8% swap fee lunc/osmo pool because the router 'thinks' it's cheaper; until swap fee is factored in it most usually isn't cheaper than directly swapping through pool 561 (.3% lunc/osmo pool) Below's images do not fully exemplify this 'bug' because the swap tab found the most optimal route; this is not always the case however it is quite a large spread because of the 8% swap fee's skew."

Hi @arispech  kindly visit the official COSMOS page to start a chat with an agent on a live chat support to report this for review and for further assistance via LIVE CHAT

This can be fixed with custom filtering before passing pools to router.


[mobile] homepage + active livestreams - 1 getting filtered

If you have 2 active livestreams on your homepage, 1 gets filtered, so only 1 shows. It briefly shows and then disappears. Works on following page where we have the horizontal view.

What's missing in my steps?  The livestream still shows:
https://user-images.githubusercontent.com/64950861/157999162-d5a0122c-141a-4382-902a-8b38ee8555fe.mp4

Sorry, should have been more specific about mobile mode. But i also had another case where it looked like it may have been working. Will try to repro again tomorrow.

I think this may be related to the ad being hidden and therefore the tile also. Similar to the fix you made in the other area?
Have 5 streams on desktop, and only 4 on mobile, still 2nd one hidden.


Fixed infinite loop, now = finite loop

Description
Fixed infinite loop.
Type of change
Please delete options that are not relevant.

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

Change Status

[X] Complete, tested, ready to review and merge

How Has This Been Tested?

[x] Test A
[x] Test B

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] My code has been reviewed by at least one peer
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] There are no merge conflicts

This is what the Network tab in 'inspect' looks like without id placed on line 33 of index.js in Project/Queries. There is an endless loop.

export const NOTE_FEED_QUERY = gql`
  query NoteFeed($id: ID!, $privatePerm: Boolean) {
    me {
      id
      email
    }
    project(id: $id) {

      projectManagers {
        name
        id
        email
        avatar
      }
      notes(orderBy: updatedAt_DESC, privatePerm: $privatePerm) {
        id
        topic
        content
        author {
          id
          email
          name
        }
        attendedBy {
          id
          name
          email
          avatar
        }
        rating
        privateNote
      }
    }
  }
`;

This is what it looks like once id is added. The infinite loop is fixed.

export const NOTE_FEED_QUERY = gql`
  query NoteFeed($id: ID!, $privatePerm: Boolean) {
    me {
      id
      email
    }
    project(id: $id) {
      id
      projectManagers {
        name
        id
        email
        avatar
      }
      notes(orderBy: updatedAt_DESC, privatePerm: $privatePerm) {
        id
        topic
        content
        author {
          id
          email
          name
        }
        attendedBy {
          id
          name
          email
          avatar
        }
        rating
        privateNote
      }
    }
  }
`;

Without id in the NOTE_FEED_QUERY (will continue infinitely):

With id in NOTE_FEED_QUERY:

This is an infinite loop, and different issue than a re-render caused by nested components and associated queries.  @lawsonarichard and I discussed this last night.


Add --slot-size to allow size checking without pad

WIP - Under test
Add the new --slot-size and make --pad a bool flag, to allow checking that firmware fits in the slot without overflowing into the trailer region even when no padding was requested.
Fixes #241
Signed-off-by: Fabio Utzig<EMAIL_ADDRESS>

CC @nvlsianpu


No --listen parameter

I run this stuff over the network off a server and am not able to access the web UI over my local network. A public instance is not ideal for me to go out of my lan and back again.

What do you mean?
By default it's not exposed to LAN, you can expose it to LAN. And if you want to connect from outside that's a firewall + router/ISP thing

I'll add a --listen flag

thanks


Add a Codacy badge to README.md

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


padding on menu (firefox on archlinux)

Fine on manjaro
https://i.imgur.com/ajWhV05.png

With the new update of Firefox it seems that the issue is gone.....
I'll wait 2 days and if the issue is really gone I'll close this issue