dispatch strategy behaves different than Dispatch itself

When using Dispatch the opened split/window closes once the given tests ran without errors. Dispatch also opens a quickfix split if the test suite threw any errors. The dispatch strategy does neither of those things.
Is this intended and if yes how can I imitate that behavior in vim-test?
Thanks.

That's strange, the dispatch strategy calls :Dispatch <cmd> directly (as you can see here), so it should behave exactly like calling it directly.
What is the behaviour you're getting? Is there some indication that the command has been run at all? Also, are you by any chance using MacVim or GVim? Because in that case a split is open in the Terminal directly and you won't be able to see it, although the quickfix is still opened.

After testing again I realized I actually do get a quickfix split. But it opens regardless of if there were any failing tests or not.
I have no idea about Vimscript but I always thought Dispatch checks the exit code of the rspec/test command and decides of opens the quickfix window. Could this be the problem here?

Yeah, I think it does do that. When you run your test command with :Dispatch directly, does it behave differently than when you run it through vim-test?

Yes: :Dispatch opens a tmux split, runs the given tests and closes the tmux split without opening the quickfix window.
:TestFile opens a tmux split, runs the given tests, closes the tmux split and opens a quickfix window containing the complete test suite output.

When you run :TestFile, try running :echo g:test#last_command, and see if it matches the command that you ran when using :Dispatch directly, otherwise try running :Dispatch with that command and see if you still get different behaviour.

:echo g:test#last_command returns zeus rspec spec/controllers/ticketings_controller_spec.rb. Running :Dispatch zeus rspec spec/controllers/ticketings_controller_spec.rb behaves exactly like :TestFile (opens quickfix).
I Dispatch seems to run !zeus rspec spec/controllers/ticketings_controller_spec.rb, too. It doesn't open a quickfix though.

This should probably then be fixed in vim-dispatch, because it should detect that it's running rspec from the command string. Or you just need to debug by looking at exit statuses, although it probably isn't related to that. I don't know why it has this behaviour, but it definitely isn't a bug in vim-test.

Maybe I'm a little late to the discussion, but @chrkau and @janko-m, I hit the same thing.  I think the problem is with @charkau's vim-dispatch configuration.  @chrkau, have you tried adding to your vimrc:
let g:dispatch_compilers = {'zeus': ''}

For more on dispatch_compilers see:
https://github.com/tpope/vim-dispatch/blob/master/doc/dispatch.txt#L53-L61


convert one-channel images (depth images) into video?

Can we convert depth frames into video?
I am using this for color images:
process = (
    ffmpeg
    .input('pipe:', framerate=15, format='rawvideo', pix_fmt='bgr24', s='640x480')
    .output('videos/movie264.mp4', pix_fmt='nv21', **config)
    .overwrite_output()
    .run_async(pipe_stdin=True)
)

How should we set format and pix_fmt?
The depth image only has one channel.

I've used a pix_fmt of gray16be which is a 16-bit mono image


[BUG] Stoix install fails with python 3.12

Describe the bug
Stoix fails to install in a clean conda environment
To Reproduce
conda create -n stoix python
...
pip install -e .

Results in
...
Collecting arch==5.0.1 (from rliable->id-marl-eval@ git+https://github.com/instadeepai/marl-eval->Stoix==0.0.1)
  Using cached arch-5.0.1.tar.gz (937 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [35 lines of output]
      <string>:10: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
      Compiling ./arch/univariate/recursions.pyx because it changed.
      Compiling ./arch/bootstrap/_samplers.pyx because it changed.
      [1/2] Cythonizing ./arch/bootstrap/_samplers.pyx
      [2/2] Cythonizing ./arch/univariate/recursions.pyx
      Traceback (most recent call last):
        File "/Users/smorad/miniforge3/envs/stoix/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/Users/smorad/miniforge3/envs/stoix/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/smorad/miniforge3/envs/stoix/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/vb/d133zwt15qn51k8m9h7b_mgm0000gn/T/pip-build-env-dv0nlsmd/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 325, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/vb/d133zwt15qn51k8m9h7b_mgm0000gn/T/pip-build-env-dv0nlsmd/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 295, in _get_build_requires
          self.run_setup()
        File "/private/var/folders/vb/d133zwt15qn51k8m9h7b_mgm0000gn/T/pip-build-env-dv0nlsmd/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 487, in run_setup
          super().run_setup(setup_script=setup_script)
        File "/private/var/folders/vb/d133zwt15qn51k8m9h7b_mgm0000gn/T/pip-build-env-dv0nlsmd/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 210, in <module>
        File "<string>", line 151, in run_setup
        File "/private/var/folders/vb/d133zwt15qn51k8m9h7b_mgm0000gn/T/pip-install-wm4lwdk0/arch_ecf8e1f1145d44bf9d790d077f996b93/versioneer.py", line 1518, in get_version
          return get_versions()["version"]
                 ^^^^^^^^^^^^^^
        File "/private/var/folders/vb/d133zwt15qn51k8m9h7b_mgm0000gn/T/pip-install-wm4lwdk0/arch_ecf8e1f1145d44bf9d790d077f996b93/versioneer.py", line 1445, in get_versions
          cfg = get_config_from_root(root)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/vb/d133zwt15qn51k8m9h7b_mgm0000gn/T/pip-install-wm4lwdk0/arch_ecf8e1f1145d44bf9d790d077f996b93/versioneer.py", line 340, in get_config_from_root
          parser = configparser.SafeConfigParser()
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      AttributeError: module 'configparser' has no attribute 'SafeConfigParser'. Did you mean: 'RawConfigParser'?
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

Expected behavior
To not crash
Context (Environment)
Stoix master (845e4f112abf93dd865e857565478fc4be9e369e)
Python 3.12.3
Possible Solution
I could pin to 3.10, but it would be nice to use the latest python version

Aah yeah, I'd like it to work with python 12 as well. I'll look into it right now.

It should be fixed now. give it a try.


@IBOutlet weak var cartBarItem: BBBadgeBarButtonItem! not working.....

How to set the bar item for IBoutlet
@IBOutlet weak var cartBarItem: BBBadgeBarButtonItem!
// Add it as the  of the navigation bar
self.cartBarItem = barButtonCart

Please see issue #7


Add App Functions Spec.

!deploy

This doesn't seem to be referenced anywhere. Should there be a new functions (or similar) item in specification/resources/apps/models/app_spec.yml?

Good callout, will add.

!deploy

!deploy

!deploy

!deploy


plot limit on disk throughput widget

This will vary greatly depending on the selected storage driver (https://docs.docker.com/engine/userguide/storagedriver/selectadriver/0). There is no default storage driver and that will vary depending on the underlying system.
Vector should handle this separately, probably with different metric groups depending on the storage driver.

Also might be a separate widgets, with the usage and limit as non-stacked area charts.


/usr/share/plymouth/themes/os1/os1.plymouth does not exist

While installing this theme and using the command plymouth-set-default-theme os1 -R, an error occurs.
$ sudo plymouth-set-default-theme os1 -R
/usr/share/plymouth/themes/os1/os1.plymouth does not exist

Using Arch Linux.

thanks for the contribution. I have merged your commits to main branch


import 'videojs-background';

I'd like you use your lib in my (browserify) project I tried adding this (http://ifandelse.com/its-not-hard-making-your-library-support-amd-and-commonjs/) to your project, but this is all still new to me and I can't get it to work...
Would you mind having a stab at it?

I have a patched fork. You could run npm install npm install git://github.com/truetone/videojs-Background.git#npm-fix --save until my pull request is merged here.


[AVOCADO-254] Open PrefilterReads up for external use.

Resolves #254. Also drops the need to specify whether contig names follow GRCh or UCSC conventions.

Coverage increased (+0.9%) to 81.958% when pulling e3f4a3d39ff2a85367748e02c6fd0c0a0aefc971 on fnothaft:issues/254-open-prefilter-reads into 7d0011d68bf3f12c1c0a75c7d7423696053e2f7a on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/avocado-prb/201/
Test PASSed.

Coverage increased (+0.07%) to 81.958% when pulling f08835a05774ee2b18cb62e09e34099f441fa33b on fnothaft:issues/254-open-prefilter-reads into 064be3ecf994996c08111addf1a0c5809f74ad18 on bigdatagenomics:master.

Coverage increased (+0.07%) to 81.958% when pulling f08835a05774ee2b18cb62e09e34099f441fa33b on fnothaft:issues/254-open-prefilter-reads into 064be3ecf994996c08111addf1a0c5809f74ad18 on bigdatagenomics:master.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/avocado-prb/202/
Test PASSed.


:construction: secondary style sheet for docs

This provides a secondary tailwind config that does not do a reset and only includes styles compiled from header and footer templates.
@nessita this style sheet is in static/css/docsstyles.css for use on the docs effort.  It should not affect the body of the page since I've stripped out the css reset and only building for the header and footer.  I can't really test fully on my local machine so I don't know what other side effects we will see yet.
You can see from this hacky test on my machine that when I add this style sheet it does not affect the body.  However I don't have the header in there.

@gregnewman I'm approving and landing this so we can iterate over my work in PR #583


refactor(src): restructured the src folder

PR Checklist
Please check if your PR fulfills the following requirements:

[ yes] The commit message follows our guidelines: https://github.com/NeoSOFT-Technologies/rest-node-nestjs/blob/main/CONTRIBUTING.md
[yes] Tests for the changes have been added (for bug fixes / features)
[yes] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[yes] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?
Currently all the modules are not grouped well like user module is in component and auth module is in src folder.
Issue Number: N/A
What is the new behavior?
Now modules have been grouped well. Moved auth module to core, User module to feature and moved http and mailer module to shared.
Does this PR introduce a breaking change?

No

Other information

refactor(src) : have refactored the src folder


Add rules for sorting imports to ruff

Why are these changes needed?
Unsorted imports often cause merge conflicts that must be resolved manually. It slows down the process of merging new pull requests by making it difficult to merge them in batches.
Existing imports have been sorted and cleaned up using the ruff linter with additional rules.

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Codecov Report
Attention: Patch coverage is 84.45946% with 23 lines in your changes are missing coverage. Please review.

Project coverage is 47.25%. Comparing base (af9b300) to head (e7dd80b).

Files
Patch %
Lines

autogen/agentchat/contrib/agent_builder.py
0.00%
6 Missing :warning:

autogen/coding/jupyter/local_jupyter_server.py
0.00%
6 Missing :warning:

autogen/agentchat/contrib/society_of_mind_agent.py
0.00%
3 Missing :warning:

autogen/coding/jupyter/jupyter_code_executor.py
0.00%
3 Missing :warning:

autogen/coding/jupyter/__init__.py
0.00%
2 Missing :warning:

...n/coding/jupyter/embedded_ipython_code_executor.py
0.00%
1 Missing :warning:

autogen/coding/jupyter/jupyter_client.py
75.00%
1 Missing :warning:

autogen/graph_utils.py
50.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #2155       +/-   ##
===========================================
+ Coverage   36.77%   47.25%   +10.48%
===========================================
  Files          69       69
  Lines        7133     7129        -4
  Branches     1556     1684      +128
===========================================
+ Hits         2623     3369      +746
+ Misses       4277     3472      -805
- Partials      233      288       +55

Flag
Coverage Δ

unittests
47.11% <84.45%> (+10.35%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Replaces with the fresh PR https://github.com/microsoft/autogen/pull/2183


Add Algolia search

Integrates with Algolia's DocSearch. Need an API key to enable.
The search bar is by default on the right side of the nav, so this also reduces the width of the other nav links to accommodate the search bar.

I'm really liking these new docs, and the search makes it even better.


DB Connection validity check

In Core Java Persistence of Persistence Module, DB connection must be checked before giving it to resource to use it.
@Override
public Connection getConnection() throws SQLException {
        if (connectionPool.isEmpty()) {
            if (usedConnections.size() < MAX_POOL_SIZE) {
                connectionPool.add(createConnection(url, user, password));
            } else {
                throw new RuntimeException("Maximum pool size reached, no available connections!");
            }
        }
        Connection connection = connectionPool.remove(connectionPool.size() - 1);
        //Todo Check Connection Validity
        usedConnections.add(connection);
        return connection;
}

Todo in Code is added for more detail.

@eugenp assign this ticket to me.....

Hi @ammaralii,
Thanks for the feedback. We'll take a look into this.
I'll keep you updated!

Hi @kwoyke,
I already provide the fix and open the merge request. You can reproduce the issue locally by down the DB service then up again then it'll give exception communication link failure or sometimes null pointer exception on connection object we are using. But if we check the connection then whenever the connection is given from pool it first checks if its death then we will give another life.

Hi @ammaralii,
We've updated the article and codes. We couldn't merge your PR directly. You use hardcoded "SELECT 1" query - that would fail on Oracle or DB2 databases for example. Instead we used a generic Connection#isValid(int) that delegates to the JDBC driver.
Again, thanks for your feedback.


Add $ to IsIdentifierFirstChar list.

Quick fix for https://github.com/Microsoft/nodejstools/issues/514. Technically JavaScript identifier names can contain many other Unicode characters such as π, Σ, λ. We may want to support these in the future.

:+1:


Can't upload to firebase storage in ios but works on android.

Can't upload to firebase storage in ios but works on android.
Unhandled Exception: [firebase_storage/unauthorized] User is not authorized to perform the desired action.
I don't thing there is any issue on the firebase backend as files can be successfully uploaded through same plugin in android.
But not in ios.
What can be the problem?

Hi @prashantspandey
Could you please provide your flutter doctor -v and flutter pub deps -- --style=compact?
Could you provide the entire stacktrace?
Thank you

Closing due to inactivity.


Fixed end time to 3:30 UTC for PD01

Session should be 2 h not 90 min.  It is correct on grid.

Looks good to me.

Merged.  Checked page: OK.
On Tue, Oct 20, 2020 at 5:58 PM Debbie Paul<EMAIL_ADDRESS>wrote:

Looks good to me.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/tdwg/website/pull/247#issuecomment-713224964, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACKZUDIHMLQFTGUKRUVPV3TSLYW4BANCNFSM4SZBUZ5Q
.


Need to know How we add sort keys in query

Hi
Need to know how we add sort keyboard in query? I am not able to get any method or parameter by we can pass sort keys in query.
Please suggest
Thanks
Gaurav

Hi dgaurav-163,
Could you provide a code sample of what you're trying to do? Thanks.

Hi Please find code sample
Document query = new Document("user_id",userId);
query.append("mode", "car");
//            Document sortParam = new Document("firstUpdateTimestamp", "-1");
//            query.append("sort", sortParam);
_mongoClient.getDatabase(DATABASE).getCollection(COLLECTION_NAME).find(query, QUERY_LIMIT)
We have to sort data according to "firstUpdateTimestamp" field so where I can add sort key in code.
Please suggest

@dgaurav-163 this is possible now as of 4.0.0-beta-1


Revert "Added GamepadEx class with mapping functionality for actions"

Reverts j5155/Expressway#5

Forgot to review it first? 😅

i merged to master instead of a new branch lmao oopsie poopsie


[Bitsong] - Bdjuno unables to parse the tx type /cosmos.authz.v1beta1.MsgGrant.

Bdjuno is running fine on Bitsong but some of the transactions can't be parsed.
Mar 13 23:10:13 bitsong-m-bdjuno-ln-atl bdjuno[764061]: 11:10PM ERR re-enqueueing failed block err="failed to get transactions for block: error while unpacking message: no concrete type registered for type URL /cosmos.authz.v1beta1.MsgGrant against interface *types.Msg" height=5050773

bitsong upgraded to sdk v0.44.5, updated bdjuno branch updated and deployed at the server.


NullPointerException in ConfigBeanImpl.isOptionalProperty when using ConfigBeanFactory.create

Thanks to Viktor Klang, discovered ConfigBeanFactory but it's not quite working against an S3 Client Configuration.
build.sbt:
libraryDependencies += "com.amazonaws" % "aws-java-sdk-s3" % "1.11.73"
libraryDependencies += "com.typesafe" % "config" % "1.3.1"

FailApp.scala:
import com.amazonaws.ClientConfiguration
import com.typesafe.config.{ConfigBeanFactory, ConfigFactory}

object FailApp extends App {
  ConfigBeanFactory.create(ConfigFactory.empty(), classOf[ClientConfiguration])
}

Running:
sbt run

Result:
java.lang.NullPointerException
	at com.typesafe.config.impl.ConfigBeanImpl.isOptionalProperty(ConfigBeanImpl.java:280)
	at com.typesafe.config.impl.ConfigBeanImpl.createInternal(ConfigBeanImpl.java:95)
	at com.typesafe.config.ConfigBeanFactory.create(ConfigBeanFactory.java:47)
	at FailApp$delayedInit$body.apply(FailApp.scala:5)

Expectation:
No exception, no output.

Here's a gist, you can just clone it using the command in the comment:
https://gist.github.com/ScalaWilliam/6f267928f222e4ac80f43bb0d663d871#gistcomment-1958918

The issue appears to be that there's no field called userAgent, but there are methods getUserAgent/setUserAgent. It seems the user agent is deprecated in favor of getUserAgentPrefix/Suffix, so maybe the user agent field is now "virtual" (defined in terms of those other properties).
The easy fix could be to modify this block:
            List<PropertyDescriptor> beanProps = new ArrayList<PropertyDescriptor>();
            for (PropertyDescriptor beanProp : beanInfo.getPropertyDescriptors()) {
                if (beanProp.getReadMethod() == null || beanProp.getWriteMethod() == null) {
                    continue;
                }
                beanProps.add(beanProp);
            }

Which ignores props without a get or set, to also ignore props without a field?
A harder fix might be, if there's no field, invoke the getter/setter instead, but maybe that's overkill.
PRs welcome, ideally with regression test case. :-) Otherwise timeline is unknown but should happen eventually.
Thanks for the report!

Fixed in #454


superH memory read/write functions bugs on sunflower for embedded systems

Describe the bug
Functions:
uchar 	superHreadbyte(Engine *, State *S, ulong addr);
ulong 	superHreadlong(Engine *, State *S, ulong addr);
ushort 	superHreadword(Engine *, State *S, ulong addr);
void 	superHwritelong(Engine *, State *S, ulong addr, ulong data);
void 	superHwriteword(Engine *, State *S, ulong addr, ulong data);
void	        superHwritebyte(Engine *, State *S, ulong addr, ulong data);
produce segmentation faults when executed since all the structures
related to numa have not been initialized or are NULL.
To Reproduce
Steps to reproduce the behavior (on PC):

Clone Helium-firmware@4f14f6 with sunflower-simulator@t8d5601e.
Apply patches for spresense (according to README).
Modify sim/main.c so that sunflower can be compiled natively.
Execute any m file.

Steps to reproduce the behavior (on spresense):

Clone Helium-firmware@4f14f6 with sunflower-simulator@t8d5601e.
Compile for spresense (according to README).
Run spresense and load an srec file in sunflower.
Run the simulator.

This is now fixed. @btsouts please tag with the corresponding issue / commit that addressed this.


Easier switch to multiple accounts?

Describe the requested feature:
So I looked thru this github page and I understood that it is possible to have multiple account based on the API keys? Which I couldn't get to work btw. But is there a way we would be able to just press a button and all my logged in accounts show up (like the app).
Additional context:
I would like to help you guys out if it is too much work.
Looking at the application all data is loaded thru the selected API key right? So I would say that if it was build with this in mind that it is really 90% UI? Correct me if I'm wrong. I am a android developer so I know how annoying it is if someone says that "But it is not that hard right?". While is basically messes up the whole core system.

https://wiki.bunqdesk.top/setup/setup-multiple-accounts/
Basically you can log out and log in to a different account. From that point on you should be able to switch instantly from the desktop and from the login screen itself.


Never return -1 from BN_exp

The return code of BN_exp, BN_mod_exp is documented as 0. However it might return -1 in the cases below which can happen if BN is not correct. This is an issue for example in  DSA_generate_key that relies on that return value.
This is actually not a theoretical issue as it happened when I was playing with DSA in PHP openssl ext:
https://github.com/php/php-src/blob/c3c90abb17017af83133f57ec5d7663c0b33fdff/ext/openssl/tests/bug72336.phpt
The problem was that we expect binary value - BN_bin2bn is used on the string (meaning user forgot to call hex2bin). What happened is that DSA_generate_key returned 1 (success) but in fact priv key wasn't generated. Currently I have to do this after porting to 1.1 which is a bit messy.
https://github.com/php/php-src/blob/c3c90abb17017af83133f57ec5d7663c0b33fdff/ext/openssl/openssl.c#L4077-L4082
I understand that it might not be good to fix it in 1.0.2 due to BC break but 1.1 would be good IMHO.

Yes, return 0 on error is what is documented, pretty clearly.  We should fix this now.

That the actual function can return -1 and always has means that this is actually a documentation error in this instance. That it should be consistent and not do so is a separate item. Taking a quick "grep" for return statements in the bn code I think this won't be the only function where a return of neither 0 or 1 is possible.

Okay, so you think we should update the doc.  I'm fine with that.

No - just pointing out that the source of truth for what an API does isn't the documentation.
Ideally the documentation and the code should match - but when they don't we cannot just automatically assume the documentation is the correct item. We need to look at it. There is only one use of BN_exp in the tree in bntest and it does a <= 0 check for failure. All the BN functions should be consistent - but they are not - and I do think this specific change makes sense to make - but the first check needs to be to see the history of the function and its usage - rather than leaping to must-match-the-doc on the assumption that the documentation is correct.
On this specific change I'm +1 for including it as it should be this way - and there is no library code impacted - and the change basically doesn't break anything (we just will never return a -1 error indication and that is okay). So it is a change to an API - but it is a compatible change.

ah, thanks.  i agree the 1//0/-1 thing needs to be looked at.  i will merge this one.

merged with commit 0818dba  thanks!


 Now that I tried recovering my password using my GitHub email address I'm getting 500 Internal Server Error message.

Closing old issue. Please open a new one if this issue is still there.


feature(wrh): taxi_dqn_config.py update

Description
update dqn params
Related Issue
TODO
Check List

[ ] merge the latest version source branch/repo, and resolve all the conflicts
[ ] pass style check
[ ] pass all the tests

This PR will be continued in #807


[PE-D][Tester E] The filterbuyers command does not work even with the example input given in the UG

Command executed: filterbuyers -p 500000 -c bright; sunny -priority HIGH (taken from the command summary section)
Expected: command to execute and buyers list to be filtered.
Actual: Error message shown, command not executed

Labels: severity.High type.FunctionalityBug
original: wrewsama/ped#9

edit UG w syntax

Fixed UG documentation error


Update lb-s3-bucket/aws to v0.16.0 added option to force ssl on buckets

what
Figured I would update this to use v0.16 of the lb s3 bucket module similar to the this
Adds an optional input variable to force SSL on the s3 bucket for security compliance.
why
[PCI.S3.5] S3 buckets should require requests to use Secure Socket Layer
references
AWS Config

/test all

@joshmello
Applied your changes in https://github.com/cloudposse/terraform-aws-nlb/pull/27
Closing the PR


Synchronous onboarding and PostOnboarding on Desktop

📝 Description
This pull request serves as a base branch for the ongoing effort to bring synchronous onboarding and the post onboarding hub to Ledger Live Desktop.
❓ Context

Impacted projects: ledger-live-desktop
Linked resource(s): FAT-99 FAT-556

✅ Checklist

[x] Test coverage
[x] Atomic delivery
[x] No breaking changes

📸 Demo

TODO
🚀 Expectations to reach
Please make sure you follow these Important Steps.
Pull Requests must pass the CI and be internally validated in order to be merged.

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots

/generate-screenshots 🤞 last hopeless try before the weekend that gh checkout runs fine in the macos runner

OMG it worked !

C'mon, please be green

/generate-screenshots


"This API can only be accessed with username and password Basic Auth" on create or pull-request

Can't seem to find anything like this in the issue list that covers this. I just installed hub on a new laptop and my usual pull-request command just shows:
❯ hub pull-request
Error creating pull request: Forbidden (HTTP 403)
This API can only be accessed with username and password Basic Auth

This was both before & after setting GITHUB_* env vars. Using password from an auth token since my account if MFA.
❯ hub version
git version 2.4.6
hub version 2.2.1

Not sure how best to debug this further.
If there's a more appropriate place for support tickets please let me know.
Thanks in advance!

Looks like hub fork also yields the same error.

What does "MFA" mean for your account?
Hub doesn't support using an OAuth token instead of your password just yet. Instead, you can drop your OAuth token manually within hub configuration so you don't get prompted for any of your credentials anymore. Create a ~/.config/hub file:
---
github.com:
- protocol: https
  user: matschaffer
  oauth_token: YOUROAUTHTOKEN

If you're using GitHub Enterprise, instead of "github.com" above put the domain of your Enterprise instance. Let me know if this works. Sorry for the trouble. We're working on improving this flow.

Sorry, MFA == two factor authentication (https://help.github.com/articles/about-two-factor-authentication/)
Maybe hub doesn't support having that enabled?
This is against github.com rather than enterprise.
Is there somewhere in the UI can I create an oauth token manually or do I need curl the API?

We do support 2FA. Just use your regular password when the hub asks you for it on the command-line. Your password will never get stored anywhere.

Ah, actually I think I'd assumed I needed a authentication token which is what I was trying to use as a password. Tried just my regular u/p today, did the 2fa and everything went well.
Thanks for the assist and apologies for the noise.


CLI Completion Scripts

Describe the problem
One thing I've been doing lately with my (unfortunately private, so I've no examples) CLI scripts is to provide a file that lets the user bootstrap tab-completion automatically, either by copying it into a shell configuration file like ~/.bashrc or copying the file entirely, like completion/_foo.zsh, into a place where it can be automatically picked up.
I think it would be useful to be able to automatically generate these from TTY::Option, even though some of the features wouldn't be translatable into every shell.
How would the new feature work?
It doesn't quite make sense to generate this during normal runtime, but since you already have #parse, perhaps we could add #to_completion(shell = :bash) or something.
Drawbacks
Mostly that it could be argued this is out of scope for the gem, and a lot of work to research and support a bunch of different shells.

Hi Michael 👋
Thank you for taking interest in tty-option and sharing your idea for generating completion scripts.
I like the idea but equally, I don't have the investment time to work on this. I have many other projects and ideas that I wish to do and this has never been in scope for this gem. I can see the utility in having the ability to automatically generate completion scripts. But my worry is the maintenance of this feature would be significant, given the possible shells and allowing configuration of what should and shouldn't be completed.
Now, if you have time and wish to see this functionality please create a proof of concept and share it. We can then collaborate and see what's the best way to handle it.
Given the above, I will close this suggestion.


Errors not bubbling up.

When trying to create a customer, I get the error undefined local variable or method 'error' for Paysimple:Module.
begin
 Paysimple::Customer.create(customer_data)
rescue => e
  puts e.message
end

When I call it directly from rails console I get the correct error
Digest::Digest is deprecated; use Digest
Paysimple::InvalidRequestError: ["The LastName field is required."]
from /Users/Trent/.rvm/gems/ruby-2.2.2@breeze_pay/gems/paysimple-ruby-0.2.0/lib/paysimple.rb:135:in 'handle_api_error'

Disregard. I see you made the fix but haven't yet updated ruby gems. I have forked the repo and am using it. Thanks.


Pack config shows secrets on final output

Even though secrets are obfuscated on input, and even if I choose to skip showing the config in an editor before saving it, the final output still shows secrets in plaintext.
vagrant@st2vagrant:/opt/stackstorm/packs/twitter/actions$ st2 pack config twitter
count (integer) [30]:
language:
access_token (secret): ***************************************************
consumer_key (secret): **************************
consumer_secret (secret): ***************************************************
query (comma-separated list): #stackstorm
access_token_secret (secret): **********************************************
---
Do you want to preview the config in an editor before saving? [y]: n
---
Do you want me to save it? [y]: y
+----------+--------------------------------------------------------------+
| Property | Value                                                        |
+----------+--------------------------------------------------------------+
| pack     | twitter                                                      |
| values   | {                                                            |
|          |     "count": 30,                                             |
|          |     "language": null,                                        |
|          |     "access_token": "                                        |
|          | fffff-fffff",         |
|          |     "access_token_secret": "                                 |
|          | fffff",              |
|          |     "consumer_secret": "                                     |
|          | fffff",         |
|          |     "query": [                                               |
|          |         "#stackstorm"                                    |
|          |     ],                                                       |
|          |     "consumer_key": "	fffff"             |
|          | }                                                            |
+----------+--------------------------------------------------------------+

Note that I obfuscated the keys in the output - normally they're shown clearly

It would be nice if this output was suppressed, even as an option.

It looks like this is not entirely consistent. When testing with st2 2.3.0-1, and the Slack pack, I see this:
vagrant@bwcvagrant:~$ st2 pack config slack
post_message_action.username: esteetwe
post_message_action.webhook_url [True]: https://hooks.slack.com/services/4385723579354983579235
post_message_action.icon_emoji [:panda_face:]:
post_message_action.channel: lkhtesting
action_token (secret): ***********
sensor.strip_formatting (boolean) [n]:
sensor.token (secret) [True]: *************************
admin.organization: foobar
admin.attempts (integer) [1]:
admin.admin_token (secret) [True]: ********************
admin.auto_join_channels (comma-separated list): stackstorm
admin.set_active (boolean) [y]:
---
Do you want to preview the config in an editor before saving? [y]: n
---
Do you want me to save it? [y]: y
+----------+--------------------------------------------------------------+
| Property | Value                                                        |
+----------+--------------------------------------------------------------+
| id       | 59480eccbb0f180d70538a1f                                     |
| pack     | slack                                                        |
| values   | {                                                            |
|          |     "post_message_action": {                                 |
|          |         "username": "esteetwe",                              |
|          |         "webhook_url":                                       |
|          | "https://hooks.slack.com/services/4385723579354983579235",   |
|          |         "icon_emoji": ":panda_face:",                        |
|          |         "channel": "lkhtesting"                              |
|          |     },                                                       |
|          |     "action_token": "********",                              |
|          |     "sensor": {                                              |
|          |         "strip_formatting": false,                           |
|          |         "token": "3485934579438574930857843"                 |
|          |     },                                                       |
|          |     "admin": {                                               |
|          |         "organization": "foobar",                            |
|          |         "attempts": 1,                                       |
|          |         "admin_token": "90384250850-48502435",               |
|          |         "auto_join_channels": [                              |
|          |             "stackstorm"                                     |
|          |         ],                                                   |
|          |         "set_active": true                                   |
|          |     }                                                        |
|          | }                                                            |
+----------+--------------------------------------------------------------+
vagrant@bwcvagrant:~$

Note that action_token is obfuscated in the output, but sensor.token and admin.admin_token are not. Something to do with the object hierarchy?
Related to our Secrets Masking configuration probably? https://docs.stackstorm.com/reference/secrets_masking.html

Tested with 2.8.1, this no longer applies:
extreme@EWC:~$ st2 pack config twitter
count (integer) [30]:
language:
access_token (secret): *******************
consumer_key (secret): ***************
consumer_secret (secret): ***************
query (comma-separated list): st2
access_token_secret (secret): *******************
---
Do you want to preview the config in an editor before saving? [y]: n
---
Do you want me to save it? [y]: y
+----------+---------------------------------------+
| Property | Value                                 |
+----------+---------------------------------------+
| id       | 5b9f428c99c96b07f8208493              |
| pack     | twitter                               |
| values   | {                                     |
|          |     "count": 30,                      |
|          |     "language": null,                 |
|          |     "access_token": "********",       |
|          |     "consumer_key": "********",       |
|          |     "consumer_secret": "********",    |
|          |     "query": [                        |
|          |         "st2"                         |
|          |     ],                                |
|          |     "access_token_secret": "********" |
|          | }                                     |
+----------+---------------------------------------+
extreme@EWC:~$```

Do we have a test for this somewhere? This would be a great regression test.


2.0 Minimap: canvas click does nothing; Initial load does not display waveform

Having trouble with minimap plugin on version 2
Wavesurfer.js version(s):
2.0 beta 2
Browser version(s):
Chrome Version 62.0.3202.94 (Official Build) (64-bit)
Code needed to reproduce the issue:

Angular 1x.
wavesurfer.js and Minimap plugin built into vendor.js file via gulp
directive loads wavesurfer and minimap.
controller inits minimap later

(1) Canvas click issue:
Clicking on minimap canvas scrolled the audio on 1x, now nothing happens. Only main WS container canvas can be intereacted with. Hopefully I am missing some minimap config detail
(2) Minimap waveform does not display on first initialization?
Minimap will not show waveform on first load.
** Soution found: Found that the sequence of loading the minimap cleared the problem.
Im not sure about the operations around promises but toggling the minimap before 'ready' seems to clear for me. code snippet:
 vm.wavesurfer.load(activeUrl || null);    vm.wavesurfer.once('ready', function () {     // do stuff       }); _toggleMinimap();

Found that changing the backend to 'mediaelement' seems to clear the minimap render issue, but creates other problems. I think I need to add a timeout or something before loading minimap. does not seem to solve canvas interaction though

@PureSqncMedia can you try with 2.0.1?

related to https://github.com/katspaugh/wavesurfer.js/commit/f08cfbded2c1eb1ff48990733dab48606c882e9b perhaps.

Soution found: Found that the sequence of loading the minimap cleared the problem.

And can you make a pull request so we can take a look at the fix?

@thijstriemstra I tried with 2.0.1 and still no interaction on the minimap canvas. I might try with 'mediaelement' backend, or is that not recommended?
The code I have is just loading sequence for wavesurfer elements. The fix for some reason seems that loading the minimap within ws 'onready' signal was
`
function _start() {
if (vm.player.track) {
vm.player.wavesurfer.load(vm.activeAudioURL || null);
vm.player.wavesurfer.once('ready', function () {
Player.playTrack(vm.player.trackId).then(playTrackSuccessFn, playTrackErrorFn);
$scope.$broadcast('rzSliderForceRender');
$scope.length = Math.floor(vm.player.wavesurfer.getDuration()).toString();
$interval(function () {
$scope.moment = Math.floor(vm.player.wavesurfer.getCurrentTime()).toString();
}, vm.momentInterval);
                vm.player.trackTime = vm.player.wavesurfer.getDuration();
                play();
                vm.player.playing = true;
            });

            _toggleMinimap();
        }
        else {
            console.error('No valid audio found during trak player start !');
        }
    }

    function _toggleMinimap() {
        //delete any minimaps first
        angular.forEach(vm.player.wavesurfer.initialisedPluginList, function (value, key) {
            if (key == 'minimap') {
                vm.player.wavesurfer.destroyPlugin('minimap');
                console.log("yesahh deleted some minimz")
            }
        });

        //init minimaps

vm.player.wavesurfer.addPlugin(MiniMapPlugin.create(detail_minimap_wave_options)).initPlugin('minimap');
    }`

code example
`
vm.player.wavesurfer.once('ready', function () {
.. do stuff on ready, like play
// loading minimap in here results in first track played not rendering minimap.
// a subsequential load and load minimap seems to display correctly
// so moved the add plugin part outside of ready block
});
//works first load
vm.wavesurfer.addPlugin(MiniMapPlugin.create(detail_minimap_wave_options)).initPlugin('minimap');
`

@thijstriemstra @roboswank  Just updated and happy to see that minimap canvas works now! Thanks


SOLR-17102: Replaced VersionBucket array with locks on-demand

https://issues.apache.org/jira/browse/SOLR-17102
DistributedUpdateProcessor some refactoring to balance locks clearly.
VersionInfo: moved locks out to UpdateLocks.
Refactor: OrderedExecutor uses generics.  Allows use of the ID now (a BytesRef) instead of a hash.

Surprisingly clean and concise refactoring!

Thanks :-). There were a few rounds of attempts before getting at this.

Did you run the ./jmh.sh CloudIndexing benchmark on it?

oh, great suggestion.  I'll do that as the next step; I remember you used that when you did the earlier VersionBucket change.
FYI I removed the documentation setting for the former timeout without replacement of the new.  I think it's even more unlikely that someone would set it.  When it was a "bucket" there was a reason and the docs elaborated on that reason.  It's reasonable to by default block indefinitely for a concurrent update for the same doc.  I'll get another opinion on this once I see the benchmarks look good.

Before
Benchmark               (nodeCount)  (numReplicas)  (numShards)  (preGenerate)  (useSmallDocs)   Mode  Cnt     Score      Error  Units
CloudIndexing.indexDoc            4              1            4          50000            true  thrpt    4  9714.465 ± 2035.592  ops/s
CloudIndexing.indexDoc            4              1            4          50000           false  thrpt    4   961.674 ±   64.489  ops/s
CloudIndexing.indexDoc            4              3            4          50000            true  thrpt    4  3766.626 ± 2087.101  ops/s
CloudIndexing.indexDoc            4              3            4          50000           false  thrpt    4   270.518 ±  273.984  ops/s

After
Benchmark               (nodeCount)  (numReplicas)  (numShards)  (preGenerate)  (useSmallDocs)   Mode  Cnt     Score      Error  Units
CloudIndexing.indexDoc            4              1            4          50000            true  thrpt    4  8876.582 ± 6487.282  ops/s
CloudIndexing.indexDoc            4              1            4          50000           false  thrpt    4   939.311 ±  140.602  ops/s
CloudIndexing.indexDoc            4              3            4          50000            true  thrpt    4  3467.660 ±  625.605  ops/s
CloudIndexing.indexDoc            4              3            4          50000           false  thrpt    4   263.458 ±  306.638  ops/s

Perhaps an 8% regression.  Hmmm.

Could you rerun the benchmark 'After'? The error range is high on the first line.
Does the benchmark outputs if there is more GC? I'm thinking of the new transient small lock objects. Not sure it would be visible though.

I simplified it a bit more when removing from the map.  And added more code comments / javadocs.
I benchmarked another time, this time with HTTP1.1 which didn't have the connection issue, thus could get a more complete run.  The findings are the same:
Baseline
Benchmark               (nodeCount)  (numReplicas)  (numShards)  (preGenerate)  (useSmallDocs)   Mode  Cnt      Score     Error  Units
CloudIndexing.indexDoc            1              1            1          50000            true  thrpt    8  28078.538 ± 841.866  ops/s
  MAX: 28546

Latest HPPC, simplify remove
Benchmark               (nodeCount)  (numReplicas)  (numShards)  (preGenerate)  (useSmallDocs)   Mode  Cnt      Score     Error  Units
CloudIndexing.indexDoc            1              1            1          50000            true  thrpt    8  28580.605 ± 857.046  ops/s
  MAX: 29310

updated the upgrade notes to point out the legacy config settings.
removed settings from any config
CHANGES.txt:

SOLR-17102: The VersionBucket indexing lock mechanism was replaced with something just as fast yet
that which consumes almost no memory, saving 1MB of memory per SolrCore.  (David Smiley)

Will merge tomorrow.


Fix 404s on Codecov File Views

This PR will close #203
Description
As per the linked issue, there was a bug regarding Codecov and accessing the API's files within the codecov portal to see the colourful display of test coverage (screenshot of the issue below). By not providing a path for the checkout action within the CI build workflow, this maps the path on the runner to the relative path on the repo. This is a slightly different solution to the 'path fixing' I identified in the issue, but changing the repo path on the CI runner has turned out to be the simpler solution. This means you can see coverage data in a colourful format!

Testing Instructions
Viewing any of the files using the following link (codecov on this branch) will allow you to view them with coverage data:
https://codecov.io/gh/ral-facilities/datagateway-api/tree/bugfix%2Fcodecov-path-fixing/datagateway_api

[x] Review code
[x] Check GitHub Actions build
[x] Review changes to test coverage
[x] You should be able to view a copy of each file, displaying code coverage data - test link provided above
[x] Codecov PR comment should be present and correct on this PR

Agile Board Tracking
Connect to #203

Yeah I can recreate the issues you're describing, not sure what the solution would be for them though. I wonder if the first one is occurring because the base branch (master) doesn't have the changes in, so there's nothing to compare to. Perhaps that's more hopeful than realistic though. I will dig a bit deeper into the sunburst issue once I've merged this

It looks like the sunburst issue has been resolved - if you navigate to a file (or directory for that matter) it'll take you to src and displays files as intended. I guess we can find out if the first issue has been resolved when a PR is opened that contains the changes from this PR


Underscore in setup.py for extras_require is (probably) not supported by pip

Description
When I run the suggested installation command:
pip install -U "jax[cuda12_pip]" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html
I get the following warning (which causes other downstream problems):
WARNING: jax 0.4.23 does not provide the extra 'cuda12-pip'
Note that cuda12-pip has a hyphen in the warning, not an underscore.
This problem occurs with pip versions since 23.3, but not earlier ones. In the release notes for pip 23.3 this change seems to be considered a bug fix:

Normalize extras according to PEP 685 from package metadata in the resolver for comparison. This ensures extras are correctly compared and merged as long as the package providing the extra(s) is built with values normalized according to the standard. Note, however, that this does not solve cases where the package itself contains unnormalized extra values in the metadata. (#11649)

The normalization mentioned is the following:
re.sub(r"[-_.]+", "-", name).lower()
i.e., all consecutive hyphens, underscores, and periods are replaced with a single hyphen.
Given these circumstances, does it make sense to change the setup.py in jax and jaxlib to not use underscores? Or at least mention in the README that an older version of pip needs to be used. This caused me a lot of problems with the installation, especially since it just gives a warning without failing.
What jax/jaxlib version are you using?
No response
Which accelerator(s) are you using?
No response
Additional system info?
No response
NVIDIA GPU info
No response

Never mind, I just realized my "downstream problems" were caused by something else. Sorry for the spam.


Remove tomcat-users.xml get packed with the 4.4.x kernel

Description:
$subject
This is default tomcat config file. And it's not used with the product. Hence it should be removed shipping with products.

Fixed with https://github.com/wso2/carbon-kernel/pull/2254


Minor fixes + More Consistent Arguments for Argument Parser

This is a really minor change and I'm not sure if the repo accepts pull requests:
I noticed that the "C" button wasn't capturing the depth map in "standard" mode (tested on my MacBook M1 on macOS Monterey 12.6.2) so I changed the if key == 32 and not self.estimator.live_render: to if key == 32 or key == ord("c") and not self.estimator.live_render: in torchcam/camera.py
I also noticed that there was a bit of inconsistency with the argument parser, since it was expecting simply mode when all other arguments had a -- in front of them.

Thank you for your interest @Sendeky, and sorry for not getting back to you sooner (I didn't get the notification).
I've migrated this project to Click CLI to make the formatting in args a bit more consistent.
I think the c as exit key is a good addition. I'm thinking of reviving back this project, so feel free to put in a PR with the updated setups.


Espresso support with most recent Enso@GraalVM 22.3

Pull Request Description
Once upon a time there was #4877 which managed to convince Enso native-image to load standard libraries via Espresso. Since then, however, the codebase changed and the integration with GraalVM 22.3.x and Espresso has to look differently right now. This PR updates the original code to most recent develop branch.
Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[ ] Unit tests have been written where possible.

To build and test this PR via sbt (btw. try ./run backend sbt if you want to get all the dependencies):
$ /graalvm-ce-java17-22.3.1/bin/gu install espresso
enso$ sbt --java-home /graalvm-ce-java17-22.3.1/ bootstrap
enso$ sbt --java-home /graalvm-ce-java17-22.3.1/ buildEngineDistribution
enso$ sbt --java-home /graalvm-ce-java17-22.3.1/ engine-runner/buildNativeImage

Let's also create a file h.enso with following content:
import Standard.Base.IO

main = IO.println <| "Hello World!"

Now we can execute the created bits using different styles...
Native Image with Espresso
enso$ JAVA_HOME=/graalvm-ce-java17-22.3.1/ ./runner --run h.enso
Hello World!

Great, it works. Enso can successfully call into Espresso and invoke System.out.println.
JVM with Espresso
enso$ JAVA_HOME=/graalvm-ce-java17-22.3.1/
    ./built-distribution/enso-engine-0.0.0-dev-linux-amd64/enso-0.0.0-dev/bin/enso --run h.enso

Useful for debugging and finding out what is going on. Pass in additional JAVA_OPTS=-agentlib:jdwp... to debug.
JVM without Espresso
enso$ ENSO_JAVA=hosted JAVA_HOME=/graalvm-ce-java17-22.3.1/
    ./built-distribution/enso-engine-0.0.0-dev-linux-amd64/enso-0.0.0-dev/bin/enso --run h.enso

Useful to compare the benefits of Espresso over the current (e.g. hosted Java) state.

The performance results are interesting. It takes about 1-2s to start Espresso:

And there is a huge difference between following programs:
main = [ "IO", "Hello World!" ]

this one starts almost instantly (as it doesn't need standard libraries, neither Espresso). The version with:
import Standard.Base.IO
main = IO.println <| "Hello World!"

is slow - it needs to prepare the Standard.Base library and also boot Espresso. However also the following version isn't fast:
import Standard.Base.IO
main = [ IO, "Hello World!" ]

and this time it is not problem of Espresso, but of preparing the Standard.Base library. The work on speeding this up such library initialization is tracked as #6100

With f3705ea one can even run the IDE with ENSO_JAVA=espresso and visualizations seem to work:

Not everything is perfect, Table.new results in something strange, but overall the state isn't completely hopeless.

Running test/Test with Espresso
With 513215f and with one more patch (to workaround https://github.com/oracle/graal/issues/7435):
enso$ git diff
diff --git engine/runtime/src/main/java/org/enso/interpreter/node/callable/resolver/HostMethodCallNode.java engine/runtime/src/main/java/org/enso/interpreter/node/callable/resolver/HostMethodCallNode.java
index 32de29eb30..5944927e62 100644
--- engine/runtime/src/main/java/org/enso/interpreter/node/callable/resolver/HostMethodCallNode.java
+++ engine/runtime/src/main/java/org/enso/interpreter/node/callable/resolver/HostMethodCallNode.java
@@ -268,7 +268,7 @@ public abstract class HostMethodCallNode extends Node {
       @Shared("hostValueToEnsoNode") @Cached HostValueToEnsoNode hostValueToEnsoNode) {
     try {
       return hostValueToEnsoNode.execute(instances.instantiate(self, args));
-    } catch (UnsupportedMessageException e) {
+    } catch (Error | UnsupportedMessageException e) {
       CompilerDirectives.transferToInterpreter();
       var ctx = EnsoContext.get(this);
       var err = ctx.getBuiltins().error().makeNotInvokable(self);

I am able to execute whole test/Tests suite:
enso$ .../enso-0.0.0-dev/bin/enso --run test/Tests
1663 tests succeeded.
541 tests failed.

Still a bunch of Espresso bugs to fix, but the result isn't completely desperate.


287_circuit for padstackinstance

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.11%. Comparing base (2de2718) to head (951cada).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #306   +/-   ##
=======================================
  Coverage   84.11%   84.11%
=======================================
  Files         105      105
  Lines       17914    17919    +5
=======================================
+ Hits        15068    15073    +5
  Misses       2846     2846


Data transformers

There should be a portable way of specifying transformations on quicktype data, and quicktype should generate code for them in all supported languages.  A wide range of transformations are possible - see jq for some inspiration.  We'll have to start small, if only because supporting the target languages will be a lot of work.
As a first step I suggest modifying individual properties by applying a simple function to them. In many cases that will also modify their types.
Things that would be easy to solve with this:

Default values for optionals, of which #41 is a special case.

Parsing strings into non-JSON types.  Special cases are #138 and #221.

For the time being the functions will be pre-defined, but we'll have to eventually allow users to implement their own functions in their target languages.

We can represent this in the IR with two new entities.  One is a new kind of type, a TransformedType, the other one describes the semantics of the transformation, and is called a Transformation.
Transformers are defined recursively.  A transformer can be either

a built-in transformer function, such as StringToDate, potentially taking arguments such as a date format.
for compound types, a set of transformers, one for for each child of the transformed type.

Each transformed type has

the JSON type that's transformed.
the type that it's transformed to.
a transformer

We need to take care to make transformers bidirectional, so that data can be serialized as well as deserialized.
Note that the same transformations can be expressed in multiple ways.  For example, let's say we have
class Foo {
    date: String -> Date (StringToDate)
}

The meaning here is that the type of Foo.date is a TransformedType which is transformed from string to date via the StringToDate transformer function.  This is semantically equivalent to
class { date: String } ->
class Foo {
    date: Date
} ( { date: StringToDate} )

Here the outer type is a TransformedType that transforms from a class containing a string member date to class Foo with date member date with a transformer applies the StringToDate transformer function to the date property.
This is actually helpful, since some programming languages might not support all different ways of expressions a transformation.  Swift, for example, doesn't seem to allow attaching a custom decoder to a single class property, like the first example demands.  Instead, we would have to implement a custom decoder for the whole class, which corresponds to the second example.  We can let the IR pick the correct way of expressing the transformation so the renderers don't have to do any heavy lifting.
Once we get into decoding unions we will have to be able to express more complex transformer arguments, which will include decision trees and IR types.

Come to think of it, maybe a transformer should always be a transformer function, and we'll have transformer functions that do the job of transforming compound types by transforming their elements/properties.  That means we'll have to be able to pass more complex arguments to transformer functions right away, including other transformers.  The second example above then would be something like
class {
    date: String
}
  ->
class Foo {
    date: Date
}
    (TransformClass({ date: StringToDate}))

meaning that we transform the class type with the transformer function TransformClass, with the argument { date: StringToDate }, which tells it to apply the transformer StringToDate to the property date.

Another thing transformers can be useful for: Let's say we add tuples to our IR (#48), but some target languages (Go comes to mind) don't support tuples.  We'd also like to bootstrap them quickly, so a way of supporting them without adapting all the renderers right away would be nice.
We start out with this tuple in the IR:
[string, number, boolean]

We can change this into this transformed type:
Array<string | number | boolean>
  ->
class {
    first: string;
    second: number;
    third: boolean;
}
    (TransformTuple([first, second, third]))

Of course we still have to implement the transformer for the target language.  Maybe we will find a way to make composable transformers so this becomes easier.  Maybe something like this:
MakeClass({
    first: IndexArray(0) >> CastUnion,
    second: IndexArray(1) >> CastUnion,
    third: IndexArray(2) >> CastUnion
})

Come to think of it some more, TransformClass from above is just a special case of MakeClass if we also add a GetProperty transformer.

Issues with generating unions and intersections with primitive transformed types
There are three non-trivial cases to distinguish when forming unions and intersections in the presence of transformed types:

There's a type T in the union/intersection as well as a transformed type whose source type is T.  Example: string and stringified number.

There is more than one transformed type with the same source type T present in the union/intersection.  Example: stringified number and stringified bool.

There is a type T in the union/intersection as well as more than one transformed type whose source type is T.  Example: string, stringified number, and stringified bool.

Unions
The easy way out is to always go down to type T.  The hard way requires having either failable transformers or predicates for when transformers will apply.  In that case the generated code will first have to try the transformers, and if none of them can apply, use the base type (or fail, if the base type isn't in the union).  We should have a defined order in which the transformers will be tried.
Intersections
For intersections, we can disregard the base type T in all cases, so cases 2 and 3 are the same.  Case 1 is trivial, since only the transformed type is left, so that's what the resulting type is.  The other cases, where there is more than one transformed type, is a question of how hard we want to work.  In some instances we'll have to hard-code the intersection type.  For example, any intersection between date, time, and date-time is always date-time.

Having started on the implementation I now believe that it's easier to not introduce a new type, but to make transformations attachable to types.  The type is the source of the transformation, and the transformation specifies the target type.  Most of the IR machinery can be adapted to this much easier, I think.
For the union of multiple transformations, I think this can be nicely solved by having a transformation that combines multiple transformations in sequence, trying them one after the other.  We might even want to bring enums into this, so that we can handle them at the same level as date-time, for example.  One might want to have a union of date-time and the string"never", for example.  The target type of such a transformation is the union of the target types of the individual transformations, of course.

Not all transformations will be bidirectional.  For example, default-value maps from T | null to T.  If you serialize again and encounter a T with the default value, do you leave it be or transform it to null?  In this case you'll probably always choose to leave it be, but other cases might be less clear.

Bidirectionality and computed properties
We can rescue bidirectionality by making the properties where we would lose it computed properties, as opposed to transforming the values at serialization time.  As an example, let's say we have this type:
class Foo {
    bar: string = "Bear"
}

The semantics being that if bar is missing, or null or whatever, it defaults to "Bear".  As opposed to generating this C# class:
class Foo {
    [JsonProperty("bar", Default = "Bear") or whatever]
    public string Bar { get; set; }
}

We could generate
class Foo {
    [JsonProperty("bar", AllowNull)]
    private string InternalBar { get; set; }

    public string Bar {
        get {
            return InternalBar == null ? "Bear" : InternalBar;
        }
        set {
            InternalBar = value;
        }
    }
}

Downsides:

Computed properties might be awkward in some languages.
They are slower, unless we retain the results, in which case they use more memory.
They do need the transformed type to be a property in most languages.  As in, if the top-level type is a string, you can't make it into a computed property because it's not contained in anything.

Computed properties might be useful for purposes other than preserving bidirectionality.

This is implemented in #845


version: post-release bump

Signed-off-by: Jiri Kuncar<EMAIL_ADDRESS>

Coverage remained the same at 99.528% when pulling 6022d7cf1b029f5ac1e368250f343c373182bfdc on jirikuncar:version-bump into 5d7df330f02d48192ecab3e2a83e72a2a009b3e4 on inveniosoftware:master.


Feature request: separate equalizers for left/right channel

Is it possible to make ability to apply equalizer to left and right channel independently? I have reactive tinnitus influenced by one sided low hearing loss in high frequency range. Ability to raise output in left channel only would be very useful.

I was considering this feature for the next release of eqMac, but I cannot seem to figure out how the UI/UX would look like, not even mentioning the code complexity. This is not something that is commonly requested from users, so would have to be a secondary functionality you enable from settings. Definitely not in the nearest future, but I will keep thinking about it and maybe it will happen some day. Sorry to hear about your issues, I have some ear troubles as well. Good luck!

IMHO one both / left / right switch will be enough. Curves for left and right ear can be disambiguated by color. It's very close to best UI for editing curves - Photoshop Curves filter. Thanks for considering this.

Hey everyone. Excuse me for the lack of updates. I have been really busy on the new version of eqMac. I am happy to announce an open beta release.
You can download it here: https://eqmac.app
There are a lot of known bugs that I could not fix just yet:

Driver not installing from the first try. Sometimes need to restart eqMac a couple times.
Complex Device switching logic is not perfect (for example when switching off the device that's currently playing, plugging in new device etc.)
UI takes long to load (a lot can be done to improve that)
UI dimensions are not synching properly
There's a noticeable audio delay in the playback, so not suitable for video yet (I know where the problem lies just haven't had the time to fix yet)
Sometimes the playback doesn't start, in that case please try to switch the output to the device you want and eqMac should try and build the audio pipeline again.

So as you can see there's still some work to be done but at least it's a completely modern tech stack that I am fully willing to work on: 100% Swift, User Space driver and a Web UI, instead of C++, Kernel Drivers. As I said, should work on Catalina and should not have any pesky ear drum blasting volume bugs (at least I didn't come across any).
I'm open to feedback and issue reports here: https://github.com/bitgapp/eqmac
That repo will become the main repo for eqMac and this repo will be archived eventually.
And yes it's called just eqMac, not eqMac3 :) And it will stay that way forever.

Hey everyone 👋
I will be open sourcing the new version of eqMac, therefore, I am closing this issue and repository as I will be transfering nodeful/eqMac2 repo to bitgapp/eqMac
Please bear with me for the next couple of days as I setup that new repo. The eqMac2 code will be living in the old-eqMac2 branch if anyone is interested in referencing it for whatever reason.
Please try out the new version and report bugs to the new repo.
Thanks for using eqMac.

Hey I have gave a lot of thought to this feature and I think it doesn't really fit into the UI/UX of eqMac and would be very hard to implement in terms of coding. I really recommend considering using https://rogueamoeba.com/audiohijack/ software it's very stable and you can achive the setup you desire like so:


Compare Miscanthus parameters to defaults

[x] Vmax estimated by meta-analysis is much smaller than default of 50.
[ ] check other parameters as well
[x] add new parameters that are used by salix PFT but not (yet) by miscanthus
[x] quantum efficiency
[x] chi_leaf

As discussed in ebimodeling/Miscanthus_water_use_manuscript#18:
Fernando's paper (Miguez et al 2009) used Vmax = 39 and ...

Hi Dave,
In the figure I showed in the last meeting I used 39 for Vmax.
On Wed, Jul 1, 2015 at 12:30 PM, David LeBauer<EMAIL_ADDRESS>wrote:

As discussed in ebimodeling/Miscanthus_water_use_manuscript#18
https://github.com/ebimodeling/Miscanthus_water_use_manuscript/issues/18
:
Fernando's paper (Miguez et al 2009
http://onlinelibrary.wiley.com/doi/10.1111/j.1757-1707.2009.01019.x/epdf)
used Vmax = 39 and ...
[image: image]
https://cloud.githubusercontent.com/assets/464871/8460747/d9317cbc-1fec-11e5-9a5b-cecffc8b70d9.png
—
Reply to this email directly or view it on GitHub
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-117758762
.

--
Dr. Ursula Ruiz-Vera
Postdoctoral Research Associate
Carl R. Woese Institute for Genomic Biology,
University of Illinois at Urbana-Champaign
1206 W. Gregory Drive, Room 1400
Urbana, Illinois 61801
<EMAIL_ADDRESS>+407-694-6405

@UrsulaRuiz are you otherwise using defaults?

Yes, I am using defaults. The values I entered before the run are: the type
of soil, the day of emerge, day for the peak of yield, the initial rhizome .
I still need to change the Tp1 tp2 in phenology.
On Thu, Jul 2, 2015 at 9:27 AM, David LeBauer<EMAIL_ADDRESS>wrote:

@UrsulaRuiz https://github.com/UrsulaRuiz are you otherwise using
defaults?
—
Reply to this email directly or view it on GitHub
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118049943
.

--
Dr. Ursula Ruiz-Vera
Postdoctoral Research Associate
Carl R. Woese Institute for Genomic Biology,
University of Illinois at Urbana-Champaign
1206 W. Gregory Drive, Room 1400
Urbana, Illinois 61801
<EMAIL_ADDRESS>+407-694-6405

What values are you using for

day of emergence
day for the peak of yield (and what parameter is this?)
initial rhizome .
Tp1, tp2,

?
On Thu, Jul 2, 2015 at 9:36 AM, UrsulaRuiz<EMAIL_ADDRESS>wrote:

Yes, I am using defaults. The values I entered before the run are: the type
of soil, the day of emerge, day for the peak of yield, the initial rhizome
.
I still need to change the Tp1 tp2 in phenology.
On Thu, Jul 2, 2015 at 9:27 AM, David LeBauer<EMAIL_ADDRESS>wrote:

@UrsulaRuiz https://github.com/UrsulaRuiz are you otherwise using
defaults?
—
Reply to this email directly or view it on GitHub
<
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118049943

.

--
Dr. Ursula Ruiz-Vera
Postdoctoral Research Associate
Carl R. Woese Institute for Genomic Biology,
University of Illinois at Urbana-Champaign
1206 W. Gregory Drive, Room 1400
Urbana, Illinois 61801
<EMAIL_ADDRESS>+407-694-6405
—
Reply to this email directly or view it on GitHub
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118052065
.

They are specified in the Miscanthus_Manuscript.RMd file in the inst folder
which is inside the Miscanthus_water_use
The values are from the energy farm data
On Thu, Jul 2, 2015 at 9:43 AM, David LeBauer<EMAIL_ADDRESS>wrote:

What values are you using for

day of emergence
day for the peak of yield (and what parameter is this?)
initial rhizome .
Tp1, tp2,

?
On Thu, Jul 2, 2015 at 9:36 AM, UrsulaRuiz<EMAIL_ADDRESS>wrote:

Yes, I am using defaults. The values I entered before the run are: the
type
of soil, the day of emerge, day for the peak of yield, the initial
rhizome
.
I still need to change the Tp1 tp2 in phenology.
On Thu, Jul 2, 2015 at 9:27 AM, David LeBauer<EMAIL_ADDRESS>wrote:

@UrsulaRuiz https://github.com/UrsulaRuiz are you otherwise using
defaults?
—
Reply to this email directly or view it on GitHub
<

https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118049943

.

--
Dr. Ursula Ruiz-Vera
Postdoctoral Research Associate
Carl R. Woese Institute for Genomic Biology,
University of Illinois at Urbana-Champaign
1206 W. Gregory Drive, Room 1400
Urbana, Illinois 61801
<EMAIL_ADDRESS>+407-694-6405
—
Reply to this email directly or view it on GitHub
<
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118052065
.

—
Reply to this email directly or view it on GitHub
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118054930
.

--
Dr. Ursula Ruiz-Vera
Postdoctoral Research Associate
Carl R. Woese Institute for Genomic Biology,
University of Illinois at Urbana-Champaign
1206 W. Gregory Drive, Room 1400
Urbana, Illinois 61801
<EMAIL_ADDRESS>+407-694-6405

both the definition of newphenoParms and its use are commented out ... are you planning to use these values?
#newphenoParms <- phenoParms(tp1=525, tp2=1226, tp3=1927, tp4=3500, tp5=3600, tp6=4000, kRhizome1=-0.1, kRhizome2=-0.08)
...
res09 <- BioGro(WetDat=weather09, iRhizome=iRhizome, day1=day1, dayn=dayn, photoControl=newphotoparameters, soilControl = soilparameters_565B)#, phenoControl=newphenoParms)

I need to change tp1 and tp2 in each year.
On Thu, Jul 2, 2015 at 11:52 AM, David LeBauer<EMAIL_ADDRESS>wrote:

both the definition of newphenoParms and its use are commented out ... are
you planning to use these values?
#newphenoParms <- phenoParms(tp1=525, tp2=1226, tp3=1927, tp4=3500, tp5=3600, tp6=4000, kRhizome1=-0.1, kRhizome2=-0.08)...res09 <- BioGro(WetDat=weather09, iRhizome=iRhizome, day1=day1, dayn=dayn, photoControl=newphotoparameters, soilControl = soilparameters_565B)#, phenoControl=newphenoParms)
—
Reply to this email directly or view it on GitHub
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118091461
.

--
Dr. Ursula Ruiz-Vera
Postdoctoral Research Associate
Carl R. Woese Institute for Genomic Biology,
University of Illinois at Urbana-Champaign
1206 W. Gregory Drive, Room 1400
Urbana, Illinois 61801
<EMAIL_ADDRESS>+407-694-6405

How do you determine tp1 and tp2?
On Friday, July 3, 2015, UrsulaRuiz<EMAIL_ADDRESS>wrote:

I need to change tp1 and tp2 in each year.
On Thu, Jul 2, 2015 at 11:52 AM, David LeBauer<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
wrote:

both the definition of newphenoParms and its use are commented out ...
are
you planning to use these values?
#newphenoParms <- phenoParms(tp1=525, tp2=1226, tp3=1927, tp4=3500,
tp5=3600, tp6=4000, kRhizome1=-0.1, kRhizome2=-0.08)...res09 <-
BioGro(WetDat=weather09, iRhizome=iRhizome, day1=day1, dayn=dayn,
photoControl=newphotoparameters, soilControl = soilparameters_565B)#,
phenoControl=newphenoParms)
—
Reply to this email directly or view it on GitHub
<
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118091461
.

--
Dr. Ursula Ruiz-Vera
Postdoctoral Research Associate
Carl R. Woese Institute for Genomic Biology,
University of Illinois at Urbana-Champaign
1206 W. Gregory Drive, Room 1400
Urbana, Illinois 61801
<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','ruizver1@illinois.edu');>
+407-694-6405
—
Reply to this email directly or view it on GitHub
https://github.com/ebimodeling/biocro_regional/issues/9#issuecomment-118397637
.

@dlebauer
We are adjusting them empirically to match observations of LAI growth observed at the Efarm.
Even if we do not change tp1 and tp2, final yields estimates are still acceptable. we plan to change them only to match pattern of LAI growth.
There should not be any need to change them if  you are, at this moment, interested in final yield.


nans in MD cookbook notebook

This seems like a really excellent project and I am excited about the potential for gradient-based optimization with your approach. However, in running the MD cookbook notebook, every simulation or minimization routine results in nans. I was surprised to find this even for simple Lennard-Jones fluid at low density and with very small time steps (it occurs even for 1e-6 sometimes). I am running directly from Google Colab so it doesn't seem like an issue on my end. Is there a stable version I could try rolling back to where this has been tested?
I also tried the jax.config.update("jax_debug_nans", True) setting and got the following output from the first Brownian dynamics simulation of the bubble raft on the first step:
Invalid value encountered in the output of a jit function. Calling the de-optimized version.

FloatingPointError                        Traceback (most recent call last)
/usr/local/lib/python3.6/dist-packages/jax/interpreters/xla.py in _xla_call_impl(fun, device, backend, name, donated_invars, *args)
528   try:
--> 529     return compiled_fun(*args)
530   except FloatingPointError:
21 frames
FloatingPointError: invalid value (nan) encountered in xla_call
During handling of the above exception, another exception occurred:
FloatingPointError                        Traceback (most recent call last)
/usr/local/lib/python3.6/dist-packages/jax/interpreters/xla.py in _check_nans(name, xla_shape, buf)
350   if dtypes.issubdtype(xla_shape.element_type(), np.inexact):
351     if np.any(np.isnan(buf.to_py())):
--> 352       raise FloatingPointError(f"invalid value (nan) encountered in {name}")
353
354 ### compiling jaxprs
FloatingPointError: invalid value (nan) encountered in mul

Hi @wfreinhart thanks a lot for your message, sorry you're finding NaNs! I just opened up / ran the JAX MD cookbook and it did not give NaNs (except, curiously for once cell in the NN example because the learning rate is slightly too high). It is strange because, as you suggest, neither of us are running locally and we are both using a fresh install.
One thing I'm curious about: in the cookbook notebook I don't actually see that we use LJ anywhere, just to confirm: you also observe NaNs with the soft sphere potential?

Thanks for the quick reply! Yes, I was talking about the soft sphere potential in the cookbook, sorry for the confusion.
I triple checked and realized my mistake -- I was looking at an old version of the code at https://github.com/jaxmd/jax-md and literally didn't realize until this moment that it was located at a different url than this one. My mistake!

Ah yes, this is confusing; that version was an anonymized version for a conference but really it shouldn't exist anymore. I'll make a note to delete it. In case you were looking there, we have a number of new notebooks now that I think are a bit more current than the cookbook notebook!


Example transport path

First simple example

FYI, I merged quite a few breaking changes to the graaf interface on main. I pushed the changes to your branch, so you should be able to continue (after pulling your branch) without any merge conflicts. Please feel free to reach out if you encounter any issues.

Hey @bobluppes! Can you take a look? For some reason compile and test doesnt pass but it works fine on my machine. I've checked the error and it's saying that cannot find breadth traversal.


Nvidia GPU provider (Cuda or Vulkan) not found

Description
No gpu providers found on a windows 10 home with an Nvidia rxt 2070 gpu. This is a very common end-user config.
Steps to reproduce

Get the go-sm Windows binaries from v0.2.7-beta.0
Run the gpu test app to list providers from PowerShell. e.g. .\gpu-setup-test.exe -l

Actual Behavior
Only cpu provider is listed
Expected Behavior
A Cuda (and in a addition Vulkan) providers should be displayed for the gpu.
Environment

Windows 10 home 10.0.19043 (fully patched)
Nvidia RTX 2070
Nvidia drivers 456.71
Local user account is a local admin (not a limited user)
AMD Ryzen 7 2700 8 cores gpu. 32GB ram.

Notes

Smapp 0.2 doesn't display the gpu option because the node api returns to it only the cpu provider but this is easier to repro and fix w/o considering smapp - just by using or console gpu-setup-test.exe

No repro on more recent builds.


Feature: Control over reloads via API

Thank you for making this library!
Using the API, I'd like control over reloads.
Suggestion: a flag that determines if automatic reloads should occur and a function to call in order to trigger a reload.

Yes, me too. And that's not a coincidence, since we're working on the same project together.

Do you mean like this?
let config = Config {
    reload: true
}
listen("<IP_ADDRESS>", 8080, "./", config).await.unwrap();

That brings us half way there. Additionally, we'd like to be able to trigger individual reloads ourselves. I know I'm not providing you with an API design at this point. It's not immediately apparent to me what that design would be.

What's the purpose for you to determine every reloads? For example, do you want to detect if some files change and then trigger reloads or want to control the reloads frequency manually?

A page may consist of multiple files. And in our case it does. If these files get updated too far apart from each other, then the developer experiences multiple reloads. This is what we're trying to prevent. If we control the triggering of reloads, we can make sure that the developer experiences one reload per change they made.

They hit save once and experience one reload. Well, at least that's the ideal. Currently, they experience two reloads for each time they save a source file. That is because we have a program that is watching the output HTML files and spitting out a CSS file whenever they change (tailwindcss). So each change to an HTML file is followed by a change to a CSS file.

I think these problems are worth considering but should be done by the library instead of the users in order to avoid the extra repetitive work.
Here are my solutions:
1. Irrelevant Reload
Use html-editor (also my library) to analyze the src/href attributes in the HTML file of the to find which assets it depends. And then refresh only when the files it depends or itself are modified.
The problem is that it will increase the response time since the parse process need extra. I may need improve the html-editor performance (it's still far from what I expected) first before implement it into this repository.
2. Extra Repetitive Reload
Set a minimum interval like 500ms. If in this interval any file is changed, it will be pushed to the queue, and then when the timer is over, the server will check the queue to trigger at most 1 reload for each file.
If you agree the solutions, I will split this issue into 2 new issues and then work on it. Or if not, feel free to give me any suggestion.

The source of truth regarding updates to the website is somewhere in the program that updates them.
When the boundary is an inconvenient one, such as a command, one is inclined to reduce interaction across that boundary and to develop such features that ignore the source of truth and try to figure out when there was an update by looking at the files and updates to individual files. This is detective work as if the source of truth is not available.
When the boundary is an API, then the truth can be propagated from the source of it conveniently to the mechanism that can tell the browser to reload.
I should be able to tell live-server via the API "there was an update to the website", which is different then individual file updates.

Ok, it makes sense. Maybe these two solutions above can be added as another update in the future.
Now for this issue, here's another possible API:
listen_manually("<IP_ADDRESS>", 8080, "./", |file: Path, behavior: Behavior| {
  if is_relevant_file(file) {
    match behavior {
      Behavior::Modify => true,
      _ => false
    }
  } else {
    false
  }
}).await.unwrap();

This closure can be expanded like returning aVec<Url> to only reload part of the pages or adding other parameters, but it will also need extra efforts to implement.

A page may consist of multiple files. And in our case it does. If these files get updated too far apart from each other, then the developer experiences multiple reloads. This is what we're trying to prevent. If we control the triggering of reloads, we can make sure that the developer experiences one reload per change they made.

This problem may have been solved in #83. Multiple file changes will only trigger one reload in the website unless the changes are separated in time too long.
Close this issue also because of no activity for a long time. Feel free to open another issue if you have other problems.

Thank you, but  time-based isn't what I'm looking for. I am looking for explicit control over reloads.


Update source of bintray dependencies

Description
Teku is now publishing to Cloudsmith but we still have two bintray repos for dependencies - discovery which we should migrate and besu which has moved to artifactory but not sure what the repo URL is.

Need to update to besu 21.1.0 when it comes out.  Repo URL is https://hyperledger.jfrog.io/artifactory/besu-maven/

besu 21.1.0 has invalid dependencies so will now need https://github.com/hyperledger/besu/pull/1950 to merge and then wait for 21.1.1 to upgrade.
Dependencies will be available to download from bintray for another 12 months so timing won't be a problem.

The other remaining dependency is jblst.  It's publishing to cloudsmith but the latest versions are giving us grief with SIGILL so a new release hasn't been cut yet.


Feature Proposal: Add items-normal and align-normal

Hey!
I've noticed that there are no utilities for a "normal" state for align-items and align-self
Styles that want to accomplish an output like items-center md:items-normal won't work currently.
That's the reason why "normal" utilities should be added. I'd create a PR for that if this feature is wanted.

Interesting I didn't even know about those values! The idea with the current set of utilities is that you'd just reset to the browser's default value to "undo" anything, so with align-items the default is items-stretch and with align-self the default is self-auto.
You can see on MDN that the initial/default/normal value for align-items is stretch, and for align-self it's auto:
https://developer.mozilla.org/en-US/docs/Web/CSS/align-items
https://developer.mozilla.org/en-US/docs/Web/CSS/align-self
The normal value isn't actually the default state for either of them and appears to be quite complex:

Is that actually what you want or does the current system accomplish what you need?

"normal" doesn't seem to be a flexbox specific property value. It looks like this is best handled in a custom way.

I've used items-stretch and self-auto in those scenarios, but I thought it might be useful to have a normal helper as well.
Though I understand that they won't be needed that often, so introducing them could be done as a separate utility by people needing them :)

Gonna close this unless there's a really compelling reason to add these; sounds like what we currently have solves this for most situations. Thanks though @manniL!


Bringing master into dev after 1.1.0 release.

Codecov Report

Merging #429 into dev will increase coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev     #429      +/-   ##
==========================================
+ Coverage   85.94%   86.06%   +0.12%
==========================================
  Files          56       56
  Lines        4391     4429      +38
  Branches      474      481       +7
==========================================
+ Hits         3774     3812      +38
  Misses        465      465
  Partials      152      152

Impacted Files
Coverage Δ

...orage-blob/azure/storage/blob/appendblobservice.py
94.38% <ø> (ø)
:arrow_up:

...-storage-queue/azure/storage/queue/queueservice.py
95.06% <ø> (ø)
:arrow_up:

...re-storage-common/azure/storage/common/__init__.py
100% <ø> (ø)
:arrow_up:

...re-storage-queue/azure/storage/queue/_constants.py
100% <100%> (ø)
:arrow_up:

...rage-common/azure/storage/common/_serialization.py
75.14% <100%> (+0.75%)
:arrow_up:

...torage-blob/azure/storage/blob/_deserialization.py
86.58% <100%> (+0.24%)
:arrow_up:

...-storage-common/azure/storage/common/_constants.py
100% <100%> (ø)
:arrow_up:

...ge-common/azure/storage/common/_deserialization.py
81.75% <100%> (+0.98%)
:arrow_up:

...zure-storage-file/azure/storage/file/_constants.py
100% <100%> (ø)
:arrow_up:

...zure-storage-common/azure/storage/common/models.py
91.27% <100%> (+0.36%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


Implement topk

This patch implements TopK which is the last missing operation to compile the SSD model.
Both TopK and NonMaxSuppression use ArgSort, so move the helper function emitArgSort to ONNXToKrnlCommon.{cpp, hpp}.

Jenkins Linux s390x Build #2468 [push] Implement topk (#954)  *... started at 18:49

Jenkins Linux amd64 Build #2469 [push] Implement topk (#954)  *... started at 17:49

Jenkins Linux ppc64le Build #1578 [push] Implement topk (#954)  *... started at 18:51

Jenkins Linux s390x Build #2468 [push] Implement topk (#954)  *... passed after 1 hr 3 min

Jenkins Linux amd64 Build #2469 [push] Implement topk (#954)  *... passed after 1 hr 9 min

Jenkins Linux ppc64le Build #1578 [push] Implement topk (#954)  *... passed after 1 hr 23 min


Added threshold option for find_eog_events() and create_eog_epochs()

What does this implement/fix?
Found that find_eog_events() and thus create_eog_epochs() weren't detecting many EOG events, despite seeing them in data.  The function used on the backend - peak_finder() - can take a threshold as argument, added option for user to pass a threshold to the function.  Initial testing found better detection of blinks.
Additional information
Perhaps more necessary in EEG recordings.

thanks @pmolfese
is there any chance to add a tiny test?
You'll need to update what's new API section to document the new parameter.

Added a very basic test to test_eog.py.
Apologies for the basic question: Where is the API document?

here :
https://github.com/mne-tools/mne-python/blob/master/doc/whats_new.rst#api

@pmolfese you still have flake8 errors
$ flake8 mne/preprocessing/eog.py
mne/preprocessing/eog.py:42:5: W191 indentation contains tabs
mne/preprocessing/eog.py:42:5: E101 indentation contains mixed spaces and tabs
mne/preprocessing/eog.py:109:80: E501 line too long (81 > 79 characters)
mne/preprocessing/eog.py:111:80: E501 line too long (82 > 79 characters)
mne/preprocessing/eog.py:161:80: E501 line too long (90 > 79 characters)
mne/preprocessing/eog.py:218:1: W293 blank line contains whitespace
mne/preprocessing/eog.py:220:5: W191 indentation contains tabs
mne/preprocessing/eog.py:220:5: E101 indentation contains mixed spaces and tabs
mne/preprocessing/eog.py:243:80: E501 line too long (86 > 79 characters)

Fixed flake errors in eog.py.  Still getting used to Python style.

thanks @pmolfese


synax check error

VSCode Version: 0.10.11
OS Version: Windows 10

my code like this :
{
prev_text: '<',
next_text:">",
items_per_page: 6,
## // behind line will fire synax check error.
link_to: 'javascript:;',
callback: function(currentPage){
}
}

I cannot reproduce this

Can you please add a screenshot of what you are seeing.

Cannot act on this with more information - closing, pls reopen when there is more information.

@egamma I'm sorry. I'm late. And the latest version is ok.