Add "nano" preset for WaveNet and LSTM

Also skew the LSTM presets toward simpler architectures.
Based on my testing I think the set of LSTM presets is pretty good, but I'm happy to discuss alternatives. In my opinion, it really isn't worth going below 1x12, and if you go bigger than 1x24 you don't really have any advantage over WaveNet.
The proposed LSTM presets are:
"standard" - 1x24
"lite" - 2x8
"feather - 1x16
"nano" - 1x12
If you want to try these out, I've got a dev version of the "easy" colab here:
https://colab.research.google.com/github/mikeoliphant/neural-amp-modeler/blob/devel/bin/train/easy_colab.ipynb
Note that you need to add model_type="LSTM" to train LSTM presets.

Oli, do you have any sample .nam files using LSTM that I can try?

So, about the proposed model architectures. I trained & ran one of each of the LSTMs. I'm finding that The RT performance of each is roughly the same:

Standard: 20.3x
Lite: 20.3x  (I'm mildly surprised that this isn't faster than feather...)
Feather: 20.9x
Nano: 21.0x

WaveNets:

Standard: 7.3x
Lite: 8.9x
Feather: 13.4x
Nano: 15.9x

Controls:

No model: 23.4x
No NAM: 24.9x
Neural DSP Archetype: Nolly Amp 3 (high oversampling, all other effects disabled): 7.5x

Everything run in Reaper, bouncing v1_1_1.wav to 48k mono with only NAM loaded up; noise gate and EQ are bypassed, no IR. VST3 built on
Is there a more meaningful difference in processor usage on some other system?
Also: is it really the case in your experience that LSTM-lite is more accurate than LSTM-feather for these? I only checked it on the files linked in the README, but feather came out a bit more accurate than lite (after only 100 epochs, though). Interested to hear.
[Don't mean to be scrutinizing this PR for any particular reason--just found myself with some spare time and some curiosity since I don't think I've taken down all the numbers at once before...figured I'd just have a look 🙂 If you can speak to the question about lite vs feather accuracy, I think I'm happy to approve otherwise.]

Is there a more meaningful difference in processor usage on some other system?

The numbers at the low end are hard to gauge well on a powerful machine. It is a lot more clear on more CPU-constrained devices.
My numbers on Raspberry Pi 4 are here:
https://github.com/mikeoliphant/neural-amp-modeler-lv2/tree/main/models
And @KaisKermani posted numbers for the MOD Dwarf over in the issue thread:
https://github.com/sdatkinson/neural-amp-modeler/issues/271#issuecomment-1591507749

Also: is it really the case in your experience that LSTM-lite is more accurate than LSTM-feather for these?

I got about 1/2 the ESR for 2x8 compared to 1x16, but those results obviously aren't as cut-and-dried as the performance numbers. ESR during LSTM training has a tendency to jump around a lot (something that seems to be improved a bit by upping "ny" - but that's another discussion).


5.4: Error: failed to create containerd task: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: exec: "/usr/bin/nice": stat /usr/bin/nice: no such file or directory: unknown

apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: cross-seed
  name: cross-seed
spec:
  selector:
    matchLabels:
      app: cross-seed
  template:
    metadata:
      labels:
        app: cross-seed
    spec:
      containers:
      - command:
        - /usr/bin/nice
        - /usr/bin/ionice
        - -c3
        - -n7
        - /usr/local/bin/cross-seed
        args:
        - daemon

We switched to alpine. nice is in /bin


add fALS

Add an entry to authors.tsv with your ORCID identifier and an entry to the xrefs.tsv to the EFO

Use the EFO namespace I just made instead
https://github.com/pharmacome/curation/blob/master/template.bel#L23


add SIMILAR TO operator

To cater the new matching operator: SIMILAR TO
https://www.postgresql.org/docs/current/functions-matching.html

Coverage remained the same at 92.27% when pulling 48b1aee77aa907627b684730737e815af9ad1b29 on rachelleannmorales:add-similar-to-operator into 2dadde4214d9ee333adccfa517089647e94d23be on knex:master.


feat: cli improvements

Proposed changes
Improved build command of the CLI
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Update (if none of the other choices apply)

Checklist

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream modules

Codecov Report

Merging #75 (2ffb4a6) into main (b1bb717) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #75   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           61        62    +1
  Lines          674       707   +33
  Branches        95        95
=========================================
+ Hits           674       707   +33

Impacted Files
Coverage Δ

packages/cli/src/utils/buildTheme.ts
100.00% <100.00%> (ø)

packages/cli/src/utils/getClassesCSS.ts
100.00% <100.00%> (ø)

packages/cli/src/utils/getComponentsCSS.ts
100.00% <100.00%> (ø)

packages/cli/src/utils/parseSlice.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Remove @Modifiable from all Immutables.

See, for example, AbstractAccountBalanceSettings.
Note, this will likely affect the ITs, so we will need to see if this change is compatible without too much rewiring of those tests.

This ticket is possibly related to the idea about in-place editing of the ILP packet. For example, it might be more performant to serialize to a Modifiable Immutable for the duration of the packet-switch, and then serialize out to bytes from the Java object to the wire via OER (though perf test will prove this out)


Screen ping for the mobile trackers

Screen ping for the mobile trackers - for discussion. Comes from customer suggestion:
Is there in the SDK a tracker to measure time on screen?

Hey @ihortom, must have missed this one!  At the moment there is not a way to do this.  However we are starting to track the android lifecycle for backgrounding and foregrounding of the application so there is scope here to try and record the time between an activity being created and then destroyed... will require some digging.

Discussion thread on iOS: https://github.com/snowplow/snowplow-objc-tracker/issues/275

Duplicate of #503


Color modifier won't type coerce

We should support:
<Rectangle
  red="0.45"
  green="0.1"
  blue="0.22"
  style={~s[Color(red: attr("red"), green: attr("green"), blue: attr("blue"))]}
/>

Color is not a modifier, but you can pass a color to a modifier like foregroundStyle.

yes, good point


CI: intermittent static analysis failures

https://github.com/commaai/openpilot/actions/runs/5747918611/job/15579785897?pr=29216

oh, failure is hidden inside warnings


Make puma log silence completely

Puma log is shown that puma server starts. Using action_dispatch_system_test_case load hook can stop its output
Before
$ bin/rspec spec/system/tasks_spec.rb
Capybara starting Puma...
* Version 3.12.0 , codename: Llamas in Pajamas
* Min threads: 0, max threads: 4
* Listening on tcp://<IP_ADDRESS>:51998
.

Finished in 1.39 seconds (files took 0.91103 seconds to load)
1 example, 0 failures

After
$ bin/rspec spec/system/tasks_spec.rb
.

Finished in 1.22 seconds (files took 0.84185 seconds to load)
1 example, 0 failures

Hello @ta1kt0me
Thanks a lot for your patch. By any chance do you think you can provide a test for this?
I will look at the CI (Puma requir Ruby 2.2+ minimum now).

Thanks for your comment, I'll try it.

Honestly I'll be happy to merge this when green (depends on #2013)

CI is now green. Do you think you can rebase a re-push? I will look at this, this afternoon. 👋

Hello
It seems your test is broken. What I did instead is:

clone the repo of https://github.com/rspec/rspec-rails/issues/1897#issuecomment-401015183
Specify your branch and other rspec ones https://github.com/benoittgt/puma_in_rspec_output/commit/df6d432da8e72a3311638396819c9d5c55a11d2b
Then when I run rspec I don't see anymore the puma log.

The test fails on:
$ bundle exec cucumber features/system_specs/system_specs.feature:38                                                                                [20:08:02]
/Users/bti/code/bundle/ruby/2.5.0/gems/gherkin-2.12.2/lib/gherkin/formatter/filter_formatter.rb:105: warning: constant ::Fixnum is deprecated
Using the default profile...
..F--

(::) failed steps (::)

Exit status was 1 but expected it to be 0. Output:

Randomized with seed 7465

Widget management
  enables me to create widgets (FAILED - 1)

Failures:

  1) Widget management enables me to create widgets
     Failure/Error: raise ActionController::RoutingError, "No route matches [#{env['REQUEST_METHOD']}] #{env['PATH_INFO'].inspect}"

     ActionController::RoutingError:
       No route matches [GET] "/stylesheets/application.css"

     # /Users/bti/code/bundle/ruby/2.5.0/gems/railties-5.2.0/lib/rails/rack/logger.rb:38:in `call_app'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/railties-5.2.0/lib/rails/rack/logger.rb:26:in `block in call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/railties-5.2.0/lib/rails/rack/logger.rb:26:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/method_override.rb:22:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/runtime.rb:22:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/sendfile.rb:111:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/railties-5.2.0/lib/rails/engine.rb:524:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/urlmap.rb:68:in `block in call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/urlmap.rb:53:in `each'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/urlmap.rb:53:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/rack-2.0.5/lib/rack/builder.rb:153:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/capybara-2.18.0/lib/capybara/server.rb:44:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/puma-3.12.0/lib/puma/configuration.rb:225:in `call'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/puma-3.12.0/lib/puma/server.rb:658:in `handle_request'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/puma-3.12.0/lib/puma/server.rb:472:in `process_client'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/puma-3.12.0/lib/puma/server.rb:332:in `block in run'
     # /Users/bti/code/bundle/ruby/2.5.0/gems/puma-3.12.0/lib/puma/thread_pool.rb:133:in `block in spawn_thread'
     # ------------------
     # --- Caused by: ---
     # Capybara::CapybaraError:
     #   Your application server raised an error - It has been raised in your test code because Capybara.raise_server_errors == true
     #   /Users/bti/code/bundle/ruby/2.5.0/gems/capybara-2.18.0/lib/capybara/session.rb:145:in `raise_server_error!'

Top 1 slowest examples (1.56 seconds, 99.8% of total time):
  Widget management enables me to create widgets
    1.56 seconds ./spec/system/widget_system_spec.rb:8

Finished in 1.56 seconds (files took 1.22 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/system/widget_system_spec.rb:8 # Widget management enables me to create widgets

Randomized with seed 7465

 (RSpec::Expectations::ExpectationNotMetError)
features/system_specs/system_specs.feature:59:in `Then the exit status should be 0'

Failing Scenarios:
cucumber features/system_specs/system_specs.feature:38 # Scenario: System specs driven by selenium_chrome_headless

1 scenario (1 failed)
5 steps (1 failed, 2 skipped, 2 passed)
0m5.098s

So fix is good for me. I don't know how much we invest to fix the test.

chromedriver isn't available on travis without installing it, if theres a driver that works built in that would work, but I can see the puma output in the output...

Are we accepting this patch without the test?

I think we need some form of integration or smoke test here really

@ta1kt0me do you think you have a look.
Otherwise I will look at it.

@benoittgt @JonRowe I'm sorry for replying so late and thank you for your advice. I found the way which resolves https://github.com/rspec/rspec-rails/pull/2011#issuecomment-406885744 's error, but I can not be sure it's good solution.
diff --git a/Rakefile b/Rakefile
index 6bb2e0c4..435e361f 100644
--- a/Rakefile
+++ b/Rakefile
@@ -60,7 +60,7 @@ namespace :generate do

       # Rails 4 cannot use a `rails` binstub generated by Bundler
       sh "rm -f #{bindir}/rails"
-      sh "bundle exec rails new ./tmp/example_app --no-rc --skip-javascript --skip-sprockets --skip-git --skip-test-unit --skip-listen --skip-bundle --template=example_app_generator/generate_app.rb"
+      sh "bundle exec rails new ./tmp/example_app --no-rc --skip-javascript --skip-git --skip-test-unit --skip-listen --skip-bundle --template=example_app_generator/generate_app.rb"

       in_example_app do
         sh "./travis_retry_bundle_install.sh 2>&1"

The above makes all spec in green in my local. Or if you have other better solution, it might be good.

I think the intent to skip sprocket was to make the app smaller.
Do you think you can add a commit with this change to check the CI?

I've added the change.

It seems their is an issue with the webdriver.
             Unable to find chromedriver. Please download the server from http://chromedriver.storage.googleapis.com/index.html and place it somewhere on your PATH. More info at https://github.com/SeleniumHQ/selenium/wiki/ChromeDriver.

As per my earlier comment:

chromedriver isn't available on travis without installing it, if theres a driver that works built in that would work, but I can see the puma output in the output...

:joy:

CI seems to be better.

@ta1kt0me Can you please rebase? CI is more stable lately.

@ta1kt0me Would you like to wrap this up before we release 4.0?

I can handle it if needed

This would further confirm your already quite intense comeback!


Safari Issue: Can't find variable: AudioContext

Apparently safari needs webkit prefix.
https://github.com/amaneureka/T-Rex/issues/5#issuecomment-217964069

Ps.. So nicely done this package.. Love it!

Are you using the node, or the web version?

CDN. If I copy down the bundle and add var AudioContext = window.AudioContext || window.webkitAudioContext; at the very top, the error goes away... Unfortunately, this causes some audio issues, as to now, the sound is eliminated.. And - it is JUST Safari...

I just made a quick CodeSandBox: https://codesandbox.io/s/playing-with-the-wavejs-package-zfgio using the NPM package. If you open it up in Chrome - No issues, In ssafari, you'll see the error again regarding AudioContext.

I'm having the same problem after adding the audio context fix, but I don't
have a apple device to debug the problem. Are there any errors after you
fix the problem?
On Wed, Jul 1, 2020, 11:10 PM Corfitz<EMAIL_ADDRESS>wrote:

CDN. If I copy down the bundle and add var AudioContext =
window.AudioContext || window.webkitAudioContext; at the very top, the
error goes away... Unfortunately, this causes some audio issues, as to now,
the sound is eliminated.. And - it is JUST Safari...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/foobar404/Wave.js/issues/7#issuecomment-652804475,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG22BHM36PAGIZJ274Z3RMDRZQQEHANCNFSM4ONPMYXQ
.

Fixing one seemed to have created a new :P I have tested the bundle with the fix here https://codesandbox.io/s/snowy-snow-p569j?file=/bundle.js , and the error does indeed go away in Safari - though now weird stuff happens where the audio doesn't even play.

Okay Ill be working on a solution in the next few days.

Have a look at #8 :D You may have to package the repo yourself for release. Not sure how to...

Okay I setup a development workflow and uploaded my source files. Look at the bottom of the readme to see details on how to work with the source files. I also worked for a while on the ios issue. Im not sure if I solved the issue because I dont have anyway of testing but I did implement something that should of fixed it. The audioContext is muted by safari, and is only unmuted when a user gesture happens. But you must also create the audioContext when the user event happens, and try to play a sound so safari unmutes. Hopefully the issue is now solved.

Great stuff this.. I have had a little browse-through... I looked into the react-test - which is failing due to the wave.fromElement only accepting strings for IDs and finding the elements automatically, whereas the thought of being able to assign an actual element as parameter, may be a good idea - especially on the react side of things.. We should look further into that..
The Safari issue has been resolved, but testing on the iPhone doesn't seem to work well.. Not sure if this is a localhost issue or bug in code, so will get back to you on that.. Great job solving the Safari bug.. cheers..

Oh.. Guess we can actually close this issue now then
/close


Fixed a typo that was causing some confusion on the Editor.

Overview
The text in the Editor property field was wrong.
Changes

Fixes: Fixed the typo error.

Verizontal is my favorite typo! XD


Link changed

The UK Census Open Atlas Project has moved to http://www.alex-singleton.com/r/2013/02/05/2011-census-open-atlas-project/

Its been fixed! Thanks!


felix agent: bind failed: Address already in use

happens sometimes:
Will update in 60 seconds...
ERROR: transport error 202: bind failed: Address already in use
ERROR: JDWP Transport dt_socket failed to initialize, TRANSPORT_INIT(510)
JDWP exit error AGENT_ERROR_TRANSPORT_INIT(197): No transports initialized [debugInit.c:750]
FATAL ERROR in native method: JDWP No transports initialized, jvmtiError=AGENT_ERROR_TRANSPORT_INIT(197)
/tmp/node-agent.sh: line 247:   132 Aborted                 (core dumped) java $props1 "$props2" -jar target.jar

fixed by INAETICS/node-agent-java@5536a77


Updates to PubSub+ event broker for August Release

The changes included in this PR:

Fix the health-check port to be accessible to the AWS load balancer
Quickstart support for Single Storage
Support Private VPC Broker HA Deployment With Broker Services Exposed Internally
Support for new Region Stockholm.

Hi @vsnyc , does this PR require a step from you or will it be auto-merged to develop? Let us know please if there is anything required from Solace side.

/do-e2e-tests


Fix some format

Fix format

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

This change is

@abiogenesis-now I tried getting the inner list to be bulleted. Can't seem to get it to work. See the two most recent commits.

It's so strange.

@stewart-yu @steveperry-53 did you try four spaces before the bullet? It should be similar to:

numbered item 1

nested bullet 1
nested bullet 2

numbered item 2
numbered item 3

1. numbered item 1
    * nested bullet 1
    * nested bullet 2
1. numbered item 2
1. numbered item 3

@heidecke Yes, we've tried four spaces.
I'm going to say we've spent enough time in this. Merging.
If any of you feel strongly that we should stick with this bulleted list problem, feel free to open another PR, and we can fiddle with it there.


Updating source maps

It might be more of a question that an issue (or it might not be related to browserify-hmr) but it looks like source maps aren't updating with browserify-hmr, is there a way to have them updating somehow ?

As far as I can tell, browserify-hmr handles sourcemaps right. It seems like browsers have issues with loading new sourcemaps for files: https://github.com/gaearon/react-hot-loader/issues/180.

That's good to know and annoying at the same time :/
I'm closing the issue since it's not related to browserify-hmr.


Add documentation

Start with repo readme, is it already outdated.
Maybe create directories for each data source/algorithm, each with a short how-to readme?

Yes, main readme can describe the purpose (since not everybody understands github actions), and then you can have links to subfolder readmes with more details. It would be good to have instructions like:

here is how to run the workflow with default parameters
here is where to change stuff to run the workflow with new parameters.

I want to work on this issue, I have checked all the things which should be present in the README file.
So please assign me, to this issue. @Molkree


feature: nominal transfer fee

This PR does two things:

Adds a nominal transfer fee to the chain config
Subtracts the fee from the transferable governance token balance.

Because we validate against the transferable balance, this means there is no need for an additional validation step as the transferable balance now takes into account the nominal transfer fee, for example:

transferable balance from the lib = 10.00005
nominal transfer fee = 0.00001
transferable balance in UI = 10.00004

@nud3l the reason this is in draft is that I want to check you're happy with the transfer fee effectively being hidden from the user. We're not flagging that we're deducting a nominal amount from the transferable balance, we're just handling it behind the scenes (e.g. a transferable balance of 0 as displayed to the user is 0.00001 as returned from the lib).
I think this is ok because the aim with this ticket is to prevent the transaction failing rather than displaying the nominal transfer fee which wouldn't necessarily match the real transfer fee in any case.

Can't we just use the actual fee estimate? It's rather straightforward to query them: you can use the payment.queryInfo or payment.queryFeeDetails rpc calls

Can't we just use the actual fee estimate? It's rather straightforward to query them: you can use the payment.queryInfo or payment.queryFeeDetails rpc calls

I don't think that's being returned from the lib yet, although I've only had a quick look - the requirement from Dom for this was pretty specific :)
https://www.notion.so/interlay/Flag-transfer-fees-806e8a0cd8ec4182936967916eca7544

Closing this as no longer required. We've been doing work to handle fees at the component level so I think this has been superseded.


Add git.commitType configuration

This controls what happens when committing with nothing staged
It has options for committing all files, committing just tracked fiels
or not committing.

@ashirley,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@ashirley, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

This is related to issue #15613 and PR https://github.com/Microsoft/vscode/pull/25364


Payal Patra

Due we close new issues on PR merge and voting is still possible on closed issues I will close this one. Nice Profile btw 🚀


Feature Request: caching using redis async

Hello,
thanks for taking back the maintainership, now cache is only file based. Could you permit redis async cache also?
Modification could need to try/catch import of redis.asyncio to be optional at import for instance

Why not! Good idea.
3 remarks:

you could make it more pythonic like using get() put() instead of camelCase cache*, getitem would be even better actually but a bit overkill for our usecase
cacheExpired method won't be useful with Redis because if some key has expired, it will return None as well.
I don't understand why you would like to store the timestamp, it's native to Redis. IMHO it should be handled directly by this class and not pass an argument since it's specific to cache system that don't provide native expiration date.

thanks, yes those are good steps forward,
initially i just used the existing logic,
your point of simply returning None when expired will simplify the current messy implementation
will cleanup ,
my current target is to be able to use a caching decorator to fully separate the caching from the data processor
and add a dbm or SQLite cache ,
the current JSON all in one file seems a bit slow in the long run

ok reworked the caching, much simpler now
only get, put required and a init on starting the cache
https://github.com/mboot-github/WhoisDomain/blob/master/whoisdomain/doWhoisCommand.py#L204

Great, just one remark:
Actually with your new code, I could pass directly a (sync only) Redis object to can't use async redis with your code if you permit to pass the cache object as parameter (default value None) and if it's not None, init it https://github.com/mboot-github/WhoisDomain/blob/master/whoisdomain/doWhoisCommand.py#L208
OR if you don't test "isinstance" and just use duck typing instead...
This way I could pass my cache even without creating a SimpleCache class for Redis!
Yet it won't work async, since the whole codebase is sync anyway that's very minor.

latest code now has: withVerifyCacheStubType default True
it is a hack for now
import whoisdomain

whoisdomain.CACHE_STUB = <your cache implementation object>

pc = ParameterContext(
    withVerifyCacheStubType=False,
    <<your other options to whoisdomain must now go here>>,
)

# no other options they will all be ingnored when using pc=pc
# only domain is now a valid param , all the rest are read from ParameterContext

data = whoisdomain.query(
    domain="yourdomain",
    pc=pc,
)

I feel this to be a bit overkill? Either one check isinstance because we must ensure conformity to specification or we don't need so. Forcing not to check seems weird for me IMHO.

ok removed that, the cache.get()/cache.put() part is now in a try/catch

try/cactch removed ;-)
timeout on cli whois gives wrong results, new code updated, more testing to done before release

testCache.py now has a example RedisCache

without further comment we will close the issue next week.


Bridge: Set up Cypress

Action Item
Set up cypress in the Bridge with a dummy test and run it as part of our nightly build.

The Nx Cypress Plugin can be used as a good reference for the setup


LoadBalancingPartition: Clean-Up Unhealthy Dangling Connections.

Description
Scope: Applicable with and without the "Advanced Replica Selection" feature.
Problem: One or our recent conversations with the IC3 team has helped us unveiling a potential issue with the RNTBD connection creation and management flow in the LoadBalancingPartition. The team is using the .NET SDK version 3.39.0-preview to connect to cosmos db. Recently they encountered a potential memory leak in some of their clusters and upon investigation, it appeared that one of the root cause is the underlying CosmosClient is keeping a high number of unhealthy and unused LbChannelStates.
In a nut shell, below are few of the account configurations and facts:
• account-1: 9 Partitions with 1 unique tenant. There are approx 4 to 8 clients for this tenant. 2 * no of replica regions is the client count. They have the connection warm-up enabled on this account.
• account-2 2592 Partitions with 249 tenants/ feds. Connections created in happy path scenario: 249 x Y (Y = number of active clients for that account). Connection warm-up disabled on this account.
• account-3: 27 Partitions with 13 tenants/ feds. CreateAndInitialize They have the connection warm-up enabled on this account.
To understand this in more detail, please take a look at the memory dump below.

[Fig-1: The above figure shows a snapshot of the memory dump taken for multiple accounts This also unviels the potential memory leak by the unhealthy connection.]
Upon further analysis of the memory dump, it is clear that:

The number of stale unhealthy connections are higher in the accounts where we have the replica validation enabled along with the connection warm-up.

Without the connection warm-up, the number of stale unhealthy connections are comparatively lower, but still good enough to increase the memory foot-print.

[Fig-2: The above figure shows how the memory footprint got increased over time, along with incoming requests. The service was finally needed to be restarted to free up the memory]

Even without the replica validation feature, the memory footprint showed a consistent increase over time.

[Fig-3: The above figure shows the memory consumption from the IC3 partner-api service which is using an older version (v 3.25.0) of the .NET SDK and the memory consumption kept increasing with time.]

Analysis :  Upon further digging in to the memory dump, and re-producing the scenario locally, it was noted that:

With Replica Validation Enabled: Each of the impacted LoadBalancingPartition was holding more than 1 Unhealthy stale LbChannelState (which is a wrapper around the Dispatcher and a Channel),  when the connection to the backend replica was closed deterministically.

With Replica Validation Disabled: Each of the impacted LoadBalancingPartition was holding exactly 1 Unhealthy stale LbChannelState (which is a wrapper around the Dispatcher and a Channel),  when the connection to the backend replica was closed deterministically.

Let's take a look at the below diagram to understand this in more detail:

[Fig-4: The above figure shows an instance of the LoadBalancingPartition holding more than one entry of unhealthy LbChannelState.]
By looking at the above memory dump snapshot, it is clear that these stale LbChannelState entries are kept in the LoadBalancingPartition, until they are removed the openChannels list, which is responsible for maintaining the number of channels (healthy or unhealthy) for that particular endpoint. If they are not cleaned up proactively (which is exactly this case), it might end up claiming extra memory overhead. With increasing number of partitions, connections over the time, things get worse with all these unused, yet low hanging LbChannelStates claiming more and more memory, and causing it to a memory leak. This is the potential root cause of the increased memory consumption.
Proposed Solution :
There are few changes proposed to fix this scenario. These are discussed briefly in the below section:

During the replica validation phase, in the OpenConnectionAsync(), proactively remove all the Unhealthy connections from the openChannels within the LoadBalancingPartition. This guarantees that any unhealthy LbChannelStates will be removed from the LoadBalancingPartition, freeing up the additional memory.
Yet to be identified: Figure out ways to avoid opening duplicate connections to the same endpoint, multiple times.

FAQs:

Is this applicable only for the newer version of the SDKs?
Ans: No, the scenario can happen with the older version of the SDK too. As discussed in the above sections, the root cause of this problem is with the connection management and clean-up. It is observed from the memory dump that with older version of the SDKs, each impacted LoadBalancingPartition holds exactly one stale connection. Thus, with increasing number of connections, the memory utilization can be increased due to these unused stale connection stying in the memory.

Does enabling "Advanced Replica Selection" make the memory consumption Worse"?
Ans: Yes, it does. The advanced replica selection feature is designed in such a way that it keeps track of any Unhealthy replica that had a connectivity issue, and temporarily quarantines it so that the incoming requests has higher chance of landing on to a Healthy replica. Additionally, the feature validates the Unhealthy replica by proactively opening a connection to check if the replica came back up. This helps dramatically to reduce the laency for read workloads, when a replica undergoes upgrades etc. However, these proactive open connections can potentially increase the number of stale connections, when that connection is closed from BE for idleness. This has a larger impact on increasing the memory footprint today.

Hi @kundadebdatta ,
It looks like we're encountering this issue in our production environment as well. In one of the dumps, we observed over 300 LbChannelState instances in an unhealthy state for a LoadBalancingPartition. Does this mean we have replica validation enabled? Is this feature turned on by the SDK or the account?
Regarding the unhealthy state, do you know under what circumstances the connection becomes unhealthy? Also, is it possible to detect this from the SDK diagnostic logs or metrics in Cosmos DB? Additionally, do you have an ETA for when the new Cosmos SDK release will be available?


Any additional work around for site requiring multiple certificates for authentication?

Thank you for your wonderful walk through. I've previously had experience installing CAC on my iPad and then a Chromebook but struggled much more trying to get CAC software working on Linux (Tuxedo OS) right now. Your solution to install the different middleware was what worked :-).  However, I'm not able to get onto some more of my complex CAC protected websites that likely require multiple certificates and for which I can get on with my CAC software on my Chromebook and iPad. I checked that I have all proper certificates installed (same as other devices) and checked the tests you recommended to determine whether the middle ware can see my CAC card and my certificates (it seems to as passes the tests).
However, one site in particular (my work virtual Citrix desktop) is giving me the most problems -- I can log onto the Citrix storefront but when it tires to actually connect and get permissions for the Virtual Desktop it fails with a not very, "You failed to trust root CA 3" message despite clearly having it marked in the browser (Chrome's) security section as trusted. (Again, the same certificates work in Chrome to connect to the website on a different device-- works on ChromeOS ARm64 Chromebook on Chrome browser. Besides the obvious difference that different type of machine not working on (x86-64 AMD machine) on Tuxedo OS Linux using Chrome as browser, I was wondering if you had any further problems with your setup you described in your walk through not be able to authenticate to certain websites that you can on non Linux devices.
Thanks!

I figured out and solved the issue :-)
Please see:
https://github.com/jdjaxon/linux_cac/issues/30


fix(config): clarify dim level parameter label for GE/Jasco 26932 / 26933 / ZW3008

Tweak the label of parameter 2 to highlight it's the dim level sent to associated lights, not the switch itself.
I missed that it said it in the description. Crossed checked with: https://drzwave.blog/2021/02/16/jasco-motion-dimmer-explained/

@blhoward2 I think I've seen this parameter in your PR (although in a different file). Might make sense to update it there too.

@zwave-js-bot rename commit


Gulp cannot find locally installed node

I have node installed in local folder and not installed globally.
For different tools I configured to run with terminal and configured node path:
"terminal.integrated.env.windows": { "PATH": "mynodepath" },
"tslint.nodePath": "mynodepath",
"npm.runInTerminal": true,
"yarn.runInTerminal": true,

I don't see option to configure gulp autodetected to use terminal and avoid installing Node globally.
'node' is not recognized as an internal or external command,
operable program or batch file.

Auto detecting gulp for folder Test failed with error: Error: Command failed: node_modules\.bin\gulp.cmd --tasks-simple --no-color
'node' is not recognized as an internal or external command,
operable program or batch file.

You should just add node to your PATH globally, that solves all the issues. Otherwise you can do it for each feature/extension:
"terminal.integrated.env.windows": { "PATH": "mynodepath;${env.PATH}" }

I have already set nose in terminal path but seems like VSCode gulp runner doesn’t use it.
Adding node globally is not a good option for me.

@NN--- you can report against the extension if you want but the real solution is to globally set it. Maybe you could try a powershell/batch script to setup the node path when launching VS Code?

According to documentation it is not an extension but part of VSCode: https://code.visualstudio.com/docs/editor/tasks

Which part? tslint is an extension, if npm.runInTerminal is true it should inherit the env from terminal.integrated.env.windows.

My bad, I thought it comes with VSCode.
Thanks.


Markdown in description

In your demo: http://rebilly.github.io/ReDoc/ you have markdown in the description field and manage to create additional menu items by markdown headlines. I try to recreate that but are not able to get the same result. Is there any special thing that have to be set?
Are using json not yaml

@runes83 probably you're missing some \n or spaces according to markdown spec.
Send me a sample of your swagger and I will try to help

Tried using your example text but still error, here is the swagger:
https://gist.github.com/runes83/35579c4faf28a9f765c3afaab28a8a9e

I can't see markdown headers in your description field
They should start from # sign.
check example here: https://github.com/Rebilly/ReDoc/blob/master/demo/swagger.yaml#L8

@RomanGotsiy I've updated the gist with your petstore markdown and still the same result

Here is a screenshot of what it looks like

Now you have extra spaces before your text which mean code block in markdown.
Please validate your markdown in any markdown editor, e.g. https://stackedit.io/editor

@runes83 here is the working gist: https://gist.github.com/RomanGotsiy/d700746a48964a63610435e572319ae1

@RomanGotsiy Thanks it look like Swaschbuckle dis somthing to the markdown it works now :-)


jackal: 0.5.2-1 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository jackal to 0.5.2-1:

upstream repository: https://github.com/jackal/jackal.git
release repository: https://github.com/clearpath-gbp/jackal-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.21
previous version for package: 0.5.1-0

jackal_control
* Removed reference to FootprintLayer.
* Increased inflation radius to account for uneven wall in Jackal_world.
* Added pointgrey camera and accessories.
* Improve robot_localiztion params
* Added Sick LMS1XX URDF.
* Fixed example calibration output.
* Added tutorials.
* Contributors: Mike Purvis, Martin Cote, Tony Baltovski, James Servos

jackal_msgs

No changes

+1


Add llama3 level0 example

Description
Background: https://github.com/analytics-zoo/nano/issues/1706#issuecomment-2437244193
Add llama3 example first.
llama3 use another prompt format, so separate to different script.
4. How to test?

[x] Application test

Shall we add openvino into ipex-llm[npu] ? May need to update setup.py ?

Shall we add openvino into ipex-llm[npu] ? May need to update setup.py ?

I think we don't need to add, [npu] option is enough.


Error in delegating from web

Describe the bug
A user in Discord reported experiencing an issue repeatedly when attempting to perform a delegation. The error message they received is:
Error: [unknown] Tendermint error 1: failed to deliver transaction: executing transaction: given
198000000 unbonded stake, expected
197722849 delegation tokens but description
produces 197791845

The user reports experiencing this over a one hour period, against multiple validators:

I tried first at anatolian team, later Penumbra labs 1 . Both gave error.

When they use the same delegation amount, the values reported in the error message remain the same.
To Reproduce
Steps to reproduce the behavior:

Unclear

Screenshots

Flagging @grod220 - is there an associated issue on the web side? Seems worth parallel investigation

I cannot replicate this on current main, and this was tested before v8.0 was released. I'll investigate the context.

Flagging @grod220 - is there an associated issue on the web side? Seems worth parallel investigation

there isn't currently an issue on the web side about this.

I also cannot replicate, I was able to delegate 1um to the Penumbra Labs 2 validator: https://app.testnet.penumbra.zone/#/tx/12f85129322f516e830a8928069605171113808378e9e0394f695c7fd299e046

Closed as non-issue, user seems to have needed to refresh their cache


Move intake form into an embed, add link to MaskForce

This addresses #77. I spoke with Danielle from Mask Force, and she agreed that because they request very specific information from hospitals, it makes sense for us to link to one another instead of attempting to take requests for home-sewn masks directly.

Ah damn. I totally rebased this PR onto the wrong thing. Going to open a new PR instead.


Genotyping

Hi,
Just a quick question, wham does not provide genotyping step itself? We need to use another program to genotype wham's result?
Thanks.
George

Hello,
Yes, I'de recommend you use SVTyper for whamg SV calls.

Hello,
Yes, I'd recommend SVTyper for WhamG calls.

Thanks!


UAA quickstart problems. Can't get it working after many many hours.

SECURITY NOTICE: If you have found a security problem in the UAA, please do not file a public github issue. Instead, please send an email to<EMAIL_ADDRESS>Thanks for taking the time to file an issue. You'll minimize back and forth and help us help you more effectively by answering all of the following questions as specifically and completely as you can.
What version of UAA are you running?
What output do you see from curl <YOUR_UAA>/info -H'Accept: application/json'
How are you deploying the UAA?
I am deploying the UAA

locally only using gradlew

What did you do?

$ git clone git://github.com/cloudfoundry/uaa.git
$ cd uaa
$ ./gradlew run
curl -H "Accept: application/json" localhost:8080/uaa/login

Then I did every possible things it recommended I could find but nothing really helped.
What did you expect to see? What goal are you trying to achieve with the UAA?
{
  "timestamp":"2012-03-28T18:25:49+0100",
  "commit_id":"111274e",
  "prompts":{"username":["text","Username"],
    "password":["password","Password"]
  }
}

What did you see instead?
FAILURE
Please include UAA logs if available.
uaa-server.log

Followed your exact steps. That checks out the develop branch
curl -H "Accept: application/json" localhost:8080/uaa/login

{"app":{"version":"4.28.0-SNAPSHOT"},"links":{"uaa":"http://localhost:8080/uaa","passwd":"/forgot_password","login":"http://localhost:8080/uaa","register":"/create_account"},"zone_name":"uaa","entityID":"cloudfoundry-saml-login","commit_id":"6c68c1a","idpDefinitions":{},"prompts":{"username":["text","Email"],"password":["password","Password"]},"timestamp":"2019-02-07T16:08:53-0800"}

Are you using Java 8? Your log shows that it works too.


FloatingActionMenu Icons

Hi, i don't see in example how can i add an icon image in buttons of floating action menu buttons, anybody help me?

Hi,
How to change fab menu button ? from plus icon  to other icon


 It's essentially a wrapper around the actual exception.
It would make it easier for end-users of the NSwag-generated API if the API would just throw the wrapped exception instead of an aggregate exception.

Where is this happening?

woops, my bad. It's the runtime doing it because my tests are waiting for the async client to run.


Got "No suitable kernel definition found for op BatchNormalization(9)"

Describe the bug
When call BatchNormalization op with opset=9, got "RuntimeError: [ONNXRuntimeError] : 1 : GENERAL ERROR : No suitable kernel definition found for op BatchNormalization(9)" error. Used inputs/outputs/attribute is no problem with opset=8 or 7. I think the difference between opset 9 and 7, 8 is spatial attribute and the value is not set.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): both Ubuntu 16.04 / 18.04
ONNX Runtime installed from (source or binary): binary, via PyPI
ONNX Runtime version: 0.2.1
Python version: both 3.5.6 / 3.6.7
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

To Reproduce
Reproduce test code example: https://gist.github.com/disktnk/f9a0b0ec12daf8f39d09fc86ade2a08e
(snippet)
    x = np.array([[[[-1, 0, 1]], [[2, 3, 4]]]]).astype(np.float32)
    s = np.array([1.0, 1.5]).astype(np.float32)
    bias = np.array([0, 1]).astype(np.float32)
    mean = np.array([0, 3]).astype(np.float32)
    var = np.array([1, 1.5]).astype(np.float32)
    y = _batchnorm_test_mode(x, s, bias, mean, var).astype(np.float32)

    node = onnx.helper.make_node(
        'BatchNormalization',
        inputs=['x', 's', 'bias', 'mean', 'var'],
        outputs=['y'],
    )
    inputs = [x, s, bias, mean, var]
    outputs = [y]
    graph = onnx.helper.make_graph(
        nodes=[node],
        name='test_batchnorm_example',
        inputs=inputs,
        outputs=outputs)
    opset_version = 8  # 9 is failed by "GENERAL ERROR", 7 and 8 are succeeded
    model = onnx.helper.make_model(
        graph,
        producer_name='backend-test',
        opset_imports=[onnx.helper.make_opsetid('', opset_version)]
    )

    import onnxruntime as rt
    sess = rt.InferenceSession(model.SerializeToString())

Expected behavior
No error
Screenshots
Additional context

The latest ONNX Runtime only supports ONNX 1.2 and 1.3, not 1.4. We are working on opset 9 support, but it's not finished yet.

Wait, I implemented it.   https://github.com/Microsoft/onnxruntime/commit/fb7be27096ee5492ef9889ce5c37a07946c6fd1a
I totally forgot that.

Ah, it's not in 0.2.x release.  Please wait for our next release, or , you can checkout the latest master and build it by yourself. Thanks.

I got it, thanks!


Restricting VPN profile creation to certain users

Hi! This may be a noob question.  Is it possible to restrict create of vpn profiles to only certain users if using Google Authentication?  I would like to have approved users be able to automatically have profiles created, but only approved users. Thanks!

This is currently possible for IPSec IKEv2 VPNs but not for OpenVPN.

I am closing this issues, unless you want this to be done with OpenVPN, we will prioritise that in that case.


Can't type on mobile when using inline

On mobile (iOS, Safari & Chrome), I can select a div that has ui-tinymce, but I can't type. The keyboard does show up, but typed letters don't show up in the div and are not added to the model.
This is the div:

These are the options:
$scope.tinymceOptions = { 	inline: true, 	plugins : 'autolink, autoresize, charmap, image, imagetools, link, paste, table', 	browser_spellcheck : true, 	autoresize_min_height: 450, 	autoresize_max_height: 500, 	autoresize_bottom_margin: '50', 	menubar: 'edit insert format table tools', 	default_link_target: '_blank' };
The css class only has some borders and height.
I'm using:

tinymce#4.3.3 (also tried 4.3.7)
angular-ui-tinymce#0.0.14

Typing does work when I change the div into a textarea, but then the textarea is filled with this text:
<br data-mce-bogus="1">
Happens on desktop & mobile. When I change it, it changes back to that string.

Can you please set up a plunker with your exact example and I can take a look into fixing it?

Thank you for your quick reply Dan! I'm leaving for holiday now but I'll make a Plunker as soon as I'm back.

On 4 mrt. 2016, at 19:23, Dan<EMAIL_ADDRESS>wrote:
Can you please set up a plunker with your exact example and I can take a look into fixing it?
—
Reply to this email directly or view it on GitHub.

Tried it in a Plunker and it worked, so I guess there's something else in my app that breaks it. I'll dive into it!

Great, feel free to open a new issue or report back here if you find it is a bug.


Geojson output

Should planet-stream output geojson? We have the full geometry, and it would serve downstream apps better if every record is geojson w/ properties.
cc @smit1678 @matthewhanson

Yes, I think it should


fix zero OID parse prepared statement

No issue to close
Description
fix handling zero OID to parse prepared statement
Is it a feature that change user experience?
no
Client output
no

Pull Request Test Coverage Report for Build 351451937

18 of 36 (50.0%)  changed or added relevant lines in 4 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 89.351%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/connection/src/lib.rs
1
3
33.33%

src/pg_model/src/statement.rs
0
2
0.0%

src/node/src/query_engine/mod.rs
11
25
44.0%

Files with Coverage Reduction
New Missed Lines
%

src/node/src/query_engine/mod.rs
7
59.31%

Totals

Change from base Build 347914731:
-0.3%

Covered Lines:
7434

Relevant Lines:
8320

💛  - Coveralls


Exe not there

There is no wfcreplay.exe file within this download.
I can't use visual studio because my computer only has a 28 gb hard drive, and I only have 5 gb left.
Can I use the linux exe though?
I'm on windows 8 32 bit

Yes there is.
https://github.com/Prof9/wfcreplay/releases


[drm] Allow user to specify which monitor to display on with the SFML_DRM_CONNECTOR environment variable

This is the first of two further drm PRs that I have that were held up pending the merge of https://github.com/SFML/SFML/pull/3136
This PR allows users with multiple monitors to tell SFML which monitor to send the drm output to.
The SFML_DRM_CONNECTOR environment variable is used to tell the library which connector to use for output.  So for example SFML_DRM_CONNECTOR=HDMI-A-2 will direct output to the monitor connected to the system's second HDMI port.
It would be great if this could be considered for 2.6.2 as it is pretty critical for having a reasonable way (on my system at least) to test the drm mode setting, etc.

Description

This PR is related to the issue #
Tasks

[ ] Tested on Linux
[ ] Tested on Windows
[ ] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

How to test this PR?

#include <SFML/Graphics.hpp>

int main()
{
    sf::RenderWindow window(sf::VideoMode({1280, 720}), "Minimal, complete and verifiable example");
    window.setFramerateLimit(60);

    while (window.isOpen())
    {
        while (const std::optional event = window.pollEvent())
        {
            if (event->is<sf::Event::Closed>())
                window.close();
        }

        window.clear();
        window.display();
    }
}

Would've been great if we had some better communication for this. I didn't get a reply on the other PR and SFML 2.6.2 has now already been tagged.

When do you plan on porting your previous DRM PR to SFML 3? SFML 2 development has been winding down for some time now so I'm not eager to see more non-trivial PRs targeting that version. This will not make it into 2.6.2 I'm afraid and there's no telling when 2.6.3 will be released, if ever.

There will likely be a SFML 2.6.3 given that it's the last SFML 2.x version, but not in the next few months. Cutting of SFML 2.6.2 was more done, so we're not releasing fixes for SFML 2 that aren't included in initial SFML 3 release. As such it would indeed be great if we could target more SFML 3.
Do you have any other DRM fixes lined up? Even if they're a dependency on this one, it might be great to open PRs for them, so we're at least aware of the work.

Would've been great if we had some better communication for this. I didn't get a reply on the other PR and SFML 2.6.2 has now already been tagged.

Hi I submitted https://github.com/SFML/SFML/pull/3136 back in June and in discussions on that thread I stated there was more to come that depended on that PR being merged (comments from July 27 to Chris).  3136 only got merged earlier today, 4 months later.  Sorry I thought that it was clear

The only other one I have at this point is an implementation of mouse support and all the sf::Cursor functionality. Right now its on 2.6.x

On Nov 9, 2024, at 4:34 PM, Lukas Dürrenberger @.***> wrote:

There will likely be a SFML 2.6.3 given that it's the last SFML 2.x version, but not in the next few months. Cutting of SFML 2.6.2 was more done, so we're not releasing fixes for SFML 2 that aren't included in initial SFML 3 release. As such it would indeed be great if we could target more SFML 3.
Do you have any other DRM fixes lined up? Even if they're a dependency on this one, it might be great to open PRs for them, so we're at least aware of the work.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

It's all connected Chris and I have spent most of our free time to get SFML 3 to the finish line, so SFML 2 things simply get a low prio.
I do appreciate the fixes and updates for the DRM implemention, especially since we don't really have anyone else with that knowledge! 🙌
Can you open a PR for the cursor implementation as well?

yes, for sure

On Nov 9, 2024, at 4:54 PM, Lukas Dürrenberger @.***> wrote:

It's all connected Chris and I have spent most of our free time to get SFML 3 to the finish line, so SFML 2 things simply get a low prio.
I do appreciate the fixes and updates for the DRM implemention, especially since we don't really have anyone else with that knowledge! 🙌
Can you open a PR for the cursor implementation as well?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

I'd like to see https://github.com/SFML/SFML/pull/3136 get ported to v3 before we go through with more DRM PRs targeting v2. Are you okay with that?


Update config.php

So there have been no big issues in 12. How about we extend the roll-out to 50%...
Or first 30 or so?

Makes total sense. Tnx!


FAQ link is not working correctly

Trello card

If i should add smooth animation for scroll, please tell me )

Perhaps good to note, you can still click the button twice, and it'll do something different both times, but that's in my opinion forgivable, if we don't like it I suggest we remove animating it in.


make code much more minifier friendly

cuts our bundle size down by ~0.2kb ish
we now beat simian!!!!!!!!!! (for me sad and epic at same time)

update: currently investigating BIG errors while using this in Discord, do not merge yet

update 2: unreproducible on a second try, if anyone else can repro please say, else i've tried to repro again ~3 times, all unsuccessfully.

I have no idea what caused this but I suspect it has something to do with the Fucklytics plugin doing weird things to the dispatcher handlers.
If nobody can repro I consider this a false alarm.

Fucklytics happens to not use the patcher to my knowledge. This can be merged.


How to change RGB stream to Grayscale?

I am a freshman of realsense sdk 2.0 and I am learning it from the example capture. I need to change the output from RGB to Grayscale . I think it might have something to do with the class colorizer, but I don't know how to set about. Can you please give me some advice?
Below are the codes of the example:
#include <librealsense2/rs.hpp> // Include RealSense Cross Platform API
#include "example.hpp"          // Include short list of convenience functions for rendering
// Capture Example demonstrates how to
// capture depth and color video streams and render them to the screen
int main(int argc, char * argv[]) try
{
// Create a simple OpenGL window for rendering:
window app(1280, 720, "RealSense Capture Example");
// Declare two textures on the GPU, one for color and one for depth
texture depth_image, color_image;
// Declare depth colorizer for pretty visualization of depth data
rs2::colorizer color_map;

// Declare RealSense pipeline, encapsulating the actual device and sensors
rs2::pipeline pipe;
// Start streaming with default recommended configuration
pipe.start();

while(app) // Application still alive?
{
    rs2::frameset data = pipe.wait_for_frames(); // Wait for next set of frames from the camera

   // rs2::frame depth = color_map(data.get_depth_frame()); // Find and colorize the depth data
	rs2::frame depth = color_map(data.get_depth_frame());
    rs2::frame color = data.get_color_frame();            // Find the color data

    // Render depth on to the first half of the screen and color on to the second
    depth_image.render(depth, { 0,               0, app.width() / 2, app.height() });
    color_image.render(color, { app.width() / 2, 0, app.width() / 2, app.height() });
}

return EXIT_SUCCESS;

}
catch (const rs2::error & e)
{
std::cerr << "RealSense error calling " << e.get_failed_function() << "(" << e.get_failed_args() << "):\n    " << e.what() << std::endl;
return EXIT_FAILURE;
}
catch (const std::exception& e)
{
std::cerr << e.what() << std::endl;
return EXIT_FAILURE;
}

Hi @xzyxzy29
Could you clarify if you are referring to the output of the depth or the color sensor?

@dorodnic

[Realsense Customer Engineering Team Comment]
The RGB sensor also support Y16 format, this is a grayscale, as below screenshot you can see that.
You can set the image format to Y16 as follows.
// Declare RealSense pipeline, encapsulating the actual device and sensors
rs2::pipeline pipe;
rs2::config c;
c.enable_stream(RS2_STREAM_COLOR, 640, 480, RS2_FORMAT_Y16, 30);
pipe.start(c);
Of course, you also need to modify the rendering method in the sample you mentioned to display the grayscale image.

[Realsense Customer Engineering Team Comment]
@xzyxzy29 Do you still have any further question about this ticket?


Failed to load module: No module named google.protobuf.internal

Hi I'm getting a Failed to load module: No module named google.protobuf.internal error when trying to load the droneapi module in mavproxy.
I did have the droneapi working but because I installed as root it put the .mavinit.scr file in /home/root. I since set up a user account. I then uninstalled the droneapi package then reinstalled from px4 login with sudo. Now the droneapi isn't working and giving the above mentioned error.
Obviously the uninstall / install has messed it up somehow but can't see where. The output at the bottom shows there is multiple instances of the api.py file. Maybe the path to the correct location is missing?
Any ideas how to fix this?
Below is the traceback when trying to load droneapi in mavproxy.
GUIDED> Traceback (most recent call last):
  File "/usr/local/bin/mavproxy.py", line 260, in load_module
    m = import_package(modpath)
  File "/usr/local/bin/mavproxy.py", line 390, in import_package
    mod = __import__(name)
ImportError: No module named mavproxy_droneapi.module.api

Traceback (most recent call last):
  File "/usr/local/bin/mavproxy.py", line 260, in load_module
    m = import_package(modpath)
  File "/usr/local/bin/mavproxy.py", line 390, in import_package
    mod = __import__(name)
  File "/usr/local/lib/python2.7/dist-packages/droneapi/module/api.py", line 8,                in <module>
    from droneapi.lib.WebClient import *
  File "/usr/local/lib/python2.7/dist-packages/droneapi/lib/WebClient.py", line                1, in <module>
    from webapi_pb2 import *
  File "/usr/local/lib/python2.7/dist-packages/droneapi/lib/webapi_pb2.py", line                4, in <module>
    from google.protobuf.internal import enum_type_wrapper
ImportError: No module named google.protobuf.internal

Failed to load module: No module named google.protobuf.internal. Use 'set moddebug 3' in the MAVProxy console to enable traceback
```

Below is output to show where the api.py file is located, in many locations:
```
px4@ubilinux:~$ sudo find / -iname *api.py*
/usr/share/pyshared/numpy/core/generate_numpy_api.py
/usr/share/pyshared/numpy/core/tests/test_api.py
/usr/lib/pymodules/python2.6/numpy/core/generate_numpy_api.py
/usr/lib/pymodules/python2.6/numpy/core/generate_numpy_api.pyc
/usr/lib/pymodules/python2.6/numpy/core/tests/test_api.py
/usr/lib/pymodules/python2.6/numpy/core/tests/test_api.pyc
/usr/lib/pymodules/python2.7/numpy/core/generate_numpy_api.py
/usr/lib/pymodules/python2.7/numpy/core/generate_numpy_api.pyc
/usr/lib/pymodules/python2.7/numpy/core/tests/test_api.py
/usr/lib/pymodules/python2.7/numpy/core/tests/test_api.pyc
/usr/local/lib/python2.7/dist-packages/requests-2.7.0-py2.7.egg/requests/api.pyc
/usr/local/lib/python2.7/dist-packages/requests-2.7.0-py2.7.egg/requests/api.py
/usr/local/lib/python2.7/dist-packages/pymavlink/generator/lib/genxmlif/xmlifApi.py
/usr/local/lib/python2.7/dist-packages/pymavlink/generator/lib/genxmlif/xmlifApi.pyc
/usr/local/lib/python2.7/dist-packages/droneapi/module/api.pyc
/usr/local/lib/python2.7/dist-packages/droneapi/module/api.py
/root/droneapi-python/droneapi/module/api.py
/root/build/pymavlink/build/lib.linux-i686-2.7/pymavlink/generator/lib/genxmlif/xmlifApi.py
/root/build/pymavlink/generator/lib/genxmlif/xmlifApi.py
/root/build/droneapi/build/lib.linux-i686-2.7/droneapi/module/api.py
/root/build/droneapi/droneapi/module/api.py
/home/px4/build/droneapi/droneapi/module/api.py
/home/px4/build/droneapi/build/lib.linux-i686-2.7/droneapi/module/api.py
```

I am also having the same problem. I attempted to manually install python-protobuf, but I still get an error that it is missing enum_type_wrapper when loading mavproxy.

I found the fix.
Mine was failing during the pip install phase because it was not finding "../src/google/protobuf/unittest.proto"
The protobuff git put out a fix 9 days a go that involved adding an if statement in the setup.py file to ignore the unittest section if you are not working on dev. Worked like a charm!
https://github.com/google/protobuf/commit/e4f4d9fe83887b08b0db67512947645324826755

@mrpollo - assigned to you - something to fix in our installation?

@codywburns thanks for the report, can you create a different issue for your problem please? even if its solved already

no problem

@ggregory8 did you uninstall both droneapi and mavproxy?, also when you installed them again under your user px4 did you re/install both using sudo?, do you know the contents of $PYTHONPATH?
echo $PYTHONPATH

@mrpollo Sorry about delay, have been away. Yes I had uninstalled both droneapi and mavproxy and reinstalled using sudo. Just tried again with same results.
I noticed during pip install that I am getting same error as @codywburns. So I followed his advice as per issue #125 and it is now working.
Also echo $PYTHONPATH hasn't showed anything, even before I was having this issue. Is there another way to check this?

This should be fixed on windows at least by #228

This will no longer be an issue since https://github.com/dronekit/dronekit-python/pull/341.


Returning a Dictionary From a Metric Class

In the KerasNLP repository, we are currently working on implementing the ROUGE metric. It inherits from keras.metrics.Metric. However, we came across this issue:
>>> rg(y_true, y_pred)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/abheesht/python_envs/keras_nlp/lib/python3.8/site-packages/keras/metrics/base_metric.py", line 200, in __call__
    return distributed_training_utils.call_replica_local_fn(
  File "/home/abheesht/python_envs/keras_nlp/lib/python3.8/site-packages/keras/distribute/distributed_training_utils.py", line 60, in call_replica_local_fn
    return fn(*args, **kwargs)
  File "/home/abheesht/python_envs/keras_nlp/lib/python3.8/site-packages/keras/metrics/base_metric.py", line 196, in replica_local_fn
    result_t._metric_obj = self  # pylint: disable=protected-access
AttributeError: 'dict' object has no attribute '_metric_obj'

Note that this issue shows up only when we call the object directly, and not when we run it like this:
>>> rg.update_state(y_true, y_pred)
>>> rg.result()
{'precision': <tf.Tensor: shape=(), dtype=float32, numpy=0.4>, 'recall': <tf.Tensor: shape=(), dtype=float32, numpy=0.2>, 'f1_score': <tf.Tensor: shape=(), dtype=float32, numpy=0.26666668>}

This gives rise to a couple of questions.

Does Keras support dictionary return type for metrics? If not, we can close this issue immediately.
If it does support dictionary return type, this is most probably a bug. In particular, this line is the cause of error: https://github.com/keras-team/keras/blob/master/keras/metrics/base_metric.py#L206.

I am of the opinion that there wasn't any intention to support dictionary return type for metrics because this comment mentions that result_t is of type tensor: https://github.com/keras-team/keras/blob/master/keras/metrics/base_metric.py#L198. However, it would be really helpful for us KerasNLP folks if we could have support for dictionary (we want to return precision, recall, F1-score for ROUGE metric). The other solution could be to return a tensor of shape (3,) containing the precision, recall, F1-score values (but this isn't very elegant).
CC: @mattdangerw, @chenmoneygithub, @fchollet
EDIT: Looks like returning a dictionary of scalars is supported according to the doc-string: https://github.com/keras-team/keras/blob/master/keras/metrics/base_metric.py#L88-L89. So, this is most likely a bug.
This is the result() function I wrote:
def result(self):
        if self._number_of_samples == 0:
            return {"precision": 0.0, "recall": 0.0, "f1_score": 0.0}

        rouge_precision = self._rouge_precision / self._number_of_samples
        rouge_recall = self._rouge_recall / self._number_of_samples
        rouge_f1_score = self._rouge_f1_score / self._number_of_samples
        return {"precision": rouge_precision, "recall": rouge_recall, "f1_score": rouge_f1_score}

@abheesht17,
To expedite the trouble-shooting process, could you please provide the complete code to reproduce the issue reported here. Thank you!

@tilakrayal, here is the notebook where I've reproduced the error: https://colab.research.google.com/drive/1dDYLLgjiSZHEHEzC5t0zHIh0ljfYV9P7?usp=sharing. Thanks!

@gadagashwini,
I was able to reproduce the issue in tensorflow v2.8, v2.9 and nightly. Kindly find the gist of it here.

@abheesht17,
I tried to execute the code on the latest tf-nightly(2.13.0-dev20230215)  and Keras(2.13.0)
and it was executed without any issue/error. Could you please check the gist of it here and let us know if it serves the purpose. Also please take a look at the base_metric.py file where the MetricDict class was updated. Thank you!
https://github.com/keras-team/keras/blob/master/keras/metrics/base_metric.py#L206
class _MetricDict(dict):
    """Wrapper for returned dictionary of metrics."""

    def __init__(self, **kwargs):
        super().__init__(**kwargs)
        self._metric_obj = None


Error "Wifi could not be disabled" on command wifi-wand off... while the command is working!

When running the command wifi-wand off, the wifi is properly turned off.
However, an error is raised:
Error: /Users/me/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/wifi-wand-2.15.0/lib/wifi-wand/models/mac_os_model.rb:209:in `wifi_off'
/Users/me/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/wifi-wand-2.15.0/lib/wifi-wand/command_line_interface.rb:377:in `cmd_of'
/Users/me/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/wifi-wand-2.15.0/lib/wifi-wand/command_line_interface.rb:479:in `block in commands'
/Users/me/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/wifi-wand-2.15.0/lib/wifi-wand/command_line_interface.rb:207:in `attempt_command_action'
/Users/me/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/wifi-wand-2.15.0/lib/wifi-wand/command_line_interface.rb:232:in `process_command_line'
/Users/me/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/wifi-wand-2.15.0/lib/wifi-wand/command_line_interface.rb:519:in `call'
/Users/me/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/wifi-wand-2.15.0/lib/wifi-wand/main.rb:65:in `call'
/Users/me/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/wifi-wand-2.15.0/exe/wifi-wand:5:in `<top (required)>'
/Users/me/.rbenv/versions/3.3.0/bin/wifi-wand:25:in `load'
/Users/me/.rbenv/versions/3.3.0/bin/wifi-wand:25:in `<main>'

Wifi could not be disabled.

To reproduce
Just run:
$ wifi-wand off

Configuration

macOS Sonoma 14.4.1 (23E224)
ruby 3.3.0 (2023-12-25 revision 5124f9ac75) [x86_64-darwin23]
wifi-wand (2.15.0)

Thank you for this great gem! 🙌

This was due to the wifi_on? method not working properly due to the airport deprecation.
I have fixed that method to use networksetup instead, and that fixes the problem.
Please gem install wifi-wand and get version 2.16.0, which fixes this and some other issues.


Paired device list disappears on Bluetooth Scanning UI sometime

When you start ANT Manager, you will face Bluetooth Scanning & Pairing UI.
If there are many candidate Bluetooth devices, paired device view is collapsed and not seen.
I have to enhance this UI's visibility.

I've resolved this issue in PR #125. So I close this issue.


Unable to restore nuget packages

Part 3 of step 3 has you running a nuget restore, this does not seem to work in VS2022, I get this error:

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: a8027df4-5e60-69aa-1c0a-249053cb6e72
Version Independent ID: c4ca0cff-b1b2-3690-00b9-af6c81d838b9
Content: Quickstart: Sign in users in web apps using the auth code flow - Microsoft Entra
Content Source: articles/active-directory/develop/web-app-quickstart.md
Service: active-directory
Sub-service: develop
GitHub Login: @Dickson-Mwendia
Microsoft Alias: dmwendia

@catmanjan
Thanks for your feedback! We will investigate and update as appropriate.

@catmanjan
For you information Please refer the below documented link
https://learn.microsoft.com/en-us/nuget/consume-packages/package-restore#restore-packages

@ManoharLakkoju-MSFT  I get the error when I attempt to restore packages

@catmanjan
Please Note, GitHub forum is dedicated for docs related issues. For any technical queries or clarifications, we encourage to utilise Microsoft Q & A platform. Kindly raise your query on Microsoft Q&A Platform

@ManoharLakkoju-MSFT how is this not a docs related issue?

@catmanjan
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly
@Dickson-Mwendia
Please review this.

@catmanjan are you still experiencing this issue on VS 2022? I've followed the guidance in this quickstart and was able to restore nugget packages successfully.

From your screenshot, I see the error occurred while finding part of the path. A suggestion that might help fix the problem and avoid other errors caused by path length limitations is extracting the archive or cloning the project repository into a directory near the root of your drive.
Let me know whether this helps. If not, I will re-open the issue and we could attempt to reproduce the error you're experiencing.
#please-close


Error on running Photon-WebUI

I followed your guide on running this sample.

Install dependencies via npm install.

Run the sample via npm run dev.

But it fails with the following error.
Error: Cannot find module '\Photon\Photon-WebUI\build\webpack.dev.conf.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:580:15)
    at Function.Module._load (internal/modules/cjs/loader.js:506:25)
    at Module.require (internal/modules/cjs/loader.js:636:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at WEBPACK_OPTIONS \Photon-WebUI\node_modules\webpack-cli\bin\convert-argv.js:133:13)
    at requireConfig (\Photon-WebUI\node_modules\webpack-cli\bin\convert-argv.js:135:6)
    at \\Photon-WebUI\node_modules\webpack-cli\bin\convert-argv.js:142:17
    at Array.forEach (<anonymous>)
    at module.exports (\Photon-WebUI\node_modules\webpack-cli\bin\convert-argv.js:140:15)
    at Object.<anonymous> (\Photon-WebUI\node_modules\webpack-dev-server\bin\webpack-dev-server.js:92:55)

I checked the configuration, there is no build folder in this repo. And your configuration .eslintignore  file specified that  /build/ is ignored.

So my question is how to fix this issue?

I migrated the project to use a template generated using Vue-CLI 3. The issue has been addressed.


Remove AngularCookieLocaleResolver

remove AngularCookieLocaleResolver due to being a leftover from angularjs, a companion of https://github.com/jhipster/generator-jhipster/pull/23818
Fix https://github.com/jhipster/generator-jhipster/issues/20106

I suppose those checks should start passing after https://github.com/jhipster/generator-jhipster/pull/23818 gets merged?

@shrralis Yes
The pipeline target main brach and not your branch, this is why it is failing here...
But do not worry, this will be merged only if on the generator side everything is working and validated

Thanks @shrralis for the contribution


Can not sell pending at Confirmation button

Customer: Every time I try to sell I get this screen and it just stays blank no matter how long I sit there. Anyone else having that issue?![image](https://user-images.githubusercontent.com/117429099/230800013-e91a025d-7042-4385-b10d-65766f2a0750.jpeg]

Look like Nova wallet Issue.

so cant do anything? @langdon0003


Prod release v0.2.0

This is a bit long and messy. I am running a full test here: /lustre/scratch123/tol/teams/tolit/users/ps22/pipelines/blobtoolkit. Will update once it completes, but please start reviewing if you can. Would be great to get it merged this week. Thanks :)

BUSCO in test and test_raw profile runs but it fails for test_full. Here is the error:
I tried with multiple busco input folder values.
Command error:
  Traceback (most recent call last):
    File "/usr/local/bin/busco", line 42, in <module>
      run_BUSCO.main()
    File "/usr/local/lib/python3.7/site-packages/busco/run_BUSCO.py", line 420, in main
      busco_run.run()
    File "/usr/local/lib/python3.7/site-packages/busco/run_BUSCO.py", line 70, in run
      self.load_config()
    File "/usr/local/lib/python3.7/site-packages/busco/run_BUSCO.py", line 62, in load_config
      self.config_manager.load_busco_config_main()
    File "/usr/local/lib/python3.7/site-packages/busco/BuscoLogger.py", line 62, in wrapped_func
      self.retval = func(*args, **kwargs)
    File "/usr/local/lib/python3.7/site-packages/busco/ConfigManager.py", line 58, in load_busco_config_main
      self.config_main.validate()
    File "/usr/local/lib/python3.7/site-packages/busco/BuscoConfig.py", line 550, in validate
      self._init_downloader()
    File "/usr/local/lib/python3.7/site-packages/busco/BuscoConfig.py", line 392, in _init_downloader
      self.downloader = BuscoDownloadManager(self)
    File "/usr/local/lib/python3.7/site-packages/busco/BuscoDownloadManager.py", line 53, in __init__
      self._obtain_versions_file()
    File "/usr/local/lib/python3.7/site-packages/busco/BuscoLogger.py", line 62, in wrapped_func
      self.retval = func(*args, **kwargs)
    File "/usr/local/lib/python3.7/site-packages/busco/BuscoDownloadManager.py", line 75, in _obtain_versions_file
      urllib.request.urlretrieve(remote_filepath, local_filepath)
    File "/usr/local/lib/python3.7/urllib/request.py", line 257, in urlretrieve
      tfp = open(filename, 'wb')
  PermissionError: [Errno 13] Permission denied: '/tmp/nxf.mWMrsqoKa2/lineages/file_versions.tsv'

Work dir:
  /lustre/scratch123/tol/teams/tolit/users/ps22/pipelines/blobtoolkit/work/a5/0dd84f642656f5b7ffc5655393c25c

@priyanka-surana : I just remembered that the Busco module lost its mandatory --offline: https://github.com/nf-core/modules/pull/4360
Did you update the module recently ? If so, you need to add --offline in your conf/modules.config. It didn't cause any issue in the genome-note pipeline yet because its Busco module hasn't been udpated.

@priyanka-surana : I just remembered that the Busco module lost its mandatory --offline: nf-core/modules#4360
Did you update the module recently ? If so, you need to add --offline in your conf/modules.config. It didn't cause any issue in the genome-note pipeline yet because its Busco module hasn't been udpated.

I added --offline only if the lineages path is provided, like this:
path busco_lineages_path              // Recommended: path to busco lineages - downloads if not set
...
def busco_lineage_dir = busco_lineages_path ? "--download_path ${busco_lineages_path} --offline" : ''

This gives a new error:
Command output:
  New AUGUSTUS_CONFIG_PATH=/tmp/nxf.5PnCMeZR0Z/tmp.3lEJgFGY09
  2023-12-19 15:36:43 INFO:	***** Start a BUSCO v5.5.0 analysis, current time: 12/19/2023 15:36:43 *****
  2023-12-19 15:36:43 INFO:	Configuring BUSCO with local environment
  2023-12-19 15:36:43 INFO:	Mode is genome
  2023-12-19 15:36:43 INFO:	Running in batch mode. 1 input files found in /tmp/nxf.5PnCMeZR0Z/input_seqs
  2023-12-19 15:36:43 INFO:	Input file is /tmp/nxf.5PnCMeZR0Z/input_seqs/GCA_927399515.1.fasta
  Short summaries were not available: No genes were found.

Command error:
  2023-12-19 15:36:43 ERROR:	Unable to run BUSCO in offline mode. Dataset /tmp/nxf.5PnCMeZR0Z/lineages/lineages/eukaryota_odb10 does not exist.
  mv: cannot stat 'GCA_927399515.1-eukaryota_odb10-busco/*/short_summary.*.json': No such file or directory
  mv: cannot stat 'GCA_927399515.1-eukaryota_odb10-busco/*/short_summary.*.txt': No such file or directory

Work dir:
  /lustre/scratch123/tol/teams/tolit/users/ps22/pipelines/blobtoolkit/work/e2/dd5a4ba76c8b173b9bed14921de01a

Removing the busco_lineages_path makes the pipeline work just fine. I think the issue must be with that dataset or how we are passing it. This is the version I am testing: /lustre/scratch123/tol/teams/tolit/users/ps22/pipelines/blobtoolkit (branch blast). Thanks :)

The whole BUSCO issue is sorted by removing the backslash / from the end of the location. This is still weird to me.
But now there is a new error, which should be more straight forward. The first line of one of the DIAMOND database is empty. I guess I need to recreate them. Unless I am missing something.
Database: ./gfLaeSulp1.1.buscoregions (type: Diamond database, sequences: 246, letters: 338784)
  Block size =<PHONE_NUMBER>
  Current RSS: 20.4 MB, Peak RSS: 20.4 MB
  Opening the input file... Error: Error detecting input file format. First line seems to be blank.
  Error: Error detecting input file format. First line seems to be blank.

But now there is a new error, which should be more straight forward. The first line of one of the DIAMOND database is empty. I guess I need to recreate them. Unless I am missing something.

The gfLaeSulp1.1*dmnd files are identical to the ones Rich has in https://github.com/blobtoolkit/blobtoolkit/tree/main/databases/uniprot
I would rather think that "input file" refers to the Fasta file. Can you check that the Fasta file has sequences in it ?

I would rather think that "input file" refers to the Fasta file. Can you check that the Fasta file has sequences in it ?

Yes right again. The chunks file is empty. The input fasta is created with this command:
btk pipeline chunk-fasta \
    --in GCA_927399515.1.fasta \
    --busco full_table.tsv \
    --out GCA_927399515.1.chunks.fasta \
    --chunk 100000 --overlap 0 --max-chunks 10 --min-length 1000

I am looking more into this now.

Yes right again. The chunks file is empty. The input fasta is created with this command:

Rich is also looking into this. For both of us, local tests also produce blank files which Rich does not think should happen.
Please let me know if any other changes not related to this step are needed. We will work on sorting the errors in parallel to other changes.

The error was:

OK, there were 2 problems. BUSCO 5.5 changed the full_table format a little so the parsing wasn't working and then I found a bug in a check for runs of unmasked bases that was checking for masked instead. I've just pushed the changes so a fixed version 4.3.1 should be building now

It is resolved now, the blast steps have all completed successfully.

There is this warningWARN: There's no process matching config selector: BLOBTOOLKIT_DEPTH -- Did you mean: BLOBTOOLKIT_CHUNK?

modules/local/blastn.nf (BLASTN) seems not used ?

The full test is complete successfully. You can view the results here: http://grit-btk.tol-dev.sanger.ac.uk/view/btk_prod_test_full_mosdepth/dataset/btk_prod_test_full_mosdepth/blob
I am only concerned about this part in the results (meta.json):
"settings": {
"blast_chunk":100000,
"blast_max_chunks":10,
"blast_min_length":1000,
"blast_overlap":0,
"pipeline":"https://github.com/blobtoolkit/blobtoolkit","release":"blobtoolkit-pipeline v4.1.6",
"software_versions": {
"blastn":"2.12.0+",
"blobtk":"0.2.4",
"blobtools":"4.3.1",
"busco":"5.3.2",
"diamond":"2.0.15",
"minimap2":"2.24-r1122",
"python":"3.9.13",
"samtools":"1.15.1",
"seqtk":"1.3-r106",
"snakemake":"7.19.1"},
"stats_chunk":1000,
"stats_windows":[0.1,0.01,100000,1000000],
"taxdump":"./taxdump","tmp":"/tmp"},
"similarity": {
"diamond_blastp":{
"evalue":1e-10,
"import_evalue":1e25,
"import_max_target_seqs":100000,
"max_target_seqs":10,
"name":"reference_proteomes",
"path":"./uniprot",
"taxrule":"blastp=buscogenes"},
"diamond_blastx": {
"evalue":1e-10,
"import_evalue":1e-25,
"max_target_seqs":10,
"name":"reference_proteomes",
"path":"./uniprot",
"taxrule":"buscogenes"}}

Is this going to cause issues for production?

@gq1 Can you please take a look at the tests for this pipeline? Thanks.

@gq1 Can you please take a look at the tests for this pipeline? Thanks.

Can you run the test locally?
Not much help from the log, run out of disk space?
2023-12-20T14:34:17.7624742Z   touch: cannot touch '.command.trace': Permission denied
2023-12-20T14:34:17.7625306Z
2023-12-20T14:34:17.7625598Z Work dir:
2023-12-20T14:34:17.7626480Z   /home/runner/work/blobtoolkit/blobtoolkit/work/d9/ce32b60e9dba5d624bc89e3bcf10c8

I looked at /lustre/scratch123/tol/teams/tolit/users/ps22/pipelines/blobtoolkit/results/GCA_927399515.1 and browsed the data on the BTK viewer.
These seem fine:

GC%
Read coverage
Lengths
Gaps (Ns)
"position" and "proportion" files
Summary files GCA*.summary.json (incl. the BUSCO score), identifiers.json, meta.json

These seem weird:

${lineage}_odb10_count*.json only have 0s for the related eukaryote lineages (fungi and below) and non-0s for bacteria/archaea ! Maybe there's some filtering happening.
I also don't know / understand what's supposed to be in the buscogenes* and buscoregions* JSON files, so can't really tell whether the counts are correct

I looked at /lustre/scratch123/tol/teams/tolit/users/ps22/pipelines/blobtoolkit/results/GCA_927399515.1 and browsed the data on the BTK viewer.
These seem fine:

GC%
Read coverage
Lengths
Gaps (Ns)
"position" and "proportion" files
Summary files GCA*.summary.json (incl. the BUSCO score), identifiers.json, meta.json

These seem weird:

${lineage}_odb10_count*.json only have 0s for the related eukaryote lineages (fungi and below) and non-0s for bacteria/archaea ! Maybe there's some filtering happening.
I also don't know / understand what's supposed to be in the buscogenes* and buscoregions* JSON files, so can't really tell whether the counts are correct

This is fixed.


C++ indexer: index base classes. [phab:D144]

Created by zarko at 2015-03-21 00:22:23:
This diff emits extends/ edges for C++.
There are a few TODOs remaining about
linking to particular template instantiations
instead of tapp specifications of these
(and one about #312). This will be better
to deal with in a separate CL, as I think we'll
want to do it consistently inside
BuildNodeIdForType.

jdennett commented inline (context lost during migration) at 2015-03-23 17:47:18:
Might it be helpful to also say (possibly parenthetically) "(The derived class.)" for this, and "(The base.") below?

jdennett commented inline (context lost during migration) at 2015-03-23 17:47:18:
The presence of AS suggests that this is for C++-based languages only, in which case we can assume that bases are also record types in the documentation above.

jdennett commented inline (context lost during migration) at 2015-03-23 17:47:18:
We should be careful to indicate that this is a "direct inheritance" relationship, not the transitive closure thereof.

jdennett commented inline (context lost during migration) at 2015-03-23 17:47:18:
Nit: the order public, private, protected looks strange to my eyes.  Public, protected, private would be less jarring.

zarko wrote at 2015-03-23 18:08:44:

Address comments.

jdennett accepted this revision at 2015-03-23 21:57:27

zarko closed via commit 0c6617e8fceb88812f45e3d47edefd86ad7faa7f at 2015-03-23 21:58:42


[optimization] child node reordering

So @developit pointed out another optimization we could do is check node reordering. Often times lists get updated; items get appended or removed. Given that lists, forms and links are the staple elements of HTML this is probably worth optimizing for.
The new node comparison algorithm would then become:

check if node is same, if yes do nothing
compare the full array of child elements with equality checks, apply the re-order algorithm
insert new children where reordering didn’t apply

I reckon this should speed up the reordering case by quite a bunch. Help welcome! :sparkles:

It seems like adding an ID check could also be a cheap optimization.

That's what keys do in react, some prior art.

Is there a way we can do children node reordering without having to force the user to supply a identifying key for every element?
So far every implementation I have seen has had this.
Problem is when the children look alike but children's-children contains the content. Apart from creating a Merkle tree or having the user supply a key I cannot see any other way of doing it.
I would love to help with this. ✨

@reminyborg well, each DOM element is just an object. If we set a property on the object we could use it as a unique identifier for the child nodes; removing the need for stuff to happen in userland. I think at least; hmmmm

@yoshuawuyts i might be wrong, in the cases of reordering with only the children's-children changing we might still end up mutating the elements instead of moving them. The effects are probably most apparent when dealing with focus.
A possible solution would be a optional "key" property. It would simplify reordering and keeping DOM state like focus intact. And if "key" is not used we can solve it optimistically.
I remember being discussed in mithril: http://mithril.js.org/mithril.html#dealing-with-focus

+1 on ID check
Also Merkel Trees is such and interesting idea we should def add an experimental branch that tries this out.

Hehe, I think for all intents and purposes cache-element / nanocomponent
achieves what I initially proposed for merkle trees ✨
On Fri, Jan 20, 2017, 16:41 kj<EMAIL_ADDRESS>wrote:

+1 on ID check
Also Merkel Trees is such and interesting idea we should def add an
experimental branch that tries this out.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yoshuawuyts/nanomorph/issues/8#issuecomment-274102927,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWleqiyrhs6Dc5Mu5ZybNlK8EU9OTJCks5rUNWsgaJpZM4JfAfi
.

I was thinking about what to name this prop, and key seems like it's the most used:

react has

vue has

inferno has

Also from https://github.com/infernojs/inferno/issues/406:

Unlike React, Inferno promotes correct usage of instances where you might want, and not want, keys. Using either in different situations can have different performance characteristics depending on what is being "patched". Inferno could warn for not using keys on lists that contain stateful DOM elements that can have big issues if not keyed (e.g. inputs, selects, iframes, video, audio etc) – but that's not in scope for now. It might be a good task to add to the issue backlog though.

This made me realize that if we want to prevent onload / unonload from going haywire in lists we need to implement reordering for sure.

blep, ok I think we might need to rename it to

; was just thinking about it and figured using the data- prefix might actually be a bit nicer (:

I was experimenting with the thought that using the element ID would be enough.
I was reading morphdoms reasonings about it in patrick-steele-idem/morphdom#3
It would require users to create document unique ID's when needing list performance or preserving DOM state. But it would also ensure that the 'children' could survive being moved.

@reminyborg ah this explains a lot - I've run into issues before where I set an id on an element and it would stop updating. Didn't know why, just considered it a morphdom gotcha.
I'm not necessarily a fan of using IDs as keys, because IDs meant to be unique per page which means there could be conflicts. Using a data- attributed on the other hand feels less intrusive imo and only serves as a hint to the algorithm on how top optimize stuff

actually IDs might make sense - if we document it well tho :sparkles:

I've found in working on an infinite list module (https://github.com/editdata/infinite-elements) that having an id or key for reordering is essential for making it work.
Using the id by default and providing the option of using an alternate key attribute might be good. morphdom has a getNodeKey option to accomplish this.

Yeah fair point about the config, though I'd like to try a no-config
version first - the more consistent user land becomes the better it is I
think. I think using IDs for reordering shouldn't give us a lot of trouble
On Sun, Mar 12, 2017, 22:05 Seth Vincent<EMAIL_ADDRESS>wrote:

I've found in working on an infinite list module (
https://github.com/editdata/infinite-elements) that having an id or key
for reordering is essential for making it work.
Using the id by default and providing the option of using an alternate key
attribute might be good. morphdom has a getNodeKey option to accomplish
this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yoshuawuyts/nanomorph/issues/8#issuecomment-285976502,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWlejkxouWA32KzCC4CXGeFkvQDHSW-ks5rlF4MgaJpZM4JfAfi
.

From IRC:
17:04 <yoshuawuyts> how about: look over list of elements, reference each child by an id in an object, reference new children by id in object, morph nodes with same id, then loop over nodes and skip the ones that have already been morphed (e.g. do the reordering part)
17:05 <julian> yeah that sounds good, but doesn't solve the on-load problem, right?
17:06 <yoshuawuyts> it would because we never remove them from the dom
17:06 <julian> how do you reorder AB to BA without removing a node?
17:06 <yoshuawuyts> oh fuck lol
17:07 <julian> maybe on-load needs to be changed to allow this
17:07 <julian> yes i think so
17:07 <julian> it should after deletion check if "the same element" doesn't exist after all, maybe like that?
17:08 <yoshuawuyts> dang, reordering lists is tricky. appending / prepending nodes is the easier case
17:08 <julian> yeah definitely
17:09 <julian> deleting and adding in between works too
17:09 <julian> just reordering
17:09 <yoshuawuyts> luckily onload is a different concern - if we solve the algo in nanomorph then we can figure out how to best do stuff with onload on top of that
17:09 <julian> ok, cool!
17:09 <yoshuawuyts> I think at least haha
17:09 <julian> it "has to be that way"
17:09 <julian> not much we can do in nanomorph otherwise
17:10 <yoshuawuyts> yeahh
17:10 <julian> on-load should work with the concept of moving elements around
17:10 <julian> that should be a basic premise
17:10 <yoshuawuyts> we could so something like: listen for onload events -> nextTick -> check if currently in DOM -> if not in DOM fire the onunload handler
17:11 <yoshuawuyts> which would allow for detaching -> immediately reattaching DOM nodes
17:11 <julian> yeah i was thinking the same
17:11 <yoshuawuyts> woot!

Patch to fix this was merged, but current master shows a regression in certain cases:

This was from my CSSconf talk slides, when upgrading to new nanomorph things failed. Oooops.
https://github.com/yoshuawuyts/css-conf-talk/tree/master

Wondering how this could affect reordered components that return a proxy.

The idea is that it'd do away with artifacts accidentally being rendered (:
On Thu, May 18, 2017 at 10:45 PM Bret Comnes<EMAIL_ADDRESS>wrote:

Wondering how this could affect reordered components that return a proxy.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yoshuawuyts/nanomorph/issues/8#issuecomment-302536203,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWlejgJt8E3MIuTdyVzm3fIWqNF-9YRks5r7K36gaJpZM4JfAfi
.

Fantastic.  I suspected so.

https://github.com/shama/bel/pull/74 fixed the regression I encountered - haven't been able to reproduce. We should stay on the lookout for possible regressions tho

v5.0.0

This is going to be shipped in choo v6, requires #71 to land


fix(comment-lint): disabled body rules

Currently bee's comment linting rules are vary strict about git commit body. This is causing CI lint job to fail for commits made by dependency bot because it is adding vary descriptive body of git commits.
This PR:

relaxes rules of commit body
changes Get Commit Message task to log short commit format

Example of dependency bot PR (https://github.com/ethersphere/bee/pull/3827)

This change is

@mrekucci this PR just relaxes body of commits while keeping the same requirements for titles. Is there any specific need to restrict body of commit to only 72 characters? Wouldn't it be beneficial to have relaxed descriptions (bodies) of commits in order to better explain / summarise commits?


Very inconsistant spacing in MPL drawer

Information
This needs to be fixed asap as the unrolled circuits now are very wide.
Related to: #3489

Qiskit Terra version: master
Python version:
Operating system:

What is the current behavior?

Steps to reproduce the problem
What is the expected behavior?
Suggested solutions

I've got a fix for this and #3489 locally that I've just to clean up and push a PR, but after the fix the circuit here becomes:

Oh yeah. You the man!


Refactor Augmentation

I've created this PR to refactor the data augmentation component of our project.
To better understand the changes, please review the example located at templates/speech_recognition/ASR/ for an overview of how the updated code will look.
Here are the key improvements proposed:

Enhanced Configurability: In the current solution, all data augmentation functions (e.g., addnoise, addrev, dropfreq, speed change) and their respective hyperparameters are defined directly in the YAML configuration. In the previous version, these were nested within a separate layer, which may have made it less intuitive for users. However, this change enhances transparency for users and simplifies the process of integrating their custom augmenters into the augmentation pipeline.

Augmentation Combinator Class: I've introduced a new class that allows us to mix these augmenters in various ways. The philosophy behind this approach is similar to our separation of the scorer from the beam searcher, where we defined scorers and enabled users to pipeline them into the beam searcher.

Improved Dataset: In the previous version, the open_rir dataset was downloaded and generated behind the scenes. With this solution, the download location is now explicit, giving users greater control.

Enhanced open-rirs Dataset: I've created a new version of the open-rirs dataset and uploaded it to our Dropbox repository. In this new version, noise sequences and impulse responses have been separated. This change eliminates the need for users to download impulse responses when they only require noise sequences.

Transparent Data Preparation: In the earlier version, the data preparation for the noisy dataset occurred behind the scenes. With this update, it is performed in train.py, aligning it with the data preparation process for other training datasets.

Note:

As a result of these changes,tests are currently failing as I will need to convert all the recipes. Before proceeding with that, I would need your feedback.
Additional tests and ablations are required, such as comparing FastDropChunk vs. DropChunk, experimenting with larger bandwidth for frequency drop, varying the number of augmentations, and exploring shuffled augmentations.
The support for speed perturbation in the augmenter, with variable-length augmentation, is still intact.

Hey @mravanelli, I really like the new changes. This improvement and the refactoring of the fit_batch fn will help a lot!

Please make sure to update the PR with the latest commits in unstable-v0.6 (it includes a new recipe Tedlium2)

I did a bunch of tests and everything seems to work. Recipe tests are fine. TIMIT recipes achieve the expected performance. I tried LibriSpeech Transformer as well and the performance is the expected one too. I think we can merge it!


Added platform-bible-react preview page, revised readmes

Resolves #865

This change is

@rolfheij @jolierabideau could you give me any hints, while TJ is out?

You tagged my private Github account. Not a bad way to get my attention on my day off 🤣
We created this small preview page to make the development of components in platform-bible-react easier.
Before the process would be: Make changes to the code of your component, (re)build the entire platform-bible-react library, run Storybook or Platform.Bible from main. (Re)building the library and spinning up Platform.Bible would take up to two minutes where you had to sit and wait. This was very time consuming, especially when you made lots of small changes/tweaks (like adding/removing Tailwind CSS directives).
This small preview page allows you to see the changes in your components live. It kinda serves the same purpose as Storybook would, but since it's a fully functional React component, it also allows you to do some more advanced operations on it, like using (React) hooks and wrapping in divs or other components.


how long does it take to capture avi file?

Hello,
I was confused that data/rawframes&opticalflow_extraction.py takes so long (>15min, my gpu is 3060, ram is 14gb) and it just stay here everytime running it.

BTW, could you open a discussion in the repo?
Thank you very much for your help!

Hi gr816ox!
The script takes a long time, especially for Optical Flow generation, a time that increases a lot if OpenCV Cuda-Enabled is not used.
As far as I remember (3070, 32gb ram) it took a whole day to generate everything, setting multiprocessing to 6 physical processors.
In case you can't generate them let me know, I can pass them directly to you!

In the meantime, I took the opportunity to open a discussion in the repo!

Hi giocoal!
THX! I'm running the code on a cloud server, and it's very complicated to debug :(.
Could you pass the data with google drive?
It's very kind of you to save me from the terrible debugging XD.

I uploaded a .zip file containing the RGB frames and Optical Flow to Google Drive. The download link and instructions for using them can be found in the README of the repository.
I hope I have been of some help, I would be very pleased if you star ⭐ my repo if you feel like it :)

Sure! This is an very useful repo to me. Thanks again for your help!