Adding tcp_port and udp_port in ares_addr_node and process with that.

This improvement allows you to specify different ports per server.

Can this be merged, please?

This change isn't done in an ABI compatible way. Older applications will use a smaller struct so we cannot just unconditionally write to the new members of the struct.

@bagder any suggestion on how I could write this in an ABI compatible way? That's because the way it is, the lib doesn't work following your docs :confused:
Just as reference, I'm using this change on this lib: https://github.com/azukiapp/libnss-resolver

@bagder any suggestion on how I could write this in an ABI compatible way?

By introducing a new struct that is used by a new function instead of expanding/changing an existing one.

That's because the way it is, the lib doesn't work following your docs

The lib does work for quite a lot of users so I guess you're talking about some specifics? If there are bugs in the docs we should work on fixing those too.

One function talking about ports is ares_set_servers_csv(), which claim that port is ignored.  But as far as my testing goes, setting the port causes it to become 1, not 53.  I would like to be able to use c-ares with Tor listening for DNS requests on a port other than 53, and this do not work currently.

@petterreinholdtsen that sounds like a bug in ares_set_servers_csv() though, which indeed is related to this issue but still will need a separate fix as well

By the way, I've got a pending change that should help with this (it adds a new structure and entrypoints for setting per-server ports) -- I'll send it round soon.

Fixed in commit 7972adc5d7ff


Integrate docs with main repository

Move docs content to the main repository & add metadata .json file for consuming them in our Vue app.

Oh, I totally forgot that all the tests will break until we "fix" the docs to pass the doc tester. If you remove "docs", from testpaths at the end of pyproject.toml, that'll do it. I can fix them up later.

I cherry-picked this into the v0.8.0 tag so we could test new docs via the GitHub API releases mechanism. See dc4c846a0a964ffe31a98ae848164c6a5dc2658e.
Usually we won't need to do this as the docs will already have the base format. But if we do need to do a retroactive sweeping change to all the docs (since each version is managed independently) this could be one way to do that.


v0.1.7

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/56750
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.7 -m "<description of version>" 4f19d3c7460f5cb7f72d9a98073ba7baa5adb621
git push origin v0.1.7


[Bug]: Password field accepts more than 42 characters

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When a new user Signs up the maximum password count that can be entered is 42 characters. But currently character count of 47 is getting accepted and user is able to sign up successfully. (Refer video)
This happens during 'Reset password' also.
https://loom.com/share/ba08dc3785f6484fb0db8bbebe8d567c
Expected Behavior: As prompted to the user a maximum of only 42 characters must be accepted.
Steps To Reproduce

Go to https://release.app.appsmith.com/user/login
Sign up > Enter new email id > Enter password with 47 character count and observe

Environment
Release
Version
Cloud

A community member reported the same issue on https://github.com/appsmithorg/appsmith/issues/12965 and has requested we take a look at the limit again.
Password generators throw out passwords with as many as 48 characters.


test: initial framework for easier integration testing

This is coming out of frustrations with the existing "integration"
testing. Most notible is the slow build/test due to lots of installs,
docker builds and bad comparison/debug. Having a proper DSL/framework
should allow parallel builds and better comparison/debug.
Issue: https://github.com/adamdecaf/cert-manage/issues/88

This passed in CI: https://travis-ci.org/adamdecaf/cert-manage/builds/318760043 I aborted it early because I don't need finish the tip testing.


Allow ADAM überscript to take .aws/credentials file

Currently, we just take .boto.

Yes, ~/.aws/credentials should be preferred for credentials since it has become the standard and is supported by multiple AWS SDKs (Boto, Java SDK to name two).


Can't write a pattern to a scroll on v0.8.3?

This is a known bug! It's fixed in 0.9, which will hopefully be out by Monday, 13th June, 2022.

Hi, I'm playing on the fabric version of 9.0, and for some reason I'm still unable to write spells to scrolls. In fact, the game actively labels them as being unwritable with the "Auditor's Purification II" spell.

If it helps at all, this was experienced in the Blanketcon 2022 modpack. Sorry for the hassle, and I hope things go smoothly with bugfixing!

You should only be writing patterns to scrolls; use Consideration, then draw a pattern, then try writing it.
(Also please open a fresh issue in the future instead of commenting on this one!)


Boolean 'not' operator does not display correctly in Numbas.jme.display.texify

I have been experimenting with writing custom scripts to use JME for checking Boolean expressions, it seems to be working OK but the input box won't accept '!' or 'not' operators. The error message reads:-
texOps[tok.name.toLowerCase(...)] is not a function
I traced this through and found that  both ops are translated to 'not', though only '!' is define in texOps.
Both the editor and runtime are affected.

Whoops, can't think how long that's been broken! Fixed it. Thanks for letting me know.

Thanks for the quick response – looks like no-one else is using Booleans!
Kind regards,
Graeme Chester
Dr. E.G. Chester, School of EEE, Newcastle University
Tel: 0191 208 7325 Email: graeme.chester<EMAIL_ADDRESS>From: Christian Lawson-Perfect<EMAIL_ADDRESS>Sent: 12 January 2016 15:22
To: numbas/Numbas<EMAIL_ADDRESS>Cc: Graeme Chester<EMAIL_ADDRESS>Subject: Re: [Numbas] Boolean 'not' operator does not display correctly in Numbas.jme.display.texify (#422)
Whoops, can't think how long that's been broken! Fixed it. Thanks for letting me know.
—
Reply to this email directly or view it on GitHubhttps://github.com/numbas/Numbas/issues/422#issuecomment-170945277.


chef-client ignores .chef/knife.rb

Description
When running chef-client, it ignores the local .chef/knife.rb.  I wanted simplify the command-line options have have knife.rb with:
local_mode true

Chef Version
ChefDK 2.5.3
Platform Version
Ubuntu 16.04
Replication Case
# .chef/knife.rb
chef_repo = File.join(File.dirname(__FILE__), '..')
infra_cookbooks = "#{chef_repo}/cookbooks"
berks_cookbooks = "#{chef_repo}/berks-cookbooks"
cookbook_path [infra_cookbooks,berks_cookbooks]
local_mode true

chef-client --runlist 'recipe[my_cookbook]'

Client Output
Actual
[2018-04-25T21:51:31-07:00] WARN: *****************************************
[2018-04-25T21:51:31-07:00] WARN: Did not find config file: /etc/chef/client.rb, using command line options.
[2018-04-25T21:51:31-07:00] WARN: *****************************************

Expected
Expectation was that chef would run in local mode as it was configured this way in .chef/knife.rb.
As a workaround, I have to run: chef-client --local-mode --runlist 'recipe[my_cookbook]', to pick up the local configuration, which defeats the purpose specifying local_mode in the first place.
Also you can do chef-client --config .chef/knife.rb --runlist 'recipe[my_cookbook]', which could then set it to local mode, but then why not use --local-mode in the first place, if you have to manually change the config path.

When you give --local-mode it uses the workstation config loader behavior for historical reasons. Without that (i.e. normal chef-client usage) it uses /etc/chef/client.rb. The historical baggage around local-mode is one of several reasons to prefer chef-solo anyway, which uses /etc/chef/solo.rb by default.

Closing this one as the behaviour is explained and this is as designed.


Create our work part

Create our work part

There is a bug
https://gyazo.com/b83b70ffa43676258038de4fb291a44a
In Responsive, Make 2 columns in tablet, then 1 column in Mobile
https://gyazo.com/356dcc32328aeaa20255c80f606cbc48
Fix categories overlap each others, if they could fit 1 line, let it be.
Remove hover effect on mobile. Could you suggest a better way display that information in mobile screen? Because in Mobile, we don't have "hover" effect.
https://gyazo.com/356dcc32328aeaa20255c80f606cbc48


How to get file/line number ?

Hello,
Is there an easy way to get file/line number in error ?
Wouldn't anyone need that as a default ?

duplicate of #63

formatter := runtime.Formatter{ChildFormatter: &log.TextFormatter{
	FullTimestamp: true,
}}
formatter.Line = true
log.SetFormatter(&formatter)
log.SetOutput(os.Stdout)
log.SetLevel(log.InfoLevel)
log.WithFields(log.Fields{
	"file": "main.go",
}).Info("Server is running...")

Note: import ( 	runtime "github.com/banzaicloud/logrus-runtime-formatter"
log "github.com/sirupsen/logrus")


If backdropClosesModal is True, onClickImage is activated and backdropClosesModal is activated

If backdropClosesModal is True, onClickImage is activated and backdropClosesModal is activated.
How to solve it

preventDefault from the function passed to onClickImage e.g.
class Seohangyu extends Component {
  handleImageClick(e) {
    e.preventDefault()
  }
  render () {
    <Lightbox
      onClickImage={this.handleImageClick}
    />
  }
}

The way you sent it was not resolved.
However,  'preventDefault' has been corrected to 'stopPropgation' and solved Your quick answer was a great help. Thank you.

@seohangyu right, yeah makes sense - my mistake. Glad your issue is resolved.


ci: Enable /azp run for submodule pipeline

Reason for Change:

Enables the use of comment triggers for the submodules pipeline. .pipelines/submodules-pipeline.yaml was excluded from the triggers and prevented /azp run Azure Container Networking PR - submodules from being called to run the pipeline.
Issue Fixed:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[x] relevant PR labels added

Notes:

/azp run Azure Container Networking PR - submodules


Make TensorCoreTiledLayout import more robust

Summary: Fixes https://github.com/pytorch/torchtune/issues/1908. Previous attempts (https://github.com/pytorch/torchtune/pull/1886) to fix this issue still break in local settings, so it's more robust and simpler to just try catch the import error.
Test Plan:
from torchtune.training.quantization import *

@joecummings @ebsmothers @jainapurva Can you please take a look?


Fix-Add droppable attribute check on target column before dropping source column

droppable attribute is not respected when  dropping source column over target column where the attribute is set to false.
Just added a check on this PR for droppable attribute on target column.
Related Issue : #494

Thanks for this! Seems like a quite a big woopsie :face_with_head_bandage:
Will update here when the version including this change is out


Mathematical Averages Calculator

Enter calculator name -
Mathematical Averages Calculator
Describe calculator functionality -
The Mathematical Averages Calculator takes a set of values and finds the Arithmetic Mean, Geometric Mean and Harmonic Mean of the same.
Under which category you wish to place it -

[ ] Computer Science
[x] Mathematics
[ ] Physics
[ ] Chemistry
[ ] Biology
[ ] Art
[ ] Finance
[ ] Scientific
[ ] Health
[ ] Sports
[x] Let project owner decide
[ ] Suggest a new category - (Answer here)

@Vedansh-Keshari I want to work on this task, kindly assign this to me

Hey @its-kumar-yash,
Assigned.
You may contribute your code under the folder path -
calc_for_everything\calculators\src\mathematical\averagesCalculator
Issue completion deadline - 26th Oct 8:00:00 am IST
NOTE - In case the issue completion deadline is not met, the issue will be assigned to a new participant


Plugins broken with GHC 8.10.1

./ghcide/Testing.hs:1:1: error:
    Bad interface file: /nix/store/y47mcwxg3xjzm1vncv8vm3km103l1kzn-ghc-typelits-knownnat-0.7.2/lib/ghc-8.10.1/x86_64-linux-ghc-8.10.1/ghc-typelits-knownnat-0.7.2-EOol1HudK6SImK1KLBAdq1/GHC/TypeLits/KnownNat.hi
        Something is amiss; requested module  ghc-typelits-knownnat-0.7.2:GHC.TypeLits.KnownNat differs from name found in the interface file sorted-list-<IP_ADDRESS>:GHC.TypeLits.KnownNat (if these names look the same, try again with -dppr-debug)

The error is specific to 8.10.1 and ghcide, GHCi does not display this error. More details in #519

Looks like - https://gitlab.haskell.org/ghc/ghc/issues/18070

we can probably close this ticket then!


crud

[ ] check deploy pass

ActionController::RoutingError (No route matches [GET] "/manifest.json"):

7.2

Whenever we have such a combination of routes, controller actions, and views that work together to perform CRUD operations on an entity, we call that entity a resource. For example, in our application, we would say an article is a resource.

inspect routes
bin/rails routes

railway crash
rails

production
[Architecture](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2)
[Observability](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/observability)
[Logs](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/logs)
[Settings](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/settings)

Share

rails
[Deployments](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/service/d25f50b2-329f-4eb8-ac7c-6f02843f4f2e)
[Variables](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/service/d25f50b2-329f-4eb8-ac7c-6f02843f4f2e/variables)
[Metrics](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/service/d25f50b2-329f-4eb8-ac7c-6f02843f4f2e/metrics)
[Settings](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/service/d25f50b2-329f-4eb8-ac7c-6f02843f4f2e/settings)
[rails-drew.up.railway.app](https://rails-drew.up.railway.app/)
[US West](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/service/d25f50b2-329f-4eb8-ac7c-6f02843f4f2e/settings#deploy-region)
[1 Replica](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/service/d25f50b2-329f-4eb8-ac7c-6f02843f4f2e/settings#deploy-replicas)

History

[rails](https://railway.app/project/7ac7363f-2207-4fc7-abd1-c84d9223a7a2/service/d25f50b2-329f-4eb8-ac7c-6f02843f4f2e)
/
0011ea7
Sep 17, 2024 10:16 AM

Crashed

Get Help

Details
Build Logs
Deploy Logs
HTTP Logs

Filter
Filter logs using "", (), AND, OR, -

You reached the start of the range → Sep 17, 2024 10:16 AM

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/cref.rb:91:in `get'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/eager_load.rb:173:in `block in actual_eager_load_dir'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/helpers.rb:47:in `block in ls'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/helpers.rb:25:in `each'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/helpers.rb:25:in `ls'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/eager_load.rb:168:in `actual_eager_load_dir'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/eager_load.rb:17:in `block (2 levels) in eager_load'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/eager_load.rb:16:in `each'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/eager_load.rb:16:in `block in eager_load'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/eager_load.rb:10:in `synchronize'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader/eager_load.rb:10:in `eager_load'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader.rb:413:in `block in eager_load_all'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader.rb:411:in `each'

	from /usr/local/bundle/ruby/3.2.0/gems/zeitwerk-2.6.18/lib/zeitwerk/loader.rb:411:in `eager_load_all'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/application/finisher.rb:80:in `block in <module:Finisher>'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/initializable.rb:32:in `instance_exec'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/initializable.rb:32:in `run'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/initializable.rb:61:in `block in run_initializers'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:228:in `block in tsort_each'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:350:in `block (2 levels) in each_strongly_connected_component'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:431:in `each_strongly_connected_component_from'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:349:in `block in each_strongly_connected_component'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:347:in `each'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:347:in `call'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:347:in `each_strongly_connected_component'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:226:in `tsort_each'

	from /usr/local/lib/ruby/3.2.0/tsort.rb:205:in `tsort_each'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/initializable.rb:60:in `run_initializers'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/application.rb:435:in `initialize!'

	from /rails/config/environment.rb:5:in `<main>'

	from config.ru:3:in `require_relative'

	from config.ru:3:in `block (2 levels) in <main>'

	from /usr/local/bundle/ruby/3.2.0/gems/rack-3.1.7/lib/rack/builder.rb:108:in `eval'

	from /usr/local/bundle/ruby/3.2.0/gems/rack-3.1.7/lib/rack/builder.rb:108:in `new_from_string'

	from /usr/local/bundle/ruby/3.2.0/gems/rack-3.1.7/lib/rack/builder.rb:97:in `load_file'

	from /usr/local/bundle/ruby/3.2.0/gems/rack-3.1.7/lib/rack/builder.rb:67:in `parse_file'

	from /usr/local/bundle/ruby/3.2.0/gems/rackup-2.1.0/lib/rackup/server.rb:354:in `build_app_and_options_from_config'

	from /usr/local/bundle/ruby/3.2.0/gems/rackup-2.1.0/lib/rackup/server.rb:263:in `app'

	from /usr/local/bundle/ruby/3.2.0/gems/rackup-2.1.0/lib/rackup/server.rb:424:in `wrapped_app'

	from /usr/local/bundle/ruby/3.2.0/gems/rackup-2.1.0/lib/rackup/server.rb:326:in `block in start'

	from /usr/local/bundle/ruby/3.2.0/gems/rackup-2.1.0/lib/rackup/server.rb:382:in `handle_profiling'

	from /usr/local/bundle/ruby/3.2.0/gems/rackup-2.1.0/lib/rackup/server.rb:325:in `start'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/commands/server/server_command.rb:38:in `start'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/commands/server/server_command.rb:145:in `block in perform'

	from <internal:kernel>:90:in `tap'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/commands/server/server_command.rb:136:in `perform'

	from /usr/local/bundle/ruby/3.2.0/gems/thor-1.3.2/lib/thor/command.rb:28:in `run'

	from /usr/local/bundle/ruby/3.2.0/gems/thor-1.3.2/lib/thor/invocation.rb:127:in `invoke_command'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/command/base.rb:178:in `invoke_command'

	from /usr/local/bundle/ruby/3.2.0/gems/thor-1.3.2/lib/thor.rb:538:in `dispatch'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/command/base.rb:73:in `perform'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/command.rb:71:in `block in invoke'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/command.rb:149:in `with_argv'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/command.rb:69:in `invoke'

	from /usr/local/bundle/ruby/3.2.0/gems/railties-7.2.1/lib/rails/commands.rb:18:in `<main>'

	from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'

	from <internal:/usr/local/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'

	from /usr/local/bundle/ruby/3.2.0/gems/bootsnap-1.18.4/lib/bootsnap/load_path_cache/core_ext/kernel_require.rb:30:in `require'

	from /rails/bin/rails:4:in `<main>'

container event container died

rails | Railway


Use IteratorResult to cleanly indicate end of stream.

Noe this builds on #785.  I'm curious to see how GitHub & Reviewable handle this situation.  If it's not convenient, let's just wait on this PR until the other one is merged.

This change is

OK, it looks like I'm proposing to merge this into async-stream-gc, which makes no sense.  But after that PR is merged into master, I can redirect this one to diff against master instead, so it will all work out as far as GitHub is concerned.  Dunno about Reviewable though.

Yeah, you might have to close this PR and start a new PR against master. Let me know when/if ready. Either way, I just took a look and this looks great - fixes bug & aligns with future standards - LGTM++

Review status: 0 of 10 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Switching the base branch to master worked, so this PR is ready for review.  Thanks!

Review status: 0 of 10 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

 Nice, this is a big step in the right direction. Left a few comments, unrelated to this change, but something we should address in future PRs.

Reviewed 3 of 12 files at r1, 7 of 9 files at r2.
Review status: all files reviewed at latest revision, 6 unresolved discussions.

src/contrib/data/dataset.ts, line 259 at r2 (raw file):

   * within that element are disposed.
   *
   * @param f A function to apply to each dataset element.

say it returns a promise that resolves when done iterating over all elements.

src/contrib/data/streams/data_stream.ts, line 278 at r2 (raw file):

  }

  async next(): Promise<IteratorResult<T>> {

How about FunctionCallStream taking a user-provided function that returns what next() returns.
This way the user is in full control when to stop iterating.
// f looks a lot like `async function* f() {...}`
const f = streamFromFunction(() => {
  const value = await someVal();
  const done = await checkIfDone();
  return {value, done};
});

Up to you if you want to do it here, or leave a TODO

src/contrib/data/streams/data_stream.ts, line 292 at r2 (raw file):

  async next(): Promise<IteratorResult<T>> {
    while (this.count++ < this.maxCount) {
      const skipped = await this.upstream.next();

we should read maxCount elements in parallel.
const reads = [];
while (cond) { reads.push(this.upstream.next()); }
const doneReads = await Promise.all(reads);
...

Up to you if in this PR, or add a TODO for later.

src/contrib/data/streams/data_stream.ts, line 353 at r2 (raw file):

    // output queue, then this stream is also exhausted.
    while (this.outputQueue.length() === 0) {
      if (!await this.pump()) {

this looks like none of the QueueStream derivatives will be able to read in parallel. E.g. BatchedStream below. If that's the case, add a TODO to fix this.

src/contrib/data/streams/data_stream.ts, line 372 at r2 (raw file):

  async pump(): Promise<boolean> {
    const item = await this.upstream.next();
    if (item.done) {

If you want to read a whole batch in parallel, can't check for item.done here. Have to do something like:
const doneReads = await Promise.all(batchOfReads);
// deal with partial batch, since last K reads may have `done: true`

And rework logic in QueueStream, or remove QueueStream altogether. See previous comment.

src/contrib/data/streams/data_stream_test.ts, line 187 at r2 (raw file):

    const func = () => ++i < 100 ? i : undefined;

    // streamFromFunction never ends, so we truncate it

see another comment above re: streamFromFunction

Comments from Reviewable

Review status: 7 of 12 files reviewed at latest revision, 6 unresolved discussions, some commit checks pending.

src/contrib/data/dataset.ts, line 259 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
say it returns a promise that resolves when done iterating over all elements.

Done.

src/contrib/data/streams/data_stream.ts, line 278 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
How about FunctionCallStream taking a user-provided function that returns what next() returns.
This way the user is in full control when to stop iterating.
// f looks a lot like `async function* f() {...}`
const f = streamFromFunction(() => {
  const value = await someVal();
  const done = await checkIfDone();
  return {value, done};
});

Up to you if you want to do it here, or leave a TODO

Good idea; done here.

src/contrib/data/streams/data_stream.ts, line 292 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
we should read maxCount elements in parallel.
const reads = [];
while (cond) { reads.push(this.upstream.next()); }
const doneReads = await Promise.all(reads);
...

Up to you if in this PR, or add a TODO for later.

This bears discussion.  Added TODO.

src/contrib/data/streams/data_stream.ts, line 353 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
this looks like none of the QueueStream derivatives will be able to read in parallel. E.g. BatchedStream below. If that's the case, add a TODO to fix this.

I'm unclear that parallel reads would have much real benefit here.  In any case an imminent PR reduces the use of QueueStreams.  Left a TODO to discuss the remaining cases.

src/contrib/data/streams/data_stream.ts, line 372 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
If you want to read a whole batch in parallel, can't check for item.done here. Have to do something like:
const doneReads = await Promise.all(batchOfReads);
// deal with partial batch, since last K reads may have `done: true`

And rework logic in QueueStream, or remove QueueStream altogether. See previous comment.

Fix already queued up in followup PR :)

src/contrib/data/streams/data_stream_test.ts, line 187 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
see another comment above re: streamFromFunction

Done.

Comments from Reviewable

Review status: 7 of 12 files reviewed at latest revision, 5 unresolved discussions, all commit checks successful.

src/contrib/data/streams/data_stream.ts, line 292 at r2 (raw file):
Previously, davidsoergel (David Soergel) wrote…
This bears discussion.  Added TODO.

Nice. The todo you added makes a great point - and not a high priority for now.

src/contrib/data/streams/data_stream.ts, line 353 at r2 (raw file):
Previously, davidsoergel (David Soergel) wrote…
I'm unclear that parallel reads would have much real benefit here.  In any case an imminent PR reduces the use of QueueStreams.  Left a TODO to discuss the remaining cases.

Good point. It's unclear. TODO is great for now. if we ever come into a real-world example being bottleneck-ed by this, we can tackle it then. Until then, the simplest thing that works is the best :)

src/contrib/data/streams/data_stream.ts, line 372 at r2 (raw file):
Previously, davidsoergel (David Soergel) wrote…
Fix already queued up in followup PR :)

Nice. Looking forward.

Comments from Reviewable


PowerToys reported an error and told me to report an error here and paste the log text here idk anything

Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.18362.0
IntPtr Length: 8
x64: True
Date: 09/14/2020 15:36:37
Exception:
System.IO.FileNotFoundException: C:\Users[USERNAME]\Desktop\jdk-10.0.2_windows-x64_bin.exe
at System.Diagnostics.FileVersionInfo.GetVersionInfo(String fileName)
at Microsoft.Plugin.Program.Programs.Win32Program.ExeProgram(String path)
at Microsoft.Plugin.Program.Programs.Win32Program.GetAppFromPath(String path)
at Microsoft.Plugin.Program.Storage.Win32ProgramRepository.OnAppCreated(Object sender, FileSystemEventArgs e)
at System.IO.FileSystemWatcher.NotifyFileSystemEventArgs(WatcherChangeTypes changeType, ReadOnlySpan`1 name)
at System.IO.FileSystemWatcher.ParseEventBufferAndNotifyForEach(Byte[] buffer)
at System.IO.FileSystemWatcher.ReadDirectoryChangesCallback(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* overlappedPointer)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
at System.Threading._IOCompletionCallback.PerformIOCompletionCallback(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* pNativeOverlapped)

What a title 😂
But more seriously I think thats a duplicate I've seen many times. What do you think @enricogior

@alekhyareddy28 is this a known issue?

I think this was what @somil55 fixed.

@arjunbalgovind This is similar to #6425. I made a fix for this in #6456

@festival0156 Closing this as duplicate of #6425


Add BhagvadGita AI chatbot as a feature

I recently contributed to an AI chatbot of Bhagavad Gita and felt it could be a fantastic addition to this website. It can be added as a redirect link to the landing page for the chatbot site. Here's the link to the site.

ok go for it and please join discord channel for discussion about this issue

Is this issue still open? Can you please assign me this issue?

@akshitagupta15june should this be closed now? I think this was not done.

I would like to work on this issue, kindly assign this to me.


v1.91 regression: Draco-encoded 3D Tiles fail to load when Cesium is running on localhost

This simple Sancastle (loaded from localhost) works fine in v1.90 but spews errors in v1.91:
Error: Failed to load model: https://assets.cesium.com/69380/2/034.b3dm?v=1
Cannot read properties of undefined (reading '__destroy__')
A 3D tile failed to load: https://assets.cesium.com/69380/2/030.b3dm?v=1

Stack trace:
a.destroy (draco_decoder_nodejs.js:84)
decodeDracoTypedArray (decodeDraco.js:152)
decodeAttribute (decodeDraco.js:196)
decodePrimitive (decodeDraco.js:325)
decode (decodeDraco.js:346)
callAndWrap (createTaskProcessorWorker.js:39)
(anonymous) (createTaskProcessorWorker.js:85)
Worker.postMessage (async)
(anonymous) (TaskProcessor.js:272)
(anonymous) (when.js:196)
when (when.js:59)
TaskProcessor.scheduleTask (TaskProcessor.js:261)
scheduleDecodingTask (DracoLoader.js:112)
DracoLoader.decodeModel (DracoLoader.js:259)
Model.update (Model.js:5340)
Batched3DModel3DTileContent.update (Batched3DModel3DTileContent.js:460)
Cesium3DTile.process (Cesium3DTile.js:1906)
processTiles (Cesium3DTileset.js:2335)
Cesium3DTileset.prePassesUpdate (Cesium3DTileset.js:2197)
PrimitiveCollection.prePassesUpdate (PrimitiveCollection.js:392)
prePassesUpdate (Scene.js:3694)
tryAndCatchError (Scene.js:3791)
Scene.render (Scene.js:3860)
CesiumWidget.render (CesiumWidget.js:790)
render (CesiumWidget.js:42)
requestAnimationFrame (async)

The tileset is Melbourne Photogrammetry.
Strangely, the problem does not occur on sandcastle.cesium.com or the automatic deployment of main, only on localhost. So maybe there's something strange about how I'm building it. NodeJS v12.22.7 in WSL, fwiw.
Anyway git bisect reports the first commit with the problem is this one:
https://github.com/CesiumGS/cesium/commit/c497cdb4d22da601827003ee2ff1779b54fbe6dd
On my system it's perfectly consistent in working before that commit and spewing those errors after that commit.

Nevermind! Once I looked at the problematic commit more closely, the cause became obvious: the draco3d library was updated in that commit, and I didn't run npm install. Sorry for the noise!


ftp download

I could unzip
cds+introns.fa.gz
but not
cds+introns+utrs.fa.gz

Was it this file?:
https://curation.pombase.org/dumps/builds/pombase-build-2024-09-01/fasta/feature_sequences/cds+introns+utrs.fa.gz
That worked OK for me.  What was the error?

Could you let me know what the file size is?  It should be 5124639.
Which URL did you use to download?

5.1 MB
following link from
https://www.pombase.org/downloads/genome-datasets
Sequences in FASTA format for:

5.1 MB

What is the file size in bytes?  ls -al cds+introns+utrs.fa.gz

following link from
https://www.pombase.org/downloads/genome-datasets
Sequences in FASTA format for:

I just tried the link from there and didn't have a problem uncompressing the file.
Could you see what gzip reports from the command line?:
gzip -tv cds+introns+utrs.fa.gz
Thanks.

gzip -tv cds+introns+utrs.fa.gz
cds+introns+utrs.fa.gz:	  OK
but it isn't unzipped

cds+introns+utrs.fa.gz: OK

Looks OK then?  Perhaps OS X has a problem with the file?

but it isn't unzipped

The "-t" flag just tests that the file is OK.  It doesn't unzip.

Maybe we should discuss this on a call?

Appear to be the "+" in the file name, and archive utility combination

It took a bit of Googling but I think this same problem:
https://apple.stackexchange.com/questions/388759/archive-utility-cant-open-some-gzipped-text-files-based-on-their-contents
This was linked to from that post: https://github.com/libarchive/libarchive/issues/1051#issuecomment-1267882936
Looks like someone is having trouble with a fasta file.
So it's a known Mac bug.
Val, what version of MacOS are you using?
I saw several posts suggesting using this instead on this buggy built in archive program: https://theunarchiver.com/

Ventura 13.2.1

Ventura 13.2.1

OK, so hopefully most Mac users will have a more recent version with fewer bugs.

yep, ignore unless anyone complains. I will update soon


proposed merge: MONDO:0044923 acute myeloid leukemia with mutated NPM1 and MONDO:0018437 acute myeloid leukemia with NPM1 somatic mutations

Proposed merge of these two terms:
[Term]
id: MONDO:0044923
name: acute myeloid leukemia with mutated NPM1
def: "An acute myeloid leukemia with mutation of the nucleophosmin gene.
It is usually associated with normal karyotype and frequently has
myelomonocytic or monocytic features. It usually responds to induction
therapy." [NCIT:C82431]
synonym: "acute myeloid leukemia with cytoplasmic nucleophosmin" EXACT
[NCIT:C82431]
synonym: "acute myeloid leukemia with mutated NPM1" EXACT [NCIT:C82431]
synonym: "AML with mutated NPM1" EXACT [NCIT:C82431]
synonym: "NPMc+ AML" EXACT [NCIT:C82431]
xref: NCIT:C82431 {source="MONDO:equivalentTo"}
xref: ONCOTREE:AMLNPM1 {source="MONDO:equivalentTo"}
xref: UMLS:C2826177 {source="MONDO:equivalentTo"}
is_a: MONDO:0018874 {source="NCIT:C82430",
source="NCIT:C82431/inferred", source="ONCOTREE:AMLNPM1/inferred"} !
acute myeloid leukemia
[Term]
id: MONDO:0018437
name: acute myeloid leukemia with NPM1 somatic mutations
def: "Acute myeloid leukemia with NPM1 somatic mutations is a subtype of
acute myeloid leukemia with recurrent genetic abnormalities
characterized by clonal proliferation of myeloid blasts harboring
mutations of the NPM1 gene in the bone marrow, blood and other tissues.
It is associated with multilineage dysplasia, involving the myeloid,
monocytic, erythroid, and megakaryocytic cell lineages. Patients usually
present with leukocytosis, thrombocytosis and nonspecific symptoms
related to ineffective hematopoesis (fatigue, bleeding and bruising,
recurrent infections, bone pain), with frequent extramedullary
involvement typically presenting as gingival hyperplasia and
lymphadenopathy." [Orphanet:402026]
subset: ordo_disease {source="Orphanet:402026"}
synonym: "AML with NPM1 somatic mutations" EXACT [Orphanet:402026]
xref: ICD10:C92.0 {source="ORDO:402026/ntbt", source="Orphanet:402026"}
xref: Orphanet:402026 {source="MONDO:equivalentTo"}
xref: SCTID:763309005 {source="MONDO:equivalentTo"}
is_a: MONDO:0020078 {source="Orphanet:402026"} ! acute myeloid leukemia
with recurrent genetic anomaly
The NCIT class doesn't explicitly say somatic mutations but can this be assumed?

cc @fragosog

Hi @nicolevasilevsky , consulted with one of the editors.  This concept (and others like it) are not specific for somatic mutations, can include germline.  Not an expert but in this particular case it doesn't look like germline variants are playing a role.  Want me to double-check?  It might be a blanket-case concept.

can you check?
It would be good if we can all agree on a single named concept for each of these diseases (across ORDO, NCIT, and others). If there exists both somatic and germline then perhaps ORDO would be open to generalizing their existing classes.
Ot if the community feels it's really important to have distinct named classes for the generic, somatic and germline versions that can be done too

OK, in this particular case, the editor is not aware of germline variants of consequence, but she can't exclude creating germline variant subclasses in the future.  Looks like a merge won't work.  Not sure that ORDO would adopt a similar generic superclass creation guideline, it might be useful but if most of the variants of interest are somatic maybe adds too much content for the bang you get back?

I think this can be closed.


Add a way to skip a test

Issue #, if available:
Description of changes:
The test case can be skipped if it returned AWS_OP_SKIP.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
Patch coverage has no change and project coverage change: -0.03 :warning:

Comparison is base (cfaa904) 81.97% compared to head (0129121) 81.95%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1012      +/-   ##
==========================================
- Coverage   81.97%   81.95%   -0.03%
==========================================
  Files          52       52
  Lines        5569     5568       -1
==========================================
- Hits         4565     4563       -2
- Misses       1004     1005       +1

see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Typed arrays break resource deserialization when some elements are null

Godot version
v4.0.beta16.official [518b9e580]
System information
MacOS 13.1
Issue description
I have two custom resources (called Collection and Item) and Collection contains an exported property with the type items: Array[Item].  My serialized Collection.items is length 2, the 0th element is an Item and the 1st is null. When I deserialize this resource, it will silently fail.
If I change the type annotation from Array[Item] to just Array and literally nothing else, it will start working. It seems like unless there's a way to specify a Maybe or Optional type, we have to allow null.
Steps to reproduce

Create two custom resources, Collection and Item.  Collection should have a property @export items: Array[Item].
Create an actual resource of type Collection and, using the inspector, add two elements to its items array
Instantiate the 0th item with a new Item
Do not instantiate the element at index 1
Try to iterate over the array after a load and see that it will never deserialize that first element.
Notice that if you literally only change the type annotation of items to be simply Array`, you can now iterate over the items whether null or not.

Minimal reproduction project
repro.zip

Is this fixed by #72492

Relevant #70657.

This appears to have been fixed by the recent typed array update.  Closing.


Audio Context is not in window object

Do you want to request a feature or report a bug?
Report Bug
What is the current behavior?
AudioContext in window is undefined.
If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can yarn install and yarn test.
What is the expected behavior?
window should return the AudioContext class/function
Please provide your exact Jest configuration and mention your Jest, node,
yarn/npm version and operating system.

This is an issue with jsdom, not jest.


FitDiagnostics with MultiSignalModel?

I'd like to make nice plots of fit_b and fit_sb where all three signal rates in my analysis are floating but I can't find a way to run FitDiagnostics with more than one POI floating. Is this possible?

Found a hack. Basically I use MultiDimFit to get the best fit signal strengths, then use FitDiagnostics twice with --customStartingPoint, once setting the parameters to zero, once setting them to their best fit value. This way my S+B fit is actually the 'background only fit' where I've fixed the signal to the best fit value.
Here's my horrible hack to get what I want:
        # get best fit signal strengths
        cmd = 'combine -M MultiDimFit --setParameters  rZZ=1,rZH=1,rHH=1 --robustFit 1 -n .fit_all -d ZZ4b/nTupleAnalysis/combine/combine_closure_SvB.root'
        execute(cmd, o.execute)
        execute('mv higgsCombine.fit_all.MultiDimFit.mH120.root combinePlots/%s/'%classifier, o.execute)
        import ROOT
        bestfit_file = ROOT.TFile('combinePlots/%s/higgsCombine.fit_all.MultiDimFit.mH120.root'%classifier, 'READ')
        bestfit_tree = bestfit_file.Get('limit')
        bestfit_tree.GetEntry(0)
        rZZ, rZH, rHH = bestfit_tree.rZZ, bestfit_tree.rZH, bestfit_tree.rHH
        bestfit_file.Close()

        # background only fit
        cmd = 'combine -M FitDiagnostics --setParameters rZZ=0.,rZH=0.,rHH=0. --customStartingPoint --robustFit 1 --saveShapes --saveWithUncertainties -d ZZ4b/nTupleAnalysis/combine/combine_closure_%s.root'%(classifier)
        execute(cmd, o.execute)
        execute('mv fitDiagnostics.root combinePlots/%s/fitDiagnostics_closure_b.root'%(classifier), o.execute)

        # best fit signal
        cmd = 'combine -M FitDiagnostics --setParameters rZZ=%f,rZH=%f,rHH=%f --customStartingPoint --robustFit 1 --saveShapes --saveWithUncertainties -d ZZ4b/nTupleAnalysis/combine/combine_closure_%s.root'%(rZZ, rZH, rHH, classifier)
        execute(cmd, o.execute)
        execute('mv fitDiagnostics.root combinePlots/%s/fitDiagnostics_closure_s.root'%(classifier), o.execute)

Solution here: https://github.com/cms-analysis/HiggsAnalysis-CombinedLimit/issues/766

Hi Patrick, are there any other freely floating parameters in your model? (e.g. a rateParam for one of the backgrounds?) If so the easiest thing to do is to add --redefineSignalPOIs <other_floating_param> and --freezeParameters rZZ,rZH,rHH. If you have no other freely floating parameters in the model I have to think a bit, as I think combine will complain in this setup if there is no POI that's floating

Yeah the idea is to do a background only fit and all my nuisances are constrained.

I was about to respond saying that I think combine won't complain provided
there are some parameters in the model (i.e you can freeze the pois without
having to redefine something else as the poi) at least with multidim fit's
global fit it should be ok
Best,
Nick
On Thu, Jul 21, 2022 at 4:51 PM Patrick Bryant @.***>
wrote:

Actually this might have worked:
@.***/uscms_data/d3/bryantp/CMSSW_11_1_0_pre5/src% combine -M MultiDimFit --setParameters rZZ=0,rZH=0,rHH=0 --freezeParameters rZZ,rZH,rHH --robustFit 1 -n .fit_b --saveWorkspace --saveFitResult -d ZZ4b/nTupleAnalysis/combine/combine_closure_SvB.root
<<< Combine >>>
Options for Robust Minimizer ::
Tolerance  0.1
Strategy   0
Type,Algo  Minuit2,Migrad

method used is MultiDimFit
random number generator seed is 123456
ModelConfig 'ModelConfig' defines more than one parameter of interest. This is not supported in some statistical methods.
Set Default Value of Parameter rZZ To : 0
Set Default Value of Parameter rZH To : 0
Set Default Value of Parameter rHH To : 0
SimNLL created with 9 channels, 0 generic constraints, 203 fast gaussian constraints, 0 fast poisson constraints, 0 fast group constraints,
Skipping rHH. Looks like the last fit did not float this parameter. You could try running --algo grid to get the errors.
Skipping rZZ. Looks like the last fit did not float this parameter. You could try running --algo grid to get the errors.
Skipping rZH. Looks like the last fit did not float this parameter. You could try running --algo grid to get the errors.

--- MultiDimFit ---
best fit parameter values:
rHH :    +0.000
rZZ :    +0.000
rZH :    +0.000
Done in 0.01 min (cpu), 0.01 min (real)
—
Reply to this email directly, view it on GitHub
https://github.com/cms-analysis/HiggsAnalysis-CombinedLimit/issues/765#issuecomment-1191656413,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMEVW3RC7J6SMEINDXWVWLVVFWXZANCNFSM5ZSCDQ6A
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***
.com>

Ok yeah that works, thanks! I feel like I had this at some point and forgot. Good to have it documented here now.


Fps dropping down

Hello hritik, i used vscode and opened a server, everything is working fine but the fps is 0 and i am not able to move or look around, how can i solve this problem

Hmm, perhaps I can look at the code and come back to you. Is it possible for you to send it in this thread?


fix: Disable unsupported generators for AsyncAPI V3 (#979)

Description

Updated the GeneratorModal component to disable unsupported generators for AsyncAPI V3.
Added a warning message for unsupported generators with a link to the tracking page.

Related issue(s)
Fixes #979

/rtm


Could we replace  the left vertical scroll bar with just a vertical line？

Thanks for this greate template.
But some of the readers think the left vertical scroll bar looks a little ugly.

Could we replace that with just a vertical line, like the following

Every browser renders scrollbars differently, I'd prefer not to customize them.
In fact on touch devices you don't even see a scrollbar.


Database implementation

Is your feature request related to a problem? Please describe.
Having to push to VPS(fly.io) everytime you want to change the channels which are blacklisted
Describe the solution you'd like
Implementing a simple database (sqlite) to allow yourself to change the blacklisted channels via the bot and not having to restart the entire bot everything
Describe alternatives you've considered
N/A
Additional context
N/A

Try either a CSV or TXT file instead of sqlite. Might be simpler and easier for people to deal with

Issue closed with PR https://github.com/patelheet30/passel-hikari/pull/4


HBASE-25275 Upgrade asciidoctor

Upgraded asciidoctor-maven-plugin and asciidoctorj-pdf to the latest versions.
Followed the migration guide of asciidoctor-maven-plugin 2.0.0 migration guide.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 10s
Docker mode activated.

_ Other Tests _

+1 :green_heart:
asflicense
0m 21s
The patch does not generate ASF License warnings.

35m 41s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2647/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2647

Optional Tests
dupname asflicense hadoopcheck xml

uname
Linux 3f438abba065 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 57d9cae480

Max. process+thread count
63 (vs. ulimit of 30000)

modules
C: . U: .

 U: .

 U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2647/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Test failures are unrelated.


Steamvr crashes

i open alvr, steamvr boots, the quest 2 connects, then i get that error

Please read the troubleshooting page in the wiki.

Please read the troubleshooting page in the wiki.


Training/validation/testing details

First of all congratulations on being accepted to EMNLP!
I'm a GNN and hyperGNN learning enthusiast. I've been playing around with the code, but it remains unclear to me on how do you pick your best model. From what I understood, I suspect it has to do sth with the USE_TEST flag, but it's still unclear to me how exactly to use the flag. Currently, if I e.g. run it for 10 epochs, validating every five, with the USE_TEST: True, it will validate 2 times on the test data, right? (My reasoning is based on lines 298 and below, combined with loops/evaluation.py)
Now, imagine I want to run the training, save the best model (to do this I set SAVE: True)  based on validation data and then run on the test set, to obtain test set performance. (Alternatively, it's ok for me to run for X epochs and then do test on test set data right after it). If I set USE_TEST: False it doesn't seem it is not going to evaluate on the test set in the end (which is put in eval_vl_data) , whereas if I set the flag to True, eval_vl_data is set to contain test_data and thus will validate on test data every 5 epochs or so.
What is the correct set of commands I should be running? Do I need to modify the code?
P.S.
ev_tr_data seems to be never used outside of lines 298-305

Hi!

it remains unclear to me on how do you pick your best model

In simple words, it's designed in a way that you run a bunch of hyperparameters optimization experiments on validation with USE_TEST: False, and once you have the best hyperparams, run one final experiment on test with the USE_TEST: True flag.

Currently, if I e.g. run it for 10 epochs, validating every five, with the USE_TEST: True, it will validate 2 times on the test data, right?

Right!

I want to run the training, save the best model ... based on validation data and then run on the test set

The SAVE flag is rather for analyzing the model guts and weights after it has been trained and evaluated on the test set, ie, after you performed all the hyperparameter optimization. As of now, there is no functionality to save the best model and load it for the test run.

ev_tr_data seems to be never used outside of lines 298-305

This is a remnant from debugging to evaluate how a model overfits on a subset of training data, it is used in the training loop  https://github.com/migalkin/StarE/blob/f40a5ee082d61851477e9870c21e991c7d91deb3/loops/loops.py#L137
Right now the CLI param for it is disabled in the run.py, but you can uncomment it and play around.
https://github.com/migalkin/StarE/blob/f40a5ee082d61851477e9870c21e991c7d91deb3/run.py#L56
Usually training sets are large compared to validation/test, so it might be a long process :)

Thanks for the detailed explanation. So, if I understand correctly, it is the model at the end of training (always) that is picked for testing. E.g. you decide best hyper-params (incl. epochs to run for) on the validation dataset, and you run once on the test dataset and results reported in the end are for the last model?

Yes, this is correct.
We used wandb to track validation experiments, selected the best performing hyperparams and ran one final experiment on the test set with those hyperparams.


supabase-cli: 1.112.0 -> 1.120.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for supabase-cli is: A CLI for interacting with supabase
meta.homepage for supabase-cli is: https://github.com/supabase/cli
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.120.0 with grep in /nix/store/9p0cafsl60bl2rvxyr9iapj01049rayp-supabase-cli-1.120.0
found 1.120.0 in filename of file in /nix/store/9p0cafsl60bl2rvxyr9iapj01049rayp-supabase-cli-1.120.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
supabase-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9p0cafsl60bl2rvxyr9iapj01049rayp-supabase-cli-1.120.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A supabase-cli https://github.com/r-ryantm/nixpkgs/archive/d6a1bffff5e5a5be748a64bb8d4e0c2878dbfe00.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d6a1bffff5e5a5be748a64bb8d4e0c2878dbfe00#supabase-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9p0cafsl60bl2rvxyr9iapj01049rayp-supabase-cli-1.120.0
ls -la /nix/store/9p0cafsl60bl2rvxyr9iapj01049rayp-supabase-cli-1.120.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

supabase-cli

Maintainer pings
cc @Gerschtli @kashw2 for testing.

Closing in favor of https://github.com/NixOS/nixpkgs/pull/273836.


Add a presentation view to projects

Aspirational idea, not sure what repo this should live in. The current view has useful information:

I think it'd be neat if there was a "presentation mode" or some kind of toggle on this page that takes this information and presents it in a way more condusive to presentations.
Project name/logo, usage of QR codes, and other layout options to make it more condusive to be put on on a single summary page designed more as a billboard.
Use cases:

Booth screens and other digital signage at events
Handouts and other print material
Sharing the URLs via mobile, etc
Slide candy (good template for a final "Here's how to find us" slide

This would encourage projects to fill out their landscape entries with the proper metadata. They would match the same look and feel of the LFX dashboards so they look good together in a project presentation.
More!
There's also some nice autogeneration we could do. The projects team constantly gets questions asked like "hey which graduated projects represent runtime". Then one of us goes to the landscape, searches, ends up with something like this:

Then we copy and paste the logos into the slides, add a URL/QR Code, and then manually add the description of each project. Then we end up sharing slides, those get out of date, etc. Appropriately sized versions of these could fit multiple projects per slide, etc. TLDR: Basically this evolves into Project Social Cards. The digital signage that we use at Kubecon to congratulate Sandbox, Incubating, and Graduating could look really slick. 😄
Then the projects will run with it. I can imagine someone will make a stylized postcard with their team photo from KubeCon on one side and the project info on the other, etc.

Hi @castrojo
About the second part of this, perhaps you can share a link like this and so you avoid a out of date image. 🙂


Can't select dropdown menu in new GitHub UI

In new GitHub UI, dropdown menus are not active when hover, and can't click it.

Temporary workaround, remove:
    .dropdown-menu {
        z-index: 9;
    }


Feature/check auth in header

Adds a service for checking if the user is logged in:

gets token from local storage or url (if coming from a social login)
checks auth
cleans URL of token
manages state of My GFW link

I think it's a correct approach; we're doing a fetch when the component mounts and let the associated components re-render once the browser is available. componentDidMount and componentDidUpdate will be triggered only on the client to it seems like the right place to do these things 🤔

Updated! I think its the simplest approach 🌴


chore(rust): fix curl test case

Current Behaviour
Test fails as there is a redirect
   $ curl -s http://ockam.io/
   Redirecting to https://ockam.io/ (308)

Proposed Changes
Change the curl command to follow redirects

Checks

[X] All commits in this Pull Request are signed.
[X] All commits in this Pull Request follow the Ockam commit message convention.
[X] I accept the Ockam Community Code of Conduct.
[X] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/contributors repository.

Thanks @polvorin
I removed the test altogether, it seem unnecessary.


Mypy badge

This PR adds a mypy badge to the README.
Also, nicer alt texts for all other badges.

Nice, thanks @headtr1ck !


[Console Output] Change all file paths from relative to absolute

With #801 the report file paths are printed as absolute path - having the entire file path printed which makes it clickable from any place from the Terminal.
We should do the same for the findings so that they're always clickable from the terminal regardless in which directory you execute detekt.

Did anyone test clickable links in IntelliJ IDEA?
I'm using Idea 2018.1.3 (latest), gradle 4.6 and detekt RC7.
I'm launching detektCheck from intelliJ's gradle menu (not from terminal) and in Build window file paths are not clickable (sorry for blackboxing project-related info)

I think the terminal emulator just doesn't support it.

@vanniktech the issue is with Build window, not the terminal emulator. And paths from, for example, exceptions are clickable there.

Those are different things.
Exceptions are handled by the ide.

Okay. Are there plans to support this? Any related issues on jetbrains issue tracker?


run BestTrack on 120m mesh v2.2

see here: https://github.com/JaimeCalzadaNOAA/coastal-act/blob/updates/tasks/test_Mesh_120m_v22/best_track_storms/Makefile

current BestTrack JSON:
https://github.com/noaa-ocs-modeling/CoupledModelDriver/blob/4ff84fa9c9672dd30d5825a209f6c917acf82110/coupledmodeldriver/configure/forcings/base.py#L164-L202

this issue is related to #56

this command should work:
initialize_adcirc \
  --output-directory run_adcirconly_hsofs_250m \
  --mesh-directory /scratch2/COASTAL/coastal/save/shared/models/meshes/hsofs/250m/v1.0 \
  --modeled-start-time 20121022T060000 \
  --modeled-duration 04:05:00:00 \
  --modeled-timestep 00:00:02 \
  --tidal-spinup-duration 12:06:00:00 \
  --platform HERA \
  --adcirc-processors 600 \
  --adcirc-executable /scratch2/COASTAL/coastal/save/shared/repositories/ADC-WW3-NWM-NEMS/ADCIRC/work/padcirc \
  --adcprep-executable /scratch2/COASTAL/coastal/save/shared/repositories/ADC-WW3-NWM-NEMS/ADCIRC/work/adcprep \
  --modulefile /scratch2/COASTAL/coastal/save/shared/repositories/ADC-WW3-NWM-NEMS/modulefiles/envmodules_intel.hera \
  --job-duration 06:00:00 \
  --forcings tidal,besttrack \
  --tidal-source TPXO \
  --tidal-path /scratch2/COASTAL/coastal/save/shared/models/forcings/tides/h_tpxo9.v1.nc

the run at /scratch2/COASTAL/coastal/save/shared/working/zach/adcirc/run_20210413_besttrack_hsofs_250m seems to have run successfully, and timed out at hotstart as normal
Here is the code to convert the forcing from a BestTrackForcingJSON into an ADCIRCpy BestTrackForcing object:
https://github.com/noaa-ocs-modeling/CoupledModelDriver/blob/4ff84fa9c9672dd30d5825a209f6c917acf82110/coupledmodeldriver/configure/forcings/base.py#L186-L193
Then, add forcings to mesh:
https://github.com/noaa-ocs-modeling/CoupledModelDriver/blob/a7bf10e8e7f7fb4b8a9a9e164823335457fd72ea/coupledmodeldriver/configure/models.py#L201-L216

the time series of fort.63.nc seems to have an empty zeta:
fort63_dataset = Dataset('/scratch2/COASTAL/coastal/save/shared/working/zach/adcirc/run_20210413_hsofs_250m_besttrack/runs/run_1/fort.63.nc')
print(fort63_dataset['zeta'].shape)

(0, 1813443)

the surface output fort.63.nc seems to have an empty zeta:
fort63_dataset = Dataset('/scratch2/COASTAL/coastal/save/shared/working/zach/adcirc/run_20210413_hsofs_250m_besttrack/runs/run_1/fort.63.nc')
print(fort63_dataset['zeta'].shape)

(0, 1813443)

Perhaps the output frequency?
This should be accessible through driver.set_surface_elevation_output(*args, **kwargs).

updated the code setting the output frequency:
https://github.com/noaa-ocs-modeling/CoupledModelDriver/blob/4eea406e35369fbac53682bb8610e2609d8e2b4b/coupledmodeldriver/configure/models.py#L284-L346
fort61_dataset = Dataset(COLDSTART_DIRECTORY / 'fort.61.nc')
fort63_dataset = Dataset(COLDSTART_DIRECTORY / 'fort.63.nc')

print({
    variable_name: variable.shape
    for variable_name, variable in fort61_dataset.variables.items()
})
print({
    variable_name: variable.shape
    for variable_name, variable in fort63_dataset.variables.items()
})

{'time': (0,), 'station_name': (397, 50), 'x': (397,), 'y': (397,), 'zeta': (0, 397)}
{'time': (0,), 'x': (5792708,), 'y': (5792708,), 'element': (11467160, 3), 'adcirc_mesh': (1,), 'neta': (), 'nvdll': (1,), 'max_nvdll': (), 'ibtypee': (1,), 'nbdv': (273,), 'nvel': (), 'nvell': (513,), 'max_nvell': (), 'ibtype': (513,), 'nbvv': (119521,), 'depth': (5792708,), 'zeta': (0, 5792708)}

From /scratch2/COASTAL/coastal/save/shared/working/zach/adcirc/run_20210414_hsofs_120m_v2.2_besttrack/coldstart/fort.15
NSPOOL**
-5 12.25 12.25 1                                                ! NOUTE TOUTSE TOUTFE NSPOOLE

-5 12.25 12.25 1                                                ! NOUTV TOUTSV TOUTFV NSPOOLV

-5 12.250000 12.250000 1                                        ! NOUTGE TOUTSGE TOUTFGE NSPOOLGE
-5 12.250000 12.250000 1                                        ! NOUTGV TOUTSGV TOUTFGV NSPOOLGV

Check you values for start, end and steps: TOUTS, TOUTFE, NSPOOLE
https://adcirc.org/home/documentation/users-manual-v53/parameter-definitions#TOUTSE
On Thu, Apr 15, 2021 at 8:40 AM Zachary Burnett @.***>
wrote:

From
/scratch2/COASTAL/coastal/save/shared/working/zach/adcirc/run_20210414_hsofs_120m_v2.2_besttrack/coldstart/fort.15
NSPOOL**
-5 12.25 12.25 1                                                ! NOUTE TOUTSE TOUTFE NSPOOLE
-5 12.25 12.25 1                                                ! NOUTV TOUTSV TOUTFV NSPOOLV
-5 12.250000 12.250000 1                                        ! NOUTGE TOUTSGE TOUTFGE NSPOOLGE
-5 12.250000 12.250000 1                                        ! NOUTGV TOUTSGV TOUTFGV NSPOOLGV
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/noaa-ocs-modeling/CoupledModelDriver/issues/77#issuecomment-820390788,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ3T4UMRAWNIHQT2QYNKYYLTI3NERANCNFSM423O6MLA
.

--
Jaime R. Calzada
UCAR
Marine Modeling and Analysis Branch
National Ocean Service / Office of Coast Survey Development Laboratory
National Oceanic and Atmospheric Administration
1315 East West Highway, N/CS13
Silver Spring, MD 20910-3282
phone:<PHONE_NUMBER> x 119 <%28301%29%20713-2809%20x103>
mobile:<PHONE_NUMBER> <%28201%29%20539-1679>

blocked by https://github.com/JaimeCalzadaNOAA/adcircpy/issues/76

issue solved by https://github.com/JaimeCalzadaNOAA/adcircpy/pull/77

added fort.22 reading with https://github.com/JaimeCalzadaNOAA/adcircpy/pull/78

run completed successfully at /scratch2/COASTAL/coastal/save/shared/working/zach/adcirc/run_20210416_hsofs_120m_v2.2_besttrack


Add license to footer

Change "Kubernetes", which is not a thing you can assign copyright to, to "The Kubernetes Authors", in line with comments in source files.  This may or may not be more appropriate to assign copyright to CNCF.
Add link to LICENSE with CC BY SA text, which will only be true when #1763 is merged.
I don't think you'll like the CSS, but without the 'light-text' tag you get a blue/purple link.

This change is

Docs LGTM, but waiting on lawyercats.

+1. CNCF requires trademark to be signed over, but copyright can still be held by the authors. In Google's case, they also assigned the copyright, but that's not impacted by saying The Kubernetes Authors.


Flex 3857 add firmware version property

All is well.

Tested using my development environment. Works as expected.


notify if eth0 not found

@ibuildthecloud

I don't think this actually provides much value as that message won't be echo'd on the screen

I'd really like to know why udev doesn't detect the device.  Have you found out why that is?


ontogpt in runner, change from os.system to direct python import

It works now, but it would be better to go from os.system("ontogpt...") to from ontogpt... import run_multilingual_analysis and then use it as a python function. See here:
https://github.com/monarch-initiative/malco/blob/0c156496ba7838c70ac62260772813080e87157a/src/malco/run/run.py#L19-L30

We changed this by now. See
https://github.com/monarch-initiative/malco/blob/af9edf389c3b3c588ae1b69ee10ae79fa1913727/src/malco/runner.py#L30-L34


Generalize util method for loading class from service loader

Why are the changes needed?

Generalize util method for loading class from service loader in kyuubi-util-scala module

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Thanks, merged to master.


Include timeout param in client example

Fixes #77.

Codecov Report

Merging #78 (30ff187) into master (6685093) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #78   +/-   ##
=======================================
  Coverage   81.20%   81.20%
=======================================
  Files           5        5
  Lines         250      250
=======================================
  Hits          203      203
  Misses         47       47

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 6685093...30ff187. Read the comment docs.

Codecov Report

Merging #78 (30ff187) into master (6685093) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #78   +/-   ##
=======================================
  Coverage   81.20%   81.20%
=======================================
  Files           5        5
  Lines         250      250
=======================================
  Hits          203      203
  Misses         47       47

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 6685093...30ff187. Read the comment docs.

Thanks for the contribution! If you're doing a lot of data publishing Socrata maintains a separate publishing API which may be better suited for your usecase: https://socratapublishing.docs.apiary.io/

Thanks for the contribution! If you're doing a lot of data publishing Socrata maintains a separate publishing API which may be better suited for your usecase: https://socratapublishing.docs.apiary.io/

@xmunoz I publish loads and loads of data. Really appreciate the tip—thanks!

@xmunoz I publish loads and loads of data. Really appreciate the tip—thanks!


Warning that language server is running

Result from CocInfo
## versions

vim version: NVIM v0.4.4
node version: v15.5.0
coc.nvim version: 0.0.80-7642d233d6
coc.nvim directory: /home/hugo/.homesick/repos/dotfiles/home/.config/nvim/plugged/coc.nvim
term: alacritty
platform: linux

## Log of coc.nvim

2020-12-27T11:46:38.887 INFO (pid:539717) [extensions] - Skipped load vim plugin from "/home/hugo/.config/nvim/plugged/coc-jedi", "coc-jedi" already global extension.
2020-12-27T11:46:39.192 INFO (pid:539717) [services] - registered service "jedi"
2020-12-27T11:46:39.196 INFO (pid:539717) [plugin] - coc.nvim 0.0.80-7642d233d6 initialized with node: v15.5.0 after 350ms
2020-12-27T11:46:40.212 INFO (pid:539717) [services] - jedi-language-server state change: stopped => starting
2020-12-27T11:46:40.217 INFO (pid:539717) [language-client-index] - Language server "jedi" started with 539794
2020-12-27T11:46:40.384 INFO (pid:539717) [services] - jedi-language-server state change: starting => running
2020-12-27T11:46:40.391 INFO (pid:539717) [services] - service jedi started
2020-12-27T11:48:12.901 INFO (pid:539717) [attach] - receive notification: showInfo []

Describe the bug
When using coc-jedi, coc shows a warning that the language server is running. This warning seems odd -- there's nothing wrong with that, and seems like an "everything is alright" warning.
This pops up every time I open a python file:
[coc.nvim] jedi: running "/home/hugo/.homesick/repos/dotfiles/home/.config/coc/extensions/node_modules/coc-je
di/.venv/bin/jedi-language-server"
Press ENTER or type command to continue

Reproduce the bug
I used this config for reproducing the bug:
set nocompatible
set runtimepath^=/home/hugo/.config/nvim/plugged/coc.nvim
set runtimepath^=/home/hugo/.config/nvim/plugged/coc-jedi.nvim
filetype plugin indent on
syntax on
set hidden

Screenshots (optional)

In your coc-settings.json:
 "jedi.startupMessage": false,

This will disable the startup message.

Hmm... I see. Maybe the default could be changed though?
E.g.: Showing a warning when that everything is okay seems like an oxymoron. This also pops up every time a file is opened -- I can't see why this would be desirable, unless debugging coc-jedi itself.

Resolved in 188a0f40acfd204e4eadac562d4d7f9a29dd18f6

Resolved in 188a0f40acfd204e4eadac562d4d7f9a29dd18f6

Sorry for lateness, I just returned from some time off

Sorry for lateness, I just returned from some time off

Thanks!

Thanks!


Fix polygon render bug

The following leaves an unfilled triangle:
val p1 = RenderedMesh(
      shader = shader,
      transform = Matrix3
        .transform(
          translation = Vector2(
            cos(1.7f * Time.current + 5f) * .2,
            sin(3f * Time.current + 3f) * .11
          ),
          rotation = cos(.8f * Time.current + 3f) * 2f * pi,
          scale = Vector2(1f, 1f) * (.2f + .1f * (cos(.8f * Time.current) + 1))
        ),
      mesh = Mesh(
        Polygon(
          Vector(
            Vector2(0, 0),
            Vector2(1, 0),
            Vector2(1, 1),
            Vector2(0, 1),
            Vector2(0, .8),
            Vector2(0.35, 0.5),
            Vector2(0, .2)
          ).map(v => Vector2(v.x - .5, v.y - .5))
        )
      ),
      tint = Color.YELLOW * .75
    )

I assume this happens since the first and last line segments found distinct valid points, and their triangles don't share an edge.
Screenshots:

The rest of the code can be found in the 9ee9e32 commit.


Graphviz - Misleading Error Message when failure generating schema due to incorrect column name

Steps to reproduce
Sample Notebook Code:
import atoti as tt
import pandas as pd
session = tt.Session()
table = session.read_pandas(pd.DataFrame({
"&": ["a"]
}), table_name="test")
session.tables.schema

Actual Result
The schema generation fails with the following error:
Missing Graphviz library which is required to display the graph. It can be installed with Conda: conda install graphviz or by following the download instructions at https://www.graphviz.org/download/.
This message is misleading as graphviz even if it is properly installed it will throw this error instead of indicating that the generation of the graph has failed...

Expected Result
A proper error message should be produced to indicate the generation failure
Environment

atoti: 0.6.5
Operating System: WinOS
Browser: NA

Hi !
You're right, I do reproduce this misleading error message.
It looks like it comes from the use of the ampersand.


color scheme convertor - vim to jedit

colorscheme guy: ok, you convinced me, i'll put that in my todo list

Just a reminder from our chat. Make it convert from Vim to jEdit.
Also maybe you can make it convert Vim to Komodo Edit.
Heck, convert between them all and everyone on the internet on the
whole planet will come to your site!  I will make a donation if you can
do this.

It should be simple enough with an OOP approach...


Allow negative x and y values

I really like this library, but it would be cool if negative x and y values were supported.

Hi Kevin, thanks for the endorsement ;).
Indeed it would be great to have this feature !
Unfortunately I am right now on vacation in Japan for three weeks and I have limited time to work on it (& crappy wifi).
Anyway I will try to take some time for a quick fix, otherwise it is gonna be done when I return.

Thank you! Don't pressure yourself, enjoy your vacation. I only use this library right now for a proof-of-concept.

Finally found some time to work on it, should be ok 👍.

Seems that there are still issues when constructing a quadtree with negative x and y values.

Ah sorry I misunderstood.
I thought you wanted negative x/y coordinates for the elements, not the quadtree itself.
I just pushed a fix which removes the constraint.


Improve web GUI error reporting with invalid raw_csr

How to reproduce
Open a raw_csr dataset with an indptr file size that is invalid for the dtype specified in the sparse.toml file in the web GUI.
What happens
The web GUI closes the file browsing/opening view without an error message. On the console libertem-server shows the following:
2023-07-12 11:11:50,602 - distributed.worker - WARNING - Compute Failed
Key:       get_nav_size-af4fd8dd-67d5-4c39-b136-7ca9f070faec
Function:  get_nav_size
args:      ()
kwargs:    {}
Exception: "ValueError('Size of available data is not a multiple of the data-type size.')"

2023-07-12 11:11:50,603 - tornado.application - ERROR - Uncaught exception GET /api/datasets/detect/?path=%2Fdata%2Ftest%2FMeasurement_jul_12_2023_10h26m07s%2Fimage%2Fsparse0.toml (<IP_ADDRESS>)
HTTPServerRequest(protocol='http', host='localhost:9000', method='GET', uri='/api/datasets/detect/?path=%2Fdata%2Ftest%2FMeasurement_jul_12_2023_10h26m07s%2Fimage%2Fsparse0.toml', version='HTTP/1.1', remote_ip='<IP_ADDRESS>')
Traceback (most recent call last):
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/tornado/web.py", line 1786, in _execute
    result = await result
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/libertem/web/dataset.py", line 147, in get
    detected_params = await sync_to_async(
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/libertem/common/async_utils.py", line 58, in sync_to_async
    return await loop.run_in_executor(pool, fn)
  File "/usr/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/libertem/common/async_utils.py", line 27, in _wrapped
    return f(*args, **kwargs)
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/libertem/io/dataset/__init__.py", line 232, in detect
    params = cls.detect_params(path, executor)
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/libertem/io/dataset/raw_csr.py", line 252, in detect_params
    image_count = executor.run_function(get_nav_size, descriptor=descriptor)
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/libertem/executor/dask.py", line 522, in run_function
    return future.result()
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/distributed/client.py", line 327, in result
    raise exc.with_traceback(tb)
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/libertem/io/dataset/raw_csr.py", line 438, in get_nav_size
    indptr = np.memmap(
  File "/home/asi/envs/libertem-venv/lib/python3.10/site-packages/numpy/core/memmap.py", line 239, in __new__
    raise ValueError("Size of available data is not a "
ValueError: Size of available data is not a multiple of the data-type size.

What should happen
The error message is shown as an error in the web GUI and the user can continue browsing.

The error message is shown as an error in the web GUI and the user can continue browsing

Rough plan:
This means that detect_params needs to be able to throw a specific exception type, which signals "this is the correct data set type for this file, but I can't handle it for some other reason". If we just return False, we will get the open dialog with RAW selected as type, which can be confusing for users.
Dataset implementations then need to be updated to catch specific errors - this may be something we do over time to improve the user experience, whenever we catch these issues. For raw_csr this means after successfully loading the toml file, any exceptions need to be converted to the new exception type.
Then, this exception type needs to be propagated as a custom message through the web API, similar to the current DATASET_DETECTION_FAILED message.
Then, this message needs to be handled in the client, basically adding a variant to the DetectDatasetResponse, which then needs to be converted to an appropriate error message in the doDetectDataset saga.


Favourites tab

Add categories for favourites tab
ie:
All Favourites - List all of the favourites from all pages
Converter Favourites
Deconvolution Favourites etc

Alternative is to allow users to create and rearrange folders in the bookmarks tab so they can store bookmarks where they like to tidy up the interface and find locations easily

bookmarks are not visible outside tab they are created
filters, sorting, subfolders
provide advice on effort/priority


Support for isNarration element

I can't find SDK class for isNarration element
This element widely used in OfficeMix extension.
Can you help me? May be have some workaround (except direct xml reading)?
Thanks

If there are no strongly typed classes, you would have to resort to XML, e.g., using Linq to XML as in the PowerTools for Open XML.


Large stack frames cause the listing to lag

Describe the bug
Large function stack frames, about 0x13f0 in size and roughly 1276 types, cause the listing to lag while scrolling.
Expected behavior
Gotta go fast
Environment (please complete the following information):

OS: Microsoft Windows [Version 10.0.19041.508]
Java Version: 15
Ghidra Version: 9.2_DEV f7f366b261589a65696c17852812566b9e6659bb (sorta)

Additional context
There really isn't much I can provide for this. I've only encountered it in large C++ programs that are subject to copyright.
Edit: Exactly how large of a function you ask?
Exception while decompiling xxxxxxxx: Decompiler results exceeded payload limit of 50 MBytes

We have seen the behavior before when a function has an excessive number of xrefs.   In your case, with the large stack frame, I'm assuming your have many xrefs for the stack variables.  Is that the case?
If you have the time, energy and desire, it would be helpful to get some insight into what the slowness is.  I examine issues like this using visualvm that comes with the jdk.    I would start Ghidra and visualvm.  Once you connect to Ghidra in visualvm, then trigger the slow scrolling and see where visualvm indicates the time was consumed.

We have seen the behavior before when a function has an excessive number of xrefs. In your case, with the large stack frame, I'm assuming your have many xrefs for the stack variables. Is that the case?
If you have the time, energy and desire, it would be helpful to get some insight into what the slowness is. I examine issues like this using visualvm that comes with the jdk. I would start Ghidra and visualvm. Once you connect to Ghidra in visualvm, then trigger the slow scrolling and see where visualvm indicates the time was consumed.

Unfortunately visualvm doesn't support jdk15 yet but I found something else that I could get a free trial for.
Apparently this is where the problem starts.
https://github.com/NationalSecurityAgency/ghidra/blob/3c683ae770cf495deb32588202b00b88e125a4a4/Ghidra/Features/Base/src/main/java/ghidra/app/util/viewer/format/FieldFormatModel.java#L502
Most of the slow down is being caused by db.KeyToRecordIterator.next() with most of the time being spent here
https://github.com/NationalSecurityAgency/ghidra/blob/3c683ae770cf495deb32588202b00b88e125a4a4/Ghidra/Framework/DB/src/main/java/db/Schema.java#L268
You could create a corresponding Supplier<Field>[] which would remove any overhead of repeatedly constructing it via reflection. Alternatively you could also get away with a static map which would map each Field Class to its corresponding new.
Call-tree-–-By-thread.html.txt

Thanks for the dump.  It shows what I suspected, that the xrefs are doing too much work for large functions (look at the section around the one you mentioned, where it points to the VariableXRefHeaderFieldFactory.  This is loading all symbols to figure out how to display xrefs in the function header.  The problem with the code is that it does not place any limitation on the number of symbols retrieved.
It needs to be smarter.  For example, it could understand that the UI is set to display a limited number of xrefs, depending upon the option, and to then stop loading symbols at that point.  For example, if the option indicates to show 5 xrefs, then stop loading xrefs after 5 are reached.   This may not be enough if the volume of the stack members is huge.  Some sort of caching may also be required.
We haven't fixed this yet, as it only affects a small number of binaries, such as what you are using.

It would be nice if the amount of the numbers in the stack frame which gets displayed is limited as well. It is about as fun as you could imagine to scroll through.
I did do the following test just to have a look and while it did improve I'm not entirely certain if it is worth making the change to it. This was done about as lazily as possible.
private static Map<Class<?>, Supplier<Field>> fieldMap;

static {
	fieldMap = new HashMap<>();
	fieldMap.put(BinaryField.class, BinaryField::new);
	fieldMap.put(BooleanField.class, BooleanField::new);
	fieldMap.put(ByteField.class, ByteField::new);
	fieldMap.put(IntField.class, IntField::new);
	fieldMap.put(LongField.class, LongField::new);
	fieldMap.put(ShortField.class, ShortField::new);
	fieldMap.put(StringField.class, StringField::new);
}

private static Field newField(Class<?> clazz) throws Exception {
	Supplier<Field> supplier = fieldMap.get(clazz);
	return supplier != null ? supplier.get() :
		(Field) clazz.getDeclaredConstructor().newInstance();
}

I then replaced all uses of Class.newInstance with the new method.
While it's not a crazy amount it is still a speed boost of around 80% on a bit of code used everywhere in Ghidra.
Live-call-tree.html.txt

I created a fork and invited you to examine changes related to this ticket in a branch named 'GH-2351-dragonmacher-slow-listing-fix'

I created a fork and invited you to examine changes related to this ticket in a branch named 'GH-2351-dragonmacher-slow-listing-fix'
Please try the changes in your problem binary.

Will do.

It is good. I ran the profiler in case there was a measurable problem somewhere but I don't see anything that stands out. I also held the pg dn key for a good 5 minutes and passed a handful of stack frames which were several times larger than the one I was initially using to test the issue and had no problem. I've attached that data just because.
While not related I came across a list of thrown exceptions and happened to notice that docking.DockingActionProxy.createMenuItem(boolean) DockingActionProxy.java:178 had thrown 3298 NoSuchFieldException 2442 of which were from docking.menu.MenuBarManager.getMenuBar() MenuBarManager.java:142. It's not relevant but am bringing it up incase it should be looked into.
Live-call-tree.html.txt
Live-call-tree.csv.txt
Held
Live-call-tree-held.zip
Live-call-tree-held.csv.txt

Thanks for testing.  This will fix a long-standing issue.

Thanks for testing. This will fix a long-standing issue.

No problem. Also don't bother looking into the exceptions. They are from BasicLookAndFeel and is Oracle's problem.


class Psy\Input\ShellInput not found

When i execute php vendor/scil/laravel-fly/bin/fly start, the ternimal display error with
Fatal error: Uncaught Error: Class 'Psy\Input\ShellInput' not found in /www/wwwroot/zdd/vendor/scil/laravel-fly/src/LaravelFly/Server/Traits/Console.php:38

This class is in package psy/psysh which is required by laravel/tinker

thanks for you reply.


Support EndpointSlices in AntreaProxy

Describe the problem/challenge you have
Starting with K8s 1.19, EndpointSlices are enabled by default (https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.19.md#endpointslices-are-now-enabled-by-default) and used by kube-proxy to implement ClusterIP Services and increase scalability. We should consider supporting EndpointSlices in AntreaProxy as well. Since we still want to support older K8s versions, we could introduce a feature gate (see EndpointSliceProxying for kube-proxy).

If no one is working on it, I would like to take this.

Fixed by #1703


Can't see followers

Steps
Follow a user:

On the followed account you can't see the follower

Could you please give me the error from console browser and if it is your local?

Its on local, if I click follow I get this error:

Thanks! I hope this change work now.

Thx 4 quick fix


Row actions overflow update

Closes #1087
Summary

Added the option to specify if an item in rowActions has a divider or is a delete type.

Change List (commits, features, bugs, etc)

Added hasDivider and isDelete prop to OverFlowMenuItem in RowActionsCell
Added hasDivider amd isDelete to default/propTypes
Added a few more elements to rowActions to incorporate the new attributes

Acceptance Test (how to verify the PR)

Add a new element to rowActions in TableBodyRow story and set hasDivider and/or isDelete to true to see the new appended styles

Do we need to expose this functionality through Table/StatefulTable? @bryancboyd

Good catch @tay1orjones.  @rawa-resul we'll need to also update the Table props to allow passing in these additional props on a table row.
To validate the design, I'd suggest updating the with row expansion and actions Table story to use the new separator and delete option:

@tay1orjones @bryancboyd
Thanks for pointing that out. I have added the required props in Table.story.jsx to include hasDivider and isDelete type. The changes should be visible in:

Stateful Example with expansion and column resize
Stateful Example with row nesting and fixed columns
With row expansion and actions
Stateful Example with I18N strings

@tay1orjones My bad, I completely missed the RowActionPropTypes. I have added these now.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Live Reloading Support for zcli apps:server

Feature
While you are able to reload the app inside the Zendesk product which will incorporate changes you have made to your app, this doesn't take in changes made to the manifest.json file. It would be good to be able to live reload changes to the manifest as well as the rest of your app assets and reload the app in browser automatically without having to restart the development server and reload the app in product similar to this
Issue details

Command: zcli apps:server
Version: @zendesk/zcli/1.0.0-beta.8 darwin-x64 node-v14.17.1
OS: macOS Big Sur 11.4 (20F71)

The enhancement fix is in PR https://github.com/zendesk/zcli/pull/90 and merged into master.


add null check for keyboardFrame variable in KeyboardAvoidingView component

This is a critical change and should be pushed to stabel asap
The reason for the pull request is to prevent crashes that can occure if the function relativeKeyboardHeight(keyboardFrame: ScreenRect) got null as an input variable which sometimes does happen.
Test plan: manually verify the crash no longer happens (done).

@facebook-github-bot shipit

@facebook-github-bot shipit


DO NOT MERGE: Revert "initial commit"

This reverts commit e23aee70e82f224eab6529506799ee9d94929673.
This is just to give us a diff to hang review comments off of, I'll fix them with followups.

Fixes applied


Support for synchronous functions

Suffixed with Sync.
Can be really convenient for some scripting scenarions.
Naive implementation to begin with later on we can iterate with something more optimized.
Applicable to:

SqlCommand (execution)
Transaction Helpers
TransactionScope Helpers: food for thought => btw, are we supposed to ship it in a different package knowing that TransactionScope still suffer from major shortcomings since .NET Core era and onwards (see https://github.com/dotnet/runtime/issues/715)

Done


Return HttpResponse Message from requests, remove exception throwing

Instead of throwing an exception on a bad HttpResponseMessage status, the response message is now returned to the caller, who can check the status and act appropriately. This is needed because the thrown exception for bad http status does not include the status code in an format except as part of a description string.
This change thus allows the callers to more easily determine the status of the request, and to act more appropriately on that statues (retry request, prompt user for reauth, throw an exception, etc).

Signed off.  Looks good!


[modules] Add metrics-scraper module

Remaining task is the documentation.

a few notes on the setup but overall ok, can also be worth documenting in the same pr?

Yes I need to add the documentation, will do in the same PR.


Flow to TypeScript conversion

Description
Project wide conversion from Flow to TypeScript.
Related issues
Closes #448
Type of change

[x] New feature (non-breaking change which adds functionality)

Changes Made

TS configurations (in eslintrc.json and tsconfig.json)
ESLint configurations with TS
Converted files to .ts and .tsx extensions
Installed necessary types
Use redux useSelector and useDispatch as custom typed methods (useAppSelector, useAppDispatch)
Module augmentations for MUI Theme and Palette to enable custom properties
Improved types throughout project tree
Removed Flow related configurations, including workflow for static types check. This is handled via eslint
Cypress TS configuration
Introduced types for custom Cypress commands
Updated README.md
Introduced TSC check in Husky pre-commit hook and GitHub Actions workflow

Testing

[x] Tests do not apply

a comment while i review this, is it possible or useful to use the tsc (https://www.typescriptlang.org/docs/handbook/compiler-options.html) to do typechecks e.g. https://github.com/typescript-eslint/typescript-eslint/blob/main/package.json#L48 so that we can implement that as part of a github action ?

I think this a great idea and in addition should be added to Husky pre-commit hooks. I'll try to suggest both workflow and GH action workflow into this PR.

a comment while i review this, is it possible or useful to use the tsc (https://www.typescriptlang.org/docs/handbook/compiler-options.html) to do typechecks e.g. https://github.com/typescript-eslint/typescript-eslint/blob/main/package.json#L48 so that we can implement that as part of a github action ?

I think this a great idea and in addition should be added to Husky pre-commit hooks. I'll try to suggest both ~workflow~ pre-commit hook and GH action workflow into this PR.

Husky pre-commit hook and GH action workflow now introduced.

I still have a question: Why do we need to put dispatch as [dispatch] in useEffect, for e.g. in App.tsx, instead of [ ] as usual ? Any documents regarding to this ?

Good call.
Empty second arguments on useEffect Hook is against react-hooks/exhaustive-deps eslint rule. In recommended eslint configuration we'd have this rule to warn on improperly used hooks.
I had this rule configured at the beginning of conversion but I decided to leave it off at the moment to help finish the conversion faster.

there are a lot of types as any, but I assume we can revisit that.

This was on purpose to save time. I think we could introduce eslint rule no-explicit-any and type all instances of any. This however can be time consuming and I suggest this to be done on different PR.

This was on purpose to save time. I think we could introduce eslint rule no-explicit-any and type all instances of any. This however can be time consuming and I suggest this to be done on different PR.

that what is what i implied as well.
also sry did not realised me merging the dependencies cause package lock to get conflicts

also sry did not realised me merging the dependencies cause package lock to get conflicts

No prob.

I continued on this PR as established wiht @saulipurhonen in RC

seems pressing the Submit Study button does not work for some reason :man_shrugging:

~seems pressing the Submit Study button does not work for some reason 🤷♂~
FormSelectField seem to be broken

--> Fixed for FormSelectField, the function responsible for changing fieldValue was broken in this branch
The PR is in quite a good shape now, there are still some issues left as mentioned: types as any, and enable eslint rule react-hooks/exhaustive-deps again, but I think this could be done separately in another PR.

then I will click merge and we can see what happens.


Menu without text

Hi,
Very often when accessing Dropbox menu there is no text in it. See the screenshot. I'm using Loki. Interface Russian.

I don't think this issue has anything to do with this project, it's best to report it directly to Dropbox developers.


[firebase_database] Order by key/ Order by child is not working properly

Steps to Reproduce
I am using Firebase Realtime Database for one of my flutter application. I've fetch many user data as a list and build a listview. The problem is that when i apply 'Order by key' or 'order by child' , the list stays same as before (random order).
Here is the code
DatabaseReference reference = FirebaseDatabase.instance.reference();
allData.length == 0
    ? reference
        .child('teachers')
        .orderByKey()
        //.orderByChild('index')
        .once()
        .then((DataSnapshot snap) {
        var keys = snap.value.keys;

        var data = snap.value;
        allData.clear();

        for (var key in keys) {
          TeachersData d = TeachersData(
              data[key]['bio link'],
              data[key]['email'],
              data[key]['name'],
              data[key]['phone'],
              data[key]['profile link'],

              data[key]['title']);
          allData.add(d);

        }

Here the screenshot of my database

@rakib205
The issue at https://github.com/flutter/flutter/issues/36661 has been closed and moved here. Future collaboration on this issue will be done here.

any progress?

Any updates please?

Looks like a duplicate of #753.

This appears to be a duplicate of #753 - I'm going to go ahead and close this issue so we can track on #753.


Add "Hello world" in ActionScript programming language.

I would love to take up this issue, can you pleas assign to this me


Saving in database issue

when I am using this statement to send messages to client side to specific connection ids :
await _chatHubContext.Clients.Clients(connectionsIds).SendMessageToAgent(message);
it's not saving any data in the database
but when use this will all :
await _chatHubContext.Clients.All.SendMessageToAgent(message);
it's saving data in the database, why this issue and how to solve it, here is my configuration :
services.AddSignalR()
.AddSqlServer(options =>
{
options.ConnectionString = Configuration.GetConnectionString("DefaultConnection");
options.AutoInstallSchema = true;
options.AutoEnableServiceBroker = true;
})
.AddJsonProtocol(options => { options.PayloadSerializerOptions.DefaultIgnoreCondition = JsonIgnoreCondition.WhenWritingNull; });

This is intended - messages sent directly to a specific connection, if that connection is connected to the server that is sending the message, are sent directly to that connection and do not need to transit through the database.


Disabled input

Is it possible to disable text or number input?
It would be very useful to create read-only fields, and to enable/disable them as needed.
Maybe via InputParamsOption with optional "enabled" flag?

You can use addMonitor to add read-only fields. https://cocopon.github.io/tweakpane/monitor.html

That will work. Thanks!


If an agent is reused, the Ontrack config will already exists

In this case, we want the existing configuration to be ignored and overridden. This might need a change in the Ontrack CLI (additional option to force a configuration to be overridden).

See nemerosa/ontrack-cli#4


Add Worker Pool concurrency.go

hi, I optimized this code by adding a worker pool. Previously, you called a separate goroutine for each task. When there are a large number of tasks, this approach consumes too many machine resources. The best approach in this case is a worker pool, which limits the number of simultaneously working goroutines.

if you want to know more, read https://doziestar.medium.com/effortlessly-tame-concurrency-in-golang-a-deep-dive-into-worker-pools-31f6ed9f2872


New path for drawing a signaling pathway

Current drawing paths only explain metabolic pathways, and we have some specific tasks that are relevant for other types of pathways like signaling pathways (listed below). But it would be good to expand these instructions into a full path relevant for signaling pathways, as they are a big part of WP and current instructions don't explain some aspects that are critical to this type of pathway.
Relevant tasks:
https://academy.wikipathways.org/stages/draw-transcriptiontranslation/index.html
https://academy.wikipathways.org/stages/draw-protein-phosphorylation/index.html
https://academy.wikipathways.org/stages/draw-graphical-line/index.html

Potential target pathways (PFOCR):
https://pfocr.wikipathways.org/figures/PMC11202427__genes-15-00798-g001.html
https://pfocr.wikipathways.org/figures/PMC10889086__ijms-25-02208-g001.html
https://pfocr.wikipathways.org/figures/PMC11079208__fcell-12-1390704-g003.html
Instead of adding a new track, the "Setting Pathway Information Free" track will be updated, since the "My First Metabolic Pathway" already covers metabolic pathway content.

https://pfocr.wikipathways.org/figures/PMC10953212__40001_2024_1708_Fig2_HTML.html