cannot reproduce benchmarks

hey @kartikk221
i cannot quite confirm your benchmarks on Node v16.13.1 / Arch Linux / 44GB RAM / Ryzen 7 Pro (16 core):

also tried with Node 17 and got similar results, except NanoExpress failed to start likely due to using an old uWS.js version:

It seems the wide variation between your results is caused by testing locally on the machine. I am not sure why testing locally on your machine would cause inconsistent results but after profiling the benchmark for uWebsockets.js and HyperExpress. I retrieved the following node profiler logs:
uws.txt
hyperexpress.txt
I then copied the command that you used with wrk but this time ran the benchmark on a 1vCPU Vultr instance that is running Ubuntu v21 and I got almost the following results from wrk with your command.
uWebsockets.js
Running 10s test @ http://SERVER_IP_REDACTED:8081/benchmark
  1 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    21.22ms   14.53ms 256.07ms   98.95%
    Req/Sec     4.98k   142.07     5.17k    93.00%
  49505 requests in 10.01s, 25.49MB read
Requests/sec:   4947.74
Transfer/sec:      2.55MB

HyperExpress
Running 10s test @ http://SERVER_IP_REDACTED:8082/benchmark
  1 threads and 100 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency    21.12ms   14.48ms 253.57ms   98.95%
    Req/Sec     5.00k   178.33     5.23k    97.00%
  49779 requests in 10.01s, 24.92MB read
Requests/sec:   4975.14
Transfer/sec:      2.49MB

Also, the results are so similar because your command is actually very lightweight and causes the 1vCPU server to only be at 20% CPU usage during the test. If the test was more extensive I'm sure uWS would pull ahead due to the lower overhead. But yeah there should not be too much of a difference between uWS, HyperExpress and NanoExpress since HyperExpress and in some cases NanoExpress only construct new instances of predefined classes and creating objects in JS is plenty fast. If you can run the benchmarks on a remote machine or a different machine then your stresser machine then you should see accurate results.

If you can run the benchmarks on a remote machine or a different machine then your stresser machine then you should see accurate results.

why would this be any better? you're just benchmarking the limits/bottleneck of a network, or a specific host.
in my test, wrk runs on a different core than the node server, so the fact that it is the same machine makes minimal difference -- this machine is far more powerful than this test exerts. more importantly, NanoExpress shows much higher numbers than hyper-express. both are less that raw uWS.js, which is expected of course. your benchmarks show that they are all close, but i'm not seeing this.

the only conclusion you can draw from your benchmark table is  that Express and Fastify hit the Vultr CPU limits before they hit the network limits, and uWS.js, NanoExpress and Hyper-Express all hit the network limits before the CPU limits.

Like I said I am not sure exactly why testing on a local machine would cause this big difference but that is precisely why I profiled the node process when testing with uWebsockets.js vs HyperExpress to see why HyperExpress was scoring so much lower in wrk and where the overhead was coming from. If you look at the profile logs that I attached above you will see that HyperExpress's Javascript logic takes up less than 5% of the total execution ticks. Unless you think that creating a new instance of a constructor in Javascript is a "heavy" operation there is practically no room of a bottleneck in HyperExpress vs uWebsockets.js or NanoExpress for that matter.
Also, you mentioned that Express/Fastify hit CPU limits and uWS.js, NanoExpress and HyperExpress hit the network limits. That is precisely what is supposed to happen as stated in the benchmarks section. Also, you are more than welcome to go out and try these benchmarks on a different server with a faster network card or your own environment with an extremely fast network adapter, you will see that these results will remain relatively the same simply due to the little overhead in HyperExpress vs. uWebsockets.js as shown in the Node profiler logs above.

interesting. very odd that i get very different results locally for NanoExpress and HyperExpress, and NanoExpress is much closer to raw uWS.js.
let me know if you'd like any of my logs to investigate this (and let me know how to generate them). otherwise, thanks for the discussion :+1:

interesting. very odd that i get very different results locally for NanoExpress and HyperExpress, and NanoExpress is much closer to raw uWS.js.

I looked a bit further into this yesterday and after reviewing the request flow between both HyperExpress and NanoExpress, the only main difference I could derive was that HyperExpress may be performing more JS/C++ bridge communications (Not even sure they can be called those) but essentially when a HyperExpress.Request instance is created, HyperExpress will fetch the method, path, query, ip, proxy_ip and all headers and store them locally as shown here: https://github.com/kartikk221/hyper-express/blob/23723e98ff4e603a90f90e19ec335dcb3b108c93/src/components/http/Request.js#L61-L67 no other operations are performed other than wrapping the middlewares/user route handler in a try/catch/promise which will catch both sync/async errors for the global error handler. All of these additional calls and actions are essentially neccessary for HyperExpress to be as user-friendly as possible and essentially provide the most "Express-like" experience even though uWebsockets.js is vastly different than the built-in Node http/https module which is what Express/Fastify use. I would say in terms of practical performance where you are running on servers and clusters, HyperExpress/NanoExpress/uWebsockets.js will perform relatively similar as the network will almost always be bottlenecked first. If for some reason you do need every single possible request/sec from a webserver than you might as well opt-out to build directly in C++ with uSockets/uWebsockets since those would provide the most raw performance without any Node overhead.

great, thanks for looking into it and that's a completely fair trade-off. i'll try to re-bench soon on e.g. a 1GB/1VCPU linode instance w/debian-unstable. still a bit odd that your numbers show ~5x higher perf vs Fastify, but locally i only get 2x faster.

The ~5x higher performance against Fastify most likely occurs because of the test scenario which simulates with 2500 concurrent connections over a pipeline of 4 requests for 30 seconds. Express/Fastify or any other Node based webserver gets heavily bottlenecked the more stress you place upon it. Your results of 43k reqs/sec for Fastify look about right since your single core performance is likely a bit faster than the Vultr instance I use for benchmarking.


Run cyclone in multiple processes or threads?

Hi, I run my server this way:
from twisted.internet import reactor
reactor.listenTCP(8887, Application(), interface="<IP_ADDRESS>")
reactor.run()

However, as I observed, requests are processed one at a time. The server takes up only around 6% of CPU and the clients need to wait very long. May I ask you where/which parameter I can change to increase the concurrent connections?
Thanks.

I suspect that this isn't related to cyclone itself, but rather something you might be doing in your application handler. Seen this before where people create new database connections and do other blocking operations in their handlers instead of keeping it async top to bottom.

Thank you @fiorix. I did exactly what you suspected. However, I am not really sure about what you meant by "async top to bottom". Could you give me an example?
Thank you very much.

You shouldn't create database connections in your request handlers. Instead, you should create them when your application is instantiated and then reuse them as much as possible. Specifically for databases, check out twisted.enterprise.adbapi.
By "keep it async from top to bottom" I mean don't do any operation that can potentially block your request handler because your web server is running single threaded and must handle requests concurrently without blocking. If you block in one request handler, you'll block the entire server.


Secrets Management: secrets not available, secrets-driver throwing "mounted but not registered in Docker"

Rancher Versions:
Server: 1.4.0
Docker Version: 1.13 (yes, it's unsupported, still a bug)
OS and where are the hosts located? (cloud, bare metal, etc): RancherOS on AWS
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB) HA rancher, using RDS
Environment Type: (Cattle/Kubernetes/Swarm/Mesos) Cattle
Steps to Reproduce:

Add secrets driver
Add some secrets
Upgrade service and mount secrets in service
Exec into container to try and find them

Results:
2/10/2017 3:21:36 PMtime="2017-02-10T21:21:36Z" level=info msg=remove.response
2/10/2017 3:22:09 PMtime="2017-02-10T21:22:09Z" level=info msg="container 37b7ccfd81e02932399fd9639d06241c47726d5d2d944b1f4976126be8517406 destroyed"
2/10/2017 3:22:10 PMtime="2017-02-10T21:22:10Z" level=error msg="Mounted but not registered in Docker: /var/lib/rancher/volumes/rancher-secrets/staging/57d74f7d3ae863ad1870aa1df337f751083294b622c9ed6ec19f62f863b64fba"
2/10/2017 3:22:10 PMtime="2017-02-10T21:22:10Z" level=info msg="Unmounting /var/lib/rancher/volumes/rancher-secrets/staging/57d74f7d3ae863ad1870aa1df337f751083294b622c9ed6ec19f62f863b64fba"
2/10/2017 3:22:36 PMtime="2017-02-10T21:22:36Z" level=info msg=unmount.request name=57d74f7d3ae863ad1870aa1df337f751083294b622c9ed6ec19f62f863b64fba
2/10/2017 3:22:36 PMtime="2017-02-10T21:22:36Z" level=info msg=unmount.response
2/10/2017 3:22:41 PMtime="2017-02-10T21:22:41Z" level=info msg=create.request name=f769fe12047c4ad98c3e9cfcb90c64008e83dc4b1711f4fa8506d2f3ac545aa2 options=map[io.rancher.secrets.token:{"value":"[redacted]"}]
2/10/2017 3:22:41 PMtime="2017-02-10T21:22:41Z" level=info msg="Updating volume f769fe12047c4ad98c3e9cfcb90c64008e83dc4b1711f4fa8506d2f3ac545aa2 (1v57552) rancher-secrets:1sd3 on 1h52, state inactive => inactive" err=<nil>
2/10/2017 3:22:41 PMtime="2017-02-10T21:22:41Z" level=info msg=create.response
2/10/2017 3:22:41 PMtime="2017-02-10T21:22:41Z" level=info msg=attach.request name=f769fe12047c4ad98c3e9cfcb90c64008e83dc4b1711f4fa8506d2f3ac545aa2
2/10/2017 3:22:42 PMtime="2017-02-10T21:22:42Z" level=error msg="Unsuccessful request: 500 Internal Server Error"
2/10/2017 3:22:42 PMtime="2017-02-10T21:22:42Z" level=info msg=attach.response
2/10/2017 3:22:42 PMtime="2017-02-10T21:22:42Z" level=info msg=mount.request name=f769fe12047c4ad98c3e9cfcb90c64008e83dc4b1711f4fa8506d2f3ac545aa2
2/10/2017 3:22:42 PMtime="2017-02-10T21:22:42Z" level=error msg="Unsuccessful request: 500 Internal Server Error"

@brettneese Can you look at the rancher/server docker logs and see if there is any errors in there?

@brettneese Also, is your secret longer than 41 bytes?  There is a known issue with the secrets needing to be less than 41 bytes.

@ibuildthecloud Ah yeah, that was it. It's just a random token so I shortened it.
What's confusing is that if you associate any secrets with a service that are longer than 41 bytes, the whole thing fails.


Update Varnish role to 2.0.0

Issue Type

Feature Idea

Summary
The upstream Varnish role now has a 2.0.0 release that changes the repository from where Varnish is installed. Because of this change, upgrading existing boxes may require a little manual intervention (or maybe we could automate this in a point release).
What this gives us, though, is Varnish 5.0 and 5.1 out of the box (the 1.x series only had up to Varnish 4.1). So the tradeoff is worth it...
I should also test the templates/drupalvm.vcl.j2 varnish VCL for Drupal VM with 5.x to see if there are any incompatibilities.

Automated tests passed, trying a manual provision to make sure the VCL is at least minimally working.

Works, verified with curl --head drupalvm.dev:81.


Updating kubeflow kustomize requirements

Fixes https://github.com/kubeflow/website/issues/870

This change is

/assign

/retest

@sarahmaddox Looks like this PR is merging into the wrong branch, please ignore.

@richardsliu For some reason the presubmit is skipped on the PR, and the Netlify preview isn't built. Do you have any idea why?

Ah I see :)


[Feature Request] Use node 20 instead of node 16

Hi,
Thanks for you work about this Github action.
It's not really a feature but could you use an up to date node version (at least version 20) in order to fit the new requirements of Github:
https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Thx

Agreed, that would be ideal.
If you're able, mind submitting a pull request with the change?

I see a fork may have already done this. Could be a good reference.
https://github.com/joshmenden/action-hide-sensitive-inputs/commit/2f7464b0d433e6ed11b064d53338893917a1c7d4

thx @levibostian for the merge.
I seen that your CI after the merge is broken. Do you think that it's possible to release and push as soon as possible a new version of the github actions ?

Deployment fixed. Latest release has been shipped.


Add support for podLabelSelector to WaitForControlledPodsRunning

It tries to solve problem that there is a race between rolling update of pods and WaitForControlledPodsRunning waiting for updated pods.
Sample usage:
First, create deployment label pod-revision: old.
- name: Starting measurement for waiting for pods
  measurements:
  - Identifier: WaitForRunningPods
    Method: WaitForControlledPodsRunning
    Params:
      action: start
      apiVersion: apps/v1
      kind: Deployment
      labelSelector: l1 = v1
      operationTimeout: 1h
- name: Create deployments with pod-revision=old label
  phases:
  - namespaceRange:
      min: 1
      max: 1
    replicasPerNamespace: 1
    tuningSet: 100qps
    objectBundle:
    - basename: foo
      objectTemplatePath: deployment.yaml
      templateFillMap:
         labelToSet: old
- name: Waiting for objects become created
  measurements:
  - Identifier: WaitForRunningPods
    Method: WaitForControlledPodsRunning
    Params:
      action: gather

Then, before we trigger update of deployments, we "limit" WaitForControllerPodsRunning's window to only updated pods by setting pod labels selector:
- name: Set pod selector
  measurements:
  - Identifier: WaitForRunningPods
    Method: WaitForControlledPodsRunning
    Params:
       action: setPodLabelSelector
       selector:
          pod-revision: new
- name: Update Foos
  phases:
  - namespaceRange:
      min: 1
      max: 1
    replicasPerNamespace: 1
    tuningSet: 100qps
    objectBundle:
    - basename: foo
      objectTemplatePath: deployment.yaml
      templateFillMap:
         labelToSet: new

At this point, each deployment object update will trigger new watch inside of WaitForControllerPodsRunning with additional selector that will limit "watch window" to new pods only. This way, we in the next step we will wait only for updated pods (not seeing old pods at all).
- name: Waiting for objects become updated
  measurements:
  - Identifier: WaitForRunningPods
    Method: WaitForControlledPodsRunning
    Params:
      action: gather

The main drawback of such approach is that we do not check if old pods were deleted at all. Potentially due to some bug, they may leak and we won't catch this.
/assign @mm4tt
/assign @wojtek-t

/hold
I still need to test if this works and we still need to discuss whether this is a right approach to take.


Since the morning of the 5th of November, in the augur client my positions are missing.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser: [e.g. stock browser, safari]
Version: [e.g. 22]

Additional context
Add any other context about the problem here.

Hello!
We've been looking into this today and it appears there is an issue with the latest warp sync checkpoint, where it's missing the positions from shares which were acquired externally.
We have code up for a workaround and will be building a new beta client.

Thanks for the update. As of now, the problem persists.
On Fri, Nov 6, 2020 at 7:52 AM Paul Gebheim<EMAIL_ADDRESS>wrote:

Hello!
We've been looking into this today and it appears there is an issue with
the latest warp sync checkpoint, where it's missing the positions from
shares which were acquired externally.
We have code up for a workaround and will be building a new beta client.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AugurProject/augur/issues/9661#issuecomment-722910073,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIW6XXYSDMDNU7AOVODFT6TSOOMJZANCNFSM4TLYSLFQ
.

--
Balázs Váradi, Ph.D.
Reguly Antal u. 56, Budapest, 1089, Hungary

If it helps (the problem is still there): I have bets (nTrump) on
catnip.exchange (those show on the catnip interface but not on augur) and
sold Yes shares on the corresponding market directly on Augur (where you
can see that my orders were filled months ago, but not the actual positions.
On Sat, Nov 7, 2020 at 3:56 AM Balazs Varadi<EMAIL_ADDRESS>wrote:

Thanks for the update. As of now, the problem persists.
On Fri, Nov 6, 2020 at 7:52 AM Paul Gebheim<EMAIL_ADDRESS>wrote:

Hello!
We've been looking into this today and it appears there is an issue with
the latest warp sync checkpoint, where it's missing the positions from
shares which were acquired externally.
We have code up for a workaround and will be building a new beta client.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AugurProject/augur/issues/9661#issuecomment-722910073,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIW6XXYSDMDNU7AOVODFT6TSOOMJZANCNFSM4TLYSLFQ
.

--
Balázs Váradi, Ph.D.
Reguly Antal u. 56, Budapest, 1089, Hungary

--
Balázs Váradi, Ph.D.
Reguly Antal u. 56, Budapest, 1089, Hungary

Hi all, having this problem also. Did not open a position on external like Catnip, opened within Augur itself. As of 5 Nov, my positions have completely disappeared also (presidential election positions). I believe there are a few users awaiting some action here on this that have positions open natively on Augur too.

I'm also having this issue. My positions have disappeared from the UI. Those shares were acquired within Augur itself, not externally. I want to close these positions and I have no way of doing it.

Dear developers, the problem is still there. Let me stress that this is not a minor annoyance: the money of clients is reported gone. We cannot withdraw our gains and deploy our inaccessible dai elsewhere, causing us additional loss you will not compensate us for. We await timely action. Solving this problem is not something you can conveniently fold into some upgrade coming up in months.

I agree. The lack of communication on this issue is unnerving to say the least. As customers, we don't have access to our funds and the opportunity cost for us is immense due to this issue. Can we have at least an expectation of when we can see this resolved or can you provide an alternative way of accessing and trading our positions?


Update the filtering of time for cloud copy

Overview
This PR update the retrieval of data from cloud copy!

Pull Request Test Coverage Report for Build 239

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals

Change from base Build 228:
0.0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls

@pshivraj Any thoughts on this credential failures?

@pshivraj Any thoughts on this credential failures?

I see one failure owing to missing funtion filter_ncurls() which apparently looks like has been removed from current pull request. Is there any reason why this module is not needed anymore ?

@pshivraj Any thoughts on this credential failures?

I see one failure owing to missing function filter_ncurls() and get_start_end_dates() which apparently looks like has been removed from current pull request. Is there any reason why this module is not needed anymore ?

I tested this locally, since this pull request uses different method to get ncurls so taking away one test case should make this build pass.

I see one failure owing to missing function filter_ncurls() and get_start_end_dates() which apparently looks like has been removed from current pull request. Is there any reason why this module is not needed anymore ?

Yea because we don't need it anymore because the new processed netcdf.

@pshivraj Does this work locally now? If so please merge. Thanks :)

Works fine with test case fix. Merging it


support for eclipse project

any possble provide support for eclipse project?just need to read the properties from .classpath
now the default output directory is the root of the project (vim-rooter)
i want the plugin can read the .classpath file ,here is my helloword project

https://github.com/scrooloose/syntastic/pull/1559

I'm afraid I can't parse what you're saying. shrug

sorry for my pool english!
i just want use the plugin for auto compile ecplise java project !  i am using vim to create my java project ,when the project is create by maven ,the plugin works well ,and can autocomplile the file into target,but when it has no pom.xml ,the plugin can not read the file named .classpath  which is created by eclipse,i hope you can understande

You can set g:syntastic_java_javac_classpath in an autocmd.

but this option can not set the where the javac output,i hope it is same as eclipse,so i can do not open eclipse,just use vim

You can use an autocmd to set g:syntastic_java_javac_classpath, to the output of an external command.

Also, if you leave g:syntastic_java_javac_classpath unset, javac will fallback to the value of the environment variable CLASSPATH (at least that's what I recall).  Thus it should work if you set the environment variable CLASSPATH before running Vim.

i will check my old setting ,thank you

i have set let g:syntastic_java_javac_classpath="bin",but it does not work,alse compile into the project root directory

and i think g:syntastic_java_javac_classpath is not the option for set the outputdir,
javac -d outputdir -cp g:syntastic_java_javac_classpath

Ah, the output directory, not the classpath.  You can (ab)use g:syntastic_java_javac_options to set it:
let g:syntastic_java_javac_options = '-Xlint -d /some/path'

Ah let me have a try

i just try with let g:syntastic_java_javac_options = ' -d  bin ' an hour ago but it does not work

it does not work let g:syntastic_java_javac_options = '-Xlint -d bin'

Did you try setting it to a full path?

i have setted ,but still does not work

https://github.com/scrooloose/syntastic/blob/50d0315835047c35a1f6f6edf214e612d6eae3dd/syntax_checkers/java/javac.vim#L99-L114

Well then, please set g:syntastic_debug to 1, run the checkert, run :mes, and post the output.

i think it is due to you have use -d out of g:syntastic_java_javac_options,you see, there are two -d
"src/com/wsdjeg/testjc/OuterClass.java" 21L, 383C written
syntastic: 14.228614: &shell = '/usr/bin/zsh', &shellcmdflag = '-c', &shellpipe = '2>&1| tee', &shellquote = '', &shellredir = '>%s 2>&1', &shell
slash = 0, &shelltemp = 1, &shellxquote = '', &shellxescape = ''
syntastic: 14.230439: UpdateErrors (auto): default checkers
syntastic: 14.231350: CacheErrors: default checkers
syntastic: 14.233363: g:syntastic_aggregate_errors = 0
syntastic: 14.233943: getcwd() = '/home/wsdjeg/workspace/TestJC2'
syntastic: 14.244238: CacheErrors: Invoking checker: java/javac
syntastic: 14.248198: SyntasticMake: called with options: {'postprocess': ['cygwinRemoveCR'], 'errorformat': '%E%f:%l: error: %m,%W%f:%l: warning
: %m,%E%f:%l: %m,%Z%p^,%-G%.%#', 'makeprg': 'javac -Xlint -d /home/wsdjeg/workspace/TestJC2/bin -d . /home/wsdjeg/workspace/TestJC2/src/com/wsdje
g/testjc/OuterClass.java'}
syntastic: 14.820233: getLocList: checker java/javac returned 0

It goes without saying that you shoul also set g:syntastic_java_javac_delete_output to 0.

i just do not want to delete the output file ,so that let the plugin help me AUTO compile the java file when save,i think i can give you an PR

Did you set g:syntastic_java_javac_delete_output to 0?

by default it is 0, and i have echo g:syntastic_java_javac_delete_output also show me 0

It isn't 0 by default.  Syntastic won't add the second -d if g:syntastic_java_javac_delete_output is 0.

Sorry, I now understanbd what's going on.  You have to set both g:syntastic_java_javac_delete_output and g:syntastic_java_javac_autoload_maven_classpath to 0.

i will add this two option in the elseif

Ok, commit cfdcdfa make g:syntastic_java_javac_delete_output = 0 take effect regardless of g:syntastic_java_javac_autoload_maven_classpath.

after this comit mvn project does not checked

hi @lcd047  if i run mvn clean .the checker will not work ,after run mvm compile it works well

i have test this  :
mvn clean
mkdir -p target/classes
then vim src/---------/Hello.java
then the checker works well

I'm afraid I don't understand.  What doesn't work?

it can not create the outputdir target/classes ,if has no the  dir ,the checker will not work, if i mkdir -p target/classes ,it works well

Ok, reverting last patch: commit ae7e00a.  One more example showing that this checker has become a labyrinthic mess.  Perhaps somebody that actually cares about Java is willing to maintain it outside syntastic?

hi by default
if s:has_maven && g:syntastic_java_javac_autoload_maven_classpath
        if !g:syntastic_java_javac_delete_output
            let javac_opts .= ' -d ' . syntastic#util#shescape(s:MavenOutputDirectory())
        endif
        let javac_classpath = s:AddToClasspath(javac_classpath, s:GetMavenClasspath())
    endif
    " }}}2

thank you i have find how to config,just add this to my vimrc
if ！filereadable("pom.xml")
let g:syntastic_java_javac_autoload_maven_classpath=0
endif

hi @lcd047 here is my config for this plugin,it works well in maven project and eclipse project and the project which has no both pom and .classpath.
only little issue is that:
in the project ,if the default  outputdirectory is not exist,the checker will not work,any possiable,
add this func,
if I set do not delete the outputfile(which means do not use the temp_directory) and the outputdirectory != ' ' ,confirm wether the default outputdirectory exist,if do not exist,create it !

Not possible.  By default syntastic doesn't know, nor care, about maven's output directory.  If you set an explicit output directory it's up to you to create it if needed.

sorry i have just read the code of this plugin ,i also find it is  not possible.thank you all the same.do you know javacomplete2, with the help of these two plugin ,I can use vim to finished my java project,


[MNT] python 3.13 support, add 3.13 to test matrix

Extends the bounds in pyproject.toml to add experimental support for python 3.13, and adds testing on 3.13 to the test matrix.
Full 3.13 support to be added with 2.7.0.

numpy seems incompatible with windows, this is the failure:
rm -rf testdir
mkdir -p testdir
cp .coveragerc testdir
cp setup.cfg testdir
python -m pytest
<frozen importlib._bootstrap>:488: Warning: Numpy built with MINGW-W64 on Windows 64 bits is experimental, and only available for
testing. You are advised not to use it for production.

CRASHES ARE TO BE EXPECTED - PLEASE REPORT THEM TO NUMPY DEVELOPERS
C:\hostedtoolcache\windows\Python\3.13.0\x64\Lib\site-packages\numpy\core\getlimits.py:225: RuntimeWarning: invalid value encountered in exp2
  epsneg_f128 = exp2(ld(-113))
C:\hostedtoolcache\windows\Python\3.13.0\x64\Lib\site-packages\numpy\core\getlimits.py:226: RuntimeWarning: invalid value encountered in exp2
  tiny_f128 = exp2(ld(-16382))
C:\hostedtoolcache\windows\Python\3.13.0\x64\Lib\site-packages\numpy\core\getlimits.py:240: RuntimeWarning: invalid value encountered in exp2
  eps=exp2(ld(-112)),
C:\hostedtoolcache\windows\Python\3.13.0\x64\Lib\site-packages\numpy\core\getlimits.py:41: RuntimeWarning: invalid value encountered in nextafter
  self._smallest_subnormal = nextafter(
C:\hostedtoolcache\windows\Python\3.13.0\x64\Lib\site-packages\numpy\core\getlimits.py:52: RuntimeWarning: invalid value encountered in log10
  self.precision = int(-log10(self.eps))
make: *** [Makefile:25: test] Error -1073741819

Hm, must be coming from one of the softdeps, because the no-extras run seems fine. I suspect scikit-survival since in the install phase it is the only dep that builds.


Pigar uses underscores instead of hyphens

For packages with hyphens in their names, pigar uses underscores instead of hyphens. This way pip and conda do not recognize these packages and cannot find them.

I believe python stores modules in this way (with underscores in the site-packages folder). But in my case pip has no issue installing packages with underscores instead of hyphens.
Possible solution could be to just replace the underscores to normalize names as described in PEP503.
re.sub(r"[-_.]+", "-", name).lower()


Enhancement - Dark Theme option

Add a button to site for switching between Regular Theme (light) and Dark Theme.  We could use a package like this one (which didn't seem to work with our site when importing it in module.exports in config.js by default, just tried quickly) https://www.npmjs.com/package/vuepress-theme-dark-new
OR borrow ideas from other style sheets to define our own dark theme.
Site would switch CSS styles it reads from based on the state of the button.
Very low priority, just a cool idea, especially for a project focused on developers who I find often prefer dark themes.

Another potential package to use - https://vuejsexamples.com/a-vuepress-theme-that-supports-multiple-color-themes/
A sample implementation - Note the Settings cog wheel that lets you choose dark theme. https://vuepress-theme-yuu.netlify.com/

Fixed in #1739.


"cb_append_data_point" leads to runtime error if no existing points are present

I initialize a LiveLinePlot, which is passed to a DataConnector for later updating. On the first iteration where I try to add a datapoint through the connecter via cb_append_data_point I receive the following error:
/home/user/.local/lib/python3.10/site-packages/pyqtgraph/debug.py:128: RuntimeWarning: Ignored exception:
Traceback (most recent call last):
  File "/home/user/path/pysideGUI.py", line 1101, in <module>
    app.exec()
  File "/home/user/.local/lib/python3.10/site-packages/pglive/sources/live_plot_widget.py", line 165, in paintEvent
    return super().paintEvent(ev)
  File "/home/user/.local/lib/python3.10/site-packages/pyqtgraph/widgets/GraphicsView.py", line 137, in paintEvent
    return super().paintEvent(ev)
  File "/home/user/.local/lib/python3.10/site-packages/pyqtgraph/debug.py", line 128, in w
    printExc('Ignored exception:')
  --- exception caught here ---
  File "/home/user/.local/lib/python3.10/site-packages/pyqtgraph/debug.py", line 126, in w
    func(*args, **kwds)
  File "/home/user/.local/lib/python3.10/site-packages/pyqtgraph/graphicsItems/PlotCurveItem.py", line 905, in paint
    p.drawLines(*self._getLineSegments())
TypeError: QPainter.drawLines() takes exactly one argument (0 given)
  printExc('Ignored exception:')

I've found that checking for the presence of data in the x or y variable in the DataConnector and adding the first datapoint twice if there is none prevents the error from appearing. E.g.:
if len(connector.x) < 1:
  *add extra datapoint*
*add datapoint like normal*

It seems like there just needs to be an additional check somewhere(?)
Using:
Python 3.10
pglive 0.5.5

Error is thrown by underlying pyqtgraph, complaining, that there is no data to be plot I believe.
Are the pglive examples running OK from your environment?
Could you confirm, that you are actually passing some data when calling cb_append_data_point from your data generator?
I think either providing some input data causing error or posting some minimal reproducible code would help me to figure out the issue. Without it, I can't reproduce the exact issue and will be just guessing what might be wrong.
Try maybe just collecting the data before are passed to callback and see if there is no None values or empty List or something like that. Pyqtgraph might ignore for example None value or empty list or string if that is found in the X or Y values.
Thanks for using pglive, I hope we can solve your problem.

Forgot to mention that I'm using PySide6.
I just tried using the cb_set_data function and received a different error:
File "/home/user/path/pysideGUI.py", line 192, in add_points
   self.lines[name][1].cb_set_data(ys, xs)
 File "/home/user/.local/lib/python3.10/site-packages/pglive/sources/data_connector.py", line 132, in cb_set_data
   self.sig_data_toggle.emit(self)
TypeError: sig_data_toggle(PyObject,bool) needs 2 argument(s), 1 given!

The line in the source code causing the new error is located here:
https://github.com/domarm-comat/pglive/blob/c5eecc2c7fd0dace5533d216a7d1912192f74acb/pglive/sources/data_connector.py#L20
Here is some example code to reproduce the errors:

from PySide6.QtCore import Qt
from PySide6.QtWidgets import *
from pyqtgraph import mkPen, InfiniteLine
from pglive.sources.data_connector import DataConnector
from pglive.sources.live_plot import LiveLinePlot
from pglive.sources.live_plot_widget import LivePlotWidget
import random

class MainWindow(QMainWindow):
    def __init__(self):
        super().__init__()

        self.layout = QVBoxLayout()

        self.testplot = TestPlot()
        self.layout.addWidget(self.testplot.plot_widget)

        testbutton = QPushButton(text="Add Point")
        testbutton.pressed.connect(self.add_point)
        self.layout.addWidget(testbutton)

        testbutton = QPushButton(text="Set Data")
        testbutton.pressed.connect(self.set_data)
        self.layout.addWidget(testbutton)

        self.main_widget = QWidget()
        self.main_widget.setLayout(self.layout)
        self.setCentralWidget(self.main_widget)

    def add_point(self):
        self.testplot.add_point(random.random(), random.random())

    def set_data(self):
        self.testplot.set_data(
            [random.random(), random.random()], [random.random(), random.random()]
        )

class TestPlot:
    def __init__(self, width=200, height=30):
        self.plot_widget = LivePlotWidget()

        tmp = LiveLinePlot(pen=mkPen((0, 8), width=2))
        self.conn = DataConnector(tmp)
        self.plot_widget.addItem(tmp)

    def clear(self):
        self.conn.cb_set_data(None, None)

    def set_data(self, xs, ys):
        self.conn.cb_set_data(ys, xs)

    def add_point(self, x, y):
        self.conn.cb_append_data_point(y, x)

if __name__ == "__main__":
    app = QApplication()  # sys.argv)

    window = MainWindow()
    window.show()

    app.exec()

FYI, you are correct that the original error is just a pyqtgraph issue (happens with the normal PlotWidget). I've created an issue on their github insead.
The error from cb_set_data is still relevant, though.

I can confirm error with cb_set_data. I found out the reason as well, but I need to do some more testing before releasing new version.

V0.5.6 is out, fixing cb_set_data error. That should work just fine now.
For cb_set_data error, it seems it's really something internal with pyqtgraph and pen as you discovered.
I tried just pure pyqtgraph code:

from PyQt6.QtWidgets import QApplication
import pyqtgraph as pg

app = QApplication(sys.argv)

plotWidget = pg.plot(title="Three plot curves")
di = pg.PlotDataItem(pen=pg.mkPen("red", width=2))
di.setData([0], [1])
plotWidget.addItem(di)

app.exec()

Which gave me the same error. It kind of make sense, since it's trying to plot line using only one point. You should have two :).
When I removed pen parameter error was gone...

FYI, the original error has been fixed on pyqtgraph's side, but not yet released (https://github.com/pyqtgraph/pyqtgraph/pull/2481).
Confirmed that the V0.5.6 fix worked for me.
Thanks for the fast response on this :)


Serve Images from GCR

We need to merge #103 to get a clear diff of the changes.
/cc @ameukam @BenTheElder

Closed in favour of https://github.com/kubernetes/k8s.io/pull/4258


Subscript and Superscript implemented

Closes #3
Created AsymmetricMarkdownTag class that differentiates from MarkdownTag in that these don't need a closing markdown tag (don't really know the scientific name for these tags, renaming of this stuff is welcomed)

Hey, I'm really sorry it took me 6 months to get to this. Had a bit of a rough start to the year.
Thank you so much for the PR! It works great and the code is nice and tidy. I think AsymmetricMarkdownTag is a great name and I'm happy to keep it for now.
The only change I'm going to make is that I'm going to remove the subscript tag. I've decided to do this for the following reasons:

The typical use case for subscript is a mathematical or chemical formula (i.e. "H2O"). This use case doesn't work with an AsymmetricMarkdownTag, because the tag requires a space after the subscript text.
The syntax is awkward and conflicts with a lot of regular text. There are plenty of reasons to have an underscore in the middle of text, and it would suck to need a \ in front of all of them.
Subscript isn't part of any other markdown renderer that I'm aware of. I think FTR should aim for consistency/familiarity.

For now, if folks want subscript I think that directly using the <sub> tag is adequate. That said, this has got me thinking that FTR should have better extension support. If you want to add subscript, or any other rendering feature, you should be able to add that without forking the project. Hm...
Once again thank you very much for the PR. Take care friend, peace and love! ✌

Okay, when I looked at this earlier I didn't realize how much of the code you added was copy+pasted from my code. I want to minimize duplicate code, so I did some refactoring.
I removed AsymmetricMarkdownTag; its functionality can now be handled by MarkdownTag. In the process, I also fixed a bug with your implementation (tag was not closed by a line ending).
See bdd0575 for details, and I also updated the live demo if you want a quick way to play around with my changes.


pdfsig.exe is missing

The build doesn't contain pdfsig.exe

Hi,
Sorry for the delay! I think if it is not in this package it is not getting built by https://github.com/conda-forge/poppler-feedstock. Maybe you can open an issue there and ask why it is not included?


Use external version of k8s.io/component-base/config.ClientConnectionConfiguration

How to categorize this PR?

/area dev-productivity
/kind cleanup
What this PR does / why we need it:
This is a prefactoring for https://github.com/gardener/gardener/issues/11043:
This PR starts using the external version of k8s.io/component-base/config.ClientConnectionConfiguration in client-related functions while maintaining another variant with the internal version (deprecated).
This will help with a smooth removal of the internal version of component config APIs with individual PRs per component.
Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/11043
Special notes for your reviewer:
Release note:

Client-related functions have been adapted to use the external version of `k8s.io/component-base/config.ClientConnectionConfiguration`. If you need a helper function for transitioning to the external version, use `pkg/client/kubernetes.ConvertClientConnectionConfigurationToExternal`.

/retest

/retest

/test all

/retest

I rebased to resolve the conflicts and addressed @marc1404's suggestions. PTAL :)


validateIri to Reject "http://" URL

Additional logic/tests to reject a URL of "http://"

This addresses the latest discussion on https://github.com/CycloneDX/cdxgen/issues/1107


bug: cannot import novel/extension in a Next.js project

Provide environment information
  System:
    OS: macOS 14.0
    CPU: (12) arm64 Apple M2 Max
    Memory: 9.06 GB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.10.0 - ~/.nvm/versions/node/v20.10.0/bin/node
    npm: 10.2.3 - ~/.nvm/versions/node/v20.10.0/bin/npm
    pnpm: 8.14.3 - ~/.nvm/versions/node/v20.10.0/bin/pnpm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman

Describe the bug
I'm struggling to import novel/extensions due to some TS error. My Next.js project has:
    "module": "esnext",
    "moduleResolution": "node",

Which is standard and hasn't caused any issue with other libraries. Specifically for novel/extensions, I get this:
Cannot find module 'novel/extensions' or its corresponding type declarations.
  There are types at '/Users/arielweinberger/Development/academia-helper/node_modules/novel/dist/extensions.d.ts', but this result could not be resolved under your current 'moduleResolution' setting. Consider updating to 'node16', 'nodenext', or 'bundler'.ts(2307)

I assume this is related to using exports in package.json?
Link to reproduction

To reproduce
Set up a fresh Next.js repo and try to use Novel:
import {
  EditorBubble,
  EditorBubbleItem,
  EditorCommand,
  EditorCommandItem,
  EditorContent,
  EditorRoot,
} from 'novel'

import {
  TiptapImage,
  TiptapLink,
  UpdatedImage,
  TaskList,
  TaskItem,
  HorizontalRule,
  StarterKit,
  Placeholder,
} from "novel/extensions";

export default async function IndexPage() {
  return (
    <EditorRoot>
      <EditorContent extensions={defaultExtensions}></EditorContent>
    </EditorRoot>
  )
}

Additional information
No response

My bad, I just checked and the default values for a fresh Next.js app are:
    "module": "ESNext",
    "moduleResolution": "Bundler",

Changed to this and it works. Thanks.


Fix some name capitalization

Feel free to squash all these web-based edits into a single commit, if we do merge.


python3 setup.py tests, unit tests fail (and also have side-effects)

log included, most tests run successfully, some fail.  others have side-effects that actually prevent and prohibit jumpscale from working (issue #18)
test.txt

commit 286dc4d: isolated the unit tests run each time to put them into a completely new (fresh, non-existent, blank) subdirectory with NO config file(s).  tests that need to test config files should fork a new process, setting up HOSTCFGDIR before-hand, pre-preparing the config, and THEN exec()ing a new process.
it's the only sane way to get round the problem of having a global import.  the insane way is to bootstrap a new jumpscale instance from the current one... which may have unintended side-effects.

test02_j_state.test008_state_reset is REALLY causing some damage.  self.client.reset() is not only destroying state, it's destroying the entire config file (which causes subsequent tests to fail as there are no dirs present for example).  disabling it for now.
it should really be modified to be run either in its own process or to take a copy of the state on-disk and restore it afterwards.

commit    fd7cd93..18107fb


Made freeTrial and InDepthAnalysis component responsive

Issue Title
Please enter the title of the issue related to your pull request.
Enter the issue title here- Made freeTrial and InDepthAnalysis component responsive

[x] I have provided the issue title.

Info about the Related Issue
What's the goal of the project?
Describe the aim of the project- The aim of the project was to make the above mentioned components responsive for all screen sizes.

[x] I have described the aim of the project.

Name
Please mention your name.
Enter your name here- Nakul Krishnakumar

[x] I have provided my name.

GitHub ID
Please mention your GitHub ID.
Enter your GitHub ID here- nakul-krishnakumar

[x] I have provided my GitHub ID.

Identify Yourself
Mention in which program you are contributing (e.g., WoB, GSSOC, SSOC, Hactoberfest).
Enter your participant role here- GSSOC

[x] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
*Closes: #599 *

[x] I have provided the issue number.

Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
Describe your changes here- Previously the components mentioned above were overflowing out of viewport for lower screen sizes. I used media queries and changed the flex-direction to column for both the components and it prevents overflow now.

[x] I have described my changes.

Type of Change
Select the type of change:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe how your changes have been tested.
Describe your testing process here. I tested this using vscode and devtools in browser. I checked for every screen size and made necessary changes.

[x] I have described my testing process.

Checklist
Please confirm the following:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.

@vanshchauhan21 u didnt assign the gssoc role, pls add them

@vanshchauhan21 you didn't add gssoc label to this PR, pls consider adding it

@vanshchauhan21 please assign this pr to me


Evaluate performance improvement over COIFv6

There seems to be a potential 15% performance improvement over COIFv6 if we replace the following code:
valLow = (val * binLowerBoundPercent);
valThresholdCheck = Math.abs(val - valLow);
if (valThresholdCheck > maximumDifferenceThreshold)
    valLow = val - maximumDifferenceThreshold;
valHigh = (val * binUpperBoundPercent);
valThresholdCheckHigh = Math.abs(val - valHigh);
if (valThresholdCheckHigh > maximumDifferenceThreshold)
    valHigh = val + maximumDifferenceThreshold;

with:
valLow = val - maximumDifferenceThreshold;
valHigh = val + maximumDifferenceThreshold;

MainCoifV6 yielded 75,995 feature matches in both cases.
MainCoifV6Panorama yielded 41,279 feature matches in both cases.

Commenting out:
//valLow = val - maximumDifferenceThreshold;
//valHigh = val + maximumDifferenceThreshold;

//if (val2 < valLow || val2 > valHigh) {

yields an additional 10% performance improvements on top of the original 15% performance improvement, while seeming to have no impact on feature matching accuracy.
MainCoifV6 yielded 75,995 feature matches in both cases.
MainCoifV6Panorama yielded 41,279 feature matches in both cases.

MainCoifV6Panorama feature matches sorted by X1, then Y1, then X2, then Y2 and written to disk are the same when comparing original COIFv6 implementation versus implementation with the 15% and 10% performance improvements.

MainCoifV6 feature matches sorted by X1, then Y1, then X2, then Y2 and written to disk are the same when comparing original COIFv6 implementation versus implementation with the 15% and 10% performance improvements.

Updated COIFv6 implementations with the above changes.


task697 is incomplete.

For any question on formal logic sentence, vocabulary, universe, and semantics should also be given to validate if the sentence evaluates to true or false. The task assumes an implied mental model of these things. For any statement, I can create a universe, vocabulary, and semantics to make all options true. I think we should remove this task.

E.g.,
         {
            "input": "Select the best translation into predicate logic: Abdul and Cleopatra are Egyptian.\n(A)Ea • Ec (B)Ea • c (C)Ae ∨ Ce (D)Ex • Ey",
            "output": [
                "A"
            ]
        },

It assumes what E, a, A, c, e, x, and y mean in the options, and chooses a correct answer based on the original authors universe, vocabulary, and the semantics.

The task assumes an implied mental model of these things.

Do you think humans (e.g., crowdworkers) would be able to solve these?
My rule-of-thumb is, if humans can do it, it's reasonable to expect machines to do it as well.

The task assumes an implied mental model of these things.

Do you think humans (e.g., crowdworkers) would be able to solve these? My rule-of-thumb is, if humans can do it, it's reasonable to expect machines to do it as well.

Some crowd workers might do it. If the bar is simply " if humans can do it, it's reasonable to expect machines to do it as well", then I think we can keep the task as is.

I am going to second @pulkitverma25's comment. The notation used in the answer choices is weird, and even for someone who knows formal logic, it might need explanation. I guess we can keep it and just increase the number of examples, to cover different operators.

Sounds fair to me.

#698 has improved the task with a few examples, bit it is still a bit odd. Closing it, for now, if you see that it can be improved in anyway, let's discuss it.


Conversion from 2-column PDF to "single column" output text

Are there an option to set when calling "pdf2txt" in command line mode, so articles with 2 columns are converted into one consecutive column text file?

There is no option.  The various PDF converters are themselves responsible for figuring out how many columns the text there are and combining them into a single column.  They are similarly responsible for combining lines and pages.  Do you know of a converter that can accept a hint about the number of columns?


Floating Promise calling DataStoreContext.realize prefetching DS after attaching remote DS

I'm seeing evidence in Office Fluid telemetry that DataStoreContext.realize is failing in this case (because the errors don't seem to bubble up, only get logged via "RealizeError" event):
https://github.com/microsoft/FluidFramework/blob/124e881fc7fb37e9fa5709fcf7b90ab3e65a9e58/packages/runtime/container-runtime/src/dataStores.ts#L241-L243
I don't know this code/scenario well enough to judge what this means exactly or how application flow will continue from here if we can't realize a newly-attached data store, but wanted to raise the issue.
All other callers to DataStoreContext.realize return or await the promise.  In the logs I can see a subset of failures hitting Summarize_cancel, for instance.
cc @vladsud, @agarwal-navin to comment on if this is a problem or not.

This code is wrong and should be removed. And I though Navin had PR to do so (or an issue).

This may help - https://github.com/microsoft/FluidFramework/issues/8494

Ok I will close this then, now that it's been tied with that issue :)


Fix #746

load should return the current configuration

Here the log from the test:
Error: Property configuration of an instance of qx.tool.cli.api.AbstractApi is not (yet) ready!
at wrapper.eval [as getConfiguration] (eval at __P_96_13 (/home/runner/work/qooxdoo-compiler/qooxdoo-compiler/lib/compiler/index.js:13154:36), :3:166)
getConfiguration is called before the class is constructed completely. I remember - that's was the reason for the breaking change.
await load();
let config = this,getConfiguration()

should work

while investigating this one, I found that node_modules contains: node_modules/<EMAIL_ADDRESS>which it probably should not, AND which is even from an older version of the code ...

Here the log from the test:
Error: Property configuration of an instance of qx.tool.cli.api.AbstractApi is not (yet) ready!
at wrapper.eval [as getConfiguration] (eval at __P_96_13 (/home/runner/work/qooxdoo-compiler/qooxdoo-compiler/lib/compiler/index.js:13154:36), :3:166)
getConfiguration is called before the class is constructed completely. I remember - that's was the reason for the breaking change.
await load();
let config = this.getConfiguration()

should work

well, I guess there is a better way ....

I tried it half a day but did not find one.
Good luck!


🐞[BUG]  PNG and Contact image is not transparent

🐞 Bug Description
A clear and concise description of what the bug is.
in this frontend , index.html .. frontend works in the Nav bar there is a setup icon and a contact icon that's png not  transparent so that is why that looks different as compared to the features icon making difference .. and one more that is API Key Setup section paragraph tag  that paragraph is too closed with downside container so alignment needed there
🔍 Steps to Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '...'
Scroll down to '...'
See error

💡 Expected Behavior
Describe what you expected to happen.
📷 Screenshots
If applicable, add screenshots to help explain your problem.

🖥 System Information

OS: [e.g., Windows, MacOS]
Python version: [e.g., 3.9]
Library versions: [e.g., speech_recognition 3.8.1]

📜 Additional Context
Add any other context about the problem here.

I solved this issue additionally, I changed one front image and also did changes with the scroll top button that was so small and merged towards the chatbot button .. I'm adding that scroll top button previous view

now after applying changes it looks like this ..

i did few changes that video
https://github.com/user-attachments/assets/b97f4239-f5ea-48e2-b11f-d88178ceb0fc


Define scope of UI template

Related to my comment https://github.com/substrate-developer-hub/substrate-ui-template/issues/22#issuecomment-528992292
It would be helpful to define what is and is not in scope for this template. Is it
1 an example and starting point for writing dapp-specific UIs
2 a stripped down version of Apps?
3 a replacement for bonds ui?
4 other?
So far I've thought of it as 1 and 3 but not 2.
Once we have consensus about what this template is for, we should add it to the readme to guide users in understanding what they can do with it and to guide contributors in knowing which features are in scope.
Here's my first attempt:
The UI template is an example and starting point for creating user interfaces for Substrate-based dApps. It uses react and polkadot-api as its primary dependencies. It does not strive to be a general purpose blockchain interaction tool, that's what apps is for.

So far I've thought of it as 1 and 3 but not 2.

This is correct to me.
I think more than that, it could be a general set of examples using the Polkadot JS API.
The principals to me are:

Completely modular design. Every component should be able to be removed or added. Users should be able to paste their "jsx" component into repos or chats, and all you need to do is copy and paste it into this template to work.

 The components in the UI should favor simplicity in how they are designed and implemented over "best practices".

It never breaks no matter the Substrate UI it is hooked up to. See https://github.com/substrate-developer-hub/substrate-ui-template/issues/14

What you wrote seems fine, although I think generally anyone can put any component they want into the assuming they follow step 3.
I was, for example, thinking of embedding our first tutorial demo component in here, such that if you have a POE module, the component would pop up. Otherwise, it would hide itself from the user.


MemcacheSessionHandler client side

Hi, congrats! Ratchet is awesome!
Please, how to implement MemcacheSessionHandler on client side ?
I get stuck...
Regards

That's OK :-) All goes well and works pretty nicely.

See the Session documentation for an example.


fix: readonly view

Merged readonly flag with sysInfo API
Removed the /readonlyinfo endpoint
Read only defaults to false, edit shared config variable server.readonly to enable it

@veds-g - Thanks!

Comment this line since by default it's false.
When it's readonly, all the Create/Update APIs should return 403 forbidden.

When it's readonly, all the Create/Update/Delete APIs should return 403 forbidden.

very valid point :)


Normalise, validate to ACGT (not U) and throw exception for ambiguous nucleotides

We do not want U in our primer.bed or reference.fasta
ambiguous nucs in NEB RSV scheme


Initial import of toolchain image sanity tests

Set of very simple tests, to verify basic usability of tools packaged in the image.

Except the one comment about checking the usage, I don't have anything else.

Thank you, LGTM.


python311Packages.friendly-traceback: init at unstable-2022-11-28

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Update of stack-data is needed for this package: https://github.com/NixOS/nixpkgs/pull/300816


disableDepthTestDistance causes weird mouse interaction

disableDepthTestDistance pushes the billboards to the near plane, which messes up the collision detection when you try to zoom in.
See discussion in https://github.com/AnalyticalGraphicsInc/cesium/pull/6802#issuecomment-405415474
var viewer = new Cesium.Viewer('cesiumContainer', {
    terrainProvider: Cesium.createWorldTerrain()
});
viewer.scene.globe.depthTestAgainstTerrain = true;

var lon = -122.1958;
var lat = 46.1915;
for (var i = 0; i < 20; i++) {
    for (var j = 0; j < 20; j++) {
        viewer.entities.add({
            position : Cesium.Cartesian3.fromDegrees(lon + i * -0.001, lat + j * -0.001),
            billboard : {
                image : '../images/facility.gif',
                heightReference : Cesium.HeightReference.RELATIVE_TO_GROUND,
                verticalOrigin: Cesium.VerticalOrigin.BOTTOM,
                disableDepthTestDistance: Number.POSITIVE_INFINITY
            }
        });

        viewer.entities.add({
            position : Cesium.Cartesian3.fromDegrees(lon + i * 0.001, lat + j * 0.001),
            billboard : {
                image : '../images/facility.gif',
                heightReference : Cesium.HeightReference.RELATIVE_TO_GROUND,
                verticalOrigin: Cesium.VerticalOrigin.BOTTOM,
                disableDepthTestDistance: Number.POSITIVE_INFINITY
            }
        });
    }
}

viewer.camera.flyTo({
    destination: Cesium.Cartesian3.fromDegrees(lon, lat, 10000)
});

Just to tie in the conversation from https://github.com/AnalyticalGraphicsInc/cesium/issues/8009, @raycrummey you mentioned your workaround was to implement a custom zoom function. In case it helps others blocked on this, did you simply disable the collision detection or did you do something else?
I tried viewer.scene.screenSpaceCameraController.enableCollisionDetection = false; but I can still see the issue there.

Omar,
One workaround is to disable Cesium zoom, this.viewer.scene.screenSpaceCameraController.enableZoom = false;
Then you could implement a scaled down version of handleZoom()

Hi @raycrummey !
I would be interested in how did you replace the handleZoom() function of cesium by yours ?
Did you modify the code and build the version with your fix ?
Or can you replace the function in your own code by setting your own function in some way ?
Btw, can you show me your own function ?
Thanks a lot for your response.

Came up again here https://github.com/AnalyticalGraphicsInc/cesium/issues/8476.

Opened https://github.com/CesiumGS/cesium/issues/8810 for rendering billboards in a separate pass, which would help fix this problem.

Looks like this commit fixes this. @kring do you happen to know if the bug you were working on for that commit is related to this issue?

Could be @YoussefV.

If this is fixed, that means we can look into enabling the billboard unburying path by default, right @hpinkos?

@mramato  #6838 needs to be fixed.  And probably #7376 and #8588 too


Screen sharing results in infinite nesting of images.

hi,How should this situation be resolved?
Screen sharing results in infinite nesting of images.

don't show local track, will solve this problem, otherwise it can't be resolved

@wanjm is right. the mirror effect is caused by your application rendering the screen share itself.


Disable New/Edit

Hey!
We have some tables that we dont want to allowing editing from the web UI. Is there a way to allow view only, and remove the "New" button?
Cheers!
Andrew

This question has been asked a thousand times. Search in the issues

I have searched, but couldn't find anything I could follow. Kindly provide a link to a posted issue.

Have a look at basic samples

Hey,
I looked at those but cant figure it out. I have managed to remove the form buttons, but not the new button on the main page using the following code:

namespace UCGWebPortal.Unilytic {
@Serenity.Decorators.registerClass()
@Serenity.Decorators.responsive()
export class EventLogDialog extends Serenity.EntityDialog<EventLogRow, any> {
    protected getFormKey() { return EventLogForm.formKey; }
    protected getIdProperty() { return EventLogRow.idProperty; }
    protected getLocalTextPrefix() { return EventLogRow.localTextPrefix; }
    protected getNameProperty() { return EventLogRow.nameProperty; }
    protected getService() { return EventLogService.baseUrl; }

    protected form = new EventLogForm(this.idPrefix);

    constructor(container: JQuery) {
        super(container);
    }

//Tried to copy this from the suggested example, but it doesn't do anything
protected getButtons() {
return [{
title: 'Perform Bulk Action on Selected Orders',
cssClass: 'send-button'
        }];
    }

//This appears to work:
protected getToolbarButtons() {
return [];
}
}

}
I take it there is no way to put an "entity" in to read only mode removing buttons and disabling controls?
Thanks!
Andrew

You can add the attribute Updatable(false) in the fields of the form (or in the row) or you can set read-only programmatically in dialog.ts
https://github.com/volkanceylan/Serenity/wiki/EnableDisableControls

That worked a treat thanks!
Apologies for all the dumb questions, struggle to find docs when searching.
Whats the best approach to disable and hide the buttons? Ideally I want to hide the "New", "Update", and "Save" button, but keep the "Delete" button. I attempted the above approach but couldn't achieve my requirement.

Thanks again!
Andrew

Override protected getButtons() in grid.ts and/or in dialog.ts
See RemovingAddButton.ts in the latest Serene template. volkanceylan has already answered you on this.
This is for add button, but work in the same way for the other buttons. Check their css classes.
This code is from basic sample in Serene:
        /**
         * This method is called to get list of buttons to be created.
         */
        protected getButtons(): Serenity.ToolButton[] {

            // call base method to get list of buttons
            // by default, base entity grid adds a few buttons,
            // add, refresh, column selection in order.
            var buttons = super.getButtons();

            // here is several methods to remove add button
            // only one is enabled, others are commented

            // METHOD 1
            // we would be able to simply return an empty button list,
            // but this would also remove all other buttons
            // return [];

            // METHOD 2
            // remove by splicing (something like delete by index)
            // here we hard code add button index (not nice!)
            // buttons.splice(0, 1);

            // METHOD 3 - recommended
            // remove by splicing, but this time find button index
            // by its css class. it is the best and safer method
            buttons.splice(Q.indexOf(buttons, x => x.cssClass == "add-button"), 1);

            return buttons;
        }

In dialog, method name is getToolButtons(), you override similarly and find buttons by their css class.

Thanks so much guys, got it all working :)

2.3.3 also has read-only dialog sample, for those interested.


Get value on onPress callback for ToggleButtonGroup

Describe the bug
How does one capture which toggle button was pressed in a toggle button group? I tried adding an onPress prop to the ToggleButtonGroup component, but didn't receive a value for it...
<ToggleButtonGroup
    onPress={val => console.log(val)} //undefined
>
    <ToggleButton />
    <ToggleButton />
</ToggleButtonGroup>

To Reproduce
Steps to reproduce the behavior:

Add onPress prop to ToggleButtonGroup

Expected behavior
Get a value back to show which one was pressed.

Looks like that's a bug. The value from ToggleButton should be passed to the onPress of the ToggleButtonGroup.
I've updated this on master, https://github.com/codypearce/material-bread/commit/bdcf3f20628109268801ae73b1868ba3f2a919cd.
You can test this by installing from master if you want.
I'm going to close this issue, but feel free to reopen if the issue persists.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Yarn Install Errors

Got a few errors from yarn during the initial clean install. Not familiar with yarn, looking for help.
Here is what I did incl. terminal output. Running on a M1
I checked the requirements first, seems fine:
Docker -v
Docker version 20.10.23, build 7155243
poetry -V
Poetry (version 1.4.2)
node -v
v18.12.1
python -V
Python 3.11.2
yarn -v
3.5.0

Then I fetch a fresh clone of the repo.
gh repo clone e2b-dev/e2b
Cloning into 'e2b'...
remote: Enumerating objects: 5913, done.
remote: Counting objects: 100% (577/577), done.
remote: Compressing objects: 100% (298/298), done.
remote: Total 5913 (delta 289), reused 522 (delta 271), pack-reused 5336
Receiving objects: 100% (5913/5913), 3.20 MiB | 9.78 MiB/s, done.
Resolving deltas: 100% (3920/3920), done.

cd e2b
Starting with the first yarn command. It has a problem and recommends to run "yarn install instead" which I did.
yarn install:all
Internal Error: ai-api@workspace:.: This package doesn't seem to be present in your lockfile; run "yarn install" to update the lockfile
at q0.getCandidates (/Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:435:5145)
at xf.getCandidates (/Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:391:1264)
at /Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:439:7693
at ff (/Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:390:11070)
at ge (/Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:439:7673)
at async Promise.allSettled (index 0)
at async to (/Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:390:10398)
at async /Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:439:8205
at async ti.startProgressPromise (/Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:390:47579)
at async je.resolveEverything (/Users/martin/.cache/node/corepack/yarn/3.5.0/yarn.js:439:6233)

It has a problem and recommends to run "yarn install instead" which I did.
yarn install
➤ YN0070: Migrating from Yarn 1; automatically enabling the compatibility node-modules linker 👍
➤ YN0000: ┌ Resolution step
➤ YN0061: │ stable@npm:0.1.8 is deprecated: Modern JS already guarantees Array#sort() is a stable sort, so this library is deprecated. See the compatibility table on MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/sort#browser_compatibility
➤ YN0032: │ fsevents@npm:2.3.2: Implicit dependencies on node-gyp are discouraged
➤ YN0061: │ @npmcli/move-file@npm:2.0.1 is deprecated: This functionality has been moved to @npmcli/fs
➤ YN0032: │ bufferutil@npm:4.0.7: Implicit dependencies on node-gyp are discouraged
➤ YN0032: │ utf-8-validate@npm:5.0.10: Implicit dependencies on node-gyp are discouraged
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (pb8aea), requested by @tiptap/core
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (pec51a), requested by @tiptap/extension-code-block
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (p15c05), requested by @tiptap/extension-dropcursor
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (p4a73e), requested by @tiptap/extension-gapcursor
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (p84589), requested by @tiptap/extension-history
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (pc2d7d), requested by @tiptap/extension-horizontal-rule
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements  for details, where  is the six-letter p-prefixed code
➤ YN0000: └ Completed in 10s 470ms
➤ YN0000: ┌ Fetch step
➤ YN0013: │ yaml@npm:1.10.2 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yargs-parser@npm:21.1.1 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yargs@npm:17.7.1 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yocto-queue@npm:0.1.0 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ zustand@npm:4.3.5 can't be found in the cache and will be fetched from the remote registry
➤ YN0000: └ Completed in 0s 440ms
➤ YN0000: ┌ Link step
➤ YN0007: │ supabase@npm:1.46.5 must be built because it never has been before or the last one failed
➤ YN0007: │ @prisma/engines@npm:4.11.0 must be built because it never has been before or the last one failed
➤ YN0007: │ bufferutil@npm:4.0.7 must be built because it never has been before or the last one failed
➤ YN0007: │ es5-ext@npm:0.10.62 must be built because it never has been before or the last one failed
➤ YN0007: │ utf-8-validate@npm:5.0.10 must be built because it never has been before or the last one failed
➤ YN0007: │ prisma@npm:4.11.0 must be built because it never has been before or the last one failed
➤ YN0007: │ @prisma/client@npm:4.11.0 [556c3] must be built because it never has been before or the last one failed
➤ YN0007: │ ai-api@workspace:. must be built because it never has been before or the last one failed
➤ YN0000: └ Completed in 9s 298ms
➤ YN0000: Done with warnings in 20s 277ms

Trying to use install:all again to make sure nothing was missed.
yarn install:all
➤ YN0000: ┌ Resolution step
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (pb8aea), requested by @tiptap/core
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (pec51a), requested by @tiptap/extension-code-block
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (p15c05), requested by @tiptap/extension-dropcursor
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (p4a73e), requested by @tiptap/extension-gapcursor
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (p84589), requested by @tiptap/extension-history
➤ YN0002: │ @tiptap/starter-kit@npm:2.0.0-beta.220 doesn't provide @tiptap/pm (pc2d7d), requested by @tiptap/extension-horizontal-rule
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements  for details, where  is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed
➤ YN0000: Done with warnings in 0s 387ms
Installing dependencies from lock file
No dependencies to install or update
Installing the current project: codegen (0.1.0)
➤ YN0000: ┌ Resolution step
➤ YN0032: │ fsevents@npm:2.3.2: Implicit dependencies on node-gyp are discouraged
➤ YN0032: │ utf-8-validate@npm:5.0.10: Implicit dependencies on node-gyp are discouraged
➤ YN0032: │ bufferutil@npm:4.0.7: Implicit dependencies on node-gyp are discouraged
➤ YN0061: │ @npmcli/move-file@npm:2.0.1 is deprecated: This functionality has been moved to @npmcli/fs
➤ YN0061: │ uuid@npm:3.4.0 is deprecated: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
➤ YN0000: └ Completed in 7s 761ms
➤ YN0000: ┌ Fetch step
➤ YN0013: │ yaeti@npm:0.0.6 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yallist@npm:4.0.0 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yamljs@npm:0.3.0 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yargs-parser@npm:21.1.1 can't be found in the cache and will be fetched from the remote registry
➤ YN0013: │ yargs@npm:17.7.1 can't be found in the cache and will be fetched from the remote registry
➤ YN0000: └ Completed in 0s 267ms
➤ YN0000: ┌ Link step
➤ YN0007: │ esbuild@npm:0.17.11 must be built because it never has been before or the last one failed
➤ YN0007: │ bufferutil@npm:4.0.7 must be built because it never has been before or the last one failed
➤ YN0007: │ es5-ext@npm:0.10.62 must be built because it never has been before or the last one failed
➤ YN0007: │ utf-8-validate@npm:5.0.10 must be built because it never has been before or the last one failed
➤ YN0000: └ Completed in 3s 281ms
➤ YN0000: Done with warnings in 11s 351ms

Ok, seems fine now. Let's try the next step.
yarn db:start
node:internal/errors:484
ErrorCaptureStackTrace(err);
^
TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension "" for /Users/martin/Code/e2b/node_modules/supabase/bin/supabase. Loading extensionless files is not supported inside of "type":"module" package.json contexts. The package.json file /Users/martin/Code/e2b/node_modules/supabase/package.json caused this "type":"module" context. Try changing /Users/martin/Code/e2b/node_modules/supabase/bin/supabase to have a file extension. Note the "bin" field of package.json can point to a file with an extension, for example {"type":"module","bin":{"supabase":"./bin/supabase.js"}}
at new NodeError (node:internal/errors:393:5)
at Object.getFileProtocolModuleFormat [as file:] (node:internal/modules/esm/get_format:79:11)
at defaultGetFormat (node:internal/modules/esm/get_format:121:38)
at defaultLoad (node:internal/modules/esm/load:81:20)
at nextLoad (node:internal/modules/esm/loader:163:28)
at ESMLoader.load (node:internal/modules/esm/loader:605:26)
at ESMLoader.moduleProvider (node:internal/modules/esm/loader:457:22)
at new ModuleJob (node:internal/modules/esm/module_job:63:26)
at #createModuleJob (node:internal/modules/esm/loader:480:17)
at ESMLoader.getModuleJob (node:internal/modules/esm/loader:434:34) {
code: 'ERR_UNKNOWN_FILE_EXTENSION'
}
Node.js v18.12.1
Got a stacktrace and no idea where I went wrong ...
Anyone got a idea how to solve this?

yarn -v 3.5.0

Mine is running on yarn version 1.22.19, which is the "Classic Stable" version on https://classic.yarnpkg.com/lang/en/
Could that be the issue?

That's it! Thanks a lot!
It was quite a struggle to activate the right Yarn version as I had multiple installs along the way, but switching to your version solved the issue.
Thanks again :)


Multilingual Issue

Dear authors,
I have two questions.
First, how can I use multilingual pre-trained BERT in pytorch?
Is it all download model to $BERT_BASE_DIR?
Second is tokenization issue.
For Chinese and Japanese, tokenizer may works, however, for Korean, it shows different result that I expected
import torch
from pytorch_pretrained_bert import BertTokenizer, BertModel, BertForMaskedLM
tokenizer = BertTokenizer.from_pretrained('bert-base-uncased')
text = "안녕하세요"
tokenized_text = tokenizer.tokenize(text)
print(tokenized_text)

` ['ᄋ', '##ᅡ', '##ᆫ', '##ᄂ', '##ᅧ', '##ᆼ', '##ᄒ', '##ᅡ', '##ᄉ', '##ᅦ', '##ᄋ', '##ᅭ']
The result is based on not 'character' but 'byte-based character'
May it comes from unicode issue. (I expect ['안녕', '##하세요'])

Hi, you can use the multilingual model as indicated in the readme with the commands:
tokenizer = BertTokenizer.from_pretrained('bert-base-multilingual')
model = BertModel.from_pretrained('bert-base-multilingual')

This will load the multilingual vocabulary (which should contain korean) that your command was not loading.


document(README): add BreakAway-specific instructions to README

Add more clear instructions on how to setup this action to ensure checkboxes in PRs are checked before merging.

Forcing merge to get this documentation in place so I can then share with rest of team.


Fix German Migration String Grammar

What it says in the title.
Some of the grammar was a little awkward and there was one typo.

This should definitely be a warning NOT to check that box unless you have a valid reason, although the "WÄHLEN SIE DIESE OPTION" made me chuckle a bit. I'll match the grammar correction in pr #22

Nvm overlooked the line break. In any case https://github.com/Yellow-Dog-Man/Locale/pull/22/commits/8d1ae4838bcf81f9367e85b95ce20c412955d7a9

Closing this now since it has been taken care of in https://github.com/Yellow-Dog-Man/Locale/commit/8d1ae4838bcf81f9367e85b95ce20c412955d7a9


Add attributeTopDown.

Stolen from @tpolecat.

Current coverage is 63.46%

Merging #9 into master will increase coverage by +0.72% as of 4103fe0

@@            master      #9   diff @@
======================================
  Files           15      15
  Stmts          204     208     +4
  Branches         4       4
  Methods          0       0
======================================
+ Hit            128     132     +4
  Partial          0       0
  Missed          76      76

Review entire Coverage Diff as of 4103fe0

Powered by Codecov. Updated on successful CI builds.

Just some minor stylistic comments in the tests. Even so, these are some great, concise, understandable tests! Nice work.

@mossprescott Updated.

LGTM

Yeah, ok … need to change how the merge script modifies the commit, looks like.


Crashing when creating world

So everytime I load into a world (in forge and fabric) it just crashes, i'll send a snip-it of the crash report:
https://pastebin.com/G5ruTrUx

interesting, can you send me a list of your mods and also send me your mod loader and mod loader version so I can reproduce this error.

it was on 1.20.6 and i was using just the defaults, i'll try to see if it still happens

yeah it still happens but it stops when i install embeddium on neoforge
i'm gonna check what happens on fabric

it was on 1.20.6 and i was using just the defaults, i'll try to see if it still happens

i was using just legacybob

okay i crash on fabric with fabric api and legacybob and when i install sodium it stops

that's interesting because during testing I never got any crashes with just bare LegacyBob. I will check and hopefully resolve the bug, if I can't then I will add sodium as a list of dependencies.

Fixed in LegacyBob 1.3


Fix - Primary Qualification Selection

Selects the primary qualification based on the "vatsim" code, rather than date added

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

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


Set correct content type of file in S3 storage

It's possible to set correct metadata in AWS by that command:
$ aws s3 cp s3://BUCKET/ s3://BUCKET/ --exclude "*" --include "*.jpg" --recursive --metadata-directive REPLACE --content-type "image/jpeg" --cache-control "public, max-age=31536000, immutable" --acl public-read


add swift package manager

Hi
please add swift package manager

In the 4.4.0 they added but I get this error:

https://github.com/Yummypets/YPImagePicker has no Package.swift manifest for version 4.4.0

@s4cha

@Maxvit5 This is supported, but you have to specify the branch. https://github.com/Yummypets/YPImagePicker branch spm. I missed the specify branch part in the readme at first and kept thinking it was broken.

https://github.com/Yummypets/YPImagePicker/releases/tag/5.0.0


scheduler: Make all schedulers watch events

This is so that the down event of the scheduler leader is forwarded to the controller by the non-leaders (previously they would miss the event as only the leader watched for events).
This also ensures that deploying a scheduler will wait for the new scheduler to be watching host events before killing old schedulers.

LGTM

LGTM


Removed deprecation warning.

There are warnings related to your configuration. If no errors occurred,
Terraform will continue despite these warnings. It is a good idea to resolve
these warnings in the near future.

Warnings:

  * module.ecs_cluster.aws_iam_instance_profile.ecs_profile: "roles": [DEPRECATED] Use `role` instead. Only a single role can be passed to an IAM Instance Profile

There are warnings related to your configuration. If no errors occurred,
Terraform will continue despite these warnings. It is a good idea to resolve
these warnings in the near future.

Warnings:

  * module.ecs_cluster.aws_iam_instance_profile.ecs_profile: "roles": [DEPRECATED] Use `role` instead. Only a single role can be passed to an IAM Instance Profile

@jasiek  thanks for this!


HOOBS config

I know this might seem like the wrong forum but I recently switched from Homebridge to HOOBS. I had had broadlink-rm working perfectly on homebridge but it seems impossible to configure it on hoobs and editing the config file directly causes everything to stop. Has anyone figured this one out?

Same for me :(

Solution located : https://www.reddit.com/r/HOOBS/comments/etk23j/cannot_find_module_hapnodejs/?utm_source=share&utm_medium=ios_app&utm_name=iossmf
Worked for me straight away

Thanks !
so for information here is what to be done before installing the Broadlink plugin.
Via Terminal :
sudo setcap cap_net_raw+ep /usr/local/bin/node
then
npm install hap-nodejs
Then install the Plugin...

Hello!
First of all - thanks a lot for bringing this plugin so far. It’s really great - in fact, it’s one of the greatest additions to the homebridge ecosystem I think. Thanks for that !!
I would like to ask if you are planning to make this plugin hoobs certified ?
It seems hoobs is really progressing fast and adds some senseful additions in terms of security and also ease of use. I would love to see this. Do you have any of this plannned?
@iprhodes

Same problem i manage to solve it configure it like this...
,
{
"platform": "BroadlinkRM",
"debug": "true",
"plugin_name": "homebridge-broadlink-rm",
"accessories": [
{
"name": "Logitech z906 Speaker",
"type": "switch",
"data": {
"on": "2600500000012990151015341510150f160f15101510150f160f15101510150f151015341510153415101510150f151015101510150f153416331634153415341534153415341510150005da0001294616000d050000000000000000",
"off": "2600500000012990151015341510150f160f15101510150f160f15101510150f151015341510153415101510150f151015101510150f153416331634153415341534153415341510150005da0001294616000d050000000000000000",
"hosts": {
"address": "<IP_ADDRESS>7",
"mac": "78:0f:78:00:5e:e7"
}
}
}
],
"plugin_map": {
"plugin_name": "homebridge-broadlink-rm"
}
}


feat(enharmonic_note): add toClosestNote method

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

23 of 23 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 92.401%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
304

Relevant Lines:
329

💛  - Coveralls


About domain overlap in the dataset

Hello!
I found that dataset Multiwoz has already been included in your pretrained data UniDA, while it is also used as your fine-tuning data. Will there be unfairness in your low-resource experiment result?
Thanks for your response in advance!

Thank you for your interest.
First, UniDA only contains the training dataset of MultiWOZ. Second, for each X% low-resource setting experiment, we re-train a GALAXY model on our pre-training data only including the X% MultiWOZ training dataset.


Only capturing first page of docsend document

I'm attempting to download https://docsend.com/view/u53utyp2j4ycg7r6 using the extension from the chrome web store v2.1.2 and when I do it attempts to scrape each page but then the final document is just the first page over and over again.

So unfortunately the extension only support capturing presentation slides, not documents. I didn't actually know this was an option on Docsend, but now I do. Let me see what it would take to capture the full document. Updating the issue title to reflect this.

Hi, I too have this issue. Would be extremely grateful if you could help to update the code for this use case as well. Many many thanks.

So unfortunately the extension only support capturing presentation slides, not documents. I didn't actually know this was an option on Docsend, but now I do. Let me see what it would take to capture the full document. Updating the issue title to reflect this.

Bumping this, it would be great if documents could be captured as well. Thanks for your effort in creating this extension : )

wen?


Allow to change default translation model namespace from config file

thanks @Gummibeer  :+1:

If @dimsav approves this I will put it into v9.1.0 #509

@Gummibeer sure
Nice work @ahmed-aliraqi !

Will merge this tomorrow, add it to changelog and release v9.1.0 after a long time as next Minor release! 😊🎉


fix(v1): mobile nav alignment

Fixes #254 for alignment

Motivation
(Write your motivation here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Closing due to staleness. Feel free to reopen in future 😄


Firefox startup is so slow it can trigger the slow JS timer

Was this with a uproxy-lib sample app or uproxy? Don't really see what we can do here...

Full uProxy. Any solution would involve figuring out how to delay, subdivide, or optimize the loading of Polymer, I think.

No plans to fix this in the forseeable future.


Rancher provisioned/managed RKE2/K3s data-dir support

Summary
Fixes #11061

Occurred changes and/or fixed issues

add k8sDistro dir to harvester cloud config path
add unit test

Technical notes summary

Areas or cases that should be tested

I just tested RKE2 cluster provisioning in general and it's working, afaik. Don't know how to test this further

Areas which could experience regressions

Screenshot/Video

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

FYI @torchiaf . This "touches" Harvester


https://www.opentech.fund/reimbursement/ is a broken link

@frjo adding https://www.opentech.fund/stipend/ to this ticket. In the interim, can you confirm whether the following pages are still operational and ok to use? cc @danblah

https://old.opentech.fund/reimbursement/
https://old.opentech.fund/stipend/

@becklm What current page should it go to? Or is the page missing completely?
Sent with GitHawk

Please use https://archive.opentech.fund/ instead of https://old.opentech.fund/. They go to the same site but "archive" is the correct one to use.
This seems to be webforms and that functionality is not in place on the new site so feel free to use the old ones for now. That site will work until OTF decides otherwise.

Got it @fredrik. In that case I've set redirects for /reimbursement and /stipend to https://archive.opentech.fund/reimbursement/ and https://archive.opentech.fund/stipend/ so they should now work cc @becklm


PYTHON ICON for virtualenv prompt

Rust, Ruby icon already support, I'm so susprise why virtualenv still don't have icon.
I get Python icon Unicode from here: https://vorillaz.github.io/devicons/#/cheat

NOTE: This is re-submmit PR of #257

Looks good! Thanks so much, @thuandt! Nice eye-candy contribution =)


Mesh's color is incorrect with rough material

Operating system or device - Godot version:
Windows7 64bit
https://bintray.com/calinou/godot/editor/2017-08-12.b034c4e#files
Issue description:
It's the same scene, but it looks different in nightly build (above) and alpha1 build (below).

Steps to reproduce:
Create a MeshInstance with CubeMesh, give it a SpatialMaterial, and set the material's roughness to 1.0.

I think the bug is fixed in today's(2017-08-18) build (https://bintray.com/calinou/godot/editor/2017-08-18.445b0ab#files)
I guess it's time to close the issue.
Thanks and good job guys!

2017-08-16 build

2017-08-18 build

Probably fixed by 3a4ff34.


Added countdown timer functionality in TimerRing

Thanks for the PR, however, I plan on deprecating this library and am not accepting new features. Really appreciate your work here but I do not have the time to continue maintaining this library.


Fixed typo in howto-conditional-access-policy-all-users-mfa.md

Fixed typo in the designation of Alex Weinert

#sign-off


add detail error message and fixed issue:#1532,#1366 - [closed]

In GitLab by @liyanq528 on Jun 7, 2018, 06:38
Merges 20180428 -> dev

In GitLab by @wknet123 on Nov 13, 2019, 11:04
closed


Add RunTimeEndian

Fixes #84
No noticeable change in benchmarks when not using RunTimeEndian. I didn't measure with it.
I changed dwarfdump to use it since performance doesn't matter much there, to test it and to see what the size change was:
Before:
   text	   data	    bss	    dec	    hex	filename
 680902	  22249	   3704	 706855	  ac927	target/release/examples/dwarfdump

After:
   text	   data	    bss	    dec	    hex	filename
 593190	  22025	   3704	 618919	  971a7	target/release/examples/dwarfdump

Coverage increased (+0.4%) to 93.857% when pulling 20dc85d80b0e69751df787a21d18d6eb1633cfce on philipc:endian into 414f8a1783da7cd6ff149c2b4d00ead1d8eac1ee on gimli-rs:master.

Coverage increased (+0.4%) to 93.856% when pulling c23c747aea8cd9ec371dd252e1e8259f072a0c67 on philipc:endian into 414f8a1783da7cd6ff149c2b4d00ead1d8eac1ee on gimli-rs:master.


Bug sur l'affichage des overflows

When page is reloaded overflow room disappears

I think that overflow state is stored in volatile variable.
Then this information must fetch from server.


Enable proguard for release build

This PR enables proguard for release build as requested in https://github.com/artem-zinnatullin/qualitymatters/issues/156

Current coverage is 77.89% (diff: 100%)

No coverage report found for master at 3bdf1e6.

Powered by Codecov. Last update 3bdf1e6...189fa59

Guys check this https://github.com/yongjhih/android-proguards

In my opinion it would be cool to have the possibility to update proguard rules by ourself without waiting for a 3rd party lib to be updated. @artem-zinnatullin What do you think about the lib mentioned about?

Well, I'm fine with small scoped libraries that provide proguard rules for specific libs.

Added the libraries instead of proguard files, @artem-zinnatullin could you please check.

Reverted the branch to user proguard files with Rx rules defined in https://github.com/artem-zinnatullin/RxJavaProGuardRules.

@Ilya-Gh thank you for PR and sorry that it took so long to review!


TCP RST : "simulate" connection failure

Currently, there is no way for libuv to terminate a TCP connection with a RST packet (simulating a connection failure).  This is a feature request.
Use case
The need for this feature arises from how different web browsers handle incomplete downloads from HTTP servers.  For example, if a file download ends prematurely (such as when the Content-Length header does not match the size of the file which was transferred), some browsers will still consider this to be a successful transfer (looking at you Chrome & Edge) so long as the TCP connection is ended gracefully.
Similarly, some web browsers will also consider an non-terminated chunked transfer to be "complete" too, so long as the connection terminates gracefully.
This has been discussed on the Chromium bugs board for years, and they've had a potential "fix" in their queue for over a year.
Today, the only universally reliable way for web browsers to "understand" that a transfer has failed is when a TCP connection is dropped ungracefully (i.e. with a RST packet).
I maintain a reverse HTTP proxy server written in NodeJS.  If an end-user is downloading a file via the proxy when suddenly a back-end file-serving web server goes down, the proxy has no reliable way to communicate to the end-user that the download has been interrupted.  Dropping the TCP connection gracefully will often result in the end-user being erroneously informed that the download has completed successfully (when it hasn't).
It also makes sense to me that if a proxy server experiences a RST from an upstream server, then it should be able to "pass" this down to the end-user.
How
In Winsock, I believe this can be achieved by enabling SO_LINGER (using setsockopt()) with a zero second linger time, before closing the connection as normal.
I don't know enough to offer any advice for how to achieve this on other platforms.

Zero SO_LINGER works on Unices too, so it should be possible to implement uv_tcp_close_reset(), or whatever a good name for it is.
That said, the interaction between zero SO_LINGER and shutdown() is not consistent across platforms.  Mixing of uv_tcp_close_reset() and uv_shutdown() would need to be avoided.

😕

I have a branch somewhere with this implemented. I’m AFK until next week.
I’ll create a PR then.
El El sáb, 10 ago 2019 a las 13:11, DDoSolitary<EMAIL_ADDRESS>escribió:

😕
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/libuv/libuv/issues/1991?email_source=notifications&email_token=AAIIVGFV5RE7X3XXO6KDSADQD2O5ZA5CNFSM4FVWLJG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4ALN2I#issuecomment-520140521,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIIVGEZ3YD3HTXRBOG2JETQD2O5ZANCNFSM4FVWLJGQ
.

Oooh, it sounds like this could be just the thing I wanted for writing JuliaLang socket tests too! (simulating network failures, to run the error code handling paths)